On 09/01/2015 02:29 PM, Ben Pfaff wrote: > [This is probably threaded wrong; somehow the patch didn't make it into > my inbox and I had to apply it from the archive.]
Weird. I use git-send-email and always give it the message-id of patch 0/N of the last revision to reply to. > It seems that, when create_br_int() creates the integration bridge, it > should return it to get_br_int() so that it can return it (instead of > NULL). Fair enough. I remember doing that on purpose, but I'm not positive why. Maybe I thought it should be committed before returning it from there for some reason. In any case, I'll fix that up. > Should we add some text to ovn-architecture(7) mentioning that > ovn-controller will create the integration bridge? It's still possible > that the platform will want to create the integration bridge before > starting ovn-controller, if it wants to add vifs to that bridge in > advance. So now it becomes optional. > > I wonder whether ovn-controller(8) should mention this behavior. These doc suggestions sound good. I'll update ovn-architecture and ovn-controller docs. -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
