summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
3 daysUpdate sphinx extension loggingHEADmasterSean McGinnis
Sphinx 1.6 deprecated using the application object to perform logging and it will be removed in the upcoming 2.0 release. This updates our extensions to use the recommended sphinx.util.logging instead. Change-Id: I3abce4e3c147befd0235820cb8850fe18f6dee42 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com> Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 19 Oct 2018 00:44:29 +0000 Reviewed-on: https://review.openstack.org/611135 Project: openstack/keystoneauth Branch: refs/heads/master
2018-09-25Reformat Adapter docstringMonty Taylor
Typing docstrings for the next patch was annoying. Reformat the docstrings to wrap at the front rather than in visual blocks. Change-Id: I08fc1e45e032197f3bb0b8311c032b471494ef80 Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Workflow+1: Gage Hugo <gagehugo@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 25 Sep 2018 18:41:41 +0000 Reviewed-on: https://review.openstack.org/605042 Project: openstack/keystoneauth Branch: refs/heads/master
2018-09-23Cache root urls with and without trailing slashesMonty Taylor
The trailng slash on a pathless url is not meaningful, but we were treating the url given to the discovery cache as if it were. In some circumstances, such as an endpoint_override that didn't match the found discovery document perfectly, a double-request could be made. Normalize root urls in the caching code so that https://example.com and https://example.com/ would be the same. Change-Id: I70a5911cf0f213a7816fe8d58c6cca4702ff71bb Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 25 Sep 2018 07:03:01 +0000 Reviewed-on: https://review.openstack.org/604635 Project: openstack/keystoneauth Branch: refs/heads/master
2018-09-12Merge "Protect against endpoint_data not existing"3.11.0Zuul
2018-09-12Merge "Add support for ironic single-version responses"Zuul
2018-09-11Protect against endpoint_data not existingMonty Taylor
It's possible in get_api_major_version that the endpoint in question is not found at all. In that case, we are documented to return None, but what we do instead is throw an exception trying to get data off of the None object. Change-Id: I06ad497854f4e95a1a2a4a93241b244fc476b139 Notes (review): Code-Review+2: ayoung <ayoung@redhat.com> Code-Review+1: Filippo Inzaghi <zhao.daqing@99cloud.net> Code-Review+2: wangxiyuan <wangxiyuan@huawei.com> Workflow+1: wangxiyuan <wangxiyuan@huawei.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 12 Sep 2018 05:29:29 +0000 Reviewed-on: https://review.openstack.org/601636 Project: openstack/keystoneauth Branch: refs/heads/master
2018-09-06Add support for ironic single-version responsesMonty Taylor
The ironic payload looks like: {'id': 'v1', 'links': [{"href": "https://bare-metal.example.com/v1/", "rel": "self"}]} This does not have version info in it, nor min/max ranges for microversion discovery. We can't really get any useful information from this document, but we can at least not fail when trying to deal with it. This should then be upwards-compatible with ironic adding version discovery information to the document that is returned. Change-Id: I47e0f9b295c24ef168f4a033faf573b953025d4c Notes (review): Code-Review+1: Robert Putt <robert.putt@rackspace.co.uk> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: wangxiyuan <wangxiyuan@huawei.com> Workflow+1: wangxiyuan <wangxiyuan@huawei.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 12 Sep 2018 05:29:29 +0000 Reviewed-on: https://review.openstack.org/595980 Project: openstack/keystoneauth Branch: refs/heads/master
2018-09-06Merge "Remove os-testr from requirements"Zuul
2018-09-05Remove os-testr from requirementsColleen Murphy
keystoneauth is switched to stestr, we don't need to depend on os-testr anymore. Change-Id: Ie3fc5e5b03f5c0865bf3f499666f504703cd0b0d Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Workflow+1: Lance Bragstad <lbragstad@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 06 Sep 2018 20:38:25 +0000 Reviewed-on: https://review.openstack.org/600004 Project: openstack/keystoneauth Branch: refs/heads/master
2018-08-29add lib-forward-testing-python3 test jobDoug Hellmann
This is a mechanically generated patch to add a functional test job running under Python 3 as part of the python3-first goal. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: I4af5c3a4abea74e371f8656535c80fc79e1e442c Story: #2002586 Task: #24304 Notes (review): Code-Review+2: Gage Hugo <gagehugo@gmail.com> Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Workflow+1: Lance Bragstad <lbragstad@gmail.com> Code-Review+1: melissaml <ma.lei@99cloud.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 05 Sep 2018 22:17:43 +0000 Reviewed-on: https://review.openstack.org/597658 Project: openstack/keystoneauth Branch: refs/heads/master
2018-08-29add python 3.6 unit test jobDoug Hellmann
This is a mechanically generated patch to add a unit test job running under Python 3.6 as part of the python3-first goal. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: I01201d3866201a04ab36c8cf92413ef6de5e8511 Story: #2002586 Task: #24304 Notes (review): Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Code-Review+1: melissaml <ma.lei@99cloud.net> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Workflow+1: Gage Hugo <gagehugo@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 05 Sep 2018 22:17:41 +0000 Reviewed-on: https://review.openstack.org/597657 Project: openstack/keystoneauth Branch: refs/heads/master
2018-08-29switch documentation job to new PTIDoug Hellmann
This is a mechanically generated patch to switch the documentation jobs to use the new PTI versions of the jobs as part of the python3-first goal. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: I30f3bc0f5b68190f72b015ae5a14251c91eeee42 Story: #2002586 Task: #24304 Notes (review): Code-Review+1: Lance Bragstad <lbragstad@gmail.com> Code-Review+1: Gage Hugo <gagehugo@gmail.com> Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 05 Sep 2018 21:43:03 +0000 Reviewed-on: https://review.openstack.org/597656 Project: openstack/keystoneauth Branch: refs/heads/master
2018-08-29import zuul job settings from project-configDoug Hellmann
This is a mechanically generated patch to complete step 1 of moving the zuul job settings out of project-config and into each project repository. Because there will be a separate patch on each branch, the branch specifiers for branch-specific jobs have been removed. Because this patch is generated by a script, there may be some cosmetic changes to the layout of the YAML file(s) as the contents are normalized. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: Iab3d02dbf7b6df016bf44f7be7b27c01f498601e Story: #2002586 Task: #24304 Notes (review): Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Code-Review+1: melissaml <ma.lei@99cloud.net> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Workflow+1: Gage Hugo <gagehugo@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sat, 01 Sep 2018 04:42:15 +0000 Reviewed-on: https://review.openstack.org/597655 Project: openstack/keystoneauth Branch: refs/heads/master
2018-08-21Add Keystone2Keystone exampleColleen Murphy
Since we removed the keystoneauth example in the main docs in favor of an openstackclient example[1] add an example of using the Keystone2Keystone auth plugin. [1] https://review.openstack.org/591587 Change-Id: I5815fecbfe53d8a191a8a64912dac17e66ca928a Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Workflow+1: Gage Hugo <gagehugo@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 27 Aug 2018 23:35:54 +0000 Reviewed-on: https://review.openstack.org/594156 Project: openstack/keystoneauth Branch: refs/heads/master
2018-08-18Merge "add release notes to readme.rst"Zuul
2018-08-17Change log hashing to SHA256Gage Hugo
With the recent Bandit update[0], the usage of SHA1 is now being tagged as an issue. This changes the hashing of logs to SHA256 instead of SHA1. Change-Id: Icde62b8d5ff78b4155e9df8231d63be3ecc53520 Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 17 Aug 2018 22:10:35 +0000 Reviewed-on: https://review.openstack.org/593094 Project: openstack/keystoneauth Branch: refs/heads/master
2018-08-15add release notes to readme.rstliushuobj
Change-Id: Ib63209b676fe9d0d949fff3ba975bcd99d98ea6b Notes (review): Code-Review+1: lvxianguo <lvxianguo@inspur.com> Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Workflow+1: Gage Hugo <gagehugo@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sat, 18 Aug 2018 07:00:05 +0000 Reviewed-on: https://review.openstack.org/591943 Project: openstack/keystoneauth Branch: refs/heads/master
2018-07-26Update reno for stable/rockyOpenStack Release Bot
Change-Id: I4bd8f5afed5fc6488144e509afe2f5c7f5df5669 Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: wangxiyuan <wangxiyuan@huawei.com> Workflow+1: wangxiyuan <wangxiyuan@huawei.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 16 Aug 2018 09:20:04 +0000 Reviewed-on: https://review.openstack.org/586083 Project: openstack/keystoneauth Branch: refs/heads/master
2018-07-24Merge "Add ability to filter version data by service-type"Zuul
2018-07-23Add ability to filter version data by service-typeMonty Taylor
The get_all_version_data method is useful for getting a full listing of what's going on with version discovery on a cloud. Sometimes though people just want to see the versions for a specific service. Add a filter to allow skipping making the version discovery call in the first place, instead of needing to do that as a post-filtering step. Change-Id: Ia3ca4be2976d1a5e7914fa8f2adbf7297e8cb1e1 Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 24 Jul 2018 18:22:43 +0000 Reviewed-on: https://review.openstack.org/585029 Project: openstack/keystoneauth Branch: refs/heads/master
2018-07-19Add netloc and version check for version discovery3.10.0wangxiyuan
If the url netloc in the catalog and service's response are not the same, we should choose the catalog's and add the version info to it if needed. Change-Id: If78d368bd505156a5416bb9cbfaf988204925c79 Closes-bug: #1733052 Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Workflow+1: Lance Bragstad <lbragstad@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 19 Jul 2018 16:52:18 +0000 Reviewed-on: https://review.openstack.org/583215 Project: openstack/keystoneauth Branch: refs/heads/master
2018-06-15Add minimum version for requirements3.9.0wangxiyuan
OpenStack requirements now checks the minimum verion for each requirements library[1]. This patch added the missing one to let the CI happy. [1]: https://review.openstack.org/#/c/574367 Change-Id: I0deaa8f2b3248955d5a7baa7b2cb531516343ffe Notes (review): Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Workflow+1: Gage Hugo <gagehugo@gmail.com> Code-Review+1: wangqi <wang.qi@99cloud.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 19 Jun 2018 19:39:27 +0000 Reviewed-on: https://review.openstack.org/575685 Project: openstack/keystoneauth Branch: refs/heads/master
2018-06-12raise_exc default in AdapterEric Fried
It can be annoying to have to say raise_exc=False (or use try/except) on every call when talking to an API where 4xx response codes are useful/normal/informative or where the preferred coding style is to use conditionals rather than try/except. With this change, the Adapter constructor takes a new kwarg, raise_exc. It defaults to None, and the existing behavior is unchanged. If set to a boolean value, that is used as the default for requests. Specifying raise_exc to the primitives (get, head, put, post, patch, delete, request) at any point along the chain will still take precedence. Change-Id: Ie291c3cb891467728d8ca33cf62afdab37c82f34 Closes-Bug: #1776501 Notes (review): Code-Review+1: Chris Dent <cdent@anticdent.org> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 12 Jun 2018 19:45:56 +0000 Reviewed-on: https://review.openstack.org/574784 Project: openstack/keystoneauth Branch: refs/heads/master
2018-06-09Merge "fix tox python3 overrides"3.8.0Zuul
2018-06-06fix tox python3 overridesDoug Hellmann
We want to default to running all tox environments under python 3, so set the basepython value in each environment. We do not want to specify a minor version number, because we do not want to have to update the file every time we upgrade python. We do not want to set the override once in testenv, because that breaks the more specific versions used in default environments like py35 and py36. Change-Id: I2db130183c0a657b281e0990593fb86be6bef017 Signed-off-by: Doug Hellmann <doug@doughellmann.com> Notes (review): Code-Review+2: Gage Hugo <gagehugo@gmail.com> Code-Review+1: Nguyen Hai <nguyentrihai93@gmail.com> Code-Review+1: zhongshengping <chdzsp@163.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sat, 09 Jun 2018 04:24:09 +0000 Reviewed-on: https://review.openstack.org/572906 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-29Add optional support for retrying certain HTTP codesDmitry Tantsur
Ironic commonly returns HTTP 409 when a node is locked by another routine and HTTP 503 when the conductor has no free threads to process the request. Currently it is managed by custom code in ironicclient and openstacksdk, this change will allow to move it to Session itself. Change-Id: I04e356e7856b020cd20aa598e291ef31e02730d2 Notes (review): Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 06 Jun 2018 19:58:45 +0000 Reviewed-on: https://review.openstack.org/570934 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-17Merge "Expose version_between as a real function"3.7.0Zuul
2018-05-17Merge "Add oslo.config option for split-loggers"Zuul
2018-05-17Merge "Collect timing information for API calls"Zuul
2018-05-16Merge "Return the endpoint_override from EndpointData"Zuul
2018-05-16Add oslo.config option for split-loggersMonty Taylor
For people setting creating Sessions via load_from_conf_options, such as the OpenStack services, turning on split-loggers needs to be done in a config file. In order to do that, we need to expose it in the conf options list. Don't add it to the argparse options for now - it would just add another command line option that is less likely to see use. Change-Id: I106c6acbe306e581d293612630ec810c11d9d61c Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Code-Review+1: Cuong Nguyen <cuongnv@vn.fujitsu.com> Code-Review+1: Neha Alhat <neha.alhat@nttdata.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 17 May 2018 21:06:52 +0000 Reviewed-on: https://review.openstack.org/568878 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-16Collect timing information for API callsMonty Taylor
python-openstackclient does this in a wrapper class around Session, and openstacksdk does something similar that could be removed if support were directly in keystoneauth. Add this so that we can remove the custom wrapper/manipulation in openstackclient and openstacksdk. Change-Id: Icf00c66f57d20d2cef724c233160d3b1e0d52102 Notes (review): Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 17 May 2018 21:06:52 +0000 Reviewed-on: https://review.openstack.org/568877 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-16Return the endpoint_override from EndpointDataEric Fried
Setting up an Adapter with an endpoint_override, calling get_endpoint_data() on that adapter, and accessing the `url` @property of the resulting EndpointData now regurgitates the endpoint_override initially set on the Adapter. Change-Id: I1033c97a7b0e19a75fa0d3c5682cf18b75918514 Closes-Bug: #1707993 Notes (review): Code-Review+1: Eric Fried <efried@us.ibm.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Workflow+1: Lance Bragstad <lbragstad@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 16 May 2018 21:52:28 +0000 Reviewed-on: https://review.openstack.org/491947 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-15Expose version_between as a real functionMonty Taylor
We expose version_to_string and version_match but not version_between. openstacksdk would really like to use version_between too for matching microversion suitability. Turn it in to a public function. Change-Id: I710f9e1441f4caeb9bd9830f9d4a3398a71249ec Notes (review): Code-Review+2: Gage Hugo <gagehugo@gmail.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 17 May 2018 21:06:53 +0000 Reviewed-on: https://review.openstack.org/568640 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-15Fix logging of encoded headers3.6.2Brian Rosmaita
A change introduced in 3.5.0 sorts headers, but runs into a problem when the headers are bytes, such as the headers provided by the python-glanceclient. requests expects headers to be str type in both python2 and python3. This means in python2 we need to encode unicode objects as ASCII (the encoding that should be used for HTTP headers) and in python3 we need to decode bytes as ASCII into str. Change-Id: Ib81497c3a873616c22ba68256c596a6fb113e11e Closes-bug: #1766235 Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 16 May 2018 20:45:07 +0000 Reviewed-on: https://review.openstack.org/568365 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-10Fix interface argument to get_all_version_data3.6.1Monty Taylor
The new get_all_version_data call lists 'public' as the default value for interface, but had None in the arguments. 'public' was the intent, and is what the similar call on the base auth plugin does. Change-Id: I5db2b66d0c130ed80b13ee0c41e0e37c6f7234a7 Notes (review): Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: Colleen Murphy <colleen@gazlene.net> Workflow+1: Colleen Murphy <colleen@gazlene.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 11 May 2018 02:54:37 +0000 Reviewed-on: https://review.openstack.org/567702 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-08Merge "Reference class variable in Status"Zuul
2018-05-08Merge "Infer version from old versioned service type aliases"3.6.0Zuul
2018-05-03Reference class variable in StatusLance Bragstad
This is trivial change to use an existing class variable instead of redefining a string. Change-Id: Ic4fbbc3b735a30babdfee74225975152476196e3 Notes (review): Code-Review+1: Eric Fried <efried@us.ibm.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 08 May 2018 20:37:19 +0000 Reviewed-on: https://review.openstack.org/564262 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-03Use Status variables in testsLance Bragstad
A recent change pushed API status definitions into a class: I93ee971125bc0c7a497e1fb839df38ebd38340e1 We can reuse this in the tests instead of duplicating the same string across assertions. Change-Id: I07a602a05f896d7cc70120bd89424e3c553baf9f Notes (review): Code-Review+1: Eric Fried <efried@us.ibm.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 08 May 2018 15:40:14 +0000 Reviewed-on: https://review.openstack.org/564258 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-03Infer version from old versioned service type aliasesMonty Taylor
The last piece of service type alias support is to handle volumev2, volumev3, workflowv2, workflowv3 and friends. Although it's an annoying scenario, luckily legacy code that uses them has a clear meaning. volumev2, version='3' is just legit not a thing. Needed-By: https://review.openstack.org/564494 Change-Id: Iec09bcb16d8e9b09e09bf12d03c2a55e679ad70c Notes (review): Code-Review+1: Eric Fried <efried@us.ibm.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Workflow+1: Gage Hugo <gagehugo@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 08 May 2018 15:40:15 +0000 Reviewed-on: https://review.openstack.org/564299 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-03Turn normalize_status into a classMonty Taylor
For better readability, make normalize_status a class that has constants for each of the status values. While in there, add a test for unknown status values. Co-Authored-By: Eric Fried <efried@us.ibm.com> Change-Id: I93ee971125bc0c7a497e1fb839df38ebd38340e1 Notes (review): Code-Review+1: Eric Fried <efried@us.ibm.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: Rodrigo Duarte <rodrigodsousa@gmail.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 08 May 2018 15:12:12 +0000 Reviewed-on: https://review.openstack.org/564110 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-03Make VersionData classMonty Taylor
We're repeating ourselves a bunch with a plain dict that contains the version data. Make a class to encapsulate it. Make the class a subclass of dict so that json translation works. Change-Id: Ic7d122487174a5b0378a8dfaa39514c5cb2a604b Notes (review): Code-Review+1: Lance Bragstad <lbragstad@gmail.com> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 08 May 2018 15:12:11 +0000 Reviewed-on: https://review.openstack.org/564469 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-03Add methods to get all of the version dataMonty Taylor
We've got great discovery support, but if someone is wanting to find out what is available and doesn't otherwise know what they're looking for, they're out of luck. Add a method to EndpointData which will return all of the version data for a given service, and then add a method to the base auth plugin that will use that method to collect all of the version discovery documents for every service in the cloud. This commit adds os-service-types so that the resulting datastructure can return only official service type keys. A followup patch will also use os-service-types to allow catalog lookups by service-type alias. There is a change to the test_identity_common.V2.get_auth_data method to remove the public and internal urls for keystone from the catalog. The V3 catalog only has keystone on admin, so this makes them have equivilent data. Change-Id: I07243edb939865a5df8b283e7c626874ffd830db Notes (review): Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 08 May 2018 12:03:52 +0000 Reviewed-on: https://review.openstack.org/559154 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-03Implement service_type alias lookupsMonty Taylor
The Service Types Authority has grown support for aliases, and the os-service-types library exposes the data. Add support for matching known aliases when matching endpoints for a user. Change-Id: Ie90c265cb17905981d877abfaaa52354a3e63692 Notes (review): Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Code-Review+2: Gage Hugo <gagehugo@gmail.com> Workflow+1: Gage Hugo <gagehugo@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 08 May 2018 12:02:16 +0000 Reviewed-on: https://review.openstack.org/462218 Project: openstack/keystoneauth Branch: refs/heads/master
2018-05-02Merge "fix a typo in session.py"Zuul
2018-05-02Merge "Trivial: Update pypi url to new url"Zuul
2018-05-01Merge "Allow tuples and sets in interface list"Zuul
2018-05-01Trivial: Update pypi url to new urlXiaojueGuan
Change-Id: I226e5df7d6bab44da643d3861f5e5b02612250bc Notes (review): Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Workflow+1: Morgan Fainberg <morgan.fainberg@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 02 May 2018 16:19:32 +0000 Reviewed-on: https://review.openstack.org/565418 Project: openstack/keystoneauth Branch: refs/heads/master
2018-04-26Allow tuples and sets in interface listMonty Taylor
While writing an SDK patch, I accidentaly sent a tuple for interface and things went boom. We already accept a list, so go ahead and accept tuples and sets too. Change-Id: I4b14792c7eda4e489c53fdc95e40bde14a61c052 Notes (review): Code-Review+2: Morgan Fainberg <morgan.fainberg@gmail.com> Code-Review+2: Lance Bragstad <lbragstad@gmail.com> Workflow+1: Lance Bragstad <lbragstad@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 01 May 2018 21:58:31 +0000 Reviewed-on: https://review.openstack.org/564495 Project: openstack/keystoneauth Branch: refs/heads/master