Re: [pve-devel] [PATCH ifupdown2] debian: postinst: ensure /etc/iproute2/rt_tables.d exists

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

Am 08.07.25 um 18:43 schrieb Friedrich Weber:
> When upgrading to Debian Trixie, iproute2 is also upgraded. Its
> postinst deletes /etc/iproute2/rt_tables.d if it is currently empty
> (or only contains a README file) [1]. After that, ifreload -a will
> always print a warning:
> 
>> error: [Errno 2] No such file or directory: '/etc/iproute2/rt_tables.d/ifupdown2_vrf_map.conf'
> 
> because it cannot create the file if the parent directory does not
> exist.
> 
> To avoid that, create the directory in ifupdown2's postinst if it
> does not exist yet.
> 
> Commit b115f7f ("add /etc/iproute2/rt_tables.d/ to managed
> directories") already added the directory to ifupdown2.dirs, but this
> does not seem to be effective, due to the following sequence of events
> on upgrade:
> 
> 1) iproute2 is unpacked
> 2) ifupdown2 is unpacked. The directory still exists, so the
>    ifupdown2.dirs entry does not have an effect
> 3) ifroute2 is set up, and its postinst empties and removes the

s/if/ip/ (can be fixed up on applying).

>    directory
> 4) ifupdown2 is set up
> 
> [1] https://sources.debian.org/src/iproute2/6.15.0-1/debian/iproute2.postinst/
> 
> Signed-off-by: Friedrich Weber <f.weber@proxmox.com>
> ---
> 
> Notes:
>     Please check the reasoning for plausibility, my understanding of dpkg
>     is very basic, this is just what I could infer from running strace on
>     the upgrade.

Seems plausible as unpacking happens before configuring the pacakges.

>     
>     An alternative solution would be to patch ifupdown2 itself to create
>     the directory (if needed) before trying to write to the
>     ifupdown2_vrf_map.conf file.

That might be slightly more robust and should be IMO what upstream should
do already, that said, it's a very specific case and not having to touch
the python code has its advantages, so the patch here would be fine for
me in general.

> 
>  debian/ifupdown2.postinst | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/debian/ifupdown2.postinst b/debian/ifupdown2.postinst
> index f9a719c..13cc987 100644
> --- a/debian/ifupdown2.postinst
> +++ b/debian/ifupdown2.postinst
> @@ -111,6 +111,9 @@ case "$1" in
>          process_udev
>          chmod +x /usr/share/ifupdown2/__main__.py
>          postinst_remove_diverts
> +        if [ ! -d /etc/iproute2/rt_tables.d ]; then
> +            mkdir /etc/iproute2/rt_tables.d || echo "could not create /etc/iproute2/rt_tables.d"
> +        fi
>          if [ -z "$2" ] && [ ! -e /proxmox_install_mode ]; then
>              proxmox_compatibility
>              echo "Reloading network config on first install"



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