summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add release notes for keystoneclient"2.0.0Jenkins2015-12-019-2/+298
|\
| * Add release notes for keystoneclientSteve Martinelli2015-11-299-2/+298
| | | | | | | | | | | | | | as mentioned in the mailing list, we need to include release notes for libraries, note that we do not include changes for liberty. Change-Id: I6497aac36720e2bea3f25316a426ea9fedb96c79
* | Remove hardcoded endpoint filter for update passwordHaneef Ali2015-11-302-2/+23
|/ | | | | | | | | | | User password update hardcoded the endpoint_filter to always use the public endpoint. This will break deployments where services behind the firewall have no access to the public endpoint. Endpoint selection should be allowed by the end user (i.e. openstack --os-interface internal user password set). Closes-Bug: 1503459 Change-Id: Ib11d60cd8e81b99aedb27f1cbbf6b79218045cf0
* Updated from global requirementsOpenStack Proposal Bot2015-11-271-1/+1
| | | | Change-Id: I0bd278c3c138f162381fae854d620afcca5246a5
* remove unnecessary FakeLog class in test codeZhiQiang Fan2015-11-262-26/+6
| | | | | | | | | | | | The FakeLog defined in keystoneclient/tests/unit/test_http.py isn't used anywhere. This patch removes it. fixtures package already provides a FakeLogger for us to use, so we really don't need to maintain a private implementation, this patch removes FakeLog in test_auth_token_middleware and replaces it with fixtures.FakeLogger Change-Id: I6aaf761a9676edf5bd799d22b79497be1d423e7c
* Merge "No keystone Endpoint now gives a valid Error Message"Jenkins2015-11-261-1/+4
|\
| * No keystone Endpoint now gives a valid Error Messagerajiv2015-11-261-1/+4
| | | | | | | | | | | | | | | | When no valid keystone endpoint exist, EndpointNotFound exception is raised with an error message Change-Id: I75b00cb73b18bc19261c061e0ae217ef251f8853 Closes-Bug: #1208991
* | Merge "Removes py26 support"Jenkins2015-11-266-22/+10
|\ \ | |/ |/|
| * Removes py26 supportDavid Stanek2015-11-256-22/+10
| | | | | | | | | | | | | | We are removing Python 2.6 support from the Keystone libraries. Change-Id: I1c7a79edd41a73946c9d77bfb8cd2075e2500760 Closes-Bug: 1519449
* | Merge "Silence most of the deprecation spam"Jenkins2015-11-263-0/+17
|\ \
| * | Silence most of the deprecation spamMonty Taylor2015-11-083-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | Running keystoneclient unittest is really painful, because it's impossible to see what failed in all of the deprecation output. Make most of it go away. There is still a chunk that is being elusive. Change-Id: I006859b016274fc1adf69653ff75b3071d2c9446
* | | Removes discover from test-reqsDave Chen2015-11-251-1/+0
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | The discover package is part of Python 2.7, so we don't need to explicitely require it. It's by the way annoying for distribution package maintainers, as the file has to be patched. Keystone team has decided to drop python 2.6 support[1], so, it's safe to remove it from test-requirements.txt. [1] http://eavesdrop.openstack.org/meetings/keystone/2015/keystone.2015-11-24-17.59.html Related-Bug: #1519449 Co-Authored-By: Thomas Goirand <thomas@goirand.fr> Change-Id: I5bb258a4b305c0084be50bc22fe7a0e6a4f65aad
* | Merge "Map keystoneclient exceptions to keystoneauth"Jenkins2015-11-232-348/+225
|\ \
| * | Map keystoneclient exceptions to keystoneauthJamie Lennox2015-11-192-348/+225
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To allow people to use a keystoneauth session with keystoneclient we need to make it so that any exceptions that keystoneclient catch are the same as what keystoneauth might throw. The only practical way to do this is to map the keystoneclient exceptions onto the keystoneauth equivalents. This is fairly easy as all these exceptions were extracted from keystoneclient initially. Closes-Bug: #1515048 Change-Id: I3b74b0ba1e1f9dda937a2d90e2d75ff0b7597a9b
* | | Fixes warning for positional arg in project createdaniel-a-nguyen2015-11-201-1/+1
| | | | | | | | | | | | | | | Change-Id: I572f5368669e33b48250799c436635d6a398271d Closes-Bug: #1518511
* | | Updated from global requirementsOpenStack Proposal Bot2015-11-202-3/+3
| | | | | | | | | | | | Change-Id: I28ff28b701ec9fc922a64f156848a48da90a461a
* | | Swap the order of username deprecationMonty Taylor2015-11-195-9/+9
|/ / | | | | | | | | | | | | | | | | | | | | | | | | The attempt at a move to user-name is an exercise in churn, and is filling everyone's logs with admonitions to change the name of their variables - which does not work if they do. Swap this, effectively reverting the attempt at a move. user-name will continue to work on the off chance anyone started consuming that path, which is unlikely because none of the consuming programs expose that as an actual option. Closes-Bug: 1498247 Change-Id: I62d991fda1df63c9cbabfde2f6836bc031f5147c
* | Merge "Last sync from oslo-incubator"Jenkins2015-11-141-0/+6
|\ \
| * | Last sync from oslo-incubatorDavanum Srinivas2015-11-131-0/+6
| | | | | | | | | | | | | | | | | | | | | oslo-incubator will cease to host common code soon. This is hopefully the very last sync from oslo-incubator. Sync'ed with oslo-incubator SHA (4bbd5edc8b235592f59a808fa33fd6a994b23b2c) Change-Id: Ie40b2e849f8ed859eb29ffe4f0c6496d7c9f9366
* | | Merge "update incorrect docstring for regions"Jenkins2015-11-121-1/+1
|\ \ \ | |/ / |/| |
| * | update incorrect docstring for regionsSteve Martinelli2015-11-101-1/+1
| | | | | | | | | | | | | | | | | | it currently says endpoints, when it's obviously regions. Change-Id: Ic080436441cb07f5be83e6037cd38aeb14577398
* | | Updated from global requirementsOpenStack Proposal Bot2015-11-112-3/+3
| | | | | | | | | | | | Change-Id: I63782a86ad782e35545a233a5562cfb8f89bd209
* | | Merge "Add missing end single quote"Jenkins2015-11-111-1/+1
|\ \ \
| * | | Add missing end single quoteHidekazu Nakamura2015-11-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | End single quote of is missing. This patch add it. Change-Id: Ibabde8922d97e1d30f7130574080f56ccb8d6be9
* | | | Merge "Iterate over copy of session.adapters keys in Python2/3"Jenkins2015-11-111-1/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Iterate over copy of session.adapters keys in Python2/3Corey Bryant2015-11-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Iterate over a copy of session.adapters keys in both Python 2.x and Python 3.x. In Python 3.x, keys() is not a copy, and therefore items can't be popped from it while iterating. Note that this patch addresses the following error message which is new in Python 3.5. That is, you need to be using Python 3.5+ to reproduce this error. RuntimeError: OrderedDict mutated during iteration https://bugs.python.org/issue24369 https://hg.python.org/cpython/rev/0d8679858272 Change-Id: Iaa2be0dc8ef26e51ce5e8f50049c9e8f84418ec0 Closes-Bug: #1483872
* | | | Merge "Add docstring validation"Jenkins2015-11-102-1/+19
|\ \ \ \ | |_|_|/ |/| | |
| * | | Add docstring validationDolph Mathews2015-11-092-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This introduces a linter for PEP257 to avoid trivial nitpicking of docstrings in code reviews. Because flake8_docstrings simply provides a plugin to add pep257 to flake8, you can run it via `tox -e pep8`. PEP257 checks which we are currently violating are ignored in tox.ini. We can remove them from the ignored list as they are fixed. Change-Id: I01ebad7b70cf61dd80d3c06c6808d8178fbdd634 Related-Bug: 1501544 Depends-On: I60adf0dca4aa32f4ef6bca61250b375c8a3703c6
* | | | Pull the endpoint from the SessionMonty Taylor2015-11-084-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the user passes a Session in, we can pull the endpoint to use for discovery from the Session itself, rather than erroring. Closes-Bug: #1513839 Co-Authored-By: Dolph Mathews <dolph.mathews@gmail.com> Change-Id: I82a41c67f80d2494f04739d82b112b7ff1dc4682
* | | | Replace repeated assertion with the lossZhengYue2015-10-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In unit tests, there are two assertions for 'adminURL', but lost one for 'internalURL'. Fixed the problem. Change-Id: I4694cc121f8ad617a8c8e93e21bf6228f183dab6 Closes-bug: #1479582
* | | | Merge "Mark abstractmethod bodies with nocover"Jenkins2015-10-227-11/+12
|\ \ \ \
| * | | | Mark abstractmethod bodies with nocoverBrant Knudson2015-10-217-11/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | abstractmethod bodies aren't going to be called by unit tests, so there's no way to get coverage. The code in an abstractmethod body should be marked with "# pragma: no cover" so that they don't show up as missed in the coverage report. Change-Id: I88a7481ab22f2ce1abfd62badc5f5048acc6929f
* | | | | Merge "Docstring spelling and function-vs-method fixes"Jenkins2015-10-212-6/+6
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Docstring spelling and function-vs-method fixesDolph Mathews2015-10-212-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are all object methods, not independent functions. Change-Id: I3c232d922e61a94c7dc2c2b9a8d3768fd42be1a7
* | | | | pass on @abc.abstractmethodsDolph Mathews2015-10-216-2/+10
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Although a bare docstring is entirely valid grammar, those new to Python find that pattern baffling, *especially* when other similar methods contain pass statements. So, for consistency, add a pass statement to otherwise bare @abc.abstractmethods. Note that the implementation of an @abc.abstractmethod (in the abstract base class) can still be called by concrete children, so suddenly raising a NotImplementedError() instead might be "surprising" to implementors. A no-op such as "pass" or "return None" is preferable. Change-Id: I79969ad1a3429516ea785c649a165ead54944225
* | | | Updated from global requirementsOpenStack Proposal Bot2015-10-191-1/+1
| | | | | | | | | | | | | | | | Change-Id: Icaf6ab75e5ca438a98c400a17ce7eacc712910aa
* | | | Merge "Fix typo that says V3 token only works for v2"Jenkins2015-10-161-1/+1
|\ \ \ \
| * | | | Fix typo that says V3 token only works for v2Steve Martinelli2015-10-151-1/+1
| | |_|/ | |/| | | | | | | | | | | | | | | | | | The V3 token plugin should work for v3 identity service Change-Id: Ic3c53dd94e1fe5d1ab69a02a7f7469735ee4c978
* | | | auto-generate release historySteve Martinelli2015-10-152-0/+9
|/ / / | | | | | | | | | | | | | | | | | | | | | currently there is no release history for keystoneclient, though sometimes the commits are lacking context, this automated approach is far better than nothing. Change-Id: Ibb865b4830cbe1e2e99688103d26f1378d2c32b1
* | | Updated from global requirements1.8.1OpenStack Proposal Bot2015-10-141-1/+1
| | | | | | | | | | | | Change-Id: I4e994e5be2b5a649c94cc866211af6f3226b6170
* | | Updated from global requirementsOpenStack Proposal Bot2015-10-131-1/+1
| | | | | | | | | | | | Change-Id: I2e09a5b66fde17db0615baf64a5b7d37b6abca4b
* | | Updated from global requirements1.8.0OpenStack Proposal Bot2015-10-131-1/+1
| | | | | | | | | | | | Change-Id: Ie2f43389ee63918071b99c55fd56420f9cfb6ea1
* | | Updated from global requirementsOpenStack Proposal Bot2015-10-091-1/+1
| | | | | | | | | | | | Change-Id: I8c0e393bae99dec6f0153c205d3cd9c3be94ddcc
* | | Merge "Make __all__ immutable"Jenkins2015-10-0716-28/+27
|\ \ \ | |/ /
| * | Make __all__ immutableDolph Mathews2015-10-0116-28/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | Using a mutable type implies that it's acceptable for the set of publicly-accessible attributes to be mutated at runtime, which defeats their intended purpose of documenting the public interface. Tuples are immutable. Change-Id: Ib3ab93224ba240040b08ece481ef5ba620c3f658
* | | Merge "Use dictionary literal for dictionary creation"Jenkins2015-10-071-3/+3
|\ \ \
| * | | Use dictionary literal for dictionary creationAnkit Agrawal2015-09-241-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Dictionary creation could be rewritten as a dictionary literal. for example: token_values = {} token_values['user_id'] = access.get('user', {}).get('id') could be rewritten as token_values = {'user_id': access.get('user', {}).get('id')} TrivialFix Change-Id: I0c5677b527d440b8faded31bf4d9d62805391ae3
* | | | Merge "List creation could be rewritten as a list literal"Jenkins2015-10-071-3/+4
|\ \ \ \ | |_|_|/ |/| | |
| * | | List creation could be rewritten as a list literalAnkit Agrawal2015-09-241-3/+4
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In _http_log_response method of session.py module, list "string_parts' is created as below. string_parts = ['RESP:'] string_parts.append('[%s]' % response.status_code) Could be rewritten as string_parts = [ 'RESP:', '[%s]' % response.status_code ] TrivialFix Change-Id: I83d04a71e030f3904c84cead4659c230393631db
* | | Updated from global requirementsOpenStack Proposal Bot2015-10-021-1/+1
| | | | | | | | | | | | Change-Id: Idfeced4ee4d3902d360a0b7d9f7b57b6d46777c5
View Lorry Controller Status