-
Notifications
You must be signed in to change notification settings - Fork 1.9k
added loadBalancerIP option to service #539
added loadBalancerIP option to service #539
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? |
💚 CLA has been signed |
Hi @adulescentulus, |
jenkins test this please |
@jmlrt I signed the CLA about 8 hours ago and got a pdf. I don't know what's next. Is there a manual processing step left? |
cla/check |
CLA check is done automatically. However it seems you signed it with your personnal email while the git commit has been done with some corporate email. Can you change your commit email to the one you used to sign CLA? You should be able to do that with the following commands: git commit --amend --author="John Doe <john@doe.org>"
git push --force-with-lease |
@jmlrt thanks for the hint. I fixed my mail address and now CLA passes and I hope the CI as it did locally |
jenkins test this please |
sorry just merged from master and so needs rebuild, didn't expect someone to be awake |
jenkins test this please |
aaahh now I get it: between two functions there need to be two blank lines, not one like I did. I need to fix this, don't I? |
Oh yes sorry, I relaunched the test without checking logs. Indeed our python lint test is using black and requires 2 new lines between each functions. |
finally hope everything is good to go now. but why did |
python lint job is using |
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.
LGTM, thanks for this PR 👍
- based on PR: elastic#539 Signed-off-by: Max Kovgan <max@opsguru.io>
${CHART}/tests/*.py
opened to address #257 feature request which I need in a project. The old #323 somehow got stale.