| *** camelCaser has quit IRC | 00:11 | |
| *** camelCas- has joined #openstack-sdks | 00:11 | |
| *** logan- has quit IRC | 00:12 | |
| *** logan- has joined #openstack-sdks | 00:13 | |
| *** ricolin has joined #openstack-sdks | 01:02 | |
| *** ricolin_ has joined #openstack-sdks | 01:17 | |
| *** ricolin has quit IRC | 01:20 | |
| *** bobh has joined #openstack-sdks | 01:40 | |
| *** ricolin_ is now known as ricolin | 02:27 | |
| *** bobh has quit IRC | 02:53 | |
| *** whoami-rajat has joined #openstack-sdks | 03:19 | |
| *** icarusfactor has joined #openstack-sdks | 05:08 | |
| *** factor has quit IRC | 05:08 | |
| *** Luzi has joined #openstack-sdks | 05:19 | |
| *** icarusfactor has quit IRC | 06:01 | |
| *** factor has joined #openstack-sdks | 06:02 | |
| *** slaweq has joined #openstack-sdks | 06:48 | |
| *** dtantsur|afk is now known as dtantsur | 06:55 | |
| *** tosky has joined #openstack-sdks | 07:22 | |
| *** ralonsoh has joined #openstack-sdks | 07:29 | |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 07:52 |
|---|---|---|
| *** jpich has joined #openstack-sdks | 07:53 | |
| *** ralonsoh has quit IRC | 07:54 | |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 07:59 |
| *** jangutter has joined #openstack-sdks | 08:05 | |
| *** e0ne has joined #openstack-sdks | 08:18 | |
| *** cdent has joined #openstack-sdks | 08:23 | |
| *** ralonsoh has joined #openstack-sdks | 08:26 | |
| *** ralonsoh has quit IRC | 08:35 | |
| *** ralonsoh has joined #openstack-sdks | 08:47 | |
| *** ricolin_ has joined #openstack-sdks | 10:01 | |
| *** ricolin has quit IRC | 10:03 | |
| dtantsur | mordred: we don't have anything to build swift temporary URLs in SDK, do we? | 10:10 |
| dtantsur | we could probably just copy https://opendev.org/openstack/python-swiftclient/src/branch/master/swiftclient/utils.py#L71 as it is :) | 10:13 |
| *** gtema has joined #openstack-sdks | 10:26 | |
| *** jangutter_ has joined #openstack-sdks | 10:55 | |
| *** jangutter has quit IRC | 10:58 | |
| *** jangutter_ has quit IRC | 11:08 | |
| *** jangutter has joined #openstack-sdks | 11:12 | |
| *** jangutter has quit IRC | 11:14 | |
| *** gtema has quit IRC | 11:16 | |
| *** jangutter has joined #openstack-sdks | 11:38 | |
| *** jangutter has quit IRC | 11:40 | |
| *** jangutter has joined #openstack-sdks | 11:43 | |
| *** jangutter has quit IRC | 11:45 | |
| *** mgariepy has joined #openstack-sdks | 11:58 | |
| *** jangutter has joined #openstack-sdks | 11:59 | |
| *** zbr has quit IRC | 12:46 | |
| *** ricolin_ is now known as ricolin | 12:48 | |
| *** zbr has joined #openstack-sdks | 13:03 | |
| mordred | dtantsur: we've got support for form_post which uses the temp-url-key infrastructure | 13:04 |
| mordred | dtantsur: so yeah, we could probably lift in that code for generating temp_urls themselves pretty easily | 13:05 |
| openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for servers https://review.opendev.org/675113 | 13:05 |
| dtantsur | makes sense. it seems that the code in swiftclient doesn't depend on anything specific to swiftclient | 13:05 |
| openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | 13:08 |
| *** mriedem has joined #openstack-sdks | 13:10 | |
| *** gtema has joined #openstack-sdks | 13:17 | |
| edleafe | Who knew that our API work extended to snail mail? https://twitter.com/shawnacscott/status/1158813468704698368 | 13:22 |
| *** gtema has quit IRC | 13:22 | |
| dtantsur | oh lol | 13:24 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for global_request_id https://review.opendev.org/674807 | 13:39 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Initalize pool_executor so close works https://review.opendev.org/675120 | 13:39 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix discovery cache sharing https://review.opendev.org/675121 | 13:39 |
| mordred | dtantsur, efried: ^^ there we go - with discovery cache fixed and also with context manager fixed | 13:39 |
| *** factor has quit IRC | 13:44 | |
| *** factor has joined #openstack-sdks | 13:45 | |
| Shrews | mordred: do we have any existing tests in sdk that exercise the large object code? I'm not finding any | 13:46 |
| mordred | Shrews: yes - becuase they sucked to write | 13:46 |
| Shrews | :) | 13:46 |
| Shrews | i must be looking in the wrong places then | 13:46 |
| mordred | Shrews: test_create_dynamic_large_object and test_create_static_large_object in openstack/tests/unit/cloud/test_object.py | 13:48 |
| Shrews | ah ha | 13:48 |
| mordred | also test_object_segment_retry_failure | 13:48 |
| Shrews | i forgot we still have mixed tests | 13:48 |
| *** jpich has quit IRC | 13:51 | |
| *** jpich has joined #openstack-sdks | 13:52 | |
| mordred | yeah. one of these days it's all going to be perfect and shiny | 13:52 |
| *** whoami-rajat has quit IRC | 13:57 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix discovery cache sharing https://review.opendev.org/675121 | 14:03 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for global_request_id https://review.opendev.org/674807 | 14:03 |
| mordred | Shrews: if you have a sec, the first two in that stack are bugfixes I noticed while working on the third for efried | 14:04 |
| openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | 14:04 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for global_request_id https://review.opendev.org/674807 | 14:05 |
| mordred | efried: you are quick at re-reviewing things | 14:05 |
| mordred | dtantsur: if we do lift in the temp-url stuff, we should probably look to see what from the form_post method we can reuse. (we're using that form-post method in zuul log uploading to swift, so I'm pretty sure it's solid) | 14:06 |
| dtantsur | makes sense, yeah | 14:07 |
| *** slaweq has quit IRC | 14:07 | |
| *** slaweq has joined #openstack-sdks | 14:08 | |
| *** dtantsur is now known as dtantsur|bbl | 14:21 | |
| *** Luzi has quit IRC | 14:30 | |
| efried | mordred: I just happened to be looking at my email at the time :) | 14:41 |
| *** altlogbot_3 has quit IRC | 14:46 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rationalize endpoint_for and get_endpoint_from_catalog https://review.opendev.org/675130 | 14:47 |
| mordred | Shrews, dtantsur|bbl: ^^ that also adds some tests and docs for a method we added last week - and makes it make sense with the existing method that was already there | 14:47 |
| *** altlogbot_2 has joined #openstack-sdks | 14:47 | |
| efried | mordred: Attempting to use sdk auto passthrough proxy for cyborg gets Sundar this error: http://paste.openstack.org/show/755594/ | 14:52 |
| efried | is that expected? | 14:52 |
| mordred | efried: does cyborg not publish a version discovery document? | 14:55 |
| efried | I was certain they did | 14:56 |
| efried | mordred: esp because they have a v1 that's dead, and are working up v2 now. | 14:56 |
| *** factor has quit IRC | 14:57 | |
| mordred | hrm. do you have a link to code where that's happening? I'd like to see the value for accelerator_api_version - or more generally the clouds.yaml / config settings and also the discovery documents | 14:58 |
| *** factor has joined #openstack-sdks | 14:58 | |
| mordred | that error basically happens when a version has been requested and discovery can't find a major api version on the service | 14:59 |
| mordred | in any cas e- let's dig in and figure out what's going on there | 15:00 |
| mordred | efried: oh - I'm sorry - I misspoke - that's what happens when there is no configured api version | 15:01 |
| efried | mordred: Okay. Sundar is Pacific so might be a bit before he's on. The conf is here http://paste.openstack.org/show/755593/ | 15:01 |
| mordred | efried: oh - blerg. this is dumb | 15:01 |
| efried | mordred: Oh, so you mean he could put a version in the conf and/or constructor and be good to go? | 15:01 |
| mordred | efried: I have NO IDEA why I thought the logic for this case would be reasonable | 15:01 |
| efried | heh | 15:02 |
| *** factor has quit IRC | 15:03 | |
| mordred | efried: fix coming. also - I think we can make a more substantial improvement - which is to make a base class and a subclass of the ServiceDescription object - one that we use for services we know about, and one for thigns that we're just creating becaues they're in service-types-authority | 15:05 |
| mordred | because there's warnings we throw in the current case for "you're only going to get passthrough REST here" - but that's a warning if we have a supported Proxy version because that means something went wrong | 15:05 |
| mordred | if we DON'T have one of those, then you're expecting a REST passthrough, so the warning is stupid | 15:06 |
| mordred | but - there's just a dumb logic error for the cyborg case | 15:06 |
| efried | mordred: So then we wouldn't actually need the placement passthrough you made earlier this week? | 15:06 |
| efried | this feels like it's kinda in the same neighborhood | 15:07 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: WIP: Retry SLO manifest upload https://review.opendev.org/675134 | 15:08 |
| Shrews | mordred: Want to check that I'm attacking this in the right way before I get too far? ^^^ | 15:09 |
| mordred | efried: it is in the same neighborhood - and perhaps | 15:10 |
| mordred | Shrews: yeah - I thnik so | 15:10 |
| mordred | efried: yah - I can make one small patch to make this whole thing much easier I think | 15:13 |
| efried | nice | 15:13 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Skip most service_description for unknown services https://review.opendev.org/675135 | 15:18 |
| mordred | efried: ^^ that patch might fix the placement and the cyborg issue | 15:18 |
| efried | mordred: asking Sundar to try it. (He's in a mtg, may be an hour or so.) | 15:21 |
| mordred | efried: ok. let him know I'm very keen to make sure his stuff is working here | 15:23 |
| efried | will do, and thanks mordred | 15:24 |
| *** slaweq_ has joined #openstack-sdks | 15:26 | |
| *** slaweq has quit IRC | 15:29 | |
| *** factor has joined #openstack-sdks | 15:30 | |
| *** dtantsur|bbl is now known as dtantsur | 15:49 | |
| *** icarusfactor has joined #openstack-sdks | 15:50 | |
| *** factor has quit IRC | 15:52 | |
| *** icarusfactor has quit IRC | 15:55 | |
| *** whoami-rajat has joined #openstack-sdks | 15:55 | |
| *** ralonsoh has quit IRC | 15:57 | |
| *** ralonsoh has joined #openstack-sdks | 15:57 | |
| Shrews | mordred: ooh, delete_autocreated_image_objects *almost* does what we need for failed upload cleanup. Maybe add an optional parameter to limit it to object name prefix? | 16:07 |
| *** efried is now known as efried_rollin | 16:12 | |
| Shrews | oh, actually, don't think i can use that at the point i need it | 16:12 |
| * Shrews tries anyway | 16:14 | |
| *** cdent has quit IRC | 16:22 | |
| *** ricolin_ has joined #openstack-sdks | 16:23 | |
| *** jpich has quit IRC | 16:25 | |
| *** ricolin has quit IRC | 16:26 | |
| *** ricolin_ is now known as ricolin | 16:29 | |
| *** factor has joined #openstack-sdks | 16:35 | |
| *** ricolin has quit IRC | 16:40 | |
| *** dtantsur is now known as dtantsur|afk | 16:55 | |
| *** e0ne has quit IRC | 17:06 | |
| *** altlogbot_2 has quit IRC | 17:17 | |
| *** gtema has joined #openstack-sdks | 17:19 | |
| *** gtema has quit IRC | 17:23 | |
| *** altlogbot_3 has joined #openstack-sdks | 17:23 | |
| *** mgariepy has quit IRC | 17:29 | |
| *** mgariepy has joined #openstack-sdks | 17:30 | |
| *** altlogbot_3 has quit IRC | 17:30 | |
| *** altlogbot_2 has joined #openstack-sdks | 17:35 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Skip most service_description for unknown services https://review.opendev.org/675135 | 17:42 |
| openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add remove_service to token fixtures https://review.opendev.org/675174 | 18:34 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Validate that connect_as connects as the project https://review.opendev.org/675178 | 18:45 |
| *** e0ne has joined #openstack-sdks | 18:49 | |
| *** mgariepy has quit IRC | 19:13 | |
| *** mgariepy has joined #openstack-sdks | 19:13 | |
| *** mgariepy has quit IRC | 19:14 | |
| *** mgariepy has joined #openstack-sdks | 19:15 | |
| *** mgariepy has quit IRC | 19:16 | |
| *** mgariepy has joined #openstack-sdks | 19:18 | |
| *** mgariepy has quit IRC | 19:19 | |
| *** mgariepy has joined #openstack-sdks | 19:20 | |
| *** gtema has joined #openstack-sdks | 19:20 | |
| *** gtema has quit IRC | 19:25 | |
| *** mgariepy has quit IRC | 19:33 | |
| *** dave-mccowan has joined #openstack-sdks | 19:33 | |
| *** mgariepy has joined #openstack-sdks | 19:34 | |
| *** camelCas- has quit IRC | 19:38 | |
| *** e0ne has quit IRC | 19:46 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Make a reusable fixture for mocking clouds https://review.opendev.org/675187 | 19:50 |
| efried_rollin | oo | 19:59 |
| *** efried_rollin is now known as efried | 19:59 | |
| mordred | efried: that's VERY VERY early and is almost completely unuseful for you yet | 20:02 |
| efried | but the train has left the station | 20:02 |
| mordred | my plan is to do some refactoring inside of sdk so that we make the fixture and then we use that fixture in our tests | 20:02 |
| mordred | so that, you know - you can count on it | 20:02 |
| mordred | I thnik it's going to make sdk's tests better too | 20:02 |
| efried | spect so | 20:03 |
| mordred | because we have a bunch of wonky hacks going on right now | 20:03 |
| * mordred looks at some kmalloc todo comments ... | 20:03 | |
| * mordred looks at some mordred todo comments | 20:03 | |
| efried | mordred: Have we done the thing yet where we run some form of nova dsvm CI on sdk patches? | 20:03 |
| *** factor has quit IRC | 20:03 | |
| efried | you start mucking with endpoint methods, makes me nervous. | 20:03 |
| *** factor has joined #openstack-sdks | 20:03 | |
| mordred | don't be nervous - we never write bugs | 20:04 |
| mordred | efried: and no - but we should | 20:04 |
| *** factor has quit IRC | 20:05 | |
| mordred | efried: we have plenty of dsvm jobs going in sdk - but nothing running a tempest-based anything because up until recently there was no need to | 20:05 |
| *** factor has joined #openstack-sdks | 20:05 | |
| kmalloc | mordred: my TODO comments are.... long ago todo ;) | 20:08 |
| mordred | kmalloc: :) | 20:08 |
| mordred | kmalloc: well - check that WIP - I think you'll immediately get where I'm going with that and will hopefully like it | 20:09 |
| kmalloc | mordred: oh that is a good direction | 20:09 |
| kmalloc | didn't even take long to see it. | 20:09 |
| kmalloc | good stuff | 20:09 |
| mordred | \o/ | 20:09 |
| kmalloc | i'll review a bit more in depth knowing it's WIP | 20:12 |
| kmalloc | but i think you're headed the right way for sure. | 20:12 |
| *** whoami-rajat has quit IRC | 20:17 | |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: WIP: Retry SLO manifest upload https://review.opendev.org/675134 | 20:20 |
| mordred | Shrews: ^^ \o/ | 20:23 |
| Shrews | mordred: i'm completely boggled on the new test_slo_manifest_fail test and what requests to mock for the object cleanup phase | 20:24 |
| Shrews | that's where it's barfing for me, atm | 20:25 |
| Shrews | i know what calls i *think* it's going to make, but not the details | 20:25 |
| mordred | hrm. | 20:26 |
| mordred | well - it *should* fire a delete for each segment it sent an upload for, right? | 20:27 |
| mordred | (in some order or another) | 20:27 |
| mordred | oh - well - except - in this case you're going to return a list of segment objects as the return value from the list_objects call it's going to make | 20:28 |
| mordred | so the json you return from list_object in your requests_mock should feed which segments it fires deletes for, yeah? | 20:28 |
| mordred | (and you could probably make it a small list of segments so you don't go crazy) | 20:28 |
| Shrews | i think that's right, just trying to figure out how to make that happen. thus far, it doesn't like my list objects uri i'm creating there | 20:29 |
| Shrews | '{endpoint}/{container}?format=json&prefix={prefix}' <<-- that one | 20:31 |
| *** factor has quit IRC | 20:32 | |
| *** factor has joined #openstack-sdks | 20:32 | |
| *** factor has quit IRC | 20:35 | |
| *** factor has joined #openstack-sdks | 20:36 | |
| mordred | Shrews: oh - so ... theres a thing with url parameters where you have to pass a thing | 20:44 |
| mordred | Shrews: complete_qs=True | 20:44 |
| mordred | Shrews: if the query string is meaningful, complete_qs causes it to be considered | 20:44 |
| Shrews | mordred: i was wondering what that meant | 20:45 |
| Shrews | i found nothing meaningful :( thx | 20:46 |
| mordred | Shrews: are you saying we should document our test framework? | 20:46 |
| mordred | Shrews: if so - you're in luck - because I'm working on extracting all this requests-mock/url stuff into a reusable fixture that openstacksdk consumers can also use - which means we'll necessarily need to give it a proper api and document that API | 20:47 |
| mordred | if that's not what your'e saying, you're still in luck, for the same reason ;) | 20:47 |
| *** ralonsoh has quit IRC | 20:58 | |
| *** slaweq_ has quit IRC | 21:06 | |
| *** camelCaser has joined #openstack-sdks | 21:09 | |
| *** slaweq_ has joined #openstack-sdks | 21:11 | |
| *** slaweq_ has quit IRC | 21:17 | |
| *** factor has quit IRC | 21:35 | |
| *** factor has joined #openstack-sdks | 21:35 | |
| openstackgerrit | Morgan Fainberg proposed openstack/keystoneauth master: Add remove_service to token fixtures https://review.opendev.org/675174 | 21:43 |
| efried | mordred: Sundar has been pretty busy today, so only sparsely working with your patch, but we seem to have at least gotten past that initial crash. | 22:05 |
| *** slaweq_ has joined #openstack-sdks | 22:11 | |
| *** slaweq_ has quit IRC | 22:16 | |
| *** mriedem has quit IRC | 22:24 | |
| mordred | efried: sweet | 22:49 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | 22:51 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | 23:08 |
| *** slaweq_ has joined #openstack-sdks | 23:11 | |
| *** slaweq_ has quit IRC | 23:16 | |
| *** tosky has quit IRC | 23:20 | |
| *** gtema has joined #openstack-sdks | 23:22 | |
| *** gtema has quit IRC | 23:26 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!