Skip to content

tests/gnrc_rpl: Disable CI test for native #19171

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 18, 2023
Merged

Conversation

maribu
Copy link
Member

@maribu maribu commented Jan 18, 2023

Contribution description

This disables the tests/grnc_rpl test run for native. This test is too flaky to be included in the CI.

Testing procedure

Not needed

Issues/PRs references

None

@maribu maribu requested a review from miri64 as a code owner January 18, 2023 18:49
@github-actions github-actions bot added the Area: tests Area: tests and testing framework label Jan 18, 2023
@maribu maribu requested review from benpicco and kaspar030 January 18, 2023 18:50
Copy link
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

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

ACK.

bors merge

@bors
Copy link
Contributor

bors bot commented Jan 18, 2023

🕐 Waiting for PR status (GitHub check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

@kaspar030 kaspar030 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Jan 18, 2023
@riot-ci
Copy link

riot-ci commented Jan 18, 2023

Murdock results

✔️ PASSED

810653d tests/gnrc_rpl: Disable CI test for native

Success Failures Total Runtime
1 0 1 01m:00s

Artifacts

@benpicco
Copy link
Contributor

maybe we should also mark RPL as broken in the doc…

bors merge

@miri64
Copy link
Member

miri64 commented Jan 18, 2023

maybe we should also mark RPL as broken in the doc…

I would elect for marking it @experimental 😅 I think the implementation never really left the experimental stage, but back then there was no way to mark it as such 😁

bors bot added a commit that referenced this pull request Jan 18, 2023
19166: boards/seeedstudio-gd32: complete and improve board definition r=benpicco a=gschorcht

### Contribution description

This PR completes and improves the board definition for the Seeedstudio GD32 RISC-V Dev board. It
- adds the user LED and Button definitions to `board.h` (bef4209)
- adds the SAUL support for user LED and Button definitions (7a5b2f1)
- changes the OpenOCD configuration (32c0c4b)
   - to be usable with Upstream OpenOCD release version 0.12.0
   - to allow different FTDI configuration and other adapters
- adds a documentation with flashing guide, feature list, support status, pinout, schematic reference (9dcc83b)

These changes is the first PR for a number of follow-up PR I will provide in next days to extend the GD32VF103 support. I have already working
- `periph_adc` support,
- `periph_spi` support,
- `tinyusb_device` support,

and almost finished
- `periph_i2c` support (implemented and working with some errors),
- `pm_layered` support (implemented but not working correctly yet), and
- `periph_usbdev` support (implemented by extending `usbdev_synopsys_dwc2 driver but bot working yet).

I will try to implement
- `periph_gpio_irq` support,
- `periph_rtc` support, and
- `periph_rtt` support.

Since I'm using the Sispeed Longan Nano board for testing, I will add the board definition for this board. I will then move some board definitions to a common folder.

### Testing procedure

Green CI.
Documentation should be generated correctly.
Flashing the `seeedstudio-gd32` should still work.
`tests/leds` should work.

### Issues/PRs references

19171: tests/gnrc_rpl: Disable CI test for native r=benpicco a=maribu

### Contribution description

This disables the `tests/grnc_rpl` test run for `native`. This test is too flaky to be included in the CI.

### Testing procedure

Not needed

### Issues/PRs references

None

Co-authored-by: Gunar Schorcht <gunar@schorcht.net>
Co-authored-by: Marian Buschsieweke <marian.buschsieweke@ovgu.de>
@bors
Copy link
Contributor

bors bot commented Jan 18, 2023

This PR was included in a batch that was canceled, it will be automatically retried

@bors
Copy link
Contributor

bors bot commented Jan 18, 2023

Build succeeded:

@bors bors bot merged commit e7a68ac into RIOT-OS:master Jan 18, 2023
@maribu maribu deleted the tests/gnrc_rpl branch January 21, 2023 21:40
@jia200x jia200x added this to the Release 2023.04 milestone Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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 CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants