Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix update condition bug in reloader #12156

Closed
wants to merge 5 commits into from

Conversation

DanielQujun
Copy link
Contributor

as mentioned in#12155 ,update condition is invalid

@DanielQujun DanielQujun requested a review from a team as a code owner May 10, 2019 03:11
@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?

@elasticcla
Copy link

Hi @DanielQujun, 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 your 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?

@DanielQujun
Copy link
Contributor Author

Hi @DanielQujun, 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 your 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?

email add done.

@DanielQujun
Copy link
Contributor Author

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?

yes

@DanielQujun
Copy link
Contributor Author

heart broken, why the test suit always got timeout error.

@exekias
Copy link
Contributor

exekias commented Feb 28, 2020

Closing as it was fixed by #16440

@exekias exekias closed this Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants