Re: [ovs-dev] [PATCH 1/2 ovn] OVN: Fix container build.

2019-09-19 Thread aginwala
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

Re: [ovs-dev] [PATCH 1/2 ovn] OVN: Fix container build.

2019-09-19 Thread Numan Siddique
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

Re: [ovs-dev] [PATCH 1/2 ovn] OVN: Fix container build.

2019-09-11 Thread aginwala
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

Re: [ovs-dev] [PATCH 1/2 ovn] OVN: Fix container build.

2019-09-11 Thread 0-day Robot
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

[ovs-dev] [PATCH 1/2 ovn] OVN: Fix container build.

2019-09-11 Thread amginwal
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