Re-adding get_site_job_data(): Now uses host platform for hostname.

Re-adding get_site_job_data(): Now uses host platform for hostname.

Update to CL http://codereview.chromium.org/6677136/
 - Changing default keyval info with get_site_job_data() allows cleaner reporting and smaller tko_machines tables when multiple machines are used on the same job.

Now hostname will only be updated in multimachine jobs when the platform of a host can be retrieved from AFE.  The platform will be used as the hostname to prevent breaking the dashboard.

Change-Id: I24c6a6937b583ca3d10f5c9bfb759346ae6a2b26

BUG=None.
TEST=Tested multimachine BVT run in the lab.

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=9dc949c

[pauldean_chromium, 2011-04-06 17:56:24]

Hi Eric,

Here is the update with platform as the hostname in multimachine jobs.

Thanks,
paul

[pauldean_chromium, 2011-04-06 20:14:15]

Eric,

Here's a revised version that reads the platform from the keyval file instead of
from the database.

I didn't use get_platform_label() because that only works when the hosts are
attached to a job, and when I create temporary hosts they are not associated
with a job.  Associating them with a job created more side-effects than it is
worth.

Thanks,
paul

[ericli, 2011-04-06 20:20:57]

LGTM.

http://codereview.chromium.org/6805015/diff/4/server/site_server_job.py
File server/site_server_job.py (right):

http://codereview.chromium.org/6805015/diff/4/server/site_server_job.py#newco...
server/site_server_job.py:25: # Only modify hostname on multimachine jobs
add a comment here:
assume all machine in the job machines list have the same platform label.