Comment 5 for bug 1843269

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

Reviewed: https://review.opendev.org/682359
Committed: https://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=a0d1babfd285c78e8c3e69d5e7e6e6c9d3cfb470
Submitter: Zuul
Branch: stable/stein

commit a0d1babfd285c78e8c3e69d5e7e6e6c9d3cfb470
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
    (cherry picked from commit e039fb1bcdf58d7fb26640089f8260abce7e3a71)