Log OpenFlow max switch scale result

Was logging to console only, so not visible in primary log.

Change-Id: I9c95f2e95db3bc89431f490ce1af1d0f3ae55e23
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

fix Verify VMs Received DHCP Lease func - if dhcp ip is empty change to
None

Change-Id: Ic0720bff4ebb9607e9a95a11fc4eeec9acb4158a
Signed-off-by: mor.zvi <mor.zvi@hpe.com>

Add IPv6 tunnel test to Genius CSIT

This test creates a vxlan tunnel between two endpoints with IPv6 addresses and checks
that the network topology configuration is updated accordingly.

I did a configuration check and not an full check with OVS because the OVS version doesn't
support IPv6 vxlan tunnels.

This tests change https://git.opendaylight.org/gerrit/#/c/49542/

Change-Id: I6d40d0e5b7a73ac934261e45a4239c156fb5c9b9
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>

Skip macspoofing tempest test if security group mode is 'transparent'

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

Add Rpc Provider Test Suites

Change-Id: Ide061771ca1ef099da58916293221001e0580202
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Add SXP performance measuring suite

Added Connectivity measuring tests

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

addind additional Tempest scenario tests
and adding the option to skip run of tests in mitaka,
like hotplig nic test that fails in mitaka due to a bug that was fixed only in Newton.

Change-Id: I7fc99d2a178fd318c3ec87dbc3d6ef9c1f499465
Signed-off-by: Mor Zvi <mor.zvi@hpe.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Remove netvirt Tempest macspoofing test

This test was recently modified causing inconsistent
results between various ACL runs.

Removing it and it will be re-added as part of a bunch
of Tempest SG tests after it is analyzed:
https://git.opendaylight.org/gerrit/#/c/48654/

Change-Id: Ie19b010a4391d29ffbef94cf2d44f52e090af05d
Signed-off-by: Alon Kochba <alonko@hpe.com>

Add Persistency Test cases:

  - Basic Environment Setup Tests
  - Add SFC Elements and restart cluster

Change-Id: I75d219b95200c6fb9e097538cbbee2983701888c
Signed-off-by: Celestino Martinez Lopez <celestino.martinez.lopez@ericsson.com>

Fix typo of ) instead of } in robot variable

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

Rework Check_Status_Code to check the code with explicitly specified codes

Change-Id: I49a7f5970464f149b8acc9b606bf9a73b3136224
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Add global rpc ha scenarion for frozen/stopped/paused node

Change-Id: I91208a259cc24fea6ed058b7f89e097244c5f571
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Add keywords for (un)freezing the karaf process

Change-Id: Ibc8be69ee00a3bdeb8bc71caedcb3da47d0eca6a
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Add new suite for adding cars during shard leader change.

Change-Id: I4840c2bc0a77163bc9c67804db7a907effe0424f
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Tweak cbench execution and availability when it's running on localhost

  seems most (guessing) all uses of this script were using cbench from a remote system
  and logging in to that system via SSH to run the tool. The changes here are fixing
  some problems I found when trying to run cbench on the same system running robot.
  in other words, running from localhost

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

Fix the problem when no real check of new leader was done

Change-Id: I396c3b2be6a0f02c86fd96a9532cc78bc96c3c5b
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Fix cluster test teardowns, allow model dump to fail

Fixes test teardowns on several netvirt clustering test failures, and
allows model dump to fail in teardowns after bringing up ODL.

Change-Id: Icd3597c30f8d7c30bcae128d4658880ddd4af156
Signed-off-by: Alon Kochba <alonko@hpe.com>

Add ietf-acls to netvirt module dump

Change-Id: Idc82d435d1f8b0d6ae0b404a97f29254046c095e
Signed-off-by: Alon Kochba <alonko@hpe.com>

Change Entity Owner suite to new cluster behavior

New cluster behavior: only entities the switch connects to show
as candidates in the entity owner API.

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

Bug 7868: Increase timeout for one request in car perf

There has been a performance regression,
but in order to see how bad it is, we need to wait longer.

Also:
+ Added a check to fail Add_Card when tool reports an error.
+ Test teardown to report known bugs.

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

Bug 7785: Remove flows with invalid action

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

Use ostestr instead of run_tempest.sh

run_tempest.sh was deprecated and removed,
instead use 'ostestr' to run tests.

in addition remove unused code in DevstackUtils

Change-Id: I790dafd2821437e4a97b851dda659c51733541a4
Signed-off-by: Alon Kochba <alonko@hpe.com>

distchanges: Add protections around patches under review

distchanges breaks for patches that are under review and not merged yet. This
is because the gerrit queries assume they are merged. When it breaks
the distchanges output stops abruptly and doesn't continue
processing all the projects. This patch prints the log and then keeps going.
There is another patch under review to get the gerrits for the unmerged patches,
so this patch here just keeps the logs going.

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

Add distchanges lookup for aggregated merges

Some distributions end up with a aggregated merge
of a number of gerrits in a patch stack. The data included in the git.properties
includes all the gerrit's commit messages. This gerrit adds a
search using the commit message of the patch at the top of the stack. [1]
is an example of one of these types of gerrits.

[1] https://git.opendaylight.org/gerrit/#/c/51304/

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

Comment out BFD and ELAN suits due to inconsistent issue.

Change-Id: I13d2a6a0a899078bd15e977d0840e8ef814d5dff
Signed-off-by: Suvitha.balu <suvitha.balu@tcs.com>

Bug 7661: Add WUKS to netconf outages suite

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

Bug 3712. SFC: SFP validation of SF types at creation time

Set of five new tests for SFP validation. This validation
verifies that SF types for SFs defined in SFP hops are (when
present), consistent with the SF types for the SFs defined in
the referenced SF chain

Change-Id: I66a5f3bc3fa2098def8397bdd256094c5c82a818
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

Add interfaces child info to model dumps

Change-Id: I60983dde4c6646e5a113107489be258451d9b319
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>

Due to Two patch merge, the Delete SecurityGroup added twice in OpenStackOperation.robot.So deleting one.

Change-Id: Ib57391d74d8fe32401d3c6251698d0935c63a9c2
Signed-off-by: suvitha.balu <suvitha.balu@tcs.com>

Add isolation scenarios for clustered car/people tests

Change-Id: Ie0e282df9e6a01aa8e0e3ef116f016d6378ac275
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Basic Elanservice Test case

Change-Id: If9fee3d7150ae830fe345457cab22ec66a53f646
Signed-off-by: suvitha.balu <suvitha.balu@tcs.com>

Bug 7791: Add test cases to Netconf Mdsal suite

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

Moving DHCP check from suite setup to test case

so that all test cases get exectuted.
Also adding 2 more testcases.

Change-Id: If0773039c3e3fbf74bb70969c8b9bfc1431ceb73
Signed-off-by: rahul.agarwal <rahul.agarwal3@tcs.com>

Add MdsalLowlevel.robot library

Change-Id: I7b4ac834f05242ce7717aa203d377e44c64b0331
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Reorder netvirt suites, start with connectivity

Move connectivity suite to beginning, as we know
exactly what is done. Tempest running first uses
various API calls that may hide issues tested by
connectivity suite, such as auto tunnels.

Change-Id: Ic56db7beb83ac5ad24a5997404bd46b599e88118
Signed-off-by: Alon Kochba <alonko@hpe.com>

Add missing teardown debugs to vpnservice CSIT suite

Change-Id: Ia5835d26b421f16cf2ef3a222de51c68719ce406
Signed-off-by: Alon Kochba <alonko@hpe.com>

Fix OpenFlow library KW

Clean up ${base_mac} variable.
Also add REST content log.

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

Add remote-url command line argument

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

Adding comparison script to list patches added to one distro compared to another

Change-Id: Iae9028304bf3407a0075b026ae22b3fd21997c76
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

configure allowed address pairs on port create

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

Fix for requesting the eos on isolated node

Getting eos details on isolated node returned 404 and the tc failed.
This caused not even calling the tested rpc.
https://logs.opendaylight.org/releng/jenkins092/controller-csit-3node-clustering-all-carbon/219/archives/log.html.gz#s1-s9
On isolated memner the Get From Uri failure will be ignored

Change-Id: I8f2a6de3fb87096959fef7fc5aa01475702a8d41
Signed-off-by: Peter Gubka <pgubka@cisco.com>

don't use known hosts file for netvirt CSIT VMs

Change-Id: Ic5891aec0b6b38a2cfb2ab1fbbe7a0b8f3b35f18
Signed-off-by: Alon Kochba <alonko@hpe.com>

Remove setting of TRACE to entire netvirt in vpnservice

Change-Id: I9b4f34fe916e17697f25b5f38512b031601607aa
Signed-off-by: Alon Kochba <alonko@hpe.com>

GENIUS - CSIT - : Replaced RequestLibrary session output from XML to JSON to have uniformity

Change-Id: I0234ae61c16c81808ff623f4a807b44b23a6ec48
Signed-off-by: <nazeerahamed.hunshyal@tcs.com>

Fix DHCP checks in netvirt CSIT, fail and log

Change-Id: I507d035053f982c4786d24df303d4764b6e489d1
Signed-off-by: Mor Zvi <mor.zvi@hpe.com>

print date in utc format rather than epoch

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

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

Add protections around changes pretty_print

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

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

Add suite for global singleton rpc testing

Change-Id: I7f3d198e0806b12c21fb75261d6da53025417869
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Two more test cases for Tunnel monitoring added

Change-Id: I8f4dd4c415bacb443e42a758264f1034b90bcb0b
Signed-off-by: rahul.agarwal <rahul.agarwal3@tcs.com>

add federation rabbit

Change-Id: I237093bf1a2bdbeb31600abf56ca3e4645857d91
Signed-off-by: Sandra Biton <sandra.biton@hpe.com>

Update the scriptplan name to match new job names.

The job names for legacy netvirt were changed. However, the file names of
corresponding scriptlan were not updated correctly.

Change-Id: Ic62191732b9bb321725b01066a0f53c45e23e3be
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Add .editorconfig file

Different file formats have separate sections.
Add rules as you see gaps.

Link to style-guide followed, present before each section

Change-Id: I4101e6bcd27f16bc554d70ade177089f9572db6d
Signed-off-by: Umesh Singla <umeshksingla@gmail.com>

distchanges fix for messages with () in them

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

Remove ODL_STREAM variable from libraries/ChangeCounter.robot

Change-Id: Ibeacf103b4c6bb68c04d78e548647544c78b8256
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Update snat model list to match latest code usage

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

Remove ODL_STREAM variable usage in if statement from 010_SM_add_upd_del_flows.robot

Change-Id: Iccc2480d1e6ea89584e04df2b4677a7004eba4aa
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Remove ODL_STREAM usage from notifications/notifications_basic.robot

Change-Id: I26abffe70d021e02bed867e2a013ff10cad65747
Signed-off-by: Peter Gubka <pgubka@cisco.com>

fix argument processing for projects

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

SFC_Basic with Logical SFF

Change-Id: Ia5cb79f8d1ce1676b1ac5388ff26261d99b7d6bb
Signed-off-by: Ursicio Martin <Ursicio.Javier.Martin@ericsson.com>

Skip working on gerrits if the includes list is empty

this could happen when the firt gerrit found is not "merged" as would
be the case when this tool is triggered with a patch test gate job.

this will actually stop parsing gerrits for the project that has an
unmerged change in the distro.

this is a quick fix to get the tool working again. a better fix needs
to come next where it can handle unmerged gerrits and then continue
to print all the merged patches that still made it in to that relevant
project

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

Fix up model dumps collection to reflect latest datastores

Change-Id: Id0340224db5020538054f85bc9212be8399c46eb
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Bug fix on the Keywords, SSHLibrary write command takes the older values.

Change-Id: If4e064b0b1cf87f00faacd0e58c49fb9645aed20
Signed-off-by: abbas <abbas.pareedkunju@tcs.com>

SXP performance test filter change

Updated filter prefix to adapt to change https://git.opendaylight.org/gerrit/#/c/49270/
as old prefix was out of scope.

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

Fix integration tests for iotdm project

Change-Id: Id6530d66a2066802b219d1608af3d7179ff6846d
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>

Add tests for symmetric paths using bidirectional flag in SF type

Change-Id: I33d39ce4fec52c5242e03751ab2cfea6dfff5acc
Signed-off-by: eabasan <santiago.abascal@ericsson.com>

Adding netvirt to list of netvirt projects

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

Enlarge nc timeout used for netvirt SNAT test

Change-Id: I94724ef655a4974ca4887ef5369df94622333d5c
Signed-off-by: Alon Kochba <alonko@hpe.com>

Remove ODL_STREAM variable from Netvirt_Vpnservice/BFD_Monitoring.robot

Change-Id: Ie3b3161cb91d36fcdef3ebcec406ecc0bbbd8205
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Remove lithium testplans for bgpcep and tsdr

Change-Id: Ic6aa89334696dd8ddd6ce8727af49497a9748736
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Create boron specific bgpcep-bgpclustering-ha-boron.txt tesplan file

Netconf feature is going to be removed from bootfeatures in carbon in
https://git.opendaylight.org/gerrit/#/c/50711/ and netconf readyness suite
is not needed in carbon because of that. In boron it remains.

Change-Id: I497f9fa254146e7dd6e69e7fddeb948573de9d8d
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Remove ODL_STREAM variable from usage in SxpLib.robot

Change-Id: I35d9794ca3db5a9a0c2ff3cbfd1af1a324b05159
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Remove ODL_STREAM from VtnCoKeywords.robot

Contains also an additional kw in CompareStream to run kw for a certain stream

Change-Id: I92f70902cdb706c88a80e3e5d3105aa5e22a3239
Signed-off-by: Peter Gubka <pgubka@cisco.com>

adding ability to download a distro from url and show patches from that distro

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

Remove ODL_STREAM variable from usage in libraries/TopoprocessingKeywords.robot

Change-Id: Ie202479ec81edb765e37bb4fa8a82ec7b8c78620
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Add SNAT test to netvirt CSIT

Change-Id: I8caba0858a9cf85c0cda8e22cf22ea8c998ca500
Signed-off-by: Alon Kochba <alonko@hpe.com>
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Update tcpmd5 suite with installation of netconf connector if needed

Change-Id: Ie8629d51d55dd381d99d9be3f663bd6e7386778c
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Remove unused resource file ConfigViaRestconf

Change-Id: Ib2a06bfccce149eb0b20aaab246ea0b4af4fada7
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Tunnel monitoring test cases

Change-Id: I7135fd9115c83debf7b8b8880fae8935340534e2
Signed-off-by: rahul.agarwal <rahul.agarwal3@tcs.com>

Fix typo in OpenFlow library

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

increasing loop count for karaf ssh as a workaround for ssh connection rejections

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

Increase Karaf SSH timeout

After few tests, this seems to stabilize the failure:

SSHException: No existing session

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

adding a few tweaks and fixes/workarounds

- looks like the ssh url for gerrit is not working from the releng VMs
  so trying with https now

- the CommandFailed class was broken and it appears to be that there
  was a missing docstring for that class, so self.__doc__ was None
  and doing a return with something None + something else was crashing.
  So, I added a basic docstring there. I know this is code similar to
  another project (e.g. git-review), so I'm not so confident it's
  the right thing, but its working for me. It was noticed because the
  ssh url to gerrit was being rejected so that class was getting
  excercised

- I did a few modifications based on what my IDE told me was good
  python

- I removed one and added one print output to make it more clear
  what's going on (to me, at least)

- fixed one variable naming typo

- defining the ssh URL in a single location

- verbose cmdline arg was overriding VERBOSE to 0 even if it was modified inline

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

Add stop test validation

Change-Id: Ief8681e0e8aef34abecc52b25e23788aa4eb7ebe
Signed-off-by: Peter Gubka <pgubka@cisco.com>

SXP filtering policy update

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

First pass at tool to list gerrits included in a karaf distribution

The usage is shown below. The tool will use the following defaults:
Using these options: branch: master, limit: 10, qlimit: 50
remote_url: ssh://git.opendaylight.org:29418
distro_path: /tmp/distribution-karaf
projects: ['genius', 'mdsal', 'netvirt', 'neutron', 'openflowjava', 'openflowplugin', 'ovsdb', 'yangtools']

This can only be used on master right now because it relies on having a git.properties
file in the jar's.

usage: changes.py [-h] [-b BRANCH] [-d DISTRO_PATH] [-l LIMIT] [-p PROJECTS]
                  [-q QLIMIT] [-r REMOTE_URL] [-v] [--license] [-V]

optional arguments:
  -h, --help            show this help message and exit
  -b BRANCH, --branch BRANCH
                        git branch for patch under test
  -d DISTRO_PATH, --distro-path DISTRO_PATH
                        path to the expanded distribution, i.e. /tmp
                        /distribution-karaf
  -l LIMIT, --limit LIMIT
                        number of gerrits to return
  -p PROJECTS, --projects PROJECTS
                        list of projects to include in output
  -q QLIMIT, --query-limit QLIMIT
                        number of gerrits to search
  -r REMOTE_URL, --remote REMOTE_URL
                        git remote url to use for gerrit
  -v, --verbose         Output more information about what's going on
  --license             Print the license and exit
  -V, --version         show program's version number and exit

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

Add karaf logging to controller-clustering.txt suites

Change-Id: I764b451dca4a49feac62ec3e290be47ace81b625
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Add new suite to test bgp md5 using exabgp

Change-Id: Ib2b67354adbe67427c2d88a781e4290f540505ee
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Remove ODL_VERSION variable, it is not needed

Change-Id: Ib15d49dfa5bea7bb5eef2bdf5477689f963c9749
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Fix up CSIT such that extra-route is cleared correctly.

Extra routes should be cleared from a router before an
attempt is made to remove subnets which are the
targets of extra-routes from router.

Also it is better to have Should Contain as the last
line of test scripts as any cleanup after a failure of
Should Contain never gets triggered.

Change-Id: I2987b60afab3d78e8ef3cb84118e72ea44b087b0
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Add a keyword to WaitForFailure Resource

+ Verify_Keyword_Never_Passes_Within_Timeout
+ Documentation updated.

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

Fix for NonExistentRouter delete TC for Newton

Change-Id: Ib645cee7c23b811e6affefc5e980d3be95d3b829
Signed-off-by: suvitha.balu <suvitha.balu@tcs.com>

Update version.robot to reflect netconf connector usage

Change-Id: I1b206601879b067026052fa0625e23d9723f39fb
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Stabilize OF Reconciliation test

- Remove ssh karaf from the suite setup. This is to avoid the ssh karaf
failure we see lately in CI.

- Kill controller if it does not stop after 2 minutes. This is required when
test runs in the -all- mode (many features loaded).

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

System test for Yangtools

+ Yang file preparation Resource introduced.
+ Suite binding-parent refactored to use that Resource.
+ The robot suite suite.
+ The testplan.
+ Artifact downloading now supports multipatch job.
*- Only as a suite keyword, probably should be moved to NexusKeywords.
+ RemoteBash.robot Resource introduced to host operations common for multiple tools.
+ Changes to existing libraries:
*+ Added TODOs to de-duplicate with respect to RemoteBash.
*+ NexusKeywords:
**+ NexusKeywords__Get_Items_To_Look_At knows yang-system-test.
**+ Deploy_From_Url to skip URL construction.
**+ Install_And_Start_Java_Artifact
**+ ${explicit_url}=${EMPTY} option added where needed
*+ TemplatedRequests:
**+ ${accept} argument no longer mandatory for Get_From_Uri
*+ New Copyright year.

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

New OF reconciliation suite + other fixes

- New reconciliation suite for groups and flows.
- Consolidate OpenFlow KWs in FlowLib.robot
- Consolidate Mininet KWs in MininetKeywords.robot
- Adjust existing test to the above changes.
- Fix Delete flow check in cluster test.
- Use org.apache.karaf vs karaf to find karaf instances.
(this allows to run pybot command in controller VM)

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

VPNservice Datapath TC and added additional validation

Change-Id: Ibd2c344ec867536b861e42d111af937422852ce6
Signed-off-by: suvitha.balu <suvitha.balu@tcs.com>

Eliminate getting model dumps when 2 ODL nodes are down

This will reduce the delay in CSIT

Change-Id: Ifb944b8cf8cfad7a0b79a223d19e636ed646a0d6
Signed-off-by: gvrangan <venkatrangang@hcl.com>

Expand documentation of SetupUtils

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

Rework of Restperfclient__Invoke_With_Timeout kw to avoid broken ssh session

In the jobs netconf-csit-1node-periodic-scale-only-<strem> it took too long
to handle 65k requests via netconf connector and ssh session became broken.
To avoid this execute command kw was replced with write and read until kws.

Minor changes to support debugging in libraries/SSHKeywords.robot and restperfclient/mdsal.robot

Change-Id: I080edb518acfe2e955fbe249e6e7bb33d8099eda
Signed-off-by: Peter Gubka <pgubka@cisco.com>

Update bgp ingest manypeer changecount suite not to use netconf connector in carbon

Change-Id: I27aa3f749e4208bb531864b846f18f00ed732b0a
Signed-off-by: Peter Gubka <pgubka@cisco.com>