integration/test.git
3 years agoAdd schemas with rpc action in augmentation 79/90879/5
Kailash Khalasi [Thu, 2 Jul 2020 16:16:29 +0000 (09:16 -0700)]
Add schemas with rpc action in augmentation

INTTEST-107
Change-Id: I4b7a1f96f2a947c30867eac10ce5cea4ab88c4f8
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
3 years agoINTTEST-83: Add testcase for restconf notification stream 76/90676/6
Kailash Khalasi [Wed, 24 Jun 2020 17:18:47 +0000 (10:18 -0700)]
INTTEST-83: Add testcase for restconf notification stream

Change-Id: Ifa68dd65afd17a97d4f85025e321cf3771d0493b
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
3 years agoFix python3 issues in Netvirt 17/90617/7
Srinivas Rachakonda [Wed, 24 Jun 2020 07:35:45 +0000 (13:05 +0530)]
Fix python3 issues in Netvirt

Also removed VTN library files

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

3 years agoINTTEST-91 - Updating list and dictionary syntax 02/90502/7
Srinivas Rachakonda [Fri, 19 Jun 2020 05:03:40 +0000 (10:33 +0530)]
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

3 years agoFix python3 cluster suites 52/90852/1
Luis Gomez [Wed, 1 Jul 2020 22:29:47 +0000 (15:29 -0700)]
Fix python3 cluster suites

Change-Id: I63d4d7906cbfc81804a2e10793e04109ddf65531
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoResolve E722 do not use bare except 63/90763/2
Thanh Ha [Fri, 26 Jun 2020 15:17:14 +0000 (11:17 -0400)]
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>
3 years agoPrint git-diff when pre-commit fails 97/90397/2
Thanh Ha [Thu, 11 Jun 2020 14:13:02 +0000 (10:13 -0400)]
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>
3 years agoMake GitLint actually run as part of pre-commit 96/90396/2
Thanh Ha [Thu, 11 Jun 2020 14:12:19 +0000 (10:12 -0400)]
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>
3 years agoFix formatting for .pre-commit-config.yaml 03/90403/2
Thanh Ha [Thu, 11 Jun 2020 20:19:25 +0000 (16:19 -0400)]
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>
3 years agoRestore forked ipaddr library 18/90718/2
Luis Gomez [Fri, 26 Jun 2020 02:39:43 +0000 (19:39 -0700)]
Restore forked ipaddr library

Change-Id: Ibdcb3c963fceeac5cd304c956d8d589aa0033912
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoIncrease HTTP timeout for apidocs test 17/90717/2
Luis Gomez [Fri, 26 Jun 2020 01:34:31 +0000 (18:34 -0700)]
Increase HTTP timeout for apidocs test

Change-Id: I180d1f34b5458326ad60161c60e9c51c43ae36d0
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoInstall ipaddr module in Tools VM 19/90619/4
Venkatrangan Govindarajan [Wed, 24 Jun 2020 08:25:43 +0000 (13:55 +0530)]
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

3 years agoINTTEST-88: Check namespace in response header 75/90675/1
Kailash Khalasi [Wed, 24 Jun 2020 17:10:58 +0000 (10:10 -0700)]
INTTEST-88: Check namespace in response header

Change-Id: Ib23bbd813c29a47a28b2d5629c9238727d39df54
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
3 years agoFix python3 issues in OFP 2 14/90614/4
Luis Gomez [Wed, 24 Jun 2020 01:37:08 +0000 (18:37 -0700)]
Fix python3 issues in OFP 2

JIRA: INTTEST-105

Change-Id: I35d00e8a98df9ad1763e9de16da7915d17ef18cd
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoFix python3 issues in OFP 82/90582/22
Luis Gomez [Mon, 22 Jun 2020 20:06:38 +0000 (13:06 -0700)]
Fix python3 issues in OFP

JIRA: INTTEST-105

Change-Id: I70deb287014630d21dbc6842678edef21d5690b7
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoFix for Genius Tests 07/90607/1
Venkatrangan Govindarajan [Tue, 23 Jun 2020 14:43:51 +0000 (20:13 +0530)]
Fix for Genius Tests

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

3 years agoRemove HTTP library 91/90591/2
Jamo Luhrsen [Mon, 22 Jun 2020 23:21:26 +0000 (16:21 -0700)]
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

3 years agoRemove HTTP library 92/90592/2
Jamo Luhrsen [Mon, 22 Jun 2020 23:24:57 +0000 (16:24 -0700)]
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

3 years agoRemove NIC related test files 88/90588/3
Jamo Luhrsen [Mon, 22 Jun 2020 21:56:24 +0000 (14:56 -0700)]
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

3 years agoRemove token auth suite and it's testplan 90/90590/2
Jamo Luhrsen [Mon, 22 Jun 2020 23:19:43 +0000 (16:19 -0700)]
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

3 years agoresp.content->resp.text in Should Contain 93/90593/1
Jamo Luhrsen [Tue, 23 Jun 2020 00:15:04 +0000 (17:15 -0700)]
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

3 years agoFix BGPCEP test for 409 code and isolate failures 62/90462/8
Luis Gomez [Tue, 16 Jun 2020 21:03:28 +0000 (14:03 -0700)]
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>
3 years agoMigrate Tests/Scripts to Python3 27/90527/14
Venkatrangan Govindarajan [Sat, 20 Jun 2020 18:28:50 +0000 (23:58 +0530)]
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

3 years agoAdjust apidocs URL for Aluminium 63/90463/2
Luis Gomez [Tue, 16 Jun 2020 21:58:02 +0000 (14:58 -0700)]
Adjust apidocs URL for Aluminium

Change-Id: I861ed4028a53274b4fa9a73bb8abb48b2717988e
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoFix cluster suite for 409 code change 61/90461/2
Luis Gomez [Tue, 16 Jun 2020 19:27:44 +0000 (12:27 -0700)]
Fix cluster suite for 409 code change

JIRA: INTTEST-102

Change-Id: Ibae69a260fc11dbc91a3b71df180ea49567b6052
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoUse wildcard *journal 41/90341/3
Jamo Luhrsen [Mon, 8 Jun 2020 23:33:47 +0000 (16:33 -0700)]
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

3 years agoRevert "Remove robot-tidy from pre-commit" 93/90393/5
Thanh Ha [Thu, 11 Jun 2020 13:28:07 +0000 (09:28 -0400)]
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>
3 years agoUse DELETED_STATUS_CODES instead of 404 83/90383/1
Jamo Luhrsen [Wed, 10 Jun 2020 21:37:10 +0000 (14:37 -0700)]
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

3 years agoConvert Python boolean to properly-encoded boolean 60/90360/4
Robert Varga [Tue, 9 Jun 2020 14:33:27 +0000 (16:33 +0200)]
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>
3 years agoRemove robot-tidy from pre-commit 64/90364/6
Jamo Luhrsen [Tue, 9 Jun 2020 22:59:56 +0000 (15:59 -0700)]
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

3 years agoNETVIRT-1599 Add support for group-add-mod-enabled param 02/86802/24
Srinivas Rachakonda [Wed, 8 Jan 2020 06:15:17 +0000 (11:45 +0530)]
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>
3 years agoUse correct kw when checking empty DS in netconf library 29/90129/1
Kailash Khalasi [Sat, 30 May 2020 01:15:06 +0000 (18:15 -0700)]
Use correct kw when checking empty DS in netconf library

Change-Id: I488168ef77f7ffd8ca73489c1093b01bbe309279
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
3 years agoFix robot bug in Should_Contain assertion 18/90118/1
Kailash Khalasi [Fri, 29 May 2020 21:00:04 +0000 (14:00 -0700)]
Fix robot bug in Should_Contain assertion

Change-Id: I4f2f8b70cb63f4d09fa2c5466dad1d6371d67909
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
3 years agoUse ${DELETED_STATUS_CODES} instead of hard-coded value 08/90108/1
Kailash Khalasi [Fri, 29 May 2020 18:33:05 +0000 (11:33 -0700)]
Use ${DELETED_STATUS_CODES} instead of hard-coded value

Change-Id: I5c992a1bb128fd1044cb2449052f85aef8555491
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
3 years agoFix typo in lisp test 07/90007/1
Luis Gomez [Wed, 27 May 2020 16:57:34 +0000 (09:57 -0700)]
Fix typo in lisp test

Change-Id: I57111a228050a9e5756444e8537d75ee5111335b
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoAllow 409 code when DS is empty 21/89921/3
Luis Gomez [Sat, 23 May 2020 01:07:22 +0000 (18:07 -0700)]
Allow 409 code when DS is empty

Change-Id: Ib4fc3a58fadc147623091daae8f99b5604e430e2
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoIncrease OF stats polling period in scale test 38/89638/2
Luis Gomez [Thu, 7 May 2020 17:25:28 +0000 (10:25 -0700)]
Increase OF stats polling period in scale test

Change-Id: I013986e4660f84b72f95e321ea0b51db05ff9246
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agoMininet clean command to use "Execute Command" vs "Write" 98/89798/2
Kailash Khalasi [Fri, 15 May 2020 04:14:23 +0000 (21:14 -0700)]
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

3 years agoUse correct system prompt variable for tools system vm in test suites 20/89620/1
Kailash Khalasi [Wed, 6 May 2020 23:00:54 +0000 (16:00 -0700)]
Use correct system prompt variable for tools system vm in test suites

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

3 years agoUse correct system default prompt for start/stop mininet 07/89607/2
Kailash Khalasi [Tue, 5 May 2020 16:03:08 +0000 (09:03 -0700)]
Use correct system default prompt for start/stop mininet

Change-Id: I872c9dec3de76de5e5c62508b013cd7102c1513c
Signed-off-by: Kailash Khalasi <kailash.khalasi@gmail.com>
3 years agoAdd missing WUKS in genius CSIT 09/89609/1
Luis Gomez [Tue, 5 May 2020 23:46:02 +0000 (16:46 -0700)]
Add missing WUKS in genius CSIT

Change-Id: Ia79c8d50d34133fc90b499d79451045b662e43b9
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
3 years agochecking vm state before grep 95/89595/1
Srinivas Rachakonda [Tue, 5 May 2020 01:30:53 +0000 (07:00 +0530)]
checking vm state before grep

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

3 years agoremove default_java_options kw usage 99/89499/1
Jamo Luhrsen [Tue, 28 Apr 2020 18:10:21 +0000 (11:10 -0700)]
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

3 years agoAdd venv/ to .gitignore 24/88724/3
Jamo Luhrsen [Thu, 26 Mar 2020 18:56:32 +0000 (11:56 -0700)]
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

3 years agoAlphabetize .gitignore 23/88723/3
Jamo Luhrsen [Thu, 26 Mar 2020 18:55:56 +0000 (11:55 -0700)]
Alphabetize .gitignore

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

4 years agoBring yang-model-validator suite/lib up to date 37/88837/31
Jamo Luhrsen [Wed, 1 Apr 2020 02:37:26 +0000 (19:37 -0700)]
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>
4 years agoUse proper yang testtool version 17/88817/6
Jamo Luhrsen [Wed, 1 Apr 2020 02:37:26 +0000 (19:37 -0700)]
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>
4 years agoRemove java 7 leftover code 56/88656/2
Jamo Luhrsen [Tue, 24 Mar 2020 18:27:23 +0000 (11:27 -0700)]
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

4 years agoCombine common keywords and fix argument issue 20/88720/7
Jamo Luhrsen [Thu, 26 Mar 2020 18:51:32 +0000 (11:51 -0700)]
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>
4 years agoIncrease connection wait time to 5m 62/88662/4
Jamo Luhrsen [Tue, 24 Mar 2020 23:42:59 +0000 (16:42 -0700)]
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>
4 years agoUpgrade to python3 for pre-commit flake config 38/88738/5
Jamo Luhrsen [Thu, 26 Mar 2020 22:40:19 +0000 (15:40 -0700)]
Upgrade to python3 for pre-commit flake config

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

4 years agoFix Flake8 errors 39/88739/4
Jamo Luhrsen [Fri, 27 Mar 2020 00:22:06 +0000 (17:22 -0700)]
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>
4 years agoFix broken glob inside quotes 58/88658/2
Jamo Luhrsen [Tue, 24 Mar 2020 20:52:12 +0000 (13:52 -0700)]
Fix broken glob inside quotes

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

4 years agoEnable rfc8040 requests for notification suite 26/88226/16
Jamo Luhrsen [Tue, 3 Mar 2020 20:20:42 +0000 (12:20 -0800)]
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>
4 years agoAlso transfer segmented-journal directory 98/88598/2
Robert Varga [Mon, 23 Mar 2020 19:17:07 +0000 (20:17 +0100)]
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>
4 years agoAdd flag to log, or not, in Get_From_URI kw 46/88446/6
Jamo Luhrsen [Mon, 16 Mar 2020 21:53:35 +0000 (14:53 -0700)]
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>
4 years agoAdd auth header to websocket client connect 77/88277/3
Jamo Luhrsen [Fri, 6 Mar 2020 01:20:50 +0000 (17:20 -0800)]
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>
4 years agoUse correct keyword to run command on tools system 02/88402/1
Kailash Khalasi [Wed, 11 Mar 2020 17:20:42 +0000 (10:20 -0700)]
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>
4 years agoFix usage of executing command on remote system 91/88391/2
Kailash Khalasi [Tue, 10 Mar 2020 17:28:58 +0000 (10:28 -0700)]
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>
4 years agoINTTEST-71: Update nexus release url link 06/88106/10
Jamo Luhrsen [Thu, 27 Feb 2020 18:32:39 +0000 (10:32 -0800)]
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>
4 years agoAdd readthedocs build config file 64/88064/1
Jamo Luhrsen [Wed, 26 Feb 2020 00:35:01 +0000 (16:35 -0800)]
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>
4 years agoAdd Aluminium releases to comparestream lib 47/87947/2
Jamo Luhrsen [Fri, 21 Feb 2020 17:45:04 +0000 (09:45 -0800)]
Add Aluminium releases to comparestream lib

Change-Id: If3f0968f400bd6bb39b03f0f6c1fd2da7554fc33
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
4 years agoAdd soft link for rfc8040 version of testplan 39/87939/3
Jamo Luhrsen [Fri, 21 Feb 2020 06:56:06 +0000 (22:56 -0800)]
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>
4 years agoAdd Library/Keywords to dynamically generate URI 97/87897/7
Jamo Luhrsen [Wed, 19 Feb 2020 01:28:52 +0000 (17:28 -0800)]
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>
4 years agoRework netconf-userfeatures testplans 38/87938/1
Jamo Luhrsen [Fri, 21 Feb 2020 06:53:02 +0000 (22:53 -0800)]
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>
4 years agoUse larger juniper schema in max device scale test 17/87517/18
Jamo Luhrsen [Sat, 8 Feb 2020 00:47:17 +0000 (16:47 -0800)]
Use larger juniper schema in max device scale test

Change-Id: Iebb059c5edc0090cb6230573961590bf4c5ac9a1
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
4 years agoRemove -XX:MaxPermSize 04/87904/1
Robert Varga [Wed, 19 Feb 2020 10:08:55 +0000 (11:08 +0100)]
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>
4 years agoINTTEST-70 : Fix JsonRpc test 14/87714/7
Richard Kosegi [Fri, 14 Feb 2020 13:31:58 +0000 (14:31 +0100)]
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

