openstack/python-tackerclient
Code Issues Proposed changes

Merge "Improved display of vnfPkgId for v2 Tacker"

Browse Source
This commit is contained in:
Zuul 2024-12-17 00:52:35 +00:00 committed by Gerrit Code Review
commit a8f4bcbdf7
2 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tackerclient/osc/v1/vnflcm/vnflcm.py
View File

@ -31,8 +31,7 @@ _attr_map = (
('vnfProvider', 'VNF Provider', tacker_osc_utils.LIST_BOTH),
('vnfSoftwareVersion', 'VNF Software Version', tacker_osc_utils.LIST_BOTH),
('vnfProductName', 'VNF Product Name', tacker_osc_utils.LIST_BOTH),
('vnfdId', 'VNFD ID', tacker_osc_utils.LIST_BOTH),
('vnfPkgId', 'VNF Package ID', tacker_osc_utils.LIST_BOTH)
('vnfdId', 'VNFD ID', tacker_osc_utils.LIST_BOTH)
)
LOG = logging.getLogger(__name__)
@ -68,8 +67,7 @@ def _get_columns(vnflcm_obj, action=None):
'vnfdVersion': 'VNFD Version',
'instantiationState': 'Instantiation State',
'_links': 'Links',
'vnfConfigurableProperties': 'VNF Configurable Properties',
'vnfPkgId': 'VNF Package ID',
'vnfConfigurableProperties': 'VNF Configurable Properties'
}
if action == 'show':
if vnflcm_obj['instantiationState'] == 'INSTANTIATED':
@ -80,6 +78,12 @@ def _get_columns(vnflcm_obj, action=None):
{'vimConnectionInfo': 'VIM Connection Info',
'_links': 'Links'}
)
# Note: To prevent it from appearing in the v2 API output,
# the 'VNF Package ID' will be output only if the vnfPkgId exists.
if 'vnfPkgId' in vnflcm_obj:
column_map.update(
{'vnfPkgId': 'VNF Package ID'}
)
return sdk_utils.get_osc_show_columns_for_sdk_resource(vnflcm_obj,
column_map)

2
tackerclient/tests/unit/osc/v1/test_vnflcm.py
View File

@ -56,7 +56,7 @@ def _get_columns_vnflcm(action='create'):
if action == 'list':
columns = [ele for ele in columns if ele not in
['VNFD Version', 'VNF Instance Description',
'VNF Configurable Properties']]
'VNF Configurable Properties', 'VNF Package ID']]
columns.remove('Links')
return columns