Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1978)

Unified Diff: bin/cbuildbot.py

Issue 3745006: Fix issue with stale cache from bad builds. (Closed) Base URL: http://git.chromium.org/git/crosutils.git
Patch Set: Nit Created 10 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | cros_mark_all_as_stable » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: bin/cbuildbot.py
diff --git a/bin/cbuildbot.py b/bin/cbuildbot.py
index 37a8b6e2691943905479a02594c9e510f9bed561..f000a576c39a57d075e6e97cc1006406d78d844c 100755
--- a/bin/cbuildbot.py
+++ b/bin/cbuildbot.py
@@ -345,6 +345,14 @@ def _UprevCleanup(buildroot, error_ok=False):
'--tracking_branch="cros/master"', 'clean'],
cwd=cwd, error_ok=error_ok)
+ workon_packages = RunCommand(['./cros_workon', 'list'],
+ enter_chroot=True, cwd=cwd, error_ok=error_ok,
+ redirect_stdout=True)
+ if workon_packages:
+ Info('Cleaning up workon packages %s' % workon_packages)
+ RunCommand(['./cros_workon', 'stop'] + workon_packages.split(),
+ enter_chroot=True, cwd=cwd, error_ok=False)
+
def _UprevPush(buildroot):
"""Pushes uprev changes to the main line."""
@@ -439,7 +447,6 @@ def main():
# Master bot needs to check if the other slaves completed.
if cbuildbot_comm.HaveSlavesCompleted(config):
_UprevPush(buildroot)
- _UprevCleanup(buildroot)
else:
# At least one of the slaves failed or we timed out.
_UprevCleanup(buildroot)
@@ -449,7 +456,7 @@ def main():
if buildconfig['important']:
cbuildbot_comm.PublishStatus(cbuildbot_comm.STATUS_BUILD_COMPLETE)
- _UprevCleanup(buildroot)
+ _UprevCleanup(buildroot)
except:
# Send failure to master bot.
if not buildconfig['master'] and buildconfig['important']:
« no previous file with comments | « no previous file | cros_mark_all_as_stable » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698