Update client-env to use openstackclient commands

Updated the only instance of a keystoneclient command, to
check if the identity service is enabled.

Change-Id: If86f71c1610a79690d6c6a8eb423b6fa234372bb
Avoid pm-is-supported spam in logs

Per the libvirt team, the only way to stop the spam in
libvirtd logs is to install pm-utils package. Details are in:

https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/994476

Closes-Bug: #1274772
Closes-Bug: #1297512

Change-Id: Ia2f7a388470dcd8a3e2c12cf2980beccd65af94c
Default RABBIT_HOST to be the SERVICE_HOST.

Using localhost for RABBIT_HOST can cause problems - particularly
on systems where IPv6 is partially broken and localhost resolves
to an IPv6 address.

The service should logically be running on SERVICE_HOST unless
otherwise overridden.

Change-Id: I6efb71b8b95174770066f38403d5d428ec88c3b6
Devstack support for Neutron VPNaaS

Includes two enhancements to support VPNaaS and allow other
vendor implementation support. Goal is to provide a solution
for VPN .ini files until a more general solution can be defined
to handle all reference and vendor .ini files.

First, the /opt/stack/neutron/etc/vpn_agent.ini file is copied
to /etc/neutron, to allow the selected device drivers for agent
to be specified. Both reference and vendor implementations can
be uncommented in this file, prior to stacking.

Second, to allow vendor VPN implementations to provide an .ini
file to the agent at start-up time, an environment variable is
defined and can be set in the localrc to specify one or more
.ini files. The desire is NOT to include vendor settings in the
global vpn_agent.ini.

Change-Id: I5370ee57bbe326eff505a53055da4fddc0838de4
Closes-Bug: 1301552
Enable colors for Sahara log

Change-Id: If915be21a7090c93e0be4c1961058c155ed9ca2c
Set flag in Tempest enabling Ironic scenario tests

When Ironic has been configured as the backing compute driver,
set the corresponding flag in Tempest that will cause a corresponding
scenario test to run.

Change-Id: I2bb6009c338cb301bd332e640a845fc1f6cb9336
only remove network states from Nova when n-net is enabled

When n-net is not enabled but q-dhcp is enabled then it's not
necessary to remove the states directory of nova-network.

Change-Id: Ib4b37aa7df2b3cda3048b278ddad24e87e3744c9
Fix openvswitch package and service names for openSUSE

Change-Id: Iaf173422e4d009bdd7ed7ed79af2269c44039060
Set parameters for OpenDaylight ML2 driver

Now that https://review.openstack.org/#/c/85589/ is merged, the
OpenDaylight ML2 driver requires that url, username and password
parameters are set.

Change-Id: Iecbdd3275387bea145a5d274d359a15eceaee6b7
Closes-Bug: #1304226
Set a correct whitelist for drivers in Ironic

Ironic recently introduced a feature called Drivers
whitelist. It allows to specify the drivers that are
enabled.

The default list is not appropriate for functional and
integration testing so devstack needs to be updated.

Change-Id: Iccc81343f4d5a7dec8b371934a8c9849d9ffac23
Enabled non-root users to read nfs_shares.conf.

If you use the cinder NFS driver, it will create a file called
nfs_shares.conf, containing a list of NFS shares cinder can
use.  The file is created with 660 permissions and owned by
root:root.  When c-vol starts, it has problems reading
nfs_shares.conf, resulting in a "[Errno 13] Permission denied:
'/etc/cinder/nfs_shares.conf'".  This patch creates
nfs_shares.conf without using sudo, so c-vol can at least
read it.

Change-Id: Ifce2bd4b9ff2191445a1964c1ce09c823d096d68
Closes-Bug: #1297916
euca-bundle-image requires a new architecture parameter

Ran into this issue while trying to enable UCA. This switch was
documented by the euca2ools team in:
https://eucalyptus.atlassian.net/browse/DOC-807

Closes-Bug: #1304571

Change-Id: I1ea7c6a760c75e67f09e566d73e571be3a1d89ea
Restart only nova-compute when deploying Ironic

lib/ironic redundantly restarts *all* Nova services as part of its bootstrap.
Only nova-compute should be restarted to ensure its resource tracker has picked
up newly enlisted Ironic nodes.  This change limits the restart to only
nova-compute and triggers it after nodes have been enlisted in Ironic.

Change-Id: I87a508d6611a16fa6b966ab18e15aa9fbcc5e8c7
Inject the domain name for devstack in tempest

A new setting for the keystone domain name is introduced in
tempest by change https://review.openstack.org/#/c/81872/.

The default value in tempest is set to None, so that it is
not devstack specific. It's up to devstack to configure
the name of the keystone domain defined in devstack.

Partially implements bp multi-keystone-api-version-tests

Change-Id: I3d1406407759e1fdce0704fa2d6aa025f23a5f08
Merge "Default RABBIT_HOST to be the SERVICE_HOST."
Merge "Inject the domain name for devstack in tempest"
Merge "Devstack support for Neutron VPNaaS"
Merge "Enabled non-root users to read nfs_shares.conf."
Revert "Devstack support for Neutron VPNaaS"

This reverts commit 8450a92a4d0299fff89717e665ce31d54731d3f0.

Change-Id: I6dd0f44a1a57735de292bc4ed4eaa929dc308297
Merge "Set parameters for OpenDaylight ML2 driver"