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.

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

Reply via email to