Clean up pep8 E125 violations

Fixed E125 errors
E125 is ignored because of false alarms
All ignores are to be removed in the next sequence of patches

Change-Id: Id9ce27d1348a9318e183591db79232a01731f8ad
Add XML support for test_security_groups.py and test_security_group_rules.py.

Change-Id: I5d521acf116e122c7b5608992c33854caad30bab
Put skip at top level.

Change-Id: I379e4364b780e7254b9ee8458c1f5603cd2bb2b6
Merge "Put skip at top level."
Refactor list servers negative test and fix remaining gate bugs

test_list_servers_negative had so many issues with it, it's hard
to list them here... but at least the following has been fixed:

* No longer will the test flatly delete all instances the tenant
  and alt tenant have during every test method run (!!)
* Fixtures -- 2 active and 1 deleted server -- are created in the
  class' setUpClass() method instead of having test methods
  constantly re-launch instances -- this reduces the test time
  down about 3 minutes
* Removed all non-negative tests
* Removed all tests that were skipped due to bugs that have been
* Skip the XML create_service() test for Identity Admin tests
  because of the upstream Keystone bug 1061738

This patch also includes patches for bug fixes that Sean Gallagher
contributed but needed to be rebased into this one in order
to get the full gate to pass.

Change-Id: I0b38cf045520d93031c7916dae2479daf69d0411
fixes: LP bug #1059802
fixes: LP bug #1060373
fixes: LP bug #1061740
fixes: LP bug #1061167
Tempest tests to cover live-block-migration

Related to bug 1051881.

Two test cases: one to cover the live block migration on XenServer, and
the other to indicate that the machine status is left in an inconsistent
state in case a non-existing host is specified as target.
To run the live-migration tests with block migration, include:

    [compute]
    live_migration_available = true
    use_block_migration_for_live_migration = true

in your tempest.conf

To run these tests, the hypervisors must support the Storage XenMotion
feature.

Change-Id: I1e6cc903fa573ecd219a08bf4dddd9cc1f4f60df
Added missing import for SkipTest in test_authorization.

Fixes bug 1058281

Change-Id: I41d706aeb67c9555bee0113247855b4a077775eb
Handle ImportError's when quantumclient is missing

Updates tempest/manager.py so that it ignore's ImportError
exceptions when quantumclient is not present.

Not all Tempest tests require a quantumclient so it makes sense not
to require it all the time. If a user tries to run quantum specific
tests they will simply get an error when trying to create the
network client instead should quantumclient not be present.

Change-Id: Iea5476eea91e0d92c3b4728f041a3ce72bb5a320
Fix 'message' is not defined errors.

Updates the Tempest common rest client so that it sets the
'message' variable value in all cases. Previously some exceptions
could slip through and cause 'message' is not defined errors.

Change-Id: I09ec8a311fd60ab3b625f41a8eb4147fe5f777eb
Initial add of Swift tests

* Added Swift client
* Added Swift configuration
* Added basic Swift tests for objects, containers, and accounts

Change-Id: I92ac9f69f3e8fe227dff55a2a5b55001f37acb73
Merge "Handle ImportError's when quantumclient is missing"
Merge "Initial add of Swift tests"
Merge "Tempest tests to cover live-block-migration"
flavors with disk sizes of 10 10 20 30 would fail unexpectedly before since flavor[1]['disk'] == flavor[0]['disk]

ammended for trailing whitespace

Fixes bug 1066121

Change-Id: Ic941d35a0d92ae37d0a8cb2123165f19392b0c8e
General cleanup/organization of compute tests

* Renamed services/nova to compute
* Added OpenStack license where missed to compute services/tests
* Broke down compute tests into separate packages by functionality

Change-Id: Ibb1f3ec74b83816205a0bed7d0e062ebf9677525
Remove kong. Fixes bug 1052511.

Change-Id: I5e027c9539fd2c594d24447866e79c8caad9aa40
Fix SyntaxError: invalid syntax - comma.

  File "/var/jenkins/workspace/openstack-tempest_on-slave_testing/tempest/tempest/services/object_storage/account_client.py", line 36
    headers = {"X-Storage-Token", self.token}
                                ^
SyntaxError: invalid syntax

Change-Id: I850aba75a28edfd2990accadfb0f3e7115eba864
Remove unnecessary test. Fixes bug 1072841.

Providing a bad status is no longer a failure so this case is not needed.

Change-Id: Id0b2282bf23c3ccd50e967111e943f93fc4ea683
Added Swift tests:
      * account: delete account metadata
      * container: retrieve/delete container metadata
      * object: retrieve/copy(2 ways) object
Syntax bug fix in container_client.py:
      return resp. body => return resp, body
Fixed passing headers parameter in head method in rest_client.py:
      return self.request('HEAD', url, headers)
Removed unused imports.

Change-Id: Ica67d6ecae80f851e9e05ebeb4c7d461eba143ee
Make assertion failures more informative.

The assertIn, assertEquals and other assertions will output more information
than assertTrue, which outputs:
False is not True
on failure.

Change-Id: Ibbf90a479174d16cac5361519643b55744678a84