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

Unified Diff: scripts/common/slave_alloc.py

Issue 1068263003: CrOS: Update public waterfall to auto-configure. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Updated from comments. Created 5 years, 8 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
Index: scripts/common/slave_alloc.py
diff --git a/scripts/common/slave_alloc.py b/scripts/common/slave_alloc.py
index 5fdc7ea4d45fd08031ef710a38c3d039df66a5ee..03d69d80d5f09d31fbea7e6389ca177519aea9c1 100644
--- a/scripts/common/slave_alloc.py
+++ b/scripts/common/slave_alloc.py
@@ -120,7 +120,7 @@ class SlaveAllocator(object):
enforce (bool): If True, raise an IOError if the state file does not
exist or a ValueError if it could not be loaded.
"""
- state = None
+ state = {}
ghost stip (do not use) 2015/04/10 22:47:04 why is this not None?
dnj 2015/04/12 15:20:38 This handles the case where a state file does not
path = path or self.DEFAULT_STATE_PATH
if not os.path.exists(path):
if enforce:

Powered by Google App Engine
This is Rietveld 408576698