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

Add support for starting stopping and refreshing pools #879

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

mulkieran
Copy link
Member

@mulkieran mulkieran commented Apr 27, 2022

Closes #725
Related stratis-storage/project#296
Related #866

@mulkieran mulkieran self-assigned this Apr 27, 2022
@mulkieran
Copy link
Member Author

coverage task will fail.

@mulkieran mulkieran force-pushed the start-stop branch 5 times, most recently from fc5d472 to 8cd6276 Compare April 28, 2022 01:18
@mulkieran
Copy link
Member Author

stratisd dependent tests should all fail.

@mulkieran mulkieran force-pushed the start-stop branch 6 times, most recently from 6de8048 to 3ac648c Compare April 29, 2022 00:52
@mulkieran
Copy link
Member Author

rebased.

@mulkieran
Copy link
Member Author

The bug in listing stopped pools should be fixed now.

@mulkieran
Copy link
Member Author

rebased.

Copy link
Member

@bmr-cymru bmr-cymru left a comment

Choose a reason for hiding this comment

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

One possible typo in a comment but looks good otherwise

"""
Get a cell value or N/A if None.
Take a value that represents possibly inconsistency via option type.
Copy link
Member

Choose a reason for hiding this comment

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

^possibly^possible?

@mulkieran
Copy link
Member Author

This passes all tests again stratis-storage/stratisd#2949 .

@mulkieran
Copy link
Member Author

mulkieran commented Jun 6, 2022

Should we allow capture_key in the "pool start" subcommand? Right now we just have an ability to specify encryption method, I think that's best, actually.

@mulkieran mulkieran marked this pull request as ready for review June 16, 2022 13:26
@mulkieran
Copy link
Member Author

mulkieran commented Jun 16, 2022

All tests and coverage succeed when run against: stratis-storage/stratisd#2949 .

@mulkieran mulkieran changed the title Start stop Add support for starting stopping and refreshing pools Jun 16, 2022
@mulkieran mulkieran requested a review from jbaublitz June 16, 2022 13:28
Copy link
Member

@jbaublitz jbaublitz left a comment

Choose a reason for hiding this comment

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

I've tested this and it looks good!

Remove global unlock command and StratisCliAggregateError.

Add a start and stop command on pools.

Add a new global debug subcommand for refresh.

Add additional options to the pool list command for showing stopped
pools and detailed views on individual pools.

Signed-off-by: mulhern <[email protected]>
@mulkieran
Copy link
Member Author

rebased and squashed. All tests are expected to pass.

@mulkieran mulkieran merged commit 3ff4397 into stratis-storage:master Jun 24, 2022
@mulkieran mulkieran deleted the start-stop branch June 24, 2022 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Display information about "locked pools" by a regular CLI command
4 participants