Use iproute instead of nameserver to get gw

The nameserver line is only included when a vm fails
to ping the gw and you see the below messages in the
console log:

=== pinging gateway failed, debugging connection ===

The iproute line is always included:

ip-route:169.254.169.254 via 10.20.20.2 dev eth0

The old code worked because at least one of the vms
in the list would hit the gateway failed block and
get the gw from the nameserver line. The old code
also would only copy the gw address once for all
the vms because it kept a list with size 1 for
the address. The new Get VM IPs tries to get the
gw for each vm and not just one for the whole list
of vms. Timing differences might end up not getting
the nameserver line, so instead use the metadata
line which is always present.

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

Add router and sg listings to Show Debugs keyword

- also refactored List Router to List Routers because it makes
  more sense to be plural

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

Move arp learning to use Get VM IPs

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

Create and delete security groups

The connectivity test suites are creating and deleting most resources
as they require, except for allowing SSH, which gets created during
l2 tests and is reused by l3, which creates a dependency. It would be
great, if l3 test suite worked independently

Change-Id: Icdf47ceaaf3b1fb0353cca454eb039795137e7c1
Signed-off-by: Tomas Jamrisko <tjamrisk@redhat.com>

Fix SFC OVS NSH installation regression

- It was still tring to install OVS 2.5.90 from Yi Yang's
  branch, but that no longer works and OVS 2.6.1 should now
  be installed.

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

INTTEST-33: ipv6 neighboring displayed from VM

The fix is fixing the syntax of the ip command to use when being a
busybox instance.

Change-Id: I1eb3cf4e0e162437671f1f0fa7e7cdf117d9ebd2
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Move sfc to use Get VM IPs

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

Move l2gw to use Get VM IPs

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

Move l3 clustering to use Get VM IPs

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

Move l2 clustering to use Get VM IPs

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

Move bgp vpn to use Get VM IPs

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

Move basic vpn suite to use Get VM IPs

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

Change Get VM IPs [Documentation] for accuracy

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

Move elan suite to use Get VM IPs

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

Use Run Keywords for multiple keywords in teardown

this was broken recently, when wanting to split out
the amount of data models we collect in our non-sfc
suite.

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

Add parens around "${status}" - fix invalid syntax

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

Handle case when Get VM IP is status FAIL

When a keyword is supposed to return some values (a list in
this case) but hits a failure before the [Return] statement,
we actually only get a status of FAIL and some error message.
Need to handle that.

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

Remove Lithium keyword from packetcable csit

There is no more lithium to worry about, so no need to keep that
keyword

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

Rework variables to keep scope limited to keyword

- remove the need to share variables across keywords
- return console log so it can be printed just once after
  the collect ip WUKS is done polling

Change-Id: I10847f064ca38d6504e2715b51ad02792e0d3811
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Fix tox problem

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

Reduce logging in collect ips

Created a new VM Collect IP keyword that reduces the
amount of logging and works faster.

Problems in old keyword:
- logged the whole console multiple times, per vm check and per
operation
- ran Verify Active extra times
- different suites replicated using code

This patch:
- moves all the common wrapping code to a single keyword
- reduces the extra Active calls
- reduces the console to a single time per vm check
- removed console log on error since the log is always captured
during the attempt to get the addresses already

This looks to have reduced the time in half for retrieving the
addresses as well as reducing the multiple loggin of long
console logs.

Only converted the connectivity suite and left the old keyword
in place. The next patches will convert the other suites.

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

Fix for multiple keywords found in Utils.robot

By specifying OperatingSystem library for 'Get file' call.

Change-Id: I0a4e8a362033b381cf88ac303b931880f419a3de
Signed-off-by: Tomas Cechvala <tcechval@cisco.com>

Reduce the number of polling iterations in WUKS

This has the potential to reduce some logs that end up in
the output.xml. We do use --removewuks when we generate the
html reports, but that is post processing. All the iterations
are still being logged in output.xml. Hopefully this will
help reduce the final log size of output.xml

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

