1. Update isilon-python sdk to 0.6.0 2. Fix info module issue

Signed-off-by: Luis Liu <[email protected]>

Author: vangork

plugins/module_utils/storage/dell/shared_library/cluster.py

@@ -63,3 +63,16 @@ def get_cluster_owner_details(self):
             error_message = f"Failed to get the details of cluster owner details with error: {utils.determine_error(e)}"
             LOG.error(error_message)
             self.module.fail_json(msg=error_message)
+
+    def get_maintenance_settings_details(self):
+        """
+        Get cluster maintenance settings
+        :return: Cluster maintenance settings details.
+        """
+        try:
+            cluster_maintenance_settings = self.cluster_api.get_maintenance_settings()
+            return cluster_maintenance_settings.to_dict()
+        except Exception as e:
+            error_message = f"Failed to get the details of cluster maintenance settings with error: {utils.determine_error(e)}"
+            LOG.error(error_message)
+            self.module.fail_json(msg=error_message)

plugins/module_utils/storage/dell/utils.py

@@ -114,7 +114,7 @@ def get_powerscale_management_host_parameters():
 def get_powerscale_connection(module_params):
     if HAS_POWERSCALE_SDK:
         if isi_sdk.__name__ == "isilon_sdk":
-            conn = isi_sdk.v9_5_0.Configuration()
+            conn = isi_sdk.v9_10_0.Configuration()
         else:
             conn = isi_sdk.Configuration()
         if module_params['port_no'] is not None:
@@ -126,7 +126,7 @@ def get_powerscale_connection(module_params):
         conn.username = module_params['api_user']
         conn.password = module_params['api_password']
         if isi_sdk.__name__ == "isilon_sdk":
-            api_client = isi_sdk.v9_5_0.ApiClient(conn)
+            api_client = isi_sdk.v9_10_0.ApiClient(conn)
         else:
             api_client = isi_sdk.ApiClient(conn)
         return api_client
@@ -266,12 +266,14 @@ def validate_module_pre_reqs(module_params):
 ''' Import compatible powerscale sdk based on onefs version '''
 
 
-def import_powerscale_sdk(sdk):
+def import_powerscale_sdk(sdk, major, minor):
     try:
         global isi_sdk
         global ApiException
         global HAS_POWERSCALE_SDK
         isi_sdk = importlib.import_module(sdk)
+        isi_sdk.major = major
+        isi_sdk.minor = minor
         ApiException = getattr(importlib.import_module(sdk + ".rest"),
                                'ApiException')
         HAS_POWERSCALE_SDK = True
@@ -291,7 +293,7 @@ def find_compatible_powerscale_sdk(module_params):
                            if pkg.key.startswith("isilon-sdk")]
     if powerscale_packages:
         powerscale_sdk = powerscale_packages[0].key.replace('-', '_')
-        import_powerscale_sdk(powerscale_sdk + ".v9_5_0")
+        import_powerscale_sdk(powerscale_sdk + ".v9_10_0", 9, 10)
         try:
             HAS_POWERSCALE_SDK = True
             api_client = get_powerscale_connection(module_params)
@@ -300,11 +302,8 @@ def find_compatible_powerscale_sdk(module_params):
             minor = str(parse_version(cluster_api.get_cluster_config().to_dict()['onefs_version']['release'].split('.')[1]))
             array_version = major + "_" + minor + "_0"
 
-            if int(minor) >= 5:
-                compatible_powerscale_sdk = "isilon_sdk.v9_5_0"
-            else:
-                compatible_powerscale_sdk = "isilon_sdk.v" + array_version
-            import_powerscale_sdk(compatible_powerscale_sdk)
+            compatible_powerscale_sdk = "isilon_sdk.v" + array_version
+            import_powerscale_sdk(compatible_powerscale_sdk, int(major), int(minor))
 
         except Exception as e:
             HAS_POWERSCALE_SDK = False

plugins/modules/info.py

@@ -3270,8 +3270,8 @@ def __init__(self):
 
         self.api_client = utils.get_powerscale_connection(self.module.params)
         self.isi_sdk = utils.get_powerscale_sdk()
-        self.major = int(str(self.isi_sdk)[21])
-        self.minor = int(str(self.isi_sdk)[23])
+        self.major = self.isi_sdk.major
+        self.minor = self.isi_sdk.minor
         LOG.info('Got python SDK instance for provisioning on PowerScale ')
 
         self.cluster_api = self.isi_sdk.ClusterApi(self.api_client)
@@ -4002,6 +4002,13 @@ def get_support_assist_settings(self):
             return SupportAssist(self.support_assist_api, self.module).get_support_assist_settings()
         else:
             return {}
+    
+    def get_maintenance_settings(self):
+        """Get maintenance settings based on the version."""
+        if self.major > 9 or (self.major == 9 and self.minor > 9):
+            return Cluster(self.cluster_api, self.module).get_maintenance_settings_details()
+        else:
+            return Events(self.event_api, self.module).get_event_maintenance()
 
     def perform_module_operation(self):
         """Perform different actions on Gatherfacts based on user parameter chosen in playbook"""
@@ -4109,7 +4116,7 @@ def perform_module_operation(self):
             'server_certificate': lambda: Certificate(self.certificate_api, self.module).get_server_certificate_with_default(),
             'roles': lambda: Auth(self.auth_api, self.module).get_auth_roles(access_zone),
             'support_assist_settings': self.get_support_assist_settings,
-            'alert_settings': lambda: Events(self.event_api, self.module).get_event_maintenance(),
+            'alert_settings': self.get_maintenance_settings,
             'alert_rules': lambda: Events(self.event_api, self.module).get_alert_rules(),
             'alert_categories': lambda: Events(self.event_api, self.module).get_alert_categories(),
             'alert_channels': lambda: Events(self.event_api, self.module).get_event_channels(),

requirements.txt

@@ -1 +1 @@
-isilon-sdk==0.3.0.1
+isilon-sdk==0.6.0