Skip to content

feat(vector): default API bind and readiness for chart-managed config#540

Open
kurochan wants to merge 1 commit intovectordotdev:developfrom
kurochan:vector-readiness
Open

feat(vector): default API bind and readiness for chart-managed config#540
kurochan wants to merge 1 commit intovectordotdev:developfrom
kurochan:vector-readiness

Conversation

@kurochan
Copy link

@kurochan kurochan commented Mar 5, 2026

This PR updates the vector chart defaults to make health checks easier to use with chart-managed configuration.

  • Changes default generated Vector config api.address from 127.0.0.1:8686 to 0.0.0.0:8686
  • Adds a default readinessProbe only for chart-managed config (no customConfig / no existingConfigMaps)
    • httpGet.path: /health
    • httpGet.port: api
  • Keeps livenessProbe opt-in (no default liveness behavior change)
  • Updates README and values comments with API exposure and probe behavior notes

Considerations

  • This is a behavioral change for users relying on chart defaults: API bind changes from loopback-only to pod-reachable (0.0.0.0).
  • Security impact: wider API reachability means access should be restricted with NetworkPolicy and careful Service/Ingress exposure (this PR intentionally does not add default NetworkPolicy resources).

@kurochan kurochan changed the title fix(vector): default API bind and readiness for chart-managed config feat(vector): default API bind and readiness for chart-managed config Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant