summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Only log application/json content typemitaka-eol2.3.2stable/mitakaTobias Diaz2017-01-114-27/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a combination of 2 commits. The first commit's message is: Prevent MemoryError when logging response bodies Response bodies are loaded into memory prior to being logged. Loading huge response bodies may result in a MemoryError. This patch proposes that only JSON and TEXT responses be logged, i.e when the Content-Type header is application/json or application/text. Responses that do not include or have a different Content-Type header will have their body omitted. This is a sort of backport of the fix for keystoneauth sessions, see I93b6fff73368c4f58bdebf8566c4948b50980cee Co-Authored-By: Samuel de Medeiros Queiroz <samueldmq@gmail.com> Closes-bug: 1616105 Change-Id: I8f43eee3a0b35041c6cf672e476f8151cf2f8d14 (cherry-picked from: 3e56e0d7e5e1a76d806a3bc1f6d5ef9070f95771) Only log application/json in session to start When whitelisting content types to debug print from session we chose application/json and application/text. application/text is not a real mime type, text is typically text/plain. Rather than guess at mime types only print application/json to start with, but make it easy for additional types to be added later. Adapted from keystoneauth: Ica5fee076cdab8b1d5167161d28af7313fad9477 Related-Bug: 1616105 Change-Id: Ieaa8fb3ea8d25e09b89498f23b70b18c0f6153f1 (cherry-picked from: 51d16fa344829aadf454faf5e0c4535a8f96a7c8)
* Merge "Do not log binary data during request" into stable/mitakaJenkins2017-01-102-7/+9
|\
| * Do not log binary data during requestSteve Martinelli2017-01-092-7/+9
| | | | | | | | | | | | | | | | | | | | | | Do not log binary data during debug logging of a session. Replace the binary data with the string <binary_data> instead. sort of a backport of: I5184002f3a21c5e0ee510b21b9a7884c8dccd1e3 Change-Id: I07ddbc3967f297597542f1975004d94c490f6e6b Related-Bug: 1616105 (cherry picked from commit af770f17b705a66bd4292b2a54df46ec5fdaa12b)
* | X-Serivce-Token should be hashed in the logTin Lam2017-01-093-2/+8
|/ | | | | | | | | | | Currently, logs display the hash values of X-Auth-Token, Authorization, and X-Subject-Token, but not the value of the X-Service-Token. This patch set adds the X-Service-Token to the list of header fields to be hashed for logging purposes. Change-Id: Iaa3a27f4b6c3baf964fa0c71328ffe9df43b2c0a Closes-Bug: #1654847 (cherry picked from commit 56af8c90ecbb3cb5d29036151108b1e4e7a69bcc)
* List system dependencies for running common testsAndreas Jaeger2016-06-302-0/+35
| | | | | | | | | | | | | | | | | | | | Add an other-requirements.txt file containing a cross-platform list of dependencies needed for running included tox-based tests. Also include a tox environment for convenience calling the bindep[*] utility to list any missing system requirements. This change is self-testing. For other-requirements.txt see also http://docs.openstack.org/infra/manual/drivers.html#package-requirements [*] http://docs.openstack.org/infra/bindep/ Also, merge Xenial followup change into this. Change-Id: Ifa3d417d9c3f8c172980a593c0a5a4a29987366a (cherry picked from commit f74d9e3d82ad46efb27069487670165e2c30dc45) (cherry picked from commit 5dc965e268a5a1fa0c39dc6bcef4ef9bf717c8c0)
* Updated from global requirementsOpenStack Proposal Bot2016-04-181-1/+1
| | | | Change-Id: I846d9e75bbdff054de00801f7af7ad52c656a391
* Update .gitreview for stable/mitakaDoug Hellmann2016-03-091-0/+1
| | | | Change-Id: Ie9a3721dfd4cab5db8dda4ec2721641f950893e5
* Merge "Revert "Support `truncated` flag returned by identity service""2.3.1Jenkins2016-02-273-95/+8
|\
| * Revert "Support `truncated` flag returned by identity service"Monty Taylor2016-02-263-95/+8
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 870be44c0ed5564101f9cf9d53e8085fb01680c8. This change breaks end users of the library. See: http://logs.openstack.org/50/285450/1/check/gate-shade-dsvm-functional-keystone2/d1093b5/console.html#_2016-02-26_20_49_32_928 For an example of a consumer of the library being broken. Change-Id: I1912003afb89579eb869767db7a411c451bc9806
* | Merge "Updated from global requirements"2.3.0Jenkins2016-02-261-1/+1
|\ \
| * | Updated from global requirementsOpenStack Proposal Bot2016-02-261-1/+1
| | | | | | | | | | | | Change-Id: Id1cf3690fab9802ca60db8a5fa9fa0e022e6927c
* | | Merge "Support `truncated` flag returned by identity service"Jenkins2016-02-263-8/+95
|\ \ \ | |/ / |/| / | |/
| * Support `truncated` flag returned by identity serviceBoris Bobrov2016-02-263-8/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create a custom list with flag `truncated` to support corresponding response from the identity service. 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. The previous attempt in commit c28d40814962b3a8ccb81e5e7d7f832c8f0a3c9a was reverted by d20b300589863bcf165945beb129ebcc3621a14f because it broke other code. This commit changes the way the flag is added and verifies that existing code will not break. Change-Id: Ia86cfd91110adae6d7ab86ff1f152a8f9be27837 Closes-Bug: 1520244
* | Merge "Support creation of domain specific roles"Jenkins2016-02-232-1/+33
|\ \
| * | Support creation of domain specific rolesHenry Nash2016-02-192-1/+33
| | | | | | | | | | | | | | | | | | | | | Add support for the domain_id attribute in the role entity. Partially Implements: blueprint domain-specific-roles Change-Id: I06af7647e15aa742609b3fe1b9b222fbeaeb5735
* | | Updated from global requirementsOpenStack Proposal Bot2016-02-201-1/+1
|/ / | | | | | | Change-Id: I9d945a1a80616f8a86c7ccac5b84d477f61b0c55
* | Merge "Implied Roles"Jenkins2016-02-194-0/+186
|\ \
| * | Implied RolesAdam Young2016-02-174-0/+186
| |/ | | | | | | | | | | | | | | | | | | | | While the entity for an inference rule should be thought of as a resource, the rules are essentially relationships between roles. The `implied_role` API is linked with the role API, and thus the client functions are part of v3/role.py. However, it does not map completely cleanly to the Crud baseclass, and requires some custom URL generation. Change-Id: I80a40e88b571fe9b0eca3af8b705ea79f28eb904
* | Add back a bandit tox jobEric Brown2016-02-181-0/+5
|/ | | | | | | | The bandit project uses an integration test to ensure it doesn't break other projects by introducing new changes. To run this integration, it is necessary to have a common tox target of 'bandit' Change-Id: If44008f6a48134a8af38e6794fb87ae09aac57b4
* Merge "add release notes for deprecated auth bits"2.2.0Jenkins2016-02-121-0/+12
|\
| * add release notes for deprecated auth bitsSteve Martinelli2016-02-111-0/+12
| | | | | | | | Change-Id: I95ff3940b35d09fd747c49baff69dc7a12451309
* | 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 "Updated from global requirements"Jenkins2016-02-121-1/+1
|\ \ \ | |_|/ |/| |
| * | Updated from global requirementsOpenStack Proposal Bot2016-02-111-1/+1
| | | | | | | | | | | | Change-Id: I469d93e30110dd90633a2ca78091e79bb573657c
* | | 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 "Make pep8 *the* linting interface"Jenkins2016-02-111-9/+2
|\ \ \
| * | | Make pep8 *the* linting interfaceAndreas Jaeger2016-02-101-9/+2
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | According to the PTI (=Python Test Interface, http://governance.openstack.org/reference/cti/python_cti.html), pep8 is the interface for codestyle checks. Move all tests from linters to pep8. Change-Id: If50b7471112151245d383b4a1d0056ce9041adf2 Depends-On: I9346e4bf8c3a50de088c6a157f0d871c0a9333a1
* | | Updated from global requirementsOpenStack Proposal Bot2016-02-101-1/+1
|/ / | | | | | | Change-Id: Ia1f8737451fff215dca109fc3fa49e643dc8e2e0
* | Updated from global requirementsOpenStack Proposal Bot2016-02-082-3/+3
| | | | | | | | Change-Id: I66be09102edcd67f022019744920485d03952e9a
* | 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
* | | Merge "Update keyring requirements"Jenkins2016-02-041-2/+3
|\ \ \ | |_|/ |/| |
| * | 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
* | Merge "Bandit profile updates"Jenkins2016-02-022-17/+2
|\ \
| * | Bandit profile updatesBrant Knudson2016-01-272-17/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We'd like to use the same "gate" profile for all OpenStack gate jobs, so the "keystone_conservative" profile which we were running as part of the gate is change to "gate. The other profiles that aren't used as part of the gate are removed. Change-Id: I931dc957b4659806027d45dfec5e61e9c7973564
* | | Update translation setupAndreas Jaeger2016-01-282-25/+5
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow new infra setup for translations, see spec http://specs.openstack.org/openstack-infra/infra-specs/specs/translation_setup.html for full details. This basically renames python-keystoneclient/locale/python-keystoneclient.pot to keystoneclient/locale/keystoneclient.pot. For this we need to update setup.cfg. The domain name is already correct in keystoneclient/_i18n.py. The project has no translations currently, let's remove the outdated pot file, the updated scripts work without them. So, we can just delete the file and once there are translations, an updated pot file together with translations can be imported automatically. Change-Id: Ie5a0c3e8fcdb485edf05f2188fce64a9fce96b5f
* | Merge "Missing defaults in the create() method in the v2 ServiceManager"Jenkins2016-01-272-2/+33
|\ \ | |/ |/|
| * 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 "Remove Babel from requirements.txt"Jenkins2016-01-271-1/+0
|\ \
| * | Remove Babel from requirements.txtBrant Knudson2016-01-251-1/+0
| |/ | | | | | | | | | | keystoneclient doesn't use Babel directly. Change-Id: Icc34344afe27330a4a896e14e70fe0e31b4e4bc6
* | Merge "Remove bandit tox environment"Jenkins2016-01-271-4/+0
|\ \ | |/ |/|
| * Remove bandit tox environmentAndreas Jaeger2016-01-181-4/+0
| | | | | | | | | | | | | | bandit is now part of linters environment, remove it from tox.ini. Depends-On: I67ebec070cd0804d310534712a3a3f382df7e48f Change-Id: Ie59ee56f15a857926eef4a7b466637686e116b07
* | Merge "Remove argparse from requirements"Jenkins2016-01-251-1/+0
|\ \
| * | Remove argparse from requirementsAndreas Jaeger2016-01-231-1/+0
| | | | | | | | | | | | | | | | | | | | | argparse was external in python 2.6 but not anymore, remove it from requirements. This project is not supporting 2.6 anymore. Change-Id: Ib7e74912b36c1b5ccb514e31fac35efeff57378d
View Lorry Controller Status