Fix Robot evaluate

It seems like latest robot version changes the way python dictionary
is evaluated.

Change-Id: I34b72190bb4f7d09aedf1cb1d46b4ec65b247164
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Fix error in CSIT due to not empty stderr

pip command shows stderr about python 2.7 deprecation

Change-Id: I79c399abb203cb4d7bcdafd208ed241829822ba7
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Qualify Keywords in Genius Suite

Change-Id: Id2c77861462d29fc5e2ab944ca0b8d8823aad2e0
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

Remoce  wrong keyword for node data

Upgrade CSIT has been failing with the below error:

No keyword with name 'DevstackUtils.Set Node Data For Control Only Node
Setup' found.

The keyword was renamed, but the keyword call can simply be removed
because the proper keyword is called during Suite Setup->
OpenstackOperations.OpenStack Suite Setup->Devstack Suite Setup->
Get DevStack nodes Data

Change-Id: I05d7470733669e5b74d57237f1c013cdf345c219
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Migrate to serviceutils/upgrade flag

JIRA: GENIUS-190

Change-Id: Iafdea3a3e74b4e1c8d2f6161ec05cca91f51d8bc
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Fix URI bad format error

In jenkins model dump for rendered-service-path
gets error response.

Change-Id: Iebb1c89c90c335fc5b2b924e250b7d876386dd2f
Signed-off-by: ananth.y <ananth.y@hcl.com>

Migrate from godep to go modules

COE CSIT has migrated to latest golang,
so we can start using go modules for
building watcher and cni.

Change-Id: Ie9c54e81a7b9917c818d234df8c6e53a83bc6349
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Enable mdsal store for testool

Change-Id: I8ee0e343f980b883d04483f1569674ddeea69877
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

SFC New testcase addition.

- SFC Flow Classifier creation with destination port range.
- SFC Flow Classifier creation with source port range.
- During Start Applications on VM Instances part, executing call
  commands inside single ssh connection.

Change-Id: I4f3c226a6ced0fb699754e57b8d9440da0d09849
Signed-off-by: pabitra_b <pabitra_b@hcl.com>

Revert "Fix OpenStack CLI Get List error"

All jobs are reported as failure, but tests are working. This patch
causes failure in Suite Teardown Causing Failure when the json outcome
has a value like False which is not a string
link:https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/
netvirt-csit-1node-0cmb-1ctl-2cmp-openstack-queens-upstream-stateful-
fluorine/1169/robot-plugin/log_02_l3.html.gz

This reverts commit e6e32a086559fbe0a59558175b6f6b3b0c99f34f.

Change-Id: I4653effc113a08b45f94d3a265dd2dd67c6c9318
Signed-off-by: gvrangan <vgovindarajan@luminanetworks.com>

Fix OpenStack CLI Get List error

In OpenStack CLI Get List error, RequestsLibrary.To Json
    ${json} will result in exception if return value of
OpenStack CLI is empty, i.e. ${json} is empty, this will
terminate current keyword/test case, it isn't what we expect.

This commit fixed this error and make sure current keyword/test
case can be run normally.

Change-Id: If36e27f092e66920004cfc09e9a6d6ec88bf5b51
JIRA: INTTEST-56
Signed-off-by: Yi Yang <yangyi01@inspur.com>

Revert "Revert Add Acl enhancement"

This reverts commit I7bf12b07e24f73d29bc90f1cff0bd90253e17915

Change id 73638 went in but  broke netvirt csit so was reverted
and this is an attempt to get that original patch in without
breaking netvirt csit.

Change-Id: I21e04f6c1492dc828fa04993c2554ec10ea2d0a5
Signed-off-by: TabassumSharieff <shariefftabassum14@gmail.com>

Include sodium in the list of branches

Change-Id: Ic497401eabbe5746a40004fdcda461b36e616f98
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

Change ping check for COE

The output of ping between pods
is checked over 0% packet loss.
This is changed to 64 bytes to
ensure proper check is being done.

Change-Id: I45923d3b2884b75cdd16baa5204427ed46ca9d40
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

GENIUS-136 : Enhance Genius-CSIT to 3 Switches

Change-Id: I39ae335caec7867e05942efa63a85d043bcb4a57
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

Modification on SFC test Cases

- Replaced Ubuntu image with Alpine Image
- Created new variables ETH_IN, ETH_OUT, CLOUD_IMAGE_USER,
  CLOUD_IMAGE_PASS, CLOULD_IMAGE_CONSOLE specific to Cloud Image.
- Globally set suit variable for OS Prompt, user, password and console.
- During Creation of VM instance Key is ommitted as it is not required.
- Downloading VXLAN Tool is not required in this case.
- Used 512 MB and 1 CPU for Alpine Image.
- Replace curl comand with nc command.
- Replace cirros instances and create 3 instances using the opnfv image.
- Leverage SimpleHTTPServer from python to check the connection.

Change-Id: I0309c2a735664f937b41f5954ea8e6d4b44f5a71
Signed-off-by: pabitra_b <pabitra_b@hcl.com>

Append ARP use cases TCs in puntpath suite

Change-Id: I121f33cf3ba85c9a9b68c80b74d5c00614b88598
Signed-off-by: shahid_calsoft <mohammad.shahid@altencalsoftlabs.com>

Do not check status code for empty inventory

Since Neon it returns 404.

Change-Id: I93df691d925fb1891c3a63cf2388c633001f9299
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Change ODL port from 6633 to 6653

Change-Id: I83a737c05c4bccd5c0a7362dca1cd077141df94d
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

SXP: Adapt/increase retry timers

- adapt/increase retry timers for cluster
routing tests

Change-Id: I2003fe719be257b4ed0c25fc26a05599bf2cea11
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

GENIUS-268: Disable BFD Greater than Neon

Changes in test cases are done because BFD
will be disabled by default for branches
greater than neon while for other branches
it will remain enabled by default.

Change-Id: I3d2819d3a198f534afce94314c5756e4cd62408c
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

Adjust XML output for empty device configuration

This change is required after this merge in master:
https://git.opendaylight.org/gerrit/#/c/79524/

Note this behavior change shows when devices
are added using RPC (vs added to datastore).

Change-Id: Iffa85482bf107b27381eabba2fa44a22567f49a1
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

SXP: Add script plan

- add script plan for cluster-binding
suite to install SXP device

JIRA: SXP-150
Change-Id: Id705526a2d95961ea494c0347d88899218b09404
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Fix node stop status checking

- use ODL system to check that node
has stopped

JIRA: SXP-150
Change-Id: I274fb83ae225b87443ee91845950018b04e0a2dc
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

COE-43: Check for stale veth ports

To check if stale veth ports are left on
the switches after pod deletion

Change-Id: I55261ec15842d7c641b76dd0ed53fe4c543a194b
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

SXP: Fix node stop status checking

- use ODL system to check that node
has stopped

JIRA: SXP-150
Change-Id: I4eab98143c20ae41a961f2654a86f5aa6a046850
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Append 3 more punt path protection TCs

Change-Id: I3d9d0c201ad20f7f79e261bcebe47356dc12eede
Signed-off-by: shahid_calsoft <mohammad.shahid@altencalsoftlabs.com>

Update list of projects

Change-Id: I86c89149872ea52dfabd7418494ca357cc94e9a1
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Additional URLs in Coe TearDown

Change-Id: I82a1a066e81ccd6df5c53134bd2ac5e5f4d28188
Signed-off-by: Faseela K <faseela.k@ericsson.com>

SXP: Check shard status

JIRA: SXP-150
Change-Id: I244d2a91027f800ae1ca595f0ebbc2521939ffc7
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Wait until cluster is ready

JIRA: SXP-150
Change-Id: I8308bb3e40efa0d90224f56f65b5ba22dfbd4745
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Introduce cluster-binding job

- move the last test file of clustering suite
to separate suite increases its possibility to pass

JIRA: SXP-150
Change-Id: I4003b919b837fbeb6b5fd12e9bc93b013d67e883
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Improve reliability of clustering suite

- increase retry timeouts

JIRA: SXP-150
Change-Id: Ib143df18b0e9badc587157e51c5618b0b8a95610
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Improve reliability of basic suite

- check that node is started after add
- check that node is stopped after delete

- increase timeouts in all connection
related tests

JIRA: SXP-150
Change-Id: I440a90c7b471594642d843ed87a9b638743a6485
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Enable DEBUG log level

- enable debug log level on cluster tests start
and revert it back to info on teardown

Change-Id: I038ece42d86432c2b6a65588fb5277ed782dbf08
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Pass default values to service_list arg

fixes bug that was introduced by:
https://git.opendaylight.org/gerrit/#/c/79385/

Change-Id: Ia8d19251d93e0cd4c3c05dfb4f315b2662599e27
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

SXP: Increase number of retries

- increase number of retries to check bindings

JIRA: SXP-150
Change-Id: I6abd25c2ec5c3d54f3036a464851a18636817f0f
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Introduce @{EMPTY_LIST} variable

- introduce @{EMPTY_LIST} variable to use
in KWs with list parameters

JIRA: SXP-150
Change-Id: I07a3b3c171370cea13d68fecf409e924a6826f1c
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Check cluster is ready after (un-)isolation

JIRA: SXP-150
Change-Id: I5e587775054a6da4ee392439727abdf0ab64d3e6
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Rename bfd suite teardown

Change-Id: I2c55a95c85a2fef04718013118e51498104101b6
Signed-off-by: Faseela K <faseela.k@ericsson.com>

SXP: Verify that all bindings are present

Change-Id: I152b6315302f8ccb975855645bbf93acb90c6133
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Increase WUKS retry time

Change-Id: I2d91227ed1c495d0c3423a84dbb5cb377538cf6a
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

GENIUS - Fix diagstatus check

Change-Id: Iaaf3308cef501e492efa3294e6a638c2d98597f6
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Fix handling of list argument

I've learned that you cannot assign default values to a list
argument (something to do with mutability), so removing
the default value in the argument line and handling the case
where the keyword is called without passing in a list for
that argument.

Change-Id: I2b37e0c7a9a7708a234562541439de8db37eda84
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Pass service_list as actual list

this logic was broken by c/79243

Change-Id: I1ee01bdabd09759fd6ca101a8f8b4689e529cbd6
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

SXP: Use not equal retry timer value

- not equal retry timer value allows
to deal with VSS switchover

JIRA: SXP-147
Change-Id: I998f1f52f8dcc543b4e2e402c0cd3f2b58dbff59
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Verify restconf is available on member startup

CSIT run
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/controller-csit-3node-clustering-tell-all-neon/106/robot-plugin/log.html.gz
failed due to the restconf availability check returning 404.
This occurred at 17:16:24 however the karaf log shows that
the restconf bundle was still in the process of starting up
at that time (it was waiting for ia AAA dependency). The
bundle eventually completed startup at 17:17:19.

The restconf check is part of the
Get_Raft_Property_From_Shard_Member keyword. However it's
wrapped in the higher-level
Verify_Leader_Exists_For_Each_Shard keyword with a deadline
of 30 sec which is enough most of the time but needs a bigger
cushion. Unfortunately, since the restconf check is deeply
nested within a Wait Until Keyword Succeeds, we can't just
simply wrap it with it's own deadline as that may be longer
than the enclosing deadline. Instead, I added the restconf
check to the Start_Members_From_List_Or_All keyword which
already has a 5 min deadline. I left
Get_Raft_Property_From_Shard_Member as is lthough I really
don't think the restconf check belongs there.

Change-Id: I687a10cbc92d8190a2bcb9a5f4ac1837ba0459a8
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

complete project telemetry system test job

Change-Id: I1ce405ff31dee6a001dc4a83f5345d3df0473b7e
Signed-off-by: Wsx25289 <10200860@zte.com.cn>

Add Floating Ip testcases to SNAT suite.

Change-Id: I9483d953369951a1524e9bb5868c64fbcea699c9
Signed-off-by: mpany <madhusmita.p@altencalsoftlabs.com>

Add two new keywords to BGP library.

Change-Id: I989f4b735e01d9bef0e68f9016eb6082fb010bb9
Signed-off-by: mpany <madhusmita.p@altencalsoftlabs.com>

SXP: Format connectivity tests

Change-Id: I883cfeeaac3d597c14b531cd8b87faabfe5839d7
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Use not random retry time

Change-Id: I07f7f866af4b0d374e74366e4ba7ce60cd70fd48
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Add openflow punt protection suite

Change-Id: I226552cbe569032881d9a16dbd1b2d0234bbf370
Signed-off-by: shahid_calsoft <mohammad.shahid@altencalsoftlabs.com>

Increase timeout for member stop

We've been seeing intermittent failures when
stopping a member where it times out waiting
for the karaf process to disappear. Eg
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/controller-csit-3node-clustering-tell-all-neon/107/robot-plugin/log.html.gz#s1-s17
In that case, it timed out at 08:45:10 - the
karaf log indicates it stopped somehwere between
08:45:11 and 8:45:32 when it was restarted. So
the 2 min deadline wasn't quite enough in this
case.

I think at least part of the problem is that
CSIT installs the "world" or at least all the
managed projects which adds to the startup and
shutdown time - coupled with random delays due to
workload in the CSIT env. In lieu of trimming down
the installed features, let's double the timeout
to 4 min to (hopefully) give plenty of cushion.

Change-Id: I1d3b85ec1600c26c6ee187ffc5036a513151f2c6
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Use proper syntax for list passing to *contain* kw

more details here:
https://git.opendaylight.org/gerrit/#/c/79287/1//COMMIT_MSG

Change-Id: I65f688ac32a568f7bfa9f318eb270fc6135911d5
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Use $ for list arg passed in to builtIn kw

If a list is passed in with the @ syntax, it's expanded first
the first two elements are used for the BuiltIn keyword. So
the result is not what we expect. I noticed it in this false
failure:

https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-1cmb-0ctl-0cmp-openstack-queens-upstream-stateful-snat-conntrack-fluorine/154/robot-plugin/log_full.html.gz#s1-s11-k1-k2-k6-k5

it needs to come in with the $ syntax so the keyword can
use it internally to loop over, and compare to the 2nd
argument.

Change-Id: Iea8cda9f8f7357cac3399cf7756822cc743cc6cd
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

SXP: Improve connectivity tests

- Add logging for tests with passwords
- Nodes are already cleaned in test:
remove unnecessary cleanings

Change-Id: Ia6c86eb7083101a59f6d9c742a9f629b93296054
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Remove check for ownership change

The "Partition And Heal" and "Global Rpc Freeze" expect the
owner not to change after rejoin but akka may report a spurious
unreachable member event that can cause an owner change after rejoin.

Change-Id: I586f0da98743bd02a5a02038a4b894b2e8929218
JIRA: CONTROLLER-1865
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Add Subnet And Multicast Suite.

Change-Id: I20f0a24c16252e6f078a98c0c96eb1382faeaabf
Signed-off-by: anuj.l <anuj.l@altencalsoftlabs.com>
Signed-off-by: mpany <madhusmita.p@altencalsoftlabs.com>

Revert "Fix wrong GERRIT_PROJECT"

This reverts commit 5856b0886f8024dd1b6cabdddfd1a68d73a42bd7.

Change-Id: Ia949c795feb0f6f9ee9b167c53d2091f200a5b0f
Signed-off-by: Sam Hague <shague@redhat.com>

Fix wrong GERRIT_PROJECT

GERRIT_PROJECT is used in the csit job to clone
the integration/test project so it can not be·
overwritten with the root tst-patch job.

Depends-on: I426b516d2b81f1f2a758552f3e97c16f08e46127
Change-Id: I7d6d5e3d3ffe76807692a7b9bdd52fbca8377fe5
Signed-off-by: Sam Hague <shague@redhat.com>

Add multiple bgp vpn suite.

Change-Id: I3ce4c0dbe6eba7899c3cd90debec9b84d5178a85
Signed-off-by: mpany <madhusmita.p@altencalsoftlabs.com>

Fix refspec for coe test patch

Depends-on: I5c67e9cf8db8b3e0ffc7ef857bae946cd3d7f489
Change-Id: I587245e1b61a0fa800d79cb920cab35667b678c5
Signed-off-by: Sam Hague <shague@redhat.com>

pretty print git log output

Change-Id: I01fd907056bea4a57211f94d6e808f115fde361f
Signed-off-by: Sam Hague <shague@redhat.com>

Remove check for leadership change

see [0] for more details, but the idea is that we
have some sporadic cases where the isolated node
will report a node unreachable while it's being
rejoined to the cluster which triggers a new leader
election. This unreachable node messages can be
expected if the timing of things is a certain way
so the new leader election is also expected. In
the spirit of creating stable CSIT results without
sporadic failures, we will remove this test case.

[0] https://jira.opendaylight.org/browse/CONTROLLER-1865

JIRA: CONTROLLER-1865
Change-Id: Ic5da2d0748915a36e11eaa841f4b2772ecf7540f
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Genius CSIT - Add AutoTunnels Suite

This change is to remove CLI based deletion of vteps and
add REST calls for all recent changes of auto tunnel.
Removed All openstack related commands and
added CLI based verification of tunnel state in ITM_Vtep_Auto_Tunnel

Test spec link for auto-tunnel:
https://docs.opendaylight.org/projects/genius/en/latest/specs/itm-tunnel-auto-config.html

ITM_Vtep_Auto_Tunnel suit contains below test scenario
verify default transport zone after enabling local ip in compute
verify tunnel status after renaming external id to TZA in compute
verify tunnel status under default transport zone after removing TZA

Change-Id: Idadf7038d6b412b358e822b57590b8319ecfbe56
Signed-off-by: srajappa <srajappa@outlook.com>

Add graceful-restart test suite

Change-Id: I370968d827149a59d0908ff0f184450c70426a9e
Signed-off-by: tomas.markovic <tomas.markovic@pantheon.tech>

COE : Pass proper GERRIT_REFSPEC

Change-Id: Idf3c9ea9305e9abae99881c274a621ab3af51055
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

Precise log statements for COE CSIT

Change-Id: Ia579b0df72d05cc087e8398d4ab498d942466ebd
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

Fix COE Model Dumps

coe_data_models was partial for the
second suite, as model derivation is done
in start suite, and that was run only for
the first suite.

Change-Id: I73c3143ded18f44bf6a10b69bc3b581410671e93
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Broken COE patch test

If we run COE patch test from netvirt,
it is currently not working, as the refspec
is getting used for building golang binaries
from COE project.

Change-Id: If90519fb035d01b4c18d3a1abcc5f06e92e8074a
Signed-off-by: Faseela K <faseela.k@ericsson.com>

s/Testcases/Test Cases/

With the newer version of robotframework the use of a single
work Testcases produces this warning in the robot .html log

Error in file '<filename>: Section name 'Testcases' is deprecated.
Use 'Test Cases' instead.

This will clean that up

Change-Id: Ibd1542b31fe2573cd20931129f29f5c4b5cf0060
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Fix change_counter

