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

Unified Diff: scripts/slave/recipes/run_presubmit.py

Issue 2463653002: Stop supporting old Gerrit CQ tryjob properties. (Closed)
Patch Set: Created 4 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 | « scripts/slave/recipes/boringssl_docs.expected/boringssl-docs-gerrit-deprecated.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipes/run_presubmit.py
diff --git a/scripts/slave/recipes/run_presubmit.py b/scripts/slave/recipes/run_presubmit.py
index 44f249a68b5af328679a444c7515044b6380dc48..02fd32cc73d4a6331cd42a4dba04a2b567c46c45 100644
--- a/scripts/slave/recipes/run_presubmit.py
+++ b/scripts/slave/recipes/run_presubmit.py
@@ -64,14 +64,8 @@ def _RunStepsInternal(api):
presubmit_args.extend(['--rietveld_email', '']) # activate anonymous mode
elif patch_storage == 'gerrit':
gerrit_url = api.properties.get('patch_gerrit_url')
Michael Achenbach 2016/11/07 12:54:00 Same as in another CL: Does it make sense to use g
tandrii(chromium) 2016/11/07 13:01:08 sorry. Simplified.
- if not gerrit_url:
- # TODO(tandrii): clean up old Gerrit patch properties.
- # Field event.patchSet.ref looks like 'refs/changes/11/338811/4'
- issue, patchset = api.properties['event.patchSet.ref'].split('/')[-2:]
- gerrit_url = api.properties['gerrit']
- else:
- issue = api.properties.get('patch_issue')
- patchset = api.properties.get('patch_set')
+ issue = api.properties.get('patch_issue')
+ patchset = api.properties.get('patch_set')
presubmit_args = [
'--issue', issue,
'--patchset', patchset,
@@ -173,18 +167,6 @@ def GenTests(api):
)
yield (
- api.test('infra_with_runhooks_and_gerrit_deprecated') +
Michael Achenbach 2016/11/07 12:54:00 Missing deletion of the file?
tandrii(chromium) 2016/11/07 13:01:08 Done.
- api.properties.tryserver_gerrit(
- full_project_name='infra/infra',
- repo_name='infra',
- mastername='tryserver.infra',
- buildername='infra_presubmit',
- runhooks=True) +
- api.step_data('presubmit', api.json.output([['infra_presubmit',
- ['compile']]]))
- )
-
- yield (
api.test('infra_with_runhooks_and_gerrit') +
api.properties.tryserver(
gerrit_project='infra/infra',
« no previous file with comments | « scripts/slave/recipes/boringssl_docs.expected/boringssl-docs-gerrit-deprecated.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698