Remove "-t -t" -- is this causing us to hang?

site_config/slave_hosts_cfg.py

 """ This file contains configuration information for the build slave host
 machines. """
 
 
 import collections
 import ntpath
 import os
 import posixpath
 
 import skia_vars
@@ skipping 26 matching lines @@
 GCE_USERNAME = skia_vars.GetGlobalVariable('gce_username')
 
 SKIALAB_ROUTER_IP = skia_vars.GetGlobalVariable('skialab_router_ip')
 SKIALAB_USERNAME = skia_vars.GetGlobalVariable('skialab_username')
 
 
 # Procedures for logging in to the host machines.
 
 def skia_lab_login(hostname, config):
   """Procedure for logging into SkiaLab machines."""
-  # The multiple "-t"s are needed in order to force tty allocation, which gives
-  # us the expected interactive behavior, even with two levels of SSH.
   return [
-    'ssh', '-t', '-t', '%s@%s' % (SKIALAB_USERNAME, SKIALAB_ROUTER_IP),
+    'ssh', '%s@%s' % (SKIALAB_USERNAME, SKIALAB_ROUTER_IP),
     'ssh', '%s@%s' % (SKIALAB_USERNAME, config['ip'])
   ]
 
 
 def compute_engine_login(hostname, config):
   """Procedure for logging into Skia GCE instances."""
   return [
     'gcutil', '--project=%s' % GCE_PROJECT,
     'ssh', '--ssh_user=%s' % GCE_USERNAME, hostname,
   ]
@@ skipping 537 matching lines @@
   """Helper function for retrieving slave host configuration information.
 
   If the given hostname is unknown, returns a default config.
 
   Args:
       hostname: string; the hostname of the slave host machine.
   Returns:
       SlaveHostConfig instance representing the given host.
   """
   return SLAVE_HOSTS.get(hostname, default_slave_host_config(hostname))