reconfigurator-cli could let you set SP versions (#8273)

Author: davepacheco

Cargo.lock

@@ -208,6 +208,7 @@ fn process_command(
         Commands::SledRemove(args) => cmd_sled_remove(sim, args),
         Commands::SledShow(args) => cmd_sled_show(sim, args),
         Commands::SledSetPolicy(args) => cmd_sled_set_policy(sim, args),
+        Commands::SledUpdateSp(args) => cmd_sled_update_sp(sim, args),
         Commands::SiloList => cmd_silo_list(sim),
         Commands::SiloAdd(args) => cmd_silo_add(sim, args),
         Commands::SiloRemove(args) => cmd_silo_remove(sim, args),
@@ -261,6 +262,8 @@ enum Commands {
     SledShow(SledArgs),
     /// set a sled's policy
     SledSetPolicy(SledSetPolicyArgs),
+    /// simulate updating the sled's SP versions
+    SledUpdateSp(SledUpdateSpArgs),
 
     /// list silos
     SiloList,
@@ -372,6 +375,20 @@ impl From<SledPolicyOpt> for SledPolicy {
     }
 }
 
+#[derive(Debug, Args)]
+struct SledUpdateSpArgs {
+    /// id of the sled
+    sled_id: SledUuid,
+
+    /// sets the version reported for the SP active slot
+    #[clap(long, required_unless_present_any = &["inactive"])]
+    active: Option<ArtifactVersion>,
+
+    /// sets the version reported for the SP inactive slot
+    #[clap(long, required_unless_present_any = &["active"])]
+    inactive: Option<ExpectedVersion>,
+}
+
 #[derive(Debug, Args)]
 struct SledRemoveArgs {
     /// id of the sled
@@ -885,18 +902,22 @@ fn cmd_sled_show(
     args: SledArgs,
 ) -> anyhow::Result<Option<String>> {
     let state = sim.current_state();
-    let planning_input = state
-        .system()
-        .description()
+    let description = state.system().description();
+    let sled_id = args.sled_id;
+    let sp_active_version = description.sled_sp_active_version(sled_id)?;
+    let sp_inactive_version = description.sled_sp_inactive_version(sled_id)?;
+    let planning_input = description
         .to_planning_input_builder()
         .context("failed to generate planning_input builder")?
         .build();
-    let sled_id = args.sled_id;
-    let sled_resources =
-        &planning_input.sled_lookup(args.filter, sled_id)?.resources;
+    let sled = planning_input.sled_lookup(args.filter, sled_id)?;
+    let sled_resources = &sled.resources;
     let mut s = String::new();
     swriteln!(s, "sled {}", sled_id);
+    swriteln!(s, "serial {}", sled.baseboard_id.serial_number);
     swriteln!(s, "subnet {}", sled_resources.subnet.net());
+    swriteln!(s, "SP active version:   {:?}", sp_active_version);
+    swriteln!(s, "SP inactive version: {:?}", sp_inactive_version);
     swriteln!(s, "zpools ({}):", sled_resources.zpools.len());
     for (zpool, disk) in &sled_resources.zpools {
         swriteln!(s, "    {:?}", zpool);
@@ -924,6 +945,46 @@ fn cmd_sled_set_policy(
     Ok(Some(format!("set sled {} policy to {}", args.sled_id, args.policy)))
 }
 
+fn cmd_sled_update_sp(
+    sim: &mut ReconfiguratorSim,
+    args: SledUpdateSpArgs,
+) -> anyhow::Result<Option<String>> {
+    let mut labels = Vec::new();
+    if let Some(active) = &args.active {
+        labels.push(format!("active -> {}", active));
+    }
+    if let Some(inactive) = &args.inactive {
+        labels.push(format!("inactive -> {}", inactive));
+    }
+
+    assert!(
+        !labels.is_empty(),
+        "clap configuration requires that at least one argument is specified"
+    );
+
+    let mut state = sim.current_state().to_mut();
+    state.system_mut().description_mut().sled_update_sp_versions(
+        args.sled_id,
+        args.active,
+        args.inactive,
+    )?;
+
+    sim.commit_and_bump(
+        format!(
+            "reconfigurator-cli sled-update-sp: {}: {}",
+            args.sled_id,
+            labels.join(", "),
+        ),
+        state,
+    );
+
+    Ok(Some(format!(
+        "set sled {} SP versions: {}",
+        args.sled_id,
+        labels.join(", ")
+    )))
+}
+
 fn cmd_inventory_list(
     sim: &mut ReconfiguratorSim,
 ) -> anyhow::Result<Option<String>> {

dev-tools/reconfigurator-cli/src/lib.rs

@@ -13,6 +13,18 @@ sled-add 90c1102a-b9f5-4d88-92a2-60d54a2d98cc
 sled-add 04ef3330-c682-4a08-8def-fcc4bef31bcd
 sled-list
 
+sled-update-sp dde1c0e2-b10d-4621-b420-f179f7a7a00a
+sled-update-sp dde1c0e2-b10d-4621-b420-f179f7a7a00a --active 1.0.0
+sled-show dde1c0e2-b10d-4621-b420-f179f7a7a00a
+sled-update-sp dde1c0e2-b10d-4621-b420-f179f7a7a00a --inactive 2.0.0
+sled-show dde1c0e2-b10d-4621-b420-f179f7a7a00a
+sled-update-sp dde1c0e2-b10d-4621-b420-f179f7a7a00a --active 3.0.0
+sled-show dde1c0e2-b10d-4621-b420-f179f7a7a00a
+sled-update-sp dde1c0e2-b10d-4621-b420-f179f7a7a00a --active 4.0.0 --inactive invalid
+sled-show dde1c0e2-b10d-4621-b420-f179f7a7a00a
+sled-update-sp dde1c0e2-b10d-4621-b420-f179f7a7a00a --active 4.0.0 --inactive 5.0.0
+sled-show dde1c0e2-b10d-4621-b420-f179f7a7a00a
+
 inventory-generate
 inventory-list
 

dev-tools/reconfigurator-cli/tests/input/cmds.txt

@@ -37,7 +37,10 @@ T ENA ID                                   PARENT
 
 > sled-show 2eb69596-f081-4e2d-9425-9994926e0832
 sled 2eb69596-f081-4e2d-9425-9994926e0832
+serial serial1
 subnet fd00:1122:3344:102::/64
+SP active version:   Some("0.0.1")
+SP inactive version: None
 zpools (10):
     088ed702-551e-453b-80d7-57700372a844 (zpool)
     SledDisk { disk_identity: DiskIdentity { vendor: "fake-vendor", model: "fake-model", serial: "serial-088ed702-551e-453b-80d7-57700372a844" }, disk_id: b2850ccb-4ac7-4034-aeab-b1cd582d407b (physical_disk), policy: InService, state: Active }
@@ -395,7 +398,10 @@ T ENA ID                                   PARENT
 
 > sled-show 89d02b1b-478c-401a-8e28-7a26f74fa41b
 sled 89d02b1b-478c-401a-8e28-7a26f74fa41b
+serial serial0
 subnet fd00:1122:3344:101::/64
+SP active version:   Some("0.0.1")
+SP inactive version: None
 zpools (4):
     44fa7024-c2bc-4d2c-b478-c4997e4aece8 (zpool)
     SledDisk { disk_identity: DiskIdentity { vendor: "fake-vendor", model: "fake-model", serial: "serial-44fa7024-c2bc-4d2c-b478-c4997e4aece8" }, disk_id: 2a15b33c-dd0e-45b7-aba9-d05f40f030ff (physical_disk), policy: InService, state: Active }

dev-tools/reconfigurator-cli/tests/output/cmds-example-stdout

@@ -0,0 +1,7 @@
+error: the following required arguments were not provided:
+  --active <ACTIVE>
+  --inactive <INACTIVE>
+
+Usage: sled-update-sp --active <ACTIVE> --inactive <INACTIVE> <SLED_ID>
+
+For more information, try '--help'.