Skip to content

Commit c9108d6

Browse files
committed
Corrected comments
1 parent c28e97d commit c9108d6

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

pkg/syncer/admissionhandler/validate_cnsfileaccessconfig.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -307,23 +307,23 @@ func validateProviderServiceAccount(ctx context.Context, namespace, serviceAccou
307307
log := logger.GetLogger(ctx)
308308
log.Infof("Validating provider service account '%s' in namespace '%s'", serviceAccountName, namespace)
309309

310-
// Extract expected cluster name from service account name
311-
// serviceAccountName format: "{cluster-name}-pvcsi"
310+
// Extract expected vsphere cluster name from service account name
311+
// serviceAccountName format: "{vsphere-cluster-name}-pvcsi"
312312
if !strings.HasSuffix(serviceAccountName, "-pvcsi") {
313313
log.Infof("Service account '%s' does not follow cluster pattern (missing -pvcsi suffix)", serviceAccountName)
314314
return false, nil
315315
}
316316

317317
clusterName := strings.TrimSuffix(serviceAccountName, "-pvcsi")
318318
if clusterName == "" {
319-
log.Warnf("Empty cluster name extracted from service account '%s'", serviceAccountName)
319+
log.Warnf("Empty vsphere cluster name extracted from service account '%s'", serviceAccountName)
320320
return false, nil
321321
}
322322

323-
log.Infof("Extracted cluster name '%s' from service account '%s', searching in namespace '%s'",
323+
log.Infof("Extracted vsphere cluster name '%s' from service account '%s', searching in namespace '%s'",
324324
clusterName, serviceAccountName, namespace)
325325

326-
// If generic Cluster not found, try VSphereCluster resource
326+
// validate VSphereCluster resource exists
327327
found, err := validateVSphereClusterResource(ctx, clusterName, namespace)
328328
if err != nil {
329329
log.Warnf("Failed to check VSphereCluster resource: %v", err)
@@ -334,7 +334,7 @@ func validateProviderServiceAccount(ctx context.Context, namespace, serviceAccou
334334
return true, nil
335335
}
336336

337-
log.Infof("Neither Cluster nor VSphereCluster '%s' found in namespace '%s', service account '%s' is not valid",
337+
log.Infof("VSphereCluster with name :'%s' not found in namespace '%s', So service account '%s' is not valid",
338338
clusterName, namespace, serviceAccountName)
339339
return false, nil
340340
}

0 commit comments

Comments
 (0)