summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix response body being omitted in debug mode incorrectlynewton-eol3.5.1stable/newtonTin Lam2017-01-172-2/+18
| | | | | | | | | | | In debug mode, when a response's header Content-Type is set to "application/json" with a parameter, i.e., "application/json; charset=UTF-8". This patch set ignores the additional parameter and only match the mimetype. Change-Id: Ie8fcb1061e0e49b039436947524cfdc704c83846 Closes-Bug: #1656981 (cherry picked from commit dcb719d0e51afa0253c144136b41f0e390c48c4c)
* Only log application/json content typeTobias 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/newtonJenkins2017-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)
* Revert "Add auth functional tests"Rodrigo Duarte Sousa2016-09-221-39/+0
| | | | | | | | | | | | This reverts commit 2a91a41142080a4f1abb8f9846f88846c6b5c6b7. This is a quick fix to unblock the python-keystoneclient gate. Not closing the bug since we can fix the test behavior by using an extra client just to run the tests operations. Change-Id: I976488243ad3c357cd2aeb604973603380858dc4 Related-Bug: 1626131 (cherry picked from commit ca669701f7ca8ef17d32ab7b56430cef1ce23431)
* Update .gitreview for stable/newtonDoug Hellmann2016-09-021-0/+1
| | | | Change-Id: I3081404d0b4992f3e30a185fa212b7b5d426a2ab
* Merge "Do not send user ids as payload"3.5.0Jenkins2016-08-222-11/+3
|\
| * Do not send user ids as payloadBoris Bobrov2016-07-292-11/+3
| | | | | | | | | | | | | | | | User ids are already in the URL. Keystone doesn't consume ids in the body. Change-Id: Ie90ebd32fe584dd1b360dc75a828316b1a9aedde Closes-Bug: 1607751
* | Updated from global requirementsOpenStack Proposal Bot2016-08-201-1/+1
| | | | | | | | Change-Id: If3bc6b1f3d66f982f18c1373f27da7ad26a2a7cc
* | Merge "Improve docs for v3 tokens"Jenkins2016-08-191-14/+15
|\ \
| * | Improve docs for v3 tokensNisha Yadav2016-08-181-14/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | In preparation to add functional tests for v3 tokens, this change proposes to detail the method docs, because the tests need to be based on them. Change-Id: I5e09a0663c5490fa4dcedeb900c7668a93a8aebd Partial-Bug: #1330769
* | | Merge "Follow up patch for Improve docs for v3 policies"Jenkins2016-08-191-4/+5
|\ \ \
| * | | Follow up patch for Improve docs for v3 policiesNisha Yadav2016-08-191-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes few nits left by the review I8fef9c2ec54e38648e1000874c6d2381b6fad576 Change-Id: I31ea3ba97c806f9bb01d33c4a5b429d309816117 Partial-Bug: #1330769
* | | | Merge "Follow up patch for Improve docs for v3 services"Jenkins2016-08-191-3/+5
|\ \ \ \
| * | | | Follow up patch for Improve docs for v3 servicesNisha Yadav2016-08-191-3/+5
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes a nit left by the review Ic7aaadcd8f8d5ebdec0cd908e7cbe4f997a17b54 Change-Id: Ia5b6ea552c37fc7ec4d82ffc2a7101fcf9b8f0e5 Partial-Bug: #1330769
* | | | Merge "Follow up patch for Improve docs for v3 domains"Jenkins2016-08-191-0/+3
|\ \ \ \
| * | | | Follow up patch for Improve docs for v3 domainsNisha Yadav2016-08-191-0/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes a nit left by the review I78e62fbd877ba32744fa3587ad2eb497bfb79fc2 Change-Id: I63bc438bca5f989ab79dc2393ca9d0a0d2e4af4e Partial-Bug: #1330769
* | | | Merge "Follow up patch for Add ec2 functional tests"Jenkins2016-08-191-0/+22
|\ \ \ \
| * | | | Follow up patch for Add ec2 functional testsNisha Yadav2016-08-191-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds the test_list_ec2 left by the review I19988661e2d92a3e501a04e2a0e2d773ddcc36e9 Change-Id: Id53fbcb3e82d3bc47babd1129d533dfbc6c13791
* | | | | Merge "Add auth functional tests"Jenkins2016-08-191-0/+39
|\ \ \ \ \
| * | | | | Add auth functional testsNisha Yadav2016-08-181-0/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds functional tests for auth. Change-Id: I62b42b7318cb0a85dee280931e44f6e5852216f0
* | | | | | Merge "Reuse Domain and Project resouce definitions"Jenkins2016-08-191-30/+4
|\ \ \ \ \ \ | |/ / / / / | | | / / / | |_|/ / / |/| | | |
| * | | | Reuse Domain and Project resouce definitionsSamuel de Medeiros Queiroz2016-08-181-30/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Domain and Project resources are defined in v3/domains.py and v3/projects.py, respectively. The v3/auth.py module has some functions that returns projects and domains. However, it is redifining those resources. This change makes the auth module re-use of the existing definitions. Change-Id: Id7bd527a4b972a4259a66e36c684f74533e57f79
* | | | | Merge "Fix no content return type doc"Jenkins2016-08-184-8/+16
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix no content return type docEduardo Magalhães2016-08-184-8/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replaces the occurrences of "204 No Content" with the correct return and rtype. Change-Id: Id782cda4d226c96c31d375fd2432e9d7e28e2f77
* | | | | Merge "Follow up patch for Improve docs for v3 ec2"Jenkins2016-08-181-6/+6
|\ \ \ \ \
| * | | | | Follow up patch for Improve docs for v3 ec2Nisha Yadav2016-08-181-6/+6
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes a nit left by the review I57bf96a60baed3a9420879f4f897e73a40ff102a Change-Id: I958e6cb1978c100d735e59c12ee662070e7fdca8 Partial-Bug: #1330769
* | | | | Merge "Add ec2 functional tests"Jenkins2016-08-182-0/+88
|\ \ \ \ \ | |/ / / / |/| | / / | | |/ / | |/| |
| * | | Add ec2 functional testsNisha Yadav2016-08-172-0/+88
| |/ / | | | | | | | | | | | | | | | Adds functional tests for ec2. Change-Id: I19988661e2d92a3e501a04e2a0e2d773ddcc36e9
* | | Improve docs for v3 authNisha Yadav2016-08-171-11/+25
|/ / | | | | | | | | | | | | | | | | In preparation to add functional tests for v3 auth, this change proposes to detail the method docs, because the tests need to be based on them. Change-Id: Idc60cb077d7e403a47cf9d6ad72f9d1a215c3f3b Partial-Bug: #1330769
* | Merge "Improve docs for v3 ec2"Jenkins2016-08-161-12/+53
|\ \
| * | Improve docs for v3 ec2Nisha Yadav2016-08-161-12/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | In preparation to add functional tests for v3 ec2, this change proposes to detail the method docs, because the tests need to be based on them. Change-Id: I57bf96a60baed3a9420879f4f897e73a40ff102a Partial-Bug: #1330769
* | | Merge "Add credential functional tests"Jenkins2016-08-162-0/+214
|\ \ \ | |/ /
| * | Add credential functional testsNisha Yadav2016-08-162-0/+214
| | | | | | | | | | | | | | | | | | Adds functional tests for credentials. Change-Id: Ia483c47956747a241bc521c0e67a874e794d709c
* | | Move other-requirements.txt to bindep.txtAndreas Jaeger2016-08-121-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default filename for documenting binary dependencies has been changed from "other-requirements.txt" to "bindep.txt" with the release of bindep 2.1.0. While the previous name is still supported, it will be deprecated. Move the file around to follow this change. Note that this change is self-testing, the OpenStack CI infrastructure will use a "bindep.txt" file to setup nodes for testing. For more information about bindep, see also: http://docs.openstack.org/infra/manual/drivers.html#package-requirements http://docs.openstack.org/infra/bindep/ As well as this announcement: http://lists.openstack.org/pipermail/openstack-dev/2016-August/101590.html Change-Id: Ibffad3665fea9be1ee6404b3f99ee0f23feb50e5
* | | Merge "Add __ne__ built-in function"Jenkins2016-08-112-3/+19
|\ \ \
| * | | Add __ne__ built-in functionyuyafei2016-08-112-3/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Python 3 __ne__ by default delegates to __eq__ and inverts the result, but in Python 2 they urge you to define __ne__ when you define __eq__ for it to work properly [1].There are no implied relationships among the comparison operators. The truth of x==y does not imply that x!=y is false. Accordingly, when defining __eq__(), one should also define __ne__() so that the operators will behave as expected. [1]https://docs.python.org/2/reference/datamodel.html#object.__ne__ Also fixes spelling errors:resoruces. Change-Id: Iae4ce0fe84fae810711cc8c3fdb94eb9ca1d772e Closes-Bug: #1586268
* | | | Merge "Improve docs for v3 credentials"Jenkins2016-08-061-32/+57
|\ \ \ \ | | |/ / | |/| |
| * | | Improve docs for v3 credentialsNisha Yadav2016-08-051-32/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In preparation to add functional tests for v3 credentials, this change proposes to detail the method docs, because the tests need to be based on them. Change-Id: Iaabf78b9b1c5e46e5812f6cf6b7896c45675be2e Partial-Bug: #1330769
* | | | Merge "Add role functional tests"Jenkins2016-08-051-0/+236
|\ \ \ \ | |/ / /
| * | | Add role functional testsNisha Yadav2016-08-051-0/+236
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds functional tests for roles. For now, the tests are created under a single class. Once we have a gate that runs against LDAP, we will create a class that only contains readonly tests and a tox call for it (e.g tox -e functional-readonly). Change-Id: I62b022044b56b6219a83e9e5cc0136c95389db16
* | | | Merge "Add Python 3.5 classifier"Jenkins2016-08-051-0/+1
|\ \ \ \ | |_|/ / |/| | |
| * | | Add Python 3.5 classifierzheng yin2016-08-051-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that there is a passing gate job, we can claim support for Python 3.5 in the classifier. Change-Id: I87c7cd72cdddad4165e1a0b24aec94fbbc61acce
* | | | Merge "Follow up patch for Improve docs for v3 roles"Jenkins2016-08-041-1/+0
|\ \ \ \ | | |/ / | |/| |
| * | | Follow up patch for Improve docs for v3 rolesNisha Yadav2016-08-041-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes a nit left by the review I67d0649f734d19e1b9547f8ebe5dced1ce8cf7f8 Change-Id: I592dd31e5ebe47e115c6e93850f31c8f4239b561
* | | | Merge "Updated from global requirements"Jenkins2016-08-041-1/+1
|\ \ \ \
| * | | | Updated from global requirementsOpenStack Proposal Bot2016-08-041-1/+1
| | | | | | | | | | | | | | | | | | | | Change-Id: I5c93bd76215050cb0f671a053d49f52c570285f4
* | | | | Merge "Improve docs for v3 roles"Jenkins2016-08-041-21/+217
|\ \ \ \ \ | |/ / / / |/| / / / | |/ / /
| * | | Improve docs for v3 rolesNisha Yadav2016-08-041-21/+217
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In preparation to add functional tests for v3 roles, this change proposes to detail the method docs, because the tests need to be based on them. Change-Id: I67d0649f734d19e1b9547f8ebe5dced1ce8cf7f8 Partial-Bug: #1330769
View Lorry Controller Status