.murdock: test 200 repeats of known flakey test#18396
Closed
MrKevinWeiss wants to merge 5 commits intoRIOT-OS:masterfrom
Closed
.murdock: test 200 repeats of known flakey test#18396MrKevinWeiss wants to merge 5 commits intoRIOT-OS:masterfrom
MrKevinWeiss wants to merge 5 commits intoRIOT-OS:masterfrom
Conversation
github-actions
bot
added
the
Area: CI
MrKevinWeiss
added
CI: full build
MrKevinWeiss
changed the title
github-actions
bot
added
the
Area: CI
Contributor
|
@MrKevinWeiss The test failed due to a syntax error. Murdock is idle, so it's good time to do some tests now. |
Contributor
Author
|
Heh, but I was not idle. I will try to hit this on the weekend when there is not much traffic. |
Contributor
Author
|
hmmm |
MrKevinWeiss
added
CI: ready for build
Contributor
|
This must be the only PR which we want to fail and keeps passing... |
leandrolanzieri
added
CI: ready for build
MrKevinWeiss
added
CI: ready for build
Contributor
Author
|
Mobi6 is down again so I will retry. |
Contributor
|
Maybe ccache is too optimistic, so repeated runs yield same result but cache miss causes hash mismatch? |
Contributor
Author
|
Ya, that will be the next phase, disable ccache... |
chrysn
changed the title
Contributor
Author
|
I wonder if the failed results are getting overwritten with passing ones... |
Contributor
Author
|
I will wait until the speedup #18423 is in before continuing with this as the feedback cycle is super long. |
Contributor
Author
|
Theory-> something to do with gitcache (so far I have only seen it on esp bootloaders or packages or things that need to checkout other repos) stm pkgFrom #18433 esp bootloaderesp app ?(many mismatches binary + partitions) |
Now trying to multiply by 200 submitted jobs using the get_app_board_toolchain_pairs
MrKevinWeiss
force-pushed
the
test/hashissue
branch
from
98ce4d7 to
f132ebf
Compare
Contributor
Author
|
Status update, we still see mismatches in the nightlies (cryptoauth package tests). This PR has been rebased and we will see if we can make it reproducible. |
Contributor
Author
|
Ugg, this is still an issue but probably does not make sense to work on it during hacknack. |
Contributor
Author
|
Hmmm I give up on this. It seems like all the changes are somehow working through it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is only a test to try to solve the occasional unrelated hash failure problem forcing CI reruns.