neutron

ovs agent : Failed reporting state!

Bug #1221054 reported by Derek Higgins
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Derek Higgins
neutron 2013.2 "havana"

Bug Description

Noticed on compute nodes, while running neutron on TripleO

[-] Failed reporting state!
TRACE neutron.plugins.openvswitch.agent.ovs_neutron_agent Traceback (most recent call last):
TRACE neutron.plugins.openvswitch.agent.ovs_neutron_agent File "/opt/stack/venvs/neutron/lib/python2.7/site-packages/neutron/plugins/openvswitch/agent/ovs_neutron_agent.py", line 222, in _report_state
TRACE neutron.plugins.openvswitch.agent.ovs_neutron_agent self.int_br_device_count
)
TRACE neutron.plugins.openvswitch.agent.ovs_neutron_agent AttributeError: 'OVSNeutronAgent' object has no attribute 'int_br_device_count'

Tags: ovs
Derek Higgins (derekh)
Changed in neutron:
assignee: nobody → Derek Higgins (derekh)
OpenStack Infra (hudson-openstack)
Changed in neutron:
status: New → In Progress
Eugene Nikanorov (enikanorov)
Changed in neutron:
importance: Undecided → Critical
Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

Can you provide more detailed trace with line numbers?
I don't see the issue in my environment.

Changed in neutron:
importance: Critical → High
importance: High → Medium
Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

Reproduced it.
Apparently it only happens at startup, so the issue is not critical, changing to Medium.

Revision history for this message
Derek Higgins (derekh) wrote :

Potential Fix https://review.openstack.org/#/c/45131/

ZhiQiang Fan (aji-zqfan)
tags: added: ovs
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/45131
Committed: http://github.com/openstack/neutron/commit/becadd39df087544ac1ce97926cc3014fd286e08
Submitter: Jenkins
Branch: master

commit becadd39df087544ac1ce97926cc3014fd286e08
Author: Derek Higgins <email address hidden>
Date: Wed Sep 4 23:05:08 2013 +0100

    Move declaration of int_br_device_count earlier

    _report_state is being called by setup_rpc so int_br_device_count needs
    to be initialized earlier. To avoid
    AttributeError: object has no attribute 'int_br_device_count'

    This wasn't caught by unit tests for 3 separate reason
    o The reference to self.int_br_device_count is wrapped in
      except Exception: log / pass
    - This reference has been moved outside of the try/except

    o Unit tests set report_interval to 0 so the heartbeat wasn't called
      during unit tests.
    - now removed

    o The function passed into FixedIntervalLoopingCall isn't started
      anyways so wasn't calling self._report_state
    - replaced FixedIntervalLoopingCall with a mock that calls the
      function once.

    Fixes bug #1221054

    Change-Id: I572af4ee017c1f7f016c8f9981ca54d1301442d1

Changed in neutron:
status: In Progress → Fix Committed
Mark McClain (markmcclain)
Changed in neutron:
milestone: none → havana-rc1
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: havana-rc1 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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