Add support for odl_l3-fdio

This patch adds support for ODL L3 FDIO scenario.

Introduces a new deploy settings parameter, odl_routing_node, to
indicate the routing node's hostname. This parameter will only
be used for this scenario.

Change-Id: I8df3eb701299007761c6cec42c4ca318b124a0b8
opnfv-tht-pr: 108
os-net-config-pr: 4
Signed-off-by: Feng Pan <fpan@redhat.com>

Merge "Fixes mismatch for Java APIs between HC and VPP and L2 pkgs"

Fixes mismatch for Java APIs between HC and VPP and L2 pkgs

This copies the same API jars used for VPP to the Honeycomb to help
workaround the issues we are facing of incompatible Java APIs between HC
and VPP versions.  It also locks vpp/honeycomb/odl to known working
packages for L2 scenarios.

Also increases overcloud disk +400MB to accomodate more vpp pkgs.

opnfv-tht-pr: 122

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

Merge "Updating repository for code containing the smoke tests"

Remove trailing folder from RESOURCES in contrib

It shouldn't be there and it breaks the script.

Change-Id: I0762707a0c55a47b1f576e88c5ff0af979efdfda
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Merge "Open additional ports for OpenDaylight services"

Open additional ports for OpenDaylight services

- port 8101 is open to all karaf ssh access
- port 6633 is open as the old default openflow port that is still
  in use by upstream OpenDaylight system tests. It is still enabled
  by default from the openflowplugin.

Add old openflow port (6633) and karaf ssh port (8101)

opnfv-tht-pr: 120

JIRA: APEX-397

Change-Id: I90a55b1d150758790cfb52231560c8cd722d94ad
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Adding "Real Time KVM" support

Adding the support to run the "Real Time KVM" kernel in the compute
node(s) to reduce the network latency caused by the virtualization of
the network functions. Since the current "Real Time KVM" kernel
doesn't support physical to logical network interface mapping yet, we
need to use the real physical interface name rather than the logical
interface name for all compute interfaces in network_settings.yaml.

Also updates tests for the network settings changes and fixes a bug
where interface was being queried on the controller nics before it
interfaces were validated.

JIRA: APEX-128

Change-Id: Iccb6c9fbdcf7e3b482f4283039ef17c38e0a6df7
Signed-off-by: davidjchou <david.j.chou@intel.com>
Signed-off-by: Tim Rozet <trozet@redhat.com>
Signed-off-by: Dan Radez <dradez@redhat.com>

Fixes syntax bug on mariadb file limit increase

There was a missing ':' in the hieradata value which was causing the
mariadb limit to never get increased.  Also only runs the limit increase
when noha is used.

opnfv-tht-pr: 119
opnfv-puppet-tripleo-pr: 19

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

Merge "Use the more tested networking-odl-v1 in bgpvpn"

Merge "Adding to build cache and removing jolokia tarball"

Adding to build cache and removing jolokia tarball

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

Updating repository for code containing the smoke tests

ci/run_smoke_tests.sh is currently pulling the test code from
the CableLabs repository. Now that OPNFV owns that code, we
should now be sourcing that from gerrit.opnfv.org.

JIRA: FUNCTEST-754

Change-Id: Ib4cd5f3596a891a94383a452eaa78c951d063bf8
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Disabling ONOS from Danube

Integration work for ONOS in Danube was not completed.
Disabling this scenario for this release.

Disabling completed mostly by commenting out sections enabling
easy re-enabling next release once integration work is completed.

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

Use the more tested networking-odl-v1 in bgpvpn

opnfv-tht-pr: 114

Change-Id: Icfa4edab139bcbf7684417649105212cc5f00a2e
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Adding Labels

Labels serve as anchors/ refs when referenced from other documents
This patch adds labels to docs/*/*/index.rst

Change-Id: Ia891cbe2866acceb626730b778585b5620cb2ff0
Signed-off-by: ShubhamRathi <shubhamiiitbackup@gmail.com>

Build cleanups

- moved patch files into patches dir
- python-etcd is packaged upstream

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

Merge "Allow RDO_IMAGE_URI env var to override RDO images source"

Adding notifier topic to ceilometer

opnfv-tht-pr: 116

JIRA: APEX-367

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

Merge "Increases open files limit for mariadb"

Fixes neutron L3 HA

We modify neutron code to allow FDIO to create regular tap ports in
router namespaces.  However, we did not add support for neutron L3 HA,
which required this extra patch to work correctly.

JIRA: APEX-393

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

Allow RDO_IMAGE_URI env var to override RDO images source

JIRA: APEX-381

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

Removes hardcoded packages from dev_deploy_check

The package versions we were downloading as beta are no longer required.
Those versions are now included in epel, so we can just use the repo.

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

Increases open files limit for mariadb

There is currently an issue where the max open files limit is hit with
MariaDB in noha deployments, because it is defaulted to 1024 by system
limits. In HA deployments the limit is bumped to 16384. This patch
introduces a flag to be able to increase the limit to 16384 for noHA
deployments.

opnfv-tht-pr: 115
opnfv-puppet-tripleo-pr: 17

JIRA: APEX-345

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

Fixing the congress post install operations

- adding comments regarding nova's api microversion, please read the
  comments in the code for explaination
- removing the ceilometer datasource creation, ceilometer has been
  replace by gnocchi as the primary telemetry source and there is not
  a gnocchi driver in newton for congress

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

Merge "Removing sctp module load from Apex"

Merge "Adding dovetail installation using docker container"

Removing sctp module load from Apex

Handled upstream by: https://review.openstack.org/#/c/438343/

opnfv-tht-pr: 112

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

Merge "Adding endpoint url definitions to congress"

Merge "Adding endpoint url definitions for tacker"

Adding dovetail installation using docker container

This patch adds dovetail option to deploy settings, installs
dovetail image to undercloud.

Change-Id: If7a763d9f6b27655dbb1f5855f438ab11bd88a82
Signed-off-by: zshi <zshi@redhat.com>

Add ipv6 support to baremetal virthost networks

This won't be used in the vast majority of cases, but where someone
wants to use an ipv6 external network on the undercloud, this will
be needed.

Change-Id: Ibb41e0fee7c112c19446d96eb195c0df552c90d1
Signed-off-by: Michael Chapman <woppin@gmail.com>

Adding endpoint url definitions to congress

opnfv-tht-pr: 109

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

Adding endpoint url definitions for tacker

opnfv-tht-pr: 110

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

Allows for specifying compute node RAM

Currently we allow specifying ram per Overcloud VM.  If the node is
detected as ODL, we bump the RAM to a minimum of 10GB.  There is a need
to be able to specify the RAM per compute node in cases where we need
controller RAM to be high, but want compute nodes to be low (like CSIT)
- in order to keep the memory footprint as small as possible.

Changes Include:
 - Adds '--virtual-compute-ram' argument that will override the
   '--virtual-ram' param for compute nodes
 - Fixes a bug where if ODL is used and RAM is overridden to 10GB for
   Control nodes, it was also accidentally being set for compute nodes
 - Modifies '--virtual-ram' to be '--virtual-default-ram' in order to
   clarify this parameter sets the default amount of RAM for all
   overcloud nodes which may be overridden by a role specific arg

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

Adding congress support back into Danube

opnfv-tht-pr: 97
opnfv-puppet-tripleo-pr: 10

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

Merge "onos feature install modified for onos 1.8"

Merge "Add support for odl-fdio scenario"

Merge "Adding Quagga to build"

Adding Quagga to build

Builds 6Wind ZRPC, Quagga, and dependencies required for OpenDaylight
with BGPVPN scenario.  Packages are built into RPMs, and left on the
overcloud disk in /root/quagga.  They are then installed at deploy time
if VPN feature flag is enabled.  Note, for proper upstream we should
create a zrpc OOO service, but that can be a follow up patch.  For that
we would also need official packaging/build system upstream somewhere.

JIRA: APEX-357

Changes Include:
 - Build thrift rpm
 - Build c-capnproto rpm
 - Build quagga rpm
 - Build zrpcd rpm
 - Configure bgpd
 - Detect deploy with VPN and install pkgs
 - Enable zrpcd with systemd

Change-Id: I9825694f46aaab48a3d1cd5fc4d9a24c7370e8fa
Signed-off-by: Tim Rozet <trozet@redhat.com>
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Add support for odl-fdio scenario

Changes:
- Kernel parameters are now set through first-boot.yaml during deployment. No custom
  images will be built. Note that all nodes will be configured the same way, we only
  use compute's kernel parameter settings currently (from deploy settings file)
- Add support for VPP interface type in network settings file, it is now possible
  to specify vpp_interface type and uio_driver for tenant nic in network settings
  file. A new example config, network_settings_vpp.yaml is added.
- Add support for odl_l2-fdio scenario.

Limitations:
- Physical NIC names must be specified in network settings file, numbered nic names
  such as nic1 are not supported for fdio scenarios.
- The same kernel parameters will be configured for all nodes. The paramters will be
  taken from compute kernel parameter section in deploy settings file.

opnfv-tht-pr: 104
opnfv-puppet-tripleo-pr: 12
os-net-config-pr: 3

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

Fix docs path in common rpm spec file

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

Merge "onos 1.8 upgradation"

Doc updates for MS6

Updated the docs structure according to directives and MS6

Change-Id: I36e92cbc58328528ebb91ff4f54ee701f5477443
Signed-off-by: Sofia Wallin <sofia.wallin@ericsson.com>

onos feature install modified for onos 1.8

Change-Id: Idc60b0304f8f37d2daa7aa6cd220be7859121352
Signed-off-by: rssaravana <saravanakumar.somaskandan@huawei.com>

onos 1.8 upgradation

Change-Id: I5e9b8dd51be347f08e8ab8205a70121b903347fd
Signed-off-by: rssaravana <saravanakumar.somaskandan@huawei.com>

Merge "Adding Gluon to post-install and bumping version"

Merge "Enables OpenDaylight HA"

Removes cinder quota show in post install

We used this a while back for additional debug output to prove the quota
setting were correct.  Now with composable services, it isn't always
guaranteed that cinder is used.  Since we don't use this output for
anything, this patch removes it.

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

Enables OpenDaylight HA

Enables ODL clustering by default for all HA ODL scenarios.

opnfv-tht-pr: 107
opnfv-puppet-tripleo: 14

JIRA: APEX-390

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

Merge "Adding tacker support back into Danube"

Adding Gluon to post-install and bumping version

Adding the creation of the Gluon dummy network and subnet
to the post installation steps.

Bumping the version of Gluon to the latest.

opnfv-tht-pr: 106

Change-Id: I5123916f0e153cbd1e70e443d9c8af38ac55fdef
Signed-off-by: Georg Kunz <georg.kunz@ericsson.com>

Removes cinder from controller with CSIT env

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

Adding tacker support back into Danube

opnfv-tht-pr: 98
opnfv-puppet-tripleo-pr: 11

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

Adding gluon to the Build and Deploysystem

In order to make gluon really deployable some more things
need to be done:
  - Add pull request 103 which add yaml files which
    point to gluon puppet
  - Rename the scenario to odl-gluon similar to bgpvpn
    scenario
  - Add tht of gluon to overcloud-deploy-functions.sh

JIRA: APEX-379

opnfv-tht-pr: 103
opnfv-puppet-tripleo-pr: 13

Change-Id: I3ccfb5a279f1df21c4da79fc47039b3c0b6e5495
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
Signed-off-by: Georg Kunz <georg.kunz@ericsson.com>

Fixes networking-bgpvpn pkg install

Also includes minor change for tox working directory to move up one dir
(build dir) for congress.  This is due to https://github.com/tox-dev/tox/issues/66

Removes networking_bgpvpn.conf, and uses neutron service_providers to
set.  This is because we do not have upstream patches to configure
bgpvpn conf file via OOO and having 2 definitions breaks deploy.

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

CSIT: adds default floating ip pool

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

Adds default root password to undercloud

Adds default password 'stack'

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

Merge "Use RDO packaged networking-odl"

Merge "Adding yardstick installation using docker container"

Use RDO packaged networking-odl

We currently use networking-odl package from opnfv fds repo, this
is no longer needed as networking-odl is packaged in RDO.

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

Merge "Add gluon scenario config file"

Merge "Fixing apex build system CentOS-7-1511 not found"

Merge "Upgrades os-net-config to stable/danube"

Add gluon scenario config file

This patch adds first version of gluon scenario config file,
as well as introducing etcd service in puppet-fdio and T-H-T.

JIRA: APEX-379

opnfv-tht-pr: 96
opnfv-puppet-tripleo-pr: 9

Change-Id: Icef29520d093e005dba9cbc5cf0e3fd1ed893cf4
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

Fixing apex build system CentOS-7-1511 not found

The build system is broken since CentOS-7-1511 is
not found on the mirror http://mirrors.cat.pdx.edu.

1511 is deprecated as written in:
http://mirror.pregi.net/centos/7.2.1511/readme

Fixes: https://build.opnfv.org/ci/job/apex-build-master/340/console

Change-Id: I20d5cfddebb4be338f8b712bbe4d02dcbab30e5d
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

Adding yardstick installation using docker container

This patch adds yardstick option to deploy settings, installs docker and
yardstick image to undercloud.

Change-Id: I3a2a1d6bcdfe5becd11d6d9f22218837c572d5da
Signed-off-by: Manjunath Ranganathaiah <manjunath.ranganathaiah@intel.com>

Upgrades os-net-config to stable/danube

stable/danube is based on latest stable/newton with our hieradata/dpdk
patches rebased on top.

JIRA: APEX-386

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

Adding networking-bgpvpn; use new neutron-opendaylight-bgpvpn.yaml

opnfv-tht-pr: 100

Change-Id: I5db74e9ec74d6d03fde1650739d934ddfeec45ec
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

Fixing glance-client and rally

Since the project_id is not in the overcloudrc
glance clinet is failing.

