diff options
| author | Kyrylo Romanenko <kromanenko@mirantis.com> | 2016-10-06 16:02:09 +0300 |
|---|---|---|
| committer | Kyrylo Romanenko <kromanenko@mirantis.com> | 2016-10-07 11:02:53 +0000 |
| commit | 42f9317360ac4a20daa2c5ef329d9fc99e8df680 (patch) | |
| tree | b49427ec8b1487fdd907ad628ec39d456355e1e5 | |
| parent | bfeecd50fdb8f1475d51ac662eb09514a1754345 (diff) | |
| download | python-openstackclient-42f9317360ac4a20daa2c5ef329d9fc99e8df680.tar.gz | |
Improve output of supported API versions
Sort supported versions properly for better look.
Change-Id: I6c2f5ecc04cf14ea5bf1b214cb303fcc9783af3f
Closes-Bug: #1630962
| -rw-r--r-- | openstackclient/shell.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/openstackclient/shell.py b/openstackclient/shell.py index 3971b6ef..be4b5283 100644 --- a/openstackclient/shell.py +++ b/openstackclient/shell.py @@ -93,10 +93,12 @@ class OpenStackShell(shell.OpenStackShell): mod_versions = getattr(mod, 'API_VERSIONS', None) if not skip_old_check and mod_versions: if version_opt not in mod_versions: + sorted_versions = sorted( + mod.API_VERSIONS.keys(), + key=lambda s: list(map(int, s.split('.')))) self.log.warning( - "%s version %s is not in supported versions %s" - % (api, version_opt, - ', '.join(list(mod.API_VERSIONS.keys())))) + "%s version %s is not in supported versions: %s" + % (api, version_opt, ', '.join(sorted_versions))) # Command groups deal only with major versions version = '.v' + version_opt.replace('.', '_').split('_')[0] |