openstack/python-tackerclient
Code Issues Proposed changes

Merge "Add client side support for error_reason"

Browse Source
This commit is contained in:
Jenkins
2016-06-18 13:39:28 +00:00
committed by Gerrit Code Review
parent 67c034b8a1 fc363ee49a
commit 8605e2d254
2 changed files with 11 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tackerclient/tacker/v1_0/vm/vnf.py
View File

@@ -26,7 +26,7 @@ class ListVNF(tackerV10.ListCommand):
resource = _VNF
list_columns = ['id', 'name', 'description', 'mgmt_url', 'status',
'vim_id', 'placement_attr']
'vim_id', 'placement_attr', 'error_reason']
class ShowVNF(tackerV10.ShowCommand):

11
tackerclient/v1_0/client.py
View File

@@ -30,6 +30,7 @@ from tackerclient.i18n import _
_logger = logging.getLogger(__name__)
DEFAULT_DESC_LENGTH = 25
DEFAULT_ERROR_REASON_LENGTH = 100
def exception_handler_v10(status_code, error_content):
@@ -390,7 +391,15 @@ class Client(ClientBase):
@APIParamsCall
def list_vnfs(self, retrieve_all=True, **_params):
return self.list('vnfs', self.vnfs_path, retrieve_all, **_params)
vnfs = self.list('vnfs', self.vnfs_path, retrieve_all, **_params)
for vnf in vnfs['vnfs']:
error_reason = vnf.get('error_reason', None)
if error_reason and \
len(error_reason) > DEFAULT_ERROR_REASON_LENGTH:
vnf['error_reason'] = error_reason[
:DEFAULT_ERROR_REASON_LENGTH]
vnf['error_reason'] += '...'
return vnfs
@APIParamsCall
def show_vnf(self, vnf, **_params):