Skip to content

tests: blacklist mtd using tests for esp32-wroom-32#15124

Merged
bergzand merged 1 commit intoRIOT-OS:masterfrom
kaspar030:blacklist_mtd_tests_esp32
Sep 30, 2020
Merged

tests: blacklist mtd using tests for esp32-wroom-32#15124
bergzand merged 1 commit intoRIOT-OS:masterfrom
kaspar030:blacklist_mtd_tests_esp32

Conversation

@kaspar030
Copy link
Contributor

@kaspar030 kaspar030 commented Sep 30, 2020

Contribution description

These tests fail on CI. Blocklisting them for now so CI won't stumble all the time.

Testing procedure

Issues/PRs references

Corresponding issue: #15123

@kaspar030 kaspar030 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) 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 Platform: ESP Platform: This PR/issue effects ESP-based platforms CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Sep 30, 2020
@benpicco benpicco requested a review from gschorcht September 30, 2020 08:48
Copy link
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

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

If nobody comes up with a better solution (those tests succeed locally) this is the easy way out :\

@kaspar030 kaspar030 force-pushed the blacklist_mtd_tests_esp32 branch from 0282a31 to cc0523e Compare September 30, 2020 11:00
@kaspar030 kaspar030 force-pushed the blacklist_mtd_tests_esp32 branch from cc0523e to d59fd84 Compare September 30, 2020 11:02
@bergzand bergzand merged commit 04d1eb5 into RIOT-OS:master Sep 30, 2020
@kaspar030 kaspar030 deleted the blacklist_mtd_tests_esp32 branch September 30, 2020 18:41
@benpicco
Copy link
Contributor

benpicco commented Oct 4, 2020

Turns out esp* did indeed break by #14754 - we should have trusted the CI on this one 😞

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 Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants