[PATCH 0/2] {ArmVirtPkg,OvmfPkg}/PlatformCI: stick with "ubuntu-18.04" for now

[PATCH 0/2] {ArmVirtPkg,OvmfPkg}/PlatformCI: stick with "ubuntu-18.04" for now

[Laszlo Ersek]

Personal build: https://github.com/tianocore/edk2/pull/1237

"vm_image: 'ubuntu-latest'" now refers to Ubuntu Focal (20.04LTS), not
Ubuntu Bionic (18.04LTS), according to
<https://github.com/actions/virtual-environments/issues/1816>.

In Focal, the "qemu" package is a dummy package with no dependencies,
and so the actual emulators are not pulled in. Compare:

  https://packages.ubuntu.com/bionic/qemu
  https://packages.ubuntu.com/focal/qemu

This causes CI runs to fail.

It would be best to switch to the "qemu-system" package name, which
continues to depend on the emulators:

  https://packages.ubuntu.com/bionic/qemu-system
  https://packages.ubuntu.com/focal/qemu-system

However, while that package does make the emulators available, the
emulators crash. So for now, stick with the previous Ubuntu environment,
which continues to be supported, per
<https://github.com/actions/virtual-environments/issues/1816>.

Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>

Thanks,
Laszlo

Laszlo Ersek (2):
  ArmVirtPkg/PlatformCI: stick with "ubuntu-18.04" for now
  OvmfPkg/PlatformCI: stick with "ubuntu-18.04" for now

 ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml | 2 +-
 OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.19.1.3.g30247aa5d201

[PATCH 1/2] ArmVirtPkg/PlatformCI: stick with "ubuntu-18.04" for now

[Laszlo Ersek]

"vm_image: 'ubuntu-latest'" now refers to Ubuntu Focal (20.04LTS), not
Ubuntu Bionic (18.04LTS), according to
<https://github.com/actions/virtual-environments/issues/1816>.

In Focal, the "qemu" package is a dummy package with no dependencies, and
so the actual emulators are not pulled in. Compare:

  https://packages.ubuntu.com/bionic/qemu
  https://packages.ubuntu.com/focal/qemu

This causes CI runs to fail.

It would be best to switch to the "qemu-system" package name, which
continues to depend on the emulators:

  https://packages.ubuntu.com/bionic/qemu-system
  https://packages.ubuntu.com/focal/qemu-system

However, while that package does make the emulators available, the
emulators crash. So for now, stick with the previous Ubuntu environment,
which continues to be supported, per
<https://github.com/actions/virtual-environments/issues/1816>.

Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
---
 ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
index b9452ec5bfaf..c5e14248bc33 100644
--- a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
+++ b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
@@ -17,7 +17,7 @@ jobs:
   - job: Platform_CI
     variables:
       package: 'ArmVirtPkg'
-      vm_image: 'ubuntu-latest'
+      vm_image: 'ubuntu-18.04'
       should_run: true
       run_flags: "MAKE_STARTUP_NSH=TRUE QEMU_HEADLESS=TRUE"
 
-- 
2.19.1.3.g30247aa5d201

[PATCH 2/2] OvmfPkg/PlatformCI: stick with "ubuntu-18.04" for now

[Laszlo Ersek]

"vm_image: 'ubuntu-latest'" now refers to Ubuntu Focal (20.04LTS), not
Ubuntu Bionic (18.04LTS), according to
<https://github.com/actions/virtual-environments/issues/1816>.

In Focal, the "qemu" package is a dummy package with no dependencies, and
so the actual emulators are not pulled in. Compare:

  https://packages.ubuntu.com/bionic/qemu
  https://packages.ubuntu.com/focal/qemu

This causes CI runs to fail.

It would be best to switch to the "qemu-system" package name, which
continues to depend on the emulators:

  https://packages.ubuntu.com/bionic/qemu-system
  https://packages.ubuntu.com/focal/qemu-system

However, while that package does make the emulators available, the
emulators crash. So for now, stick with the previous Ubuntu environment,
which continues to be supported, per
<https://github.com/actions/virtual-environments/issues/1816>.

Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
---
 OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
index a47d273217ab..df6a62cd6107 100644
--- a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
+++ b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
@@ -17,7 +17,7 @@ jobs:
   - job: Platform_CI
     variables:
       package: 'OvmfPkg'
-      vm_image: 'ubuntu-latest'
+      vm_image: 'ubuntu-18.04'
       should_run: true
       run_flags: "MAKE_STARTUP_NSH=TRUE QEMU_HEADLESS=TRUE"
 
-- 
2.19.1.3.g30247aa5d201

Re: [edk2-devel] [PATCH 0/2] {ArmVirtPkg,OvmfPkg}/PlatformCI: stick with "ubuntu-18.04" for now

[Michael D Kinney]

Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Laszlo Ersek
> Sent: Thursday, December 17, 2020 12:41 PM
> To: edk2-devel-groups-io <devel@edk2.groups.io>
> Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Leif Lindholm
> <leif@nuviainc.com>; Philippe Mathieu-Daudé <philmd@redhat.com>
> Subject: [edk2-devel] [PATCH 0/2] {ArmVirtPkg,OvmfPkg}/PlatformCI: stick with "ubuntu-18.04" for now
> 
> Personal build: https://github.com/tianocore/edk2/pull/1237
> 
> "vm_image: 'ubuntu-latest'" now refers to Ubuntu Focal (20.04LTS), not
> Ubuntu Bionic (18.04LTS), according to
> <https://github.com/actions/virtual-environments/issues/1816>.
> 
> In Focal, the "qemu" package is a dummy package with no dependencies,
> and so the actual emulators are not pulled in. Compare:
> 
>   https://packages.ubuntu.com/bionic/qemu
>   https://packages.ubuntu.com/focal/qemu
> 
> This causes CI runs to fail.
> 
> It would be best to switch to the "qemu-system" package name, which
> continues to depend on the emulators:
> 
>   https://packages.ubuntu.com/bionic/qemu-system
>   https://packages.ubuntu.com/focal/qemu-system
> 
> However, while that package does make the emulators available, the
> emulators crash. So for now, stick with the previous Ubuntu environment,
> which continues to be supported, per
> <https://github.com/actions/virtual-environments/issues/1816>.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> 
> Thanks,
> Laszlo
> 
> Laszlo Ersek (2):
>   ArmVirtPkg/PlatformCI: stick with "ubuntu-18.04" for now
>   OvmfPkg/PlatformCI: stick with "ubuntu-18.04" for now
> 
>  ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml | 2 +-
>  OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml    | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> --
> 2.19.1.3.g30247aa5d201
> 
> 
> 
> 
> 

Re: [PATCH 0/2] {ArmVirtPkg,OvmfPkg}/PlatformCI: stick with "ubuntu-18.04" for now

[Philippe Mathieu-Daudé]

On 12/17/20 9:40 PM, Laszlo Ersek wrote:
> Personal build: https://github.com/tianocore/edk2/pull/1237
> 
> "vm_image: 'ubuntu-latest'" now refers to Ubuntu Focal (20.04LTS), not
> Ubuntu Bionic (18.04LTS), according to
> <https://github.com/actions/virtual-environments/issues/1816>.
> 
> In Focal, the "qemu" package is a dummy package with no dependencies,
> and so the actual emulators are not pulled in. Compare:
> 
>   https://packages.ubuntu.com/bionic/qemu
>   https://packages.ubuntu.com/focal/qemu
> 
> This causes CI runs to fail.
> 
> It would be best to switch to the "qemu-system" package name, which
> continues to depend on the emulators:
> 
>   https://packages.ubuntu.com/bionic/qemu-system
>   https://packages.ubuntu.com/focal/qemu-system
> 
> However, while that package does make the emulators available, the
> emulators crash. So for now, stick with the previous Ubuntu environment,
> which continues to be supported, per
> <https://github.com/actions/virtual-environments/issues/1816>.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> 
> Thanks,
> Laszlo
> 
> Laszlo Ersek (2):
>   ArmVirtPkg/PlatformCI: stick with "ubuntu-18.04" for now
>   OvmfPkg/PlatformCI: stick with "ubuntu-18.04" for now
> 
>  ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml | 2 +-
>  OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml    | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Series:
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>

