Adds detection for virsh default network IP

In case a user already has a default virsh network and the default IP is
not 192.168.122.1, this patch will allow detecting and using the found
value.

Change-Id: Id5d46a820d89e86c9b5897f5d29b83ed32f9d6f9
Signed-off-by: Tim Rozet <trozet@redhat.com>

Add kubernetes repo and install RPMS into overcloud images

This change introduces k8s into overcloud images. It just installs
kubernetes rpms only, so further configuration is required to
activate k8s at overcloud nodes and it will be taken care of other
diffs.

Change-Id: I606d176812705802790243ae4e95988a591c5fe6
Signed-off-by: Tomofumi Hayashi <s1061123@gmail.com>

Merge "Revert "Adds Calipso scenario""

Fixes removing inventory keys

Inventory keys that we want to delete were not being removed for
instackenv.json.  Also fixes unit tests.

Change-Id: I7d6c752342db050d651953633f18e236f60334e3
Signed-off-by: Tim Rozet <trozet@redhat.com>

Revert "Adds Calipso scenario"

This reverts commit 433961f8c0b816f19aa858654261fa87d2f02a19.

Change-Id: Ic8a14d2e0ad8d1006625af401fde2f365890650b
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Adds Calipso scenario"

Merge "Improves clean to cleanup virsh nets and and other bridges"

Removing duplicate lib path in ansible_path

- deploy line 283 lib is joined with ANSIBLE_PATH
  then line 285, 347, 418 and 435 it was joined again with
  ansible_path from 283 already containing lib

Change-Id: I6612967d626eeb0668eb43294831447c387f8e93
Signed-off-by: Dan Radez <dradez@redhat.com>

Merge "Set aarch64 deploys to UEFI"

Improves clean to cleanup virsh nets and and other bridges

Currently only the admin and external bridges will be removed from the
jumphost which is only really clean for a baremetal deployment.  Virtual
deployments will leave br-tenant, br-api, br-storage.  This patch
removes those as well as deletes all Apex virsh networks from libvirt.

Also checks if authorized_keys exists before trying to edit it.

Change-Id: I1affe2733ba1dc38fd3b91fb14398454137f00a4
Signed-off-by: Tim Rozet <trozet@redhat.com>

Adds Calipso scenario

Calipso uses docker containers and is post deployment feature that can
be used to monitor pieces of the stack.  This patch installs it to the
undercloud rather than the jumphost as it is easier to manage for
cleanup and dependencies.

Change-Id: Ie2dacc4f14b391a864fafb2e75ebcd15facb19f1
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Fixes missing libvirt-client in ISO"

Merge "Removes building OVS 2.6.1 with NSH"

Merge "Freezes RDO images for Euphrates"

Set aarch64 deploys to UEFI

Change-Id: I6b037ccd38fa68e5f863566a1c9a3ed4399bab84
Signed-off-by: Dan Radez <dradez@redhat.com>

Removes building OVS 2.6.1 with NSH

For master(Pike) we wont need this anymore since OVS 2.8 will have built
in support for NSH.

Change-Id: I51e2f4abf282ddb83d3edef9e5b827363a4d68be
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fixes missing libvirt-client in ISO

This was causing ISO install to fail because the libvirt modules were
missing for python-libvirt.

JIRA: APEX-517

Change-Id: Iaf127340864da58b2bace720293e515c8e9db426
Signed-off-by: Tim Rozet <trozet@redhat.com>

Freezes RDO images for Euphrates

Change-Id: Ie20e7d0c52efc2497c7c9975f88be0a9bfa7aa07
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Fixes jumphost pubkey error in undercloud playbook"

Minor fixes to clean

Also adds hidden files (like IDE configs) to gitignore

Change-Id: Ief595c41c5b4f0fe058f7b774f7a2aefdab733a4
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fix broken jumphost detach method when orig file exists

In the case where the backup ifcfg file exists (.orig) the functionality
to detach the interface from ovs was broken.  This fixes it and adds
unit test case to cover that scenario.

Change-Id: If6d8ca0ba5cf80df71bb82f52e9d204435374479
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fixes wrong argument for inventory with clean

Change-Id: I3ff145fa0f500182744fd310bb844e4bc5854663
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Cleaning up yamllint warnings"

Merge "Removes clean.sh"

Removes clean.sh

CI has been migrated to use opnfv-clean for verify jobs, which is now
installed as python.  There is no reason to keep ci/clean.sh anymore.

Change-Id: I2edf23901c5debaba0674e3506fd0392d317dc6f
Signed-off-by: Tim Rozet <trozet@redhat.com>

baro: updating refs to collectd-ceilometer

Since Ceilometer support was removed from the
collectd-ceilometer-plugin[1], the repo name and ceiloemter are
misleading. A change was submitted to openstack-infra to rename
the repo [2]

This change removes references to collectd-ceilometer from apex.
These changes are independant of the repo rename.

[1] https://review.openstack.org/#/c/490823/
[2] https://review.openstack.org/#/c/500768/

Depends-On: I6cf22c081aa1e40f64995be40b6225855faf4092
Change-Id: Ib3a7766e28fd03a86a995eea4b9abc996498200e
Signed-off-by: Emma Foley <emma.l.foley@intel.com>

Cleaning up yamllint warnings

yamllint output:
build/baremetal-environment.yaml
  1:1       warning  missing document start "---"  (document-start)

build/opnfv-environment.yaml
  6:3       warning  comment not indented like content  (comments-indentation)
  14:4      warning  missing starting space in comment  (comments)
  123:1     warning  comment not indented like content  (comments-indentation)
  128:1     warning  comment not indented like content  (comments-indentation)
  156:1     warning  comment not indented like content  (comments-indentation)

config/deploy/os-odl-fdio-dvr-ha.yaml
  1:1       warning  missing document start "---"  (document-start)

config/deploy/os-odl-fdio-dvr-noha.yaml
  1:1       warning  missing document start "---"  (document-start)

config/network/network_settings_v6.yaml
  225:6     warning  missing starting space in comment  (comments)

Change-Id: I3985ce0b6740ce4694b5d75b477cb03c3dc49c28
Signed-off-by: Dan Radez <dradez@redhat.com>

Fix broken build dependency

libguestfs-tools is failing to install because libvirt-client is
installed and there is a package conflict.  Removing libvirt-client from
image before installing libguestfs-tools.

NSH build is failing in overcloud because the CentOS repos updated with
newer kernel source and headers, and we are still using older kernel.
This patch adds the old kernel source and headers.  Upgrading to new
kernel causes NSH OVS to fail when compiling.

Change-Id: Ifd500ad138116cfecafb2268ccb580bd44d5efd5
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Migrates clean to python"

Migrates clean to python

ci/clean.sh will be removed in a future patch after releng is updated to
use python.

JIRA: APEX-509
JIRA: APEX-319

Change-Id: If890db2fc5a31833ad28ec6f04589e25457bd380
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Updates SFC to use nitrogen"

Show ansible tasks as they complete

Change-Id: I1b68d70fd97076a7f2ca68091a6e94d87b72efa2
Signed-off-by: Dan Radez <dradez@redhat.com>

Updates SFC to use nitrogen

Also fixes issue where openvswitch NSH rpm was not getting installed
because we needed to downgrade now that default OVS is 2.7 and NSH OVS
is 2.6.1

Change-Id: I1b639ca80214032e93aacb3332e9588d49e74127
Signed-off-by: Tim Rozet <trozet@redhat.com>

Dup owner, changing to group

Change-Id: I8ca1501949a0e272e928c6b31b08eba2519951ec
Signed-off-by: Dan Radez <dradez@redhat.com>

Restarts libvirtd after stopping firewalld

When stopping firewalld, libvirtd needs to be restarted in order for
libvirt to use/populate iptables with the NAT rules for the default
virsh network.

JIRA: APEX-510

Change-Id: Ia5b4515fd961baa70de58814e9eae4b397db28a7
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Adjustments to tmp dir handling"

Merge "Adds missing packages for undercloud"

Adjustments to tmp dir handling

- prefixing with apex_tmp
- removing overcloud qcow on failure

JIRA: APEX-513

Change-Id: I766df10c294c31622917bbd5b0597e3373716d8b
Signed-off-by: Dan Radez <dradez@redhat.com>

Merge "Conditionalize VM console arg based on arch"

Adds missing packages for undercloud

JIRA: APEX-511

Change-Id: If8ed2b5409e450c39f4fffd5a0b3cb96560c5d26
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "More cleanup for ci directory"

Merge "Output ansible's output to info log"

More cleanup for ci directory

Changes Include:
 - Moves dev_dep_check.sh to contrib dir as it is not used by our CI
 - Removes test.sh as it is not used by CI.  tox can be invoked
   directly for developers.
 - Removes run_smoke_tests.sh which is not used anymore

Change-Id: I59e39be0ce8b23234c70bf066aebf894b6d73388
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Skip introspection for aarch64 BM deployments"

Output ansible's output to info log

Change-Id: I27ad54785716677a885ae954a2af6dbaf8610ff1
Signed-off-by: Dan Radez <dradez@redhat.com>

Merge "Enabling the aarch check for deployment"

Merge "Adds parser tests and cleanup"

Conditionalize VM console arg based on arch

Change-Id: Ide9159e5ba7d5388c7bf62bf865b0015422e9159
Signed-off-by: Dan Radez <dradez@redhat.com>

Enabling the aarch check for deployment

Change-Id: Ic5f1fc8a61f74f7aa6b2fabf67169c601a603367
Signed-off-by: Dan Radez <dradez@redhat.com>

Skip introspection for aarch64 BM deployments

Change-Id: I630e90065f9cc76ef42ca9b23aea920c2493b53f
Signed-off-by: Dan Radez <dradez@redhat.com>

Adds parser tests and cleanup

Adds unit testing for the common/parsers library and cleans up some of
the test files syntax.

Change-Id: I7ff9d7ba20b028fba410af900a0c3107a5806d8f
Signed-off-by: Tim Rozet <trozet@redhat.com>

Remove deploy.sh and references

Deploy is now pip/rpm installed as opnfv-deploy in /bin.  Alternatively
the deploy.py script can be invoked via python.  The legacy deploy.sh
was only being used for CI, which has now been updated to remove the
need for the Apex repo containing ci/deploy.sh.

Change-Id: I1fb93bd9de23a97a09bedb8ac014ef81955875f2
Signed-off-by: Tim Rozet <trozet@redhat.com>

Removes build.sh

We now have build.py which handles all the building and build.sh was
just a wrapper to include ansible playbook.  Now that we have built in
apex libs to run ansible there is no reason to keep build.sh anymore.

Also simplifies the logic to determine apex root by using the git
properties.

Change-Id: I00d2e5c7c198e549d21936bb1e9b562ba93d3010
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Adds Undercloud retry and log fetch"

Merge "Fix deployment errors for odl-fdio scenarios"

Adds Undercloud retry and log fetch

Retry to install the undercloud if it fails due to sometimes failing.
Also copy undercloud log over to host temp dir.

Change-Id: I50f98c4f0d52fa6bb56d9dcd0766bbe745160452
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fix deployment errors for odl-fdio scenarios

- Changed NIC template generation to only configure VPP interface for
  external network on compute nodes
- Fixed overcloud deploy environment file error when deploying odl-fdio

Change-Id: I4fb07073b0dffc35ef09a5e32deb8470f5ee8347
Signed-off-by: Feng Pan <fpan@redhat.com>

Adds showing overcloud failures

Change-Id: Ic09a9b6d57d2ab05562f64a75f9732f76ac61bb7
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Fix nosdn-fdio deployment errors"

Merge "Change qemu-img resize unit"

Fix nosdn-fdio deployment errors

Also fixes kernel parameter configuration issue for fdio and ovs-dpdk
scenarios

Change-Id: I84162eb07d1427f62ac3c84fd32e78af0e3b8456
Signed-off-by: Feng Pan <fpan@redhat.com>

Merge "Fixes missing iso packages and idempotency of virt resources"

Merge "Removes ignoring errors on congress config"

Fixes missing iso packages and idempotency of virt resources

Python packages for ansible (py2.x) were missing causing ansible to
fail.  This patch adds those packages to the ISO and deploy
dependencies playbook.  Also includes fixes for deploy_dependencies
playbook for virt resources to make them idempotent by removing
using 'command' and instead just declaring 'state'.

Change-Id: I9311184a6f3c40ead0a8ba16fa0b61667876cf12
Signed-off-by: Tim Rozet <trozet@redhat.com>

Removes ignoring errors on congress config

Now that congress has been fixed this should always work.

Change-Id: I701707b8b91b340289613bdb31fa91456743d233
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Updates preliminary docs for Euphrates"

Updates preliminary docs for Euphrates

Change-Id: I4e3e78e60756824c1294ab067a325311bc2ac8f2
Signed-off-by: Tim Rozet <trozet@redhat.com>

Change qemu-img resize unit

Docs read:
size
    is the disk image size in bytes. Optional suffixes "k" or "K"
    (kilobyte, 1024) "M" (megabyte, 1024k) and "G" (gigabyte, 1024M)
     and T (terabyte, 1024G) are supported.  "b" is ignored.

MB is technically invalid and for somereason only aarch has complained
thus far

Change-Id: Ic152a0edab17ac1cdb5afc0791bb0353b95f94b8
Signed-off-by: Dan Radez <dradez@redhat.com>

Merge "Updates to create_vm for aarch64"

Merge "Fixes baremetal noha deployments"

Merge "Add argument types for deploy.py"

Fixes jumphost pubkey error in undercloud playbook

Change-Id: I71eca92e305b3bddc35691db8b8b9ebd3bc1b9fb
Signed-off-by: Tim Rozet <trozet@redhat.com>

Updates to create_vm for aarch64

- serial console was being duplicated in libvirt xml template
- force undercloud VM arch to jumphost arch
- assign diskbus to scsi for aarch64

Change-Id: I5208883c2001497fc7e958f9ffcdf14f0be5f600
Signed-off-by: Dan Radez <dradez@redhat.com>

Fixes baremetal noha deployments

There is a bug where when we have 3 nodes tagged as control in inventory
we are automatically using all 3, even for a noha deployment.

JIRA: APEX-507

Change-Id: Id2a9be2c52d0c7c81b9c5dcbc93006a476abb537
Signed-off-by: Tim Rozet <trozet@redhat.com>

Passing deploy_dir through to create_vm

Change-Id: I07a1c899d43c5edf1054fc819fff4336d9b3ed16
Signed-off-by: Dan Radez <dradez@redhat.com>

Add argument types for deploy.py

Add data type for integer arguments to avoid str to int comparison exceptions.

Change-Id: Ifa40e1cc8588df6d618185882bba71f7ce7b9a1d
Signed-off-by: Feng Pan <fpan@redhat.com>

Fixes bridge checking on jumphost for bm

Change-Id: I5e90dd84383c7348d2cbe58a18370ca10d915598
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fixes external network with baremetal interface

Change-Id: Icd65ffbd14dee603e5b2a0a42e9d0cdfd98c694c
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fixes baremetal interface attach

Change-Id: If0e501c218b1ea981caa255b5e217b5118b772b0
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Add missing iso build dependencies"

Add missing iso build dependencies

Change-Id: Ie7c07d34819ea29575f94ecac3da046758415da6
Signed-off-by: Feng Pan <fpan@redhat.com>

Skip the python3 vbmc patch

- vbmc has already been imported so we know it's there
  if vbmc is located in a different spot than the
  hardcoded location just skip the patch

Change-Id: I765fb94c09e0775084ecd37659f4900411eb0e3e
Signed-off-by: Dan Radez <dradez@redhat.com>

Merge "Updates and additions to handle aarch64"

Fixes issues with baremetal

Change-Id: I60470611a70261d57ccbfd0e499f4336c7a12fe1
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Updates RPM specs and building"

Updates and additions to handle aarch64

- move patches from x86 only to common virt-customize
- updates to undercloud to configure pxe and external network
- initial updates to overcloud deploy functions

JIRA: APEX-381

Change-Id: I496e196535e63bfd31654d0f501fe3f00fa5ca3f
Signed-off-by: Dan Radez <dradez@redhat.com>

Updates RPM specs and building

Changes Include:
 - Disables ONOS from rpm and image building
 - Fixes opnfv-apex package dependency for python34-opnfv-apex
 - Updates requirements for python34-opnfv-apex
 - Updates docs
 - Removes temporary build qcow2 for odl which was being bundled
   into our dev tars unnecessarily

Change-Id: Ief668067181fc35a54553ae118b7950bddce1940
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Change destination of snmp mibs files from /usr/share/mibs/ietf to /usr/share/snmp/mibs, and create symbolic link from ietf to snmp/mibs."

Merge "Migrates Apex to Python"

Migrates Apex to Python

Removes all bash libraries and converts almost all of the code to a
mixture of Python and Ansible.  utils.sh and clean.sh still exist.
clean.sh will be migrated fully to clean.py in another patch.

The Apex Python package is now built into the opnfv-apex-common RPM.  To
install locally do 'pip3 install .'.  To deploy:

opnfv-deploy -d <file> -n <file> --image-dir /root/apex/.build  -v --debug

Non-python files (THT yaml, settings files, ansible playbooks) are all
installed into /usr/share/opnfv-apex/.  The RPM will copy settings files
into /etc/opnfv-apex/.

JIRA: APEX-317

Change-Id: I3232f0329bcd13bce5a28da6a8c9c84d0b048024
Signed-off-by: Tim Rozet <trozet@redhat.com>

Change destination of snmp mibs files from
/usr/share/mibs/ietf to /usr/share/snmp/mibs,
and create symbolic link from ietf to snmp/mibs.

Change-Id: Icf3ff3659f63222cc3a5e8f1f93fe183a2b86f45
Signed-off-by: jhinman1 <john.hinman@intel.com>

No change to apex, change is to apex-puppet-tripleo
Bumps the step for barometer from 4 to 5.
apex-puppet-tripleo: I486c4045e29c7032526be6e19d11e7979070c2d9

Change-Id: I263d687e90e747e3fdfb9690ca9209553a1a2890
Signed-off-by: jhinman1 <john.hinman@intel.com>

Bumps OVS version to 2.8 for OVN

OVS 2.7 does not seem to be available anymore in the copr repo and has
been replaced by 2.8.

Change-Id: I7ba98f14e9c9c34f6802f8a890876363c839ab9b
Signed-off-by: Tim Rozet <trozet@redhat.com>

Small correction in docs.

Change-Id: Iec9a401759de5953d4a8ca05c4f1c32b163343ba
Signed-off-by: Taseer Ahmed <taseer94@gmail.com>

Merge "Fixes OpenDaylight version"

Merge "Adds Doctor DS driver to Congress"

Fixes OpenDaylight version

The default version is now Carbon with the option of using
nitrogen (master) for puppet-odl and opendaylight.

JIRA: APEX-500

Change-Id: I44d02ec0db5fb8e6342e5ab6802a5a3f302b8511
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Change install directory for collectd-ceilometer from /opt to /opt/stack."

Merge "Backport of Congress parallel execution"

Backport of Congress parallel execution

JIRA: APEX-480
JIRA: DOCTOR-78

Change-Id: I6b5b3e7f2daaec7e2ead76d74f8d3713378a5200
Signed-off-by: Carlos Goncalves <carlos.goncalves@neclab.eu>