Skip to content

remove I/O from signal handler. (backport #2986)#2995

Closed
mergify[bot] wants to merge 1 commit intokiltedfrom
mergify/bp/kilted/pr-2986
Closed

remove I/O from signal handler. (backport #2986)#2995
mergify[bot] wants to merge 1 commit intokiltedfrom
mergify/bp/kilted/pr-2986

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Nov 21, 2025

Description

Fixes #2982

Is this user-facing behavior change?

Yes this fixes the possible deadlock via IO.

Did you use Generative AI?

No.

Additional Information

Note

backport required for all downstream distros.

This is an automatic backport of pull request #2986 done by Mergify.

@fujitatomoya @mergify
remove I/O from signal handler. (#2986)
7767008 
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 95cb964)
@mergify mergify bot mentioned this pull request Nov 21, 2025
Merged
@fujitatomoya
Copy link
Copy Markdown
Collaborator

fujitatomoya commented Nov 21, 2025

Pulls: #2995, ros2/system_tests#577
Gist: https://gist.githubusercontent.com/fujitatomoya/1a3250e73075c2377ce164d91d221cef/raw/2de334166f0763af83c04a8b426de0c3dd356208/ros2.repos
BUILD args: --packages-above-and-dependencies ‎rclcll test_rclcpp
TEST args: --packages-above ‎rclcll test_rclcpp
ROS Distro: kilted
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17561

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@fujitatomoya fujitatomoya mentioned this pull request Nov 21, 2025
Closed
@fujitatomoya
Copy link
Copy Markdown
Collaborator

fujitatomoya commented Nov 21, 2025 •
edited by ahcorde
Loading

Pulls: #2995, ros2/system_tests#577
Gist: https://gist.githubusercontent.com/fujitatomoya/1a919e0b8d65564c60068e863de10e4a/raw/2de334166f0763af83c04a8b426de0c3dd356208/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp test_rclcpp
TEST args: --packages-above rclcpp test_rclcpp
ROS Distro: kilted
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17564

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@fujitatomoya
Copy link
Copy Markdown
Collaborator

fujitatomoya commented Nov 21, 2025

This is ABI breaking change, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fujitatomoya fujitatomoya fujitatomoya approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@fujitatomoya