-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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 healthcheck in ES compose service #20511
Fix healthcheck in ES compose service #20511
Conversation
Python is not included anymore on Elasticsearch images, change the healthcheck to be based on curl and the easier to parse cat API.
Pinging @elastic/integrations (Team:Integrations) |
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 fixing this!
💔 Tests FailedExpand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
Failure in CI is not related, and previous failure is fixed. Merging. |
Python is not included anymore on Elasticsearch images, change the healthcheck to be based on curl and the easier to parse cat API. (cherry picked from commit 368d31b)
Python is not included anymore on Elasticsearch images, change the healthcheck to be based on curl and the easier to parse cat API.
Python is not included anymore on Elasticsearch images, change the
healthcheck to be based on curl and the easier to parse cat API.
This fixes failures in CI with master branch.