[ovs-dev] [PATCH] XenServer: Don't reset on xe-toolstack-restart

Ben Pfaff blp at nicira.com
Sat Feb 7 07:17:31 UTC 2015


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 <blp at 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.



More information about the dev mailing list