Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-12-06
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
18:51:58 openstackgerrit Gergely Csatari proposed openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485
19:16:20 mriedem mordred: done re https://review.openstack.org/#/c/621695/
19:27:35 openstackgerrit Gergely Csatari proposed openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485
19:38:02 openstackgerrit Yuval Shalev proposed openstack/openstacksdk master: Add host aggregate missing functions https://review.openstack.org/621695
19:45:46 mordred mriedem: thank you
20:09:25 openstackgerrit Merged openstack/python-openstackclient master: Don't display router's is_ha and is_distributed attributes always https://review.openstack.org/567620
20:11:07 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Don't try to upload to images in saving or queued state https://review.openstack.org/624188
22:31:42 openstackgerrit Merged openstack/openstacksdk master: Add host aggregate missing functions https://review.openstack.org/621695
#openstack-sdks - 2018-12-11
02:22:14 openstackgerrit Gergely Csatari proposed openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485
07:33:20 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
07:36:36 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
13:51:54 openstackgerrit Telles Mota Vidal Nóbrega proposed openstack/os-api-ref master: Fixing duplicate label issue. https://review.openstack.org/624400
14:47:13 openstackgerrit Merged openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485

Earlier   Later