On Sat, Feb 07, 2015 at 07:11:13AM +0000, Jason Kölker wrote: > On Sat, Feb 7, 2015 at 6:58 AM, Ben Pfaff <b...@nicira.com> wrote: > > I'm pretty sure that this version will not parse "ssl:1.2.3.4:5" > > correctly; won't it return "1.2.3.4:5" instead of just "1.2.3.4"? > > It handles bolth cases. The split tuple will be ('ssl', '1.2.3.4', > '5') and it will return the middle value, else it will return first > value: > > In [11]: [parse_controller(c) for c in ('ssl:1.2.3.4:5', '6.7.8.9:0')] > Out[11]: ['1.2.3.4', '6.7.8.9']
You're right of course, thanks. I applied this to master then backported both patches to branch-2.3, branch-2.1, and branch-2.0. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev