kotovalexarian-likes-github/moby--moby
1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00
Code Releases Activity

Merge pull request #18418 from aaronlehmann/no-head-requests

Browse source 
Avoid a HEAD request for each layer in a v2 pull
This commit is contained in:
David Calavera 2015-12-04 14:51:54 -08:00
commit 82c47088db
1 changed files with 15 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

23
distribution/pull_v2.go
View file

@ -130,14 +130,6 @@ func (p *v2Puller) download(di *downloadInfo) {
blobs := p.repo.Blobs(context.Background())
desc, err := blobs.Stat(context.Background(), di.digest)
if err != nil {
logrus.Debugf("Error statting layer: %v", err)
di.err <- err
return
}
di.size = desc.Size
layerDownload, err := blobs.Open(context.Background(), di.digest)
if err != nil {
logrus.Debugf("Error fetching layer: %v", err)
@ -146,6 +138,21 @@ func (p *v2Puller) download(di *downloadInfo) {
}
defer layerDownload.Close()
di.size, err = layerDownload.Seek(0, os.SEEK_END)
if err != nil {
// Seek failed, perhaps because there was no Content-Length
// header. This shouldn't fail the download, because we can
// still continue without a progress bar.
di.size = 0
} else {
// Restore the seek offset at the beginning of the stream.
_, err = layerDownload.Seek(0, os.SEEK_SET)
if err != nil {
di.err <- err
return
}
}
verifier, err := digest.NewDigestVerifier(di.digest)
if err != nil {
di.err <- err