address context shutdown racy condition.

[fujitatomoya]

Description

retarget #3217 to lyrical

Fixes # (issue)

Is this user-facing behavior change?

Did you use Generative AI?

Additional Information

[fujitatomoya]

this is ABI compatibility change, so that we can put this to lyrical and downstream branches.

[mergify]

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

• Queue this pull request

[fujitatomoya]

Pulls: #3219
Gist: https://gist.githubusercontent.com/fujitatomoya/977ef70875dc4467b6f510f9db8e39a5/raw/2cb0f02adf8bf5ad334675d3d51445cf204e17ed/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp
ROS Distro: lyrical
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/20058

• Linux
• Linux-aarch64
• Linux-rhel
• Windows

[skyegalaxy]

lgtm with green CI

[fujitatomoya]

• Linux
• Linux-aarch64
• Linux-rhel
• Windows

[skyegalaxy]

https://github.com/Mergifyio backport humble jazzy kilted

[mergify]

backport humble jazzy kilted

✅ Backports have been created

Details

• #3224 address context shutdown racy condition. (backport #3219) has been created for branch humble but encountered conflicts

Cherry-pick of b723878 has failed:

On branch mergify/bp/humble/pr-3219
Your branch is up to date with 'origin/humble'.

You are currently cherry-picking commit b723878.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   rclcpp/src/rclcpp/signal_handler.cpp

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	both modified:   rclcpp/src/rclcpp/context.cpp
	deleted by us:   rclcpp/test/rclcpp/test_context.cpp

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

• #3225 address context shutdown racy condition. (backport #3219) has been created for branch jazzy but encountered conflicts

Cherry-pick of b723878 has failed:

On branch mergify/bp/jazzy/pr-3219
Your branch is up to date with 'origin/jazzy'.

You are currently cherry-picking commit b723878.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   rclcpp/src/rclcpp/signal_handler.cpp
	modified:   rclcpp/test/rclcpp/test_context.cpp

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   rclcpp/src/rclcpp/context.cpp

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

• #3226 address context shutdown racy condition. (backport #3219) has been created for branch kilted but encountered conflicts

Cherry-pick of b723878 has failed:

On branch mergify/bp/kilted/pr-3219
Your branch is up to date with 'origin/kilted'.

You are currently cherry-picking commit b723878.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   rclcpp/src/rclcpp/signal_handler.cpp
	modified:   rclcpp/test/rclcpp/test_context.cpp

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   rclcpp/src/rclcpp/context.cpp

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally