fix(deployment): typo regarding topologySpreadConstraints

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

fix(deployment): typo regarding topologySpreadConstraints #523

Closed cpockrandt closed 2 months ago

cpockrandt commented 7 months ago

Description

There's a bug introduced by a typo in the latest helm chart release.

Issues Resolved

[List any issues this PR will resolve. You should likely open an issue if one does not already exist.]

Check List

[x] Commits are signed per the DCO using --signoff

For any changes to files within Helm chart directories:

[x] Helm chart version bumped
[x] Helm chart CHANGELOG.md updated to reflect change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

cpockrandt commented 6 months ago

Could someone take a look at this PR and review it please? :)

dazoakley commented 2 months ago

Hi folks, any chance this could be looked at, at the moment the dashboards chart is completely broken.

We'd like to update the chart we're using and include TSC's... :smile:

gaiksaya commented 2 months ago

Hi folks, any chance this could be looked at, at the moment the dashboards chart is completely broken.

We'd like to update the chart we're using and include TSC's... 😄

Looks like there are conflicts in the files. Can you please rebase and resolve the conflicts? Thanks!

peterzhuamazon commented 2 months ago

I will update this PR and review.

peterzhuamazon commented 2 months ago

Doesnt seem it requires any backports. Thanks @cpockrandt .