make "sled failures only" the default policy

oxidecomputer · Sep 2, 2024 · 637b780 · 637b780
1 parent 9ca4c5e
commit 637b780
Show file tree

Hide file tree

Showing 7 changed files with 498 additions and 8 deletions.
diff --git a/nexus/db-model/src/instance.rs b/nexus/db-model/src/instance.rs
@@ -109,7 +109,7 @@ impl Instance {
             ncpus: params.ncpus.into(),
             memory: params.memory.into(),
             hostname: params.hostname.to_string(),
-            auto_restart_policy: InstanceAutoRestart::Never,
+            auto_restart_policy: InstanceAutoRestart::default(),
             runtime_state,
 
             updater_gen: Generation::new(),

diff --git a/nexus/db-model/src/instance_auto_restart.rs b/nexus/db-model/src/instance_auto_restart.rs
@@ -18,18 +18,26 @@ impl_enum_type!(
 
     // Enum values
     Never => b"never"
+    SledFailuresOnly => b"sled_failures_only"
     AllFailures => b"all_failures"
 );
 
 impl InstanceAutoRestart {
     pub fn label(&self) -> &'static str {
         match self {
-            InstanceAutoRestart::Never => "never",
-            InstanceAutoRestart::AllFailures => "all_failures",
+            Self::Never => "never",
+            Self::SledFailuresOnly => "sled_failures_only",
+            Self::AllFailures => "all_failures",
         }
     }
 }
 
+impl Default for InstanceAutoRestart {
+    fn default() -> Self {
+        Self::SledFailuresOnly
+    }
+}
+
 impl fmt::Display for InstanceAutoRestart {
     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         write!(f, "{}", self.label())