Skip to content

Demo to show the working of the incompatible_qos event callbacks.#416

Merged
ivanpauno merged 12 commits intoros2:masterfrom
aws-ros-dev:jaisontj/incompatible_qos
Apr 1, 2020
Merged

Demo to show the working of the incompatible_qos event callbacks.#416
ivanpauno merged 12 commits intoros2:masterfrom
aws-ros-dev:jaisontj/incompatible_qos

Conversation

@jaisontj
Copy link
Copy Markdown
Contributor

@jaisontj jaisontj commented Nov 15, 2019

Related to this feature request. The design and implementation details can also be found there.

Signed-off-by: Jaison Titus jaisontj92@gmail.com

@jaisontj jaisontj force-pushed the jaisontj/incompatible_qos branch 2 times, most recently from dae0e45 to 586626a Compare November 19, 2019 17:50
@jaisontj jaisontj marked this pull request as ready for review November 22, 2019 01:38
Copy link
Copy Markdown

@piraka9011 piraka9011 left a comment

Choose a reason for hiding this comment

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

Small nits, but LGTM! 👍

@mm318 mm318 force-pushed the jaisontj/incompatible_qos branch from 115b22c to 0bff4ec Compare February 24, 2020 19:38
jaisontj and others added 6 commits March 22, 2020 10:09
Signed-off-by: Jaison Titus <jaisontj92@gmail.com>
Signed-off-by: Jaison Titus <jaisontj92@gmail.com>
Signed-off-by: Jaison Titus <jaisontj92@gmail.com>
Signed-off-by: Jaison Titus <jaisontj92@gmail.com>
Signed-off-by: Jaison Titus <jaisontj92@gmail.com>
Signed-off-by: Miaofei <miaofei@amazon.com>
@mm318 mm318 force-pushed the jaisontj/incompatible_qos branch from 0bff4ec to a178cd5 Compare March 22, 2020 18:21
…istent

Signed-off-by: Miaofei <miaofei@amazon.com>
mm318 added 3 commits March 24, 2020 15:56
Signed-off-by: Miaofei <miaofei@amazon.com>
Signed-off-by: Miaofei <miaofei@amazon.com>
Signed-off-by: Miaofei <miaofei@amazon.com>
Signed-off-by: Miaofei <miaofei@amazon.com>
Copy link
Copy Markdown
Member

@ivanpauno ivanpauno left a comment

Choose a reason for hiding this comment

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

LGTM!

Signed-off-by: Miaofei <miaofei@amazon.com>
@mm318
Copy link
Copy Markdown
Member

mm318 commented Apr 1, 2020

@ros2/aws-oncall - please run this CI job
Gist: https://gist.githubusercontent.com/mm318/576eae9c880a670ce32d73793fb78a91/raw/7274e2f29d97e2ddbec6805759a3de29e5d30c1b/ros2_qos_demos.repos
BUILD args: --packages-up-to quality_of_service_demo_cpp quality_of_service_demo_py
TEST args: --packages-select quality_of_service_demo_cpp quality_of_service_demo_py
Job: ci_launcher

@dabonnie
Copy link
Copy Markdown

dabonnie commented Apr 1, 2020

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

@mm318
Copy link
Copy Markdown
Member

mm318 commented Apr 1, 2020

@ivanpauno, there is a random buildfarm failure for the Linux-aarch64 run. Can we merge this pull request?

@ivanpauno
Copy link
Copy Markdown
Member

@ivanpauno, there is a random buildfarm failure for the Linux-aarch64 run. Can we merge this pull request?

Yes, I think it's safe to merge (considering that the last commit are just style changes and this PR has already passed CI before).
Thanks for the contribution!

@ivanpauno ivanpauno merged commit df9a85e into ros2:master Apr 1, 2020
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.

7 participants