-
Notifications
You must be signed in to change notification settings - Fork 42
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
Make test_tracetools ping pubs/subs transient_local #125
Make test_tracetools ping pubs/subs transient_local #125
Conversation
This will make sure that the initial `/ping` message is received no matter the launch order of the `*ping` and `*pong` executables. Also, given this guarantee, cancel the timer after the initial `/ping` message. Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
Once this is approved, I'll trigger CI and will use |
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.
i think setting the durability with transient local is the best option here, lgtm!
Alright, then let's go ahead with this. This way we'll have the long weekend to see if it works. |
@Mergifyio backport jazzy |
✅ Backports have been created
|
This will make sure that the initial `/ping` message is received no matter the launch order of the `*ping` and `*pong` executables. Also, given this guarantee, cancel the timer after the initial `/ping` message. Finally, add some helpful debug logs. Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai> (cherry picked from commit 00a4e99)
This will make sure that the initial `/ping` message is received no matter the launch order of the `*ping` and `*pong` executables. Also, given this guarantee, cancel the timer after the initial `/ping` message. Finally, add some helpful debug logs. Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai> (cherry picked from commit 00a4e99) Co-authored-by: Christophe Bedard <christophe.bedard@apex.ai>
Fixes #124
See an overview of the tests in #124 (comment)
This will make sure that the initial
/ping
message is received no matter the launch order of the*ping
and*pong
executables.Also, given this guarantee, cancel the timer after the initial
/ping
message.Finally, add some helpful debug logs.