Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-02-28
15:54:10 gtema elmiko: understand
15:56:01 elmiko that said, i am definitely open to the idea of including this information into the guidance, if we can find an appropriate location
15:56:52 elmiko gtema: like, maybe in here somewhere? http://specs.openstack.org/openstack/api-sig/guidelines/consuming-catalog.html
15:58:35 elmiko API SIG office hours have begun! (a little early too)
15:59:13 gtema elmiko: might be an option
15:59:48 elmiko gtema: totally open to suggestions too
15:59:53 gtema since it is under api-sig - for sure. On the other hand it is more related to configuring catalog and not consuming
16:00:11 elmiko right
16:00:23 elmiko that's why i'm a little stumped about where we should put it
16:00:42 elmiko should this be part of the keystone docs?
16:01:06 gtema well...
16:01:11 gtema might be
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 gtema mrhillsman: unfortunately not going to the summit
16:04:58 mrhillsman apologies for not being around, elmiko you volunteered to help lead sdk validation stuff right?
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"

Earlier   Later