Skip to content
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.

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

feat(vector): Support enable for both the normal and headless service #257

Merged
merged 2 commits into from
Oct 20, 2022

Conversation

spencergilbert
Copy link
Contributor

@spencergilbert spencergilbert commented Oct 19, 2022

Previously both a standard and headless service were enabled and configured by the same values. This PR allows for the headless service to be disabled separately.

The headless service is particularly useful for the in-chart HAProxy loadbalancer, but for production deploys users should leverage the standard service with a LoadBalancer type (and relevant configurations, generally annotation driven, their cloud provider requires).

Closes #253

Signed-off-by: Spencer Gilbert spencer.gilbert@datadoghq.com

@spencergilbert spencergilbert self-assigned this Oct 19, 2022
@spencergilbert spencergilbert force-pushed the spencer/separate-headless-service-conf branch 2 times, most recently from 6ddb1d1 to 8aba8d0 Compare October 19, 2022 18:32
…service

The headless service can be disabled and configured separately, but the default behavior will be to reuse the config from the service.

Signed-off-by: Spencer Gilbert <spencer.gilbert@datadoghq.com>
@spencergilbert spencergilbert force-pushed the spencer/separate-headless-service-conf branch from 8aba8d0 to 7e8355f Compare October 19, 2022 19:12
Signed-off-by: Spencer Gilbert <spencer.gilbert@datadoghq.com>
@spencergilbert spencergilbert marked this pull request as ready for review October 20, 2022 15:25
@spencergilbert spencergilbert requested a review from a team October 20, 2022 15:25
charts/vector/templates/service-headless.yaml Show resolved Hide resolved
charts/vector/README.md Show resolved Hide resolved
@spencergilbert spencergilbert changed the title feat(vector): Support configuration for both the normal and headless service feat(vector): Support enable for both the normal and headless service Oct 20, 2022
@spencergilbert spencergilbert merged commit 9842060 into develop Oct 20, 2022
@spencergilbert spencergilbert deleted the spencer/separate-headless-service-conf branch October 20, 2022 18:37
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.

Separate configuration for the Service and Headless Service
2 participants