Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-09-17
07:49:50 gtema let's see whether it helps :)
07:50:48 gtema I didn't know the image is so deep in OSC. The change is now similar huge to the what it was in SDK :D
07:50:59 gtema it would be fun to wait for review
08:02:19 mordred gtema: image is such a fun topic
08:02:37 gtema yeah
11:20:55 openstackgerrit Vishakha Agarwal proposed openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272
11:34:20 gtema what the heck is going on, that still every change fails. Argh....
11:40:21 mordred gtema: there was a mirror issue for a second - although I think that's fixed now
11:40:48 gtema really?
11:41:20 gtema ah, yeah. lets see whether 3rd or 5th round of rechecks helps
13:11:13 openstackgerrit Merged openstack/openstacksdk master: Properly convert baremetal fields to server-side values https://review.opendev.org/680649
13:19:01 dtantsur mordred: we may need to release openstacksdk with stable/train, otherwise https://review.opendev.org/#/c/682412/ will happen
13:19:44 gtema we need to get image tags patch in, so that I finish OSC change
13:19:58 gtema s/I/I can/
13:40:43 gtema oh god, my change is again going to fail. What a wonderful day full of sporadic failures everywhere
13:53:06 dtantsur \o/
13:53:16 dtantsur it's been like that in the ironic world for weeks, I know your pain
13:53:34 dtantsur so, we need to release ASAP or at least communicate the plans to the release team
13:53:36 dtantsur mordred: ^^
13:54:17 gtema dtantsur: I need https://review.opendev.org/#/c/682454/ in the nearest release, so if you mind ...
13:54:30 gtema even tests are again failing (unrelated)
13:56:03 dtantsur gtema: please see if you want to fix my comment https://review.opendev.org/#/c/682454/
13:57:16 gtema well, I can quickly do this, if you both agree to put it into release ASAP, otherwise followup
13:59:31 dtantsur I can approve it, I think. it's a trivial modification to the already approved patch
13:59:38 dtantsur I mean, approve the updated version
14:00:01 gtema ok, gimme a sec. The current test run is anyway failing
14:02:12 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Fix image create with tags https://review.opendev.org/682454
14:03:10 gtema done
14:03:12 dtantsur +A
14:03:32 gtema thks
14:06:09 mordred dtantsur, gtema: let me go ping the release team
14:06:16 gtema cool
14:14:09 mordred dtantsur, gtema: seems like bugfixes -so this might be an 0.35.1 - yeah?
14:14:23 gtema yupp, think so
14:52:43 openstackgerrit Merged openstack/openstacksdk master: baremetal-introspection: fix passing a Node to start_introspection https://review.opendev.org/681229
14:57:25 johnsom gtema: the bind_ip issue is related to a recent OSC change that changed the output format for networking fixed IPs. We merged an Octavia patch for that about 12 hours ago.
14:59:07 gtema johnsom: yeah, I asked today morning in the octavia channel. Thanks
14:59:26 johnsom Ah, ok
14:59:31 gtema perhaps it make sense to add a devstack-networking type of job in the octavia jobs?
14:59:44 gtema to avoid it in future
15:01:15 johnsom devstack-networking?
15:02:21 gtema in the SDK we have a functional-devstack-networking jobs, which is executing all the tests with Octavia and rest of "extended" networking things
15:03:09 gtema https://opendev.org/openstack/openstacksdk/src/branch/master/.zuul.yaml#L120
15:03:10 johnsom Yes
15:03:43 gtema where basically the devstack installation itself failed
15:04:47 johnsom As did all of ours when OSC merged the breaking change.
15:05:08 gtema aaah, it was the OSC change, got it
15:06:37 johnsom Yeah, the client output for neutron ports significantly changed in a non-backwards compatible way. This broke our devstack plugin script.
15:06:53 gtema ok
15:07:40 johnsom We wrote a big awk script to handle both the old and new output formats to work around it.
15:08:40 gtema yeah, does not look like a perfect approach ;-)
15:16:42 frickler maybe for such complex data manipulation a python script that uses the sdk would be a better solution
17:09:17 openstackgerrit Merged openstack/openstacksdk master: Fix image create with tags https://review.opendev.org/682454
21:06:59 mordred frickler, johnsom: clarkb did some experiments with using sdk in a python script in devstack for some other things. looking at that octavia chunk, it might be a more pleasant set of things to do as an sdk script
21:07:41 johnsom mordred You are proposing changing the devstack plugins from bash to python?
21:08:09 mordred johnsom: not generally - but it might be easier to just add a python script that the bash plugin calls to do the interaction
21:08:24 mordred like - there's a bunch of bash you're having to do there
21:08:25 johnsom Would work... Not sure what all we would need to bring in for some of that other system stuff, or have a bunch of command calls.
21:09:50 johnsom It's a pretty simple awk script really. My guess is faster than starting up a python environment. Really it's just a bummer that the -f value output format changed such that we need to deal with both.
21:11:24 johnsom Would be an interesting exercise to compare our OSC calls to non-OSC calls. If it's OSC heavy then just changing the devstack plugin to python might make sense.
21:18:24 mordred johnsom: yeah - I think it would only really be a win if you were converting a bunch of calls into a single script - not just the awk line - a single awk invocation is almost certainly cheaper
21:19:03 mordred but a single python script vs 8 osc invocations *might* be a win. otoh - it adds another layer
21:19:25 johnsom Yep
22:30:35 lifeless starting python is slowish
22:30:39 lifeless importing python code is slowish
22:30:44 lifeless running python code is slowish
22:30:57 lifeless doing all three is about the same time as running 10 or so awks
#openstack-sdks - 2019-09-18
04:55:51 openstackgerrit Duc Truong proposed openstack/openstacksdk master: [WIP] Add clustering update_action https://review.opendev.org/682787
04:58:25 openstackgerrit Duc Truong proposed openstack/openstacksdk master: [WIP] Add clustering update_action https://review.opendev.org/682787
06:21:22 openstackgerrit inspurericzhang proposed openstack/shade master: fix "How To Contribute" url https://review.opendev.org/682801
06:34:37 openstackgerrit inspurericzhang proposed openstack/openstacksdk master: fix "How To Contribute" url https://review.opendev.org/682803
07:57:55 openstackgerrit Merged openstack/openstacksdk master: Build PDF docs https://review.opendev.org/601659
08:03:32 frickler a python script might be easier to understand, debug and test than awk, even if it is slower
08:53:11 openstackgerrit Merged openstack/openstacksdk master: Cleanup doc/source/conf.py https://review.opendev.org/679910
09:10:51 mordred yay the pdf docs patches actually landed
09:11:17 gtema yes, it took quite some dozens of rechecks
09:54:20 openstackgerrit inspurericzhang proposed openstack/shade master: fix "How To Contribute",bugs,review url https://review.opendev.org/682801
14:28:30 efried Can openstacksdk image do fancy things like download at this point?
14:29:41 efried https://docs.openstack.org/openstacksdk/latest/user/proxies/image_v2.html#openstack.image.v2._proxy.Proxy.download_image looks like it -- does this work?
14:31:38 openstackgerrit Bram proposed openstack/openstacksdk master: support restore from volume backup in cloud layer https://review.opendev.org/682909
15:41:14 openstackgerrit Duc Truong proposed openstack/openstacksdk master: Add clustering update_action https://review.opendev.org/682787
15:56:52 mriedem dtroyer: i'm +1 on this osc change https://review.opendev.org/#/c/673725/ but found a way to tighten up one of the tests which was a false positive before - i left the diff inline and was wondering if you'd want that in the same change or in a follow up?
15:57:15 mriedem i'm thinking follow up...
15:59:27 dtroyer mriedem: follow-up is fine, the usual worry is it never getting done, I trust you will see to that :)
15:59:38 mriedem i've got the patch locally, pushing
15:59:39 dtroyer er, see to it getting done
16:01:10 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Provide stderr in exception when check_parser fails https://review.opendev.org/682939
16:06:37 openstackgerrit Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374
17:24:35 openstackgerrit Merged openstack/openstacksdk master: baremetal-introspection: add manage_boot argument to start_introspection https://review.opendev.org/681225
#openstack-sdks - 2019-09-19
11:57:17 openstackgerrit Andreas Florath proposed openstack/python-openstackclient master: Fix osc-lib interface change: catch osc-lib Forbidden https://review.opendev.org/683118
11:58:52 openstackgerrit Andreas Florath proposed openstack/osc-lib master: Fix osc-lib interface change: throw osc-lib Forbidden https://review.opendev.org/683119
11:59:43 openstackgerrit Andreas Florath proposed openstack/python-openstackclient master: Fix osc-lib interface change: catch osc-lib Forbidden https://review.opendev.org/683118
16:00:10 elmiko API SIG office hour is now starting
16:00:25 mordred you're an office hour
16:01:09 elmiko haha
16:04:39 dtantsur o/
16:04:52 dtantsur I'm here for a short time since I'm still on a crazy post-relocation schedule
16:05:37 mordred dtantsur: I have missed things - where did you relocation?
16:05:50 dtantsur mordred: Düsseldorf (west Germany)
16:06:03 mordred dtantsur: oh wow

Earlier   Later