zuul
/
zuul
Code Issues Proposed changes

Merge "Rename project to project_name in getPullBySha"

This commit is contained in:
Zuul 2019-02-19 18:51:34 +00:00 committed by Gerrit Code Review
parent 40feb0a13d 06ccb71dca
commit 8d40ea3b3a
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
zuul/driver/github/githubconnection.py
View File

@ -1181,19 +1181,18 @@ class GithubConnection(BaseConnection):
return True
def getPullBySha(self, sha, project, log):
cached_pr_numbers = self._sha_pr_cache.get(project, sha)
def getPullBySha(self, sha, project_name, log):
cached_pr_numbers = self._sha_pr_cache.get(project_name, sha)
if len(cached_pr_numbers) > 1:
raise Exception('Multiple pulls found with head sha %s' % sha)
if len(cached_pr_numbers) == 1:
for pr in cached_pr_numbers:
pr_body, pr_obj = self.getPull(project, pr, log)
pr_body, pr_obj = self.getPull(project_name, pr, log)
return pr_body
pulls = []
project_name = project
owner, project = project.split('/')
github = self.getGithubClient("%s/%s" % (owner, project))
github = self.getGithubClient(project_name)
owner, project = project_name.split('/')
repo = github.repository(owner, project)
for pr in repo.pull_requests(state='open'):
pr_dict = pr.as_dict()
@ -1204,7 +1203,7 @@ class GithubConnection(BaseConnection):
continue
pulls.append(pr_dict)
log.debug('Got PR on project %s for sha %s', project, sha)
log.debug('Got PR on project %s for sha %s', project_name, sha)
self.log_rate_limit(self.log, github)
if len(pulls) > 1:
raise Exception('Multiple pulls found with head sha %s' % sha)