From: Yumei Huang <yuhuang@redhat.com>
To: Stefano Brivio <sbrivio@redhat.com>
Cc: David Gibson <david@gibson.dropbear.id.au>,
passt-dev@passt.top, dgibson@redhat.com
Subject: Re: [PATCH] test: Remove unused distro images
Date: Mon, 8 Sep 2025 19:39:25 +0800 [thread overview]
Message-ID: <CANsz47=th1a0=iq=L4Ptiaxnc_M1F1eAroV=dPVL8zRTQCinkA@mail.gmail.com> (raw)
In-Reply-To: <20250908112328.1be76e36@elisabeth>
Just sent a new patch to fix the broken link. Tested on my test
machine, and all images got downloaded successfully.
Please help review, thanks!
On Mon, Sep 8, 2025 at 5:23 PM Stefano Brivio <sbrivio@redhat.com> wrote:
>
> On Mon, 8 Sep 2025 18:19:51 +1000
> David Gibson <david@gibson.dropbear.id.au> wrote:
>
> > On Mon, Sep 08, 2025 at 10:02:19AM +0200, Stefano Brivio wrote:
> > > On Mon, 8 Sep 2025 17:09:57 +1000
> > > David Gibson <david@gibson.dropbear.id.au> wrote:
> > >
> > > > On Mon, Sep 08, 2025 at 02:09:21PM +0800, Yumei Huang wrote:
> > > > > Some of the download links are broken, and none of the images are
> > > > > used in current tests. Removing them so it won't block make.
> > > > >
> > > > > Signed-off-by: Yumei Huang <yuhuang@redhat.com>
> > > >
> > > > Ah, sorry. I realised I forgot some information when I suggested you
> > > > make this change.
> > > >
> > > > Although we're not using these images at the moment, we hope to use
> > > > them again some time in future. For that reason it's useful to keep
> > > > the information about what the images were and how they worked in
> > > > place.
> > > >
> > > > So, rather than removing this completely, it would be better to keep
> > > > the {DEBIAN,...}_IMGS variables and the download targets, but just
> > > > don't include them in DOWNLOAD_ASSETS for now. That will make it
> > > > easier to put a fixed version of this back when we're ready.
> > >
> > > My suggestion from a couple of days ago:
> > >
> > > https://archives.passt.top/passt-dev/20250820134427.6a16ad19@elisabeth/
> > >
> > > was rather that we update the links. Not many of them are broken, and
> > > it would be valuable to have test/distro working again. Is that
> > > unfeasible for some other reason?
> >
> > Not in principle, I think. It is, however, pretty miserable for me to
> > test: even before the links broke it was achingly slow for me to
> > download all the images in Australia. Don't know what it's like in
> > China.
>
> I guess it's similar: if you have mirrors nearby, then it's fast,
> otherwise it's slow.
>
> I actually ran all this (without your changes to cache things, so
> downloading every time) on a machine in Beijing a few years ago, and
> most downloads were very fast (all the Debian ones at least), some
> other ones (some of the old Fedora ones, I think) pretty slow.
>
> > This was just the fastest way to stop this being a blocker for
> > anyone new to the project.
>
> Sure, that's the most important thing. We can proceed in two ways, I
> guess:
>
> 1. we go ahead and apply this and make it my priority to fix the rest
> myself (old Fedora downloads are slow for me as well, but I don't
> care that much)
>
> 2. Yumei gives it a try with fixing download links, if it's doable from
> China, and after that (probably) broken tests. If it's not doable, 1.
> is absolutely fine!
>
> ...let me know if 2. might make sense.
>
> --
> Stefano
>
--
Thanks,
Yumei Huang
prev parent reply other threads:[~2025-09-08 11:39 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-08 6:09 [PATCH] test: Remove unused distro images Yumei Huang
2025-09-08 7:09 ` David Gibson
2025-09-08 7:56 ` Yumei Huang
2025-09-08 8:02 ` Stefano Brivio
2025-09-08 8:19 ` David Gibson
2025-09-08 9:23 ` Stefano Brivio
2025-09-08 11:39 ` Yumei Huang [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CANsz47=th1a0=iq=L4Ptiaxnc_M1F1eAroV=dPVL8zRTQCinkA@mail.gmail.com' \
--to=yuhuang@redhat.com \
--cc=david@gibson.dropbear.id.au \
--cc=dgibson@redhat.com \
--cc=passt-dev@passt.top \
--cc=sbrivio@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://passt.top/passt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).