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>

Fix bmp_basic test on neon

On neon we no longer require empty list
in the output, and this line can be completely
removed.

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

Scalability of VMs in COE

To increase the number of nodes and check
how many pods can be brought up on
the nodes.

Change-Id: I63a067c258b32e39b3a1086b4c0c2ae5a3716da0
Signed-off-by: Karthika Panneer <karthikapaneer97@gmail.com>

Scalability Of Pods In COE

Change-Id: I9047165c62bca08f9d7c6b586ffec35ce3316bb8
Signed-off-by: Karthika Panneer <karthikapaneer97@gmail.com>

Ignore failure on trace:transactions w/ karaf cli

this is part of a common test teardown keyword which is
blindly run on all ODL nodes. However, sometimes those
nodes are actually down in our 3node jobs. We don't
want to fail the test case in that instance.

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

Add comments to ARP Learning suite

Change-Id: I3f0528c859507d62d7211a922584c5c7a8bd5749
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Fix failing sfc test case

[0] broke a test case. The syntax was wrong and not using
enough arguments. I only fixed the syntax and arguments.
I have no idea if the functionality intended will work
or not.

[0] https://git.opendaylight.org/gerrit/#/c/77258

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

Do not close connections on tcpdump nodes

there is some hard to nail down bug in the test code
such that closing a single tcpdump connection is
actually closing a different one and when that different
one is attempted to be used, we fail on connection
not found. Until that is figured out, this should
work around that. Every suite that uses this tcpdump
functionality will have a suite teardown that uses
close all connections which should eventually close
these

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

Log validations to file

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

Add tunnel validation

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

Change kill -9 to graceful stop command

Since it's accepted that 'kill -9' is the extreme and
uncommon way to stop ODL karaf, and we have nobody
that can currently try to figure out and fix the problems
we run in to when restarting that way, we'll move
to using the graceful (bin/stop) method of stopping
karaf.

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

List open transactions in test case teardown

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

Add Elan service and Interface recovery suite

Change-Id: I550f246a34204f861746dcbe7efcd36b92d23d15
Signed-off-by: sanjothapatil <sanjotha.it@gmail.com>

Fixes to SFC CSIT for neon-mri changes

Change-Id: Id8790628c237ccf542978a3995a40d4d9fe0fc92
Signed-off-by: Brady Johnson <bjohnson@inocybe.com>

Use same tests before and after downing two nodes

Sometimes we notice a failure in connectivity after the step
to take down two of three nodes, and the node that is left
is in a bad state (maybe because it's the only node up and
cannot sync with anyone). If there is a real connectivity
issue we want to see it while a healthy cluster is running

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

Subnet Routing-Remaining Testcases

Change-Id: I93fa7ea2f3889642c38700fcfa20921bc9b06835
Signed-off-by: rssaravana <saravanakumar.somaskandan@gmail.com>

Use 3.5.0 version of flake8

Version 3.6.0 was released today and it fails many
existing files.

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

Mark CONTROLLER-1849 bug in cluster suite

Also add akka "light" debugging.

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

SXP: Use correct virtual interface name

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

SXP: Update ping parameters

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

SXP: Log routing

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

Fix netconf cluter test

- Update model name.
- Adjust perf test duration and tool timeout.
- Remove netconf ready suite.
- Comment failing suite (adding jira ticket)
- Also fix tox issue in docs/conf.py

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

update odltools cli to use netvirt subcommand

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

Fix env removal

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

Change gate skipping logic to use tags

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

Add bgp funct rt constrain validation test suite

This suite tests advertising rt-constrain routes to odl.
It runs with 3 bgp peers configured. It tests whether app peer
advertizes l3vpn route to bgp peers based on whether app peer
received route containing route-target argument from given bgp peer.

Change-Id: I2e56acb0a4982de182669d3a6750a79bb63ef92e
Signed-off-by: Dasa Simkova <dasa.simkova@pantheon.tech>

Use tags to skip tests in gate job

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

Skip some tests to help speed up gate job

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

Add basic odltools csit

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