diff --git a/docs/develop-resources/api-conventions/validations.md b/docs/develop-resources/api-conventions/validations.md index 8381d63525..53c5f86bf8 100644 --- a/docs/develop-resources/api-conventions/validations.md +++ b/docs/develop-resources/api-conventions/validations.md @@ -101,7 +101,7 @@ If the field contains more complicated GCP behavior, for example the GCP server Parent field should be required and immutable. -Note that parent is not a field name. It refers to the pre-requisite of the resource in question. You can identify the parent field through to the cloud resource's parent segment in the URL. For example, for GKE clusters, the [GET request URL](https://cloud.google.com/kubernetes-engine/docs/reference/rest/v1/projects.zones.clusters/get) `https://container.googleapis.com/v1/projects/{projectId}/zones/{zone}/clusters/{clusterId}` has `projectId` and `zone` in the parent segment. Therefore, for GKE clusters, projectRef and zone will be the parent fields. +Note that parent is not a field name. It refers to the prerequisite of the resource in question. You can identify the parent field through to the cloud resource's parent segment in the URL. For example, for GKE clusters, the [GET request URL](https://cloud.google.com/kubernetes-engine/docs/reference/rest/v1/projects.zones.clusters/get) `https://container.googleapis.com/v1/projects/{projectId}/zones/{zone}/clusters/{clusterId}` has `projectId` and `zone` in the parent segment. Therefore, for GKE clusters, projectRef and zone will be the parent fields. ### Required diff --git a/pkg/test/controller/reconcile.go b/pkg/test/controller/reconcile.go index ae92f01812..802f59f20b 100644 --- a/pkg/test/controller/reconcile.go +++ b/pkg/test/controller/reconcile.go @@ -171,7 +171,7 @@ func isTransientError(t *testing.T, err error) bool { // We don't know the exact error currently, use string matching for now... // // Example error: - // reconcile.go:175: error was not considered transient; chain is [[*errors.errorString: Update call failed: error applying desired state: summary: failed pre-requisites: missing permission on "billingAccounts/0162D7-7B0CB6-ED962E": billing.resourceAssociations.create]] + // reconcile.go:175: error was not considered transient; chain is [[*errors.errorString: Update call failed: error applying desired state: summary: failed prerequisites: missing permission on "billingAccounts/0162D7-7B0CB6-ED962E": billing.resourceAssociations.create]] if strings.Contains(errorMessage, "missing permission on") { t.Logf("internal error found; considered transient; chain is %v", chain) return true