summaryrefslogtreecommitdiff
path: root/keystoneclient
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Handle exception on UnicodeDecodError in logging of request"Jenkins2016-02-122-2/+50
|\
| * Handle exception on UnicodeDecodError in logging of requestTrevor McKay2016-02-102-2/+50
| | | | | | | | | | | | | | | | | | | | | | If the logging of an HTTP request causes a UnicodeDecodeError, modify the log entry using oslo_utils.encodeutils.safe_decode with errors='replace' and try again Co-Authored-By: Nikita Konovalov <nkonovalov@mirantis.com> Change-Id: Ic365c654ebca4045208c6c30e232665145db7b4c Closes-Bug: #1453953
* | Merge "Deprecate adapter"Jenkins2016-02-122-1/+8
|\ \
| * | Deprecate adapterJamie Lennox2016-02-032-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | This involves switching keystoneclient to use the adapter on keystoneauth. Change-Id: I02780b0c00e3865f083b4bca98bff81127ed2277 Implements: bp deprecate-to-ksa
* | | Merge "Deprecate auth plugins from keystoneclient"Jenkins2016-02-1220-39/+117
|\ \ \ | |/ /
| * | Deprecate auth plugins from keystoneclientJamie Lennox2016-02-0320-39/+117
| | | | | | | | | | | | | | | | | | | | | Deprecate auth plugins in favour of those from keystoneauth. Change-Id: I8963ded9b68569717d7a6e30623ee78301b59a4a Implements: bp deprecate-to-ksa
* | | Merge "Deprecate Session"Jenkins2016-02-1216-21/+85
|\ \ \ | |/ / | | / | |/ |/|
| * Deprecate SessionJamie Lennox2016-02-0316-21/+85
| | | | | | | | | | | | | | | | Deprecate the keystoneclient Session object in favour of keystoneauth's Session. Change-Id: I26e000d626a466f63d10d2a961adc698f8de0636 Implements: bp deprecate-to-ksa
* | Merge "Remove python 2.5 workaround"Jenkins2016-02-041-6/+0
|\ \
| * | Remove python 2.5 workaroundBrant Knudson2016-01-311-6/+0
| |/ | | | | | | | | | | | | keystoneclient doesn't support python 2.5 so remove this workaround code. Messing with global variables is a mistake anyways. Change-Id: I1567508cb1931b8c5365cfe6584e5a732c459db0
* | Update keyring requirementsBrant Knudson2016-01-311-2/+3
|/ | | | | | | As of commit 05c2bcef the minimum keyring version is 5.5.1, so update the version that httpclient will use. Change-Id: I877b9d40b8bc4f89754d64d29d82a9e3a3f2dd04
* Missing defaults in the create() method in the v2 ServiceManagerJude Job2016-01-272-2/+33
| | | | | | | | This patch include a new method for testing, without description value. Change-Id: Icd04e4479a341e7691fc562b3e09d5aa41a354e7 Closes-Bug: #1318438
* Merge "Replace TestResponse with requests_mock"Jenkins2016-01-258-45/+16
|\
| * Replace TestResponse with requests_mockJamie Lennox2016-01-258-45/+16
| | | | | | | | | | | | | | | | The TestResponse object doesn't do the right thing with regards to content vs text. Just reuse the one from requests_mock rather that try and fix it. Change-Id: Ia8bcae126babb0e616329928c57f875a50a957d6
* | use positional library instead of utilsSteve Martinelli2016-01-2531-97/+117
| | | | | | | | | | | | swap instances of utils.positional with the positional library. Change-Id: Id8a9961e68d287a802f25512fc970829e9feb5c2
* | Use positional library instead of local codeMorgan Fainberg2016-01-232-230/+1
|/ | | | | | | | Use the positional library instead of the local copy of the code. This will prevent duplicating code across repositories without needing to chase down weird dependency graphs. Change-Id: I7fef334666e93e358870173d29018e5d47312d2b
* Merge "Adds an option to include names in role assignment lists"Jenkins2016-01-232-1/+17
|\
| * Adds an option to include names in role assignment listsTom Cocozzello2016-01-232-1/+17
| | | | | | | | | | | | | | | | | | | | Allow the client to take advantage of the include_names with list role assignments. Change-Id: I4aa77c08660a0cbd021502155938a46121ca76ef Depends-On: I0a1cc986b8a35aeafe567e5e7fee6eeb848ae113 Closes-Bug: #1479569 Implements: blueprint list-assignment-with-names
* | Merge "Address hacking check H405"Jenkins2016-01-2321-129/+175
|\ \ | |/ |/|
| * Address hacking check H405lin-hua-cheng2016-01-1321-129/+175
| | | | | | | | | | | | | | | | | | Previously, there were a string of commits to keystone that addresed ignored hacking checks. This commit does the same for H405 in keystoneclient. This also modifies our tox.ini so that we no longer ignore H405 violations. Change-Id: I2af152e5425a0e9c82314039fdbb90d661c22680 Closes-Bug: 1482773
* | Mark password/secret options as secretBrant Knudson2016-01-145-5/+7
| | | | | | | | | | | | | | | | | | Password, token, and secret options should be marked as secret=True so that when the value is logged the logger knows to obfuscate the value. Change-Id: I6ebdfa3bf6faf37bc11640a5826b3b55bb920fc4 Closes-Bug: 1534299
* | Merge "Deprecate the baseclient.Client"Jenkins2016-01-142-1/+8
|\ \
| * | Deprecate the baseclient.ClientJamie Lennox2015-12-172-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't want to support other clients subclassing this. Deprecate it for removal. As far as I know noone else is using this class. Change-Id: Iad0ea860bd07c657d7094018c037b003ea92dc28 Implements: bp deprecate-to-ksa
* | | Merge "Revert "Support `truncated` flag returned by keystone""2.1.2Jenkins2016-01-142-55/+8
|\ \ \
| * | | Revert "Support `truncated` flag returned by keystone"2.1.1Brant Knudson2016-01-132-55/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit c28d40814962b3a8ccb81e5e7d7f832c8f0a3c9a. This is causing stable keystone to fail. keystone has tests that verify that the returned value of the list operation == [], which fails since the return value is now an object and not a list. Change-Id: Ieb143574271b991d3e19e864497073fbedf46bcb
* | | | Revert "Change default endpoint for Keystone v3 to public"Brant Knudson2016-01-134-19/+3
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit d3b11d674d6539a0a09e0c432983ebf172e8ad79. This is causing auth_token middleware tests to fail. The error is like: EndpointNotFound: public endpoint for identity service in east region not found So this is going to potentially affect customers. Change-Id: I5ad917e48c9b140709dd3bf95e89c07ea58d6a66
* | | Merge "Support `truncated` flag returned by keystone"Jenkins2016-01-072-8/+55
|\ \ \ | |/ /
| * | Support `truncated` flag returned by keystoneBoris Bobrov2015-12-282-8/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wrap a list of objects into custom class with additional attributes. This is wanted by Horizon, that wants to know that the list returned from keystone is not full and that more strict filters need to be applied. Change-Id: Icfabfd055aed1648dc4130b03ec3dbf9bad4e45a Closes-Bug: 1520244
* | | Merge "Replace textwrap with fast standard code"Jenkins2016-01-061-2/+1
|\ \ \
| * | | Replace textwrap with fast standard codeBernhard M. Wiedemann2015-12-161-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This improves on commit 4350c176048b8d159d08b82b915e9544ac9dee6f We found a major performance regression in keystoneclient when using PKI tokens, related to http://bugs.python.org/issue25870 It can be tested with time python -c "import textwrap; textwrap.wrap('x'*9000, 64)" which has a complexity of O(n*n) because it uses certain regexps in python versions before 3.5. Closes-Bug: #1526686 Related-Bug: #1404402 Change-Id: Ibc81907c4d9db2c09fff41ccf21345fbdb19202d
* | | | Merge "Remove "deprecated" internal method"Jenkins2016-01-0610-21/+16
|\ \ \ \
| * | | | Remove "deprecated" internal methodBrant Knudson2015-12-2910-21/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method says it's deprecated but it's an internal method, so there's no need to deprecate it, just remove it. Change-Id: I916b4a5c9dc2320a4bff7f966fd0c7d5a4957ff9
* | | | | Merge "Fix Resource.__eq__ mismatch semantics of object equal"Jenkins2016-01-062-3/+6
|\ \ \ \ \
| * | | | | Fix Resource.__eq__ mismatch semantics of object equalRui Chen2015-12-162-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The __eq__ of apiclient.base.Resource will return True, if the two objects have same id, even if they have different other attributes value. The behavior is weird and don't match the semantics of object equal. The objects that have different value should be different objects. Fix this issue and add some test cases in this patch. Change-Id: I1565c6e66d6dd6cf7bd2b7e8526157190deb45de Closes-Bug: #1499369
* | | | | | Merge "Change default endpoint for Keystone v3 to public"Jenkins2016-01-064-3/+19
|\ \ \ \ \ \
| * | | | | | Change default endpoint for Keystone v3 to publicRoxana Gherle2015-12-234-3/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All of the other Openstack services have a 'public' default endpoint type. Keystone has 'admin' default endpoint type. Why not make Keystone compliant and change the default for Keystone v3 from 'admin' to 'public'. Keystone v2 will remain the same with an 'admin' default. Closes-Bug: #1457702 Change-Id: I515438477dba72c2a0c4595603000690511b5700
* | | | | | | Merge "Docstring: Mark optional parameter as optional."Jenkins2016-01-061-1/+1
|\ \ \ \ \ \ \
| * | | | | | | Docstring: Mark optional parameter as optional.Rushi Agrawal2015-12-161-1/+1
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In client.Client, providing version number is optional. So write '(optional)' in the docstring describing this parameter. Change-Id: I5e355515de8d043bda85b5abf43c549b4ba22d61
* | | | | | | Merge "Wrong usage of "a/an""Jenkins2016-01-062-2/+2
|\ \ \ \ \ \ \
| * | | | | | | Wrong usage of "a/an"hgangwx2015-12-302-2/+2
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wrong usage of "a/an" in the messages: "string that is the id field for an pre-existing" "build a etree.XML object filling certain" Should be: "string that is the id field for a pre-existing" "build an etree.XML object filling certain" Totally 2 occurrences in python-keystoneclient base code. Change-Id: Icef5247672f95af87375a4a135a961aefb0a4906
* | | | | | | Merge "Implements base classes for functional tests"Jenkins2016-01-064-0/+87
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | |
| * | | | | | Implements base classes for functional testsSamuel de Medeiros Queiroz2016-01-044-0/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the cross project workshop "Establishing key themes for the Mitaka cycle" [1] during the Mitaka summit, one of the agreed point was to implement functional tests for the client libraries. [2] Later on, this will serve to improve our backward compatibility for client libraries. See I72e4e9cfa0539f6b326a0296c065fa3cb754f8ae In preparation to implement the functional tests for both v2 and v3 clients, this patch initializes the new directories and adds the base test classes: V3ClientTestCase and V2ClientTestCase. Each class instantiates its specific keystoneclient version via os-client-config either based on a clouds.yaml config file or from the environment variables. [1] https://mitakadesignsummit.sched.org/event/edd78ade4c7c92581a7cabc26019a85a [2] https://etherpad.openstack.org/p/mitaka-crossproject-themes Change-Id: Ia73d72d5f87051fb46d733782275b548874a1def
* | | | | | | remove keystoneclient.apiclient.exceptionsSteve Martinelli2015-12-292-75/+0
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | these have been deprecated in favor of keystoneclient.exceptions for a very long time. let's finally remove them. Change-Id: I0fc06a12647a0faac5ba98ed83118269efc304a6 Closes-Bug: 1526651
* | | | | | Make tests run against original client and sessionsJamie Lennox2015-12-187-28/+244
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The managers have a bad habit of reaching back into the client and making assumptions about the values that are saved there. These assumptions are not always correct when we use the session object. Test all the versioned managers against a client that was constructed with the old method and with a keystoneclient session object and a keystoneauth1 session object. Change-Id: I93a26db7ae7e4d887aa815108be71c72b4a1f2bb
* | | | | | Seperate Client base test classJamie Lennox2015-12-1832-67/+66
| |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Client tests are broadly seperated into v2 and v3. These folders don't distinguish between the tests that are on v2/v3 auth and those that are CRUD related. This means that the base v2 and v3 test class always create a client object for the tests to use, even if they don't need it. Whilst this isn't a big deal now, we want to be able to seperate only those tests that require a client so we can test them with multiple different styles of client. Add a new ClientTestCase class that will construct a client object for the tests that need it. Change-Id: I61f463ac5e878107c1889672f4d4cf5874821363
* | | | | Merge "move hacking to tests folder"Jenkins2015-12-163-1/+1
|\ \ \ \ \
| * | | | | move hacking to tests folderSteve Martinelli2015-12-153-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | hacking should go under tests, no reason to pollute the top level directory with non-keystoneclient related bits. Change-Id: Ib4e9a3acfbab5a115c76bbc8bbf7241f42c6eac8
* | | | | | Merge "remove oslo-incubator's memorycache"Jenkins2015-12-161-103/+0
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | remove oslo-incubator's memorycacheSteve Martinelli2015-12-141-103/+0
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | with middleware removed from keystoneclient we no longer need to keep memorycache around. Change-Id: I98d8ddd1398699f8821e4c36fe911c9d211c46a9 Related-Bug: #1449066
* | | | | Add include_subtree to role_list_assignments calldaniel-a-nguyen2015-12-152-1/+21
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is needed for Domain Admin to list role assignments. Related-Bug: 1437407 Depends-On: I3495c7cab3b40811b2722ac7d70ddda30410b62b Closes-Bug: #1462694 Change-Id: I63849d5f39d090fec3ef6b9182f339e198e0c551
View Lorry Controller Status