On Wed, Mar 24, 2021 at 2:32 PM Frode Nordahl <[email protected]> wrote: > > On Wed, Mar 24, 2021 at 1:54 PM Numan Siddique <[email protected]> wrote: > > I applied the patches 6 and 7 to the main branch. > > > > There are some issues with patch 9. I didn't apply patch 8 as it > > seems related to patch 9. > > > > If I configure like below and run "make check" it fails for me. Can > > you please take a look. > > > > $mkdir _gcc > > $cd _gcc > > $../configure --enable-Werror --enable-sparse --with-ovs-source=... > > $cd .. > > $make -C _gcc check > > make[2]: Entering directory /tmp/ovn/_gcc' > > make[2]: 'tests/atlocal' is up to date. > > make[2]: 'tests/testpki-cacert.pem' is up to date. > > make[2]: 'tests/testpki-test-cert.pem' is up to date. > > make[2]: 'tests/testpki-test-privkey.pem' is up to date. > > make[2]: 'tests/testpki-test-req.pem' is up to date. > > make[2]: 'tests/testpki-test2-cert.pem' is up to date. > > make[2]: 'tests/testpki-test2-privkey.pem' is up to date. > > make[2]: 'tests/testpki-test2-req.pem' is up to date. > > cp /tmp/ovn/_gcc/tests/pki/main-cert.pem tests/testpki-main-cert.pem > > cp: cannot stat '/tmp/ovn/_gcc/tests/pki/main-cert.pem': No such file > > or directory > > make[2]: *** [Makefile:3512: tests/testpki-main-cert.pem] Error 1 > > make[2]: Leaving directory '/tmp/ovn/_gcc' > > Thank you for finding this issue, as it hid itself from me when > checking with a plain `make distcheck`. > > I'll investigate and put up a v3.
Do you have any more details to share about your environment? I created a clean container and ran your steps with patch 8 and 9 and it succeeds with no issues here: https://pastebin.ubuntu.com/p/WXryXktqCh/ There could still be an issue here, but I need some more detail to figure out what is happening on your end. -- Frode Nordahl > -- > Frode Nordahl > > > > Thanks > > Numan > > > > > > > > > > -- > > > Frode Nordahl > > > > > > > Thanks. > > > > > > > > On 3/5/21 7:16 AM, Frode Nordahl wrote: > > > > > The current version of the test attempts to simulate chassis > > > > > registration prior to starting `ovn-controller`, however it does > > > > > not set the `hostname` field. > > > > > > > > > > The RBAC role for `ovn-controller` does not allow for a chassis to > > > > > change its own name or hostname, which makes sense as this is used > > > > > for authentication. > > > > > > > > > > Update the test to set the `hostname` field when simulating chassis > > > > > registration so that `ovn-controller` does not attempt to update it > > > > > and subsequently make the test fail. > > > > > > > > > > Fixes b6b3823d4 ("ovn-controller: Fix I-P for SB Port_Binding and OVS > > > > > Interface") > > > > > > > > > > Signed-off-by: Frode Nordahl <[email protected]> > > > > > --- > > > > > tests/ovn.at | 2 +- > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > diff --git a/tests/ovn.at b/tests/ovn.at > > > > > index bec593dcc..ca9623fee 100644 > > > > > --- a/tests/ovn.at > > > > > +++ b/tests/ovn.at > > > > > @@ -21572,7 +21572,7 @@ ovn-nbctl --wait=sb lsp-add ls1 lsp1 > > > > > > > > > > # Simulate the fact that lsp1 had been previously bound on hv1. > > > > > ovn-sbctl --id=@e create encap chassis_name=hv1 ip="192.168.0.1" > > > > > type="geneve" \ > > > > > - -- --id=@c create chassis name=hv1 encaps=@e \ > > > > > + -- --id=@c create chassis hostname=hv1 name=hv1 encaps=@e \ > > > > > -- set Port_Binding lsp1 chassis=@c > > > > > > > > > > as hv1 > > > > > > > > > > > > _______________________________________________ > > > dev mailing list > > > [email protected] > > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > > > > > > -- > Frode Nordahl _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
