On Mon, Aug 15, 2016 at 5:05 PM, Andy Zhou <az...@ovn.org> wrote: > > > On Mon, Aug 15, 2016 at 1:50 PM, Russell Bryant <russ...@ovn.org> wrote: > >> >> >> On Mon, Aug 15, 2016 at 4:45 PM, Andy Zhou <az...@ovn.org> wrote: >> >>> >>> >>> On Mon, Aug 15, 2016 at 1:36 PM, Russell Bryant <russ...@ovn.org> wrote: >>> >>>> >>>> >>>> On Mon, Aug 15, 2016 at 2:54 PM, Andy Zhou <az...@ovn.org> wrote: >>>> >>>>> >>>>> >>>>> On Thu, Aug 11, 2016 at 1:22 PM, Russell Bryant <russ...@ovn.org> >>>>> wrote: >>>>> >>>>>> >>>>>> >>>>>> On Tue, Aug 9, 2016 at 7:49 PM, Andy Zhou <az...@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 <az...@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? >
No, I don't think that should be the goal. I think providing bare minimum to enable basic experimentation is better. What you've done in the patch already is perfect for that. -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
