functest.git
8 years agoPrint all the Copper test output to a log file 49/20349/1
Carlos Goncalves [Thu, 1 Sep 2016 15:40:33 +0000 (17:40 +0200)]
Print all the Copper test output to a log file

To keep the jenkins console clean and readable,
dump all the output that the Copper test case
generates to a log file and push it to artifact
repository at the end.

Change-Id: If4812c7e92b4d050757ca07333f7c353a0220056
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
Signed-off-by: Carlos Goncalves <carlos.goncalves@neclab.eu>
(cherry picked from commit 7442f6fb3ec962481d45a8ee0d12ad96242d9c0e)

8 years ago[Doctor test case] Redirect output to a log file 45/20345/1
Carlos Goncalves [Mon, 5 Sep 2016 12:06:25 +0000 (14:06 +0200)]
[Doctor test case] Redirect output to a log file

The output shown in the Jenkins console will now be
redirected to a file which will be available and linked
to the artifacts repository.

Change-Id: I2e9b5c2ab681a787aa0fc249844cc13feb581764
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
Signed-off-by: Carlos Goncalves <carlos.goncalves@neclab.eu>
(cherry picked from commit de6ca443b6ea60111f57bfe1cc6153463847c209)

8 years agoSwitch print to logging.info 37/20337/1
Cédric Ollivier [Thu, 1 Sep 2016 11:52:28 +0000 (13:52 +0200)]
Switch print to logging.info

It follows the recommendations described in the next review [1].

[1] https://gerrit.opnfv.org/gerrit/#/c/19795/

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

8 years agoRevert excluding floating IP cases from odl_l3 35/20335/1
vitikkan [Mon, 5 Sep 2016 09:33:13 +0000 (12:33 +0300)]
Revert excluding floating IP cases from odl_l3

Only restrictions table in release notes is updated.

Change-Id: I8f27a876058eabe1c490fc5980de5626aed665a3
Signed-off-by: vitikkan <viktor.tikkanen@nokia.com>
(cherry picked from commit beb0b0c8fddbdfd45fef0a7e9548b52e48f94c45)

8 years agoExclude test cases that use floating IP from odl_l3 scenarios 87/20287/1
vitikkan [Fri, 2 Sep 2016 09:33:09 +0000 (12:33 +0300)]
Exclude test cases that use floating IP from odl_l3 scenarios

All the tempest_smoke_serial test cases that use floating IP
addresses for accessing instances are excluded because of
ODL bug https://bugs.opendaylight.org/show_bug.cgi?id=5586

JIRA: FUNCTEST-445

Change-Id: I19468be108a25e989c0cfef031bbe5f2b9db0964
Signed-off-by: vitikkan <viktor.tikkanen@nokia.com>
(cherry picked from commit 0e29b1b97d1b996706aca692872fca5488a3acf7)

8 years agostop hardcoded FUNCTEST_REPO path everywhere 19/20219/1
SerenaFeng [Thu, 1 Sep 2016 02:19:58 +0000 (10:19 +0800)]
stop hardcoded FUNCTEST_REPO path everywhere

functest repo is used by almost every testcase, each place we get it like:
REPOS_DIR = os.getenv('repos_dir')
FUNCTEST_REPO = ("%s/functest" % REPOS_DIR)
provide a common interface in functest_util.py

JIRA: FUNCTEST-453

Change-Id: Ie0635dacc761ed2d05b7e606530368844f32ebaf
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit 470f573a3ac2eb1783efec6da01a3bee33360c65)

8 years agoMerge "Refactor SFC testcase" into stable/colorado
Jose Lausuch [Fri, 2 Sep 2016 07:56:13 +0000 (07:56 +0000)]
Merge "Refactor SFC testcase" into stable/colorado

8 years agounify functest_yaml obtain process 13/20113/2
SerenaFeng [Wed, 31 Aug 2016 02:22:33 +0000 (10:22 +0800)]
unify functest_yaml obtain process

functest_yaml is needed in almost all of the testcases, the obtain process is the same:
with open(os.environ["CONFIG_FUNCTEST_YAML"]) as f:
functest_yaml = yaml.safe_load(f)
f.close()
abstract a method to unify the process, and provide the interface

JIRA: FUNCTEST-447

Change-Id: I96a9a9962d7b466f10bbd3b5ab2495957524e22a
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit f4d0a05011e961a363c6d99522684ecdeb959716)

8 years agoFixes cleaning up HA Neutron objects 11/20211/1
Tim Rozet [Thu, 1 Sep 2016 19:05:07 +0000 (15:05 -0400)]
Fixes cleaning up HA Neutron objects

This fixes a race condition where the cleanup method is trying to remove
HA networks, while Neutron is also trying to remove the network - after
the HA router is removed.  This only applies to installations where
Neutron L3 HA is enabled (nosdn cases only).

JIRA: FUNCTEST-454

Change-Id: If06b6f6297e99dcce4b0f2aff8ec80b47488ee90
Signed-off-by: Tim Rozet <trozet@redhat.com>
(cherry picked from commit c7effe57d86f77e1d824b9c927d9e8ab70a59695)

8 years agoAdd router association utility function 75/20175/1
George Paraskevopoulos [Thu, 1 Sep 2016 13:27:44 +0000 (16:27 +0300)]
Add router association utility function

Add function to create an association between a router and a bgpvpn

Change-Id: I23449a0363f9d8f3c77ea3be1e104586d3c111d1
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
(cherry picked from commit 1d01ece307a130bdd0c85ea811b3b72cc6d53156)

8 years agoFix some bugs 41/20141/1
boucherv [Thu, 1 Sep 2016 13:04:42 +0000 (15:04 +0200)]
Fix some bugs

Change-Id: Iaf0859775933ba09cbbe170af3ce4ed1e6d3e9a1
Signed-off-by: boucherv <valentin.boucher@orange.com>
(cherry picked from commit 26f493bf3bb06a0ceb959294397ff653b92da202)

8 years agoAdd a specific logger for orchestrator class 09/20109/1
boucherv [Wed, 31 Aug 2016 15:43:08 +0000 (17:43 +0200)]
Add a specific logger for orchestrator class

JIRA: FUNCTEST-428

Change-Id: I1aab08f8f92df3b5527e6f8121cdf45aec2f5c17
Signed-off-by: boucherv <valentin.boucher@orange.com>
(cherry picked from commit dbbf8da5725f18eede541c82f6adac90a955f43d)

8 years agoRemove copper from lxd scenario 75/20075/1
Morgan Richomme [Wed, 31 Aug 2016 15:34:40 +0000 (17:34 +0200)]
Remove copper from lxd scenario

JIRA: FUNCTEST-452

Change-Id: I35c6417e6650ce190e499384f9cf9cca1fa35eb1
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
(cherry picked from commit bb699b3cfc146566ba54aa96111cf5a05839322a)

8 years agoRefactor SFC testcase 55/20055/1
George Paraskevopoulos [Mon, 29 Aug 2016 08:13:17 +0000 (11:13 +0300)]
Refactor SFC testcase

- Change the way the test checks if the vxlan_tool has started.
  Now the vxlan_tool is detected even if another python process is up.
- Log the exceptions in the vxlan_tool check block. The exceptions that
  occur are not destructive for the testcase but it's better to be
  visible.
- Add a 60 seconds sleep before the testcase starts to give plenty of
  time to ODL to add the correct flows.
- Move the testcase success/fail check blocks inside the try blocks.
  The checks are performed on the stdout variable which is in scope
  only inside the try blocks.

Change-Id: I49dec176fb83117e9896ac1d19e8a5bd95e3d7cf
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
(cherry picked from commit 563a540585768d900d70ce79f2a871a9d1fab03e)

8 years agoUse a global logger instead of method args 21/20021/1
Cédric Ollivier [Wed, 31 Aug 2016 07:38:24 +0000 (09:38 +0200)]
Use a global logger instead of method args

It's the first step to safely remove all logers as method args.
Outputs printed by bash scripts called by execute_command are no longer
written in functest.log. Else logs printed by python scripts will
be duplicated in it.

JIRA: FUNCTEST-428

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

8 years agounify testcases.yaml obtain process 87/19987/1
SerenaFeng [Wed, 31 Aug 2016 03:07:40 +0000 (11:07 +0800)]
unify testcases.yaml obtain process

hardcoded "''FUNCTEST_REPO + "/ci/testcases.yaml"''' appears wherever
testcases.yaml is needed, abstract a method to provide unified interface

JIRA: FUNCTEST-449

Change-Id: Ida8e3d90926d3e2245883a29f577ecb121bd5d6c
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit f5afb93a4a2ed9547757655c1405ef27fabed11b)

8 years agodelete useless FUNCTEST_REPO variable 85/19985/1
SerenaFeng [Wed, 31 Aug 2016 02:37:15 +0000 (10:37 +0800)]
delete useless FUNCTEST_REPO variable

FUNCTEST_REPO appears in lots of feature testcases, but never be used, delete it clean the code

JIRA: FUNCTEST-448

Change-Id: I4a9a10eac29e09b481563ed2c33f4122dd9ee3de
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit 834f81f0ad708b930f66aec1fd8b36e2cfb8b32b)

8 years agoRelease note: add restriction for joid/domino 83/19983/1
Morgan Richomme [Tue, 30 Aug 2016 07:01:31 +0000 (09:01 +0200)]
Release note: add restriction for joid/domino

JIRA: FUNCTEST-436

Change-Id: I44fb50a2ef58095aff54f03b837bf456850f3057
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
(cherry picked from commit e1cf8564b1f06a64863ee463b817935ec58f834c)

8 years agoUpdate Release note 81/19981/1
Morgan Richomme [Fri, 26 Aug 2016 10:16:05 +0000 (12:16 +0200)]
Update Release note

JIRA: FUNCTEST-436

Change-Id: I3cbf6b276494c77afb10802af6735647ea6e62d5
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
(cherry picked from commit 2d3b3b01e9c53930ff4bfa9d1142a3a25bba42a0)

8 years agoparser fails on virtual environement 75/19975/1
SerenaFeng [Wed, 31 Aug 2016 09:06:23 +0000 (17:06 +0800)]
parser fails on virtual environement

Parser tests needs lots of resources, such as vcpu/memory/disk,
virtuals pod may not meet the requirements, so exclude Parser
running on virtual pods by exclude 'noha' scenario

JIRA: FUNCTEST-444

Change-Id: I6f2832fa0f727f762987c41d7a3485dede041151
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit 35cf15a87126b6947b59899cd726f4b92599fe79)

8 years agoFix problem with SFC test 65/19965/1
Brady Johnson [Tue, 30 Aug 2016 15:50:10 +0000 (17:50 +0200)]
Fix problem with SFC test

- If its not possible to get the IPs, then exit
- Patch Set 2: addressed code review comments

Change-Id: I16e22513b32e2b0da311ebcc280f3611570634a3
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>
(cherry picked from commit 429ea989e665ebc3dac381c2629662f327dd9a46)

8 years agoCommitter promotion: Cedric Ollivier 63/19963/1
Morgan Richomme [Tue, 23 Aug 2016 15:07:17 +0000 (17:07 +0200)]
Committer promotion: Cedric Ollivier

Cedric joind Functest for Colorado
He fully refactored the ODL section including work upstream within ODL community
In addition of ODL skills, Cedric is a great benevolent reviewer.
Even if he is famous in the project for his gerrit -1, his reviews are always
usefull and clearly consolidate the code baseline for functest.
He is very relevant on all the technical aspects and we always need a Debianist
among commiters

Change-Id: I4bd0f778edfaa0d8969bdda3387d5a5ebc8b0f93
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
(cherry picked from commit bb88bedfd4410ca6057fe9a2512537b25ba7c326)

8 years agoCommitter promotion: Serena Feng 55/19955/1
Morgan Richomme [Tue, 23 Aug 2016 14:59:48 +0000 (16:59 +0200)]
Committer promotion: Serena Feng

Serena has joined the project for Colorado
She managed 22% of the open JIRAs (Ast rank in Functest)
She fully managed the refactoring of the testapi used by most of the projects
She also was key for the integration of several feature projects
She replaced the PTL during Summer

Change-Id: Id2a1ca253f80a425d7db4cb64591a44b4bcc5898
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
(cherry picked from commit 648a32fcf32390a301c75dd21aa7bae2ff5580d0)

8 years agodisable test case does not make sense for LXD. 13/19913/1
Narinder Gupta [Tue, 30 Aug 2016 15:28:28 +0000 (10:28 -0500)]
disable test case does not make sense for LXD.

This patch will disable the test cases which does not make sense
to run on LXD deployments.

Change-Id: I683b3714aa5af3c750ca57fab9fceff907b7d646
Signed-off-by: Narinder Gupta <narinder.gupta@canonical.com>
(cherry picked from commit 029a659b4198611dc93b6b41f42c3939a98de5bd)

8 years agoupdate fuel ref in tacker setup script 81/19881/1
Ferenc Cserepkei [Tue, 30 Aug 2016 10:10:47 +0000 (12:10 +0200)]
update fuel ref in tacker setup script

Since the poc script is fixed it's fuel commit hash in the setup
script have to be updated to select the fixed poc script.

Change-Id: I1fd22e4dc216fd2889adea0c9889df1d4a1bf476
Signed-off-by: Ferenc Cserepkei <ferenc.cserepkei@ericsson.com>
(cherry picked from commit 0d1faa954ec5a5e2911a2ec6df2f0a5f8782a623)

8 years agoExclude healthcheck from lxd scenarios 79/19879/1
Morgan Richomme [Tue, 30 Aug 2016 10:17:09 +0000 (12:17 +0200)]
Exclude healthcheck from lxd scenarios

JIRA: FUNCTEST-442

Change-Id: I1f6888dbddebf28af4b565ca87d9791a21cd4d7e
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
(cherry picked from commit 9b77f96822457e20bd5c5249c41bf498a3351b2b)

8 years agoAllow realtime Jenkins' console output 71/19871/1
Cédric Ollivier [Mon, 29 Aug 2016 20:50:53 +0000 (22:50 +0200)]
Allow realtime Jenkins' console output

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

8 years agoForce stdin, stdout and stderr to be totally unbuffered 69/19869/1
Cédric Ollivier [Mon, 29 Aug 2016 19:57:03 +0000 (21:57 +0200)]
Force stdin, stdout and stderr to be totally unbuffered

It protects against mixed logs as lots of python scripts directly call
print which doesn't flush stdout as opposed to log.info(). This
requirement is induced by subprocess.Popen() initiated by run_tests.py.

The following programs could switch to logging instead of print [1]

[1] $ find . -name "*.py" |xargs grep -v ^[[:space:]]*# \
        |grep \ print\  |cut -d: -f 1 |sort -u
./testcases/Controllers/ODL/OpenDaylightTesting.py
./testcases/Controllers/ONOS/Sfc/Sfc_fun.py
./testcases/Controllers/ONOS/Sfc/Sfc.py
./testcases/Controllers/ONOS/Teston/adapters/client.py
./testcases/Controllers/ONOS/Teston/adapters/connection.py
./testcases/Controllers/ONOS/Teston/adapters/environment.py
./testcases/Controllers/ONOS/Teston/adapters/foundation.py
./testcases/OpenStack/vPing/vping_util.py
./testcases/security_scan/connect.py
./utils/functest_utils.py
./utils/functest_vacation.py
./utils/openstack_clean.py

JIRA: FUNCTEST-441

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

8 years agoStop hard-coding db url 47/19847/1
Cédric Ollivier [Tue, 30 Aug 2016 08:33:39 +0000 (10:33 +0200)]
Stop hard-coding db url

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

8 years agoFlush stdout as it can be redirected to log 45/19845/1
Cédric Ollivier [Mon, 29 Aug 2016 19:24:52 +0000 (21:24 +0200)]
Flush stdout as it can be redirected to log

It avoids mixed logs in Jenkins' console by flushing every message
printed by the forked programs as logger.info() does.
It must be complete by refactoring logger management in util functions
[1] and by preventing output buffering in forked programs.

[1] https://jira.opnfv.org/browse/FUNCTEST-428

JIRA: FUNCTEST-441

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

8 years agoExclude floating-ip related tempest tests to blacklist for bgpvpn scenario 41/19841/1
jose.lausuch [Mon, 29 Aug 2016 13:35:35 +0000 (15:35 +0200)]
Exclude floating-ip related tempest tests to blacklist for bgpvpn scenario

Change-Id: I7115018e64a2413209313b9a432d3d1b59e841d1
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
(cherry picked from commit fbe4b77d39e0520e2f34a81dae7010b0f01d2aa2)

8 years agoSet env vars as mandatory to push to DB 35/19835/1
Cédric Ollivier [Fri, 26 Aug 2016 14:53:41 +0000 (16:53 +0200)]
Set env vars as mandatory to push to DB

Now env vars are mandatory in push_results_to_db. It avoids testing
programs to check them even if they are useless from their points of
view.
The related getters aren't removed to help debugging via
logger_test_results which could be run without report.

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

8 years agoReview dev guide 31/19731/1
Morgan Richomme [Wed, 24 Aug 2016 14:37:14 +0000 (16:37 +0200)]
Review dev guide

JIRA: FUNCTEST-433

Change-Id: Ic141a5b74d9b3afc165e1f1f65f3dc1b05d77f4e
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
(cherry picked from commit 94c22c662fa057a181f76ef28bddc0c5ac949e7e)

8 years agoconfiguration guide review 47/19647/1
Morgan Richomme [Thu, 25 Aug 2016 16:00:55 +0000 (18:00 +0200)]
configuration guide review

JIRA: FUNCTEST-435

Change-Id: I5b18f1223a5892d084c761f174ac5f713ceed476
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
(cherry picked from commit e9876bd337af54883fe46e49343abc22cfd0404f)

8 years agorefactor create shared network process to eliminate reduplicate 43/19643/1
SerenaFeng [Thu, 25 Aug 2016 03:20:17 +0000 (11:20 +0800)]
refactor create shared network process to eliminate reduplicate

Some places shared network is created through create_network_full first,
then update it to shared, abstract a uniformed method to eliminate reduplicate

JIRA: FUNCTEST-437

Change-Id: I13c20ab061323188c5eb3b8c861674df38083c62
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit 007868e4810aaf9a2664b030281fe7c818fc3494)

8 years agorefactor create or get image process to eliminate reduplicate 39/19639/1
SerenaFeng [Tue, 23 Aug 2016 05:44:36 +0000 (13:44 +0800)]
refactor create or get image process to eliminate reduplicate

Some places when create image is referred, they check if image exist,
if exist use it directly, else create a new one.
Abstract a method to integrate the process

JIRA: FUNCTEST-432

Change-Id: I83d1112aa5cb0d3cdfab92fd49cd5f2c1cceff82
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit 5f5af6b1d8d0af19db143e67f04f67a643af863c)

8 years agoStop calling exec_test.sh in the middle of python scripts 37/19637/1
Cédric Ollivier [Wed, 24 Aug 2016 15:55:10 +0000 (17:55 +0200)]
Stop calling exec_test.sh in the middle of python scripts

Now run_tests.py calls directly OpenDaylightTesting.functest_run().
It mainly reads env vars as exec_test.sh does.

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

8 years ago[Fuel] Skip test_server_basic_ops tempest test 35/19635/1
Michael Polenchuk [Tue, 23 Aug 2016 07:49:25 +0000 (10:49 +0300)]
[Fuel] Skip test_server_basic_ops tempest test

Skip TestServerBasicOps which fails when deleting server
and floating ip almost at the same time.

JIRA: FUEL-173
JIRA: FUEL-176
Change-Id: I24f3397096757028bed9d32d6353aa689e8eaa9a
Signed-off-by: Michael Polenchuk <mpolenchuk@mirantis.com>
(cherry picked from commit 44819aec4a70a75abc9fbcff23edd9d904170bf7)

8 years agoUse by default the default ODL web port 41/19541/1
Cédric Ollivier [Wed, 24 Aug 2016 15:31:01 +0000 (17:31 +0200)]
Use by default the default ODL web port

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

8 years agoFixing commit f1054f8f 11/19511/1
Ferenc Cserepkei [Wed, 24 Aug 2016 14:24:11 +0000 (16:24 +0200)]
Fixing commit f1054f8f

fix missed the apt-get action

Change-Id: Iac94576a2f712133eee92e4b5460f85d59e90e94
Signed-off-by: Ferenc Cserepkei <ferenc.cserepkei@ericsson.com>
(cherry picked from commit 2002f166154ca5a9380f9d2da2843477669af8f5)

8 years agoIncluded correct links (Colorado) to Doctor/Promise user guides 75/19475/1
Bertrand Souville [Wed, 24 Aug 2016 08:37:04 +0000 (10:37 +0200)]
Included correct links (Colorado) to Doctor/Promise user guides

Change-Id: I9f6a3d806407f088bd97826c978417d23b18613a
Signed-off-by: Bertrand Souville <souville@docomolab-euro.com>
(cherry picked from commit 218e49c9729f3ecfe3661a6efce0a46a6ba4e278)

8 years ago[SFC] Fixing CI 63/19463/1
Nikolas Hermanns [Wed, 24 Aug 2016 08:17:09 +0000 (10:17 +0200)]
[SFC] Fixing CI

The fix https://gerrit.opnfv.org/gerrit/#/c/19351 will need
more time for dicussion. In the meanwhile we can use this
one so that CI does not break anymore.

Change-Id: Icbd54200bd62e5019c2a556699ceb41afdb229ca
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit f1054f8f7c787e50df0c0e27ecb10752393071e2)

8 years agoAvoid duplicating logs in run_tests.py 47/19347/1
Cédric Ollivier [Tue, 23 Aug 2016 11:25:02 +0000 (13:25 +0200)]
Avoid duplicating logs in run_tests.py

When run_tests.py sets logger as execute_command arg, logs are
duplicated in functest.log and not shown in console (as every DEBUG
message). As logger is mainly managed by the python scripts in charge
of testing, it can be safely removed here.
stderr is also redirected to stdout in execute_command to print
possible relevant data.

JIRA: FUNCTEST-431

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

8 years agowhen create/get/update failed return None rather than False 45/19345/1
SerenaFeng [Tue, 23 Aug 2016 06:27:51 +0000 (14:27 +0800)]
when create/get/update failed return None rather than False

In openstack_util.py, when create/get/update network or other resources failed,
False is returned, which is incompatible with success return value such as id, network_nic.

JIRA: FUNCTEST-427

Change-Id: Id2f55d8524e5aff150ba6bfb799085377a63baa4
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit 3e95a1cd0721924e89e2cfd2960d4736bc6c5588)

8 years agodelete useless image_exists in vping 41/19341/1
SerenaFeng [Tue, 23 Aug 2016 03:05:52 +0000 (11:05 +0800)]
delete useless image_exists in vping

