Skip to content

Commit

Permalink
Add omdb db validate subcommands (#4376)
Browse files Browse the repository at this point in the history
In order to diagnose if customer-support#57 is caused by _not_ having
the fixes for omicron#3866, add a few commands to omdb to validate the
contents of the database:

    Usage: omdb db validate <COMMAND>

    Commands:
      validate-volume-references  Validate each `volume_references` column in the region snapshots table
      validate-region-snapshots   Find either region snapshots Nexus knows about that the corresponding Crucible agent says were deleted, or
                                      region snapshots that Nexus doesn't know about
      help                        Print this message or the help of the given subcommand(s)

This commit also adds an environment variable OMDB_FETCH_LIMIT, which
overrides the default fetch limit.
  • Loading branch information
jmpesp authored Nov 20, 2023
1 parent 8238581 commit 7f42bc9
Show file tree
Hide file tree
Showing 6 changed files with 473 additions and 5 deletions.
1 change: 1 addition & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion dev-tools/omdb/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ anyhow.workspace = true
async-bb8-diesel.workspace = true
chrono.workspace = true
clap.workspace = true
crucible-agent-client.workspace = true
diesel.workspace = true
dropshot.workspace = true
futures.workspace = true
Expand Down Expand Up @@ -39,10 +40,10 @@ tokio = { workspace = true, features = [ "full" ] }
uuid.workspace = true
ipnetwork.workspace = true
omicron-workspace-hack.workspace = true
nexus-test-utils.workspace = true

[dev-dependencies]
expectorate.workspace = true
nexus-test-utils.workspace = true
nexus-test-utils-macros.workspace = true
omicron-nexus.workspace = true
omicron-test-utils.workspace = true
Expand Down
Loading

0 comments on commit 7f42bc9

Please sign in to comment.