Bug 8223: Fixed incorrect enable-flow-removed-notification check. 44/55044/1
authorShigeru Yasuda <s-yasuda@da.jp.nec.com>
Thu, 13 Apr 2017 11:12:19 +0000 (20:12 +0900)
committerShigeru Yasuda <s-yasuda@da.jp.nec.com>
Fri, 14 Apr 2017 01:58:09 +0000 (01:58 +0000)
flow-removed notification needs to be published if
DeviceManager.isFlowRemovedNotificationOn() returns true.

Change-Id: I10c025f11e14670303b70bf31d3a9cefa5a676ec
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 644f1bbd99d4fd949800fb3167d11d4ce968c950)

openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/DeviceContextImpl.java

index f5faf5b8df2623b7976b269878294b313cfb2e7b..60e7f1a959ad9ef1d82ff610bf5bc7503b15433e 100644 (file)
@@ -331,7 +331,7 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi
         final org.opendaylight.yang.gen.v1.urn.opendaylight.flow.service.rev130819.FlowRemoved flowRemovedNotification =
                 flowRemovedTranslator.translate(flowRemoved, deviceInfo, null);
 
-        if(!myManager.isFlowRemovedNotificationOn()) {
+        if(myManager.isFlowRemovedNotificationOn()) {
             // Trigger off a notification
             notificationPublishService.offerNotification(flowRemovedNotification);
         } else if(LOG.isDebugEnabled()) {