| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-12-06 | |||
| 16:10:23 | cdent | it's too hipster to live long | |
| 16:10:41 | elmiko | live fast, die young, leave a great spec? | |
| 16:11:01 | edleafe | You and your hipster jokes | |
| 16:11:10 | edleafe | I was hating hipsters before it was cool | |
| 16:11:16 | elmiko | ooh, nice | |
| 16:13:37 | dtantsur | wow, such news | |
| 16:39:50 | edleafe | Hey elmiko, dtantsur - regarding https://review.openstack.org/#/c/616610/ (DELETE guidance), I'd like to take a crack at rewording it a bit. Not changing the meaning, but just making it read better. Once that's done and you guys OK it, should we consider freezing for the liaisons to review? | |
| 16:40:05 | elmiko | edleafe: sgtm | |
| 16:40:12 | dtantsur | I'm kind of -0 on the whole idea, but not blocking | |
| 16:40:23 | dtantsur | explicit better than implicit etc | |
| 16:40:41 | elmiko | i like the clarification in general, but i agree that wording it more clearly would be nice | |
| 16:41:03 | edleafe | dtantsur: yeah, you've said that before. That's why I'd like the liaisons to give their opinions, too. | |
| 16:41:10 | elmiko | ++ | |
| 16:44:39 | cdent | since when has a guideline every changed existing behavior? | |
| 16:44:45 | elmiko | exactly... | |
| 16:48:26 | edleafe | I was thinking the same thing :) | |
| 16:50:49 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: Add propagate_uplink_status to port https://review.openstack.org/586687 | |
| 17:00:52 | elmiko | catch ya next week api-sig folks =) | |
| 17:02:08 | dtantsur | o/ | |
| 17:02:13 | edleafe | \o | |
| 20:38:00 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Deal with double-normalization of host_id https://review.openstack.org/623308 | |
| 20:51:06 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Deal with double-normalization of host_id https://review.openstack.org/623308 | |
| #openstack-sdks - 2018-12-07 | |||
| 09:51:35 | openstackgerrit | Yuval Shalev proposed openstack/openstacksdk master: Add host aggregate missing functions https://review.openstack.org/621695 | |
| 13:11:36 | openstackgerrit | melissaml proposed openstack/cliff master: Change openstack-dev to openstack-discuss https://review.openstack.org/623482 | |
| 13:22:37 | openstackgerrit | Gergely Csatari proposed openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485 | |
| 16:06:43 | cdent | elmiko: I think sdk and api sig is good match. we don't want api to be just about api good behavior, it should be about using apis in general | |
| 16:06:58 | cdent | otherwise sigs are too special and will fade away (like api-sig pretty much did) | |
| 16:09:13 | mordred | ++ | |
| 16:13:13 | dtantsur | this | |
| 17:05:41 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Fix requesting specific fields from ironic https://review.openstack.org/623059 | |
| 17:27:24 | openstackgerrit | Merged openstack/openstacksdk master: Deal with double-normalization of host_id https://review.openstack.org/623308 | |
| 17:27:25 | openstackgerrit | Merged openstack/openstacksdk master: Change openstack-dev to openstack-discuss https://review.openstack.org/622947 | |
| 18:14:08 | elmiko | cdent: i agree. additionally, i think the noise and process around creating another sig just adds confusion. i'm happy to include the sdk in the api sig's mission, i just wanted to add some context around what i think we need to do to make it a success. | |
| 18:14:13 | elmiko | edleafe dtantsur|afk ^^ | |
| 18:14:22 | cdent | ✔ | |
| 18:15:00 | elmiko | my main reasoning for even bringing up the working group thing, was just around the initial effort that Melvin is proposing. it seemed really well scoped. | |
| 18:15:18 | edleafe | elmiko: we discussed this at the last few PTGs. The SDK work is a good fit for our SIG. | |
| 18:16:11 | elmiko | no disagreement from me | |
| 18:16:57 | cdent | elmiko: the reason I brought it up was because it was vaguely sounding the thread like somebody (not sure if it was you or not) was thinking a separate sig was the way to go. I admit to not reading as closely as I should have | |
| 18:17:55 | elmiko | cdent: yeah, i think second sig was definitely amongst the ideas tossed out there | |
| 18:18:36 | elmiko | i don't really have a _strong_ opinion on it, but we did talk about this a few PTGs ago (and maybe more that i missed) and it still makes sense to me. | |
| 18:18:39 | edleafe | Yeah, I think Melvin thought that would be the easiest approach | |
| 18:18:48 | elmiko | i think my main concern is getting more bodies to step up and participate | |
| 18:19:26 | elmiko | dtantsur|afk's work aside, we need someone to represent the sdk side of this | |
| 18:19:38 | edleafe | If it's gonna work, it's gonna need the SDK people to do the bulk of the organizing and work. We can help with our experience, but it will need to be new blood. | |
| 18:19:42 | cdent | Can't we just get dtantsur|afk to do everything, since he's afk? | |
| 18:20:05 | edleafe | Done! dtantsur|afk now owns the responsibility for All The Things | |
| 18:20:06 | elmiko | edleafe: yeah, i remember him approaching us about adding more sdk/operator focused stuff to our agenda, and i thought we were all fine with it back then | |
| 18:20:11 | elmiko | cdent: ++ | |
| 18:20:19 | elmiko | SIG All-The-Things? | |
| 18:20:38 | elmiko | edleafe: ++ agreed about the workload | |
| 18:20:51 | elmiko | i guess that's what i wanted to capture in my response to Thierry | |
| 18:21:20 | edleafe | I was going to reply, but thought it would be just "what elmiko said" | |
| 18:21:59 | elmiko | edleafe: =) | |
| 18:28:11 | elmiko | cdent: i hope it didn't sound like i was advocating the other way. it sounds like we have a good consensus though | |
| 18:28:38 | cdent | somebody may wish to clarify? not sure, I'm not caught up on the thread | |
| 18:29:08 | elmiko | good idea, i'll take another look to see if i was ambiguous and could potentially make it clearer | |
| 18:29:15 | elmiko | or maybe just link to this talk in chat | |
| 18:32:15 | cdent | but we have our professional reputations to consider!! | |
| 18:34:40 | openstackgerrit | Ed Leafe proposed openstack/api-sig master: Add idempotent DELETE guidance https://review.openstack.org/616610 | |
| 18:35:06 | edleafe | elmiko: cdent: ^^ Cleaned up the wording. Let me know if you think it's better | |
| 18:36:06 | elmiko | edleafe: ack | |
| #openstack-sdks - 2018-12-10 | |||
| 11:26:37 | dtantsur | cdent, edleafe, /me git rm -f /all/the/things && git commit -m "My work here is done" && git push -f | |
| 11:26:54 | cdent | yes!!! | |
| 12:31:14 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Fix requesting specific fields from ironic https://review.openstack.org/623059 | |
| 13:38:00 | edleafe | dtantsur: | |
| 13:38:32 | edleafe | dtantsur: oh, crap - now we're all out of a job. Nothing left to do! | |
| 13:47:30 | dtantsur | what do you mean? we still can rewrite everything in rust! | |
| 13:48:07 | cdent | I think you just used up openstack's ninth life. game over. | |
| 13:48:19 | dtantsur | lol | |
| 13:58:30 | dtantsur | mordred: morning! new ironic bug fix (sigh :) ready for review: https://review.openstack.org/#/c/623059/ | |
| 14:50:22 | mordred | yay bug fixes! | |
| 14:53:40 | mordred | dtantsur: in https://review.openstack.org/#/c/623059/4/openstack/baremetal/v1/chassis.py you set _max_microversion = 1.8 ... is that supposed to be min? | |
| 14:58:09 | dtantsur | mordred: it's max, because we don't know/support higher versions for chassis (there is none so far) | |
| 15:00:29 | mordred | kk. just checking | |
| 15:01:36 | cdent | APPARENTLY MY HINT WAS TOO PASSIVE. GIVE ME COFFEE | |
| 15:09:22 | mordred | dtantsur: if you have a sec, https://review.openstack.org/#/c/621153 from gtema looks good to me, but it touches many files | |
| 15:09:31 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154 | |
| 15:09:41 | dtantsur | mordred: will check | |
| 15:09:50 | mordred | and it's in support of that patch, that I just rebased | |
| 15:16:23 | mordred | mriedem: would you look at https://review.openstack.org/#/c/621695 and let me know if the api calls added to the compute proxy seem like the right thing to add? (they look fine to me, but I've never worked with host aggregates) | |
| 15:19:16 | mriedem | ack | |
| 15:19:55 | mordred | dtantsur, Shrews: also, eyeballs on https://review.openstack.org/#/c/618993 would be welcome | |
| 15:22:40 | cdent | make sure you form that query right or you'll get a partial cdent resource | |
| 15:23:09 | dtantsur | OMG | |
| 15:26:24 | mordred | cdent: did you know you can use graphql inside of react javascript just as a form to manipulate in-memory data objects? | |
| 15:26:48 | cdent | well, thank goodness for that, all my problems are solved | |
| 15:30:30 | mordred | cdent: I only know this because it's used in the gatsby static site generator tool: https://github.com/emonty/jubilant-fortnight/blob/master/src/templates/blog-index.js#L55 (and which might be the best and most natural use I've seen for the concept, tbh) | |
| 15:31:41 | mordred | the underlying framework sucks in all of the markdown files that exist and puts them into memory, and then you an graphql that data structure as needed to generate content rather than needing to nested for loop a bunch | |
| 15:32:22 | cdent | sounds rather reasonable | |
| 16:17:02 | openstackgerrit | Merged openstack/openstacksdk master: Fix requesting specific fields from ironic https://review.openstack.org/623059 | |
| 16:23:01 | openstackgerrit | Merged openstack/openstacksdk master: Add propagate_uplink_status to port https://review.openstack.org/586687 | |
| 16:35:00 | openstackgerrit | Merged openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 16:48:22 | openstackgerrit | Merged openstack/openstacksdk master: block_storage.backup func tests to use configurable timeout https://review.openstack.org/622933 | |
| 16:57:15 | openstackgerrit | Merged openstack/openstacksdk master: Register proxy directly in add_service https://review.openstack.org/620398 | |
| 16:57:16 | openstackgerrit | Merged openstack/openstacksdk master: Fix query parameters of network.port_forwarding https://review.openstack.org/622963 | |
| 17:20:42 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Supports router gateway IP QoS https://review.openstack.org/622130 | |
| 18:04:00 | openstackgerrit | Merged openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993 | |
| 18:21:23 | openstackgerrit | Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139 | |
| 18:50:16 | openstackgerrit | Gergely Csatari proposed openstack/os-api-ref master: Adding suport for readthedocs.org https://review.openstack.org/624165 | |