[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 52/56] docker: docker.py add check sub-command
From: |
Alex Bennée |
Subject: |
[Qemu-devel] [PULL 52/56] docker: docker.py add check sub-command |
Date: |
Tue, 19 Jun 2018 16:44:31 +0100 |
This command allows you to check if we need to re-build a docker
image. If the image isn't in the repository or the checksums don't
match then we return false and some text (for processing in
makefiles).
Signed-off-by: Alex Bennée <address@hidden>
---
- fix prints to be "modern"
- PEP8 fixes
- split inspect into function
diff --git a/tests/docker/docker.py b/tests/docker/docker.py
index 0a0155c941..770b22424b 100755
--- a/tests/docker/docker.py
+++ b/tests/docker/docker.py
@@ -181,8 +181,14 @@ class Docker(object):
stderr=subprocess.STDOUT,
**kwargs)
+ def inspect_tag(self, tag):
+ try:
+ return self._output(["inspect", tag])
+ except subprocess.CalledProcessError:
+ return None
+
def get_image_dockerfile_checksum(self, tag):
- resp = self._output(["inspect", tag])
+ resp = self.inspect_tag(tag)
labels = json.loads(resp)[0]["Config"].get("Labels", {})
return labels.get("com.qemu.dockerfile-checksum", "")
@@ -444,6 +450,36 @@ class CcCommand(SubCommand):
return Docker().command("run", cmd, args.quiet)
+class CheckCommand(SubCommand):
+ """Check if we need to re-build a docker image out of a dockerfile.
+ Arguments: <tag> <dockerfile>"""
+ name = "check"
+
+ def args(self, parser):
+ parser.add_argument("tag",
+ help="Image Tag")
+ parser.add_argument("dockerfile",
+ help="Dockerfile name")
+
+ def run(self, args, argv):
+ dockerfile = open(args.dockerfile, "rb").read()
+ tag = args.tag
+
+ dkr = Docker()
+ info = dkr.inspect_tag(tag)
+ if info is None:
+ print("Image does not exist")
+ return 1
+
+ if dkr.image_matches_dockerfile(tag, dockerfile):
+ if not args.quiet:
+ print("Image is up to date")
+ return 0
+ else:
+ print("Image needs updating")
+ return 1
+
+
def main():
parser = argparse.ArgumentParser(description="A Docker helper",
usage="%s <subcommand> ..." % os.path.basename(sys.argv[0]))
--
2.17.1
- [Qemu-devel] [PULL 27/56] tests/tcg: move MIPS specific tests into subdir, (continued)
- [Qemu-devel] [PULL 27/56] tests/tcg: move MIPS specific tests into subdir, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 28/56] tests/tcg: enable building for MIPS, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 51/56] docker: docker.py don't conflate checksums for extra_files, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 20/56] tests/tcg/i386/test-i386: use modern vector_size attributes, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 30/56] tests/tcg/mips: include common mips hello-mips, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 36/56] tests/tcg: enable building for m68k, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 47/56] tests/tcg: override runners for broken tests, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 39/56] tests/tcg: enable building for mips64, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 55/56] tests/docker/Makefile.include: only force SID to NOCACHE if old, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 48/56] tests/tcg/i386: extend timeout for runcom test, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 52/56] docker: docker.py add check sub-command,
Alex Bennée <=
- [Qemu-devel] [PULL 41/56] docker: move debian-powerpc-cross to sid based build, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 43/56] tests/tcg/Makefile: update to be called from Makefile.target, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 40/56] tests/tcg: enable building for RISCV64, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 38/56] tests/tcg: enable building for sparc64, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 50/56] docker: docker.py use "version" to probe usage, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 49/56] tests: add top-level make dependency for docker builds, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 54/56] docker: docker.py adding age check command, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 35/56] tests/tcg: enable building for HPPA, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 15/56] tests/tcg: enable building for i386, Alex Bennée, 2018/06/19
- [Qemu-devel] [PULL 26/56] tests/tcg/arm: add fcvt test cases for AArch32/64, Alex Bennée, 2018/06/19