Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Backport: Don't destroy persistent volumes when killing unreachable resident tasks #5304

Conversation

timcharper
Copy link
Contributor

Summary:
Killing an unreachable resident task will do nothing, rather than
destroy the reservations.

Fixes #5207

Also-By: tharper@mesosphere.com

Test Plan: sbt test

Reviewers: unterstein, meichstedt, jasongilanfarr, jenkins

Reviewed By: meichstedt, jasongilanfarr, jenkins

Subscribers: jdef, marathon-team

Differential Revision: https://phabricator.mesosphere.com/D529

@timcharper timcharper mentioned this pull request Mar 6, 2017
2 tasks
@marcomonaco marcomonaco added the pr label Mar 6, 2017
@timcharper timcharper requested review from meichstedt and jdef March 6, 2017 20:10
Copy link
Contributor

@meichstedt meichstedt left a comment

Choose a reason for hiding this comment

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

LGTM

There are conflicts to resolve though :|

Copy link
Contributor

@unterstein unterstein left a comment

Choose a reason for hiding this comment

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

Conflicting files
src/main/scala/mesosphere/marathon/core/task/termination/impl/KillServiceActor.scala

Otherwise LGTM

Summary:
Killing an unreachable resident task will do nothing, rather than
destroy the reservations.

Fixes #5207

Also-By: tharper@mesosphere.com

Test Plan: sbt test

Reviewers: unterstein, meichstedt, jasongilanfarr, jenkins

Reviewed By: meichstedt, jasongilanfarr, jenkins

Subscribers: jdef, marathon-team

Differential Revision: https://phabricator.mesosphere.com/D529
@timcharper timcharper force-pushed the tharper/backport-d529-dont-destroy-unreachable-persistent-vols branch from fec2729 to 1455fdd Compare March 9, 2017 14:24
@timcharper timcharper merged commit 5f34abd into releases/1.4 Mar 9, 2017
@timcharper timcharper deleted the tharper/backport-d529-dont-destroy-unreachable-persistent-vols branch March 9, 2017 14:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants