| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-12-05 | |||
| 15:47:50 | stephenfin | zigo: yes please | |
| 15:50:08 | croelandt | Hello! I see upload_image() is marked as deprecated in the SDK and that I should switch to using "create_image". I'm not sure I can use create_image to upload data to an existing image though | |
| 15:50:21 | croelandt | Also I could use the import workflow, but I'd like for the data upload to work in an env where that is not configured | |
| 15:50:24 | croelandt | Any idea? | |
| 15:54:16 | stephenfin | croelandt: the issue with that method was that it was doing too much: it should have only been handling the upload step of the legacy flow but it was instead doing create (which afaik is also part of the image import flow) + upload | |
| 15:54:36 | stephenfin | mrjoshi was working on it in the past but never got to the bottom of it https://review.opendev.org/c/openstack/openstacksdk/+/881939 | |
| 15:55:21 | stephenfin | I still think that's the correct path to take but someone needs to pick it up again | |
| 15:56:54 | croelandt | OK, I see she's answered comments by Artem, Maxim and yourself and then uploaded a patchset that passes the CI. Do you know whether there is something specific to change from patchset #21? | |
| 15:58:49 | stephenfin | croelandt: It's a really hard patch to review, and I struggle each time I've to try. I think it needs to be broken up to reviewable and mergeable, if I'm being honest :( | |
| 15:59:01 | stephenfin | For example, this could be a separate patch https://review.opendev.org/c/openstack/openstacksdk/+/881939/21/openstack/image/v2/image.py | |
| 16:00:29 | stephenfin | I also think we'd benefit from a precursor patch that tests calling upload_image with arguments in various forms | |
| 16:00:37 | stephenfin | e.g. upload_image('ami', None) | |
| 16:00:41 | stephenfin | upload_image('ami', disk_format=None) | |
| 16:00:58 | stephenfin | upload_image(container_format='ami', disk_format=None) | |
| 16:01:24 | stephenfin | etc. etc. That would give us confidence that the code to handle the old deprecated signature works as intended and we don't break users | |
| 16:03:39 | stephenfin | croelandt: You and I have reason to be cautious around this code too: we have made mistakes here in the recent past 😅 https://review.opendev.org/c/openstack/openstacksdk/+/910218 https://review.opendev.org/c/openstack/openstacksdk/+/960136 | |
| 16:04:39 | croelandt | yes, this is all fairly complex | |
| 16:05:15 | croelandt | but ok now I got an idea of what is needed to move forward | |
| 16:05:18 | croelandt | great, thanks! | |
| 16:06:09 | stephenfin | nw, let me know when it's good to go and I'll give it a run through 🫡 | |
| 16:10:48 | croelandt | stephenfin: so (provided I can make this work) we'd be undeprecating upload_image? | |
| 16:10:58 | stephenfin | yes | |
| 16:11:45 | stephenfin | but it would be refocused purely on the legacy image upload flow | |
| 16:12:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Implement conflict resolution https://review.opendev.org/c/openstack/python-openstackclient/+/968178 | |
| 16:13:23 | stephenfin | fwiw I'd also like to do the same in `create_image` but I think that ship has sailed... | |
| 16:14:39 | croelandt | does create_image do two things? | |
| 16:14:53 | stephenfin | create_image does about 5 things | |
| 16:15:06 | stephenfin | 😅 | |
| 16:15:07 | croelandt | oh yeah you can have create_image(..., data='stuff') so it creates the image and uploads data to it | |
| 16:15:10 | stephenfin | yup | |
| 16:15:36 | stephenfin | I get why it does that but I'd much rather people manually chose between the two flows | |
| 16:17:51 | croelandt | indeed | |
| 16:29:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Deprecate rewriting of is_public property https://review.opendev.org/c/openstack/openstacksdk/+/969964 | |
| 16:29:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Move some code around https://review.opendev.org/c/openstack/openstacksdk/+/969965 | |
| 19:30:24 | opendevreview | cid proposed openstack/openstacksdk master: Add Node.instance_name support https://review.opendev.org/c/openstack/openstacksdk/+/955103 | |
| 21:09:10 | opendevreview | cid proposed openstack/openstacksdk master: Add Node.instance_name support https://review.opendev.org/c/openstack/openstacksdk/+/955103 | |
| #openstack-sdks - 2025-12-08 | |||
| 07:53:33 | opendevreview | Thomas Goirand proposed openstack/openstacksdk master: Fix Python 3.14 annotation https://review.opendev.org/c/openstack/openstacksdk/+/970100 | |
| 07:57:40 | opendevreview | Thomas Goirand proposed openstack/openstacksdk master: Python 3.14: fix timeout wait https://review.opendev.org/c/openstack/openstacksdk/+/970101 | |
| 10:50:56 | opendevreview | Thomas Goirand proposed openstack/openstacksdk master: Python 3.14: fix timeout wait https://review.opendev.org/c/openstack/openstacksdk/+/970101 | |
| 11:21:14 | opendevreview | Thomas Goirand proposed openstack/openstacksdk master: Python 3.14: fix timeout wait https://review.opendev.org/c/openstack/openstacksdk/+/970101 | |
| 12:07:23 | rpittau | hi all! we've removed/renamed some jobs from bifrost, please look at https://review.opendev.org/c/openstack/openstacksdk/+/969079 to avoid any disruption, thanks! | |
| 15:03:55 | opendevreview | Andriy Kurilin proposed openstack/openstacksdk master: Define tenant_id as a server-side filter for Port resource https://review.opendev.org/c/openstack/openstacksdk/+/970135 | |
| 16:20:49 | croelandt | stephenfin: haven't looked into improving the image upload support in the SDK, but I did rework the md-tags quite heavily (https://review.opendev.org/c/openstack/openstacksdk/+/897276), I could use your opinion on that | |
| 17:41:47 | stephenfin | rpittau: thanks, done | |
| 17:42:08 | stephenfin | croelandt: ack, will take a look shortly | |
| 18:08:02 | croelandt | stephenfin: thanks! I'm not too happy about reimplementing some methods but it'll have to do | |
| 18:22:27 | opendevreview | Miro Tomaska proposed openstack/python-openstackclient master: Moving tapas osc client code from neutronclient https://review.opendev.org/c/openstack/python-openstackclient/+/963445 | |
| 18:24:01 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Implement conflict resolution https://review.opendev.org/c/openstack/python-openstackclient/+/968178 | |
| 18:24:33 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Implement conflict resolution https://review.opendev.org/c/openstack/python-openstackclient/+/968178 | |
| 18:40:35 | opendevreview | Miro Tomaska proposed openstack/python-openstackclient master: Improve help strings for tap services https://review.opendev.org/c/openstack/python-openstackclient/+/968546 | |
| 19:00:45 | opendevreview | Miro Tomaska proposed openstack/python-openstackclient master: Improve help strings for tap services https://review.opendev.org/c/openstack/python-openstackclient/+/968546 | |
| 19:11:14 | opendevreview | Merged openstack/openstacksdk master: Update name of bifrost CI job https://review.opendev.org/c/openstack/openstacksdk/+/969079 | |
| 19:11:46 | opendevreview | Merged openstack/openstacksdk master: Fix Python 3.14 annotation https://review.opendev.org/c/openstack/openstacksdk/+/970100 | |
| 19:26:04 | opendevreview | Merged openstack/openstacksdk master: Add SDK support for adding/removing metadef tags https://review.opendev.org/c/openstack/openstacksdk/+/897276 | |
| 23:35:08 | M0weng[m] | stephenfin: Saw this in the python-openstackclient bugs (while looking for something else), looks related to the case sensitivity stuff - https://bugs.launchpad.net/python-openstackclient/+bug/2131938 | |
| 23:35:48 | M0weng[m] | I still haven't gotten to test on my own system after messing up my devstack while trying to set up magnum 😓 | |
| #openstack-sdks - 2025-12-09 | |||
| 00:01:58 | M0weng[m] | I forgot, I have an old backup devstack VM that should work fine for this purpose. In terms of testing python-openstackclient with and without a patch, where on my devstack machine would I be able to apply the patch? I.e. where are the python-openstackclient files located? (Or maybe it works differently... I'm only going off of experience patching the keystone files in /opt/stack/keystone) | |
| 07:34:34 | frickler | M0weng[m]: a standard devstack install comes with OSC installed from pypi. what I'd suggest to do is: git clone it to /opt/stack, checkout the branch or tag you want to test, run "tox -e venv" to install it, run e.g. ".tox/venv/bin/openstack user show Demo" | |
| 07:35:20 | frickler | the above works for 6.6.1, so the above bug is confirmed by that. not sure whether it is intentional or not, though, maybe gtema knows more | |
| 10:18:10 | opendevreview | Andriy Kurilin proposed openstack/openstacksdk master: Define tenant_id as a server-side filter for Port resource https://review.opendev.org/c/openstack/openstacksdk/+/970135 | |
| 22:17:36 | M0weng[m] | Thanks frickler ! | |
| 23:15:14 | opendevreview | Doug Goldstein proposed openstack/keystoneauth master: Add v3websso OpenID Connect Web SSO authentication plugin https://review.opendev.org/c/openstack/keystoneauth/+/970328 | |
| #openstack-sdks - 2025-12-10 | |||
| 09:20:07 | opendevreview | Thomas Goirand proposed openstack/openstacksdk master: Python 3.14: fix timeout wait https://review.opendev.org/c/openstack/openstacksdk/+/970101 | |
| 12:42:30 | opendevreview | Merged openstack/openstacksdk master: Define tenant_id as a server-side filter for Port resource https://review.opendev.org/c/openstack/openstacksdk/+/970135 | |
| 14:34:42 | opendevreview | Doug Goldstein proposed openstack/keystoneauth master: Add v3websso OpenID Connect Web SSO authentication plugin https://review.opendev.org/c/openstack/keystoneauth/+/970328 | |
| 14:38:07 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: uff: Enable E5 check https://review.opendev.org/c/openstack/keystoneauth/+/970399 | |
| 15:25:23 | opendevreview | Thomas Goirand proposed openstack/openstacksdk master: Python 3.14: fix timeout wait https://review.opendev.org/c/openstack/openstacksdk/+/970101 | |
| 15:56:50 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Fix help for cloud.port_create https://review.opendev.org/c/openstack/openstacksdk/+/970413 | |
| 16:16:37 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: ruff: Enable E5 check https://review.opendev.org/c/openstack/keystoneauth/+/970399 | |
| 16:29:11 | opendevreview | Takashi Kajinami proposed openstack/osc-lib master: ruff: Enable E5 check https://review.opendev.org/c/openstack/osc-lib/+/970422 | |
| 16:35:55 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: ruff: Enable E5 check https://review.opendev.org/c/openstack/keystoneauth/+/970399 | |
| 16:41:14 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Use consistent name for logger instance https://review.opendev.org/c/openstack/keystoneauth/+/970427 | |
| 16:52:47 | opendevreview | Merged openstack/keystoneauth master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/keystoneauth/+/959452 | |
| 16:52:49 | opendevreview | Merged openstack/keystoneauth master: reno: Update master for unmaintained/2024.1 https://review.opendev.org/c/openstack/keystoneauth/+/965771 | |
| 17:07:46 | opendevreview | Takashi Kajinami proposed openstack/os-client-config master: ruff: Enable E5 check https://review.opendev.org/c/openstack/os-client-config/+/970444 | |
| 17:09:20 | opendevreview | Takashi Kajinami proposed openstack/os-service-types master: ruff: Enable E5 check https://review.opendev.org/c/openstack/os-service-types/+/970447 | |
| 17:17:00 | opendevreview | Stephen Finucane proposed openstack/os-client-config master: ruff: Enable E5 check https://review.opendev.org/c/openstack/os-client-config/+/970444 | |
| 17:42:17 | opendevreview | Merged openstack/os-service-types master: ruff: Enable E5 check https://review.opendev.org/c/openstack/os-service-types/+/970447 | |
| 17:44:33 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: ruff: Enable E5 check https://review.opendev.org/c/openstack/python-openstackclient/+/970452 | |
| 18:15:11 | opendevreview | Austin Jamias proposed openstack/ansible-collections-openstack master: Add port_forwarding modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/970456 | |
| 18:23:48 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: ruff: Enable S checks https://review.opendev.org/c/openstack/keystoneauth/+/970458 | |
| 18:23:49 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: docs: Update note on v2 API https://review.opendev.org/c/openstack/keystoneauth/+/970459 | |
| 18:23:50 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Simplify mypy configuration https://review.opendev.org/c/openstack/keystoneauth/+/970460 | |
| 18:23:52 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Run mypy from tox https://review.opendev.org/c/openstack/keystoneauth/+/970461 | |
| 18:23:54 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: WIP: typing: Add hints to fixtures https://review.opendev.org/c/openstack/keystoneauth/+/970462 | |
| 19:16:28 | opendevreview | Merged openstack/osc-lib master: ruff: Enable E5 check https://review.opendev.org/c/openstack/osc-lib/+/970422 | |
| 19:43:39 | opendevreview | Merged openstack/python-openstackclient master: ruff: Enable E5 check https://review.opendev.org/c/openstack/python-openstackclient/+/970452 | |
| 20:04:27 | opendevreview | Merged openstack/keystoneauth master: ruff: Enable E5 check https://review.opendev.org/c/openstack/keystoneauth/+/970399 | |
| 20:04:29 | opendevreview | Merged openstack/keystoneauth master: Use consistent name for logger instance https://review.opendev.org/c/openstack/keystoneauth/+/970427 | |
| 21:02:20 | opendevreview | Merged openstack/os-client-config master: ruff: Enable E5 check https://review.opendev.org/c/openstack/os-client-config/+/970444 | |
| 23:21:41 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| #openstack-sdks - 2025-12-11 | |||
| 06:39:35 | joelm21 | Howdy Stackers. I'm messing around with the openstackclient, and trying to determine whether in it's current state it works with OS_PASSCODE. From what I can tell, it's a supported parameter, but when I attempt to authenticate with an OS_PASSCODE exported, I get an error: "BaseGenericPlugin.__init__() got an unexpected keyword argument 'passcode'". Looking at the source, it looks like it doesn't know how to parse OS_PASSCODE, but I might be missi | |
| 06:40:09 | joelm21 | (Is this a convenient thing to do, having to authenticate TOTP every time? shh, it's not about convenient, it's about having fun!) | |
| 06:51:57 | joelm21 | Ah, found what I was looking for; neither authentication.rst or man/openstack.rst really have this in a clear example, but both `export OS_AUTH_TYPE="v3multifactor"` and `export OS_AUTH_METHODS="v3password,v3totp"` needs to be specified, and that list _must_ not have a space in it. then it accepts OS_PASSCODE just fine! | |
| 06:52:22 | joelm21 | Is it totally dumb? Yes. Is it awesome? Also yes. | |
| 07:03:11 | joelm21 | A wee little script to prompt for MFA, issue a token, then switch to token auth and we're off to the races. | |
| 08:21:52 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Enable logging related ruff checks https://review.opendev.org/c/openstack/keystoneauth/+/970587 | |
| 10:42:06 | opendevreview | cid proposed openstack/openstacksdk master: Add Node.instance_name support https://review.opendev.org/c/openstack/openstacksdk/+/955103 | |
| 13:15:22 | opendevreview | Stephen Finucane proposed openstack/openstackclient master: Respect conflict resolution https://review.opendev.org/c/openstack/openstackclient/+/970618 | |
| 13:15:22 | opendevreview | Stephen Finucane proposed openstack/openstackclient master: Migrate to pyproject.toml and ruff https://review.opendev.org/c/openstack/openstackclient/+/970619 | |