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

yet another refactor #14

Closed
wants to merge 6 commits into from
Closed

yet another refactor #14

wants to merge 6 commits into from

Conversation

olga-mir
Copy link
Owner

@olga-mir olga-mir commented Dec 13, 2022

clusterctl in a GitOps way.

https://kubernetes.slack.com/archives/C8TSNPY4T/p1666177315009279

Going to park this, most likely permanently.

  • CAPI (and providers) do not provide helm charts or package management. Extracting manifests is hacky (patching env vars by manipulating yaml file), realease upgrades come with no guarantee (will have to download new manifests, compare and watch for potential changes)
  • CAPA manifests have AWS credentials as part of the manifest. They could be zeroed out in favor of IRSA, but this has some complications. There is no IRSA on kind cluster, nor in plain AWS clusters that this project is currently working with. Even if swapping out IRSA for kiam (which is now deprecated due to prevalence of EKS and IRSA) I would still have to commit ARNs of the roles that contain AWS account ID (even though it is not strictly sensitive I prefer to avoid this).
  • There is a lot of work in the area of the project that least interests me. Especially given the fact that the current installation and tear down way work perfectly fine (even though not very pretty and 100% written in bash). There is more value in improving this process by removing CRS in favor of (not yet released) CAPI Helm Provider.

@olga-mir olga-mir closed this Dec 24, 2022
@olga-mir olga-mir deleted the develop branch December 24, 2022 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant