Revert "[manager/scheduler] Ignore unassigned tasks with desired state beyond running." [WIP]

[anshulpundir]

This reverts commit e3fcf6d.(#2574)

This is likely the root cause for #2705

[GordonTheTurtle]

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "rev" git@github.com:anshulpundir/swarmkit.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

[codecov]

Codecov Report

Merging #2710 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2710      +/-   ##
==========================================
- Coverage   61.89%   61.79%   -0.11%     
==========================================
  Files         134      134              
  Lines       21771    21769       -2     
==========================================
- Hits        13476    13452      -24     
- Misses       6836     6864      +28     
+ Partials     1459     1453       -6

[thaJeztah]

can we / should we write an (e2e) test based on
#2705 to verify the behavior. and prevent regressions?

[anshulpundir]

can we / should we write an (e2e) test based on
#2705 to verify the behavior. and prevent regressions?

I am going to try to add a unit-test that verifies the expected scheduler behavior.

[anshulpundir]

This is likely the root cause for #2705

Not the root cause. Actual root cause is captured in #2705