Update progenitor from v0.4.0 -> v0.5.0

clients/dns-service-client/src/lib.rs

@@ -29,8 +29,10 @@ pub fn is_retryable(error: &DnsConfigError<crate::types::Error>) -> bool {
     let response_value = match error {
         DnsConfigError::CommunicationError(_) => return true,
         DnsConfigError::InvalidRequest(_)
-        | DnsConfigError::InvalidResponsePayload(_)
-        | DnsConfigError::UnexpectedResponse(_) => return false,
+        | DnsConfigError::InvalidResponsePayload(_, _)
+        | DnsConfigError::UnexpectedResponse(_)
+        | DnsConfigError::InvalidUpgrade(_)
+        | DnsConfigError::ResponseBodyError(_) => return false,
         DnsConfigError::ErrorResponse(response_value) => response_value,
     };
 

common/src/api/external/error.rs

@@ -523,6 +523,7 @@ impl<T: ClientError> From<progenitor::progenitor_client::Error<T>> for Error {
             // our version constraints (i.e. that the call was to a newer
             // service with an incompatible response).
             progenitor::progenitor_client::Error::InvalidResponsePayload(
+                _bytes,
                 ee,
             ) => Error::internal_error(&format!(
                 "InvalidResponsePayload: {}",
@@ -539,6 +540,12 @@ impl<T: ClientError> From<progenitor::progenitor_client::Error<T>> for Error {
                     r.status(),
                 ))
             }
+            progenitor::progenitor_client::Error::InvalidUpgrade(e) => {
+                Error::internal_error(&format!("InvalidUpgrade: {e}",))
+            }
+            progenitor::progenitor_client::Error::ResponseBodyError(_) => {
+                Error::internal_error(&format!("ResponseBodyError: {e}",))
+            }
         }
     }
 }

nexus/src/app/sagas/switch_port_settings_apply.rs

@@ -223,7 +223,7 @@ async fn spa_undo_ensure_switch_port_settings(
     let log = sagactx.user_data().log();
 
     let port_id: PortId = PortId::from_str(&params.switch_port_name)
-        .map_err(|e| external::Error::internal_error(e))?;
+        .map_err(|e| external::Error::internal_error(e.to_string().as_str()))?;
 
     let orig_port_settings_id = sagactx
         .lookup::<Option<Uuid>>("original_switch_port_settings_id")

nexus/src/app/sagas/switch_port_settings_clear.rs

@@ -179,7 +179,7 @@ async fn spa_undo_clear_switch_port_settings(
     let log = sagactx.user_data().log();
 
     let port_id: PortId = PortId::from_str(&params.port_name)
-        .map_err(|e| external::Error::internal_error(e))?;
+        .map_err(|e| external::Error::internal_error(e.to_string().as_str()))?;
 
     let orig_port_settings_id = sagactx
         .lookup::<Option<Uuid>>("original_switch_port_settings_id")

openapi/bootstrap-agent.json

@@ -355,6 +355,7 @@
         ]
       },
       "Certificate": {
+        "description": "Certificate\n\n<details><summary>JSON schema</summary>\n\n```json { \"type\": \"object\", \"required\": [ \"cert\", \"key\" ], \"properties\": { \"cert\": { \"type\": \"string\" }, \"key\": { \"type\": \"string\" } } } ``` </details>",
         "type": "object",
         "properties": {
           "cert": {
@@ -903,6 +904,7 @@
         "format": "uuid"
       },
       "RecoverySiloConfig": {
+        "description": "RecoverySiloConfig\n\n<details><summary>JSON schema</summary>\n\n```json { \"type\": \"object\", \"required\": [ \"silo_name\", \"user_name\", \"user_password_hash\" ], \"properties\": { \"silo_name\": { \"$ref\": \"#/components/schemas/Name\" }, \"user_name\": { \"$ref\": \"#/components/schemas/UserId\" }, \"user_password_hash\": { \"$ref\": \"#/components/schemas/NewPasswordHash\" } } } ``` </details>",
         "type": "object",
         "properties": {
           "silo_name": {
@@ -967,7 +969,7 @@
         ]
       },
       "UserId": {
-        "description": "Names must begin with a lower case ASCII letter, be composed exclusively of lowercase ASCII, uppercase ASCII, numbers, and '-', and may not end with a '-'. Names cannot be a UUID though they may contain a UUID.",
+        "description": "Names must begin with a lower case ASCII letter, be composed exclusively of lowercase ASCII, uppercase ASCII, numbers, and '-', and may not end with a '-'. Names cannot be a UUID though they may contain a UUID.\n\n<details><summary>JSON schema</summary>\n\n```json { \"title\": \"A name unique within the parent collection\", \"description\": \"Names must begin with a lower case ASCII letter, be composed exclusively of lowercase ASCII, uppercase ASCII, numbers, and '-', and may not end with a '-'. Names cannot be a UUID though they may contain a UUID.\", \"type\": \"string\", \"maxLength\": 63, \"minLength\": 1, \"pattern\": \"^(?![0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$)^[a-z]([a-zA-Z0-9-]*[a-zA-Z0-9]+)?$\" } ``` </details>",
         "type": "string"
       }
     },