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
stack.sh fails for Cisco plugin with missing cisco_plugins.ini error

When the Cisco Nexus monolithic plugin is configured in the localrc,
stack.sh fails with the error message:
     sed: can't read /etc/neutron/plugins/cisco/cisco_plugins.ini:
          No such file or directory

This failure was introduced with the merge of change set 76992
(bug #1285884). Change set 76992 removed the copying of
Q_PLUGIN_EXTRA_CONF_FILES from the neutron project area to directories
under /etc/neutron. This copy from neutron to /etc is required by the
Cisco Nexus monolithic plugin, since this plugin follows the original
model for generating extra config files based on localrc settings:
 - Copy default config file(s) from neutron project to /etc using
   a relative path (relative to neutron and relative to /)
 - Modify the copies of the config file(s) using localrc settings
 - Add a --config-file command line setting for neutron server which
   points to the modified config file(s) under /etc

The fix is to add the copying of the default versions of the extra
config files from neutron to /etc into the cisco devstack script.

Change-Id: Ifbae197ed591f7cccfd6d51eae2b21e31eb0a409
Closes-Bug: #1295411
Remove keystone_ec2_uri flag from lib/heat

As per I9908e17bed33fdc64d058a6d6db7b29f9c8d53d6 keystone_ec2_uri
is now derived from auth_uri. Thus keystone_ec2_uri is useless
now

Change-Id: I97954fdbbd74a6920b307f33873bcc8d70331bf4
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
Delete unused neutron port

For purpose of creating fake baremetal nodes environment for ironic
functional/integration testing the unused neutron port to be created.
Adding the removal part and helpful comments.

Change-Id: I855ba1dbbf13f343d513ba387716996d6e4c20a6
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
increase heat guests

heat is booting a real fedora, which makes m1.tiny completely not
appropriate. We want to limit disk io, so boot as a 2 vcpu 1G guest
to try to get some better performance out of these guests when
tested in tempest.

Change-Id: I80de67080a5106c2d8921e146c37c614ee5f8abf
Fix typo in nova_admin_username

The last parameter is unneeded, the value is always
set to "nova".

Change-Id: I7167d3559e8dfdb51ac8264e274f7ba167c18dfe
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
Add missing function neutron_plugin_configure_service to
brocade specific file

Closes bug: 1304701

Change-Id: I18f90865a015b9dc6db4123d46492140ca67a38f
Merge "Inject the domain name for devstack in tempest"