openstack/deb-python-senlinclient
Code Issues Proposed changes

Add short id support to cluster-policy-list

Browse Source 
We are supposed to support short IDs everywhere for consistency. This
patch fixes the cluster-policy-list command.

Change-Id: I5a040ff928d5d953674d66b86975788600c59f9b
Closes-Bug: #1520863
This commit is contained in:
tengqm 2015-11-28 22:04:05 -05:00
parent f027c04777
commit e4075653d2
1 changed files with 10 additions and 1 deletions

11
senlinclient/v1/shell.py

@ -854,6 +854,8 @@ def do_cluster_scale_in(sc, args):
help=_('Name of keys used for sorting the returned events.'))
@utils.arg('-s', '--sort-dir', metavar='<DIR>',
help=_('Direction for sorting, where DIR can be "asc" or "desc".'))
@utils.arg('-F', '--full-id', default=False, action="store_true",
help=_('Print full IDs in list.'))
@utils.arg('id', metavar='<CLUSTER>',
help=_('Name or ID of cluster to query on.'))
def do_cluster_policy_list(sc, args):
@ -880,7 +882,14 @@ def do_cluster_policy_list(sc, args):
sortby_index = 3
policies = sc.cluster_policies(cluster.id, **queries)
utils.print_list(policies, fields, sortby_index=sortby_index)
formatters = {}
if not args.full_id:
formatters = {
'policy_id': _short_id,
}
utils.print_list(policies, fields, formatters=formatters,
sortby_index=sortby_index)
@utils.arg('-p', '--policy', metavar='<POLICY>', required=True,