Merge pull request #1185 from tobias-urdin/ci

New CI fixes
gnocchixyz · Feb 20, 2022 · 94ae8c0 · 94ae8c0
2 parents 1207c4a + 476702f
commit 94ae8c0
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 13 deletions.
diff --git a/.mergify.yml b/.mergify.yml
@@ -1,26 +1,46 @@
+queue_rules:
+  - name: default
+    conditions:
+    - check-success=doc (docs)
+    - check-success=doc (docs-gnocchi-web)
+    - check-success=check (pep8)
+    - check-success=test (py36, postgresql-file-upgrade-from-4.3)
+    - check-success=test (py36, mysql-file)
+    - check-success=test (py36, mysql-swift)
+    - check-success=test (py36, mysql-s3)
+    - check-success=test (py36, postgresql-file)
+    - check-success=test (py36, postgresql-swift)
+    - check-success=test (py36, postgresql-s3)
+    - check-success=test (py38, mysql-ceph-upgrade-from-4.3)
+    - check-success=test (py38, postgresql-file-upgrade-from-4.3)
+    - check-success=test (py38, mysql-file)
+    - check-success=test (py38, mysql-swift)
+    - check-success=test (py38, mysql-s3)
+    - check-success=test (py38, mysql-ceph)
+    - check-success=test (py38, postgresql-file)
+    - check-success=test (py38, postgresql-swift)
+    - check-success=test (py38, postgresql-s3)
+    - check-success=test (py38, postgresql-ceph)
+
 pull_request_rules:
   - name: automatic merge
     actions:
-      merge:
-        method: rebase
-        rebase_fallback: merge
-        strict: true
+      queue:
+        method: merge
+        name: default
     conditions:
     - label!=work-in-progress
     - '#approved-reviews-by>=1'
     - check-success=doc (docs)
     - check-success=doc (docs-gnocchi-web)
-
     - check-success=check (pep8)
-
     - check-success=test (py36, postgresql-file-upgrade-from-4.3)
     - check-success=test (py36, mysql-file)
     - check-success=test (py36, mysql-swift)
     - check-success=test (py36, mysql-s3)
     - check-success=test (py36, postgresql-file)
     - check-success=test (py36, postgresql-swift)
     - check-success=test (py36, postgresql-s3)
-
     - check-success=test (py38, mysql-ceph-upgrade-from-4.3)
     - check-success=test (py38, postgresql-file-upgrade-from-4.3)
     - check-success=test (py38, mysql-file)
@@ -34,10 +54,9 @@ pull_request_rules:
 
   - name: automatic merge backports from Mergify
     actions:
-      merge:
-        method: rebase
-        rebase_fallback: merge
-        strict: true
+      queue:
+        method: merge
+        name: default
     conditions:
     - base~=^stable/.*
     - label!=work-in-progress

diff --git a/gnocchi/rest/api.py b/gnocchi/rest/api.py
@@ -1296,7 +1296,7 @@ class QueryStringSearchAttrFilter(object):
     @classmethod
     def _parsed_query2dict(cls, parsed_query):
         result = None
-        while parsed_query:
+        while len(parsed_query.asList()):
             part = parsed_query.pop()
             if part in cls.binary_operator:
                 result = {part: {parsed_query.pop(): result}}

diff --git a/gnocchi/tests/functional/gabbits/aggregates-with-metric-ids.yaml b/gnocchi/tests/functional/gabbits/aggregates-with-metric-ids.yaml
@@ -645,7 +645,7 @@ tests:
         $.code: 400
         $.description.cause: "Invalid operations"
         $.description.reason: "/^Fail to parse the operations string/"
-        $.description.detail: '/^Expected "\)", found end of text  \(at char 15\), \(line:1, col:16\)/'
+        $.description.detail: /^Expected \'\)\', found end of text  \(at char 15\), \(line:1, col:16\)/
 
     - name: get invalid metric operations
       POST: /v1/aggregates