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

Unified Diff: scripts/slave/recipe_modules/chromite/config.py

Issue 1250393003: CrOS: Update Chromite recipe module. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Fix coverage. Created 5 years, 4 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 | « scripts/slave/recipe_modules/chromite/api.py ('k') | scripts/slave/recipe_modules/chromite/example.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/chromite/config.py
diff --git a/scripts/slave/recipe_modules/chromite/config.py b/scripts/slave/recipe_modules/chromite/config.py
index 8a950e1dc0abb701bb0c246b7085326303f6945b..310e32dae2cc1e5ac372ac5b36a8b53126960408 100644
--- a/scripts/slave/recipe_modules/chromite/config.py
+++ b/scripts/slave/recipe_modules/chromite/config.py
@@ -13,13 +13,14 @@ import DEPS
path_api = DEPS['path'].api
-def BaseConfig(**_kwargs):
+def BaseConfig(CBB_CONFIG=None, CBB_BRANCH=None, CBB_BUILD_NUMBER=None,
+ CBB_DEBUG=False, CBB_CLOBBER=False, **_kwargs):
return ConfigGroup(
# Base mapping of repository key to repository name.
repositories = Dict(value_type=Set(basestring)),
# Checkout Chromite at this branch. "origin/" will be prepended.
- chromite_branch = Single(basestring),
+ chromite_branch = Single(basestring, empty_val=CBB_BRANCH or 'master'),
# Should the Chrome version be supplied to cbuildbot?
use_chrome_version = Single(bool),
@@ -29,6 +30,9 @@ def BaseConfig(**_kwargs):
read_cros_manifest = Single(bool),
cbb = ConfigGroup(
+ # The Chromite configuration to use.
+ config = Single(basestring, empty_val=CBB_CONFIG),
+
# The buildroot directory name to use.
builddir = Single(basestring),
@@ -36,7 +40,7 @@ def BaseConfig(**_kwargs):
build_id = Single(basestring),
# If supplied, forward to cbuildbot as '--buildnumber'.
- build_number = Single(int),
+ build_number = Single(int, empty_val=CBB_BUILD_NUMBER),
# If supplied, forward to cbuildbot as '--chrome-rev'.
chrome_rev = Single(basestring),
@@ -45,14 +49,22 @@ def BaseConfig(**_kwargs):
chrome_version = Single(basestring),
# If True, add cbuildbot flag: '--debug'.
- debug = Single(bool),
+ debug = Single(bool, empty_val=CBB_DEBUG),
# If True, add cbuildbot flag: '--clobber'.
- clobber = Single(bool),
+ clobber = Single(bool, empty_val=CBB_CLOBBER),
# The (optional) configuration repository to use.
config_repo = Single(basestring),
),
+
+ # A list of branches whose Chromite version is "old". Old Chromite
+ # buildbot commands reside in the "buildbot" subdirectory of the Chromite
+ # repository instead of the "bin".
+ old_chromite_branches = Set(basestring),
+
+ # A list of branches whose builders should not use a shared buildroot.
+ non_shared_root_branches = Set(basestring),
)
config_ctx = config_item_context(BaseConfig)
@@ -63,7 +75,15 @@ def base(c):
c.repositories['tryjob'] = []
c.repositories['chromium'] = []
c.repositories['cros_manifest'] = []
- c.chromite_branch = 'master'
+
+ c.old_chromite_branches.update((
+ 'firmware-uboot_v2-1299.B',
+ 'factory-1412.B',
+ ))
+ c.non_shared_root_branches.update(c.old_chromite_branches)
+ c.non_shared_root_branches.update((
+ 'factory-2305.B',
+ ))
# If running on a testing slave, enable "--debug" so Chromite doesn't cause
# actual production effects.
« no previous file with comments | « scripts/slave/recipe_modules/chromite/api.py ('k') | scripts/slave/recipe_modules/chromite/example.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698