Add functionality required for perf dashboard.

src/scripts/run_remote_tests.sh

 #!/bin/bash
 
 # Copyright (c) 2009 The Chromium OS Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Script to run client or server tests on a live remote image.
  
 # Load common constants.  This should be the first executable line.
 # The path to common.sh should be relative to your script's location.
 
 . "$(dirname $0)/common.sh"
 . "$(dirname $0)/remote_access.sh"
 
 DEFAULT_OUTPUT_FILE=test-output-$(date '+%Y%m%d.%H%M%S')
 
 DEFINE_boolean cleanup ${FLAGS_TRUE} "Clean up temp directory"
 DEFINE_integer iterations 1 "Iterations to run every top level test" i
 DEFINE_string output_file "${DEFAULT_OUTPUT_FILE}" "Test run output" o
 DEFINE_boolean verbose ${FLAGS_FALSE} "Show verbose autoserv output" v
 DEFINE_boolean update_db ${FLAGS_FALSE} "Put results in autotest database" u
 DEFINE_string machine_desc "" "Machine description used in database"
+DEFINE_string build_desc "" "Build description used in database"
 
 function cleanup() {
   if [[ $FLAGS_cleanup -eq ${FLAGS_TRUE} ]]; then
     rm -rf "${TMP}"
   else
     echo "Left temporary files at ${TMP}"
   fi
 }
 
 # Returns an error if the test_result_file has text which indicates
@@ skipping 17 matching lines @@
 
 # Removes single quotes around parameter
 # Arguments:
 #   $1 - string which optionally has surrounding quotes
 # Returns:
 #   None, but prints the string without quotes.
 function remove_quotes() {
   echo "$1" | sed -e "s/^'//; s/'$//"
 }
 
+# Adds attributes to all tests run
+# Arguments:
+#   $1 - results directory
+#   $2 - attribute name (key)
+#   $3 - attribute value (value)
+function add_test_attribute() {
+  local results_dir="$1"
+  local attribute_name="$2"
+  local attribute_value="$3"
+  if [[ -z "$attribute_value" ]]; then
+    return;
+  fi
+
+  for status_file in $(echo "${results_dir}"/*/status); do
+    local keyval_file=$(dirname $status_file)/keyval
+    echo "Updating ${keyval_file}"
+    echo "${attribute_name}=${attribute_value}" >> "${keyval_file}"
+  done
+}
+
 function main() {
   assert_outside_chroot
 
   cd $(dirname "$0")
 
   FLAGS "$@" || exit 1
 
   if [[ -z "${FLAGS_ARGV}" ]]; then
     echo "Please specify tests to run, like:"
     echo "  $0 --remote=MyMachine SystemBootPerf"
@@ skipping 61 matching lines @@
     if [ "${type}" == "client" ]; then
       option="-c"
     elif [ "${type}" == "server" ]; then
      option="-s"
     else
       echo "Unknown type of test (${type}) in ${control_file}"
       exit 1
     fi
     echo "Running ${type} test ${control_file}"
     local short_name=$(basename $(dirname "${control_file}"))
-    local timestamp=$(date '+%s')
-    local results_dir="${TMP}/${short_name},${FLAGS_machine_desc},${timestamp}"
+    local start_time=$(date '+%s')
+    local results_dir="${TMP}/${short_name},${FLAGS_machine_desc},${start_time}"
     rm -rf "${results_dir}"
     local verbose=""
     if [[ ${FLAGS_verbose} -eq $FLAGS_TRUE ]]; then
       verbose="--verbose"
     fi
     ${autoserv} -m "${FLAGS_remote}" "${option}" "${control_file}" \
       -r "${results_dir}" ${verbose}
     local test_status="${results_dir}/status"
     local test_result_dir="${results_dir}/${short_name}"
     local keyval_file="${test_result_dir}/results/keyval"
     if is_successful_test "${test_status}"; then
       echo "${control_file} succeeded." | tee -a "${FLAGS_output_file}"
       if [[ -f "${keyval_file}" ]]; then
         echo "Keyval was:" | tee -a "${FLAGS_output_file}"
         cat "${keyval_file}" | tee -a "${FLAGS_output_file}"
       fi
     else
       echo "${control_file} failed:" | tee -a "${FLAGS_output_file}"
       cat "${test_status}" | tee -a "${FLAGS_output_file}"
         # Leave around output directory if the test failed.
       FLAGS_cleanup=${FLAGS_FALSE}
     fi
+    local end_time=$(date '+%s')
 
     # Update the database with results.
     if [[ ${FLAGS_update_db} -eq ${FLAGS_TRUE} ]]; then
+      add_test_attribute "${results_dir}" machine-desc "${FLAGS_machine_desc}"
+      add_test_attribute "${results_dir}" build-desc "${FLAGS_build_desc}"
+      add_test_attribute "${results_dir}" server-start-time "${start_time}"
+      add_test_attribute "${results_dir}" server-end-time "${end_time}"
       if ! "${parse_cmd}" -o "${results_dir}"; then
         echo "Parse failed." | tee -a "${FLAGS_output_file}"
         FLAGS_cleanup=${FLAGS_FALSE}
       fi
     fi
   done
 
   echo "Output stored to ${FLAGS_output_file}"
 }
 
 main $@