openstack
/
tempest
Code Issues Proposed changes

Use --os-endpoint-type for Cinder CLI 

The --endpoint-type option has been deprecated in Cinder for several
years and was just recently removed. It was missed that the tempest lib
CLI code was still using that to specify the public URL. This updates
the Cinder CLI handling to use the more conventional --os-endpoint-type.

Change-Id: Icb81710a6b905d80c2e75b29be449dc029e01bdd
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
Closes-bug: #1783885
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
This commit is contained in:
Sean McGinnis 2018-07-26 15:27:31 -05:00
parent 5a501744d2
commit 2e354d7a03
No known key found for this signature in database
GPG Key ID: CE7EE4BFAF8D70C8
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
releasenotes/notes/cinder-use-os-endpoint-type-c11f63fd468ceb4c.yaml Normal file
View File

@ -0,0 +1,6 @@
---
upgrade:
- |
Cinder CLI calls have now been updated to use the ``--os-endpoint-type``
option instead of ``--endpoint-type``. The latter had been deprecated in
Cinder and has been removed in the Rocky release.

2
tempest/lib/cli/base.py
View File

@ -269,7 +269,7 @@ class CLIClient(object):
:param merge_stderr: if True the stderr buffer is merged into stdout
:type merge_stderr: boolean
"""
flags += ' --endpoint-type %s' % endpoint_type
flags += ' --os-endpoint-type %s' % endpoint_type
return self.cmd_with_auth(
'cinder', action, flags, params, fail_ok, merge_stderr)