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>

Revert "Enable check for status after member starts up"

This reverts commit c7b7b369035a373808d5a8253f92923e184b7420.

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

Optimization variable name in host_route_handling.robot

Chang variable name to singular format:
${NETWORK_1_VMS} --->  ${NETWORK_1_VM}

Change-Id: I6ce46966b7f2b1eff4f81bacdf22a27328494e5f
Signed-off-by: Ning Zhang <zhangninglc@inspur.com>

Get vm ip failed during test host_route_handling.robot

Increase timeout for Get VM IP.

JIRA:INTTEST-66

Change-Id: If446618553ba2646168676c9601d431eaa649c89
Signed-off-by: Ning Zhang <zhangninglc@inspur.com>

Enable check for status after member starts up

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

ODL Bgpvpn function test failed

The case in bgpvpn.robot test failed: Verify ODL supports VPN creation via neutron bgpvpn create command

Change-Id: Ib786f1e276c12c5df394d4e7bf04b20273acf0f1
Signed-off-by: Ning Zhang <zhangninglc@inspur.com>

increase the timeout for Get VM IP

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

Fix jsonrpc suite name

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

Stabilize OF Stats Perf test

We see some failures in master. This will help identifying
if it is timing or real controller issue.

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

Add NETVIRT-1640 to whitelist

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

INTTEST-64: Fix BGPCEP 3node HA Test

Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Change-Id: I1fd6bf303025892d8137286aa05255e52f0ee392

Remove trailing whitespace in all files

Trailing whitespace is a waste of space and does not have any
visual impact. This patch adds a hook that will automatically
remove trailing whitespace in all files.

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

Move pep8 to pre-commit

This final patch removes our dependency on coala which hasn't had
a release in quite some time.

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

Switch to pre-commit for rstcheck

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

Move jjb/set_elasticsearch_attribute_*.sh and jjb/set_jvm_common_attribute.sh to the repo releng/builder

JIRA: INTTEST-62
Signed-off-by: Stanislav Chlebec <stanislav.chlebec@pantheon.tech>
Change-Id: I0375c5b7bbe676ec96a8f924c13b585444316d60

add support for JVM monitoring

- delete
   .../test/csit/scripts/set_elasticsearch_attribute_short.sh (or ...long.sh)
   .../test/csit/scripts/set_jvm_common_attribute.sh
   in scriptplans files (in the folder integration/test/csit/scriptplans)
- delete
  integration/test/csit/suites/integration/Create_JVM_Plots.robot
  in testplans    (in the folder integration/test/csit/testplans)

These explicit mentions are moved to the scripts in releng/builder repo
see related  gerrit patch 85398

JIRA: INTTEST-14
Signed-off-by: Stanislav Chlebec <stanislav.chlebec@pantheon.tech>
Change-Id: I4b6133ee8c2e4c595ee42859a888edf2fc73deea

Remove COE Netvirt CSIT Scripts

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

INTTEST-61 lispflowmapping failing

LISP: fix 200 -> 204 status code in perf tests
Signed-off-by: BhuvanaVignesh <bvignesh@luminanetworks.com>
Change-Id: I97a4b1cc1dd092d68e526d7f6558b57358f2592f

Magnesium support for mdsal netconf

Signed-off-by: BhuvanaVignesh <bvignesh@luminanetworks.com>
Change-Id: I6366185c8050762f020969835bee4d51e16aeb79

Test not using JDK-11.

The robot keyword uses \$JAVA_HOME instead of JAVA_HOME (this is passed as
argument to the robot keyword).

Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Change-Id: Ibfbd63b6bc227ca10f26000465b8f608c1d582f5

Fix Typo for sodium

Signed-off-by: BhuvanaVignesh <bvignesh@luminanetworks.com>
Change-Id: I53b44e391249d63b7a3117d9d56d628d38b000b7

Change getter.py script so it does not use token

If scope variable is not set, the script will use
basic authn vs token. This will fix netconf scale
tests in sodium and beyond.

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

CSIT for Yang 1.1 actions

Change-Id: I49a579fd0231ba74ebbe96ef653fbd6af0a9b65a
JIRA: NETCONF-636
Signed-off-by: mariusz.sobucki@est.tech

correct check_for_null for l2gw delete

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

LISP: fix stable/sodium CSIT jobs

Change-Id: I22297049929750d9345fa9bc00699696993bc21b
Signed-off-by: Lori Jakab <lojakab@cisco.com>

Fix for add floating ip command change in rocky

With change of openstack version the command for add float ip
has changed

Signed-off-by: JayaPr <jaya.priyadarshini@ericsson.com>
Change-Id: I1309b140760e71eb5c93046a33d7493b7a90da71

Move l3vpn_bgp_multipath_maxpath_cli from tdd to dcgw job

Signed-off-by: JayaPr <jaya.priyadarshini@ericsson.com>
Change-Id: Ic1ef6f9028681485e2d5f070d22daad9f14fb173

Add testcase to Verify The Subnet Route When The Network Is Removed From The Vpn

Signed-off-by: JayaPr <jaya.priyadarshini@ericsson.com>
Change-Id: I958ff933c476f0fac7dedd642227b56c9df409e9

Increase WUKS timeout due to change of behavior

See details in OPNFLWPLUG-1074

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

Remove test deleting port in OVS

This produces an OVS crash and OVS (and controller) may not behave correctly
after that.

|00178|bridge|INFO|bridge s2: deleted interface s2-eth1 on port 1
|00179|util|EMER|lib/netdev.c:2281: assertion ifidx failed in
netdev_ports_remove()
|00005|daemon_unix(monitor)|ERR|2 crashes: pid 13613 died, killed (Aborted),
core dumped, restarting

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

Modified the description for bgp_multipath_maxpath

Change-Id: Ifa5e505cd9d497f97ba71a79383a6a4bd240c467
Signed-off-by: JayaPr <jaya.priyadarshini@ericsson.com>

Add testcheck for Subnet Route For single/multiple Subnet On
Single/Multiple` Vswitch

Signed-off-by: JayaPr <jaya.priyadarshini@ericsson.com>
Change-Id: Ide31666f36829f8ea5ca0ee87cf69b791c90450f

Adjust BGPCEP test to BGPCEP-877 bug fix

Also repair Sum_Hex_Message Keyword.

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