[autotest] Swap order of updating label/attrs when doing an mach install.
We'll want to update the label last during a machine install since the
tests use that as a sign that provision succeeded even though it may
have failed updating the host attributes with the job_repo_url.
BUG=chromium:637876
TEST=None
Change-Id: I07b655055f60d1271e2b54dbe03b5ef1a638c7ee
Reviewed-on: https://chromium-review.googlesource.com/370401
Commit-Ready: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
Reviewed-by: Dan Shi <dshi@chromium.org>
diff --git a/server/afe_utils.py b/server/afe_utils.py
index f6997bd..b6e000e 100644
--- a/server/afe_utils.py
+++ b/server/afe_utils.py
@@ -235,9 +235,9 @@
else:
image_name, host_attributes = host.machine_install_by_devserver(
*args, **dargs)
- add_version_label(host, image_name)
for attribute, value in host_attributes.items():
update_host_attribute(host, attribute, value)
+ add_version_label(host, image_name)
def get_os(host):