]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
tests: skip rbdmirror tests on non-secondary daemon
authorGuillaume Abrioux <gabrioux@redhat.com>
Mon, 1 Aug 2022 21:12:13 +0000 (23:12 +0200)
committerGuillaume Abrioux <gabrioux@redhat.com>
Wed, 3 Aug 2022 04:44:51 +0000 (06:44 +0200)
the daemon is not running on the 'primary' daemon.
Therefore, these tests are not needed.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit a12f67e1388b35974842f4fa8f7c56c99f56cd0a)

tests/conftest.py
tests/functional/tests/rbd-mirror/test_rbd_mirror.py

index 415f1031dfb0743283588d2cc2c7a68bb6932c2b..42e37ca1df59f560e99d10f68e87f97eb52132ae 100644 (file)
@@ -104,6 +104,7 @@ def node(host, request):
     docker = ansible_vars.get("docker")
     dashboard = ansible_vars.get("dashboard_enabled", True)
     radosgw_num_instances = ansible_vars.get("radosgw_num_instances", 1)
+    ceph_rbd_mirror_remote_user = ansible_vars.get('ceph_rbd_mirror_remote_user', '')
     ceph_release_num = {
         'jewel': 10,
         'kraken': 11,
@@ -129,6 +130,9 @@ def node(host, request):
             request.function, group_names)
         pytest.skip(reason)
 
+    if request.node.get_closest_marker('rbdmirror_secondary') and not ceph_rbd_mirror_remote_user:  # noqa E501
+        pytest.skip('Not a valid test for a non-secondary rbd-mirror node')
+
     if request.node.get_closest_marker('ceph_crash') and group_names in [['nfss'], ['iscsigws'], ['clients'], ['monitoring']]:
         pytest.skip('Not a valid test for nfs, client or iscsigw nodes')
 
index f72d0a505a4a11b305a59432ae84c13cac527366..eb5b7498167c89669b54f2c439883aa8fcf8791d 100644 (file)
@@ -4,10 +4,12 @@ import json
 
 class TestRbdMirrors(object):
 
+    @pytest.mark.rbdmirror_secondary
     @pytest.mark.no_docker
     def test_rbd_mirror_is_installed(self, node, host):
         assert host.package("rbd-mirror").is_installed
 
+    @pytest.mark.rbdmirror_secondary
     def test_rbd_mirror_service_enabled_and_running(self, node, host):
         service_name = "ceph-rbd-mirror@rbd-mirror.{hostname}".format(
             hostname=node["vars"]["inventory_hostname"]
@@ -16,6 +18,7 @@ class TestRbdMirrors(object):
         assert s.is_enabled
         assert s.is_running
 
+    @pytest.mark.rbdmirror_secondary
     def test_rbd_mirror_is_up(self, node, host, setup):
         hostname = node["vars"]["inventory_hostname"]
         cluster = setup["cluster_name"]