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

SdfGenerator_TEST failures on master #322

Closed
scpeters opened this issue Sep 1, 2020 · 3 comments
Closed

SdfGenerator_TEST failures on master #322

scpeters opened this issue Sep 1, 2020 · 3 comments
Labels
🔮 dome Ignition Dome tests Broken or missing tests / testing infra

Comments

@scpeters
Copy link
Member

scpeters commented Sep 1, 2020

There have been several recurring UNIT_SdfGenerator_TEST failures on the master branch since the following build on August 21:

The following test failure provides a clue of the difference:

The original world file has the following uri:

<uri>https://fuel.ignitionrobotics.org/1.0/openrobotics/models/Backpack/1/files/meshes/Backpack.dae</uri>`

The generated world file appears to have 2 URI's concatenated together:

<uri>file:///root/.ignition/fuel/fuel.ignitionrobotics.org/openrobotics/models/Backpack/1/https://fuel.ignitionrobotics.org/1.0/openrobotics/models/Backpack/1/files/meshes/Backpack.dae</uri>

I added some debugging print statements to the test in scpeters@a1490fb that confirm the other failures are connected to this same issue.

I think this may be related to gazebosim/gz-fuel-tools#85

@scpeters
Copy link
Member Author

scpeters commented Sep 1, 2020

@ahcorde noticed this same thing in #314 (comment)

@chapulina chapulina added the tests Broken or missing tests / testing infra label Sep 1, 2020
@chapulina
Copy link
Contributor

They should be fixed when #271 is merged forward, see #321

@chapulina chapulina added the 🔮 dome Ignition Dome label Sep 1, 2020
@chapulina
Copy link
Contributor

Latest master branch build is blue: https://build.osrfoundation.org/job/ignition_gazebo-ci-master-bionic-amd64/29/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 dome Ignition Dome tests Broken or missing tests / testing infra
Projects
None yet
Development

No branches or pull requests

2 participants