[ovs-dev] [PATCH 4/7] ovn-controller: Update man page comment on mid-run changes.

Justin Pettit jpettit at nicira.com
Tue Aug 25 19:51:40 UTC 2015


> On Aug 25, 2015, at 12:40 PM, Russell Bryant <rbryant at redhat.com> wrote:
> 
> On 08/24/2015 09:12 PM, Justin Pettit wrote:
>> It's no longer necessary to restart ovn-controller when the chassis name
>> or integration bridge changes.
>> 
>> Signed-off-by: Justin Pettit <jpettit at nicira.com>
>> ---
>> ovn/controller/ovn-controller.8.xml |    5 ++---
>> 1 files changed, 2 insertions(+), 3 deletions(-)
>> 
>> diff --git a/ovn/controller/ovn-controller.8.xml b/ovn/controller/ovn-controller.8.xml
>> index 19bb5b7..2d657fb 100644
>> --- a/ovn/controller/ovn-controller.8.xml
>> +++ b/ovn/controller/ovn-controller.8.xml
>> @@ -115,9 +115,8 @@
>>       </ul>
>>       <p>
>>         Currently, <code>ovn-controller</code> does not support changing
>> -        the chassis name, integration bridge, or OVN database mid-run.
>> -        If these values need to change, the daemon must be restarted.
>> -        This behavior should be changed.
>> +        OVN database mid-run.  If the value needs to change, the daemon
>> +        must be restarted.  This behavior should be improved.
>>       </p>
>>     </p>
>> </manpage>
>> 
> 
> "changing OVN database mid-run" would read better for me as "changing
> the OVN database mid-run".  Otherwise, lgtm.
> 
> Acked-by: Russell Bryant <rbryant at redhat.com>

Agreed.  Thanks.

--Justin





More information about the dev mailing list