On 7/30/24 13:52, Dumitru Ceara wrote:
> On 7/29/24 22:47, Rosemarie O'Riorden wrote:
>> All other OVN daemons have the --unixctl option to specify the control
>> socket file. It was missing from ovn-controller and ovn-controller-vtep,
>> which this patch fixes.
>>
>> The ovn-controller-vtep docume
On 7/29/24 22:47, Rosemarie O'Riorden wrote:
> All other OVN daemons have the --unixctl option to specify the control
> socket file. It was missing from ovn-controller and ovn-controller-vtep,
> which this patch fixes.
>
> The ovn-controller-vtep documentation was also missing the already
> suppor
Bleep bloop. Greetings Rosemarie O'Riorden, 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 83 characters long (recommended limit is 79)
#32 FILE: controller-vtep/
All other OVN daemons have the --unixctl option to specify the control
socket file. It was missing from ovn-controller and ovn-controller-vtep,
which this patch fixes.
The ovn-controller-vtep documentation was also missing the already
supported --help and --version options, which are now reflected