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

should arlon detect illegal/unsupported settings or changes to CAPI base cluster? #103

Open
bcle opened this issue Jun 23, 2022 · 0 comments
Labels
Needs Review For all new issues that are unplanned, have no milestone or project. size:M
Milestone

Comments

@bcle
Copy link
Collaborator

bcle commented Jun 23, 2022

Arlon and ArgoCD apply CAPI manifests "as is". The user is fully responsible for the manifest contents. Examples of bad things that can happen:

  • It's not always intuitive to know what kubernetes versions are supported by the current version of CAPI providers. It's also provider specific. For example, for AWS, clusterawsadm ami list is the best way. An unsupported version will result in a non-converging cluster that fails silently.
  • During an update/upgrade caused by modifying a shared base cluster, existing cluster manifests disallow some resource fields (in the spec) to change. I forgot which one(s) I tried (instance type?). What'll happen is, in the best case, argocd will flag some resources as out of sync, and in the worst case, they will appear in sync and healthy, but actually unhealthy until you look at CAPI/CAPA pod logs. (duplicate of Changes to "immutable" fields of base cluster results in poor UX for existing clusters (OutOfSync) #137)

Aha! Link: https://pf9.aha.io/features/ARLON-134

@bcle bcle added this to 0.9.0 Jun 23, 2022
@bcle bcle moved this to Todo in 0.9.0 Jun 23, 2022
@bcle bcle added the size:M label Jul 1, 2022
@cre8minus1 cre8minus1 added this to the 0.10.0 milestone Jul 6, 2022
@cre8minus1 cre8minus1 removed this from 0.9.0 Jul 6, 2022
@cre8minus1 cre8minus1 added this to 0.10.0 Jul 6, 2022
@cre8minus1 cre8minus1 added the Needs Review For all new issues that are unplanned, have no milestone or project. label Jul 6, 2022
@bcle bcle removed this from 0.10.0 Sep 30, 2022
@bcle bcle removed this from the 0.10.0 milestone Sep 30, 2022
@cre8minus1 cre8minus1 added this to the Not Planned milestone Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review For all new issues that are unplanned, have no milestone or project. size:M
Projects
None yet
Development

No branches or pull requests

2 participants