yardstick.git
6 years agoMerge "Bugfix in role enable_hugepages_on_boot"
Ross Brattain [Fri, 22 Dec 2017 11:42:15 +0000 (11:42 +0000)]
Merge "Bugfix in role enable_hugepages_on_boot"

6 years agoMerge "Bugfix:proxy env, ansible multinode support"
Ross Brattain [Fri, 22 Dec 2017 10:52:13 +0000 (10:52 +0000)]
Merge "Bugfix:proxy env, ansible multinode support"

6 years agoMerge "Remove "flake8" and "pep8" from test-requirements"
Ross Brattain [Fri, 22 Dec 2017 10:48:33 +0000 (10:48 +0000)]
Merge "Remove "flake8" and "pep8" from test-requirements"

6 years agoMerge "Variable local_iface_name is read before it is set"
Ross Brattain [Fri, 22 Dec 2017 10:46:36 +0000 (10:46 +0000)]
Merge "Variable local_iface_name is read before it is set"

6 years agoBugfix in role enable_hugepages_on_boot 15/47615/10
Jan Malanik [Tue, 21 Nov 2017 19:05:07 +0000 (11:05 -0800)]
Bugfix in role enable_hugepages_on_boot

Make role idempotent.
Current regex doesn't match for:
linux   /boot/vmlinuz-4.10.0-30-generic root=/dev/nvme0n1p1 ro vga=788 text

Change-Id: If5eb20fce4ddc35bb109c00eb7565853b3763950
Signed-off-by: Jan Malanik <janx.malanik@intel.com>
Signed-off-by: Malanik Jan <janx.malanik@intel.com>
6 years agoMerge "Fix the pylint version for pep8 tox target"
Ross Brattain [Fri, 22 Dec 2017 10:40:49 +0000 (10:40 +0000)]
Merge "Fix the pylint version for pep8 tox target"

6 years agoAdd doc about installing YardStick on OpenSUSE 61/46961/5
rossella [Wed, 8 Nov 2017 17:33:38 +0000 (18:33 +0100)]
Add doc about installing YardStick on OpenSUSE

Add guide to install YardStick on OpenSUSE and modify the README
to state that OpenSUSE is supported.

Change-Id: Id107b24e0ec1ab441426bb150d9e863986d20fc1
Signed-off-by: Rossella Sblendido <rsblendido@suse.com>
6 years agoFix the pylint version for pep8 tox target 73/49273/2
Martin Banszel [Tue, 19 Dec 2017 09:48:11 +0000 (10:48 +0100)]
Fix the pylint version for pep8 tox target

Change-Id: I5e1c169a6ca13b3f46129b2c4a15ccbab2677a63
Signed-off-by: Martin Banszel <martinx.banszel@intel.com>
6 years agoVariable local_iface_name is read before it is set 83/47583/6
Jiri Prokes [Mon, 20 Nov 2017 12:09:11 +0000 (04:09 -0800)]
Variable local_iface_name is read before it is set

Moving variable setting before reading
Verified via TC:
yardstick/samples/vnf_samples/nsut/ping/tc_ping_heat_context.yaml

JIRA: YARDSTICK-899

Change-Id: Ia2feac1ed4e67dccd02446ba27afc9d40e87be35
Signed-off-by: Jiri Prokes <jirix.x.prokes@intel.com>
6 years agoMerge "pretty cli format: runner/scenario/testcase list"
Rex Lee [Mon, 18 Dec 2017 01:33:02 +0000 (01:33 +0000)]
Merge "pretty cli format: runner/scenario/testcase list"

6 years agoupdate release notes for Euphrates 5.1.0 31/49131/1
Ross Brattain [Sat, 16 Dec 2017 10:18:31 +0000 (02:18 -0800)]
update release notes for Euphrates 5.1.0

Change-Id: I11b248882dd314baebd3ab4e7e676a1f3e0b50cd
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoMerge "Addition of NSB Prox test documentation of vPE and LW-AFTR test cases"
Ross Brattain [Sat, 16 Dec 2017 09:39:25 +0000 (09:39 +0000)]
Merge "Addition of NSB Prox test documentation of vPE and LW-AFTR test cases"

6 years agobugfix: tc078 have no data stored or dashboard to show results 49/47749/4
rexlee8776 [Fri, 24 Nov 2017 07:37:20 +0000 (07:37 +0000)]
bugfix: tc078 have no data stored or dashboard to show results

JIRA: YARDSTICK-867

1.print result in the log, showing the txt format result

Change-Id: I6deeb4cf44895ef7a6c6d231c77b3843eec285d7
Signed-off-by: rexlee8776 <limingjiang@huawei.com>
6 years agoMerge "bugfix: tc078 fails in some situations"
Ross Brattain [Sat, 16 Dec 2017 09:21:57 +0000 (09:21 +0000)]
Merge "bugfix: tc078 fails in some situations"

6 years agoMerge "bugfix: kill process do not accurately kill "nova-api""
Ross Brattain [Sat, 16 Dec 2017 08:44:07 +0000 (08:44 +0000)]
Merge "bugfix: kill process do not accurately kill "nova-api""

6 years agosamples: Add generic L3 forwarder tests 95/44295/16
Dino Madarang [Tue, 3 Oct 2017 01:02:17 +0000 (01:02 +0000)]
samples: Add generic L3 forwarder tests

A generic throughput test case that can be used as a
stub code for a Linux-based VNF configured as an L3 forwarder.

Supported context:
* Standalone OVSDPDK and SRIOV
* Baremetal

Code changes:
* Allow pmd-cpu-mask and lcore mask for OVS DPDK
* router_vnf.py - configures interface IP addresses and static arp entries
  using ip command
* NFVi KPIs
* Allow cputune tag for standalone context to be able to PIN on NUMA 1 cpus

SRIOV Test cases:
* RFC2544 Ethernet framesizes, 128K Flows
* 2,4 and 6 ports
* 2 and 3 vcpus per port
*
OVSDPDK Test cases:
* RFC2544 Ethernet framesizes, 128K Flows
* 2 and 4 ports
* 2 vcpus per port
* 2 PMD threads per port

TODO:
* Documentation
* Add 6 ports tests

References:
* router_vnf.py is based on sample_vnf.py
* tc_*.yaml files are based on acl/vfw test case files

Added unitests
Added get_stats to parse ip -s link

Change-Id: Id1b969d5420dfcab7c1e695acbd2cd1655747efe
Signed-off-by: Dino Simeon Madarang <dinox.madarang@intel.com>
Signed-off-by: Kavindya Deegala <kavindya.s.deegala@intel.com>
Reviewed-by: Alain Jebara <alain.jebara@intel.com>
Reviewed-by: Deepak S <deepak.s@linux.intel.com>
Reviewed-by: Emma Foley <emma.l.foley@intel.com>
Reviewed-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
Reviewed-by: Ross Brattain <ross.b.brattain@intel.com>
Reviewed-by: Edward MacGillivray <edward.s.macgillivray@intel.com>
Signed-off-by: Dhaval Patel <dhaval.r.patel@intel.com>
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agopretty cli format: runner/scenario/testcase list 73/47473/13
rexlee8776 [Mon, 20 Nov 2017 03:28:31 +0000 (03:28 +0000)]
pretty cli format: runner/scenario/testcase list

JIRA: YARDSTICK-855

1. using prettytable to reformat cli below:
yardstick runner list
yardstick scenario list
yardstick testcase list
2. remove redundant function print_hbar
3. fix pep8 problems, the framework using do_list dynamicly,
so disable unused-argument check for now.

Change-Id: I79e1226e21fca4624bf4436414917bd07ecabca0
Signed-off-by: rexlee8776 <limingjiang@huawei.com>
6 years agoNSB: Fix standalone.model.Libvirt SR-IOV modeling 31/46531/10
Dino Madarang [Wed, 1 Nov 2017 16:10:16 +0000 (09:10 -0700)]
NSB: Fix standalone.model.Libvirt SR-IOV modeling

Fixed standalone.model.Libvirt SR-IOV XML interface modeling, acording
to [1]:
- All PCI attributes now are printed in hexadecimal format.
- The PCI address is now added in the correct section, 'interface'.

network_services.utils.PciAddress was refactored to accept both 'domain:
bus:slot:function' and 'bus:slot:function' format inputs. This class is
used as input in the previous class, Libvirt, to print in XML the PCI
address of a SR-IOV interface.

network_services.utils.PciAddress.parse_address is now deprecated. Instead
the class standard instantiation must be used:

    libvirt_obj = utils.PciAddress(text_with_address)

A deprecation decorator is implemented along with this patch. This
decorator is used for the first time in the previously mentioned function.
This decorator stores every decorated function name and deprecation message
and raises a logging warning message the first time this function is used.

[1] https://goo.gl/so2Mrp

Change-Id: I22e95c488e27d6e2a8fdf6c1a07faab275fa6bba
Signed-off-by: Dino Simeon Madarang <dinox.madarang@intel.com>
Reviewed-by: Alain Jebara <alain.jebara@intel.com>
Reviewed-by: Deepak S <deepak.s@linux.intel.com>
Reviewed-by: Ross Brattain <ross.b.brattain@intel.com>
Reviewed-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agobugfix: kill process do not accurately kill "nova-api" 93/47793/6
rexlee8776 [Sat, 25 Nov 2017 06:44:33 +0000 (06:44 +0000)]
bugfix: kill process do not accurately kill "nova-api"

JIRA: YARDSTICK-849

1. update the kill process RegEx pattern to be more accurate
2. make attack_process to be parameter so it can be customized if
the SUT is using a different name of attack_process

Change-Id: I569730ced6c24aafbffcf2fc5752d3560d0adac5
Signed-off-by: rexlee8776 <limingjiang@huawei.com>
6 years agoBugfix:proxy env, ansible multinode support 57/47957/14
Malanik Jan [Tue, 28 Nov 2017 16:22:17 +0000 (08:22 -0800)]
Bugfix:proxy env, ansible multinode support

JIRA: YARDSTICK-893

- hosts and plays in ansible/nsb_setup.yml refactored
Fixed proxy configuration from env or /etc/environment in nsb_setup.sh
Fixed ansible parameter 'environment: "{{ proxy_env }}"' in:
 - roles for components download.
 - roles for components dependencies install.

Now, you can use ansible to modify all yardstick-standalone hosts at once.
1) Configure http[s]_proxy in
- /etc/environment
or
- env variables
2) Modify ansible/yardstick-install-inventory.ini:
  [jumphost]
  localhost  ansible_connection=local

  [yardstick-standalone]
  yardstick-standalone-node ansible_host=192.168.1.2
  yardstick-standalone-node-2 ansible_host=192.168.1.3

  [all:vars]
  ansible_user=root
  ansible_pass=root

3) start:
./nsb_setup.sh # for baremetal/sriov
./nsb_setup.sh <path to admin-openrc.sh> #for openstack heat

Change-Id: I167db8bae8303210f21e8ae6f3870ef054bbedce
Signed-off-by: Malanik Jan <janx.malanik@intel.com>
6 years agoMerge "bugfix: tc006, tc079, tc082 miss grafana dashboard in local deployment"
Ross Brattain [Thu, 14 Dec 2017 08:39:26 +0000 (08:39 +0000)]
Merge "bugfix: tc006, tc079, tc082 miss grafana dashboard in local deployment"

6 years agoMerge "Refactor userguide "Yardstick Installation""
Ross Brattain [Thu, 14 Dec 2017 08:35:56 +0000 (08:35 +0000)]
Merge "Refactor userguide "Yardstick Installation""

6 years agoChange testcases 'cirros' image to 'yardstick-image' 25/48025/2
Rodolfo Alonso Hernandez [Wed, 29 Nov 2017 16:21:12 +0000 (16:21 +0000)]
Change testcases 'cirros' image to 'yardstick-image'

The default OpenStack 'cirros-0.3.5' image doesn't exist anymore. The
image included by default in any testing OpenStack deployment is now
called 'cirros-0.3.5-x86_64-disk', but this name can change.

Because Yardstick, during the installation, deploys a customized image,
'yardstick-image', it makes sense to use it as the default image.

JIRA: YARDSTICK-873

Change-Id: I8fd7cbc01d4fa52e9107f2fac7d2d9ff5d1091c7
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoMerge "Ansible: fix lowercasing issue with ConfigParser"
Ross Brattain [Thu, 14 Dec 2017 07:54:55 +0000 (07:54 +0000)]
Merge "Ansible: fix lowercasing issue with ConfigParser"

6 years agoRemove "flake8" and "pep8" from test-requirements 31/48731/3
Rodolfo Alonso Hernandez [Tue, 12 Dec 2017 13:35:58 +0000 (13:35 +0000)]
Remove "flake8" and "pep8" from test-requirements

Removed "flake8" and "pep8" from test-requirements. Since
44070670c798d2cde2824f3150b2b11bf35c6319, "pylint" is the
code style checker, instead of "flake8" or "pep8".

JIRA: YARDSTICK-832

Change-Id: Iee598b0b1793d71dc74be36fa1875387b17d1e25
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agobugfix: tc078 fails in some situations 17/47317/7
rexlee8776 [Thu, 16 Nov 2017 08:49:16 +0000 (08:49 +0000)]
bugfix: tc078 fails in some situations

1. make  "SPECint_benchmark, runspec_iterations, runspec_tune, runspec_size,
runspec_rate" parameterized and change the default value "int^429" to "int"
2. remove "become: true" in some roles since it does not need root privilege

JIRA: YARDSTICK-852

Change-Id: Icb384bddc12911e2681a981d0504e0e142d1a8ec
Signed-off-by: rexlee8776 <limingjiang@huawei.com>
6 years agoMerge "NSB: Fix IndexError in pin_vcpu_for_perf"
Ross Brattain [Tue, 12 Dec 2017 16:01:12 +0000 (16:01 +0000)]
Merge "NSB: Fix IndexError in pin_vcpu_for_perf"

6 years agoAvoid pylint warning for "protected-access" 75/48475/5
Rodolfo Alonso Hernandez [Wed, 6 Dec 2017 16:52:36 +0000 (16:52 +0000)]
Avoid pylint warning for "protected-access"

This warning should be disabled in order to access protected
methods during testing. Any access to protected methods done
outside the testing classes, must be catched during the reviewing
process.

Change-Id: I2127b41edf704d0cb4f68b33d98705039e058d1a
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoAvoid pylint warning for "missing-docstring" 63/48463/5
Rodolfo Alonso Hernandez [Wed, 6 Dec 2017 16:23:35 +0000 (16:23 +0000)]
Avoid pylint warning for "missing-docstring"

Disabled pylint message for missing doctring in methods, classes and
modules. Modules must be auto-explanatory, and some methods are trivial
an doesn't need an explanation.

Change-Id: I01d80737d92a7345bc6a2a6f607eb35cbf28da67
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoMerge "Adding vFirewall HTTP Traffic Profiles for tests including Concurrency, CPS...
Ross Brattain [Tue, 12 Dec 2017 07:19:11 +0000 (07:19 +0000)]
Merge "Adding vFirewall HTTP Traffic Profiles for tests including Concurrency, CPS, TPUT, TPS for 2Ports and 4Ports"

6 years agoAdding vFirewall HTTP Traffic Profiles. 59/45259/5
Phani Kiran Thaticharla [Mon, 16 Oct 2017 17:39:22 +0000 (10:39 -0700)]
Adding vFirewall HTTP Traffic Profiles.

Modified http_locator location for 64K, 256K, 512K and 1024K
concurrency files.

The traffic profiles include various http locator image sizes i.e
4K,64K,256K,512K and 1024K

The traffic schema is changed to "nsb" from "isb" for all the
traffic profiles

Change-Id: I012da8daaa880ca80db2d3b08cebd9a73c19cd47
Signed-off-by: Phani Kiran Thaticharla <phani.kiran.thaticharla@intel.com>
Reviewed-by: Oscar Medina-Duran <oscar.medina-duran@intel.com>
Reviewed-by: Deepak S <deepak.s@linux.intel.com>
Reviewed-by: Ross Brattain <ross.b.brattain@intel.com> 
6 years agoheat: allow overriding Heat/Neutron private IP for DPDK tests 93/45393/12
Ross Brattain [Tue, 17 Oct 2017 21:30:37 +0000 (14:30 -0700)]
heat: allow overriding Heat/Neutron private IP for DPDK tests

For some L2/L3 DPDK testcases we need to use a custom
IP address space different from what Heat provides.

These testcases require port_security_enabled = False so
Neutron should allow for unrestricted L2 traffic.

This will work because we bind the ports to DPDK and thus
don't need DHCP.

  vnf_0:
    floating_ip: true
    placement: "pgrp1"
    network_ports:
      mgmt:
        - mgmt
      uplink_0:
        - xe0:
            local_ip: 10.44.0.20
            netmask: 255.255.255.0
      downlink_0:
        - xe1:
            local_ip: 10.44.0.30
            netmask: 255.255.255.0

Also fixup flake8 errors in unittests

Change-Id: Id29dfffa692f16fb1f526d208db43e476e2f7830
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoAdding vFirewall HTTP Traffic Profiles for tests including Concurrency, CPS, TPUT... 45/48345/1
Phani Kiran Thaticharla [Mon, 4 Dec 2017 22:02:10 +0000 (14:02 -0800)]
Adding vFirewall HTTP Traffic Profiles for tests including Concurrency, CPS, TPUT, TPS for 2Ports and 4Ports

Change-Id: I3c7f047dc6d1545093aed6f64ccaed9f90d85af3
Signed-off-by: Phani Kiran Thaticharla <phani.kiran.thaticharla@intel.com>
Reviewed-by: Oscar Medina-Duran <oscar.medina-duran@intel.com>
Reviewed-by: Deepak S <deepak.s@linux.intel.com>
Reviewed-by: Ross Brattain <ross.b.brattain@intel.com> 
6 years agoMerge "bugfix: tc023 miss description and tc050-58 wrong description"
Ross Brattain [Fri, 1 Dec 2017 19:18:02 +0000 (19:18 +0000)]
Merge "bugfix: tc023 miss description and tc050-58 wrong description"

6 years agoMerge "KVMFORNFV: Reverting LiveMigration changes"
Jing Lu [Fri, 1 Dec 2017 05:40:23 +0000 (05:40 +0000)]
Merge "KVMFORNFV: Reverting LiveMigration changes"

6 years agoKVMFORNFV: Reverting LiveMigration changes 29/48129/2
Navya Bathula [Thu, 30 Nov 2017 21:46:15 +0000 (21:46 +0000)]
KVMFORNFV: Reverting LiveMigration changes

This patch consists of reverting the changes of patch 45227 and
incudes redirecting the console output of the LiveMigration
execution to /dev/null as the stdout contains only the statistics,
i.e., totaltime, downtime and setuptime.

This reverts commit 5a1f65d3e7d67488ee6f558dccfa5ca5581ddb65.

Change-Id: I252b5a4045657cfa8362e9aae755249480cd3b77
Signed-off-by: Navya <navyax.bathula@intel.com>
6 years agoMerge "Remove img_modify_playbook assignation in build_yardstick_image.yml"
Ross Brattain [Wed, 29 Nov 2017 11:51:15 +0000 (11:51 +0000)]
Merge "Remove img_modify_playbook assignation in build_yardstick_image.yml"

6 years agoMerge "Move test requirements into test-requirements.txt"
Ross Brattain [Wed, 29 Nov 2017 11:50:53 +0000 (11:50 +0000)]
Merge "Move test requirements into test-requirements.txt"

6 years agoMerge "Move test scripts to tools/"
Ross Brattain [Wed, 29 Nov 2017 11:50:30 +0000 (11:50 +0000)]
Merge "Move test scripts to tools/"

6 years agoMerge "tox: Split tests into separate tox environments"
Ross Brattain [Wed, 29 Nov 2017 11:50:07 +0000 (11:50 +0000)]
Merge "tox: Split tests into separate tox environments"

6 years agoMerge "Tox: add a pep8 target"
Ross Brattain [Wed, 29 Nov 2017 11:50:04 +0000 (11:50 +0000)]
Merge "Tox: add a pep8 target"

6 years agoRemove img_modify_playbook assignation in build_yardstick_image.yml 35/47935/1
Rodolfo Alonso Hernandez [Tue, 28 Nov 2017 14:02:34 +0000 (14:02 +0000)]
Remove img_modify_playbook assignation in build_yardstick_image.yml

Variable "img_modify_playbook" is being assigned in first context of
./ansible/build_yardstick_image.yml [1]. However this variable is never used.
In second context [2], the variable used is a CLI input variable.

[1]https://github.com/opnfv/yardstick/blob/stable/euphrates/ansible/build_yardstick_image.yml#L187
[2]https://github.com/opnfv/yardstick/blob/stable/euphrates/ansible/build_yardstick_image.yml#L205

JIRA: YARDSTICK-871

Change-Id: I7bd5063c5e86520abcafdc782222ee53faf58f45
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoAddition of NSB Prox test documentation of vPE and LW-AFTR test cases 89/47589/2
DanielMartinBuckley [Tue, 21 Nov 2017 15:35:31 +0000 (15:35 +0000)]
Addition of NSB Prox test documentation of vPE and LW-AFTR test cases

YARDSTICK-840

NSB Prox test documentation of vPE and LW-AFTR test cases documenation.

Change-Id: Ic6da6546faf0494fdee71962a515083e6e10a410
Signed-off-by: Daniel Martin Buckley <daniel.m.buckley@intel.com>
6 years agofix docker client don't have trex_client 03/47503/1
Ace Lee [Mon, 20 Nov 2017 10:47:19 +0000 (10:47 +0000)]
fix docker client don't have trex_client

JIRA: YARDSTICK-854

Add trex_client in docker,
so we could run NSB testcase in docker container.

Change-Id: Iac1393d6e3a175137973190936ab69a1f2ce8272
Signed-off-by: Ace Lee <liyin11@huawei.com>
6 years agoMerge "update docker version to 16.04"
Ross Brattain [Mon, 20 Nov 2017 08:57:01 +0000 (08:57 +0000)]
Merge "update docker version to 16.04"

6 years agoNSB: Fix IndexError in pin_vcpu_for_perf 85/45885/3
Dino Madarang [Thu, 19 Oct 2017 20:41:05 +0000 (20:41 +0000)]
NSB: Fix IndexError in pin_vcpu_for_perf

This addresses the IndexError in Standalone SRIOV/OVSDPDK context.
The correct sys_cpu number should be sys_cpu-1

Error:
    threads = "%s-%s" % (soc_cpu[socket][sys_cpu], soc_cpu[socket][-1])
IndexError: list index out of range

Tested platform: (Hyper-Threading disabled)
NUMA node0 CPU(s):     0-21
NUMA node1 CPU(s):     22-43

With socket equal to 0, sys_cpu value was 22 which resulted to IndexError.

Change-Id: I36b045cacce4caff4b5ddafaa9b6737f1c6df0a1
Signed-off-by: Dino Simeon Madarang <dinox.madarang@intel.com>
Reviewed-by: Alain Jebara <alain.jebara@intel.com>
Reviewed-by: Deepak S <deepak.s@linux.intel.com>
Reviewed-by: Ross Brattain <ross.b.brattain@intel.com>
Reviewed-by: Edward MacGillivray <edward.s.macgillivray@intel.com>
6 years agobugfix: tc023 miss description and tc050-58 wrong description 39/47239/4
rexlee8776 [Wed, 15 Nov 2017 09:40:21 +0000 (09:40 +0000)]
bugfix: tc023 miss description and tc050-58 wrong description

also modify tc019,025,045-49 descrition to be unified

JIRA: YARDSTICK-850

Change-Id: I3f775f1ebff4063edc53b9bc26d7ba10b8af8a9d
Signed-off-by: rexlee8776 <limingjiang@huawei.com>
6 years agoMove test requirements into test-requirements.txt 33/47133/6
Emma Foley [Tue, 7 Nov 2017 22:52:42 +0000 (22:52 +0000)]
Move test requirements into test-requirements.txt

Move the packages required for testing into test-requirements.txt
This follows conventions used in OpenStack projects.

JIRA: YARDSTICK-837
Change-Id: If08fc416aa21b11250066bc19a54daa35e863407
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
6 years agoMove test scripts to tools/ 75/46875/6
Emma Foley [Tue, 7 Nov 2017 21:04:08 +0000 (21:04 +0000)]
Move test scripts to tools/

Move the scripts that tox uses for running tests
to the tools/ directory:
* run_tests.sh -> tools/run_tests.sh
* tests/ci/cover.{sh,awk} -> tools/

JIRA: YARDSTICK-837
Change-Id: I817c9e5f58c06b45f1de3f369318f4bb24168677
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
6 years agotox: Split tests into separate tox environments 73/46873/6
Emma Foley [Tue, 7 Nov 2017 20:29:03 +0000 (20:29 +0000)]
tox: Split tests into separate tox environments

* Add argument parsing to run_tests.sh so tests can be selected
* Define tox environments for:
  * Coverage
  * Unit tests
  * Functional tests
    * for py27 and py3
* Add environments to default env
  * Behaviour is the same when the tests are run
* Updates the py27 environment config, and adds a py3 config.
  * Commands from testenv are moved to py27
  * py3 env uses the same commands, but runs with python3

Change-Id: I22f17c0103101f01ec268b1de9d146780ae67a6d
JIRA: YARDSTICK-837
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
6 years agoTox: add a pep8 target 75/46575/14
Emma Foley [Thu, 2 Nov 2017 16:50:14 +0000 (16:50 +0000)]
Tox: add a pep8 target

Adds a pep8, pep8-full targets to tox:
* pep8 target: runs pylint on the changed files
* pep8-full target: runs pylint on whole repo

Adds scripts to run pylint:
* stored in tools/

Documentation: changeed the command for running tests
(run_tests.sh -> tox). This runs the same set of tests as before.

JIRA: YARDSTICK-832

Change-Id: I652da87723682d958f3fcbc2eb4cd88422636a3b
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
6 years agoMerge "Dockerfile: apt-get clean to save layer space"
Ross Brattain [Fri, 17 Nov 2017 17:16:51 +0000 (17:16 +0000)]
Merge "Dockerfile: apt-get clean to save layer space"

6 years agoMake GenericTrafficGen class abstract 39/47339/2
Rodolfo Alonso Hernandez [Thu, 16 Nov 2017 08:56:56 +0000 (08:56 +0000)]
Make GenericTrafficGen class abstract

GenericTrafficGen class is now an abstract class. Only optional methods
are implemented.
  - 'run_traffic' and 'terminate' are mandatory.
  - 'listen_traffic', 'verify_traffic' and 'wait_for_instance' are
    optional. By default these methods doesn't execute any action.

JIRA: YARDSTICK-853

Change-Id: I2befdaa337af79cc2364bdd7c66183c31c5ab69a
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoMerge "Add vfw ixload testcase for heat"
Ross Brattain [Fri, 17 Nov 2017 10:36:43 +0000 (10:36 +0000)]
Merge "Add vfw ixload testcase for heat"

6 years agoMerge "NSB: update vfw tc_heat_external tests"
Ross Brattain [Fri, 17 Nov 2017 10:34:28 +0000 (10:34 +0000)]
Merge "NSB: update vfw tc_heat_external tests"

6 years agoMerge "Check requirements against OpenStack requirements"
Ross Brattain [Fri, 17 Nov 2017 09:55:41 +0000 (09:55 +0000)]
Merge "Check requirements against OpenStack requirements"

6 years agoMerge "NSB Prox BM test case fixes for scale up"
Ross Brattain [Fri, 17 Nov 2017 09:48:39 +0000 (09:48 +0000)]
Merge "NSB Prox BM test case fixes for scale up"

6 years agoMerge "Fix controller IP parsing in "nsb_setup.sh" script"
Ross Brattain [Fri, 17 Nov 2017 09:46:46 +0000 (09:46 +0000)]
Merge "Fix controller IP parsing in "nsb_setup.sh" script"

6 years agoMerge "Clean functional tests in "test_cli_scenario""
Ross Brattain [Fri, 17 Nov 2017 09:46:00 +0000 (09:46 +0000)]
Merge "Clean functional tests in "test_cli_scenario""

6 years agoMerge "Create get_description and get_scenario_type for Scenario"
Ross Brattain [Fri, 17 Nov 2017 09:45:55 +0000 (09:45 +0000)]
Merge "Create get_description and get_scenario_type for Scenario"

6 years agoFix controller IP parsing in "nsb_setup.sh" script 11/46311/6
Rodolfo Alonso Hernandez [Fri, 27 Oct 2017 10:43:29 +0000 (11:43 +0100)]
Fix controller IP parsing in "nsb_setup.sh" script

Fixed the controller IP parsing in "nsb_setup.sh" script. The new
regex expression now accepts "OS_AUTH_URL" as:
  * http://10.237.223.168/identity
  * http://10.237.223.168:5000/identity
  * http://hostname/identity
  * http://hostname:5000/identity

JIRA: YARDSTICK-829

Change-Id: Ifaeab991bcbb6aa7b22c3533ab7554527184878b
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agobugfix: tc006, tc079, tc082 miss grafana dashboard in local deployment 01/47401/1
rexlee8776 [Fri, 17 Nov 2017 08:41:06 +0000 (08:41 +0000)]
bugfix: tc006, tc079, tc082 miss grafana dashboard in local deployment

JIRA: YARDSTICK-847

Change-Id: I0750f2d17bac9e0214caf640e4ea1141ea3b4a03
Signed-off-by: rexlee8776 <limingjiang@huawei.com>
6 years agoupdate docker version to 16.04 23/46623/6
Ace Lee [Fri, 3 Nov 2017 06:39:00 +0000 (06:39 +0000)]
update docker version to 16.04

JIRA: YARDSTICK-690

This patch is for Yardstick Docker base image upgrade to 16.04

This patch will enable ansible build image in yardstick env prepare.

Add qemu-img convert to ansible build image.

Change-Id: I43127b6020bd20bd9f4aac4fca0df75353b24346
Signed-off-by: Ace Lee <liyin11@huawei.com>
6 years agoNSB "Prox" : Cleanup duplicated traffic profile 67/47367/1
Abhijit Sinha [Thu, 16 Nov 2017 17:40:02 +0000 (17:40 +0000)]
NSB "Prox" : Cleanup duplicated traffic profile

JIRA: YARDSTICK-848

The NSB PROX MPLS test uses Binsearch traffic
profile and the mpls traffic profile is a duplicate.

Change-Id: Ie2124cebf306fd6917b70ecd7c23ae12ef4850dc
Signed-off-by: Abhijit Sinha <abhijit.sinha@intel.com>
6 years agoNSB TRex: convert DPDK port number to logical TRex port number 99/46599/10
Ross Brattain [Thu, 2 Nov 2017 23:54:56 +0000 (16:54 -0700)]
NSB TRex: convert DPDK port number to logical TRex port number

when we create TRex config we sort based on PCI bus address
and create a logical port ordering.

We need to save this port ordering and re-use it everywhere.

redirect vnfd_helper.port_num() to resource_helper.port_num() to
use the logical mapping

Change-Id: Ibff628556d5e11e686e15716a66a3210758c4ff0
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoNSB Prox BM test case fixes for scale up 31/47131/2
Abhijit Sinha [Mon, 13 Nov 2017 17:23:27 +0000 (17:23 +0000)]
NSB Prox BM test case fixes for scale up

YARDSTICK-839

In a scale-up test the VNF number can start from
0 to MAX (num_vnf-1)

The NSB PROX BM test nodes are currently marked as:
    tg__0: trafficgen_1.yardstick
    vnf__0: vnf.yardstick
They need to be changed to the below:
    tg__0: tg_0.yardstick
    vnf__0: vnf_0.yardstick

Changes done are:
- All BM Prox test cases have tg_0 and vnf_0 as default
  test nodes
- The prox-baremetal-1/2/4.yaml sample pod file
- Cleaned Up some unused Prox test/config files

Change-Id: I819011bcb35eda62b17dd3e1035918918c582b13
Signed-off-by: Abhijit Sinha <abhijit.sinha@intel.com>
6 years agoRefactor userguide "Yardstick Installation" 07/46307/6
Rodolfo Alonso Hernandez [Thu, 26 Oct 2017 16:36:39 +0000 (17:36 +0100)]
Refactor userguide "Yardstick Installation"

[1] must be refactored in order to:
  * Include proxy options.
  * Use OpenStack client, instead of project CLI.
  * Use sudo in some commands.
  * Adapt the document to rst coding style.
  * Fix minor errors.

JIRA: YARDSTICK-828

[1] https://github.com/opnfv/yardstick/blob/923aa1738cbf5a8053e77e00937fc26f2142f6a3/docs/testing/user/userguide/04-installation.rst

Change-Id: Ice63f344a8a87221bdf4daa6153b9ccac321e5b7
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoMerge "Remove network_services.vnf_generic.vnf.prox_helpers.ProxSocketHelper.rx_stats"
Ross Brattain [Fri, 10 Nov 2017 15:09:57 +0000 (15:09 +0000)]
Merge "Remove network_services.vnf_generic.vnf.prox_helpers.ProxSocketHelper.rx_stats"

6 years agoMerge "tc053 kill haproxy wrong mismatch"
Ross Brattain [Fri, 10 Nov 2017 15:06:20 +0000 (15:06 +0000)]
Merge "tc053 kill haproxy wrong mismatch"

6 years agoMerge "Heat: add availability_zone"
Ross Brattain [Fri, 10 Nov 2017 15:05:51 +0000 (15:05 +0000)]
Merge "Heat: add availability_zone"

6 years agoMerge "Update links to documentation in README.rst"
Ross Brattain [Fri, 10 Nov 2017 15:04:57 +0000 (15:04 +0000)]
Merge "Update links to documentation in README.rst"

6 years agoMerge "NSB: fix vpe_config license"
Ross Brattain [Fri, 10 Nov 2017 14:59:30 +0000 (14:59 +0000)]
Merge "NSB: fix vpe_config license"

6 years agoMerge "pmu: remove event_download_local workaround script"
Ross Brattain [Fri, 10 Nov 2017 14:58:03 +0000 (14:58 +0000)]
Merge "pmu: remove event_download_local workaround script"

6 years agoNSB: fix vpe_config license 31/47031/1
Ross Brattain [Thu, 9 Nov 2017 15:40:23 +0000 (15:40 +0000)]
NSB: fix vpe_config license

Use Apache 2.0

Change-Id: I199d09f04a5c84a25bd91dceba5e4ed1f93075f6
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoCheck requirements against OpenStack requirements 75/46975/3
Rodolfo Alonso Hernandez [Wed, 8 Nov 2017 17:55:54 +0000 (17:55 +0000)]
Check requirements against OpenStack requirements

Added a new tox environment to check the project requirements against
the OpenStack requirements. The openstack/requirements branch used will
depends on Yardstick release development.

At this point, Yardstick F release must be compliant with OpenStack Pike
release.

Change-Id: I77a4cfa4f99674176c3f5711dc1b6787bb400ecf
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoAddition of IPv6 icpm, udp and tcp to Neutron Security Group 61/46661/4
DanielMartinBuckley [Fri, 3 Nov 2017 11:28:23 +0000 (11:28 +0000)]
Addition of IPv6 icpm, udp and tcp to Neutron Security Group

When creating a Neutron security group for a heat template. Add
IPv6 security group rules to access the resouces.

Change-Id: I0b541dfe400121e35908a70778665ee1576f0622
Signed-off-by: Daniel Martin Buckley <daniel.m.buckley@intel.com>
6 years agoMerge "NSB Prox LW_AFTR Test"
Ross Brattain [Thu, 9 Nov 2017 10:18:21 +0000 (10:18 +0000)]
Merge "NSB Prox LW_AFTR Test"

6 years agoRemove network_services.vnf_generic.vnf.prox_helpers.ProxSocketHelper.rx_stats 79/46879/1
Emma Foley [Wed, 8 Nov 2017 17:48:59 +0000 (17:48 +0000)]
Remove network_services.vnf_generic.vnf.prox_helpers.ProxSocketHelper.rx_stats

* Remove the method which is unused and is marked as deprecated.

Change-Id: Ie64084fcd26985283c664445b173a757d3d908ab
JIRA: YARDSTICK-838
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
6 years agopmu: remove event_download_local workaround script 13/46813/3
Ross Brattain [Tue, 7 Nov 2017 19:49:47 +0000 (19:49 +0000)]
pmu: remove event_download_local workaround script

intel_pmu needs to download a config file based on the
CPU model.  When generating VNF images we don't have
access to the actual vCPU that will be used, so we
can't predownload.  This code was meant to be a fix
for that by downloading all the configs and then
selecting one that matched the vCPU.

However we have license issues with intel_pmu enven GPLv2 code,
so remove it for now.

Change-Id: I5257ff7c4ddc1d40537dadb29efa40d1d68cb852
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoFixed incorrect Copyright Notice 07/46807/3
DanielMartinBuckley [Tue, 7 Nov 2017 17:48:18 +0000 (17:48 +0000)]
Fixed incorrect Copyright Notice

Changed Copyright notice on tuples.lua from "Intel Copyright" to Apache 2.0

Change-Id: I85e6f9eb86c116d552f86cddf05b3b368b32f776
Signed-off-by: Daniel Martin Buckley <daniel.m.buckley@intel.com>
6 years agoClean functional tests in "test_cli_scenario" 79/46979/1
Rodolfo Alonso Hernandez [Tue, 7 Nov 2017 16:48:15 +0000 (16:48 +0000)]
Clean functional tests in "test_cli_scenario"

Changed how "test_cli_scenario" tests check the presence
of a string.

Fixed "test_scenario_show_Lmbench": logic in [1] was incorrectly
assigning an string value instead of the result of the "in"
clause.

[1] https://github.com/opnfv/yardstick/blob/stable/euphrates/tests/functional/test_cli_scenario.py#L35-L36

Change-Id: I71fceabad972e4c0d9e78cbfeb85dcd6a5ae4bda
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoCreate get_description and get_scenario_type for Scenario 77/46977/1
Rodolfo Alonso Hernandez [Tue, 7 Nov 2017 16:27:43 +0000 (16:27 +0000)]
Create get_description and get_scenario_type for Scenario

Created two new class functions for Scenario class:

* "get_description" will return the first line of the docstring,
  if exists.
* "get_scenario_type" will return the value of "__scenario_type__"
  variable.

Change-Id: I8e3b47e33e0bae101b874c3975bb9c383397d188
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
6 years agoNSB Prox LW_AFTR Test 75/44775/14
DanielMartinBuckley [Wed, 11 Oct 2017 11:05:54 +0000 (12:05 +0100)]
NSB Prox LW_AFTR Test

JIRA: YARDSTICK-802

Addition of PROX LW_AFTR basked on PROX/DATS v037 test_104_lw_aftr.py
- This support BM and Openstack Heat
- This supports 4 Ports ONLY
- Grafana Dashboards included
- Code Coverage / Unit testing

Change-Id: If2170ab458bf687256d5f1a1e840a3b9d2788ef7
Signed-off-by: Daniel MArtin Buckley <daniel.m.buckley@intel.com>
Signed-off-by: Abhijit Sinha <abhijit.sinha@intel.com>
6 years agoAnsible: fix lowercasing issue with ConfigParser 13/46613/5
Ross Brattain [Fri, 3 Nov 2017 03:21:39 +0000 (20:21 -0700)]
Ansible: fix lowercasing issue with ConfigParser

by default ConfigParser will lowercase everything,
unless you override optionxform.

also sort key value in inventory line for consistency

https://docs.python.org/3/library/configparser.html#configparser.ConfigParser.optionxform

Transforms the option name option as found in an input file or as passed in by
client code to the form that should be used in the internal structures. The
default implementation returns a lower-case version of option; subclasses may
override this or client code can set an attribute of this name on instances to
affect this behavior.

You don’t need to subclass the parser to use this method, you can also set it
on an instance, to a function that takes a string argument and returns a
string. Setting it to str, for example, would make option names case sensitive:

cfgparser = ConfigParser()
cfgparser.optionxform = str

Note that when reading configuration files, whitespace around the option names
is stripped before optionxform() is called.

YARDSTICK-833

Change-Id: Ia1810b0c77922d84e11c9e538540b38816338593
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agotest_sample_vnf: mock time 01/46601/1
Ross Brattain [Fri, 3 Nov 2017 01:12:33 +0000 (18:12 -0700)]
test_sample_vnf: mock time

Change-Id: Id8aa734fee431d90cbdc1e0eb2173784ada822fe
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoUpdate links to documentation in README.rst 83/46583/1
Emma Foley [Tue, 31 Oct 2017 15:53:05 +0000 (15:53 +0000)]
Update links to documentation in README.rst

The links to the docs in README were pointing to Colorodo documentation.

For master, this should not be pinned to a particular release, but
should show the most recent docs at all times.

Change-Id: I3186ea0dbc5105812407a0f87e02d99899a951a0
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
6 years agoHeat: add availability_zone 79/40179/7
Ross Brattain [Thu, 24 Aug 2017 22:06:50 +0000 (15:06 -0700)]
Heat: add availability_zone

Users would like to ensure placement of VMs on specific compute nodes so
that the measurements are meaningful. Examples: Measure network
performance in different scenarios (VMs in same host, in different
hosts, across fabric, across tenants)

Example:

