Tidy after recent merges 23/45723/1
authorVratko Polak <vrpolak@cisco.com>
Fri, 16 Sep 2016 09:41:29 +0000 (11:41 +0200)
committerVratko Polak <vrpolak@cisco.com>
Fri, 16 Sep 2016 09:41:29 +0000 (11:41 +0200)
Change-Id: Ib938ba8c10dfe952001dba317b6cfde39e1888a0
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
18 files changed:
csit/libraries/LISPFlowMapping.robot
csit/libraries/OpenStackOperations.robot
csit/libraries/VtnCoKeywords.robot
csit/suites/bgpcep/bgpfunct/050_bmp_basic.robot
csit/suites/genius/01_Interface_manager.robot
csit/suites/genius/02_ID_manager.robot
csit/suites/openstack/clustering/01_l2_tests.robot
csit/suites/openstack/clustering/02_l3_tests.robot
csit/suites/openstack/connectivity/01_l2_tests.robot
csit/suites/openstack/tempest/tempest.robot
csit/suites/sfc/Full_Deploy/010__sfc_full_deploy.robot
csit/suites/sfc/SFC_Basic/020__sfc_service_forwarders.robot
csit/suites/sfc/SFC_Basic/050__sfc_service_schedule_types.robot
csit/suites/sfc/SFC_Basic/060__sfc_service_paths.robot
csit/suites/unimgr/uni_operations/010_unimanager_restconf.robot
csit/suites/vtn/VTN_Mgr_OF10/010__vtn_vbrif_vlanmap.robot
csit/suites/vtn/VTN_Mgr_OF13/010__vtn_vbrif_vlanmap.robot
csit/variables/sfc/Variables.robot

index 570341a081f085941f66f6cf6a8e5450fc8cfaf1..9dad54ef0fad7e061376c0ec787fb5c3d83308e9 100644 (file)
@@ -57,11 +57,11 @@ Check Mapping Removal
     Post Log Check    ${LFM_RPC_API}:get-mapping    ${json}    404
 
 Get Mapping JSON
-    [Documentation]    Returns mapping record JSON dict
     [Arguments]    ${eid}    ${rloc}
+    [Documentation]    Returns mapping record JSON dict
     ${loc_record}=    Get LocatorRecord Object    ${rloc}
     ${lisp_address}=    Get LispAddress Object    ${eid}
     ${loc_record_list}=    Create List    ${loc_record}
     ${mapping_record_json}=    Get MappingRecord JSON    ${lisp_address}    ${loc_record_list}
     ${mapping}=    Wrap input    ${mapping_record_json}
-    [return]    ${mapping}
+    [Return]    ${mapping}
index 2f7704fdc8746891d1a84960ba2cce6a58d39477..83ba84a6db2aea3ac42012116285db40e827d3fe 100644 (file)
@@ -162,7 +162,7 @@ Get Port Id
     [Return]    ${port_id}
 
 Create Vm Instances
-    [Arguments]    ${net_name}    ${vm_instance_names}    ${image}=cirros-0.3.4-x86_64-uec    ${flavor}=m1.nano      ${sg}=default
+    [Arguments]    ${net_name}    ${vm_instance_names}    ${image}=cirros-0.3.4-x86_64-uec    ${flavor}=m1.nano    ${sg}=default
     [Documentation]    Create X Vm Instance with the net id of the Netowrk.
     ${devstack_conn_id}=    Get ControlNode Connection
     Switch Connection    ${devstack_conn_id}
@@ -396,14 +396,14 @@ Show Debugs
     List Ports
 
 Create Security Group
-    [Arguments]    ${sg_name}     ${desc}
+    [Arguments]    ${sg_name}    ${desc}
     ${devstack_conn_id}=    Get ControlNode Connection
     Switch Connection    ${devstack_conn_id}
-    ${output}=    Write Commands Until Prompt    nova secgroup-create ${sg_name} ${desc}      40s
+    ${output}=    Write Commands Until Prompt    nova secgroup-create ${sg_name} ${desc}    40s
     Close Connection
 
 Create Security Rule
-    [Arguments]    ${direction}     ${protocol}      ${min_port}     ${max_port}     ${remote_ip}     ${sg_name}
+    [Arguments]    ${direction}    ${protocol}    ${min_port}    ${max_port}    ${remote_ip}    ${sg_name}
     ${devstack_conn_id}=    Get ControlNode Connection
     Switch Connection    ${devstack_conn_id}
     ${output}=    Write Commands Until Prompt    neutron security-group-rule-create --direction ${direction} --protocol ${protocol} --port-range-min ${min_port} --port-range-max ${max_port} --remote-ip-prefix ${remote_ip} ${sg_name}
index edd39bbfcbdc3373a1580603d25479794684e0f6..229ce53409c89215a54e9ad50b129a7260ad3e9e 100644 (file)
@@ -12,10 +12,10 @@ Resource          ./Utils.robot
 
 *** variable ***
 ${vlan_topo}      sudo mn --controller=remote,ip=${ODL_SYSTEM_IP} --custom vlan_vtn_test.py --topo vlantopo
-${nexus_path}    https://nexus.opendaylight.org/content/repositories/opendaylight.snapshot/org/opendaylight/vtn/distribution.vtn-coordinator
-${boron}    6.3.0-SNAPSHOT
-${carbon}    6.4.0-SNAPSHOT
-${vtn_dist}    distribution.vtn-coordinator
+${nexus_path}     https://nexus.opendaylight.org/content/repositories/opendaylight.snapshot/org/opendaylight/vtn/distribution.vtn-coordinator
+${boron}          6.3.0-SNAPSHOT
+${carbon}         6.4.0-SNAPSHOT
+${vtn_dist}       distribution.vtn-coordinator
 
 *** Keywords ***
 Get VtnCo
@@ -43,13 +43,13 @@ Start SuiteVtnCo
     SSHLibrary.Execute Command    sudo yum install -q -y http://yum.postgresql.org/9.3/redhat/rhel-7-x86_64/pgdg-centos93-9.3-1.noarch.rpm
     SSHLibrary.Execute Command    sudo yum install -q -y postgresql93-libs postgresql93 postgresql93-server postgresql93-contrib postgresql93-odbc-09.03.0400
     Run Keyword If    '${ODL_STREAM}' == 'boron'    SSHLibrary.Execute Command    wget "${nexus_path}/${boron}/maven-metadata.xml"
-    ...    ELSE IF    '${ODL_STREAM}' == 'carbon'   SSHLibrary.Execute Command    wget "${nexus_path}/${carbon}/maven-metadata.xml"
-    Run Keyword If    '${ODL_STREAM}' == 'boron' or '${ODL_STREAM}' == 'carbon'   SSHLibrary.Get_file    maven-metadata.xml
+    ...    ELSE IF    '${ODL_STREAM}' == 'carbon'    SSHLibrary.Execute Command    wget "${nexus_path}/${carbon}/maven-metadata.xml"
+    Run Keyword If    '${ODL_STREAM}' == 'boron' or '${ODL_STREAM}' == 'carbon'    SSHLibrary.Get_file    maven-metadata.xml
     ${time_stamp}=    XML.Get Element Text    maven-metadata.xml    xpath=.//snapshot/timestamp
     ${build_number}=    XML.Get Element Text    maven-metadata.xml    xpath=.//snapshot/buildNumber
     Run Keyword If    '${ODL_STREAM}' == 'boron'    SSHLibrary.Execute Command    wget '${nexus_path}/${boron}/${vtn_dist}-6.3.0-${time_stamp}-${build_number}-bin.tar.bz2'
     ...    ELSE IF    '${ODL_STREAM}' == 'carbon'    SSHLibrary.Execute Command    wget '${nexus_path}/${carbon}/${vtn_dist}-6.4.0-${time_stamp}-${build_number}-bin.tar.bz2'
-    Run Keyword If    '${ODL_STREAM}' == 'boron' or '${ODL_STREAM}' == 'carbon'   SSHLibrary.Execute Command    tar -C/ -jxvf ${vtn_dist}*-bin.tar.bz2
+    Run Keyword If    '${ODL_STREAM}' == 'boron' or '${ODL_STREAM}' == 'carbon'    SSHLibrary.Execute Command    tar -C/ -jxvf ${vtn_dist}*-bin.tar.bz2
     ...    ELSE    SSHLibrary.Execute Command    tar -C/ -jxvf ${WORKSPACE}/${BUNDLEFOLDER}/externalapps/${vtn_dist}*-bin.tar.bz2
     SSHLibrary.Execute Command    /usr/local/vtn/sbin/db_setup
     SSHLibrary.Execute Command    /usr/local/vtn/bin/vtn_start
index e4239c60cd54131b2cf44be7b3693d4387472e8e..fb05935a89be2b637091cbb743368cc59407f35e 100644 (file)
@@ -25,7 +25,6 @@ Resource          ${CURDIR}/../../../libraries/NexusKeywords.robot
 Resource          ${CURDIR}/../../../libraries/Utils.robot
 Resource          ${CURDIR}/../../../libraries/TemplatedRequests.robot
 
-
 *** Variables ***
 ${CONFIG_SESSION}    config-session
 ${BGP_BMP_DIR}    ${CURDIR}/../../../variables/bgpfunctional/bmp_basic
@@ -38,10 +37,9 @@ Start_Bmp_Mock
     BuiltIn.Log    ${command}
     SSHLibrary.Set_Client_Configuration    timeout=30s
     SSHLibrary.Write    ${command}
-    ${until_phrase}=    CompareStream.Set_Variable_If_At_Least_Boron       successfully established.    sucesfully established.
+    ${until_phrase}=    CompareStream.Set_Variable_If_At_Least_Boron    successfully established.    sucesfully established.
     SSHLibrary.Read_Until    ${until_phrase}
 
-
 Verify Data Reported
     [Documentation]    Verifies if the tool reported expected data
     &{mapping}    BuiltIn.Create_Dictionary    TOOL_IP=${TOOLS_SYSTEM_IP}
index cb400ad4fca2caee2870e51600ed5a0f51a945e8..01146d2b865062843db015a93a6962d728910554 100644 (file)
@@ -145,7 +145,7 @@ no goto_table entry
 
 table0 entry
     [Arguments]    ${connection-id}    ${bridgename}
-    [Documentation]    After Creating the trunk interface , checking for  table 0 entry exist in the flow dumps
+    [Documentation]    After Creating the trunk interface , checking for table 0 entry exist in the flow dumps
     switch connection    ${connection-id}
     log    switch connection
     ${ovs-check}    execute command    sudo ovs-ofctl -O OpenFlow13 dump-flows ${bridgename}
@@ -155,7 +155,7 @@ table0 entry
 
 ovs check for member interface creation
     [Arguments]    ${connection-id}    ${bridgename}
-    [Documentation]    This keyword verifies the member interface created on OVS by checking the table0 ,vlan and action=pop_vlan entries    
+    [Documentation]    This keyword verifies the member interface created on OVS by checking the table0 ,vlan and action=pop_vlan entries
     switch connection    ${connection-id}
     ${ovs-check}    execute command    sudo ovs-ofctl -O OpenFlow13 dump-flows ${bridgename}
     log    ${ovs-check}
@@ -165,7 +165,7 @@ ovs check for member interface creation
 
 Create Interface
     [Arguments]    ${json_file}    ${interface_mode}
-    [Documentation]    Creates an trunk/transparent interface based on input provided to the json body                           
+    [Documentation]    Creates an trunk/transparent interface based on input provided to the json body
     ${body}    OperatingSystem.Get File    ${genius_config_dir}/${json_file}
     log    ${genius_config_dir}/${json_file}
     ${body}    replace string    ${body}    "l2vlan-mode":"trunk"    "l2vlan-mode":"${interface_mode}"
index 285e81b9380528d033595f0b3e7d005a9135fa7d..8a3ee31ac2c7ab2650d308170c85c836392466a7 100644 (file)
@@ -47,7 +47,7 @@ Neg_Allocate ids of size 10 from the same pool
     should be equal as strings    ${resp.status_code}    500
 
 Allocate IDs of size 3 from the pool
-    [Documentation]    This testcase allocates 3 Ids from the created pool in test case 1 
+    [Documentation]    This testcase allocates 3 Ids from the created pool in test case 1
     ${body}    OperatingSystem.Get File    ${genius_config_dir}/${allocaterange_json}
     ${body}    replace string    ${body}    test-key    ${test_keys[2]}
     ${body}    replace string    ${body}    5    3
index 075a7c865b5457ec1280522d1aa98b1c2ceec08e..6ec23a44b018767b02044059a80de9b90dff87d1 100644 (file)
@@ -88,7 +88,7 @@ Delete the Bridge Manually and Verify After Fail
 Create Vm Instances For l2_net_1
     [Documentation]    Create Vm instances using flavor and image names for a network.
     Log    ${devstack_conn_id}
-    OpenStackOperations.Create Vm Instances    l2_net_1    ${NET_1_VM_INSTANCES}     sg=csit
+    OpenStackOperations.Create Vm Instances    l2_net_1    ${NET_1_VM_INSTANCES}    sg=csit
 
 Bring Up ODL1
     [Documentation]    Bring up ODL1 again
@@ -112,7 +112,7 @@ Take Down ODL2
 
 Create Vm Instances For l2_net_2
     [Documentation]    Create Vm instances using flavor and image names for a network.
-    OpenStackOperations.Create Vm Instances    l2_net_2    ${NET_2_VM_INSTANCES}      sg=csit
+    OpenStackOperations.Create Vm Instances    l2_net_2    ${NET_2_VM_INSTANCES}    sg=csit
 
 Bring Up ODL2
     [Documentation]    Bring up ODL2 again
index 88c9835d9f5d517cfd80963a32eb147712d65042..8681d6a56b01bc1d21e35a59cb92c9d3c0d39772 100644 (file)
@@ -50,7 +50,7 @@ Take Down ODL1
 
 Create Vm Instances For l3_net_1
     [Documentation]    Create Four Vm instances using flavor and image names for a network.
-    OpenStackOperations.Create Vm Instances    l3_net_1    ${NET_1_VM_INSTANCES}     sg=csit
+    OpenStackOperations.Create Vm Instances    l3_net_1    ${NET_1_VM_INSTANCES}    sg=csit
 
 Bring Up ODL1
     [Documentation]    Bring up ODL1 again
@@ -62,7 +62,7 @@ Take Down ODL2
 
 Create Vm Instances For l3_net_2
     [Documentation]    Create Four Vm instances using flavor and image names for a network.
-    OpenStackOperations.Create Vm Instances    l3_net_2    ${NET_2_VM_INSTANCES}     sg=csit
+    OpenStackOperations.Create Vm Instances    l3_net_2    ${NET_2_VM_INSTANCES}    sg=csit
 
 Bring Up ODL2
     [Documentation]    Bring up ODL2 again
@@ -75,7 +75,7 @@ Take Down ODL3
 Create Router router_2
     [Documentation]    Create Router and Add Interface to the subnets. this fails sometimes.
     OpenStackOperations.Create Router    router_2
-    [Teardown]    Report_Failure_Due_To_Bug     6117
+    [Teardown]    Report_Failure_Due_To_Bug    6117
 
 Create Router router_3
     [Documentation]    Create Router and Add Interface to the subnets.
index 4ff01b4f0f720053d1d4471604e9c1e99ca1a654..f07f257c4a8b6aed71b28f8f1464bbc8ac217a92 100644 (file)
@@ -41,17 +41,17 @@ Create Subnets For l2_network_2
 
 Add Ssh Allow Rule
     [Documentation]    Allow all TCP packets for testing
-    Create Security Group      csit    "CSIT SSH Allow"
-    Create Security Rule     ingress      tcp     1     65535     0.0.0.0/0      csit
-    Create Security Rule     egress       tcp     1     65535     0.0.0.0/0      csit
+    Create Security Group    csit    "CSIT SSH Allow"
+    Create Security Rule    ingress    tcp    1    65535    0.0.0.0/0    csit
+    Create Security Rule    egress    tcp    1    65535    0.0.0.0/0    csit
 
 Create Vm Instances For l2_network_1
     [Documentation]    Create Four Vm instances using flavor and image names for a network.
-    Create Vm Instances    l2_network_1    ${NET_1_VM_INSTANCES}     sg=csit
+    Create Vm Instances    l2_network_1    ${NET_1_VM_INSTANCES}    sg=csit
 
 Create Vm Instances For l2_network_2
     [Documentation]    Create Four Vm instances using flavor and image names for a network.
-    Create Vm Instances    l2_network_2    ${NET_2_VM_INSTANCES}     sg=csit
+    Create Vm Instances    l2_network_2    ${NET_2_VM_INSTANCES}    sg=csit
 
 Ping Vm Instance1 In l2_network_1
     [Documentation]    Check reachability of vm instances by pinging to them.
index d55c1690ce149a19f0d96dc782573ffd6deac14a..b50f5619ccb976a1fffed0c50c6eea72e44e0270 100644 (file)
@@ -75,7 +75,7 @@ Add Config To File On Existing SSH Connection
 
 Clean Up After Running Tempest
     [Documentation]    Clean up any extra leftovers that were created to allow tempest tests to run.
-    Delete SubNet     external-subnet
+    Delete SubNet    external-subnet
     Delete Network    external
     List Networks
-    Close All Connections
\ No newline at end of file
+    Close All Connections
index a0c38574ce82203f1e30879a33cf420d883e1b12..84a7ca3354328c10be222d8d4fd70f45435f3f85 100644 (file)
@@ -10,9 +10,11 @@ Variables         ../../../variables/Variables.py
 Resource          ../../../variables/sfc/Variables.robot
 Resource          ../../../libraries/Utils.robot
 Resource          ../../../libraries/TemplatedRequests.robot
+
 *** Variables ***
 ${CREATE_RSP1_INPUT}    {"input":{"parent-service-function-path":"SFP-1","name":"SFP-1-Path-1"}}
 ${CREATE_RSP_FAILURE_INPUT}    {"input":{"parent-service-function-path":"SFC1-empty","name":"SFC1-empty-Path-1"}}
+
 *** Test Cases ***
 Basic Environment Setup Tests
     [Documentation]    Prepare Basic Test Environment
@@ -44,7 +46,7 @@ Clean Datastore After Tests
 Post Elements To URI As JSON
     [Arguments]    ${uri}    ${data}
     ${resp}    RequestsLibrary.Post Request    session    ${uri}    data=${data}    headers=${headers}
-    Should Contain    ${ALLOWED_STATUS_CODES}    ${resp.status_code}    
+    Should Contain    ${ALLOWED_STATUS_CODES}    ${resp.status_code}
 
 Get JSON Elements From URI
     [Arguments]    ${uri}
@@ -77,4 +79,3 @@ Init Suite
     Set Suite Variable    ${SERVICE_FUNCTION_PATHS_FILE}    ${CURDIR}/../../../variables/sfc/${VERSION_DIR}/full-deploy/service-function-paths.json
     Set Suite Variable    ${SERVICE_RANDOM_SCHED_TYPE_FILE}    ${CURDIR}/../../../variables/sfc/${VERSION_DIR}/full-deploy/service-random-schedule-type.json
     Set Suite Variable    ${SERVICE_ROUNDROBIN_SCHED_TYPE_FILE}    ${CURDIR}/../../../variables/sfc/${VERSION_DIR}/full-deploy/service-roundrobin-schedule-type.json
-
index b3a5a6fed461311119a3f0cabd014f740e780f33..7f128bb343b4739dee25066bd5adf63f9c1ade6f 100644 (file)
@@ -49,7 +49,7 @@ Delete A Service Function Forwarder
     [Documentation]    Delete A Service Function Forwarder
     Add Elements To URI From File    ${SERVICE_FORWARDERS_URI}    ${SERVICE_FORWARDERS_FILE}
     ${resp}    RequestsLibrary.Get Request    session    ${SERVICE_FORWARDERS_URI}service-function-forwarder/SFF-bootstrap
-    Should Contain    ${ALLOWED_STATUS_CODES}    ${resp.status_code} 
+    Should Contain    ${ALLOWED_STATUS_CODES}    ${resp.status_code}
     Remove All Elements At URI    ${SERVICE_FORWARDERS_URI}service-function-forwarder/SFF-bootstrap
     ${resp}    RequestsLibrary.Get Request    session    ${SERVICE_FORWARDERS_URI}service-function-forwarder/SFF-bootstrap
     Should Be Equal As Strings    ${resp.status_code}    404
index 3f1451ebfc8d6df5a6280b98258839e8246e4813..657026b0e78e744821bc5dbf4863e98688e11ed7 100644 (file)
@@ -74,7 +74,6 @@ Put one Service Function Schedule Algorithm Type
     Check For Elements At URI    ${SERVICE_WSP_SCHED_TYPE_URI}    ${elements}
     Check For Elements At URI    ${SERVICE_SCHED_TYPES_URI}    ${elements}
 
-
 *** keywords ***
 Init Suite
     [Documentation]    Initialize session and ODL version specific variables
index a1281276a607ec7d6f36e51efb95ad57a8d9cb3c..4468576f243b6ef3d3e036025057f9c71360c4d4 100644 (file)
@@ -81,7 +81,6 @@ Put one Service Function
     Should Contain    ${ALLOWED_STATUS_CODES}    ${resp.status_code}
     Should Contain    ${resp.content}    SFC1-400
 
-
 *** keywords ***
 Init Suite
     [Documentation]    Initialize session and ODL version specific variables
index 34db8163d4e257560a7151ec98be384c1f589dd5..cf01a97f5551f6b0d481bb7189ee57477348ea06 100644 (file)
@@ -47,7 +47,7 @@ Update UNI Speed
     Should Contain    ${ALLOWED_STATUS_CODES}    ${resp.status_code}
     ${resp}    RequestsLibrary.Put Request    session    ${CONFIG_TOPO_API}/${Uni_topo_API}${Mininet2_IP}/cl-unimgr-mef:speed    data=${speedJson}
     Log    ${resp.content}
-    Should Contain    ${ALLOWED_STATUS_CODES}    ${resp.status_code}    
+    Should Contain    ${ALLOWED_STATUS_CODES}    ${resp.status_code}
     ${elements}    Create List    speed-10G
     Wait Until Keyword Succeeds    16s    2s    Check For Elements At URI    ${OPERATIONAL_TOPO_API}/topology/unimgr:uni/    ${elements}
 
index 32232a62cf46d46fe9f1480e9975d0ab2cb397d5..3f76ba47d20af39428c2cfff9709de1d3f191a6c 100644 (file)
@@ -59,8 +59,8 @@ Add a flowcondition
 
 Add a vbrif flowfilter with vlanpcp
     [Documentation]    Create a flowfilter with vlanpcp and Verify ping
-    Add a vbrif flowfilter    Tenant1    vBridge1    if1    ${flowfiltervlanpcp}
     [Tags]    exclude
+    Add a vbrif flowfilter    Tenant1    vBridge1    if1    ${flowfiltervlanpcp}
     Wait_Until_Keyword_Succeeds    20s    1s    Mininet Ping Should Succeed    h1    h5
 
 Verify vlanpcp of vbrif flowfilter
index 901b497cc514a7275cfeded7a1139a3c1b720c05..e32af73743c6c18e19775c1fe88eed0aeca3f7bd 100644 (file)
@@ -59,8 +59,8 @@ Add a flowcondition
 
 Add a vbrif flowfilter with vlanpcp
     [Documentation]    Create a flowfilter with vlanpcp and Verify ping
-    Add a vbrif flowfilter    Tenant1    vBridge1    if1    ${flowfiltervlanpcp}
     [Tags]    exclude
+    Add a vbrif flowfilter    Tenant1    vBridge1    if1    ${flowfiltervlanpcp}
     Wait_Until_Keyword_Succeeds    20s    1s    Mininet Ping Should Succeed    h1    h5
 
 Verify vlanpcp of vbrif flowfilter
index 3bcd1f732be65056e4df5cd308f8f8a18bebae21..c663a2a7fc33c69003c1940b0fd210c8656b21f5 100644 (file)
@@ -12,4 +12,3 @@ ${SERVICE_ROUNDROBIN_SCHED_TYPE_URI}    SERVICE_SCHED_TYPE_URI_BASE+'round-robin
 ${OPERATIONAL_RSPS_URI}    /restconf/operational/rendered-service-path:rendered-service-paths/
 ${OPERATIONS_CREATE_RSP_URI}    /restconf/operations/rendered-service-path:create-rendered-path/
 ${OPERATIONS_DELETE_RSP_URI}    /restconf/operations/rendered-service-path:delete-rendered-path/
-