[omdb] Add a couple simple Sled Agent queries #4126
Conversation
omdb/src/bin/omdb/main.rs
Outdated
| OmdbCommands::Db(db) => db.run_cmd(&log).await, | ||
| OmdbCommands::Nexus(nexus) => nexus.run_cmd(&log).await, | ||
| OmdbCommands::Sled(sled) => sled.run_cmd(&log).await, |
There was a problem hiding this comment.
I only wonder if this should be sled-agent (there are other things on the sled). If you do I'd carry that through everywhere -- e.g., the URL env var name too.
omdb/src/bin/omdb/sled.rs
Outdated
|
|
||
| #[derive(Debug, Subcommand)] | ||
| enum ZoneCommands { | ||
| /// Print list of all zones |
There was a problem hiding this comment.
| /// Print list of all zones | |
| /// Print list of control plane zones that the sled agent is configured to run |
(just trying to be precise about what it's printing so that someone doesn't interpret this as the same as zoneadm list)
edit: I just checked and it looks like this is only the running zones that are the subset of zones the sled agent is configured to run, I think.
There was a problem hiding this comment.
Updated, specifying that these are "running control plane zones". I think we could also expose the set of zones the sled was told to run, but this will require modifying the sled agent API.
omdb/src/bin/omdb/sled.rs
Outdated
|
|
||
| #[derive(Debug, Subcommand)] | ||
| enum ZpoolCommands { | ||
| /// Print list of all zpools |
There was a problem hiding this comment.
| /// Print list of all zpools | |
| /// Print list of all zpools managed by the sled agent |
|
I forgot to mention: you will need to merge with #4101 and regenerate the output with EXPECTORATE. |
Related to #1881