Since os_tenant_name is not in overcloudrc
rally setup is failing.

Change-Id: Iaec4031d3419e8380fb8789288e587c44d600971
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

[BGPVPN] Remove l2 in Name; Enable ODL Boron

Since Boron ODL servers always the l3 usecase
so the l2 differentiation does not make sense
anymore.

Depends: https://gerrit.opnfv.org/gerrit/27605

Change-Id: I8bb55387001dfc81845134639ab4e913072f9a27
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

Merge "Fixes yum update on undercloud"

Merge "Changes to Installation Procedure docs"

Fixes yum update on undercloud

We had already disabled this via puppet, but the code upstream was moved
into python.  This patch now disables it in python.

JIRA: APEX-382

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

Merge "[BGPVPN] Remove setup_TEPS"

Merge "fix post-install.sh"

Merge "Some bugfixes in simple_deploy.sh"

fix post-install.sh

If sahara and swift are not configured, the post install script will
fail during removals of endpoints for sahara and swift.

This patch adds checks whether endpoints to be removed exist or not.

Change-Id: I8e417dbe6e51461d7c4ef1e2368865917b2f8279
Signed-off-by: Ryota MIBU <r-mibu@cq.jp.nec.com>

[BGPVPN] Remove setup_TEPS

with boron the setup teps script is not needed anymore

Change-Id: I72cbd2201089ede4fca4ece63c126b7f4f826f1e
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

removeing congress requirements.txt

- upstream removes requirements.txt during setup like this
  seems we're getting bitten by letting pip try and handle
  deps. This shoudl fix it for now and we can clean out this
  spec once upstream distgit lands

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

Some bugfixes in simple_deploy.sh

Change-Id: I553beedc6195dd4714ba6cb388036da2bd7a355d
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

Change commands to print PublicVip to fix format issue.

Latest 'openstack stack output' commands prints output with
ascii table format, so opnfv-deploy output breaks as following:
"Overcloud dashboard available at http://+---------"

This change fix it with json+jq.

Change-Id: I594f31254caeee1dc2bc3c6d39d9640cba039f05
Signed-off-by: Tomofumi Hayashi <tohayash@redhat.com>

Removes previous patch for osc auth

This should be fixed now upstream

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

Merge "Adding installation of ansible and dependencies to host attempting to execute <apex>/ci/run_smoke_tests.sh. Also fixed the filename that contains the test stdout/stderr output from snoke-tests.out to smoke-tests.out"

Adding an easy install Script

This script is for new commers. It deployes apex in the most
easiest way. People can just execute it and see what happens.
As well the script is so short and easy that they directly
see what needs to be done.

Change-Id: I02c9773ad6df154c735024de2ffd02254c8f67bb
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

Fixes default floating ip pool

We need the default pool to match the default external network we create
so that using nova to create floating ip pools without specifying the
newtwork will work in temptest tests.

JIRA: APEX-380

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

Revert "Locks vpp to 17.01" and correct python-api pkg

This reverts commit 3237c8cd2fd50d72a867c9bd5a58ed1a9f43f1a8.

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

Locks vpp to 17.01

This is needed because honeycomb is incompatible with 17.04 vpp.

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

Merge "Fixing typo"

Fixing typo

Command argmument needs two dashes.

Change-Id: I6582dec83f68fe91b0f221ff610aa4149547b32a
JIRA: APEX-296
Signed-off-by: Mark Beierl <mark.beierl@dell.com>

Updating Congress version to Newton

- Congress is not in THT now so this will not install yet
- Removing doctor driver sideload in build
  The doctor driver is upstream in Newton. No need to side load the
  driver into the congress codebase anymore.
- moving to upstream puppet module

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

Merge "Updating ENV var names to better reflect what they point to"

Merge "Adds declaring disk device to use on overcloud nodes"

Updating ENV var names to better reflect what they point to

- Also removing a stale file

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

Merge "Fixes ODL spelling error on carbon"

Merge "Updating stack-debug to use stack failures list"

Merge "remove odl sdnvpn config file"

Fixes ODL spelling error on carbon

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

Adds declaring disk device to use on overcloud nodes

Now in inventory file a user can declare 'disk_device' which will allow
the user to specify which disk on their overcloud server to use for
installation.  The variable can be a comma delimited list, which will
search in order post-introspection for the first device on the node that
exists.  The default disk used will be sda for deployments.

Currently defining a per node disk is not supported by OSCLI, so
although defined that way in Apex inventory, only the last definition
will be used for all nodes.

Other changes include:
 - Various fixes in inventory parsing
 - Makes bash writing a common function
 - Introspection now enabled for baremetal deployments

JIRA: APEX-296

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

Updating stack-debug to use stack failures list

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