Fix the order of assertEqual arguments(keystoneclient, kvs, etc)

assertEqual method's arguments should be in
('expected', 'actual') order.

Change-Id: Ic429c9264a823c3b03ca3b5c898b4aeb96891504
Partial-Bug: #1277104
remove the unused variable in test_sql_upgrade

There is an unused variable in tests/test_sql_upgrade.py. It needs
to be removed.

Change-Id: Ia50a4e91319d1a4dce9515c8b910de28fa68fcb5
Add slowest output to tox runs (testr)

Add --slowest for all tox test runs that don't use other various
test commands.

Change-Id: I9f9fdcdda8895ba33c817f87b566794dbcd6ccb9
Remove unused db_sync from extensions

The db_sync in the extensions isn't used, either by the tests or
by keystone-manage db_sync, so remove them.

Change-Id: Ia0598e8dae81e258a05767f47525d5c11acce8db
Reduce environment logging

The log message for configuring the environment as eventlet was
at the info level. Operators don't find this detail interesting
so this should be logged at debug level.

Change-Id: Ic78c5af0c781f6e1e427dbf40da0a8bee9762ea2
Make service catalog include service name

If a service name is available then it should be included as part of a
token's service catalog. This functionality was present in the V2 api
and is already relied upon by various clients. It will make it much
easier to transition these clients if we can keep this functionality.

Change-Id: I0d824d73c375ffbc5a6b44c10992967f59742b01
Closes-Bug: #1288098
Clean up config help text

These are suggestions to clean up the help text.

Change-Id: If22c6491b530da92ee1ce6411b57f0585817de38
Cleanup config.py

Trailing , on last entry for each section.

Put ] on its own line.

This will make reviews look better since don't have to change as
much to add a config option and also lead to fewer merge conflicts
since fewer lines are changed.

Change-Id: I58d8d826404f0b13975f114ddb3465f2773d6942
Enable concurrent testing by default

Enable concurrent testing by default, 1 worker per available core.

Change-Id: I552bca32d953833a48fcf70ccc0daaf50e2aa09a
Keystone doesn't use pam

The requirements.txt file had pam, but Keystone doesn't use pam.

Change-Id: Ib617146d4ec75532f994ae85837ecbf383647306
Remove _delete_tokens function from federation controller

There was a TODO to delete tokens based on idps and mappings, but
since this will be done at the middleware level of keystoneclient
(bug 1291157 then there is no include this piece of code.

Change-Id: I0e198c2b7a05e283409aef25e1d7ab93247814fe
support conventional domain name with one or more dot

Formal domain name will have one or more dot within the name,
meantime, it's common for one company own several domains,
for example abc.com and abc.net. But currently keystone
multi-domain support only supports domain name without '.',
because the wrong way that get the domain name from domain
config file name.

Change-Id: I152cd3b513353e3d4c565e71c8755789d24e1a0e
Closes-Bug: 1257171
For ldap, API wrongly reports user is in group

When the ldap identity backend is configured,
HEAD v3/groups/​{group_id}​/users/​{user_id}
always returns 200, even if the user is not actually in the group.
This is because the sql and kvs backend will raise NotFound
exception if the user is not in the group, but the ldap backend
just return result.

Change-Id: Ie1585c8aebe054091bd76fded666bf41125ff9ca
Closes-Bug: 1245247
Fix create_region_with_id raise 500 Error bug

1. add sql.handle_conflicts wrapper in sql backend driver.
2. check duplicate region id in kvs backend driver.
3. add 2 test cases to check exception.Conflict when create duplicate id region.

Closes-Bug: #1283904

Change-Id: Icfc5edf661501a5c134f324a76824812dbda2760
Merge "Fix the order of assertEqual arguments(keystoneclient, kvs, etc)"
Merge "Enable concurrent testing by default"
Sanitizes authentication methods received in requests.

When a user authenticates against Identity V3 API, he can specify
multiple authentication methods. This patch removes duplicates, which
could have been used to achieve DoS attacks.

Change-Id: Iec9a1875a4ff6e2fac0fb2c3db6f3ce34a5dfd1d
Closes-Bug: 1300274
Merge "Remove _delete_tokens function from federation controller"
Cleanup of test_cert_setup tests

- Removed the redundant calls to load backends and fixtures
- Only create a controller in the one test that needs it

Change-Id: I6c068452479ceb23010396f14fe6d155ac954b04
Removes useless wrapper from manager base class

There is no value to adding this extra layer to each method call.
Instead just return the boundmethod object itself.

Change-Id: I0eddb9fcb25bf9e20ae60d200c9538fbc426123c