Skip to content

tests/gnrc_sixlowpan: enable test on murdock#9650

Merged
miri64 merged 1 commit intoRIOT-OS:masterfrom
cladmi:pr/tests/gnrc_sixlowpan
Aug 1, 2018
Merged

tests/gnrc_sixlowpan: enable test on murdock#9650
miri64 merged 1 commit intoRIOT-OS:masterfrom
cladmi:pr/tests/gnrc_sixlowpan

Conversation

@cladmi
Copy link
Copy Markdown
Contributor

@cladmi cladmi commented Jul 31, 2018 •
edited
Loading

Contribution description

Enable testing gnrc_sixlowpan on murdock.

This should normally fail according to the release tests.
Should be fixed.

Issues/PRs references

Would help testing #9649 and #9648

@cladmi cladmi added Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware for the labeled PR labels Jul 31, 2018
@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Jul 31, 2018

Without the fix I get for native:

/tmp/dwq.0.37149671447736266/70d82b62cadeaff4d6834871563b2845/build/tests_gnrc_sixlowpan.elf: open(/dev/net/tun): No such file or directory
/tmp/dwq.0.37149671447736266/70d82b62cadeaff4d6834871563b2845/Makefile.include:494: recipe for target 'term' failed

So added the first commit that should solve this.

@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 31, 2018
@cladmi cladmi added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 31, 2018
@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 31, 2018
@miri64
Copy link
Copy Markdown
Member

miri64 commented Jul 31, 2018

@cladmi can you redo your cherry-pick. When I rebased, I squashed my changes to the apps BOARD_INSUFFICIENT_MEMORY list into the commit.

@cladmi cladmi force-pushed the pr/tests/gnrc_sixlowpan branch from 419c0d1 to e023be3 Compare July 31, 2018 15:08
@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 31, 2018
@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Jul 31, 2018

Done

@cladmi cladmi force-pushed the pr/tests/gnrc_sixlowpan branch from e023be3 to 82dbbd7 Compare July 31, 2018 15:46
@cladmi cladmi added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 31, 2018
@cladmi cladmi force-pushed the pr/tests/gnrc_sixlowpan branch from 82dbbd7 to 25b958a Compare July 31, 2018 17:09
@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 31, 2018
@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 1, 2018
@cgundogan cgundogan added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 1, 2018
@cgundogan
Copy link
Copy Markdown
Member

cgundogan commented Aug 1, 2018

Somehow the build is stuck (I think on one of the tests) :-/

I did restart murdock. However, @smlng or @kaspar030 should have a closer inspection on why these issues happen lately.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Aug 1, 2018

I think it's this PR (+ #9654) specifically that are the ones getting stuck. I'm looking at the page since the reload and the build is stuck at fail: 0 pass: 23439 done: 23439/23521.

@kaspar030
Copy link
Copy Markdown
Contributor

kaspar030 commented Aug 1, 2018

Seems like the samr21-xpro test node is down, which makes all PR's with "CI: run tests" get stuck. Quickfix is to remove that label from all PRs.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Aug 1, 2018

Quickfix is to remove that label from all PRs.

Seems pointless, since both PRs test that behavior

@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Aug 1, 2018

Tests will still be run for native. samr21-xpro should just need to be run manually.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Aug 1, 2018 •
edited
Loading

@cladmi can you have a look at the node? I've sent you the details to your phone.

@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Aug 1, 2018

ok

@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Aug 1, 2018

I rebooted the given node.

@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 1, 2018
@cladmi cladmi added this to the Release 2018.07 milestone Aug 1, 2018
@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware for the labeled PR Area: tests Area: tests and testing framework and removed CI: run tests If set, CI server will run tests on hardware for the labeled PR CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: tests Area: tests and testing framework labels Aug 1, 2018
@cladmi cladmi added the Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch label Aug 1, 2018
@miri64
Copy link
Copy Markdown
Member

miri64 commented Aug 1, 2018 •
edited
Loading

I'll test this one for samr21-xpro locally

miri64
miri64 approved these changes Aug 1, 2018
View reviewed changes
Copy link
Copy Markdown
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait... we did that already in #9649 ^^, so ACK and go.

@miri64 miri64 merged commit 1fcf23c into RIOT-OS:master Aug 1, 2018
@cladmi cladmi mentioned this pull request Aug 1, 2018
Merged
@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Aug 1, 2018

Backport provided in #9673

@cladmi cladmi deleted the pr/tests/gnrc_sixlowpan branch August 1, 2018 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@miri64 miri64 miri64 approved these changes

Assignees

No one assigned

Labels

Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch

Projects

None yet

Milestone

Release 2018.07

Development

Successfully merging this pull request may close these issues.

4 participants

@cladmi @miri64 @cgundogan @kaspar030