| opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: WIP: note for setting project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | 08:56 |
|---|---|---|
| opendevreview | Merged openstack/python-openstackclient stable/2025.1: identity: Fix 'trust' commands to work with SDK https://review.opendev.org/c/openstack/python-openstackclient/+/944991 | 09:37 |
| 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 | 10:32 |
| opendevreview | Merged openstack/keystoneauth stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/keystoneauth/+/943704 | 11:08 |
| opendevreview | Merged openstack/keystoneauth stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/keystoneauth/+/943705 | 11:17 |
| 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 | 11:57 |
| opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Define the correct QoS rules deletion method https://review.opendev.org/c/openstack/openstacksdk/+/945205 | 12:06 |
| opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add QoS packet rate limit rule https://review.opendev.org/c/openstack/python-openstackclient/+/944784 | 12:10 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Refactor loader magic fixes https://review.opendev.org/c/openstack/openstacksdk/+/944123 | 13:39 |
| opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | 14:13 |
| 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:33 |
| 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:35 |
| 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:38 |
| stephenfin | So as I noted, I've broken https://review.opendev.org/c/openstack/ironic/+/928921/5 up to address those three things: | 14:39 |
| stephenfin | The first moves API versioning to a decorator | 14:40 |
| stephenfin | The second reworks request path, query string and body parameter validation to use JSONSchema instead of the homegrown stuff ironic has | 14:40 |
| stephenfin | The third adds request body parameter validation, since that doesn't exist (at least at runtime) yet | 14:40 |
| 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:42 |
| 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:43 |
| stephenfin | for above message @ :35 : s/and extra (a)/and extract (b)/ | 14:44 |
| opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Make image job non-voting https://review.opendev.org/c/openstack/python-openstackclient/+/945226 | 15:03 |
| 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:04 |
| gtema | agreed stephenfin. This job is afaik anyway only required by infra folks | 15:07 |
| stephenfin | Sweet. I've sent it on its way | 15:07 |
| 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:11 |
| opendevreview | Merged openstack/python-openstackclient master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/python-openstackclient/+/944835 | 15:18 |
| opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | 16:12 |
| opendevreview | Merged openstack/python-openstackclient stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/python-openstackclient/+/944833 | 16:24 |
| opendevreview | Merged openstack/openstacksdk master: Refactor loader magic fixes https://review.opendev.org/c/openstack/openstacksdk/+/944123 | 16:41 |
| opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | 17:05 |
| opendevreview | Merged openstack/python-openstackclient master: zuul: Make image job non-voting https://review.opendev.org/c/openstack/python-openstackclient/+/945226 | 17:16 |
| 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 | 17:29 |
| opendevreview | Vasyl Saienko proposed openstack/openstacksdk stable/2024.2: Fix baremetal get_node_console https://review.opendev.org/c/openstack/openstacksdk/+/942619 | 18:06 |
| opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Specifying project-domain for project https://review.opendev.org/c/openstack/python-openstackclient/+/945193 | 20:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!