Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CAFV-449][1.3.z] update CLUSTERCTL.md with updated versions #614

Merged
merged 3 commits into from
Mar 18, 2024

Conversation

rliang88
Copy link
Contributor

@rliang88 rliang88 commented Mar 14, 2024

Description

Please provide a brief description of the changes proposed in this Pull Request

  • add 1.3.z and 1.2.z to table
  • updated core capi compatibility to v1.5.4 for 1.3.z and main

Checklist

  • tested locally
  • updated any relevant dependencies
  • updated any relevant documentation or examples

API Changes

Are there API changes?

  • Yes
  • No

If yes, please fill in the below

  1. Updated conversions?
    • Yes
    • No
    • N/A
  2. Updated CRDs?
    • Yes
    • No
    • N/A
  3. Updated infrastructure-components.yaml?
    • Yes
    • No
    • N/A
  4. Updated ./examples/capi-quickstart.yaml?
    • Yes
    • No
    • N/A
  5. Updated necessary files under ./infrastructure-vcd/v1.0.0/?
    • Yes
    • No
    • N/A

Issue

If applicable, please reference the relevant issue

Fixes #


This change is Reviewable

Copy link
Contributor

@lzichong lzichong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed all commit messages.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @rliang88 and @sahithi)


docs/CLUSTERCTL.md line 22 at r1 (raw file):

1. Run the below command to initialize the management cluster with the Cluster API and the associated provider for VMware Cloud Director. Ensure the version interop between CAPVCD, Core CAPI, and Clusterctl are correct from the [interop version table](#version-interop)
   1. For CAPVCD `main`, use `clusterctl init --core cluster-api:v1.4.0 -b kubeadm:v1.4.0 -c kubeadm:v1.4.0 -i vcd:v1.1.0`

I think the versions should be updated here as well

Copy link
Contributor

@lzichong lzichong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: 0 of 1 files reviewed, 2 unresolved discussions (waiting on @rliang88 and @sahithi)


docs/CLUSTERCTL.md line 46 at r1 (raw file):

- All of the templates to generate the cluster manifests are located at `templates` directory under the root of the github repository.
- All the flavors listed support only v1beta1 API versions of CAPVCD and Core CAPI.
- Currently, we have v1.20.8, v1.21.8, v1.22.9, v1.23.10 as template flavors, and they each have their own etcd/dns versions pre-populated.

Possibly we can also update this list of versions to the latest kubernetes versions that we have in the templates folder

Copy link
Contributor

@lzichong lzichong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm: after addressing comments

Reviewable status: 0 of 1 files reviewed, 2 unresolved discussions (waiting on @rliang88 and @sahithi)

| 1.1.z | v1.4.0 |
| 1.0.z | v1.1.3 |

<a name="clusterctl_set_up"></a>
## Set up
Install [clusterctl](https://cluster-api.sigs.k8s.io/user/quick-start.html#install-clusterctl) according to the [version interop table](#version-interop).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please keep this line of clusterctl installation

## Initialize Management cluster

1. Run the below command to initialize the management cluster with the Cluster API and the associated provider for VMware Cloud Director. Ensure the version interop between CAPVCD, Core CAPI, and Clusterctl are correct from the [interop version table](#version-interop)
1. For CAPVCD `main`, use `clusterctl init --core cluster-api:v1.4.0 -b kubeadm:v1.4.0 -c kubeadm:v1.4.0 -i vcd:v1.1.0`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vcd:v1.3.0

Copy link
Contributor Author

@rliang88 rliang88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: 0 of 1 files reviewed, 4 unresolved discussions (waiting on @lzichong and @sahithi)


docs/CLUSTERCTL.md line 22 at r1 (raw file):

Previously, lzichong wrote…

I think the versions should be updated here as well

Done.


docs/CLUSTERCTL.md line 46 at r1 (raw file):

Previously, lzichong wrote…

Possibly we can also update this list of versions to the latest kubernetes versions that we have in the templates folder

Done.


docs/CLUSTERCTL.md line 22 at r2 (raw file):

Previously, sahithi (Sahithi Ayloo) wrote…

vcd:v1.3.0

Done.

@rliang88 rliang88 requested a review from sahithi March 18, 2024 21:44
@rliang88 rliang88 merged commit 268a164 into vmware:1.3.z Mar 18, 2024
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants