Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(alertmanager): 🚑 resolve replicaCount comparision type issue #2231

Merged

Conversation

prashant-shahi
Copy link
Contributor

@prashant-shahi prashant-shahi commented Jul 5, 2022

What this PR does / why we need it:

Alertmanager HA setup with mutiple replicas is broken because of the issue tagged below. This PR resolves it.

Which issue this PR fixes

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name

Signed-off-by: Prashant Shahi [email protected]

@naseemkullah naseemkullah merged commit fd06408 into prometheus-community:main Jul 5, 2022
@prashant-shahi prashant-shahi deleted the prashant/ha-setup branch July 6, 2022 05:39
stazdx pushed a commit to stazdx/helm-charts that referenced this pull request Jul 6, 2022
…metheus-community#2231)

* fix(alertmanager): 🚑 resolve replicaCount comparision type issue

Signed-off-by: Prashant Shahi <[email protected]>

* chore(release): 🔖 Bump up alertmanager to 0.18.3

Signed-off-by: Prashant Shahi <[email protected]>

* fix(alertmanager): 🚑 backward compatiblity with older version of Helm

Signed-off-by: Prashant Shahi <[email protected]>
Signed-off-by: Staz <[email protected]>
ikarlashov pushed a commit to ikarlashov/helm-charts that referenced this pull request Jul 8, 2022
…metheus-community#2231)

* fix(alertmanager): 🚑 resolve replicaCount comparision type issue

Signed-off-by: Prashant Shahi <[email protected]>

* chore(release): 🔖 Bump up alertmanager to 0.18.3

Signed-off-by: Prashant Shahi <[email protected]>

* fix(alertmanager): 🚑 backward compatiblity with older version of Helm

Signed-off-by: Prashant Shahi <[email protected]>
Signed-off-by: Evgen Karlashov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[alertmanager] multiple replicas result in error: incompatible types for comparison
2 participants