From 399b58fe776d7b97ddf098d28093ad49c089aaea Mon Sep 17 00:00:00 2001 From: Vratko Polak Date: Wed, 15 Aug 2018 09:51:40 +0200 Subject: [PATCH] PAL: Log and continue when out of colors 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 --- resources/tools/presentation/generator_CPTA.py | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/resources/tools/presentation/generator_CPTA.py b/resources/tools/presentation/generator_CPTA.py index 7279ea345a..a4459bc003 100644 --- a/resources/tools/presentation/generator_CPTA.py +++ b/resources/tools/presentation/generator_CPTA.py @@ -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 -- 2.16.6