ovs agent : Failed reporting state!

[Derek Higgins]

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'

[Eugene Nikanorov (enikanorov)]

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

[Eugene Nikanorov (enikanorov)]

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

[Derek Higgins (derekh)]

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

[OpenStack Infra (hudson-openstack)]

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