remove unneeded sled agent param

oxidecomputer · Dec 19, 2024 · 7197618 · 7197618
1 parent 4aac06a
commit 7197618
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 12 deletions.
diff --git a/nexus/src/app/sagas/instance_migrate.rs b/nexus/src/app/sagas/instance_migrate.rs
@@ -539,7 +539,6 @@ async fn sim_instance_migrate(
             InstanceStateChangeRequest::Migrate(
                 InstanceMigrationTargetParams {
                     src_propolis_addr: src_vmm_addr.to_string(),
-                    src_propolis_id,
                 },
             ),
         )

diff --git a/openapi/sled-agent.json b/openapi/sled-agent.json
@@ -4004,16 +4004,10 @@
           "src_propolis_addr": {
             "description": "The address of the Propolis server that will serve as the migration source.",
             "type": "string"
-          },
-          "src_propolis_id": {
-            "description": "The Propolis ID of the migration source.",
-            "type": "string",
-            "format": "uuid"
           }
         },
         "required": [
-          "src_propolis_addr",
-          "src_propolis_id"
+          "src_propolis_addr"
         ]
       },
       "InstanceProperties": {

diff --git a/sled-agent/types/src/instance.rs b/sled-agent/types/src/instance.rs
@@ -184,10 +184,6 @@ pub struct VmmUnregisterResponse {
 /// migration.
 #[derive(Copy, Clone, Debug, Deserialize, Serialize, JsonSchema)]
 pub struct InstanceMigrationTargetParams {
-    /// The Propolis ID of the migration source.
-    // TODO(gjc) try to get rid of this, migration targets don't use it anymore
-    pub src_propolis_id: Uuid,
-
     /// The address of the Propolis server that will serve as the migration
     /// source.
     pub src_propolis_addr: SocketAddr,