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

Add more info to debug Kibana client error #4669

Merged
merged 2 commits into from Jul 14, 2017

Conversation

Projects
None yet
3 participants
@andrewkroh
Copy link
Member

andrewkroh commented Jul 13, 2017

Log the raw response with errors to make it easier to understand the cause of the error.

I also enhanced the docker healthcheck to look at the /api/status response from Kibana rather than just a plain HTTP 200 from /.

For example:

Exiting: Error importing Kibana dashboards: fail to create the Kibana loader: Error creating Kibana client: fail to get the Kibana version:fail to unmarshal the response from GET http://localhost:5601/api/status: invalid character '<' looking for beginning of value, response=<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1"... (truncated)

andrewkroh added some commits Jul 13, 2017

Modify healthcheck to check /api/status for green
Tests were failing when accessing :5601/api/status and getting back an HTML page that says "Kibana did not load properly. Check the server output for more information."
Add more info to debug Kibana client error
Log the raw response with errors.

@andrewkroh andrewkroh added the review label Jul 13, 2017

@@ -6,7 +6,9 @@ ARG DOWNLOAD_URL
ARG ELASTIC_VERSION
ARG XPACK

HEALTHCHECK --retries=6 CMD curl -f http://localhost:5601
RUN apt-get update && apt-get install -y jq && apt-get clean

This comment has been minimized.

Copy link
@ruflin

ruflin Jul 14, 2017

Collaborator

This will probably slow down our builds quite a bit. Is that something we should contribute back to the official image?

@ruflin ruflin merged commit 71a1ad2 into elastic:master Jul 14, 2017

6 checks passed

CLA Commit author has signed the CLA
Details
beats-ci Build finished.
Details
codecov/patch 15.38% of diff hit (within 100% threshold of 58.35%)
Details
codecov/project 62.83% (+4.48%) compared to 573eb67
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@monicasarbu monicasarbu added the libbeat label Jul 17, 2017

@andrewkroh andrewkroh deleted the andrewkroh:bugfix/kibana-response-log branch Apr 20, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.