[Qemu-commits] [qemu/qemu] d47e37: tests/docker: force NOUSER=1 for base images

[Richard Henderson]

  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: d47e3751b5b642b9a3d9604f48717a51f1cb0c01
      
https://github.com/qemu/qemu/commit/d47e3751b5b642b9a3d9604f48717a51f1cb0c01
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-11-16 (Tue, 16 Nov 2021)

  Changed paths:
    M tests/docker/Makefile.include

  Log Message:
  -----------
  tests/docker: force NOUSER=1 for base images

As base images are often used to build further images like toolchains
ensure we don't add the local user by accident. The local user should
only exist on local images and not anything that gets pushed up to the
public registry.

Reported-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20211115142915.3797652-2-alex.bennee@linaro.org>


  Commit: 81c9b06ea0bf7d5e0c0b00aae41df9060c063cfa
      
https://github.com/qemu/qemu/commit/81c9b06ea0bf7d5e0c0b00aae41df9060c063cfa
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-11-16 (Tue, 16 Nov 2021)

  Changed paths:
    M tests/vm/Makefile.include

  Log Message:
  -----------
  tests/vm: sort the special variable list

Making the list alphabetical makes it easier to find the config option
you are looking for.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20211115142915.3797652-3-alex.bennee@linaro.org>


  Commit: ebd654aabc1a9128b2ea390faa23c2fb6b4b13b8
      
https://github.com/qemu/qemu/commit/ebd654aabc1a9128b2ea390faa23c2fb6b4b13b8
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-11-16 (Tue, 16 Nov 2021)

  Changed paths:
    M tests/vm/Makefile.include

  Log Message:
  -----------
  tests/vm: don't build using TCG by default

While it is useful to run these images using TCG their performance
will not be anything like the native guests. Don't do it by default.

Fixes: https://gitlab.com/qemu-project/qemu/-/issues/393
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20211115142915.3797652-4-alex.bennee@linaro.org>


  Commit: a399f9143e801efbc574495c7df0eba3bd729e9f
      
https://github.com/qemu/qemu/commit/a399f9143e801efbc574495c7df0eba3bd729e9f
  Author: Paolo Bonzini <pbonzini@redhat.com>
  Date:   2021-11-16 (Tue, 16 Nov 2021)

  Changed paths:
    M accel/tcg/meson.build
    M meson.build

  Log Message:
  -----------
  meson: remove useless libdl test

dlopen is never used after it is sought via cc.find_library, because
plugins use gmodule instead; remove the test.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20211110092454.30916-1-pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20211115142915.3797652-5-alex.bennee@linaro.org>


  Commit: d7c2e2b3f447133175be5817663147d1309bde2d
      
https://github.com/qemu/qemu/commit/d7c2e2b3f447133175be5817663147d1309bde2d
  Author: Cleber Rosa <crosa@redhat.com>
  Date:   2021-11-16 (Tue, 16 Nov 2021)

  Changed paths:
    M .gitlab-ci.d/custom-runners.yml
    M docs/devel/ci-jobs.rst.inc
    A scripts/ci/org.centos/stream/8/build-environment.yml
    A scripts/ci/org.centos/stream/8/x86_64/configure
    A scripts/ci/org.centos/stream/8/x86_64/test-avocado
    A scripts/ci/org.centos/stream/README
    M scripts/ci/setup/build-environment.yml

  Log Message:
  -----------
  Jobs based on custom runners: add CentOS Stream 8

This introduces three different parts of a job designed to run
on a custom runner managed by Red Hat.  The goals include:

  a) propose a model for other organizations that want to onboard
     their own runners, with their specific platforms, build
     configuration and tests.

  b) bring awareness to the differences between upstream QEMU and the
     version available under CentOS Stream, which is "A preview of
     upcoming Red Hat Enterprise Linux minor and major releases".

  c) because of b), it should be easier to identify and reduce the gap
     between Red Hat's downstream and upstream QEMU.

The components of this custom job are:

  I) OS build environment setup code:

     - additions to the existing "build-environment.yml" playbook
       that can be used to set up CentOS/EL 8 systems.

     - a CentOS Stream 8 specific "build-environment.yml" playbook
       that adds to the generic one.

 II) QEMU build configuration: a script that will produce binaries with
     features as similar as possible to the ones built and packaged on
     CentOS stream 8.

III) Scripts that define the minimum amount of testing that the
     binaries built with the given configuration (point II) under the
     given OS build environment (point I) should be subjected to.

 IV) Job definition: GitLab CI jobs that will dispatch the build/test
     jobs (see points #II and #III) to the machine specifically
     configured according to #I.

Signed-off-by: Cleber Rosa <crosa@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Tested-by: Willian Rampazzo <willianr@redhat.com>
Message-Id: <20211111160501.862396-2-crosa@redhat.com>
Message-Id: <20211115142915.3797652-6-alex.bennee@linaro.org>


  Commit: 60bec83e8a5c7cbaa47648f4b3370c8d53e48982
      
https://github.com/qemu/qemu/commit/60bec83e8a5c7cbaa47648f4b3370c8d53e48982
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-11-16 (Tue, 16 Nov 2021)

  Changed paths:
    M .gitlab-ci.d/custom-runners.yml
    A .gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
    A .gitlab-ci.d/custom-runners/ubuntu-18.04-s390x.yml
    A .gitlab-ci.d/custom-runners/ubuntu-20.04-aarch64.yml

  Log Message:
  -----------
  gitlab-ci: Split custom-runners.yml in one file per runner

To ease maintenance, add the custom-runners/ directory and
split custom-runners.yml in 3 files, all included by the
current custom-runners.yml:
 - ubuntu-18.04-s390x.yml
 - ubuntu-20.04-aarch64.yml
 - centos-stream-8-x86_64.yml

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20211115095608.2436223-1-philmd@redhat.com>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Message-Id: <20211115142915.3797652-7-alex.bennee@linaro.org>


  Commit: 9968de0a4a5470bd7b98dcd2fae5d5269908f16b
      
https://github.com/qemu/qemu/commit/9968de0a4a5470bd7b98dcd2fae5d5269908f16b
  Author: Daniel P. Berrangé <berrange@redhat.com>
  Date:   2021-11-16 (Tue, 16 Nov 2021)

  Changed paths:
    M .gitlab-ci.d/cirrus.yml

  Log Message:
  -----------
  gitlab: skip cirrus jobs on master and stable branches

On the primary QEMU repository we want the CI jobs to run on the staging
branch as a gating CI test.

Cirrus CI has very limited job concurrency, so if there are too many
jobs triggered they'll queue up and hit the GitLab CI job timeout before
they complete on Cirrus.

If we let Cirrus jobs run again on the master branch immediately after
merging from staging, that just increases the chances jobs will get
queued and subsequently timeout.

The same applies for merges to the stable branches.

User forks meanwhile should be allowed to run Cirrus CI jobs freely.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Message-Id: <20211116112757.1909176-1-berrange@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>


  Commit: 9104bd89dadd0b17c414ba24b0841d0042179a2e
      
https://github.com/qemu/qemu/commit/9104bd89dadd0b17c414ba24b0841d0042179a2e
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-11-16 (Tue, 16 Nov 2021)

  Changed paths:
    M .gitlab-ci.d/cirrus.yml
    M .gitlab-ci.d/custom-runners.yml
    A .gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
    A .gitlab-ci.d/custom-runners/ubuntu-18.04-s390x.yml
    A .gitlab-ci.d/custom-runners/ubuntu-20.04-aarch64.yml
    M accel/tcg/meson.build
    M docs/devel/ci-jobs.rst.inc
    M meson.build
    A scripts/ci/org.centos/stream/8/build-environment.yml
    A scripts/ci/org.centos/stream/8/x86_64/configure
    A scripts/ci/org.centos/stream/8/x86_64/test-avocado
    A scripts/ci/org.centos/stream/README
    M scripts/ci/setup/build-environment.yml
    M tests/docker/Makefile.include
    M tests/vm/Makefile.include

  Log Message:
  -----------
  Merge tag 'pull-for-6.2-161121-1' of https://github.com/stsquad/qemu into 
staging

Misc build and test fixes:

  - force NOUSER for base docker images
  - don't run TCG VM tests by default
  - remove useless meson test
  - add Centos 8 custom runner
  - split up custom-runners to individual files
  - skip cirrus checks on master/stable branches

# gpg: Signature made Tue 16 Nov 2021 05:22:09 PM CET
# gpg:                using RSA key 6685AE99E75167BCAFC8DF35FBD0DB095A9E2A44
# gpg: Good signature from "Alex Bennée (Master Work Key) 
<alex.bennee@linaro.org>" [full]

* tag 'pull-for-6.2-161121-1' of https://github.com/stsquad/qemu:
  gitlab: skip cirrus jobs on master and stable branches
  gitlab-ci: Split custom-runners.yml in one file per runner
  Jobs based on custom runners: add CentOS Stream 8
  meson: remove useless libdl test
  tests/vm: don't build using TCG by default
  tests/vm: sort the special variable list
  tests/docker: force NOUSER=1 for base images

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Compare: https://github.com/qemu/qemu/compare/871c71b1bad2...9104bd89dadd