kotovalexarian-likes-github
/
moby--moby
mirror of https://github.com/moby/moby.git
1
0
Fork 0
Code Releases Activity

Merge pull request #26504 from tonistiigi/frozen-fix 

Fix already loaded detection for frozen images
This commit is contained in:
Alexander Morozov 2016-09-14 08:22:48 -07:00 committed by GitHub
parent 8ac2000f50 ed747e5a9e
commit 397df22963
2 changed files with 31 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
integration-cli/fixtures/load/frozen.go
View File

@ -21,16 +21,26 @@ var frozenImgDir = "/docker-frozen-images"
// the passed in images
func FrozenImagesLinux(dockerBinary string, images ...string) error {
imgNS := os.Getenv("TEST_IMAGE_NAMESPACE")
var loadImages []string
var loadImages []struct{ srcName, destName string }
for _, img := range images {
if imgNS != "" {
img = imgNS + "/" + img
}
if err := exec.Command(dockerBinary, "inspect", "--type=image", img).Run(); err != nil {
loadImages = append(loadImages, img)
srcName := img
// hello-world:latest gets re-tagged as hello-world:frozen
// there are some tests that use hello-world:latest specifically so it pulls
// the image and hello-world:frozen is used for when we just want a super
// small image
if img == "hello-world:frozen" {
srcName = "hello-world:latest"
}
if imgNS != "" {
srcName = imgNS + "/" + srcName
}
loadImages = append(loadImages, struct{ srcName, destName string }{
srcName: srcName,
destName: img,
})
}
}
if len(loadImages) == 0 {
// everything is loaded, we're done
return nil
@ -38,32 +48,33 @@ func FrozenImagesLinux(dockerBinary string, images ...string) error {
fi, err := os.Stat(frozenImgDir)
if err != nil || !fi.IsDir() {
if err := pullImages(dockerBinary, loadImages); err != nil {
srcImages := make([]string, 0, len(loadImages))
for _, img := range loadImages {
srcImages = append(srcImages, img.srcName)
}
if err := pullImages(dockerBinary, srcImages); err != nil {
return errors.Wrap(err, "error pulling image list")
}
} else {
if err := loadFrozenImags(dockerBinary); err != nil {
if err := loadFrozenImages(dockerBinary); err != nil {
return err
}
}
if imgNS != "" {
for _, img := range loadImages {
target := strings.TrimPrefix(img, imgNS+"/")
if target != img {
if out, err := exec.Command(dockerBinary, "tag", img, target).CombinedOutput(); err != nil {
return errors.Errorf("%v: %s", err, string(out))
}
if out, err := exec.Command(dockerBinary, "rmi", img).CombinedOutput(); err != nil {
return errors.Errorf("%v: %s", err, string(out))
}
for _, img := range loadImages {
if img.srcName != img.destName {
if out, err := exec.Command(dockerBinary, "tag", img.srcName, img.destName).CombinedOutput(); err != nil {
return errors.Errorf("%v: %s", err, string(out))
}
if out, err := exec.Command(dockerBinary, "rmi", img.srcName).CombinedOutput(); err != nil {
return errors.Errorf("%v: %s", err, string(out))
}
}
}
return nil
}
func loadFrozenImags(dockerBinary string) error {
func loadFrozenImages(dockerBinary string) error {
tar, err := exec.LookPath("tar")
if err != nil {
return errors.Wrap(err, "could not find tar binary")

19
integration-cli/fixtures_linux_daemon.go
View File

@ -16,30 +16,13 @@ import (
)
func ensureFrozenImagesLinux(t *testing.T) {
images := []string{"busybox:latest", "hello-world:latest", "debian:jessie"}
images := []string{"busybox:latest", "hello-world:frozen", "debian:jessie"}
err := load.FrozenImagesLinux(dockerBinary, images...)
if err != nil {
t.Log(dockerCmdWithError("images"))
t.Fatalf("%+v", err)
}
// hello-world:latest gets re-tagged as hello-world:frozen
// there are some tests that use hello-world:latest specifically so it pulls
// the image and hello-world:frozen is used for when we just want a super
// small image
if out, err := exec.Command(dockerBinary, "tag", "hello-world:latest", "hello-world:frozen").CombinedOutput(); err != nil {
t.Log(dockerCmdWithError("images"))
t.Fatal(string(out))
}
if out, err := exec.Command(dockerBinary, "rmi", "hello-world:latest").CombinedOutput(); err != nil {
t.Log(dockerCmdWithError("images"))
t.Fatal(string(out))
}
for _, img := range images {
if img == "hello-world:latest" {
img = "hello-world:frozen"
}
protectedImages[img] = struct{}{}
}
}