| opendevreview | Doug Goldstein proposed openstack/ironic master: doc: define the shape of inspection inventory https://review.opendev.org/c/openstack/ironic/+/940277 | 01:35 |
|---|---|---|
| cardoe | so thinking of something like ^ but then also maybe jsonschema... we don't have anything to generate tables from schema do we? | 01:36 |
| opendevreview | Vasyl Saienko proposed openstack/networking-generic-switch master: Remove unneeded hasattr check for bond methods https://review.opendev.org/c/openstack/networking-generic-switch/+/940296 | 11:05 |
| jssfr | Jan 28 11:24:07 ubuntu ironic-python-agent[1923]: 2025-01-28 11:24:07.111 1923 ERROR root ironic_python_agent.errors.InvalidImage: The provided image is not valid for use: Security: Detected image format was gpt, but only raw, qcow2, iso are allowed | 11:39 |
| jssfr | so this is a raw image with a gpt partition table... is that check a bit overzealous? | 11:39 |
| jssfr | looks like https://docs.openstack.org/ironic/2024.1/configuration/config.html#conductor.permitted_image_formats is the way to configure this | 11:42 |
| jssfr | converting the raw image to qcow2 helps :D | 13:00 |
| TheJulia | jssfr: is that appearing in 2024.1? | 14:20 |
| jssfr | no, that's actually zed, I found out. | 14:31 |
| jssfr | I thought we were on 2024.1, but it's zed. | 14:31 |
| jssfr | so may not be relevant really. | 14:31 |
| jssfr | (I only found that out later tho) | 14:32 |
| TheJulia | The check was backported, but I didn't think it had the distingishment between gpt and raw | 14:32 |
| TheJulia | I thought that came in the later versions upstream | 14:32 |
| jssfr | I wouldn't put it past our image build process to accidentally pull in some newer dependency somewhere... | 14:33 |
| TheJulia | well, it was included code, not a library in that version | 14:33 |
| TheJulia | so it just seems confusing, but at the same time it was all a blur | 14:33 |
| jssfr | hmmm | 14:34 |
| JayF | this looks like you pullewd in newer ipa/oslo utils to an older ironic? | 15:05 |
| JayF | full backtrace there would be useful, along with a `pip freeze` in the IPA install in the venv of the ramdisk | 15:05 |
| jssfr | JayF, I'll see if I can get that tomorrow (I'm off my work systems already). | 15:14 |
| jssfr | pip freeze should be easy, not so sure about the full backtrace. That might be gone already. | 15:14 |
| jssfr | OH | 15:14 |
| jssfr | this is about the **ipa** version | 15:14 |
| jssfr | in that case we might actually be on 2024.1 not on zed, I'll have to confirm that. I was only looking at conductor/api/etc. versions. The IPA image is a separate process which may source different code. (cc TheJulia ) | 15:15 |
| TheJulia | Well, the check exists internally in ironic *and* the ramdisk | 15:29 |
| TheJulia | so sort of depends on where it gets disqualified at | 15:29 |
| opendevreview | cid proposed openstack/ironic master: Apply Rules: inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939218 | 15:48 |
| opendevreview | cid proposed openstack/ironic master: API/Testing: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 15:48 |
| jssfr | TheJulia, ramdisk | 16:37 |
| jssfr | (the log line is from there) | 16:37 |
| TheJulia | ahh, yeah | 16:46 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Bootable container support https://review.opendev.org/c/openstack/ironic-python-agent/+/940178 | 16:55 |
| cardoe | JayF: you were looking for pre-commit on IPA right? | 17:00 |
| *** priteau is now known as Guest7267 | 17:04 | |
| *** priteau2 is now known as priteau | 17:04 | |
| cardoe | Or was it another repo? Cause I did IPA. | 17:08 |
| TheJulia | I'm guessing Jay is still feeling under the weather | 17:17 |
| TheJulia | And likely the only way to illicit a response is for a hockey game to appear outside his home ;) | 17:18 |
| JayF | I'm here just busy | 17:19 |
| JayF | yesterday was less illness and more allergies+insomnia working together to knock me outta commission | 17:19 |
| JayF | My meetings for the day are complete :) | 17:19 |
| JayF | cardoe: I think it was ironic-specs? | 17:23 |
| JayF | cardoe: IDR at this point, my context of that conversation is long gone :) | 17:23 |
| cardoe | Okay. I’ve got patches for both in gerrit. | 17:24 |
| cardoe | Sounds like you’ve had a rough go of it. | 17:24 |
| JayF | ack; I'll do a review pass once I'm to the bottom of the list | 17:24 |
| JayF | eh; I had COVID over the winter break; I recovered from most of it but have been lethargic/having leftover sinus issues | 17:24 |
| JayF | so lack of sleep + allergy flare up basically just kicked my butt yesterday but I'm feeling really good today (thanks; sudafed) | 17:25 |
| cardoe | I’m trying to clean up the node cleaning docs and the inspection ones. I’m honestly more confused than I was when I started about behavior. | 17:25 |
| JayF | give me like, 30-60 minutes to get to the bottom of my todo list | 17:26 |
| JayF | maybe a little more | 17:26 |
| JayF | and I'm happy to sync up with you on it and find answers | 17:26 |
| TheJulia | ugh, sinus issues from covid seem to last a tremendously long time | 17:26 |
| cardoe | No worries. I’m around on mobile. I’ll sit in front of my laptop in a bit. | 17:27 |
| JayF | yeah; I just decided I'll do what my doc has told me to do in the past and just sudafed for two weeks and try to reset things | 17:27 |
| cardoe | I’m in Puerto Rico today but it’s a work day for my wife. And my “job” is to follow her with camera gear and scene lists and make sure we get them. But I can have her stop and edit anytime. | 17:28 |
| cardoe | I’m on PTO from my real job to be an assistant to my wife. | 17:29 |
| JayF | your wife some kinda influencer? | 17:30 |
| cardoe | Kinda? She doesn’t actually post the stuff. She creates content that others get from her. | 17:32 |
| cardoe | And it’s not influencers. But like brands and travel agencies. | 17:33 |
| TheJulia | That sounds awesome | 17:37 |
| cardoe | It started years ago when she was a real estate agent. She’s good at digital video. So she made walk throughs before the 360 camera was a thing. Then added drone footage before it was a thing. Ended up getting others interested in having her do it. So that was more consistent work. So she did that for a while. Then she made some montage of a vacation. | 17:37 |
| TheJulia | as an FYI, I posted https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/940247 to try and reduce some of our test failures in CI | 17:37 |
| cardoe | And our travel agent said they’d give us a free or reduced trip if she made them a bunch of ads. So we did that. And now she’s a part time travel agent and the brand themselves send her on trips to generate content for them to post. | 17:39 |
| cardoe | So we go a few weeks a year and record a metric ton of footage. It’s like stock photos on steroids. | 17:39 |
| JayF | that is really freakin' cool seeming | 17:41 |
| JayF | that being said: you should be enjoying Puerto Rico without the glow of a laptop :) nothing we need here can't wait until you're back workin' | 17:42 |
| cardoe | It’s funny cause I don’t have any social media apps. | 17:42 |
| JayF | the older I get, the more I think IRC counts | 17:45 |
| cardoe | I’m literally downloading files from her cameras and tagging them for her. So I’ve already got a laptop. | 17:45 |
| JayF | maybe that's more reflective of how many of my friends/hobbies are IRC-centric though :) | 17:45 |
| TheJulia | I'm pretty much done with social media at this point, except the blue skys | 17:46 |
| JayF | my X account was closed last week, even though for a while it was just podcast shilling. I'm trying to look at everything like that less and less, and heavily filter any ... non-personal information | 17:47 |
| JayF | like I wanna hear how Julia is doing; I don't need 45 people boosting news into my thread | 17:47 |
| TheJulia | I've been a lot better | 17:47 |
| TheJulia | but i'll be okay... for now | 17:47 |
| cardoe | Exactly. I think we’ve now seen that social media has been a net negative on society. | 17:47 |
| cardoe | Though I shouldn’t bash it cause it’s given me some cool trips since COVID. | 17:48 |
| JayF | I'm hoping I'm retirement age before I am convinced s/social media/internet/ | 17:48 |
| JayF | otherwise my job gets a lot harder ;) | 17:48 |
| TheJulia | heh | 17:48 |
| TheJulia | So! I'm curious. Any thoughts/feelings on doing some auto-middle of error truncation with error messages? | 17:49 |
| JayF | In the beginning, the internet was created. This has made a lot of people very angry and has been widely regarded as a bad move. | 17:49 |
| JayF | I have no idea what you mean by that | 17:49 |
| TheJulia | specifically, when doing my bootc testing last week, I had podman throw a stupidly long error | 17:49 |
| TheJulia | such that the tail end got truncated because it exceeded the field length | 17:49 |
| cardoe | The closest we’ve come to being in any of our videos is some Royal Caribbean video where I slide off the end of a slide wearing a GoPro and my legs come into view for a second. | 17:49 |
| JayF | TheJulia: how big is the field? is the answer not "make it big enough to hold a giant freakin' error"? | 17:50 |
| TheJulia | JayF: That is a stupidly good question | 17:50 |
| TheJulia | I think it is 1k | 17:50 |
| * TheJulia looks i tup | 17:50 | |
| cardoe | We throw some serious back traces. That’s surprising. | 17:50 |
| TheJulia | L + 2 bytes, where L < 216 | 17:51 |
| TheJulia | so we might be truncating elsewhere | 17:51 |
| TheJulia | I'll have to dig | 17:51 |
| TheJulia | I just found the trail end truncation semi-annoying | 17:52 |
| TheJulia | It *helped* I had the logs on a terminal window and I could see why my deploy of a container failed | 17:52 |
| TheJulia | but yeah | 17:52 |
| JayF | I don't have a good sense of if that's already extremely long | 17:53 |
| JayF | my preference is always to not truncate and instead show it all | 17:53 |
| * JayF points at node faults and regrets not implementing it again | 17:53 | |
| TheJulia | sort of why I put history in | 17:54 |
| TheJulia | I'll dig through the code and see what might be truncating it | 17:55 |
| JayF | I had a fleeting thought that I buried deep inside | 17:55 |
| JayF | of node.last_error being a pointer to a node history entry | 17:55 |
| JayF | so we could make the field larger without ballooning nodes | 17:55 |
| TheJulia | its not a bad idea, except I think the error is likely pre-truncated | 17:55 |
| JayF | but I'm sure that's a bad idea for other reasons I don't have the words for | 17:55 |
| TheJulia | at least, that is my gut feeling | 17:55 |
| TheJulia | well | 17:57 |
| TheJulia | So the fun thing with text and longtext is they are not actually in the table | 17:57 |
| TheJulia | the db keeps a pointer *and* like the first so many bytes | 17:57 |
| TheJulia | for index queries, if you query in such a way to table scan, then it goes and resolves all the pointers | 17:57 |
| TheJulia | and is overall just "mean" to the DB | 17:58 |
| * TheJulia is stupidly familiar with DB internals thanks to spending *many* years supporting https://en.wikipedia.org/wiki/Basis_database | 17:58 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!