diff mbox series

.gitlab-ci.d/cirrus.yml: Fix the NetBSD and OpenBSD Cirrus-CI KVM jobs

Message ID 20240424074202.245646-1-thuth@redhat.com (mailing list archive)
State New, archived
Headers show
Series .gitlab-ci.d/cirrus.yml: Fix the NetBSD and OpenBSD Cirrus-CI KVM jobs | expand

Commit Message

Thomas Huth April 24, 2024, 7:42 a.m. UTC
Pulling the "master" libvirt-ci containers does not work anymore,
so we have to switch to the "latest" instead. See also:
https://gitlab.com/libvirt/libvirt/-/commit/5d591421220c850aa64a640
https://gitlab.com/libvirt/libvirt-ci/-/commit/6e3c5ccac77714be70c0

The effects were dormant for a year as the old ':master' tags were
not removed until some weeks ago.

The other Cirrus-CI jobs were already fixed in commit 1d2f2b35bc86b7a1
("gitlab-ci/cirrus: switch from 'master' to 'latest'"), but apparently
nobody noticed the KVM jobs so far that need to be triggered manually.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 .gitlab-ci.d/cirrus.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Daniel P. Berrangé April 24, 2024, 8:23 a.m. UTC | #1
On Wed, Apr 24, 2024 at 09:42:02AM +0200, Thomas Huth wrote:
> Pulling the "master" libvirt-ci containers does not work anymore,
> so we have to switch to the "latest" instead. See also:
> https://gitlab.com/libvirt/libvirt/-/commit/5d591421220c850aa64a640
> https://gitlab.com/libvirt/libvirt-ci/-/commit/6e3c5ccac77714be70c0
> 
> The effects were dormant for a year as the old ':master' tags were
> not removed until some weeks ago.
> 
> The other Cirrus-CI jobs were already fixed in commit 1d2f2b35bc86b7a1
> ("gitlab-ci/cirrus: switch from 'master' to 'latest'"), but apparently
> nobody noticed the KVM jobs so far that need to be triggered manually.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  .gitlab-ci.d/cirrus.yml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


With regards,
Daniel
Philippe Mathieu-Daudé April 24, 2024, 8:37 a.m. UTC | #2
On 24/4/24 09:42, Thomas Huth wrote:
> Pulling the "master" libvirt-ci containers does not work anymore,
> so we have to switch to the "latest" instead. See also:
> https://gitlab.com/libvirt/libvirt/-/commit/5d591421220c850aa64a640
> https://gitlab.com/libvirt/libvirt-ci/-/commit/6e3c5ccac77714be70c0
> 
> The effects were dormant for a year as the old ':master' tags were
> not removed until some weeks ago.
> 
> The other Cirrus-CI jobs were already fixed in commit 1d2f2b35bc86b7a1
> ("gitlab-ci/cirrus: switch from 'master' to 'latest'"), but apparently
> nobody noticed the KVM jobs so far that need to be triggered manually.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   .gitlab-ci.d/cirrus.yml | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
diff mbox series

Patch

diff --git a/.gitlab-ci.d/cirrus.yml b/.gitlab-ci.d/cirrus.yml
index 4671f069c3..8b45f72910 100644
--- a/.gitlab-ci.d/cirrus.yml
+++ b/.gitlab-ci.d/cirrus.yml
@@ -95,7 +95,7 @@  aarch64-macos-14-base-build:
 .cirrus_kvm_job:
   extends: .base_job_template
   stage: build
-  image: registry.gitlab.com/libvirt/libvirt-ci/cirrus-run:master
+  image: registry.gitlab.com/libvirt/libvirt-ci/cirrus-run:latest
   needs: []
   timeout: 80m
   script: