| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-02-28 | |||
| 16:01:59 | mrhillsman | proposal: forum session on sdk validation discussion | |
| 16:02:32 | gtema | however if "consuming" page guide describes how to treat uncertancies or strict requirements - is also good | |
| 16:04:52 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | |
| 16:04:58 | mrhillsman | apologies for not being around, elmiko you volunteered to help lead sdk validation stuff right? | |
| 16:04:58 | gtema | mrhillsman: unfortunately not going to the summit | |
| 16:05:03 | elmiko | gtema: ack, i'm good with that | |
| 16:05:16 | mrhillsman | or was that gtema | |
| 16:05:23 | gtema | yupp | |
| 16:05:30 | elmiko | mrhillsman: i'm definitely willing to help as i can, i won't be at the forum though | |
| 16:05:36 | gtema | but was also terribly busy | |
| 16:05:38 | mrhillsman | ok cool, no worries, i will be there | |
| 16:06:39 | gtema | elmiko: shoul I the suggest a change to "consuming catalog" guide to adress the issue? | |
| 16:08:53 | gtema | hmm, current state doesn't disallow bad region-name (refering to current keystone) | |
| 16:09:10 | openstackgerrit | Dean Troyer proposed openstack/osc-lib master: Add release note for --os-user-id fix https://review.openstack.org/640082 | |
| 16:09:28 | gtema | but it is exactly a problem for me with current keystone, so we might "re-phrase" requirements here | |
| 16:38:14 | elmiko | doh, gtema disappeared | |
| 16:39:04 | elmiko | gtema: if you read back later, i think making a PR to the guide is definitely welcome and a good place to further this discussion | |
| 17:23:33 | openstackgerrit | Merged openstack/osc-lib master: Add release note for --os-user-id fix https://review.openstack.org/640082 | |
| 17:43:44 | dtroyer | mordred: I'm going to release osc-lib 1.12.1 at that point ^^^ | |
| 17:44:04 | dtroyer | there's only 1 bug fix for this cycle, this should be it | |
| 17:45:34 | mordred | dtroyer: sounds great! | |
| 17:49:30 | kmalloc | mordred: lets revisit landing the fair semaphore and just lean on SDK consuming it as the functional test. | |
| 17:49:42 | kmalloc | mordred: looking into why the SDK consuming patch is unhappy/failing now. | |
| 17:49:47 | kmalloc | (had to recheck) | |
| 17:50:01 | mordred | kmalloc: ++ I think that's likely a good idea | |
| 17:50:21 | mordred | kmalloc: it's possible that the consuming patch isn't properly actually installing the right versions of things | |
| 17:50:26 | kmalloc | yeah that is my guess. | |
| 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? | |