adding some logging and checking Intent type more carefully
Checking for switch == null when install flow entries
(effect is we only install fe's on switches for which we are master)
Change-Id: I1645caf68ec6082737157737f3e12752a7515f5d
diff --git a/src/main/java/net/onrc/onos/intent/runtime/PlanInstallModule.java b/src/main/java/net/onrc/onos/intent/runtime/PlanInstallModule.java
index adef594..4d4e437 100644
--- a/src/main/java/net/onrc/onos/intent/runtime/PlanInstallModule.java
+++ b/src/main/java/net/onrc/onos/intent/runtime/PlanInstallModule.java
@@ -67,7 +67,8 @@
while(true) {
try {
IntentOperationList intents = intentQueue.take();
- //TODO: drain the remaining intent lists
+ //TODO: consider draining the remaining intent lists
+ // and processing in one big batch
processIntents(intents);
} catch (InterruptedException e) {
log.warn("Error taking from intent queue: {}", e.getMessage());
@@ -162,13 +163,13 @@
@Override
public Collection<Class<? extends IFloodlightService>> getModuleServices() {
- // TODO Auto-generated method stub
+ // no services, for now
return null;
}
@Override
public Map<Class<? extends IFloodlightService>, IFloodlightService> getServiceImpls() {
- // TODO Auto-generated method stub
+ // no services, for now
return null;
}