feat(helm): add OpenSearch Operator Helm chart #513
Conversation
- Adds generated plugindef with all values --------- Signed off by: Simon Olander ([email protected])
- Alters the values.yaml to remove redundant fields (should be default) - Updates the plugindef to only allow edits to high-level settings. --------- Signed off by: Simon Olander ([email protected])
| @@ -15,6 +15,7 @@ on: | |||
| - kubeconfig-generator/chart/** | |||
| - logshipper/chart/** | |||
| - obenbao/charts/** | |||
| - opensearch/chart/** | |||
There was a problem hiding this comment.
You can do the same for the workflow/ci-pr-build.yaml to benefit from testing charts already during an open PR
|
|
||
| ## Architecture | ||
|
|
||
|  |
There was a problem hiding this comment.
I like the README so far. Your plan here was to come up with the architecture image/draw.io at a later stage, but maybe you can add it already?
| dependencies: | ||
| - name: opensearch-operator | ||
| alias: opensearchOperator | ||
| repository: https://opensearch-project.github.io/opensearch-k8s-operator/ # TODO: Move to -> oci://ghcr.io/cloudoperators/greenhouse-extensions/charts |
There was a problem hiding this comment.
As I understand it you will use the upstream charts? Then this will likely stay and will have the depedency from there, no?
| version: 2.7.0 | ||
| - name: opensearch-cluster | ||
| alias: opensearchCluster | ||
| repository: https://opensearch-project.github.io/opensearch-k8s-operator/ # TODO: Move to -> oci://ghcr.io/cloudoperators/greenhouse-extensions/charts |
There was a problem hiding this comment.
Is there another helm-chart used for the clusters? Then this repository likely needs to point to that one.
| password: YWRtaW4= # admin (testing only) | ||
| username: YWRtaW4= # admin (testing only) |
There was a problem hiding this comment.
security-wise and going forward I think it makes sense to template the secrets here and add them as a field in the plugindefinition?
Pull Request Details
Provide details about your pull request and what it adds, fixes, or changes.
This PR adds a Helm chart for the OpenSearch Operator