gerrit.opnfv Code Review - armband.git/log

Makefile: Trim OPNFV_PRODNO to fit in ISO volid

genisoimage has a limit of 32 chars for volume ID, which was
reached during Jenkins builds (which sets REVSTATE to a datestring),
but not while building outside Jenkins (in which case REVSTATE was
simply set to "P000"), hence the recent broken builds despite
manully testing the changes beforehand.

Shorten the product name (PRODNO var), and revert git SHA to full
format, as that was not the culprit.

Related-bug: ARMBAND-53

Change-Id: I0a530982893af1b6a0243b55f6b1d2eea66ac5bf
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

BUILD_ID: Fix git SHA one last time.

Previous commit which was supposed to fix the BUILD_ID being too
long mistakenly altered the submodule code for computing the
short SHA, instead of altering the Makefile.

Fixes: ARMBAND-53

Change-Id: Ib1c1b3dc9592714fd86a59ae1a64001bd2930bd3
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Merge "BUILD_ID: Fix build ID too long err"

Updated armband
Project: fuel 6d01c223e8bab2f5bab4941d12a1f09c5d5123b0

Add templating support for generating config files

- Remove unneeded method
- Write result to a file and not STDOUT
- Add documentation
- Remove trailing whitespace
- Documentation corrections

Change-Id: I7532222d3512380c4f1129bd05dc2ba37b409dc2
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>

BUILD_ID: Fix build ID too long err

Previous change altered the BUILD_ID to include Armband commit info
instead of Fuel@OPNFV info, and also a marker to indicate it comes
from Armband repo.

This lead to the ISO volume ID becoming too long:

"Making ISO...
genisoimage: Volume ID string too long
make[1]: *** [iso] Error 255"

While at it, change our marker from "FUEL_AARCH64" to "ARMBAND_FUEL",
as our ISO should also support x86 in the future, not only arm64.

Fixes: ARMBAND-53

Change-Id: I52526f1f7861aca71d16d1e6b6b2a9566a6c7d3f
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: vswitchperf 891488bb5e0047c7062198a9241511c7fcedeb6f

docs: Remove trailing whitespaces

Wrong whitespace characters breaks automatic generation of VSPERF
documentation. It is necessary to remove wrong characters before
merge of the draft into VSPERF doc, e.g. by filtering through
tr -d '\014'

JIRA: VSPERF-324

Change-Id: Ie280f5a443c89d4235cae5eb89fdceae74ea4213
Signed-off-by: Martin Klozik <martinx.klozik@intel.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Al Morton <acmorton@att.com>
Reviewed-by: Christian Trautman <ctrautma@redhat.com>
Reviewed-by: <bmichalo@redhat.com>

Updated armband
Project: yardstick 6ce3a97548e95a62cf39f37c76bf5b618f5575c4

Bug fix in SFC test case

It was importing novaclientv2 and it should import version 1

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

Updated armband
Project: yardstick 8be72d7d2e27554b8c640946ff79ed9d70c80e51

Add test case description and task file for TC069

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

Rebase: update patch context after upstream change

Change-Id: Ia268e17a7a8b0c2c47209e47feb6f77394036283
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Rebase: Trim changes for obsoleted mirror patch

Upstream change [1] removed Czech mirror patch.

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

Change-Id: Id986899aa83b3cb7d228a266b1b8a8f5a942ba49
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: fuel b89e203b875737b0f251bf13ceb300b90e14b14f

Pinning down exact Fuel 9.0.1 repo versions

stable/mitaka branch is open for new commits for Fuel 9.1. To not be
impacted by new upstream changes point to specific commits which were
used for Fuel 9.0 General Availabilit (9.0.1).

JIRA: https://jira.opnfv.org/browse/FUEL-153
verify: no-cache

Change-Id: I7fd8d2f11e4be6f47e39924913c48bfe7d4f2eb3
Signed-off-by: Michal Skalski <mskalski@mirantis.com>

Merge "BUILD_ID: Fix incomplete git build info"

Updated armband
Project: yardstick 0c6ec33f372cfafb55aa2df12511fb11848a69e4

Fix a bug for node context

Change-Id: Idff42e95f5ade160c00adc13c158dc1746e8d049
Signed-off-by: Songnon <songnon.zhang@ericsson.com>

Updated armband
Project: fuel a64cb984707ca6259c4f4641fbfa6f6ef85fd88d

Temporary fix for bgpvpn plugin builds

JIRA: FUEL-154

Change-Id: Iad554aef8db9a6d01c408c0014dabf45e4c1a0e6
Signed-off-by: Michal Skalski <mskalski@mirantis.com>

Updated armband
Project: fuel 67e958e90a412db2054390a049d1e482d6cd8b1f

Update the commit ID of kvmfornfv upstream for fuel-plugin-kvm

This version of kvmfornfv upstream fixed the issue of failing to launch
instances in Horizon after OPNFV environment deployed.

Change-Id: I05fa9f977924ee679312c599ee0bccabaf7bf1a9
Signed-off-by: davidjchou <david.j.chou@intel.com>

Updated armband
Project: vswitchperf 5109fb49432be5bb0075b6b55e8b5304fa6172be

pkt_gen: Moongen class name not consistent with name in conf files

Need to change naming to 'Moongen' for consistent naming
between conf files and class

JIRA: VSPERF-323

Change-Id: Ic580b68323a759b3c2ef7ca3c07006243a3acf7d
Signed-off-by: bmichalo <bmichalo@redhat.com>

Updated armband
Project: vswitchperf 3f54fe3750eef81f5fb44b90ab6a53e60e63f870

ci: CI report generation

Script for generation of documentation has been moved from releng
into new opnfvdocs repository. VSPERF script for a daily CI job is
calling this script for overall PDF report generation. Thus it must
be updated to use new script location.

JIRA: VSPERF-324

Change-Id: I173a2139c57c39acb25c14f7a082e0ff0a888b2f
Signed-off-by: Martin Klozik <martinx.klozik@intel.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Al Morton <acmorton@att.com>
Reviewed-by: Christian Trautman <ctrautma@redhat.com>
Reviewed-by: <bmichalo@redhat.com>

Updated armband
Project: fuel 0d028383c5b5df7b73683bcdbd7062ef893b6c3f

Build new cache when list of packages has changed

Create a new cache of f_repobuild job when list of packages required by
plugins has changed.

JIRA: FUEL-150

Change-Id: I079950eaa9e3b32ea7baa3f6d4a37cc37f99f3a7
Signed-off-by: Michal Skalski <mskalski@mirantis.com>

Updated armband
Project: yardstick ebb9bc33134a90ea707f163950487056dc64e729

add support for suite to support constraints and task_args

suite.yaml has constraint and task_args parameter, so
each task_case can be able to add constaints (currently
support pod_name and installer), also support parameters
to be replace for special test case like tc043.

JIRA: YARDSTICK-299

Change-Id: I2b2c8f9858893da03aaa8bcac20a474a2d434509
Signed-off-by: rexlee8776 <limingjiang@huawei.com>

Updated armband
Project: yardstick e14fbf021bdf90dc160685c23b3cbbb320e9cfbf

Modify ComputeCapacity scenario host key name

Adjust target node key name.

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

Updated armband
Project: vswitchperf 380a6cd33d7b1d85bbc46f5c90532ed55c7aaf27

conf/integration: Fix typo on Integration testcase example.

This patch fixes a typo in the integration testcase example,
otherwise settings like
{"guest_loopback" : "linux_bridge"}
are ignored.

JIRA: ONP-6635

Change-Id: Iab61d36e70f96f4a6c1f98a506753dc7fa090140
Signed-off-by: Antonio Fischetti <antonio.fischetti@intel.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Martin Klozik <martinx.klozik@intel.com>

Updated armband
Project: vswitchperf bd8bb26c67d696aba7b30bf4eb1616a7258b6f5d

ovs-appctl fix: Fixes ovs-appctl by placing pid file appropriately

ovs-appctl was broken due to a previous commit which changed the
location of the ovs-vswitchd pid file. This fixes it by placing
the pid back in the default location where ovs-appctl is looking
for.

JIRA: VSPERF-315

Change-Id: I4ae87ad566acb69ac9d565a3a77865f5a6861b93
Signed-off-by: MartinX Klozik <martinx.klozik@intel.com>
Signed-off-by: Christian Trautman <ctrautma@redhat.com>

Updated armband
Project: vswitchperf 34a550ed0c852d4472b17e9ee565f20d344052bc

New IETF Draft WG version

JIRA: VSPERF-??

Change-Id: I1f07a70bf3c8604df890defd8493f107550bf8f3
Signed-off-by: Al Morton <acmorton@att.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Billy O'Mahony<billy.o.mahony@intel.com>

Updated armband
Project: vswitchperf 568524a40ab6fa4d570c65c6c42cd68c02c9dd22

ovs/ofctl: Fix validation method for complex flows.

The current validation method for 'add_flow' commands
does not work properly with complex flows, like UDP flows.
It always returns that the flow was not set, even though
the add_flow command was successful.

JIRA: ONP-6579

Change-Id: Iaad77551326ff495cb68a033e1303450a99386bd
Signed-off-by: Antonio Fischetti <antonio.fischetti@intel.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Martin Klozik <martinx.klozik@intel.com>

Merge "fuel-library: Fix virtlogd not started"

Merge "deploy: Delete old Fuel env if present"

Merge "fuel-astute: Erase EFI boot entry on EFI systems"

Merge "deploy.sh: Make submodules-clean"

Updated armband
Project: yardstick 1e6ca0bfaaeadff7ce8ac9b1693b03d122e58a79

Minor fixes in fuel plugin
1. update yardstick tasks
2. fix typo
3. update comments in pre_build_hook

Change-Id: Ib7fc87838d121dc35fb84220fbb0a9909c6592e1
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Updated armband
Project: yardstick 1f17dacf4548d8a939b134ab573108d15432c162

Bug fix and extended unit tests

There was a bug in how the mock was created and a new unit test is added
in order to cover the "else" of the if conditions

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

Updated armband
Project: yardstick cbb35b49da2d8d491ded038a525e4107b37fae74

code clean up

JIRA: YARDSTICK-272

Change-Id: Id2f1b3d1beff8843700cdfcaaa1e8f496f445ed9
Signed-off-by: tjuyinkanglin <14_ykl@tongji.edu.cn>
revise opnfv_yardstick_tc046.rst

JIRA:YARDSTICK-292

Change-Id: I0212768d56d661edf1a41209a155c7b4dcdf892b
Signed-off-by: tjuyinkanglin <14_ykl@tongji.edu.cn>

BUILD_ID: Fix incomplete git build info

Closes: ARMBAND-53

Change-Id: I92c1541a22a837c281d9b86f6ee8da327305884d
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

deploy.sh: Make submodules-clean

Allow starting ci/deploy.sh multiple times from the same repository.

Change-Id: Ibc9d766640a611ce1d56454096c9f750da956c79
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

deploy: Delete old Fuel env if present

In order to clean up old partitioning and boot entries on target
nodes, before removing the Fuel Master, try ssh-ing into it and
removing all environments/nodes.

This is especially important for EFI systems, where old boot
entries are left behind without a proper env delete, leading to
target nodes failing to PXE boot on Fuel Master re-install.

This change assumes that:
- all Fuel Master information is unchanged between old and new
deploy (fuel IP, password etc.);
- Fuel Master is up and running, in a sane state (target nodes
are also online), when deploy starts;

Closes: ARMBAND-51

Change-Id: I52b853b37a0d226a9cb33179f7aeb65eff74da4e
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

fuel-astute: Erase EFI boot entry on EFI systems

On EFI-enabled systemd, grub-install from grub-efi-* package
installs a boot entry named "ubuntu".

Since this boot entry is saved in board flash memory, erasing
the MBR bootloader code and/or partition signature will not
clear the ubuntu boot entry, leaving it pointing to a loader
on the ESP (EFI System Partition) that will try to load the
kernel/initrd from an erased partition.

In Fuel 8.0, the whole disk was erased, so the ubuntu EFI boot
entry was skipped due to missing ESP, while for Fuel 9.0 and
above we have to explicitly remove it to keep the system in a
sane state.

Note: efibootmgr is installed automatically on EFI systems
as a dependency of grub-efi-*.

Closes: ARMBAND-47

Change-Id: If715e960dfd96df807abb0e26f4b7604624ac408
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

submodule: Add fuel-astute.

Change-Id: I6362dbdfabe9f3fa0d371b3ad3a6c2c6d139b684

Updated armband
Project: fuel  264623809055c5ade26fc615d1255302f0b7c305

Update fuel yardstick plugin
  1. update fuel plugin builder
  2. change install directory in yardstick plugin
  3. update README

Change-Id: I8d717ef568aeaa7958cd1a3161d4ef8ca17a1de1
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Updated armband
Project: yardstick 3a3ccbd79d47333ce7626674c68a4cdfac44020d

change ci from base-on-pod to base-on-scenario (in progress)

1. first need to change in yardstick
2. then change in releng to modify the input parameter

JIRA: YARDSTICK-299

Change-Id: I3d707721acbc1d63be34da4e98138f845dc93466
Signed-off-by: rexlee8776 <limingjiang@huawei.com>

Updated armband
Project: yardstick 428c72a39bde85d29fae024e1f3d38f5a177f292

throughput between nodes (in progress)

JIRA: YARDSTICK-256

Change-Id: I48d66081dcfabf4462ef1ff15c9a3ad28132aaf9
Signed-off-by: rexlee8776 <limingjiang@huawei.com>

Revert "Bugfix backport: sysctl.conf vrouter namespace"

Merged upstream.

This reverts commit 31087865b416f0eab066377c76a0d580d3fd8bd1.

Change-Id: I60bd902207bb35e244e0e10646226d9f611bd68a

fuel-library: Fix virtlogd not started

libvirt >= 1.3.0 installs additional services, including virtlogd,
which need to be started explicitly after install, as the Ubuntu UCA
package is currently broken.

Fuel code already carries a fix for this, but only for UCA libvirt.
Refactor the condition for the existing fix to check package version
instead of origin.

Closes-bug: https://jira.opnfv.org/browse/ARMBAND-37

Change-Id: I651ab48072ddcbfb88ce78a66c25cb524e442981
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Bugfix backport: sysctl.conf vrouter namespace

Upstream bug report [1], proposed change backported from [2].
Temporary backport proposed fix until fuel-library merges it.

Currently net.ipv4.ip_nonlocal_bind is not set inside vrouter
namespace.

This can lead to deployment failure with p_dns error:

ocf-ns_dns: INFO: dnsmasq daemon is not running
ocf-ns_dns: ERROR: dnsmasq: failed to create listening socket for
10.109.5.1: Cannot assign requested address
ocf-ns_dns: ERROR: Error. dnsmasq daemon returned error 0.

We had very similar issue in the past:
https://bugs.launchpad.net/fuel/+bug/1500871

We need to add
ocf_run $RUN_IN_NS /sbin/sysctl -w net.ipv4.ip_nonlocal_bind=1

To get_ns() in ns_dns and ns_vrouter

[1] https://bugs.launchpad.net/fuel/+bug/1595957
[2] https://review.openstack.org/#/c/334415/

Change-Id: Id8d3a2e58de4dccc9a79536bd443294e085f70ff
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: vswitchperf f35edfc71ee92d67e6cb587b116b64c3a005ac71

dpdk: Support of DPDK16.07

DPDK 16.07 changed the default log level of DPDK enabled applications.
Detection of the end of OVS initialization procedure has been changed
to work for various DPDK versions and for both DPDK and Vanilla OVS.

JIRA: VSPERF-316

Signed-off-by: Martin Klozik <martinx.klozik@intel.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Al Morton <acmorton@att.com>
Reviewed-by: Christian Trautman <ctrautma@redhat.com>
Reviewed-by: <bmichalo@redhat.com>
Reviewed-by: <sridhar.rao@spirent.com>

Updated armband
Project: fuel dfda6c2155a647bd78467c6cc8926c5d9c3a3234

Add fuel collectd ceilometer plugin

Change-Id: I2f95d9166e2d0d7508551de8b87dc6c84800e3c1
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Updated armband
Project: yardstick e6277570188829687b2ecf48adbd10bc3c9d46bd

Add test case description and task file for TC049

JIRA: YARDSTICK-295

Change-Id: I9c13bee1bb986f515f4bc41e4455176240e2f19d
Signed-off-by: tjuyinkanglin <14_ykl@tongji.edu.cn>

Updated armband
Project: yardstick 086bd65535027be14a23b4296539900b0220c1b6

Add test case description and task file for TC048

JIRA: YARDSTICK-294

Change-Id: Ie6a9271a4cbc81bce47f7f59e609b90a9de1e909
Signed-off-by: tjuyinkanglin <14_ykl@tongji.edu.cn>

Updated armband
Project: yardstick bd67f539aa7ad9a659b7d945942e1eeda96c45d7

Add test case description and task file for TC047

JIRA: YARDSTICK-293

Change-Id: I23a0e78a297cdc5742fa8d1234e6f271e076f359
Signed-off-by: tjuyinkanglin <14_ykl@tongji.edu.cn>

Updated armband
Project: yardstick 0f1b872b3bbff6e0f2c8d56f3037064206dd068d

Add test case description and task file for TC046

JIRA: YARDSTICK-292

Change-Id: I552c8357bd9378bea70a12872497d09711c91eef
Signed-off-by: tjuyinkanglin <14_ykl@tongji.edu.cn>

Updated armband
Project: yardstick e416cdd3339ca283216e7af9a162641534360e52

bugfix:fix all generic test cases in grafana

update the pod name and scenario name, and extend the duration of graph
JIRA:-

Change-Id: Ic611dc8d49e666433c2191c4098d56d95d40f1af
Signed-off-by: kubi <jean.gaoliang@huawei.com>

fuel@opnfv: honor pod's interfaces/transforms

Change-Id: I493279d81b32a58b918f2b6caa0cf869a0caa674
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>

Updated armband
Project: vswitchperf 8494dca647761592d7dd822849e56546f107dcb1

Update fuel plugin builder installation

Change-Id: Ife16738ea8717749c52e950404257936994fe273
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Updated armband
Project: vswitchperf 555c74d4aff580b8115d6e79ee985af9a986ff94

docs: Update yardstick userguide

Documentation update based on review comments received after
the merge of original patch.

JIRA: VSPERF-288

Change-Id: I86305fa064cc07e2e0797276a7b89b1d4b23d126
Signed-off-by: Martin Klozik <martinx.klozik@intel.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Al Morton <acmorton@att.com>
Reviewed-by: Christian Trautman <ctrautma@redhat.com>
Reviewed-by: <bmichalo@redhat.com>
Reviewed-by: <sridhar.rao@spirent.com>

Updated armband
Project: fuel 3647eb632066358736b55cd79a799e1a20a8e38e

Update NSH patches

1. update patches from commit 307b06d

Change-Id: Ibaba101833a934840b4491925ff944f1420fce2e
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

leveldb: Use odl_package var for ODL pkg version.

Change-Id: I220b364d2aecc5e08b3c90c451acf26e5d8ea0d7
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: yardstick 55e2e867e9a3f01b39f619251253470948e250d4

Update README in fuel plugin

Change-Id: Ice5e394ebdd734b10833b90e84427601fec173ad
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Updated armband
Project: fuel  d4ae056501ed0778a19beb6f207c00439f95cff3

Aligning machine type over Ubuntu and Centos

We have previously used "pc-1.0" as the qemu-kvm machine definition
for virtual Fuel deploys. On Ubuntu this points to:

pc-1.0               Standard PC (i440FX + PIIX, 1996)

On CentOS 7 this machine definition does not exist, but there's a
match in "pc":

pc                   RHEL 7.0.0 PC (i440FX + PIIX, 1996) (alias of
pc-i440fx-rhel7.0.0

So, finally, pc is also in Ubuntu:

pc                   Ubuntu 14.04 PC (i440FX + PIIX, 1996) (alias of
pc-i440fx-trusty)

This change is about exploring whether moving to "pc" will make the
auto deploy work on both platforms with the "pc" machine type.

What still needs to be done in CentOS is to create a symlink from
/usr/libexec/qemu-kvm and /usr/bin/kvm though.

Change-Id: I1cce52a21075950cb59b75709b12d724d759e3e6
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>

Merge "Fuel VM: set virtio cache to writeback"

Merge "iptables: Move SSH rules to post-scripts."

Fuel VM: set virtio cache to writeback

Change-Id: Icbbba0bee68a8fe9de76dbf3c339e01da43fb3a2
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>

iptables: Move SSH rules to post-scripts.

Previous change [1] introduced the addition of an iptables rule
that was supposed to allow SSH access on all ifaces (not only
admin iface) when additional ifaces are configured.

However, Fuel installer is flushing the rules after transplant
adds our SSH config, overwriting it.

Move iptables SSH config to post-install section, as standalone
script. In order to keep the same behavior, test ifcfg-eth0
(admin interface is expected to be called eth0 by convention)
for "DEFROUTE=no" and only whitelist SSH on all ifaces if it matches.

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

Change-Id: I086b75461daa62671cad10494fe34acfd77757ae
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

ODL wget: --no-check-certificate for local cache.

Change-Id: Idbc45b4d1aeddf5450b756d23faf56c95fccc975

Merge "transplant.py: Allow SSH on all interfaces."

dea_base: use grub-efi-arm64 instead of grub-pc.

Change-Id: Iae462ebe89ff3bc351addd12a8f5cc5ecdef2b11
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>

Updated armband
Project: yardstick cd9d1296ab33b0193bfd37e77241291065d9a06b

Update fuel plugin

1. update fuel plugin builder
2. change install directory in yardstick plugin

Change-Id: Ice560955bb659ca86476b9dd02b3028b853ac01c
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Updated armband
Project: vswitchperf dd47e4e2ea55f01235c20bed8e07d69fb51c292e

Enable BURST_MODE for l2fwd

JIRA: VSPERF-267

This change adds optional burst mode to l2fwd module that makes use of
skb->xmit_more API available in Linux Kernel > 3.18 in order to batch
transmission of packets out of the NIC, increasing forwarding performance
significantly. By default burst mode is disabled. If a value greater than
1 is provided, burst mode is enabled to send that many packets at once.
Typical values would be burst=8 or burst=16.

Change-Id: I8ef5f86cf73d4cb5a8e4c618a86111ebf411dca8
Signed-off-by: Mesut Ali Ergin <mesut.a.ergin@intel.com>
Signed-off-by: Mallesh Koujalagi <malleshx.koujalagi@intel.com>

Updated armband
Project: yardstick 567694e4db2d0776db6c115b98f54a4737e52124

Creating Director and related codes.

Add a new scenario type 'ScenarioGeneral' that support orchestrating general HA test scenarios.
Director, ActionPlayer and RollbackPlayer are uesed to execute the test scenario (or test flow).

JIRA: YARDSTICK-288

Change-Id: Ied2ccd4712f3c3efde6771bfa4538c1e9e137c11
Signed-off-by: lihuan <lihuansse@tongji.edu.cn>

Updated armband
Project: yardstick de26decf97c64b4a9a624d0acbf3882af4916f39

Bug fix:change pod name and extend graph duration

Because there were some change with community pod name.
We need to change it with new pod name so that all the result
could be shown in the grafana

JIRA:-

Change-Id: I584294a5ead4337160c1ce2fcca8a53901f03a07
Signed-off-by: kubi <jean.gaoliang@huawei.com>

Merge "patches: removed patches upstreamed to fuel@opnfv"

Updated armband
Project: fuel f7f13befab5bf9a585f4f24018fd9561ed61b1ae

[BGPVPN] Do not include temporary files in plugin

Change-Id: Idd2e2f403a2dd5be44c480e8b493e4281ade1994
Signed-off-by: Michal Skalski <mskalski@mirantis.com>

patches: removed patches upstreamed to fuel@opnfv

Change-Id: I2cd1899d82878dfece22ed546e1ff24d5387e8c1
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>

Updated armband
Project: fuel 16ea7e06170ba0bb62a164de5b33f19ccd1b1d12

deploy: ISO rebuild: use -joliet-long names.

Armband adds similar files, which tend to have long names and only
differ in suffixes (e.g. DEBs for amd64 vs arm64).

Previous change [1] fixed ISO build errors, but failed to address
the deploy.py ISO rebuild (which modifies and recreates the ISO,
calling mkisofs, which complains about clashing file indexes).

This works around ISO rebuild error(s) like:

"genisoimage: Error:
./ubuntu/pool/main/g/golang-gogoprotobuf/golang-gogoprotobuf\
-dev_0.0~git20150828.0.6cab0cc-1~u14.04+mos1_arm64.deb and
./ubuntu/pool/main/g/golang-gogoprotobuf/golang-gogoprotobuf\
-dev_0.0~git20150828.0.6cab0cc-1~u14.04+mos1_amd64.deb
have the same Joliet name"

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

Change-Id: If58a18ba46343a52ce6e9a0a6a0482c1a3079451
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: fuel  7c600840185d9b454c1995e35a9704db823fb5c6

virtual_fuel: prepare class to allow multiple bridges

The VirtualFuel class has now two new methods:

    del_vm_nics: Deletes all interfaces from the VM.

    add_vm_nic: Adds a NIC to the VM, attached to the specified bridge.

The following method has been deleted:

    set_vm_nic: implemented with the two new methods

Apart from the deleted method, no functionality has been changed.

This is just a small but necessary step towards adding support for
configuring more than one NIC in the fuel VM.

Change-Id: I9f02c8163dfb9768510e78d5d5e77a0bb43306fb
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
deploy: add support for multiple bridges

deploy.py:
Some Fuel VMs may need more than one network interface. To be able to
provide that, we now allow the user to specify the "-b" paramter
(bridge) multiple times, creating a new NIC for each one of them.

The NICs are created in the same order as they are given in the command
line.

There is no change in behavior from earlier versions, pxebr will still
be the default bridge if none is specified in the command line.

deploy.sh:
To reflect the new capabilities of deploy.py, we introduce the
possibility to specify -B more than once in deploy.sh, and honor that
when calling deploy.py. We also make it possible to specify a comma
separated list of bridges, as in: -B br1,br2.

Change-Id: I1a0100f2cfe755ec6adfeedafb391c2357f46f51
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>

Updated armband
Project: vswitchperf 64823f173b4fae6921bfde642fb68eac55a470ea

yardstick: Documentation and YAML testcase definitions

Yardstick supports execution of vsperf specific testcases.
These testcases are specified by YAML files located
inside yardstick/tests directory. Documentation describing
yardstick usage and execution of vsperf testcases is
part of the userguide.

JIRA: VSPERF-288

Change-Id: I5a575d28d344e0f3e0a1c8434e2f3231fd312d31
Signed-off-by: Martin Klozik <martinx.klozik@intel.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Al Morton <acmorton@att.com>
Reviewed-by: Christian Trautman <ctrautma@redhat.com>
Reviewed-by: <bmichalo@redhat.com>
Reviewed-by: <sridhar.rao@spirent.com>

Updated armband
Project: yardstick f5acf5d40684080f1633a3c4eafebfe5e6ab69b1

bug fix for huawei-pod2

Since TC027 depends on specific pod configure,
so it only should be run at huawei-pod1

JIRA:-

Change-Id: Ib85daae64ce97cad907dbaa8309c80cf4f0153b5
Signed-off-by: kubi <jean.gaoliang@huawei.com>

Updated armband
Project: fuel f59b4e0786f11e2799990beeaf0e4a148c3572af

Point to newer commit in ODL plugin repo

Use small fix for setting ovsdb managers.

Change-Id: Ie405d4a2db59ea490ff0a690bb310c0c74d8de79
Signed-off-by: Michal Skalski <mskalski@mirantis.com>

Updated armband
Project: yardstick 55af4f473d60838b57b14670423717b7f2e901ea

Enhence CPUload scenario

Enhence CPUload to measure maximum, minimum and average CPU usage.

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

Updated armband
Project: yardstick 65d32056b14f5f34c2aba186f7ccb3e34c67a9a5

Creating result checker classes

JIRA: YARDSTICK-276

Change-Id: I63c4f2c36108e95f5d3b7da42e66cb8c9b16c817
Signed-off-by: lihuan <lihuansse@tongji.edu.cn>

Updated armband
Project: yardstick 7b2923df78a0820df9e968ae1909d838c07ebd83

vsperf: Initial integration with VSPERF

New scenario class allows to execute vsperf installed inside VM.
Vsperf is executed in trafficgen mode. It means, it will only
configure and execute traffic by external HW traffic generator
based on custom configuration file. After traffic generator stops,
then test results will be converted from vsperf CSV format into
JSON and passed to yardstick for further processing.
Currently, traffic is passed only through external bridge. In
the future, test scenarios will launch VNFs and traffic will
be properly routed to them. Proper routing can be ensured
by OVS flows configuration via setup-script executed during
setup phase.
Testcase definition yaml files inside vsperf/ directory won't be
pushed to the yardstick, but they will be stored inside vsperf
repository. They are part of the draft review only to show
how it is done.

JIRA: VSPERF-288

Change-Id: I13a519ed39091fe89d1a43cc522738044fb3c609
Signed-off-by: Martin Klozik <martinx.klozik@intel.com>
Reviewed-by: <sunshine.wang@huawei.com>
Reviewed-by: <lvjing5@huawei.com>
Reviewed-by: <jean.gaoliang@huawei.com>
Reviewed-by: <david.j.chou@intel.com>

Updated armband
Project: vswitchperf 7cded121ab8291d742ea9d5324f93cae34dc10e5

Spirent-TestCenter: Code Cleanup.

Removal of redunant code in testcenter.py.

No logic Changes.
Cosmetic changes - removing redundant code of 150+ lines.

JIRA: VSPERF-308

Change-Id: I4766890cdc9f3d23103b2d2090285361ac4a5197
Signed-off-by: Sridhar K. N. Rao <sridhar.rao@spirent.com>

Updated armband
Project: vswitchperf 7a16aa0bc7875b5b3fe6385cf99afa076e1e1cb3

rstp-stp: Add basic functions for stp/rstp enable on ovs

Add basic functions to enable/disable spanning tree
protocols on the bridge.

Also adds bridge info function to retrieve other bridge
information.

JIRA: VSPERF-314

Change-Id: Ic72c5a2a9d16aab1b95428ce37042a5b536481aa
Signed-off-by: Christian Trautman <ctrautma@redhat.com>

Updated armband
Project: vswitchperf cead9a3d0571b8afc5ed680475e9bc3ea92c7d79

Namespace_veth: Add funtionality for network namespace, veth ports

Adds functionality for network namespaces and veth/peer ports
to connect the namespaces. The namespaces are tracked outside
of default Linux namespaces to prevent possible disruption of
work outside of VSPerf where a user may create a namespace and
the cleanup code deletes it. The cleanup code only removes
namespaces and veth ports created inside the testcase.

JIRA: VSPERF-310

Change-Id: If8881cafb119f38f052403a1de497e9660187d2e
Signed-off-by: Christian Trautman <ctrautma@redhat.com>

Updated armband
Project: yardstick 3cf1c2162d800e4f5a5596bdf54732ba906c95f6

Add Yardstick TC044 TEST CASE

JIRA:yardstick-290

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

Updated armband
Project: yardstick 609e6ea459afc1aff901ecb1cd38c403ef8f8c13

Add test case description and task file for TC045

JIRA: YARDSTICK-291

Change-Id: I85081c075fb795ee7839009952e49ad13b9cf2bf
Signed-off-by: tjuyinkanglin <14_ykl@tongji.edu.cn>

Updated armband
Project: yardstick 4d78a2c02653f4918b55e0becad9094518e1afc3

Add fuel test suite

This file is used for verifing yardstick in fuel integration.

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

transplant.py: Allow SSH on all interfaces.

Previously (in Fuel 8.0), SSH used to listen only on 10.20.0.2
(admin interface), which required editing sshd_config and restarting
SSH server for allowing SSH connections over the public IP on eth1
(just an example, which corresponds to Armband PODs use cases).

In Fuel 9.0, SSH server on Fuel Master listens on all ifaces,
but connections are filtered by iptables.

This change piggy-backs on a previous Armband addition that allows
transplant.py script to configure additional interfaces (e.g. public).

In case additional interfaces have been configured by transplant,
this change will instruct iptables to accept SSH connections on ANY
interface, not only the admin one.

Possible improvements:
- limit SSH access to admin + interfaces added by transplant instead
of ANY;

Change-Id: I0923496e1b23f6dc826c8afbbe9805956c2d4b34
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Temporary: HACK: SR-IOV: Fix VNIC filtering.

FIXME: Revert this when upstream Fuel 9.1 closes bug [1].

[1] https://bugs.launchpad.net/fuel/+bug/1597775

Change-Id: I692b0ef63ee3e035deadef9d4e47b43f4c53e4aa
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: yardstick e112b517bdca5f685dd250b9b0a7f6a18f4fb6f8

Add zte-pod1 info in Yardstick test cases Grafana page.

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

Updated armband
Project: yardstick 991aaea442d2a36d43871224a7d96a7e7eea6910

fix the ssh bug when run tc043 in CI

Change-Id: I93ffe038ee128976db001e4f2ece79366aa3826c
Signed-off-by: rexlee8776 <limingjiang@huawei.com>

Backport: dpkg-checkbuilddeps to mk-build-deps.

For some reason, our docker sandbox builder already has dh-systemd
when checking for "fuel-agent" build-deps, and therefore skips
adding it as a build dep inside the chroot, leading to debuild fail.

Backporte upstream fix for this from [1].

[1] https://review.openstack.org/#/c/325210/

Change-Id: Ieece9b2250bd91e688bd0f203240da5234930558
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: fuel fdef1b47e7c2fe894b12e1d0976cc101c09939a1

Point to latest commit in bgpvpn plugin repo

Currently we use commit before uplift to mitaka.

Change-Id: Ie08a548b133aff01cc2457bd74d0fe7a86a18eff
Signed-off-by: Michal Skalski <mskalski@mirantis.com>

Updated armband
Project: yardstick eacef205efa9289bccbf38d82df07cdc63486d35

Creating a generic opertion

Operation class is used to do some work on the target system such as creating a VM instance.

JIRA: YARDSTICK-275

Change-Id: Ib62846824b74dcdae51f143bc59fba385cc7d84c
Signed-off-by: lihuan <lihuansse@tongji.edu.cn>

Updated armband
Project: yardstick 51b902b9d9d7a641357dd615dcd5a5529bffc8bb

Add Yardstick TC004 TEST CASE

JIRA:yardstick-14

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

Merge "[BGPVPN] Enable/add plugin build to ISO."

Merge "ODL: Rework opnfv-quagga building/packaging."

[BGPVPN] Enable/add plugin build to ISO.

Change-Id: I4d01e9f9aeea3b0ab5bcc7fbe6840b8960fff5d9

Updated armband
Project: fuel 50ebdb5d193ad7085ac1ba906622ae4cd0dd8f3c

[BGPVPN] fpm: Build multiarch DEBs (arch <all>)

Update upstream reference to include support for building DEB
packages for architecture <all> instead of the native one, since
apart from one small hardcoded check on Fedora for java version,
most code is arch-independant (python).

TODO:
Extend check for `uname -m` result for aarch64 and other platforms
in <networking_odl/devstack/setup_java.sh>.

While at it, make git info variable overrideable, just in case
we want to patch this out of upstream tree locally.

Change-Id: Ia139c8d957f65bb120dca1727fd4a422e2510ea0
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: fuel d1466b0f8792bde0a1e39d2071a03d0317e0f7ee

Fix ODL deployments

Change-Id: Idae02b340d3a64109deeda7ef67574430b928219
Signed-off-by: Michal Skalski <mskalski@mirantis.com>

ODL: Rework opnfv-quagga building/packaging.

My initial goal was to split the ODL patch in two patches:
- leveldb + leveldbjni arm64 support;
- opnfv-quagga building/packing for multiple archs (UBUNTU_ARCH);

However, due to the design of opnfv-packaging repo, it is easier to:
- patch upstream sources to support native arm64 builds (see [1]);
- add prebuilt arm64 binaries to output dir in git repo (like amd64),
submitted as pull request upstream [2];

Until above OPNFV-QUAGGA changes land upstream or are refactored,
rely on forked repository that contains above patches [3] on
branch "stable/colorado".

opnfv-quagga patch adding UBUNTU_ARCH parsing is more or less there
to detect missing DEBs, rather than filter prebuilt DEBs, which for
now only cover "amd64 arm64".

[1] https://github.com/nikolas-hermanns/opnfv-quagga-packaging/pull/1
[2] https://github.com/nikolas-hermanns/opnfv-quagga-packaging/pull/2
[3] https://github.com/alexandruavadanii/opnfv-quagga-packaging

Change-Id: I022c1e82531d82422486f2041c70a3f50b6ca386
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Updated armband
Project: yardstick 12f7ee47f39091be7b139f9bc083dbc675099937

SFC Yardstick test

Modifications of the SFC Yardstick test

The test creates two chains. One chain blocks HTTP the other one blocks SSH.
We doublecheck that HTTP works in one but not in the other and the same for
SSH.

There are some things that must be modified manually as ODL is not yet ready
for ovs 2.5.90. Here are the instructions:

https://wiki.opnfv.org/display/sfc/Yardstick

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