4 years agoMoving vpn-instance yang from VPNMgr > NeutronVPN 63/87663/1
Srinivas Rachakonda [Thu, 13 Feb 2020 09:44:42 +0000 (15:14 +0530)]
Moving vpn-instance yang from VPNMgr > NeutronVPN

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

4 years agoMoving vpn-instance yang from VPNMgr > NeutronVPN 14/87614/2
Srinivas Rachakonda [Wed, 12 Feb 2020 10:42:35 +0000 (16:12 +0530)]
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

4 years agoNETVIRT-1572 94/87094/27
Srinivas Rachakonda [Thu, 23 Jan 2020 08:05:08 +0000 (13:35 +0530)]
NETVIRT-1572

Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
Change-Id: Ifc33be3605d080ee3dea2eb733f3b1d5412b2834
Signed-off-by: Srinivas Rachakonda <srinivas.rachakonda@gmail.com>
4 years agoupdate cirros image for stein and master 11/87311/2
Srinivas Rachakonda [Fri, 31 Jan 2020 09:58:09 +0000 (15:28 +0530)]
update cirros image for stein and master

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

4 years agoAdd back [Template] override for rest of files 08/87308/1
Jamo Luhrsen [Fri, 31 Jan 2020 02:09:59 +0000 (18:09 -0800)]
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>
4 years agoAdd back [Template] override 07/87307/1
Jamo Luhrsen [Fri, 31 Jan 2020 01:30:02 +0000 (17:30 -0800)]
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>
4 years agoadded stein in exclusion 82/87282/1
Srinivas Rachakonda [Thu, 30 Jan 2020 08:56:09 +0000 (14:26 +0530)]
added stein in exclusion

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

4 years agoPin python setuptools version to 44.0.0 42/87242/2
Luis Gomez [Wed, 29 Jan 2020 04:30:53 +0000 (20:30 -0800)]
Pin python setuptools version to 44.0.0

Change-Id: I2a56b521041815c5bb5f82bbee01b51c12e9d5f9
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
4 years agoMake a neon specific aaa testplan 20/87120/1
Jamo Luhrsen [Thu, 23 Jan 2020 23:46:59 +0000 (15:46 -0800)]
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>
4 years agoClean up new AAA auth suite 18/87118/1
Jamo Luhrsen [Thu, 23 Jan 2020 21:18:42 +0000 (13:18 -0800)]
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>
4 years agoAdd new restconf auth suite 87/87087/5
Jamo Luhrsen [Wed, 22 Jan 2020 22:10:22 +0000 (14:10 -0800)]
Add new restconf auth suite

Change-Id: I4136c93f43489c4c720cf42b62707246355cae89
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
4 years agoadd wait and check for flow entry for a node 37/86837/26
Srinivas Rachakonda [Fri, 10 Jan 2020 08:23:56 +0000 (13:53 +0530)]
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>
4 years agoSet the value we use in Jenkins by default 46/86846/1
Luis Gomez [Fri, 10 Jan 2020 18:56:21 +0000 (10:56 -0800)]
Set the value we use in Jenkins by default

Change-Id: I89dfaaf828886cfe9f511166a104be9b33b9f872
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
4 years agoFix docker run command 13/86813/1
Luis Gomez [Wed, 8 Jan 2020 23:11:38 +0000 (15:11 -0800)]
Fix docker run command

USER_HOME is reserved for robot image home path.

Change-Id: I991e453173f1a41fd80c24741002e7ad37603e60
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
4 years agoebgp yang model change 33/86733/5
Srinivas Rachakonda [Mon, 6 Jan 2020 08:54:45 +0000 (14:24 +0530)]
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>
4 years agoUpgrade RF syntax for v3.2 compatibility 50/86550/2
Jamo Luhrsen [Mon, 23 Dec 2019 22:30:46 +0000 (14:30 -0800)]
Upgrade RF syntax for v3.2 compatibility

Change-Id: I3db4425c219f955ba8ec98cd26246a5561b77c64
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
4 years agoINTTEST-65: use resp.text instead of .content 82/86682/1
Jamo Luhrsen [Fri, 3 Jan 2020 01:23:22 +0000 (17:23 -0800)]
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>
4 years agocorrection to NETVIRT-1599 16/86516/1
Srinivas Rachakonda [Fri, 20 Dec 2019 07:40:16 +0000 (13:10 +0530)]
correction to NETVIRT-1599

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

4 years agocorrection to NETVIRT-1599 06/86506/1
Srinivas Rachakonda [Thu, 19 Dec 2019 14:04:44 +0000 (19:34 +0530)]
correction to NETVIRT-1599

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

4 years agoNETVIRT-1599 - Added rpc commit call after upgrade flag is set 30/85930/8
Srinivas Rachakonda [Thu, 21 Nov 2019 11:24:59 +0000 (16:54 +0530)]
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>
4 years agoRepair PCEP test 22/86422/5
Luis Gomez [Sat, 14 Dec 2019 00:35:08 +0000 (16:35 -0800)]
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>
4 years agoRevert "Enable check for status after member starts up" 11/86411/2
Luis Gomez [Fri, 13 Dec 2019 04:34:21 +0000 (04:34 +0000)]
Revert "Enable check for status after member starts up"

This reverts commit c7b7b369035a373808d5a8253f92923e184b7420.

Change-Id: I51c2c59b5d5f42cb28f1defedd2ab1ef9d3bb8e5
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
4 years agoOptimization variable name in host_route_handling.robot 41/86341/2
Ning Zhang [Wed, 11 Dec 2019 01:58:48 +0000 (09:58 +0800)]
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>
4 years agoGet vm ip failed during test host_route_handling.robot 56/86056/7
Ning Zhang [Fri, 29 Nov 2019 07:33:08 +0000 (15:33 +0800)]
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>
4 years agoEnable check for status after member starts up 90/86190/6
Luis Gomez [Wed, 4 Dec 2019 19:04:37 +0000 (11:04 -0800)]
Enable check for status after member starts up

Change-Id: I02d3dfb7e5612e8d17aa01348b19787240ad7cea
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
4 years agoODL Bgpvpn function test failed 47/86247/1
Ning Zhang [Fri, 6 Dec 2019 02:54:00 +0000 (10:54 +0800)]
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>
4 years agoincrease the timeout for Get VM IP 23/85923/6
Srinivas Rachakonda [Thu, 21 Nov 2019 07:41:53 +0000 (13:11 +0530)]
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>
4 years agoFix jsonrpc suite name 96/85896/2
Richard Kosegi [Fri, 15 Nov 2019 08:34:44 +0000 (09:34 +0100)]
Fix jsonrpc suite name

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

4 years agoStabilize OF Stats Perf test 70/85870/1
Luis Gomez [Tue, 19 Nov 2019 20:39:49 +0000 (12:39 -0800)]
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>
4 years agoAdd NETVIRT-1640 to whitelist 72/85772/2
Srinivas Rachakonda [Fri, 15 Nov 2019 09:49:31 +0000 (15:19 +0530)]
Add NETVIRT-1640 to whitelist

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

4 years agoINTTEST-64: Fix BGPCEP 3node HA Test 21/85821/1
Venkatrangan Govindarajan [Mon, 18 Nov 2019 12:03:33 +0000 (17:33 +0530)]
INTTEST-64: Fix BGPCEP 3node HA Test

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

4 years agoRemove trailing whitespace in all files 82/85782/1
Thanh Ha [Fri, 15 Nov 2019 18:22:08 +0000 (13:22 -0500)]
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>