-
Notifications
You must be signed in to change notification settings - Fork 364
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
Test 12 (testing urlhelper fails) is racy #3210
Comments
I just got
Bumping this to priority, this seems to come up far more often than some of the other racy tests we've had. |
pmatilai
added a commit
to pmatilai/rpm
that referenced
this issue
Jul 30, 2024
/bin/false exits so fast that racing with the parent output order is a fifty-sixty situation at best. Use a purpose-made shell script that does a bit of IO and even sleeps a bit, which together with shell startup time should be plenty enough to guarantee stable output for most practical purposes. If it *still* bugs us after this, we can always sort the output. Fixes: rpm-software-management#3210
ffesti
pushed a commit
that referenced
this issue
Aug 5, 2024
/bin/false exits so fast that racing with the parent output order is a fifty-sixty situation at best. Use a purpose-made shell script that does a bit of IO and even sleeps a bit, which together with shell startup time should be plenty enough to guarantee stable output for most practical purposes. If it *still* bugs us after this, we can always sort the output. Fixes: #3210
dmnks
pushed a commit
to dmnks/rpm
that referenced
this issue
Aug 29, 2024
/bin/false exits so fast that racing with the parent output order is a fifty-sixty situation at best. Use a purpose-made shell script that does a bit of IO and even sleeps a bit, which together with shell startup time should be plenty enough to guarantee stable output for most practical purposes. If it *still* bugs us after this, we can always sort the output. Fixes: rpm-software-management#3210 (cherry picked from commit 8e1e935)
dmnks
pushed a commit
that referenced
this issue
Aug 30, 2024
/bin/false exits so fast that racing with the parent output order is a fifty-sixty situation at best. Use a purpose-made shell script that does a bit of IO and even sleeps a bit, which together with shell startup time should be plenty enough to guarantee stable output for most practical purposes. If it *still* bugs us after this, we can always sort the output. Fixes: #3210 (cherry picked from commit 8e1e935)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There is an empty line / new line moving around.
Using
sort
as above doesn't fix it.This has been mentioned in #3196 previously but deserves it's own ticket.
The text was updated successfully, but these errors were encountered: