Add missing , for model

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

Add negative test and happy path for router delete

- there is some situation where deleting a router that is
associated with a VPN causes a cascading effect of failing
test cases. We need to first disassociate the vpn before
deleting the router. However, this seems like it's not
handled correctly. Should we be rejecting the router
delete? So that negative test flow is also added here.

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

Remove hardcoded line nos in status output matches

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

Fix robot syntax

Additional dot in test.

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

Add timeout for rib out checks

Because route-reflect can take up to
10 seconds when using play.py tool.

Also changed test case name to better reflect
functionality.

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

Application peer oxygen route key

Output was the same for both oxygen,
and fluorine versions.

Add fluorine output and change the default
output to match oxygen values.

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

Local as peer test

Testing local-as configuration.
Configures external peer with local-as,
and internal peer without local-as.

We connect play.py python tool to both
peers, and we wait for them to exchange
routes.

Local-as from external peer is expected
on outputs of both peers, and each peer
should contain the others peer-as.

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

Adjust UM projects CSIT documentation

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

Update of Check_Karaf_Log_Message_Count API

Enabling user option to use either console or karaf log

Change-Id: I42e23647c608e91a0c8d7d3d271f571f727c0511
Signed-off-by: eafatha <a.fathima.thasneem@ericsson.com>

Multipath route-key config

Addition of mandatory fluorine
argument, but invalid for old
versions of odl, as ipv4/6 and linkstate
route-key is not supported.

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

Route refresh config update

Route-key parameter is now mandatory
when sending routes to bgp on fluorine.

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

Add route-key into policies

Mandatory fluorine argument.

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

Cluster id test case

In fluorine mutliple cluster-ids are
supported, therefore we add new test case
to test this functionality.

Two ibgp rr-client peers introduce prefixes
and we check if controller adds respective
cluster-ids as argument to these routes.

Also fixing play.py type hex for cluster-id
argument.

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

Bgp app peer route key

Addition of mandatory fluorine argument.

Removing support for lithium, beryllium.

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

Count threads in suite teardown

- in light of the recent discovery that we get a lot of threads
running when doing karaf shell commands (especially via ssh),
this info might be useful to have down the road.

see: https://jira.opendaylight.org/browse/INFRAUTILS-29

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

Test to cover No FIP VM to FIP VM traffic.

1)This covers NFIP to FIP same compute and different compute.
2)NFIP-FIP from a NAPT switch and Non-NAPT switch(Assumes NAPT switch is
scheduled on a compute).

Change-Id: I9956aa3a52245daf1ab9a7a31e145669168c5a96
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Whitelist karaf ssh interrupted timeout Exception

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

Adjust OF stats test

Do not push flows until switch is really connected.

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

Check_Karaf_Log_Message_Count API added

Change-Id: If0d9e685e3363b26bb4ebe3b69d546bd1eae23cf
Signed-off-by: eafatha <a.fathima.thasneem@ericsson.com>

Remove ':' OLFE exception in whitelist

seems that we sometimes get OptimisticLockFailedExceptions
with and without the ':' on the end. Remove it for now so
we can still ignore OLFEs in CSIT

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

Add mandatory configuration

Result of: BGPCEP-743
On Fluorine we have to add default policies
when reconfiguring example-bgp-rib, which does
contain in fluorine config located in
odl-bgpcep-bgp-config-example feature.

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

Fail test cases on Exception in karaf.log

There will be a global variable to control whether this check will
happen or not. a -v FAIL_ON_EXCEPTIONS:False (or True) on the
pybot command line can be used.

Also moved some keywords that are focused on parsing karaf.log
messages to KarafKeywords.robot. They did not seem to fit in
the OpenStackOperations.robot library

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

Update CSIT doc for Unmanaged projects

Also update JJB project examples.

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

Exabgp ipv6 connect test

Tests simple ipv6 functionality.

Configures neighbor as ibgp peer with ipv6
address. Starts exabgp with various ipv6
configurations, and verifies all of them report
expected data.

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

BGPCEP-781: Adjust evpn routes

Due to addition of path-id to evpn,
so we can use multiple evpn routes at once,
tests have to be adjusted since path-id is
now mandatory for evpn

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

Bgp policy add AS number

Due to developement on fluorine,
we add this Autonomous System
number to output of odl.

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

OVSDB.robot library file update

Change-Id: Id5a74ceee47061cb09d4912511ef42b792b81b2b
Signed-off-by: eafatha <a.fathima.thasneem@ericsson.com>

Revert "Exclude failing tempest test until nova bu

This reverts commit 21bff84a69111758a8bb9ea74028002a9584c425.

Reason for revert: nova patch was merged in queens which should allow
this test case to pass.

  https://review.openstack.org/#/c/553817/

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

Connectivity test with blocking ports

Adding a new clustering test which checks how the cluster behaves
when we lose connectivity between ODL nodes while keeping them
available on other interfaces.

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

BGPCEP-781: Adjust l3vpn routes

path-id is now mandatory for l3vpn routes,
and should be expected and configured as well.

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

BGPCEP-781: Add fluorine outputs

Mandatory path-id argument is now visible
in policies results.

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

Additional cases for clustering HA L3 tests

Taking down controllers one-by-one doesn't guarantee that a leader
will be killed. This patch explicitly tests recovery after removal
of a leader. It also adds recovery after all nodes in the cluster
are restarted.

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

Remove version feature test in master

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

Parse taglist.log for autorelease builds

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

Peer group reconfig test case

Add test cases for peer-group
configuration and reconfiguration.

Configures peer-group assigns peer to it.
Checks working peer with filled topology.
Reconfigures peer-group without appropriate
afi-safi, and checks empty topology.

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

Reference OVSDB-413 instead of bz 8529

eventually we need to move all CSIT bug references to
their JIRA counterparts, then change the bug reporting
keyword to make the link proper. For now, I just want
to stop having to figure it out when I'm vetting release
CSIT failures.

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

BGPCEP-700: Log message for expected error.

This error is always in karaf.log due to
test intentionally sending routes without
mandatory arguments.

Add log message after start of the test case
to clarify what we are doing.

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

Use Variables.robot in two main libraries

We need to keep moving on this conversion. This patch
is to fix a bug that was very irritating to track down.

this line in Variables.robot is pretty simple:
  @{AUTH}           ${ODL_RESTCONF_USER}    ${ODL_RESTCONF_PASSWORD}

But, changing the two ODL_ variables on the command line
so that we can have AUTH be something different for
other installations was not working. AUTH was still taking
admin:admin as user and password.

That's because we were still sourcing Variables.py in
these two libs (Utils and TemplatedRequests) where this
line exists:

 AUTH = [u'admin', u'admin']

pybot brings in those .py variables files last and they will
take final precendence, even over the command line overrides.

very annoying, and also very hard to figure out unless
you are thinking about it.

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

BGP RIB policies test

This test suite is base for policies tests
which will be filtering routes and peers based on
policies set.

It is running with 6 exabgps in background and app peer.
Configurations of exabgp are subject to change based on
policies applied.

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

Refactor Genius

Removing unnecessary logging statements and global varibales

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

Fix for VTN Coordinator CSIT failure

- CSIT is downloading a wrong version of VTN Coordinator
- This Patch fixes the same by ading the correct index for download.

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

Reference AAA-101 instead of bz 5838

eventually we need to move all CSIT bug references to
their JIRA counterparts, then change the bug reporting
keyword to make the link proper. For now, I just want
to stop having to figure it out when I'm vetting release
CSIT failures.

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

Add new testcase to ITM_Direct_Tunnel

Verifying Tunnels after restarting both OVS and CONTROLLER

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

Add pcep and bmp ha to fluorine

Pcep-ha and Bmp-ha are relevant to versions
Oxygen and above, therefore I am adding
new testplan specific for fluorine to
enable testing of these suites in fluorine.

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

Fix sporadic genius 3 node CSIT failures

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

Add get sysstat keyword

Sysstat is running on all children nodes and
this keyword can be used to get this sysstat
usage statistics when investigating
performance issues on odl nodes.

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

Fix link to releng/builder Jenkins guide

Change-Id: I61c01d5d6895e3946181553801db96c88ad841ea
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix RTD not building robot libdoc

ReadTheDocs does not run tox so needs to be run as part of conf.py
code to force it to build the libdocs.

Change-Id: I04a713319d846659784e9e2099b110323a80ccd8
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Move libdocs script to integration/test

Since this is used by integration/test it should live in
the integration/test project.

Change-Id: I4cbaaa52f82d29947d53b3992dd6bfe113905dbc
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Switch to using lfdocs-conf to pull in theme

lfdocs-conf is a new project that allows us to centrally manage
project theme configuration without having to duplicate it
everywhere or have the "docs" project become a large submodule
project.

We will push to ReadTheDocs separately and then create a new path
for this doc under docs.opendaylight.org/projects/integration-test

For further details refer to:
http://docs.releng.linuxfoundation.org/en/latest/project-documentation.html

Change-Id: I6e88b86cb4089d1410be6f8cd5f2b36a90a33162
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Exclude failing tempest test until nova bug fixed

with this test skipped, we are expecting 100%
pass which will give more confidence to devs
when looking at -gate- results

nova bug:
  https://review.openstack.org/#/c/553817/

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

Test cases  for Bug 8882

Tests the PNF connectivity from SNAT vm

Change-Id: I3a181da8c70f618a83767f940995245f5b510b48
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Increase time for version feature verification

Current time with all featurres loaded is not sufficient

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

NETVIRT-1060:vpn_basic_dualstack: fix extra route

Configure extra-routes IPv4 and IPv6 only in one VM, then update router.
Check, that these Ipv4 and IPv6 extra route addresses are reachable from
all other spawned VMs.

JIRA: NETVIRT-1060

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

Remove newton code

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

Allow offsetting index for raft queries

Our downstream CI uses tripleO for deployment of OS. This can cause
differences in deployment. In this case it's related to akka.conf
which indexes member roles from 0, instead of 1. This results in
some queries failing with 404.

This patch allows us to add an offset for indexes so that we query
the appropriate member-${index}

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

Move Karaf Home variable

Karaf_home variable is used in Variables.robot
without definition, which causes test execution error
in some test suites.

Moved it from Clustermanagement.robot library to
Variables.robot

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

Modify data-store URLs for NetVirt data collection

1) Added config/ebgp:bgp
2) operational/l3vpn:vpn-interfaces is not used anymore from Oxygen. It
has been replaced by operational/odl-l3vpn:vpn-interface-op-data. Keep
both models and just let it be empty for the different branches.

Change-Id: Ie82adead893060f1dc70ae39b4fd28556148e9d0
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Remove dummy.robot

Now that itm-direct-tunnels suite is added here, removing dummy.robot

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

NETCONF-521: Fix netconf test

There is new API for add/remove netconf device keys in Oxygen.
There is new API for configuring netconf device in Oxygen.

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

Test Suite For ITM_Direct_Tunnels

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

Change Genius IdManager Start and Stop Suite

Idmanager does not have any testcase where switches are needed.
Changing the start suite for this, so that we can save
some time during CSIT runs as it saves a lot of unnecessary
operations.

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

include I as part of Change-Id

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

Deploy Openstack Using Robot Keywords

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

Strip Change-Id from regex

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

NETCONF-520: Fix netconf test in oxygen

Looks like a yang model changed during oxygen so new RPC URI and data
for adding and deleting device are required from oxygen onwards.

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

Fix L2GW test case 16

Should be 3 ports, not 1.

Fix a typo.

Check port_mac[2], which is the remote OVSPORT2 MAC,
instead of port_mac[4], which is the local HWVTEPPORT3
MAC.

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

Add OVS dumps to Test Tear Down

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

Add queens tempest exclusions

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

Use fix URL for netconf test-tool from Oxygen

This is because test-tool is broken in Oxygen.

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

Add queens cirros flavor

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

Fix VTN CSIT Issues

- VTN Manager CSIT tests OF10 cases, so modified
  start Keyword to start mininet with OF10

- VTN Coordiantor does not download Oxygen for testing.
  Currently fixed to do so.

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

Remove genius DEBUG

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

Revert "Add trace:transactions to suite teardown"

This reverts commit 360dabe9726142f778b9ae623059277adde0e144.

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

Run command on existing connection

Adding a convenience function to Utils.robot to run a command
on an already open and existing connection

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

Add fluorine keywords to CompareStream lib

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

Add trace:transactions to suite teardown

this will only add the execution of the command. Only
the output will be logged, and no pass/fail criteria
yet.

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

Expect RSP to be persisted from oxygen onward (II)

Previous patch did not account for the fact that CSIT is non versioned
and that RSPs would not be persisted on nitrogen and before.

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

Add to git Ignore

log files
csv files
out files

Change-Id: Ie21e008a199c4d5ee12fa2092ca30c76e451d5bc
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Cleanup ovsdb cluster suites

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

Cleanup ovsdb suites

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

Fix yangtools test tool installation

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

Fix some L2GW test cases for existing connections

Fix some issues in newly added test cases that are
now able to run in queens.

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

Fix mistake [0] in location of Variables.robot

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

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

Add Variables.robot to get visibility to ${AUTH}

in the case when a suite uses this cluster management (CM)
library, but has not imported the global Variables.robot
file, certain variables are not available (e.g. ${AUTH})
which is used in this CM lib. This can cause problems
like unauthorized access when sending REST calls to the
controller.

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

Bgp variables cleanup

Removed all folders which are no longer used.
In particular mostly with .beryllium and .boron
extensions.
The rest is mostly tcpmd5user requests, which were
replaced by variables.py in same folder.

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

Cleanup DevstackUtils.robot

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

GENIUS-111 Genius CSIT - IdManager Failure

Child pool name logic converts host IP to String and
appends it to parent pool name. This can result in
a negative number if IP is 128.x.x.x+ Fixing it in
IdManager can have upgradabuility impact. So modifying
CSIT to allow -ve numbers too.

Refer: 67332

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

Remove installation of features.

Installed features are already in odl compatible all.

Change-Id: I04c82fd70585f672b73e8bd9750fa0f20076648e
Signed-off-by: Andrej Záň <andrej.zan@pantheon.tech>

Breakout complicated setup from suite setup

The arp tests have very complicated setup in the
suite setup. This makes it hard to debug if there are
any failures. Breaking the steps out makes it easier to
debug and allows the debug logging to happen.

Also cleaned up other style issues in the file.

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

Add retries for HWVTEP emulator setup

In queens, I can see that the HVWTEP emulator
shows connected for 0 secs, so let's add some
retries/timeout to give it time to show up in the
config.

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

sfc: dont use internal pattern for rsp names

Avoid using in the tests the same pattern for RSP names
(<sfp-name>-Path-<id>) as that used internally for the
RSPs that are automatically created from SFPs. It can
produce false positive/negative matches on the tests.

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

Fix L2GW cleanup test case

Enhance cleanup test case to cleanup optional resources
only when appropriate.

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

Delete data folder

Delete data along with journal and snapshots folder.
This is needed for ODL upgrade CSIT

Change-Id: I893accaecf6931701b9b27c321a99c47e62798a6
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

Increase timeout and retries for Genius

POST requests are sporadically failing with READ_TIMEOUT
error. This increases default timeout from 1 to 5 secs
and max_retries from 0 to 5 to avoid such issues. Note
that for success scenarios there is no impact as they
will return response anyhow. This just avoids wrongly
marked failures coz we gave up too soon.

Similarly add WUKS for other operations that expect
a change in value. Waiting for a valid response is not
enough, need to wait some more for response to return
a different value.

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

Add Netvirt L2GW test cases

Add test cases to cover the use case where a port
is added to an L2GW that has an existing L2GW
connection.

This is only supported on ODL oxygen and later.
As well, OpenStack must be queens or later.

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

Use L2GW Security Group when creating VMs

There are multiple default SGs in Pike, which cause
failures when creating VMs in L2GW CSIT. So, don't
use default SG, use L2GW SG when creating the VMs.

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

Add Change-Id to regex

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

Tests for Bug #8400

This patch tests the status of traffic when a releated rule
is removed when the communication is happening.

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

Move external net variables to common location

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