Skip to content

[foxy backport #1635] Fix action server deadlock issue that caused by other mutexes locked in CancelCallback#1654

Merged
ivanpauno merged 1 commit intoros2:foxyfrom
KavenYau:foxy-backport-PR1635
May 5, 2021
Merged

[foxy backport #1635] Fix action server deadlock issue that caused by other mutexes locked in CancelCallback#1654
ivanpauno merged 1 commit intoros2:foxyfrom
KavenYau:foxy-backport-PR1635

Conversation

@KavenYau
Copy link
Contributor

@KavenYau KavenYau commented May 2, 2021

foxy backport for #1635.

Signed-off-by: Kaven Yau kavenyau@foxmail.com

…in CancelCallback (ros2#1635)

Signed-off-by: Kaven Yau <kavenyau@foxmail.com>
Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

change lgtm

@fujitatomoya
Copy link
Collaborator

@ivanpauno is this supposed to be done by using mergify like #1635 (comment)?

@clalancette
Copy link
Contributor

@ivanpauno is this supposed to be done by using mergify like #1635 (comment)?

Using mergify isn't required; it just makes backports easier. If someone does the backport by hand, that's fine too.

@ivanpauno ivanpauno requested a review from jacobperron May 3, 2021 13:05
@jacobperron
Copy link
Member

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno ivanpauno merged commit 791c23a into ros2:foxy May 5, 2021
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