Make unable to ping log messages appropriate level

The ping from an ssh connection checks in _check_remote_connectivity()
aren't necessarily expected to pass on the first attempt. However,
every time the ping was unsuccessful the method would log an
exception which is too severe a log for an expected error. This
commit uses a more appropriate log level for this case.

Change-Id: I8068fa85472259676acebd99b9d191065661e101
Move baremetal API tests to an /admin subdir

To avoid confusion, move all baremetal API tests to api/baremetal/admin and
add a README.rst explaining that the API provided by Ironic is meant to be used
as an admin and service level API.

Change-Id: Ib49b6d5937da6eedc3028a1f831552cf02d9f69e
Update compute attach tests to use "volume_device_name"

Some attach tests in the compute suites are using hard-coded
values for the volume device name. These should be replaced
with the compute.volume_device_name in tempest.conf.

Change-Id: I2f5da22b5f1bbd3910e4226b96e73106aa75c65a
If no volume, don't delete volume in teardown

If exception is thrown in setupClass before volume is created,
tearDownClass needn't delete volume.

Change-Id: I91c7eff8ec147b0ce3a9ea3e6c8d906ba68412c2
Closes-Bug: 1350167
Add min_client_version decorator for CLI tests

This adds the min_client_version decorator to be used in the CLI tests.
The decorator is used skip CLI tests where the command being tested
isn't available in the version of the client we're testing against.

The cmd method is pulled out of the ClientTestBase class so it can be
used by the decorator and is renamed so it does not conflict with the
cmd argument passed to it.

To start using the decorator and show how it works, two known CLI tests
are decorated for minimum version checks on their clients.

Part of blueprint minversion-check-for-cli-tests

Change-Id: I8a0ee390048870d450772fa064d0dc2daa2b7036
Add a new credential provider to use a list of creds

This commit adds a new credential provider to use credentials provided
from a list of available credentials.

Partially implements bp test-accounts
Change-Id: I69c3105f8b413102ea567ccb92965572ce03d679
Add image-list to glance cli help test

In the glance client, there is 'image-list' that is not deprecated.
So this should be added to wanted_commands.

Change-Id: Ic14a9aa1ac24a7756e390a4f6e26e3af2952a52d
Stop leaking identity resources in teardown_all()

teardown_all() tries to delete all resources. But if an exception
happens, the rest of resources are remained and it seems to happen
frequently.
This commit fixes it.

Partially Implements: blueprint stop-leaking
Change-Id: I671b2262ea5ed5bf7d78149cf88f2894b6bf6508
Add interface_attach compute feature flag

Dynamic attachment of network interfaces is not currently a supported feature
in the baremetal case.  This adds feature flag to skip related API tests.

Change-Id: Ia09e84d7acfcd7291b3c9b58bc163603d6a0fb34
Do not isolate networks for baremetal

Network isolaton is not currently a supported feature for baremetal
deployments.  This adjusts the isolated credentials to avoid creation of
isolated network environments for created tenants.  Instead, we rely on
a shared private network serving addresses from a common subnet.

Change-Id: I34b16ed5a6f9221707f3a9d0d4619bb3b261e14f
Router API Tests Enhancements

Add test to creation of router with snat rule
  - Create a router with default attributes
  - Verify the default snat attributes
  - Creating a router using admin priviledges for a tenant
     with snat value True/False
  - Verify routers for snat value True/False by overwriting
     through admin tenant.

Change-Id: Icf2f56f7aa7a2dc736c57daad6d2d5c37e0d9e32
Fix typo in comment

Fixes a typo in flavor response validation.

Change-Id: Ib492e291f5c64a18256cefbda11f81ff33f03861
Merge "Add image-list to glance cli help test"
Merge "Do not isolate networks for baremetal"
Remove skipping flavor_access_add/remove related tests

At change Ibf741059c3a7a94543f1a2230fe7048692e1db0c, skip the
flavor_access_add/remove related tests for changing 'tenant' to
'project' in nova v3 REST API. But the v3 API plan is changed,
this kind of change will be introduce back when we have micro-version.
So remove the skip decorator for now.

Change-Id: I9ba1f0c10b7e976df303a9800bea2eea62f43891
Remove skipping test_list_servers_by_admin_with_all_tenants test

At change Ibb71c3a966f6c09b93e821781d4266d56d65950b, skip the
test_list_servers_by_admin_with_all_tenants test for changing
'tenant' to 'project' in nova v3 REST API. But the v3 API plan is
changed, this kind of change will be introduce back when we have
micro-version. So remove the skip decorator.

Change-Id: Ife983aab3cd79531c18ba5104a6dd8f1aaca6cd4
Merge "Add interface_attach compute feature flag"
Merge "Fix typo in comment"
Merge "Move baremetal API tests to an /admin subdir"
Merge "Router API Tests Enhancements"