Use StrictVersion class to avoid wrong version comparisons that happen in some cases using LooseVersion class which results in TypeError (#10178)

* Use StrictVersion class to avoid wrong version comparisons that happen in some cases using LooseVersion class

* Refactor code

* Add changelog for PR number 10178

* Update changelog to be more precise

* Use LooseVersion instead of StrictVersion to check cobbler's version in cobbler system module

* Update PR 10178 changelog description to be more accurate

Author: umiruka

changelogs/fragments/10178-update-minimum-version-number-to-avoid-wrong-comparisons-cobbler-xmlrpc.yml

@@ -0,0 +1,2 @@
+bugfixes:
+  - cobbler_system - update minimum version number to avoid wrong comparisons that happen in some cases using LooseVersion class which results in TypeError (https://github.com/ansible-collections/community.general/issues/8506, https://github.com/ansible-collections/community.general/pull/10145, https://github.com/ansible-collections/community.general/pull/10178).

plugins/modules/cobbler_system.py

@@ -163,7 +163,6 @@
 )
 from ansible_collections.community.general.plugins.module_utils.version import LooseVersion
 
-
 IFPROPS_MAPPING = dict(
     bondingopts='bonding_opts',
     bridgeopts='bridge_opts',
@@ -280,8 +279,8 @@ def main():
 
         if system:
             # Update existing entry
-            system_id = None
-            if LooseVersion(str(conn.version())) >= LooseVersion('3.4.0'):
+            system_id = ''
+            if LooseVersion(str(conn.version())) >= LooseVersion('3.4'):
                 system_id = conn.get_system_handle(name)
             else:
                 system_id = conn.get_system_handle(name, token)