Skip to content

Increase elasticsearch start timeout #217

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

Merged
merged 1 commit into from
Mar 29, 2016
Merged

Increase elasticsearch start timeout #217

merged 1 commit into from
Mar 29, 2016

Conversation

adamstegman
Copy link
Contributor

We're installing a few plugins into Elasticsearch by configuring them in elasticsearch.plugins:

properties:
  elasticsearch:
    plugins:
    - license: license
    - shield: shield
    - cloud-aws: cloud-aws
    - marvel: marvel-agent

At this point Elasticsearch takes well past 30 seconds to create its pidfile and monit gives up on it every time.

I saw in 96002b3 that the plugin installation was deliberately moved to a different pidfile, so I thought I'd address it by increasing the start timeout. I timed it; on an m3.medium this set of plugins delays startup of Elasticsearch by about 45 seconds, so I rounded to 60 and doubled it out of caution.

Plugin installation delays Elasticsearch starting
@cf-gitbot
Copy link
Collaborator

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/115982667.

@mrdavidlaing
Copy link
Member

@adamstegman - Thanks for this patch - looks most useful!

I can't seem to find a signed CLA for you; which we'll need before merging.

Please could you print & sign the relevant CLA from https://github.com/logsearch/licensing#signing-the-contributors-license-agreement and forward a scanned copy to [email protected]

Thanks!

@adamstegman
Copy link
Contributor Author

@mrdavidlaing I just sent it. Thanks for your patience!

@kirederik kirederik merged commit 8382184 into logsearch:develop Mar 29, 2016
@mrdavidlaing
Copy link
Member

CLA received. Safe to merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants