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

Infrom user why rt policy could not be set, infrom if is set. #1705

Merged

Conversation

mamueluth
Copy link
Member

Print out information why setting of rt policy could not be set. Additionally inform if successfully set the rt policy

Copy link

codecov bot commented Aug 22, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 86.80%. Comparing base (40ea191) to head (ce60aff).
Report is 3 commits behind head on master.

Files Patch % Lines
controller_manager/src/ros2_control_node.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1705      +/-   ##
==========================================
- Coverage   86.81%   86.80%   -0.01%     
==========================================
  Files         116      116              
  Lines       10595    10596       +1     
  Branches      975      975              
==========================================
  Hits         9198     9198              
- Misses       1047     1048       +1     
  Partials      350      350              
Flag Coverage Δ
unittests 86.80% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
controller_manager/src/ros2_control_node.cpp 89.28% <0.00%> (-3.31%) ⬇️

Copy link
Contributor

@christophfroehlich christophfroehlich left a comment

Choose a reason for hiding this comment

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

LGTM. I guess that there is no way to check this part of the code in our github CI, or would there be a possibility to fake the RT kernel inside a container? #610

@bmagyar bmagyar added backport-humble This label should be used by maintaines only! Label triggers PR backport to ROS2 humble. backport-iron This label should be used by maintaines only! Label triggers PR backport to ROS2 Iron. labels Aug 22, 2024
@bmagyar bmagyar merged commit dd352eb into ros-controls:master Aug 22, 2024
20 checks passed
mergify bot pushed a commit that referenced this pull request Aug 22, 2024
#1705)

(cherry picked from commit dd352eb)

# Conflicts:
#	controller_manager/src/ros2_control_node.cpp
mergify bot pushed a commit that referenced this pull request Aug 22, 2024
#1705)

(cherry picked from commit dd352eb)

# Conflicts:
#	controller_manager/src/ros2_control_node.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-humble This label should be used by maintaines only! Label triggers PR backport to ROS2 humble. backport-iron This label should be used by maintaines only! Label triggers PR backport to ROS2 Iron.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants