qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: qemu-devel@nongnu.org
Cc: fam@euphon.net, "Thomas Huth" <thuth@redhat.com>,
	berrange@redhat.com, "Cornelia Huck" <cohuck@redhat.com>,
	f4bug@amsat.org,
	"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
	"Willian Rampazzo" <willianr@redhat.com>,
	stefanha@redhat.com, crosa@redhat.com, pbonzini@redhat.com,
	"Alex Bennée" <alex.bennee@linaro.org>,
	aurelien@aurel32.net
Subject: [PATCH  v1 3/8] gitlab: add special rule for the hexagon container
Date: Thu, 20 May 2021 18:42:58 +0100	[thread overview]
Message-ID: <20210520174303.12310-4-alex.bennee@linaro.org> (raw)
In-Reply-To: <20210520174303.12310-1-alex.bennee@linaro.org>

The hexagon container is always manually built but of course not
everyone will be building it themselves and pushing to their
registries. We still need to create a "local" registry copy for the
actual gitlab tests to run. We don't build it in this case, just pull
it across from the upstream registry. We disable this rule from
running on the qemu-project itself so it doesn't accidentally wipe out
our master copy.

Fixes: 910c40ee94 ("gitlab: add build-user-hexagon test")
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: Cornelia Huck <cohuck@redhat.com>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Cc: Cornelia Huck <cohuck@redhat.com>

---
v2
  - fix silly typo
---
 .gitlab-ci.d/containers.yml | 27 +++++++++++++++++++++++++++
 .gitlab-ci.yml              |  2 ++
 2 files changed, 29 insertions(+)

diff --git a/.gitlab-ci.d/containers.yml b/.gitlab-ci.d/containers.yml
index 3fb3c14f06..088c7e68c3 100644
--- a/.gitlab-ci.d/containers.yml
+++ b/.gitlab-ci.d/containers.yml
@@ -101,6 +101,33 @@ armhf-debian-cross-container:
   variables:
     NAME: debian-armhf-cross
 
+# We never want to build hexagon in the CI system and by default we
+# always want to refer to the master registry where it lives.
+hexagon-cross-container:
+  image: docker:stable
+  stage: containers
+  except:
+    variables:
+      - $CI_PROJECT_NAMESPACE == 'qemu-project'
+  variables:
+    NAME: debian-hexagon-cross
+    GIT_DEPTH: 1
+  services:
+    - docker:dind
+  before_script:
+    - export TAG="$CI_REGISTRY_IMAGE/qemu/$NAME:latest"
+    - export COMMON_TAG="$CI_REGISTRY/qemu-project/qemu/qemu/$NAME:latest"
+    - docker info
+    - docker login $CI_REGISTRY -u "$CI_REGISTRY_USER" -p "$CI_REGISTRY_PASSWORD"
+  script:
+    - echo "TAG:$TAG"
+    - echo "COMMON_TAG:$COMMON_TAG"
+    - docker pull $COMMON_TAG
+    - docker tag $COMMON_TAG $TAG
+    - docker push "$TAG"
+  after_script:
+    - docker logout
+
 hppa-debian-cross-container:
   extends: .container_job_template
   stage: containers-layer2
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index f718b61fa7..b2f929c758 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -421,6 +421,8 @@ build-user-static:
 # declared. The image is manually uploaded.
 build-user-hexagon:
   extends: .native_build_job_template
+  needs:
+    job: hexagon-cross-container
   variables:
     IMAGE: debian-hexagon-cross
     TARGETS: hexagon-linux-user
-- 
2.20.1



  parent reply	other threads:[~2021-05-20 17:51 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-20 17:42 [PATCH v1 0/8] various misc fixes (gitlab, gdbstub, plugins) Alex Bennée
2021-05-20 17:42 ` [PATCH v1 1/8] tests/tcg: add a multiarch signals test to stress test signal delivery Alex Bennée
2021-05-24 21:05   ` Richard Henderson
2021-05-20 17:42 ` [PATCH v1 2/8] gitlab: explicitly reference the upstream registry Alex Bennée
2021-05-24 19:15   ` Willian Rampazzo
2021-05-25  4:04   ` Philippe Mathieu-Daudé
2021-05-20 17:42 ` Alex Bennée [this message]
2021-05-25  3:54   ` [PATCH v1 3/8] gitlab: add special rule for the hexagon container Philippe Mathieu-Daudé
2021-05-25  9:40   ` Philippe Mathieu-Daudé
2021-05-25 11:25     ` Philippe Mathieu-Daudé
2021-05-25 15:06       ` Alex Bennée
2021-05-20 17:42 ` [PATCH v1 4/8] gdbstub: Constify GdbCmdParseEntry Alex Bennée
2021-05-24 21:06   ` Richard Henderson
2021-05-20 17:43 ` [PATCH v1 5/8] gdbstub: Replace GdbCmdContext with plain g_array() Alex Bennée
2021-05-20 17:43 ` [PATCH v1 6/8] hmp-commands: expand type of icount to "l" in replay commands Alex Bennée
2021-05-25  3:55   ` Philippe Mathieu-Daudé
2021-05-20 17:43 ` [PATCH v1 7/8] gdbstub: tidy away reverse debugging check into function Alex Bennée
2021-05-24 21:08   ` Richard Henderson
2021-05-25  3:57   ` Philippe Mathieu-Daudé
2021-05-20 17:43 ` [PATCH v1 8/8] plugins/syscall: Added a table-like summary output Alex Bennée
2021-05-24 17:50 ` [PATCH v1 0/8] various misc fixes (gitlab, gdbstub, plugins) Alex Bennée

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210520174303.12310-4-alex.bennee@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=aurelien@aurel32.net \
    --cc=berrange@redhat.com \
    --cc=cohuck@redhat.com \
    --cc=crosa@redhat.com \
    --cc=f4bug@amsat.org \
    --cc=fam@euphon.net \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=thuth@redhat.com \
    --cc=wainersm@redhat.com \
    --cc=willianr@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).