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

feat(k8sd): introduce feature packages and update feature apply functions #1119

Draft
wants to merge 1 commit into
base: KU-2730/helm-charts-microcluster
Choose a base branch
from

Conversation

berkayoz
Copy link
Member

  • Added the FeatureManifest struct that contains the required charts, images and default values
  • Features are updated to use feature manifests when reconciling through the feature controller.
  • Implementation modules under features are updated to contain sub modules per feature.
  • Values handling is separated from the Apply functions and is performed via helper functions instead.
  • Introduced helper Values types and related functions for merging values together using mergo.

@berkayoz berkayoz force-pushed the KU-2730/helm-charts-microcluster branch from 6c4f9cd to 3ae6337 Compare February 28, 2025 10:26
@berkayoz berkayoz force-pushed the KU-2731/feature-manifest branch from adca897 to d20d506 Compare February 28, 2025 11:25
@berkayoz berkayoz force-pushed the KU-2730/helm-charts-microcluster branch from 3ae6337 to 4dbf70a Compare March 3, 2025 09:05
…ions

- Added the FeatureManifest struct that contains the required charts and images for the feature.
- Features are updated to use feature manifests when reconciling through the feature controller.
- Implementation modules under `features` are updated to contain sub modules per feature.
- Values handling is separated from the `Apply` functions and is performed via helper functions instead.
- Introduced helper `Values` types and related functions for merging values together using mergo.
@berkayoz berkayoz force-pushed the KU-2731/feature-manifest branch from d20d506 to 0e7cae0 Compare March 3, 2025 11:15
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