Thursday, 2019-04-18

*** TheJulia_ has joined #openstack-sdks00:54
*** kmalloc_ has joined #openstack-sdks00:54
*** fyx_ has joined #openstack-sdks00:54
*** kmalloc has quit IRC01:02
*** TheJulia has quit IRC01:02
*** Dinesh_Bhor has quit IRC01:02
*** fyx has quit IRC01:02
*** melwitt has quit IRC01:02
*** kmalloc_ is now known as kmalloc01:02
*** TheJulia_ is now known as TheJulia01:02
*** fyx_ is now known as fyx01:02
*** irclogbot_1 has quit IRC01:05
*** ricolin has joined #openstack-sdks01:05
*** irclogbot_2 has joined #openstack-sdks01:06
*** Dinesh_Bhor has joined #openstack-sdks01:08
*** whoami-rajat has joined #openstack-sdks01:24
*** melwitt has joined #openstack-sdks01:25
*** bobh has joined #openstack-sdks01:50
*** bobh has quit IRC01:51
*** holser_ has joined #openstack-sdks03:50
*** holser_ has quit IRC03:54
*** holser_ has joined #openstack-sdks03:57
*** holser_ has quit IRC04:09
*** slaweq has joined #openstack-sdks04:45
*** slaweq has quit IRC04:50
*** gkadam has joined #openstack-sdks05:01
*** gkadam_ has joined #openstack-sdks05:07
*** gkadam has quit IRC05:09
*** slaweq has joined #openstack-sdks05:56
*** e0ne has joined #openstack-sdks06:17
*** e0ne has quit IRC06:18
*** Luzi has joined #openstack-sdks06:23
*** gtema has joined #openstack-sdks06:56
*** gkadam_ has quit IRC07:01
*** ianw is now known as ianw_pto07:15
*** slaweq has quit IRC07:16
*** slaweq has joined #openstack-sdks07:28
*** tosky has joined #openstack-sdks07:42
*** jpich has joined #openstack-sdks07:56
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image  https://review.openstack.org/65153408:08
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Update baremetal to use proxy logger  https://review.openstack.org/65208308:11
*** ralonsoh has joined #openstack-sdks08:16
*** dtantsur|afk is now known as dtantsur08:25
*** e0ne has joined #openstack-sdks08:33
*** e0ne has quit IRC08:34
*** ttsiouts has joined #openstack-sdks08:45
*** e0ne has joined #openstack-sdks08:45
*** e0ne has quit IRC08:47
*** holser_ has joined #openstack-sdks08:57
*** e0ne has joined #openstack-sdks08:59
*** markvoelker has joined #openstack-sdks09:12
*** gtema has quit IRC09:17
*** markvoelker has quit IRC09:41
openstackgerritRodolfo Alonso Hernandez proposed openstack/openstacksdk master: Add "name" filter in "list" call when retrieving a single register  https://review.openstack.org/63723809:41
*** ttsiouts has quit IRC09:51
*** ttsiouts has joined #openstack-sdks09:53
*** e0ne has quit IRC09:54
*** e0ne has joined #openstack-sdks10:00
*** ttsiouts has quit IRC10:09
*** ttsiouts has joined #openstack-sdks10:10
*** gtema has joined #openstack-sdks10:15
*** Luzi has quit IRC10:28
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image  https://review.openstack.org/65153410:30
*** markvoelker has joined #openstack-sdks10:38
*** zigo has joined #openstack-sdks10:51
*** cdent has joined #openstack-sdks11:09
*** markvoelker has quit IRC11:11
*** slaweq is now known as slaweq|lunch11:15
*** slaweq|lunch is now known as slaweq11:33
*** ttsiouts has quit IRC11:59
*** bobh has joined #openstack-sdks11:59
*** markvoelker has joined #openstack-sdks12:09
*** ttsiouts has joined #openstack-sdks12:09
*** ttsiouts has quit IRC12:12
*** ttsiouts_ has joined #openstack-sdks12:12
*** cdent has quit IRC12:15
*** bobh has quit IRC12:23
*** markvoelker has quit IRC12:41
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Remove now unused task_manager file  https://review.openstack.org/65373113:03
*** ttsiouts_ has quit IRC13:06
*** ttsiouts has joined #openstack-sdks13:13
*** markvoelker has joined #openstack-sdks13:38
*** markvoelker has quit IRC13:42
gtemamordred: do you have an opinion on https://review.openstack.org/#/c/651534/12/openstack/resource.py@425?13:43
gtemado we want to support this logic everywhere?13:43
gtemadtantsur: https://review.openstack.org/#/c/651534/12/openstack/image/v1/image.py@103 - an existing logic to give possibility to the requestor to self iterate over the result13:46
mordredgtema: yes - I replied13:47
gtemathanks13:47
gtemaI can also move "output" logic under the Image.download method if we want it now13:48
mordredyeah - I think that's a good idea13:48
gtemaok. Will also add few tests on the proxy.download_image (were missing so far)13:49
dtantsurgtema: then why not change the second place to just 'return resp'?13:50
gtemahmm13:50
gtemamaybe13:50
gtemawhat do we want for chunk_size?13:54
openstackgerritDmitry Tantsur proposed openstack/openstacksdk master: Support for the baremetal introspection service  https://review.openstack.org/65004413:54
dtantsurmordred: a new service ^^^ \o/13:54
dtantsurgtema: I'd put at least 4k, but I don't have any reliable reasoning behind it13:55
dtantsurideally, we should benchmark it :)13:55
gtema:D Benchmark will actually depend on the chosen setup/platform/image/etc13:56
gtemadtantsur: returning resp instead of content breaks current behavior. This is not something I wanted to get with this change14:01
*** cdent has joined #openstack-sdks14:10
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image  https://review.openstack.org/65153414:17
dtantsurgtema: ah, I see14:21
*** gtema has quit IRC14:22
*** markvoelker has joined #openstack-sdks14:26
*** samueldmq has joined #openstack-sdks14:29
*** ricolin has quit IRC14:32
*** yolanda_ has quit IRC14:40
*** ttsiouts has quit IRC14:44
*** ttsiouts has joined #openstack-sdks14:44
openstackgerritMerged openstack/openstacksdk master: Move Orchestration methods into Proxy  https://review.openstack.org/64927314:49
openstackgerritMerged openstack/openstacksdk master: Allow replacing service implementation  https://review.openstack.org/64870415:02
*** e0ne has quit IRC15:08
*** e0ne has joined #openstack-sdks15:09
*** e0ne has quit IRC15:16
*** cdent has quit IRC15:18
*** ttsiouts has quit IRC15:27
*** ttsiouts has joined #openstack-sdks15:29
*** bobh has joined #openstack-sdks15:41
elmikoAPI SIG office hour is now open!16:00
*** ttsiouts has quit IRC16:00
edleafeYay!!!!16:00
elmiko\o/16:00
* edleafe pops champagne16:00
elmikoooh la la16:00
edleafeI did want to go over some outstanding api-sig patches, if that's ok with you16:01
elmikosure, i'm game!16:02
edleafeI think the DNS guideline is ready for freeze https://review.openstack.org/#/c/651222/16:02
elmiko++16:02
edleafeSo is the version discovery doc, but it has a merge conflict https://review.openstack.org/#/c/459405/16:02
elmikoi guess we should just send custom freeze emails to the ml since we don't have the newsletter?16:03
elmikooh wow, so exciting to see the version discovery finally getting there =)16:03
edleafeGenerally that's what the tools do: send an email to the liaisons, and add them to the reviewer list for the patch16:03
elmikook, sweet16:04
edleafeBut sure, we could also send out an email16:04
elmikoi suppose we don't have to, i was just curious and apparently the cob-webs have gotten the better of that part of my memory XD16:04
edleafedtantsur had a suggestion on the liaisons file patch https://review.openstack.org/#/c/645223/16:04
edleafeOnce I fix that, I think that that should be a quick approve16:04
edleafeI.e., it's not changing a guideline or anything16:05
elmikook, cool. i will remove my +2 with a note just for clarity sake16:05
elmikoi agree with dtantsur's idea16:05
edleafeschweet16:05
*** jpich has quit IRC16:06
edleafeI thought that the tags update was ready, but after thinking a bit more about Akihiro's comment, I want to revisit it https://review.openstack.org/#/c/642497/16:06
edleafeAnd I was going to fast-approve the emai list name change https://review.openstack.org/#/c/632255/116:07
edleafebut it was stated that it was a dupe without referencing what it is a dupe of16:07
edleafeDo you have any idea?16:07
* dtantsur w00t at freezing the DNS guideline16:07
elmikoedleafe: i agree about Akihiro's comment, thanks for taking that on16:07
dtantsuredleafe: I don't see a dup, let's approve it..16:08
edleafeI just posted a reply to the dupe patch a few minutes ago. There's no rush, so if there is no good answer by next week, let's approve it then16:08
elmikoedleafe: agreed16:09
edleafeThat's all I had on my plate to discuss16:10
elmikoedleafe ++16:11
*** cdent has joined #openstack-sdks16:14
*** bobh has quit IRC16:16
edleafehttps://review.openstack.org/#/c/651222 has been frozen16:17
elmikothanks edleafe16:17
dtantsurw00t!16:19
openstackgerritEd Leafe proposed openstack/api-sig master: Added Placement to the liaisons file  https://review.openstack.org/64522316:19
edleafedtantsur: elmiko: ^^ Fixed the ordering16:19
* dtantsur expects an avalanche of "Are you crazy?" comments16:19
edleafeEither of you want to fast-approve that?16:19
* dtantsur plans on proudly answering YES16:19
elmikoedleafe: looking now16:19
edleafeNobody will ask such a dumb question. We already know the answer is yes16:19
efriedDoes the API SIG deal with things like adoption of OpenStack SDK?16:20
dtantsurefried: it borders with our responsibilities, at least16:20
elmiko++16:20
efriednot to interrupt; feel free to come back to me when you reach an appropriate spot in the agenda16:20
edleafeefried: not specifically, but we do say things like "yes, that's a good thing to do"16:21
elmikono real agenda, this is just open office hours16:21
elmikodtantsur edleafe , +wf on the liaison file16:21
edleafeelmiko: kewl16:21
dtantsurefried: I think it's a good topic for right now, but what exactly do you have in mind?16:21
efriedOkay, so this is mostly FYI: I'm kind of involved in / leading a charge to make nova use the sdk.16:21
dtantsurif you need some to do some work, we have mordred to appoint16:22
elmikonice16:22
elmikoLOL16:22
efriedhttps://blueprints.launchpad.net/nova/+spec/openstacksdk-in-nova16:22
dtantsurbecause we don't do work here, it's not the goal of API SIG16:22
efriedmordred has already done some work.16:22
dtantsuroh?16:22
elmikodtantsur ++16:22
edleafeWe pride ourselves on convincing others to do the work16:22
elmikoXD16:22
efriedthere's this: https://review.openstack.org/643601 which needs to be cleaned up and have UT added16:23
efriedbut has been integration-tested and works.16:23
dtantsurefried: I'd really like to get to switching ironic-inspector to use SDK for talking to ironic and swift16:23
efriedtl;dr: lets you set up Connection with oslo.config ksa opts instead of clouds.yaml16:23
dtantsurI mean, get to it in Train16:23
dtantsurefried: I'm very much in favor of that. very.16:23
efried...so we can make the transition transparent to operators initially.16:23
efriedand then there's this: https://review.openstack.org/#/c/643664/ which introduces the plumbing for sdk in nova16:24
efriedand the patch on top of it https://review.openstack.org/#/c/642899/ which starts the process of swapping out calls into ironicclient.16:24
efriedidea being to stack several of those ^ up until we can rip out ironicclient entirely.16:24
efried(from nova)16:25
efriedI've got a new guy helping out with the nova things16:25
efriedbut could use some help on the sdk side16:25
efriedthere's the patch noted above, already started, could use closure16:25
mordredyeah. I'll definitely be digging in to help16:26
mordredand I need to finish that patch as soon as I can finish up all the meetings I've been stuck in this week16:26
efriedand then there's a need for a fixture to facilitate testing on the nova side16:26
efriedmordred: Totally understood, I was actually fishing for other volunteers to help :)16:26
dtantsurefried: note that future openstacksdk 0.26 will add patch_node for ironic, which will substantially simplify transition (no need to get rid of patcher right away)16:26
efriedhere's a story for the fixtures: https://storyboard.openstack.org/#!/story/200547516:26
efrieddtantsur: ack, good to know.16:27
efriedSo16:27
efriedbesides soliciting help if possible for the sdk items noted above16:27
efriedit would be nice if the API SIG (or whoever) could lend support to this idea in general.16:28
efriedbecause nova cores are old and crusty and afraid of change16:28
efriedand sdk is shiny and new and fancy16:28
efriedAny questions, comments, etc?16:28
*** bobh has joined #openstack-sdks16:29
dtantsurefried: what kind of support would you imagine?16:29
dtantsuresp. around making old nova cores a bit less crusty?16:29
efriedI dunno. Looking for opportunities to tell key people that this is a good idea?16:29
edleafeefried: Sure, I'm sure that my opinion will sway the Nova cores. It *always* does!16:30
efriedheh16:30
efriedwell actually16:30
dtantsurwe should have had a Forum session16:30
mordredyeah16:30
efriedit sort of dovetails with the (ill-fated) community goal for osc.16:30
mordredwell - tim bell has consistently raised his hands in support of this16:30
mordrediirc16:30
mordredso - I think we just say "tim bell says it's a good idea"16:30
dtantsur++ should work16:31
mordredbecause nobody wants to argue with tim bell16:31
efriedUsing sdk over python-*client or even raw ksa in $project could be a community goal.16:31
mordredI mean - *I* don't want to argue with tim and I like arguing with anybody16:31
dtantsurheh16:31
elmikoefried: fwiw, i think the api sig could definitely help with moving the discussions forward and provide solid support in that respect. as the others have noted, i'm not sure how much we could lend in terms of coding or contributions to nova.16:31
mordredefried: yeah - but I'd honestly still focus it on using sdk - even if people just want to use raw ksa16:31
openstackgerritEd Leafe proposed openstack/api-sig master: Add document describing consuming version discovery  https://review.openstack.org/45940516:31
mordredlargely because it'll help us align on operator-centric config16:31
edleafeelmiko: dtantsur: ^^ rebase the version discovery doc16:32
dtantsurmany thanks edleafe!16:32
mordredefried: and if people don't want to use the object layer of sdk in their code - that's fine ... they can still just do things like conn.compute.get('/servers')16:32
efriedmordred: ack, that's been my thinking. I alluded to that in the nova blueprint16:32
mordred++16:32
elmikoedleafe: ack, thanks16:32
efriede.g. I don't think placement has any object model code in sdk, right?16:32
mordrednope, not to my knowledge16:33
mordredso I think nova would just be using conn.placement.get('/things')16:33
dtantsurI think the biggest reason to NOT use ironicclient is that we largely screwed microversioning there :)16:33
efriedbut we ought to be able to swap out the ksa adapter with Connection.placement and not have to do much else except some fixture stuff.16:33
efriedyes.16:33
mordredefried: exactly16:33
dtantsurand this is also my argument against recommending people to use raw ksa: microversioning requires careful handling..16:34
efriedduring transition, operators can continue using ksa oslo.config opts, but can cut over to clouds.yaml any time they want16:34
efriedokay, dtantsur and mordred these arguments are going to be helpful.16:34
efriedBecause we "just" (queens) got done converting (almost) everything to ksa.16:35
efriedexcept we really didn't16:35
efriedwe just enabled ksa opts, but are still using clients under the covers16:35
dtantsurditto for ironic16:35
cdentIs using clouds.yaml going to be a requirement of this thing, or just an option? Because I know that having more than one thing to config has been sad-making for some in the past.16:35
efriedcdent: mordred wrote a patch to enable oslo.config ksa opts, so it's an option16:36
cdentkewl16:36
efriedhttps://review.openstack.org/64360116:36
efriedI still have an open question16:36
efriedabout how to bootstrap use of clouds.yaml16:36
efriedWhat I've inferred is that there would be a (single) oslo.config opt (per service-I'm-talking-to) to denote which, um, section of the clouds.yaml to use for that connection.16:37
efriedso iow to cut over to clouds.yaml for [service], operator would replace [service].$ksa_opts with [service].use_clouds_yaml_section=admin or similar.16:38
openstackgerritMerged openstack/api-sig master: Added Placement to the liaisons file  https://review.openstack.org/64522316:38
efriedand the proj code would pick that up and do Connection(cloud=CONF.service.use_clouds_yaml_section) instead of Connection(oslo_config_stuff)16:38
efriedmordred: does that gel with your thinking?16:39
mordredefried: yeah - I think that's likely right - hand waving about the actual details16:39
efriedright16:39
efriedoh, I am not going to budge on those conf opt names, though.16:39
efriedMY bikeshed16:39
efriedokay, cool, thanks for the airtime folks.16:40
edleafeefried: we'll mail you the bill16:40
elmikogladly efried =)16:40
elmikoerr... yeah, what edleafe said XD16:40
* mordred paints efried's bikeshed16:42
dtantsurlol16:42
cdentI knew mordred was saucy, but had no idea he was so rude.16:44
edleafeI'm thinking that the image doesn't have pymysql. Does it?16:47
edleafewhoops, wrong channel16:48
*** e0ne has joined #openstack-sdks16:48
*** dtantsur is now known as dtantsur|afk16:50
*** e0ne has quit IRC16:52
*** e0ne has joined #openstack-sdks16:52
*** tosky has quit IRC16:56
*** bobh has quit IRC16:57
*** e0ne has quit IRC16:58
edleafeelmiko: ugh, the version discovery rebase is getting a docs error16:58
edleafeI'll see if I can figure that out later. Gotta run for a bit now16:59
elmikodoh17:00
elmikolater o/17:00
elmikohave a good weekend folks =)17:00
openstackgerritEric Fried proposed openstack/api-sig master: Add document describing consuming version discovery  https://review.openstack.org/45940517:18
efriededleafe, elmiko: ftfy ^17:18
elmikoefried: thanks!17:39
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion  https://review.openstack.org/65268217:41
*** ralonsoh has quit IRC17:43
*** holser_ has quit IRC17:54
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion  https://review.openstack.org/65268218:06
edleafeelmiko: Since Eric's fix looks good, shall I freeze https://review.openstack.org/#/c/459405/18:07
*** cdent has quit IRC18:12
elmikoedleafe ++, just added my review18:19
edleafek18:20
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion  https://review.openstack.org/65268218:29
elmikothanks for handling all the freezes edleafe18:32
edleafeThat's 'cause I'm the cool one18:32
elmiko++18:32
*** e0ne has joined #openstack-sdks19:14
*** jangutter has joined #openstack-sdks19:15
*** e0ne has quit IRC19:18
*** jangutter has quit IRC19:49
*** jangutter has joined #openstack-sdks20:04
*** jangutter has quit IRC20:09
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion  https://review.openstack.org/65268220:12
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion  https://review.openstack.org/65268220:49
*** efried has quit IRC20:55
openstackgerritMonty Taylor proposed openstack/openstacksdk master: DNM Format code with black and gate on it  https://review.openstack.org/65387622:41
*** holser_ has joined #openstack-sdks22:46
*** efried has joined #openstack-sdks23:00
*** whoami-rajat has quit IRC23:03
*** holser_ has quit IRC23:23
*** holser_ has joined #openstack-sdks23:29
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Continue refactoring of the image  https://review.openstack.org/65153423:36

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