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

DAOS-16013 tools: Query all pool targets by default #14783

Merged
merged 4 commits into from
Jul 30, 2024
Merged

Conversation

mjmac
Copy link
Contributor

@mjmac mjmac commented Jul 18, 2024

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

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>
@mjmac mjmac requested review from a team as code owners July 18, 2024 15:52
Copy link

Ticket title is 'daos pool query-targets outputs nothing'
Status is 'In Review'
Labels: 'GCP'
https://daosio.atlassian.net/browse/DAOS-16013

kjacque
kjacque previously approved these changes Jul 18, 2024
@mjmac mjmac requested review from tanabarr, kccain and knard38 July 19, 2024 13:52
@kccain
Copy link
Contributor

kccain commented Jul 22, 2024

For symmetry should the dmg pool query-targets command support the same?

@mjmac
Copy link
Contributor Author

mjmac commented Jul 22, 2024

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.

mjmac added 2 commits July 23, 2024 20:34
Required-githooks: true

Change-Id: Ia89439318f767b386f29fdc3be7b32b6d229ee07
Required-githooks: true

Change-Id: I8e6f475c41e055c5aa687df0a398082daa06f887
Signed-off-by: Michael MacDonald <mjmac@google.com>
Copy link
Contributor

@tanabarr tanabarr left a 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>
@mjmac
Copy link
Contributor Author

mjmac commented Jul 24, 2024

Can you please add some unit tests to verify this change.

It's not easy to thoroughly test this, but I've added what I can without heroics.

@mjmac mjmac requested a review from tanabarr July 24, 2024 18:33
Copy link
Contributor

@tanabarr tanabarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mjmac mjmac merged commit ce2e20e into master Jul 30, 2024
52 checks passed
@mjmac mjmac deleted the mjmac/DAOS-16013 branch July 30, 2024 19:28
mjmac added a commit that referenced this pull request Jul 31, 2024
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>
mjmac added a commit that referenced this pull request Aug 5, 2024
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>
mjmac added a commit that referenced this pull request Aug 6, 2024
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>
@mjmac mjmac mentioned this pull request Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants