From 8bbea95f32b8073b9108b54eb80e24e71333d0fc Mon Sep 17 00:00:00 2001 From: xialinjuan Date: Fri, 22 Jan 2016 21:41:28 +0800 Subject: [PATCH] make aggregation-method argument as a mandatory field When create the gnocchi alarm, the aggregation-method is required. Closes-Bug: #1531602 Change-Id: I65b5038053d69dc727639fbfeaa5a23108845083 (cherry picked from commit 63d629f423441ac7d3435370e6108897f23f3486) --- ceilometerclient/v2/shell.py | 1 + 1 file changed, 1 insertion(+) diff --git a/ceilometerclient/v2/shell.py b/ceilometerclient/v2/shell.py index b1f87344..4395ecd0 100644 --- a/ceilometerclient/v2/shell.py +++ b/ceilometerclient/v2/shell.py @@ -519,6 +519,7 @@ def common_alarm_gnocchi_arguments(rule_namespace, create=False): help='Number of periods to evaluate over.') @utils.arg('--aggregation-method', metavar='', dest=rule_namespace + '/aggregation_method', + required=create, help=('Aggregation method to use, one of: ' + str(GNOCCHI_AGGREGATION) + '.')) @utils.arg('--comparison-operator', metavar='',