Skip to content

Fix alertmanager URL#2178

Merged
petersutter merged 1 commit intomasterfrom
bug/alertmanager-url
Nov 14, 2024
Merged

Fix alertmanager URL#2178
petersutter merged 1 commit intomasterfrom
bug/alertmanager-url

Conversation

@petersutter
Copy link
Member

@petersutter petersutter commented Nov 13, 2024

What this PR does / why we need it:
This PR fixes the alertmanager URL. The alertmanager is a statefulset and follows the same convention as the Prometheus statefulset, listing the index of the statefulset pod.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

Fixed alertmanager URL

@gardener-robot gardener-robot added needs/review Needs review size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 13, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 13, 2024
@ghost ghost added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Nov 13, 2024
Copy link
Member

@holgerkoser holgerkoser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/review Needs review labels Nov 14, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 14, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 14, 2024
@petersutter petersutter merged commit 2376bf3 into master Nov 14, 2024
@petersutter petersutter deleted the bug/alertmanager-url branch November 14, 2024 09:52
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Nov 14, 2024
holgerkoser added a commit that referenced this pull request Nov 15, 2024
* master:
  temporarily disable sass major and minor updates
  Update Monthly Dev Dependencies (#2181)
  Update fsfe/reuse-action action to v5 (#2182)
  fix alertmanager url (#2178)
  temporarily disable dockerfile-ast and sass updates
  replaced set editor readonly state instead of editable view (#2176)
  Migrate config .github/renovate.json5 (#2177)
  Update renovate.json5
  Update dependency express-static-gzip to v2.2.0 (#2172)
  Do not wrap lodash because of linter (#2174)

# Conflicts:
#	.pnp.cjs
#	frontend/src/views/GAdministration.vue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) reviewed/lgtm Has approval for merging size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants