Your message dated Mon, 21 Oct 2024 12:18:58 +0000
with message-id <[email protected]>
and subject line Bug#1081374: fixed in autopkgtest 5.40
has caused the Debian Bug report #1081374,
regarding /usr/share/autopkgtest/lib/autopkgtest_qemu.py: autopkgtest-virt-qemu 
not setting qemu-* options correctly to boot with UEFI
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
1081374: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1081374
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: autopkgtest
Version: 5.39
Severity: normal
File: /usr/share/autopkgtest/lib/autopkgtest_qemu.py
X-Debbugs-Cc: [email protected]

Dear Maintainer,

as I was trying to use an Ubuntu image for autopkgtest that can only be
booted with UEFI, I recognized that it was not booting since it was
using SeaBIOS even though I used --efi or --boot=efi option.

After debugging I found out that the new_session function in
autopkgtest_qemu.py does not pass efi_code and efi_data to the
QemuSession. After passing those variables, the image boots with UEFI.
These are the changes the I made:

diff --git a/lib/autopkgtest_qemu.py b/lib/autopkgtest_qemu.py
index 870633b3..b64c38db 100644
--- a/lib/autopkgtest_qemu.py
+++ b/lib/autopkgtest_qemu.py
@@ -662,6 +662,8 @@ class QemuFactory:
 
         return QemuSession(
             boot=self.boot,
+            efi_code=self.efi_code,
+            efi_data=self.efi_data,
             qemu_architecture=self.qemu_architecture,
             qemu_command=self.qemu_command,
             qemu_options=qemu_options,


-- System Information:
Debian Release: trixie/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 6.10.6-amd64 (SMP w/16 CPU threads; PREEMPT)
Locale: LANG=C.UTF-8, LC_CTYPE=C.UTF-8 (charmap=locale: Cannot set LC_ALL to 
default locale: No such file or directory
UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages autopkgtest depends on:
ii  apt-utils       2.9.8
ii  libdpkg-perl    1.22.11
ii  mawk            1.3.4.20240819-3
ii  procps          2:4.0.4-5
ii  python3         3.12.5-1
ii  python3-debian  0.1.49
ii  retry           1.0.5-3

Versions of packages autopkgtest recommends:
ii  autodep8  0.28+nmu1
ii  fakeroot  1.36-1

Versions of packages autopkgtest suggests:
pn  docker-cli | docker.io  <none>
pn  docker.io               <none>
pn  fakemachine             <none>
ii  genisoimage             9:1.1.11-3.5
pn  incus                   <none>
pn  lxc                     <none>
pn  lxd                     <none>
ii  ovmf                    2024.05-2
pn  ovmf-ia32               <none>
pn  podman                  <none>
ii  python3-distro-info     1.7
pn  qemu-efi-aarch64        <none>
pn  qemu-efi-arm            <none>
pn  qemu-efi-riscv64        <none>
pn  qemu-system             <none>
ii  qemu-utils              1:9.0.2+ds-2+b1
ii  schroot                 1.6.13-5
ii  util-linux              2.40.2-7
pn  vmdb2                   <none>
pn  zerofree                <none>

-- debconf information:
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
        LANGUAGE = (unset),
        LC_ALL = (unset),
        LC_TIME = "de_DE.UTF-8",
        LC_MONETARY = "de_DE.UTF-8",
        LC_ADDRESS = "de_DE.UTF-8",
        LC_TELEPHONE = "de_DE.UTF-8",
        LC_MESSAGES = "POSIX",
        LC_NAME = "de_DE.UTF-8",
        LC_MEASUREMENT = "de_DE.UTF-8",
        LC_IDENTIFICATION = "de_DE.UTF-8",
        LC_NUMERIC = "de_DE.UTF-8",
        LC_PAPER = "de_DE.UTF-8",
        LANG = "C.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("C.UTF-8").
locale: Cannot set LC_ALL to default locale: No such file or directory

-- debsums errors found:
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
        LANGUAGE = (unset),
        LC_ALL = (unset),
        LC_TIME = "de_DE.UTF-8",
        LC_MONETARY = "de_DE.UTF-8",
        LC_ADDRESS = "de_DE.UTF-8",
        LC_TELEPHONE = "de_DE.UTF-8",
        LC_MESSAGES = "POSIX",
        LC_NAME = "de_DE.UTF-8",
        LC_MEASUREMENT = "de_DE.UTF-8",
        LC_IDENTIFICATION = "de_DE.UTF-8",
        LC_NUMERIC = "de_DE.UTF-8",
        LC_PAPER = "de_DE.UTF-8",
        LANG = "C.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("C.UTF-8").
debsums: changed file /usr/share/autopkgtest/lib/autopkgtest_qemu.py (from 
autopkgtest package)

--- End Message ---
--- Begin Message ---
Source: autopkgtest
Source-Version: 5.40
Done: Paride Legovini <[email protected]>

We believe that the bug you reported is fixed in the latest version of
autopkgtest, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Paride Legovini <[email protected]> (supplier of updated autopkgtest package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Mon, 21 Oct 2024 12:09:29 +0200
Source: autopkgtest
Architecture: source
Version: 5.40
Distribution: unstable
Urgency: medium
Maintainer: Debian CI team <[email protected]>
Changed-By: Paride Legovini <[email protected]>
Closes: 1011204 1078760 1079216 1080981 1081374
Changes:
 autopkgtest (5.40) unstable; urgency=medium
 .
   [ Johannes Schauer Marin Rodrigues ]
   * lib/adt_binaries.py: run 'apt update' before 'apt install'
 .
   [ Simon McVittie ]
   * build-qemu(1): Separate recommended and deprecated invocation
   * build-lxc, build-lxd: Convert command-line parsing to Python
   * build-lxc: Check presence of lxc-templates
   * build-docker: Label Docker/Podman images with some of their properties
   * virt-podman: Try to enable --init automatically if it would work
   * build-docker: Tag images for default architecture with/without prefix
 .
   [ Paride Legovini ]
   * Implement support for adding PPAs via --add-apt-source=ppa:OWNER/PPA
   * Refactor and improve "get priority from apt policy" logic
   * Rework and simplify the package pinning logic
     - man: clarify documentation on how pinning works
   * Refactor handling of synthesized dependencies
   * Get the testbed release from the APT data sources
   * Always get the testbed release when setting up the testbed
   * Trust find_source_version_to_download even in presence of src: pins
   * Refactor how APT releases and pockets are added
     - Handle sources.list [ options ] when doing --add-apt-release
     - Update manpage after refactor of APT source addition
   * Refactor _run_apt_install() and rename to _run_apt_command()
     - Handle --apt-upgrade via _run_apt_command() (Closes: #1011204)
     - Update testbed package index (apt-get update) using _run_apt_command()
   * Revert "lxc: define /tmp mount via lxc-start"
   * build_source: do not generate debug symbol packages
   * build_source: do not build udeb packages
     - tests: test that nocheck, noudeb profiles are set at build time
   * tests: check if CAP_SYS_ADMIN is EFFECTIVE (Closes: #1079216)
   * tests: Archive: support specifying the release Codename
   * tests: Archive: add support for accessing dists via codename
   * tests: Archive: rename series to suite, require more explicit config
   * tests: test_apt_pocket_pkg_with_proposed_dep: use expect_success=True
   * tests: expand the pocket tests to cover --add-apt-release --pin-packages
   * tests: run the apt_pocket tests with NotAutomatic: yes in -proposed
   * tests: add test_debian_layout_* to test pinning on a Debian-like layout
   * tests: add new test: test_offline_success
   * tests: ChrootRunner: get testbed_apt_version from the chroot
   * tests: mockup-commands: bump version of mock apt command
   * d/control: move python3-distro-info from Suggests to Depends
   * pre-commit: add rst-* hooks from pygrep-hooks
   * CI: add Salsa CI pipeline recipe for Ubuntu
 .
   [ Stefano Rivera ]
   * incus/lxd: Set a default value for PATH
 .
   [ Guilhem Moulin ]
   * build-qemu: Turn off ‘orphan_file’ for older target suites
     (Closes: #1078760)
 .
   [ Lee Garrett ]
   * Increase msize for 9p mount to 512KB (~30% read performance increase)
 .
   [ Ural Tunaboyu ]
   * t/autopkgtest-buildvm-ubuntu-cloud: add check for required space
   * t/autopkgtest-buildvm-ubuntu-cloud: fix image url for some arches
     (LP: #2077552)
   * t/autopkgtest-buildvm-ubuntu-cloud: add image cache (LP: #2073938)
   * lib/autopkgtest_deps: add dependency for package
   * lib/autopkgtest_deps: add dependency for directory
   * t/autopkgtest-buildvm-ubuntu-cloud: add directory dependency on output_dir
   * tests/autopkgtest: make test_apt_source_arch_conflict pass on non-amd64
   * debian/tests: enable lxd tests for all arches for which there are images
 .
   [ Blair Noctis ]
   * lib/testdec: use lists in dep set manipulation (Closes: #1080981)
   * tests/testdesc: test recommends from multiple packages
 .
   [ Kai Oberbeckmann ]
   * qemu: Fix boot of UEFI-only virtual machines (Closes: #1081374)
 .
   [ Christian Kastner ]
   * reboot-required: Also monitor /boot/{grub,efi}
 .
   [ Paul Gevers ]
   * VirSubproc.py: prevent UnboundLocalError on timeout
 .
   [ Antonio Terceiro ]
   * autopkgtest-build-lxc: avoid lxc-attach messing with file permissions
Checksums-Sha1:
 89560c7287dc44a1cd2f05552cbbfdc64f46ff10 2152 autopkgtest_5.40.dsc
 167056aa229ef0fdab61a0570206dfee959897ca 237932 autopkgtest_5.40.tar.xz
 185fe2369dfd29a757e4cd714bf3003f98093866 7714 autopkgtest_5.40_source.buildinfo
Checksums-Sha256:
 327e4b30062d130270a1e7db29e552f258b8a1f9e25cebfee489480ac0c48482 2152 
autopkgtest_5.40.dsc
 988ce6164d4f62a206f977d2603f4c3cd68d34dda1aa09364752218d26854f98 237932 
autopkgtest_5.40.tar.xz
 0f9f1c2748f1fd82e3a498bdf35cc07883a9a62da5bbd5cbf17571bf7d1abcaa 7714 
autopkgtest_5.40_source.buildinfo
Files:
 e447fb8fd0144cc3c9b0aab599395043 2152 devel optional autopkgtest_5.40.dsc
 2497a3668e896fd50c6a7e05b5aecfd2 237932 devel optional autopkgtest_5.40.tar.xz
 9e580a53a298b53fbd0a16fde53c7700 7714 devel optional 
autopkgtest_5.40_source.buildinfo

-----BEGIN PGP SIGNATURE-----

iQFGBAEBCgAwFiEEVhrVhe7XZpIbqN2W1lhhiD4BTbkFAmcWRAESHHBhcmlkZUBk
ZWJpYW4ub3JnAAoJENZYYYg+AU25/AIIAKwZmxLt15lfFtzliOJKgNzRkdgoXvVV
5u14+DOh+SBf2zwwfNckHXkKHUqEHjXtjfBXu6IYD6FQdJSv+jgwsx9dKa6WEgmb
HZPlNll/4LimdBFA/NpQ/3ETYMDy137hzT1drrKwdW6u/ZOQanCw/Bjk/YY+sUA3
DZhpppD+ssU9QziYTffJKWnqGcTscHVAWuL+KUTkzf2kQQGxALscnezTHPieUHfQ
tciEhg9kYROKEceST9cjUcdx2U3Tti6vcHhvqcp0IxzGJoNwao3c23DyE07qKXJ0
7w4sbSrY1UjQiKxIpliFsqAvXLpuOfz2rNIDEbS+K6aDxAA5UXuXe7g=
=QfSO
-----END PGP SIGNATURE-----

Attachment: pgpPBqNYikoYm.pgp
Description: PGP signature


--- End Message ---

Reply via email to