*** slaweq has joined #openstack-sdks | 00:11 | |
*** slaweq has quit IRC | 00:15 | |
*** tosky has quit IRC | 00:18 | |
*** whoami-rajat has joined #openstack-sdks | 01:04 | |
*** slaweq has joined #openstack-sdks | 01:11 | |
*** slaweq has quit IRC | 01:16 | |
*** ttsiouts has quit IRC | 01:17 | |
openstackgerrit | Adrian Turjak proposed openstack/service-types-authority master: Add Adjutant as admin-logic https://review.openstack.org/633854 | 01:17 |
---|---|---|
openstackgerrit | Merged openstack/keystoneauth master: Add support for client-side rate limiting https://review.openstack.org/605043 | 01:58 |
*** markvoelker has quit IRC | 02:26 | |
*** slaweq has joined #openstack-sdks | 03:11 | |
*** slaweq has quit IRC | 03:15 | |
*** dave-mccowan has joined #openstack-sdks | 03:18 | |
*** markvoelker has joined #openstack-sdks | 03:27 | |
openstackgerrit | Doug Wiegley proposed openstack/python-openstackclient master: Add 'security_group' type support to network rbac commands https://review.openstack.org/636760 | 03:43 |
*** markvoelker has quit IRC | 04:00 | |
*** dave-mccowan has quit IRC | 04:04 | |
*** slaweq has joined #openstack-sdks | 04:11 | |
openstackgerrit | Merged openstack/service-types-authority master: Add Adjutant as admin-logic https://review.openstack.org/633854 | 04:12 |
*** slaweq has quit IRC | 04:16 | |
openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Drop py35 jobs https://review.openstack.org/639915 | 04:29 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/640255 | 04:36 |
*** markvoelker has joined #openstack-sdks | 04:57 | |
*** markvoelker has quit IRC | 05:31 | |
*** slaweq has joined #openstack-sdks | 06:11 | |
*** slaweq has quit IRC | 06:15 | |
*** markvoelker has joined #openstack-sdks | 06:28 | |
*** Luzi has joined #openstack-sdks | 06:43 | |
*** markvoelker has quit IRC | 07:00 | |
*** slaweq has joined #openstack-sdks | 07:11 | |
*** slaweq has quit IRC | 07:15 | |
*** e0ne has joined #openstack-sdks | 07:28 | |
*** slaweq has joined #openstack-sdks | 07:32 | |
*** gkadam has joined #openstack-sdks | 07:37 | |
*** markvoelker has joined #openstack-sdks | 07:57 | |
*** tssurya has joined #openstack-sdks | 08:25 | |
*** markvoelker has quit IRC | 08:31 | |
*** tbarron has quit IRC | 08:37 | |
*** tosky has joined #openstack-sdks | 08:42 | |
*** jpena|off is now known as jpena | 08:48 | |
*** jpich has joined #openstack-sdks | 08:52 | |
*** holser_ has joined #openstack-sdks | 08:55 | |
*** ttsiouts has joined #openstack-sdks | 08:56 | |
*** ralonsoh has joined #openstack-sdks | 09:02 | |
*** markvoelker has joined #openstack-sdks | 09:28 | |
*** dtantsur|afk is now known as dtantsur | 09:37 | |
dtantsur | mordred: very nice of you, thanks :) | 09:38 |
*** markvoelker has quit IRC | 10:01 | |
*** cdent has joined #openstack-sdks | 10:15 | |
*** e0ne has quit IRC | 10:22 | |
*** e0ne has joined #openstack-sdks | 10:23 | |
openstackgerrit | Colleen Murphy proposed openstack/keystoneauth master: Drop py35 jobs https://review.openstack.org/639915 | 10:33 |
*** gtema has joined #openstack-sdks | 10:39 | |
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:49 |
frickler | also, clouds.yaml seems to be lacking support for this variant | 10:50 |
frickler | cmurphy: ^^ do you happen to know whether this is planned or is there some reason no to support it? | 10:50 |
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:53 |
cmurphy | frickler: er sorry, was your question to me about version detection or clouds.yaml? | 10:54 |
cmurphy | i think clouds.yaml should already support application credentials if you use auth_type: v3applicationcredential | 10:55 |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | 10:55 |
*** markvoelker has joined #openstack-sdks | 10:58 | |
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:09 |
frickler | cmurphy: so I'll create a bug for the first one and try to dig into it a bit | 11:10 |
frickler | or maybe first wait for mordred to see this and have the answer ready ;) | 11:10 |
*** e0ne has quit IRC | 11:11 | |
*** cdent has quit IRC | 11:12 | |
*** e0ne has joined #openstack-sdks | 11:14 | |
*** ttsiouts has quit IRC | 11:18 | |
*** cdent has joined #openstack-sdks | 11:20 | |
*** tbarron has joined #openstack-sdks | 11:21 | |
cmurphy | :) | 11:21 |
*** markvoelker has quit IRC | 11:30 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Move object methods to object_store proxy https://review.openstack.org/638731 | 11:38 |
*** e0ne has quit IRC | 11:49 | |
*** Luzi has quit IRC | 11:50 | |
*** ttsiouts has joined #openstack-sdks | 11:51 | |
*** gtema has quit IRC | 11:58 | |
openstackgerrit | Merged openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/640255 | 12:16 |
*** gtema has joined #openstack-sdks | 12:18 | |
mordred | cdent: in a fit of reasonableness - that ^^ seems to have gotten itself resolved | 12:22 |
cdent | mordred: aye. as expected it just needed a bit of additional voice and on it sails | 12:22 |
*** markvoelker has joined #openstack-sdks | 12:28 | |
mordred | mnaser: I know you have tons of free time and are usually sitting around bored looking for things to do ... | 12:28 |
cmurphy | lol | 12:29 |
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:29 |
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 |
mordred | dtantsur: upon further reflection, I think you're totally right (obviously that'll want to be a train thing) | 12:33 |
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:34 |
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 |
mordred | we already have flags like _dirty on resource objects | 12:35 |
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:36 |
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:37 |
mordred | "... and set placeholder=False at the end of the fetch" | 12:38 |
mordred | we obviously don't want to just run fetch anytime a property is empty - that would be horrible | 12:38 |
dtantsur | yeah, kind of lazy loading in sqlalchemy | 12:39 |
mordred | but it could give us a building block to do nicely nested objects | 12:39 |
mordred | yeah | 12:39 |
mordred | I think we might need to be careful, once we have nested objects, to think about how to_json works ... | 12:39 |
dtantsur | indeed | 12:39 |
dtantsur | a topic for the PTG? | 12:39 |
mordred | like - maybe to_json should NOT expand nested resources and instead just render the id or something | 12:39 |
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 |
dtantsur | well, it's a start, right? ;) | 12:40 |
dtantsur | then we can propose our drunk thoughts to the community | 12:40 |
mordred | wcpgw? | 12:40 |
* dtantsur has learned a new acronym :D | 12:41 | |
mordred | \o/ | 12:41 |
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:41 |
mordred | yah | 12:43 |
mordred | we do already have "original_names" as an arg for to_dict | 12:45 |
*** e0ne has joined #openstack-sdks | 12:50 | |
*** jpena is now known as jpena|lunch | 12:51 | |
mordred | gtema: the pep8 gods are angry | 12:57 |
*** markvoelker has quit IRC | 13:01 | |
*** holser_ has quit IRC | 13:19 | |
*** holser_ has joined #openstack-sdks | 13:23 | |
*** holser_ has quit IRC | 13:23 | |
*** holser_ has joined #openstack-sdks | 13:24 | |
Shrews | mordred: i have booked the 1st part of summit week | 13:27 |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | 13:32 |
mordred | Shrews: oh yay! | 13:32 |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Move object methods to object_store proxy https://review.openstack.org/638731 | 13:35 |
gtema | gtema is here, but still in a call, so with just one ear and one finger | 13:37 |
mordred | gtema: you're going to love the patch I'm about to push uo | 13:40 |
mordred | up | 13:40 |
gtema | am I, or would I be not happy? | 13:41 |
mordred | gtema: I'm hoping the first --- but it's possible also you might eye-roll me :) | 13:41 |
gtema | is it about Swift stuff? | 13:42 |
mordred | nope. it's a followup to my comment on your dns patch about setUpClass | 13:42 |
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 | 13:43 |
*** jpena|lunch is now known as jpena | 13:54 | |
*** markvoelker has joined #openstack-sdks | 13:58 | |
*** nicholas has joined #openstack-sdks | 14:09 | |
*** holser_ has quit IRC | 14:10 | |
*** TheJulia is now known as needssleep | 14:14 | |
*** cmurphy is now known as cmorpheus | 14:17 | |
*** holser_ has joined #openstack-sdks | 14:20 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Get rid of setUpClass and block it for forever https://review.openstack.org/640379 | 14:22 |
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 |
dtantsur | well, why not? :) | 14:23 |
*** jpich has quit IRC | 14:24 | |
* mordred thanks the nova team for providing code to cargo-cult | 14:24 | |
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 |
mordred | dtantsur: fair | 14:26 |
mordred | dtantsur: I suppose some people want ACTUAL INFORMATION | 14:27 |
*** jpich has joined #openstack-sdks | 14:27 | |
dtantsur | wait, WUT?? | 14:27 |
*** markvoelker has quit IRC | 14:31 | |
Shrews | how is making baby pandas cry NOT enough info???? | 14:37 |
dtantsur | I've heard about heartless people out there | 14:40 |
gtema | so, I'm now finally back. | 14:46 |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | 14:58 |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | 15:01 |
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:03 |
gtema | Shrews: ^^ | 15:13 |
*** markvoelker has joined #openstack-sdks | 15:28 | |
*** e0ne has quit IRC | 15:38 | |
*** e0ne has joined #openstack-sdks | 15:39 | |
Shrews | gtema: will take a look shortly | 15:44 |
gtema | oki, thanks | 15:45 |
*** tssurya has quit IRC | 15:59 | |
*** markvoelker has quit IRC | 16:01 | |
*** holser_ has quit IRC | 16:03 | |
*** e0ne has quit IRC | 16:14 | |
*** e0ne has joined #openstack-sdks | 16:15 | |
*** e0ne has quit IRC | 16:17 | |
*** kmalloc is now known as needscoffee | 16:22 | |
*** gkadam has quit IRC | 16:27 | |
openstackgerrit | Morgan Fainberg proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | 16:29 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Get rid of setUpClass and block it for forever https://review.openstack.org/640379 | 16:40 |
openstackgerrit | Doug Wiegley proposed openstack/python-openstackclient master: Add 'security_group' type support to network rbac commands https://review.openstack.org/636760 | 16:44 |
*** markvoelker has joined #openstack-sdks | 16:58 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add DNS support https://review.openstack.org/623009 | 17:05 |
*** ttsiouts has quit IRC | 17:06 | |
*** gtema has quit IRC | 17:12 | |
*** cdent has quit IRC | 17:28 | |
*** markvoelker has quit IRC | 17:31 | |
*** dtantsur is now known as dtantsur|afk | 18:00 | |
*** jpena is now known as jpena|off | 18:03 | |
*** jpich has quit IRC | 18:10 | |
*** irclogbot_2 has joined #openstack-sdks | 18:11 | |
*** ralonsoh has quit IRC | 18:22 | |
* dtroyer read that as "with a keystone conspiracy" and thought "that already happened…" | 18:27 | |
*** mrhillsman is now known as mrhillsman_lunch | 18:28 | |
*** markvoelker has joined #openstack-sdks | 18:28 | |
openstackgerrit | Merged openstack/openstacksdk master: Add image tasks schema methods https://review.openstack.org/639332 | 18:36 |
mordred | needscoffee: probably want to update the depends-on in the task manager patch to https://review.openstack.org/#/c/640389/ (the ksa constraints patch) | 18:46 |
*** cdent has joined #openstack-sdks | 18:58 | |
*** cdent has quit IRC | 18:58 | |
*** markvoelker has quit IRC | 19:01 | |
*** e0ne has joined #openstack-sdks | 19:12 | |
*** purplerbot has quit IRC | 19:18 | |
*** purplerbot has joined #openstack-sdks | 19:18 | |
needscoffee | mordred: ahh fair enough. | 19:22 |
openstackgerrit | Morgan Fainberg proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | 19:22 |
needscoffee | mordred: but also YAY the rate limiting patch landed in KSA. | 19:23 |
*** mrhillsman_lunch is now known as mrhillsman | 19:29 | |
*** e0ne has quit IRC | 19:31 | |
*** tosky has quit IRC | 19:42 | |
*** tosky has joined #openstack-sdks | 19:43 | |
*** dmellado has quit IRC | 19:44 | |
*** dmellado has joined #openstack-sdks | 19:45 | |
*** irclogbot_2 has quit IRC | 19:49 | |
*** markvoelker has joined #openstack-sdks | 19:58 | |
*** irclogbot_2 has joined #openstack-sdks | 20:03 | |
*** e0ne has joined #openstack-sdks | 20:05 | |
mordred | needscoffee: IKR??? | 20:18 |
*** markvoelker has quit IRC | 20:31 | |
*** markvoelker has joined #openstack-sdks | 21:28 | |
*** irclogbot_2 has quit IRC | 21:38 | |
*** tobiash has quit IRC | 21:49 | |
*** tobiash has joined #openstack-sdks | 21:51 | |
*** markvoelker has quit IRC | 22:01 | |
openstackgerrit | Merged openstack/keystoneauth master: Drop py35 jobs https://review.openstack.org/639915 | 22:16 |
*** e0ne has quit IRC | 22:24 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | 22:38 |
mordred | needscoffee: ^^ updated ... we'd added in a couple of calls to task_manager.stop() since we wrote the patch that weren't covered in the rebase (cause new) | 22:39 |
mordred | those are just causing exceptions, because there is no object to call stop on | 22:39 |
mordred | so I removed them | 22:39 |
mordred | fingers crossed for this time | 22:40 |
*** guilhermesp__ has joined #openstack-sdks | 22:45 | |
*** odyssey4me_ has joined #openstack-sdks | 22:45 | |
*** odyssey4me has quit IRC | 22:52 | |
*** guilhermesp has quit IRC | 22:52 | |
*** melwitt has quit IRC | 22:52 | |
*** guilhermesp__ is now known as 18WAAFJGJ | 22:52 | |
*** odyssey4me_ is now known as odyssey4me | 22:52 | |
*** melwitt has joined #openstack-sdks | 22:53 | |
*** markvoelker has joined #openstack-sdks | 22:59 | |
*** frickler has quit IRC | 23:06 | |
*** frickler has joined #openstack-sdks | 23:06 | |
needscoffee | mordred: ah yeah | 23:14 |
needscoffee | mordred: that makes sense | 23:14 |
needscoffee | mordred: thankfully that becomes a non-issue (no more of it) soon. | 23:16 |
mordred | ++ | 23:22 |
* needscoffee keeping eyes on that SDK patch | 23:25 | |
needscoffee | i really hope it passes so we can get that in and stop playing the "this might need more love" game. | 23:25 |
*** markvoelker has quit IRC | 23:31 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!