Hi,
this series contains the sole patch that I made to 1.4.0
when preparing to upload 1.4.0-1 to Debian.
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
This is necessary as openvswitch-test makes use of debhelper.
As reported by lintian.
Signed-off-by: Simon Horman
---
debian/control |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/debian/control b/debian/control
index 5861447..4f71465 100644
--- a/debian/control
+++ b
openvswitch_1.4.0-1_amd64.changes uploaded successfully to localhost
along with the files:
openvswitch_1.4.0-1.dsc
openvswitch_1.4.0.orig.tar.gz
openvswitch_1.4.0-1.debian.tar.gz
openvswitch-common_1.4.0-1_amd64.deb
openvswitch-switch_1.4.0-1_amd64.deb
openvswitch-ipsec_1.4.0-1_amd64.de
Version: 1.4.0-1
On Tue, Jan 24, 2012 at 02:38:00PM -0800, Ben Pfaff wrote:
> # Update Debian BTS.
> tags 656448 + confirmed upstream fixed-upstream
> thanks
>
> Thanks, I pushed this fix to master, branch-1.2, branch-1.3,
> branch-1.4, branch-1.5. It will appear in the next Debian upload.
I h
On Wed, Dec 07, 2011 at 08:22:28PM -0800, Ben Pfaff wrote:
> OpenFlow 1.1 was released about 9 months ago. Open vSwitch doesn't
> yet have support for this protocol version. Furthermore, OpenFlow 1.2
> will likely be released by the end of 2011.
[snip]
> How to contribute
> -
>
openvswitch-brcompat_1.4.0-1_amd64.deb
to main/o/openvswitch/openvswitch-brcompat_1.4.0-1_amd64.deb
openvswitch-common_1.4.0-1_amd64.deb
to main/o/openvswitch/openvswitch-common_1.4.0-1_amd64.deb
openvswitch-controller_1.4.0-1_amd64.deb
to main/o/openvswitch/openvswitch-controller_1.4.0-1_amd
Your message dated Fri, 3 Feb 2012 17:45:14 +0900
with message-id <20120203084513.gd17...@verge.net.au>
and subject line Re: [ovs-dev] Bug#637802: Bug #637802: DKMS support now
implemented upstream
has caused the Debian Bug report #637802,
regarding Please add DKMS support
to be marked as done.
T
Your message dated Fri, 3 Feb 2012 17:38:48 +0900
with message-id <20120203083843.ga17...@verge.net.au>
and subject line Re: [ovs-dev] Bug#651798: Bug#651798: Fails to build against
Linux 3.1
has caused the Debian Bug report #651798,
regarding Fails to build against Linux 3.1
to be marked as done.
Your message dated Fri, 3 Feb 2012 17:40:19 +0900
with message-id <20120203084018.gb17...@verge.net.au>
and subject line Re: [ovs-dev] Bug#656448: [PATCH] debian: Delete log files
correctly when packages are purged.
has caused the Debian Bug report #656448,
regarding openvswitch-pki: unowned files
Your message dated Fri, 3 Feb 2012 17:41:54 +0900
with message-id <20120203084154.gc17...@verge.net.au>
and subject line Re: [ovs-dev] Bug#657115: Bug#657115:
openvswitch-datapath-dkms: Failed to compile with kernel 3.2
has caused the Debian Bug report #657115,
regarding openvswitch-datapath-dkms:
On Tue, Jan 31, 2012 at 01:53:49PM -0800, Gurucharan Shetty wrote:
> + case ${OVSDHCPINTERFACES} in
> + ${OVSINTF#ifcfg-} | \
> + "${OVSINTF#ifcfg-},"* | \
> + *",${OVSINTF#ifcfg-}" | \
> +
On Thu, Feb 2, 2012 at 7:05 PM, Jesse Gross wrote:
> On Thu, Feb 2, 2012 at 6:58 AM, Pravin B Shelar wrote:
>> cleanups:
>> - removes flex_array_shrink() as there is no caller.
>> - add flex_array_init()
>> - Add few 'const's.
>>
>> Signed-off-by: Pravin B Shelar
>
> Since w
On Fri, Feb 03, 2012 at 05:51:22PM +0900, Simon Horman wrote:
> On Wed, Dec 07, 2011 at 08:22:28PM -0800, Ben Pfaff wrote:
> > OpenFlow 1.1 was released about 9 months ago. Open vSwitch doesn't
> > yet have support for this protocol version. Furthermore, OpenFlow 1.2
> > will likely be released b
On Fri, Feb 03, 2012 at 05:34:17PM +0900, Simon Horman wrote:
> This is necessary as openvswitch-test makes use of debhelper.
>
> As reported by lintian.
>
> Signed-off-by: Simon Horman
Thanks. I pushed this to master, branch-1.5, branch-1.4.
___
dev
On Fri, Feb 03, 2012 at 05:34:16PM +0900, Simon Horman wrote:
> this series contains the sole patch that I made to 1.4.0
> when preparing to upload 1.4.0-1 to Debian.
Thank you very much for the upload, and for the patch.
___
dev mailing list
dev@openvsw
Suggested-by: "Alexey I. Froloff"
Signed-off-by: Ben Pfaff
---
Not tested, please scrutinize!
rhel/etc_sysconfig_network-scripts_ifup-ovs |9 +++--
1 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/rhel/etc_sysconfig_network-scripts_ifup-ovs
b/rhel/etc_sysconfig_network-sc
On Fri, Feb 03, 2012 at 04:30:36PM +0400, Alexey I. Froloff wrote:
> On Tue, Jan 31, 2012 at 01:53:49PM -0800, Gurucharan Shetty wrote:
> > + case ${OVSDHCPINTERFACES} in
> > + ${OVSINTF#ifcfg-} | \
> > + "${OVSINTF#ifcfg-},"* |
On Thu, Feb 02, 2012 at 05:51:22PM -0800, Justin Pettit wrote:
> Ethan already covered the code part, but just a couple of questions on the
> documentation:
>
> > +Bitwise matches on transport ports are rarely useful in isolation, but
> > +a group of them can be used to reduce the number of flows
Looks good.
On Fri, Feb 3, 2012 at 9:19 AM, Ben Pfaff wrote:
> Suggested-by: "Alexey I. Froloff"
> Signed-off-by: Ben Pfaff
> ---
> Not tested, please scrutinize!
>
> rhel/etc_sysconfig_network-scripts_ifup-ovs |9 +++--
> 1 files changed, 3 insertions(+), 6 deletions(-)
>
> diff --gi
Thanks.
I dropped the update to the copyright notice (which happened
accidentally) and pushed this.
On Fri, Feb 03, 2012 at 09:33:11AM -0800, Gurucharan Shetty wrote:
> Looks good.
>
> On Fri, Feb 3, 2012 at 9:19 AM, Ben Pfaff wrote:
>
> > Suggested-by: "Alexey I. Froloff"
> > Signed-off-by:
>
>
> Guru, should we make that change? I suggested a comma originally
> because I though it would be easier to write the shell fragments (no
> need for quotes) but I didn't realize that there was precedent for
> spaces.
>
> For consistency, it may be a good idea. I will come up with the patch for
On Fri, Feb 03, 2012 at 09:37:18AM -0800, Gurucharan Shetty wrote:
> >
> >
> > Guru, should we make that change? I suggested a comma originally
> > because I though it would be easier to write the shell fragments (no
> > need for quotes) but I didn't realize that there was precedent for
> > spaces
On Fri, Feb 03, 2012 at 09:21:29AM -0800, Ben Pfaff wrote:
> My understanding is that ordinary Linux bridges don't work that way
> with the RHEL network scripts. Is that correct? I think that it is
> a good idea for OVS and the Linux bridge to act the same way, to the
> extent that it is possible
> My intention in inventing the dl_, nw_, and tp_ prefixes in struct flow
> way back when was for them to stand for "data link", "network", and
> "transport", the names of layer 2 through 4. So "tp" seems a little
> truncated to me. How about "tp_port"?
Ahhh, I had thought tp stands for "transpo
Processing commands for cont...@bugs.debian.org:
> # Fri Feb 3 19:03:08 UTC 2012
> # Tagging as pending bugs that are closed by packages in NEW
> # http://ftp-master.debian.org/new.html
> #
> # Source package in NEW: openvswitch
> tags 653645 + pending
Bug #653645 [openvswitch-common] openvswitch
There's a redundant newline at the end of xenserver/automake.mk
Otherwise looks good to me assuming you tried the new instructions.
Ethan
On Thu, Feb 2, 2012 at 16:38, Ben Pfaff wrote:
> I got tired of trying to follow them properly myself.
>
> Signed-off-by: Ben Pfaff
> ---
> INSTALL.XenServ
Use space separated list of DHCP interfaces for OVSDHCPINTERFACES
instead of comma separated list. This is done to maintain
consistency with BOND_IFACES.
Signed-off-by: Gurucharan Shetty
---
rhel/README.RHEL|2 +-
rhel/etc_sysconfig_network-scripts_ifup-ovs |8
On Fri, Feb 03, 2012 at 12:27:37PM -0800, Gurucharan Shetty wrote:
> Use space separated list of DHCP interfaces for OVSDHCPINTERFACES
> instead of comma separated list. This is done to maintain
> consistency with BOND_IFACES.
>
> Signed-off-by: Gurucharan Shetty
Looks good to me. Thank you.
__
Found by valgrind.
Signed-off-by: Ben Pfaff
---
utilities/ovs-ofctl.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c
index 9b1a1b2..7df0a90 100644
--- a/utilities/ovs-ofctl.c
+++ b/utilities/ovs-ofctl.c
@@ -447,6 +447,8 @
Looks good,
Ethan
On Fri, Feb 3, 2012 at 12:38, Ben Pfaff wrote:
> Found by valgrind.
>
> Signed-off-by: Ben Pfaff
> ---
> utilities/ovs-ofctl.c |4
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c
> index 9b1a1b2..7df0a
This is a revised version of the of1.1 draft series originally
posted starting here:
http://openvswitch.org/pipermail/dev/2011-December/013585.html
A few of the original patches have since been polished up and
committed, so those are not present here. This is also true
of some parts of the conten
This prepares for a gradual introduction of definitions from OpenFlow
1.1 and later, by making it clearer that the current definitions are
specific to OpenFlow 1.0.
---
build-aux/check-structs | 51 +-
include/openflow/automake.mk| 26 +-
include/ope
The intention is that, as each OpenFlow 1.1 feature is added to Open
vSwitch, the corresponding protocol definitions will be broken up this way:
- Definitions that are the same in OF1.0 and OF1.1 will retain the "OFP"
or "ofp" prefix and move to openflow-common.h.
- Definitions that are s
OpenFlow 1.1 extends port numbers to 32 bits. Initially we plan to support
only port numbers in the 16-bit range in Open vSwitch. However the OF1.1
reserved ports have high-valued fixed numbers that require translation to
high fixed values in the 16-bit range for OF1.0. These new functions
provi
---
include/openflow/openflow-1.0.h| 41 ++-
include/openflow/openflow-1.1.h| 23 +++-
include/openflow/openflow-common.h | 22 +++
lib/ofp-print.c| 18
lib/ofp-util.c
Thank you. I pushed it.
On Fri, Feb 03, 2012 at 12:40:06PM -0800, Ethan Jackson wrote:
> Looks good,
>
> Ethan
>
> On Fri, Feb 3, 2012 at 12:38, Ben Pfaff wrote:
> > Found by valgrind.
> >
> > Signed-off-by: Ben Pfaff
> > ---
> > utilities/ovs-ofctl.c |4
> > 1 files changed, 4 inse
Hi,
please note openvswitch-test contains files for python2.4, which has not been
a supported version for lots of time.
Cheers,
Luca
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Python 2.4 is obsolete, not present in Debian squeeze or sid, so don't
install Python modules for it.
It would be better to just put the files directly into
/usr/share/pyshared/ovs/, instead of in site-packages for some specific
Python version, but this causes problems for builds on squeeze, as
do
On Fri, Feb 03, 2012 at 11:37:03AM -0800, Ethan Jackson wrote:
> There's a redundant newline at the end of xenserver/automake.mk
Thanks, I removed it.
> Otherwise looks good to me assuming you tried the new instructions.
They work for me.
I'll push this soon.
Thanks,
Ben.
On Fri, Feb 03, 2012 at 07:56:56AM -0800, Ben Pfaff wrote:
> On Fri, Feb 03, 2012 at 05:51:22PM +0900, Simon Horman wrote:
> > On Wed, Dec 07, 2011 at 08:22:28PM -0800, Ben Pfaff wrote:
> > > OpenFlow 1.1 was released about 9 months ago. Open vSwitch doesn't
> > > yet have support for this protoco
Looks good, thanks.
Ethan
On Fri, Feb 3, 2012 at 13:33, Ben Pfaff wrote:
> Python 2.4 is obsolete, not present in Debian squeeze or sid, so don't
> install Python modules for it.
>
> It would be better to just put the files directly into
> /usr/share/pyshared/ovs/, instead of in site-packages f
Accepted:
openvswitch-brcompat_1.4.0-1_amd64.deb
to main/o/openvswitch/openvswitch-brcompat_1.4.0-1_amd64.deb
openvswitch-common_1.4.0-1_amd64.deb
to main/o/openvswitch/openvswitch-common_1.4.0-1_amd64.deb
openvswitch-controller_1.4.0-1_amd64.deb
to main/o/openvswitch/openvswitch-controlle
Your message dated Fri, 03 Feb 2012 21:42:41 +
with message-id
and subject line Bug#653645: fixed in openvswitch 1.4.0-1
has caused the Debian Bug report #653645,
regarding openvswitch-common: Please depend on "python (>= 2.7) |
python-argparse"
to be marked as done.
This means that you clai
Thank you for the review. I pushed this to "master". I'll push it to
maintenance branches once the autobuilder gives me the thumbs-up.
Thanks Ethan and Luca,
Ben.
On Fri, Feb 03, 2012 at 01:37:55PM -0800, Ethan Jackson wrote:
> Looks good, thanks.
>
> Ethan
>
> On Fri, Feb 3, 2012 at 13:33,
Nothing actually uses call_vsctl(), so remove it and its dependencies.
Signed-off-by: Ben Pfaff
---
.../usr_share_openvswitch_scripts_ovs-xapi-sync| 10 --
1 files changed, 0 insertions(+), 10 deletions(-)
diff --git a/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync
b/xense
Looks good to me.
Ethan
On Fri, Feb 3, 2012 at 15:40, Ben Pfaff wrote:
> Nothing actually uses call_vsctl(), so remove it and its dependencies.
>
> Signed-off-by: Ben Pfaff
> ---
> .../usr_share_openvswitch_scripts_ovs-xapi-sync| 10 --
> 1 files changed, 0 insertions(+), 10 del
Otherwise coverage log messages will eventually get rotated out of the
system logs and never reappear (before restarting a given daemon).
Bug #1452.
Signed-off-by: Ben Pfaff
---
lib/coverage.c | 13 +++--
1 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/lib/coverage.c b/
Seems fine to me.
Ethan
On Thu, Jan 26, 2012 at 15:53, Ben Pfaff wrote:
> I don't remember why we disallow this. It seems reasonable to me to enable
> it. A test in a upcoming commit relies upon the ability.
>
> Signed-off-by: Ben Pfaff
> ---
> ofproto/ofproto.c |4
> 1 files chang
On Fri, Feb 3, 2012 at 5:45 AM, Pravin Shelar wrote:
> On Thu, Feb 2, 2012 at 7:05 PM, Jesse Gross wrote:
>> It's not clear to me that flex_array_init() is that big of a win.
>> struct flex_array stores the base array inside of it, so it is defined
>> to be PAGE_SIZE. Since it is so large, stori
I don't think the question marks add much and they look a bit funny. Don't
feel strongly about it thought.
OFPT_FLOW_MOD in->is confusing
In the table "match on out port?" -> "match on out_port?" to be consistent
with the description above.
Otherwise looks good,
Ethan
On Thu, Jan 26, 2012 at
50 matches
Mail list logo