In rally, image_exists is used to decide whether the image
resource should be cleaned or not. But unlike rally,
image_exists in vping is only assigned but not never be actually used.

JIRA: FUNCTEST-426

Change-Id: Id2089dd2a9734ba957cc1c80d4dc31bc27907b0a
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit 73053cd38fed875e3e5d64cabfb7fbfeb2839a80)

8 years agoEnable security scan for apex scenarios 19/19319/1
vitikkan [Tue, 23 Aug 2016 08:17:30 +0000 (11:17 +0300)]
Enable security scan for apex scenarios

Change-Id: I6c2925237d7ae437f49d60d37ef906ed8029eb03
Signed-off-by: vitikkan <viktor.tikkanen@nokia.com>
(cherry picked from commit da6398642446f98240d9d1bf924e959e64a73d63)

8 years agorefactor push_result_to_db print thing 11/19311/1
SerenaFeng [Mon, 22 Aug 2016 10:23:25 +0000 (18:23 +0800)]
refactor push_result_to_db print thing

when Exceptions happen, the print code is too difficult
to understand and maintain, modify to make it clear

jira: FUNCTEST-425

Change-Id: Ie12af989e711df3791167955642f2edf1164b146
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
(cherry picked from commit 2588e2af6c2b609ab08957268f8163954cd8eae1)

8 years agoCreate tempest resources before generating tempest.conf 09/19309/1
vitikkan [Tue, 23 Aug 2016 07:32:24 +0000 (10:32 +0300)]
Create tempest resources before generating tempest.conf

During generating tempest.conf file rally tries to download
cirros image from the internet if no suitable images exist in
the configuration. This change ensures that cirros image is
created before generating tempest.conf file.

JIRA: FUNCTEST-195

Change-Id: I9e2df8af40bd02db9b78b8b6b730b4f4168a8b2d
Signed-off-by: vitikkan <viktor.tikkanen@nokia.com>
(cherry picked from commit 93f4a6576d58522dfbb458f184399767ef8764aa)

8 years agodoctor test support fuel installer 93/19293/1
dongwenjuan [Tue, 23 Aug 2016 02:27:26 +0000 (10:27 +0800)]
doctor test support fuel installer

Change-Id: I9229d4de823568bfefd4f2c1ab5c5c1a1ded6d2c
Signed-off-by: dongwenjuan <dong.wenjuan@zte.com.cn>
(cherry picked from commit e0a17212d21ae7d76c55791ab8bcde1c66df3158)

8 years agoAdd logger to push_results_to_db args 91/19291/1
Cédric Ollivier [Tue, 23 Aug 2016 06:23:08 +0000 (08:23 +0200)]
Add logger to push_results_to_db args

It conforms to the current implementation of functest_utils

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

8 years agoPoint tempest to stable 12.1.0 89/19289/1
Michael Polenchuk [Mon, 22 Aug 2016 09:38:15 +0000 (12:38 +0300)]
Point tempest to stable 12.1.0

Checkout tempest to the latest stable tag
instead of master development branch.

Change-Id: I20c7e83bf6441e84b9441f4cb193681753f81300
Signed-off-by: Michael Polenchuk <mpolenchuk@mirantis.com>
(cherry picked from commit 132774cfdca1f8c4cb6f62baed15314f657e0e0b)

8 years agoFix typos in OpenDaylightTesting 27/19227/1
Cédric Ollivier [Mon, 22 Aug 2016 13:39:42 +0000 (15:39 +0200)]
Fix typos in OpenDaylightTesting

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

8 years agoMend create_image() function 05/19205/1
Michael Polenchuk [Mon, 22 Aug 2016 12:37:06 +0000 (15:37 +0300)]
Mend create_image() function

Change-Id: I91681736817f881ca53e5124328bd4399aa54740
Signed-off-by: Michael Polenchuk <mpolenchuk@mirantis.com>
8 years agoMerge changes from topics 'dont_open_twice', 'add_logs', 'manage_return_codes'
Morgan Richomme [Mon, 22 Aug 2016 12:09:06 +0000 (12:09 +0000)]
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

8 years agoMerge "bugfix: push_result_to_db exception show"
Morgan Richomme [Mon, 22 Aug 2016 12:08:13 +0000 (12:08 +0000)]
Merge "bugfix: push_result_to_db exception show"

8 years agoAvoid opening tmp files twice 91/19191/2
Cédric Ollivier [Mon, 22 Aug 2016 11:40:34 +0000 (13:40 +0200)]
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>
8 years agoMerge "Bug fix - Better way to grab file from gerrit"
valentin boucher [Mon, 22 Aug 2016 11:57:36 +0000 (11:57 +0000)]
Merge "Bug fix - Better way to grab file from gerrit"

8 years agoAdd logs (info) 83/19183/1
Cédric Ollivier [Mon, 22 Aug 2016 10:51:22 +0000 (12:51 +0200)]
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>
8 years agobugfix: push_result_to_db exception show 37/19137/4
SerenaFeng [Mon, 22 Aug 2016 02:50:31 +0000 (10:50 +0800)]
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>
8 years agoImprove errors and status codes management 77/19177/2
Cédric Ollivier [Mon, 22 Aug 2016 09:48:44 +0000 (11:48 +0200)]
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>
8 years agoCatch exception if output.xml failed 71/19171/1
Cédric Ollivier [Mon, 22 Aug 2016 09:06:12 +0000 (11:06 +0200)]
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>
8 years agoMerge "Add reporting to DB in OpenDaylightTesting"
Morgan Richomme [Mon, 22 Aug 2016 08:43:46 +0000 (08:43 +0000)]
Merge "Add reporting to DB in OpenDaylightTesting"

8 years agoMerge "Protect again changes of unix permissions"
Morgan Richomme [Mon, 22 Aug 2016 08:06:46 +0000 (08:06 +0000)]
Merge "Protect again changes of unix permissions"

8 years agoMerge "refactor vping"
Morgan Richomme [Mon, 22 Aug 2016 07:14:00 +0000 (07:14 +0000)]
Merge "refactor vping"

8 years agorefactor vping 17/18717/10
SerenaFeng [Tue, 16 Aug 2016 06:14:28 +0000 (14:14 +0800)]
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>
8 years agoProtect again changes of unix permissions 21/19121/1
Cédric Ollivier [Sun, 21 Aug 2016 10:22:42 +0000 (12:22 +0200)]
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>
8 years agoChmod 755 all py files which can be executed 19/19119/1
Cédric Ollivier [Sun, 21 Aug 2016 10:08:39 +0000 (12:08 +0200)]
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>
8 years agoAdd reporting to DB in OpenDaylightTesting 83/19083/4
Cédric Ollivier [Fri, 19 Aug 2016 15:39:50 +0000 (17:39 +0200)]
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>
8 years agoAdd exec right 87/19087/1
Cédric Ollivier [Fri, 19 Aug 2016 16:06:41 +0000 (18:06 +0200)]
Add exec right

Change-Id: I9c3fd786574622977870b12d38b92f8bed8a929c
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
8 years agoMerge "Fix print when push_results_to_db fails"
Morgan Richomme [Fri, 19 Aug 2016 15:41:35 +0000 (15:41 +0000)]
Merge "Fix print when push_results_to_db fails"

8 years agoFix print when push_results_to_db fails 81/19081/1
Cédric Ollivier [Fri, 19 Aug 2016 15:30:18 +0000 (17:30 +0200)]
Fix print when push_results_to_db fails

Change-Id: I82741a8727fc17bc6648e1b1e92511b3efd93b8a
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
8 years agoMerge "Modify data sent to testAPI"
Morgan Richomme [Fri, 19 Aug 2016 12:05:23 +0000 (12:05 +0000)]
Merge "Modify data sent to testAPI"

8 years agoBug fix - Better way to grab file from gerrit 37/19037/2
Manuel Buil [Fri, 19 Aug 2016 08:16:16 +0000 (10:16 +0200)]
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>
8 years agoPull down default instance flavor 47/19047/1
Michael Polenchuk [Fri, 19 Aug 2016 09:02:20 +0000 (12:02 +0300)]
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>
8 years agoMerge "functest compass usage document"
Morgan Richomme [Fri, 19 Aug 2016 07:42:47 +0000 (07:42 +0000)]
Merge "functest compass usage document"

8 years agoMerge "Remove useless odlreport2db args"
Morgan Richomme [Fri, 19 Aug 2016 07:41:50 +0000 (07:41 +0000)]
Merge "Remove useless odlreport2db args"

8 years agoMerge "Colorado Release note"
Morgan Richomme [Fri, 19 Aug 2016 07:41:41 +0000 (07:41 +0000)]
Merge "Colorado Release note"

8 years agoModify data sent to testAPI 39/18939/3
Cédric Ollivier [Thu, 18 Aug 2016 08:57:34 +0000 (10:57 +0200)]
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>
8 years agoColorado Release note 53/18853/6
Morgan Richomme [Wed, 17 Aug 2016 15:41:37 +0000 (17:41 +0200)]
Colorado Release note

FUNCTEST-420

Change-Id: I3fab0637fedc04bcb61a76e61f91da5df5c732c1
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
8 years agofix Pillow version 3.3.0 05/19005/1
Ryota MIBU [Fri, 19 Aug 2016 02:49:57 +0000 (11:49 +0900)]
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>
8 years agofunctest compass usage document 75/18775/5
MatthewLi [Wed, 17 Aug 2016 02:33:14 +0000 (22:33 -0400)]
functest compass usage document

JIRA: FUNCTEST-333

add some information in configuration file

Change-Id: I6ec91d15c859b9739904d602aef3a0927f6561b6
Signed-off-by: MatthewLi <matthew.lijun@huawei.com>
8 years agoMerge "complement Parser related userguide"
Morgan Richomme [Thu, 18 Aug 2016 14:54:00 +0000 (14:54 +0000)]
Merge "complement Parser related userguide"

8 years agocomplement Parser related userguide 43/18943/2
SerenaFeng [Thu, 18 Aug 2016 09:07:23 +0000 (17:07 +0800)]
complement Parser related userguide

JIRA: FUNCTEST-423

Change-Id: Ib52e04a873a2a924102832fe5461ebab2296464d
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
8 years agoMerge "Config guide workflow simplification"
Morgan Richomme [Thu, 18 Aug 2016 09:52:40 +0000 (09:52 +0000)]
Merge "Config guide workflow simplification"

8 years agoConfig guide workflow simplification 79/18479/7
Juha Haapavirta [Thu, 11 Aug 2016 18:47:12 +0000 (21:47 +0300)]
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>
8 years agoRemove useless odlreport2db args 17/18917/2
Cédric Ollivier [Thu, 18 Aug 2016 07:34:04 +0000 (09:34 +0200)]
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>
8 years agoMerge "Add report flag for moon"
Morgan Richomme [Thu, 18 Aug 2016 08:49:10 +0000 (08:49 +0000)]
Merge "Add report flag for moon"

8 years agoMerge "resume Parser test run on fuel onos"
Morgan Richomme [Thu, 18 Aug 2016 08:23:54 +0000 (08:23 +0000)]
Merge "resume Parser test run on fuel onos"

8 years agoAdd report flag for moon 27/18927/1
Morgan Richomme [Thu, 18 Aug 2016 08:18:18 +0000 (10:18 +0200)]
Add report flag for moon

JIRA: FUNCTEST-210

Change-Id: I97be5542ae4ae87f1b8d62394ff66068e77f4082
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
8 years agoMerge "Minor fixes for Doctor Project"
Morgan Richomme [Thu, 18 Aug 2016 07:56:36 +0000 (07:56 +0000)]
Merge "Minor fixes for Doctor Project"

8 years agoMerge "Convert to Unix format"
Morgan Richomme [Thu, 18 Aug 2016 07:56:19 +0000 (07:56 +0000)]
Merge "Convert to Unix format"

8 years agoMinor fixes for Doctor Project 99/18899/2
Bertrand Souville [Thu, 18 Aug 2016 04:45:47 +0000 (06:45 +0200)]
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>
8 years agodelete extra slash while execute cases test in exec_test.sh 95/18895/2
SerenaFeng [Thu, 18 Aug 2016 02:15:11 +0000 (10:15 +0800)]
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>
8 years agoresume Parser test run on fuel onos 01/18901/1
SerenaFeng [Thu, 18 Aug 2016 03:48:15 +0000 (11:48 +0800)]
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>
8 years agoReorder imports 49/18849/1
Cédric Ollivier [Wed, 17 Aug 2016 15:17:08 +0000 (17:17 +0200)]
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>
8 years agoConvert to Unix format 41/18841/1
Cédric Ollivier [Wed, 17 Aug 2016 13:42:00 +0000 (15:42 +0200)]
Convert to Unix format

Change-Id: I3ddb9c38010c6f19ebd507990b57c9c498f49da8
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
8 years agoCheck the response status code 39/18839/1
Cédric Ollivier [Wed, 17 Aug 2016 13:26:35 +0000 (15:26 +0200)]
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>
8 years agoMerge "Add reporting flag to push reuslts only from CI"
Morgan Richomme [Wed, 17 Aug 2016 11:22:50 +0000 (11:22 +0000)]
Merge "Add reporting flag to push reuslts only from CI"

8 years agobugfix: wrong directory 23/18823/1
meimei [Wed, 17 Aug 2016 10:37:38 +0000 (18:37 +0800)]
bugfix: wrong directory

Change-Id: I0b39277118ff26496553f9639a1c219fd5783b8f
Signed-off-by: meimei <meimei@huawei.com>
8 years agoRemove parser tests from fuel onos 21/18821/1
wuwb1989 [Wed, 17 Aug 2016 09:25:39 +0000 (17:25 +0800)]
Remove parser tests from fuel onos

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