Temporarily replace compute hard reboot by soft reboot test due to BZ1890895

Due to BZ1890895, test_reboot_computes_recovery is not stable. This test
was applying a hard reboot to the compute nodes. The test is replaced by
test_soft_reboot_computes_recovery until BZ1890895 is resolved

Change-Id: If2d68bfbfed3c1b149929ec05a8e83f3720cc174

tobiko/tests/faults/ha/test_cloud_recovery.py

@@ -68,9 +68,9 @@ class DisruptTripleoNodesTest(testtools.TestCase):
         cloud_disruptions.reset_all_controller_nodes()
         overcloud_health_checks()
 
-    def test_reboot_computes_recovery(self):
+    def test_soft_reboot_computes_recovery(self):
         overcloud_health_checks()
-        cloud_disruptions.reset_all_compute_nodes(hard_reset=True)
+        cloud_disruptions.reset_all_compute_nodes(hard_reset=False)
         # verify VM status is updated after reboot
         nova.wait_for_all_instances_status('SHUTOFF')
         # start all VM instance
@@ -78,6 +78,17 @@ class DisruptTripleoNodesTest(testtools.TestCase):
         nova.start_all_instances()
         overcloud_health_checks(passive_checks_only=True)
 
+    # TODO(eolivare): the following test is skipped due to rhbz#1890895
+    # def test_hard_reboot_computes_recovery(self):
+    #     overcloud_health_checks()
+    #     cloud_disruptions.reset_all_compute_nodes(hard_reset=True)
+    #     # verify VM status is updated after reboot
+    #     nova.wait_for_all_instances_status('SHUTOFF')
+    #     # start all VM instance
+    #     # otherwise sidecar containers will not run after computes reboot
+    #     nova.start_all_instances()
+    #     overcloud_health_checks(passive_checks_only=True)
+
     def test_reboot_controller_main_vip(self):
         overcloud_health_checks()
         cloud_disruptions.reset_controller_main_vip()