chore: replace control-plane promtail with alloy#595
Open
ma-hartma wants to merge 12 commits into
Open
Conversation
28 tasks
78c5ad6 to
04731eb
Compare
This was referenced May 6, 2026
9182a67 to
413e4cc
Compare
413e4cc to
8ef1443
Compare
4d6686b to
8597f82
Compare
ma-hartma
commented
May 12, 2026
ma-hartma
commented
May 13, 2026
1040276 to
d92490b
Compare
ma-hartma
commented
May 19, 2026
|
|
||
| rule { | ||
| target_label = "job" | ||
| replacement = "monitoring/event-exporter" |
Contributor
Author
There was a problem hiding this comment.
The event-exporter used with promtail had the events labelled with job=monitoring/event-exporter.
I did actually re-label the logs from job=kubernetes-events to also match the legacy job name.
I haven't found any usage in metal-roles, but other users may rely on it.
ma-hartma
commented
May 19, 2026
| logging_ingress_dns: "loki.{{ metal_control_plane_ingress_dns }}" | ||
| logging_ingress_loki_tls: yes | ||
| logging_ingress_loki_basic_auth_user: promtail | ||
| logging_ingress_loki_basic_auth_user: promtail # TODO rename to alloy or generic |
Contributor
Author
There was a problem hiding this comment.
I'll leave the old username for now.
ma-hartma
commented
May 19, 2026
| # Each entry: {url, remote_timeout?: duration, basic_auth?: {username, password}} | ||
| # Thanos Receive endpoint is included automatically when monitoring_thanos_receive_enabled: true. | ||
| logging_alloy_prometheus_write_endpoints: >- | ||
| {{ [{'url': 'http://thanos-receive.' ~ logging_namespace ~ '.svc.cluster.local:19291/api/v1/receive'}] |
Contributor
Author
There was a problem hiding this comment.
In the shooted seeds, we do not have an in-cluster prometheus. Here we could actually use ServiceMonitors but I decided against it, because we would introduce a dependency on prometheus already existing before deploying the helm chart.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Replaces promtail with alloy in the control-plane.
The partition replacement is treated separately in #592 and should go first.
Question Marks:
After successful replacement
Manually tested in the mini-lab:
References:
Closes #552
Used AI-Tools ✨
Release Notes
Required Actions
Noteworthy