[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL v2 45/50] tests/docker: fall back more gracefully when pull fails
From: |
Alex Bennée |
Subject: |
[PULL v2 45/50] tests/docker: fall back more gracefully when pull fails |
Date: |
Sat, 11 Jul 2020 18:07:27 +0100 |
I only spotted this in the small window between my testing with my
registry while waiting for the gitlab PR to go in. As we pre-pull the
registry image we know if that fails there isn't any point attempting
to use the cache. Fall back to the way we used to do it at that point.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20200709141327.14631-12-alex.bennee@linaro.org>
diff --git a/tests/docker/docker.py b/tests/docker/docker.py
index 9684f07bde..2d67bbd15a 100755
--- a/tests/docker/docker.py
+++ b/tests/docker/docker.py
@@ -306,13 +306,14 @@ class Docker(object):
checksum = _text_checksum(_dockerfile_preprocess(dockerfile))
if registry is not None:
- dockerfile = dockerfile.replace("FROM qemu/",
- "FROM %s/qemu/" %
- (registry))
# see if we can fetch a cache copy, may fail...
pull_args = ["pull", "%s/%s" % (registry, tag)]
- self._do(pull_args, quiet=quiet)
-
+ if self._do(pull_args, quiet=quiet) == 0:
+ dockerfile = dockerfile.replace("FROM qemu/",
+ "FROM %s/qemu/" %
+ (registry))
+ else:
+ registry = None
tmp_df = tempfile.NamedTemporaryFile(mode="w+t",
encoding='utf-8',
--
2.20.1
- [PULL v2 00/50] testing updates (vm, gitlab, misc build fixes), Alex Bennée, 2020/07/11
- [PULL v2 04/50] tests/vm: Add configuration to basevm.py, Alex Bennée, 2020/07/11
- [PULL v2 42/50] docs/devel: convert and update MTTCG design document, Alex Bennée, 2020/07/11
- [PULL v2 43/50] docs/devel: add some notes on tcg-icount for developers, Alex Bennée, 2020/07/11
- [PULL v2 44/50] docs: Add to gdbstub documentation the PhyMemMode, Alex Bennée, 2020/07/11
- [PULL v2 45/50] tests/docker: fall back more gracefully when pull fails,
Alex Bennée <=
- [PULL v2 46/50] tests/docker: update toolchain set in debian-xtensa-cross, Alex Bennée, 2020/07/11
- [PULL v2 47/50] .cirrus.yml: add bash to the brew packages, Alex Bennée, 2020/07/11
- [PULL v2 48/50] tests: fix "make check-qtest" for modular builds, Alex Bennée, 2020/07/11
- [PULL v2 49/50] Revert "vga: build virtio-gpu as module", Alex Bennée, 2020/07/11
- [PULL v2 50/50] iotests: Set LC_ALL=C for sort, Alex Bennée, 2020/07/11
- Re: [PULL v2 00/50] testing updates (vm, gitlab, misc build fixes), Peter Maydell, 2020/07/13