cover the case of ac without labels in selectACForVolume

Signed-off-by: Shi, Crane <crane.shi@emc.com>
dell · Jun 23, 2023 · b953f24 · b953f24
1 parent cba700d
commit b953f24
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/pkg/base/capacityplanner/node_capacity.go b/pkg/base/capacityplanner/node_capacity.go
@@ -154,7 +154,11 @@ func (nc *nodeCapacity) selectACForVolume(vol *genV1.Volume) *accrd.AvailableCap
  }
 
  for _, ac := range nc.acsOrder[vol.StorageClass] {
- if requiredSize <= nc.acs[ac].Spec.Size && nc.acs[ac].Labels[v1.StorageGroupLabelKey] == vol.StorageGroup {
+ var acSGLabel string
+ if nc.acs[ac].Labels != nil {
+ acSGLabel = nc.acs[ac].Labels[v1.StorageGroupLabelKey]
+ }
+ if requiredSize <= nc.acs[ac].Spec.Size && acSGLabel == vol.StorageGroup {
  // check if AC is reserved
  reservation, ok := nc.reservedACs[ac]