[pve-devel] [PATCH installer v2] install: don't select a debian mirror based on the country anymore

[Fiona Ebner]

Am 31.07.25 um 11:15 AM schrieb Shannon Sterz:
> newer isos ship with new deb822 style repository configurations in
> `/etc/apt/sources.list.d/debian.sources` which already use the now
> prefered cdn [1]. since setting a mirror like this does not work
> anymore due to this change, just remove it to keep using the prefered
> cdn.
> 
> [1]:
> https://www.debian.org/releases/trixie/release-notes/upgrading.en.html#adding-apt-internet-sources
> 
> Signed-off-by: Shannon Sterz <s.sterz at proxmox.com>

Reviewed-by: Fiona Ebner <f.ebner at proxmox.com>

We could also go ahead and remove adding the mirror information to the
country hash in country.pl after this change.

html/country.htm also should be updated:

The Proxmox Installer automatically makes location-based optimizations,
like choosing the nearest mirror to download files from.

since it's not the Installer itself doing that anymore.

> ---
> 
> changes since v1:
> 
> - instead of adapting how we set mirrors to the new deb822 format,
>   simply remove this code altogether in favor of using the cdn.
> 
>  Proxmox/Install.pm | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/Proxmox/Install.pm b/Proxmox/Install.pm
> index 3159dcc..5d77821 100644
> --- a/Proxmox/Install.pm
> +++ b/Proxmox/Install.pm
> @@ -1316,12 +1316,6 @@ _EOD
>          symlink("/usr/share/zoneinfo/$timezone", "$targetdir/etc/localtime");
>          file_write_all("$targetdir/etc/timezone", "$timezone\n");
> 
> -        # set apt mirror
> -        if (my $mirror = $iso_env->{locales}->{country}->{$country}->{mirror}) {
> -            my $fn = "$targetdir/etc/apt/sources.list";
> -            syscmd("sed -i 's/ftp\\.debian\\.org/$mirror/' '$fn'");
> -        }
> -
>          # create extended_states for apt (avoid cron job warning if that
>          # file does not exist)
>          file_write_all("$targetdir/var/lib/apt/extended_states", '');
> --
> 2.47.2