[ovs-dev] [PATCH] sandbox: launch SB backup server when running in OVN mode

[Andy Zhou]

On Mon, Aug 15, 2016 at 1:50 PM, Russell Bryant <russell at ovn.org> wrote:

>
>
> On Mon, Aug 15, 2016 at 4:45 PM, Andy Zhou <azhou at ovn.org> wrote:
>
>>
>>
>> On Mon, Aug 15, 2016 at 1:36 PM, Russell Bryant <russell at ovn.org> wrote:
>>
>>>
>>>
>>> On Mon, Aug 15, 2016 at 2:54 PM, Andy Zhou <azhou at ovn.org> wrote:
>>>
>>>>
>>>>
>>>> On Thu, Aug 11, 2016 at 1:22 PM, Russell Bryant <russell at ovn.org>
>>>> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Tue, Aug 9, 2016 at 7:49 PM, Andy Zhou <azhou at ovn.org> wrote:
>>>>>
>>>>>> Automatically launch backup server for OVN SB database that replicates
>>>>>> all transactions of the active server. This can be handy for
>>>>>> experimenting with the newly added replication feature.
>>>>>>
>>>>>> Signed-off-by: Andy Zhou <azhou at ovn.org>
>>>>>>
>>>>>
>>>>> Sure, seems fine to me.  You could also add some notes to
>>>>> OVN-Tutorial.md about how to poke at the backup replica.
>>>>>
>>>>
>>>> How about the following incremental? I'd like to fold it in before
>>>> pushing
>>>> diff --git a/tutorial/OVN-Tutorial.md b/tutorial/OVN-Tutorial.md
>>>> index a1fff0b..be9805c 100644
>>>> --- a/tutorial/OVN-Tutorial.md
>>>> +++ b/tutorial/OVN-Tutorial.md
>>>> @@ -25,12 +25,17 @@ environment:
>>>>    1. Creates the `OVN_Northbound` and `OVN_Southbound` databases as
>>>> described in
>>>>       [ovn-nb(5)] and [ovn-sb(5)].
>>>>
>>>> -  2. Creates the `hardware_vtep` database as described in [vtep(5)].
>>>> +  2. Creates a backup server for `OVN_Southbond` database. Sandbox
>>>> launch
>>>> +     screen provides the instructions on accessing the backup database.
>>>> +     However access to the backup server is not required to go through
>>>> the
>>>> +     tutorial.
>>>>
>>>> -  3. Runs the [ovn-northd(8)], [ovn-controller(8)], and
>>>> [ovn-controller-vtep(8)]
>>>> +  3. Creates the `hardware_vtep` database as described in [vtep(5)].
>>>> +
>>>> +  4. Runs the [ovn-northd(8)], [ovn-controller(8)], and
>>>> [ovn-controller-vtep(8)]
>>>>       daemons.
>>>>
>>>> -  4. Makes OVN and VTEP utilities available for use in the environment,
>>>> +  5. Makes OVN and VTEP utilities available for use in the environment,
>>>>       including [vtep-ctl(8)], [ovn-nbctl(8)], and [ovn-sbctl(8)].
>>>>
>>>>  Note that each of these demos assumes you start with a fresh sandbox
>>>>
>>>
>>> This incremental looks good.
>>>
>>> Any reason not to add a backup for northbound, as well?
>>>
>> It can be easily added. I know we need to replicate SB for sure, but was
>> not sure about NB.
>> For the purpose of experimenting with the backup feature, one back server
>> seems to be sufficient.
>> We can certainly expend the set up to be closer to real life usage down
>> the road.
>>
>
> I agree that 1 is fine for experimentation.
>

Thanks, pushed to master.

>
> In practice, I think we need the same replication and backup for both NB
> and SB.  I'd like to avoid having to repopulate the NB db in the event of a
> host failure.
>
If our goal is to make sandbox more life like, should we consider adding an
HA manager in the picture and provide an floating IP for ovsdb clients to
connect with?

>
>