Revert "ACL Service Recovery"

This reverts commit 9d7edc291030d091a056c7f16678a3d2276f0a28.

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

ACL Service Recovery

Change-Id: I4f9495d3cd3072f909f0839381dd91d17ab85409
Signed-off-by: SrinivasRachakonda <srinivas.rachakonda@gmail.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Revert "Test for Bug Netvirt-1080"

This reverts commit 14615b37bd847b344f1f3eacbbcf2eceb4e63df4.

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

Use default prompt variable, not hardcoded one

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

Add host route suite to 1node job

suite was merged [0] and worked well in the sandbox for the
3node job. Having it run successfully in this tdd-wip
1node job can get us closer to adding this suite to
the gate/upstream job that we pay close attention to.

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

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

Add host route suite for host route handling.

Change-Id: I412ad117aeb0d20e6ef9288e217bb971c65b0a1e
Signed-off-by: Arunkumar Murukesan <arunkumar.murukesan@gmail.com>

Test for Bug Netvirt-1080

Here, the FIP is detatched from VM and the Operations
are attempted again.

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

Enhance Genius Tests to Check Service Status

This adds a keyword to verify Service Status in 1node
and 3node ODL deployments

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

Test lsp output more precisely

Attributes can change between versions,
therefore we want to see these changes
and address them when necessary.

Currently it's just checking for one line
which is actually several times in the output.

Adding fulltext for comparison for rib which
matters the most in this case.

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

Update ITM_DIRECT_TUNNEL suite

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

Remove variables.py

In flowspec it's not necessary,
and now we can safely delete it.

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

Adjust prefix counts

Due to addition of new features
in fluorine, it is no longer possible
to achieve old counts.

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

Remove variables.py

It's no longer necessary in several
test suites, as result verification
was changed long ago.

Bgp_app_peer reworked with templatedrequests.

Refactor BgpOperations session usage.
Remove unused libraries.

Add RKAIE to karaf console write, because this
is just informative and is not critical.

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

Update Genius.Teardown

Increased timeout to restart controller
added interface list in teardown dumps

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

Update Verfiy Tunnel Status As UP

In ITM Direct Tunnels one testcase has been updated.

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

Revert "Set bundle resync to true in configplan"

This reverts commit 4d795609ee5d5b8fc23a84a8317c7d77c7561650.

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

Set bundle resync to true in configplan

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

Add Keyword to Check /diagstatus

JIRA: INTTEST-45
JIRA: NETVIRT-1177
Change-Id: I59dacf1230faa8cbdb651084b757849bfde78709
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Add exceptions to whitelist for NEUTRON-157

These jobs hit NEUTRON-157 and a new exception is coming out
from an mdsal change in the api. The job below has an example.

https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-fluorine/290/odl_1/odl1_karaf.log.gz

https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-fluorine/281/odl_1/odl1_karaf.log.gz

JIRA: NETVIRT-1090
JIRA: NEUTRON-157
Change-Id: I3d2cb216e896bfd168e85cfe073f019b3aed713a
Signed-off-by: Sam Hague <shague@redhat.com>

Add exception for NEUTRON-157

This job hit NEUTRON-157 and a new exception is coming out
from an mdsal change in the api. The job below has an example.

https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-fluorine/281/odl_1/odl1_karaf.log.gz

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

Local address peer test

Test case meant for validation of
local-address functionality.
When set, bgp peer can connect only
on this address.

Refactor test suite for newer approach
to comparing topology with
TemplatedRequests robot library.
Remove unnecessary code.

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

Fix bgp-cli test

ShellTable has now been migrated to
oxygen as well, so I unified test
with fluorine, since the only difference
was the divider and trailing
white spaces.

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

Use Variables.robot instead of Variables.py

Also change resources to alphabet order in affected
test suites.

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

Refactor keyword Verify Tunnel Status As UP

Tunnel Status As UP keyword is just verifys tunnels are Up or not.
In this we are verifying no.of tunnels created are equal to
no.of expected tunnel

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

Change comparestream usage

This patch makes lines in code shorter and cleaner.
Also robot logs are less pain to examine.

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

Revert "Do not fail on ODLPARENT-152"

original patch works for most cases, but has some corner
cases that don't work. We can ignore the ssh NPE entirely
from showing up with this patch:
  https://git.opendaylight.org/gerrit/#/c/71423/

This reverts commit 5cc59cbad39a4d759d8814089e7283aecfdae06c.

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

Fix itm-direct-tunnels OVS restart TCs

Genius itm-direct-tunnels suite was using the wrong IP
to restart the OVS.

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

Fix itm-direct-tunnels suite tear down.

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

Add scriptplan for genius-verify

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

Fix NIC basic test.

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

Genius scriptplans and testplans

After the recent refactoring of genius CSIT yaml in
builder repo, there was a need for changing some of the
scriptplans and testplans. Adding the new plans
with soft links to the existing plans.

Change-Id: I93479f054dfd3b506135f93e5868dcf3cad4d946
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Bgp app peer xml change

Test was sending POST into wrong uri,
although it did use to work.

Patch adjusts xml template when we are
sending POST requests, so it
works with that uri.

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

Change naming to future proof

With next version of odl, I would have
to do this anyway, so might as well just
do it now.

Since it's only test for oxygen and fluorine
right now. It will take .oxygen files
and the default files for other versions.

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

Disable the ssh exception logging

JIRA: ODLPARENT-152
Change-Id: If2d6fc330e6f3836bd01795ed159bb0eed4d9fa8
Signed-off-by: Sam Hague <shague@redhat.com>

Allow connection to karaf port during suite setup

Typical tripleo deployment doesn't allow external connections
to karaf consoles. As a result the setup and thus all the other
tests fail.

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

Add script-plan for genius-tdd-wip

test-genius-tdd-wip from genius patches is not executing the
script plan, and hence test cases are failing.

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

SFC-219: Fix incorrect transport id

Change-Id: I379ed3394475e6a4d7649cfa18ab35a8f3d1bb84
JIRA: SFC-219
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

Update test docs

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

Genius test-plan refactoring per stream

The corresponding builder repo patch :
https://git.opendaylight.org/gerrit/#/c/71283/

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

Add some missing model dumps

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

Do not fail on ODLPARENT-152

this is a special case exception we want to ignore and
not cause failures in the robot test cases.

The Exception prints a generic 2nd line that was falling
through our normal whitelist process. So this special case
will be caught IFF those two lines appear back to back.

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

INTTEST-35 Explicitly install os-testr

os-testr is no longer being installed as a dependency
when tempest is installed. Eventually we should figure
out the right/new/supported way to do this, but for now
we need our tempest.api.network tests to run

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

Bgp-cli debug

Output of cli has changed in fluorine.
Also reworked some of the test cases
to be clearer and more universal.

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

Bmp initiation timeout

On all versions bmp currently does
not start within 50sec currently assigned
to it.

Increasing this value.

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

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>