Skip to content
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

Spurious testsuite failures when run in parallel #233

Closed
andhe opened this issue May 6, 2016 · 3 comments
Closed

Spurious testsuite failures when run in parallel #233

andhe opened this issue May 6, 2016 · 3 comments

Comments

@andhe
Copy link

andhe commented May 6, 2016

Hello!

Filing this issue mostly to give feedback so that you're aware of it. Please feel free to close this issue at any time when you feel there's nothing more you'd like to fix up here, anyway...

While building libical 2.0.0 packages on Debian I ran into testsuite problems with "test 8" (icalrecur -r) on a seemingly random set of architectures: arm64, ppc64el, mips64el. The problems where not reproducible when running a manual build on Debian porter boxes for these architectures. I quickly assumed based on experience that it was parallel build related and can confirm that switching from "--parallel" to "--no-parallel" in our debian/rules file which invokes the build machinery when building packages made the problems go away.

I have not confirmed but my suspicion is that both the icalrecur tests (8 & 9) using the same test.out output filename is part of the problem. Likely the tests run in parallell and the test9 gets to overwrite the file before test8 has a chance to verify it against its expected results in certain racy conditions which happened to trigger reliably on the previously mentioned architecture buildds. Maybe the icalrecur test program can be changed to take output filename as an argument (and then be handed unique names per test) instead of hardcoding test.out in the fopen call? For now I'll live with marking the package as not being able to build in parallel to work around the issue.

Regards,
Andreas Henriksson

@winterz
Copy link
Member

winterz commented May 6, 2016

I don't recall running make test in a parallel mode, so I wouldn't be surprised that this is something we need to fix.

@winterz
Copy link
Member

winterz commented Sep 5, 2016

in the master branch I just commited 825e25a that might fix this issue. I tested every combination I could think of. Please test.

@winterz
Copy link
Member

winterz commented Nov 27, 2016

no response. closing. please re-open if you still encounter this problem in master branch.

@winterz winterz closed this as completed Nov 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants