From a081c3d769b47a2213325c7ff2e0ce8352072198 Mon Sep 17 00:00:00 2001 From: Gil Forcada Date: Fri, 13 Jul 2018 12:52:30 +0200 Subject: [PATCH] Only run deps job on core packages --- src/mr.roboto/src/mr/roboto/subscriber.py | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/mr.roboto/src/mr/roboto/subscriber.py b/src/mr.roboto/src/mr/roboto/subscriber.py index 925c0b3..486a8fc 100644 --- a/src/mr.roboto/src/mr/roboto/subscriber.py +++ b/src/mr.roboto/src/mr/roboto/subscriber.py @@ -441,8 +441,12 @@ def run(self): """Trigger the dependencies jenkins job""" branch = self.pull_request['head']['ref'] - if self.repo_name == 'buildout.coredev': - self.log('Do not run dependencies jenkins job on buildout.coredev') + if not self.is_core_package(): + base_branch = self.pull_request['base']['ref'] + key = f'{self.repo_full_name}/{base_branch}' + self.log( + f'No dependencies jenkins job run on a non-core package {key}', + ) return settings = self.event.request.registry.settings @@ -465,6 +469,12 @@ def run(self): f'pull request {self.short_url}', ) + def is_core_package(self): + base_branch = self.pull_request['base']['ref'] + key = f'{self.repo_full_name}/{base_branch}' + sources = self.event.request.registry.settings['sources_file'] + return key in sources + @subscriber(MergedPullRequest) class UpdateCoredevCheckouts(PullRequestSubscriber):