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

[17.06] progress: Show progress of replicated tasks before they are assigned #97

Merged
merged 1 commit into from
Jul 11, 2017
Merged

Conversation

andrewhsu
Copy link
Contributor

Backport fix:

This was only showing tasks that belong to nodes that are currently up,
so that tasks on down nodes don't appear to be stuck. But this
unintentionally excludes tasks that haven't been assigned yet, so if a
task is stuck before assignment, for example because no nodes meet its
constraints, a progress bar won't even be shown. The check should only
apply to tasks that have a node assignment.

Signed-off-by: Aaron Lehmann <[email protected]>
(cherry picked from commit d3d09f67b18673ec457c5f6634d346a937c58728)
Signed-off-by: Andrew Hsu <[email protected]>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

ping @aaronlehmann

@aaronlehmann
Copy link
Contributor

LGTM

@thaJeztah
Copy link
Member

@andrewhsu looks ready to merge

@andrewhsu andrewhsu merged commit 38f7bac into docker-archive:17.06 Jul 11, 2017
@andrewhsu andrewhsu deleted the fix-progress branch July 11, 2017 22:56
@andrewhsu andrewhsu modified the milestone: 17.06.1 Jul 12, 2017
docker-jenkins pushed a commit that referenced this pull request Mar 29, 2018
Add building code for Ubuntu Bionic (18.04) LTS
Upstream-commit: f2312f5
Component: packaging
silvin-lubecki pushed a commit to silvin-lubecki/docker-ce that referenced this pull request Jan 29, 2020
[17.06] progress: Show progress of replicated tasks before they are assigned
silvin-lubecki pushed a commit to silvin-lubecki/docker-ce that referenced this pull request Jan 30, 2020
Add building code for Ubuntu Bionic (18.04) LTS
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants