Friday, 2019-03-01

*** slaweq has joined #openstack-sdks00:11
*** slaweq has quit IRC00:15
*** tosky has quit IRC00:18
*** whoami-rajat has joined #openstack-sdks01:04
*** slaweq has joined #openstack-sdks01:11
*** slaweq has quit IRC01:16
*** ttsiouts has quit IRC01:17
openstackgerritAdrian Turjak proposed openstack/service-types-authority master: Add Adjutant as admin-logic  https://review.openstack.org/63385401:17
openstackgerritMerged openstack/keystoneauth master: Add support for client-side rate limiting  https://review.openstack.org/60504301:58
*** markvoelker has quit IRC02:26
*** slaweq has joined #openstack-sdks03:11
*** slaweq has quit IRC03:15
*** dave-mccowan has joined #openstack-sdks03:18
*** markvoelker has joined #openstack-sdks03:27
openstackgerritDoug Wiegley proposed openstack/python-openstackclient master: Add 'security_group' type support to network rbac commands  https://review.openstack.org/63676003:43
*** markvoelker has quit IRC04:00
*** dave-mccowan has quit IRC04:04
*** slaweq has joined #openstack-sdks04:11
openstackgerritMerged openstack/service-types-authority master: Add Adjutant as admin-logic  https://review.openstack.org/63385404:12
*** slaweq has quit IRC04:16
openstackgerritVishakha Agarwal proposed openstack/keystoneauth master: Drop py35 jobs  https://review.openstack.org/63991504:29
openstackgerritOpenStack Proposal Bot proposed openstack/os-service-types master: Updated from OpenStack Service Type Authority  https://review.openstack.org/64025504:36
*** markvoelker has joined #openstack-sdks04:57
*** markvoelker has quit IRC05:31
*** slaweq has joined #openstack-sdks06:11
*** slaweq has quit IRC06:15
*** markvoelker has joined #openstack-sdks06:28
*** Luzi has joined #openstack-sdks06:43
*** markvoelker has quit IRC07:00
*** slaweq has joined #openstack-sdks07:11
*** slaweq has quit IRC07:15
*** e0ne has joined #openstack-sdks07:28
*** slaweq has joined #openstack-sdks07:32
*** gkadam has joined #openstack-sdks07:37
*** markvoelker has joined #openstack-sdks07:57
*** tssurya has joined #openstack-sdks08:25
*** markvoelker has quit IRC08:31
*** tbarron has quit IRC08:37
*** tosky has joined #openstack-sdks08:42
*** jpena|off is now known as jpena08:48
*** jpich has joined #openstack-sdks08:52
*** holser_ has joined #openstack-sdks08:55
*** ttsiouts has joined #openstack-sdks08:56
*** ralonsoh has joined #openstack-sdks09:02
*** markvoelker has joined #openstack-sdks09:28
*** dtantsur|afk is now known as dtantsur09:37
dtantsurmordred: very nice of you, thanks :)09:38
*** markvoelker has quit IRC10:01
*** cdent has joined #openstack-sdks10:15
*** e0ne has quit IRC10:22
*** e0ne has joined #openstack-sdks10:23
openstackgerritColleen Murphy proposed openstack/keystoneauth master: Drop py35 jobs  https://review.openstack.org/63991510:33
*** gtema has joined #openstack-sdks10:39
fricklerversion detection seems to be broken when using application_credentials as auth method, I need to specify the /v3 variant of the auth_url explicitly10:49
frickleralso, clouds.yaml seems to be lacking support for this variant10:50
fricklercmurphy: ^^ do you happen to know whether this is planned or is there some reason no to support it?10:50
cmurphyfrickler: 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 plugin10:53
cmurphyfrickler: er sorry, was your question to me about version detection or clouds.yaml?10:54
cmurphyi think clouds.yaml should already support application credentials if you use auth_type: v3applicationcredential10:55
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add DNS support  https://review.openstack.org/62300910:55
*** markvoelker has joined #openstack-sdks10:58
fricklercmurphy: 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. now11:09
fricklercmurphy: so I'll create a bug for the first one and try to dig into it a bit11:10
frickleror maybe first wait for mordred to see this and have the answer ready ;)11:10
*** e0ne has quit IRC11:11
*** cdent has quit IRC11:12
*** e0ne has joined #openstack-sdks11:14
*** ttsiouts has quit IRC11:18
*** cdent has joined #openstack-sdks11:20
*** tbarron has joined #openstack-sdks11:21
cmurphy:)11:21
*** markvoelker has quit IRC11:30
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Move object methods to object_store proxy  https://review.openstack.org/63873111:38
*** e0ne has quit IRC11:49
*** Luzi has quit IRC11:50
*** ttsiouts has joined #openstack-sdks11:51
*** gtema has quit IRC11:58
openstackgerritMerged openstack/os-service-types master: Updated from OpenStack Service Type Authority  https://review.openstack.org/64025512:16
*** gtema has joined #openstack-sdks12:18
mordredcdent: in a fit of reasonableness - that ^^ seems to have gotten itself resolved12:22
cdentmordred: aye. as expected it just needed a bit of additional voice and on it sails12:22
*** markvoelker has joined #openstack-sdks12:28
mordredmnaser: I know you have tons of free time and are usually sitting around bored looking for things to do ...12:28
cmurphylol12:29
mordredmnaser: 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
mordreddtantsur: 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 whatever12:33
mordreddtantsur: upon further reflection, I think you're totally right (obviously that'll want to be a train thing)12:33
mordreddtantsur: 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" fields12:34
mordredit 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 there12:35
mordredwe already have flags like _dirty on resource objects12:35
mordredmaybe 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
mordredthen 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
mordredwe obviously don't want to just run fetch anytime a property is empty - that would be horrible12:38
dtantsuryeah, kind of lazy loading in sqlalchemy12:39
mordredbut it could give us a building block to do nicely nested objects12:39
mordredyeah12:39
mordredI think we might need to be careful, once we have nested objects, to think about how to_json works ...12:39
dtantsurindeed12:39
dtantsura topic for the PTG?12:39
mordredlike - maybe to_json should NOT expand nested resources and instead just render the id or something12:39
mordredyeah. 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 resources12:40
dtantsurwell, it's a start, right? ;)12:40
dtantsurthen we can propose our drunk thoughts to the community12:40
mordredwcpgw?12:40
* dtantsur has learned a new acronym :D12:41
mordred\o/12:41
dtantsuras 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
mordredyah12:43
mordredwe do already have "original_names" as an arg for to_dict12:45
*** e0ne has joined #openstack-sdks12:50
*** jpena is now known as jpena|lunch12:51
mordredgtema: the pep8 gods are angry12:57
*** markvoelker has quit IRC13:01
*** holser_ has quit IRC13:19
*** holser_ has joined #openstack-sdks13:23
*** holser_ has quit IRC13:23
*** holser_ has joined #openstack-sdks13:24
Shrewsmordred: i have booked the 1st part of summit week13:27
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add DNS support  https://review.openstack.org/62300913:32
mordredShrews: oh yay!13:32
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Move object methods to object_store proxy  https://review.openstack.org/63873113:35
gtemagtema is here, but still in a call, so with just one ear and one finger13:37
mordredgtema: you're going to love the patch I'm about to push uo13:40
mordredup13:40
gtemaam I, or would I be not happy?13:41
mordredgtema: I'm hoping the first --- but it's possible also you might eye-roll me :)13:41
gtemais it about Swift stuff?13:42
mordrednope. it's a followup to my comment on your dns patch about setUpClass13:42
gtemaahh, 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 welcome13:43
*** jpena|lunch is now known as jpena13:54
*** markvoelker has joined #openstack-sdks13:58
*** nicholas has joined #openstack-sdks14:09
*** holser_ has quit IRC14:10
*** TheJulia is now known as needssleep14:14
*** cmurphy is now known as cmorpheus14:17
*** holser_ has joined #openstack-sdks14:20
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Get rid of setUpClass and block it for forever  https://review.openstack.org/64037914:22
mordredShrews, 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 is14:23
dtantsurwell, why not? :)14:23
*** jpich has quit IRC14:24
* mordred thanks the nova team for providing code to cargo-cult14:24
dtantsurmordred: :) a reference to crying pandas is convincing enough, but I wonder if we should add a bit more details to HACKING.rst14:26
mordreddtantsur: fair14:26
mordreddtantsur: I suppose some people want ACTUAL INFORMATION14:27
*** jpich has joined #openstack-sdks14:27
dtantsurwait, WUT??14:27
*** markvoelker has quit IRC14:31
Shrewshow is making baby pandas cry NOT enough info????14:37
dtantsurI've heard about heartless people out there14:40
gtemaso, I'm now finally back.14:46
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add DNS support  https://review.openstack.org/62300914:58
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add DNS support  https://review.openstack.org/62300915:01
gtemamordred, 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
gtemaShrews: ^^15:13
*** markvoelker has joined #openstack-sdks15:28
*** e0ne has quit IRC15:38
*** e0ne has joined #openstack-sdks15:39
Shrewsgtema: will take a look shortly15:44
gtemaoki, thanks15:45
*** tssurya has quit IRC15:59
*** markvoelker has quit IRC16:01
*** holser_ has quit IRC16:03
*** e0ne has quit IRC16:14
*** e0ne has joined #openstack-sdks16:15
*** e0ne has quit IRC16:17
*** kmalloc is now known as needscoffee16:22
*** gkadam has quit IRC16:27
openstackgerritMorgan Fainberg proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency  https://review.openstack.org/60492616:29
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Get rid of setUpClass and block it for forever  https://review.openstack.org/64037916:40
openstackgerritDoug Wiegley proposed openstack/python-openstackclient master: Add 'security_group' type support to network rbac commands  https://review.openstack.org/63676016:44
*** markvoelker has joined #openstack-sdks16:58
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add DNS support  https://review.openstack.org/62300917:05
*** ttsiouts has quit IRC17:06
*** gtema has quit IRC17:12
*** cdent has quit IRC17:28
*** markvoelker has quit IRC17:31
*** dtantsur is now known as dtantsur|afk18:00
*** jpena is now known as jpena|off18:03
*** jpich has quit IRC18:10
*** irclogbot_2 has joined #openstack-sdks18:11
*** ralonsoh has quit IRC18:22
* dtroyer read that as "with a keystone conspiracy" and thought "that already happened…"18:27
*** mrhillsman is now known as mrhillsman_lunch18:28
*** markvoelker has joined #openstack-sdks18:28
openstackgerritMerged openstack/openstacksdk master: Add image tasks schema methods  https://review.openstack.org/63933218:36
mordredneedscoffee: 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-sdks18:58
*** cdent has quit IRC18:58
*** markvoelker has quit IRC19:01
*** e0ne has joined #openstack-sdks19:12
*** purplerbot has quit IRC19:18
*** purplerbot has joined #openstack-sdks19:18
needscoffeemordred: ahh fair enough.19:22
openstackgerritMorgan Fainberg proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency  https://review.openstack.org/60492619:22
needscoffeemordred: but also YAY the rate limiting patch landed in KSA.19:23
*** mrhillsman_lunch is now known as mrhillsman19:29
*** e0ne has quit IRC19:31
*** tosky has quit IRC19:42
*** tosky has joined #openstack-sdks19:43
*** dmellado has quit IRC19:44
*** dmellado has joined #openstack-sdks19:45
*** irclogbot_2 has quit IRC19:49
*** markvoelker has joined #openstack-sdks19:58
*** irclogbot_2 has joined #openstack-sdks20:03
*** e0ne has joined #openstack-sdks20:05
mordredneedscoffee: IKR???20:18
*** markvoelker has quit IRC20:31
*** markvoelker has joined #openstack-sdks21:28
*** irclogbot_2 has quit IRC21:38
*** tobiash has quit IRC21:49
*** tobiash has joined #openstack-sdks21:51
*** markvoelker has quit IRC22:01
openstackgerritMerged openstack/keystoneauth master: Drop py35 jobs  https://review.openstack.org/63991522:16
*** e0ne has quit IRC22:24
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency  https://review.openstack.org/60492622:38
mordredneedscoffee: ^^ 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
mordredthose are just causing exceptions, because there is no object to call stop on22:39
mordredso I removed them22:39
mordredfingers crossed for this time22:40
*** guilhermesp__ has joined #openstack-sdks22:45
*** odyssey4me_ has joined #openstack-sdks22:45
*** odyssey4me has quit IRC22:52
*** guilhermesp has quit IRC22:52
*** melwitt has quit IRC22:52
*** guilhermesp__ is now known as 18WAAFJGJ22:52
*** odyssey4me_ is now known as odyssey4me22:52
*** melwitt has joined #openstack-sdks22:53
*** markvoelker has joined #openstack-sdks22:59
*** frickler has quit IRC23:06
*** frickler has joined #openstack-sdks23:06
needscoffeemordred: ah yeah23:14
needscoffeemordred: that makes sense23:14
needscoffeemordred: thankfully that becomes a non-issue (no more of it) soon.23:16
mordred++23:22
* needscoffee keeping eyes on that SDK patch23:25
needscoffeei 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 IRC23:31

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!