context:
  name: yardstick
  placement_groups:
    pgrp1:
      policy: "availability"
  servers:
    tg_0:
      floating_ip: true
      placement: "pgrp1"
      availability_zone: "zone2"
    vnf_0:
      floating_ip: true
      placement: "pgrp1"
      availability_zone: "zone1"

Change-Id: I28a757c25ae3f5b3571ab3edd82d51ceba32c302
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoNSB: remove validate_cpu_cfg and all app cpu methods 49/46349/2
Ross Brattain [Fri, 27 Oct 2017 20:43:28 +0000 (13:43 -0700)]
NSB: remove validate_cpu_cfg and all app cpu methods

We don't really know what core we are going to use for the
VNF before we generate the MultiPortConfig, so these
methods are incorrect.

We may need some more advanced method to validate
the vCPU topology, but that will probably happen
in the MultiPortConfig

Change-Id: Ifee7ae4589ce0fce67771fb8d8b8dce0a0f06409
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoMerge "resource: remove cores args"
Ross Brattain [Tue, 31 Oct 2017 22:37:43 +0000 (22:37 +0000)]
Merge "resource: remove cores args"

6 years agoixia: add 1024 and 1280 byte packet size to Ixia ipv4 traffic profile 45/46145/2
Ross Brattain [Mon, 23 Oct 2017 21:07:01 +0000 (14:07 -0700)]
ixia: add 1024 and 1280 byte packet size to Ixia ipv4 traffic profile

This brings it to parity with its non-Ixia counterpart (ipv4_throughput.yaml).

Change-Id: I4c91d1ae507c7b5e0a23dcf49606b16891b400bd
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agotc053 kill haproxy wrong mismatch 97/46297/2
rexlee8776 [Fri, 27 Oct 2017 09:23:16 +0000 (17:23 +0800)]
tc053 kill haproxy wrong mismatch

JIRA: YARDSTICK-831

Change-Id: I587c486572f439952b8a31da664d1e2c1fb8dc27
Signed-off-by: rexlee8776 <limingjiang@huawei.com>
6 years agoresource: remove cores args 67/45267/6
Ross Brattain [Mon, 16 Oct 2017 21:30:13 +0000 (14:30 -0700)]
resource: remove cores args

We used to try to probe all the cores that the VNF was using
and only dump CPU stats for those cores.

We can't really detect those core accurately and we would rather
dump all the core information and let influxdb and grafana filter
the information.

We do end up with excessive KPI output, especially on systems with 88
cores, but this is manageable.

The core logic was partially removed, this finishes the removal.

Change-Id: I5cbb694fd982519e2df54db49a21ed5948e13537
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoMerge "KVMFORNFV: To avoid livemigration result update to json"
Jing Lu [Thu, 26 Oct 2017 07:50:50 +0000 (07:50 +0000)]
Merge "KVMFORNFV: To avoid livemigration result update to json"

6 years agoKVMFORNFV: To avoid livemigration result update to json 27/45227/15
Navya [Mon, 16 Oct 2017 14:26:41 +0000 (07:26 -0700)]
KVMFORNFV: To avoid livemigration result update to json

This patch is used to remove the livemigration testcase result loaded
to json as there is no dashboard implemented for the testcase

Change-Id: I7a9589a0bbc5f2a28587c2878da042fc50af18e0
Signed-off-by: Navya Bathula <navyax.bathula@intel.com>
6 years agoRemoving trailing whitespace 95/46195/1
JingLu5 [Wed, 25 Oct 2017 02:13:01 +0000 (02:13 +0000)]
Removing trailing whitespace

Change-Id: I0380deec3e41a6dc02571ed9c4fa16e304f7f85e
Signed-off-by: JingLu5 <lvjing5@huawei.com>
6 years agoAdd "How to work with test case" into devguide 05/46105/2
kubi [Mon, 23 Oct 2017 07:51:03 +0000 (15:51 +0800)]
Add "How to work with test case" into devguide

JIRA: YARDSTICK-73

add a new chapter about how to work with test case, now just add three sub chapter,

sample test case, release test case, test case description file, will update more for

how to write a new test case.

Change-Id: I44906917e76ad6d4890f345c963b9e4440f6be39
Signed-off-by: kubi <jean.gaoliang@huawei.com>
6 years agoAdd vfw ixload testcase for heat 89/45389/6
Jennifer Li [Tue, 17 Oct 2017 19:58:31 +0000 (12:58 -0700)]
Add vfw ixload testcase for heat

    - concurrency
    - connections/second
    - throughput

Change-Id: Idb977462952fba2e6a9684e916dfeca810f1ee94
Signed-off-by: Jennifer Li <jennifer.li@intel.com>
6 years agoNSB: update vfw tc_heat_external tests 47/46147/1
Ross Brattain [Mon, 23 Oct 2017 21:51:04 +0000 (14:51 -0700)]
NSB: update vfw tc_heat_external tests

rename xe0, xe1 to uplink_0, downlink_0
enumerate tgs/vnfs starting from 0.
add disable_dhcp

Change-Id: Ia4f08b3ad5ab298192c249d04b10a6efbe4f210e
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
6 years agoBugFix: Fix to pod filename ext and prox path 13/46113/1
Abhijit Sinha [Mon, 23 Oct 2017 09:27:27 +0000 (10:27 +0100)]
BugFix: Fix to pod filename ext and prox path

As per defect:
- the "context:" stanza of every tc_prox_baremetal*.yaml"
 file has an incorrect pathname for the config file

-  tc_prox_baremetal_binsearch.yaml and
tc_prox_baremetal_ramp.yaml refer to
"prox-baremetal.yml", which idoes not exist

- incorrect prox bin path

Resolution:
- Unify the pod file name and path for all BM tests
- Prox bin path changed to "/opt/nsb_bin/prox"

Change-Id: I16b25189104ad5585ddc1c2c0dd71b3c4c2aecd3
Signed-off-by: Abhijit Sinha <abhijit.sinha@intel.com>
6 years agoMerge "Fix Malformed Table in User Guide docs"
Jing Lu [Fri, 20 Oct 2017 23:42:36 +0000 (23:42 +0000)]
Merge "Fix Malformed Table in User Guide docs"