[ovs-build] Still Failing: openvswitch/ovs#2933 (master - fdb3c70)

Travis CI builds at travis-ci.org
Mon Jul 18 23:05:59 UTC 2016


Build Update for openvswitch/ovs
-------------------------------------

Build: #2933
Status: Still Failing

Duration: 37 minutes and 10 seconds
Commit: fdb3c70 (master)
Author: Russell Bryant
Message: ovn-controller: Drop remove_local_datapath_by_binding().

ovn-controller has an hmap called 'local_datapaths' which tracks
all OVN datapaths that have at least one port binding on the local
chassis.  This patch corrects the case where a port binding row is
deleted from the southbound DB while it's still bound to the chassis,
meaning it was deleted before the ovs interface was deleted.

The previous code tried to handle this case by calling
remove_local_datapath_by_binding().  The function appears to try
to look up local_datapath by the binding UUID.  If it finds it,
it will delete the local datapath entry.  On the surface, this
looks like a bug where it deletes a local datapath entry even
when there could be other ports still bound to the chassis.
The reality is that this function was always a no-op.  It was
doing a lookup using a different hash value than how local_datapath
entries are actually hashed. In practice, this wasn't a big problem
because local_datapaths are correctly cleaned in in the
process_full_binding case after an ovs interface is added or removed.

The new change ensures that we run the process_full_binding code
in this case right away, even if the interface is not deleted.

Fixes: 263064aeaa31 ("Convert binding_run to incremental processing.")
Signed-off-by: Russell Bryant <russell at ovn.org>
Acked-by: Ryan Moats <rmoats at us.ibm.com>

View the changeset: https://github.com/openvswitch/ovs/compare/564666e9db0e...fdb3c70c3ee5

View the full build log and details: https://travis-ci.org/openvswitch/ovs/builds/145661223

--

You can configure 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://www.openvswitch.org/pipermail/ovs-build/attachments/20160718/54389fb8/attachment-0002.html>


More information about the build mailing list