add cleanup for con sg suites

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

Cleanup on neutron sg suite

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

Reduce logging in vpnservice

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

Don't process the prompt line

The Write Commands Until Expected Prompt Keyword
executes a command and then reads output until it sees
the prompt. It then returns all that read output. This
output is then split into lines - which means you get
the prompt line also.

This patch instructs Split To Lines to ignore the last
line (-1).

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

Don't dump the err/warn/exception

No need to collect this information as
it is also collected by the builder at the end of the test run.

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

Reduce logging in model dumps

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

Reduce logging in SG suite

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

Reduce logging in OpenStackOperations

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

Redirect ostestr stderr output to /dev/null

comments in the code, but for completeness here is the reason:
There are tons of deprecation error messages when we use
ostestr in our CSIT environment (openstack via devstack)
The robot log files are very large and one culprit is all
these deprecation warnings. If we redirect stderr to /dev/null
we should be able to ignore them. We will miss any other
errors, however.

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

Remove extra Logging in VPNService suites

this is an ongoing effort to reduce our log sizes,
specifically the output.xml file which can grow
very large and takes a lot of resources to parse.

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

Collect debugs before tempest teardown

- the tempest.api.network bucket is still run with ostestr so it
  can take advantage of the parallel execution and will not pause
  on teardown.

- all scenario tests are broken out to be run one at a time and
  will collect debugs on teardown.

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

Remove logging of ${output} of tempest run

The robot keyword to Read Until Prompt is also giving the
full output so it's unnecessary and duplicate to also Log
it. This will help reduce our log sizes

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

Add verification for OPNFLWPLUG-930

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

Add floating_network_name to tempest.conf

in pike, the tempest tests need to know the name of the
external network to use for floating ips. This will
resolve one of the failures we have had since bringing
in the pike jobs

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

Migrate AAA CSIT to use Variables.robot from .py

Also removing unused, empty suite.

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

IPv6: get IPV6_ADDR_POOL values from Variables

Get values for NET1_IPV6_ADDR_POOL and NET2_IPV6_ADDR_POOL from
netvirt/Variables.robot, instead of declare them at the beginning.

Change-Id: Idac6f6df96e95e7193bb756c3d5f657dd8e74034
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>

CSIT: move NET1_IPV6_ADDR_POOL in Variables

Change-Id: If49d1179cdab507c351cbfab39afd4732a1ed75e
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>

Fix VTN Coordinator Oxygen Job Failure

VTN Coordinator Oxygen fails to install VTN Coordinator.
* Path added to the test file to be able to download and install
  VTN Coordinator.
* Patch also modifies the image path/name for Nitrogen/Carbon also.

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

Remove VPN service and BGP VPN service

VPN service suite is already being run in
netvirt job, so don't run it again in the
l2gw job.

A new job will be created for BGP VPN
test suite.

Change-Id: Ia84de3da41c8e9155b217aeccd032c719bc06d6c
Signed-off-by: Victor Pickard <vpickard@redhat.com>

Remove unused BFD suite

this suite is not in use. Email discussion here:
https://lists.opendaylight.org/pipermail/genius-dev/2017-October/001929.html

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

Ignore pep8 E722

Existing code has started to fail on this error and causing
verify jobs to fail that have nothing to do with said existing
code. For now, we can ignore this to let work continue in
the int/test repo.

A jira ticket will be opened for someone to fix those related
erorrs (example below) and then remove this ignore = E722
statement.

Example:
./csit/libraries/AAAJsonUtils.py:208:13: E722 do not use bare except'
            except:
            ^
Change-Id: Iee989ea6f2bcc3149988eb2e430fc7246c2d97e2
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Add a snippet about using jjb-deploy keyword

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

Remove robot tags for skipping irrelevant things

we used to skip things if the openstack was mitaka or
if odl sg mode was transparent or learn. All of those
things no longer exist in our CSIT, so no need to
have them

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

Fix script typo. Argument is not used.

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

Add AKKA debug in OF cluster test

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

common tempest in DevstackUtils

Moved the tempest-specific functions from the test.robot
file over to the othr tempest functions in DevstackUtils.
This is in prep to add a single tempest test suite later.

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

Revert "Bug 9294:"

This reverts commit 4c84f63737c9e2ff189a28d1f59b5a88b7a17059.

Change-Id: I735033c33ae8facca4c1f24969e2e2a630ab6c10
Signed-off-by: Yrineu Rodrigues <yrineu.rodrigues@serro.com>

Fix BGPCEP TCPMD5 tests

Change-Id: Id21ec4fe2a4debf9b5d1ea2111e6126388a11869
Signed-off-by: Yrineu Rodrigues <yrineu.rodrigues@serro.com>

Bug 9294:
Well known mandatory attribute missing: ORIGIN

Change-Id: I4ca1a6ade0c46dd6ccdb499173aa8300490c9888
Signed-off-by: Yrineu Rodrigues <yrineu.rodrigues@serro.com>

Remove Beryllium specific test code

specifically this commit is targeted at removing the compare
stream keyword calls that refer to Beryllium

also removed a few bits that referred to initializing variables
based on Lithium, which is also no longer relevant

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

Clean up and rename OF CSIT jobs

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

Remove all test references to boron

There are still references to boron in some comments and
docs sections, but the intent here is to remove all test
code (lots of conditional stuff like IF boron do this or
that...).

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

Add Oxygen keywords to CompareStream lib

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

Cleanup unsupported releases from CompareStream

we no longer test against anything older than Carbon
so removing everything else

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

Remove idmlite CSIT job(s)

per AAA PTL's wishes:
    https://lists.opendaylight.org/pipermail/release/2017-October/012653.html

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

Alphabetize tempest tests to match subunit

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

Add more printouts to debug cluster issues

Netstat will run for all members after member is killed,
stopped or started.

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

Remove skip_if_boron tag from ofp test cases

boron is no longer being validated in CSIT

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

Delete tesplans specific to Boron

Change-Id: Idda6bd2163a8ca65c0a93b0459a318a35987f8a6
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Change tab to spaces

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

Clean up openflow test plans

FRS jobs are removed now but there are other test plans not used
in jenkins today.

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

Add netstat check in Teardown

This KW should also execute when the Member start fails.

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

Enable printout to debug OF cluster issues

For now netstat will give some info on akka open ports
when cluster member is started.

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

lispflowmapping: remove Boron references

Change-Id: Ie1d7aa1e2fab7393aa3598afe6a498351654311b
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Revert Fix Recent Failurs in VTN Coordinator CSIT

This reverts commit f0bcc5d0d17b9f6d2335411b5ead9e1b378d6c7e.

Change-Id: I4861a1e8096fb4a3823fcc03c505564585962734
Signed-off-by: Y Ananth <ananth.y@hcl.com>

Fix the recent csit failure

