On Fri, Jun 24, 2016 at 3:52 PM, Russell Bryant <russ...@ovn.org> wrote:
> Some previous commits broke ovn-controller binding handling such that > ovn-controller never cleared out the chassis column of the Port_Binding > table. This broke OpenStack CI for OVN. This patch adds an OVN test > case that would have caught this issue. > > Signed-off-by: Russell Bryant <russ...@ovn.org> > --- > tests/ovn.at | 46 ++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 46 insertions(+) > > diff --git a/tests/ovn.at b/tests/ovn.at > index 297070c..5aa1fd6 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -3182,3 +3182,49 @@ OVS_APP_EXIT_AND_WAIT([ovs-vswitchd]) > OVS_APP_EXIT_AND_WAIT([ovsdb-server]) > > AT_CLEANUP > + > +# 1 hypervisor, 1 port > +# make sure that the port state is properly set to up and back down > +# when created and deleted. > +AT_SETUP([ovn -- port state up and down]) > +AT_KEYWORDS([ovn]) > +ovn_start > + > +ovn-nbctl ls-add ls1 > +ovn-nbctl lsp-add ls1 lp1 > +ovn-nbctl lsp-set-addresses lp1 unknown > + > +net_add n1 > +sim_add hv1 > +echo 1 > +as hv1 ovs-vsctl add-br br-phys > +echo 2 > +as hv1 ovn_attach n1 br-phys 192.168.0.1 > +echo 3 > + > +as hv1 ovs-vsctl add-port br-int vif1 -- set Interface vif1 > external-ids:iface-id=lp1 > +OVS_WAIT_UNTIL([test x`ovn-nbctl lsp-get-up lp1` = xup]) > +echo 4 > + > +as hv1 ovs-vsctl del-port br-int vif1 > +OVS_WAIT_UNTIL([test x`ovn-nbctl lsp-get-up lp1` = xdown]) > + > +echo 5 > All these debug "echo" commands should be removed ... oops. -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
