Browse Source

Merge "Move isChangeReadyToBeEnqueued after pipeline requirements"

tags/3.6.0
Zuul 5 months ago
parent
commit
e692bf530d
1 changed files with 5 additions and 5 deletions
  1. 5
    5
      zuul/manager/__init__.py

+ 5
- 5
zuul/manager/__init__.py View File

@@ -277,11 +277,6 @@ class PipelineManager(object):
277 277
                            "ignoring" % change)
278 278
             return True
279 279
 
280
-        if not self.isChangeReadyToBeEnqueued(change):
281
-            self.log.debug("Change %s is not ready to be enqueued, ignoring" %
282
-                           change)
283
-            return False
284
-
285 280
         if not ignore_requirements:
286 281
             for f in self.ref_filters:
287 282
                 if f.connection_name != change.project.connection_name:
@@ -293,6 +288,11 @@ class PipelineManager(object):
293 288
                                    "requirement %s" % (change, f))
294 289
                     return False
295 290
 
291
+        if not self.isChangeReadyToBeEnqueued(change):
292
+            self.log.debug("Change %s is not ready to be enqueued, ignoring" %
293
+                           change)
294
+            return False
295
+
296 296
         with self.getChangeQueue(change, change_queue) as change_queue:
297 297
             if not change_queue:
298 298
                 self.log.debug("Unable to find change queue for "

Loading…
Cancel
Save