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

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

[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@proxmox.com>
---

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



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [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@proxmox.com>

Reviewed-by: Fiona Ebner <f.ebner@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


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel