Skip to content

Commit

Permalink
Merge pull request #1190 from kahirokunn/fix
Browse files Browse the repository at this point in the history
fix: The subject was incorrectly used in some parts of the document.
  • Loading branch information
k8s-ci-robot authored Mar 24, 2024
2 parents f6c2c4f + 50d3147 commit fccd828
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions docs/proposals/20221123-managed-kubernetes-in-capg.md
Original file line number Diff line number Diff line change
Expand Up @@ -107,9 +107,9 @@ So that I don't have to worry about the management of these instances

#### Functional Requirements

**FR1:** CAPA MUST support creating GKE clusters.
**FR1:** CAPG MUST support creating GKE clusters.

**FR2:** CAPA MUST validate the input an user specifies for a GKE cluster.
**FR2:** CAPG MUST validate the input an user specifies for a GKE cluster.

#### Non-Functional Requirements

Expand Down Expand Up @@ -234,7 +234,7 @@ GCP permission also needs to be updated so the controllers can call GKE APIs.

### Risks and Mitigations

The risk is that the implemented interface is different from CAPA and CAPZ. However it follows the recommended approach in the [Managed Kubernetes in CAPI](https://github.com/kubernetes-sigs/cluster-api/blob/main/docs/proposals/20220725-managed-kubernetes.md) proposal.
The risk is that the implemented interface is different from CAPG and CAPZ. However it follows the recommended approach in the [Managed Kubernetes in CAPI](https://github.com/kubernetes-sigs/cluster-api/blob/main/docs/proposals/20220725-managed-kubernetes.md) proposal.

## Alternatives

Expand Down

0 comments on commit fccd828

Please sign in to comment.