Skip to content

Don't schedule SLM jobs when services have been stopped#48658

Merged
dakrone merged 1 commit intoelastic:masterfrom
dakrone:slm-dont-schedule-during-shutdown
Oct 30, 2019
Merged

Don't schedule SLM jobs when services have been stopped#48658
dakrone merged 1 commit intoelastic:masterfrom
dakrone:slm-dont-schedule-during-shutdown

Conversation

@dakrone
Copy link
Copy Markdown
Member

@dakrone dakrone commented Oct 29, 2019

This adds a guard for the SLM lifecycle and retention service that
prevents new jobs from being scheduled once the service has been
stopped. Previous if the node were shut down the service would be
stopped, but a cluster state or local master election would cause a job
to attempt to be scheduled. This could lead to an uncaught
RejectedExecutionException.

Resolves #47749

This adds a guard for the SLM lifecycle and retention service that
prevents new jobs from being scheduled once the service has been
stopped. Previous if the node were shut down the service would be
stopped, but a cluster state or local master election would cause a job
to attempt to be scheduled. This could lead to an uncaught
`RejectedExecutionException`.

Resolves elastic#47749
@dakrone dakrone added >bug :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. v8.0.0 v7.5.0 v7.6.0 v7.4.2 labels Oct 29, 2019
@dakrone dakrone requested a review from AthenaEryma October 29, 2019 21:19
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-features (:Core/Features/ILM+SLM)

Copy link
Copy Markdown
Contributor

@AthenaEryma AthenaEryma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find! LGTM

@dakrone dakrone merged commit f1f1dda into elastic:master Oct 30, 2019
@dakrone dakrone deleted the slm-dont-schedule-during-shutdown branch October 30, 2019 15:00
dakrone added a commit to dakrone/elasticsearch that referenced this pull request Oct 30, 2019
This adds a guard for the SLM lifecycle and retention service that
prevents new jobs from being scheduled once the service has been
stopped. Previous if the node were shut down the service would be
stopped, but a cluster state or local master election would cause a job
to attempt to be scheduled. This could lead to an uncaught
`RejectedExecutionException`.

Resolves elastic#47749
dakrone added a commit to dakrone/elasticsearch that referenced this pull request Oct 30, 2019
This adds a guard for the SLM lifecycle and retention service that
prevents new jobs from being scheduled once the service has been
stopped. Previous if the node were shut down the service would be
stopped, but a cluster state or local master election would cause a job
to attempt to be scheduled. This could lead to an uncaught
`RejectedExecutionException`.

Resolves elastic#47749
dakrone added a commit to dakrone/elasticsearch that referenced this pull request Oct 30, 2019
This adds a guard for the SLM lifecycle and retention service that
prevents new jobs from being scheduled once the service has been
stopped. Previous if the node were shut down the service would be
stopped, but a cluster state or local master election would cause a job
to attempt to be scheduled. This could lead to an uncaught
`RejectedExecutionException`.

Resolves elastic#47749
dakrone added a commit that referenced this pull request Oct 30, 2019
* Don't schedule SLM jobs when services have been stopped (#48658)

This adds a guard for the SLM lifecycle and retention service that
prevents new jobs from being scheduled once the service has been
stopped. Previous if the node were shut down the service would be
stopped, but a cluster state or local master election would cause a job
to attempt to be scheduled. This could lead to an uncaught
`RejectedExecutionException`.

Resolves #47749

* Fix test for backport
@polyfractal polyfractal added v7.4.3 and removed v7.4.2 labels Oct 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. v7.4.3 v7.5.0 v7.6.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HistoryTemplateSearchInputMappingsTests failure on master with RejectedExecutionException

5 participants