diff mbox series

[1/1] gitlab-ci: Add KVM x86 cross-build jobs

Message ID 20210512044228.3254073-2-philmd@redhat.com (mailing list archive)
State New, archived
Headers show
Series gitlab-ci: Add KVM x86 cross-build jobs | expand

Commit Message

Philippe Mathieu-Daudé May 12, 2021, 4:42 a.m. UTC
Add a new job to cross-build the x86 target without the
TCG/XEN accelerators (IOW: only KVM accelerator enabled).

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

Comments

Thomas Huth May 12, 2021, 4:57 a.m. UTC | #1
On 12/05/2021 06.42, Philippe Mathieu-Daudé wrote:
> Add a new job to cross-build the x86 target without the
> TCG/XEN accelerators (IOW: only KVM accelerator enabled).
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>   .gitlab-ci.d/crossbuilds.yml | 8 ++++++++
>   1 file changed, 8 insertions(+)
> 
> diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitlab-ci.d/crossbuilds.yml
> index e44e4b49a25..a8eac2cddba 100644
> --- a/.gitlab-ci.d/crossbuilds.yml
> +++ b/.gitlab-ci.d/crossbuilds.yml
> @@ -198,6 +198,14 @@ cross-win64-system:
>     variables:
>       IMAGE: fedora-win64-cross
>   
> +cross-i386-kvm-only:
> +  extends: .cross_accel_build_job
> +  needs:
> +    job: i386-fedora-cross-container
> +  variables:
> +    IMAGE: fedora-i386-cross
> +    ACCEL_CONFIGURE_OPTS: --disable-tcg --disable-xen

We already have build-tcg-disabled in the main .gitlab-ci.yml file ... so I 
fail to see what additional test coverage we get by this job?

  Thomas
Philippe Mathieu-Daudé May 12, 2021, 5:06 a.m. UTC | #2
On 5/12/21 6:57 AM, Thomas Huth wrote:
> On 12/05/2021 06.42, Philippe Mathieu-Daudé wrote:
>> Add a new job to cross-build the x86 target without the
>> TCG/XEN accelerators (IOW: only KVM accelerator enabled).
>>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>>   .gitlab-ci.d/crossbuilds.yml | 8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>> diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitlab-ci.d/crossbuilds.yml
>> index e44e4b49a25..a8eac2cddba 100644
>> --- a/.gitlab-ci.d/crossbuilds.yml
>> +++ b/.gitlab-ci.d/crossbuilds.yml
>> @@ -198,6 +198,14 @@ cross-win64-system:
>>     variables:
>>       IMAGE: fedora-win64-cross
>>   +cross-i386-kvm-only:
>> +  extends: .cross_accel_build_job
>> +  needs:
>> +    job: i386-fedora-cross-container
>> +  variables:
>> +    IMAGE: fedora-i386-cross
>> +    ACCEL_CONFIGURE_OPTS: --disable-tcg --disable-xen
> 
> We already have build-tcg-disabled in the main .gitlab-ci.yml file ...
> so I fail to see what additional test coverage we get by this job?

Doh I missed build-tcg-disabled was the x86 job, sorry.

Please disregard this patch!
diff mbox series

Patch

diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitlab-ci.d/crossbuilds.yml
index e44e4b49a25..a8eac2cddba 100644
--- a/.gitlab-ci.d/crossbuilds.yml
+++ b/.gitlab-ci.d/crossbuilds.yml
@@ -198,6 +198,14 @@  cross-win64-system:
   variables:
     IMAGE: fedora-win64-cross
 
+cross-i386-kvm-only:
+  extends: .cross_accel_build_job
+  needs:
+    job: i386-fedora-cross-container
+  variables:
+    IMAGE: fedora-i386-cross
+    ACCEL_CONFIGURE_OPTS: --disable-tcg --disable-xen
+
 cross-amd64-xen-only:
   extends: .cross_accel_build_job
   needs: