Add Elastic eck-operator Helm Chart. #1128
Conversation
Signed-off-by: Michael Montgomery <[email protected]>
| HelmRepo: https://helm.elastic.co | ||
| HelmChart: eck-operator | ||
| Vendor: Elastic | ||
| ChartMetadata: |
There was a problem hiding this comment.
There's no need to include ChartMetadata.kubeVersion, since the upstream chart defines kubeVersion.
It is better that this is defined upstream anyways. If the value were to change upstream, you'd have to remember to update the value here as well, or chart versions that are included in the partner charts repo would have the value specified here set. It is likely that this would be forgotten.
Don't forget to delete the changes from charts/ and assets and re-run bin/partner-charts-ci update after you make this change.
There was a problem hiding this comment.
@adamkpickering this is complete. note I was forced to manually edit the index.yaml to get the bin/partner-charts-ci update command to work.
There was a problem hiding this comment.
Ah yeah, I should have mentioned that you need to delete the changes from index.yaml as well.
It's best to make a commit where you add the necessary files to the packages/ directory, and then a second commit where you add the changes from bin/partner-charts-ci update. That way you can simply do a git reset --hard to the first commit if you ever need to change things. Sorry about the confusion.
Signed-off-by: Michael Montgomery <[email protected]>
Signed-off-by: Michael Montgomery <[email protected]>
|
Signed-off-by: Michael Montgomery <[email protected]>
2 participants
This adds the Elastic ECK operator Chart .
Tested on: