| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-09-06 | |||
| 14:17:08 | opendevreview | Merged openstack/keystoneauth master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/keystoneauth/+/928396 | |
| 14:33:51 | opendevreview | Merged openstack/keystoneauth stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/keystoneauth/+/928394 | |
| 14:33:52 | opendevreview | Merged openstack/keystoneauth stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/keystoneauth/+/928395 | |
| 14:44:54 | opendevreview | Merged openstack/codegenerator master: Undo hacking identity auth catalog command https://review.opendev.org/c/openstack/codegenerator/+/928338 | |
| 15:46:50 | opendevreview | Merged openstack/python-openstackclient master: quota: Allow 'quota set' to function without volume service https://review.opendev.org/c/openstack/python-openstackclient/+/925861 | |
| 17:15:48 | frickler | autocomplete for cloud names, that's a nice idea | |
| 19:25:16 | opendevreview | Merged openstack/keystoneauth master: Pass oidc client id in the body if no client secret provided https://review.opendev.org/c/openstack/keystoneauth/+/927581 | |
| 21:18:16 | zigo | fungi: I'll add oscs as recommends for the python3-openstackclient package, and as depends for the openstack-clients package, I think it's fine to have these separated. | |
| 21:18:57 | zigo | Though true, it could actually live within upstream openstackclient git... | |
| 21:19:56 | zigo | I've just added screenshot and docs. | |
| #openstack-sdks - 2024-09-07 | |||
| 15:01:21 | opendevreview | KyeongNa Park proposed openstack/openstacksdk master: Add create, delete, find, list and UPDATE example connect code - create zone, recordset - list zone - delete zone, recordset - example connect https://review.opendev.org/c/openstack/openstacksdk/+/928129 | |
| #openstack-sdks - 2024-09-08 | |||
| 13:56:03 | opendevreview | KyeongNa Park proposed openstack/openstacksdk master: Add create, delete, find, list and UPDATE example connect code - create zone, recordset - list zone - delete zone, recordset - example connect https://review.opendev.org/c/openstack/openstacksdk/+/928129 | |
| 16:52:22 | opendevreview | Slawek Kaplonski proposed openstack/openstacksdk master: Add "trusted" attribute to "port" resource https://review.opendev.org/c/openstack/openstacksdk/+/927723 | |
| 16:57:39 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add "trusted" attribut to the "port" https://review.opendev.org/c/openstack/python-openstackclient/+/927727 | |
| #openstack-sdks - 2024-09-09 | |||
| 05:36:14 | opendevreview | Jun Woo Park proposed openstack/openstacksdk master: Add getting info about the Limit in a DNS SDK. https://review.opendev.org/c/openstack/openstacksdk/+/927854 | |
| 05:48:54 | opendevreview | Jun Woo Park proposed openstack/openstacksdk master: Add getting info about the Limit in a DNS SDK. https://review.opendev.org/c/openstack/openstacksdk/+/927854 | |
| 06:29:31 | opendevreview | Jun Woo Park proposed openstack/openstacksdk master: Add getting info about the Limit in a DNS SDK. https://review.opendev.org/c/openstack/openstacksdk/+/927854 | |
| 12:08:41 | slaweq | gtema (Artem Goncharov) hi, how are You? Can you check https://review.opendev.org/c/openstack/openstacksdk/+/927723 and https://review.opendev.org/c/openstack/python-openstackclient/+/927727 when you will have few minutes? Thx in advance | |
| 12:15:41 | gtema | slaweq - I wanted to keep on hold that till all release branches happens | |
| 12:16:01 | gtema | anyway - opened as tabs in my browser to process in the course of today | |
| 12:33:12 | slaweq | gtema (Artem Goncharov) sure, it can definitely be after release will be done | |
| 12:33:16 | slaweq | thx a lot | |
| 14:07:46 | opendevreview | Josephine Seifert proposed openstack/python-openstackclient master: WIP: Add admin-editable metadata field to volume types https://review.opendev.org/c/openstack/python-openstackclient/+/928132 | |
| 14:52:24 | opendevreview | KyeongNa Park proposed openstack/openstacksdk master: Add create, delete, find, list and UPDATE example connect code - create zone, recordset - list zone - delete zone, recordset - example connect https://review.opendev.org/c/openstack/openstacksdk/+/928129 | |
| 16:34:07 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Start building Ironic OpenAPI https://review.opendev.org/c/openstack/codegenerator/+/928664 | |
| #openstack-sdks - 2024-09-10 | |||
| 07:16:25 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Start building Ironic OpenAPI https://review.opendev.org/c/openstack/codegenerator/+/928664 | |
| 09:00:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Escape commit message https://review.opendev.org/c/openstack/codegenerator/+/928761 | |
| 14:06:32 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Update betamax extra deps https://review.opendev.org/c/openstack/keystoneauth/+/928801 | |
| 14:06:33 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Add pre-commit https://review.opendev.org/c/openstack/keystoneauth/+/928802 | |
| 14:06:34 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: docs: Remove cruft from configuration files https://review.opendev.org/c/openstack/keystoneauth/+/928803 | |
| 14:06:35 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: trivial: Remove errant trailing commas https://review.opendev.org/c/openstack/keystoneauth/+/928804 | |
| 14:06:37 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Apply ruff, ruff-format https://review.opendev.org/c/openstack/keystoneauth/+/928805 | |
| 14:06:39 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Enable ruff, ruff-format https://review.opendev.org/c/openstack/keystoneauth/+/928806 | |
| 14:06:41 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Make abstract method actually abstract https://review.opendev.org/c/openstack/keystoneauth/+/928807 | |
| 14:06:43 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Synchronise overridden method signatures https://review.opendev.org/c/openstack/keystoneauth/+/928808 | |
| 14:06:45 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Add variable for auth plugin opts https://review.opendev.org/c/openstack/keystoneauth/+/928809 | |
| 14:06:47 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Resolve remaining initial issues https://review.opendev.org/c/openstack/keystoneauth/+/928810 | |
| 14:06:49 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Enable mypy https://review.opendev.org/c/openstack/keystoneauth/+/928811 | |
| 14:06:51 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: discover: Expand use of kwargs https://review.opendev.org/c/openstack/keystoneauth/+/928812 | |
| 14:06:53 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: discover: Rework normalization https://review.opendev.org/c/openstack/keystoneauth/+/928813 | |
| 14:06:55 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Annotate keystoneauth1.discover https://review.opendev.org/c/openstack/keystoneauth/+/928814 | |
| 14:19:00 | stephenfin | gtema: jfyi, I have keystoneauth fully typed (except for tests and test fixtures). Might be looking for your help getting them merged in the next while. I want them to continue my typing SDK journey | |
| 14:19:35 | stephenfin | Also, I was going to ask if you'd seen https://review.opendev.org/c/openstack/codegenerator/+/927704 but it seems you have (y) | |
| 14:19:42 | gtema | sure stephenfin, thanks. I got reformat and typing merged in keystone itself | |
| 14:19:51 | gtema | so adding keystoneauth is great | |
| 14:20:05 | gtema | I am on your change | |
| 14:20:37 | gtema | https://review.opendev.org/c/openstack/codegenerator/+/928761 fixes it (you used ' in the commit message and it broke the job preparing the commit) | |
| 14:21:53 | gtema | sadly there is apparently also one race that I was not able to reproduce locally ever that causes codegenerator to produce "broken" code and only recheck helps - I am scratching my head with no ideas why is that happening and how I can reproduce it reliably | |
| 14:23:33 | gtema | btw stephenfin - we started looking at ironic. They use different decorators and not everything is easy, but at least I hope they will decide how to deal with it (whether to switch to decorators like everywhere else or to tweak codegenerator to support that deviation as well) | |
| 14:23:49 | stephenfin | sweet \o/ | |
| 14:24:23 | stephenfin | I did talk to them at the last PTG about doing this and TheJulia noted that they would just like to ensure adding decorators didn't introduce a significant performance regression but were otherwise open to it | |
| 14:25:10 | stephenfin | I've been focused on Manila and Nova (plus, to a lesser degree, Cinder) since though, so I haven't had a chance to work on Ironic | |
| 14:25:47 | gtema | JayF asked his colleague to start looking at openapi@ironic so hopefully we get a better traction. It's however still not trivial due to use of pecan with a very dynamic routing | |
| 14:26:55 | JayF | adamcarthur5 is in here FYI | |
| 14:27:20 | gtema | right, thanks for bringing us together JayF | |
| 14:28:40 | JayF | Absolutely. We're going to be chatting today and he's going to bring me up to date. | |
| 14:29:10 | gtema | perfect. I told him also that we could have another joint meeting to further align on strategy | |
| 14:30:25 | JayF | I'm more of a person who likes to find a specific action we can take and take it rather than talking more about strategy. Such as maybe putting a point on that question for how ironic is going to implement this, so that we can pilot it and get an answer from the other cores | |
| 14:31:47 | gtema | you speak to my soul JayF ;-) | |
| 14:32:01 | stephenfin | JayF: Agreed. The main thing we need is the ability to retrieve a schema for a given microversion. | |
| 14:33:31 | JayF | I don't know any of the mechanics for what y'all been working on so far, but I'll have a better idea after talking to Adam this afternoon. | |
| 14:33:56 | JayF | I think you'll find that ironic tends to let the implementers have a lot of free reign on implementation details. We're not dogmatic about specific ways of doing things | |
| 14:34:25 | JayF | So if you find a way that works, as long as it makes sense and is updatable I don't think anyone is going to have a problem with it [knock on wood] | |
| 14:34:29 | stephenfin | idk how up-to-speed Adam is, but most of nova's logic lives here nova/api/validation/__init__.py | |
| 14:35:11 | gtema | stephenfin - ironic has something similar and though deviating heavily | |
| 14:35:16 | stephenfin | It's not complicated and relies on a webob request object being the first parameter to any controller method | |
| 14:35:20 | stephenfin | gtema: good to know | |
| 14:35:48 | stephenfin | so does manila and cinder, but those are a lot close to Nova that Ironic is (all use webob + routes) | |
| 14:36:00 | stephenfin | ...hence my focus on those three together so far :) | |
| 14:36:30 | gtema | and I "copied" the same into Keystone (which is similarly as ironic relies on a completely different framework) | |
| 14:37:18 | gtema | from that pov it would be strategically more helpful if ironic would adopt the same decorators (at least syntaxically) | |
| 14:40:43 | stephenfin | yeah, flask (with plugins) iirc. Nova et al probably need to switch themselves at some point since webob seems to be going the way of the dinosaur, but I already have more work that I can keep on top of so... | |
| 14:57:51 | JayF | Ironic uses some webob headers too -- you're saying those need to change? | |
| 14:58:15 | JayF | Let me put it this way; if Ironic has tech debt in our API controllers around how we pass around headers/information, I'm not aware of it and so please help if you know any :D | |
| 15:06:28 | opendevreview | Merged openstack/codegenerator master: Escape commit message https://review.opendev.org/c/openstack/codegenerator/+/928761 | |
| 16:45:46 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/keystoneauth/+/928833 | |
| 16:48:35 | opendevreview | Merged openstack/codegenerator master: Add getting started guide to README https://review.opendev.org/c/openstack/codegenerator/+/927704 | |
| 16:49:54 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/keystoneauth/+/928833 | |
| 16:50:28 | stephenfin | gtema: Are we ready to merge the "drop python3.8" patches now that we've branched? https://review.opendev.org/q/topic:%22drop-py37-py38%22 | |
| 16:51:57 | gtema | yupp, shoot it | |
| 16:52:25 | gtema | oh but not in the cli | |
| 16:52:41 | stephenfin | huh? | |
| 16:52:48 | gtema | cli is not branched, not that I see a branch | |
| 16:52:57 | stephenfin | I think it's independent | |
| 16:53:20 | stephenfin | nope | |
| 16:54:24 | stephenfin | only thing I can see is that it's 'type: other' | |
| 16:55:13 | gtema | ehm, I am confused | |
| 16:56:41 | stephenfin | looks like this has happened before https://review.opendev.org/c/openstack/releases/+/913494 | |
| 16:56:45 | stephenfin | I think there's a tooling issue | |
| 17:03:15 | gtema | fwik it should have happened in https://review.opendev.org/c/openstack/releases/+/927944 where sdk and osc-lib were branched | |
| 17:04:09 | stephenfin | discussing on #openstack-release now. elodilles confirmed it's because of the type | |
| 17:04:43 | stephenfin | osc-lib and sdk are 'type: library' so we can't do the same there. client-library doesn't really make sense either since it's not a library | |
| 17:49:12 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add "trusted" attribut to the "port" https://review.opendev.org/c/openstack/python-openstackclient/+/927727 | |
| 18:25:06 | opendevreview | Slawek Kaplonski proposed openstack/openstacksdk master: Add "trusted" attribute to "port" resource https://review.opendev.org/c/openstack/openstacksdk/+/927723 | |
| 19:15:13 | samcat116 | Has --os-profile been removed from the OpenStack CLI? Its not a recognized argument for anything on client version 6.3.0 | |
| 20:29:29 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 20:30:09 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 21:16:14 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 21:26:28 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 21:49:47 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |