diff options
| author | Jiří Suchomel <jsuchome@suse.cz> | 2015-04-16 16:04:09 +0200 |
|---|---|---|
| committer | Jiří Suchomel <jsuchome@suse.cz> | 2015-04-29 11:54:23 +0200 |
| commit | 4de6e14357d9ec7940d6f18d9be1c419b5db1550 (patch) | |
| tree | d1470164f3a3e0b612fc59588a593fbacffdd53b | |
| parent | 62e3573fceb6c6120d8c14a761927c2979d9c132 (diff) | |
| download | python-ceilometerclient-4de6e14357d9ec7940d6f18d9be1c419b5db1550.tar.gz | |
Allow changing project-id and user-id by alarm-threshold-update
Ceilometer alarm-threshold-update should support updating
project-id and user-id, but the update function was silently
ignoring changes of these options.
Change-Id: I33231bff44b90f3c58e5ef1646349b6dc63a3ce7
Closes-Bug: 1395932
| -rw-r--r-- | ceilometerclient/tests/unit/v2/test_alarms.py | 6 | ||||
| -rw-r--r-- | ceilometerclient/v2/alarms.py | 5 |
2 files changed, 7 insertions, 4 deletions
diff --git a/ceilometerclient/tests/unit/v2/test_alarms.py b/ceilometerclient/tests/unit/v2/test_alarms.py index 4239d2f..ce3e1c9 100644 --- a/ceilometerclient/tests/unit/v2/test_alarms.py +++ b/ceilometerclient/tests/unit/v2/test_alarms.py @@ -78,6 +78,7 @@ DELTA_ALARM_RULE = {u'comparison_operator': u'lt', DELTA_ALARM_TC = [{u'name': u'cons1', u'duration': 500}] DELTA_ALARM['time_constraints'] = DELTA_ALARM_TC +DELTA_ALARM['user_id'] = u'new-user-id' UPDATED_ALARM = copy.deepcopy(AN_ALARM) UPDATED_ALARM.update(DELTA_ALARM) UPDATED_ALARM['threshold_rule'].update(DELTA_ALARM_RULE) @@ -90,7 +91,7 @@ UPDATED_ALARM['time_constraints'] = [{u'name': u'cons1', DELTA_ALARM['threshold_rule'] = DELTA_ALARM_RULE UPDATE_ALARM = copy.deepcopy(UPDATED_ALARM) UPDATE_ALARM['remove_time_constraints'] = 'cons2' -del UPDATE_ALARM['user_id'] +UPDATE_ALARM['user_id'] = u'new-user-id' del UPDATE_ALARM['project_id'] del UPDATE_ALARM['name'] del UPDATE_ALARM['alarm_id'] @@ -128,11 +129,12 @@ DELTA_LEGACY_ALARM = {u'alarm_actions': ['url1', 'url2'], u'threshold': 42.1} DELTA_LEGACY_ALARM['time_constraints'] = [{u'name': u'cons1', u'duration': 500}] +DELTA_LEGACY_ALARM['user_id'] = u'new-user-id' DELTA_LEGACY_ALARM['remove_time_constraints'] = 'cons2' UPDATED_LEGACY_ALARM = copy.deepcopy(AN_LEGACY_ALARM) UPDATED_LEGACY_ALARM.update(DELTA_LEGACY_ALARM) UPDATE_LEGACY_ALARM = copy.deepcopy(UPDATED_LEGACY_ALARM) -del UPDATE_LEGACY_ALARM['user_id'] +UPDATE_LEGACY_ALARM['user_id'] = u'new-user-id' del UPDATE_LEGACY_ALARM['project_id'] del UPDATE_LEGACY_ALARM['name'] del UPDATE_LEGACY_ALARM['alarm_id'] diff --git a/ceilometerclient/v2/alarms.py b/ceilometerclient/v2/alarms.py index 18cdbb0..80f0d5b 100644 --- a/ceilometerclient/v2/alarms.py +++ b/ceilometerclient/v2/alarms.py @@ -32,9 +32,10 @@ UPDATABLE_ATTRIBUTES = [ 'ok_actions', 'insufficient_data_actions', 'repeat_actions', + 'project_id', + 'user_id' ] -CREATION_ATTRIBUTES = UPDATABLE_ATTRIBUTES + ['project_id', 'user_id', - 'time_constraints'] +CREATION_ATTRIBUTES = UPDATABLE_ATTRIBUTES + ['time_constraints'] class Alarm(base.Resource): |