]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph-volume: fix batch functional tests, idempotent test must check stderr 29689/head
authorJan Fajerski <jfajerski@suse.com>
Thu, 15 Aug 2019 11:28:42 +0000 (13:28 +0200)
committerJan Fajerski <jfajerski@suse.com>
Fri, 16 Aug 2019 08:42:15 +0000 (10:42 +0200)
Fixes: https://tracker.ceph.com/issues/41295
Signed-off-by: Jan Fajerski <jfajerski@suse.com>
(cherry picked from commit 88807110f3014e7a2a39c37ae5c64bb3ab107d80)

src/ceph-volume/ceph_volume/tests/functional/batch/playbooks/test.yml
src/ceph-volume/ceph_volume/tests/functional/batch/playbooks/test_explicit.yml

index c9375e2b7fb7ffa868fa1e4efe63ce1e6e039b49..1b0fe1f8cebfdbe0bea3f77742cb9cddcb1ce511 100644 (file)
@@ -46,7 +46,7 @@
         msg: "lvm batch failed idempotency check"
       when:
          - batch_cmd.rc != 0
-         - "'strategy changed' not in batch_cmd.stdout"
+         - "'strategy changed' not in batch_cmd.stderr"
 
     - name: run batch --report to see if devices get filtered
       command: "ceph-volume --cluster {{ cluster }} lvm batch --report --format=json --{{ osd_objectstore|default('bluestore') }} {{ '--dmcrypt' if dmcrypt|default(false) else '' }} {{ devices | join(' ') }}"
@@ -59,5 +59,5 @@
       fail:
         msg: "lvm batch --report failed idempotency check"
       when:
-         - batch_cmd.rc != 0
-         - "'strategy changed' not in batch_cmd.stdout"
+         - report_cmd.rc != 0
+         - "'strategy changed' not in report_cmd.stderr"
index 853f6a1cb6861d725c0acc0d672702b04a26c6e7..4c5cbfc9a3b3dbbb57ed06e069359508408f42cf 100644 (file)
@@ -46,7 +46,7 @@
         msg: "lvm batch failed idempotency check"
       when:
          - batch_cmd.rc != 0
-         - "'strategy changed' not in batch_cmd.stdout"
+         - "'strategy changed' not in batch_cmd.stderr"
 
     - name: run batch --report to see if devices get filtered
       command: "ceph-volume --cluster {{ cluster }} lvm batch --report --format=json --{{ osd_objectstore|default('bluestore') }} {{ '--dmcrypt' if dmcrypt|default(false) else '' }} {{ devices[:2] | join(' ') }} --db-devices {{ devices[2:] | join(' ') }}"
@@ -59,5 +59,5 @@
       fail:
         msg: "lvm batch --report failed idempotency check"
       when:
-         - batch_cmd.rc != 0
-         - "'strategy changed' not in batch_cmd.stdout"
+         - report_cmd.rc != 0
+         - "'strategy changed' not in report_cmd.stderr"