-
Notifications
You must be signed in to change notification settings - Fork 0
feat: get helm config from a well known configmap #27
Conversation
876157a
to
5770b04
Compare
5770b04
to
2fca7f9
Compare
charts/cluster-api-runtime-extensions-nutanix/templates/helm-config.yaml
Outdated
Show resolved
Hide resolved
charts/cluster-api-runtime-extensions-nutanix/templates/helm-config.yaml
Outdated
Show resolved
Hide resolved
charts/cluster-api-runtime-extensions-nutanix/templates/helm-config.yaml
Outdated
Show resolved
Hide resolved
fdca540
to
d2410f1
Compare
Is there any consideration about how many versions (X-??) of an add-on helm chart CAREN should be supporting between creating cluster and upgrading it to next version? |
No and there was none previously either. Each DKP release will have set addons that we support |
81efb0c
to
7969d1b
Compare
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.
Great changes, just had a question on aws-ccm
charts/cluster-api-runtime-extensions-nutanix/templates/helm-config.yaml
Outdated
Show resolved
Hide resolved
charts/cluster-api-runtime-extensions-nutanix/templates/helm-config.yaml
Outdated
Show resolved
Hide resolved
charts/cluster-api-runtime-extensions-nutanix/templates/helm-config.yaml
Outdated
Show resolved
Hide resolved
charts/cluster-api-runtime-extensions-nutanix/templates/helm-config.yaml
Show resolved
Hide resolved
a dev doc or readme about how a developer can change add-on helm configuration would be nice. |
7969d1b
to
ed362df
Compare
I added the warning. As far dev docs go, we don't really have a structure for using them and I don't really think its needed at this point either |
319d204
to
d57fa75
Compare
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.
Thanks for the improvement.
aa0f5de
to
eba2351
Compare
* ci: adds tooling to create configmap * feat: use a configmap to get helmchart info * fix: precommit issues * fix: typo in cilium * fix: remove workspace files * build: template name for configmap * refactor: names for helm chart info getter * refactor: use nutanix-storage name instead of nutnaix-csi * refactor: move to globaloptions * fix: adds snapshot to helm config * fix: comments after review * fix: adds a warning and removes ebs csi * fix: typo * fix: adds missing script file * fix: precommit
This deploys a helm chart with our helm addon configuration for CAREN's configuration. Other components are not expected to interact with this on a per cluster basis. This also allows the user some flexibility/insight into what addons are deployed.
The config map contains information such as version, chart name, and repository url.
The next step would be adding a field under
clusterConfig.addons
calledhelmRegistry
which would be used in conjunction with this configmap to support pulling helmCharts from private registries.See this example below for examples of the configmap and it being used to generate helmchartproxy.
here is an example of a generate helm chart proxy