Revert "[reclient] Limit local resource tuning on windows to cloudtops"

This reverts commit 90d54ff344.

Reason for revert: b/335525655 Racing with higher resource fraction on physical windows machines seems to also be a problem.

Original change's description:
> [reclient] Limit local resource tuning on windows to cloudtops
>
> Bug: b/330173550
> Change-Id: I65ce93eeb216a7a4b77506fbcb515697fdecde2c
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5420670
> Commit-Queue: Ben Segall <bentekkie@google.com>
> Reviewed-by: Michael Savigny <msavigny@google.com>

Bug: b/330173550
Change-Id: I94479584bd2a041e0e621a8cf3f3cb7f65600a57
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5462390
Commit-Queue: Michael Savigny <msavigny@google.com>
Reviewed-by: Ben Segall <bentekkie@google.com>
changes/90/5462390/2
Michael Savigny 1 year ago committed by LUCI CQ
parent b7ed76a09d
commit 65faa11896

@ -274,30 +274,13 @@ def set_mac_defaults():
os.environ.setdefault("RBE_deps_cache_max_mb", "1024")
def is_win_ctop():
if not sys.platform.startswith("win"):
return False
import winreg
try:
bios = winreg.OpenKeyEx(winreg.HKEY_LOCAL_MACHINE,
r"SOFTWARE\\GWindows")
model = winreg.QueryValueEx(bios, "Model")
val = len(model) > 0 and model[0] == 'Google Compute Engine'
if bios:
winreg.CloseKey(bios)
return val
except OSError:
return False
def set_win_defaults():
# Enable the deps cache on windows. This makes a notable improvement
# in performance at the cost of a ~200MB cache file.
os.environ.setdefault("RBE_enable_deps_cache", "true")
if is_win_ctop():
# Reduce local resource fraction used to do local compile actions on
# windows, to try and prevent machine saturation.
os.environ.setdefault("RBE_local_resource_fraction", "0.05")
# Reduce local resource fraction used to do local compile actions on
# windows, to try and prevent machine saturation.
os.environ.setdefault("RBE_local_resource_fraction", "0.05")
def workspace_is_cog():

Loading…
Cancel
Save