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(charts): composer PrometheusRule #1733

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

quasystaty1
Copy link
Contributor

@quasystaty1 quasystaty1 commented Oct 24, 2024

Summary

adds prometheusrule object to composer chart

Background

Prometheus rule object file was overlooked in prior #1601

Changes

  • add prometheusRule object to enable alerts on monitoring stack
  • also fixes issue with composer metric port violating targetPort length

@quasystaty1 quasystaty1 requested a review from a team as a code owner October 24, 2024 11:46
@quasystaty1 quasystaty1 requested a review from aajimal October 24, 2024 11:46
@github-actions github-actions bot added the cd label Oct 24, 2024
@quasystaty1 quasystaty1 changed the title fix(charts): composer metric port fix(charts): composer PrometheusRule Oct 24, 2024
@quasystaty1 quasystaty1 force-pushed the quasystaty1/charts/composer-metric-port branch from 3907f7b to 20d01a4 Compare October 24, 2024 12:10
@quasystaty1 quasystaty1 added this pull request to the merge queue Oct 24, 2024
Merged via the queue into main with commit d5f7a19 Oct 24, 2024
46 checks passed
@quasystaty1 quasystaty1 deleted the quasystaty1/charts/composer-metric-port branch October 24, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants