[autotest] Drone -> drone.hostname
Forgot this wasn't an __str__, and it's messing with graphite.
BUG=None
DEPLOY=scheduler
TEST=Ran
Change-Id: Id8bcb24471e2e4ae03fb4ff92ba7ad31c3cbfb06
Reviewed-on: https://chromium-review.googlesource.com/200306
Tested-by: Alex Miller <milleral@chromium.org>
Reviewed-by: Prashanth B <beeps@chromium.org>
diff --git a/scheduler/drone_manager.py b/scheduler/drone_manager.py
index 7d25098..2caa66d 100644
--- a/scheduler/drone_manager.py
+++ b/scheduler/drone_manager.py
@@ -401,8 +401,9 @@
for drone, results_list in all_results.iteritems():
results = results_list[0]
+ drone_hostname = drone.hostname.replace('.', '_')
- with self._timer.get_client('%s.results' % drone):
+ with self._timer.get_client('%s.results' % drone_hostname):
for process_info in results['all_processes']:
if process_info['comm'] == 'autoserv':
self._add_autoserv_process(drone, process_info)
@@ -412,10 +413,10 @@
for process_info in results['parse_processes']:
self._add_process(drone, process_info)
- with self._timer.get_client('%s.pidfiles' % drone):
+ with self._timer.get_client('%s.pidfiles' % drone_hostname):
self._process_pidfiles(drone, results['pidfiles'],
self._pidfiles)
- with self._timer.get_client('%s.pidfiles_second' % drone):
+ with self._timer.get_client('%s.pidfiles_second' % drone_hostname):
self._process_pidfiles(drone, results['pidfiles_second_read'],
self._pidfiles_second_read)