Removing the implicit delete from Terminate and Kill.

testing/legion/examples/subprocess/subprocess_test.py

 #!/usr/bin/env python
 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """A host test module demonstrating interacting with remote subprocesses."""
 
 # Map the legion directory so we can import the host controller.
 import sys
 sys.path.append('../../')
@@ skipping 14 matching lines @@
     self.task = None
 
   def SetUp(self):
     """Creates the task machine and waits until it connects."""
     parser = argparse.ArgumentParser()
     parser.add_argument('--task-hash')
     args, _ = parser.parse_known_args()
 
     self.task = self.CreateNewTask(
         isolated_hash=args.task_hash,
-        dimensions={'os': 'Ubuntu-14.04', 'pool': 'Legion'},
+        dimensions={'os': 'Ubuntu-14.04', 'pool': 'Chromoting'},

[Mike Meade, 2015/03/06 17:18:43]

We merged the Legion machines in with the Chromoting machines since they are
configured to be able to run the browser_tests. There was no reason to maintain
2 sets of machines with different pool names.

         idle_timeout_secs=90, connection_timeout_secs=90,
         verbosity=logging.DEBUG)
     self.task.Create()
     self.task.WaitForConnection()
 
   def RunTest(self):
     """Main method to run the test code."""
     self.TestLs()
     self.TestTerminate()
     self.TestMultipleProcesses()
@@ skipping 11 matching lines @@
     self.task.rpc.subprocess.Wait(sleep20)
     elapsed = time.time() - start
     assert elapsed >= 20
 
     self.task.rpc.subprocess.Delete(sleep20)
     self.task.rpc.subprocess.Delete(sleep10)
 
   def TestTerminate(self):
     start = time.time()
     proc = self.task.rpc.subprocess.Popen(['sleep', '20'])
-    self.task.rpc.subprocess.Terminate(proc)  # Implicitly deleted
+    self.task.rpc.subprocess.Terminate(proc)
     try:
       self.task.rpc.subprocess.Wait(proc)
     except xmlrpclib.Fault:
       pass
+    finally:
+      self.task.rpc.subprocess.Delete(proc)
     assert time.time() - start < 20
 
   def TestLs(self):
     proc = self.task.rpc.subprocess.Popen(['ls'])
     self.task.rpc.subprocess.Wait(proc)
     assert self.task.rpc.subprocess.GetReturncode(proc) == 0
     assert 'task.isolate' in self.task.rpc.subprocess.ReadStdout(proc)
     self.task.rpc.subprocess.Delete(proc)
 
 
 if __name__ == '__main__':
   ExampleTestController().RunController()