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

Add tests to conda-forge job #3264

Closed
wants to merge 9 commits into from

Conversation

traversaro
Copy link
Collaborator

@traversaro traversaro commented Sep 22, 2022

In conda-forge/gazebo-feedstock#148 we debugged quite a tricky issue that only appeared at runtime. To avoid having problems like this in the future, I tried to enable tests (skipping the one that are currently failing) in the conda-forge job.

As the conda-forge CI job is already quite slow (~1 hour and 15 minutes), we can also think of changing the job to run the tests only on a given schedule, but for the time being I just implemented the simplest solution.

@traversaro
Copy link
Collaborator Author

It seems that there is some sort of cross-talking, so even if after one round I disable all failing test, and the following round more test fail. So probably for the moment it is a better idea just to run a few specified tests.

@traversaro
Copy link
Collaborator Author

Ok, by running just the INTEGRATION_transport_msg_count I reproduced the problem that we have in conda-forge/gazebo-feedstock#152 .

@traversaro
Copy link
Collaborator Author

Apparently the problem in INTEGRATION_transport_msg_count on macOS will be solved by #3269 , but even with that fix the test is not passing in CI for some reason. Anyhow, we can close the PR for now.

@traversaro traversaro closed this Oct 26, 2022
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.

1 participant