Fix condition for verify_disks_lso_attached

This PR: red-hat-storage#10120 Broke original condition: if config.ENV_DATA.get("platform") != constants.BAREMETAL_PLATFORM: and changed to: if config.ENV_DATA.get("platform") in [ constants.BAREMETAL_PLATFORM, constants.HCI_BAREMETAL, ]: There should be `not in`. Which is fixed in this commit. Signed-off-by: Petr Balogh <[email protected]>
openshift-cherrypick-robot · Aug 2, 2024 · 37336e3 · 37336e3
1 parent 4e6a974
commit 37336e3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/ocs_ci/ocs/ui/base_ui.py b/ocs_ci/ocs/ui/base_ui.py
@@ -365,7 +365,7 @@ def check_element_text(self, expected_text, element="*", take_screenshot=False):
 
         Args:
             expected_text (string): The expected text.
-            element (str): element (default: *)
+            element (str): element
             take_screenshot (bool): if screenshot should be taken
 
         return:

diff --git a/ocs_ci/ocs/ui/deployment_ui.py b/ocs_ci/ocs/ui/deployment_ui.py
@@ -237,7 +237,7 @@ def install_lso_cluster(self):
             self.do_click(
                 locator=self.dep_loc["all_nodes_create_sc"], enable_screenshot=True
             )
-        if config.ENV_DATA.get("platform") in [
+        if config.ENV_DATA.get("platform") not in [
             constants.BAREMETAL_PLATFORM,
             constants.HCI_BAREMETAL,
         ]: