feat(proton-bridge): metrics port, PodMonitor, Recreate, digest (0.2.0)#6
Merged
Merged
Conversation
Make the metrics exporter first-class so consumers don't need postRenderer / extraEnv workarounds: - declare a `metrics` containerPort (container.metricsPort, default 9154) and set CONTAINER_METRICS_PORT; default bridge.mode to grpc (serves the exporter API) - add an optional PodMonitor (podMonitor.enabled) - default update strategy to Recreate (single ReadWriteOnce volume; rolling updates deadlock on volume multi-attach) - support image.digest for immutable pinning (repo:tag@digest) - bump appVersion to 3.25.0 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Makes the proton-bridge metrics exporter first-class so the home-cluster deployment can drop its postRenderer/extraEnv/digest-in-tag workarounds.
Changes (chart 0.1.8 → 0.2.0)
metricscontainerPort (container.metricsPort, default 9154) and setCONTAINER_METRICS_PORT. A PodMonitor needs the port declared to scrape it.bridge.modenow defaults togrpc(serves mail + the gRPC API the exporter reads).noninteractive/clistill available.podMonitor.enabled(default off), withinterval/path/labels.strategy.type: Recreate— the single ReadWriteOnce volume makes rolling updates deadlock on multi-attach.image.digest→ referenced asrepo:tag@digest(immutable; avoids stale mirror pulls of a re-pushed mutable tag).Validation
helm templaterenders cleanly forgrpc/noninteractive/cli; default has ports[smtp, imap, metrics],strategy: Recreate,CONTAINER_METRICS_PORT=9154, no PodMonitor.image.digestset →…:tag@sha256:…; withpodMonitor.enabled=true→ PodMonitor onport: metrics.helm lintclean.Follow-up: a home-cluster PR bumps to 0.2.0 and removes the workarounds.
🤖 Generated with Claude Code