-
Notifications
You must be signed in to change notification settings - Fork 112
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
Added info on ClusterClass use #385
base: main
Are you sure you want to change the base?
Added info on ClusterClass use #385
Conversation
Hi @p-strusiewiczsurmacki-mobica. Thanks for your PR. I'm waiting for a metal3-io member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
3209226
to
1b93996
Compare
/ok-to-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
Holding until the CAPM3 PR is merged
/hold
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lentzi90 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits
@@ -0,0 +1,73 @@ | |||
# Using ClusterClass with CAPM3 | |||
|
|||
ClusterClass is a feature of Cluster API that enables the cluster operators to create multiple clusters using a single general template. You can find detailed explanation of how to use a `ClusterClass` in the [Cluster API documentation](https://github.com/kubernetes-sigs/cluster-api/blob/main/docs/proposals/20210526-cluster-class-and-managed-topologies.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line length is not (yet) enforced, but it would be appreciated if you could wrap them at 80.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrapped at 80 where possible (some URLs might be longer).
git clone https://github.com/metal3-io/cluster-api-provider-metal3.git | ||
``` | ||
|
||
2. Generate Tilt settings that will enable ClusterClass support in CAPI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2. Generate Tilt settings that will enable ClusterClass support in CAPI. | |
1. Generate Tilt settings that will enable ClusterClass support in CAPI. |
Auto numbering should work here, and avoids issue of mis-numbering.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed :)
make tilt-settings-clusterclass | ||
``` | ||
|
||
2. Start Tilt. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2. Start Tilt. | |
1. Start Tilt. |
...Like here, 2 should be 3.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed :)
af9e07f
to
3cac548
Compare
Signed-off-by: Patryk Strusiewicz-Surmacki <[email protected]>
3cac548
to
131afef
Compare
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with /lifecycle stale |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/remove-lifecycle stale |
This PR introduces some basic info on how to use ClusterClass with CAPM3, and how to deploy example ClusterClass using Tilt.
Should be only merged after CAPM3 PR #1405 gets merged.