| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-02-28 | |||
| 17:50:41 | kmalloc | i *think* the current implementation of fair semaphore is fine. | |
| 17:50:52 | kmalloc | can you drop the -2 on it plz | |
| 17:50:55 | mordred | kmalloc: but yeah - a receheck is a good idea at this point | |
| 17:50:57 | mordred | yeah | |
| 17:51:00 | kmalloc | thanks. | |
| 17:51:20 | kmalloc | we might be able to drop it in today/release it if it all... isn't insane | |
| 17:52:19 | mordred | kmalloc: \o/ | |
| 17:52:21 | kmalloc | ah bah the SDK patch needs rebase. | |
| 17:52:23 | mordred | that would be so exciting | |
| 17:52:26 | mordred | blerg | |
| 17:52:37 | openstackgerrit | Merged openstack/python-openstackclient master: Fix: Restore output 'VolumeBackupsRestore' object is not iterable https://review.openstack.org/624860 | |
| 18:07:52 | openstackgerrit | Morgan Fainberg proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | |
| 18:08:08 | kmalloc | mordred: ^ i think that is a cleaned up rebase... we shall see. | |
| 18:08:29 | mordred | kmalloc: fingers crossed! | |
| 18:09:28 | kmalloc | i expect it to explode pretty spectacularly | |
| 18:27:18 | openstackgerrit | Merged openstack/openstacksdk master: Move image methods to sdk image proxy https://review.openstack.org/609684 | |
| 18:33:09 | openstackgerrit | Merged openstack/openstacksdk master: Use retriable_status_codes in image upload https://review.openstack.org/639679 | |
| 18:40:37 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Paginate over usage list to return all usages https://review.openstack.org/639652 | |
| 18:41:56 | openstackgerrit | Merged openstack/openstacksdk master: Add glance image import support https://review.openstack.org/639122 | |
| 19:06:16 | openstackgerrit | Merged openstack/openstacksdk master: Tweak find_image method to search in hidden images https://review.openstack.org/639613 | |
| 19:20:34 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add image tasks schema methods https://review.openstack.org/639332 | |
| 19:20:35 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add image.service_info resources https://review.openstack.org/638227 | |
| 20:25:17 | kmalloc | mordred: btw, second confirmation (direct testing) that fair_semaphore works great at least in single-threaded mode for rate_limiting | |
| 20:26:19 | kmalloc | mordred: and yeah we can't easily test the SDK consumption until we release KSA with the sempaphore | |
| 21:04:28 | openstackgerrit | Merged openstack/openstacksdk master: Use mock context in test_fwaas https://review.openstack.org/639421 | |
| 22:14:35 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add support for client-side rate limiting https://review.openstack.org/605043 | |
| 22:41:19 | mordred | dtantsur|afk: 0.25.0 of openstacksdk released with your ironic patches in it | |
| #openstack-sdks - 2019-03-01 | |||
| 01:17:33 | openstackgerrit | Adrian Turjak proposed openstack/service-types-authority master: Add Adjutant as admin-logic https://review.openstack.org/633854 | |
| 01:58:36 | openstackgerrit | Merged openstack/keystoneauth master: Add support for client-side rate limiting https://review.openstack.org/605043 | |
| 03:43:09 | openstackgerrit | Doug Wiegley proposed openstack/python-openstackclient master: Add 'security_group' type support to network rbac commands https://review.openstack.org/636760 | |
| 04:12:57 | openstackgerrit | Merged openstack/service-types-authority master: Add Adjutant as admin-logic https://review.openstack.org/633854 | |
| 04:29:07 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Drop py35 jobs https://review.openstack.org/639915 | |
| 04:36:12 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/640255 | |
| 09:38:19 | dtantsur | mordred: very nice of you, thanks :) | |
| 10:33:29 | openstackgerrit | Colleen Murphy proposed openstack/keystoneauth master: Drop py35 jobs https://review.openstack.org/639915 | |
| 10:49:49 | frickler | version detection seems to be broken when using application_credentials as auth method, I need to specify the /v3 variant of the auth_url explicitly | |
| 10:50:12 | frickler | also, clouds.yaml seems to be lacking support for this variant | |
| 10:50:42 | frickler | cmurphy: ^^ do you happen to know whether this is planned or is there some reason no to support it? | |
| 10:53:21 | cmurphy | frickler: it's not intentional on my part, I noticed that behavior too and I think it has something to do with how we're using a v3 specific identity plugin | |
| 10:54:25 | cmurphy | frickler: er sorry, was your question to me about version detection or clouds.yaml? | |
| 10:55:54 | cmurphy | i think clouds.yaml should already support application credentials if you use auth_type: v3applicationcredential | |
| 10:55:58 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | |
| 11:09:46 | frickler | cmurphy: actually both questions ;) but with your help I solved the second one now, the "auth_type" has to go outside of the auth section, where it produces an error, so that one is o.k. now | |
| 11:10:03 | frickler | cmurphy: so I'll create a bug for the first one and try to dig into it a bit | |
| 11:10:55 | frickler | or maybe first wait for mordred to see this and have the answer ready ;) | |
| 11:21:45 | cmurphy | :) | |
| 11:38:59 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Move object methods to object_store proxy https://review.openstack.org/638731 | |
| 12:16:58 | openstackgerrit | Merged openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/640255 | |
| 12:22:10 | mordred | cdent: in a fit of reasonableness - that ^^ seems to have gotten itself resolved | |
| 12:22:48 | cdent | mordred: aye. as expected it just needed a bit of additional voice and on it sails | |
| 12:28:17 | mordred | mnaser: I know you have tons of free time and are usually sitting around bored looking for things to do ... | |
| 12:29:39 | cmurphy | lol | |
| 12:29:48 | mordred | mnaser: if you'd like to fill that free time with code review - https://review.openstack.org/#/c/623009 is up adding support for designate to sdk - and I think you know designate more better than I do - wanna take a look? | |
| 12:33:39 | mordred | dtantsur: you made a comment the other day about on-demand fetching of resource attributes rather than having empty resources just with id in them or whatever | |
| 12:33:58 | mordred | dtantsur: upon further reflection, I think you're totally right (obviously that'll want to be a train thing) | |
| 12:34:29 | mordred | dtantsur: I was just looking at https://review.openstack.org/#/c/623009/18/openstack/dns/v2/recordset.py from gtema - specifically the "zone_id" and "zone_name" fields | |
| 12:35:15 | mordred | it struck me that it woudl be way cooler if that was just a zone field that was a Zone with the id and name filled in- but then the thign you said about empty resources would be there | |
| 12:35:31 | mordred | we already have flags like _dirty on resource objects | |
| 12:36:59 | mordred | maybe we add a flag (or maybe it turns out we don't need one- but I think we will) - so that we could have something do the equiv of self.zone = zone.Zone(id=body['id'], placeholder=True) ... | |
| 12:37:45 | mordred | then if that's true, it would be saying "this represents an existing thing on the cloud, but we don't want to fetch it just now. when someone goes to read a property that's missing - run a fetch" | |
| 12:38:07 | mordred | "... and set placeholder=False at the end of the fetch" | |
| 12:38:26 | mordred | we obviously don't want to just run fetch anytime a property is empty - that would be horrible | |
| 12:39:00 | dtantsur | yeah, kind of lazy loading in sqlalchemy | |
| 12:39:01 | mordred | but it could give us a building block to do nicely nested objects | |
| 12:39:03 | mordred | yeah | |
| 12:39:23 | mordred | I think we might need to be careful, once we have nested objects, to think about how to_json works ... | |
| 12:39:33 | dtantsur | indeed | |
| 12:39:43 | dtantsur | a topic for the PTG? | |
| 12:39:48 | mordred | like - maybe to_json should NOT expand nested resources and instead just render the id or something | |
| 12:40:20 | mordred | yeah. I think that's actually a really good one - although I don't think we'll have gtema or Shrews there - so it might just be you and me getting drunk talking about nested resources | |
| 12:40:34 | dtantsur | well, it's a start, right? ;) | |
| 12:40:45 | dtantsur | then we can propose our drunk thoughts to the community | |
| 12:40:48 | mordred | wcpgw? | |
| 12:41:43 | mordred | \o/ | |
| 12:41:55 | dtantsur | as to to_json.. we may want to have different calls with more specific names? like to_api_json (close to what server returns), serialize (our JSON representation, ideally we should be able to deserialize), etc.. | |
| 12:43:32 | mordred | yah | |
| 12:45:19 | mordred | we do already have "original_names" as an arg for to_dict | |
| 12:57:31 | mordred | gtema: the pep8 gods are angry | |
| 13:27:17 | Shrews | mordred: i have booked the 1st part of summit week | |
| 13:32:16 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | |
| 13:32:33 | mordred | Shrews: oh yay! | |
| 13:35:28 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Move object methods to object_store proxy https://review.openstack.org/638731 | |
| 13:37:25 | gtema | gtema is here, but still in a call, so with just one ear and one finger | |
| 13:40:41 | mordred | gtema: you're going to love the patch I'm about to push uo | |
| 13:40:43 | mordred | up | |
| 13:41:11 | gtema | am I, or would I be not happy? | |
| 13:41:33 | mordred | gtema: I'm hoping the first --- but it's possible also you might eye-roll me :) | |
| 13:42:01 | gtema | is it about Swift stuff? | |
| 13:42:16 | mordred | nope. it's a followup to my comment on your dns patch about setUpClass | |
| 13:43:32 | gtema | ahh, I haven't read it so far - still talking. But if you have suggestion how to bypass this crap of not being able to properly delete zone for new test - you are welcome | |
| 14:22:12 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Get rid of setUpClass and block it for forever https://review.openstack.org/640379 | |
| 14:23:17 | mordred | Shrews, gtema, dtantsur: I decided review comments of "don't use setUpClass" were a thing that can totally be a local hacking check, rather than just a human review comment. possibly too much of a rabbithole for the morning, but there it is | |
| 14:23:49 | dtantsur | well, why not? :) | |
| 14:26:37 | dtantsur | mordred: :) a reference to crying pandas is convincing enough, but I wonder if we should add a bit more details to HACKING.rst | |
| 14:26:47 | mordred | dtantsur: fair | |
| 14:27:07 | mordred | dtantsur: I suppose some people want ACTUAL INFORMATION | |
| 14:27:34 | dtantsur | wait, WUT?? | |
| 14:37:12 | Shrews | how is making baby pandas cry NOT enough info???? | |
| 14:40:44 | dtantsur | I've heard about heartless people out there | |
| 14:46:46 | gtema | so, I'm now finally back. | |