New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 672727 link

Starred by 1 user
Status: Archived
Owner:
ayatane@chromium.org
Closed: Mar 2018
Cc:
█ dshi@chromium.org
█ jrbarnette@chromium.org
pprabhu@chromium.org
█ sbasi@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Autotest: Unfork Site* classes.

Project Member Reported by pprabhu@chromium.org, Dec 9 2016 
Autotest has the Site* facility for extending functionality of core objects.
We used this facility in the past for CrOS extensions. 

But, we're long past the point where we'll ever upstream our changes / rebase to a more recent upstream autotest codebase.

If we unfork some of our changes from Site* to the base object, we can get rid of some files and simplify code readability.

Comment 1 by autumn@chromium.org, Dec 13 2016 

Is this part of Skylabs? If not, I'll mark it as a fixit candidate.

Comment 2 by pprabhu@chromium.org, Dec 13 2016

Labels: -current-issue Hotlist-Fixit

Comment 3 by akes...@chromium.org, Jan 13 2017

Cc: pprabhu@chromium.org dshi@chromium.org jrbarnette@chromium.org sbasi@chromium.org
Labels: -Pri-2 Pri-1
Status: Available (was: Untriaged)
Is this fixit sized? If so, I think it would be a really great one to fix.

Comment 4 by sbasi@chromium.org, Jan 13 2017 

This may be OKR SubKr sized IMO but depends on how many conflicts there are. You might be able to mostly just concat the site class to the base class and things work fine.

Comment 5 by ayatane@chromium.org, Feb 2 2017

Owner: ayatane@chromium.org
Project Member

Comment 7 by bugdroid1@chromium.org, Feb 4 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b

commit cdd00f20ac7607ff89a97e83e2483a4c8feddb7b
Author: Allen Li <ayatane@chromium.org>
Date: Sat Feb 04 04:01:41 2017

[autotest] Remove site_rpc_interface

BUG= chromium:672727 
TEST=Run unittests

Change-Id: I8535b121ba7f2317cccec981bf2a8a36ca2639cf
Reviewed-on: https://chromium-review.googlesource.com/435850
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

[modify] https://crrev.com/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b/server/site_utils_unittest.py
[modify] https://crrev.com/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b/frontend/afe/views.py
[delete] https://crrev.com/b774aa277a9e351708d9a8396ecee846dd0a8e91/frontend/afe/site_rpc_interface.py
[modify] https://crrev.com/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b/frontend/afe/direct_afe.py
[delete] https://crrev.com/b774aa277a9e351708d9a8396ecee846dd0a8e91/frontend/afe/site_rpc_interface_unittest.py
[modify] https://crrev.com/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b/scheduler/shard/shard_client.py
[modify] https://crrev.com/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b/frontend/afe/control_file.py
[modify] https://crrev.com/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b/server/site_utils.py
[modify] https://crrev.com/cdd00f20ac7607ff89a97e83e2483a4c8feddb7b/frontend/afe/rpc_interface.py
Project Member

Comment 8 by bugdroid1@chromium.org, Feb 4 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/fce2064b1078bc7c29bbaedb2cdf0997285bcb4d

commit fce2064b1078bc7c29bbaedb2cdf0997285bcb4d
Author: Allen Li <ayatane@chromium.org>
Date: Sat Feb 04 04:01:41 2017

[autotest] Merge site modules in utils/

BUG= chromium:672727 
TEST=None

Change-Id: Ia4462d369dc60c1298f253f645218831dcbc6e17
Reviewed-on: https://chromium-review.googlesource.com/435945
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[delete] https://crrev.com/dfb4e30592ddeaceb270df3fbab3ff3faefa5396/utils/site_test_importer_attributes.py
[modify] https://crrev.com/fce2064b1078bc7c29bbaedb2cdf0997285bcb4d/utils/test_importer.py
Project Member

Comment 9 by bugdroid1@chromium.org, Feb 4 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/2e68f186d24153994c8776d5b563921cf6d1d6cb

commit 2e68f186d24153994c8776d5b563921cf6d1d6cb
Author: Allen Li <ayatane@chromium.org>
Date: Sat Feb 04 04:01:42 2017

[autotest] Merge site_job into job

BUG= chromium:672727 
TEST=None

Change-Id: I4da8896a0b64bcb8e4cee6fe78b5affacff69243
Reviewed-on: https://chromium-review.googlesource.com/436345
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/2e68f186d24153994c8776d5b563921cf6d1d6cb/client/bin/job.py
[delete] https://crrev.com/046a14f0acb61fc08ca77e842adef3ff650af00d/client/bin/site_job.py
Project Member

Comment 10 by bugdroid1@chromium.org, Feb 4 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/046a14f0acb61fc08ca77e842adef3ff650af00d

commit 046a14f0acb61fc08ca77e842adef3ff650af00d
Author: Allen Li <ayatane@chromium.org>
Date: Sat Feb 04 04:01:42 2017

[autotest] Simplify direct_afe proxy logic

BUG= chromium:672727 
TEST=None

Change-Id: Idcc9627e5c53a0348ceb58b00b13afc000d8f16e
Reviewed-on: https://chromium-review.googlesource.com/436564
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/046a14f0acb61fc08ca77e842adef3ff650af00d/frontend/afe/direct_afe.py
Project Member

Comment 11 by bugdroid1@chromium.org, Feb 4 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/097e0371b6f462a9ed2aea43dcfcb2fbbb67396b

commit 097e0371b6f462a9ed2aea43dcfcb2fbbb67396b
Author: Allen Li <ayatane@chromium.org>
Date: Sat Feb 04 04:01:43 2017

[autotest] Make status_indenter.job private

BUG= chromium:672727 
TEST=None

Change-Id: I812a3a81f31ba80ce0edbef8f47a3b66dc19ab32
Reviewed-on: https://chromium-review.googlesource.com/437664
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/097e0371b6f462a9ed2aea43dcfcb2fbbb67396b/client/bin/job.py
Project Member

Comment 14 by bugdroid1@chromium.org, Apr 7 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0

commit 2c32d6b593c4987a525ef162d6704fa6d6d7c0b0
Author: Allen Li <ayatane@chromium.org>
Date: Fri Apr 07 21:15:36 2017

[autotest] Merge client/bin/site_utils

BUG= chromium:672727 
TEST=None

Change-Id: I03a05a3ae3163437109010106ad988ff101a3998
Reviewed-on: https://chromium-review.googlesource.com/437589
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/desktopui_MashLogin/desktopui_MashLogin.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/video_WebRtcPerf/video_WebRtcPerf.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/hardware_SsdDetection/hardware_SsdDetection.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/cros/power_utils.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/hardware_Smartctl/hardware_Smartctl.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/bin/fio_util.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/common_lib/cros/dev_server.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/video_PlaybackPerf/video_PlaybackPerf.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/cros/multimedia/system_facade_native.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/bin/site_utils.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/bin/utils.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/power_Consumption/power_Consumption.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/platform_RootPartitionsNotMounted/platform_RootPartitionsNotMounted.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/hardware_DiskSize/hardware_DiskSize.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/server/site_tests/platform_CorruptRootfs/platform_CorruptRootfs.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/platform_PartitionCheck/platform_PartitionCheck.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/hardware_Usb30Throughput/hardware_Usb30Throughput.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/longevity_Tracker/longevity_Tracker.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/bin/net/net_utils.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/enterprise_KioskPerf/enterprise_KioskPerf.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/server/hosts/cros_host.py
[delete] https://crrev.com/01d689a5f87b19f473ab517a7b167320ffdc80be/client/bin/base_utils.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/bin/net/net_utils_unittest.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/performance_Tracker/performance_Tracker.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/common_lib/cros/dev_server_unittest.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/bin/update_intel_pci_ids
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/hardware_Badblocks/hardware_Badblocks.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/hardware_StorageTrim/hardware_StorageTrim.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/cli/host.py
[modify] https://crrev.com/2c32d6b593c4987a525ef162d6704fa6d6d7c0b0/client/site_tests/hardware_StorageFio/hardware_StorageFio.py
Project Member

Comment 15 by bugdroid1@chromium.org, Apr 18 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/0adbce500c5e86cc89cfb0d51fb1f3b446202547

commit 0adbce500c5e86cc89cfb0d51fb1f3b446202547
Author: Allen Li <ayatane@chromium.org>
Date: Tue Apr 18 05:50:23 2017

Restore a copy of base_utils.py for backward compatibility

https://chromium-review.googlesource.com/c/437589/

BUG= chromium:672727 
TEST=None

Change-Id: Ib3f1f73dd74b2d0d114c878be29e6f6ead833a53
Reviewed-on: https://chromium-review.googlesource.com/473527
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Daniel Wang <wonderfly@google.com>

[add] https://crrev.com/0adbce500c5e86cc89cfb0d51fb1f3b446202547/client/bin/base_utils.py
Project Member

Comment 16 by bugdroid1@chromium.org, Apr 22 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6

commit 5ed7e63aeec90dc0c6b4229a66bf161410aa86a6
Author: Allen Li <ayatane@chromium.org>
Date: Sat Apr 22 04:57:55 2017

[autotest] Merge client/common_lib/site_utils

BUG= chromium:672727 
TEST=None

Change-Id: I15859c0e05e99d244eec5b470a984ced0c874f19
Reviewed-on: https://chromium-review.googlesource.com/437727
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/site_tests/power_VideoDetector/power_VideoDetector.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/site_utils/gs_offloader.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/cros/cellular/pseudomodem/pseudomodem_context.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/site_tests/network_TwoShills/network_TwoShills.py
[delete] https://crrev.com/240e14e3d873a64a18bf600b2bce22d8ef65924f/client/common_lib/base_utils_unittest.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/common_lib/base_utils.py
[delete] https://crrev.com/240e14e3d873a64a18bf600b2bce22d8ef65924f/client/common_lib/site_utils_unittest.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/cros/liststorage.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/site_tests/brillo_WifiInterfaceTest/brillo_WifiInterfaceTest.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/cros/multimedia/usb_facade_native.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/site_tests/brillo_PingTest/brillo_PingTest.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/bin/utils.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/cros/chaos_lib/chaos_runner.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/site_tests/policy_PluginsBlockedForUrls/policy_PluginsBlockedForUrls.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/site_utils/tester_feedback/audio_query_delegate_impl.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/site_tests/platform_USBHIDWake/platform_USBHIDWake.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/hosts/sonic_host.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/site_tests/power_LoadTest/power_LoadTest.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/cros/storage.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/site_tests/brillo_PlaybackAudioTest/brillo_PlaybackAudioTest.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/site_tests/policy_PluginsAllowedForUrls/policy_PluginsAllowedForUrls.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/site_utils/gs_offloader_unittest.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/docs/best-practices.md
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/site_tests/network_VPNConnect/network_VPNConnect.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/site_utils/server_manager_utils.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/frontend/afe/models.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/common_lib/site_utils.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/cros/dynamic_suite/suite.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/cros/network/wifi_client.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/cros/tradefed_test.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/server/site_tests/brillo_DecodingAudioTest/brillo_DecodingAudioTest.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/cros/power_suspend.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/common_lib/perf_expectations/expectation_checker.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/cros/power_utils.py
[modify] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/common_lib/utils.py
[add] https://crrev.com/5ed7e63aeec90dc0c6b4229a66bf161410aa86a6/client/common_lib/utils_unittest.py
Project Member

Comment 17 by bugdroid1@chromium.org, May 2 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/e7dbe651ce80de1b4e158a635ae79143577ce98a

commit e7dbe651ce80de1b4e158a635ae79143577ce98a
Author: Allen Li <ayatane@chromium.org>
Date: Tue May 02 02:18:11 2017

[autotest] Merge scheduler/archive_results

We should get rid of the archiving step entirely, but thats beyond
the scope of the current fixit.

 crbug.com/689244 

BUG= chromium:672727 
TEST=None

Change-Id: Ibd1a9e2fa980f9147327a5910fb13d795ee55baa
Reviewed-on: https://chromium-review.googlesource.com/437733
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/e7dbe651ce80de1b4e158a635ae79143577ce98a/scheduler/archive_results.control.srv
[delete] https://crrev.com/357103679fde0e7ff3ea6b2909ac71b4b690939a/scheduler/archive_results.py
[delete] https://crrev.com/357103679fde0e7ff3ea6b2909ac71b4b690939a/scheduler/site_archive_results.py
Project Member

Comment 18 by bugdroid1@chromium.org, May 3 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/9bd3eca726f54773c942ca2c27446ffb0f571a9b

commit 9bd3eca726f54773c942ca2c27446ffb0f571a9b
Author: Allen Li <ayatane@chromium.org>
Date: Wed May 03 03:24:19 2017

[autotest] Merge scheduler/drones

BUG= chromium:672727 
TEST=None

Change-Id: I64c58513e72c505e88016db2552ce188d0da785f
Reviewed-on: https://chromium-review.googlesource.com/438669
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/9bd3eca726f54773c942ca2c27446ffb0f571a9b/scheduler/drones.py
[delete] https://crrev.com/f7e87754b3296be922083842c537be2e56a2e9f8/scheduler/site_drones.py
Project Member

Comment 19 by bugdroid1@chromium.org, May 20 2017 

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/overlays/overlay-stumpy-private/+/3e1ca43e787561d33db64a0c7c764add43d7f289

commit 3e1ca43e787561d33db64a0c7c764add43d7f289
Author: Hung-Te Lin <hungte@chromium.org>
Date: Sat May 20 10:55:14 2017

Comment 21 by ayatane@chromium.org, Jun 20 2017

Labels: Type-Feature

Comment 22 by ayatane@chromium.org, Jun 20 2017

Labels: -Pri-1 Pri-2
Project Member

Comment 23 by bugdroid1@chromium.org, Jun 22 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/4ba6f243ea162ea7c154174da666cb81c3a94010

commit 4ba6f243ea162ea7c154174da666cb81c3a94010
Author: Allen Li <ayatane@chromium.org>
Date: Thu Jun 22 01:48:12 2017

[autotest] Remove unused site code from monitor_db

BUG= chromium:672727 
TEST=None

Change-Id: Ic48c99aae1d741cbfe282ab46015df353eddcdef
Reviewed-on: https://chromium-review.googlesource.com/438786
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/4ba6f243ea162ea7c154174da666cb81c3a94010/scheduler/monitor_db.py

Comment 24 by ayatane@chromium.org, Jun 30 2017

Status: Assigned (was: Available)
Project Member

Comment 25 by bugdroid1@chromium.org, Jul 26 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/3e6944709f789c44af7bc3d258f73932d25310b5

commit 3e6944709f789c44af7bc3d258f73932d25310b5
Author: Allen Li <ayatane@chromium.org>
Date: Wed Jul 26 23:46:52 2017

[autotest] Remove overridden import

This code doesnt have my name on it, so it existed prior to any
site_utils merging I have done.  Whatever its supposed to do, it
doesnt seem broken now, so remove the spurious import.

BUG= chromium:672727 
TEST=None

Change-Id: I9f414017bfaea91bc9c12486f7c91bfef41f57d2
Reviewed-on: https://chromium-review.googlesource.com/438838
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/client/cros/cros_ui.py
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/client/site_tests/login_LogoutProcessCleanup/login_LogoutProcessCleanup.py
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/server/autoserv
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/client/site_tests/p2p_ConsumeFiles/p2p_ConsumeFiles.py
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/site_utils/lxc/config.py
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/server/site_tests/telemetry_AFDOGenerate/telemetry_AFDOGenerate.py
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/server/cros/network/wifi_cell_test_base.py
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/client/cros/kernel_trace.py
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/client/site_tests/autoupdate_CannedOmahaUpdate/autoupdate_CannedOmahaUpdate.py
[modify] https://crrev.com/3e6944709f789c44af7bc3d258f73932d25310b5/client/site_tests/logging_UdevCrash/logging_UdevCrash.py
Project Member

Comment 26 by bugdroid1@chromium.org, Jul 28 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/24381f335143599d4b24a7dfe440e9467d8ac484

commit 24381f335143599d4b24a7dfe440e9467d8ac484
Author: Allen Li <ayatane@chromium.org>
Date: Fri Jul 28 00:55:44 2017

[autotest] Merge server/autotest._Run

BUG= chromium:672727 
TEST=None

Change-Id: I23fcd8316f645bc71ec9a901c3cd1f9b2b33b84b
Reviewed-on: https://chromium-review.googlesource.com/438761
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/24381f335143599d4b24a7dfe440e9467d8ac484/server/autotest.py
[modify] https://crrev.com/24381f335143599d4b24a7dfe440e9467d8ac484/server/site_autotest.py
Project Member

Comment 27 by bugdroid1@chromium.org, Aug 4 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/944ac462462553e6248732a6ff85377e55fae295

commit 944ac462462553e6248732a6ff85377e55fae295
Author: Allen Li <ayatane@chromium.org>
Date: Fri Aug 04 02:23:18 2017

[autotest] Merge server/autotest.client_logger

BUG= chromium:672727 
TEST=None

Change-Id: Iacb5d7a73832b0fb5fd12c1e07f7c77246ded073
Reviewed-on: https://chromium-review.googlesource.com/438762
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

[modify] https://crrev.com/944ac462462553e6248732a6ff85377e55fae295/server/autotest.py
[modify] https://crrev.com/944ac462462553e6248732a6ff85377e55fae295/server/site_autotest.py

Comment 29 by ayatane@chromium.org, Mar 10 2018

Status: Archived (was: Assigned)
►

Sign in to add a comment