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

[#5333 2/6] Region snapshot replacement omdb cmds #6279

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

jmpesp
Copy link
Contributor

@jmpesp jmpesp commented Aug 9, 2024

This commit adds some commands to omdb related to region snapshot replacement:

Usage: omdb db region-snapshot-replacement [OPTIONS] <COMMAND>

Commands:
  list     List region snapshot replacement requests
  status   Show current region snapshot replacements and their status
  info     Show detailed information for a region snapshot replacement
  request  Manually request a region snapshot replacement

list will list all region snapshot replacement requests, along with request time and state

status will show a summary of all non-complete region snapshot replacements, along with their state and progress.

info will show details for a single region snapshot replacement.

Finally, request will request that a region snapshot be replaced, and return the ID of the replacement.

This commit adds some commands to omdb related to region snapshot
replacement:

    Usage: omdb db region-snapshot-replacement [OPTIONS] <COMMAND>

    Commands:
      list     List region snapshot replacement requests
      status   Show current region snapshot replacements and their status
      info     Show detailed information for a region snapshot replacement
      request  Manually request a region snapshot replacement

`list` will list all region snapshot replacement requests, along with
request time and state

`status` will show a summary of all non-complete region snapshot
replacements, along with their state and progress.

`info` will show details for a single region snapshot replacement.

Finally, `request` will request that a region snapshot be replaced, and
return the ID of the replacement.
@jmpesp jmpesp requested review from andrewjstone and leftwo August 9, 2024 20:29
Copy link
Contributor

@leftwo leftwo 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 been using the full version of these omdb changes for a while now.


datastore
.insert_region_snapshot_replacement_request_with_volume_id(
opctx,
Copy link
Contributor

Choose a reason for hiding this comment

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

This just puts an entry in the table, but there is no action taken on it yet, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct, no action is taken until the "region snapshot replacement start" background task picks it up.

@jmpesp
Copy link
Contributor Author

jmpesp commented Aug 9, 2024

I've been using the full version of these omdb changes for a while now.

Note that all sub command is now omdb db region-snapshot-replacement, not omdb db snapshot-replacement :)

@jmpesp jmpesp merged commit de3d777 into oxidecomputer:main Aug 12, 2024
16 checks passed
@jmpesp jmpesp deleted the snapshot_replacement_part_2 branch August 12, 2024 16:54
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.

2 participants