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

🧑‍🌾 Gz-transport11-homebrew UNIT_gz_TEST ignTest.TopicInfo failed #510

Closed
Crola1702 opened this issue Jul 15, 2024 · 3 comments
Closed
Assignees
Labels
bug Something isn't working 🏯 fortress Ignition Fortress

Comments

@Crola1702
Copy link
Contributor

Environment

  • OS Version: MacOS
  • Source build

Steps to reproduce

  1. Run a build in gz-transport11
  2. See ignTest.TopicInfo fail

Output

Reference build: https://build.osrfoundation.org/job/gz_transport-ci-ign-transport11-homebrew-amd64/59/consoleFull

Log output:

33: [ RUN      ] ignTest.TopicInfo
33: /Users/jenkins/jenkins-agent/workspace/gz_transport-ci-ign-transport11-homebrew-amd64/gz-transport/src/cmd/gz_TEST.cc:135: Failure
33: Value of: infoFound
33:   Actual: false
33: Expected: true
33: OUTPUT[No publishers on topic [/foo]
33: ] Size[30]. Expected Size=50
33: 
33: /Users/jenkins/jenkins-agent/workspace/gz_transport-ci-ign-transport11-homebrew-amd64/gz-transport/src/cmd/gz_TEST.cc:138: Failure
33: Value of: output.find("ignition.msgs.Vector3d") != std::string::npos
33:   Actual: false
33: Expected: true
33: [  FAILED  ] ignTest.TopicInfo (26598 ms)

Might be related to #509

@scpeters
Copy link
Member

I suspect that this is a flaky test. It is now failing consistently on macOS since build 58, though it still passes on Ubuntu. There were no changes in the ign-transport11 branch from build 57 to build 58, and I compared the log files and did not see any significant changes in dependencies. Looking at the code for this failing test case, I see it starts a publisher and waits a few seconds, which seems like a race condition to me.

Crola1702 added a commit to osrf/buildfarm-tools that referenced this issue Jul 17, 2024
@azeey azeey mentioned this issue Aug 9, 2024
@Crola1702
Copy link
Contributor Author

This is still a consistent issue in gz-transport-11

@azeey
Copy link
Contributor

azeey commented Nov 5, 2024

Fixed by #551.

@azeey azeey closed this as completed Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🏯 fortress Ignition Fortress
Projects
Status: Done
Development

No branches or pull requests

3 participants