x/turbo-hipster
Code Issues Proposed changes

fix checking reporting overall results based off multiple datasets

Browse Source
This commit is contained in:
Joshua Hesketh 2013-09-11 14:41:35 +10:00
parent c7e963bad4
commit 0b67108deb
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
turbo_hipster/task_plugins/gate_real_db_upgrade/task.py
View File

@ -165,13 +165,14 @@ class Runner(threading.Thread):
def _check_all_dataset_logs_for_errors(self): def _check_all_dataset_logs_for_errors(self):
self.log.debug("Check logs for errors") self.log.debug("Check logs for errors")
success = False success = True
for i, dataset in enumerate(self.job_datasets): for i, dataset in enumerate(self.job_datasets):
# Look for the beginning of the migration start # Look for the beginning of the migration start
success, message = \ dataset_success, message = \
handle_results.check_log_for_errors(dataset['log_file_path'], handle_results.check_log_for_errors(dataset['log_file_path'],
self.git_path) self.git_path)
self.job_datasets[i]['result'] = message self.job_datasets[i]['result'] = message
success = False if not dataset_success else success
if success: if success:
self.work_data['result'] = "SUCCESS" self.work_data['result'] = "SUCCESS"