*** TheJulia_ has joined #openstack-sdks | 00:54 | |
*** kmalloc_ has joined #openstack-sdks | 00:54 | |
*** fyx_ has joined #openstack-sdks | 00:54 | |
*** kmalloc has quit IRC | 01:02 | |
*** TheJulia has quit IRC | 01:02 | |
*** Dinesh_Bhor has quit IRC | 01:02 | |
*** fyx has quit IRC | 01:02 | |
*** melwitt has quit IRC | 01:02 | |
*** kmalloc_ is now known as kmalloc | 01:02 | |
*** TheJulia_ is now known as TheJulia | 01:02 | |
*** fyx_ is now known as fyx | 01:02 | |
*** irclogbot_1 has quit IRC | 01:05 | |
*** ricolin has joined #openstack-sdks | 01:05 | |
*** irclogbot_2 has joined #openstack-sdks | 01:06 | |
*** Dinesh_Bhor has joined #openstack-sdks | 01:08 | |
*** whoami-rajat has joined #openstack-sdks | 01:24 | |
*** melwitt has joined #openstack-sdks | 01:25 | |
*** bobh has joined #openstack-sdks | 01:50 | |
*** bobh has quit IRC | 01:51 | |
*** holser_ has joined #openstack-sdks | 03:50 | |
*** holser_ has quit IRC | 03:54 | |
*** holser_ has joined #openstack-sdks | 03:57 | |
*** holser_ has quit IRC | 04:09 | |
*** slaweq has joined #openstack-sdks | 04:45 | |
*** slaweq has quit IRC | 04:50 | |
*** gkadam has joined #openstack-sdks | 05:01 | |
*** gkadam_ has joined #openstack-sdks | 05:07 | |
*** gkadam has quit IRC | 05:09 | |
*** slaweq has joined #openstack-sdks | 05:56 | |
*** e0ne has joined #openstack-sdks | 06:17 | |
*** e0ne has quit IRC | 06:18 | |
*** Luzi has joined #openstack-sdks | 06:23 | |
*** gtema has joined #openstack-sdks | 06:56 | |
*** gkadam_ has quit IRC | 07:01 | |
*** ianw is now known as ianw_pto | 07:15 | |
*** slaweq has quit IRC | 07:16 | |
*** slaweq has joined #openstack-sdks | 07:28 | |
*** tosky has joined #openstack-sdks | 07:42 | |
*** jpich has joined #openstack-sdks | 07:56 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534 | 08:08 |
---|---|---|
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Update baremetal to use proxy logger https://review.openstack.org/652083 | 08:11 |
*** ralonsoh has joined #openstack-sdks | 08:16 | |
*** dtantsur|afk is now known as dtantsur | 08:25 | |
*** e0ne has joined #openstack-sdks | 08:33 | |
*** e0ne has quit IRC | 08:34 | |
*** ttsiouts has joined #openstack-sdks | 08:45 | |
*** e0ne has joined #openstack-sdks | 08:45 | |
*** e0ne has quit IRC | 08:47 | |
*** holser_ has joined #openstack-sdks | 08:57 | |
*** e0ne has joined #openstack-sdks | 08:59 | |
*** markvoelker has joined #openstack-sdks | 09:12 | |
*** gtema has quit IRC | 09:17 | |
*** markvoelker has quit IRC | 09:41 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Add "name" filter in "list" call when retrieving a single register https://review.openstack.org/637238 | 09:41 |
*** ttsiouts has quit IRC | 09:51 | |
*** ttsiouts has joined #openstack-sdks | 09:53 | |
*** e0ne has quit IRC | 09:54 | |
*** e0ne has joined #openstack-sdks | 10:00 | |
*** ttsiouts has quit IRC | 10:09 | |
*** ttsiouts has joined #openstack-sdks | 10:10 | |
*** gtema has joined #openstack-sdks | 10:15 | |
*** Luzi has quit IRC | 10:28 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534 | 10:30 |
*** markvoelker has joined #openstack-sdks | 10:38 | |
*** zigo has joined #openstack-sdks | 10:51 | |
*** cdent has joined #openstack-sdks | 11:09 | |
*** markvoelker has quit IRC | 11:11 | |
*** slaweq is now known as slaweq|lunch | 11:15 | |
*** slaweq|lunch is now known as slaweq | 11:33 | |
*** ttsiouts has quit IRC | 11:59 | |
*** bobh has joined #openstack-sdks | 11:59 | |
*** markvoelker has joined #openstack-sdks | 12:09 | |
*** ttsiouts has joined #openstack-sdks | 12:09 | |
*** ttsiouts has quit IRC | 12:12 | |
*** ttsiouts_ has joined #openstack-sdks | 12:12 | |
*** cdent has quit IRC | 12:15 | |
*** bobh has quit IRC | 12:23 | |
*** markvoelker has quit IRC | 12:41 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove now unused task_manager file https://review.openstack.org/653731 | 13:03 |
*** ttsiouts_ has quit IRC | 13:06 | |
*** ttsiouts has joined #openstack-sdks | 13:13 | |
*** markvoelker has joined #openstack-sdks | 13:38 | |
*** markvoelker has quit IRC | 13:42 | |
gtema | mordred: do you have an opinion on https://review.openstack.org/#/c/651534/12/openstack/resource.py@425? | 13:43 |
gtema | do we want to support this logic everywhere? | 13:43 |
gtema | dtantsur: 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 result | 13:46 |
mordred | gtema: yes - I replied | 13:47 |
gtema | thanks | 13:47 |
gtema | I can also move "output" logic under the Image.download method if we want it now | 13:48 |
mordred | yeah - I think that's a good idea | 13:48 |
gtema | ok. Will also add few tests on the proxy.download_image (were missing so far) | 13:49 |
dtantsur | gtema: then why not change the second place to just 'return resp'? | 13:50 |
gtema | hmm | 13:50 |
gtema | maybe | 13:50 |
gtema | what do we want for chunk_size? | 13:54 |
openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Support for the baremetal introspection service https://review.openstack.org/650044 | 13:54 |
dtantsur | mordred: a new service ^^^ \o/ | 13:54 |
dtantsur | gtema: I'd put at least 4k, but I don't have any reliable reasoning behind it | 13:55 |
dtantsur | ideally, we should benchmark it :) | 13:55 |
gtema | :D Benchmark will actually depend on the chosen setup/platform/image/etc | 13:56 |
gtema | dtantsur: returning resp instead of content breaks current behavior. This is not something I wanted to get with this change | 14:01 |
*** cdent has joined #openstack-sdks | 14:10 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534 | 14:17 |
dtantsur | gtema: ah, I see | 14:21 |
*** gtema has quit IRC | 14:22 | |
*** markvoelker has joined #openstack-sdks | 14:26 | |
*** samueldmq has joined #openstack-sdks | 14:29 | |
*** ricolin has quit IRC | 14:32 | |
*** yolanda_ has quit IRC | 14:40 | |
*** ttsiouts has quit IRC | 14:44 | |
*** ttsiouts has joined #openstack-sdks | 14:44 | |
openstackgerrit | Merged openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273 | 14:49 |
openstackgerrit | Merged openstack/openstacksdk master: Allow replacing service implementation https://review.openstack.org/648704 | 15:02 |
*** e0ne has quit IRC | 15:08 | |
*** e0ne has joined #openstack-sdks | 15:09 | |
*** e0ne has quit IRC | 15:16 | |
*** cdent has quit IRC | 15:18 | |
*** ttsiouts has quit IRC | 15:27 | |
*** ttsiouts has joined #openstack-sdks | 15:29 | |
*** bobh has joined #openstack-sdks | 15:41 | |
elmiko | API SIG office hour is now open! | 16:00 |
*** ttsiouts has quit IRC | 16:00 | |
edleafe | Yay!!!! | 16:00 |
elmiko | \o/ | 16:00 |
* edleafe pops champagne | 16:00 | |
elmiko | ooh la la | 16:00 |
edleafe | I did want to go over some outstanding api-sig patches, if that's ok with you | 16:01 |
elmiko | sure, i'm game! | 16:02 |
edleafe | I think the DNS guideline is ready for freeze https://review.openstack.org/#/c/651222/ | 16:02 |
elmiko | ++ | 16:02 |
edleafe | So is the version discovery doc, but it has a merge conflict https://review.openstack.org/#/c/459405/ | 16:02 |
elmiko | i guess we should just send custom freeze emails to the ml since we don't have the newsletter? | 16:03 |
elmiko | oh wow, so exciting to see the version discovery finally getting there =) | 16:03 |
edleafe | Generally that's what the tools do: send an email to the liaisons, and add them to the reviewer list for the patch | 16:03 |
elmiko | ok, sweet | 16:04 |
edleafe | But sure, we could also send out an email | 16:04 |
elmiko | i 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 XD | 16:04 |
edleafe | dtantsur had a suggestion on the liaisons file patch https://review.openstack.org/#/c/645223/ | 16:04 |
edleafe | Once I fix that, I think that that should be a quick approve | 16:04 |
edleafe | I.e., it's not changing a guideline or anything | 16:05 |
elmiko | ok, cool. i will remove my +2 with a note just for clarity sake | 16:05 |
elmiko | i agree with dtantsur's idea | 16:05 |
edleafe | schweet | 16:05 |
*** jpich has quit IRC | 16:06 | |
edleafe | I 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 |
edleafe | And I was going to fast-approve the emai list name change https://review.openstack.org/#/c/632255/1 | 16:07 |
edleafe | but it was stated that it was a dupe without referencing what it is a dupe of | 16:07 |
edleafe | Do you have any idea? | 16:07 |
* dtantsur w00t at freezing the DNS guideline | 16:07 | |
elmiko | edleafe: i agree about Akihiro's comment, thanks for taking that on | 16:07 |
dtantsur | edleafe: I don't see a dup, let's approve it.. | 16:08 |
edleafe | I 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 then | 16:08 |
elmiko | edleafe: agreed | 16:09 |
edleafe | That's all I had on my plate to discuss | 16:10 |
elmiko | edleafe ++ | 16:11 |
*** cdent has joined #openstack-sdks | 16:14 | |
*** bobh has quit IRC | 16:16 | |
edleafe | https://review.openstack.org/#/c/651222 has been frozen | 16:17 |
elmiko | thanks edleafe | 16:17 |
dtantsur | w00t! | 16:19 |
openstackgerrit | Ed Leafe proposed openstack/api-sig master: Added Placement to the liaisons file https://review.openstack.org/645223 | 16:19 |
edleafe | dtantsur: elmiko: ^^ Fixed the ordering | 16:19 |
* dtantsur expects an avalanche of "Are you crazy?" comments | 16:19 | |
edleafe | Either of you want to fast-approve that? | 16:19 |
* dtantsur plans on proudly answering YES | 16:19 | |
elmiko | edleafe: looking now | 16:19 |
edleafe | Nobody will ask such a dumb question. We already know the answer is yes | 16:19 |
efried | Does the API SIG deal with things like adoption of OpenStack SDK? | 16:20 |
dtantsur | efried: it borders with our responsibilities, at least | 16:20 |
elmiko | ++ | 16:20 |
efried | not to interrupt; feel free to come back to me when you reach an appropriate spot in the agenda | 16:20 |
edleafe | efried: not specifically, but we do say things like "yes, that's a good thing to do" | 16:21 |
elmiko | no real agenda, this is just open office hours | 16:21 |
elmiko | dtantsur edleafe , +wf on the liaison file | 16:21 |
edleafe | elmiko: kewl | 16:21 |
dtantsur | efried: I think it's a good topic for right now, but what exactly do you have in mind? | 16:21 |
efried | Okay, so this is mostly FYI: I'm kind of involved in / leading a charge to make nova use the sdk. | 16:21 |
dtantsur | if you need some to do some work, we have mordred to appoint | 16:22 |
elmiko | nice | 16:22 |
elmiko | LOL | 16:22 |
efried | https://blueprints.launchpad.net/nova/+spec/openstacksdk-in-nova | 16:22 |
dtantsur | because we don't do work here, it's not the goal of API SIG | 16:22 |
efried | mordred has already done some work. | 16:22 |
dtantsur | oh? | 16:22 |
elmiko | dtantsur ++ | 16:22 |
edleafe | We pride ourselves on convincing others to do the work | 16:22 |
elmiko | XD | 16:22 |
efried | there's this: https://review.openstack.org/643601 which needs to be cleaned up and have UT added | 16:23 |
efried | but has been integration-tested and works. | 16:23 |
dtantsur | efried: I'd really like to get to switching ironic-inspector to use SDK for talking to ironic and swift | 16:23 |
efried | tl;dr: lets you set up Connection with oslo.config ksa opts instead of clouds.yaml | 16:23 |
dtantsur | I mean, get to it in Train | 16:23 |
dtantsur | efried: I'm very much in favor of that. very. | 16:23 |
efried | ...so we can make the transition transparent to operators initially. | 16:23 |
efried | and then there's this: https://review.openstack.org/#/c/643664/ which introduces the plumbing for sdk in nova | 16:24 |
efried | and the patch on top of it https://review.openstack.org/#/c/642899/ which starts the process of swapping out calls into ironicclient. | 16:24 |
efried | idea being to stack several of those ^ up until we can rip out ironicclient entirely. | 16:24 |
efried | (from nova) | 16:25 |
efried | I've got a new guy helping out with the nova things | 16:25 |
efried | but could use some help on the sdk side | 16:25 |
efried | there's the patch noted above, already started, could use closure | 16:25 |
mordred | yeah. I'll definitely be digging in to help | 16:26 |
mordred | and I need to finish that patch as soon as I can finish up all the meetings I've been stuck in this week | 16:26 |
efried | and then there's a need for a fixture to facilitate testing on the nova side | 16:26 |
efried | mordred: Totally understood, I was actually fishing for other volunteers to help :) | 16:26 |
dtantsur | efried: 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 |
efried | here's a story for the fixtures: https://storyboard.openstack.org/#!/story/2005475 | 16:26 |
efried | dtantsur: ack, good to know. | 16:27 |
efried | So | 16:27 |
efried | besides soliciting help if possible for the sdk items noted above | 16:27 |
efried | it would be nice if the API SIG (or whoever) could lend support to this idea in general. | 16:28 |
efried | because nova cores are old and crusty and afraid of change | 16:28 |
efried | and sdk is shiny and new and fancy | 16:28 |
efried | Any questions, comments, etc? | 16:28 |
*** bobh has joined #openstack-sdks | 16:29 | |
dtantsur | efried: what kind of support would you imagine? | 16:29 |
dtantsur | esp. around making old nova cores a bit less crusty? | 16:29 |
efried | I dunno. Looking for opportunities to tell key people that this is a good idea? | 16:29 |
edleafe | efried: Sure, I'm sure that my opinion will sway the Nova cores. It *always* does! | 16:30 |
efried | heh | 16:30 |
efried | well actually | 16:30 |
dtantsur | we should have had a Forum session | 16:30 |
mordred | yeah | 16:30 |
efried | it sort of dovetails with the (ill-fated) community goal for osc. | 16:30 |
mordred | well - tim bell has consistently raised his hands in support of this | 16:30 |
mordred | iirc | 16:30 |
mordred | so - I think we just say "tim bell says it's a good idea" | 16:30 |
dtantsur | ++ should work | 16:31 |
mordred | because nobody wants to argue with tim bell | 16:31 |
efried | Using sdk over python-*client or even raw ksa in $project could be a community goal. | 16:31 |
mordred | I mean - *I* don't want to argue with tim and I like arguing with anybody | 16:31 |
dtantsur | heh | 16:31 |
elmiko | efried: 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 |
mordred | efried: yeah - but I'd honestly still focus it on using sdk - even if people just want to use raw ksa | 16:31 |
openstackgerrit | Ed Leafe proposed openstack/api-sig master: Add document describing consuming version discovery https://review.openstack.org/459405 | 16:31 |
mordred | largely because it'll help us align on operator-centric config | 16:31 |
edleafe | elmiko: dtantsur: ^^ rebase the version discovery doc | 16:32 |
dtantsur | many thanks edleafe! | 16:32 |
mordred | efried: 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 |
efried | mordred: ack, that's been my thinking. I alluded to that in the nova blueprint | 16:32 |
mordred | ++ | 16:32 |
elmiko | edleafe: ack, thanks | 16:32 |
efried | e.g. I don't think placement has any object model code in sdk, right? | 16:32 |
mordred | nope, not to my knowledge | 16:33 |
mordred | so I think nova would just be using conn.placement.get('/things') | 16:33 |
dtantsur | I think the biggest reason to NOT use ironicclient is that we largely screwed microversioning there :) | 16:33 |
efried | but 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 |
efried | yes. | 16:33 |
mordred | efried: exactly | 16:33 |
dtantsur | and this is also my argument against recommending people to use raw ksa: microversioning requires careful handling.. | 16:34 |
efried | during transition, operators can continue using ksa oslo.config opts, but can cut over to clouds.yaml any time they want | 16:34 |
efried | okay, dtantsur and mordred these arguments are going to be helpful. | 16:34 |
efried | Because we "just" (queens) got done converting (almost) everything to ksa. | 16:35 |
efried | except we really didn't | 16:35 |
efried | we just enabled ksa opts, but are still using clients under the covers | 16:35 |
dtantsur | ditto for ironic | 16:35 |
cdent | Is 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 |
efried | cdent: mordred wrote a patch to enable oslo.config ksa opts, so it's an option | 16:36 |
cdent | kewl | 16:36 |
efried | https://review.openstack.org/643601 | 16:36 |
efried | I still have an open question | 16:36 |
efried | about how to bootstrap use of clouds.yaml | 16:36 |
efried | What 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 |
efried | so 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 |
openstackgerrit | Merged openstack/api-sig master: Added Placement to the liaisons file https://review.openstack.org/645223 | 16:38 |
efried | and 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 |
efried | mordred: does that gel with your thinking? | 16:39 |
mordred | efried: yeah - I think that's likely right - hand waving about the actual details | 16:39 |
efried | right | 16:39 |
efried | oh, I am not going to budge on those conf opt names, though. | 16:39 |
efried | MY bikeshed | 16:39 |
efried | okay, cool, thanks for the airtime folks. | 16:40 |
edleafe | efried: we'll mail you the bill | 16:40 |
elmiko | gladly efried =) | 16:40 |
elmiko | err... yeah, what edleafe said XD | 16:40 |
* mordred paints efried's bikeshed | 16:42 | |
dtantsur | lol | 16:42 |
cdent | I knew mordred was saucy, but had no idea he was so rude. | 16:44 |
edleafe | I'm thinking that the image doesn't have pymysql. Does it? | 16:47 |
edleafe | whoops, wrong channel | 16:48 |
*** e0ne has joined #openstack-sdks | 16:48 | |
*** dtantsur is now known as dtantsur|afk | 16:50 | |
*** e0ne has quit IRC | 16:52 | |
*** e0ne has joined #openstack-sdks | 16:52 | |
*** tosky has quit IRC | 16:56 | |
*** bobh has quit IRC | 16:57 | |
*** e0ne has quit IRC | 16:58 | |
edleafe | elmiko: ugh, the version discovery rebase is getting a docs error | 16:58 |
edleafe | I'll see if I can figure that out later. Gotta run for a bit now | 16:59 |
elmiko | doh | 17:00 |
elmiko | later o/ | 17:00 |
elmiko | have a good weekend folks =) | 17:00 |
openstackgerrit | Eric Fried proposed openstack/api-sig master: Add document describing consuming version discovery https://review.openstack.org/459405 | 17:18 |
efried | edleafe, elmiko: ftfy ^ | 17:18 |
elmiko | efried: thanks! | 17:39 |
openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682 | 17:41 |
*** ralonsoh has quit IRC | 17:43 | |
*** holser_ has quit IRC | 17:54 | |
openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682 | 18:06 |
edleafe | elmiko: Since Eric's fix looks good, shall I freeze https://review.openstack.org/#/c/459405/ | 18:07 |
*** cdent has quit IRC | 18:12 | |
elmiko | edleafe ++, just added my review | 18:19 |
edleafe | k | 18:20 |
openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682 | 18:29 |
elmiko | thanks for handling all the freezes edleafe | 18:32 |
edleafe | That's 'cause I'm the cool one | 18:32 |
elmiko | ++ | 18:32 |
*** e0ne has joined #openstack-sdks | 19:14 | |
*** jangutter has joined #openstack-sdks | 19:15 | |
*** e0ne has quit IRC | 19:18 | |
*** jangutter has quit IRC | 19:49 | |
*** jangutter has joined #openstack-sdks | 20:04 | |
*** jangutter has quit IRC | 20:09 | |
openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682 | 20:12 |
openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682 | 20:49 |
*** efried has quit IRC | 20:55 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: DNM Format code with black and gate on it https://review.openstack.org/653876 | 22:41 |
*** holser_ has joined #openstack-sdks | 22:46 | |
*** efried has joined #openstack-sdks | 23:00 | |
*** whoami-rajat has quit IRC | 23:03 | |
*** holser_ has quit IRC | 23:23 | |
*** holser_ has joined #openstack-sdks | 23:29 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534 | 23:36 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!