pkt_gen: STC - Handle Results DB file synchronization failures.

If the downloading of results-database file fails, the CSV file and
corresponding results are not generated - eventhough the test was
successful.
This small patch handles such cases.

Updated the logging to be more explicit.

JIRA: VSPERF-435

Change-Id: I14bf519fbca2ea362777e29920bc1759494965e0
Signed-off-by: Sridhar K. N. Rao <sridhar.rao@spirent.com>

diff --git a/tools/pkt_gen/testcenter/testcenter-rfc2544-rest.py b/tools/pkt_gen/testcenter/testcenter-rfc2544-rest.py
index 538d8a8..8da8ed1 100644 (file)
--- a/tools/pkt_gen/testcenter/testcenter-rfc2544-rest.py
+++ b/tools/pkt_gen/testcenter/testcenter-rfc2544-rest.py
@@ -490,8 +490,13 @@ def main():
         resultsdb = args.results_dir + \
             lab_server_resultsdb.split("/Results")[1]
 
-        logger.info(
-            "The local summary DB file has been saved to %s", resultsdb)
+        if not os.path.exists(resultsdb):
+            resultsdb = lab_server_resultsdb
+            logger.info("Failed to create the local summary DB File, using"
+                        " the remote DB file instead.")
+        else:
+            logger.info(
+                "The local summary DB file has been saved to %s", resultsdb)
 
         # The returns the "RFC2544ThroughputTestResultDetailedSummaryView"
         # table view from the results database.

diff --git a/tools/pkt_gen/testcenter/testcenter-rfc2889-rest.py b/tools/pkt_gen/testcenter/testcenter-rfc2889-rest.py
index cfa425e..ddb6456 100644 (file)
--- a/tools/pkt_gen/testcenter/testcenter-rfc2889-rest.py
+++ b/tools/pkt_gen/testcenter/testcenter-rfc2889-rest.py
@@ -265,8 +265,13 @@ def main():
         resultsdb = args.results_dir + \
             lab_server_resultsdb.split("/Results")[1]
 
-        logger.info(
-            "The local summary DB file has been saved to %s", resultsdb)
+        if not os.path.exists(resultsdb):
+            resultsdb = lab_server_resultsdb
+            logger.info("Failed to create the local summary DB file, using"
+                        " the remote DB file instead.")
+        else:
+            logger.info(
+                "The local summary DB file has been saved to %s", resultsdb)
 
         resultsdict = (
             stc.perform("QueryResult",