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
localhost_only prevails auto discovery options if enabled. #1069
Conversation
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@clalancette @sloretz can you take a look when you have time?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small changes to the wording of the error messages, then I think this is good for me.
@fujitatomoya Thank you so much for doing this.
@clalancette thanks for the review, comments are addressed with 6fa485b |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Arg, sorry. I missed one other typo.
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
6fa485b
to
9951e88
Compare
@Mergifyio backport iron |
✅ Backports have been created
|
* localhost_only prevails auto discovery options if enabled. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> (cherry picked from commit 101e406)
@clalancette thanks for the review. |
* localhost_only prevails auto discovery options if enabled. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
follow up of ros2/ros2#1359.
during deprecation period for
locaohost_only
option for Iron,locaohost_only
prevails over discovery option which has been added with ros2/ros2#1359 as new feature.see ros2/ros2_documentation#3519 (comment) for more details and discussion.