From 3c43b87571a5f46b73c62ed92f11b5868e9f38c5 Mon Sep 17 00:00:00 2001 From: Han Shen Date: Fri, 18 Sep 2015 11:02:30 -0700 Subject: Fixed one runtime error and enabled more output for schedv2. One bug fixed - runtime error in nightly - command_executor is not imported, although this does not break crosperf functionality. Enabled more output - I enabled thread status reporting even under non-verbose, the output is concise and it is helpful in case of debugging. TEST=manually launched crosperf. Change-Id: I775069d92b84e8f069d537df884ee07057abd3bf Reviewed-on: https://chrome-internal-review.googlesource.com/231387 Reviewed-by: Han Shen Tested-by: Han Shen --- crosperf/experiment_status.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'crosperf/experiment_status.py') diff --git a/crosperf/experiment_status.py b/crosperf/experiment_status.py index ae18dbca..ce562b95 100644 --- a/crosperf/experiment_status.py +++ b/crosperf/experiment_status.py @@ -97,15 +97,16 @@ class ExperimentStatus(object): (key, self._GetNamesAndIterations(val))) thread_status = "" - if self.experiment.log_level == "verbose": - thread_status_format = "Thread Status: \n{}\n" - if self.experiment.schedv2() is None: + thread_status_format = "Thread Status: \n{}\n" + if (self.experiment.schedv2() is None and + self.experiment.log_level == "verbose"): # Add the machine manager status. thread_status = thread_status_format.format( self.experiment.machine_manager.AsString()) - else: + elif self.experiment.schedv2(): + # In schedv2 mode, we always print out thread status. thread_status = thread_status_format.format( - self.experiment.schedv2().threads_status_as_string()) + self.experiment.schedv2().threads_status_as_string()) result = "{}{}".format(thread_status, "\n".join(status_strings)) -- cgit v1.2.3