Re: [ovs-dev] [rhel-5.6 2/2] Add RHEL-5.6 spec file and instructions.

2011-06-20 Thread Ben Pfaff
On Mon, Jun 20, 2011 at 12:39:33AM -0700, Justin Pettit wrote: > I haven't had a chance to read through the entire patch yet, but I > think you stripped out punching a hole in iptables for GRE tunnels > from the base ovs-ctl script since you sent this patch out. Did you > want to add a rule to the

Re: [ovs-dev] [rhel-5.6 2/2] Add RHEL-5.6 spec file and instructions.

2011-06-20 Thread Justin Pettit
I haven't had a chance to read through the entire patch yet, but I think you stripped out punching a hole in iptables for GRE tunnels from the base ovs-ctl script since you sent this patch out. Did you want to add a rule to the RHEL-5.6 init script to do this? Also, do you have thoughts on how

Re: [ovs-dev] [rhel-5.6 2/2] Add RHEL-5.6 spec file and instructions.

2011-06-17 Thread Ben Pfaff
On Fri, Jun 17, 2011 at 01:29:10PM -0700, Andrew Evans wrote: > Thanks, this looks good. I have only a few minor comments: > > On Thu, 2011-06-16 at 12:25 -0700, Ben Pfaff wrote: > > diff --git a/INSTALL.RHEL-5.6 b/INSTALL.RHEL-5.6 > > Should we change the RHEL 5.6 references to RHEL 5? RHEL 5.7

Re: [ovs-dev] [rhel-5.6 2/2] Add RHEL-5.6 spec file and instructions.

2011-06-17 Thread Andrew Evans
Thanks, this looks good. I have only a few minor comments: On Thu, 2011-06-16 at 12:25 -0700, Ben Pfaff wrote: > diff --git a/INSTALL.RHEL-5.6 b/INSTALL.RHEL-5.6 Should we change the RHEL 5.6 references to RHEL 5? RHEL 5.7 is currently in beta, and there will probably be future RHEL 5 minor relea

[ovs-dev] [rhel-5.6 2/2] Add RHEL-5.6 spec file and instructions.

2011-06-16 Thread Ben Pfaff
--- INSTALL.RHEL-5.6 | 53 + Makefile.am|2 + README |9 +- rhel-5.6/automake.mk | 13 ++ rhel-5.6/etc_init.d_openvswitch