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

Handle exit code of 1 when terminating Kibana process on windows #24151

Merged
merged 5 commits into from
Oct 22, 2018

Conversation

liza-mae
Copy link
Contributor

@liza-mae liza-mae commented Oct 17, 2018

Addresses #22312, part of #21315.

Similar fix to #23441

@liza-mae liza-mae changed the title Handle exit code of 1 on windows Handle exit code of 1 when terminating Kibana process on windows Oct 17, 2018
@liza-mae liza-mae self-assigned this Oct 17, 2018
@liza-mae liza-mae requested a review from spalger October 17, 2018 17:19
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@liza-mae
Copy link
Contributor Author

CI Failure is not related to this PR and is a known issue
18:06:46 Chrome 70.0.3538 (Linux 0.0.0) normalizeBreakdown returns correct breakdown FAILED

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Co-Authored-By: liza-mae <ldayoub@gmail.com>
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@liza-mae liza-mae merged commit f9927cb into elastic:master Oct 22, 2018
@liza-mae liza-mae deleted the fix/issue-22312 branch October 22, 2018 18:11
liza-mae added a commit to liza-mae/kibana that referenced this pull request Oct 22, 2018
…stic#24151)

* Handle exit code of 1 on windows

* Remove space to fix intake failure

* Update using solution from Spencer

* Update debug log message

Co-Authored-By: liza-mae <ldayoub@gmail.com>
liza-mae added a commit to liza-mae/kibana that referenced this pull request Oct 22, 2018
…stic#24151)

* Handle exit code of 1 on windows

* Remove space to fix intake failure

* Update using solution from Spencer

* Update debug log message

Co-Authored-By: liza-mae <ldayoub@gmail.com>
liza-mae added a commit that referenced this pull request Oct 23, 2018
) (#24357)

* Handle exit code of 1 on windows

* Remove space to fix intake failure

* Update using solution from Spencer

* Update debug log message

Co-Authored-By: liza-mae <ldayoub@gmail.com>
liza-mae added a commit that referenced this pull request Oct 23, 2018
) (#24358)

* Handle exit code of 1 on windows

* Remove space to fix intake failure

* Update using solution from Spencer

* Update debug log message

Co-Authored-By: liza-mae <ldayoub@gmail.com>
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.

3 participants