Re: [PATCH 0/2] {ArmVirtPkg,OvmfPkg}/PlatformCI: stick with "ubuntu-18.04" for now

[Ard Biesheuvel]

On 12/17/20 9:40 PM, Laszlo Ersek wrote:
> Personal build: https://github.com/tianocore/edk2/pull/1237
> 
> "vm_image: 'ubuntu-latest'" now refers to Ubuntu Focal (20.04LTS), not
> Ubuntu Bionic (18.04LTS), according to
> <https://github.com/actions/virtual-environments/issues/1816>.
> 
> In Focal, the "qemu" package is a dummy package with no dependencies,
> and so the actual emulators are not pulled in. Compare:
> 
>   https://packages.ubuntu.com/bionic/qemu
>   https://packages.ubuntu.com/focal/qemu
> 
> This causes CI runs to fail.
> 
> It would be best to switch to the "qemu-system" package name, which
> continues to depend on the emulators:
> 
>   https://packages.ubuntu.com/bionic/qemu-system
>   https://packages.ubuntu.com/focal/qemu-system
> 
> However, while that package does make the emulators available, the
> emulators crash. So for now, stick with the previous Ubuntu environment,
> which continues to be supported, per
> <https://github.com/actions/virtual-environments/issues/1816>.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> 
> Thanks,
> Laszlo
> 
> Laszlo Ersek (2):
>   ArmVirtPkg/PlatformCI: stick with "ubuntu-18.04" for now
>   OvmfPkg/PlatformCI: stick with "ubuntu-18.04" for now
> 

Reviewed-by: Ard Biesheuvel <ard.biesheuvel@arm.com>

Re: [edk2-devel] [PATCH 0/2] {ArmVirtPkg,OvmfPkg}/PlatformCI: stick with "ubuntu-18.04" for now

[Laszlo Ersek]

On 12/17/20 21:40, Laszlo Ersek wrote:
> Personal build: https://github.com/tianocore/edk2/pull/1237
> 
> "vm_image: 'ubuntu-latest'" now refers to Ubuntu Focal (20.04LTS), not
> Ubuntu Bionic (18.04LTS), according to
> <https://github.com/actions/virtual-environments/issues/1816>.
> 
> In Focal, the "qemu" package is a dummy package with no dependencies,
> and so the actual emulators are not pulled in. Compare:
> 
>   https://packages.ubuntu.com/bionic/qemu
>   https://packages.ubuntu.com/focal/qemu
> 
> This causes CI runs to fail.
> 
> It would be best to switch to the "qemu-system" package name, which
> continues to depend on the emulators:
> 
>   https://packages.ubuntu.com/bionic/qemu-system
>   https://packages.ubuntu.com/focal/qemu-system
> 
> However, while that package does make the emulators available, the
> emulators crash. So for now, stick with the previous Ubuntu environment,
> which continues to be supported, per
> <https://github.com/actions/virtual-environments/issues/1816>.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> 
> Thanks,
> Laszlo
> 
> Laszlo Ersek (2):
>   ArmVirtPkg/PlatformCI: stick with "ubuntu-18.04" for now
>   OvmfPkg/PlatformCI: stick with "ubuntu-18.04" for now
> 
>  ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml | 2 +-
>  OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml    | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 

Thanks for the reviews, merged as commit range
e6ae24e1d676..c487970ac89d, via
<https://github.com/tianocore/edk2/pull/1240>.

Laszlo