Deduplicate status call checks

This commit updates the jsonschema validation method validate_response()
to use the rest client method expected_success() to check that the
correct status code response is received. Previously validate response
was doing the same check despite there already being a method available
for doing the check. As a part of using expected_success() to validate
the status codes using the jsonschema it was necessary to expand
expected_successs() to handle both single expected codes as well as a
list of codes. This is because it's perfectly valid for the jsonschema
to specify a list of expected responses.

Change-Id: I400bb9aff89c18f3964a1d8ec45bfb8dfd75301d
Enhanced Trove (database) flavor API tests

 - Added test to make sure trove flavors matched OS (nova) flavors
 - Added response status checks to existing tests

Partially implements: blueprint trove-tempest

Change-Id: I59cc551ef5ec58fe43b794ae8dbd6a89c40c774b
Reworked scenario tests to use their own non-default security groups

Moved _create_security_group_nova to base OfficialClientTest class, so
that now it can be used by all scenario tests. All scenario tests now
create security group with "loginable" rules explicitly and attach it to
servers instead of using "default" security group.

Change-Id: If82c681b4a7f62808f46b03c57d113f8f5c97f6f
Closes-Bug: #1247060
Add unittests for remote client

Remote client is used in servers tests to provide some common operations into a
remote server. It's backended by the ssh class, which already has tests, so all
of its calls are mocked and due to that there are some 'useless' tests that are
pretty much checking response's format and checking if the ssh call didn't
change to avoid inconsistencies in the future.

Partially implements bp unit-tests

Change-Id: I5dabc170a557c81bb9b0605bf101719585d7c53b
Adds test for Floating Ips bulk Nova V2 API

This patch adds the test for following Nova V2 APIs-
- Create Floating Ips bulk
- Delete Floating Ips bulk
- List Floating Ips bulk

This patch also adds the config option for unallocated
Floating IP Range

Change-Id: I6af52bb2ce2caadf8491740097df71d041643e78
Add total runtime to summary output

This commit adds the total runtime in secs to the summary view from
subunit-trace.py

Change-Id: If56f2a048b3d1ab3d51425d5b3560e1f365a6c3a
Set default to false for nova v3_api config option

This commit switches the default value for the nova v3 api config
option from true to false. This changes the default behavior to not
run the Nova v3 API tests. This is because the v3 is still marked as
experimental and is able to change in a non-backwards compatible way
which will cause issues when running with branchless tempest.
Additionally, the nova team decided not to do a standalone v3 api so
at some point this will disappear and instead become v2.1 of the api
with microversions. When you couple this with the fact that tempest's
gate budget is basically exhausted (tests shouldn't take over an hour
to run in the gate) this should be defaulted off. For more discussion
see:

http://lists.openstack.org/pipermail/openstack-dev/2014-June/037370.html

Change-Id: I13dbc4b433a0f6defbeebce958b2e1cbcf64ba7a
Unbreak baremetal API tests that were not updated

A recent merge (88233) changed the baremetal test base to return (resp, body)
tuples from its create_* methods instead of the created objects.  This updates
the test cases that were missed in this patch and unbreaks all Ironic CI jobs
that were overlooked in that review.

Change-Id: I691e36d0678a15737614b7867a4c6778e714f299
Closes-bug: #1330834
Merge "Enhanced Trove (database) flavor API tests"
Merge "Add unittests for remote client"
Merge "Set default to false for nova v3_api config option"
Merge "Unbreak baremetal API tests that were not updated"
Merge "Add total runtime to summary output"
Merge "Reworked scenario tests to use their own non-default security groups"
Bump hacking to 0.9.x series

Hacking 0.9.0 had a minor bug so require 0.9.1 or higher.

In order to keep this patch to just a requirements bump, ignore new and
stricter hacking rules that are being triggered. Fixing up the code and
turning these on is out of scope of this patch and is for future patches.

Change-Id: Iafbd4adae8c6b446d53b62dfc490eab8e76a43ac
Merge "Deduplicate status call checks"
Enable E113,E111,H302,F812,E713 rules

After bumping the hacking version to the 0.9.x series ignores were
added for several rules. This commit fixes the violations for a subset
of these rules and re-enables the checks.

Change-Id: I8cc92a5de5acf43a2143751851cf6fddab401ba4
Merge "Bump hacking to 0.9.x series"
Merge "Enable E113,E111,H302,F812,E713 rules"
Merge "Adds test for Floating Ips bulk Nova V2 API"