Thanks Numan for review:
The issue was create_ovn_dbs.sh needs to be deleted from automake.mk as per
new patch. While testing, my env var used master branch vs my custom
branch ovn_docker_unix and hence it had create_ovn_dbs.sh and it built fine
as per new build scripts. I submitted v2
https://pa
On Thu, Sep 12, 2019 at 7:00 AM aginwala wrote:
> Signed-off-by: Aliasgar Ginwala
>
>
Hi Ali,
This patch is failing when I compile.
Can you please take a look.
> utilities/ovn-lib.tmp
make[1]: *** No rule to make target 'utilities/docker/create_ovn_dbs.sh',
needed by 'all-am'. Stop.
make[1
Signed-off-by: Aliasgar Ginwala
On Wed, Sep 11, 2019 at 6:00 PM 0-day Robot wrote:
> Bleep bloop. Greetings aginwala aginwala, I am a robot and I have tried
> out your patch.
> Thanks for your contribution.
>
> I encountered some error that I wasn't expecting. See the details below.
>
>
> che
Bleep bloop. Greetings aginwala aginwala, I am a robot and I have tried out
your patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
checkpatch:
ERROR: Author Aliasgar Ginwala needs to sign off.
Lines checked: 157, Warnings: 0, Errors
From: Aliasgar Ginwala
Current code changes in ovs/ovn split replaced openvswitch with ovn and
caused container patches to break.
Also, as per ff4439d, one should build ovn specifying ovs source.
This patch fixes the same.
---
utilities/docker/create_ovn_dbs.sh | 18 --
utilities