openstack/python-ceilometerclient
Code Issues Proposed changes

Merge "make aggregation-method argument as a mandatory field"

Browse Source
This commit is contained in:
Jenkins
2016-02-24 16:09:37 +00:00
committed by Gerrit Code Review
parent 75aa9a6ed4 63d629f423
commit f28adc7a83
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
ceilometerclient/v2/shell.py
View File

@@ -523,6 +523,7 @@ def common_alarm_gnocchi_arguments(rule_namespace, create=False):
help='Number of periods to evaluate over.') help='Number of periods to evaluate over.')
@utils.arg('--aggregation-method', metavar='<AGGREATION>', @utils.arg('--aggregation-method', metavar='<AGGREATION>',
dest=rule_namespace + '/aggregation_method', dest=rule_namespace + '/aggregation_method',
required=create,
help=('Aggregation method to use, one of: ' + help=('Aggregation method to use, one of: ' +
str(GNOCCHI_AGGREGATION) + '.')) str(GNOCCHI_AGGREGATION) + '.'))
@utils.arg('--comparison-operator', metavar='<OPERATOR>', @utils.arg('--comparison-operator', metavar='<OPERATOR>',