Convert bgpcep/throughpcep suite to use NexusKeywords

Change-Id: I1e9928d2bef7b510286f58b9ea5b35833b8f4533
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Changes to fix SFC clustering tests

- updates to SFC yang files to reflect changed model
- Patch Set 2 : rebase
- Patch Set 3 : Fixed SFF config after manually testing
                all the changed JSONs locally.
- Patch Set 4 : rebase
- Patch Set 5 : Fixed other erroneous SFC JSON files

Change-Id: Ibba9a310c159735c54c0952639d5370a13912000
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Adding ovsdb-deletebridge functionality to ovsdbconfigblaster

Change-Id: I901752bcef488ea2297656526af1e915c2f241bb
Signed-off-by: Praveen Mala <praveen.mala@intel.com>

Tidy robot files.

robotframework==3.0

Change-Id: I92d7cb221c5864e570bf607e098e6b68b6ac349c
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

iBGP functional suite check for updates

- correct timing of checks for no updates

Change-Id: I3d56ef8f11eb6ac95fe3b598cfd488b08f317cfb
Signed-off-by: Radovan Sajben <rsajben@cisco.com>

Add more time for CPqD switch in openflow suites

Fix in https://bugs.opendaylight.org/show_bug.cgi?id=4614
produces delays in CPqD switch connection (CPqD bug).

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

Bug 4922 Disable out of date test

This test fails due to change in the way authentication is disabled in
the Beryllium release.  In prior releases, authentication could be disabled
by adding "authEnabled=false" to "etc/org.opendaylight.aaa.authn.cfg". Now,
authentication is disabled by editing shiro.ini and changing:
"/** = authcBasic" to "/** = anon".

Because Shiro properties are not currently tracked through the config admin
service, disabling AAA requires a controller restart.  This test is removed
as it is out of date and no longer reflects how to disable authentication.
A new test is not added to this suite, as controller restart is required.

Change-Id: Ia745c6d722d6a843eac69f8f3998766a8ce48c5d
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Comment non implemented features test

Also make ping test more robust

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

Added BUG ID for failed cases.

Change-Id: I51bbf73c82ca63192101ff0f74b3816ef4febe2a
Signed-off-by: premnath <prem.annauniv@gmail.com>

Moved calls to "RequestLibrary.<action> Request" in suites/openflowplugin/

+ RequestsLibrary.Get Request
+ RequestsLibrary.Post Request
+ RequestsLibrary.Delete Request
+ RequestsLibrary.get Request
+ RequestsLibrary.Put Request

Change-Id: I0168df379cc51abdf7aa81647b3ec3866a32d849
Signed-off-by: Junaid Zulfiqar <junaid.rao21@gmail.com>

Moved calls to "RequestLibrary.<action> Request" in suites/aaa/

+ RequestsLibrary.Get Request
+ RequestsLibrary.Post Request
+ RequestsLibrary.Delete Request
+ RequestsLibrary.GET Request
+ RequestsLibrary.Put Request

Change-Id: I13d83ce842e361f6c97a70ae06188b596a6c9d09
Signed-off-by: Junaid Zulfiqar <junaid.rao21@gmail.com>

FIXED typo in NIC cli keyword

Change-Id: I009882a1d877d6b3ce6ce9c3a0fb5d48c99600ad
Signed-off-by: adetalhouet <adetalhouet@inocybe.com>

Fixing a variable for HBase scripts to run .
Changed Cassandra procedures to fix failures due to metricpath table removal.

Change-Id: I80275bb4a43b4c5b87fed6a522ef6d2a9ee40158
Signed-off-by: premnath <prem.annauniv@gmail.com>

Many iBGP routing reflector clients

- 10 iBGP rrc peers send and receive updates

Change-Id: I839b6eefcde4588cfabec020fa837dd2a8813050
Signed-off-by: Radovan Sajben <rsajben@cisco.com>

Add support for testing release builds

It turns out the only change needed is to introduce the
BUNDLE_URL variable which is supposed to hold the URL from
which the tested ODL distribution is downloaded.
The releng jobs shall use ACTUALBUNDLEURL environment
variable to fill this Robot variable. The code then gets the
URL, snips out everything starting from "org/opendaylight"
and uses the result as the base URL. This makes the "type"
argument of the deployment keyword unnecessary so it was
removed.

Change-Id: I7457b05af25a517d5cdc21011dbdae1e4b34be2a
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Handle missing maven-metadata.xml at artifact location better

When there is no maven-metadata.xml file in the directory
where the artifact is located, assume the version part in the
artifact name is equal to the name of the directory where the
artifact is located. This is needed to support releases which
don't use maven-metadata.xml in the directories with
additional artifacts.

Change-Id: I593305471107c02c66c6c1bb004c3d2a8c6842ac
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Use test tools from release that is under test

The old practice was "use the test tool with the latest
version" which almost always means "use the test tool
from the latest release". However this will break the tests
for the older releases if incompatible change is being made
to these tools. This is currently the case as the latest
release (Boron) is built with Java 8 and the older releases
(Lithium, Beryllium) are built with Java 7 and tested on
Java 7 VMs.

This adds an ugly piece of code to try to deploy and use
test tools from the release that is being tested. The
problem that prompts for the ugly code was reported as
bug 5206.

Change-Id: Id683c352ffd7e506255cbc7e0e28d612c2803360
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Replace deprecated variables in csit/libraries (2).

+ MININET -> TOOLS_SYSTEM_IP
+ MININET_USER -> TOOLS_SYSTEM_USER
+ MININET_PASSWORD -> TOOLS_SYSTEM_PASSWORD
+ MININET_PROMPT -> TOOLS_SYSTEM_PROMPT
+ CONTROLLER_USER -> ODL_SYSTEM_USER

Change-Id: I7ecab0307245c29823ce2baa0d7f0402894ec480
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Moved calls to "RequestLibrary.<action> Request" in csit/libraries/

+ RequestsLibrary.Put Request
+ RequestsLibrary.put Request
+ RequestsLibrary.Get Request
+ RequestsLibrary.GET Request
+ RequestsLibrary.Delete Request
+ RequestsLibrary.Post Request
+ RequestsLibrary.POST Request

Change-Id: Id801b2ecb709368a36b3806ac22640eea9890529
Signed-off-by: Junaid Zulfiqar <junaid.rao21@gmail.com>

Split "name" of test tool to "component" and "artifact"

The "name" of the test tool was an ad-hoc specification of
the test tool that the deployment framework was needed to
somehow use to locate the artifact in Nexus. After reviewing
the code that searches for the artifact I decided to separate
the "component" and the "artifact" into their own arguments
to make this distinction explicit. This is also necessary
to support the increasing complexity of the artifact search.

Change-Id: I3c6677c4aa191c03cc01ef35ad54a359d2320a1d
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Add missing error checks to the artifact deployer

When the artifact is not found or not downloaded, abort the
test suite with a fatal error rather than returning some
bogus file name and letting the Robot wasting time waiting
for timeouts while the Java is trying to run a nonexistent
tool.

Change-Id: I5c8b227057a25caef2e30d0cad28e1a2f76dd6e6
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Add Nexus artifact deployment initialization keyword

Currently this Nexus artifact deployment initialization
keyword contains the code to connect to the tools system,
which was extracted into a keyword in SSHKeywords. This
artifact deployment initialization keyword is then called
in all suites that need to deploy test tools so any code
that explicitly opens this connection was removed from the
suites that need the Nexus artifact deployment facility.

For Netconf test suite this setup keyowrd gets called
automatically from NetconfKeywords setup keyword so
no explicit Nexus keywords initialization is done there.

This is a part of working around the bug 5206.

Change-Id: If46823d7aba92657b99d050e3098ec7c99895824
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Move the testtool start in clustered CRUD into a testcase

The unclustered CRUD suite has test tool deployment as the
first test case. Make the clustered CRUD suite follow that
pattern to improve consistency of the code in these test
suites.

Change-Id: If940ea8aaa20c12997c04d8783402dcf7d608d19
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Added an useful URL to the TODO in northbound.robot

Change-Id: Ib2ab92ce2710c4e0b85911c6fa3f31d2f965bff4
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Fix MDSAL nonexistent namespace error being too restrictive

All that is needed is that the response contains a
<rpc-error> tag and that it mentions the bogus namespace.
Insisting on having also "does-not-exist" or other texts in
the error message is superfluous and easily leads to
spurious failures.

Change-Id: I202f4bf2a81573106763136a0d9bf4cc1bb5d8a2
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Tidy updated test suites

Change-Id: I21f45dcde5e23c18020ad55c065e2812c3de9816
Signed-off-by: Jozef Behran <jbehran@cisco.com>

adding automation around bug 5221

- a few refactors and robot tidy done as well.

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

Rename NIC features

Feature file was remodel in NIC: https://git.opendaylight.org/gerrit/#/c/33996/
This patch address the feature name changed.

Change-Id: I287ac82f787bf19e768f0558d4fd71c70532b767
Signed-off-by: adetalhouet <adetalhouet@inocybe.com>

Add extra check in case we modify a flow

We actually verify old flow has been properly deleted.
Modify existing comment to be python-style.

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

refactoring Bug suite

-  renamed to indicate that more than one bug is validated
-  separated checks in to two test cases which cover one
   bug each

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

Topoprocessing tests - Keywords reworked

Unification TP Inside Inventory test reworked.
This patch should be submitted after this change
https://git.opendaylight.org/gerrit/#/c/33854/
will be merged

Change-Id: I1b4a1d17cb2bcc0a18475df4dc859d60b5088e7a
Signed-off-by: Andrej Záň <andrej.zan@pantheon.sk>
Signed-off-by: Branislav Janošík <branislav.janosik@pantheon.sk>

Improved integration/basic code quality

It is an example we recommend every project to start with [0],
so it should conform to code quality recommendations [1].

[0] https://wiki.opendaylight.org/view/Integration/Test/Step_by_Step_Guide_To_Creating_System_Tests#Move_your_robot_suites_.28test_folders.29_into_the_project_folder
[1] https://wiki.opendaylight.org/view/Integration/Test/Test_Code_Guidelines

Change-Id: Ic66a594eba8113c562d62a99ac41c8019252fdc2
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Added link to bug 5089

The bug 5089 prevents the clustered CRUD suite from working.
Namely it prevents the suite from configuring the device.

Change-Id: I6d6c6d42df01afd6f8cf6e867ac2d9551610caa1
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Remove FastFail from the CRUD suite

The FastFail in the CRUD suite does not save much time and
there are many testcases which don't depend on the previous
ones. The CRUD suite without FastFail thus has potential
to catch more bugs and catch them faster without wasting too
much time. This is true for both, the clustered and the
non-clustered versions of the CRUD suite so the FastFail is
removed from both.

Change-Id: Ie2bccab7a41127f6b0b8e7f4585ff24fa15e267e
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Create common resource for handling CLI tools

Adds common resource for handling CLI tools in bgpcep suite according to
in process creating a keyword library named BGPcliKeywords.robot.
https://trello.com/c/OteFdQXz/263-int-test-create-a-common-resource-for-handling-interactive-command-line-tools

Change-Id: I91edd2c97b906b333130569b10d4ad5618739b5d
Signed-off-by: Kumar Rishabh <shailrishabh@gmail.com>

Ovsdb 3-node data shard csit test

    Put a bridge in config topology via PUT
    Retrieve the same bridge from operational topology via GET
    after clustering
    Create Port and verfiy both in config and operational topology

Change-Id: I04e2b61b7847fd5571f72d1df78a4812f7cebbf9
Signed-off-by: priya.ramasubbu <priya.ramasubbu@hcl.com>

Replace deprecated variables in suites/l2switch (2).

+ MININET -> TOOLS_SYSTEM_IP
+ MININET_USER -> TOOLS_SYSTEM_USER
+ MININET_PASSWORD -> TOOLS_SYSTEM_PASSWORD
+ MININET_PROMPT -> TOOLS_SYSTEM_PROMPT

