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

Unified Diff: scripts/slave/recipe_modules/swarming/api.py

Issue 2102153004: Revert "Add CIPD package support to swarming recipe api" (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 6 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/slave/recipe_modules/swarming/api.py
diff --git a/scripts/slave/recipe_modules/swarming/api.py b/scripts/slave/recipe_modules/swarming/api.py
index 4a6bbd11324f6fd0b1a467bd1b4258ff27ab5e5b..ceac2ef5b217dce86083142a9c3ace958beee04c 100644
--- a/scripts/slave/recipe_modules/swarming/api.py
+++ b/scripts/slave/recipe_modules/swarming/api.py
@@ -10,7 +10,7 @@ from recipe_engine import recipe_api
# Minimally supported version of swarming.py script (reported by --version).
-MINIMAL_SWARMING_VERSION = (0, 8, 6)
+MINIMAL_SWARMING_VERSION = (0, 4, 10)
def parse_time(value):
@@ -327,7 +327,7 @@ class SwarmingApi(recipe_api.RecipeApi):
}[platform]
def task(self, title, isolated_hash, shards=1, task_output_dir=None,
- extra_args=None, idempotent=None, cipd_packages=None):
+ extra_args=None, idempotent=None):
"""Returns a new SwarmingTask instance to run an isolated executable on
Swarming.
@@ -347,14 +347,6 @@ class SwarmingApi(recipe_api.RecipeApi):
extra_args: list of command line arguments to pass to isolated tasks.
idempotent: whether this task is considered idempotent. Defaults
to self.default_idempotent if not specified.
- cipd_packages: list of 3-tuples corresponding to CIPD packages needed for
- the task: ('path', 'package_name', 'version'), defined as follows:
- path: Path relative to the Swarming root dir in which to install
- the package.
- package_name: Name of the package to install,
- eg. "infra/tools/authutil/${platform}"
- version: Version of the package, either a package instance ID,
- ref, or tag key/value pair.
"""
if idempotent is None:
idempotent = self.default_idempotent
@@ -374,8 +366,7 @@ class SwarmingApi(recipe_api.RecipeApi):
idempotent=idempotent,
extra_args=extra_args,
collect_step=self._default_collect_step,
- task_output_dir=task_output_dir,
- cipd_packages=cipd_packages)
+ task_output_dir=task_output_dir)
def gtest_task(self, title, isolated_hash, test_launcher_summary_output=None,
extra_args=None, **kwargs):
@@ -516,10 +507,6 @@ class SwarmingApi(recipe_api.RecipeApi):
if task.user:
args.extend(['--user', task.user])
- if task.cipd_packages:
- for path, pkg, version in task.cipd_packages:
- args.extend(['--cipd-package', '%s:%s:%s' % (path, pkg, version)])
-
# What isolated command to trigger.
args.append(task.isolated_hash)
@@ -831,8 +818,7 @@ class SwarmingApi(recipe_api.RecipeApi):
]
if self.verbose:
args.append('--verbose')
- script_version = self.m.swarming_client.get_script_version('swarming.py')
- if script_version < (0, 5): # pragma: nocover
+ if self.m.swarming_client.get_script_version('swarming.py') < (0, 5):
args.extend(('--shards', str(task.shards)))
args.append(task.task_name)
else:
@@ -905,7 +891,7 @@ class SwarmingTask(object):
def __init__(self, title, isolated_hash, dimensions, env, priority,
shards, buildername, buildnumber, expiration, user, io_timeout,
hard_timeout, idempotent, extra_args, collect_step,
- task_output_dir, cipd_packages=None):
+ task_output_dir):
"""Configuration of a swarming task.
Args:
@@ -914,16 +900,6 @@ class SwarmingTask(object):
isolated_hash: hash of isolated file that describes all files needed to
run the task as well as command line to launch. See 'isolate' recipe
module.
- cipd_packages: list of 3-tuples corresponding to CIPD packages needed for
- the task: ('path', 'package_name', 'version'), defined as follows:
- path: Path relative to the Swarming root dir in which to install
- the package.
- package_name: Name of the package to install,
- eg. "infra/tools/authutil/${platform}"
- version: Version of the package, either a package instance ID,
- ref, or tag key/value pair.
- collect_step: callback that will be called to collect and processes
- results of task execution, signature is collect_step(task, **kwargs).
dimensions: key-value mapping with swarming dimensions that specify
on what Swarming slaves task can run. One important dimension is 'os',
which defines platform flavor to run the task on. See Swarming doc.
@@ -948,13 +924,15 @@ class SwarmingTask(object):
idempotent: True if the results from a previous task can be reused. E.g.
this task has no side-effects.
extra_args: list of command line arguments to pass to isolated tasks.
+ collect_step: callback that will be called to collect and processes
+ results of task execution, signature is collect_step(task, **kwargs).
task_output_dir: if defined, the directory where task results are placed
during the collect step.
"""
self._trigger_output = None
self.buildnumber = buildnumber
self.buildername = buildername
- self.cipd_packages = cipd_packages
+ self.task_output_dir = task_output_dir
self.collect_step = collect_step
self.dimensions = dimensions.copy()
self.env = env.copy()
@@ -967,7 +945,6 @@ class SwarmingTask(object):
self.priority = priority
self.shards = shards
self.tags = set()
- self.task_output_dir = task_output_dir
self.title = title
self.user = user

Powered by Google App Engine
This is Rietveld 408576698