[pve-devel] applied: [pve-common 1/3] INotify: use cidr for address on config change

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Mar 4 15:14:01 CET 2020


On 2/20/20 12:55 PM, Alexandre Derumier wrote:
> netmask/broadcast options are deprecated since buster
> https://manpages.debian.org/buster/ifupdown/interfaces.5.en.html
> 
> This still read both "old address/netmask" or "address cidr" format,
> but always convert it to cidr address format
> 
> Signed-off-by: Alexandre Derumier <aderumier at odiso.com>
> ---
>  src/PVE/INotify.pm                            | 24 ++++++-------------
>  test/etc_network_interfaces/t.bridge-v4-v6.pl | 11 ++-------
>  .../t.create_network.pl                       | 12 +++-------
>  .../t.list-interfaces.pl                      | 21 +++++++---------
>  .../t.ovs_bridge_allow.pl                     |  7 +-----
>  .../t.parsed_options.pl                       |  3 +--
>  .../etc_network_interfaces/t.unknown_order.pl | 15 ++++--------
>  .../t.update_network.pl                       | 16 ++++---------
>  8 files changed, 31 insertions(+), 78 deletions(-)
> 

applied, with a few cleanups + adding missing semicolon, patch 3/3 will
merge conflict now. Thanks!





More information about the pve-devel mailing list