8000 [elasticsearch] add loadBalancer externalTrafficPolicy option by luanguimaraesla · Pull Request #810 · elastic/helm-charts · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on May 16, 2023. It is now read-only.

[elasticsearch] add loadBalancer externalTrafficPolicy option #810

Merged
merged 1 commit into from
Sep 16, 2020
Merged

[elasticsearch] add loadBalancer externalTrafficPolicy option #810

merged 1 commit into from
Sep 16, 2020

Conversation

luanguimaraesla
Copy link
Contributor
  • Chart version not bumped (the versions are all bumped and released at the same time)
  • README.md updated with any new values or changes
  • Updated template tests in ${CHART}/tests/*.py
  • Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Hello,

This PR adds a new option in the elasticsearch service spec in order to support configuration for externalTrafficPolicy if the service is a loadBalancer.

Reference: https://kubernetes.io/docs/tasks/access-application-cluster/create-external-load-balancer/#preserving-the-client-source-ip

@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
cla-checker-service bot commented Sep 15, 2020 •
edited
Loading

💚 CLA has been signed

@luanguimaraesla
Copy link
Contributor Author

I've just signed the CLA, should I do anything else?

@jmlrt
Copy link
Member
jmlrt commented Sep 16, 2020

jenkins test this please

jmlrt
jmlrt approved these changes Sep 16, 2020
View reviewed changes
Copy link
Member
@jmlrt jmlrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM⛴

@jmlrt jmlrt merged commit 6ddd3e3 into elastic:master Sep 16, 2020
jmlrt pushed a commit that referenced this pull request Sep 16, 2020
@luanguimaraesla @jmlrt
[elasticsearch] add loadBalancer externalTrafficPolicy option (#810)
65bf107
jmlrt pushed a commit that referenced this pull request Sep 16, 2020
@luanguimaraesla @jmlrt
[elasticsearch] add loadBalancer externalTrafficPolicy option (#810)
981a4db
jmlrt pushed a commit that referenced this pull request Sep 16, 2020
@luanguimaraesla @jmlrt
[elasticsearch] add loadBalancer externalTrafficPolicy option (#810)
1c31642
@jmlrt
Copy link
Member
jmlrt commented Sep 16, 2020

backported to 6.8, 7.9 and 7.x branches

@nkammah nkammah mentioned this pull request Sep 24, 2020
Merged
@nkammah nkammah mentioned this pull request Oct 19, 2020
Closed
@elasticmachine elasticmachine mentioned this pull request Oct 21, 2020
Merged
@jmlrt jmlrt mentioned this pull request Oct 28, 2020
Closed
This was referenced Nov 17, 2020
Closed
Closed
Closed
Closed
@elasticmachine elasticmachine mentioned this pull request Nov 20, 2020
Closed
@jmlrt jmlrt mentioned this pull request Nov 23, 2020
Closed
@elasticmachine elasticmachine mentioned this pull request Nov 23, 2020
Merged
@jmlrt jmlrt mentioned this pull request Feb 8, 2021
Closed
This was referenced Mar 15, 2021
Closed
Closed
@jmlrt jmlrt mentioned this pull request May 25, 2021
Closed