Skip to content

Commit 03924bc

Browse files
author
Daniel Hiltgen
committed
Fix progress reporting for containerd pulls
During refactoring of the implementation PR progress reporting was broken. This gets the progress reporting back in action. Signed-off-by: Daniel Hiltgen <daniel.hiltgen@docker.com> (cherry picked from commit 7f4c842) Signed-off-by: Daniel Hiltgen <daniel.hiltgen@docker.com>
1 parent 6dbe8ea commit 03924bc

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

internal/containerizedengine/progress.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ outer:
6464
}
6565
}
6666

67-
err := updateNonActive(ctx, ongoing, cs, statuses, keys, activeSeen, &done, start)
67+
err := updateNonActive(ctx, ongoing, cs, statuses, &keys, activeSeen, &done, start)
6868
if err != nil {
6969
continue outer
7070
}
@@ -92,11 +92,11 @@ outer:
9292
}
9393
}
9494

95-
func updateNonActive(ctx context.Context, ongoing *jobs, cs content.Store, statuses map[string]statusInfo, keys []string, activeSeen map[string]struct{}, done *bool, start time.Time) error {
95+
func updateNonActive(ctx context.Context, ongoing *jobs, cs content.Store, statuses map[string]statusInfo, keys *[]string, activeSeen map[string]struct{}, done *bool, start time.Time) error {
9696

9797
for _, j := range ongoing.jobs() {
9898
key := remotes.MakeRefKey(ctx, j)
99-
keys = append(keys, key)
99+
*keys = append(*keys, key)
100100
if _, ok := activeSeen[key]; ok {
101101
continue
102102
}

0 commit comments

Comments
 (0)