Harden check for running ovs in sfc full deploy

Change-Id: I195d527ff6f101208dbf21b2a3b089ab56194abb
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Dump module data at the end of full deploy suite

Change-Id: I40e2cbc5c385347b2cfacbf21b2142525492b0c2
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Fix for Get Router ID

Instead of listing the routers which can be a problem when
routers have similar names - using openstack router show instead.

Change-Id: I759da4efa304820e26ace87bcde7be12da245780
Signed-off-by: Itzik Brown <itzikb@redhat.com>

Fix VTN Coordiantor Test Failures

- There are only 2 active binaries in Nexus
- Fix line parsing Nitrogen Version
- Make Oxygen test using Nitrogen binary

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

enable tcpdump on port 6653 in suite setup

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

Check ServiceStatus For Genius

Change-Id: Iabd83facf300676ed53d6a9e4fe13f6015c20542
Signed-off-by: B Sathwik <b.sathwik@ericsson.com>

Adjust for OVS-28 and disk IO issues

- Add argument ${ofversion} in MininetKeywords library.
- For old test code, just set required OpenFlow version.
- Add new variable ${DEFAULT_TIMEOUT_HTTTP}=5.
- Use the default HTTP timeout in KWs opening HTTP connections.
- KWs invoking HTTP methods should use default timeout=${EMPTY},
which will ultimately default to HTTP connection timeout.

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

Remove unused bgp suite in netvirt folder

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

Add header for distchanges to find easier in log

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

Check classifier interfaces in sfc full deploy

In sfc full deploy, check that the interface are registered in OVSDB
topology before proceeding with the configuration of the classifiers.
The classifier logic in SFC does not intend to be production ready and
does not check for this.

Change-Id: I72a744b1b86711f19fec55240089cb796fac8416
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Increase HTTP timeout for cluster test

Slow disk I/O impacts configuration operations in cluster.

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

Add testplan for tdd-wip genius job

Refer: https://git.opendaylight.org/gerrit/67802

Change-Id: I2273218fda9c523a38372f993eef2425de871b0a
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Bgp HA, code refactoring

Used BGPcliKeywords to remove some replication.
Fixed Variables.robot path Error.

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

CSIT dualstack: fix Test Teardown With Tcpdump

Fix argument list in 'VpnOperations.Test Teardown With Tcpdump Stop'
token call.

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

Add 6653 and 6633 specific port vars

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

Move remaining tcpdump users to use wrapper

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

Add missing itm tunnel-list URL to teardowns

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

Check service function types are removed

Check service function types are removed at suite/test teardown before
continuing with the nex test/suite. This prevents service funtion types
being removed during the nex test/suite making it to fail.

Change-Id: Ie6f2bc82b0933c70c99b8e83f44ffe36f48704d7
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Fixes in Openstack Keywords

- Fixed the list access in
  Is Feature Installed

- Fixed the Clean All Keyword
  Made it work for features that does not
  support L3, by ignoring the listing failure

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

Remove dualstack from basic testplan

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

Add missing folder in dualstack csit suite path

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

Add multiple routers to SNAT scenario

Creating another internal network and another router
connected to the new internal network, then launch an instance without
Floating IP and check connectivity from this instance.

Change-Id: I47fd916e61c7c09d1fd3caa3d9eb57109f5af5d6
Signed-off-by: Itzik Brown <itzikb@redhat.com>

Use OpenStack CLI for nova cli commands

- covers hypervisor list and console show
- also deletes deprecated and unused keyword
  "Collect VM Ip Addresses", as we now use a
  newer/better one "Get VM IP"

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

Add dualstack csit suite to tdd-wip job's testplan

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

CSIT: add dualstack test for one-router solution

Introduce new basic dualstack vpnservice test-suite to check
support of dualstack VMs inside L3 BGPVPN. Implemented
neutron network config includes one dualstack router.
It allows to test one-router solution, defined in
l3vpn-dual-stack-vm.rst specification: commit
07ef3e3db56610a878e0a00ada175fa70f05ffe6 "Spec to support
dualstack VMs in L3VPN"

