Re: [ovs-dev] InterfaceReconfigureVswitch.py fix didn't make it into xen-api?

2011-02-11 Thread Ian Campbell
On Fri, 2011-02-11 at 20:43 +, Ben Pfaff wrote: > Before I re-sent this patch I made sure to check whether you'd moved > to Git (since I'd heard that it was in the works), The switch was made this Monday. > but the "Xen Cloud > Platform Source" webpage doesn't say a thing about it: >

Re: [ovs-dev] [PATCH 2/2] Remove /proc/net compatibility support.

2011-02-11 Thread Ben Pfaff
Thanks. I hadn't actually tried distcheck but it's running now. Assuming it passes I'll push this. On Fri, Feb 11, 2011 at 01:59:27PM -0800, Ethan Jackson wrote: > Yay! That stuff was so ugly, glad it's gone. > > I didn't closely review all of the deleted files, but I did look > closely at the

Re: [ovs-dev] [PATCH 2/2] Remove /proc/net compatibility support.

2011-02-11 Thread Ethan Jackson
Yay! That stuff was so ugly, glad it's gone. I didn't closely review all of the deleted files, but I did look closely at the vswitchd/bridge.c changes. Assuming this passes make distcheck I think it's good. Ethan On Fri, Feb 11, 2011 at 1:48 PM, Ben Pfaff wrote: > This feature was included on

Re: [ovs-dev] [PATCH 1/2] xenserver: Remove usr_share_openvswitch_scripts_refresh-xs-network-uuids.

2011-02-11 Thread Ben Pfaff
Thanks, I pushed just this patch. On Fri, Feb 11, 2011 at 01:55:24PM -0800, Ethan Jackson wrote: > Looks Good. > > On Fri, Feb 11, 2011 at 1:48 PM, Ben Pfaff wrote: > > This file is not distributed, which breaks the build with the message: > > > > ? ?The distribution is missing the following fil

Re: [ovs-dev] [PATCH 1/2] xenserver: Remove usr_share_openvswitch_scripts_refresh-xs-network-uuids.

2011-02-11 Thread Ethan Jackson
Looks Good. On Fri, Feb 11, 2011 at 1:48 PM, Ben Pfaff wrote: > This file is not distributed, which breaks the build with the message: > >    The distribution is missing the following files: >    xenserver/usr_share_openvswitch_scripts_refresh-xs-network-uuids > > Removing it from the repository

[ovs-dev] [PATCH 2/2] Remove /proc/net compatibility support.

2011-02-11 Thread Ben Pfaff
This feature was included only to allow Citrix QA to run some tests that interacted directly with the bridge. This feature hasn't been turned on for some time, so it should not be necessary any longer. Signed-off-by: Ben Pfaff --- datapath/brc_procfs.c | 186

[ovs-dev] [PATCH 1/2] xenserver: Remove usr_share_openvswitch_scripts_refresh-xs-network-uuids.

2011-02-11 Thread Ben Pfaff
This file is not distributed, which breaks the build with the message: The distribution is missing the following files: xenserver/usr_share_openvswitch_scripts_refresh-xs-network-uuids Removing it from the repository fixes the problem. Signed-off-by: Ben Pfaff --- ...re_openvswitch_scr

Re: [ovs-dev] [RPMs v2 4/4] xenserver: Don't create XAPI cache at install.

2011-02-11 Thread Andrew Evans
On 2/11/11 12:59 PM, Ben Pfaff wrote: > It would be customary to add a few words describing how we updated it > after Bob's sign-off, and then sign it off ourselves also, e.g.: > > Signed-off-by: Bob Ball > [removed sourcing /etc/xensource-inventory since it is no longer needed] > Signed-off-by:

Re: [ovs-dev] [RPMs v2 3/4] xenserver: Rename ovs-external-ids to ovs-xapi-sync.

2011-02-11 Thread Andrew Evans
On 2/11/11 1:03 PM, Ben Pfaff wrote: > This looks fine to me. Thank you! Thanks. I pushed this on Justin's behalf. -Andrew ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev_openvswitch.org

Re: [ovs-dev] [RPMs v2 1/4] xenserver: Remove support for XenServer versions older than 5.6 FP1.

2011-02-11 Thread Andrew Evans
On 2/11/11 1:03 PM, Ben Pfaff wrote: > This looks fine to me. Thank you! Thanks. I pushed this on Justin's behalf. -Andrew ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev_openvswitch.org

Re: [ovs-dev] [RPMs v2 2/4] xenserver: Remove brctl wrapper script.

2011-02-11 Thread Andrew Evans
On 2/11/11 1:03 PM, Ben Pfaff wrote: > This looks fine to me. Thank you! Thanks. I pushed this on Justin's behalf. -Andrew ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev_openvswitch.org

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Andrew Evans
On 2/11/11 1:02 PM, Ben Pfaff wrote: > On Fri, Feb 11, 2011 at 12:39:51PM -0800, Andrew Evans wrote: >> Ok to push this? > > Yes, it's fine, thanks. If I'm correct about the > InterfaceReconfigureBridge.py change, that can always go in another patch. Thanks. I pushed this. -Andrew ___

Re: [ovs-dev] [RPMs v2 3/4] xenserver: Rename ovs-external-ids to ovs-xapi-sync.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 12:35:37PM -0800, Justin Pettit wrote: > With commit 5692e3 (xenserver: Set fail_mode on internal bridges.), the > responsibilities of ovs-external-ids got expanded beyond just monitoring > external-ids. This commit renames the script to more accurately > describe its job.

Re: [ovs-dev] [RPMs v2 2/4] xenserver: Remove brctl wrapper script.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 12:35:36PM -0800, Justin Pettit wrote: > The OVS "brctl" wrapper script was added to aid Citrix's QA and help > XenCenter display link status and statistics of bridges. Since the Citrix > integrated OVS ships without this wrapper script, it's clearly no longer > needed in t

Re: [ovs-dev] [RPMs v2 1/4] xenserver: Remove support for XenServer versions older than 5.6 FP1.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 12:35:35PM -0800, Justin Pettit wrote: > The commit "xenserver: Merge changes to scripts from XenServer 5.6.100." > breaks compatibility with XenServers older than 5.6 FP1. This > commit removes the last vestiges of support for those older XenServer > versions. > > Signed-

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 12:39:51PM -0800, Andrew Evans wrote: > Ok to push this? Yes, it's fine, thanks. If I'm correct about the InterfaceReconfigureBridge.py change, that can always go in another patch. ___ dev mailing list dev@openvswitch.org http:/

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 12:37:43PM -0800, Ethan Jackson wrote: > > I doubt it. ?Ethan? > > I added the hashing-algorithm along with the following logic as part > of the lacp changes. > I'm pretty sure it's required, but if the logic pasted below isn't > there it won't do anything. > > if mode

Re: [ovs-dev] [RPMs v2 4/4] xenserver: Don't create XAPI cache at install.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 12:35:38PM -0800, Justin Pettit wrote: > From: Bob Ball > > There are two mechanisms of installing a supplemental pack ??? one is > after installation (using the xe-install-supplemental-pack command) and > one is during installation by either creating a new install ISO wit

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Andrew Evans
On 2/11/11 12:37 PM, Ethan Jackson wrote: >> >> I doubt it. Ethan? >> > > I added the hashing-algorithm along with the following logic as part > of the lacp changes. > I'm pretty sure it's required, but if the logic pasted below isn't > there it won't do anything. > > if mode == 'lacp': >

Re: [ovs-dev] InterfaceReconfigureVswitch.py fix didn't make it into xen-api?

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 08:02:40PM +, Ian Campbell wrote: > BTW xen-api development has recently moved to git at github.com. Would > you be willing to use the github tools to create pull requests against > xen-api.git, which is supposed to help track things and stop them > getting forgotten abo

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Andrew Evans
On 2/11/11 12:17 PM, Ben Pfaff wrote: > On Fri, Feb 11, 2011 at 11:44:11AM -0800, Andrew Evans wrote: >> On 2/11/11 10:40 AM, Ben Pfaff wrote: >>> This commit should update the documentation to remove references to >>> XenServer 5.5 and 5.6 support. At least INSTALL.Linux and >>> INSTALL.XenServer

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Ethan Jackson
> > I doubt it.  Ethan? > I added the hashing-algorithm along with the following logic as part of the lacp changes. I'm pretty sure it's required, but if the logic pasted below isn't there it won't do anything. if mode == 'lacp': argv += ['lacp=active'] if halgo == 'src_mac':

Re: [ovs-dev] [PATCH 4/4] xenserver: Don't require /etc/xensource-inventory at install.

2011-02-11 Thread Justin Pettit
On Feb 11, 2011, at 11:26 AM, Ben Pfaff wrote: > On Fri, Feb 11, 2011 at 10:55:21AM -0800, Justin Pettit wrote: >> From: Bob Ball >> >> There are two mechanisms of installing a supplemental pack ??? one is >> after installation (using the xe-install-supplemental-pack command) and >> one is durin

[ovs-dev] [RPMs v2 4/4] xenserver: Don't create XAPI cache at install.

2011-02-11 Thread Justin Pettit
From: Bob Ball There are two mechanisms of installing a supplemental pack – one is after installation (using the xe-install-supplemental-pack command) and one is during installation by either creating a new install ISO with the supplemental pack bundled or creating an HTTP directory which serves

