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

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

Merged
merged 12 commits into from
Apr 1, 2020

Conversation

jaisontj
Copy link
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

Copy link

@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! 👍

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>
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
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
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

dabonnie commented Apr 1, 2020

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

@mm318
Copy link
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
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.

5 participants