opensearch-project / helm-charts

:wheel_of_dharma: A community repository for Helm Charts of OpenSearch Project.
https://opensearch.org/docs/latest/opensearch/install/helm/
Apache License 2.0
170 stars 228 forks source link

[Enhancement][opensearch-dashboards] topologySpreadConstraints for dashboards #505

Closed cpockrandt closed 7 months ago

cpockrandt commented 10 months ago

I was wondering whether it would be beneficial to add topologySpreadConstraints to the opensearch-dashboards Helm charts, similarly to the opensearch Helm chart (https://github.com/opensearch-project/helm-charts/issues/222):

https://github.com/opensearch-project/helm-charts/blob/5bb05318ea08b8ae83c8a83e22a368a3aa6133b1/charts/opensearch/templates/statefulset.yaml#L144-L147

E.g., to distribute the pods across multiple availability zones.

kinoute commented 9 months ago

Interested by this as well. I was about to create a pull request for that.

cpockrandt commented 9 months ago

I was just about to start a PR and saw that you already opened one! Thanks @kinoute!

prudhvigodithi commented 9 months ago

[Untraige] Thanks @cpockrandt for raising the issue and thanks @kinoute for the contribution, I will take a look at the PR.

cpockrandt commented 7 months ago

I'm closing this issue, since it was already merged three months ago.