tests/storage/linstor: Added test_linstor_sr_expand_host to validate the expansion of Linstor SR by integrating a new host into the storage pool.

The test:
- Ensures the new host has available disks and installs required Linstor packages.
- Configures LVM and integrates it into Linstor SR pool.
- Expands SR capacity and validates the increase.
- Handles cleanup in case of failure, including removing the node, ejecting the host, and uninstalling Linstor packages.

Signed-off-by: Rushikesh Jadhav <[email protected]>

Author: rushikeshjadhav

tests/storage/linstor/test_linstor_sr.py

@@ -2,7 +2,7 @@
 import pytest
 import time
 
-from .conftest import GROUP_NAME, LINSTOR_PACKAGE
+from .conftest import GROUP_NAME, LINSTOR_PACKAGE, LINSTOR_RELEASE_PACKAGE
 from lib.commands import SSHCommandFailed
 from lib.common import wait_for, vm_image
 from tests.storage import vdi_is_open
@@ -137,6 +137,95 @@ def test_linstor_sr_expand_disk(self, linstor_sr, provisioning_type, storage_poo
         # Ensure VM is able to start and shutdown on expanded SR
         self.test_start_and_shutdown_VM(vm)
 
+    @pytest.mark.small_vm
+    def test_linstor_sr_expand_host(self, linstor_sr, host, hostB1, provisioning_type,
+                                    storage_pool_name, vm_on_linstor_sr):
+        """
+        Join new host in the same pool, detect LINSTOR packages,
+        detect free disks, create LVM, and integrate it into LINSTOR SR.
+        """
+        sr = linstor_sr
+        vm = vm_on_linstor_sr
+        vm.start()
+        sr_size = sr.pool.master.xe('sr-param-get', {'uuid': sr.uuid, 'param-name': 'physical-size'})
+        resized = False
+        # Ensure that its a single host pool and not multi host pool
+        assert len(hostB1.pool.hosts) == 1, "This test requires second host to be a single host"
+        # Ensure that the host has disks available to use, we do not care about disks symmetry across pool
+        available_disks = hostB1.available_disks()
+        assert len(available_disks) >= 1, "This test requires second host to have free disk(s)"
+        if not hostB1.is_package_installed(LINSTOR_PACKAGE):
+            logging.info(f"Installing {LINSTOR_PACKAGE} on host {hostB1}...")
+            hostB1.yum_install([LINSTOR_RELEASE_PACKAGE])
+            hostB1.yum_install([LINSTOR_PACKAGE], enablerepo="xcp-ng-linstor-testing")
+            # Needed because the linstor driver is not in the xapi sm-plugins list
+            # before installing the LINSTOR packages.
+            hostB1.ssh(["systemctl", "restart", "multipathd"])
+            hostB1.restart_toolstack(verify=True)
+
+        devices = [f"/dev/{disk}" for disk in available_disks]
+
+        for disk in available_disks:
+            logging.info("* Disk is {}*".format(disk))
+            device = "/dev/" + disk
+            hostB1.ssh(['pvcreate', '-ff', '-y', device])
+
+        hostB1.ssh(['vgcreate', GROUP_NAME] + devices)
+
+        sr_group_name = "xcp-sr-" + storage_pool_name.replace("/", "_")
+        hostname_hostB1 = hostB1.xe('host-param-get', {'uuid': hostB1.uuid,
+                                    'param-name': 'name-label'})
+
+        controller_option = "--controllers="
+        for member in host.pool.hosts:
+            controller_option += f"{member.hostname_or_ip},"
+
+        hostB1_pool = hostB1.pool # Saving the hostB1 pool info before overwrite in join_pool.
+        try:
+            logging.info(f"Join host {hostB1} to pool {host}.")
+            # This will cause hostB1 pool to overwrite itself as host.pool creating issues on next run.
+            hostB1.join_pool(host.pool)
+            logging.info(f"Current list of linstor nodes.")
+            logging.info(host.ssh_with_result(["linstor", controller_option, "node", "list"]).stdout)
+            logging.info(f"Creating linstor node")
+            host.ssh(["linstor", controller_option, "node", "create", "--node-type", "combined",
+                      "--communication-type", "plain", hostname_hostB1, hostB1.hostname_or_ip]) # Linstor Node Create
+            logging.info(hostB1.ssh_with_result(['systemctl', 'restart', 'linstor-satellite.service']).stdout)
+            time.sleep(45) # Wait for node to come online
+            logging.info(f"New list of linstor nodes.")
+            logging.info(host.ssh_with_result(["linstor", controller_option, "node", "list"]).stdout)
+            logging.info(f"Expanding with linstor node")
+
+            if provisioning_type == "thin":
+                hostB1.ssh(['lvcreate', '-l', '+100%FREE', '-T', storage_pool_name])
+                host.ssh_with_result(["linstor", controller_option, "storage-pool", "create", "lvmthin",
+                                      hostname_hostB1, sr_group_name, storage_pool_name]).stdout # Expand linstor
+            else:
+                host.ssh_with_result(["linstor", controller_option, "storage-pool", "create", "lvm",
+                                      hostname_hostB1, sr_group_name, storage_pool_name]).stdout # Expand linstor
+        except Exception as e:
+            logging.info("* Exception: {}*".format(e))
+            host.ssh(["linstor", controller_option, "node", "delete", hostname_hostB1]) # Linstor Node Delete
+            host.pool.eject_host(hostB1)
+            hostB1.ssh(['vgremove', '-y', GROUP_NAME])
+            hostB1.ssh(['pvremove', '-y'] + devices) # Device cleanup
+            hostB1.yum_remove([LINSTOR_PACKAGE]) # Package cleanup
+
+        resized = True
+        sr.scan()
+        new_sr_size = sr.pool.master.xe('sr-param-get', {'uuid': sr.uuid, 'param-name': 'physical-size'})
+        assert int(new_sr_size) > int(sr_size) and resized is True, \
+            f"Expected SR size to increase but got old size: {sr_size}, new size: {new_sr_size}"
+        logging.info(f"* SR expansion completed from {sr_size} to {new_sr_size}*")
+        vm.shutdown(verify=True)
+        # Ensure VM is able to start and shutdown on expanded SR
+        self.test_start_and_shutdown_VM(vm)
+
+        host.ssh_with_result(["linstor", controller_option, "node", "delete", hostname_hostB1]).stdout
+        host.pool.eject_host(hostB1)
+        hostB1.pool = hostB1_pool # Post eject, reset hostB1.pool for next run ("thick")
+        hostB1.yum_remove([LINSTOR_PACKAGE]) # Package cleanup
+
     # *** tests with reboots (longer tests).
 
     @pytest.mark.reboot