[RESEND v1 0/1] CI: Use Fedora 37 / GCC 12 for Linux jobs

[RESEND v1 0/1] CI: Use Fedora 37 / GCC 12 for Linux jobs

[Oliver Steffen]

Switch the CI (Linux) to use GCC 12 providied by a new Fedora 37 based
container image.

New container image: https://github.com/tianocore/containers/pull/60

Test PR: https://github.com/tianocore/edk2/pull/4108

Oliver Steffen (1):
  CI: Use Fedora-37 (gcc12) image for Linux jobs

 .azurepipelines/templates/defaults.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.39.2

[RESEND v1 1/1] CI: Use Fedora-37 (gcc12) image for Linux jobs

[Oliver Steffen]

Switch to the new Fedora-37 CI docker image, and with it to gcc12, for
Linux jobs.

Signed-off-by: Oliver Steffen <osteffen@redhat.com>
Reviewed-by: Sunil V L <sunilvl@ventanamicro.com>
Reviewed-by: Michael Kubacki <michael.kubacki@microsoft.com>
---
 .azurepipelines/templates/defaults.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.azurepipelines/templates/defaults.yml b/.azurepipelines/templates/defaults.yml
index 74d6b417839d..190d9a09d2a7 100644
--- a/.azurepipelines/templates/defaults.yml
+++ b/.azurepipelines/templates/defaults.yml
@@ -9,4 +9,4 @@
 
 variables:
   default_python_version: ">=3.10.6"
-  default_linux_image: "ghcr.io/tianocore/containers/fedora-35-test:47addc9"
+  default_linux_image: "ghcr.io/tianocore/containers/fedora-37-test:14d2aba"
-- 
2.39.2

Re: [edk2-devel] [RESEND v1 1/1] CI: Use Fedora-37 (gcc12) image for Linux jobs

[Rebecca Cran]

Reviewed-by: Rebecca Cran <rebecca@bsdio.com>


On 3/14/23 9:21 AM, Oliver Steffen wrote:
> Switch to the new Fedora-37 CI docker image, and with it to gcc12, for
> Linux jobs.
>
> Signed-off-by: Oliver Steffen <osteffen@redhat.com>
> Reviewed-by: Sunil V L <sunilvl@ventanamicro.com>
> Reviewed-by: Michael Kubacki <michael.kubacki@microsoft.com>
> ---
>   .azurepipelines/templates/defaults.yml | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.azurepipelines/templates/defaults.yml b/.azurepipelines/templates/defaults.yml
> index 74d6b417839d..190d9a09d2a7 100644
> --- a/.azurepipelines/templates/defaults.yml
> +++ b/.azurepipelines/templates/defaults.yml
> @@ -9,4 +9,4 @@
>   
>   variables:
>     default_python_version: ">=3.10.6"
> -  default_linux_image: "ghcr.io/tianocore/containers/fedora-35-test:47addc9"
> +  default_linux_image: "ghcr.io/tianocore/containers/fedora-37-test:14d2aba"

Re: [edk2-devel] [RESEND v1 1/1] CI: Use Fedora-37 (gcc12) image for Linux jobs

[Dhaval Sharma]

[-- Attachment #1: Type: text/plain, Size: 267 bytes --]

Hi Oliver,
when is this patch expected to get merged? I need to use CI with GCC12 in order to get my changes through for this
https://edk2.groups.io/g/devel/message/101795?p=%2C%2C%2C20%2C0%2C0%2C0%3A%3Arecentpostdate%2Fsticky%2C%2Ccmo%2C20%2C2%2C0%2C97826395
=D

[-- Attachment #2: Type: text/html, Size: 464 bytes --]

Re: [edk2-devel] [RESEND v1 1/1] CI: Use Fedora-37 (gcc12) image for Linux jobs

[Oliver Steffen]

[-- Attachment #1: Type: text/plain, Size: 463 bytes --]

On Thu, Mar 30, 2023 at 3:48 PM Dhaval Sharma <dhaval@rivosinc.com> wrote:

> Hi Oliver,
> when is this patch expected to get merged? I need to use CI with GCC12 in
> order to get my changes through for this
>
> https://edk2.groups.io/g/devel/message/101795?p=%2C%2C%2C20%2C0%2C0%2C0%3A%3Arecentpostdate%2Fsticky%2C%2Ccmo%2C20%2C2%2C0%2C97826395
> =D


Hi Dhaval,

I am not sure. All good from my side. And we have some R-Bs  already...

-Oliver

[-- Attachment #2: Type: text/html, Size: 1308 bytes --]

Re: [edk2-devel] [RESEND v1 1/1] CI: Use Fedora-37 (gcc12) image for Linux jobs

[Rebecca Cran]

This isn't a problem with _this_ patch, but just a note that ReadMe.rst 
is outdated because it mentions Ubuntu_GCC5, ArmVirtPkg_Ubuntu_GCC5 etc. 
while it looks like we've been using Fedora images for a while.


-- 
Rebecca Cran


On 3/14/23 9:21 AM, Oliver Steffen wrote:
> Switch to the new Fedora-37 CI docker image, and with it to gcc12, for
> Linux jobs.
>
> Signed-off-by: Oliver Steffen <osteffen@redhat.com>
> Reviewed-by: Sunil V L <sunilvl@ventanamicro.com>
> Reviewed-by: Michael Kubacki <michael.kubacki@microsoft.com>
> ---
>   .azurepipelines/templates/defaults.yml | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.azurepipelines/templates/defaults.yml b/.azurepipelines/templates/defaults.yml
> index 74d6b417839d..190d9a09d2a7 100644
> --- a/.azurepipelines/templates/defaults.yml
> +++ b/.azurepipelines/templates/defaults.yml
> @@ -9,4 +9,4 @@
>   
>   variables:
>     default_python_version: ">=3.10.6"
> -  default_linux_image: "ghcr.io/tianocore/containers/fedora-35-test:47addc9"
> +  default_linux_image: "ghcr.io/tianocore/containers/fedora-37-test:14d2aba"

Re: [RESEND v1 0/1] CI: Use Fedora 37 / GCC 12 for Linux jobs

[Gerd Hoffmann]

On Tue, Mar 14, 2023 at 04:21:13PM +0100, Oliver Steffen wrote:
> Switch the CI (Linux) to use GCC 12 providied by a new Fedora 37 based
> container image.
> 
> New container image: https://github.com/tianocore/containers/pull/60
> 
> Test PR: https://github.com/tianocore/edk2/pull/4108

PING CI maintainers.

It got three reviews and no concerns raised.
Can we please get this merged?
The riscv folks need gcc-12 in CI to make progress.

thanks,
  Gerd

Re: [edk2-devel] [RESEND v1 0/1] CI: Use Fedora 37 / GCC 12 for Linux jobs

[Dhaval Sharma]

[-- Attachment #1: Type: text/plain, Size: 42 bytes --]

Ping..Do we have approval yet for GCC12?

[-- Attachment #2: Type: text/html, Size: 42 bytes --]

回复: [edk2-devel] [RESEND v1 0/1] CI: Use Fedora 37 / GCC 12 for Linux jobs

[gaoliming]

[-- Attachment #1: Type: text/plain, Size: 464 bytes --]

Sharma:

 I see edk2 commit e5e1cd1a83e2e7aa2179db3de5fc00d76713ec6f Switch linux image default to Fedora-37.

 

Thanks

Liming

发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Dhaval Sharma
发送时间: 2023年5月7日 1:29
收件人: Oliver Steffen <osteffen@redhat.com>; devel@edk2.groups.io
主题: Re: [edk2-devel] [RESEND v1 0/1] CI: Use Fedora 37 / GCC 12 for Linux jobs

 

Ping..Do we have approval yet for GCC12? 




[-- Attachment #2: Type: text/html, Size: 3892 bytes --]