| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-09-16 | |||
| 18:46:29 | gtema_ | right | |
| 18:46:32 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Fix image create with tags https://review.opendev.org/682454 | |
| 18:46:47 | gtema_ | fixed | |
| 18:47:29 | gtema_ | I hoped to escape issues with followup SDK changes, since I need to raise SDK in lower-constraints as well (and would require release) | |
| 18:49:28 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 19:03:23 | gtema | otherwise remaining tests in the OSC are green, so the change is on the good track | |
| #openstack-sdks - 2019-09-17 | |||
| 00:09:17 | openstackgerrit | Merged openstack/keystoneauth master: Fix misspell word https://review.opendev.org/680600 | |
| 07:04:56 | gtema | morning, was anyone able to find the reason for devstack-networking failures? It blames octavia (argument --bind_ip: Invalid IPAddress value:), but I can't find any recent suspicious change | |
| 07:23:28 | gtema | solved, the fix already landed | |
| 07:45:44 | mordred | gtema: oh yeah? good - I was just looking at that | |
| 07:45:54 | mordred | gtema: was just a devstack issue? | |
| 07:46:05 | gtema | https://review.opendev.org/#/c/682327/ | |
| 07:48:30 | mordred | oh. hah | |
| 07:48:34 | mordred | yeah | |
| 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 | |