Fixing IntentPushTestCommand

Change-Id: Ia6cc2f4be2e8087a903e660067626c5225f20216

Fixing IntentPushTestCommand
Change-Id: Ia6cc2f4be2e8087a903e660067626c5225f20216
Brian O'Connor
Commit 5b9dfdc03804003282903084a86d3f826fb718d3 5b9dfdc0 1 parent bb4d827c
Showing 1 changed file with 35 additions and 70 deletions
cli/src/main/java/org/onosproject/cli/net/IntentPushTestCommand.java
--- a/cli/src/main/java/org/onosproject/cli/net/IntentPushTestCommand.java
View file @5b9dfdc
+++ b/cli/src/main/java/org/onosproject/cli/net/IntentPushTestCommand.java
View file @5b9dfdc
@@ -15,14 +15,13 @@
  */
 package org.onosproject.cli.net;
-import com.google.common.collect.ArrayListMultimap;
+import com.google.common.collect.Lists;
-
 import org.apache.karaf.shell.commands.Argument;
 import org.apache.karaf.shell.commands.Command;
 import org.apache.karaf.shell.commands.Option;
+import org.onlab.packet.Ethernet;
+import org.onlab.packet.MacAddress;
 import org.onosproject.cli.AbstractShellCommand;
-import org.onosproject.core.ApplicationId;
-import org.onosproject.core.CoreService;
 import org.onosproject.net.ConnectPoint;
 import org.onosproject.net.DeviceId;
 import org.onosproject.net.PortNumber;
@@ -35,13 +34,12 @@ import org.onosproject.net.intent.IntentEvent;
 import org.onosproject.net.intent.IntentEvent.Type;
 import org.onosproject.net.intent.IntentListener;
 import org.onosproject.net.intent.IntentService;
+import org.onosproject.net.intent.Key;
 import org.onosproject.net.intent.PointToPointIntent;
-import org.onlab.packet.Ethernet;
-import org.onlab.packet.MacAddress;
+import java.util.Collections;
 import java.util.EnumSet;
-import java.util.HashSet;
+import java.util.List;
-import java.util.Set;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
@@ -66,21 +64,15 @@ public class IntentPushTestCommand extends AbstractShellCommand
               required = true, multiValued = false)
     String egressDeviceString = null;
