On 12/1/20 2:18 PM, Daniel P. Berrangé wrote:
> The travis container that we have no longer matches what travis
> currently uses. As all x86 jobs are being moved to GitLab CI too,
> there is no compelling reason to update the travis container. It
> is simpler to just remove it.
IMHO above rationale makes sense.
>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
> tests/docker/dockerfiles/travis.docker | 17 ----------
> tests/docker/travis | 22 ------------
> tests/docker/travis.py | 47 --------------------------
> 3 files changed, 86 deletions(-)
> delete mode 100644 tests/docker/dockerfiles/travis.docker
> delete mode 100755 tests/docker/travis
> delete mode 100755 tests/docker/travis.py
The tests/docker/Makefile.include file should be cleaned up too.
>
> diff --git a/tests/docker/dockerfiles/travis.docker b/tests/docker/dockerfiles/travis.docker
> deleted file mode 100644
> index cd1435a7e9..0000000000
> --- a/tests/docker/dockerfiles/travis.docker
> +++ /dev/null
> @@ -1,17 +0,0 @@
> -#
> -# Travis Image - this is broadly the same image that we run our CI
> -# tests on.
> -#
> -FROM travisci/ci-sardonyx:packer-1552557266-f909ac5
> -ENV DEBIAN_FRONTEND noninteractive
> -ENV LANG en_US.UTF-8
> -ENV LC_ALL en_US.UTF-8
> -RUN sed -i "s/# deb-src/deb-src/" /etc/apt/sources.list
> -RUN apt-get update
> -RUN apt-get -y build-dep qemu
> -RUN apt-get -y install device-tree-compiler python3 python3-yaml dh-autoreconf gdb strace lsof net-tools gcovr ninja-build
> -# Travis tools require PhantomJS / Neo4j / Maven accessible
> -# in their PATH (QEMU build won't access them).
> -ENV PATH /usr/local/phantomjs/bin:/usr/local/phantomjs:/usr/local/neo4j-3.2.7/bin:/usr/local/maven-3.5.2/bin:/usr/local/cmake-3.9.2/bin:/usr/local/clang-5.0.0/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
> -ENV FEATURES clang pyyaml docs
> -USER travis
> diff --git a/tests/docker/travis b/tests/docker/travis
> deleted file mode 100755
> index 47c03677d6..0000000000
> --- a/tests/docker/travis
> +++ /dev/null
> @@ -1,22 +0,0 @@
> -#!/bin/bash -e
> -#
> -# Mimic a travis testing matrix
> -#
> -# Copyright (c) 2016 Red Hat Inc.
> -#
> -# Authors:
> -# Fam Zheng <famz@redhat.com>
> -#
> -# This work is licensed under the terms of the GNU GPL, version 2
> -# or (at your option) any later version. See the COPYING file in
> -# the top-level directory.
> -
> -. common.rc
> -
> -requires pyyaml
> -cmdfile=/tmp/travis_cmd_list.sh
> -$QEMU_SRC/tests/docker/travis.py $QEMU_SRC/.travis.yml > $cmdfile
> -chmod +x $cmdfile
> -cd "$QEMU_SRC"
> -unset BUILD_DIR SRC_DIR
> -$cmdfile
> diff --git a/tests/docker/travis.py b/tests/docker/travis.py
> deleted file mode 100755
> index 37307ac366..0000000000
> --- a/tests/docker/travis.py
> +++ /dev/null
> @@ -1,47 +0,0 @@
> -#!/usr/bin/env python3
> -#
> -# Travis YAML config parser
> -#
> -# Copyright (c) 2016 Red Hat Inc.
> -#
> -# Authors:
> -# Fam Zheng <famz@redhat.com>
> -#
> -# This work is licensed under the terms of the GNU GPL, version 2
> -# or (at your option) any later version. See the COPYING file in
> -# the top-level directory.
> -
> -import sys
> -import yaml
> -import itertools
> -
> -def load_yaml(fname):
> - return yaml.safe_load(open(fname, "r").read())
> -
> -def conf_iter(conf):
> - # If "compiler" is omitted from the included env then Travis picks the
> - # first entry of the global compiler list.
> - default_compiler = conf["compiler"][0]
> - def env_to_list(env):
> - return env if isinstance(env, list) else [env]
> - for entry in conf["matrix"]["include"]:
> - yield {"env": env_to_list(entry["env"]),
> - "compiler": entry.get("compiler", default_compiler)}
> -
> -def main():
> - if len(sys.argv) < 2:
> - sys.stderr.write("Usage: %s <travis-file>\n" % sys.argv[0])
> - return 1
> - conf = load_yaml(sys.argv[1])
> - print("\n".join((": ${%s}" % var for var in conf["env"]["global"])))
> - for config in conf_iter(conf):
> - print("(")
> - print("\n".join(config["env"]))
> - print("alias cc=" + config["compiler"])
> - print("\n".join(conf["before_script"]))
> - print("\n".join(conf["script"]))
> - print(")")
> - return 0
> -
> -if __name__ == "__main__":
> - sys.exit(main())