Hi Aleksander,
> Yes. This fix still needed. Please take a look at the test below:
>
> > import ipaddress
> > ipaddress.IPv4Network((u'192.168.0.222', 24), False)
> IPv4Network(u'192.168.0.0/24')
>
> The IPv4Network method should not be used here, because in functions like an
> ip_address_dump or ip_route_dump we expect to receive the host address with
> network prefix, instead of it's network address.
Right, those should be separated by the following commit: 75761b93
E.g. ip_address_details uses vl_api_address_with_prefix_t , which maps in
Python to the IPv6Interface class.
ip_route_details is a 'pure' prefix, with the anything after prefix length
zero'ed out. Or at least it should be.
Do you have the above patch?
Otherwise I will not discount that there are issues here. ;-)
Cheers,
Ole
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#15142): https://lists.fd.io/g/vpp-dev/message/15142
Mute This Topic: https://lists.fd.io/mt/69665543/21656
Mute #vpp: https://lists.fd.io/mk?hashtag=vpp&subid=1480452
Group Owner: vpp-dev+ow...@lists.fd.io
Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
