commit be6872be5a8b59c0eba222879404a2803c7e441d
Author: Terry Wilson <email address hidden>
Date: Wed Sep 1 14:19:19 2021 +0000
Fix neutron_pg_drop-related startup issues
There is a python-ovs bug that allows transactions to be executed
before it has started monitoring the db for changes. This breaks
ovsdbapp's AddCommand behavior, which looks up a row in memory
post-commit to return it to the caller, because we process the
reply from ovsdb-server that has the UUID in it, but when we try to
look it up in memory, it isn't there since that is handled via the
monitor notifications.
Since we don't care about the return value, we can just ignore the
KeyError. In addition, the request to Idl.cond_change() to monitor
only the neutron_pg_drop table was failing for a similar reason--
it is called before the initial monitor request is sent, so
directly setting table.condition will allow the condition to be
added to the initial monitor request.
Reviewed: https:/ /review. opendev. org/c/openstack /neutron/ +/808864 /opendev. org/openstack/ neutron/ commit/ be6872be5a8b59c 0eba222879404a2 803c7e441d
Committed: https:/
Submitter: "Zuul (22348)"
Branch: stable/victoria
commit be6872be5a8b59c 0eba222879404a2 803c7e441d
Author: Terry Wilson <email address hidden>
Date: Wed Sep 1 14:19:19 2021 +0000
Fix neutron_ pg_drop- related startup issues
There is a python-ovs bug that allows transactions to be executed
before it has started monitoring the db for changes. This breaks
ovsdbapp's AddCommand behavior, which looks up a row in memory
post-commit to return it to the caller, because we process the
reply from ovsdb-server that has the UUID in it, but when we try to
look it up in memory, it isn't there since that is handled via the
monitor notifications.
Since we don't care about the return value, we can just ignore the
KeyError. In addition, the request to Idl.cond_change() to monitor
only the neutron_pg_drop table was failing for a similar reason--
it is called before the initial monitor request is sent, so
directly setting table.condition will allow the condition to be
added to the initial monitor request.
Conflicts: plugins/ ml2/drivers/ ovn/mech_ driver/ ovsdb/ovsdb_ monitor. py
neutron/
Closes-Bug: #1938766 3fb810e903d3863 4b6e60a8fe4 207872d5de9613f 93f208fdd4)
Change-Id: I78af15970a8692
(cherry picked from commit 76ee64f4a8095ba