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-transport12-homebrew UNIT_gz_TEST gzTest.TopicInfo failing #509

Open
Crola1702 opened this issue Jul 12, 2024 · 2 comments
Open
Assignees
Labels
bug Something isn't working 🌱 garden Ignition Garden

Comments

@Crola1702
Copy link
Contributor

Environment

  • OS Version: MacOS
  • Source gz-transport12 build

Steps to reproduce

  1. Run a build in gz-transport-12
  2. See UNIT_gz_TEST fail

Output

Reference build: https://build.osrfoundation.org/job/gz_transport-ci-gz-transport12-homebrew-amd64/59/

Test regression:

Log output:

33: [ RUN      ] gzTest.TopicInfo
33: /Users/jenkins/jenkins-agent/workspace/gz_transport-ci-gz-transport12-homebrew-amd64/gz-transport/src/cmd/gz_TEST.cc:124: Failure
33: Value of: output
33:   Actual: false
33: Expected: true
33: OUTPUT[���\0`\0\0Д��\0`\0\0z�����\0\0�\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0�Ս���\0\0\0p����\0\0@����\0\0\0�\0\0\0\0\0\0\0�@����\0\0�] Size[80]. Expected Size=50
33: 
33: [  FAILED  ] gzTest.TopicInfo (26576 ms)

And then:

Failed to read test report file /Users/jenkins/jenkins-agent/workspace/gz_transport-ci-gz-transport12-homebrew-amd64/build/test_results/UNIT_gz_TEST.xml
org.dom4j.DocumentException: Invalid byte 1 of 1-byte UTF-8 sequence.

Repository doesn't have any new changes.

@Crola1702
Copy link
Contributor Author

Crola1702 commented Sep 16, 2024

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

@azeey azeey assigned azeey and unassigned scpeters Nov 4, 2024
@azeey
Copy link
Contributor

azeey commented Nov 5, 2024

The ign-transport11 version of this bug was fixed in #551. But this is for Garden, which is going EOL this month. It is not trivial to forward port #551 and since this is really a timing issue in the test and not a bug in the gz-transport library, I'm inclined to close this issue without fixing it. If we have the same issue appear in gz-transport13, we can fix it there.

@azeey azeey mentioned this issue Nov 7, 2024
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🌱 garden Ignition Garden
Projects
Status: In progress
Development

No branches or pull requests

3 participants