T3: Adding summary to pingall, false if any of the path in the trace fails and minor fixes
Change-Id: Ifa2777a37dc91ccec3401d1080bd3849878f9734
(cherry picked from commit 6207715317b140dd8d6e324a974c1d8a449c0fb8)
diff --git a/src/main/java/org/onosproject/t3/api/StaticPacketTrace.java b/src/main/java/org/onosproject/t3/api/StaticPacketTrace.java
index 084b25e..22db007 100644
--- a/src/main/java/org/onosproject/t3/api/StaticPacketTrace.java
+++ b/src/main/java/org/onosproject/t3/api/StaticPacketTrace.java
@@ -43,7 +43,7 @@
private Map<DeviceId, List<FlowEntry>> flowsForDevice;
private StringBuilder resultMessage;
private Pair<Host, Host> hosts;
- private boolean success = false;
+ private List<Boolean> success = new ArrayList<>();
/**
* Builds the trace with a given packet and a connect point.
@@ -197,21 +197,21 @@
}
/**
- * Return if this trace is successful.
+ * Return if all the possible paths of this trace are successful.
*
- * @return true if successful
+ * @return true if all paths are successful
*/
public boolean isSuccess() {
- return success;
+ return !success.contains(false);
}
/**
- * Sets if this trace is successful.
+ * Sets if a path from this trace is successful.
*
- * @param success true if trace is successful.
+ * @param success true if a path of trace is successful.
*/
public void setSuccess(boolean success) {
- this.success = success;
+ this.success.add(success);
}