diff options
author | Pradeep Kilambi <pkilambi@redhat.com> | 2016-01-25 10:11:43 -0500 |
---|---|---|
committer | Mehdi Abaakouk <sileht@redhat.com> | 2016-04-29 11:06:06 +0200 |
commit | 3a1fcaf712a7cdcffafe2ae33bc4c2508bab8a49 (patch) | |
tree | 7f6e4cf4c668cc77112c8d32c9b888991877a5fe | |
parent | af3cb7b5245a572763501502ba33b21a36a26492 (diff) | |
download | ceilometer-3a1fcaf712a7cdcffafe2ae33bc4c2508bab8a49.tar.gz |
Lookup meter definition fields correctly5.0.3
Change-Id: I9db77b9bc196c06c409fcfe252c172230fee4274
Closes-Bug: #1536699
(cherry picked from commit 903a0a527cb240cfd9462b7f56d3463db7128993)
-rw-r--r-- | ceilometer/meter/notifications.py | 2 | ||||
-rw-r--r-- | releasenotes/notes/lookup-meter-def-vol-correctly-0122ae429275f2a6.yaml | 8 |
2 files changed, 9 insertions, 1 deletions
diff --git a/ceilometer/meter/notifications.py b/ceilometer/meter/notifications.py index dc360fb8..c2dd1b03 100644 --- a/ceilometer/meter/notifications.py +++ b/ceilometer/meter/notifications.py @@ -191,7 +191,7 @@ def load_definitions(config_def): continue try: - if (event_def['volume'] != 1 or + if (event_def.get('volume') != 1 or not cfg.CONF.notification.disable_non_metric_meters): md = MeterDefinition(event_def) meter_defs[event_def['name']] = md diff --git a/releasenotes/notes/lookup-meter-def-vol-correctly-0122ae429275f2a6.yaml b/releasenotes/notes/lookup-meter-def-vol-correctly-0122ae429275f2a6.yaml new file mode 100644 index 00000000..9bb5c5b1 --- /dev/null +++ b/releasenotes/notes/lookup-meter-def-vol-correctly-0122ae429275f2a6.yaml @@ -0,0 +1,8 @@ +--- +fixes: + - > + [`bug 1536699 <https://bugs.launchpad.net/ceilometer/+bug/1536699>`_] + Patch to fix volume field lookup in meter definition file. In case + the field is missing in the definition, it raises a keyerror and + aborts. Instead we should skip the missing field meter and continue + with the rest of the definitions. |