-
Notifications
You must be signed in to change notification settings - Fork 1.9k
[apm-server] Add option loadBalancerIP to service #1075
Conversation
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? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM⛴
jenkins test this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some changes are required to make Black formatter happy in https://devops-ci.elastic.co/job/elastic+helm-charts+pull-request+lint-python/828/label=docker&&virtual&&ubuntu-18.04/console.
Can you run black <CHART>/tests/<CHART>_test.py
?
jenkins test this please |
I think this will also need to be rebased on master to make tests pass. |
Signed-off-by: ebuildy <ebuildy@gmail.com>
jenkins test this please |
${CHART}/tests/*.py
${CHART}/examples/*/test/goss.yaml
See #1031 - this PR add "service.loadBalancerIP" field. (copied from Kibana chart).