-
Notifications
You must be signed in to change notification settings - Fork 26
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] Reusable CI #89
[CI] Reusable CI #89
Conversation
59d64e9
to
ffdc15f
Compare
I see that this test failure is occurring in the CI:
and that's extremely alarming since I cannot reproduce that error locally, and it's not something that should be able to change no matter the distro. One issue I'm noticing with the new CI pipeline is that all the |
Investigating.
Thank you for pointing out this! Fixed. |
ffdc15f
to
719213b
Compare
I can't either reproduce this in a local docker container. I'm doing a step-by-step migration from the old CI to the reusable one to see where the problem comes. I'm doing it in this branch/PR: #93 I have found that the problem comes from setting CXX: clang++ environment variable in the ros-tooling/[email protected] that in the reusable build workflow is set by default for all the other rmf workflows |
7fbb575
to
719213b
Compare
fails with:
|
Signed-off-by: Esteban Martinena <[email protected]>
719213b
to
7d5e13a
Compare
Signed-off-by: Yadunund <[email protected]>
* [CI] Reusable asan Signed-off-by: Esteban Martinena <[email protected]> * Remove push and add emf_traffic_examples Signed-off-by: Yadunund <[email protected]> * Reusable tsan Signed-off-by: Yadunund <[email protected]> --------- Signed-off-by: Esteban Martinena <[email protected]> Signed-off-by: Yadunund <[email protected]> Co-authored-by: Yadunund <[email protected]>
colcon test for Specifically I think it's this test
Update: If I comment out that specific test case, all tests pass
|
Signed-off-by: Yadunund <[email protected]>
…f_traffic into emartinena/reusable_ci
Signed-off-by: Yadunund <[email protected]>
Signed-off-by: Michael X. Grey <[email protected]>
Modified build.yaml workflow to use reusable workflow.