On 18/04/18 08:09, D Arunprakash wrote: > Hello, > > The following review in mdsal might have impacted openflowplugin > functionality. > > https://git.opendaylight.org/gerrit/#/c/70769/
Sorry about that. > https://jenkins.opendaylight.org/releng/job/openflowplugin-maven-verify-fluorine-mvn33-openjdk8/259/console > > > > org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.Ipv4Address<Ipv4Address{_value=0.1.2.3}> > but was: > org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.Ipv4AddressNoZone<Ipv4Address{_value=0.1.2.3}> > > is it new expectation on openflowplugin to change from Ipv4Address to > Ipv4AddressNoZone ? There are two aspects to this. As an interim, the codecs from wire need to be updated to convert Ipv4AddressNoZone to Ipv4Address, so that equality works as expected and the breakage is recovered -- https://git.opendaylight.org/gerrit/71072 does that. Going forward, though, I believe the openflow models need to be updated to require ipv4-address-no-zone rather than ipv4-address (and same goes for ip-address and ipv6-address). This really is the correct thing to do -- ipv4-address is not really the IPv4 address used in OpenFlow protocol. Regards, Robert
signature.asc
Description: OpenPGP digital signature
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
