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

Running Deployments are deleted from ZK during leader abdication #4085

Closed
aquamatthias opened this issue Jul 12, 2016 · 0 comments
Closed

Running Deployments are deleted from ZK during leader abdication #4085

aquamatthias opened this issue Jul 12, 2016 · 0 comments
Assignees
@aquamatthias
Milestone
1.2

Comments

@aquamatthias
Copy link
Contributor

If the current Marathon leader abdicates, it cancels all running deployments.
Cancellation here means: mark the deployment as failed, publish a message and delete the deployment from the repository.

Expected behavior: stop the running deployment, but do not delete the deployment from the repository. The new leader will read and restart the Deployment.
@aquamatthias aquamatthias added this to the 1.2 milestone Jul 12, 2016
@aquamatthias aquamatthias self-assigned this Jul 12, 2016
aquamatthias added a commit that referenced this issue Jul 12, 2016
@aquamatthias
Fixes #4085 by stopping all running deployments instead of cancelling…
cd7df86 
… all running deployments, which would remove the deployment from the ZK repository.

Renamed CancelAllDeployments to StopAllDeployments
aquamatthias added a commit that referenced this issue Jul 18, 2016
@aquamatthias
Fixes 4085 by stopping all running deployments instead of cancelling … (
fd29ebf 
#4086)

* Fixes #4085 by stopping all running deployments instead of cancelling all running deployments, which would remove the deployment from the ZK repository.
Renamed CancelAllDeployments to StopAllDeployments

* Add test case for StopAllDeployments - make sure all deployments are stopped.
unterstein pushed a commit that referenced this issue Jul 25, 2016
@aquamatthias @unterstein
Fixes 4085 by stopping all running deployments instead of cancelling … (
eb118fc 
#4086)

* Fixes #4085 by stopping all running deployments instead of cancelling all running deployments, which would remove the deployment from the ZK repository.
Renamed CancelAllDeployments to StopAllDeployments

* Add test case for StopAllDeployments - make sure all deployments are stopped.
@d2iq-archive d2iq-archive locked and limited conversation to collaborators Mar 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@aquamatthias aquamatthias

Labels
None yet
Projects
None yet
Milestone
1.2
Development

No branches or pull requests
1 participant
@aquamatthias