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

UnreachableStrategy configuration has no effect; should not be used by resident tasks #5163

Closed
timcharper opened this issue Feb 10, 2017 · 1 comment
Assignees

Comments

@timcharper
Copy link
Contributor

timcharper commented Feb 10, 2017

We can specify an UnreachableStrategy on an AppDefinition, but it actually has no effect. When we instantiate the Instance, we don't pass the value specified in RunSpec, so the parameter default is used.

@timcharper
Copy link
Contributor Author

timcharper pushed a commit that referenced this issue Feb 10, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan: create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
@marcomonaco marcomonaco added bug and removed sprint-4 labels Feb 13, 2017
timcharper pushed a commit that referenced this issue Feb 13, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan: create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
timcharper pushed a commit that referenced this issue Feb 13, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan: create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
timcharper pushed a commit that referenced this issue Feb 13, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan: create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
timcharper pushed a commit that referenced this issue Feb 15, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan: create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
timcharper pushed a commit that referenced this issue Feb 15, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan: create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
timcharper pushed a commit that referenced this issue Feb 16, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan: create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
@timcharper timcharper mentioned this issue Feb 22, 2017
2 tasks
timcharper pushed a commit that referenced this issue Feb 22, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan:
create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Reviewers: aquamatthias, jdef, meichstedt, jenkins

Reviewed By: aquamatthias, jdef, meichstedt, jenkins

Subscribers: jdef, marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
timcharper pushed a commit that referenced this issue Feb 22, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Test Plan:
create resident task. Make it get lost. Ensure that it
doesn't come go inactive.

Reviewers: aquamatthias, jdef, meichstedt, jenkins

Reviewed By: aquamatthias, jdef, meichstedt, jenkins

Subscribers: jdef, marathon-team

Differential Revision: https://phabricator.mesosphere.com/D488
timcharper pushed a commit that referenced this issue Feb 22, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Backport of 5ff4ca0

Test Plan: unit tests; fully test all features when patch series is merged

Reviewers: jasongilanfarr, jdef

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D542
timcharper pushed a commit that referenced this issue Feb 23, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Backport of 5ff4ca0

Test Plan: unit tests; fully test all features when patch series is merged

Reviewers: jasongilanfarr, jdef

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D542
timcharper pushed a commit that referenced this issue Feb 24, 2017
Summary:
Require disabled for resident tasks. Fixes #5163. Partially addresses

Backport of 5ff4ca0

Test Plan: unit tests; fully test all features when patch series is merged

Reviewers: jasongilanfarr, jdef

Subscribers: marathon-team

Differential Revision: https://phabricator.mesosphere.com/D542
@timcharper timcharper changed the title UnreachableStrategy configuration has no effect UnreachableStrategy configuration has no effect; should not be used by resident tasks Mar 1, 2017
@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.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants