Handle exception when compiling fails
- Change arguments of the constructor of PathNotFoundException
- Change the catched exception in Compiling.execute()
Change-Id: I3b639ffd585900c2a6dd99aeeb313bf20c6104f4
diff --git a/core/net/src/main/java/org/onosproject/net/intent/impl/Compiling.java b/core/net/src/main/java/org/onosproject/net/intent/impl/Compiling.java
index 11f9a28..0673ee4 100644
--- a/core/net/src/main/java/org/onosproject/net/intent/impl/Compiling.java
+++ b/core/net/src/main/java/org/onosproject/net/intent/impl/Compiling.java
@@ -47,14 +47,9 @@
List<Intent> installables = (current != null) ? current.installables() : null;
pending.setInstallables(intentManager.compileIntent(pending.intent(), installables));
return Optional.of(new InstallCoordinating(intentManager, pending, current));
- } catch (PathNotFoundException e) {
- log.debug("Path not found for intent {}", pending.intent());
- // TODO: revisit to implement failure handling
- return Optional.of(new CompilingFailed(pending)); //FIXME failed state transition
} catch (IntentException e) {
- log.warn("Unable to compile intent {} due to:", pending.intent().id(), e);
- // TODO: revisit to implement failure handling
- return Optional.of(new CompilingFailed(pending)); //FIXME failed state transition
+ log.debug("Unable to compile intent {} due to: {}", pending.intent(), e);
+ return Optional.of(new CompilingFailed(pending));
}
}
}