fuel.git
8 years agoMerge "Mask impitool credentials"
Jonas Bjurel [Thu, 16 Jun 2016 13:52:09 +0000 (13:52 +0000)]
Merge "Mask impitool credentials"

8 years agoMerge "Add command line argument masking for exec_cmd"
Jonas Bjurel [Thu, 16 Jun 2016 13:51:55 +0000 (13:51 +0000)]
Merge "Add command line argument masking for exec_cmd"

8 years agoMerge "Remove unused function: usage()"
Jonas Bjurel [Thu, 16 Jun 2016 13:40:35 +0000 (13:40 +0000)]
Merge "Remove unused function: usage()"

8 years agoMask impitool credentials 37/15737/2
Peter Barabas [Thu, 16 Jun 2016 11:44:17 +0000 (13:44 +0200)]
Mask impitool credentials

Change-Id: I0103082672259eaf4b1c2be8ca3a1236fc6137eb
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoAdd command line argument masking for exec_cmd 33/15733/2
Peter Barabas [Thu, 16 Jun 2016 11:19:43 +0000 (13:19 +0200)]
Add command line argument masking for exec_cmd

exec_cmd() now takes 2 additional optional arguments: mask_args and
mask_str. The former expects an array of positions to mask, the latter
expects a string to be used as mask.

Change-Id: I445141a68929a0d2837e7692ce8b4d071154cfa7
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoMerge "Introducing collection of all fuel and stack deployment logs."
Jonas Bjurel [Thu, 16 Jun 2016 10:17:04 +0000 (10:17 +0000)]
Merge "Introducing collection of all fuel and stack deployment logs."

8 years agoMerge "Clarify docker installation instructions"
Jonas Bjurel [Thu, 16 Jun 2016 10:15:04 +0000 (10:15 +0000)]
Merge "Clarify docker installation instructions"

8 years agoIntroducing collection of all fuel and stack deployment logs. 45/13945/15
Jonas Bjurel [Wed, 11 May 2016 11:38:31 +0000 (13:38 +0200)]
Introducing collection of all fuel and stack deployment logs.

The purpose of this patch is to collect all available Fuel snapshots- and
stack/node ldeployment logs for later off-line troubleshooting.
The intention is that Jenkins, or other deployment robots will be able to
collect all logs from the deployment and store it at some repository where
developers can fetch it and perform off-line post deployment trouble-shooting.
Following script arguments have been added:

CI Arg changes:
Added an argument to ci/deploy.sh:
-L [Deploy log path and file name], E.g.
   -L ~/jenkins/deploy/deploy-888.log.tar.gz
   This will create an tar gzip archive at the path and filename pointed out.
   If -L is not specified, the log archive will be placed under the CI directory
   with the following name convention: deploy-YYMMDD-HHMMSS.log.tar.gz

Fuel Internal deploy changes:
Added an argument to ci/deploy.py
-log [Deploy log path and file name], E.g.
     -log ~/jenkins/deploy/deploy-888.log.tar.gz
     This will create an tar gzip archive at the path and filename pointed out.
     If -log is not specified, the log archive will be placed under the CI
     directory with the following name convention:
     deploy-YYMMDD-HHMMSS.log.tar.gz

READY TO MERGE!
VERIFIED!

Change-Id: Icb75d9d2e66bdd47f75dcca29071943444d5c823
Signed-off-by: Jonas Bjurel <jonas.bjurel@ericsson.com>
8 years agoRemove unused function: usage() 27/15727/1
Peter Barabas [Thu, 16 Jun 2016 10:01:14 +0000 (12:01 +0200)]
Remove unused function: usage()

Change-Id: I648821262a7bfd77e59f4411380020c49306ff5e
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoClarify docker installation instructions 49/5449/4
Peter Barabas [Mon, 4 Jan 2016 08:13:33 +0000 (09:13 +0100)]
Clarify docker installation instructions

Change-Id: I8738798eac64c44a66e4fcab9758094e26a5a69f
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoAdd missing package dependency: libyaml-dev 67/5667/3
Peter Barabas [Wed, 6 Jan 2016 12:08:06 +0000 (13:08 +0100)]
Add missing package dependency: libyaml-dev

Change-Id: I058f6bc54e1c6b0a0c20eeaa2ea09f2f9a2e80ce
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoMerge "virtual_fuel: initial support for remote libvirt"
Jonas Bjurel [Wed, 15 Jun 2016 17:19:54 +0000 (17:19 +0000)]
Merge "virtual_fuel: initial support for remote libvirt"

8 years agoMerge changes from topic 'remote-libvirt-support'
Jonas Bjurel [Wed, 15 Jun 2016 17:19:37 +0000 (17:19 +0000)]
Merge changes from topic 'remote-libvirt-support'

* changes:
  virtual_fuel: make vm_template an attribute
  virtual_fuel: factor out image creation into a method
  VirtualFuel: Add temp_dir and vm_name attributes

8 years agoMerge changes from topic 'simplify-ipmi-adapter'
Jonas Bjurel [Wed, 15 Jun 2016 17:19:03 +0000 (17:19 +0000)]
Merge changes from topic 'simplify-ipmi-adapter'

* changes:
  ipmi_adapter: simplify, retry if command fails
  common.py: allow specifying number of attempts in exec_cmd

8 years agovirtual_fuel: initial support for remote libvirt 13/14713/3
Josep Puigdemont [Wed, 4 May 2016 12:27:23 +0000 (14:27 +0200)]
virtual_fuel: initial support for remote libvirt

With this patch it should be possible to create a fuel VM on a remote
libvirt server by properly defining the LIBVIRT_DEFAULT_URI [1]
environment variable. If the variable is not defined, then there should
be no percievable change in behaviour for the script.

This patch introduces the ability to create volumes (images) on a
remote libvirt host where the Fuel VM is to be deployed. For now
the volumes are created by default in a pool named jenkins, but
the idea is to allow this to be configured, probably in the POD's
DHA file.

Since all virsh commands honor LIBVIRT_DEFAULT_URI, we use this
environment variable to detect wheter we should create a volume or not.
The rationale being that the variable will only be set if the user wants
to to do the VM deployment on a remote libvirt host.

We need to create a volume because we can not rely on being able to
access the remote server's file system directly.

The images are then transferred to the libvirt host using virsh
commands. All this could also be done using scp and a user directory
on the host machine, but using pools allows us to take advantage of
libvirt's policies and file permissions.

CHANGE: when LIBVIRT_DEFAULT_URI is defined, the script will not check
for the presence of the required PXE bridge. This will still be checked
when the Fuel VM is started and the bridge not found, but this happens
at a later point than it does today.

CHANGE: before this patch, the file system image was named like the VM:
vm_name.raw. This patch introduces a change and adds a timestamp suffix
to the image: vm_name-timestamp.raw. This is so to avoid collisions with
an image with the same name on the remote pool (two PODs may be using
the same pool). It may also be useful to keep around old file system
images.

FIXME: This patch requires a pool named "jenkins" in the remote libvirt
server, and it will fail if it is not present. This should be
configurable.

Notice though that we can still define LIBVIRT_DEFAULT_URI as
"qemu:///system" to create the Fuel VM on the local host.

[1] https://libvirt.org/remote.html#Remote_URI_reference

Change-Id: I40925ed31337d3ad9cf505f284f5c3d14e9129a0
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agovirtual_fuel: make vm_template an attribute 09/15009/2
Josep Puigdemont [Wed, 4 May 2016 12:27:23 +0000 (14:27 +0200)]
virtual_fuel: make vm_template an attribute

Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agovirtual_fuel: factor out image creation into a method 07/15007/2
Josep Puigdemont [Wed, 4 May 2016 12:27:23 +0000 (14:27 +0200)]
virtual_fuel: factor out image creation into a method

Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agoVirtualFuel: Add temp_dir and vm_name attributes 05/15005/2
Josep Puigdemont [Wed, 4 May 2016 12:27:23 +0000 (14:27 +0200)]
VirtualFuel: Add temp_dir and vm_name attributes

These two variables are defined in one of the methods right now. They
will be useful to other methods too, so we add them as attributes to the
object here.

Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agoipmi_adapter: simplify, retry if command fails 03/15003/3
Josep Puigdemont [Fri, 6 May 2016 10:09:58 +0000 (12:09 +0200)]
ipmi_adapter: simplify, retry if command fails

The method get_node_state has been added to the the IpmiAdapter class.

In addition, now the power on/off methods will try several times to
perform their IPMI command before giving up, instead of bailing out at
the first error.

After the power on/off command is completed, the method will wait until
the node is in the desired state.

NOTE: a command could potentially take several minutes if the defaults
are used; each IPMI command can take up to 1 minute, and there can be 3
commands issued per operation, one of them may be retried 20 times with
the current defaults. Ideally we would use eventlet or something similar
to allow each command a limited time to execute, instead:

    with eventlet.timeout.Timeout(seconds) as t:
        power_on/off_command

Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agoODL build: Allow customizing JAVA URL 15/14915/2
Alexandru Avadanii [Tue, 31 May 2016 20:08:25 +0000 (22:08 +0200)]
ODL build: Allow customizing JAVA URL

For Armband, JAVA8_URL hardcode needs to be patched outside of the
build system, so make related var overrideable.

Change-Id: I308074a4ae0c5f8b22e5e5128965ce90fea3734e
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
Signed-off-by: Stanislaw Kardach <Stanislaw.Kardach@cavium.com>
8 years agocommon.py: allow specifying number of attempts in exec_cmd 01/15001/2
Josep Puigdemont [Fri, 6 May 2016 01:28:26 +0000 (03:28 +0200)]
common.py: allow specifying number of attempts in exec_cmd

Some commands executed by exec_cmd may fail because of a temporary
cause, and it may be desirable to retry the same command several times
until it succeeds. One example of this are the ipmitool commands, which
may fail temorarily on some targets if they get too many requests
simultaneously.

In this patch three new optional parameters are introduced to the
function signature, which do not break backward compatibility:
  attempts: which indicates how many times the command should be run if
            it returns a non-zero value*, and defaults to 1 (as today).
  delay:    which indicates the delay in seconds between attempts, and
            defaults to 5 seconds.
  verbose:  It will print the remaining attempts left for the current
            command if set to True.

* It may be desirable to add yet another parameter to indicate what
  return value should be considered an error, but non-zero for now
  seems a reasonable default.

Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agoMerge "repobuild: Allow customizing fuel-mirror repo URL"
Jonas Bjurel [Wed, 15 Jun 2016 12:31:22 +0000 (12:31 +0000)]
Merge "repobuild: Allow customizing fuel-mirror repo URL"

8 years agorepobuild: Allow customizing fuel-mirror repo URL 13/14913/3
Alexandru Avadanii [Wed, 24 Feb 2016 19:02:25 +0000 (20:02 +0100)]
repobuild: Allow customizing fuel-mirror repo URL

For Armband, fuel-mirror code needs to be patched outside of the build
system, so we factor out the repo URL into an overrideable variable
(FUEL_MIRROR_URL).

Change-Id: Icf26f1b84f5a653f541819a42980377f51c7f299
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
Signed-off-by: Stanislaw Kardach <Stanislaw.Kardach@cavium.com>
8 years agoBuild: Allow customizing FUEL_MAIN_TAG (commit) 11/14911/4
Alexandru Avadanii [Sun, 21 Feb 2016 15:28:02 +0000 (16:28 +0100)]
Build: Allow customizing FUEL_MAIN_TAG (commit)

FUEL_MAIN_TAG is currently hard set to 9.0 (not overridable).

While this is a sane assumption for the usual scenarios,
Armband applies a series of patches on top of Fuel@OPNFV, hence
requiring us to override this variable.

WARNING: FUEL_MAIN_TAG is reused for fuel-mirror tag, so if you
override this variable, make sure the same tag exists in fuel-mirror.

Change-Id: Ided75cf0c3b5ad18cf7ef1ec88b5d2dc3aada511
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
Signed-off-by: Stanislaw Kardach <Stanislaw.Kardach@cavium.com>
8 years agoMerge "Build local mirrors for stable/mitaka"
Michal Skalski [Wed, 15 Jun 2016 08:33:34 +0000 (08:33 +0000)]
Merge "Build local mirrors for stable/mitaka"

8 years agodeploy: ipmi adapter: Add <port> config support. 07/14907/2
Alexandru Avadanii [Mon, 23 May 2016 20:06:09 +0000 (22:06 +0200)]
deploy: ipmi adapter: Add <port> config support.

Sometimes the IPMI lanplus protocol listens on a non-standard
remote port, e.g. when target nodes are interfaced through a
fake IPMI BMC application that listens on multiple ports on the
same IP address.

Therefore, allow setting IPMI port in the DHA using a new
property named `ipmiPort`, and pass it along to `ipmitool` when set.

CHANGE: get_access_info now also supports specifying the IPMI
port to use with `ipmitool` by configuring the `ipmiPort`
property in the DHA.

hp_adapter.py: updated `get_access_info` return signature with
the new (unused there) `ipmiport`.

Change-Id: I620176bd7f466aa460518cf12d15ccbe86a22560
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
8 years agobuild and deploy onos plugin for fuel9 63/15263/3
wuwb1989 [Wed, 8 Jun 2016 03:42:31 +0000 (11:42 +0800)]
build and deploy onos plugin for fuel9

Change-Id: I04dd8b4bdddc5678b158d7287c6ffc52d1bce135
Signed-off-by: wuwb1989 <wuwenbin2@huawei.com>
8 years agoMerge "Enable vsperf plugin in fuel 9.0"
Jonas Bjurel [Tue, 14 Jun 2016 18:27:33 +0000 (18:27 +0000)]
Merge "Enable vsperf plugin in fuel 9.0"

8 years agoMerge "Update configuration for fuel 9 virtual deployment in Huawei-China."
Jonas Bjurel [Tue, 14 Jun 2016 18:24:15 +0000 (18:24 +0000)]
Merge "Update configuration for fuel 9 virtual deployment in Huawei-China."

8 years agoEnable vsperf plugin in fuel 9.0 25/14925/3
Guo Ruijing [Sat, 11 Jun 2016 16:29:53 +0000 (00:29 +0800)]
Enable vsperf plugin in fuel 9.0

Change-Id: I96b59e22fcc0269aa3ae0b04587c8ca7d0fab867
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>
8 years agoFixing erroneous hypervisor type 37/15537/1
Stefan K. Berg [Tue, 14 Jun 2016 13:52:16 +0000 (15:52 +0200)]
Fixing erroneous hypervisor type

A "qemu" snuck in instead of "kvm".

Change-Id: Ibe704103cd1bab6e127a31d08d53f53518033539
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>
8 years agoBuild local mirrors for stable/mitaka 59/15259/3
Michal Skalski [Tue, 7 Jun 2016 22:54:57 +0000 (00:54 +0200)]
Build local mirrors for stable/mitaka

Change-Id: I3128652aeb87cb2cfaa91ded1b8d9ebeeb70d33b
Signed-off-by: Michal Skalski <mskalski@mirantis.com>
8 years agoUpdate configuration for fuel 9 virtual deployment in Huawei-China. 07/15507/1
wuwenbin2 [Tue, 14 Jun 2016 07:41:08 +0000 (15:41 +0800)]
Update configuration for fuel 9 virtual deployment in Huawei-China.

Change-Id: I9d2fc979886510c165af8dbac93ddcdc954727cf
Signed-off-by: wuwenbin2 <wuwenbin2@huawei.com>
8 years agoUpdated devel-pipeline DEA overrides for Fuel 9.0 73/15473/2
Stefan K. Berg [Mon, 13 Jun 2016 16:29:13 +0000 (18:29 +0200)]
Updated devel-pipeline DEA overrides for Fuel 9.0

Minor fix in the ELX version.
Update to Fuel 9.0 in the default version.

Change-Id: Ic084b86e7f6d2dfc3d15b10f0ef72e04ef2b7bf6
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>
8 years agoMerge "Modification of deployment config for Fuel 9 and ELX lab"
Stefan K. Berg [Mon, 13 Jun 2016 14:55:10 +0000 (14:55 +0000)]
Merge "Modification of deployment config for Fuel 9 and ELX lab"

8 years agoMerge "Revert "Temporary fix for PXE booting from the wrong NIC""
Stefan Berg [Mon, 13 Jun 2016 12:38:14 +0000 (12:38 +0000)]
Merge "Revert "Temporary fix for PXE booting from the wrong NIC""

8 years agoRevert "Temporary fix for PXE booting from the wrong NIC" 49/15449/1
Stefan Berg [Mon, 13 Jun 2016 12:32:07 +0000 (12:32 +0000)]
Revert "Temporary fix for PXE booting from the wrong NIC"

This reverts commit 5926bcddca0eca28a33ae43aadf90b9263ae6b84.

Change-Id: Ibea4ca5d38ce2e10ab6d5bb1d7ccdf666ee149d5
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>
8 years agoMerge "Temporary fix for PXE booting from the wrong NIC"
Daniel Smith [Mon, 13 Jun 2016 11:45:58 +0000 (11:45 +0000)]
Merge "Temporary fix for PXE booting from the wrong NIC"

8 years agoTemporary fix for PXE booting from the wrong NIC 45/15445/1
Peter Barabas [Mon, 13 Jun 2016 11:18:04 +0000 (13:18 +0200)]
Temporary fix for PXE booting from the wrong NIC

Change-Id: I1f6d38da24d5a4e66061b9bdfe576fbbda7fb624
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoModification of deployment config for Fuel 9 and ELX lab 29/15429/4
Stefan K. Berg [Mon, 13 Jun 2016 07:26:57 +0000 (09:26 +0200)]
Modification of deployment config for Fuel 9 and ELX lab

Change-Id: I380087889cda079a56c8cea3acc13145dcd49046
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>
8 years agoMerge "ISO: Use --joliet-long names."
Stefan Berg [Wed, 8 Jun 2016 14:26:15 +0000 (14:26 +0000)]
Merge "ISO: Use --joliet-long names."

8 years agoMerge "Do not create files when check connectivity"
Michal Skalski [Wed, 8 Jun 2016 14:24:53 +0000 (14:24 +0000)]
Merge "Do not create files when check connectivity"

8 years agoMerge "Script to clean expired build cache items"
Stefan K. Berg [Wed, 8 Jun 2016 14:23:18 +0000 (14:23 +0000)]
Merge "Script to clean expired build cache items"

8 years agoDo not create files when check connectivity 77/15277/1
Michal Skalski [Wed, 8 Jun 2016 13:18:35 +0000 (15:18 +0200)]
Do not create files when check connectivity

Change-Id: If0ab25ee843f9892510da3d3c8d1f10a54440545
Signed-off-by: Michal Skalski <mskalski@mirantis.com>
8 years agoFix for not reaping 'ssh_network' 75/15275/1
Peter Barabas [Wed, 8 Jun 2016 13:10:31 +0000 (15:10 +0200)]
Fix for not reaping 'ssh_network'

Change-Id: Ib225701a808211e50554c8f1762325aa75ecc33f
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoScript to clean expired build cache items 73/15273/2
Stefan K. Berg [Wed, 8 Jun 2016 11:50:32 +0000 (13:50 +0200)]
Script to clean expired build cache items

Change-Id: I5e466c89c05eb4d637778218cfe03b88d569f331
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>
8 years agoMerge "Build odl plugin for mitaka"
Michal Skalski [Mon, 6 Jun 2016 09:52:09 +0000 (09:52 +0000)]
Merge "Build odl plugin for mitaka"

8 years agoBuild odl plugin for mitaka 85/14385/4
Michal Skalski [Thu, 19 May 2016 14:52:18 +0000 (16:52 +0200)]
Build odl plugin for mitaka

Change-Id: I59b96a424a753f880b4ac16abd806851ad3f9533
Signed-off-by: Michal Skalski <mskalski@mirantis.com>
8 years agoDownload deployment config after modification 69/15069/3
Peter Barabas [Thu, 2 Jun 2016 08:29:29 +0000 (10:29 +0200)]
Download deployment config after modification

Modified network or interface configurations were not reflected in
the deployment config, resulting in faulty node configurations.

Change-Id: I4ca20702c0171e7995f2b4f46317557ec9d5beac
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoMerge "Auto deploy corrections for Fuel 9.0"
Stefan K. Berg [Thu, 2 Jun 2016 07:19:28 +0000 (07:19 +0000)]
Merge "Auto deploy corrections for Fuel 9.0"

8 years agoMerge "docs: Mitaka Build req: Add p7zip-full."
Jonas Bjurel [Wed, 1 Jun 2016 18:23:33 +0000 (18:23 +0000)]
Merge "docs: Mitaka Build req: Add p7zip-full."

8 years agoISO: Use --joliet-long names. 73/14973/1
Alexandru Avadanii [Wed, 1 Jun 2016 14:43:17 +0000 (16:43 +0200)]
ISO: 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).

This works around ISO build error(s) like:

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

Change-Id: I89d9816d86176d80bcc76f5f90c8391b0664ab82
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
8 years agodocs: Mitaka Build req: Add p7zip-full. 71/14971/1
Alexandru Avadanii [Wed, 1 Jun 2016 13:43:06 +0000 (15:43 +0200)]
docs: Mitaka Build req: Add p7zip-full.

Build system uplift to Fuel 9.0/Mitaka now requires `7z` cmd utility
also on the builder host (not only inside docker containers), so
document this as a prereq in the build instructions.

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

Change-Id: Icb57ac58c499272fce0a27bb2d25383e933c7814
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
8 years agoAuto deploy corrections for Fuel 9.0 59/14959/1
Stefan K. Berg [Wed, 1 Jun 2016 11:25:37 +0000 (13:25 +0200)]
Auto deploy corrections for Fuel 9.0

- The auto deployer's detection of nodes being up corrected as "fuel node"
  now returns "1" instead of "True" to denote that the node is up.

- The location of bootstrap_admin_node.sh has changed so the detection of
  whether the Fuel node installation needed a correction for the deployer
  not to throw an exit code and terminate prematurely.

- Small fix: deploy.py is now executable (just a chmod change).

Change-Id: I8fed7bafe6912f8b4278619bbdaa16577a82737b
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>
8 years agoClean up after ourselves 47/14947/1
Peter Barabas [Wed, 1 Jun 2016 08:31:54 +0000 (10:31 +0200)]
Clean up after ourselves

Change-Id: Id7579ef618b8cd922de325d9dc1c0b7a6c5587a7
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoA small edit for the uplift to Mitaka from Liberty
Daiel Smith [Wed, 1 Jun 2016 03:49:46 +0000 (23:49 -0400)]
A small edit for the uplift to Mitaka from Liberty

Signed-off-by: Daiel Smith <daniel.smith@ericsson.com>
JIRA:0

8 years agocommon.py: catch stderr in exec_cmd 61/14561/3
Josep Puigdemont [Wed, 4 May 2016 12:27:23 +0000 (14:27 +0200)]
common.py: catch stderr in exec_cmd

When running commands with exec_cmd(), the stderr of the command is
sent to /dev/null and ignored, and only stdout is retrieved. Thus, when
a command fails and check is enabled, only the output of stdout is
presented to the user, which normally holds no information about the
error.
In this patch we retrieve stderr, and when an error occurs, an exception
is raised with that message.

Fixes https://jira.opnfv.org/browse/FUEL-142

Change-Id: I3940e1a43963a6abec362481b1d4ce7bd7cb816d
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agoMerge "Build system uplift to Fuel 9.0/Mitaka"
Stefan K. Berg [Tue, 31 May 2016 14:49:36 +0000 (14:49 +0000)]
Merge "Build system uplift to Fuel 9.0/Mitaka"

8 years agoMerge "Remove residual Lab & configuration DEA and DHA from the unprotected Fuel...
Jonas Bjurel [Mon, 30 May 2016 19:26:12 +0000 (19:26 +0000)]
Merge "Remove residual Lab & configuration DEA and DHA from the unprotected Fuel repo"

8 years agoBuild system uplift to Fuel 9.0/Mitaka 19/13919/7
Stefan K. Berg [Wed, 11 May 2016 10:48:40 +0000 (12:48 +0200)]
Build system uplift to Fuel 9.0/Mitaka

Switching to Fuel 9.0/Mitaka for the build system.

Overhaul of the patching mechanism. As bootstrap_admin_node.sh has
been transitioned into an RPM (fuel-support), the lazy designer found
it more simple to patch that script during the Fuel build phase than
at the OPNFV ISO generation. The patch mechanism has been changed to a
normal context diff instead of the orig/modified file tuples
previously used. Hopefully this will require fewer manual rebases (may
the fuzz be with us!).

Also the ks.cfg patching has transitioned to a context based ordinary
patch for the same reasons, but this is as before taking place during
OPNFV ISO generation.

Patch naming made more descriptive.

The reaping mechanism has been slightly modified due to a change in
the naming of the node files when these are generated by the Fuel CLI.

IMPORTANT 1: The package cache mechanism is currently disabled, it is
only possible to install Fuel with a direct internet connection. This
will be fixed in a later change set!

IMPORTANT 2: All plugins has been disabled! As you have re-certified
your plugin with Fuel 9.0, please re-enable it in build/Makefile!

Change-Id: Ia918d16a74b68f89d178e06befe6e8a7a9367bf9
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>
8 years agoMerge "More rigorous Fuel environment check"
Jonas Bjurel [Wed, 25 May 2016 21:30:32 +0000 (21:30 +0000)]
Merge "More rigorous Fuel environment check"

8 years agoMerge "docs/install-instr: Add target specific cfg sect."
Jonas Bjurel [Tue, 24 May 2016 17:41:48 +0000 (17:41 +0000)]
Merge "docs/install-instr: Add target specific cfg sect."

8 years agoMore rigorous Fuel environment check 27/14427/1
Peter Barabas [Fri, 20 May 2016 09:10:35 +0000 (11:10 +0200)]
More rigorous Fuel environment check

get_env() used to raise an unhandled exception when reap.py was run on a
Fuel node which didn't yet have an environment set up.

Change-Id: I07c37db2d80e416d26fa4fb4907f4e438f1c44e5
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoRemove coexistence configuration from post script 67/14367/2
Manuel Buil [Thu, 19 May 2016 11:30:25 +0000 (13:30 +0200)]
Remove coexistence configuration from post script

This is not more needed as the config comes from FUEL ODL plug-in

Change-Id: Ie09a52ac3919ac641924d41431dc000910e6d4f4
Signed-off-by: Manuel Buil <manuel.buil@ericsson.com>
8 years agodocs/install-instr: Add target specific cfg sect. 57/14057/2
Alexandru Avadanii [Thu, 12 May 2016 16:44:28 +0000 (18:44 +0200)]
docs/install-instr: Add target specific cfg sect.

Fuel 8.0 adds support for setting up "Offloading Modes" for
target nodes.

Document how this features should be used (and why/when it's
necessary), and also its common limitation of not being applied
during Verify Networks step (see related Fuel bug report [1]).

While at it, fix minor reference to branch in git tag checkout
instructions, as we'd be in detached HEAD and not on a new branch.

[1] https://jira.opnfv.org/browse/FUEL-134

[Alexandru.Avadanii@enea.com]
Only minor edits and prepared for upstreaming, this is mostly
based on previous work from Florin.

Change-Id: If7a8ca44541903dbeb6db36bc64fe56b132165cb
Signed-off-by: Florin Dumitrascu <florin.dumitrascu@enea.com>
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
8 years agoMerge "docs/release-notes: Add protocol handler to ref."
Jonas Bjurel [Thu, 12 May 2016 16:31:53 +0000 (16:31 +0000)]
Merge "docs/release-notes: Add protocol handler to ref."

8 years agoMerge "Adjust kernel numbers comparison"
Michal Skalski [Thu, 12 May 2016 14:46:37 +0000 (14:46 +0000)]
Merge "Adjust kernel numbers comparison"

8 years agodocs/release-notes: Add protocol handler to ref. 45/14045/1
Alexandru Avadanii [Wed, 11 May 2016 20:28:49 +0000 (22:28 +0200)]
docs/release-notes: Add protocol handler to ref.

Doc build fails to create a link for www.opnfv.org because it
does not explicitly specify the protocol handler.

Make doc build happy by adding "http://" prefix.

Change-Id: I7dc9c362f13bac1687d139942826eb1208556a37
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
8 years agodocs/install-instr: Fix git clone URL for HTTPS. 43/14043/1
Alexandru Avadanii [Wed, 11 May 2016 21:34:25 +0000 (23:34 +0200)]
docs/install-instr: Fix git clone URL for HTTPS.

Drop unnecessary <user>@, not needed for http downloads.
Also, fix typo (missing letter) in opnfv.org domain name.

While we're at it, fix small typo in Ceilometer name too.

Change-Id: I2c3341689b7a7808863906700342e62e707246e9
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
8 years agoMerge "docs/build-instr: Fix git clone URL for HTTPS."
Jonas Bjurel [Thu, 12 May 2016 12:44:06 +0000 (12:44 +0000)]
Merge "docs/build-instr: Fix git clone URL for HTTPS."

8 years agoRemove residual Lab & configuration DEA and DHA from the unprotected Fuel repo 05/9205/2
Jonas Bjurel [Wed, 3 Feb 2016 23:31:04 +0000 (00:31 +0100)]
Remove residual Lab & configuration DEA and DHA from the unprotected Fuel repo

This needs testing!!!!
On a follow-up release, this also needs rebasement, the libvirt templates
are still in here - they shouldnt, sinse we dont want Lab specific
configs in the Fuel repo.

NOT VERIFIED
DO NOT MERGE

Change-Id: I069ced81b886405463f27f37a6ec78e3748b37b7
Signed-off-by: Jonas Bjurel <jonas.bjurel@ericsson.com>
8 years agoMerge "deployment.py: stdout not consumed when deploying changes"
Jonas Bjurel [Wed, 11 May 2016 19:28:27 +0000 (19:28 +0000)]
Merge "deployment.py: stdout not consumed when deploying changes"

8 years agodocs/build-instr: Fix git clone URL for HTTPS. 31/13931/1
Alexandru Avadanii [Wed, 11 May 2016 13:54:02 +0000 (15:54 +0200)]
docs/build-instr: Fix git clone URL for HTTPS.

Fix wrong https git clone URL, 29418 port should be used only for SSH.

While we're at it, add some bash code-block markup to bash commands.

Change-Id: Ie95a4c170b70bcc3da5d4d59260d61fbd5f2a9f3
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
8 years agoAdjust kernel numbers comparison 23/13923/1
Michal Skalski [Wed, 11 May 2016 12:41:58 +0000 (14:41 +0200)]
Adjust kernel numbers comparison

With current solution kernel 4.2.0 is treated as earlier version,
final comparison is: 42 < 319. This change will compare coresponding
numbers of installed kernel with required version.

Change-Id: Iac3aed0eb21ac8ad3138c2169299ca3b43e6e663
Signed-off-by: Michal Skalski <mskalski@mirantis.com>
8 years agoMerge "The configuration of the coexistence between SFC and Netvirt is added"
Nikolas Hermanns [Wed, 11 May 2016 08:11:52 +0000 (08:11 +0000)]
Merge "The configuration of the coexistence between SFC and Netvirt is added"

8 years agoMerge "deploy.sh: do not expect a parameter for -h"
Jonas Bjurel [Tue, 10 May 2016 18:33:56 +0000 (18:33 +0000)]
Merge "deploy.sh: do not expect a parameter for -h"

8 years agoThe configuration of the coexistence between SFC and Netvirt is added 37/13837/3
Manuel Buil [Tue, 10 May 2016 09:05:58 +0000 (11:05 +0200)]
The configuration of the coexistence between SFC and Netvirt is added

Change-Id: I3ce03c38677ac67df3aae0363048be21b7939bde
Signed-off-by: Manuel Buil <manuel.buil@ericsson.com>
8 years agoMerge "Reflect new deploy.py options in README"
Daniel Smith [Tue, 10 May 2016 16:02:13 +0000 (16:02 +0000)]
Merge "Reflect new deploy.py options in README"

8 years agoMerge "Use US keyboard layout on the console"
Daniel Smith [Tue, 10 May 2016 16:02:06 +0000 (16:02 +0000)]
Merge "Use US keyboard layout on the console"

8 years agoMerge "Fix handling of relative ISO path on the CLI"
Stefan Berg [Mon, 9 May 2016 11:39:13 +0000 (11:39 +0000)]
Merge "Fix handling of relative ISO path on the CLI"

8 years agoUse US keyboard layout on the console 83/13783/1
Peter Barabas [Mon, 9 May 2016 11:26:51 +0000 (13:26 +0200)]
Use US keyboard layout on the console

Change-Id: I40942c7181daf5efd1640a03471e91df82548073
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoReflect new deploy.py options in README 07/13707/1
Peter Barabas [Fri, 6 May 2016 08:05:41 +0000 (10:05 +0200)]
Reflect new deploy.py options in README

Change-Id: Id22ae685f324b58d07bd0c5256f3dbf55672776e
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agodeployment.py: stdout not consumed when deploying changes 27/13627/1
Josep Puigdemont [Tue, 3 May 2016 10:44:07 +0000 (12:44 +0200)]
deployment.py: stdout not consumed when deploying changes

During the automatic deployment, when the environment is ready to be
deployed, the deploy.py script will spawn a shell process that will
perform the command "fuel deploy-changes". The standard output of this
process is then piped to a "tee" process, which redirects the output
to the standard output of the shell process, and to a file named
cloud.log. The file is monitored by the deploy script to find out the
status of the deployment, and print it to the log file of the automatic
deployment script, including percentages for each node being
provisioned. However, the deploy script never consumes the standard
output of the shell process. If the shell process produces enough
output, its standard output buffer will fill up, thus making the tee
process block trying to write to its standard output, and the cloud.log
file will not be updated. At this point, the deploy process, which is
monitoring cloud.log, will not detect any progress in the deployment,
and eventually it will time out and assume the deployment failed,
although it might have finished fine after that.

The solution here is to remove the "tee" process from the shell command,
and instead redirect standard output to the cloud.log file.
Another solution would be to actually parse the standard output of the
shell command from the deploy script itself, but that would require a
bit more work, as reading a line at a time might block the script.

Finally, with this patch the cloud.log file won't be deleted unless the
shell process has already finished.

Change-Id: I03a77be42d220b1606e48fc4ca35e22d73a6e583
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agodeploy.sh: do not expect a parameter for -h 79/13479/1
Josep Puigdemont [Sat, 30 Apr 2016 17:16:10 +0000 (19:16 +0200)]
deploy.sh: do not expect a parameter for -h

If -h was given as a parameter to the script, it would report an error
as it expected a parameter, and if it was called as the only parameter,
it would run deploy.py as if "old style" parameters had been given, thus
showing the usage for the python script, instead of the expected usage
message for this script.

Update the usage message to include -h.

Change-Id: I0930936962c1cb479ec4409ff114cd60a386b276
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agoMerge "Updating project lead, and link of approval"
Jonas Bjurel [Fri, 29 Apr 2016 19:50:40 +0000 (19:50 +0000)]
Merge "Updating project lead, and link of approval"

8 years agoMerge "Add a dha adapter python file for ZTE hardware."
Jonas Bjurel [Fri, 29 Apr 2016 17:01:32 +0000 (17:01 +0000)]
Merge "Add a dha adapter python file for ZTE hardware."

8 years agoUpdating project lead, and link of approval 47/13447/1
Aric Gardner [Fri, 29 Apr 2016 15:04:58 +0000 (11:04 -0400)]
Updating project lead, and link of approval

Change-Id: Id265bb022a4c509e642e8f8b61a65c5cfb05c1b7
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
8 years agoMerge "Clean up scenario files"
Jonas Bjurel [Thu, 28 Apr 2016 21:53:11 +0000 (21:53 +0000)]
Merge "Clean up scenario files"

8 years agoINFO file updated with Greg and Nikolas as committers 07/13407/1
Jonas Bjurel [Thu, 28 Apr 2016 17:14:44 +0000 (19:14 +0200)]
INFO file updated with Greg and Nikolas as committers

Change-Id: I9a64e5948e159362cb851777db6de41eccb2a599
Signed-off-by: Jonas Bjurel <jonas.bjurel@ericsson.com>
8 years agoFix handling of relative ISO path on the CLI 03/13403/2
Peter Barabas [Thu, 28 Apr 2016 15:25:28 +0000 (17:25 +0200)]
Fix handling of relative ISO path on the CLI

Change-Id: I32878726432c3d883f0b33bdd2c836b0770e734f
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoClean up scenario files 87/13387/6
Peter Barabas [Thu, 28 Apr 2016 08:32:02 +0000 (10:32 +0200)]
Clean up scenario files

 - remove release from scenario files
 - remove old versions of scenario files
 - remove versions from scenario file names and update scenario.yaml
 - update README

Change-Id: I5203ea0794b96fb44f6a9db25b33e5066c1d9574
Signed-off-by: Peter Barabas <peter.barabas@ericsson.com>
8 years agoAdd a dha adapter python file for ZTE hardware. 79/13379/1
wu.zhihui [Thu, 28 Apr 2016 07:27:15 +0000 (03:27 -0400)]
Add a dha adapter python file for ZTE hardware.

ZTE hardware need use IPMI command "chassis power cycle"
on function node_reset().

JIRA: FUEL-127

Change-Id: I286fd9cda43d2e1799b134f0a391f57d08cd1928
Signed-off-by: wu.zhihui <wu.zhihui1@zte.com.cn>
8 years agoUpdated documentation for Brahmaputra 3.0 follow-up release 85/12685/3
Jonas Bjurel [Tue, 26 Apr 2016 19:06:02 +0000 (21:06 +0200)]
Updated documentation for Brahmaputra 3.0 follow-up release

READY TO MERGE

Change-Id: Ic481f02375af9be1642791fa6d96856a453c4f29
Signed-off-by: Jonas Bjurel <jonas.bjurel@ericsson.com>
8 years agoMerge "configure_environment.py: quote environment name"
Michal Skalski [Tue, 26 Apr 2016 15:27:17 +0000 (15:27 +0000)]
Merge "configure_environment.py: quote environment name"

8 years agoMerge "Using VLAN segmentation for the NFV-OVS scenarios. Unfortunately this scenario...
Jonas Bjurel [Tue, 26 Apr 2016 13:18:07 +0000 (13:18 +0000)]
Merge "Using VLAN segmentation for the NFV-OVS scenarios. Unfortunately this scenario is hardwired to work with Ericsson POD-2 only"

8 years agoUsing VLAN segmentation for the NFV-OVS scenarios. 83/12483/7
Jonas Bjurel [Wed, 20 Apr 2016 12:59:49 +0000 (14:59 +0200)]
Using VLAN segmentation for the NFV-OVS scenarios.
Unfortunately this scenario is hardwired to work with Ericsson POD-2 only

Change-Id: I3a0b56d7ff71e0ec9cd97b8ef5946fb438d43e62
Signed-off-by: Jonas Bjurel <jonas.bjurel@ericsson.com>
8 years agoconfigure_environment.py: quote environment name 09/12409/2
Josep Puigdemont [Sun, 17 Apr 2016 09:22:27 +0000 (11:22 +0200)]
configure_environment.py: quote environment name

The Fuel environment name may contain spaces, putting the name in quote
marks prevents the second and subsequent words from being interpreted
as other parameters by the fuel command.

The name could contain double quotes too, so this doesn't solve all
problems, but arguably the most common case.

Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
8 years agoCorrection due to changed cpio behavior 69/12569/2
Stefan K. Berg [Fri, 22 Apr 2016 10:48:25 +0000 (12:48 +0200)]
Correction due to changed cpio behavior

As part of a security fix for cpio, its behavior has changed in terms
of how symlinks are handled.

This is affecting the Fuel build process, for which this commit is a
fix (reverting cpio back to its original behavior by adding the
argument "--extract-over-symlinks" in those instances where this is
needed).

For details, see https://jira.opnfv.org/browse/FUEL-125

Change-Id: I455b11a16ad52d8dd09165165447e4c4a661b45a
Signed-off-by: Stefan K. Berg <stefan.k.berg@ericsson.com>
8 years ago[VPNSERVICE] Bugfix Broadcast Group 41/12341/1
Nikolas Hermanns [Fri, 15 Apr 2016 13:22:34 +0000 (15:22 +0200)]
[VPNSERVICE] Bugfix Broadcast Group

The local broadcast group was not updated
correctly in ODL. This commit includes the
bugfix.

Change-Id: I7857bcdc543217d513683bf91ab13fa93ba2ed42

8 years agoAdding patch for Broadcast Storm of vpnservice 89/12289/1
Nikolas Hermanns [Thu, 14 Apr 2016 14:15:07 +0000 (16:15 +0200)]
Adding patch for Broadcast Storm of vpnservice

There is no persisten release from stable/beryllium
with the bugfix https://git.opendaylight.org/gerrit/#/c/37566/
included. So we have to overwrite the file we a own
build.

Change-Id: I637c333fa2175544f182cd1f7b774501d3fc2c25