Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-02-28
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.
14:58:10 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009
15:01:57 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009
15:03:48 gtema mordred, dtantsur: what do you think about https://review.openstack.org/#/c/638731/ in general? There is still some issue with broken ansible tests, which I can't figure out yet.
15:13:35 gtema Shrews: ^^
15:44:51 Shrews gtema: will take a look shortly
15:45:38 gtema oki, thanks
16:29:23 openstackgerrit Morgan Fainberg proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926
16:40:18 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Get rid of setUpClass and block it for forever https://review.openstack.org/640379
16:44:16 openstackgerrit Doug Wiegley proposed openstack/python-openstackclient master: Add 'security_group' type support to network rbac commands https://review.openstack.org/636760
17:05:27 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009

Earlier   Later