diff --git a/manilaclient/tests/unit/v2/fakes.py b/manilaclient/tests/unit/v2/fakes.py index 8d3473866..1d4b62b26 100644 --- a/manilaclient/tests/unit/v2/fakes.py +++ b/manilaclient/tests/unit/v2/fakes.py @@ -927,6 +927,7 @@ class FakeHTTPClient(fakes.FakeHTTPClient): return (200, {}, { 'share_type': {'id': 1, 'name': 'test-type-1', + 'description': "test share type desc", 'extra_specs': {'test': 'test'}, 'required_extra_specs': {'test': 'test'}}}) diff --git a/manilaclient/v2/shell.py b/manilaclient/v2/shell.py index 0a5646c40..4f410e2df 100644 --- a/manilaclient/v2/shell.py +++ b/manilaclient/v2/shell.py @@ -170,7 +170,7 @@ def _print_type_show(stype, default_share_type=None): 'name': stype.name, 'visibility': _is_share_type_public(stype), 'is_default': is_default, - 'description': None, + 'description': stype.description, 'required_extra_specs': _print_type_required_extra_specs(stype), 'optional_extra_specs': _print_type_optional_extra_specs(stype), } diff --git a/releasenotes/notes/bug-1830677-fix-13b30d6a89f43246.yaml b/releasenotes/notes/bug-1830677-fix-13b30d6a89f43246.yaml new file mode 100644 index 000000000..4195a60b5 --- /dev/null +++ b/releasenotes/notes/bug-1830677-fix-13b30d6a89f43246.yaml @@ -0,0 +1,6 @@ +--- +fixes: + - | + `Launchpad bug 1830677 `_ + has been fixed and the client now correctly get description of share type + instead of ``None``.