From 9f8a6e6a222ad3a06c3356c08927a5b8178c48fc Mon Sep 17 00:00:00 2001
From: Jeremy Stanley <fungi@yuggoth.org>
Date: Thu, 6 Mar 2014 03:02:39 +0000
Subject: [PATCH] Submitted is _not_ necessarily merged in Gerrit

* zuul/trigger/gerrit.py(Gerrit._isMerged): Only consider MERGED
status to indicate a change is successfully merged. SUBMITTED can
exist on changes which Gerrit was unable to merge due to nonexistent
parent/dependent commits, and possibly for other reasons too.

Change-Id: I673be5010031f8bc26ab31db9afcc7c28854db85
---
 zuul/trigger/gerrit.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/zuul/trigger/gerrit.py b/zuul/trigger/gerrit.py
index 48bf0dd231..f15d8bf514 100644
--- a/zuul/trigger/gerrit.py
+++ b/zuul/trigger/gerrit.py
@@ -236,7 +236,7 @@ class Gerrit(object):
         if not status:
             return False
         self.log.debug("Change %s status: %s" % (change, status))
-        if status == 'MERGED' or status == 'SUBMITTED':
+        if status == 'MERGED':
             return True
 
     def canMerge(self, change, allow_needs):