commit 80ae97ca7f2bf377a0a67d8599992583db8f0153
author Ray Milkey <ray@onlab.us> Fri Feb 19 19:04:13 2016 -0800
committer Gerrit Code Review <gerrit@onlab.us> Sat Feb 20 22:55:03 2016 +0000
tree 606e71c6cf5e5507153309ab696d0b35fe342ef6
parent 870c56f408032b9c3bb1c656b8e35a6e6249d8a9

Fix a bug in link up/down handling discovered by STC

Change-Id: I8e1257b8ea3592b7695e5a5dded43286f5b29d3b

core/net/src/main/java/org/onosproject/net/intent/impl/phase/Compiling.java

diff --git a/core/net/src/main/java/org/onosproject/net/intent/impl/phase/Compiling.java b/core/net/src/main/java/org/onosproject/net/intent/impl/phase/Compiling.java
index 1f5eb38..6743be9 100644
--- a/core/net/src/main/java/org/onosproject/net/intent/impl/phase/Compiling.java
+++ b/core/net/src/main/java/org/onosproject/net/intent/impl/phase/Compiling.java
@@ -60,7 +60,7 @@
             return Optional.of(new Installing(processor, new IntentData(data, compiled), stored));
         } catch (IntentException e) {
             log.debug("Unable to compile intent {} due to: {}", data.intent(), e);
-            if (stored.filter(x -> x.installables().isEmpty()).isPresent()) {
+            if (stored.filter(x -> !x.installables().isEmpty()).isPresent()) {
                 // removing orphaned flows and deallocating resources
                 return Optional.of(new Withdrawing(processor, new IntentData(data, stored.get().installables())));
             } else {