Change-Id: Ie9b048c1ce7329680a064d6fca572b6930244d71
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Add testplan for tdd-wip netvirt job.

see this patch for more details:
  https://git.opendaylight.org/gerrit/c/67707/

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

Allow isolation of specific ports

Adding arguments to isolate and rejoin to allow specifying protocol
and port. Useful for testing behaviour of cluster when the only
loss of connectivity happens between the nodes.

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

Give time for reboot in sfc persistency test

Change-Id: I6797dd0489df403b32d7eade9062ac48abd6e8b8
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Check service function types are present (II)

Similar to commit [1], but that one only took care of the SFC full
deploy suite, this one takes care of the SFC basic suite.

Check service function types are present before attempting to create any
RSP making use of them. Otherwise RSP creation will fail if attempted
before the service function types exist. Service funtion types are
created by SFC as soon as the first SF of that type is added.

[1] a77477cfcab80d3060e796cf255b3323b9bb003d

Change-Id: I7bebba9a4b634a54a75b1cd6066de77cc515d3a3
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Clean up SFC CSIT

- move to Variables.robot from Variables.py
- use args for Check Service Function Types so the keyword
  can be commonly used across suites
- moves some seemingly useful keywords from a suite to
  the new SfcKeywords lib

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

Move remaining tcpdump users to use wrapper

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

Check service function types are present

Check service function types are present before attempting to create any
RSP making use of them. Otherwise RSP creation will fail if attempted
before the service function types exist. Service funtion types are
created by SFC as soon as the first SF of that type is added.

Change-Id: Iaa5d0bc966d0bb190e64ef8084b2243078d10fc6
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Fix ip forwarding in sfc full deploy suite

Another docker update, another default change ;) This time ACCEPT is no
longer setup as the default target of the FORWARD chain, so do it
ourselves.

Change-Id: Iab2e875183f6adc20864797d06bdd51e446f01d4
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Pcep HA tests

Using Pcep-Pcc-Mock tool on cluster

Similar to bgp-ha, and bmp-ha

Stopping karaf, waiting for tool to reconnect
Killing karaf, waiting for tool to reconnect

Reworked finding owner with singleton.
Add test templates for pcep-topology with volatiles.

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

Move variable definitions

Move variable definitions from Variables.py to Variables.robot

Change-Id: Ie1c66425ca83801bd38ff2b65b3c487b4f09333a
Signed-off-by: Sai Sindhur Malleni <smalleni@redhat.com>

Create OpenStack wrapper for tcpdump

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

Use intended security group in instance create

there is some environment problem where we end
up with multiple "default" security groups and
if we don't use the unique sg defined for
that suite, we get a failure to create the
instance

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

Add BUG tag to Loop of 3 test under l2switch

Change-Id: I2b264d5520245a9b92cc07a9edd7289b4f2fbb1f
Signed-off-by: Chethana Lakshmanappa <chethana@luminanetworks.com>

Remove the 2>/dev/null workaround

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

Pop VM from list after deleting it

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

Pop VM from list after deleting it

now that we are checking nova commands for proper return
codes, we cannot just happily delete things that are already
deleted. So, for the test case where we delete a single
instance to make sure ping traffic fails, we also need to
remove that instance from the list and let further cleanup
skip it.

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

Correct Genius idmanager test suite name

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

Enhance genius CSIT Teardown dumps

The teardown dumps were missing some of the critical
REST URLs for debugging

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

Redirect openstack cli stderr > /dev/null

Something is slightly broken with our current setup of using
latest openstackclient 13.0.0 with one version back of
novaclient 9.0.0 such that when doing calls like "server
list" we are gettins some stderr output that looks like
some of the debug output when using -v or --debug.

this will supress it for now. Probably a bug should be
raised with openstackclient that it's not handling the
older novaclient properly. Not sure if they are supposed
to work nicely or not.

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

Add debug logging

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

Change Karaf prompt to root

This should fix bump delay issues in CSIT while working with
old distribution provided the overlap between prompts is minimal:

[36mopendaylight-user[0m[1m@[0m[34mroot[0m>
[?1l>[?2004l[?1h=[?2004hopendaylight-user@root>

- We can use string 'opendaylight-user' for session login
- We can use regexp 'opendaylight-user.*root.*>' for read until prompt

Change-Id: Id1d9161025d4d0910d21cbbf2cea358ce9258684
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Change the Genius Test Suite Name

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

BMP HA test

bmp_ha_karaf_stop
Simulate HA with BMP Mock tool
Stopping karaf on one node and verifying reconnection.
Restarting karaf, verifying its presence.

bmp_ha_karaf_restart
Killing karaf on one node and verifying reconnection.
Restarting karaf, verifying its presence.

Reconfiguring timers for HA tests

Removed Isolation tests, because right now they are failing.

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

Fix L2GW CSIT

Some recent changes to OpenStackOperations require adjustments
in this suite.

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

Expand HN to HOSTNAME

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

Place VMs deterministically

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

BGP Peer Reset Session Test

Using new reset-session command on oxygen

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

Add ability/flag to clean OS in suite setup

it will make it easier for other environments to adopt our
upstream CSIT if we can also easily provide the ability to
clean up the environment from any left overs that their
previous tests may have left over.

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

Fix list typo in tcpdump keyword

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

Add tcpdump filter

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

Add additional OVS logs

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

Add bound-services-state-list to model dump

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

Increase topology recover timeout

In cluster setup, topology can take some time to recover.

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

Tcpmd5 password reconfig

Additional password change on pcc-mock side.
Additional related tests.

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

disable ofp stats in netvirt csit

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

PCEP Speaker Entity Identifier

Additional configuration for PCEP Speaker.
PCEP Speaker can skip session sync.
Test Cases functionality doesn't change.

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

tcpmd5user test update

Updated test to use new pcep MD5 authentication for oxygen.
Added backwards compability for carbon and nitrogen.
Basic tests specified for path-computation-client only
Oxygen tests include pcep-session-state comparison

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

Use Variables.robot instead of Variables.py

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

IPv6: refactor Collect VM IPv6 SLAAC Addresses

Make Collect VM IPv6 SLAAC Addresses test more reliable: at first it
will take IPv6 address, which was saved for given VM in neutron
database, after that it will try to connect to VM at this address and
check an output of ip -6 a command.

Update arguments for 'Collect VM IPv6 SLAAC Addresses' call in
netvirt/vpnservice/vpn_basic_ipv6.robot test suite.

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

Improve RCORS KW to return stdout, stderr or both

- adds two default arguments to "Run Command On Remote System"
  to control which output (or both) is returned from the keyword

This removes the duplicate code we ended up with when wanting to
have the keyword return stderr

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

Fix cbench test

- Command requires -S option (not -s)
- Command results shows in stderr

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

Basic clustering tests for daexim

  - Executing pre-existing tests on cluster
  - Adding basic export testcase to cluster
  - Adding a new kw and testsuite for localized export
  - Swapping out initial state with skipped and removed unwanted json
  - Modifying kw to verify files
  - Changing the order of test cases
  - Cleaning up the export files before export call
  - Adding basic import to a cluster
  - Correcting a typo
  - Modifying the kw as per comments

Change-Id: I06cc3570e413edfa527a87ac4944d916fa5bc39e
Signed-off-by: Sreekalyan Devaraj <kalyan.ds@gmail.com>

Fix Docker troubles in aaa keystone suite

- the main problem was that filesystem mounts were failing due to
  permission issues. Adding :Z to the end of each mount is the
  fix for that.

- a few doc format issues and a TODO were also added

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

Set AUTH variable in natapp basic suite

Using ODL_RESTCONF_USER and ODL_RESTCONF_PASSWORD
to set the AUTH variable

Change-Id: Ibfda4f3ed38abfe360042c2ef5b3c29412b56974
Signed-off-by: Itzik Brown <itzikb@redhat.com>

Extract the shard name in full

Drop the datastore type off the end of the shard name, to avoid issues
with shard names containing the datastore type
(“prefix-configuration-shard” for example).

Change-Id: I6d7af717b0bc6c7490b4e1334c8c380046b11b95
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Ensure everything fits on screen

Depending on the window size, the controller names and number and
names of shards, we might run off the end of the screen, which aborts
curses in a rather inelegant fashion. We reduce the column widths
until everything fits.

Change-Id: I2924c005646694368b2eaa220ae3b5f3313b5c6c
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix bug in ip -> hypervisor retrieval

the bug was being hit when the ip address of the compute node
in question matched the substring of the ip address of another
compute node in the hypervisor list output.

example:
10.30.170.5 was matching on 10.30.170.5 -and- 10.30.170.56.

This was probably not happening often, but it did happen here:
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-ocata-upstream-stateful-carbon/245/log_full.html.gz#s1-s8-t7-k3-k3-k4

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

React quickly to quit requests

This patch checks keypresses in both loops inside the main loop,
ensuring that the monitor quits soon after the user presses 'q'.

Change-Id: I5889d1958e6c3887cde715feac4bc3cb87b67f34
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Support starting up with a partial cluster

This patch reworks the monitor so that it can start without a working
cluster:
* controller and shard information retrieval is done in the main loop;
* unknown controllers are displayed using their IP address and port
  (and the “controller” column has its own size);
* whenever information becomes available (controller name, shards) it
  is added to the working set.

Change-Id: Ic50aac9b835e4aabeb5edccb588835d0823e2e47
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Avoid tracebacks on initial probe

rest_get is called to retrieve shard information twice, once without a
try block, and again in a try block. When a controller is down, the
first one fails, so the try block is useless (and the user ends up
with an obscure error message).

This patch removes the first, useless call.

Change-Id: I2f6753d71250e3911fbcbbdd31a8b2f595c663c9
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Track controller names in the main controller map

Instead of building a separate array of controller names, track them
as an additional property in the map (this will simplify handling
controllers which are down at startup).

Change-Id: I97d19bb384ae6f816ff3a4dd97f49f33f659904f
Signed-off-by: Stephen Kitt <skitt@redhat.com>

pcepuser update

On Oxygen there is new session-state output
Reworked Test Cases to compare more specific output

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

bgp flowspec test fix

Unifying number of connection retries across tests
Reconfiguring exabgp config file

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

Increase json http timeout

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

s/CFEATURES/CONTROLLERFEATURES

looks like this patch [0] is using CONTROLLERFEATURES on
the pybot command line. I don't see anywhere where it
gets copied to CFEATURES

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

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

increase template timeout

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

cleanup connections

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

increase tempest timeout

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

Refactor Is Feature Installed

Removed the need to query the odl system by
using the CONTROLLERFEATURES parameter.

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

Cleanup unused netvirt suites

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

Increase Get Vm IPs from 120s to 180s

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

bgp_functional_route_ref fix

Addressing variety in order in cli output.
Devided template into multiple sections.

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

Copy dhcp files

Depends-on: https://git.opendaylight.org/gerrit/66364

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

Add [Documentation] to openstack Suite Teardown kw

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

Add OpenStack grenade

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

add project p4plugin system test job

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

Migrate Variables.py to Variables.robot

per our long lived task:
https://jira.opendaylight.org/projects/INTTEST/issues/INTTEST-31

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

Stabilize netconf test

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

bgp_functional_route test update

verify odl stats using restconf bgp operational-state
verify odl stats using odl-bgpcep-bgp-cli

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

Change return code checker to be more clear

Should Be True    '${rc}' == '0'

  is more clear (to me at least) than

Should Not Be True    ${rc}

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

Do not restart/cleanup OVS in init code

Do not restart/cleanup OVS on compute node during
L2GW CSIT init.

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

vpnservice-ipv6: fix calls in neutron port

When calling neutron port creation, the calls are internal and do not
refer to external libraries.

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

Add keyword for iptables cmd on remote system

this should make it easier to run any arbitrary iptables
command on a remote system

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

Add 'bug' tag so this TC can be easily excluded

Single 'bug' tag is easy to exclude in case the suite is used
for any sanity or smoke test.

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

Use consistent names for variables

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