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

Make RHEL CI happy! #730

Merged
merged 3 commits into from
Jun 17, 2022
Merged

Make RHEL CI happy! #730

merged 3 commits into from
Jun 17, 2022

Conversation

VX792
Copy link
Contributor

@VX792 VX792 commented Jun 7, 2022

Possible solution for issue #725

Haven't tested the build on Rhel, but based on the logs the error could be that the variables declared with the auto keyword in the descriptions.hpp file are deducted into initializer lists of const char*, and the compiler for some reason couldn't convert them into vectors of const char*..

Copy link
Contributor

@erickisos erickisos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@jaron-l
Copy link
Member

jaron-l commented Jun 7, 2022

Why is the RHEL workflow not running this PR?

@bmagyar
Copy link
Member

bmagyar commented Jun 8, 2022

🤦 because we made it master-only

@bmagyar
Copy link
Member

bmagyar commented Jun 8, 2022

Ok @jaron-l you are right... it's on a PR that was not followed-up on

@bmagyar
Copy link
Member

bmagyar commented Jun 8, 2022

ok, someone else please also confirm this but it seems to me that RHEL fails on master but succeeds to build here (don't check the status, look also at the logs pls)

@mergify
Copy link
Contributor

mergify bot commented Jun 8, 2022

This pull request is in conflict. Could you fix it @VX792?

@jaron-l
Copy link
Member

jaron-l commented Jun 8, 2022

ok, someone else please also confirm this but it seems to me that RHEL fails on master but succeeds to build here (don't check the status, look also at the logs pls)

Looks like controller manager is still failing tests. I'll run it locally and post the logs as my RHEL workflow doesn't post the error for some reason.

But it does build if that's all we care about.

@jaron-l
Copy link
Member

jaron-l commented Jun 8, 2022

ok, someone else please also confirm this but it seems to me that RHEL fails on master but succeeds to build here (don't check the status, look also at the logs pls)

Looks like controller manager is still failing tests. I'll run it locally and post the logs as my RHEL workflow doesn't post the error for some reason.

But it does build if that's all we care about.

So I think this PR does cause a new test to fail. It's in the controller_manager pkg at test/test_controller_manager_srvs.cpp::TestControllerManagerSrvs.list_controllers_srv. Here's the failed output:

4: /home/src/ros2_control/controller_manager/test/test_controller_manager_srvs.cpp:148: Failur
e
4: Value of: result->controller[0].claimed_interfaces
4: Expected: has 6 elements where
4: element #0 is equal to "joint2/velocity",
4: element #1 is equal to "joint3/velocity",
4: element #2 is equal to "joint2/max_acceleration",
4: element #3 is equal to "configuration/max_tcp_jerk",
4: element #4 is equal to "joint1/position",
4: element #5 is equal to "joint1/max_velocity"
4:   Actual: { "joint1/position", "joint1/max_velocity", "joint2/velocity", "joint3/velocity",
 "joint2/max_acceleration", "configuration/max_tcp_jerk" }, whose element #0 doesn't match
4: [  FAILED  ] TestControllerManagerSrvs.list_controllers_srv (175 ms)

Looks like the order of the vector changed.

@bmagyar
Copy link
Member

bmagyar commented Jun 17, 2022

Thanks @VX792 for the fix, as we discussed, UnorderedElementsAre fixed the problem!

@bmagyar bmagyar merged commit d632e89 into ros-controls:master Jun 17, 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.

5 participants