[ovs-build] Failed: numansiddique/ovn#190 (mark_test_fix - 86fad88)

[Travis CI]

Build Update for numansiddique/ovn
-------------------------------------

Build: #190
Status: Failed

Duration: 11 mins and 11 secs
Commit: 86fad88 (mark_test_fix)
Author: Mark Michelson
Message: Clear port binding flows when datapath CT zone changes.

In commit f9cab11d5fabe2ae321a3b4bad5972b61df958c0, a LOG_TO_PHY flow
was changed so that it was no longer associated with a particular port
binding. The logic there was that the particular flow contains data
pertaining to the port binding's peer's datapath, so it didn't make
sense to associate the flow with the port binding. This change was
necessary in order for flows to be recalculated properly if the
requested SNAT CT zone on a gateway router was changed. Since the
datapath was changed but no port bindings were changed, that particular
flow needed to be cleared so it could be recalculated with the new CT
zones put in place.

Unfortunately, that change broke some other behavior. Specifically, if a
router was changed from a distributed router to a gateway router, then
its port bindings and its port bindings' peers would be updated so that
they were no longer type "patch" but instead type "l3gateway". They
would attempt to remove all associated physical flows and then install
the newly relevant ones. Since the LOG_TO_PHY flow was no longer
associated with a port binding, that flow would remain. The result was
that traffic could be sent to the gateway router on chassis where the
gateway router was not pinned.

This commit seeks to fix both behaviors. Now if CT zones are changed on
a particular datapath, then all port bindings on that datapath, as well
as all of those port bindings' peers will have their physical flows
removed. When physical flows are recomputed, all of the appropriate
flows will be added.

Signed-off-by: Mark Michelson <mmichels at redhat.com>

View the changeset: https://github.com/numansiddique/ovn/commit/86fad8859aab

View the full build log and details: https://travis-ci.com/github/numansiddique/ovn/builds/203036918?utm_medium=notification&utm_source=email


--

You can unsubscribe from build emails from the numansiddique/ovn repository going to https://travis-ci.com/account/preferences/unsubscribe?repository=9252656&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://travis-ci.com/account/preferences/unsubscribe?utm_medium=notification&utm_source=email.
Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-build/attachments/20201120/223d8345/attachment.html>