Docs-only follow-up from the v1.7.1 egress-enforcement check review (chart 1.7.1; client-runtime#104, shipped via #253 / #255 / #256).
Context
The helm test enforcement check (templates/egress-enforcement-check.yaml) decides "egress blocked" from curl's exit code: 7 (connection refused) and 28 (timeout) ⇒ TCP never established ⇒ "blocked → enforced → test passes".
The caveat
exit 7 also occurs when the probe host's :443 is closed/refusing. With the default enforcementProbeHost: 1.1.1.1 (443 always open) this is safe — only a real network block yields 7/28. But if an operator overrides enforcementProbeHost to a host whose :443 is not reachable when egress is allowed, the test would false-pass: report the lockdown enforced while training pods can still reach the internet — the exact false-sense-of-security #104 exists to prevent.
Ask (docs only, no code change)
client/docs/SECURITY.md (§8.2) and the values.yaml comment for enforcementProbeHost: document that a custom probe host MUST accept TCP on :443 from outside the cluster (reachable whenever egress is allowed). Recommend keeping the default 1.1.1.1 unless there's a specific reason.
- Optionally note that only curl exit
28/7 count as "blocked".
Acceptance
Related
Docs-only follow-up from the v1.7.1 egress-enforcement check review (chart 1.7.1; client-runtime#104, shipped via #253 / #255 / #256).
Context
The
helm testenforcement check (templates/egress-enforcement-check.yaml) decides "egress blocked" from curl's exit code:7(connection refused) and28(timeout) ⇒ TCP never established ⇒ "blocked → enforced → test passes".The caveat
exit 7also occurs when the probe host's :443 is closed/refusing. With the defaultenforcementProbeHost: 1.1.1.1(443 always open) this is safe — only a real network block yields 7/28. But if an operator overridesenforcementProbeHostto a host whose :443 is not reachable when egress is allowed, the test would false-pass: report the lockdown enforced while training pods can still reach the internet — the exact false-sense-of-security #104 exists to prevent.Ask (docs only, no code change)
client/docs/SECURITY.md(§8.2) and thevalues.yamlcomment forenforcementProbeHost: document that a custom probe host MUST accept TCP on:443from outside the cluster (reachable whenever egress is allowed). Recommend keeping the default1.1.1.1unless there's a specific reason.28/7count as "blocked".Acceptance
:443-reachability requirement.values.yamlenforcementProbeHostcomment notes the same.Related