multiple ipv4_addresses not working

[mvogt1]

When specifing two IP addresses the documentation in connection.pp states:

$ipv4_address = semicolon separated list of the IPv4 addresses to assign to the interface in format 127.0.0.1/8

  networkmanager::ifc::connection {
     'ens192':
        ensure                  => present,
        interface_name          => "ens192",
        ipv4_method             => 'auto',
        ipv4_address            => "10.136.16.57/20;10.136.16.234/20",
        ipv6_method             => 'disabled',
  }

the file is written:

# THIS FILE IS CONTROLLED BY PUPPET

[connection]
interface-name=ens192
id=ens192
uuid=63c11cbc-f5a4-5a84-b217-11368fde0f1f
type=ethernet

[ipv4]
method=auto
may-fail=true
address=10.136.16.57/20;10.136.16.234/20

[ipv6]
method=disabled
addr-gen-mode=0
ip6-privacy=0
may-fail=true

But networkmanager does not active the IPs:

2: ens192: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq state UP group default qlen 1000
    link/ether 00:50:56:bd:09:e6 brd ff:ff:ff:ff:ff:ff
    altname enp11s0
    inet 10.136.16.97/20 brd 10.136.31.255 scope global dynamic noprefixroute ens192
       valid_lft 42183sec preferred_lft 42183sec

Changing the [ipv4] section in the file ens192.nmconnection to

[ipv4]
method=auto
may-fail=true
address1=10.136.16.57/20
address2=10.136.16.234/20

reload+up:

2: ens192: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq state UP group default qlen 1000
    link/ether 00:50:56:bd:09:e6 brd ff:ff:ff:ff:ff:ff
    altname enp11s0
    inet 10.136.16.57/20 brd 10.136.31.255 scope global noprefixroute ens192
       valid_lft forever preferred_lft forever
    inet 10.136.16.234/20 brd 10.136.31.255 scope global secondary noprefixroute ens192
       valid_lft forever preferred_lft forever
    inet 10.136.16.97/20 brd 10.136.31.255 scope global secondary dynamic noprefixroute ens192
       valid_lft 43198sec preferred_lft 43198sec

Thus it looks that the semicolon separated format cannot be handled by NM.

[mvogt1]

The attached patch addresses the problem.

patch_issue33.txt