Skip to content

Conversation

@dakrone
Copy link
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
@elasticmachine
Copy link
Collaborator

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

Copy link
Contributor

@gwbrown gwbrown 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HistoryTemplateSearchInputMappingsTests failure on master with RejectedExecutionException

5 participants