tests/functional: fix URLs in PCI hotplug test for aarch64

[PATCH] tests/functional: fix URLs in PCI hotplug test for aarch64

Posted by Stefan Hajnoczi 3 months ago

Debian trixie has been released. The "stable" alias no longer refers to
the Debian bookworm release, so URLs referring to bookworm artifacts via
the "stable" alias no longer work.

Switch to explicit release naming ("bookworm") to make a permalink so
the test passes again.

Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Fixes: 374a245573b8 ("tests/functional: Add PCI hotplug test for aarch64")
Resolves: #3073 ("PCI hotplug test for aarch64 fails due to broken Debian installer URL")
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
 tests/functional/test_aarch64_hotplug_pci.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/functional/test_aarch64_hotplug_pci.py b/tests/functional/test_aarch64_hotplug_pci.py
index c9bb7f1d97..0c67991b89 100755
--- a/tests/functional/test_aarch64_hotplug_pci.py
+++ b/tests/functional/test_aarch64_hotplug_pci.py
@@ -15,12 +15,12 @@
 class HotplugPCI(LinuxKernelTest):
 
     ASSET_KERNEL = Asset(
-        ('https://ftp.debian.org/debian/dists/stable/main/installer-arm64/'
+        ('https://ftp.debian.org/debian/dists/bookworm/main/installer-arm64/'
          '20230607+deb12u11/images/netboot/debian-installer/arm64/linux'),
          'd92a60392ce1e379ca198a1a820899f8f0d39a62d047c41ab79492f81541a9d9')
 
     ASSET_INITRD = Asset(
-        ('https://ftp.debian.org/debian/dists/stable/main/installer-arm64/'
+        ('https://ftp.debian.org/debian/dists/bookworm/main/installer-arm64/'
          '20230607+deb12u11/images/netboot/debian-installer/arm64/initrd.gz'),
          '9f817f76951f3237bca8216bee35267bfb826815687f4b2fcdd5e6c2a917790c')
 
-- 
2.50.1

Re: [PATCH] tests/functional: fix URLs in PCI hotplug test for aarch64

Posted by Stefan Hajnoczi 3 months ago

On Mon, Aug 11, 2025 at 12:24 PM Stefan Hajnoczi <stefanha@redhat.com> wrote:
>
> Debian trixie has been released. The "stable" alias no longer refers to
> the Debian bookworm release, so URLs referring to bookworm artifacts via
> the "stable" alias no longer work.
>
> Switch to explicit release naming ("bookworm") to make a permalink so
> the test passes again.
>
> Suggested-by: Peter Maydell <peter.maydell@linaro.org>
> Fixes: 374a245573b8 ("tests/functional: Add PCI hotplug test for aarch64")
> Resolves: #3073 ("PCI hotplug test for aarch64 fails due to broken Debian installer URL")
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
>  tests/functional/test_aarch64_hotplug_pci.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to master.

>
> diff --git a/tests/functional/test_aarch64_hotplug_pci.py b/tests/functional/test_aarch64_hotplug_pci.py
> index c9bb7f1d97..0c67991b89 100755
> --- a/tests/functional/test_aarch64_hotplug_pci.py
> +++ b/tests/functional/test_aarch64_hotplug_pci.py
> @@ -15,12 +15,12 @@
>  class HotplugPCI(LinuxKernelTest):
>
>      ASSET_KERNEL = Asset(
> -        ('https://ftp.debian.org/debian/dists/stable/main/installer-arm64/'
> +        ('https://ftp.debian.org/debian/dists/bookworm/main/installer-arm64/'
>           '20230607+deb12u11/images/netboot/debian-installer/arm64/linux'),
>           'd92a60392ce1e379ca198a1a820899f8f0d39a62d047c41ab79492f81541a9d9')
>
>      ASSET_INITRD = Asset(
> -        ('https://ftp.debian.org/debian/dists/stable/main/installer-arm64/'
> +        ('https://ftp.debian.org/debian/dists/bookworm/main/installer-arm64/'
>           '20230607+deb12u11/images/netboot/debian-installer/arm64/initrd.gz'),
>           '9f817f76951f3237bca8216bee35267bfb826815687f4b2fcdd5e6c2a917790c')
>
> --
> 2.50.1
>
>

Re: [PATCH] tests/functional: fix URLs in PCI hotplug test for aarch64

Posted by Alex Bennée 3 months ago

Stefan Hajnoczi <stefanha@redhat.com> writes:

> Debian trixie has been released. The "stable" alias no longer refers to
> the Debian bookworm release, so URLs referring to bookworm artifacts via
> the "stable" alias no longer work.
>
> Switch to explicit release naming ("bookworm") to make a permalink so
> the test passes again.
>
> Suggested-by: Peter Maydell <peter.maydell@linaro.org>
> Fixes: 374a245573b8 ("tests/functional: Add PCI hotplug test for aarch64")
> Resolves: #3073 ("PCI hotplug test for aarch64 fails due to broken Debian installer URL")
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro

Re: [PATCH] tests/functional: fix URLs in PCI hotplug test for aarch64

Posted by Philippe Mathieu-Daudé 3 months ago

On 11/8/25 18:23, Stefan Hajnoczi wrote:
> Debian trixie has been released. The "stable" alias no longer refers to
> the Debian bookworm release, so URLs referring to bookworm artifacts via
> the "stable" alias no longer work.
> 
> Switch to explicit release naming ("bookworm") to make a permalink so
> the test passes again.
> 
> Suggested-by: Peter Maydell <peter.maydell@linaro.org>
> Fixes: 374a245573b8 ("tests/functional: Add PCI hotplug test for aarch64")
> Resolves: #3073 ("PCI hotplug test for aarch64 fails due to broken Debian installer URL")
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
>   tests/functional/test_aarch64_hotplug_pci.py | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH] tests/functional: fix URLs in PCI hotplug test for aarch64

Posted by Peter Maydell 3 months ago

On Mon, 11 Aug 2025 at 17:23, Stefan Hajnoczi <stefanha@redhat.com> wrote:
>
> Debian trixie has been released. The "stable" alias no longer refers to
> the Debian bookworm release, so URLs referring to bookworm artifacts via
> the "stable" alias no longer work.
>
> Switch to explicit release naming ("bookworm") to make a permalink so
> the test passes again.
>
> Suggested-by: Peter Maydell <peter.maydell@linaro.org>
> Fixes: 374a245573b8 ("tests/functional: Add PCI hotplug test for aarch64")
> Resolves: #3073 ("PCI hotplug test for aarch64 fails due to broken Debian installer URL")
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
>  tests/functional/test_aarch64_hotplug_pci.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>


Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

thanks
-- PMM