Fix typos in OpenDaylightTesting

Change-Id: Ia70c1db358b727a910a38b679de0e3c6febdb0cd
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 43a98904445a7151b586f996ae5d0e858fa24b98)

Mend create_image() function

Change-Id: I91681736817f881ca53e5124328bd4399aa54740
Signed-off-by: Michael Polenchuk <mpolenchuk@mirantis.com>

Merge changes from topics 'dont_open_twice', 'add_logs', 'manage_return_codes'

* changes:
  Avoid opening tmp files twice
  Add logs (info)
  Improve errors and status codes management

Merge "bugfix: push_result_to_db exception show"

Avoid opening tmp files twice

It avoids opening stdout.txt twice and removes it at the end.

Change-Id: I7e006b52f49dd32543676259e0a1be2473670487
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Bug fix - Better way to grab file from gerrit"

Add logs (info)

It adds logs (info) when status is ok and fixes an exception printing
too. It removes one useless log (debug).

Change-Id: Ia59c3985ae749b80d0e0c07f331220bd33cd8007
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

bugfix: push_result_to_db exception show

When pod is not exist in DB, exception will be shown as
'Error pushing results into Database '<type 'exceptions.TypeError'>'',
very unclear, make it very difficult to fix the problem.
Modify it to show as the testapi returned
'HTTPError: HTTP 404: Not Found (Could not find pod [zte-no])'

JIRA: FUNCTEST-424

Change-Id: I55fc4639d745b84f6d32a148bd13793d236e2a09
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Improve errors and status codes management

Run returns the number of errors which is irrelevant to catch as
they are published to DB. Useless generated files are desactivated too.

Change-Id: I2baf6d5242c5f4095302a47ff90272fb95103457
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Catch exception if output.xml failed

It protects against a direct call to push_to_db without a previous run
even if it can not happen in the current implementation.

Change-Id: Ifa00aed60912fa8b96b80d581067548ac6727fb5
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Add reporting to DB in OpenDaylightTesting"

Merge "Protect again changes of unix permissions"

Merge "refactor vping"

refactor vping

There are lots of common processes in vPing_userdata.py
and vPing_ssh.py, abstract and refactor them.

JIRA: FUNCTEST-414

Change-Id: I4cd2c635318c063319d2a3c31f9a1e512eeea6c8
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Protect again changes of unix permissions

It adds chmods in Dockerfile to ensure python and bash scripts can be
executed.

Change-Id: I9d75e87fcb1eb2e6fa37039fa5a070ef21cf0efb
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Chmod 755 all py files which can be executed

All files where __main__ is checked can now be executed by everyone.
It results from [1] like the next commit [2].

[1] find . -name "*.py" |xargs grep __main__ |cut -d\: -f 1 |xargs chmod 755
[2] https://gerrit.opnfv.org/gerrit/#/c/16737/

Change-Id: I75395892e9e6afbf8ac9c995258496be557aad5f
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Add reporting to DB in OpenDaylightTesting

It now relies on Robotframework API instead of parsing output.xml.
The type of critical and elapsedime fields are now bool and int
respectively. start_date and stop_date has been fixed too.

JIRA: FUNCTEST-367

Change-Id: I59f3ad2109345395ccf01a714301a14f9323f088
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Add exec right

Change-Id: I9c3fd786574622977870b12d38b92f8bed8a929c
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Fix print when push_results_to_db fails"

Fix print when push_results_to_db fails

Change-Id: I82741a8727fc17bc6648e1b1e92511b3efd93b8a
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Modify data sent to testAPI"

Bug fix - Better way to grab file from gerrit

We changed the way to download a single file from gerrit
We point now to the original file as bugs in tacker were fixed

Change-Id: I566b2f95ac1ab6032f22180bb42d0fa49482058c
Signed-off-by: Manuel Buil <manuel.buil@ericsson.com>

Pull down default instance flavor

Set default instance flavor to m1.tiny in order to keep
hardware resources requirements at the minimum.

Change-Id: If070e25389e1d7978c34dec8621414c8938a262f
Signed-off-by: Michael Polenchuk <mpolenchuk@mirantis.com>

Merge "functest compass usage document"

Merge "Remove useless odlreport2db args"

Merge "Colorado Release note"

Modify data sent to testAPI

It mainly removes prefixes, '#' and '@' and changes details to tests.
It deletes useless comments too.

Change-Id: I7807a7429a90e7c7c15f8662008a5a768f96be98
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Colorado Release note

FUNCTEST-420

Change-Id: I3fab0637fedc04bcb61a76e61f91da5df5c732c1
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>

fix Pillow version 3.3.0

The latest docker build for functest container has failed while installing
Pillow-3.3.1 [1], where the previous build had succeeded [2].

Let's pin the version of Pillow to 3.3.0 .

[1] https://build.opnfv.org/ci/view/functest/job/functest-docker-build-push-master/589
[2] https://build.opnfv.org/ci/view/functest/job/functest-docker-build-push-master/588

Change-Id: I18e46340aa8f43ec56aef5e3d208cac59af10a9c
Signed-off-by: Ryota MIBU <r-mibu@cq.jp.nec.com>

functest compass usage document

JIRA: FUNCTEST-333

add some information in configuration file

Change-Id: I6ec91d15c859b9739904d602aef3a0927f6561b6
Signed-off-by: MatthewLi <matthew.lijun@huawei.com>

Merge "complement Parser related userguide"

complement Parser related userguide

JIRA: FUNCTEST-423

Change-Id: Ib52e04a873a2a924102832fe5461ebab2296464d
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Merge "Config guide workflow simplification"

Config guide workflow simplification

Workflow simplification for config guide. Several good improvement
ideas taken into account. Work is not totally complete yet and may
need separate handling.
Added link to chapter "Apex installer tips" which was accidentally
dropped off.
Moved commands specific to apex to appropriate section.
Took away the Rally as an example of the test case execution behind
a proxy. Rally does not nesessarily need internet connectivity.
I finally removed the dreaded "Whiteschpace" at EOL in several
places.
Adding minor tweaks and finetuning.
Corrected the phrasing of docker installation.

JIRA: FUNCTEST-347

Change-Id: I35c094d1d89538a2991a384bbc70b48b0d18e75d
Signed-off-by: Juha Haapavirta <juha.haapavirta@nokia.com>

Remove useless odlreport2db args

It removes useless odlreport2db args as functest_utils parses env vars
too and erases this previous values.
All mandatory env vars are still checked in exec_test.sh.

Change-Id: I3dd299dcad1758b994cf9d53ef65a5b83dc46b97
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Add report flag for moon"

Merge "resume Parser test run on fuel onos"

Add report flag for moon

JIRA: FUNCTEST-210

Change-Id: I97be5542ae4ae87f1b8d62394ff66068e77f4082
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>

Merge "Minor fixes for Doctor Project"

Merge "Convert to Unix format"

Minor fixes for Doctor Project

Note: The correct links to Doctor/Promise user guides
will be added once we cut colorado branch

Change-Id: Ib3a75027e7e8a94f8cb31ab827208ec391f2e2a8
Signed-off-by: Bertrand Souville <souville@docomolab-euro.com>

delete extra slash while execute cases test in exec_test.sh

There are two slashes appeared in the path when we execute case test:
/home/opnfv/functest/conf//openstack.creds
/home/opnfv/repos/functest//testcases/OpenStack/vPing/vPing_userdata.py

JIRA: FUNCTEST-421

Change-Id: I4fa8a11499e301dd84d1e1de7c72189b8d84459b
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

resume Parser test run on fuel onos

Parser tests was removed by onos people in
change https://gerrit.opnfv.org/gerrit/#/c/18821/.
After we talk about this, we found it was a mis-removal.

JIRA: FUNCTEST-422

Change-Id: I76da8786cac0fc5c874aa69e937d6ae19b20c6b1
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Reorder imports

It conforms to OpenStack Style Guidelines [1]

[1] http://docs.openstack.org/developer/hacking/

Change-Id: I995ff7fa3be995ca59e59c070b93394cd7200066
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Convert to Unix format

Change-Id: I3ddb9c38010c6f19ebd507990b57c9c498f49da8
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Check the response status code

Now an HTTPError exception is raised in case of a 4XX or 5XX error
response.

Change-Id: Idd91c1d2b2a1b81a1cf7d075527cb302099cdb4e
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Add reporting flag to push reuslts only from CI"

bugfix: wrong directory

Change-Id: I0b39277118ff26496553f9639a1c219fd5783b8f
Signed-off-by: meimei <meimei@huawei.com>

Remove parser tests from fuel onos

Change-Id: I72467575ca3eb79cbd0094882d3a5be73c2cf308
Signed-off-by: wuwb1989 <wuwenbin2@huawei.com>

Add reporting flag to push reuslts only from CI

JIRA: FUNCTEST-416

Change-Id: I1a5800be0dcf75573f44e6214b23917aadc1cbf7
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>

Merge "[multisite] set the criteria as 100%"

[moon] install moonclient in docker

Change-Id: Ie8b46620dfa995038c0f40e23133ac301496438a
Signed-off-by: meimei <meimei@huawei.com>

[multisite] set the criteria as 100%

Change-Id: I8e6a274c0f250b63c3e6f698b527fc476e1162b8
Signed-off-by: meimei <meimei@huawei.com>

Modify joid tempest list for onos

Change-Id: I0ae03b1f13bbc8801d8c14fa6ff1b93117c80e91
Signed-off-by: wuwb1989 <wuwenbin2@huawei.com>

Merge "blacklist.txt file format corrected"

Merge "output logger info for Parser tests"

blacklist.txt file format corrected

Only last part of blacklist.txt file was loaded by yaml.safe_load()
function. File format is corrected.

JIRA: FUNCTEST-418

Change-Id: I9c922b1a7b739e7d0c0d5afa3e0ea700749f9d50
Signed-off-by: vitikkan <viktor.tikkanen@nokia.com>

output logger info for Parser tests

Parser testcase fails in Functest. Because
there's on Parser debug info, make it difficult
to track failures.

JIRA: FUNCTEST-417

Change-Id: I04152de702f216dc053bdd7922a5672b5712aaff
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Add Domino user guide link, modified description text, remove trailing spaces in index.rst

Change-Id: I357274c8cbe5f312e2ed5b0ee719158e16c2cd60
Signed-off-by: Ulas Kozat <ulas.kozat@gmail.com>

Merge "Update user guide for Colorado"

Merge "Repoint security_scanning to its own repository"

Repoint security_scanning to its own repository

JIRA: FUNCTEST-415

Change-Id: I5ac2bd9b417ae045f4a1c0fa3f4fd2e7c545dbf3
Signed-off-by: Luke Hinds <lukehinds@gmail.com>

Add onos blacklist for tempest

Change-Id: If722e0d3376b8d04766b1eb871caadc5bc6e7c18
Signed-off-by: CNlucius <lukai1@huawei.com>

Fix joid test bug that creation of vm fails

Change-Id: I54d56cb2266ae627e4518f06b14705b672e6f74e
Signed-off-by: wuwb1989 <wuwenbin2@huawei.com>

[multisite]push result to db

Change-Id: Idda3a3a5c8fb20f6894675eb087417e150f8646d
Signed-off-by: meimei <meimei@huawei.com>

Call moon python script to run the moon tests

JIRA: FUNCTEST-210

Change-Id: Ic0a5f5620f9ceb42cb443b2509a2376e48d0cc42
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>

Merge "copper push to DB fix"

Merge "Update cli output text"

Merge "add parser to feature tier to make it run daily"

Update cli output text

Show correct command for creating snapshot in output when snapshot is
required but not yet created.

Change-Id: I3418406ca3370968248749ec1fdb30196b7d3a00
Signed-off-by: Juha Kosonen <juha.kosonen@nokia.com>

add parser to feature tier to make it run daily

For now parser belongs to vnf tier, but this tier
is defined to run weekly, which is not parser wanted,
so I move it to feature tier to make it run daily.

JIRA: FUNCTEST-413

Change-Id: I5b1fe3e7b140d26fe2e71f0a625e0ffad7937a13
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Update user guide for Colorado

- Automated section
- troubleshooting
- add new feature projects

JIRA: FUNCTEST-406

Change-Id: I6186b287b4defb9dff4547dc228fe5a9326fb6f3
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>

Provides connection check for package download.

JIRA: FUNCTEST-388

Performs simple connection test, using socket from std python
library. If no connection can be made, socket will timeout and
gracefully move to trying the next compute / control node.

Change-Id: I0048d74d607c8824688c73791c9646f9e9186756
Signed-off-by: Luke Hinds <lukehinds@gmail.com>

copper push to DB fix

JIRA: FUNCTEST-67

Change-Id: I03620f7f41f78da0ab004d55f72da88e9a30a1ea
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>

Fixing a typo

Multisite jobs kept failing because of a typo.

Signed-off-by: Dimitri Mazmanov <dimitri.mazmanov@ericsson.com>
Change-Id: I6e484e6457c7b8bae7c0343e9307bd15222e4fe3

Merge "Bug fix to eliminate undeterministic behaviour"

Merge "Improve LXD scenario support"

Improve LXD scenario support

JIRA: FUNCTEST-268

Change-Id: I8c8a3eff4a7e94b963c0307010a60723dc13a537
Signed-off-by: boucherv <valentin.boucher@orange.com>

Merge "Tacker post script link changed"

Tacker post script link changed

The tacker post script was modified in the fuel plug-in and thus the link
must be changed

Change-Id: I541eb43baf70171cb7c7ceb3fae284309a1a458b
Signed-off-by: Manuel Buil <manuel.buil@ericsson.com>

[bugfix] should create result folder for multisite

Change-Id: I004225131de57f1597edc62558a3a3e42f31b504
Signed-off-by: meimei <meimei@huawei.com>

[bugfix] pass the wrong argument to rally

Change-Id: I23ff06b53e45e9aa226f430e7d2fb75025281b9e
Signed-off-by: meimei <meimei@huawei.com>

Add conf directory to Dockerfile

Currently mounting the credentials file cannot be done at launch
time since the parent folder doesn't exist. This is usually not
noticed because prepare env will get the credentials file
via ssh, but the docker mount will silently fail.

Change-Id: I2e33c3c218a01ac6e1e5bf09609083dec3647b95
Signed-off-by: Michael Chapman <woppin@gmail.com>

Bug fix to eliminate undeterministic behaviour

When changing the classifier, ODL needs some seconds to update
the switches and sometimes they are not changed when the script
executes the second part of the test ==> test fails

Change-Id: I1fa3306eb7b283d708dbd736d2365456559db9b9
Signed-off-by: Manuel Buil <manuel.buil@ericsson.com>

Merge "Create config_functest patch to update the conf with scenario"

add report flag to parser

parser has no report flag, every time we run the test the results will be reported to db

JIRA: FUNCTEST-410

Change-Id: I6336b4c367fd8edc70e076b81d01d9356580bc22
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Merge "[multisite] refactor the scripts of multiste"

Merge "update pip version of functest docker image"

Merge "Bug fix in functest ODL-SFC"

[multisite] refactor the scripts of multiste

Change-Id: I07f0f5e253b04c9ac70e6dc4ea210e7cecf22c6a
Signed-off-by: meimei <meimei@huawei.com>

change parser path from RNC to vRNC

JIRA: FUNCTEST-409

Change-Id: I83887160a328c6f022efaa1547ca214571a9a01c
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

update pip version of functest docker image

JIRA: FUNCTEST-408

Change-Id: Ie14bbd0e2fa6f76957217d27cb8b0c76d4d28a03
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Create config_functest patch to update the conf with scenario

Adding support of LXD scenario

JIRA: FUNCTEST-268

Change-Id: I5891ff9de674d99d32d8db747be501ee8a2e6609
Signed-off-by: boucherv <valentin.boucher@orange.com>

Merge "parser integration bugfix"

Add SKIP status if Domino Test Case is skipped

Change-Id: I099010496dadabc18952de88fd6b3740592fc583
Signed-off-by: Ulas Kozat <ulas.kozat@gmail.com>

parser integration bugfix

JIRA: FUNCTEST-407

Change-Id: I9f202cb09935d9b77c5a977234bec6b505e296fd
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Bug fix in functest ODL-SFC

Change-Id: I496ade3ea5888967e933231f515a340369bffe8f
Signed-off-by: Manuel Buil <manuel.buil@ericsson.com>

Merge "Use Kingbird Bind host URL for Tempest"

Update test with the new SF and new tacker script

- A new SF image was created with a new name. Several files must be updated
- There is a new tacker branch for OPNFV Colorado, an update was made

Change-Id: Ib5ed364467baca32fa204155655318cb1ec189f5
Signed-off-by: Manuel Buil <manuel.buil@ericsson.com>

Use Kingbird Bind host URL for Tempest

Earlier, there was external URL for accessing KB services
in tempest.conf which is not accessible from functest docker.

Use bind host URL for KB service to access from the docker.

Change-Id: I147ffc2cd9883a6913080f9e1ab245807e0ca707
Signed-off-by: ashish singh <ashishsingh2k8@gmail.com>