summaryrefslogtreecommitdiff
path: root/keystoneclient
Commit message (Collapse)AuthorAgeFilesLines
* Replace OpenStack LLC with OpenStack Foundation0.4.1ZhiQiang Fan2013-10-1734-34/+34
| | | | | | | | | | Some files still use trademark OpenStack LLC in header, which should be changed to OpenStack Foundation. NOTE: tools/install_venv.py is not touched, should sync with oslo Change-Id: I01d4f6b64cf1a152c4e190407799ce7d53de845f Fixes-Bug: #1214176
* Add AssertRequestHeaderEqual test helper and make use of itJamie Lennox2013-10-162-17/+18
| | | | | | | This will be a fairly common pattern and so should be a helper of its own. Change-Id: Ic06add6dfe40c7d3ab88dfdd99a9087fdebd9b67
* Merge "Allow v2 client authentication with trust_id"Jenkins2013-10-154-4/+29
|\
| * Allow v2 client authentication with trust_idSteven Hardy2013-09-264-4/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It should be possible to authenticate against the v2 tokens API with a trust_id, because it suports rescoping an existing token to a trust, this patch adds client support for this. Note with the current keystone code it's necessary to pass the trustor tenant ID when rescoping with a trust where impersonation==True, e.g: c = client_v2.Client(username=TRUSTEE_USERNAME, password=TRUSTEE_USERNAME, tenant_name=TRUSTEE_TENANT_NAME, auth_url=OS_AUTH_URL_V2) c.authenticate(trust_id=trust_i.id, tenant_id=TRUSTOR_TENANT_ID) Change-Id: I177c41af298b7437e2c6fb437aa9ce9a09773b9d Closes-Bug: #1231483
* | Merge "python3: Make iteritems py3k compat"Jenkins2013-10-156-8/+17
|\ \
| * | python3: Make iteritems py3k compatChuck Short2013-10-106-8/+17
| | | | | | | | | | | | | | | | | | | | | | | | Use six.iteritems to replace dictionary.iteritems() on python2 or dictionary.items() on python3. Change-Id: I972f065414e22d287bd7e00ab2c6e754f17afb75 Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | Merge "Fixes print error for keystone action with non-English characters"Jenkins2013-10-111-1/+3
|\ \ \
| * | | Fixes print error for keystone action with non-English characterschenxiao2013-09-261-1/+3
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | When processing keystone tenant-delete action with non-English characters, printing the exception traceback will result in a UnicodeEncodeError, so should encode text with safe_encode. Fixes bug #1231472 Change-Id: Ic2599c51513189c132e31aa87d0d15df81802688
* | | Merge "assertEquals is deprecated, use assertEqual (H602)"Jenkins2013-10-119-127/+125
|\ \ \ | |_|/ |/| |
| * | assertEquals is deprecated, use assertEqual (H602)Dirk Mueller2013-10-089-127/+125
| | | | | | | | | | | | Change-Id: Iefd087320659909b89864018ccb4ccf50a75aff6
* | | Merge "Normalize datetimes to account for tz"0.4.0Jenkins2013-10-092-7/+236
|\ \ \
| * | | Normalize datetimes to account for tzBryan Davidson2013-10-092-7/+236
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch makes sure that datetimes in the auth_token middleware are normalized to account for timezone offsets. Some of the old tests were changed to ensure that the expires string stored in the cache is in ISO 8601 format and not a random float. Fixes bug 1195924 Change-Id: I5917ab728193cd2aa8784c4860a96cdc17f3d43f
* | | Merge "Fix H202 assertRaises Exception"Jenkins2013-10-094-13/+22
|\ \ \ | |/ / |/| |
| * | Fix H202 assertRaises ExceptionKui Shi2013-10-044-13/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Align the hacking version between test-requirement and global requirement. The change of H202 detection from 0.6 to 0.7 in hacking is: - if logical_line.startswith("self.assertRaises(Exception)"): + if logical_line.startswith("self.assertRaises(Exception,"): then more cases are detected by this change. Fix the exposed H202 error. There is a special test case: tests/v3/test_endpoints.py:test_update_invalid_interface ref = self.new_ref(interface=uuid.uuid4().hex) this line can not generate proper parameter for self.manager.update, add a parameter "endpoint" for it, according to the definition in keystoneclient/v3/endpoints.py:EndpointManager.update. Otherwise, there will be following error after changing the Exception to exceptions.ValidationError: TypeError: update() takes at least 2 arguments (6 given) Fixes Bug #1220008 Change-Id: I8f7ed7a6eebf8576a6db5fecd86b9d19a15c8d60
* | | Merge "Refactor for testability of an upcoming change"Jenkins2013-10-072-23/+45
|\ \ \ | |/ / |/| |
| * | Refactor for testability of an upcoming changeBryan Davidson2013-10-012-23/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | confirm_token_not_expired() in keystoneclient/middleware/auth_token.py has been moved out of the class to make it a function and be more testable. Currently, there is no need to keep it within the class. An upcoming commit makes fixes that rely on this refactor to be tested. Change-Id: I8460a2ee663dec8be0f339735208779a3b988040
* | | Merge "Add auth_uri in conf to avoid unnecessary warning"Jenkins2013-10-041-2/+6
|\ \ \
| * | | Add auth_uri in conf to avoid unnecessary warningZhiQiang Fan2013-09-261-2/+6
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Unittest code for auth_token.py doesn't specify auth_uri in conf, which will cause lots of warnings, since auth_token.py will use '%(protocol)s://%(host)s:%(port)s' to generate auth_uri when it is None, we can specify this value directly to avoid unnecessary warnings. Closes-Bug: #1219493 Change-Id: I26b636bcddabd91b06479c6a42a6a48d74ac9431
* | | Merge "Log user info in auth_token middleware"Jenkins2013-10-021-0/+3
|\ \ \ | |_|/ |/| |
| * | Log user info in auth_token middlewareAmalaBasha2013-09-101-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | Add logging for user information (like user name, tenant_id, roles) in the auth_token middleware. This would make tracking down issues much easier. Change-Id: Ife4ce29d2f8e1a338a025dda4afbd7b563f6b8c1 Implements: blueprint user-info-logging-in-auth-token-middleware
* | | Fix misused assertTrue in unit testsChang Bo Guo2013-09-251-2/+2
| |/ |/| | | | | | | | | | | | | | | Refactored unit tests to use assertEqual instead of assertTrue where needed. Fixes bug #1226374 Change-Id: I678b2e7fcc522c8776c7fc0a554c1fc229ab781e
* | Move tests in keystoneclientJamie Lennox2013-09-2449-0/+8387
| | | | | | | | | | | | | | | | | | This is the suggested location for tests and is adopted by most projects. As part of this change relative imports to package imports. Fix all the test running and coverage code to point to the new location. Change-Id: I01264aed14f396ab9a7242e3e72b71e1bc332675
* | Merge "Allow blank to email in user-update"Jenkins2013-09-191-1/+1
|\ \
| * | Allow blank to email in user-updateYuuichi Fujioka2013-09-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | User cannot set blank to email address when use "user-update". This patch fixes this problem. Change-Id: I3b9494a1c668358a529031c366c55b303f43c533 Closes-bug: #1226439
* | | Merge "Decode the non-english username str to unicode"Jenkins2013-09-181-0/+2
|\ \ \ | |/ / |/| |
| * | Decode the non-english username str to unicodeYangLei2013-09-061-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | convert the non-english username str to unicode, then the converted username can match the username from db. Fixes bug #1221576 Change-Id: I9e5f941fe43f081d75750e3f4754e8beea8210db
* | | Merge "Deprecation warning should be 'pending'"Jenkins2013-09-141-3/+3
|\ \ \
| * | | Deprecation warning should be 'pending'Dolph Mathews2013-09-051-3/+3
| |/ / | | | | | | | | | | | | | | | Until python-openstackclient is ready for more mainstream users. Change-Id: Ic90daa38cd7e8e2f6d6b23c30c3d83935cf6c1a6
* | | Merge "Replace HttpConnection in auth_token with Requests"Jenkins2013-09-131-68/+61
|\ \ \
| * | | Replace HttpConnection in auth_token with RequestsJamie Lennox2013-09-121-68/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Requests is becoming the standard way of doing http communication, it also vastly simplifies adding other authentication mechanisms. Use it in the auth_token middleware. This adds the ability to specify a CA file that will be used to verify a HTTPS connections or insecure to specifically ignore HTTPS validation. SecurityImpact DocImpact Partial-Bug: #1188189 Change-Id: Iae94329e7abd105bf95224d28f39f4b746b9eb70
* | | | Merge "Support client generate literal ipv6 auth_uri base on auth_host"Jenkins2013-09-131-2/+8
|\ \ \ \ | |/ / / | | / / | |/ / |/| |
| * | Support client generate literal ipv6 auth_uri base on auth_hostDazhao2013-09-111-2/+8
| |/ | | | | | | | | | | | | | | | | | | This patch is for fix bug 1208784. Support keystone client generate literal ipv6 auth_uri via provided auth_host, when the auth_uri is None. Fixes bug 1208784 Change-Id: I226881a35a74ef668d4cd1c6829a64c94ff185d9
* | Deprecation warning for the CLIDolph Mathews2013-09-031-1/+5
|/ | | | | | | | | keystoneclient/shell.py's docstr is rendered by argparse as the description for: $ keystone --help Change-Id: I7355cdf533800086027824fb729eb52cdd8bbc33
* Merge "Fix and enable gating on F811"Jenkins2013-08-301-9/+0
|\
| * Fix and enable gating on F811Morgan Fainberg2013-08-292-12/+0
| | | | | | | | | | | | Fixes duplicate method definitions and enables gating on F811 Change-Id: Ic360e70d13da864b8b5c10fc5e731b3952b2d2e8
* | Merge "Standardize base.py with novaclient"Jenkins2013-08-301-42/+141
|\ \
| * | Standardize base.py with novaclientAlessio Ababilov2013-08-271-42/+141
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Main changes: * deprecate api variable in favour of client * add documentation * use to_slug from oslo strutils * remove Python 2.4 support * other small fixes from novaclient Change-Id: Ife54fd3207798ee03101a48bc1cda3b3f62cc5e4
* | | Merge "Fixing potential NameErrors"Jenkins2013-08-302-1/+2
|\ \ \
| * | | Fixing potential NameErrorsMorgan Fainberg2013-08-302-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixing a couple locations that could potentially raise NameErrors instead of intended exceptions. These were found by looking at flake8 F821 (undefined name) errors. Added in unit tests to verify NameErrors would not be raised. Change-Id: I8619cb0be495b814335a5aea23daca025484d3c6
* | | | Merge "Add apiclient.exceptions hierarchy"Jenkins2013-08-305-162/+464
|\ \ \ \
| * | | | Add apiclient.exceptions hierarchyAlessio Ababilov2013-08-205-162/+464
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new apiclient.exceptions hierarchy is more comprehensive and it covers almost all HTTP error status codes. These exceptions can be used in novaclient, keystoneclient, glanceclient, and other client projects thus providing a single inteface. Users can have benefit from OpenStack clients raising exceptions of known classes while now every client has its own classes making difficult to catch, e.g., NotFound for every client. Change-Id: Ia7b25880e0ffca3526525a0f0e77c7e77c4f0076
* | | | | Merge "Sync py3kcompat from oslo-incubator"Jenkins2013-08-302-0/+66
|\ \ \ \ \
| * | | | | Sync py3kcompat from oslo-incubatorChuck Short2013-08-232-0/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Python3 reorganized the standard library and moved several functions to different modules and combined modules. Six provides a consistent interface to the module through six.moves However urllib/urlparse is not covered by six.moves so py3kcompat adds python2/python3 compatibility layer for urllib/urlparse. Change-Id: I98d45a84e8c2dcd965a9e5cc08d7bb341eb08bfc Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | | | | Merge "Use OSLO jsonutils instead of json module"Jenkins2013-08-303-14/+9
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Use OSLO jsonutils instead of json moduleJamie Lennox2013-08-263-14/+9
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For most things there is very little difference, but as we have a jsonutils module we should probably be using it. Change-Id: I406ea81bb56ad90cc9ff9b8b58b0d35b694dc802
* | | | | Merge "Fix and enable gating on F841"Jenkins2013-08-301-2/+2
|\ \ \ \ \ | | |_|_|/ | |/| | |
| * | | | Fix and enable gating on F841Morgan Fainberg2013-08-291-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | F841 local variable is assigned to but never used Change-Id: I4b54489386deb655821192b4ec1e9c0ea596a9b7
* | | | | Merge "Fix License Headers and Enable Gating on H102"Jenkins2013-08-296-1/+84
|\ \ \ \ \ | |/ / / /
| * | | | Fix License Headers and Enable Gating on H102Morgan Fainberg2013-08-276-1/+84
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Add ASLv2 headers to files that were missing it. fixes bug #1211587 Change-Id: Iede918e1ce84993cee4ecbb2d9c2606627fa412e
* | | | Merge "Remove duplicate method in AccessInfo"Jenkins2013-08-291-3/+0
|\ \ \ \
View Lorry Controller Status