-
Notifications
You must be signed in to change notification settings - Fork 310
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
DAOS-16013 tools: Query all pool targets by default #14783
Conversation
Running `daos pool query-targets` with just a rank argument should query all targets on that rank. Required-githooks: true Change-Id: Ib5fff358769679f8c282372fdfa8eaba2b5471cd Signed-off-by: Michael MacDonald <mjmac@google.com>
Ticket title is 'daos pool query-targets outputs nothing' |
For symmetry should the dmg pool query-targets command support the same? |
Hmm. Missed that! Yes, it should. Thanks for pointing it out, will push an update. |
Required-githooks: true Change-Id: Ia89439318f767b386f29fdc3be7b32b6d229ee07
Required-githooks: true Change-Id: I8e6f475c41e055c5aa687df0a398082daa06f887 Signed-off-by: Michael MacDonald <mjmac@google.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.
Can you please add some unit tests to verify this change.
Required-githooks: true Change-Id: I413d017d400540ebbcd3322608bffcadf87e2c46 Signed-off-by: Michael MacDonald <mjmac@google.com>
It's not easy to thoroughly test this, but I've added what I can without heroics. |
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
Running `(daos|dmg) pool query-targets` with just a rank argument should query all targets on that rank. Signed-off-by: Michael MacDonald <mjmac@google.com>
Running `(daos|dmg) pool query-targets` with just a rank argument should query all targets on that rank. Signed-off-by: Michael MacDonald <mjmac@google.com>
Running
daos pool query-targets
with just a rankargument should query all targets on that rank.
Required-githooks: true
Change-Id: Ib5fff358769679f8c282372fdfa8eaba2b5471cd
Signed-off-by: Michael MacDonald mjmac@google.com