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>

Fix Global Rpc Kill et al failures

Global Rpc (Kill|Freeze|Isolate) have been failing for quite some time
(in Neon and Fluorine) with a 500 response when executing RPC
 /restconf/operations/basic-rpc-test:basic-global. It's a no-input RPC
and the robot code passes an empty strng for the content-type and accept
parans to Post_To_Uri. In comparing with another RPC that has no input,
that code passes the ${HEADERS_XML} and ${ACCEPT_XML} constants so I did
the same in the Glocal Rpc tests. It seems something changed in Fluorine
where passng empty content-type/accept headers no longer works.

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

SXP: wait for node start/stop

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

Exclude test until OPNFLWPLUG-1052 is resolved

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

Wrap checks after deletes in a WUKS

Example here shows that a delete was done but the
uuid still showed up in the config store. In the
example below, you can see the openstack CLI to
delete the l2gw connection completed at timestamp
00:39. Looking in the 3 karaf logs (it's a cluster
job) you can see the delete made it by 00:41 or
00:42 depending on which log you check. So, polling
for up to 5 seconds should give a little time for
that to propagate.

example:
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/builder-copy-sandbox-logs/508/jamo-netvirt-csit-hwvtep-3node-0cmb-1ctl-2cmp-openstack-queens-upstream-stateful-fluorine/14/robot-plugin/log_full.html.gz#s1-s1-s1-t20-k1-k4

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

Make ${OS_NODE_CNT} a suite variable

Without making it a suite variable, it seems the changed
value is only local to that keyword, and outside of that
keyword it will remain the value as assigned in the
above *** Variables *** section, which is ${1}

example of the value getting changed to 3:
https://logs.opendaylight.org/sandbox/vex-yul-odl-jenkins-2/jamo-netvirt-csit-1node-0cmb-1ctl-2cmp-apex-queens-gate-snat-conntrack-oxygen/42/robot-plugin/log_full.html.gz#s1-k1-k1-k4-k2-k4

two keywords later in that same job, you can see
it's incorrectly back to 1:
https://logs.opendaylight.org/sandbox/vex-yul-odl-jenkins-2/jamo-netvirt-csit-1node-0cmb-1ctl-2cmp-apex-queens-gate-snat-conntrack-oxygen/42/robot-plugin/log_full.html.gz#s1-k1-k1-k4-k2-k6-k2

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

Replace BR1 and BR2 with br-int in Genius-CSIT

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

Fix flowspec empty uri

In neon, family ribs, are not initialized
until we fill them, and therefore we have
to check whether they are empty differently

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

Fix dpkg frontend locking

Getting following error while installing debian packages in CSIT
dpkg: error: dpkg status database is locked by another process
due to this locking failure dcgw script could not install all
required Quagga debian packges in tools system hence suite failed
refer following fix for the same
https://bugs.launchpad.net/ubuntu/+source/dpkg/+bug/1796081

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

Remove vtep-auto-tunnels from tdd-wip

This script is broken, and cannot even run
from genius patch test. Shankar needs
to update the test suite as per comments
on https://git.opendaylight.org/gerrit/#/c/73225/
and then add the suite to main test plan.

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

SXP: Wait for RPC registration

- SXP has RPC services registered on owner
node only, after owner change it can take
some time for them to be registered on new
owner

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

SXP: Fix WUKS KW usage

- BuiltIn.Wait Until Keyword Seceeds KW is used in wrong
way in some places

- the first argument should be number of retries
- the second argument should be retry interval
- it has no sense to retry 1x

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

SXP: Run binding origins test on Fluorine

- binding origins test can be run on Fluorine
and later streams only

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

Fix cluster suites after latest updates

- For OFP inventory returns empty when no switch is connected.
- For Netconf there is a change in the singleton entity ID.

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

Fix bgp l2vpn evpn test

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

Fix route-key argument in ibgp test

The route-key on neon is now in proper base64
enconding so we can change test to reflect this.

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

Log output from ovs commands for easier debugging

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

Remove Create Resources from first test case

[0] refactored some test code to unify some suite
setups, which moved "Create Resources" to the suite
setup. Doing it again in the first test case results
in a failure, since openstack create commands are
failing trying to create a duplicate

[0] https://git.opendaylight.org/gerrit/#/c/74698/7/csit/suites/netvirt/upgrade/upgrade.robot

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

Fix bgp_app_peer test

Refactor test

In result we also fix bgp_app_peer.py
script, which should increase health
of most of the performance tests.

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

Add BGP multipath support suite for ODL

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

Add DC gateway configplan

Change-Id: I16130e39a7ecb536cf1dc025335c413f7c49fe09
Signed-off-by: shahid_calsoft <mohammad.shahid@altencalsoftlabs.com>
Signed-off-by: mpany <madhusmita.p@altencalsoftlabs.com>

Fix bgpcep flowspec test

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

Dont check tunnels on single node

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

Fix bgp multipath test

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

Fix bgpcep lsp test

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

Fix bgpcep tcpmd5user test

Remove irrelevant tc's

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

Fix bgp route refresh test

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

SXP: Add RPC redirecting tests

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

Add False check to validation

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

SXP: Use ip instead of arp

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

Use defined tools ips

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

Fix bgp default policies test

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