Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-21 Thread Joe Stringer
5:24 AM >> To: Darrell Ball <db...@vmware.com>, Joe Stringer <j...@ovn.org> >> Cc: ovs dev <d...@openvswitch.org>, Ben Pfaff <b...@ovn.org> >> Subject: Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address >> variability retries. >> &g

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-21 Thread Joe Stringer
> Cc: ovs dev <d...@openvswitch.org>, Ben Pfaff <b...@ovn.org> > Subject: Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address > variability retries. > > On 21.07.2017 05:40, Darrell Ball wrote: > > The discussion about the ‘Area’ prefix has come

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-21 Thread Darrell Ball
oe Stringer <j...@ovn.org>, Darrell Ball <dlu...@gmail.com> > Cc: ovs dev <d...@openvswitch.org> > Subject: Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries. > > Please don’t update the prefix > &g

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-21 Thread Ilya Maximets
gt; that when debugging test failures, no packet gets processed by > partially set-up flow table, which may seem confusing. > > Signed-off-by: Jarno Rajahalme <ja...@ovn.org> > Acked-by: Ben Pfaff <b...@ovn.org> > > / > &g

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-20 Thread Darrell Ball
From: Darrell Ball <db...@vmware.com> Date: Thursday, July 20, 2017 at 11:17 AM To: Joe Stringer <j...@ovn.org>, Darrell Ball <dlu...@gmail.com> Cc: ovs dev <d...@openvswitch.org> Subject: Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-20 Thread Joe Stringer
On 20 July 2017 at 11:17, Darrell Ball wrote: > Please don’t update the prefix OK. As discussed offline, I'll wait for Darrell to respin this series. ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-20 Thread Darrell Ball
Please don’t update the prefix -Original Message- From: <ovs-dev-boun...@openvswitch.org> on behalf of Joe Stringer <j...@ovn.org> Date: Thursday, July 20, 2017 at 11:15 AM To: Darrell Ball <dlu...@gmail.com> Cc: ovs dev <d...@openvswitch.org> Subject: Re: [ovs-de

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-20 Thread Joe Stringer
On 16 July 2017 at 11:27, Darrell Ball wrote: > Three of the SNAT tests allow for wget retries, which occasionally > happen. However, these tests did not allow for SNAT address > variability for the retries, which is now tolerated. > > Signed-off-by: Darrell Ball

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-17 Thread Ben Pfaff
On Mon, Jul 17, 2017 at 10:03:47AM +0300, Ilya Maximets wrote: > Same thing here. Not a full review. > Please, use 'system-traffic' for these patches as a prefix. I'm seeing a lot more focus on area prefixes than I expect. These are supposed to be helpful to the reader, but I don't think it's

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

2017-07-17 Thread Ilya Maximets
Same thing here. Not a full review. Please, use 'system-traffic' for these patches as a prefix. Details are in previous e-mail: https://mail.openvswitch.org/pipermail/ovs-dev/2017-July/335751.html Best regards, Ilya Maximets. ___ dev mailing list