diff options
author | Zuul <zuul@review.opendev.org> | 2023-04-29 21:20:01 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-04-29 21:20:01 +0000 |
commit | bbdbe81790f4926e5e00085309589a2c52e5230b (patch) | |
tree | ab93ab7d69cfd8cfa15afe164ff932b82408ec5e /zuul/driver/gerrit/gerritsource.py | |
parent | 8dcc69fbf0b20a9d50ca5dc0c2cbe866abc0bb3e (diff) | |
parent | 546ad5353a89d2fe7f47636ef781c44a6e8ff975 (diff) | |
download | zuul-bbdbe81790f4926e5e00085309589a2c52e5230b.tar.gz |
Merge "Add Gerrit pipeline trigger requirements"
Diffstat (limited to 'zuul/driver/gerrit/gerritsource.py')
-rw-r--r-- | zuul/driver/gerrit/gerritsource.py | 28 |
1 files changed, 12 insertions, 16 deletions
diff --git a/zuul/driver/gerrit/gerritsource.py b/zuul/driver/gerrit/gerritsource.py index b0bd3c448..4e7a32b83 100644 --- a/zuul/driver/gerrit/gerritsource.py +++ b/zuul/driver/gerrit/gerritsource.py @@ -20,7 +20,7 @@ from urllib.parse import urlparse from zuul.source import BaseSource from zuul.model import Project from zuul.driver.gerrit.gerritmodel import GerritRefFilter -from zuul.driver.util import scalar_or_list, to_list +from zuul.driver.util import scalar_or_list from zuul.lib.dependson import find_dependency_headers from zuul.zk.change_cache import ChangeKey @@ -209,21 +209,15 @@ class GerritSource(BaseSource): return self.connection._getGitwebUrl(project, sha) def getRequireFilters(self, config): - f = GerritRefFilter( - connection_name=self.connection.connection_name, - open=config.get('open'), - current_patchset=config.get('current-patchset'), - wip=config.get('wip'), - statuses=to_list(config.get('status')), - required_approvals=to_list(config.get('approval')), - ) + f = GerritRefFilter.requiresFromConfig( + self.connection.connection_name, + config) return [f] def getRejectFilters(self, config): - f = GerritRefFilter( - connection_name=self.connection.connection_name, - reject_approvals=to_list(config.get('approval')), - ) + f = GerritRefFilter.rejectFromConfig( + self.connection.connection_name, + config) return [f] def getRefForChange(self, change): @@ -247,11 +241,13 @@ def getRequireSchema(): 'current-patchset': bool, 'wip': bool, 'status': scalar_or_list(str)} - return require def getRejectSchema(): - reject = {'approval': scalar_or_list(approval)} - + reject = {'approval': scalar_or_list(approval), + 'open': bool, + 'current-patchset': bool, + 'wip': bool, + 'status': scalar_or_list(str)} return reject |