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

distribution/xfer: fix pull progress message

Browse source 
This message accidentally changed in ac2a028dcc
because my IDE's "refactor tool" was a bit over-enthusiastic. It also went and
updated the tests accordingly, so CI didn't catch this :)

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This commit is contained in:
Sebastiaan van Stijn 2024-02-29 14:02:55 +01:00
parent a242208be3
commit ebf3f8c7fe
No known key found for this signature in database
GPG key ID: 76698F39D527CE8C
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 return
} }
progress.Update(progressOutput, descriptor.ID(), "PullOptions complete") progress.Update(progressOutput, descriptor.ID(), "Pull complete")
if withRegistered, ok := descriptor.(DigestRegisterer); ok { if withRegistered, ok := descriptor.(DigestRegisterer); ok {
withRegistered.Registered(d.layer.DiffID()) 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" { if receivedProgress[d.ID()].Action != "Already exists" {
t.Fatalf("did not get 'Already exists' message for %v", d.ID()) t.Fatalf("did not get 'Already exists' message for %v", d.ID())
} }
} else if receivedProgress[d.ID()].Action != "PullOptions complete" { } else if receivedProgress[d.ID()].Action != "Pull complete" {
t.Fatalf("did not get 'PullOptions complete' message for %v", d.ID()) t.Fatalf("did not get 'Pull complete' message for %v", d.ID())
} }
if rootFS.DiffIDs[i] != descriptor.expectedDiffID { if rootFS.DiffIDs[i] != descriptor.expectedDiffID {