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

[CI] Created build CI #443

Closed
wants to merge 2 commits into from
Closed

[CI] Created build CI #443

wants to merge 2 commits into from

Conversation

orensbruli
Copy link
Contributor

Signed-off-by: Esteban Martinena [email protected]

@luca-della-vedova
Copy link
Member

luca-della-vedova commented Sep 19, 2022

I noticed test failures due to QT on both this action and, as a consequence, the nightly.

It seems the workflow here uses a flag for colcon test:

QT_QPA_PLATFORM=offscreen colcon test --packages-select rmf_traffic_editor

I wonder if it would fix the issue?

Edit: Noted it's already in the workflow, I wonder if either the flag is not being passed to colcon test or there is some other difference in the image I'm not catching

@orensbruli
Copy link
Contributor Author

Edit: Noted it's already in the workflow, I wonder if either the flag is not being passed to colcon test or there is some other difference in the image I'm not catching

For some reason, it's not setting the QT specific env variable. Investigating.
https://github.com/open-rmf/rmf_traffic_editor/actions/runs/3071883178/jobs/5012288157#step:7:42

@codebot
Copy link
Contributor

codebot commented Sep 21, 2022

Setting QT_QTA_PLATFORM was an attempt to make a test work even though the CI machines don't have an X server running. It didn't work, and I didn't know why and failed to investigate further. I thought that was the variable that Qt says to set in order to run tests without graphics, but 🤷 I probably messed it up somehow.

Signed-off-by: Esteban Martinena <[email protected]>
Signed-off-by: Esteban Martinena <[email protected]>
@luca-della-vedova
Copy link
Member

I believe this is superseded by #457.

@luca-della-vedova luca-della-vedova deleted the emartinena/build_ci branch March 26, 2024 03:49
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

Successfully merging this pull request may close these issues.

3 participants