Comment 4 for bug 1843269

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-ovn (master)

Reviewed: https://review.opendev.org/680792
Committed: https://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=e039fb1bcdf58d7fb26640089f8260abce7e3a71
Submitter: Zuul
Branch: master

commit e039fb1bcdf58d7fb26640089f8260abce7e3a71
Author: Terry Wilson <email address hidden>
Date: Fri Sep 6 20:39:38 2019 -0500

    Only notify nova of port status changes if configured

    Although notify_nova_on_port_status_changes defaults to true, it
    could be set to false, making the nova_notifier attribute unsafe to
    use without checking. In fact, networking-ovn functional tests
    inherit from the neutron base *unit* test which disables this
    option.

    This patch checks both the config option and that the attribute
    exists, since the config could be changed after the plugin is
    already initialized without the nova_notifier attribute being set.

    Partial-Bug: 1843269
    Change-Id: Ia89283c1f6569340a891b40c92405941cbda5800