[PATCH 2/3] docker: EDK2 build job depends on EDK2 container

Philippe Mathieu-Daudé posted 3 patches 4 years, 11 months ago
[PATCH 2/3] docker: EDK2 build job depends on EDK2 container
Posted by Philippe Mathieu-Daudé 4 years, 11 months ago
Add missing dependency build-edk2 -> docker-edk2.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 .gitlab-ci.d/edk2.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitlab-ci.d/edk2.yml b/.gitlab-ci.d/edk2.yml
index e1e04524166..ba7280605c4 100644
--- a/.gitlab-ci.d/edk2.yml
+++ b/.gitlab-ci.d/edk2.yml
@@ -25,6 +25,7 @@ docker-edk2:
 
 build-edk2:
  stage: build
+ needs: ['docker-edk2']
  rules: # Only run this job when ...
  - changes: # ... roms/edk2/ is modified (submodule updated)
    - roms/edk2/*
-- 
2.26.2

Re: [PATCH 2/3] docker: EDK2 build job depends on EDK2 container
Posted by Daniel P. Berrangé 4 years, 11 months ago
On Wed, Mar 03, 2021 at 02:06:45PM +0100, Philippe Mathieu-Daudé wrote:
> Add missing dependency build-edk2 -> docker-edk2.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  .gitlab-ci.d/edk2.yml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/.gitlab-ci.d/edk2.yml b/.gitlab-ci.d/edk2.yml
> index e1e04524166..ba7280605c4 100644
> --- a/.gitlab-ci.d/edk2.yml
> +++ b/.gitlab-ci.d/edk2.yml
> @@ -25,6 +25,7 @@ docker-edk2:
>  
>  build-edk2:
>   stage: build
> + needs: ['docker-edk2']
>   rules: # Only run this job when ...
>   - changes: # ... roms/edk2/ is modified (submodule updated)
>     - roms/edk2/*

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


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|


Re: [PATCH 2/3] docker: EDK2 build job depends on EDK2 container
Posted by Laszlo Ersek 4 years, 11 months ago
On 03/03/21 14:06, Philippe Mathieu-Daudé wrote:
> Add missing dependency build-edk2 -> docker-edk2.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  .gitlab-ci.d/edk2.yml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/.gitlab-ci.d/edk2.yml b/.gitlab-ci.d/edk2.yml
> index e1e04524166..ba7280605c4 100644
> --- a/.gitlab-ci.d/edk2.yml
> +++ b/.gitlab-ci.d/edk2.yml
> @@ -25,6 +25,7 @@ docker-edk2:
>  
>  build-edk2:
>   stage: build
> + needs: ['docker-edk2']
>   rules: # Only run this job when ...
>   - changes: # ... roms/edk2/ is modified (submodule updated)
>     - roms/edk2/*
> 

Acked-by: Laszlo Ersek <lersek@redhat.com>


Re: [PATCH 2/3] docker: EDK2 build job depends on EDK2 container
Posted by Willian Rampazzo 4 years, 11 months ago
On Wed, Mar 3, 2021 at 10:10 AM Philippe Mathieu-Daudé
<philmd@redhat.com> wrote:
>
> Add missing dependency build-edk2 -> docker-edk2.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  .gitlab-ci.d/edk2.yml | 1 +
>  1 file changed, 1 insertion(+)
>

Reviewed-by: Willian Rampazzo <willianr@redhat.com>