-
+    @Argument(index = 2, name = "numberOfIntents",
-    @Argument(index = 2, name = "IntentsPerAppId",
+            description = "Number of intents to install/withdraw",
-            description = "Number of intents per appId",
             required = true, multiValued = false)
-    String intentsPerAppId = null;
+    String numberOfIntents = null;
-
-    @Argument(index = 3, name = "apps",
-            description = "Number of appIds",
-            required = false, multiValued = false)
-    String appIds = null;
-    @Argument(index = 4, name = "appIdBase",
+    @Argument(index = 3, name = "keyOffset",
-            description = "Base Value for Application IDs",
+            description = "Starting point for first key (default: 1)",
             required = false, multiValued = false)
-    String appIdBaseStr = null;
+    String keyOffsetStr = null;
     @Option(name = "-i", aliases = "--install",
             description = "Install intents",
@@ -95,12 +87,9 @@ public class IntentPushTestCommand extends AbstractShellCommand
     private IntentService service;
     private CountDownLatch latch;
     private volatile long start, end;
-    private int apps;
-    private int intentsPerApp;
-    private int appIdBase;
     private int count;
+    private int keyOffset;
     private boolean add;
-    private final Set<ApplicationId> myAppIds = new HashSet<>();
     @Override
     protected void execute() {
@@ -115,64 +104,55 @@ public class IntentPushTestCommand extends AbstractShellCommand
         PortNumber egressPortNumber = portNumber(getPortNumber(egressDeviceString));
         ConnectPoint egress = new ConnectPoint(egressDeviceId, egressPortNumber);
-        apps = appIds != null ? Integer.parseInt(appIds) : 1;
+        count = Integer.parseInt(numberOfIntents);
-        intentsPerApp = Integer.parseInt(intentsPerAppId);
+        keyOffset = (keyOffsetStr != null) ? Integer.parseInt(keyOffsetStr) : 1;
-        appIdBase = appIdBaseStr != null ? Integer.parseInt(appIdBaseStr) : 1;
-
-        count = intentsPerApp * apps;
         service.addListener(this);
-        ArrayListMultimap<Integer, Intent> operations = generateIntents(ingress, egress);
+        List<Intent> operations = generateIntents(ingress, egress);
         boolean both = !(installOnly ^ withdrawOnly);
         if (installOnly || both) {
             add = true;
-            latch = new CountDownLatch(count);
             submitIntents(operations);
         }
         if (withdrawOnly || both) {
-            if (withdrawOnly && !both) {
-                print("This should fail for now...");
-            }
             add = false;
-            latch = new CountDownLatch(count);
             submitIntents(operations);
         }
         service.removeListener(this);
     }
-    private ArrayListMultimap<Integer, Intent> generateIntents(ConnectPoint ingress, ConnectPoint egress) {
+    private List<Intent> generateIntents(ConnectPoint ingress, ConnectPoint egress) {
-        TrafficSelector.Builder selector = DefaultTrafficSelector.builder()
+        TrafficSelector.Builder selectorBldr = DefaultTrafficSelector.builder()
                 .matchEthType(Ethernet.TYPE_IPV4);
         TrafficTreatment treatment = DefaultTrafficTreatment.builder().build();
-        ArrayListMultimap<Integer, Intent> intents = ArrayListMultimap.create();
+        List<Intent> intents = Lists.newArrayList();
-        for (int app = 0; app < apps; app++) {
+        for (int i = 0; i < count; i++) {
-            for (int i = 1; i <= intentsPerApp; i++) {
+            TrafficSelector selector = selectorBldr
-                TrafficSelector s = selector
+                    .matchEthSrc(MacAddress.valueOf(i + keyOffset))
-                        .matchEthSrc(MacAddress.valueOf(i + (app + appIdBase) * intentsPerApp))
+                    .build();
-                        .build();
+            intents.add(new PointToPointIntent(appId(), Key.of(i + keyOffset, appId()),
-                intents.put(app, new PointToPointIntent(appId(app), s, treatment,
+                                               selector, treatment,
-                                                        ingress, egress));
+                                               ingress, egress,
+                                               Collections.emptyList()));
-            }
         }
         return intents;
     }
-    private void submitIntents(ArrayListMultimap<Integer, Intent> intents) {
+    private void submitIntents(List<Intent> intents) {
+        latch = new CountDownLatch(count);
         start = System.currentTimeMillis();
-        for (Integer app : intents.keySet()) {
+        for (Intent intent : intents) {
-            for (Intent intent : intents.get(app)) {
+            if (add) {
-                if (add) {
+                service.submit(intent);
-                    service.submit(intent);
+            } else {
-                } else {
+                service.withdraw(intent);
-                    service.withdraw(intent);
-                }
             }
         }
@@ -193,21 +173,6 @@ public class IntentPushTestCommand extends AbstractShellCommand
         print("Time to %s %d intents: %d ms", text, count, delta);
     }
-
-    /**
-     * Returns application ID for the CLI.
-     *
-     * @param id application id
-     * @return command-line application identifier
-     */
-    protected ApplicationId appId(Integer id) {
-        ApplicationId appId = get(CoreService.class)
-                                .registerApplication("org.onosproject.cli-"
-                                                     + (id + appIdBase));
-        myAppIds.add(appId);
-        return appId;
-    }
-
     /**
      * Extracts the port number portion of the ConnectPoint.
      *
@@ -240,7 +205,7 @@ public class IntentPushTestCommand extends AbstractShellCommand
             = EnumSet.of(Type.INSTALL_REQ, Type.WITHDRAW_REQ);
     @Override
     public synchronized void event(IntentEvent event) {
-        if (!myAppIds.contains(event.subject().appId())) {
+        if (!appId().equals(event.subject().appId())) {
             // not my event, ignore
             return;
         }