diff --git a/lower-constraints.txt b/lower-constraints.txt index 3555586d..b1f66ade 100644 --- a/lower-constraints.txt +++ b/lower-constraints.txt @@ -19,6 +19,7 @@ os-client-config==1.28.0 oslo.config==5.2.0 oslo.i18n==3.15.3 oslotest==3.2.0 +packaging==20.4 pbr==2.0.0 pyparsing==2.1.0 python-mimeparse==1.6.0 diff --git a/oslo_utils/versionutils.py b/oslo_utils/versionutils.py index 311965d2..db8ed0c3 100644 --- a/oslo_utils/versionutils.py +++ b/oslo_utils/versionutils.py @@ -19,7 +19,7 @@ Helpers for comparing version strings. .. versionadded:: 1.6 """ -import pkg_resources +import packaging.version import six from oslo_utils._i18n import _ @@ -39,17 +39,14 @@ def is_compatible(requested_version, current_version, same_major=True): True. :returns: True if compatible, False if not """ - requested_parts = pkg_resources.parse_version(requested_version) - current_parts = pkg_resources.parse_version(current_version) + requested = packaging.version.Version(requested_version) + current = packaging.version.Version(current_version) if same_major: - # NOTE(jlvillal) pkg_resources issues a warning if we try to access - # portions of the version, for example requested_parts[0] will issue a - # warning message. So get the major_version from the string instead. - if requested_version.split('.')[0] != current_version.split('.')[0]: + if requested.major != current.major: return False - return current_parts >= requested_parts + return current >= requested def convert_version_to_int(version): diff --git a/requirements.txt b/requirements.txt index 30aa6a8f..22d2b786 100644 --- a/requirements.txt +++ b/requirements.txt @@ -16,3 +16,4 @@ netaddr>=0.7.18 # BSD netifaces>=0.10.4 # MIT debtcollector>=1.2.0 # Apache-2.0 pyparsing>=2.1.0 # MIT +packaging>=20.4 # BSD