This repository has been archived by the owner on Oct 23, 2024. It is now read-only.
Marathon fails to release reserved resources for deleted apps #5142
Assignees
Comments
timcharper
pushed a commit
that referenced
this issue
Feb 9, 2017
Summary: Address #5142 Test Plan: add tests Reviewers: jasongilanfarr, meichstedt Subscribers: marathon-team Differential Revision: https://phabricator.mesosphere.com/D498
|
I agree - the same logic as |
Closed
6 tasks
This was referenced Feb 22, 2017
Closed
timcharper
pushed a commit
that referenced
this issue
Feb 23, 2017
Summary: If a resident app is deleted before Marathon has a chance to release the associated resources, then later attempts to release them fail because because we ForceExpunge the task-ID when processing our UnreserveAndDestroyVolumes operation. This change makes ForceExpunge for a non-existent task respond with no-op, rather than failure. Fixes #5142 Backport of f441c57 Test Plan: n/a Reviewers: jasongilanfarr, meichstedt Subscribers: marathon-team Differential Revision: https://phabricator.mesosphere.com/D547
timcharper
pushed a commit
that referenced
this issue
Feb 23, 2017
Summary: If a resident app is deleted before Marathon has a chance to release the associated resources, then later attempts to release them fail because because we ForceExpunge the task-ID when processing our UnreserveAndDestroyVolumes operation. This change makes ForceExpunge for a non-existent task respond with no-op, rather than failure. Fixes #5142 Backport of f441c57 Test Plan: n/a Reviewers: jasongilanfarr, meichstedt Subscribers: marathon-team Differential Revision: https://phabricator.mesosphere.com/D547
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
As of this writing, with the head of e62695b, I've encountered a case in which Marathon cannot release reserved resources because they are identified as spurious, which results in a force expunge. Down the line, the [force expunge fails because the app does not exist].
Expunge should probably be made idempotent, such that if it doesn't exist, then the result is successful.
The text was updated successfully, but these errors were encountered: