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

✨ Add clusteraddon controller #28

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Conversation

janiskemper
Copy link
Member

What this PR does / why we need it:
Add clusteraddon controller which is responsible for reconciling ClusterAddons. It templates the ClusterAddon Helm Chart from the cluster stack assets and applies its objects. It also updates the objects if the Helm Chart has been updated and only then - not necessarily on each update of a ClusterStack version.

Which issue(s) this PR fixes:
Fixes #15

TODOs:

  • squash commits
  • include documentation
  • add unit tests

Add clusteraddon controller which is responsible to reconcile
ClusterAddons. It templates the ClusterAddon Helm Chart from the cluster
stack assets and applies its objects. It also updates the objects if the
Helm Chart has been updates and only then - not necessarily on each
update of a ClusterStack version.

Signed-off-by: janiskemper <janis.kemper@syself.com>
@janiskemper
Copy link
Member Author

this has to wait until the cluster-stacks repo has releases containing all relevant release assets for cluster addons until it can run successfully

@janiskemper
Copy link
Member Author

this is technically independent of the cluster-stacks repo and I will try merging this one to not block further work.

@janiskemper janiskemper marked this pull request as ready for review November 21, 2023 08:14
@janiskemper janiskemper merged commit dde4841 into main Nov 21, 2023
7 checks passed
@janiskemper janiskemper deleted the clusteraddon-controller branch November 21, 2023 08:18
@jschoone jschoone added the Container Issues or pull requests relevant for Team 2: Container Infra and Tooling label Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Container Issues or pull requests relevant for Team 2: Container Infra and Tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ClusterAddon feature
3 participants