PAL: Log and continue when out of colors 75/14175/11
authorVratko Polak <vrpolak@cisco.com>
Wed, 15 Aug 2018 07:51:40 +0000 (09:51 +0200)
committerVratko Polak <vrpolak@cisco.com>
Wed, 15 Aug 2018 07:51:40 +0000 (09:51 +0200)
Previously, IndexError happened in worker, getting the job stuck.

Graphs should not need that many colors, so there is a bug to fix still.

Change-Id: I1af415119cf06004be5febbd03636f847a7001be
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
resources/tools/presentation/generator_CPTA.py

index 7279ea3..a4459bc 100644 (file)
@@ -318,13 +318,21 @@ def _generate_all_charts(spec, input_data):
                 logs.append(("WARNING", "No data for the test '{0}'".
                              format(test_name)))
                 continue
+            message = "index: {index}, test: {test}".format(
+                index=index, test=test_name)
             test_name = test_name.split('.')[-1]
-            trace, rslt = _generate_trending_traces(
-                test_data,
-                job_name=job_name,
-                build_info=build_info,
-                name='-'.join(test_name.split('-')[2:-1]),
-                color=COLORS[index])
+            try:
+                trace, rslt = _generate_trending_traces(
+                    test_data,
+                    job_name=job_name,
+                    build_info=build_info,
+                    name='-'.join(test_name.split('-')[2:-1]),
+                    color=COLORS[index])
+            except IndexError:
+                message = "Out of colors: {}".format(message)
+                logs.append(("ERROR", message))
+                logging.error(message)
+                continue
             traces.extend(trace)
             res.append(rslt)
             index += 1