Skip to content

fix(common): serviceDiscovery.env derives siblings from legacy configmap.SD_* without global #1779

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.

Sign up for GitHub

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

Merged
guimoreirar merged 3 commits into from
Jul 31, 2026
Merged

fix(common): serviceDiscovery.env derives siblings from legacy configmap.SD_* without global #1779

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a69ec3c
fix(common): serviceDiscovery.env derives siblings from legacy config…
guimoreirar Jul 31, 2026
58df8fb
docs(common): document configmap.SD_ADDRESS as alt activation source …
guimoreirar Jul 31, 2026
ba61e50
fix(common): serviceDiscovery.env uses hasKey precedence so explicit …
guimoreirar Jul 31, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 11 additions & 6 deletions charts/lerian-common/templates/_service_discovery.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,15 +48,20 @@ global.serviceDiscovery (all optional except address when enabled):
no-configmap caller renders byte-identical to before. */ -}}
{{- $c := .configmap | default dict -}}
{{- /*
Derive ONLY when enabled AND global.serviceDiscovery.address is configured.
Derive ONLY when enabled AND an SD address is configured — via EITHER the
environment-wide `global.serviceDiscovery.address` OR the legacy flat
`configmap.SD_ADDRESS` (on-prem/client values that predate global). Gating on
both preserves the documented backward-compat contract: a chart deployed with
`configmap.SD_ENABLED=true` + `configmap.SD_ADDRESS` (no global) still derives
the full sibling block, matching pre-refactor output.
This keeps adoption backward-compatible: a chart carrying this helper but
deployed against a not-yet-migrated environment (SD_* still hand-set in
extraEnvVars, no global.serviceDiscovery) stays INERT — extraEnvVars drives
SD exactly as before, no duplicate keys, no render break. The environment
opts into derivation by setting global.serviceDiscovery + stripping the
per-app SD_* block down to just SD_ENABLED.
extraEnvVars, no global.serviceDiscovery, no configmap.SD_ADDRESS) stays
INERT — extraEnvVars drives SD exactly as before, no duplicate keys, no render
break. The environment opts into derivation by setting global.serviceDiscovery
(or configmap.SD_ADDRESS) + stripping the per-app SD_* block down to SD_ENABLED.
*/ -}}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
{{- if and .enabled $sd.address -}}
{{- if and .enabled (or $sd.address (index $c "SD_ADDRESS")) -}}
{{- /* SD_ENABLED is NOT emitted here: it is the app's single knob and is
rendered by the component's own extraEnvVars passthrough. Emitting it again
would produce a duplicate key. This helper only adds the derived siblings. */ -}}
Expand Down
Loading