summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
9 daysChange openstack-dev to openstack-discussHEADmasterVieri
Mailinglists have been updated. Openstack-discuss replaces openstack-dev. Change-Id: Ifd1ee825acd098a6525b5c5f3932ce200cfe730e Notes (review): Code-Review+2: Gage Hugo <gagehugo@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: Thu, 06 Dec 2018 00:16:17 +0000 Reviewed-on: https://review.openstack.org/622765 Project: openstack/keystoneauth Branch: refs/heads/master
13 daysFix version discovery for clouds with int project_ids3.11.2Monty Taylor
On a cloud that has inaccessible version discovery documents AND uses integer project ids, the discovery fallback logic can fail because the project id parses as a (very large) version. Check to see that the url segment in the fallback code begins with a v, so that we're only attempting to parse versions from actual candidate segments. Closes-Bug: #1806109 Change-Id: Id90b3b9e4852494a4678b0a9bb67362babdc971c 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: Sat, 01 Dec 2018 16:25:41 +0000 Reviewed-on: https://review.openstack.org/621257 Project: openstack/keystoneauth Branch: refs/heads/master
2018-11-18Merge "Replacing the HTTP protocal with HTTPS in using-sessions.rst."Zuul
2018-11-17Replacing the HTTP protocal with HTTPS in using-sessions.rst.zhouxinyong
Change-Id: Ib4712414c48ed922ea62460730a4cd5749a3d481 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: Sun, 18 Nov 2018 19:56:40 +0000 Reviewed-on: https://review.openstack.org/617811 Project: openstack/keystoneauth Branch: refs/heads/master
2018-11-17Merge "Add py36 tox environment"Zuul
2018-11-06Add py36 tox environmentColleen Murphy
We already run python3.6 unit tests in CI. Add the py36 environment to the tox file so that developers with python3.6 available locally can opt into running that version too. Change-Id: I499db960450b9628636d503a4d7f1cc163d38e3a Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+1: Vishakha Agarwal <agarwalvishakha18@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: Sat, 17 Nov 2018 00:58:18 +0000 Reviewed-on: https://review.openstack.org/615845 Project: openstack/keystoneauth Branch: refs/heads/master
2018-10-30fix wrong spelling of "unnecessary"wangqiangbj
Change-Id: I1db36cab2a3dce4c1ae1c1de0d17c945e779bd9d Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Workflow+1: Colleen Murphy <colleen@gazlene.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 02 Nov 2018 15:01:35 +0000 Reviewed-on: https://review.openstack.org/614178 Project: openstack/keystoneauth Branch: refs/heads/master
2018-10-24Add missing release note for ironic discovery fixMonty Taylor
Change-Id: I3f0a6047ecb0569553d17eb7a066ac7023817c66 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: Fri, 26 Oct 2018 01:36:33 +0000 Reviewed-on: https://review.openstack.org/612872 Project: openstack/keystoneauth Branch: refs/heads/master
2018-10-23Make new-style single endpoint version discovery actually work for ironic3.11.1Dmitry Tantsur
For (unclear) historical reasons the root single version endpoint also contains "id" and "links" fields. This makes the current workaround for old-style endpoints take priority over the correct algorithm. This change reorders the code, so that if "version" is present, it always take priority over the workaround. The unit tests are updated to be closer to real output from ironic. Change-Id: I743b954c6c5b2f986c213acb6ec6af7e08c9f5f8 Notes (review): Code-Review+1: Julia Kreger <juliaashleykreger@gmail.com> 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: Tue, 23 Oct 2018 17:06:14 +0000 Reviewed-on: https://review.openstack.org/612680 Project: openstack/keystoneauth Branch: refs/heads/master
2018-10-16Update sphinx extension loggingSean 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