[pve-devel] [PATCH ifupdown2 2/2] add openvswitch addon

[Thomas Lamprecht]

On 2/18/20 7:08 AM, Alexandre DERUMIER wrote:
>>> Oh, and yes, if this breaks the old way of specifying ovs interfaces some 
>>> compatibillity would be great, e.g., with your mentioned postinst script 
> 
> What do you think about this simple postinst script ?
> 
> "
> use PVE::INotify;
> my $config = PVE::INotify::read_file('interfaces', 1);
> my $configdata = $config->{data};
> 
> if(defined($config->{changes})) {
>     PVE::INotify::write_file('interfaces', $configdata);
>     print "network config changes has been detected and generated in /etc/network/interfaces.new\n";
>     print "Please reload network configuration in gui\n";
> }
> "
> 
> Like this, user can still review the changes, reload it manually, on get applied on next reboot

good idea! As else we could make not yet applied pending changes look like
they got already applied, so sounds sensible to me.