Unblock netvirt CSIT issues 56/91356/1
authorKarthikeyan Krishnan <karthikeyangceb007@gmail.com>
Fri, 17 Jul 2020 07:59:37 +0000 (13:29 +0530)
committerKarthikeyan Krishnan <karthikeyangceb007@gmail.com>
Fri, 17 Jul 2020 07:59:37 +0000 (13:29 +0530)
Issue:
======
New exceptions found: 3
[{'warnerr_list': [6, 38], 'lines': ['2020-07-17T05:22:39,604 | ERROR |
NeutronFloatingToFixedIpMappingChangeListener-0 |
NeutronFloatingToFixedIpMappingChangeListener | 359 -
org.opendaylight.netvirt.neutronvpn-impl - 0.11.0.SNAPSHOT |
addToFloatingIpInfo failed for floating IP: 10.10.10.187 ',
'java.lang.NullPointerException: null', '\tat
org.opendaylight.netvirt.neutronvpn.NeutronFloatingToFixedIpMappingChangeListener
.addToFloatingIpInfo(NeutronFloatingToFixedIpMappingChangeListener.java:167)
[bundleFile:?]', '\tat
org.opendaylight.netvirt.neutronvpn.NeutronFloatingToFixedIpMappingChangeListener
.update(NeutronFloatingToFixedIpMappingChangeListener.java:121)
[bundleFile:?]', '\tat

https://logs.opendaylight.org/sandbox/vex-yul-odl-jenkins-2/srini-netvirt-csit-1node-0cmb-1ctl-2cmp-openstack-rocky-gate-stateful-aluminium/16/robot-plugin/log_full.html.gz

Solution:
=========
Provided null check in appropriate place to avoid NPE

Signed-off-by: Karthikeyan Krishnan <karthikeyangceb007@gmail.com>
Change-Id: I812463516f7fe773e47fcd80d169880e5519a1fa

neutronvpn/impl/src/main/java/org/opendaylight/netvirt/neutronvpn/NeutronFloatingToFixedIpMappingChangeListener.java

index 7b6046c04139d928016a546779331b9b142d4c06..29f56dcd56858abc02167e54bb15075e39301d8b 100644 (file)
@@ -164,7 +164,7 @@ public class NeutronFloatingToFixedIpMappingChangeListener extends AbstractAsync
                 if (fixedIpAddress != null) {
                     List<InternalToExternalPortMap> intExtPortMapList
                             = new ArrayList<InternalToExternalPortMap>(fixedNeutronPortBuilder
-                            .getInternalToExternalPortMap().size() != 0 ? fixedNeutronPortBuilder
+                            .getInternalToExternalPortMap() != null ? fixedNeutronPortBuilder
                             .getInternalToExternalPortMap().values() : null);
                     if (intExtPortMapList == null) {
                         intExtPortMapList = new ArrayList<>();