Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[elasticsearch] Add "service.enabled" parameter #998

Closed
ebuildy opened this issue Dec 23, 2020 · 3 comments · Fixed by #1115
Closed

[elasticsearch] Add "service.enabled" parameter #998

ebuildy opened this issue Dec 23, 2020 · 3 comments · Fixed by #1115
Labels

Comments

@ebuildy
Copy link
Contributor

ebuildy commented Dec 23, 2020

Describe the feature:

service.yaml should be optional.

Describe a specific use case for the feature:

In case of a pure data nodes Helm release, service is useless (I believe ..).

@jmlrt jmlrt added elasticsearch enhancement New feature or request labels Feb 24, 2021
@jmlrt
Copy link
Member

jmlrt commented Feb 24, 2021

Hi @ebuildy, Elasticsearch chart use 2 different services, the "standard" service which is used for external access and the "headless" service which is mandatory for statefulsets (we had to add one for Logstash charts, see #695 for more details).

While the "headless" need to stay, the "standard" service could be optional, I think we'll need to investigate a bit more to ensure that's OK before creating a PR for that.

@botelastic
Copy link

botelastic bot commented May 25, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@botelastic
Copy link

botelastic bot commented Jun 24, 2021

This issue has been automatically closed because it has not had recent activity since being marked as stale.

@botelastic botelastic bot closed this as completed Jun 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants