-
Notifications
You must be signed in to change notification settings - Fork 23
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
trigger dump via command #3295
trigger dump via command #3295
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## devel #3295 +/- ##
==========================================
- Coverage 43.40% 43.36% -0.05%
==========================================
Files 97 97
Lines 11357 11381 +24
==========================================
+ Hits 4930 4935 +5
- Misses 6427 6446 +19 ☔ View full report in Codecov by Sentry. |
…adical.pilot into feature/command_msg
f8bdd8a
to
24525b4
Compare
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.
LGTM! Thanks!
This PR adds support for
which will result in
$ ls rp.session.thinkie.merzky.020078.0069/*json rp.session.thinkie.merzky.020078.0069/pmgr.0000.foo.json rp.session.thinkie.merzky.020078.0069/rp.session.thinkie.merzky.020078.0069.reg.foo.json rp.session.thinkie.merzky.020078.0069/rp.session.thinkie.merzky.020078.0069.reg.init.json rp.session.thinkie.merzky.020078.0069/tmgr.0000.foo.json
If
name=foo
is ommited, the normal registry and manager dumps occur, and the session then can be used for analytics.Note that the first argument must point to a client session sandbox - the
<SID>.reg.init.json
is used to extract the ZMQ endpoints of theCONTROL
pubsub over which the request message is sent.