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

2015-02-06 Thread Ben Pfaff
On Sat, Feb 07, 2015 at 07:11:13AM +, Jason Kölker wrote: > On Sat, Feb 7, 2015 at 6:58 AM, Ben Pfaff 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 w

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

2015-02-06 Thread Jason Kölker
On Sat, Feb 7, 2015 at 6:58 AM, Ben Pfaff 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

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

2015-02-06 Thread Ben Pfaff
On Sat, Feb 07, 2015 at 06:47:51AM +, Jason Kölker wrote: > With XenServer only 1 manager is configured in the pool, which may not > be the first manager returned from `get-manager` as it returns in > lexicographical order. > > v2: Fixes vswitchCurrentControllers() to always return a list > v3