Merge #1782

1782: Cherry pick PR #1780 r=dsharma-dc a=dsharma-dc When nexus is being created with a single child, and that child goes into retire path before nexus is open, then the child gets persisted as unhealthy. This will cause volume to never be able to attach later on. Co-authored-by: Diwakar Sharma <[email protected]>
openebs · Dec 11, 2024 · d6fcd63 · d6fcd63
2 parents 0d8ac9c + dbf97b6
commit d6fcd63
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/io-engine/src/bdev/nexus/nexus_persistence.rs b/io-engine/src/bdev/nexus/nexus_persistence.rs
@@ -104,6 +104,15 @@ impl<'n> Nexus<'n> {
                     };
                     nexus_info.children.push(child_info);
                 });
+                // We started with this child because it was healthy in etcd, or isn't there at all.
+                // Being unhealthy here means it is undergoing a fault/retire before nexus is open.
+                if nexus_info.children.len() == 1 && !nexus_info.children[0].healthy {
+                    warn!("{self:?} Not persisting: the only child went unhealthy during nexus creation");
+                    return Err(Error::NexusCreate {
+                        name: self.name.clone(),
+                        reason: "only child is unhealthy".to_string(),
+                    });
+                }
             }
             PersistOp::AddChild {
                 child_uri,