x/browbeat
Code Issues Proposed changes

Fix Rally metadata

Browse Source 
We need to strip all whitespace from actions. Also need to match action
in the _type: error.

Change-Id: I931ff583652a84040e286c08e3cf8db381d91125
This commit is contained in:
Joe Talerico 2016-09-26 10:47:01 -04:00 committed by Sai Sindhur Malleni
parent 56bcf10964
commit ce5d34ffd4
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/Rally.py
View File

@ -142,8 +142,14 @@ class Rally(WorkloadBase.WorkloadBase):
rally_data[value] = [] rally_data[value] = []
rally_data[value].append(metrics[workload][value]) rally_data[value].append(metrics[workload][value])
if len(metrics['error']) > 0: if len(metrics['error']) > 0:
error = {'action_name': value, iteration = 1
workload_name = value
if value.find('(') is not -1:
iteration = re.findall('\d+', value)
workload_name = value.split('(')[0]
error = {'action': workload_name.strip(),
'browbeat_rerun': run, 'browbeat_rerun': run,
'iteration': iteration,
'error_type': metrics['error'][0], 'error_type': metrics['error'][0],
'error_msg': metrics['error'][1], 'error_msg': metrics['error'][1],
'result': task_id, 'result': task_id,
@ -161,7 +167,7 @@ class Rally(WorkloadBase.WorkloadBase):
iteration = re.findall('\d+', workload) iteration = re.findall('\d+', workload)
workload_name = workload.split('(')[0] workload_name = workload.split('(')[0]
rally_stats = {'result': task_id, rally_stats = {'result': task_id,
'action': workload_name, 'action': workload_name.strip(),
'browbeat_rerun': run, 'browbeat_rerun': run,
'iteration': iteration, 'iteration': iteration,
'timestamp': str(es_ts).replace(" ", "T"), 'timestamp': str(es_ts).replace(" ", "T"),