| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-03-21 | |||
| 11:17:22 | opendevreview | Merged openstack/keystoneauth stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/keystoneauth/+/943705 | |
| 11:57:13 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add QoS packet rate limit rule object and CRUD operations https://review.opendev.org/c/openstack/openstacksdk/+/945202 | |
| 12:06:07 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Define the correct QoS rules deletion method https://review.opendev.org/c/openstack/openstacksdk/+/945205 | |
| 12:10:10 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add QoS packet rate limit rule https://review.opendev.org/c/openstack/python-openstackclient/+/944784 | |
| 13:39:25 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Refactor loader magic fixes https://review.opendev.org/c/openstack/openstacksdk/+/944123 | |
| 14:13:15 | opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:33:58 | stephenfin | adamcarthur5: o/ I spent this morning splitting https://review.opendev.org/c/openstack/ironic/+/928921/5 up into smaller parts so the following explanation would make sense | |
| 14:35:43 | stephenfin | We want to get to the point that we can inspect controller functions, which map to a given route, and extra (a) the min/max microversion and the schemas for path, query string, request body and response body parameters | |
| 14:38:05 | stephenfin | ironic is starting from a slightly different place than other projects we've worked on (nova, cinder, keystone) since you're not doing things that are common across those, namely using jsonschema for request validation, using decorators for versioning, and using webob/routes. It's all solveable though | |
| 14:39:12 | stephenfin | So as I noted, I've broken https://review.opendev.org/c/openstack/ironic/+/928921/5 up to address those three things: | |
| 14:40:04 | stephenfin | The first moves API versioning to a decorator | |
| 14:40:06 | stephenfin | The second reworks request path, query string and body parameter validation to use JSONSchema instead of the homegrown stuff ironic has | |
| 14:40:32 | stephenfin | The third adds request body parameter validation, since that doesn't exist (at least at runtime) yet | |
| 14:42:07 | stephenfin | We will need to do those same 3 step for all APIs. Some will be more difficult than others, however, I was able to do all of the nova API (significantly larger) is down time, so it's far from insurmountable :) | |
| 14:43:13 | stephenfin | As for your other question: to the best of my knowledge, Tempest currently only has response body schemas for Nova and Cinder, and neither are comprehensive (Cinder particularly). I think these will go away once all the schemas have been added to Nova. | |
| 14:44:08 | stephenfin | for above message @ :35 : s/and extra (a)/and extract (b)/ | |
| 15:03:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Make image job non-voting https://review.opendev.org/c/openstack/python-openstackclient/+/945226 | |
| 15:04:24 | stephenfin | gtema: That keeps popping up ^ I think we'll end up removing it before long, but making it non-voting should do for now? | |
| 15:07:12 | gtema | agreed stephenfin. This job is afaik anyway only required by infra folks | |
| 15:07:28 | stephenfin | Sweet. I've sent it on its way | |
| 15:11:45 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/2025.1: zuul: Make image job non-voting https://review.opendev.org/c/openstack/python-openstackclient/+/945227 | |
| 15:18:39 | opendevreview | Merged openstack/python-openstackclient master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/python-openstackclient/+/944835 | |
| 16:12:49 | opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 16:24:14 | opendevreview | Merged openstack/python-openstackclient stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/python-openstackclient/+/944833 | |
| 16:41:47 | opendevreview | Merged openstack/openstacksdk master: Refactor loader magic fixes https://review.opendev.org/c/openstack/openstacksdk/+/944123 | |
| 17:05:04 | opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 17:16:03 | opendevreview | Merged openstack/python-openstackclient master: zuul: Make image job non-voting https://review.opendev.org/c/openstack/python-openstackclient/+/945226 | |
| 17:29:15 | opendevreview | Merged openstack/python-openstackclient stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/python-openstackclient/+/944834 | |
| 18:06:09 | opendevreview | Vasyl Saienko proposed openstack/openstacksdk stable/2024.2: Fix baremetal get_node_console https://review.opendev.org/c/openstack/openstacksdk/+/942619 | |
| 20:44:14 | opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| #openstack-sdks - 2025-03-22 | |||
| 13:36:05 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 13:39:59 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 13:46:04 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:10:33 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:21:22 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:27:21 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:29:11 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:42:05 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:47:22 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:51:31 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 14:59:04 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 15:04:39 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 15:11:29 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 15:17:29 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 15:28:56 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 15:44:39 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 15:57:20 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 20:45:57 | opendevreview | Vasyl Saienko proposed openstack/openstacksdk stable/2024.1: Fix baremetal get_node_console https://review.opendev.org/c/openstack/openstacksdk/+/942611 | |
| 23:57:11 | adamcarthur5 | Thanks for all that stephenfin. I will read all of this in more detail and get back to you | |
| #openstack-sdks - 2025-03-23 | |||
| 09:38:24 | opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 11:09:06 | opendevreview | Merged openstack/osc-lib stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/osc-lib/+/943698 | |
| 11:09:07 | opendevreview | Merged openstack/osc-lib stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/osc-lib/+/943699 | |
| #openstack-sdks - 2025-03-24 | |||
| 07:57:42 | ralonsoh | stephenfin, hello! A trivial patch: https://review.opendev.org/c/openstack/openstacksdk/+/945205. We are not calling the correct delete methods and we are leaving leftovers | |
| 09:03:38 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add QoS packet rate limit rule object and CRUD operations https://review.opendev.org/c/openstack/openstacksdk/+/945202 | |
| 09:10:54 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: [Neutron] Add "qos-policy" parameter to router creation command https://review.opendev.org/c/openstack/python-openstackclient/+/945200 | |
| 11:26:33 | stephenfin | ralonsoh: sent on its way | |
| 11:26:48 | ralonsoh | stephenfin, thanks! | |
| 11:28:00 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/2025.1: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945305 | |
| 11:30:40 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/2024.2: zuul: Make image job non-voting https://review.opendev.org/c/openstack/python-openstackclient/+/945306 | |
| 11:32:23 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/2025.1: Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/945307 | |
| 11:51:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: zuul: Replace removed jobs https://review.opendev.org/c/openstack/openstacksdk/+/945308 | |
| 13:04:52 | opendevreview | Merged openstack/openstacksdk master: Define the correct QoS rules deletion method https://review.opendev.org/c/openstack/openstacksdk/+/945205 | |
| 13:04:53 | opendevreview | Merged openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | |
| 13:04:56 | opendevreview | Merged openstack/python-openstackclient stable/2025.1: zuul: Make image job non-voting https://review.opendev.org/c/openstack/python-openstackclient/+/945227 | |
| 13:12:43 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient stable/2024.2: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945312 | |
| 13:17:52 | stephenfin | slaweq: fyi I left comments on https://review.opendev.org/c/openstack/python-openstackclient/+/944782, when you a chance | |
| 13:23:20 | opendevreview | Merged openstack/python-openstackclient stable/2024.2: zuul: Make image job non-voting https://review.opendev.org/c/openstack/python-openstackclient/+/945306 | |
| 14:34:25 | opendevreview | Merged openstack/openstacksdk stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/openstacksdk/+/943696 | |
| 14:59:25 | opendevreview | Merged openstack/openstacksdk master: zuul: Replace removed jobs https://review.opendev.org/c/openstack/openstacksdk/+/945308 | |
| 15:36:31 | opendevreview | Alfredo Moralejo proposed openstack/python-openstackclient master: Replace description-content-type by its underscore name https://review.opendev.org/c/openstack/python-openstackclient/+/945332 | |
| 16:03:46 | opendevreview | Alfredo Moralejo proposed openstack/python-openstackclient master: Replace description-content-type by its underscore name https://review.opendev.org/c/openstack/python-openstackclient/+/945332 | |
| 18:32:58 | opendevreview | Merged openstack/python-openstackclient master: Replace description-content-type by its underscore name https://review.opendev.org/c/openstack/python-openstackclient/+/945332 | |
| 22:16:56 | adamcarthur5 | stephenfin Read this through, looks great. I will start bashing this work up :). I might email you to make communication async easier, and update the bug ticket I have for it, I'll review these change as well | |
| #openstack-sdks - 2025-03-25 | |||
| 05:58:41 | opendevreview | Michael Still proposed openstack/openstacksdk master: Add support for spice-direct console types. https://review.opendev.org/c/openstack/openstacksdk/+/940479 | |
| 07:40:12 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add QoS packet rate limit rule object and CRUD operations https://review.opendev.org/c/openstack/openstacksdk/+/945202 | |
| 08:03:47 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Make sensitive fields be SecretString https://review.opendev.org/c/openstack/codegenerator/+/945425 | |
| 08:52:51 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Fix networking quota usage show https://review.opendev.org/c/openstack/python-openstackclient/+/944782 | |
| 10:38:39 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Make sensitive fields be SecretString https://review.opendev.org/c/openstack/codegenerator/+/945425 | |
| 11:13:14 | opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support for designate resuorces quota https://review.opendev.org/c/openstack/openstacksdk/+/945450 | |
| 11:18:23 | opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support for designate resuorces quota https://review.opendev.org/c/openstack/openstacksdk/+/945450 | |
| 11:49:06 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Make sensitive fields be SecretString https://review.opendev.org/c/openstack/codegenerator/+/945425 | |
| 11:50:36 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Format generated Rust code https://review.opendev.org/c/openstack/codegenerator/+/945451 | |
| 12:17:37 | stephenfin | adamcarthur5: wfm! | |
| 12:21:31 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Return the ``port`` column headers expected in the list command https://review.opendev.org/c/openstack/python-openstackclient/+/945453 | |
| 13:13:29 | opendevreview | Merged openstack/python-openstackclient master: Fix networking quota usage show https://review.opendev.org/c/openstack/python-openstackclient/+/944782 | |
| 13:19:45 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Return the ``port`` column headers expected in the list command https://review.opendev.org/c/openstack/python-openstackclient/+/945453 | |
| 13:38:39 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: [Neutron] Add "qos-policy" parameter to router creation command https://review.opendev.org/c/openstack/python-openstackclient/+/945200 | |
| 13:44:13 | opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support for designate resuorces quota https://review.opendev.org/c/openstack/openstacksdk/+/945450 | |
| 14:08:57 | opendevreview | Jan Ueberacker proposed openstack/python-openstackclient master: Add filters to search for enabled/disabled users and projects https://review.opendev.org/c/openstack/python-openstackclient/+/945460 | |
| 15:22:12 | opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support for designate resuorces quota https://review.opendev.org/c/openstack/openstacksdk/+/945450 | |
| 15:30:59 | opendevreview | Merged openstack/codegenerator master: Make sensitive fields be SecretString https://review.opendev.org/c/openstack/codegenerator/+/945425 | |
| 15:40:37 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Format generated Rust code https://review.opendev.org/c/openstack/codegenerator/+/945451 | |
| 17:01:31 | opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support for designate resuorces quota https://review.opendev.org/c/openstack/openstacksdk/+/945450 | |
| 20:39:00 | opendevreview | Merged openstack/codegenerator master: Format generated Rust code https://review.opendev.org/c/openstack/codegenerator/+/945451 | |
| 21:02:22 | opendevreview | Merged openstack/openstacksdk master: tests: Drop support for testing identity v2 https://review.opendev.org/c/openstack/openstacksdk/+/942898 | |
| #openstack-sdks - 2025-03-26 | |||
| 07:46:41 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: [Neutron] Add "qos-policy" parameter to router creation command https://review.opendev.org/c/openstack/python-openstackclient/+/945200 | |
| 08:11:40 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Return the ``port`` column headers expected in the list command https://review.opendev.org/c/openstack/python-openstackclient/+/945453 | |
| 13:42:46 | opendevreview | Jan Ueberacker proposed openstack/python-openstackclient master: Add filters to search for enabled/disabled users and projects https://review.opendev.org/c/openstack/python-openstackclient/+/942103 | |
| 14:18:39 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Return the ``port`` column headers expected in the list command https://review.opendev.org/c/openstack/python-openstackclient/+/945453 | |
| 14:20:10 | ralonsoh | stephenfin, ohhh I didn't know about ``suggestion`` | |