Skip to content

Commit 6888288

Browse files
committed
enhance validation to check refID empty and cloudCredRefID
1 parent c682ea5 commit 6888288

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

pkg/v1/instance.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,10 +88,16 @@ func ValidateListCreatedInstance(ctx context.Context, client CloudCreateTerminat
8888
if foundInstance == nil {
8989
validationErr = errors.Join(validationErr, fmt.Errorf("instance not found: %s", i.CloudID))
9090
}
91-
if foundInstance.Location != i.Location {
91+
if foundInstance.Location != i.Location { //nolint:gocritic // fine
9292
validationErr = errors.Join(validationErr, fmt.Errorf("location mismatch: %s != %s", foundInstance.Location, i.Location))
93+
} else if foundInstance.RefID == "" {
94+
validationErr = errors.Join(validationErr, fmt.Errorf("refID is empty"))
9395
} else if foundInstance.RefID != i.RefID {
9496
validationErr = errors.Join(validationErr, fmt.Errorf("refID mismatch: %s != %s", foundInstance.RefID, i.RefID))
97+
} else if foundInstance.CloudCredRefID == "" {
98+
validationErr = errors.Join(validationErr, fmt.Errorf("cloudCredRefID is empty"))
99+
} else if foundInstance.CloudCredRefID != i.CloudCredRefID {
100+
validationErr = errors.Join(validationErr, fmt.Errorf("cloudCredRefID mismatch: %s != %s", foundInstance.CloudCredRefID, i.CloudCredRefID))
95101
}
96102
return validationErr
97103
}

0 commit comments

Comments
 (0)