-
Notifications
You must be signed in to change notification settings - Fork 44
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
Deflake X-display tests on GitHub actions #58
Comments
I would prefer to use a different approach that the one in Gazebo and add have an option in the build/test system to indicate that GUI tests needs to be compiled/executed or not. This way is easier to detect failures when the display is not working well since the build will fail instead of silently report success while hiding errors. |
X display tests have been fixed on GitHub actions on #98, but they're still flaky. This is the new error:
|
As a reference,
It doesn't always happen to the same test. I've not been able to identify a pattern (i.e. it's always the first test, etc). It looks like the display just can't be found for one test, but then it's found again. I'm trying out different Xvfb arguments, and also trying to make the failure more verbose. The thing is that this failure isn't very common, so so it's hard to reproduce. |
Got a new error today that may help debug this a bit more:
It's possible that Xvfb is being killed due to high memory usage. |
Using EGL may solve this issue. |
We could revisit this idea and expose a CMake argument that lets us disable the tests which require a display on GitHub actions, but leave them enabled on Jenkins. Another alternative that @mjcarroll brought up was to try using one of the other platform plugins suggested in one of the errors above:
|
Another option potentially? https://github.com/uwerat/qpagbm |
Many
ign-gui
tests are failing like this on GitHub actions:We should prevent these tests from running when no display is detected. As a reference, this is how Gazebo-classic detects it: https://github.com/osrf/gazebo/blob/6fd426b3949c4ca73fa126cde68f5cc4a59522eb/cmake/CheckDRIDisplay.cmake
The text was updated successfully, but these errors were encountered: