| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-07-25 | |||
| 09:21:16 | frickler | a possible workaround might be to push forward with the release of 6.7.0, but IMO it would be better to root-cause this anyway | |
| 09:23:37 | opendevreview | Kaloyan Kotlarski proposed openstack/ansible-collections-openstack master: Add a list of Neutron port to attach when deploying an Ironic node https://review.opendev.org/c/openstack/ansible-collections-openstack/+/924914 | |
| 10:05:51 | opendevreview | Valentin Chassignol proposed openstack/python-openstackclient master: Add retrieve password with optional decryption https://review.opendev.org/c/openstack/python-openstackclient/+/923621 | |
| 10:54:50 | gtema | fricker: ack | |
| 11:48:54 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add callback on plugin load failure https://review.opendev.org/c/openstack/python-openstackclient/+/924923 | |
| 11:49:29 | stephenfin | frickler: Do you have a local reproducer? Can you cherry-pick/manually apply that commit ^ to see if it yields any info? | |
| 11:50:18 | stephenfin | I wrote that yesterday due to issues I was seeing with osc-placement (and previously python-ironicclient), both due to use of distutils when installed under Python 3.12 | |
| 11:55:08 | frickler | ah, I was already looking at that code, but didn't know how to add debugging. you can simply rebase your patch onto 6.6.0 and get: WARNING: Failed to import plugin openstack.cli.base:object_store: No module named 'simplejson'. | |
| 11:55:55 | frickler | so the fix is https://review.opendev.org/c/openstack/python-openstackclient/+/920001 | |
| 11:56:15 | frickler | which is also what my git bisect led me to, but I couldn't find the connection | |
| 11:57:15 | stephenfin | frickler: Yeah, I was having commands disappear for seemingly no reason too, which led me to that patch (after a few other attempts at fixes in osc-lib and cliff). Good timing it seems :) | |
| 11:57:43 | frickler | so 6.7.0 will be fine and maybe we need to backport to stable branches | |
| 11:58:50 | stephenfin | Yup. I suspect this is coming about due to https://review.opendev.org/c/openstack/osc-lib/+/920000, which is included in osc-lib 3.1.0 | |
| 12:00:05 | stephenfin | (simplejson was a direct dependency that we were including transitively from osc-lib) | |
| 12:06:55 | frickler | ah, right, downgrading my broken venv to osc-lib-3.0.1 pulls in simplejson and that fixes the whole issue | |
| 12:12:46 | rpittau | so need to wait for osc 6.7.0 to land for the fix ? | |
| 12:13:43 | stephenfin | rpittau: We don't have an easy way to release 6.6.1 so yes | |
| 12:13:53 | frickler | another workaround is to install simplejson into the venv | |
| 12:14:05 | frickler | we could do that in devstack for now, I guess | |
| 12:14:13 | rpittau | right | |
| 12:14:28 | rpittau | that workaround should work | |
| 12:15:23 | frickler | yes, I just confirmed it in my local setup | |
| 12:15:31 | rpittau | great | |
| 12:17:08 | frickler | stephenfin: we can do a 6.6.1 from stable/2024.1, can't we? | |
| 12:18:00 | stephenfin | oh, yes we can. I had thought 6.6.6 was a D release but apparently not | |
| 12:18:15 | stephenfin | do you want to do that backport and I can approve? | |
| 12:18:21 | frickler | rpittau: do you want to update your devstack patch with the simplejson addition? | |
| 12:18:31 | frickler | stephenfin: ack, just a sec | |
| 12:19:44 | ganso | stephenfin: hi! have you been able to think more about https://review.opendev.org/c/openstack/openstacksdk/+/922965 ? | |
| 12:20:15 | stephenfin | ganso: Oh, sorry, I thought we'd agreed that fixing Location would be nice but that we could treat it as out-of-scope for now | |
| 12:20:54 | stephenfin | ganso: fwiw I have some largeish typing-related rework under way that should make fixing it easier, but it's not complete yet and won't be complete for a while yet | |
| 12:21:34 | ganso | stephenfin: yea I thought we did agree but the gerrit comment history seemed to indicate otherwise. I'm happy if we are ready to move forward | |
| 12:21:40 | opendevreview | Jens Harbott proposed openstack/python-openstackclient stable/2024.1: Drop direct dependency on simplejson https://review.opendev.org/c/openstack/python-openstackclient/+/924927 | |
| 12:22:09 | ganso | stephenfin: hmm if it is a large rework shouldn't we merge that small patch now so the customers that need it don't have to wait that much? | |
| 12:34:34 | rpittau | frickler: yeah, gimme a sec to find the proper place | |
| 12:39:05 | rpittau | frickler: done | |
| 12:54:26 | frickler | rpittau: thx, will approve once CI passes | |
| 16:22:57 | opendevreview | Merged openstack/python-openstackclient stable/2024.1: Drop direct dependency on simplejson https://review.opendev.org/c/openstack/python-openstackclient/+/924927 | |
| 19:26:21 | opendevreview | Tim Bishop proposed openstack/python-openstackclient master: Respect --skip-resource when deleting. https://review.opendev.org/c/openstack/python-openstackclient/+/924659 | |
| #openstack-sdks - 2024-07-26 | |||
| 15:58:36 | opendevreview | Merged openstack/keystoneauth master: Update requests of device flow for public client https://review.opendev.org/c/openstack/keystoneauth/+/909561 | |
| 18:45:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Drop anyhow in cli templates https://review.opendev.org/c/openstack/codegenerator/+/925041 | |
| 19:46:53 | opendevreview | Merged openstack/codegenerator master: Drop anyhow in cli templates https://review.opendev.org/c/openstack/codegenerator/+/925041 | |
| #openstack-sdks - 2024-07-27 | |||
| 11:03:49 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 11:18:40 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 11:28:45 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 12:31:27 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 12:34:03 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 13:26:58 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 14:13:10 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 14:44:27 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 15:58:55 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 16:54:39 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 17:47:04 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 20:07:08 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| #openstack-sdks - 2024-07-28 | |||
| 06:38:03 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 07:25:24 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 08:52:47 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 14:28:58 | opendevreview | Baek Seung Ju proposed openstack/openstacksdk master: Added missing stack documentation and rewrite stack proxy document https://review.opendev.org/c/openstack/openstacksdk/+/921863 | |
| 14:32:02 | opendevreview | Baek Seung Ju proposed openstack/openstacksdk master: Implemented snapshot-related features of the stack. https://review.opendev.org/c/openstack/openstacksdk/+/921471 | |
| 14:32:19 | opendevreview | Baek Seung Ju proposed openstack/openstacksdk master: Added missing stack documentation and rewrite stack proxy document https://review.opendev.org/c/openstack/openstacksdk/+/921863 | |
| 14:59:39 | opendevreview | Baek Seung Ju proposed openstack/openstacksdk master: Implement heat 'stack resource signal' https://review.opendev.org/c/openstack/openstacksdk/+/894203 | |
| #openstack-sdks - 2024-07-29 | |||
| 05:35:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 07:13:08 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 07:42:29 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 08:44:42 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 08:59:14 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 09:41:18 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 10:04:22 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 10:29:53 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 10:51:13 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 11:28:33 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Propose changes for rust projects https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 13:16:30 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 14:17:13 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 14:18:30 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 15:22:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 16:38:56 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 17:55:03 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| #openstack-sdks - 2024-07-30 | |||
| 07:57:15 | opendevreview | Gaudenz Steinlin proposed openstack/openstacksdk master: Use 2.87 Nova microversion to fetch hypervisors https://review.opendev.org/c/openstack/openstacksdk/+/925168 | |
| 10:35:13 | stephenfin | gtema: I have a large number of purely typing-related reviews for SDK starting here https://review.opendev.org/c/openstack/openstacksdk/+/924847. Are you interested in reviewing them or will I just self-approve the lot? | |
| 10:35:47 | gtema | I'll look over | |
| 10:36:47 | stephenfin | ack | |
| 11:05:06 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: [WIP]: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 11:46:42 | frickler | gtema: stephenfin: can you decide on either https://review.opendev.org/c/openstack/openstacksdk/+/923671 or https://review.opendev.org/c/openstack/openstacksdk/+/922390 ? I'm fine with either choice but I really hate to regularly see failing, non-voting jobs being run in gate | |
| 11:49:12 | gtema | fricker: ack, looking | |
| 11:50:22 | gtema | You are right, non voting in gate is pointless | |
| 11:58:53 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: [WIP]: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 12:01:01 | opendevreview | Merged openstack/openstacksdk master: Added missing stack documentation and rewrite stack proxy document https://review.opendev.org/c/openstack/openstacksdk/+/921863 | |
| 12:11:54 | opendevreview | Merged openstack/openstacksdk master: cloud: Replace reference to removed variable https://review.opendev.org/c/openstack/openstacksdk/+/924847 | |
| 12:11:56 | opendevreview | Merged openstack/openstacksdk master: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/923221 | |
| 12:13:06 | opendevreview | Merged openstack/openstacksdk master: cloud: Remove check for nova extensions https://review.opendev.org/c/openstack/openstacksdk/+/923213 | |
| 12:40:30 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 13:05:38 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (1/2) https://review.opendev.org/c/openstack/openstacksdk/+/923216 | |
| 13:05:39 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Misc type fixes https://review.opendev.org/c/openstack/openstacksdk/+/923218 | |
| 13:05:39 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (2/2) https://review.opendev.org/c/openstack/openstacksdk/+/923217 | |
| 13:05:40 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable checks for openstack.cloud https://review.opendev.org/c/openstack/openstacksdk/+/923219 | |
| 13:05:40 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Combine networking-related classes https://review.opendev.org/c/openstack/openstacksdk/+/923220 | |
| 13:05:41 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable checks for openstack.tests.functional https://review.opendev.org/c/openstack/openstacksdk/+/923224 | |
| 13:05:41 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Remove TestClustering functional tests https://review.opendev.org/c/openstack/openstacksdk/+/923223 | |
| 13:05:42 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: mypy: Enable stricter type checking https://review.opendev.org/c/openstack/openstacksdk/+/925199 | |
| 13:05:42 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable warn_return_any https://review.opendev.org/c/openstack/openstacksdk/+/923226 | |