Change-Id: Ia7da401ae1e61f3c67052cf02a623334b3907408
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/controller (2).

+ MININET -> TOOLS_SYSTEM_IP
+ MININET_USER -> TOOLS_SYSTEM_USER
+ MININET_PASSWORD -> TOOLS_SYSTEM_PASSWORD
+ MININET_PROMPT -> TOOLS_SYSTEM_PROMPT

Change-Id: I7e1399863549a24f824f2c51a59b490e983e4a0a
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/openflowplugin (2).

+ MININET -> TOOLS_SYSTEM_IP
+ MININET_USER -> TOOLS_SYSTEM_USER
+ MININET_PASSWORD -> TOOLS_SYSTEM_PASSWORD
+ MININET_PROMPT -> TOOLS_SYSTEM_PROMPT

Change-Id: I27f370b74550bca54872d070e4e397efad500d41
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Include file names in bgpcep-userfeatures.txt

This will specify order of userfeatures sub-suites.
Also added TODO for cases.robot file name change.

Change-Id: I955597379e72d8d68759a552daa2662742d77564
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Bug 3780: Use rib-instance as type in bgp-peer data

Change-Id: I79058e915ed9f792d425f4db30c2d64c91164ec1
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Added Test Case For OVSDB Southbound Clustering Net-virt.

Change-Id: If965738cf8542335e38d7ae33b1f3e0c7f347fd5
Signed-off-by: manimuthazhagan <manimuthazhagan.r@hcl.com>

Added Scripts for Openflow Validation in Cassandra DB
Modified TsdrUtils to fix some minor comments

Change-Id: If6d20009c45cea16561b4c8fb7be7b1dff90ce4b
Signed-off-by: premnath <prem.annauniv@gmail.com>

LISP: remove one test

For performance reasons we now don't differentiate between add and
update, so add is allowed to overwrite existing keys.

Change-Id: I14895ace2a44e7a80cfe97b3e1dd0030518976e4
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Add log for REST response

This is useful in case we get REST error status code.

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

Replace deprecated variables in suites/ (2).

Suites modified: centinel, nemo, nic, snmp, ttp

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: I3052a2932b023323272710c1915eac211d274168
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/openflowplugin (1).

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: I93689ae884be29fb37173ddc90fea45f40171a34
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/lispflowmapping (1).

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: I9fdeeecb3a7692c1d09413581acc588ccb88c1d1
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/l2switch (1).

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: Ic27e5fbbd1617f07f9b523435c1c186ad6bb5184
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/vtn (1).

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: I74766518606fc6bd38e04e3139c798fd73b48fb0
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/iotdm (1).

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: Ib8232f0fcb19379ae7b99ed35abf4203945f2e4c
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/sfc (1).

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: I156e532a8767684f3bc3c5195408f9fdb5acefbc
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in suites/controller (1).

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: I566022374a993f72fb63195f5c070d26ecf038bf
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Replace deprecated variables in csit/libraries (1).

+ CONTROLLER -> ODL_SYSTEM_IP
+ CONTROLLER1 -> ODL_SYSTEM_2_IP
+ CONTROLLER2 -> ODL_SYSTEM_3_IP

Change-Id: I68cf7f323701973ca4dfb334cfb2f917a8ea92e7
Signed-off-by: Deepak <deepak.mk17@gmail.com>

Adds 503 as a possible error code along with 500.

Change-Id: I769b96db1c5fb04f5b67bd8ee8fb28dc1ce798d2
Refer: https://bugs.opendaylight.org/show_bug.cgi?id=4140
Signed-off-by: Ashish <ashishk.iiit@gmail.com>

Add Netconf CRUD suite to library verify list

Change-Id: I3ece9d53b9e1eae5160151e7aa406804522f2b91
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Added JSON modification request to CRUD suite

This was created as a result of investigation of failure
of restperfclient to send valid requests.

Change-Id: Ia1c2006a6abd37bd2f92ee5d5159033538153523
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Restconf change event notification subscription basic test

- check basic notification functionality
- cover bug_3934

Change-Id: Iff408ef9a3fb26a84dfbd110578737f13b7b427a
Signed-off-by: Radovan Sajben <rsajben@cisco.com>

Add FIXMEs to point out maintenance that becomes due

Added FIXMEs to point out that NetconfViaRestconf.robot and
ConfigViaRestconf.robot modules need merging into one module
and that the code is using deprecated keywords from
RequestsLibrary.

Change-Id: Ib7713b1efd241fdb3759d7c062d3b8d68f5b21bc
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Tidied updated test suites and Robot resources.

Change-Id: I49bfd071f8a784439de867f0c3bd38eeb5405664
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Removed cruft left behind by "refactoring" commit.

Removed outdated documentation cruft left behind by the
7ebd203 "Refactoring MDSAL to use the cars/people model"
commit.

Change-Id: I20f4d2fcb0cae7aebb072c918f34f095a0a3e0c2
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Detect bug 5125

Attempt to send element from a nonexistent namespace into the
configuration will cause NullPointerException. This was
reported as bug 5125. Make sure this bug does not return back
by adding a test against it.

Change-Id: I7754c2a30c5d8940a9c4f554d64111ead1d1a96b
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Fix "Variable not found" in Netconf scaling test

The commit 6acfed01ccc4bf0ce66688bb9d0b51beda5a2265 removed
the ENABLE_GLOBAL_TEST_DEADLINES, causing the Netconf scaling
tests to fail with "Variable ${ENABLE_NETCONF_TEST_TIMEOUT}
not found". Revert that bogus change.

Change-Id: I8b305bbd126aa9fde2294cc5219a502f26067145
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Refactoring MDSAL to use the cars/people model

The original MDSAL suite was using the <modules>
configuration as its test data which caused it to start
failing when the format of this data changed. The problem
is that the suite shall use the cars/people model which was
specifically designed for tasks like this.

Change-Id: I286fea431fca30dffbac6357c77fb87c948c4d9c
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Detect bug 5014

Netconf gained another bad habit. It appears to be down until
the user tries to configure a device, then it wakes up and
starts to be visible.

This bad habit was reported as bug 5014 as it breaks the
Netconf readiness suite (and probably other production
workflows that depend on the ability to determine when
Netconf becomes available). This adds a test case that
detects this bug by trying to configure a dummy device when
Netconf appears to be down after the wait is completed.
If Netconf mysteriously raises from the dead, the test case
fails, reporting that bug 5014 was found.

Change-Id: I29e8be136aaddaa13169b533805415729fcc4959
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Modify CSIT tests for VTN Coordinator of audit feature

    * Improve test scenario of audit cases and modify 040__vtn_vbrif_audit.robot
      files of both OF10 and OF13
    * Modify create vbr function in VtnCoKeyword library file
    * Modify Mininet ping function by reducing ping interval and modified wait time in Mininet ping test cases.

Change-Id: Ic0eb70df1e3251d6d11720f7db50bed60fcd1f9d
Signed-off-by: Karthik.S <siva-karthik@hcl.com>

Fix for VTN Manager CSIT test failures in VTN jenkins job

    * Reduce ping cycle from 10 to 1 in Mininet Ping Should Succeed and Mininet Ping Should Not Succeed function in VtnMaKeyword_li.robot file
    * Add Wait Until Keyword Succeeds(20s) in Verify flow test cases to get flowentries response from mininet
    * Add Wait Until Keyword Succeeds(20s) for Mininet Ping Should Succeed test cases.

Change-Id: Id99dcb8b7a5d1d99b664ddb93955ac4ce2f6d601
Signed-off-by: Karthik.S <siva-karthik@hcl.com>

Add system test for the northbound API in NEMO project.

The main test includes:
- Add predefined role of users, which inludes tenant and admin
- Add predefined node type and connection type
- register a user whose role is tenant (create a vn space)
- In this vn space, issue an intent, that is, create two nodes and one connection
between them.

TODO list:
- Use customized switch to consruct a physical topology
- Test the intent result on the physical network, such as, ping

Change-Id: I185447428fe6c5e78176a210687e6f1f48008cb3
Signed-off-by: Zhang Yali <zhangyali369@huawei.com>

remove feature installation from robot setup

the paradigm in OpenDaylight CSIT is to normally
have the features installed when the controller
is deployed.

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

Decouple netconf-ready suite and netconf connector.

Netconf connector is going out of favor with Beryllium
implementing the "topology way" of Netconf configuration [0]
which offers vastly improved performance and easier setup of
the devices. Additionally the Netconf connector is optional,
making Netconf invisible for the suite if not installed.
Thus change the suite to query the netconf topology for a
list of configured devices.

This adds a configuration variable USE_NETCONF_CONNECTOR that
allows the netconf-ready suite to use a weaker condition for
the "Netconf is ready" check that does not depend on netconf
connector. The default of the variable is True, which makes
the netconf-ready suite to depend on the netconf connector
and it shall be set to False for test jobs which don't use
the netconf connector.

[0] https://wiki.opendaylight.org/view/OpenDaylight_Controller:Config:Examples:Netconf#Spawning_netconf_connectors_via_topology_configuration

Change-Id: Iac8bd7502014d0f897b3e92b351c2f8f61da9199
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Tidied new and updated test suites

Change-Id: I11c98cfa0cec00a8e402bd1439a64fdcf544ec25
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Delete outdated comments

Change-Id: Ibc3d6b45f5b362ae4f697c78c0c853e5396416ad
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Fix "netconf_is_ready" not always  reflecting netconf status

If the "Check_Whether_Netconf_Is_Up_And_Running" test case
fails or is skipped, the "netconf_is_ready" is left set to
False even when "Wait_For_Netconf" concludes that Netconf IS
ready. This will break further test cases if they rely on
"netconf_is_ready" to determine whether the Netconf was seen
to be up and running or not. The fix simply tells the
"Wait_For_Netconf" test case to update the variable.

Change-Id: I05e18cd491dc414b036cb68504dcf3b39e0c2490
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Added SXP filtering tests

Change-Id: Id627fc8b9c8480f4cf8563d88e7870f7ebd129cb
Signed-off-by: Martin Mihálek <mamihale@cisco.com>

Revert "removing bgp tests from verify-library job"

This reverts commit 899da71719ded7336d97d985cbce03fcf38e53d4.

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

removing bgp tests from verify-library job

bgp features were having trouble and the controller
was not coming UP in verify job

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

Fixed spurious MDSAL commit tests failures.

The fixed "commit" operation always returns OK when the
candidate configuration is exactly the same as the running
configuration. Which makes sense as the commit's task is to
turn the running configuration to be exactly the same as the
(currently present) active configuration. However the tests
were expecting the commit to fail in this situation and thus
are now producing spurious failures.

Change-Id: I2f25e3d53a9519a5987e386e7ebe4d7161c0b767
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Do not use FailFast in MDSAL northbound test suite

The MDSAL northbound tests each take only a few tenths of
second to execute no matter whether passing or failing
(unless things in ODL get awry) so there is no benefit from
FailFast. Actually, the FailFast suite tends to mask bugs
as the tests after the first failure won't run at all.

So remove the FailFast support from the MDSAL northbound
test suite.

Change-Id: Ieae3cb87eee678b6e64c18c91cdb93849d4a2915
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Readding deleted line for connecting to spec hosts

Change-Id: I350a64e0435e37e1580fcf5dd1df5a3e554e0387
Signed-off-by: mahesh.manivasagam <mahesh_manivasagam@dell.com>

Added keyword to get the rloc object pointed to by hop_index.

Change-Id: Ieb0211d98e198ae94cddc5cad479b07454077769
Signed-off-by: Ashish <ashishk.iiit@gmail.com>

Test suite to reproduce bug 4794 in CI

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

using yang_json headers because data given to keyword is json obj

the new RequestsLibrary has caused the current implementation to
fail, so need to pass application/yang.data+json

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

Done the following changes :
TsdrUtils.robot - Addressed the review comments
All HSQLDB Suites - Addressed the review comments
All HBase Suites - Modified Query format based on new Design
Added HSQLDB Testplan
Changed HBase scriptplan as per new environmental variables
Removed the Flowmeter and Groupmeter status (refer bugzilla#3466)
Modified keywords which were used with deprecated format in Dictionary and Request Library

Change-Id: I3ea717671d84a66df3903d66ba24de4212a600b7
Signed-off-by: premnath <prem.annauniv@gmail.com>

Remove RequestsLibrary.py

Remove RequestsLibrary.py as all the tests use the public
RequestsLibrary
https://trello.com/c/LpxuYiEc/271-remove-requestslibrary-py-file

Change-Id: I8710367ad14d3e88c419e70f3bb2bd04af06f973
Signed-off-by: Kumar <shailrishabh@gmail.com>

Refactor OF cluster test

- Move entity owner keyword to OpenFlowCluster library (plugin specific)
- Add entity-owner shard check
- Increase timers so we know when there is delay or real failure
- Continue test when entity owner does not update list of candidates
- Add support for He and Li plugins

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

removing explicit header in RequestsLibrary.Put Request

the new library version deals with json differently than
before and broke this packetcable system test.  This will
make it work again

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

upstream ovs version to validate is 2.0.0 not 2.3.1

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

Tidied new and updated test suites

Change-Id: I9ba32a6262d3cab40c23bc5902bb34f02fb4aaf3
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Rename "first_case_ok" to "netconf_is_ready"

The new name gives much better meaning to the variable,
improving code readability quite a lot. Additionally, I want
to add some more test cases which shall not run if netconf
is detected to be ready. The code of these new test cases
is going to be a lot more readable if the variable has this
new name.

Change-Id: I8254443aad3d0ea9e039fa1c4051bf56ce5053a3
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Upgrade pep8 version for tox to 1.7.0

That is the version currently installed if no version override is used,
and since Change 32530 all upgrade blockers seem to be fixed.

The pep8 is still fixed to a constant version (1.7.0)
so that we do not get broken when newer version is released.

Change-Id: Ibb75862cf5bde142b936b1cbddc3d8ddec636fcc
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Refactor OVSDB Cluster test

- Add entity-owner shard check.
- Increase shard timers so we know when there is delay or real failure.
- Continue on entity-owner failure to update candidate list.
- Small fixes to suites variables.

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

change tempest test timeout to 15m

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

Fix pep8 1.7.0 W503 in play.py

"line break before binary operator"

Change-Id: Ifdb89e06f0c0b4a146848cad6ec81f483abf33ee
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Fix REST headers in Cluster test

Simplify headers usage
When PUT use ${HEADERS_YANG_JSON}

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

removing snapshots in addition to journals

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

Ovsdb 3node clustering csit corner cases.

    Add Test Cases for Southbound Clustering.
    Create bridge in switch and verify in all the cluster instances.

Change-Id: Iabcd6908bd7830409b417d5beee27156ae6c8af2
Signed-off-by: priya.ramasubbu <priya.ramasubbu@hcl.com>

LISP: Add Service Path Datatype test

Change-Id: I3a37ffda613594cccd9c683fbd254e261c52d8ae
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Clustered Netconf CRUD

A suite that runs a Netconf device through Create, Update
and Delete requests, checking that the changes are visible
on all three nodes in the cluster.

Change-Id: Ia464b5c7ba2b66fce8b582a82af009d1e76d82fe
Signed-off-by: Jozef Behran <jbehran@cisco.com>

New way of device configuration/deconfiguration

Added a new device type called "configured-via-topology"
which uses the new "topology way" of configuring the device.

Change-Id: I9410bf8524d687caf19c43be7eec8bb70dfda2d8
Signed-off-by: Jozef Behran <jbehran@cisco.com>

Created operational data access for NVR sessions

The NVR sessions now use 2 requests library sessions, one
for config data and the other one for operational data.
This was necessary to allow the clustered setup to check
the operational data on all three nodes of the cluster.

Change-Id: I405c6ee86b6d4721963e69f1988b433533a40678
Signed-off-by: Jozef Behran <jbehran@cisco.com>