| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-10-09 | |||
| 14:34:57 | Uggla | @stephenfin, oh ok cool ! | |
| 14:35:41 | Uggla | stephenfin, tbh I did not know the purpose of tips jobs until now. :) | |
| 14:35:52 | opendevreview | Tobias Urdin proposed openstack/ansible-collections-openstack master: Add Neutron trunk module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923025 | |
| 15:05:29 | opendevreview | Artem Goncharov proposed openstack/openapi master: Initial import of the code https://review.opendev.org/c/openstack/openapi/+/931509 | |
| 15:11:20 | opendevreview | Artem Goncharov proposed openstack/openapi master: Initial import of the code https://review.opendev.org/c/openstack/openapi/+/931509 | |
| 15:21:29 | opendevreview | Artem Goncharov proposed openstack/openapi master: Initial import of the code https://review.opendev.org/c/openstack/openapi/+/931509 | |
| 17:04:32 | opendevreview | Artem Goncharov proposed openstack/openapi master: Initial import of the code https://review.opendev.org/c/openstack/openapi/+/931509 | |
| 18:45:45 | opendevreview | Artem Goncharov proposed openstack/openapi master: Initial import of the code https://review.opendev.org/c/openstack/openapi/+/931509 | |
| 19:13:41 | opendevreview | Merged openstack/openstacksdk master: Avoid race condition between rule removal and share deletion in tearDown https://review.opendev.org/c/openstack/openstacksdk/+/931895 | |
| 19:14:56 | opendevreview | Merged openstack/openapi master: Initial import of the code https://review.opendev.org/c/openstack/openapi/+/931509 | |
| 20:53:50 | opendevreview | Takashi Kajinami proposed openstack/ansible-collections-openstack master: Drop compat implementations for tests https://review.opendev.org/c/openstack/ansible-collections-openstack/+/907515 | |
| 20:54:02 | opendevreview | Pierre Riteau proposed openstack/ansible-collections-openstack master: Fix typo in parameter description https://review.opendev.org/c/openstack/ansible-collections-openstack/+/930221 | |
| 20:54:32 | 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 | |
| 20:54:35 | opendevreview | Yuki Hirano proposed openstack/ansible-collections-openstack master: Fix openstack.cloud.port module failure in check mode https://review.opendev.org/c/openstack/ansible-collections-openstack/+/927569 | |
| 20:54:38 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 20:54:41 | opendevreview | Tom Clark proposed openstack/ansible-collections-openstack master: Add loadbalancer quota options https://review.opendev.org/c/openstack/ansible-collections-openstack/+/916805 | |
| 20:54:44 | opendevreview | Maximilian Sesterhenn proposed openstack/ansible-collections-openstack master: Add address_group, address_scope and subnetpool to Neutron RBAC https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923839 | |
| 20:55:44 | opendevreview | James Denton proposed openstack/ansible-collections-openstack master: Allow wait: false when auto_ip is false https://review.opendev.org/c/openstack/ansible-collections-openstack/+/905308 | |
| 20:55:55 | opendevreview | Kevin Honka proposed openstack/ansible-collections-openstack master: add an option to use only IPv4 only for ansible_host and ansible_ssh_host https://review.opendev.org/c/openstack/ansible-collections-openstack/+/912715 | |
| 20:56:10 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Ensure coe_cluster_template compare labels properly https://review.opendev.org/c/openstack/ansible-collections-openstack/+/882089 | |
| 21:46:08 | opendevreview | Merged openstack/ansible-collections-openstack master: Add Neutron trunk module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923025 | |
| #openstack-sdks - 2024-10-10 | |||
| 03:03:36 | opendevreview | Ghanshyam proposed openstack/python-openstackclient master: Add cross services jobs in osc gate https://review.opendev.org/c/openstack/python-openstackclient/+/931858 | |
| 08:16:25 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Remove default value for interface https://review.opendev.org/c/openstack/ansible-collections-openstack/+/889729 | |
| 08:48:39 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Fix volume backup restore response https://review.opendev.org/c/openstack/openstacksdk/+/931755 | |
| 09:20:43 | opendevreview | Simon Hensel proposed openstack/ansible-collections-openstack master: Allow to specify multiple allocation pools when creating a subnet https://review.opendev.org/c/openstack/ansible-collections-openstack/+/912775 | |
| 09:53:00 | opendevreview | Artem Goncharov proposed openstack/os-api-ref master: [WIP] Move to bootstrapv5 https://review.opendev.org/c/openstack/os-api-ref/+/932048 | |
| 10:41:29 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Fix: Volume backup restore output https://review.opendev.org/c/openstack/python-openstackclient/+/931756 | |
| 11:35:29 | opendevreview | LEE GEUM YOUNG proposed openstack/openstacksdk master: Add TSIG key support for DNS zones in SDK https://review.opendev.org/c/openstack/openstacksdk/+/930980 | |
| 13:36:28 | opendevreview | Artem Goncharov proposed openstack/os-api-ref master: [WIP] Move to bootstrapv5 https://review.opendev.org/c/openstack/os-api-ref/+/932048 | |
| 14:21:41 | opendevreview | Artem Goncharov proposed openstack/os-api-ref master: [WIP] Move to bootstrapv5 https://review.opendev.org/c/openstack/os-api-ref/+/932048 | |
| 15:03:21 | opendevreview | Artem Goncharov proposed openstack/os-api-ref master: Series of job fixes https://review.opendev.org/c/openstack/os-api-ref/+/916582 | |
| 16:25:41 | opendevreview | Merged openstack/os-api-ref master: Series of job fixes https://review.opendev.org/c/openstack/os-api-ref/+/916582 | |
| 16:27:56 | opendevreview | Artem Goncharov proposed openstack/os-api-ref master: [WIP] Move to bootstrapv5 https://review.opendev.org/c/openstack/os-api-ref/+/932048 | |
| 16:41:25 | opendevreview | Artem Goncharov proposed openstack/os-api-ref master: [WIP] Move to bootstrapv5 https://review.opendev.org/c/openstack/os-api-ref/+/932048 | |
| 17:03:16 | opendevreview | Merged openstack/ansible-collections-openstack master: Ensure coe_cluster_template compare labels properly https://review.opendev.org/c/openstack/ansible-collections-openstack/+/882089 | |
| 19:02:56 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 19:11:05 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 21:20:15 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix typo in parameter description https://review.opendev.org/c/openstack/ansible-collections-openstack/+/930221 | |
| #openstack-sdks - 2024-10-11 | |||
| 06:56:52 | opendevreview | Artem Goncharov proposed openstack/os-api-ref master: [WIP] Move to bootstrapv5 https://review.opendev.org/c/openstack/os-api-ref/+/932048 | |
| 07:19:56 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Remove default value for interface https://review.opendev.org/c/openstack/ansible-collections-openstack/+/889729 | |
| 07:55:53 | opendevreview | Artem Goncharov proposed openstack/os-api-ref master: Move to bootstrap5 https://review.opendev.org/c/openstack/os-api-ref/+/932048 | |
| 08:04:12 | opendevreview | Artem Goncharov proposed openstack/openapi master: Add tips job to see rendering with tip openstackdcstheme https://review.opendev.org/c/openstack/openapi/+/932166 | |
| 08:20:30 | opendevreview | Artem Goncharov proposed openstack/openapi master: Add tips job to see rendering with tip openstackdocstheme https://review.opendev.org/c/openstack/openapi/+/932166 | |
| 16:20:02 | opendevreview | Artem Goncharov proposed openstack/openapi master: Add link to the spec file https://review.opendev.org/c/openstack/openapi/+/932210 | |
| 16:22:49 | opendevreview | Artem Goncharov proposed openstack/openapi master: Add tips job to see rendering with tip openstackdocstheme https://review.opendev.org/c/openstack/openapi/+/932166 | |
| #openstack-sdks - 2024-10-12 | |||
| 12:34:35 | opendevreview | Takashi Natsume proposed openstack/python-openstackclient master: Replace deprecated datetime.utcnow() https://review.opendev.org/c/openstack/python-openstackclient/+/930810 | |
| #openstack-sdks - 2024-10-13 | |||
| 06:02:16 | opendevreview | LEE GEUM YOUNG proposed openstack/openstacksdk master: Add TSIG key support for DNS zones in SDK https://review.opendev.org/c/openstack/openstacksdk/+/930980 | |
| #openstack-sdks - 2024-10-14 | |||
| 10:14:38 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add method to create network resource tags https://review.opendev.org/c/openstack/openstacksdk/+/927779 | |
| 13:01:53 | opendevreview | yosef proposed openstack/python-openstackclient master: remove project from network flavor profile https://review.opendev.org/c/openstack/python-openstackclient/+/932328 | |
| 13:05:48 | opendevreview | yosef proposed openstack/python-openstackclient master: remove project from network flavor profile https://review.opendev.org/c/openstack/python-openstackclient/+/932328 | |
| 14:11:23 | opendevreview | Niklas Schwarz proposed openstack/osc-lib master: Prevent calls to get resources when no valid uuid provided https://review.opendev.org/c/openstack/osc-lib/+/932336 | |
| 16:23:57 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: WIP: identity: Migrate 'trust' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/931694 | |
| #openstack-sdks - 2024-10-15 | |||
| 04:23:46 | opendevreview | Jay Jahns proposed openstack/ansible-collections-openstack master: Enable glance-direct interop image import https://review.opendev.org/c/openstack/ansible-collections-openstack/+/932372 | |
| 05:29:50 | opendevreview | Jay Jahns proposed openstack/ansible-collections-openstack master: Enable glance-direct interop image import https://review.opendev.org/c/openstack/ansible-collections-openstack/+/932372 | |
| 06:47:04 | whoami-rajat | stephenfin, hey, can you take another look at this patch? the dependent patches have merged in all stable branches https://review.opendev.org/c/openstack/openstacksdk/+/920464 | |
| 06:53:08 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add method to create network resource tags https://review.opendev.org/c/openstack/openstacksdk/+/927779 | |
| 07:59:51 | opendevreview | yosef proposed openstack/python-openstackclient master: remove project from network flavor profile https://review.opendev.org/c/openstack/python-openstackclient/+/932328 | |
| 12:13:18 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932400 | |
| 12:13:57 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/2024.2: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932401 | |
| 12:14:20 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/2024.1: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932402 | |
| 12:14:43 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/2023.2: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932403 | |
| 12:17:38 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/2023.1: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932404 | |
| 14:23:07 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add method to create network resource tags https://review.opendev.org/c/openstack/openstacksdk/+/927779 | |
| 15:10:58 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 15:16:45 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 16:33:27 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate `region` commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/930937 | |
| 17:08:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: clientmanager: Check for 'block-storage' service type https://review.opendev.org/c/openstack/python-openstackclient/+/932435 | |
| 17:39:36 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'trust' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/931694 | |
| 19:45:22 | opendevreview | Merged openstack/python-openstackclient master: clientmanager: Check for 'block-storage' service type https://review.opendev.org/c/openstack/python-openstackclient/+/932435 | |
| 21:30:48 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| #openstack-sdks - 2024-10-16 | |||
| 06:50:17 | opendevreview | yosef proposed openstack/python-openstackclient master: remove project from network flavor profile https://review.opendev.org/c/openstack/python-openstackclient/+/932328 | |
| 07:10:52 | 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 | |
| 13:07:55 | opendevreview | Takashi Natsume proposed openstack/python-openstackclient master: Replace deprecated datetime.utcnow() https://review.opendev.org/c/openstack/python-openstackclient/+/930810 | |
| #openstack-sdks - 2024-10-17 | |||
| 09:01:55 | ralonsoh | hello folks, if you have some minutes, please check https://review.opendev.org/c/openstack/openstacksdk/+/927779 | |
| 09:01:59 | ralonsoh | thanks in advance! | |
| 12:33:54 | tkajinam | so this is likely the first problem we see after volumev3 service type was removed from devstack https://bugs.launchpad.net/horizon/+bug/2084794 | |
| 12:34:14 | tkajinam | fyi . I'm fixing it now. | |
| 14:26:25 | opendevreview | Tobias Urdin proposed openstack/python-openstackclient master: Show Created At column for volume backups in v3 https://review.opendev.org/c/openstack/python-openstackclient/+/932618 | |
| 16:22:31 | stephenfin | tkajinam: Thanks. Also hit it in OSC. I fixed that earlier this week https://github.com/openstack/python-openstackclient/commit/7c6b47b451f30d1d3965358c515baae87955d7dc | |
| 16:23:22 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/2024.2: clientmanager: Check for 'block-storage' service type https://review.opendev.org/c/openstack/python-openstackclient/+/932640 | |
| 18:09:49 | opendevreview | Merged openstack/python-openstackclient stable/2024.2: clientmanager: Check for 'block-storage' service type https://review.opendev.org/c/openstack/python-openstackclient/+/932640 | |
| #openstack-sdks - 2024-10-18 | |||
| 05:41:43 | ralonsoh | stephenfin, hello! if you have some minutes: https://review.opendev.org/c/openstack/openstacksdk/+/927779 | |
| 05:41:48 | ralonsoh | thanks in advance! | |
| 09:02:42 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add ``trunk_details`` to ``ports`` resource https://review.opendev.org/c/openstack/openstacksdk/+/926609 | |
| 09:58:57 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add the trunk subports information to the port list command https://review.opendev.org/c/openstack/python-openstackclient/+/926611 | |
| 10:07:20 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add the trunk subports information to the port list command https://review.opendev.org/c/openstack/python-openstackclient/+/926611 | |
| 11:27:49 | stephenfin | ralonsoh: I'm slightly concerned that we now have two similarly named methods that will only work with some services | |
| 11:28:15 | stephenfin | How come neutron is using POST instead of PUT? Is PUT an option? | |
| 11:28:32 | stephenfin | And what happens if you POST multiple times? Does that result in it being overridden or...? | |
| 11:38:10 | ralonsoh | stephenfin, PUT was the initial implementation. Was used to add a single tag to a resource. But there are some problems with the tag names, for example if you use slash / | |
| 11:38:41 | ralonsoh | post multiple times: that is tested in the patch. This is idempotent | |
| 11:38:59 | ralonsoh | you can post several times the same tag but it will be created once only | |
| 11:39:29 | ralonsoh | POST is the new method and in the CLI it will replace PUT | |
| 11:40:23 | stephenfin | I thought the issue was that we were passing the parameters in the URI rather than in the body? | |
| 11:40:29 | ralonsoh | yes | |
| 11:40:30 | stephenfin | (btw, I realize I'm straying into Neutron API design rather than openstacksdk stuff here, but I'd like to understand this so please bear with me :)) | |
| 11:40:44 | stephenfin | Okay, how come we didn't just...start passing it in the body? | |
| 11:40:54 | stephenfin | iiuc, that's what nova et al do? | |
| 11:40:55 | ralonsoh | that is the main reason: not to add the tag name in the URL | |
| 11:41:15 | stephenfin | https://docs.openstack.org/api-ref/compute/#replace-tags | |