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

2015-02-06 Thread Jason Kölker
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: Use str.replace instead of lstrip for dns name saftey Signed-off-by:

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: Use

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 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

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

2015-02-06 Thread Jason Kölker
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: Use str.replace instead of lstrip for dns name saftey v4: Use filter

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 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