summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2014-08-16 00:40:24 +0000
committerGerrit Code Review <review@openstack.org>2014-08-16 00:40:24 +0000
commit9a95e715eca6d483e7531c5cb47cfc44bf171ba9 (patch)
tree0297e9f778021d9475cf5dd100f80216eb95df9e
parente2259ea455273cdb71e86753cda776f3054ecc06 (diff)
parentf3348a7620eab97ee473f53321206a5ccc158202 (diff)
downloadzuul-9a95e715eca6d483e7531c5cb47cfc44bf171ba9.tar.gz
Merge "Update gerrit change attributes even if merged"
-rw-r--r--zuul/trigger/gerrit.py8
1 files changed, 4 insertions, 4 deletions
diff --git a/zuul/trigger/gerrit.py b/zuul/trigger/gerrit.py
index 754c94270..4d4deb8b6 100644
--- a/zuul/trigger/gerrit.py
+++ b/zuul/trigger/gerrit.py
@@ -364,6 +364,10 @@ class Gerrit(object):
change.is_current_patchset = False
change.is_merged = self._isMerged(change)
+ change.approvals = data['currentPatchSet'].get('approvals', [])
+ change.open = data['open']
+ change.status = data['status']
+
if change.is_merged:
# This change is merged, so we don't need to look any further
# for dependencies.
@@ -386,10 +390,6 @@ class Gerrit(object):
if not dep.is_merged and dep.is_current_patchset:
change.needed_by_changes.append(dep)
- change.approvals = data['currentPatchSet'].get('approvals', [])
- change.open = data['open']
- change.status = data['status']
-
return change
def getGitUrl(self, project):