[autotest] Refactor reimage type selection to support >2 types.
One now passes a string that is tied to the name of the type of
reimaging that one would like to do. More reimaging types can now
be easily added.
BUG=chromium-os:38250
TEST=run_suite with trybot'd control.dummy changed to use reimage_type
Change-Id: I276556ff6822800cc846b20b564ffc36ec07e61b
Reviewed-on: https://gerrit.chromium.org/gerrit/42748
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Simran Basi <sbasi@chromium.org>
Commit-Queue: Alex Miller <milleral@chromium.org>
Tested-by: Alex Miller <milleral@chromium.org>
diff --git a/server/cros/dynamic_suite/constants.py b/server/cros/dynamic_suite/constants.py
index 47d9682..d8fda28 100644
--- a/server/cros/dynamic_suite/constants.py
+++ b/server/cros/dynamic_suite/constants.py
@@ -17,3 +17,9 @@
ARTIFACT_FINISHED_TIME = 'artifact_finished_time'
DOWNLOAD_STARTED_TIME = 'download_started_time'
PAYLOAD_FINISHED_TIME = 'payload_finished_time'
+
+# Reimage type names
+# Please be very careful in changing or adding to these, as one needs to
+# maintain backwards compatibility.
+REIMAGE_TYPE_OS = 'os'
+REIMAGE_TYPE_FIRMWARE = 'firmware'