The container jobs are all optional in upstream now, only
running if dockerfile changes were made. Thus any build
jobs which reference the container jobs need to mark their
dependency as optional too.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 .gitlab-ci.yml | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 0fbdd82aae..013f821881 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -62,7 +62,8 @@ website:
   stage: builds
   image: $CI_REGISTRY_IMAGE/ci-almalinux-8:latest
   needs:
-    - x86_64-almalinux-8-container
+    - job: x86_64-almalinux-8-container
+      optional: true
   before_script:
     - *script_variables
   script:
@@ -82,7 +83,8 @@ codestyle:
   stage: sanity_checks
   image: $CI_REGISTRY_IMAGE/ci-opensuse-leap-153:latest
   needs:
-    - x86_64-opensuse-leap-153-container
+    - job: x86_64-opensuse-leap-153-container
+      optional: true
   before_script:
     - *script_variables
   script:
@@ -98,7 +100,8 @@ potfile:
   stage: builds
   image: $CI_REGISTRY_IMAGE/ci-almalinux-8:latest
   needs:
-    - x86_64-almalinux-8-container
+    - job: x86_64-almalinux-8-container
+      optional: true
   rules:
     - if: "$CI_COMMIT_BRANCH == 'master'"
   before_script:
-- 
2.36.1

On Fri, May 27, 2022 at 07:49:04AM -0400, Daniel P. Berrangé wrote:
> The container jobs are all optional in upstream now, only
> running if dockerfile changes were made. Thus any build
> jobs which reference the container jobs need to mark their
> dependency as optional too.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>  .gitlab-ci.yml | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 0fbdd82aae..013f821881 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -62,7 +62,8 @@ website:
>    stage: builds
>    image: $CI_REGISTRY_IMAGE/ci-almalinux-8:latest
>    needs:
> -    - x86_64-almalinux-8-container
> +    - job: x86_64-almalinux-8-container
> +      optional: true
>    before_script:
>      - *script_variables
>    script:
> @@ -82,7 +83,8 @@ codestyle:
>    stage: sanity_checks
>    image: $CI_REGISTRY_IMAGE/ci-opensuse-leap-153:latest
>    needs:
> -    - x86_64-opensuse-leap-153-container
> +    - job: x86_64-opensuse-leap-153-container
> +      optional: true
>    before_script:
>      - *script_variables
>    script:
> @@ -98,7 +100,8 @@ potfile:
>    stage: builds
>    image: $CI_REGISTRY_IMAGE/ci-almalinux-8:latest
>    needs:
> -    - x86_64-almalinux-8-container
> +    - job: x86_64-almalinux-8-container
> +      optional: true
>    rules:
>      - if: "$CI_COMMIT_BRANCH == 'master'"
>    before_script:
> -- 
> 2.36.1
> 

The coverity job needs the same treatment otherwise we'll see the same failure
tomorrow morning.

With that:
Reviewed-by: Erik Skultety <eskultet@redhat.com>

Erik