0ct0pu5/moby
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #47475 from thaJeztah/nothing_to_see_here_move_along_move_along

Browse source 
distribution/xfer: fix pull progress message
This commit is contained in:
Paweł Gronowski 2024-02-29 14:46:41 +01:00 • committed by GitHub
commit d19d98b136
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
distribution/xfer/download.go
View file

@ -364,7 +364,7 @@ func (ldm *LayerDownloadManager) makeDownloadFunc(descriptor DownloadDescriptor,
return
}
progress.Update(progressOutput, descriptor.ID(), "PullOptions complete")
progress.Update(progressOutput, descriptor.ID(), "Pull complete")
if withRegistered, ok := descriptor.(DigestRegisterer); ok {
withRegistered.Registered(d.layer.DiffID())

4
distribution/xfer/download_test.go
View file

@ -316,8 +316,8 @@ func TestSuccessfulDownload(t *testing.T) {
if receivedProgress[d.ID()].Action != "Already exists" {
t.Fatalf("did not get 'Already exists' message for %v", d.ID())
}
} else if receivedProgress[d.ID()].Action != "PullOptions complete" {
t.Fatalf("did not get 'PullOptions complete' message for %v", d.ID())
} else if receivedProgress[d.ID()].Action != "Pull complete" {
t.Fatalf("did not get 'Pull complete' message for %v", d.ID())
}
if rootFS.DiffIDs[i] != descriptor.expectedDiffID {