Move site_chrome_test, site_logging, site_log_reader, site_crash_test into cros dir.

client/bin/site_job.py

 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import os
-
-from autotest_lib.client.bin import boottool, site_logging, utils
+from datetime import datetime
+from autotest_lib.client.bin import boottool, utils
 from autotest_lib.client.bin.job import base_client_job
 from autotest_lib.client.common_lib import error
-from datetime import datetime
+from autotest_lib.client.cros import log_reader
+
 
 LAST_BOOT_TAG = object()
 
 class site_job(base_client_job):
-    def __init__(self, *args, **kwargs):
-        base_client_job.__init__(self, *args, **kwargs)
-
 
     def run_test(self, url, *args, **dargs):
-        log_pauser = site_logging.LogRotationPauser()
+        log_pauser = log_reader.LogRotationPauser()

[kmixter1, 2010/12/16 03:12:57]

If you collapse site_log_reader and site_logging, I would suggest logging.  Of
course that conflicts with the internal python name, so maybe adopt cros_logging

[ericli, 2010/12/17 21:48:39]

I agree. I have some initial concern about prefix cros_* to the name, but since
you guys OK with it, I am all for it.

I will also change ui to cros_ui in a separate CL.


On 2010/12/16 03:12:57, kmixter1 wrote:

         passed = False
         try:
             log_pauser.begin()
             passed = base_client_job.run_test(self, url, *args, **dargs)
             if not passed:
                 # Save the VM state immediately after the test failure.
                 # This is a NOOP if the the test isn't running in a VM or
                 # if the VM is not properly configured to save state.
                 group, testname = self.pkgmgr.get_package_name(url, 'test')
                 now = datetime.now().strftime('%I:%M:%S.%f')
@@ skipping 15 matching lines @@
 
         # sync first, so that a sync during shutdown doesn't time out
         utils.system("sync; sync", ignore_status=True)
 
         utils.system("(sleep 5; reboot) </dev/null >/dev/null 2>&1 &")
         self.quit()
 
 
     def require_gcc(self):
         return False