Skip flavor_access_add/remove related tests

This patch add skip decorator for flavor_access_add/remove related tests.
After use project instead of tenant in the flavor_access extensions, will
fix those tests.

Related patch:
https://review.openstack.org/#/c/58450/

Change-Id: Ibf741059c3a7a94543f1a2230fe7048692e1db0c
Related-Bug: #1265416
Add list server filter with extra limits

Add list servers filter with zero and exceeding limits

Change-Id: I81bfd831025af1ad42e26ef8a25985c06120aaf7
Fix logic for config file env variables

This commit fixes the logic around the 2 env variables to set the
location of the tempest config file. Previously the check to see
whether the config variables should be used instead of the default
path would be too selective and the default path etc/tempest.conf
would be used. The only check needed is whether the specified file
exists or not. The presence of the env variables shouldn't affect
that logic.

Change-Id: Id6be0f067673d600bf04dea700e69c2baffb9d9f
Fix attach_interfaces tests

Tempest's interfaces_client was not calling Nova "os-interface" API correctly.
The POST parameter interfaceAttachment was ill formed.

Change-Id: I3db0381d48919c62f3f3c6d60fc14cf4dee9c4a3
Closes-Bug: #1271642
Remove baremetal xml support

Ironic decided to not to support xml in API, therefore we remove
corresponding workpiece from the tests.

Change-Id: I35849526778e3ad9428ee9c791a78cf0b86b190f
Do not assume volume metadata is identical to POST request

This change attempts to fix bug 1264418 for Cinder; metadata can
be altered by create/update so it's wrong to assertEqual.

Change-Id: I05d13996fb5dd8c12a81576d7a4fffe2aa198dcc
Partial-Bug: 1264418
Update 'Member' with option 'operator_role' in tempest.conf

In function _get_object_storage_client(), it is better to use the
option 'operator_role' in tempest.conf than hard code the role
to 'Member' here.

Change-Id: Id1de7ee55159f6583a0a93ee87e26a5deaacfd6e
Closes-Bug: #1267857
Cleanup exceptions

There are some invalid usages about exceptions. They put some invalid
error messages to the log like this.
  --------------------------
  BuildErrorException: Server %(server_id)s failed to build and is in
  ERROR status)
  --------------------------
This commit fixes them.

NOTE: 'RateLimitExceeded' is only used in tempest/common/rest_client.py.
 This was refactored in I6c38c343618f92e0e79843aaa662a3d24026702b . But
 the class was not. And the last usage of 'SQLException' was removed in
 I0fac7b030c51985f9d6d93129bf9bab75c18cd11 . But the class was not.

Change-Id: I1477e093a56210c0c9a65fa32900c2c0b876d9a4
Fixed up an error message

Removed the comma and added spaces

Change-Id: I04977f955ed6a4575fea17b8e97b675a483cc7a9
Move common _delete_volume cleanup method in base compute test class

The test_volumes_get and test_volumes_list v2 compute tests were doing
similar volume cleanup.  This patch moves that code into the base class
so other tests that need to do volume cleanup, i.e. test_server_rescue,
can use it.

Related-Bug: #1254772

Change-Id: I59f4d638f8ab6e95736c78a384607ca89f1dcb67
Removed unnecessary lock from a aggregate test

Lock is unnecessary here, as the test is not adding
a host to an aggregate.Previoulsy this lock was
added to ensure the failure will not occur where
a host is added to an availability zone.
Reference:-
Iaf8a903d92c88c7c3092eaa00a9f7a96e6348ddc

Change-Id: I9d6d610302406d5aa7bee9135c27dbc83896e005
Move negative tests for v3 test_server_metadata

Move negative tests from test_server_metadata.py to its negative file

Partially implements blueprint negative-test-files

Change-Id: I02a05828e26be357728ff73adf75b469fce1b594
test_rescued_vm_add_remove_security_group missing addCleanup

test_rescued_vm_add_remove_security_group on failure caused
other test cases fail.

Change-Id: I17ebb23f31bc10d669208b64a66e89fa415f96ac
Convert compute api tests to global CONF object

This commit takes all the uses of config in the compute api tests and
converts them to use the global CONF object.

Partially implements bp config-cleanup

Change-Id: Iac92d6ac88200222392a131b4c95411594c509ba
Convert image and identity api tests to use global CONF object

This commit takes all the uses of config in the image and identity api
tests and converts them to use the global CONF object.

Partially implements bp config-cleanup

Change-Id: Ic305ce82d2bd5d0ef8293fe25c6a4c835633b0e8
Convert network api tests to use global CONF object

This commit takes all the uses of config in the network api tests
and converts them to use the global CONF object.

Partially implements bp config-cleanup

Change-Id: I4fc99d44073598513653cd87fd73a82a0b0093da
Add bool and integer support to XML parser

XML parser always returns value of all simple types as a string.
This change makes XML parser to convert it to boolean/integer/long
value if necessary.

Change-Id: I350178ba9820dd2579cf8063f7dedb756c651ab5
Add alias as prefix for flavor_rxtx v3

As v3 api rules, the extended attribute should be with extension
alias as prefix. So fix this case for flavor-rxtx v3 extension.

Change-Id: Ie96f88434563e50de4862358be068b975b07cb05
Related-Bug: #1226732
Removes vim headers 4th round

This is a follow up on:
   Change-Id: Ie8097db6c60481749c88991864187ade64f39ecb

The vim headers in all the files are being removed. We've missed these
files because of delays in the patches.
I hope this is the last one for Tempest.
Closes-Bug: #1229324

Change-Id: If56b28d0efb6364fe2105f7d62d6a2b05b906070
Merge "Update 'Member' with option 'operator_role' in tempest.conf"