| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-03-26 | |||
| 12:39:08 | sshnaidm | odyssey4me, I can just abandon mines, you have a full solution in yours | |
| 12:40:00 | openstackgerrit | Sagi Shnaidman proposed openstack/openstacksdk stable/train: Run fetch-subunit-output role conditionally https://review.opendev.org/714770 | |
| 12:40:16 | openstackgerrit | Igor Malinovskiy proposed openstack/python-openstackclient master: Add 'address_scope' type support to network rbac commands https://review.opendev.org/709124 | |
| 12:40:37 | openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstacksdk stable/stein: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715193 | |
| 12:40:40 | sshnaidm | odyssey4me, train and stein should be fine, rocky is waiting for pep8 fix | |
| 12:41:15 | openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstacksdk stable/train: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715192 | |
| 12:41:16 | openstackgerrit | Igor Malinovskiy proposed openstack/python-openstackclient master: Add 'subnetpool' type support to rbac commands https://review.opendev.org/712705 | |
| 12:41:31 | tremble | mordred, zo/ | |
| 12:41:32 | odyssey4me | sshnaidm: either way works - I've picked them already | |
| 12:41:34 | tremble | mordred, \o/ | |
| 12:42:11 | tremble | mordred, Anything I can watch to see it go through? | |
| 12:42:56 | mordred | tremble: https://review.opendev.org/#/c/715090/ | |
| 12:43:03 | mordred | although it looks like it just went through | |
| 12:44:03 | mordred | tremble: there's release jobs in https://zuul.opendev.org/t/openstack/status for the openstack/releases project that will actually publish to pypi | |
| 12:45:44 | tremble | Will the test jobs pick it up already? | |
| 12:47:26 | openstackgerrit | Sean McGinnis proposed openstack/openstacksdk master: Raise hacking to 2.x https://review.opendev.org/715155 | |
| 12:48:06 | smcginnis | tremble: If you filter on "release", there is a job that just started in teh release-post queue. | |
| 12:48:19 | smcginnis | That will tag the repo, among other things. | |
| 12:48:37 | smcginnis | Once that tagging happens, that triggers yet another job to run that will package it up and publish to pypi. | |
| 12:48:55 | smcginnis | So probably 15-20 minutes or so until it's published and available. | |
| 12:49:57 | tremble | Jobs triggering jobs triggering jobs. That looks like the kind of crazy I'd set up. | |
| 12:51:08 | smcginnis | ;) | |
| 12:58:48 | sshnaidm | mordred, can we merge? https://review.opendev.org/#/c/714771/ odyssey4me set his backport on top of that | |
| 13:00:08 | mordred | sshnaidm: done | |
| 13:00:29 | sshnaidm | mordred, thanks, odyssey4me ^^ | |
| 13:00:41 | odyssey4me | np :) | |
| 13:05:13 | openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512 | |
| 13:15:01 | openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_mapping https://review.opendev.org/713643 | |
| 13:15:02 | openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686 | |
| 13:15:03 | openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632 | |
| 13:15:04 | openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633 | |
| 13:15:05 | openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991 | |
| 13:15:06 | openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992 | |
| 13:28:37 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add link to image sharing api docs https://review.opendev.org/677266 | |
| 13:47:18 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Add libc6-dev to bindep https://review.opendev.org/715210 | |
| 13:50:14 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Be explicit about python version in image https://review.opendev.org/715211 | |
| 13:50:50 | mordred | amoralej|lunch: ^^ the libc6-dev patch should fix the https://review.opendev.org/#/c/697444/ patch | |
| 13:51:06 | mordred | gah | |
| 13:51:13 | mordred | amotoki: ^^ | |
| 13:51:24 | mordred | amoralej|lunch: sorry - bad tab-complete | |
| 13:51:41 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444 | |
| 14:28:51 | gtema | nodepool builds seem to be failing everywhere in SDK | |
| 14:29:01 | gtema | I mean nodepool jobs | |
| 14:30:17 | mordred | there's an issue in setuptools | |
| 14:30:22 | mordred | that has been fixed | |
| 14:30:25 | gtema | ah, ok | |
| 14:30:42 | mordred | but it's gonna be a minute before it winds its way through the layers of things to dib | |
| 14:30:56 | gtema | oki, keep waiting ;) | |
| 14:30:59 | mordred | (the bug caused non-entrypoint scripts to lose their executable bit when they were installed) | |
| 14:31:12 | gtema | cool | |
| 14:31:17 | mordred | yeah. so awesome | |
| 14:31:51 | smcginnis | Hah, oops. | |
| 14:31:53 | gtema | but from our logs you don't see this really - just that it failed | |
| 14:55:38 | mordred | gtema_: have a sec for 2 easy patches: https://review.opendev.org/#/c/715210/ https://review.opendev.org/#/c/715211/ ? | |
| 14:58:00 | gtema | yupp, looking now | |
| 14:59:30 | mordred | gtema: thanks! | |
| 14:59:45 | gtema | should I +A for the py3.7? | |
| 15:00:03 | mordred | gtema: sure. it's a pretty mechanical change | |
| 15:00:13 | gtema | done | |
| 15:00:18 | mordred | I think "let's upgrade to 3.8" would be one where we shoudl actually discuss :) | |
| 15:00:29 | gtema | agree | |
| 15:01:12 | gtema | btw this libc - today figured out, that in my zuul F30 image does not include by default bzip2 - pretty weird | |
| 15:01:34 | mordred | gtema: yeah - I think we just had a patch for that in zuul | |
| 15:02:11 | mordred | oh even weirder ... | |
| 15:02:12 | gtema | can you please point me to it? I just did a dirty hack to keep it running | |
| 15:02:16 | mordred | https://review.opendev.org/#/c/715041/ | |
| 15:02:25 | mordred | but the issue was that it was only installing for rpm | |
| 15:02:32 | mordred | but I'd expect it to be installed on F30 | |
| 15:02:54 | mordred | oh - wait ... | |
| 15:03:02 | gtema | yeah, should have been there | |
| 15:03:15 | mordred | your zuul F30 image - you mean an F30 image you're using with zuul - not that you're running zuul in | |
| 15:03:34 | gtema | right - F30 image with jobs | |
| 15:04:04 | mordred | yeah - so - in opendev we try to install as little as possible in the images so that projects can express their depends in their bindep files | |
| 15:04:34 | mordred | so if you have a job fail due to lack of bzip2, you might need to add it to the bindep of the project in question? | |
| 15:04:41 | gtema | I remember it was working always fine, but today finally reenabled building docs and that is where I noticed it | |
| 15:04:47 | gtema | exact | |
| 15:05:00 | gtema | https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/fetch-sphinx-tarball/tasks/html.yaml | |
| 15:05:00 | mordred | yeah - tox-docs is the thing that people were discussing yesterday | |
| 15:05:09 | mordred | there's a giant scrollback in the #zuul channel log about it | |
| 15:05:19 | mordred | I didn't follow the whole thing | |
| 15:05:24 | gtema | oh god, no interest for scroll | |
| 15:06:41 | mordred | gtema: https://review.opendev.org/#/c/715028 is what mnaser put together - I believe the idea is that gzip is more common | |
| 15:07:25 | gtema | yeah, this seem reasonable. I use zuul-jobs, so whenever it lands I get it as well | |
| 15:07:28 | mordred | however - if you had the bzip issue with tox-docs - it's likely that your zuul executor did not have bzip - and if you're running a zuul from the container images - they were missing bzip2 because of that bindep line. it's all a bit confusing | |
| 15:08:02 | vphilippon | Hello! I think I've stumbled on an issue with Python openstacksdk 0.44.0 dependencies. In short, it depends on futurist>=2.1.0, which is Py3-only, effectively making openstacksdk also Py3-only. | |
| 15:08:14 | gtema | exact - running zuul itself in containers | |
| 15:08:33 | mnaser | seems like everyone is hitting that heh | |
| 15:08:39 | gtema | underneath podman as non-root - so even more cool bels and whistles | |
| 15:14:22 | vphilippon | I barged in with that message, sorry. Is there a channel better suited for Python openstacksdk specific issues? | |
| 15:15:01 | mordred | vphilippon: yes - starting with this openstack release openstacksdk is python3 only | |
| 15:15:16 | mordred | vphilippon: let me find the version you should pin to if you're using py2 | |
| 15:16:12 | vphilippon | @mordred: Thanks. Although, the missing Requires-Python metadata will make a poor user experience with pip then :( | |
| 15:16:58 | mordred | vphilippon: agree - I'll get that fixed | |
| 15:17:11 | vphilippon | The only solution I know for this would be to pull v0.44.0 from pypi and republish it with the metadata | |
| 15:17:13 | mordred | vphilippon: 0.40.0 is the first release in which python2 is no longer tested/supported | |
| 15:17:38 | mordred | we can't do that - republishing a release breaks the universe. we can publish a fixed 0.45 though | |
| 15:18:00 | vphilippon | Because of 0.44.1 is released, pip will go back to 0.44.0, that is purely broken | |
| 15:18:15 | vphilippon | Because if* | |
| 15:19:18 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add requires-python entry indicating 3.6 is required https://review.opendev.org/715243 | |
| 15:19:32 | mordred | vphilippon: yeah - I think we can remove 0.44 once 0.44.1 is there | |