Update exception file output

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

Enable logging for SXP clustering tests

- and use library name with imported keywords

Change-Id: I94d88d5b6bad14723539a4cf08cb6e0efc751568
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Fail Sphinx build on warnings

Also remove redundant echo statement. Sphinx already prints the
docs location at the end of the build and echo breaks Windows devs.

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

Update exception whitelist

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

Remove duplicate context

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

Remove useless steps in Genius CSIT

Sorry to see this, but I have no clue what these steps
had to do in this particular TC. Noticed it now,
because of some recent changes in MDSAL, these GET requests
started throwing error as there is no content over there.

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

Add logout before closing Karaf SSH connection

Not doing this results in a Karaf ssh NPE

See bottom of this log:

https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1
/genius-csit-1node-upstream-all-fluorine/102/odl1_karaf.log.gz

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

Add network datapath check to vpnbasic suite.

Change-Id: I7d819335149781a7259b0b3880c37e4d68cd8e5e
Signed-off-by: mpany <madhusmita.p@altencalsoftlabs.com>

Write exceptions to file

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

Update BFD TestCase & Add Verify Tunnel Status

Verifying tunnel status after disabling BFD and
stopping OVS.New keyword will verify the
tunnel status on given Ip & State of Tunnel.

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

Fix decoding error

   .json() on the response is better than manual decoding with .decode()

Change-Id: Iff6864882cc51491a8564cc2ae67389e817c7cb6
Signed-off-by: Dibya Prakash Das <dibyadascool@gmail.com>

sfc: enable vault repo for kernel packages

It may happen that kernel version running in the provided docker system
image is not current and associated devel packages are not available in
official repo. Enable vault archive repos to fetch them.

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

Update exception whitelist

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

Update exception whitelist

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

Remove all exceptions whitelist

we have this now, which is better:

https://git.opendaylight.org/gerrit/#/c/72135/

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

Update exception whitelist

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

Update exceptions whitelist

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

Update exceptions whitelist

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

exception checker with context

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

Update Verify Tunnel Status As Up

Updation of this keyword is done to add a
transport zone variable and pass it as an argument,
instead of hard coding it.

Change-Id: I0d13688dd97cdd4d198772399ee0c63abf35ae8a
Signed-off-by: Karthika Panneer <r.p.karthika@ericsson.com>

Remove testplans

Due to carbon and nitrogen removal, some of the test
plans are no longer necessary. Keeping the fluorine
userfeatures one though, because I plan to add more
suites to that one soon.

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

Check connectivity after reboot

Add a connectivity check to VMs after taking down all nodes.

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

Fix open files

Change-Id: I6ae9a167c6c7f1cabf52871f22c18ef48c10aa78
Signed-off-by: Dibya Prakash Das <dibyadascool@gmail.com>

CSIT script for bundle based reconciliation

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

Remove unused coe testplans

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

Revert "Revert "Test for Bug Netvirt-1080""

This reverts commit ecf2215e73c49455599f26a612c5a1a17434ff42.

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

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>