]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/cephadm: adjust host drain test to handle explicit placement warning 56092/head
authorAdam King <adking@redhat.com>
Mon, 6 Nov 2023 16:19:09 +0000 (11:19 -0500)
committerAdam King <adking@redhat.com>
Wed, 13 Mar 2024 02:42:25 +0000 (22:42 -0400)
Since we're adding a warning if any host is listed explicitly
in the placement of any service when removing the host,
we need to adjust the host drain test that removes a host
without the --force flag to not have the explicit hostname
in the placement for the mon service.

Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit b4db5e4ffcf0fb345c99986718b16853f76b148a)

qa/suites/orch/cephadm/workunits/task/test_host_drain.yaml

index 83173c9f8b22ee353ee789963562488b63bff738..c195bc052b7e326662314a1d80e96536308fffae 100644 (file)
@@ -28,6 +28,16 @@ tasks:
             HOST_C="${host}"
           fi
         done
+        # One last thing to worry about before draining the host
+        # is that the teuthology test tends to put the explicit
+        # hostnames in the placement for the mon service.
+        # We want to make sure we can drain without providing
+        # --force and there is a check for the host being removed
+        # being listed explicitly in the placements. Therefore,
+        # we should remove it from the mon placement.
+        ceph orch ls mon --export > mon.yaml
+        sed /"$HOST_C"/d mon.yaml > mon_adjusted.yaml
+        ceph orch apply -i mon_adjusted.yaml
         # now drain that host
         ceph orch host drain $HOST_C --zap-osd-devices
         # wait for drain to complete