The discussion about the ‘Area’ prefix has come up again, even after Ben had 
commented about it
and after I had pointed folks to the submitting-patches.rst, which allows 
flexibility in choosing an
‘Area’ prefix by the patch submitter.

In this thread, it was again suggested that the use of ‘Area’ prefix ‘System 
Tests’ needs to change to ‘System-traffic’
Below, I list some the history regarding previous commits of system-traffic.at 
as a single patch.
Different people have had different preferences and those seem to have been 
tolerated in the past.

Hence, I would like know what has changed recently such that the documented
(submitting-patches.rst) and historical flexibility (see previous patches) 
regarding the
‘Area’ prefix is no longer tolerated ?

My suggestion is that we don’t continue along these new lines and rather stay 
flexible, as the 
work will be more productive in such an environment.

Darrell


/////////////////////

commit 9d3e0e5c196c0a91ea23d8d9254b1487cb58b58e
Author: Jarno Rajahalme <ja...@ovn.org>
Date:   Wed Mar 8 17:18:23 2017 -0800

    tests: Add an FTP test without conntrack.
    
    If FTP tests with conntrack fail, it is informative to know if the
    problem is with the FTP client and/or server, or with conntrack
    itself.
    
    Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
    Acked-by: Joe Stringer <j...@ovn.org>


/////////////////////

commit d0e4206230b31ab8dde44b6e8896c10b6317b1a8
Author: Jarno Rajahalme <ja...@ovn.org>
Date:   Fri Mar 10 16:10:41 2017 -0800

    tests: ICMP related to original direction test.
    
    Normally ICMP responses are in the reply direction of a conntrack
    entry.  This test exercises an ICMP response to the original direction
    of the conntrack entry.
    
    Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
    Acked-by: Joe Stringer j...@ovn.org

/////////////////////

commit 2fa3e06d35988ee24ce1cc0f62ccceb3862038a1
Author: Jarno Rajahalme <ja...@ovn.org>
Date:   Wed Nov 25 16:04:59 2015 -0800

    system-tests: Add IPv6 FTP system test.
    
    Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
    Acked-by: Joe Stringer <j...@ovn.org>


/////////////////////

commit e5cf8cce275934549ee1b1ed41d60d5b6ce7918d
Author: Daniele Di Proietto <diproiet...@vmware.com>
Date:   Mon Apr 25 19:06:40 2016 -0700

    system-tests: Add ping through conntrack test.
    
    Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com>
    Acked-by: Joe Stringer j...@ovn.org

/////////////////////

commit e0b9270136465f1c5379c274ca41081980bcd076
Author: Daniele Di Proietto <diproiet...@vmware.com>
Date:   Mon Apr 11 14:02:10 2016 -0700

    system-tests: Add tcp simple test.
    
    Useful to test the datapath ability to forward tcp packets without the
    complexity of connection tracking.
    
    Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com>
    Acked-by: Joe Stringer <j...@ovn.org>
    Acked-by: Flavio Leitner f...@sysclose.org


/////////////////////

commit 6cfa8ec3e3c6bfdda3982e50721549d7d7853a91
Author: Jarno Rajahalme <ja...@ovn.org>
Date:   Tue Nov 24 13:33:22 2015 -0800

    system-tests: Use '--bundle'
    
    Use OpenFlow bundles for setting up flow tables.  This has the benefit
    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>

/////////////////////







-----Original Message-----
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.

    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-dev] [patch_v1 1/2] System Tests: Allow SNAT address 
variability retries.
    
        On 16 July 2017 at 11:27, Darrell Ball <dlu...@gmail.com> 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 <dlu...@gmail.com>
        
        Thanks for the patch, I noticed this behaviour occasionally myself but
        hadn't dug into it yet.
        
        Per Ilya's request, I updated the prefix. Will apply to master shortly.
        _______________________________________________
        dev mailing list
        d...@openvswitch.org
        
https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=bMwLwPwNQR3XbY6KCxI54HZPRTBWPcrSF4_ikMQj44Q&s=9m-0DFAgXSLjCzA6DDzcTgCX-Evii2MGmQsDncTuikM&e=
 
        
    
    

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to