fix: Add comment to explain field

dlipovetsky · dlipovetsky · commit 1217beb310a7 · 2025-07-24T12:07:29.000-07:00
diff --git a/pkg/webhook/preflight/nutanix/credentials.go b/pkg/webhook/preflight/nutanix/credentials.go
@@ -222,6 +222,8 @@ func newCredentialsCheck(
 				"Failed to validate credentials: %s. This is usually a temporary error. Please retry.", ///nolint:lll // Message is long.
 				err,
 			),
+			// We do not add ".url" or ".credentials.secretRef" to the field, because we do not know
+			// if the error is related to the URL, or the credentials.
 			Field: "$.spec.topology.variables[?@.name==\"clusterConfig\"].value.nutanix.prismCentralEndpoint",
 		},
 	)

Original file line number	Diff line number	Diff line change
`@@ -222,6 +222,8 @@ func newCredentialsCheck(`
`222`	`222`	`"Failed to validate credentials: %s. This is usually a temporary error. Please retry.", ///nolint:lll // Message is long.`
`223`	`223`	`err,`
`224`	`224`	`),`
	`225`	`+ // We do not add ".url" or ".credentials.secretRef" to the field, because we do not know`
	`226`	`+ // if the error is related to the URL, or the credentials.`
`225`	`227`	`Field: "$.spec.topology.variables[[email protected]==\"clusterConfig\"].value.nutanix.prismCentralEndpoint",`
`226`	`228`	`},`
`227`	`229`	`)`