| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-10-24 | |||
| 15:13:02 | gtema | perfect, again - I do not really care how you do your stuff I just need to be able to programmatically extract info about request, response, parameters of the operation | |
| 15:13:33 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Start building Ironic OpenAPI https://review.opendev.org/c/openstack/codegenerator/+/928664 | |
| 21:22:54 | opendevreview | Thobias Salazar Trevisan proposed openstack/openstacksdk master: Fix missing 'f' prefix in exception message https://review.opendev.org/c/openstack/openstacksdk/+/933375 | |
| #openstack-sdks - 2024-10-25 | |||
| 09:23:23 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Fix redundant domain id lookup https://review.opendev.org/c/openstack/python-openstackclient/+/933403 | |
| 09:29:08 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Fix ignored --user-domain in role assignment list https://review.opendev.org/c/openstack/python-openstackclient/+/933404 | |
| 09:30:07 | tkajinam | tkajinam------- | |
| 09:33:00 | tkajinam | stephenfin, sorry I definitely did something wrong with my previous patch... ^^^ | |
| 09:55:49 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Fix ignored --user-domain in role assignment list https://review.opendev.org/c/openstack/python-openstackclient/+/933404 | |
| 09:56:21 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Fix ignored --user-domain in role assignment list https://review.opendev.org/c/openstack/python-openstackclient/+/933404 | |
| 11:40:01 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Fix ignored --user-domain in role assignment list https://review.opendev.org/c/openstack/python-openstackclient/+/933404 | |
| #openstack-sdks - 2024-10-26 | |||
| 16:08:43 | opendevreview | Takashi Kajinami proposed openstack/openstackclient master: Test more plugins in plugin tests https://review.opendev.org/c/openstack/openstackclient/+/906195 | |
| 16:23:31 | opendevreview | Merged openstack/openstackclient master: Test more plugins in plugin tests https://review.opendev.org/c/openstack/openstackclient/+/906195 | |
| #openstack-sdks - 2024-10-27 | |||
| 12:04:34 | Yosef | Hi, I have been working on this patch: https://review.opendev.org/c/openstack/python-openstackclient/+/932328 | |
| 12:05:06 | Yosef | I think it is missing workflow +1, can somebody check it? | |
| #openstack-sdks - 2024-10-28 | |||
| 10:23:40 | frickler | gtema: stephenfin: can you please check https://review.opendev.org/c/openstack/openstacksdk/+/933278? looks reasonable to me | |
| 10:24:00 | gtema | Ack | |
| 12:26:53 | opendevreview | Merged openstack/openstacksdk stable/2023.2: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/933278 | |
| 12:34:56 | servagem | Hello, Could someone please review this small patch (https://review.opendev.org/c/openstack/openstacksdk/+/933375) at your convenience? Thank you | |
| #openstack-sdks - 2024-10-29 | |||
| 07:54:55 | tkajinam | hmm something is wrong with log upload but idk how I can debug it without any logs recorded ... https://review.opendev.org/c/openstack/python-openstackclient/+/933404 | |
| 08:10:52 | frickler | tkajinam: there seems to be some issue with our registry, let's move discussion to #opendev (I do see some log with timeout in "Push tag to intermediate registry") | |
| 08:12:04 | tkajinam | frickler, thanks for checking ! ok let me repost it (with a few more details) in #opendev | |
| 11:42:33 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Fix ignored --user-domain in role assignment list https://review.opendev.org/c/openstack/python-openstackclient/+/933404 | |
| 12:02:31 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Fix ignored --user-domain in role assignment list https://review.opendev.org/c/openstack/python-openstackclient/+/933404 | |
| 12:20:16 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Fix ignored --user-domain in role assignment list https://review.opendev.org/c/openstack/python-openstackclient/+/933404 | |
| 13:01:42 | opendevreview | Niklas Schwarz proposed openstack/python-openstackclient master: Prevent additional calls from identity client when searching by the name for a resource. https://review.opendev.org/c/openstack/python-openstackclient/+/933644 | |
| 15:04:10 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix test for server shelve https://review.opendev.org/c/openstack/ansible-collections-openstack/+/933657 | |
| 15:04:51 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM: just test latest CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/933658 | |
| #openstack-sdks - 2024-10-30 | |||
| 10:13:23 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix test for server shelve https://review.opendev.org/c/openstack/ansible-collections-openstack/+/933657 | |
| 13:22:13 | yosef | Hi guys, should I RECHECK this patch? https://review.opendev.org/c/openstack/python-openstackclient/+/932328 it is failed at gate job, I don't think it is related to my changes! | |
| 17:31:17 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix identity resource link code https://review.opendev.org/c/openstack/codegenerator/+/933765 | |
| 18:31:34 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix identity resource link code https://review.opendev.org/c/openstack/codegenerator/+/933765 | |
| 18:32:14 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix identity resource link code https://review.opendev.org/c/openstack/codegenerator/+/933765 | |
| 19:15:36 | opendevreview | Merged openstack/codegenerator master: Start building Ironic OpenAPI https://review.opendev.org/c/openstack/codegenerator/+/928664 | |
| 19:24:03 | stephenfin | yosef: rechecked. It looks like a timeout | |
| 19:52:24 | adamcarthur5 | Hey stephenfin and gtema, after talking about getting what you need in Ironic for code-generator at the PTG, I am planning on dedicating a decent chunk of time to it going forward. | |
| 19:52:24 | adamcarthur5 | I have the Ironic Spec generated from master since it was merged last week, and it seems to be working fine. My plan is: | |
| 19:52:24 | adamcarthur5 | - Setup a Ticket/Bug explaining the work + the "openapi" topic that I can reference when doing the required commits | |
| 19:52:24 | adamcarthur5 | - Continue adding required elements based on the work done in https://review.opendev.org/c/openstack/ironic/+/928919?tab=comments (Do you want me to get CI passing on this? Happy to do that as a quick starter task) | |
| 19:52:24 | adamcarthur5 | Let me know if you guys have anything else you would be interested in me doing, I imagine there is more to do than just repeating the change in 928919, but it'll keep me busy for the coming weeks | |
| 19:52:25 | adamcarthur5 | - | |
| 20:12:06 | gtema | adamcarthur5: sounds great | |
| 20:13:04 | adamcarthur5 | Cool, I will keep in touch with you guys here, and we can put more permanent discussions of work to complete in the bug | |
| 21:23:42 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix identity resource link code https://review.opendev.org/c/openstack/codegenerator/+/933765 | |
| 21:27:34 | adamcarthur5 | https://bugs.launchpad.net/ironic/+bug/2086121 | |
| 21:27:34 | adamcarthur5 | And gtema would you be interested in me reviewing codegenerator patches from you? More of a way for me to learn/stay up to date with the project than anything else | |
| 21:31:32 | opendevreview | Merged openstack/python-openstackclient master: remove project from network flavor profile https://review.opendev.org/c/openstack/python-openstackclient/+/932328 | |
| 22:36:10 | gtema | adamcarthur5: sure | |
| 22:51:57 | opendevreview | Kevin Carter proposed openstack/ansible-collections-openstack master: Allow wait: false when auto_ip is false https://review.opendev.org/c/openstack/ansible-collections-openstack/+/905308 | |
| #openstack-sdks - 2024-10-31 | |||
| 09:31:05 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix identity resource link code https://review.opendev.org/c/openstack/codegenerator/+/933765 | |
| 11:23:47 | frickler | stephenfin: gtema: there have been multiple sdk job failures now on the reqs bump to latest sdk, so I'm worried that there is an actual regression there, maybe one of you can take a closer look? https://review.opendev.org/c/openstack/requirements/+/932687 | |
| 11:32:18 | frickler | hmm, interestingly no failure in the combined u-c bump, so maybe only bad luck after all https://review.opendev.org/c/openstack/requirements/+/924547 | |
| 11:55:15 | gtema | frickler: ack, looking | |
| 13:38:44 | stephenfin | adamcarthur5: If you could get 928919 passing, that would be super. Alternatively, if you can think of a better approach then please feel free to explore that | |
| 13:40:39 | stephenfin | Virtually all Python web framework appear to provide a routing mechanism and the concept of controller for API endpoints/resources | |
| 13:41:14 | stephenfin | gtema can obviously explain codegenerator better than I can, but both that and the schema validation tests I'm adding to various projects rely on iterating through the routes on a router to find the controllers and ultimately the methods the handle a given HTTP verb + path | |
| 13:43:49 | gtema | correct. For Ironic I can easily access all the routes (http verb + path) and only need to get supported request body, query parameters, information about microversion (either one function that supports different bodies per microversion or one function per microversion) and response description | |
| 13:47:04 | gtema | things like https://opendev.org/openstack/ironic/src/branch/master/ironic/api/controllers/v1/chassis.py#L46 or https://opendev.org/openstack/ironic/src/branch/master/ironic/api/controllers/v1/runbook.py#L72 make it incredibly hard to access the schema and params individually | |
| 13:51:30 | opendevreview | Merged openstack/ansible-collections-openstack master: Enable glance-direct interop image import https://review.opendev.org/c/openstack/ansible-collections-openstack/+/932372 | |
| 15:04:51 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Remove default value for interface https://review.opendev.org/c/openstack/ansible-collections-openstack/+/889729 | |
| 15:53:56 | opendevreview | Merged openstack/ansible-collections-openstack master: Allow wait: false when auto_ip is false https://review.opendev.org/c/openstack/ansible-collections-openstack/+/905308 | |
| 16:07:35 | adamcarthur5 | Yeah stephenfin I can get those tests passing. And I'm with gtema, the way Ironic does micro versions makes it tricky to do anything "smarter" than what the current plan is | |
| 16:08:16 | adamcarthur5 | It's also a plan that the Ironic folks were happy with at the PTG, so let's just stay the course. | |
| 16:08:23 | gtema | adamcarthur5 - what about adding decorators that do not influence ironic but expose the data for the openapi generator? | |
| 16:21:55 | adamcarthur5 | Hmm.. decorators that are there purely for the generator? Or that actually do something in the code? | |
| 16:23:01 | adamcarthur5 | If I am not mistaken, stephenfins commit is using the ironic validation decorator already, and my plan was to follow suit? | |
| 16:24:51 | gtema | yes sure, I mean if we can't enforce changing way of MV handling in ironic it is still more useful to have decorators only for openapi rather then producing wrong specs (or not producing at all) | |
| 16:25:49 | gtema | very initially we were talking also about validation decorators with possibility to switch off validation. This is surely not the most ideal solution, but if there is no other way - why not | |
| 16:39:34 | JayF | I thought this trick might be useful to sdk cores; I looked into it after pushing a oslo.utils change that failed lint: https://jay.jvf.cc/posts/openstack-pre-commit/ tl;dr you can setup the pre-commit hooks to automatically go into new repos | |
| 16:50:32 | gtema | JayF - we already use pre-commit in all repos (maybe missed some of the rarely used) | |
| 16:50:45 | JayF | Yeah; this is about not having to configure the local hook everywhere | |
| 16:50:57 | gtema | ah ok | |
| 16:51:05 | JayF | so people like me who work across lots of repos won't forget to run `pre-commit install` | |
| 16:51:37 | gtema | indeed | |
| 16:51:38 | gtema | thks | |
| 17:16:52 | adamcarthur5 | Hmm. I feel like having decorators that are not enforced just adds even more change for error? Unless I am misunderstanding something | |
| 17:18:03 | adamcarthur5 | I think for us in Ironic, that is the biggest thing we want to fix. We want as much validation as we can get. If we find a particularly hard API to model through validation, I would consider having a validation-off decorator | |
| 17:19:34 | JayF | Eh, some of the value is forcing us to identify the changes between microversions more easily. I'd be sad if we had to punt any cases in Ironic. | |
| 17:23:20 | adamcarthur5 | +1 | |
| 17:57:16 | gtema | sounds great for me. If you want to have validation in the form that can be introspected - I am only supporting that | |
| 17:59:36 | JayF | Yeah I told adam I basically had two goals for this: 1) the actual schema/docs generation but also 2) having ironic microversions look more like other projects, and no longer be a mess of `if x:` | |
| 17:59:59 | tom | hello sirs | |
| 18:00:15 | tom | can you help me understand whi can't i open oftc.net from my ipv6 | |
| 18:00:22 | tom | checked my ip on https://mioip.info and it's ok | |
| 18:00:35 | tom | also on ifconfig.io | |
| 21:01:21 | adamcarthur5 | stephenfin looks like a recheck is all that was needed: https://review.opendev.org/c/openstack/ironic/+/928919 | |
| 21:01:21 | adamcarthur5 | JayF and me suspect you got caught in the CI wave for latest openstack release. | |
| 21:01:34 | adamcarthur5 | I am going to quickly test locally and then we can get this one merged, and I will move on to the next one :)) | |
| 21:01:56 | JayF | adamcarthur5: I think you meant to say you tirelessly worked to resolve CI issues so his job would pass /s ;) | |
| 21:11:43 | adamcarthur5 | Don't worry JayF, I am spending all that extra time I saved re-writing Ironic into Rust :) | |
| 21:13:10 | JayF | nice, we'll finally be Secure(tm) | |
| 21:25:56 | gtema | Welcome to the Rust gang adamcarthur5 ;-) Hopefully we can finally convince TC majority to approve Rust | |
| 21:27:25 | JayF | Honestly don't hate it as an additional idea, just nobody get crazy about rewriting things :D | |
| 21:27:34 | JayF | I only volunteered to help write Ironic once ;) | |
| 21:28:31 | gtema | I doubt people are crazy to simplify rewrite openstack at once. But we shouldn't block introduction where it makes sense | |
| 21:29:18 | JayF | fwiw, I believe you can use rust in any openstack project now at your option; we made the point that the platform we publish is not meant to exclude other stuff, just cover things that have to be handled | |
| 21:30:05 | JayF | In any event, do good things and let the policy follow. Most fear of approval of /anything/ in openstack is that we don't have enough people to do the work. | |
| 21:30:08 | gtema | Well, unless rust is accepted as language I don't see how we can achieve that | |
| 21:53:16 | adamcarthur5 | > Well, unless rust is accepted as language I don't see how we can achieve that | |
| 21:53:35 | adamcarthur5 | What do you mean by accepted here (sorry, still adjusting to IRC, ignore my quote attempt) | |
| 21:59:57 | JayF | I assume the fact we don't have a rust version specified here: https://governance.openstack.org/tc/reference/runtimes/2025.1.html -- but I don't agree that means it can't be used. | |