summaryrefslogtreecommitdiff
path: root/keystoneclient
Commit message (Collapse)AuthorAgeFilesLines
* fix: remove error python2 defense code5.0.1Wenxiang Wu2022-07-061-13/+3
| | | | | | bug #1980177 Change-Id: I5e0a79b44e5cf80c82137f000a9dbd078fc931f2
* Merge "use importlib.metadata to get keyring version"Zuul2022-04-291-5/+13
|\
| * use importlib.metadata to get keyring versionDoug Hellmann2022-04-221-5/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Importing pkg_resources has a side-effect of scanning the metadata of all of the installed python modules to build an in-memory cache. That cache isn't used anywhere in keystoneclient, and it can be expensive to build. The importlib.metadata module in the 3.8 standard library (and the importlib_metadata library for earlier versions) provides the same version lookup service using a more efficient scanning implementation. Switching from pkg_resources to importlib.metadata will help application startup time, which is especially important for command line programs such as python-openstackclient. Change-Id: Ia89044ff1876eeb2793cd08ed9095ce2ffe89e09 Depends-On: Ic6db7af34c87a636bfe55bacae03c42154f4b9c7 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* | Merge "Stop to use the __future__ module."Zuul2022-04-295-10/+0
|\ \
| * | Stop to use the __future__ module.Hervé Beraud2022-04-225-10/+0
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The __future__ module [1] was used in this context to ensure compatibility between python 2 and python 3. We previously dropped the support of python 2.7 [2] and now we only support python 3 so we don't need to continue to use this module and the imports listed below. Imports commonly used and their related PEPs: - `division` is related to PEP 238 [3] - `print_function` is related to PEP 3105 [4] - `unicode_literals` is related to PEP 3112 [5] - `with_statement` is related to PEP 343 [6] - `absolute_import` is related to PEP 328 [7] [1] https://docs.python.org/3/library/__future__.html [2] https://governance.openstack.org/tc/goals/selected/ussuri/drop-py27.html [3] https://www.python.org/dev/peps/pep-0238 [4] https://www.python.org/dev/peps/pep-3105 [5] https://www.python.org/dev/peps/pep-3112 [6] https://www.python.org/dev/peps/pep-0343 [7] https://www.python.org/dev/peps/pep-0328 Change-Id: I0173b210e343ccf6b00c3b66959c001fdb1d699b
* | Merge "Stop using an admin endpoint by default"Zuul2022-04-233-7/+17
|\ \
| * | Stop using an admin endpoint by defaultDr. Jens Harbott2021-11-053-7/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | With V3 of the identity API, we no longer need to have a dedicated admin endpoint, so stop requesting one by default, allowing deployments to actually work without one. Signed-off-by: Dr. Jens Harbott <harbott@osism.tech> Change-Id: I96cc9c14008bcc59992d06c89f8f50895390f11e
* | | Merge "remove unicode from code"Zuul2022-04-223-9/+9
|\ \ \ | |_|/ |/| |
| * | remove unicode from codezhangtongjian2022-04-193-9/+9
| |/ | | | | | | Change-Id: I2d2d025b0d8bda2ffc7be4d30489728c05f53c8e
* | Add access to /v3/auth/systemsRadomir Dopieralski2021-11-223-0/+63
|/ | | | | Closes-bug: 1945649 Change-Id: I7df5d9bf3cfb0e58e0e129a56170c8fe33523a4c
* Replace assertItemsEqual with assertCountEqualgugug2020-07-122-8/+8
| | | | | | | | | | | | | | | assertItemsEqual was removed from Python's unittest.TestCase in Python 3.3 [1][2]. We have been able to use them since then, because testtools required unittest2, which still included it. With testtools removing Python 2.7 support [3][4], we will lose support for assertItemsEqual, so we should switch to use assertCountEqual. [1] - https://bugs.python.org/issue17866 [2] - https://hg.python.org/cpython/rev/d9921cb6e3cd [3] - testing-cabal/testtools#286 [4] - testing-cabal/testtools#277 Change-Id: Ib5985049235ee1b6018fc172a67e3b05970a6c42
* Merge "Use unittest.mock instead of third party mock"Zuul2020-06-1914-14/+15
|\
| * Use unittest.mock instead of third party mockSean McGinnis2020-04-1814-14/+15
| | | | | | | | | | | | | | | | Now that we no longer support py27, we can use the standard library unittest.mock module instead of the third party mock lib. Change-Id: I7498ea2353cccca7b23d9ef74015a566ac431f90 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | Fix the typo on attribute word.Daniel Bengtsson2020-04-211-2/+2
|/ | | | | | Fix the typo for the attribute word in comment and docstring. Change-Id: Ic4a841c333d712a22503c1d7dc2d21619c601408
* Hacking: Fix F601Andreas Jaeger2020-03-311-5/+1
| | | | | | | Fix F601 failures: F601 dictionary key '...' repeated with different values Change-Id: I4c12f5a2348371966215e1ae05b726e6d04b0c8f
* Update hacking for Python3Andreas Jaeger2020-03-315-1/+8
| | | | | | | | | The repo is Python 3 now, so update hacking to version 3.0 which supports Python 3. Fix problems found. Change-Id: Ic161a8f88c28d88898863e5b9d9380016fbb0d08
* Merge "Add support for app cred access rules"3.22.0Zuul2019-10-105-1/+189
|\
| * Add support for app cred access rulesColleen Murphy2019-08-215-1/+189
| | | | | | | | | | | | | | | | | | | | | | | | This change adds access_rules as a parameter for creating application credentials, and also adds the ability to list access rules and to retrieve and delete individual rules. Directly creating an access rule or updating one is not supported. bp whitelist-extension-for-app-creds Depends-On: https://review.opendev.org/671374 Change-Id: I490f1e6b421d4f36f588f83a511ce39b9b4204e2
* | Merge "Add parent project filter for listing projects"Zuul2019-09-103-3/+42
|\ \
| * | Add parent project filter for listing projectsTakashi Kajinami2019-08-293-3/+42
| |/ | | | | | | | | | | | | | | | | | | This patch introduces the interface into listing project, to specify parent_id to filter projects which has the given project as their parent[1]. [1] https://docs.openstack.org/api-ref/identity/v3/?expanded=list-projects-detail#list-projects Change-Id: If78030425468d4f99cba708540142871a2bf9190
* | Fix unit tests broken by requests-mockTakashi Kajinami2019-09-062-3/+1
|/ | | | | | | | | | | | | Now requests-mock records request url in log[1], so it is invalid to check that the logger output does NOT contain request url. Also, fix url passed to request mock as now it requires complete url is passed [1] https://github.com/jamielennox/requests-mock/pull/93 Change-Id: I4bab30a6705b7cab6b5a569dd61c442263e39995
* Merge "Add support for app cred access rules header"Zuul2019-07-101-3/+18
|\
| * Add support for app cred access rules headerColleen Murphy2019-02-251-3/+18
| | | | | | | | | | | | | | | | | | | | | | This header is set to indicate to the keystone server that the client, usually keystonemiddleware, will validate application credential access rules. If not provided and the token uses access rules, the server will return a 401. bp whitelist-extension-for-app-creds Change-Id: I64ac952d663e916150fbf7e5a8f70b76dddf3319
* | Follow bandit B105: hardcoded_password_stringVishakha Agarwal2019-06-261-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | tox -e bandit failing due to the string 'token' in [1]. According to the bandit 105 any password assigned to a string should not contain any of the variables in [2] [1]https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/common/cms.py#L41 [2]https://bandit.readthedocs.io/en/latest/plugins/b105_hardcoded_password_string.html Change-Id: I822e1195532df2b701f10087cabceda458211986
* | Merge "Update json module to jsonutils"Zuul2019-05-023-12/+13
|\ \
| * | Update json module to jsonutilscao.yuan2019-02-253-12/+13
| |/ | | | | | | | | | | | | | | | | | | oslo project provide jsonutils, and keystoneclient use it in many place[1], this PS to update the remained json module to oslo jsonutils for consistency. [1]: https://github.com/openstack/python-keystoneclient/search?utf8=%E2%9C%93&q=jsonutils&type= Change-Id: Id5275b5e6b5bf8f6d54406dac7ab95a30828cf58
* | Make tests pass in 2020Bernhard M. Wiedemann2019-02-286-21/+21
|/ | | | | | | | | | | Without this patch, build failed after 2019-12-31 with Traceback (most recent call last): File "keystoneclient/tests/unit/v3/test_auth.py", line 226, in test_authenticate_success_password_unscoped self.assertRequestBodyIs(json=self.TEST_REQUEST_BODY) File "keystoneclient/tests/unit/utils.py", line 72, in assertRequestBodyIs self.assertEqual(json, val) Change-Id: I0e44d9896c5970f0ca07438c372aec826aeb5c77
* Merge "Add return-request-id-to-caller function(v3/contrib)"Zuul2019-01-167-19/+31
|\
| * Add return-request-id-to-caller function(v3/contrib)openstack2019-01-167-19/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added return-request-id-to-caller function to resources and resource managers in the following files. * keystoneclient/v3/projects.py * keystoneclient/v3/registered_limits.py * keystoneclient/v3/roles.py * keystoneclient/v3/limits.py * keystoneclient/v3/contrib/federation/saml.py Also made changes in base.py for _put() method so that if include_metadata is True, the response data should include request_id instead of returning None as response. Change-Id: Ifc0ec9a9d666cccfee3b08ac61596a3692307f23 Implements: blueprint return-request-id-to-caller
* | Update hacking versionZhijunWei2019-01-041-3/+3
|/ | | | | | | 1. update hacking version to latest 2. fix pep8 failed Change-Id: Iecc112206633a7e771c5e45547c573d74cce6f67
* Merge "Add return-request-id-to-caller function(v3/contrib)"Zuul2018-11-299-29/+364
|\
| * Add return-request-id-to-caller function(v3/contrib)openstack2018-10-229-29/+364
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added return-request-id-to-caller function tovresources and resource managers in the following files. * keystoneclient/v3/contrib/simple_cert.py * keystoneclient/v3/contrib/endpoint_policy.py * keystoneclient/v3/contrib/oauth1/access_tokens.py * keystoneclient/v3/contrib/oauth1/request_tokens.py Adding request-id to below V3 contrib API's is covered in base patch [1] but this patch is specifically for V3 API's so covered their test cases in this patch. * keystoneclient/v3/contrib/endpoint_filter.py * keystoneclient/v3/contrib/federation/identity_providers.py * keystoneclient/v3/contrib/federation/mappings.py * keystoneclient/v3/contrib/federation/protocols.py * keystoneclient/v3/contrib/federation/service_providers.py The methods in the resource class and resource manager return a 'base.Response' class that has 'request_ids' property. The caller can get request ids of the callee via that property. [1] https://review.openstack.org/#/c/329913 Change-Id: I5f90c31020e0dd672a160c7b587f41ba8f2b596c Co-authored-by: Dinesh Bhor <dinesh.bhor@nttdata.com> Co-authored-by: Ankit Agrawal <ankit11.agrawal@nttdata.com> Co-authored-by: Neha Alhat <neha.alhat@nttdata.com> Implements: blueprint return-request-id-to-caller
* | Convert functional tests to Zuulv3Colleen Murphy2018-10-281-50/+0
|/ | | | | | | Use the devstack-tox-functional-consumer job as a parent job to allow us to remove the custom playbooks and devstack hooks. Change-Id: Ide6e6da3cbb479b5149fb44f0ef392dc80202910
* Merge "Update IdentityProviderManager docstring"Zuul2018-10-201-2/+2
|\
| * Update IdentityProviderManager docstringchenxing2018-06-201-2/+2
| | | | | | | | | | Change-Id: Ie386caf4fc9ad660581558406cd72fafc391379d Closes-Bug: #1763475
* | Merge "Add return-request-id-to-caller function(v3)"Zuul2018-10-192-2/+87
|\ \
| * | Add return-request-id-to-caller function(v3)Maho Koshiya2018-03-262-2/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added return-request-id-to-caller function to resources and resource managers in the following files. * keystoneclient/v3/projects.py The methods in the resource class and resource manager return a wrapper class that has 'request_ids' property. The caller can get request ids of the callee via the property. NOTE: Remaining resources from the V3 package are already covered in the base patch: https://review.openstack.org/#/c/329913/ Change-Id: I0133d51cfadc02e2dd926b8b0419b2e1dd0fa92a Co-authored-by: Ankit Agrawal <ankit11.agrawal@nttdata.com> Co-authored-by: Dinesh Bhor <dinesh.bhor@nttdata.com> Implements: blueprint return-request-id-to-caller
* | | Merge "Add Response class to return request-id to caller"Zuul2018-10-193-30/+318
|\ \ \ | |/ /
| * | Add Response class to return request-id to callerDavid Stanek2018-03-263-30/+318
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change is required to return 'request_id' from client to log request_id mappings of cross-project requests. Instantiating class 'keystoneclient.v3.client.Client' using 'include_metadata=True' will cause manager response to return a new 'Response' class instead of just the data. This 'Response' class is going to have additional metadata properties available like 'request_ids' and the original data will be available as property 'data' to it. This change is backward compatible since user has to set a new parameter 'include_metadata=True' to client in order to get the request_id returned. Co-author: Dinesh Bhor <dinesh.bhor@nttdata.com> Partially Implements: blueprint return-request-id-to-caller Change-Id: Ibefaa484158ff08bfcacc1e2802d87fc26fd76a5
* | Deprecate region enabled parameterVishakha Agarwal2018-09-282-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | We don't check for "enabled" in the region anywhere thus deprecating it from the create() and update calls of the v3/region.py. We dont use it in schema [1] as well as [2]. [1] https://github.com/openstack/keystone/blob/master/keystone/catalog/schema.py#L34 [2] https://github.com/openstack/keystone/blob/master/keystone/catalog/backends/sql.py#L33-L49 Change-Id: I0257d5d42916e3b4d008e592d54eeeebec591633 Partial-Bug: #1615076
* | Merge "fix misspelling of 'default'"Zuul2018-08-161-1/+1
|\ \
| * | fix misspelling of 'default'lvxianguo2018-08-161-1/+1
| | | | | | | | | | | | Change-Id: I12ffcf28d05655f1f60038dcf1c42e43a516978e
* | | refactor the getid method in keystoneclient/base.pyzhubx0072018-08-081-10/+4
|/ / | | | | | | | | | | | | | | | | Refer to a merged commit. https://review.openstack.org/#/c/588983/ TrivialFix Change-Id: Ie3a02843e35382dd24230e91534b6ed72846957d
* | Merge "Add support for project-specific limits"rocky-em3.17.0Zuul2018-06-203-0/+231
|\ \
| * | Add support for project-specific limitsLance Bragstad2018-06-143-0/+231
| | | | | | | | | | | | | | | | | | | | | | | | Thsi commit adds client support for managing limits in keystone. bp unified-limits Change-Id: I33251dbd4d3bfaf178ca86a2f5d564ac94879dd2
* | | Merge "Add support for registered limits"Zuul2018-06-203-0/+241
|\ \ \ | |/ /
| * | Add support for registered limitsLance Bragstad2018-06-143-0/+241
| |/ | | | | | | | | | | | | | | | | | | | | This change add client support for creating, reading, updating, and deleting registered limits. A subsequent patch will do the same for project-specific limits. bp unified-limits Depends-On: https://review.openstack.org/#/c/569741/ Change-Id: I6b5d106d08af53c2ad41ed3f799e9e71d370c6dd
* | Merge "fix a typo in docstring"Zuul2018-06-151-1/+1
|\ \
| * | fix a typo in docstringmelissaml2018-06-071-1/+1
| |/ | | | | | | Change-Id: I56e9c3b03ed85c1c6031390b835d678c43e51e17
* | Fix python3 test compatMorgan Fainberg2018-06-101-1/+1
|/ | | | | | | | Python3 test requirement means that the unicode type does not exist instead we use six.unicode to get `unicode` in py27 and `str` in python3. Change-Id: I2cbf2e63a9d93c232b6928acf002a45b7fbcec8e
View Lorry Controller Status