network notifications throwing keyerror

Bug #1317900 reported by Emilien Macchi
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
High
Liusheng
Icehouse
Fix Released
High
Liusheng

Bug Description

Running tempest tests in Icehouse, I can see that some network notifications seem broken:
http://paste.openstack.org/show/2TpNXOCbewVGhrRNGBPc/

2014-05-09 09:19:58.286 4691 TRACE stevedore.extension Traceback (most recent call last):
2014-05-09 09:19:58.286 4691 TRACE stevedore.extension File "/usr/lib/python2.7/dist-packages/stevedore/extension.py", line 247, in _invoke_one_plugin
2014-05-09 09:19:58.286 4691 TRACE stevedore.extension response_callback(func(e, *args, **kwds))
2014-05-09 09:19:58.286 4691 TRACE stevedore.extension File "/usr/lib/python2.7/dist-packages/ceilometer/notification.py", line 164, in _process_notification_for_ext
2014-05-09 09:19:58.286 4691 TRACE stevedore.extension p(list(ext.obj.to_samples(notification)))
2014-05-09 09:19:58.286 4691 TRACE stevedore.extension File "/usr/lib/python2.7/dist-packages/ceilometer/network/notifications.py", line 80, in process_notification
2014-05-09 09:19:58.286 4691 TRACE stevedore.extension message['payload'] = message['payload'][self.resource_name]
2014-05-09 09:19:58.286 4691 TRACE stevedore.extension KeyError: 'port'
2014-05-09 09:19:58.286 4691 TRACE stevedore.extension

My ceilometer.conf: http://paste.openstack.org/show/EwG0Yxbnw1c3huIu88eg/

Julien Danjou (jdanjou)
Changed in ceilometer:
status: New → Triaged
importance: Undecided → High
tags: added: icehouse-backport-potential
Revision history for this message
Liusheng (liusheng) wrote :

almost all bulk creating in nuetron can lead to this issue, bulk creating port, bulk creating network.
Hi Emilien Macchi , are you working on this bug? can I upload a patch to fix it?

Changed in ceilometer:
assignee: nobody → Liusheng (liusheng)
assignee: Liusheng (liusheng) → nobody
Liusheng (liusheng)
Changed in ceilometer:
assignee: nobody → Liusheng (liusheng)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.openstack.org/93702

Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/93702
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=bda44d592e02d95254d14819dbf5ffaeefa9f552
Submitter: Jenkins
Branch: master

commit bda44d592e02d95254d14819dbf5ffaeefa9f552
Author: liu-sheng <email address hidden>
Date: Thu May 15 19:17:34 2014 +0800

    Fix network notifications of neutron bulk creation

    When bulk creating ports, networks, subnets in neutron,
    ceilometer-notification-agent will be broken and raise an KeyError.

    Change-Id: Id5737f8ec8aa46561fe66bd56251d2a8d55f2548
    Closes-Bug: #1317900

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/icehouse)

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/95988

Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → juno-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/icehouse)

Reviewed: https://review.openstack.org/95988
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=c52904df99feaff5e65eedbbdcd8a25598052d71
Submitter: Jenkins
Branch: stable/icehouse

commit c52904df99feaff5e65eedbbdcd8a25598052d71
Author: liu-sheng <email address hidden>
Date: Thu May 15 19:17:34 2014 +0800

    Fix network notifications of neutron bulk creation

    When bulk creating ports, networks, subnets in neutron,
    ceilometer-notification-agent will be broken and raise an KeyError.

    Change-Id: Id5737f8ec8aa46561fe66bd56251d2a8d55f2548
    Closes-Bug: #1317900
    (cherry picked from commit bda44d592e02d95254d14819dbf5ffaeefa9f552)

Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: juno-1 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.