| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-06-09 | |||
| 13:58:15 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.config.cloud_config https://review.opendev.org/c/openstack/openstacksdk/+/950847 | |
| 13:58:15 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.config.loader https://review.opendev.org/c/openstack/openstacksdk/+/950899 | |
| 13:58:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.config.openstackcloud https://review.opendev.org/c/openstack/openstacksdk/+/950900 | |
| 13:58:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Handle interface list in CloudRegion.get_all_version_data https://review.opendev.org/c/openstack/openstacksdk/+/951878 | |
| 13:58:17 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/952133 | |
| 18:37:15 | opendevreview | Doug Goldstein proposed openstack/openstacksdk master: expose the l2_adjacency flag of networks from neutron https://review.opendev.org/c/openstack/openstacksdk/+/952158 | |
| 21:54:35 | cardoe | So merging into the SDK is broken cause of some warnings on block_storage. The gate doesn’t allow warnings. | |
| #openstack-sdks - 2025-06-10 | |||
| 07:57:46 | opendevreview | Rajesh Tailor proposed openstack/python-openstackclient master: Add support for image properties in server show https://review.opendev.org/c/openstack/python-openstackclient/+/940799 | |
| 09:24:44 | stephenfin | cardoe: which gate? | |
| 09:26:57 | stephenfin | cardoe: If you're talking about https://review.opendev.org/c/openstack/openstacksdk/+/952158, I don't think that's the issue. The issue is that you currently need to do a little dance if adding new fields to SDK | |
| 09:27:02 | stephenfin | cardoe: See https://meetings.opendev.org/irclogs/%23openstack-sdks/%23openstack-sdks.2025-05-21.log.html#openstack-sdks.2025-05-21.log.html#t2025-05-21T12:17:09 for more | |
| 09:27:41 | stephenfin | We are aware of the issue. We just don't have any good solutions don't involve a lot of work | |
| 13:17:40 | cardoe | stephenfin: no I was talking about https://review.opendev.org/c/openstack/openstacksdk/+/948552?usp=search | |
| 13:20:00 | stephenfin | I've seen that issue pop up a few times recently but it's not happening regularly and I can't reproduce it :/ https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-docs&project=openstack/openstacksdk | |
| 13:23:24 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Fix typo https://review.opendev.org/c/openstack/openstacksdk/+/952250 | |
| 13:23:38 | stephenfin | cardoe: nvm, that was easy ^ | |
| 13:25:07 | cardoe | okay cool. sorry I didn't dig in. on my ever growing list. I feel like I fix one thing and 3 more things are on it. | |
| 13:25:21 | cardoe | Trying to not forget to ask some questions during the neutron meeting :-D | |
| 13:26:05 | cardoe | But thank you for the instructions on how to add that field. I'll work on it this week,. | |
| 13:27:09 | frickler | oh, nice catch. but how does that only sometimes break the doc build? | |
| 14:08:02 | opendevreview | Merged openstack/openstacksdk master: docs: Fix typo https://review.opendev.org/c/openstack/openstacksdk/+/952250 | |
| 15:30:48 | stephenfin | frickler: not a clue but I'm guessing it's a race | |
| 15:31:05 | stephenfin | (likely due to us using parallel builds) | |
| 17:27:29 | opendevreview | Merged openstack/openstacksdk master: Follow-up: Add support for runbooks https://review.opendev.org/c/openstack/openstacksdk/+/948552 | |
| 18:40:08 | opendevreview | Merged openstack/openstacksdk master: typing: Annotate openstack.config.loader https://review.opendev.org/c/openstack/openstacksdk/+/950899 | |
| 18:46:47 | opendevreview | Merged openstack/openstacksdk master: typing: Annotate openstack.config.cloud_config https://review.opendev.org/c/openstack/openstacksdk/+/950847 | |
| 19:36:24 | opendevreview | Merged openstack/openstacksdk master: config: Handle interface list in CloudRegion.get_all_version_data https://review.opendev.org/c/openstack/openstacksdk/+/951878 | |
| 19:36:27 | opendevreview | Merged openstack/openstacksdk master: typing: Annotate openstack.config.openstackcloud https://review.opendev.org/c/openstack/openstacksdk/+/950900 | |
| #openstack-sdks - 2025-06-11 | |||
| 14:56:23 | opendevreview | cid proposed openstack/openstacksdk master: Add support for inspection rules https://review.opendev.org/c/openstack/openstacksdk/+/945700 | |
| 17:04:33 | cardoe | @stephenfin I pinged ya on the change above but I'll do it on IRC as well. Any chance we can get that reviewed and landed before flamingo? That's for something Ironic got in for epoxy but really wanna drive migration from the old stuff with flamingo. | |
| #openstack-sdks - 2025-06-12 | |||
| 09:48:30 | stephenfin | cardoe: ack, will do | |
| 12:19:26 | opendevreview | Vincent Lequertier proposed openstack/openstacksdk master: Bump python version from 3.9 to 3.10 in testing docs https://review.opendev.org/c/openstack/openstacksdk/+/952445 | |
| 12:22:06 | opendevreview | Vincent Lequertier proposed openstack/openstacksdk master: Add backup export command https://review.opendev.org/c/openstack/openstacksdk/+/952446 | |
| 14:42:51 | opendevreview | Vincent Lequertier proposed openstack/openstacksdk master: Bump python version from 3.9 to 3.13 in testing docs https://review.opendev.org/c/openstack/openstacksdk/+/952445 | |
| 15:39:36 | opendevreview | Merged openstack/openstacksdk master: Bump python version from 3.9 to 3.13 in testing docs https://review.opendev.org/c/openstack/openstacksdk/+/952445 | |
| #openstack-sdks - 2025-06-17 | |||
| 06:24:27 | opendevreview | djp proposed openstack/openstacksdk master: Fix volume attachments limit https://review.opendev.org/c/openstack/openstacksdk/+/951517 | |
| 11:58:15 | opendevreview | Jake Hutchinson proposed openstack/ansible-collections-openstack master: Remove MAC address requirement for node registration https://review.opendev.org/c/openstack/ansible-collections-openstack/+/952752 | |
| 19:04:15 | opendevreview | Merged openstack/openstacksdk master: Add support for inspection rules https://review.opendev.org/c/openstack/openstacksdk/+/945700 | |
| #openstack-sdks - 2025-06-18 | |||
| 04:16:48 | opendevreview | djp proposed openstack/openstacksdk master: Fix volume attachments limit https://review.opendev.org/c/openstack/openstacksdk/+/951517 | |
| 09:15:36 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Replace deprecated datetime.utcfromtimestamp https://review.opendev.org/c/openstack/python-openstackclient/+/952808 | |
| 09:16:01 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Replace deprecated datetime.utcfromtimestamp https://review.opendev.org/c/openstack/python-openstackclient/+/952808 | |
| 16:39:51 | stephenfin | gtema: If I do `volume list --limit 1`, what do you think that should do? | |
| 16:40:15 | gtema | hehe, I know what you mean | |
| 16:40:29 | stephenfin | and likewise for `volume list --limit 1000` | |
| 16:41:32 | gtema | due to that sort of "conflict" I added --max-items dedicated param in the rust cli | |
| 16:42:03 | gtema | and use limit as an API page limit, but still fetch all pages until I reach max-items | |
| 16:42:06 | stephenfin | yeah, I've been looking at https://review.opendev.org/c/openstack/openstacksdk/+/951517. idk if `limit` means "fetch all the resources in batches/pages of size N" or "fetch N resources" | |
| 16:42:18 | stephenfin | s/means/should mean/ | |
| 16:42:26 | stephenfin | good to hear it's not just me that's confused though :) | |
| 16:42:41 | gtema | we are passing it to the API as page size fetching all pages | |
| 16:43:15 | gtema | there are few other things that I explicitly decided to drop this old "convention" to cleanup the UX | |
| 16:43:33 | stephenfin | yup, which probably makes more sense from a library perspective, but not from a CLI perspective | |
| 16:45:10 | gtema | right. Keeping backward compatibility in lot of things limit us making a good UX | |
| 16:45:31 | gtema | so time to start switching to the new stuff ;-) | |
| 16:55:31 | gtema | I think now it would be good to rename limit to page_size in the rust cli explicitly to erase any confusion | |
| 17:02:24 | stephenfin | that would be a good clal | |
| 17:02:26 | stephenfin | *call | |
| 18:53:20 | opendevreview | Merged openstack/python-openstackclient master: Replace deprecated datetime.utcfromtimestamp https://review.opendev.org/c/openstack/python-openstackclient/+/952808 | |
| #openstack-sdks - 2025-06-20 | |||
| 04:13:26 | opendevreview | Jay Jahns proposed openstack/ansible-collections-openstack master: Add image import module and support web-download https://review.opendev.org/c/openstack/ansible-collections-openstack/+/952956 | |
| 12:52:52 | opendevreview | Jay Jahns proposed openstack/ansible-collections-openstack master: Add image import module and support web-download https://review.opendev.org/c/openstack/ansible-collections-openstack/+/952956 | |
| 15:48:18 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Inmprove UX clarity wrt "limit" query parameter https://review.opendev.org/c/openstack/codegenerator/+/952998 | |
| 17:22:27 | opendevreview | djp proposed openstack/openstacksdk master: Fix volume attachments limit https://review.opendev.org/c/openstack/openstacksdk/+/951517 | |
| 17:32:26 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Improve UX clarity of "limit" query parameter https://review.opendev.org/c/openstack/codegenerator/+/952998 | |
| 18:12:14 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Improve UX clarity of "limit" query parameter https://review.opendev.org/c/openstack/codegenerator/+/952998 | |
| 18:23:07 | opendevreview | Jay Jahns proposed openstack/ansible-collections-openstack master: Add image import module and support web-download https://review.opendev.org/c/openstack/ansible-collections-openstack/+/952956 | |
| #openstack-sdks - 2025-06-21 | |||
| 06:32:59 | opendevreview | jihyun huh proposed openstack/openstacksdk master: title: Add support for dns blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/953033 | |
| 06:50:40 | opendevreview | jihyun huh proposed openstack/openstacksdk master: Add support for dns blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/953033 | |
| 06:53:10 | opendevreview | jihyun huh proposed openstack/openstacksdk master: Add support for dns blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/953033 | |
| #openstack-sdks - 2025-06-23 | |||
| 07:53:00 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Un-vendor identity.group schema https://review.opendev.org/c/openstack/codegenerator/+/953077 | |
| 08:32:48 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Un-vendor identity.group schema https://review.opendev.org/c/openstack/codegenerator/+/953077 | |
| 09:23:42 | opendevreview | Arnaud Morin proposed openstack/openstacksdk master: Clean small nit comment https://review.opendev.org/c/openstack/openstacksdk/+/953079 | |
| 09:26:14 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Un-vendor identity.group schema https://review.opendev.org/c/openstack/codegenerator/+/953077 | |
| 10:02:49 | opendevreview | Merged openstack/codegenerator master: Un-vendor identity.group schema https://review.opendev.org/c/openstack/codegenerator/+/953077 | |
| 10:09:53 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Improve UX clarity of "limit" query parameter https://review.opendev.org/c/openstack/codegenerator/+/952998 | |
| 10:11:17 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Make keystone job voting again https://review.opendev.org/c/openstack/codegenerator/+/953083 | |
| 12:30:19 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Ensure schemas are not duplicated https://review.opendev.org/c/openstack/codegenerator/+/953093 | |
| 12:53:53 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Ensure schemas are not duplicated https://review.opendev.org/c/openstack/codegenerator/+/953093 | |
| 13:18:46 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Ensure schemas are not duplicated https://review.opendev.org/c/openstack/codegenerator/+/953093 | |
| 13:22:52 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Ensure schemas are not duplicated https://review.opendev.org/c/openstack/codegenerator/+/953093 | |
| 19:00:09 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Ensure schemas are not duplicated https://review.opendev.org/c/openstack/codegenerator/+/953093 | |
| #openstack-sdks - 2025-06-24 | |||
| 02:31:43 | opendevreview | Tony Breeds proposed openstack/openstacksdk master: Switch to new devstack based testing https://review.opendev.org/c/openstack/openstacksdk/+/953164 | |
| 03:45:24 | opendevreview | Tony Breeds proposed openstack/openstacksdk stable/2025.1: Switch to new devstack based testing https://review.opendev.org/c/openstack/openstacksdk/+/953170 | |
| 03:45:39 | opendevreview | Tony Breeds proposed openstack/openstacksdk stable/2024.2: Switch to new devstack based testing https://review.opendev.org/c/openstack/openstacksdk/+/953171 | |
| 03:45:53 | opendevreview | Tony Breeds proposed openstack/openstacksdk stable/2024.1: Switch to new devstack based testing https://review.opendev.org/c/openstack/openstacksdk/+/953172 | |
| 04:44:28 | opendevreview | djp proposed openstack/openstacksdk master: Fix volume attachments limit https://review.opendev.org/c/openstack/openstacksdk/+/951517 | |
| 07:44:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-masakari voting https://review.opendev.org/c/openstack/openstacksdk/+/951396 | |
| 07:44:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-ironic voting https://review.opendev.org/c/openstack/openstacksdk/+/942705 | |
| 07:44:54 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-magnum voting https://review.opendev.org/c/openstack/openstacksdk/+/942627 | |
| 07:44:54 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: zuul: Make dib-nodepool-functional-openstack-ubuntu-noble-src voting https://review.opendev.org/c/openstack/openstacksdk/+/942628 | |
| 07:47:41 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Migrate 'volume create' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/945876 | |
| 07:47:42 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Migrate 'volume list' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/945878 | |
| 07:47:42 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Migrate 'volume set', 'volume unset' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/945877 | |
| 07:47:43 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Migrate 'volume show' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/945881 | |
| 07:47:43 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Migrate 'volume migrate' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/945880 | |
| 08:46:49 | opendevreview | jihyun huh proposed openstack/openstacksdk master: Add support for dns blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/953033 | |
| 08:51:18 | opendevreview | jihyun huh proposed openstack/openstacksdk master: Add support for dns blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/953033 | |
| 09:32:46 | opendevreview | jihyun huh proposed openstack/openstacksdk master: Add support for dns blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/953033 | |
| 10:17:07 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Improve UX clarity of "limit" query parameter https://review.opendev.org/c/openstack/codegenerator/+/952998 | |
| 10:40:46 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Ensure schemas are not duplicated https://review.opendev.org/c/openstack/codegenerator/+/953093 | |
| 11:04:30 | opendevreview | jihyun huh proposed openstack/openstacksdk master: Add support for dns blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/953033 | |