Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

add missing become: root #461

Merged
merged 2 commits into from
Jun 28, 2018
Merged

add missing become: root #461

merged 2 commits into from
Jun 28, 2018

Conversation

bilsch
Copy link
Contributor

@bilsch bilsch commented Jun 26, 2018

I'm not sure if there are other become statements missing but these were the ones I needed initially ;)

Are these known? I saw at least one in 460

@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?

@karmi
Copy link

karmi commented Jun 26, 2018

Hi @bilsch, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in yout Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

@itsmed
Copy link
Contributor

itsmed commented Jun 26, 2018

jenkins test this please

@Crazybus
Copy link
Contributor

Jenkins test this please

Copy link
Contributor

@Crazybus Crazybus left a comment

Choose a reason for hiding this comment

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

Thanks for this! This is clearly something affecting lots of other people that wasn't caught by our testing.

@Crazybus Crazybus merged commit 27fcaa1 into elastic:master Jun 28, 2018
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.

5 participants