There was a failure caused by the patch
(https://git.opendaylight.org/gerrit/#/c/63858/)
The min and max arguments were modified but the
instance names was left unchanged. This caused some
failures in today's run. The failure was due to a
non-exsting instance name in the list. This patch removes
the instance that is not relevent anymore.

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

Use openstack cli instead of nova console-log cli

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

Fix syntax ERROR in beir suite

when using a .robot variables file we still need to referece
it as "Resource" not "Variables". example of the ERROR it
produces:

https://logs.opendaylight.org/releng/jenkins092/bier-csit-1node-basic-all-nitrogen/156/log.html.gz

- also removed the "Suite Teardown" from __init__.robot as
it's not defined anywhere.

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

Update up tempest exclusion regexes

Remove mitaka regex as we no longer have those jobs
Add ${master_exclusion_regexes) to account for Queens

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

Use test domain for email address in CSIT

as requested from:
  https://git.opendaylight.org/gerrit/#/c/55029/5/csit/suites/aaa/idmlite/010_CRUD_Rest_CommandsTests.robot

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

Modify min and max arguments for server create

* Too many debug messages makes debugging difficult.
* This patch reduces min and max to 3, so that only 3
  instances are creaetd in parallel

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

Update yang-model-validator test suite

Update this suite to reflect the changes that have been made
in the YANG Tools artifact yang-model-validator (previously known
as yang-system-test) since the test suite was first created.

The -p option previously denoted the directories with YANG files
that should be parsed as one big collection. However, the artifact
is meant to be used as a single YANG file validator similarly to
pyang. That is why the -p option is now used to specify directories
with YANG dependencies, i.e. YANG files which might be needed to
successfully validate the single YANG file specified as argument.

Therefore, adjust the suite to iterate through the set of YANG
files one by one while using the whole set of YANG models as
dependencies within the -p option.

Change-Id: I6225242bddf5dbf612a5e73153fe41c3ffa3f990
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

Test for Bug #7596

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

Fix openstack netvirt sfc suite

typo in instance creation s/groups/group

migrate some neutron cli to the openstack cli

also, since I was here, I moved all of the cleanup steps
out of their own individual test cases and into a suite
teardown keyword. This lets robot run through every step
regardless if any previous step had failed. If left in a
test case, robot bails on the test case and wont run any
other steps in that test case if any single step fails.

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

Set DS tell-based protocol for cluster perf

As suggested this improves cluster performance in carbon.
See https://bugs.opendaylight.org/show_bug.cgi?id=6755

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

Remove SNMP CSIT

the job(s) for SNMP CSIT have never worked, and have been ignored
for several (if not all) releases. it's a waste of time and
resources to have them around.

see the status pages for Nitrogen, Carbon, Boron
and Beryllium for history:
  https://docs.google.com/spreadsheets/d/1MYyGLFWN2RzUkJl8XMzXQ-3zWuOrUCQpIS6ORbmf4_U/edit#gid=421626541
  https://docs.google.com/spreadsheets/d/1VcB12FBiFV4GAEHZSspHBNxKI_9XugJp-6Qbbw20Omk/edit#gid=1226505712
  https://docs.google.com/spreadsheets/d/1zImtd764e-hOgJAxoJKl85fxHCPu2agLfqsBtf13zQY/edit#gid=1037896712
  https://docs.google.com/spreadsheets/d/1hci5TMUPyB6PX8Al-fwfVqvs5SQVa2wZLja_7rKWN6o/edit#gid=843418255

partner JJB patch in releng/builder:
  https://git.opendaylight.org/gerrit/#/c/63427/

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

Add timeout/retry to L2GW CSIT TC05

Add timeout and retry to Verify Ovs Tunnel in test case
TC05. This was failing in 3 node tests, as it takes a
little longer for config to be applied in in cluster.

Change-Id: If9c2aa798c5b879f426837b05066e21d4e2bd85d
Signed-off-by: Victor Pickard <vpickard@redhat.com>

Fix for Genius CSIT failure on Oxygen

Description : This patch has keyword Get Tunnel modifed with
code to get tunnel interface name between dpns

Change-Id: I6bac03d6d3c244f4ef40b2d5ad1e8f0f046eeeaf
Signed-off-by: vidya sathyanarayanachar <s.vidya990@gmail.com>

Bug 8999: Add entity isolation netconf suite

Added a suite based on entity.robot which
will verify that a 3 node netconf cluster recovers after
isolation and rejoin of the entity owner.

Change-Id: I2a5510338dfa323c71d3ce4820036b4bfe0342a2
Signed-off-by: Richard Hill <rhill@frinx.io>
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Add a WUKS to netconf connector check

The check for netconf connector is not able to finish in time,
though the subsequent test passed. Adding the check.

Change-Id: Ib6f0ece50643ebab7e8e32dc6023941ce6d2c4e2
Signed-off-by: Atul Gosain <atul.gosain@gmail.com>

Get compute node hostname from hypervisor list

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

Default BUNDLEFOLDER var to /opt/opendaylight

this currently will have zero effect on upstream ODL CSIT,
because we always pass this variable on the pybot command
line.

However, some other CSIT consumers (e.g. OPNFV) are seeing
lots of warnings [0] because they are not *always* passing
BUNDLEFOLDER in their robot argument list. This will remove
that error for them, while giving us a decent default of
/opt/opendaylight which is where the rpm package installs
to.

[0] https://git.opendaylight.org/gerrit/63293

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

Convert Sleep to WUKS in lispflowmapping CSIT

Part 2, 020_RPC_CRUD.robot

Change-Id: Ibdce362fbb6e87e0221878832be57474717e59e9
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Support https endpoints (Keystone and Neutron)

It simply replaces http://${NEUTRON}:9696 by ${NEUTRONURL} and
http://${KEYSTONE}:5000 by ${KEYSTONEURL}.

It also switches to Variables.robot and defines
${NEUTRON_SUBNETS_API} in it.

Change-Id: I76702facfbfb068160933fbba4dbb5f2cbc41d15
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Stabilize OF split connection test

It is possible that during unstable scenario of 2 connection
fighting each other there is no flow being pushed.

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

Rename yang-system-test module

 * rename yang-system-test module in Yangtools
   to odl-yang-validator

Change-Id: I67909babb2c61446327d7d7b1bae33f9029b4777
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Convert Sleep to WUKS in lispflowmapping CSIT

Part 1, 030_RPC_Datatypes.robot

Change-Id: I6885a831f40ee2b1ff555de0f67b456ca6e9650e
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Fix typo in OF test plan

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

Add Carbon SR2 cluster test document.

Include reports on previous releases as drafts.

Move ReST coala check to come before Python,
because the Python check takes longer to finish.

Change-Id: I84c34570c329f8793b76d07503db800999549da8
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Repair OF test

- Specify bugs to be tested in OF suite.
- Change compare stream to At Most (more safe).
- Add oxygen stream in CompareStream library.

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

Remove NIC-console tests from CSIT

That feature was removed from NIC

Change-Id: Icfe5551302423748d5a93cdd34bb809bcdc86b8c
Signed-off-by: Yrineu Rodrigues <yrineu.rodrigues@serro.com>

Bug 7855: Update singleton rates

As Bug 7855 is fixed, there is no reason to tolerate
rate of 5 on Carbon. But tests on -all- job
just after cluster reset are slower than
thecorresponding longevity job, so tolerate rate of 50 for now.

TODO added to set back to 100
when better ODL readiness detector is available.

Change-Id: I92561f617afd3bac037be236c9bad7dd90e48216
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Fix lispflowmapping 3-node CSITs

Due to propagation delay in the cluster, datastore writes are not
immediately avalable and test fail because of that. Looking at recent
test failure logs delays as long as 300+ ms can be found. This patch
adds a 500 ms pause after each datastore write, to account for cluster
propagation delay.

Change-Id: I97389dfdd1bd61ce53e8bb53be5a380aacb04c07
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Update Bier CSIT with template

Change-Id: I4e6914216482100a875b5676ef433d0c49774eb0
Signed-off-by: Quan Xiong <xiong.quan@zte.com.cn>

Create nano flavor for l2gw csit

Devstack fails with tempest and l2gw-plugin on pike.
For now, just go ahead and create the nano flavor,
and do not enable the tempest service in the
l2gw csit test suite.

Change-Id: Ia81ab327ad79ac1e429655ac64723963de7f8f66
Signed-off-by: Victor Pickard <vpickard@redhat.com>

Isolate OF bug 9145

- This new bug is impacting reconciliation suite so this change
will isolate the bug in its own test.

- Also remove unnecessary check in member isolation test.

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

Bug 9044 workaround: Delete etc/host.key

When testing locally with bin/client,
deleting etc/host.key file before starting ODL again
is enough to avoid Bug 9044 symptom.

Change-Id: I6993f543a710a88bbfdbb2a2c44fa97833229b8e
Signed-off-by: Vratko Polak <vrpolak@cisco.com>