Change-Id: I793d57c50e0742eba80dc1107f158458a19f8663
Signed-off-by: Tomas Markovic <tomas.markovic@pantheon.tech>

SXP: Wait for bindings to be send/received

Change-Id: Ide429e7a77fe89840a8b740b4640d61e428e58c0
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Fix "Check Connections" KW usage

Change-Id: Ib52a732dafbbdb66d6cf97b7be448454fc490728
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Fix teardown in 040_Custom_Passwords

- in setup only 3 nodes are created
- in teardown delete only 3 nodes (not 4)

Change-Id: I7a878dd1610c0470591e2ee8adbca8d174df6b9f
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Fix tidy for Coe

Change-Id: I7a47e2e18e29f20906d3424a9c4a1b48d1b112c6
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Enhance HA suite teardown

Change-Id: Icfb88c8b26983483dcaf4fa27b30faa58f56e4c7
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Add ping retries and break on success

Change the ping command to take a retries value. This
allows the ping to be more independent of timing issues
with flow additions.

The cirros images and the SSHLibary have some weird quirks
that the patch had to workaround:

- The cirros images use a dash like shell and not bash. This
required catering the shell command to be posix compliant.
- The typical prompt of $ is in the shell command so it can't
be used to read until. This was changed to read until regexp
to look for the final rc=n from the command.
- The SSHLibrary Write keyword actually consumes and returns
the command itself, which was causing an early match. The regex
helped here also since the regex would only match on the rc=n -
so a number had to be in the output for it to match.

JIRA: NETVIRT-1399
Change-Id: I06d43fbd2ce7a0f32ccc60cdfd2783abaadd260b
Signed-off-by: Sam Hague <shague@redhat.com>

SXP: Reorder cluster tests

- reorder cluster tests to minimize
impacts of frequent cluster owner isolation

JIRA: SXP-154
Change-Id: I3636167c55e13a20427ed0432dc667339af59d94
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Disable prefix based sharding cluster suites

The below comment block is also in the testplan file:

/ disabling all "prefix based" suites. This functionality was newly
/ introduced with these tests in 2017. The assumption is that
/ development stopped in that area, but may continue at a later
/ time. Still, the tests are failing and no resources to solve the
/ failures at this time. If/When this changes, we can uncomment
/ those suites. Note that every suite is commented out with it's
/ accompanying "restart_with_tell_based_true" setup suite.

Change-Id: If624bc303d1d12625c0f7f79f9590bc29caccaf2
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Use OS_CMP format for compute ips

The devstackutils library now will generate proper ips
for our OS_CMP1_IP and OS_CMP2_IP variables depending
on the deployment. In the case of an all-in-one deployment
the CMP1 and CMP2 ips are going to be the same. That
was done for all other suites back when we moved to this
method, but I think elan was missed somehow.

example of how it fails:
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-1cmb-0ctl-0cmp-openstack-queens-upstream-stateful-fluorine/116/robot-plugin/log_full.html.gz#s1-s11-k1-k2-k7-k1-k5-k3-k1

you can see that the ssh session had no destination ip
listed, because that variable used was not available

Change-Id: I3cb2f63828bad2ee1fd4f3a03ce08eab30159295
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Use openstack cli instead of neutron

neutron cli is deprecated and we don't install that client
in our apex job (yet) and would prefer not to keep maintaining
the deprecated cli.

Change-Id: I119dd160b04fcd1570d48c5849a999b77bbe6d92
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Add 10s polling to allow transactions to complete

In some cases the check to ensure that all cars/people
were populated in the config store happened just before
the last few were there. This could be because of
any number of reasons (slow external environment,
testing in the middle of a GC operation maybe, etc)
Adding a polling period will account for this and
not give a false failure.

Change-Id: Ic9c7456da37b5c2cca694b2affc77e048a2f7b31
JIRA:INTTEST-54
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Strict check on kubectl get nodes

1.Replacing the existing check where
status 'Ready' is checked for all
lines to checking status for number
of nodes.
2.Removal of unused hard coded variables

Change-Id: If0b086a549932188bf2ad95cdf6a654087793375
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

Use openstack CLI for router delete

the neutron cli is deprecated and would live to stop
maintaining it. Also, the apex jobs that are coming
online don't have the neutron cli installed by default.

Change-Id: Ib3daf3ef79e6609cac30b4c54d7a93a9fc18a2c8
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Check for 404 instead of empty list

As part of Neon platform upgrade, read on an empty
list now returns a null [404] instead of 200 with
emtpy list.

Refer: https://wiki.opendaylight.org/view/Neon_platform_upgrade#Datastore_lifecycle

Change-Id: I4762749ed8f54eb5a8fb78bbc11bba3ba4a3524e
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Use gerrit patch for coe csit

Change-Id: I93ee4bce847a86adaa89409b12dd8a3ae760dfc1
Signed-off-by: Sam Hague <shague@redhat.com>

Allow coe to use single suite

Change-Id: I0dc13d02c3adc76aa7f4ab0e92cf9a4039274376
Signed-off-by: Sam Hague <shague@redhat.com>

Fix outputs in bgp-ha

Change-Id: Iec24e3a285cf7af6008e7d2bb8949f16527a24ed
Signed-off-by: Tomas Markovic <tomas.markovic@pantheon.tech>

Disable Tunnel Validation for SFC Jobs

- SFC uses the flow based tunnels
- odltools needs to be enhanced for
  validating of tunnels
- Until the enhancement is complete, this
  patch blocks the tunnel validation for
  SFC jobs.

Change-Id: Ie9542d6ea878d6ea5e4d5f9a8f120f9ab1d01acf
Signed-off-by: gvrangan <venkatrangang@hcl.com>

Fix ibgp peer lsp base64 encoding

Change-Id: I702c4a377c05103272b28108e7866463b360c21e
Signed-off-by: Tomas Markovic <tomas.markovic@pantheon.tech>

Fix empty uri for functional tests

- fix as_path across functional tests

Change-Id: Ifb00546e0cd277ea18417f2e819778b294b6010c
Signed-off-by: tomas.markovic <tomas.markovic@pantheon.tech>

Append more BGP multipath testcases for ODL

Change-Id: Iff2bff7ae6a058f5a51de20350d1ae37cb8e950e
Signed-off-by: shahid_calsoft <mohammad.shahid@altencalsoftlabs.com>

Use rate of 5 to avoid false failures

See the comment in the file for more details

Change-Id: I4a54fbc693747aa0a6d49e56700a9e61b25fe99d
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Fix bundle sync test

Just need some time to install group before flow.

Change-Id: If6fd9f6f1f9c34240974f7265bd0996d181bc7a0
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

SXP: Increase node start timeout

JIRA: SXP-154
Change-Id: I1863c887cb5955511efcd91fc85455351d64b1a6
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

SXP: Check cluster is in sync

- check that all cluster nodes are
in sync after leader isolation is
reverted

JIRA: SXP-154
Change-Id: I770c148d55c7b046eb72f85a13f8d4902854f6db
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Fix bgp_route_refresh line ending

Karaf cli line ending is now updated even on oxygen.
Change test to reflect these changes.

Change-Id: Id80209750572e1f49649b53bd8b0b77d3b6ffee1
Signed-off-by: Tomas Markovic <tomas.markovic@pantheon.tech>

Lower Chasing_The_Leader rate threshold

With the rate threshold set at 50, this
test fails frequently. Not sure why that
threshold was picked but lowered it to 10.
Looking at past runs, the actual rate
fluctuates anywhere between 20-80, which is
indicative of the unpredictablity of load in
the CS environment as the related
production code is mostly network I/O bound.

Change-Id: Id579c9553e0e48c85c039de5fddb0d1e56a5f2e6
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>