kubermatic icon indicating copy to clipboard operation
kubermatic copied to clipboard

Update MLA Envoy to 1.29.2, improve alertmanager-proxy Helm Chart

Open xrstf opened this issue 1 year ago • 4 comments

What this PR does / why we need it: This builds upon #13221, but mainly makes a major version bump for Envoy, from 1.18 to 1.29. I tested the config changes on dev and besides one warning that I could not get rid of (see the comment in the envoy ConfigMap), it seems to work fine and allowed me to access the Alertmanager on dev.

What type of PR is this? /kind cleanup /kind chore

Does this PR introduce a user-facing change? Then add your Release Note here:

- Update Alertmanager Authorization Envoy to v1.29.2
- Improve alertmanager-proxy Helm Chart: do not require root permissions, drop capabilities and make logging/ports configurable

Documentation:

NONE

xrstf avatar Mar 25 '24 19:03 xrstf

looks nice and clean, thank you!

/lgtm

wurbanski avatar Mar 26 '24 20:03 wurbanski

LGTM label has been added.

Git tree hash: 65f677243466d96e3bde069c99fd873a7f6b5596

kubermatic-bot avatar Mar 26 '24 20:03 kubermatic-bot

It's broken though.

/lgtm cancel

Needs #13224 first. ^^

This new 2-step process of building images is more hassle than I thought... :-/ hmmm...

xrstf avatar Mar 26 '24 20:03 xrstf

/lgtm /approve

wurbanski avatar Apr 02 '24 07:04 wurbanski

LGTM label has been added.

Git tree hash: 900a09149cf76e738a67b7b7c04a4d33b06648f7

kubermatic-bot avatar Apr 02 '24 07:04 kubermatic-bot

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: wurbanski

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

kubermatic-bot avatar Apr 02 '24 07:04 kubermatic-bot