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

set protocol to http #1738

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

chakrabandla
Copy link

As per https://github.com/elastic/helm-charts/tree/main/elasticsearch#configuration , default value of protocol as http
Also, it clearly mentions "Change this to https if you have xpack.security.http.ssl.enabled"

protocol The protocol that will be used for the readiness probe. Change this to https if you have xpack.security.http.ssl.enabled

https://github.com/elastic/helm-charts/tree/main/elasticsearch#configuration says default value of protocol as http

protocol	The protocol that will be used for the readiness probe. Change this to https if you have xpack.security.http.ssl.enabled
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

❌ Author of the following commits did not sign a Contributor Agreement:
a204ddd

Please, read and sign the above mentioned agreement if you want to contribute to this project

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants