commit 0164c1c9f2c3faeb989a1bc9d04304db274240ed
author Yuta HIGUCHI <y-higuchi@ak.jp.nec.com> Thu May 04 15:43:55 2017 -0700
committer Yuta HIGUCHI <y-higuchi@onlab.us> Fri May 05 20:54:57 2017 +0000
tree be9a1cf760880c04862586e8421673134180bc4f
parent 8e258116dcc93121739c764122090bcfd3c07f94

Log some context when Intent failed.

Change-Id: Ib30f78f970680144f920ca4754825f78eedb658b

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

diff --git a/core/net/src/main/java/org/onosproject/net/intent/impl/phase/WithdrawRequest.java b/core/net/src/main/java/org/onosproject/net/intent/impl/phase/WithdrawRequest.java
index 4062fc62..db7509d 100644
--- a/core/net/src/main/java/org/onosproject/net/intent/impl/phase/WithdrawRequest.java
+++ b/core/net/src/main/java/org/onosproject/net/intent/impl/phase/WithdrawRequest.java
@@ -17,6 +17,8 @@
 
 import org.onosproject.net.intent.IntentData;
 import org.onosproject.net.intent.impl.IntentProcessor;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.util.Optional;
 
@@ -28,6 +30,8 @@
  */
 final class WithdrawRequest implements IntentProcessPhase {
 
+    private static final Logger log = LoggerFactory.getLogger(WithdrawRequest.class);
+
     private final IntentProcessor processor;
     private final IntentData data;
     private final Optional<IntentData> stored;
@@ -57,6 +61,8 @@
         if (!stored.isPresent() || stored.get().installables().isEmpty()) {
             switch (data.request()) {
                 case INSTALL_REQ:
+                    // illegal state?
+                    log.warn("{} was requested to withdraw during installation?", data.intent());
                     return Optional.of(new Failed(data));
                 case WITHDRAW_REQ:
                 default: //TODO "default" case should not happen