Merge "Adding three missing test case descriptions"

Merge "Fix i40vf driver download URL"

Fix i40vf driver download URL

Fix the download URL of i40vf driver introduced in
15e122a770d6349c84801dd05891e21661cbad2c.

In order to avoid the redirection, the URL should be:
  https://netix.dl.sourceforge.net/project/e1000/
  i40evf%20stable/{{ i40evf_version }}/{{ i40evf_gzfile }}

JIRA: YARDSTICK-993

Change-Id: Ie19d11d36417914e823b81867046123204111d76
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Removed zero from tc number

Originally tc number was TC0042, after fix TC042.

Change-Id: I3c36e39a141ec8eb74f9f00d7218bd511a56fbc9
Signed-off-by: Mika Rautakumpu <mika.rautakumpu@nokia.com>

Merge "Adding new vFirewall Test cases based on Concurrency, Connections per second, Throughput and Transactions per second for various http locator image sizes including 1b, 4k, 64k, 256k, 1024k"

Merge "Add "img_property" when building Yardstick image"

Adding three missing test case descriptions

HA tests TC056, TC057 and TC058 are missing from Yardstick
test case list. Missing white space added to tc056.rst.

Change-Id: Id56a22ad96af8c3d05d95fb58259e9683ce9fab0
Signed-off-by: Mika Rautakumpu <mika.rautakumpu@nokia.com>

Merge "Modify fuel_baremetal pod config file"

Add sample OpenStack/SRIOV RFC2544 based TC

Add sample OpenStack/SRIOV RFC2544 based test case which can be
run using external Trex.

Console log: http://paste.openstack.org/show/658002/
Result output: http://paste.openstack.org/show/658004/

JIRA: YARDSTICK-967

Change-Id: Id0e2d6d47f459c7e6a3370735bab74f7ce502b24
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>

Merge "Add Intel Fortville 40G VF driver to sample VNF image"

Merge "Change CivetWeb download URL"

Merge "Fix up tests for Parser scenario"

Merge "Move tests: unit/cmd/"

Add Intel Fortville 40G VF driver to sample VNF image

1) Add common YAML config file to add kernel drivers to sample VNF image.
2) Add Intel Fortville 40G VF driverto sample VNF image using
   common YAML config file for drivers.
3) Configure i40evf driver to load on boot

JIRA: YARDSTICK-968

Change-Id: I223c8a10d77352baf4f41a481f465970655ba251
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>

Merge "Adding support to pull docker images behind proxy"

Fix up tests for Parser scenario

* Check the results at the test methods
* Remove print statement from Parser scenario
* Replace assertEquals(x, True) with assertTrue(x)

Change-Id: I5c2612692e625fc888c7ce7637b9e1625440724a
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Adding support to pull docker images behind proxy

This patch creates a http_proxy.conf in docker service,
to enable docker commands behind the proxy

Change-Id: Iac38a086026c11e89e534728467dc7da0e01ba59
Signed-off-by: Deepak S <deepak.s@linux.intel.com>

Merge "Add python-openstackclient in docker image"

Merge "Replace assertEqual(x, True|False) with assert[True|False](x)"

Merge changes from topic 'bug/yardstick-864'

* changes:
  Replace assertEquals with assertEqual
  Make files pep8 compliant before using assertTrue|False

Merge "Remove generated file in 'test_build_vm_xml'"

Modify fuel_baremetal pod config file

The current HA testcases failed on fuel baremetal installer
with reason "SSHTimeout". This is due to incorrect ssh login
info in the pod config file. The change is to by running
'prepare_env.sh' script to

- read env vars 'USER_NAME' and 'SSH_KEY'
- replace the 'node_username' to env var 'USER_NAME'
- replace the "node_keyfile" to env var 'SSH_KEY'

JIRA: ARMBAND-317

Change-Id: I0239892f447ae99ab4025eedcd133e1b6f04e5a5
Signed-off-by: ting wu <ting.wu@enea.com>

Add python-openstackclient in docker image

"openstack: command not found" error occurred in [1].
The version of python-openstackclient is listed in [2].
This program should be installed in the container image
to make CI work for now.

[1]: https://build.opnfv.org/ci/job/yardstick-daisy-baremetal-daily-master/114/
[2]: https://github.com/openstack/requirements/blob/stable/pike/global-requirements.txt

Change-Id: Idb50e26bb2ca36f2dd6b6dddf94680d479cb9c9f
Signed-off-by: Alex Yang <yangyang1@zte.com.cn>

Replace assertEqual(x, True|False) with assert[True|False](x)

Change-Id: Id19df79b4d27aab1a83d8350ce6395e87c13472b
JIRA: YARDSTICK-903
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Replace assertEquals with assertEqual

unittest.assertEquals is deprecated[1], and has been replaced with
unittest.assertEqual.

[1] https://docs.python.org/2/library/unittest.html#deprecated-aliases

Change-Id: I9c6320e3a9ec5528036b529a9c32fc48b0bcfd62
JIRA: YARDSTICK-864
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Make files pep8 compliant before using assertTrue|False

JIRA: YARDSTICK-903
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
Change-Id: Id7912b5ddee36e7366bcfa824379853efd0a89f1

Move tests: unit/cmd/

* Fix pylint errors
* Add TODOs

Some errors are ignored locally, as they were a symptom of other problems.
These issues have been flagged with a TODO, and should be fixed later.

Change-Id: I505d72e1c646fea1976a0bf854369c6b6308bbb1
Jira: YARDSTICK-837
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Remove img_modify_playbook from nsb_setup.sh script

The `img_modify_playbook` variable is not used anymore by the
build_yardstick_image.yml. The `img_property` variable is used
instead. The `img_modify_playbook` usage has been removed and
the `img_property` has been introduced in
YARDSTICK-879 JIRA story.

JIRA: YARDSTICK-964

Change-Id: I0239544c4f4078de7a4817292ab6bd789c21e944
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>
Signed-off-by: Alonso Hernandez, Rodolfo <rodolfo.alonso.hernandez@intel.com>

Remove generated file in 'test_build_vm_xml'

Change I22e95c488e27d6e2a8fdf6c1a07faab275fa6bba introduced a change in
test case 'test_build_vm_xml' [1]. This test case generates a file which
is stored locally but never removed.

[1]https://github.com/opnfv/yardstick/blob/80dc9034edeb05f66bfa6264ae0adf0f6aea60da/tests/unit/benchmark/contexts/standalone/test_model.py

JIRA: YARDSTICK-918

Change-Id: I91d39bb37c3d8a273fe68155bb308c5a3bc1ca9d
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Merge "'libvirt' guest console in Standalone model"

Merge "Check RS for virtualized OpenStack/Yardstick"

Merge "Cleanup of redundant files from Prox directory"

Check RS for virtualized OpenStack/Yardstick

Draft for env requirements reading and validation
Script can be run from tools/virt_ci_rampup.sh. First requirement file is
parsed/stored. This file must be created in ahead and named as
'infra_deploy.yaml'. Then separate ansible role is used to validate
resources. Ansible-role fails if there is not enough resources on the host
for cpu, ram, disk space.

JIRA: YARDSTICK-945

Change-Id: I86cd4bdc963de3aa51392fc73a239cbd426bc64a
Signed-off-by: Stepan Andrushko <stepanx.andrushko@intel.com>

Merge "Use assertIn(x, y) instead of other variations"

Cleanup of redundant files from Prox directory

JIRA: YARDSTICK-965

Removed the redundant files and directory from
Prox test folder.

Change-Id: I6167f0a4ccdd581153dad3ec9f5eaaf2619aacbd
Signed-off-by: Abhijit Sinha <abhijit.sinha@intel.com>

Merge "Enable iommu support in grub"

Rename ResourceProfile.check_if_sa_running()

* Update log message and rename method to make debugging easier
* Fix pep8 violations in touched files

JIRA: YARDSTICK-859

Change-Id: Id138d72cd8557ffacd5dcc81d06feeb484e13625
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Enable iommu support in grub

JIRA: YARDSTICK-895
It's required for sriov standalone yardstick tests.
This part of commit message will be deleted:
This code will be enabled when https://gerrit.opnfv.org/gerrit/#/c/47957/ will be merged,
because it requires host reboot.

Change-Id: I0ec9cbf90a9d83d54e545fcf421b8baf206601a4
Signed-off-by: Jan Malanik <janx.malanik@intel.com>
Signed-off-by: Malanik Jan <janx.malanik@intel.com>

Merge "Clean unit tests in "benchmark.scenarios.lib.test_create_volume.py"

Merge "Move tests: unit/dispatcher"

Merge "Move tests: unit/orchestrator/"

Merge "Replace "oslo_utils.importutils" with standard library "importlib""

Clean unit tests in "benchmark.scenarios.lib.test_create_volume.py

* Use mock_object.assert_called_with instead of
    self.assertTrue(mock_object.called)
* Remove an unneeded if clause in CreateVolume
* Add individual tests for each method in CreateVolume

Change-Id: I54cc56434d316f059b3822879145e6a299866fcb
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Merge "update docs for procedure after host reboot"

Merge "Vincenzo stepping down as Yardstick committer"

Change ansible version to 2.4.2

Modified requirements.txt to update ansible version to 2.4.2

For the ansible virt module [1] XML templates are required.

To edit, the XML templates the XML module is required.
This is new in ansible 2.4 [2]

There is no max version restriction for ansible in OpenStack [3].

[1] http://docs.ansible.com/ansible/latest/virt_module.html
[2] https://docs.ansible.com/ansible/2.4/xml_module.html
[3] https://github.com/openstack/kolla-ansible/blob/stable/pike/doc/source/user/quickstart.rst

JIRA: YARDSTICK-948

Change-Id: I19b178f695f7381a3ecfbdb74f953a1276250958
Signed-off-by: John O Loughlin <john.oloughlin@intel.com>

Replace "oslo_utils.importutils" with standard library "importlib"

The current implementation of dynamic library importation is prone
to failure [1]:
- "sys.modules" is modified manually, which is something not
  recommended [2].
- When a module is imported is added to "sys.modules"; that means
  there is no need to manually create an entry in this object.
- "importlib" library is part of the standard library and is now
  available in PY3 and PY2 (backported). This library contains a
  function called "import_module" to import a module in runtime.

[1]https://github.com/opnfv/yardstick/blob/d2c7cc4e9768ed003257a95c92cdb278d516761b/yardstick/common/utils.py#L72-L93
[2]http://justus.science/blog/2015/04/19/sys.modules-is-dangerous.html

JIRA: YARDSTICK-949

Change-Id: Ide3b74f98858d06fa275fb6c9b78ceeaa64feed5
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Vincenzo stepping down as Yardstick committer

From: opnfv-tech-discuss-bounces@lists.opnfv.org [mailto:opnfv-tech-discuss-bounces@lists.opnfv.org] On Behalf Of Riccobene, Vincenzo M
Sent: Thursday, January 18, 2018 2:35 AM
To: opnfv-tech-discuss@lists.opnfv.org
Subject: [opnfv-tech-discuss] Stepping down as Yardstick committer

Hello,
I am currently a committer for the OPNFV Yardstick project, but as I see unfortunately in the last months I have been absent from the project since other commitments took higher priorities and all my time, resulting in no major contributions to Yardstick, and I consequently decided to step down from the positon of committer.

It has been a great pleasure to contribute to the community and I hope I will be able to contribute again in the future.
Thank you very much all!

Best Regards.
Vince

Change-Id: I809021b4081caddb60549204a742b93134190a87
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>

Merge "Fix Cirros image name in CI script"

Move tests: unit/orchestrator/

* Fix pylint errors
* Add notes where additional action is needed.

Some errors are ignored locally, as they were a symptom of other problems.
These issues have been flagged for follow-up, and should be fixed later.

JIRA: YARDSTICK-837
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
Change-Id: Idbf56d0cd3b29b70691acd6af4d5ac3c400d2ccc

Proposing Emma Foley as a committer in Yardstick

Emma has contributed to Yardstick in many ways
Achievement/Contributions include:
- Provided high-quality code review comments.
- Tox improvements
- Refactored unittest to OpenStack conventions
- Improved unittest code coverage
- Improved code quality by fixing warnings and errors
- Documentation review and update
- Split out test-requirements.txt
- Attending OPNFV Euphrates Plugfest Portland
- Starting Traffic-Generator-as-a-service initiative with NFVBench and VSPerf
- Contributions:
  https://gerrit.opnfv.org/gerrit/#/q/owner:emma.l.foley%2540intel.com

Change-Id: I639fa27a7271bf9996b2ee0e19a689644af54afe
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>

Proposing Rodolfo Alonso Hernandez as a committer in Yardstick

Rodolfo has contributed to Yardstick in many ways
Achievement/Contributions include:
- Userguide review and update
- Provided high-quality code review comments.
- Improved unittest code coverage
- Improved code quality by fixing warnings and errors
- Jenkins gate check improvements
- Pip requirements checking and updating
- Migration to Shade removing OpenStack project clients.
- Regularly attending Yardstick meetings
- JIRA personal dashboard:
  https://jira.opnfv.org/secure/Dashboard.jspa?selectPageId=11400
- Contributions:
  https://gerrit.opnfv.org/gerrit/#/q/owner:rodolfo.alonso.hernandez%2540intel.com

Change-Id: I3e5f14a3f99fffcd4bd4973aedc44ce4beeb22be
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>

Merge "Proposing Jack Chan as a committer in Yardstick"

Fix Cirros image name in CI script

Fixed the name of Cirros image used in OpenStack. The image name
is written in lowercase letters.

JIRA: YARDSTICK-842

Change-Id: Iff0b494125be61a055da753eaf57903b64f18509
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Merge "bugfix: Fix duplicate 'address' in standlone SRIOV"

bugfix: Fix duplicate 'address' in standlone SRIOV

* Add unit test per Rodolfo's comment

Creating a VM using generted xml file /tmp/vm_sriov_0.xml returns
an XML error: missing source address type.

This fix modifies generted xml from:
<source>
<address>
<address bus="0x1a" domain="0x0000" function="0x0" slot="0x10" type="pci" />
</address>
</source>

to:

<source>
<address bus="0x1a" domain="0x0000" function="0x0" slot="0x10" type="pci" />
</source>

Change-Id: I694153e7468986bacb19ba3e09e761993aad7184
Signed-off-by: Dino Simeon Madarang <dinox.madarang@intel.com>
Reviewed-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
Reviewed-by: Ross Brattain <ross.b.brattain@intel.com>
Reviewed-by: Alain Jebara <alain.jebara@intel.com>
Reviewed-by: Emma Foley <emma.l.foley@intel.com>

Merge "bugfix:Incorrect title for testcase 081"

Merge "Update Dockerfile.aarch64.patch"

update docs for procedure after host reboot

Added instructions to restart nginx and uwsgi after the host is rebooted.
Added instructions to start yardstick container if the host is rebooted.
Removed section on virtual python environment as it is not needed.

JIRA: YARDSTICK-919

Change-Id: I6e8bd1d52954c2aa138ce7879211b91b3644ecb2
Signed-off-by: John O Loughlin <john.oloughlin@intel.com>

Proposing Jack Chan as a committer in Yardstick

Jack Chan has contributed to the Yardstick project in many ways
Achievements/Contributions include:
- Kubernetes context
- GUI
- Yardstick API
- Reporting fixes
- Attending OPNFV Summit Beijing
- Regularly attending Yardstick meetings

Change-Id: Ie46a99619365893f3b877822f82f626cf7a36deb
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>

bugfix:Incorrect title for testcase 081

JIRA: YARDSTICK-915

fix the wrongly numbered testcase title

Change-Id: I61f94b66e3f9dac655b5c71646524b19bae7307c
Signed-off-by: JingLu5 <lvjing5@huawei.com>

Merge "Testcase 084 documentation"

Do not hide unit tests execution fail

While moving tests from ./tests/ to ./yardstick/tests, both unit tests
directories are executed. If the first test run fails, do not hide
the error and exit immediately.

Change-Id: Ib6c64ccf2f4b19722722eb43b3d45d5ad5f0169d
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Add "img_property" when building Yardstick image

Since [1], when "build_yardstick_image.yml" is executed, a variable called
"img_property" must be set. This variable could be "normal" or "nsb",
depending on the type of image to generate.

[1] I9fdf30084511ca9a008df3b53887b06273bbd549

JIRA: YARDSTICK-921

Change-Id: Ie3ca6fcf76b94eb9604847aefa77c6a644dddd14
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Change CivetWeb download URL

Changed CivetWeb download URL to [1].

[1] https://iweb.dl.sourceforge.net/project/civetweb/1.9.1/CivetWeb_V1.9.1.zip

JIRA: YARDSTICK-920

Change-Id: Ida014a57bd5b87d26c6cb8f5d36d43060f08026d
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Use assertIn(x, y) instead of other variations

unittest.assertIn(x,y) should be used instead of:
* unittest.assertTrue(x in y)
* unittest.assertEqual(x in y, True)

Also fixes pep8 violations existing in the file.

Change-Id: I705c1bd4af74757b5c928995894aede436d66817
JIRA: YARDSTICK-901
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Update Dockerfile.aarch64.patch

The docker image was not built for aarch64 due to recent changes in the
Dockerfile. The updates to the Dockerfile.aarch64.patch are:
- update docker version to 16.04
- install dependencies for pyopenssl==17.5.0 (libssl-dev and libffi-dev)

JIRA: ARMBAND-353

Change-Id: I915172d4e7ce21b1f5de044b20301b076cb6fe20
Signed-off-by: Cristina Pauna <cristina.pauna@enea.com>

Move tests: unit/dispatcher

* Fix pylint errors
* Add TODOs

Some errors are ignored locally, as they were a symptom of other problems.
These issues have been flagged with a TODO, and should be fixed later.

JIRA: YARDSTICK-837
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
Change-Id: I1f8a6041424c4c69f46ef30bd5c66d2d51d24347

Move tests: unit/common

* Fix pylint errors
* Add TODOs

Some errors are ignored locally, as they were a symptom of other problems.
These issues have been flagged with a TODO, and should be fixed later.

JIRA: YARDSTICK-837
Change-Id: I2847c380556985fae2c92bb8a7da9c47e91e1338
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

pylint: try to change formatting for Jenkins again

looks like we have to add var to tox passenv

Change-Id: Iba2da2123ae531e026897308769aa832c7eb6d04
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>

Merge "Move tests: functional/"

Merge "Move tests: unit/apiserver/"

Move tests: unit/apiserver/

* Fix pylint errors
* Add TODOs

Some errors are ignored locally, as they were a symptom of other problems.
These issues have been flagged with a TODO, and should be fixed later.

JIRA: YARDSTICK-837
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
Change-Id: I30d3f61e5ea479758f9a2f39cf415da18e49b9d6

Move tests: unit/test_ssh.py

Change-Id: I0eabdd5d7e918dcf0df747119a104dedf125606f
Jira: YARDSTICK-837
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Move tests: functional/

All tests in tests/functional have been moved to yardstick/tests/functional

JIRA: YARDSTICK-837
Change-Id: I89276b36635de74dd7b8f70e961c71aa0d2c494e
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

pylint: make Jenkins output parseable

Jenkins doesn't handle the color and the output is hard to read in
Jenkins logs

Change-Id: Ie8fc5a78e89f72c829e4bbbcbd1291a723dfde72
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>

Merge "Run tests in yardstick/tests/"

Run tests in yardstick/tests/

* Edit .testr.conf and tools/run_tests.sh
  * Run functional and unit tests in tests/ and yardstick/tests/
* Add yardstick/tests/unit/__init__.py and yardstick/tests/__init__.py
* Update pylintrc rules to disable:
  * missing-docstring
  * fixme
  * protected-access

Test will run in both tests/ and yardstick/tests/ initially

Change-Id: Iccd0df0d1b85b5d08f7faa56669bc32687925f7e
JIRA: YARDSTICK-837
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Merge "pyOpenSSL missing: cause 'git review' failed"

pyOpenSSL missing: cause 'git review' failed

JIRA: YARDSTICK-905

In the latest Yardstick container, when use git review to upload patches, an Attribute Error will cause failure.

root@a80081a4dccd:/home/opnfv/repos/yardstick# git review
Traceback (most recent call last):
  File "/usr/bin/git-review", line 6, in <module>
    from git_review.cmd import main
  File "/usr/lib/python2.7/dist-packages/git_review/cmd.py", line 32, in <module>
    import requests
  File "/usr/local/lib/python2.7/dist-packages/requests/_init_.py", line 52, in <module>
    from .packages.urllib3.contrib import pyopenssl
  File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/contrib/pyopenssl.py", line 46, in <module>
    import OpenSSL.SSL
  File "/usr/lib/python2.7/dist-packages/OpenSSL/_init_.py", line 8, in <module>
    from OpenSSL import rand, crypto, SSL
  File "/usr/lib/python2.7/dist-packages/OpenSSL/SSL.py", line 118, in <module>
    SSL_ST_INIT = _lib.SSL_ST_INIT
AttributeError: 'module' object has no attribute 'SSL_ST_INIT'

pyOpenSSL is required.

Change-Id: I88a4916c9c2dfcdee45320f6b1a37eaae0803671
Signed-off-by: JingLu5 <lvjing5@huawei.com>

Merge "Bump requirements to match OpenStack Pike release"

Merge "Remove unnecessary function definition in IxLoadTrafficGen"

Testcase 084 documentation

JIRA: YARDSTICK-906

TC084 is to evaluate the IaaS compute performance by using SPEC CPU 2006 benchmark

Change-Id: I628edf1d73e12d9f18eaf9609fa862ca456a1c71
Signed-off-by: JingLu5 <lvjing5@huawei.com>

Merge "Rename test/unit/cmd directory"

Remove unnecessary function definition in IxLoadTrafficGen

Removed call to 'instantiate' base class in IxLoadTrafficGen.

JIRA: YARDSTICK-894

Change-Id: I0ebee1f5e0f22cfd3a67eaac8accc67ecd35ad0c
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Merge "ansible image build improvement"

Merge "delete obsoleted zte pod info"

Merge "SPEC cpu2006 test case for VM"

delete obsoleted zte pod info

Delete zte pod info in yardstick/etc/yardstick/nodes.

Change-Id: Ie22fd9fa4b84529f0b2a7d113776d3559106e67a
Signed-off-by: zhihui wu <wu.zhihui1@zte.com.cn>

Remove tool provisioning in PROX helper

Tool provisioning in PROX setup environment helper is not needed [1]. The
tool (PROX traffic injector) is already provided during the VNF building:
  ./ansible/nsb_setup.yml -->
  ./ansible/build_yardstick_image.yml -->
  ./ansible/ubuntu_server_cloudimg_modify_samplevnfs.yml -->
  ./ansible/roles/install_samplevnf

[1]https://github.com/opnfv/yardstick/blob/master/yardstick/network_services/vnf_generic/vnf/prox_helpers.py#L641

JIRA: YARDSTICK-872

Change-Id: I0f925a7967a35a97901fbe5053793a791a7b1b01
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Make GenericVNF an abstract class

GenericVNF class is now an abstract class. Only optional methods are
implemented. Mandatory methods:
  - instantiate
  - wait_for_instantiate
  - terminate
  - scale
  - collect_kpi.

JIRA: YARDSTICK-866

Change-Id: Ia8766f9f98816e11894d1e72b0f3bd573d091d99
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Remove 'inconsistent-return-statements' check from Pylint

There are several bugs in Pylint [1] [2] affecting
'inconsistent-return-statements' check. It will be temporarily
removed.

[1] https://github.com/PyCQA/pylint/issues/1782
[2] https://github.com/PyCQA/pylint/issues/1794

JIRA: YARDSTICK-911

Change-Id: Ib655ef1befdc734c646cdfb9b48f1db0ccdf676d
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>

Merge "Fix inventory.ini for build_yardstick_image"

Merge "Adding vFW RFC2544 and ixload test cases with various packet sizes"

ansible image build improvement

JIRA: YARDSTICK-879

Add some image build vars
Add automatic image build of nsb
yardstick_image: ansible-playbook -e img_property="normal" -vvv -i inventory.ini build_yardstick_image.yml

nsb_image_build: ansible-playbook -e img_property="nsb" -vvv -i inventory.ini build_yardstick_image.yml

prepare for yardstick image uploading

Change-Id: I9fdf30084511ca9a008df3b53887b06273bbd549
Signed-off-by: Ace Lee <liyin11@huawei.com>

SPEC cpu2006 test case for VM

JIRA: YARDSTICK-906

This test case is for benchmarking VM(VNF) compute performance using SPEC CPU2006.

Change-Id: Ida87aa4d394e38b398acb0af3176dd1b4c858c35
Signed-off-by: JingLu5 <lvjing5@huawei.com>

Fix inventory.ini for build_yardstick_image

In [1], 'no hosts matched' occured because 'jumphost' was not defined
in inventory.ini.

[1]: https://build.opnfv.org/ci/job/yardstick-daisy-baremetal-daily-master/72/console

Change-Id: I8cb36bf083ce589e94f5f8b115e99556f0bcb053
Signed-off-by: Alex Yang <yangyang1@zte.com.cn>

Add get function in ssh.py

JIRA: YARDSTICK-904

This task is about to implement a get function in ssh.py to
fetch desired files/folder in a remote VM to a local path.

In some test cases, test tools will produce results in files.
We want retain the original result files and archive them in
Yardstick docker for future usage.

Change-Id: Ifd333eb044ce31cffa6f5b2a8f6a46648a8858d1
Signed-off-by: JingLu5 <lvjing5@huawei.com>

Merge "Replace subprocess "check_output" with "Popen""

Merge changes from topics 'YARDSTICK-875', 'YARDSTICK-874'

* changes:
  Check if multiverse repository is available in Ubuntu
  Specify supported architectures for Ubuntu backports repository