Re: [ovs-dev] [PATCH ovn 1/4] Add support for using OVN specific rundirs

2019-08-19 Thread Numan Siddique
On Fri, Aug 16, 2019 at 10:45 PM Mark Michelson wrote: > On 8/13/19 12:27 PM, nusid...@redhat.com wrote: > > From: Numan Siddique > > > > Until now, OVN uses the openvswitch rundirs (rundir, logdir, etcdir). > > The commit [1] changed the package name from openvswitch to ovn, but > > it didn't t

Re: [ovs-dev] [PATCH ovn 1/4] Add support for using OVN specific rundirs

2019-08-19 Thread Dumitru Ceara
On Tue, Aug 13, 2019 at 6:28 PM wrote: > > From: Numan Siddique > > Until now, OVN uses the openvswitch rundirs (rundir, logdir, etcdir). > The commit [1] changed the package name from openvswitch to ovn, but > it didn't take into the account the effects of it. When "make install" > is run ovn-ct

Re: [ovs-dev] [PATCH ovn 1/4] Add support for using OVN specific rundirs

2019-08-16 Thread Mark Michelson
On 8/13/19 12:27 PM, nusid...@redhat.com wrote: From: Numan Siddique Until now, OVN uses the openvswitch rundirs (rundir, logdir, etcdir). The commit [1] changed the package name from openvswitch to ovn, but it didn't take into the account the effects of it. When "make install" is run ovn-ctl u

Re: [ovs-dev] [PATCH ovn 1/4] Add support for using OVN specific rundirs

2019-08-13 Thread 0-day Robot
Bleep bloop. Greetings Numan Siddique, 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: WARNING: Line is 99 characters long (recommended limit is 79) #650 FILE: utilities/ovn-ctl:34

[ovs-dev] [PATCH ovn 1/4] Add support for using OVN specific rundirs

2019-08-13 Thread nusiddiq
From: Numan Siddique Until now, OVN uses the openvswitch rundirs (rundir, logdir, etcdir). The commit [1] changed the package name from openvswitch to ovn, but it didn't take into the account the effects of it. When "make install" is run ovn-ctl utility is copied to /usr/local/share/ovn/scripts f