Re: [ovs-dev] [PATCH 1/2] xenserver: Always update the bridge ID in ovs-xapi-sync.

2012-02-29 Thread Justin Pettit
On Feb 29, 2012, at 8:10 PM, Ethan Jackson wrote: >> This seems fine as a fix for now, but I think that it causes >> ovs-xapi-sync to fetch all of the network records from XAPI on every >> iteration, so I think that we should consider caching the >> xs-network-uuids to bridge-id mapping in the sam

Re: [ovs-dev] [PATCH 1/2] xenserver: Always update the bridge ID in ovs-xapi-sync.

2012-02-29 Thread Ethan Jackson
> This seems fine as a fix for now, but I think that it causes > ovs-xapi-sync to fetch all of the network records from XAPI on every > iteration, so I think that we should consider caching the > xs-network-uuids to bridge-id mapping in the same way that we now cache > the iface-id mapping. I've u

Re: [ovs-dev] [PATCH 1/2] xenserver: Always update the bridge ID in ovs-xapi-sync.

2012-02-29 Thread Justin Pettit
On Feb 29, 2012, at 7:53 PM, Ben Pfaff wrote: > On Wed, Feb 29, 2012 at 07:49:32PM -0800, Ben Pfaff wrote: >> On Wed, Feb 29, 2012 at 07:17:24PM -0800, Ethan Jackson wrote: >>> In some cases we were seeing this column get stale. >>> >>> Bug #9929. >>> Signed-off-by: Ethan Jackson >> >> This see

Re: [ovs-dev] [PATCH 1/2] xenserver: Always update the bridge ID in ovs-xapi-sync.

2012-02-29 Thread Ben Pfaff
On Wed, Feb 29, 2012 at 07:49:32PM -0800, Ben Pfaff wrote: > On Wed, Feb 29, 2012 at 07:17:24PM -0800, Ethan Jackson wrote: > > In some cases we were seeing this column get stale. > > > > Bug #9929. > > Signed-off-by: Ethan Jackson > > This seems fine as a fix for now, but I think that it causes

Re: [ovs-dev] [PATCH 1/2] xenserver: Always update the bridge ID in ovs-xapi-sync.

2012-02-29 Thread Ben Pfaff
On Wed, Feb 29, 2012 at 07:17:24PM -0800, Ethan Jackson wrote: > In some cases we were seeing this column get stale. > > Bug #9929. > Signed-off-by: Ethan Jackson This seems fine as a fix for now, but I think that it causes ovs-xapi-sync to fetch all of the network records from XAPI on every ite

[ovs-dev] [PATCH 1/2] xenserver: Always update the bridge ID in ovs-xapi-sync.

2012-02-29 Thread Ethan Jackson
In some cases we were seeing this column get stale. Bug #9929. Signed-off-by: Ethan Jackson --- .../usr_share_openvswitch_scripts_ovs-xapi-sync|5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync b/xenserver/usr_sh