Skip to content

tests/pkg_mbedtls-blacklist: blacklist samr21-xpro#17588

Merged
fjmolinas merged 1 commit intoRIOT-OS:masterfrom
benpicco:tests/pkg_mbedtls-blacklist
Jan 31, 2022
Merged

tests/pkg_mbedtls-blacklist: blacklist samr21-xpro#17588
fjmolinas merged 1 commit intoRIOT-OS:masterfrom
benpicco:tests/pkg_mbedtls-blacklist

Conversation

@benpicco
Copy link
Contributor

@benpicco benpicco commented Jan 28, 2022

Contribution description

The ENTROPY test always fails on this board

main(): This is RIOT! (Version: buildtest)
mbedtls test

  SHA-224 test #1: passed
  SHA-224 test #2: passed
  SHA-224 test #3: passed
  SHA-256 test #1: passed
  SHA-256 test #2: passed
  SHA-256 test #3: passed

  ENTROPY test: failed

Testing procedure

Issues/PRs references

@benpicco
tests/pkg_mbedtls-blacklist: blacklist samr21-xpro
3c6241b 
The ENTROPY test always fails on this board

main(): This is RIOT! (Version: buildtest)
mbedtls test

  SHA-224 test #1: passed
  SHA-224 test #2: passed
  SHA-224 test #3: passed
  SHA-256 test #1: passed
  SHA-256 test #2: passed
  SHA-256 test #3: passed

  ENTROPY test: failed
@github-actions github-actions bot added the Area: tests Area: tests and testing framework label Jan 28, 2022
@benpicco benpicco added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer labels Jan 28, 2022
@PeterKietzmann
Copy link
Member

PeterKietzmann commented Jan 28, 2022

test always fails on this board

Does it hurt anyone? I would like to fix this differently (as indicated here) but am currently busy. By next week should be doable.

@benpicco
Copy link
Contributor Author

benpicco commented Jan 28, 2022 •
edited
Loading

Does it hurt anyone?

Yes, every instance of "Run Tests" on CI does always fail.

By next week should be doable.

You can then revert this commit 😉

@benpicco benpicco added the CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs label Jan 28, 2022
fjmolinas
fjmolinas approved these changes Jan 31, 2022
View reviewed changes
Copy link
Contributor

@fjmolinas fjmolinas left a comment

Choose a reason for hiding this comment

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

ACK, this is wasting CI time, it can be removed with the fix.

@fjmolinas fjmolinas enabled auto-merge January 31, 2022 08:26
@fjmolinas fjmolinas merged commit 5bde360 into RIOT-OS:master Jan 31, 2022
@benpicco benpicco deleted the tests/pkg_mbedtls-blacklist branch January 31, 2022 09:43
@PeterKietzmann PeterKietzmann mentioned this pull request Feb 2, 2022
Merged
@OlegHahm OlegHahm added this to the Release 2022.04 milestone Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fjmolinas fjmolinas fjmolinas approved these changes

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@MichelRottleuthner MichelRottleuthner Awaiting requested review from MichelRottleuthner MichelRottleuthner is a code owner

@smlng smlng Awaiting requested review from smlng

@PeterKietzmann PeterKietzmann Awaiting requested review from PeterKietzmann

@maribu maribu Awaiting requested review from maribu

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 CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer

Projects

None yet

Milestone

Release 2022.04

Development

Successfully merging this pull request may close these issues.

4 participants

@benpicco @PeterKietzmann @fjmolinas @OlegHahm