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.
Reviewed: https:/ /review. opendev. org/680792 /git.openstack. org/cgit/ openstack/ networking- ovn/commit/ ?id=e039fb1bcdf 58d7fb26640089f 8260abce7e3a71
Committed: https:/
Submitter: Zuul
Branch: master
commit e039fb1bcdf58d7 fb26640089f8260 abce7e3a71
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 40a891b40c92405 941cbda5800
Change-Id: Ia89283c1f65693