[ovs-dev] [RPMs v2 1/4] xenserver: Remove support for XenServer versions older than 5.6 FP1.

2011-02-11 Thread Justin Pettit
The commit "xenserver: Merge changes to scripts from XenServer 5.6.100." breaks compatibility with XenServers older than 5.6 FP1. This commit removes the last vestiges of support for those older XenServer versions. Signed-off-by: Justin Pettit --- INSTALL.Linux

[ovs-dev] [RPMs v2 2/4] xenserver: Remove brctl wrapper script.

2011-02-11 Thread Justin Pettit
The OVS "brctl" wrapper script was added to aid Citrix's QA and help XenCenter display link status and statistics of bridges. Since the Citrix integrated OVS ships without this wrapper script, it's clearly no longer needed in the versions of XenServer we now support. Signed-off-by: Justin Pettit

[ovs-dev] [RPMs v2 3/4] xenserver: Rename ovs-external-ids to ovs-xapi-sync.

2011-02-11 Thread Justin Pettit
With commit 5692e3 (xenserver: Set fail_mode on internal bridges.), the responsibilities of ovs-external-ids got expanded beyond just monitoring external-ids. This commit renames the script to more accurately describe its job. Signed-off-by: Justin Pettit --- xenserver/README

Re: [ovs-dev] [PATCH 2/4] xenserver: Remove brctl wrapper script.

2011-02-11 Thread Justin Pettit
On Feb 11, 2011, at 11:22 AM, Ben Pfaff wrote: > On Fri, Feb 11, 2011 at 10:55:19AM -0800, Justin Pettit wrote: >> The OVS "brctl" wrapper script was added to aid Citrix's QA and help >> XenCenter display link status and statistics of bridges. Since the Citrix >> integrated OVS ships without this

Re: [ovs-dev] [PATCH 1/4] xenserver: Remove support for XenServer versions older than 5.6 FP1.

2011-02-11 Thread Justin Pettit
On Feb 11, 2011, at 11:21 AM, Ben Pfaff wrote: > On Fri, Feb 11, 2011 at 10:55:18AM -0800, Justin Pettit wrote: >> The commit "xenserver: Merge changes to scripts from XenServer 5.6.100." >> breaks compatibility with XenServers older than 5.6 FP1. This >> commit removes the last vestiges of suppo

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 11:44:11AM -0800, Andrew Evans wrote: > On 2/11/11 10:40 AM, Ben Pfaff wrote: > > This commit should update the documentation to remove references to > > XenServer 5.5 and 5.6 support. At least INSTALL.Linux and > > INSTALL.XenServer need to be updated. > > Justin was doin

Re: [ovs-dev] [PATCH 3/4] xenserver: Rename ovs-external-ids to ovs-xapi-sync.

2011-02-11 Thread Ethan Jackson
Thanks for doing this, I probably should have done this at the time I wrote the patch that added fail_mode to ovs-external-ids. Ethan On Fri, Feb 11, 2011 at 10:55 AM, Justin Pettit wrote: > With commit 5692e3 (xenserver: Set fail_mode on internal bridges.), the > responsibilities of ovs-extern

Re: [ovs-dev] InterfaceReconfigureVswitch.py fix didn't make it into xen-api?

2011-02-11 Thread Ian Campbell
On Fri, 2011-02-11 at 18:23 +, Ben Pfaff wrote: > The following fix to InterfaceReconfigureVswitch.py was committed to > the OVS repository in July 2010. It has an ack from Ian and I CC'd it > to Dominic, but it looks to me like it didn't make it upstream into > xen-api.hg. Could you check o

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Andrew Evans
On 2/11/11 10:40 AM, Ben Pfaff wrote: > On Thu, Feb 10, 2011 at 10:09:00PM -0800, Andrew Evans wrote: > This should be in the patch subject, since it is important. Maybe: > xenserver: Merge upstream changes and drop pre-5.6.100 support. Ok, done. > This commit should update the documentation to

Re: [ovs-dev] [PATCH 4/4] xenserver: Don't require /etc/xensource-inventory at install.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 10:55:21AM -0800, Justin Pettit wrote: > From: Bob Ball > > There are two mechanisms of installing a supplemental pack ??? one is > after installation (using the xe-install-supplemental-pack command) and > one is during installation by either creating a new install ISO wit

Re: [ovs-dev] [PATCH 3/4] xenserver: Rename ovs-external-ids to ovs-xapi-sync.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 10:55:20AM -0800, Justin Pettit wrote: > With commit 5692e3 (xenserver: Set fail_mode on internal bridges.), the > responsibilities of ovs-external-ids got expanded beyond just monitoring > external-ids. This commit renames the script to more accurately > describe its job.

Re: [ovs-dev] [PATCH 2/4] xenserver: Remove brctl wrapper script.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 10:55:19AM -0800, Justin Pettit wrote: > The OVS "brctl" wrapper script was added to aid Citrix's QA and help > XenCenter display link status and statistics of bridges. Since the Citrix > integrated OVS ships without this wrapper script, its clearly no longer s/its/it's/

Re: [ovs-dev] [PATCH 1/4] xenserver: Remove support for XenServer versions older than 5.6 FP1.

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 10:55:18AM -0800, Justin Pettit wrote: > The commit "xenserver: Merge changes to scripts from XenServer 5.6.100." > breaks compatibility with XenServers older than 5.6 FP1. This > commit removes the last vestiges of support for those older XenServer > versions. > > Signed-

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Justin Pettit
On Feb 11, 2011, at 10:40 AM, Ben Pfaff wrote: > On Thu, Feb 10, 2011 at 10:09:00PM -0800, Andrew Evans wrote: >> Citrix have made modifications to the various interface configuration scripts >> that OVS replaces, so at present those modifications are lost when OVS is >> installed on e.g. XenServe

[ovs-dev] [PATCH 3/4] xenserver: Rename ovs-external-ids to ovs-xapi-sync.

2011-02-11 Thread Justin Pettit
With commit 5692e3 (xenserver: Set fail_mode on internal bridges.), the responsibilities of ovs-external-ids got expanded beyond just monitoring external-ids. This commit renames the script to more accurately describe its job. Signed-off-by: Justin Pettit --- xenserver/README

[ovs-dev] [PATCH 2/4] xenserver: Remove brctl wrapper script.

2011-02-11 Thread Justin Pettit
The OVS "brctl" wrapper script was added to aid Citrix's QA and help XenCenter display link status and statistics of bridges. Since the Citrix integrated OVS ships without this wrapper script, its clearly no longer needed in the versions of XenServer we now support. Signed-off-by: Justin Pettit

[ovs-dev] [PATCH 1/4] xenserver: Remove support for XenServer versions older than 5.6 FP1.

2011-02-11 Thread Justin Pettit
The commit "xenserver: Merge changes to scripts from XenServer 5.6.100." breaks compatibility with XenServers older than 5.6 FP1. This commit removes the last vestiges of support for those older XenServer versions. Signed-off-by: Justin Pettit --- INSTALL.Linux

[ovs-dev] [PATCH 4/4] xenserver: Don't require /etc/xensource-inventory at install.

2011-02-11 Thread Justin Pettit
From: Bob Ball There are two mechanisms of installing a supplemental pack – one is after installation (using the xe-install-supplemental-pack command) and one is during installation by either creating a new install ISO with the supplemental pack bundled or creating an HTTP directory which serves

Re: [ovs-dev] [PATCH] xenserver: Merge changes to scripts from XenServer 5.6.100.

2011-02-11 Thread Ben Pfaff
On Thu, Feb 10, 2011 at 10:09:00PM -0800, Andrew Evans wrote: > Citrix have made modifications to the various interface configuration scripts > that OVS replaces, so at present those modifications are lost when OVS is > installed on e.g. XenServer 5.6.100. This commit applies those changes while >

[ovs-dev] HI

2011-02-11 Thread favour williams
Hello Dear, My name is favour. a female. I am a nice loving and good looking young lady. I came across your  page in www.ayola.net and I picked interest in you. Can we be friends? Contact me back direct with this my private email address (favour_willi...@att.net) so that i can send you my photos as

[ovs-dev] InterfaceReconfigureVswitch.py fix didn't make it into xen-api?

2011-02-11 Thread Ben Pfaff
The following fix to InterfaceReconfigureVswitch.py was committed to the OVS repository in July 2010. It has an ack from Ian and I CC'd it to Dominic, but it looks to me like it didn't make it upstream into xen-api.hg. Could you check on that for me? As background, we'd like OVS to not have to r

Re: [ovs-dev] switch local port

2011-02-11 Thread Ben Pfaff
On Fri, Feb 11, 2011 at 06:05:08PM +0100, Gaetano Catalli wrote: > I have some doubts to resolve about switch local port. > Firstly, in the ovs-openflowd manpage I can read: "...The Open???Flow > local port is a virtual network port that ovs-openflowd bridges to the > physical switch ports...". I'm

[ovs-dev] switch local port

2011-02-11 Thread Gaetano Catalli
Hi all, I have some doubts to resolve about switch local port. Firstly, in the ovs-openflowd manpage I can read: "...The Open‐Flow local port is a virtual network port that ovs-openflowd bridges to the physical switch ports...". I'm searching through the sources but I can't find out where this brid