Correcting Get Value from JSON

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: Iff74a4a70536b4c233bdf2d2b8c014c8c724eafd
Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>

INTTEST-107: Add test cases to cover rpc augmentation action

Change-Id: I8c9b32aea038d7f7e2003dc0171bc36da4f85cc9
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

python3 changes in openstack scripts

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I2bd8d821f3967f40f031d379bc2642d9d694a7fb

Increase timeout for apidocs URI to 90s

It seems that 60s might not be long enough in all cases as we
see this test fail sometimes. Let's try 90s to see if it helps.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: Id3eb7547dd4be8bac6d3a96ad384d950614a55e8

Fix BGPCEP ingest test

Python3 floor division is //

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

Fix robot syntax with robot.tidy tool

- have to skip .robot files with the trailing-whitespace hook because
it will remove the blank line at the end of a file whereas robot tidy
will add one which causes a failure

This change is coming from using the latest RF and
robotframework-tidy-wrapper.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I7ad30196d4ad3641901578614e3b417e4772498d

Move robot-tidy precommit back to master

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I1e58cdb94da31e98614561f2c00d4473aaae4622

Remove neon version of netconf testplans

neon is no longer maintained, so remove the files
specific to that.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I65cce28bd05ecb3d26d1b5010a0f40df15ad5806

Tests for BGPCEP-901

- This patch have the flapping pcc sessions test cases and modified as to run 1+ more pcc sessions

JIRA: BGPCEP-901
Signed-off-by: BhuvanaVignesh <bvignesh@luminanetworks.com>
Change-Id: Ifb73a165244ddb7fbbd77327479edf0331af774f

Add gate testplan for rfc8040 job

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I0da8aed19459dca15f376acc66969970f475bafd

Fix python3 issues in OVSDB

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I57bcf759f96c2d7d47ffd9acc9e21599d58c2c36

Add schemas with rpc action in augmentation

INTTEST-107
Change-Id: I4b7a1f96f2a947c30867eac10ce5cea4ab88c4f8
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

INTTEST-83: Add testcase for restconf notification stream

Change-Id: Ifa68dd65afd17a97d4f85025e321cf3771d0493b
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

Fix python3 issues in Netvirt

Also removed VTN library files

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: Ifcfbc76cc5c308f0e7126ac5b494c7e80ca64a18

INTTEST-91 - Updating list and dictionary syntax

Accessing list and dictionary items
using @{var}[item] and &{var}[item] is deprecated
from robotframework 3.2b1

https://github.com/robotframework/robotframework/blob/master/doc/releasenotes/rf-3.2b1.rst#accessing-list-and-dictionary-items-using-varitem-and-varitem-is-deprecated

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: Ic2e9bc0b7edc150c003171022da987b61bd9999f

Fix python3 cluster suites

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

Resolve E722 do not use bare except

Bare exceptions violates one of Pythons idioms that bare exceptions
should not be used. The reason for this is that a bare except catches
all exceptions and can make programming errors look like runtime
problems making debugging more difficult.

Change-Id: I7006639302ef2efc9301c384c19218dd6783c825
Ref: https://docs.python.org/3.1/howto/doanddont.html#except
Signed-off-by: Thanh Ha <zxiiro@gmail.com>

Print git-diff when pre-commit fails

When pre-commit fails and automatically fixes issues for us in Jenkins
we are not able to see what pre-commit automatically fixed. Adding
--show-diff-on-failure makes pre-commit perform a git-diff immediately
after failing to ensure more useful output is printed to inform the
contributor what changes are needed to pass the check.

Change-Id: I7d97b823e401bb737564a3936afa08085dcb6975
Signed-off-by: Thanh Ha <zxiiro@gmail.com>

Make GitLint actually run as part of pre-commit

The tox environment for pre-commit currently does not actually run
the gitlint hook.

Change-Id: Ia22d2d1fed8aed3689690e3f51e06905f409f82e
Signed-off-by: Thanh Ha <zxiiro@gmail.com>

Fix formatting for .pre-commit-config.yaml

File should start with `repos:` as the root level. Without this things
appear to still work anyway but we should probably have the right
formatting.

Change-Id: I437ae2b0164b85125fdeb323d27f025861da9f6b
Signed-off-by: Thanh Ha <zxiiro@gmail.com>

Restore forked ipaddr library

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

Increase HTTP timeout for apidocs test

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

Install ipaddr module in Tools VM

The ipaddr.py was removed in an earlier merged patch, causing
the tests to fail. This patch uses pip to install ipaddr to TOOLS VM.

JIRA: INTTEST-105
Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Change-Id: Id0dcfa0b59f1326baa3a0999ac0a7647dc26a0fa

INTTEST-88: Check namespace in response header

Change-Id: Ib23bbd813c29a47a28b2d5629c9238727d39df54
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

Fix python3 issues in OFP 2

JIRA: INTTEST-105

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

Fix python3 issues in OFP

JIRA: INTTEST-105

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

Fix for Genius Tests

JIRA: INTTEST-105
Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Change-Id: I10e536d74518605c79326509b4245ae4cc87c8e4

Remove HTTP library

The library is old, does not work with python3. I also
do not see where it is even used in the TSDR keywords
so might have been a copy/paste relic anyway.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I6e19ebfb5e07890b5090e7ce184196578bad7c2f

Remove HTTP library

The library is old and does not work with python3. It appears
that the *** Settings *** section in the affected files may
have just been copy/paste from the same source and likely the
library is not used. It's possible that some tests would break
if the library was being used. In that case, things will have
to be ported to the more commonly used RequetsLibrary keywords.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I55e5d02818b0f2bfaf106cb26ecf43df9791c6e3

Remove NIC related test files

The NIC project has long since been archived as a project
and no longer active or part of OpenDaylight. So, remove
these files.

JIRA: INTTEST-105
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I3884a5744bca2ef48db92262a6089d1bd46159a2

Remove token auth suite and it's testplan

Neon is no longer supported or tested and was the
last release that had token authentication. So,
remove the relevant resources.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I324f3de5ce83b6b6ff9ce437bf604e772c448cdd

resp.content->resp.text in Should Contain

The new python3 upgrade breaks our old checks for
Should Contain when using ${resp.content}. The
erorr is:

TypeError: a bytes-like object is required, not 'str'

There will probably be lots of places to fix this.

JIRA: INTTEST-105
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I14726cef95e2d2e7355d6974eacb4646566aa7a5

Fix BGPCEP test for 409 code and isolate failures

Some tests fail because exabgp route-refresh capability but this
capability is only required in the route-refresh test, so this
patch disables this capability if not required.

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

Migrate Tests/Scripts to Python3

- Used 2to3 to migrate scripts to python3
- Fixed some issues due to robot 3.2.1

JIRA: INTTEST-105
Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Change-Id: I1028a2090ae616b192119d0eff8d27c7f95bfddd

Adjust apidocs URL for Aluminium

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

Fix cluster suite for 409 code change

JIRA: INTTEST-102

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

Use wildcard *journal

In Aluminium the journal/ folder is now named segmented-journal/
Using a wildcard in the tests should let it work for all branches

this was partially done already with:
https://git.opendaylight.org/gerrit/c/integration/test/+/88658

Signed-off-by: Jamo Luhrsen <jamoluhrsen@ip-192-168-86-41.us-west-1.compute.internal>
Change-Id: I4cf26da11e388c986ab5343ca1ad93d0beb352f7

Revert "Remove robot-tidy from pre-commit"

Pin to a specific hash ID instead of 'master' to workaround failure
due to:

    "error: pathspec 'master' did not match any file(s) known to git."

Not sure why this is suddenly failing with this repo but using a
specific commit hash instead appears to resolve the issue.

This also changes the clone URL from git:// to https:// to match the
other repos in this file.

This reverts commit c300e14ebc35e94f62ebc2056fad7da81a54ab40.

JIRA: RELENG-149
Change-Id: I42a1ef3e8c347820a5a482b1b530616de75cfc27
Signed-off-by: Thanh Ha <zxiiro@gmail.com>

Use DELETED_STATUS_CODES instead of 404

data-missing will now give 409 in Aluminium and beyond
by default so we can look for both 404 and 409 which is
in DELETED_STATUS_CODES

JIRA: INTTEST-102
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I6f979e3a46b5121f8c98db1511e734c400a70f90

Convert Python boolean to properly-encoded boolean

'True' and 'False' are not valid values for boolean where either
XML (RFC6020/RFC7950) or JSON (RFC7951) encoding is concerned. Fix
this up by explicitly converting booleans.

JIRA: INTTEST-101
Change-Id: Iefbce681e8df3f1ca7701cd89da8b3d7c00a8b1d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove robot-tidy from pre-commit

Nothing wrong with the hook or the repo, but something
has changed recently with pre-commit or some dependency
that causes it to break. More details in:

JIRA: RELENG-149

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I43bba25e27e195224aee4e2763d2259899da8346

NETVIRT-1599 Add support for group-add-mod-enabled param

We have a flag group-add-mod-enabled in /etc/opendaylight/
datastore/initial/config/default-openflow-connection-config.xml
which needs to be enabled for upgrade scripts. Once we wipe ODL
data and then connect back the switch, this flag will enable the
ONFT_BUNDLE_ADD_MESSAGE to be accepted by the openvswitch.

Change-Id: Ie952d5ca5280fbe750495cd34dfbc065bc9c3e2f
Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>

Use correct kw when checking empty DS in netconf library

Change-Id: I488168ef77f7ffd8ca73489c1093b01bbe309279
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

Fix robot bug in Should_Contain assertion

Change-Id: I4f2f8b70cb63f4d09fa2c5466dad1d6371d67909
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

Use ${DELETED_STATUS_CODES} instead of hard-coded value

Change-Id: I5c992a1bb128fd1044cb2449052f85aef8555491
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

Fix typo in lisp test

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

Allow 409 code when DS is empty

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

Increase OF stats polling period in scale test

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

Mininet clean command to use "Execute Command" vs "Write"

"EC" executes and waits until complete. "Write" requires a "read until"

Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
Change-Id: Id417f4bf6afa1dcdc89dcfac13357e0830691a4e

Use correct system prompt variable for tools system vm in test suites

Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
Change-Id: I4490e27cc05c1f4c59ec53981361772a2ba7cca1

Use correct system default prompt for start/stop mininet

Change-Id: I872c9dec3de76de5e5c62508b013cd7102c1513c
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

Add missing WUKS in genius CSIT

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

checking vm state before grep

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I99243f2dc52c0c9f2f4c3a71b854dc5036e84fc6

remove default_java_options kw usage

It was used before when we had java 7 to worry about, but
that is long since gone. The keyword was removed in
https://git.opendaylight.org/gerrit/c/integration/test/+/88656

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I7813d294dbea66a5a2807974e4f27193b8867568

Add venv/ to .gitignore

some IDE/environments commonly create virtual environments by default
at the base of the repo. We don't want to worry about those.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I20e7c1a34404541265c6928182d987e5ae7d0065

Alphabetize .gitignore

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: Ibe8e49156aa63de6d0a1a7e6c6fb10f6996ebd31

Bring yang-model-validator suite/lib up to date

- using new version of the yang model repos to run the
  tool against.
- removed all files/paths for models that will fail
  validation. Each file removed was seen to have
  failed for what looked like valid reasons with
  a few exceptions which will be opened as bugs
  against yangtools to double check the results.
- moved keywords to the main test case to prevent the
  need for SUITE variables. This resulted in the loss
  of a full test case which was not really a test
  case and only collecting a log file when a failure
  happens.
- added more docs to hopefully help the next
  maintainer that works on this suite.
- modified docs to reflect the new state of this suite

JIRA: INTTEST-80
Change-Id: I08815cd10de7ad97745688d50c7f8ed37f9aebbd
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Use proper yang testtool version

Code was sticking the testtool version to something really old and not
letting the EXPLICIT_YANG_SYSTEM_TEST_URL ever be empty, which is
needed in order to dynamically find the right version of the tool
for the actual branch being tested.

JIRA: INTTEST-78
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I62fa8e5276a1306d510c78ac71888449fc9ea54c
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Remove java 7 leftover code

java 7 is long since dead with ODL CSIT so cleaning up

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I860e75556b29bd18313e597fa93eb4864bd388c3

Combine common keywords and fix argument issue

https://git.opendaylight.org/gerrit/c/integration/test/+/88446
broke the getsingle.robot suite because it was duplicating some
work that should be common. So combined the duplication between
it and max_devices in to the netconfkeywords lib. getmulti.robot
is also sharing keywords now.

Also add back 300s timeout for Wait_Connected keyword that was
migrated to NetconfKeywords.robot lib in parallel with the
review/merge of https://git.opendaylight.org/gerrit/c/integration/test/+/88662

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I063b6e13cdbafff778186043a2250da284365091
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Increase connection wait time to 5m

Sodium takes longer for that initial schema compilation and
now that we are using a large juniper based schema, 2m is
not long enough. In the worst case scenario where it would
never connect, then 5m is the max as the first device would
fail and the rest of the test will bail.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I1e57bd7301b3c5f44d267489a1f8ce1c5e64531f
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Upgrade to python3 for pre-commit flake config

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: Ieed91c45a5db9586e18551752507b58940aba0dd

Fix Flake8 errors

flake8 now running with python3 was turning up a lot of
errors about print statements. This fixes some of those
errors, and seems to be enough to get flake8 passing
again.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: Ie02f5266391702e425147b2981aad4bb828bdb0a
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Fix broken glob inside quotes

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I72a959d8735d3eae059d2403fedc62fc10b05b74

Enable rfc8040 requests for notification suite

also improves restconf lib to allow for rpc operations in
addition to config and operational paths

Change-Id: Ied029ef3f5d15b09aa54de7b2dd443633e567d57
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Also transfer segmented-journal directory

When using segmented-journal persistence plugin, we need to move
not 'journal' but 'segmented-journal'. Update the test suite to handle
both cases.

JIRA: CONTROLLER-1411
Change-Id: I1d05692280ae2f0437e1edb969bdf956fdd7edb3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add flag to log, or not, in Get_From_URI kw

the problem fixed here was noticed in the netconf max
device scale job. The GET was retrieving a semi-large
output each time it was checking and looping until
success (which could be minutes) and over hundreds of
devices was causing the logs to grow so large that
robot itself would crash (assuming kernel killed OOM)

Note: some keywords were given an additional argument
${log_response}=True even though the body of the keyword
does not use it. This is because there is a wrapper
keyword that calls other keywords. That wrapper
(Perform_Operation_On_Each_Device) is now passing
this log_response flag every time, even for keywords
that don't need it (e.g., Configure_Device).

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: Ia9828be85e1fc2a7e4392f58c421f49f507d611b
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Add auth header to websocket client connect

by default admin:admin will be used in a Basic auth
header, but users can override with --credentials
arg.

Change-Id: I8fbffce0223ae8aa74c4426a884a4c8296020356
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Use correct keyword to run command on tools system
- generic "Run Command on RS" uses default parameters such as user/pass/key

Change-Id: I4b2657068bbfd5cde6db7e4b40f4d2b40b663ffd
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

Fix usage of executing command on remote system

- generic keyword vs mininet system specific

Change-Id: Ibe9db8d5d9310b4ec7db2ed725be99184834eb91
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>

INTTEST-71: Update nexus release url link

post-carbon the release URL changed from /distribution-karaf
to just /karaf. The controller upgrade suite was still pulling
a carbon release to install an "older" ODL because of this.
Since Magnesium CSIT is configured to only use jdk11 and
Carbon will not start with jdk11 the job was broken.

The changes here will also ensure that the suite will find
and use the previous major version as the "old" controller
install.

Change-Id: I1312fdb95a5969fc86fd1c20afd90e637bbc6043
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Add readthedocs build config file

rtd projects are failing builds. one reason is that some
are using python2 which can't find the sphinx version
that lfdocs-conf is requiring. There may be other
reasons, but this config file should help control which
python version is used when rtd does the build.

Change-Id: I681a32dd9ef81be50df8abec030f59c4bc40d6d4
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Add Aluminium releases to comparestream lib

Change-Id: If3f0968f400bd6bb39b03f0f6c1fd2da7554fc33
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Add soft link for rfc8040 version of testplan

releng/builder code defines it's basic csit jobs to
use a testplan name of the format {project}-{functionality}.txt
with a new job flavor using a {functionality} of
userfeatures-rfc8040, but still wanting to use the same
testplan, the easiest way to accomplish that is with this
soft link.

Change-Id: Ie269ec4430a9c2b8e06b02e582d392df15d8b447
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Add Library/Keywords to dynamically generate URI

The intention is to have a flag for USE_RFC8040 that can
be set to True or False and the final REST endpoint will
be gererated automatically

Change-Id: Ic6514154f59fb2a73b0cc2dab9bec94f8f353ea6
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Rework netconf-userfeatures testplans

neon is only release using something different and all
other releasess are using the same testplan.

Change-Id: I8f0676a2996d35289878094661c3f1bc872aca79
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Use larger juniper schema in max device scale test

Change-Id: Iebb059c5edc0090cb6230573961590bf4c5ac9a1
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Remove -XX:MaxPermSize

We have not supported anything below Java 8 for a long time,
hence we do not need to muck with -XX:MaxPermSize, as it has no
effect since Java 8.

Change-Id: Ie68a311e3260f149764a9fdb485561e69090a044
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

INTTEST-70 : Fix JsonRpc test

Basic suite of JSONRPC was assuming presence of ietf-interfaces
YANG model in global schema context. This is no longer case for some time.
Instead, test service should depend on YANG model provided by JSONRPC,
such as jsonrpc-config.

Signed-off-by: Richard Kosegi <richard.kosegi@gmail.com>
Change-Id: Iac960d771cdff4a9dc814751ad3094f9320fcbc2

Moving vpn-instance yang from VPNMgr > NeutronVPN

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I1f9aecbce34bf572e07eaa52812f62018ed45dcd

Moving vpn-instance yang from VPNMgr > NeutronVPN

Vpn-instance and Vpn-interface Config Datastore are predominantly
used by the neutron-VPN layer, but the datastore is defined in
VPN manager. This review will take care of moving Vpn-instance
and associated yang models to neutron-VPN layer.

The second part of this change[1] will take care of moving
VPN-interface DS to neutronvpn

https://git.opendaylight.org/gerrit/c/netvirt/+/87418/
https://git.opendaylight.org/gerrit/c/netvirt/+/87445/
https://git.opendaylight.org/gerrit/c/integration/test/+/87613

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I054fc3f3104b792a2ad54206d273ec0c9c8eb37b

NETVIRT-1572

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: Ifc33be3605d080ee3dea2eb733f3b1d5412b2834
Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>

update cirros image for stein and master

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: Iac42f70e92db290683a8c5949b2aa56de52ac66e

Add back [Template] override for rest of files

This should clean up what's left after:
  https://git.opendaylight.org/gerrit/c/integration/test/+/87307

Change-Id: I28da3bc07ff1acdb92482a37fed7e3d544119d64
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Add back [Template] override

it seems our robot code cleanup using the tidy tool
for v3.2a1 actually removed empty [Template] lines
thinking they are a mistake? But, in those cases we
really wanted those test cases to skip the default
test template.

this was introduced with:
  https://git.opendaylight.org/gerrit/c/integration/test/+/86550/2/csit/suites/controller/dom_data_broker/clean_leader_shutdown.robot

potentially other test cases were affected, but this
should address some new failures we are debugging
for the sodium SR2 release.

Change-Id: Ib1b4f3126e82e703048dd6ef3f0fdf50336fc4be
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

added stein in exclusion

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I0bcc7542314ba30ea9fa0f1f14f852b832032a18

Pin python setuptools version to 44.0.0

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

Make a neon specific aaa testplan

token auth doesn't work after neon so we have to
split it up for the newer releases.

Change-Id: I11b1267f8544b7114c2462efc9ec58bdcc1f49ef
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Clean up new AAA auth suite

- make test case steps reflect test case names
- enhance variable scheme

Change-Id: I130669a7e29ecf17604924f3d03bd88666d45e28
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Add new restconf auth suite

Change-Id: I4136c93f43489c4c720cf42b62707246355cae89
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

add wait and check for flow entry for a node
Test case changes to look for floating IP of
VM instead of SNAT IP.:
1. Verify Floating Ip
Provision And Reachability From External
Network Via Neutron Router Through L3vpn
2. Verify Floating Ip Re-provision And
Reachability From External Network Via
Neutron Router Through L3vpn

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I3f640f3da4be79a0bf0529f62a76309d429c35e9
Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>

Set the value we use in Jenkins by default

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

Fix docker run command

USER_HOME is reserved for robot image home path.

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

ebgp yang model change
netvirt ref - https://git.opendaylight.org/gerrit/c/netvirt/+/86039

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I7f81a4c4b93001a1afefd9f7b8e1db0b6079d068
Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>

Upgrade RF syntax for v3.2 compatibility

Change-Id: I3db4425c219f955ba8ec98cd26246a5561b77c64
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

INTTEST-65: use resp.text instead of .content

when running with python3, .content may not come
in pure str() format and python3 is a little more
restrictive with that. but the requests library
will always send back resp.text as a string which
is what Should Contain is looking at

Change-Id: Ic6b02c058d702bdb4192ebe701ac8f68af7158b7
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

correction to NETVIRT-1599

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I476c065a11717b8a307bc796ff4d1d5252a180d5

correction to NETVIRT-1599

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: Ia037fe3dd9de1187cc154931e2f408f834e070dd

NETVIRT-1599 - Added rpc commit call after upgrade flag is set

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: I9dfaa84349b4bcb12d4f45e283f66dd519ace829
Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>

Repair PCEP test

Add sleep between the PCC connection and request LSP
sync optimization. See BGPCEP-890.

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