[ovs-build] Passed: ovsrobot/ovn#875 (series_192618 - e5eb717)
builds at travis-ci.com
Mon Jul 27 18:16:28 UTC 2020
Build Update for ovsrobot/ovn
Duration: 19 mins and 2 secs
Commit: e5eb717 (series_192618)
Author: Numan Siddique
Message: ovn-controller: Clear flows not associated with db rows in physical flow change handler.
The commit in the Fixes tag while handling changes for OVS interface changes and ct zone
changes, called physical_run() without clearing the flow table. This works ok for existing
flows in the flow table which are associated with ovsdb rows. physical_run() ensures that
such flows are deleted by calling ofctrl_delete_flows() by adding them back again.
But flows not associated with ovsdb rows (whose OF flow cookie is set to 0) are not cleared
at all until a full recompute is triggered. Particularly flows in table 33 which set
various conntrack zone registers still remain. Suppose a lport is claimed again and if
the ct-zone id for this lport changes, then the old flow for this lport still remains and
this causes the packet to enter the conntrack with a wrong zone id.
Such flows are stored indexed with the uuid 'hc_uuid' in the flow table and it is easy
to clear them. This patch clears such flows before calling physical_run() to fix this issue.
A more accurate fix would be to store the logical and physical flows in separate flow tables
and clear all the flows in the physical flow table before calling physical_run().
This patch is good enough for now until we implement separate flow tables.
Fixes: a3005f0dc777("ovn-controller: I-P for ct zone and OVS interface changes in flow output stage.")
Signed-off-by: Numan Siddique <numans at ovn.org>
Signed-off-by: 0-day Robot <robot at bytheb.org>
View the changeset: https://github.com/ovsrobot/ovn/commit/e5eb7174dfb6
View the full build log and details: https://travis-ci.com/github/ovsrobot/ovn/builds/177332573?utm_medium=notification&utm_source=email
You can unsubscribe from build emails from the ovsrobot/ovn repository going to https://travis-ci.com/account/preferences/unsubscribe?repository=9136199&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...
More information about the build