commit 9ea8ecb5eb6ecd3f93ab89f4cc691e77ab860c78
author Priyanka B <priyanka.b@huawei.com> Tue Jul 26 09:53:22 2016 +0530
committer Priyankab-Huawei <priyanka.b@huawei.com> Tue Jul 26 08:19:51 2016 +0000
tree 3ea046d9535f5901a989e2fd3d5a3f974e9976bf
parent 4c3b451b5ca973bb88a3a4a8e6f6cc0ce2e174b8

[ONOS] update command defect fix for cost type

Change-Id: I6cf35b6cd699904d15df7f08ca47d54e0b9c2138
(cherry picked from commit 7e3bfa4c3f96600a37182fd581d9df08026aa19e)

apps/pce/app/src/main/java/org/onosproject/pce/cli/PceUpdatePathCommand.java

diff --git a/apps/pce/app/src/main/java/org/onosproject/pce/cli/PceUpdatePathCommand.java b/apps/pce/app/src/main/java/org/onosproject/pce/cli/PceUpdatePathCommand.java
index 536eda8..058a17f 100644
--- a/apps/pce/app/src/main/java/org/onosproject/pce/cli/PceUpdatePathCommand.java
+++ b/apps/pce/app/src/main/java/org/onosproject/pce/cli/PceUpdatePathCommand.java
@@ -47,7 +47,7 @@
 
     @Option(name = "-c", aliases = "--cost", description = "The cost attribute IGP cost (1) or TE cost (2).",
             required = false, multiValued = false)
-    int cost = 0;
+    int cost = -1;
 
     @Option(name = "-b", aliases = "--bandwidth", description = "The bandwidth attribute of path. "
             + "Data rate unit is in Bps.", required = false, multiValued = false)
@@ -66,12 +66,14 @@
         }
 
         // Cost validation
-        if ((cost < 1) || (cost > 2)) {
-            error("The cost attribute value is either IGP cost(1) or TE cost(2).");
-            return;
+        if (cost != -1) {
+            if ((cost < 1) || (cost > 2)) {
+                error("The cost attribute value is either IGP cost(1) or TE cost(2).");
+                return;
+            }
+            CostConstraint.Type costType = CostConstraint.Type.values()[cost - 1];
+            constrntList.add(CostConstraint.of(costType));
         }
-        CostConstraint.Type costType = CostConstraint.Type.values()[cost - 1];
-        constrntList.add(CostConstraint.of(costType));
 
         if (!service.updatePath(TunnelId.valueOf(id), constrntList)) {
             error("Path updation failed.");