address review comments

Features: control Required-githooks: true Signed-off-by: Tom Nabarro <[email protected]>
daos-stack · Dec 19, 2024 · 3d13ba6 · 3d13ba6
1 parent 20d26e0
commit 3d13ba6
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/src/control/server/storage/scm/ipmctl.go b/src/control/server/storage/scm/ipmctl.go
@@ -201,16 +201,12 @@ func (cr *cmdRunner) handleFreeCapacity(sockSelector int, nrNsPerSock uint, regi
 		return nil, nil, errors.Wrap(err, "createNamespaces")
 	}
 
-	switch len(numaIDs) {
-	case 0:
+	if len(numaIDs) == 0 {
 		return nil, nil, errors.New("no namespaces created on regions with free capacity")
-	case 1:
-	default:
-		if sockSelector != sockAny {
-			return nil, nil,
-				errors.Errorf("unexpected number of numa nodes processed, want 1 got %d",
-					len(numaIDs))
-		}
+	}
+	if len(numaIDs) > 1 && sockSelector != sockAny {
+		return nil, nil, errors.Errorf("unexpected number of numa nodes processed, want 1 got %d",
+			len(numaIDs))
 	}
 	cr.log.Tracef("namespaces created on %v numa-nodes, fetching updated region details", numaIDs)