[CHORE] Spelling

CONTRIBUTING.md

@@ -2,7 +2,7 @@
 
 We'd love to accept your patches and contributions to this project. We use this
 GitHub project as our primary source of truth and the main development
-repository for Config Connetor. The source code in this project is also
+repository for Config Connector. The source code in this project is also
 mirrored to internal Google repository for the purposes of releases.
 
 ## Contributor License Agreement
@@ -114,7 +114,7 @@ repo to quickly set up a local dev environment.
     ./docker-setup.sh
     ```
 
-1.  Exit your current session, then SSH back in to the VM. Then run the
+1.  Exit your current session, then SSH back into the VM. Then run the
     following to ensure you have set up sudoless docker correctly:
 
     ```shell
@@ -149,7 +149,7 @@ repo to quickly set up a local dev environment.
     ```shell
     ./gcp-setup.sh
     ```
-1.  _(Optional)_ Verify that worload identity federation is [setup correctly](https://cloud.google.com/kubernetes-engine/docs/how-to/workload-identity#verify_the_setup).
+1.  _(Optional)_ Verify that workload identity federation is [setup correctly](https://cloud.google.com/kubernetes-engine/docs/how-to/workload-identity#verify_the_setup).
 
 1.  Now that you have everything set up, you can build your own images and then
     deploy the Config Connector CRDs and workloads (including controller
@@ -200,7 +200,7 @@ repo to quickly set up a local dev environment.
 The script `gcp-setup.sh` annotates your `default` namespace in the GKE cluster
 with a
 [project-id](https://cloud.google.com/config-connector/docs/how-to/organizing-resources/project-scoped-resources#annotate_namespace_configuration)
-annotation equals to your default GCP project id in gcloud. This enables Config
+annotation equal to your default GCP project id in gcloud. This enables Config
 Connector to create GCP resources in that default GCP project. We can validate
 by creating an Artifact Registry resource through Config Connector.
 
@@ -232,8 +232,7 @@ by creating an Artifact Registry resource through Config Connector.
 
 #### Looking for error logs
 
-You can look for error logs by checking the controller logs following the steps
-[here](https://cloud.google.com/config-connector/docs/troubleshooting#check-controller-logs).
+You can look for error logs by checking the controller logs following the [troubleshooting](https://cloud.google.com/config-connector/docs/troubleshooting#check-controller-logs).
 
 #### Pods fail to pull image
 When the cluster is created without providing a service account, a Compute Engine service account is created for the cluster. Users must grant the service account permission to pull images from the project registry.
@@ -367,14 +366,14 @@ can run it locally on your dev machine with the steps below.
 If you are adding a new resource, you need to follow the steps in [NewResourceFromTerraform.md](README.ChangingTerraform.md)
 to make code changes, add test data, and run the tests for your resource.
 
-If you are working on a existing resource, test yaml should exist under
+If you are working on an existing resource, test yaml should exist under
 ./pkg/test/resourcefixture/testdata/basic, you can run the test command directly
 to make sure the test can still pass. Example command:
 
 ```bash
-   # Export the environment variables needed in the dynamic tests if you haven't done it.
-   TEST_FOLDER_ID=123456789 go test -v -tags=integration ./pkg/controller/dynamic/ -test.run TestCreateNoChangeUpdateDelete -run-tests cloudschedulerjob -timeout 900s
- ```
+# Export the environment variables needed in the dynamic tests if you haven't done it.
+TEST_FOLDER_ID=123456789 go test -v -tags=integration ./pkg/controller/dynamic/ -test.run TestCreateNoChangeUpdateDelete -run-tests cloudschedulerjob -timeout 900s
+```
 Replace `cloudschedulerjob` with your test target.
 
 ### Submit a Pull Request
@@ -387,5 +386,5 @@ you can first validate the change locally:
 make ready-pr
 ```
 
-You can then commit your change and make a pull request. See more details
-[here](https://docs.github.com/en/get-started/quickstart/contributing-to-projects#making-and-pushing-changes).
+You can then commit your change and make a pull request. See [GitHub's contributing
+to projects: making and pushing changes](https://docs.github.com/en/get-started/quickstart/contributing-to-projects#making-and-pushing-changes).

Makefile

@@ -376,7 +376,7 @@ TEST_TARGET ?= mock
 e2e-sample-tests:
 	RUN_E2E=1 E2E_KUBE_TARGET=envtest E2E_GCP_TARGET=${TEST_TARGET} KCC_USE_DIRECT_RECONCILERS="ComputeForwardingRule" \ go test -test.count=1 -timeout 3600s -v ./tests/e2e -run ${SAMPLE_TESTCASE}
 
-# orgnization ID for google.com
+# organization ID for google.com
 ORG_ID ?= 433637338589
 # billing account for ACP
 BILLING_ACCOUNT ?= 010E8D-490B6B-088E1C

README.ConfigureResourceReferences.md

@@ -62,7 +62,7 @@ Once it's determined, note down the field name and the referenced GCP resource n
 
 Before you can identify what the resource is based on, you must convert
 the Google Cloud resource name or Terraform type name into a
-Config Connector kind name. Create a Github question in this repo for help
+Config Connector kind name. Create a GitHub question in this repo for help
 if you still can't find the Config Connector kind name after following these steps:
 
 1.  If you know the TF type name, you can do project-scoped search in your local
@@ -80,7 +80,7 @@ if you still can't find the Config Connector kind name after following these ste
     [KCC reference doc](https://cloud.google.com/config-connector/docs/reference/overview).
 
 1.  If you can't find a KCC kind with the steps above, it's possible that the
-    referenced resource is not supported in KCC. Create a Github question in this repo
+    referenced resource is not supported in KCC. Create a GitHub question in this repo
     to clarify. If you've determined that the KCC kind is not supported in
     KCC, skip ahead to
     [Configure reference resource in the service mappings](#configure-reference-in-the-service-mappings).
@@ -119,10 +119,10 @@ You can find a `Type` field for each argument, and two most common types are
 
     1.  If the field is not required, mark it as an ignored field following
         [this example](https://github.com/GoogleCloudPlatform/k8s-config-connector/blob/b746248cd5a9b30669380513de8fdc6b4c43018d/config/servicemappings/cloudbuild.yaml#L204).
-    1.  If the field is required, create a Github question in this repo for
+    1.  If the field is required, create a GitHub question in this repo for
         further discussion.
 
-1.  If the field isn't a string or a list, create a Github question in this repo
+1.  If the field isn't a string or a list, create a GitHub question in this repo
     for further discussion.
 
 ## Configure reference resource in the service mappings

apis/bigquery/v1beta1/dataset_types.go

@@ -42,7 +42,7 @@ type BigQueryDatasetSpec struct {
 	//  afterwards, and does not alter the existing tables.
 	//  The following values are supported:
 	//
-	//  * 'und:ci': undetermined locale, case insensitive.
+	//  * 'und:ci': undetermined locale, case-insensitive.
 	//  * '': empty string. Default to case-sensitive behavior.
 	DefaultCollation *string `json:"defaultCollation,omitempty"`
 

apis/bigqueryconnection/v1alpha1/connection_reference.go

@@ -62,7 +62,7 @@ func NewBigQueryConnectionConnectionRef(ctx context.Context, reader client.Reade
 		}
 		id.parent = "projects/" + tokens[1] + "/locations/" + tokens[3]
 
-		// Validate spec parent and resourceID field if the resource is already reconcilied with a GCP Connection resource.
+		// Validate spec parent and resourceID field if the resource is already reconciled with a GCP Connection resource.
 		if tokens[1] != projectID {
 			return nil, fmt.Errorf("spec.projectRef changed, expect %s, got %s",
 				tokens[1], projectID)

apis/compute/v1beta1/firewallpolicyrule_types.go

@@ -108,7 +108,7 @@ type ComputeFirewallPolicyRuleSpec struct {
 	/* A match condition that incoming traffic is evaluated against. If it evaluates to true, the corresponding 'action' is enforced. */
 	Match *FirewallPolicyRuleMatch `json:"match"`
 
-	/* Immutable. An integer indicating the priority of a rule in the list. The priority must be a positive value between 0 and 2147483647. Rules are evaluated from highest to lowest priority where 0 is the highest priority and 2147483647 is the lowest prority. */
+	/* Immutable. An integer indicating the priority of a rule in the list. The priority must be a positive value between 0 and 2147483647. Rules are evaluated from highest to lowest priority where 0 is the highest priority and 2147483647 is the lowest priority. */
 	Priority int64 `json:"priority"`
 
 	// +optional

apis/compute/v1beta1/forwardingrule_types.go

@@ -186,7 +186,7 @@ type ComputeForwardingRuleSpec struct {
 
 	A Forwarding Rule with protocol L3_DEFAULT can attach with target instance or
 	backend service with UNSPECIFIED protocol.
-	A forwarding rule with "L3_DEFAULT" IPProtocal cannot be attached to a backend service with TCP or UDP. Possible values: ["TCP", "UDP", "ESP", "AH", "SCTP", "ICMP", "L3_DEFAULT"]. */
+	A forwarding rule with "L3_DEFAULT" IPProtocol cannot be attached to a backend service with TCP or UDP. Possible values: ["TCP", "UDP", "ESP", "AH", "SCTP", "ICMP", "L3_DEFAULT"]. */
 	// +optional
 	IpProtocol *string `json:"ipProtocol,omitempty"`
 

apis/dataflow/v1beta1/flextemplatejob_types.go

@@ -256,7 +256,7 @@ type DataflowFlexTemplateJobObservedState struct {
 	//  Flexible resource scheduling jobs are started with some delay after job
 	//  creation, so start_time is unset before start and is updated when the
 	//  job is started by the Cloud Dataflow service. For other jobs, start_time
-	//  always equals to create_time and is immutable and set by the Cloud Dataflow
+	//  always equals create_time and is immutable and set by the Cloud Dataflow
 	//  service.
 	StartTime *string `json:"startTime,omitempty"`
 

apis/dataform/v1alpha1/repository_types.go

@@ -125,7 +125,7 @@ type DataformRepositoryStatus struct {
 	// +optional
 	ObservedGeneration *int64 `json:"observedGeneration,omitempty"`
 
-	// A unique specifier for the DataformReposity resource in GCP.
+	// A unique specifier for the DataformRepository resource in GCP.
 	// +optional
 	ExternalRef *string `json:"externalRef,omitempty"`
 

apis/dataform/v1beta1/repository_types.go

@@ -125,7 +125,7 @@ type DataformRepositoryStatus struct {
 	// +optional
 	ObservedGeneration *int64 `json:"observedGeneration,omitempty"`
 
-	// A unique specifier for the DataformReposity resource in GCP.
+	// A unique specifier for the DataformRepository resource in GCP.
 	// +optional
 	ExternalRef *string `json:"externalRef,omitempty"`
 

apis/discoveryengine/v1alpha1/discoveryenginedatastore_types.go

@@ -98,7 +98,7 @@ type DiscoveryEngineDataStoreStatus struct {
 // +kcc:proto=google.cloud.discoveryengine.v1.DataStore
 type DiscoveryEngineDataStoreObservedState struct {
 	// Output only. The id of the default
-	// [Schema][google.cloud.discoveryengine.v1.Schema] asscociated to this data
+	// [Schema][google.cloud.discoveryengine.v1.Schema] associated to this data
 	// store.
 	DefaultSchemaID *string `json:"defaultSchemaID,omitempty"`
 
@@ -112,7 +112,7 @@ type DiscoveryEngineDataStoreObservedState struct {
 
 // +genclient
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
-// TODO(user): make sure the pluralizaiton below is correct
+// TODO(user): make sure the pluralization below is correct
 // +kubebuilder:resource:categories=gcp,shortName=gcpdiscoveryenginedatastore;gcpdiscoveryenginedatastores
 // +kubebuilder:subresource:status
 // +kubebuilder:metadata:labels="cnrm.cloud.google.com/managed-by-kcc=true";"cnrm.cloud.google.com/system=true"

apis/refs/v1beta1/spannerdatabaseref.go

@@ -87,9 +87,9 @@ func ResolveSpannerDatabaseRef(ctx context.Context, reader client.Reader, obj cl
 	})
 	if err := reader.Get(ctx, key, spannerdatabase); err != nil {
 		if apierrors.IsNotFound(err) {
-			return nil, fmt.Errorf("referenced Spanner databse %v not found", key)
+			return nil, fmt.Errorf("referenced Spanner database %v not found", key)
 		}
-		return nil, fmt.Errorf("error reading referenced Spanner databse %v: %w", key, err)
+		return nil, fmt.Errorf("error reading referenced Spanner database %v: %w", key, err)
 	}
 
 	resourceID, _, err := unstructured.NestedString(spannerdatabase.Object, "spec", "resourceID")

apis/sql/v1beta1/instance_types.go

@@ -171,7 +171,7 @@ type InstanceIpConfiguration struct {
 	// +optional
 	RequireSsl *bool `json:"requireSsl,omitempty"`
 
-	/* Specify how SSL connection should be enforced in DB connections. This field provides more SSL enforcment options compared to requireSsl. To change this field, also set the correspoding value in requireSsl if it has been set. */
+	/* Specify how SSL connection should be enforced in DB connections. This field provides more SSL enforcement options compared to requireSsl. To change this field, also set the corresponding value in requireSsl if it has been set. */
 	// +optional
 	SslMode *string `json:"sslMode,omitempty"`
 }

config/cloudcodesnippets/identityplatform_v1beta1_identityplatformconfig.yaml

@@ -119,4 +119,4 @@ insertText: |
       forwardInboundCredentials:
         idToken: \${58:true}
         accessToken: \${59:true}
-        refereshToken: \${60:true}
+        refreshToken: \${60:true}

config/cloudcodesnippets/monitoring_v1beta1_monitoringalertpolicy.yaml

@@ -9,7 +9,7 @@ insertText: |
       \${3:oncall-treatment}: \${4:stay-aware}
     name: \${5:monitoringalertpolicy-name}
   spec:
-    displayName: \${6:Sample Website Aetwork Connectivity Alert Policy}
+    displayName: \${6:Sample Website Network Connectivity Alert Policy}
     enabled: \${7:true}
     notificationChannels:
     - name: \${8:monitoringalertpolicy-dep1-networkconnectivity}

config/crds/resources/apiextensions.k8s.io_v1_customresourcedefinition_bigquerydatasets.bigquery.cnrm.cloud.google.com.yaml

@@ -210,7 +210,7 @@ spec:
                    afterwards, and does not alter the existing tables.
                    The following values are supported:
 
-                   * 'und:ci': undetermined locale, case insensitive.
+                   * 'und:ci': undetermined locale, case-insensitive.
                    * '': empty string. Default to case-sensitive behavior.
                 type: string
               defaultEncryptionConfiguration: