Skip to content

[orchestrator/task reaper] Backport: Clean up tasks in dirty list for which the service has been deleted.#2667

Merged
dperny merged 1 commit into
moby:bump_v17.06from
anshulpundir:b1706
Jun 19, 2018
Merged

[orchestrator/task reaper] Backport: Clean up tasks in dirty list for which the service has been deleted.#2667
dperny merged 1 commit into
moby:bump_v17.06from
anshulpundir:b1706

Conversation

@anshulpundir
Copy link
Copy Markdown
Contributor

@anshulpundir anshulpundir commented Jun 19, 2018

cherry-pick 592e8ed (#2666)

Cherry-pick was clean.

… service has been deleted.

Signed-off-by: Anshul Pundir <anshul.pundir@docker.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 19, 2018

Codecov Report

Merging #2667 into bump_v17.06 will decrease coverage by 0.74%.
The diff coverage is 0%.

@@               Coverage Diff               @@
##           bump_v17.06    #2667      +/-   ##
===============================================
- Coverage        61.71%   60.97%   -0.75%     
===============================================
  Files               42      121      +79     
  Lines             5692    20203   +14511     
===============================================
+ Hits              3513    12319    +8806     
- Misses            1840     6531    +4691     
- Partials           339     1353    +1014

Copy link
Copy Markdown
Contributor

@dani-docker dani-docker left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@cyli cyli left a comment

Choose a reason for hiding this comment

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

LGTM

@dperny dperny merged commit a20aab0 into moby:bump_v17.06 Jun 19, 2018
anshulpundir pushed a commit that referenced this pull request Jun 19, 2018
[orchestrator/task reaper] Backport: Clean up tasks in dirty list for which the service has been deleted.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants