| opendevreview | Steve Baker proposed openstack/ironic master: Replace cinderclient usage with openstacksdk https://review.opendev.org/c/openstack/ironic/+/900265 | 01:12 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic master: Replace cinderclient usage with openstacksdk https://review.opendev.org/c/openstack/ironic/+/900265 | 01:12 |
| janders | dtantsur ACK regarding client patch, thank you | 03:00 |
| *** jph8 is now known as jph | 03:06 | |
| rpittau | good morning ironic! o/ | 07:18 |
| masghar | o/ | 07:55 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix redifsh detach generic vmedia device method https://review.opendev.org/c/openstack/ironic/+/914978 | 09:03 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2024.1: Fix generating local paths when connecting virtual media https://review.opendev.org/c/openstack/ironic/+/915153 | 09:05 |
| dtantsur | rpittau: you still have the typo in the commit message | 09:07 |
| rpittau | \o/ | 09:07 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix redfish detach generic vmedia device method https://review.opendev.org/c/openstack/ironic/+/914978 | 09:08 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Add timeout to SERVICEWAIT https://review.opendev.org/c/openstack/ironic/+/915318 | 11:22 |
| dtantsur | janders: ^^^ | 11:22 |
| dtantsur | TheJulia: hey, do you remember why we even have https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/utils.py#L487-L504 ? | 11:28 |
| dtantsur | Is it something to do with taking over? | 11:28 |
| dtantsur | As you can see, it's not aware of servicing, so I'm wondering if we should fix it or just nuke | 11:28 |
| dtantsur | I mean, the comment does mention take over, but I'm not sure why we'd go to prepare_ramdisk in take over.. ramdisk deploy? | 11:30 |
| iurygregory | good morning Ironic | 11:50 |
| TheJulia | Good morning | 12:07 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Fix servicing clean-up https://review.opendev.org/c/openstack/ironic/+/915321 | 12:10 |
| dtantsur | TheJulia: morning! I might have misunderstood the intention behind the code in https://review.opendev.org/c/openstack/ironic/+/915321/1/ironic/drivers/modules/deploy_utils.py but it does not look right | 12:12 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add states.SERVICING and SERVICEWAIT to need_prepare_ramdisk https://review.opendev.org/c/openstack/ironic/+/915323 | 12:15 |
| dtantsur | mmm, I need to put more effort into it | 12:18 |
| janders | setting W-1 on 915323 above while we figure out the preferred way forward | 12:20 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Fix servicing clean-up https://review.opendev.org/c/openstack/ironic/+/915321 | 12:21 |
| dtantsur | janders: okay, this ^^ is probably correct | 12:22 |
| dtantsur | I could merge your change in, but it touches a different logical part | 12:22 |
| iurygregory | 35min for the PTG? | 12:25 |
| rpittau | yup | 12:25 |
| iurygregory | ack, time to update zoom | 12:26 |
| iurygregory | oh wait | 12:27 |
| iurygregory | now is meetpad lol | 12:28 |
| rpittau | I beleive we're using meetpad this time :) | 12:28 |
| iurygregory | yay, no need to update zoom \o/ | 12:28 |
| rpittau | you can connect directly clicking on the ironic reservation in the ptg page | 12:29 |
| iurygregory | yeah, I clicked and I was waiting to open zoom and complains about the version I have installed :D | 12:29 |
| TheJulia | Okay, finally sipping coffeeeeeeeee | 12:32 |
| rpittau | coffee FTW | 12:32 |
| * rpittau goes to make another espresso | 12:32 | |
| TheJulia | dtantsur: I suspect the form is more driven by unit tests failing and me going "well, that is a reasonable state, why it no work, oh!" | 12:33 |
| dtantsur | I can imagine | 12:34 |
| * dtantsur is not looking forward towards fixing the tests | 12:34 | |
| TheJulia | I copied a ton of tests and had to disqualify some but a number were like "oh, wow, yeah, that is a possible case" | 12:34 |
| TheJulia | dtantsur: w/r/t the utils change, it was because the conditionals were starting to stack up if memory serves when dealing with the ramdisk cases that could exist moving around the state machine, including in cases such as takeover | 12:37 |
| dtantsur | yeah | 12:41 |
| rpittau | Ironic PTG starting in 5 minutes https://etherpad.opendev.org/p/ironic-ptg-october-2023 | 12:56 |
| rpittau | of course I put the wrong link :D | 12:56 |
| rpittau | this is the correct one https://etherpad.opendev.org/p/ironic-ptg-april-2024 | 12:57 |
| opendevreview | Merged openstack/ironic master: Fix redfish detach generic vmedia device method https://review.opendev.org/c/openstack/ironic/+/914978 | 14:57 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2024.1: Fix redfish detach generic vmedia device method https://review.opendev.org/c/openstack/ironic/+/915341 | 14:59 |
| stephenfin | rpittau: JayF: Just to confirm, the OpenAPI discussion is scheduled at this time (after the current docs discussion), yes? I might drop to discuss the OpenAPI stuff with Cinder until the docs discussion concludes and rejoin at :30. Does that work? | 15:06 |
| JayF | > Add response schema validation (and fix any gaps in our request body and query string validation) | 15:06 |
| rpittau | stephenfin: that works, yes | 15:06 |
| JayF | is that openapi stuff? | 15:06 |
| stephenfin | Yup | 15:06 |
| JayF | I think we can ping you before we go into the topic | 15:06 |
| JayF | thanks :D | 15:07 |
| stephenfin | Sweet. Thanks | 15:07 |
| * stephenfin jumps to Cinder room | 15:07 | |
| JayF | stephenfin: I think we're getting close. Maybe 5 minute warning :) | 15:21 |
| JayF | stephenfin: we're good now :) | 15:23 |
| stephenfin | JayF: I'll be another 10-15, unfortunately | 15:25 |
| rpittau | good night! o/ | 16:33 |
| iurygregory | PTG is over? | 16:35 |
| iurygregory | lol I went to grab lunch and just got back :D | 16:35 |
| TheJulia | heh | 17:35 |
| TheJulia | iurygregory: we assigned all of the work to you ;) | 17:35 |
| iurygregory | TheJulia, oh noooooooo :D | 17:46 |
| * iurygregory runs | 17:47 | |
| dking | Is it time, guys? | 18:03 |
| JayF | PTG was this morning, starting about 6am PST aka 1300 UTC | 18:05 |
| JayF | we have more sessions later the week | 18:05 |
| dking | ...fail on my part. | 18:05 |
| dking | I must have a bad timezone. I put in 1300 UTC and Google calendar gave me now. | 18:06 |
| ikoliveira | hello everyone, I'm deploying a bm via bifrost, I've create an image for it using "diskimage-create ubuntu baremetal dhcp-all-interfaces grub2 devuser openssh-server -o deployment_image" and setting the following env vars DIB_DEV_USER_USERNAME and DIB_DEV_USER_AUTHORIZED_KEYS. The node reaches the provisioning state available, but i can't ssh to it, so I'm wondering if I'm using the right command to build the image... | 18:06 |
| JayF | dking: I usually just toss "1300 UTC in pacific time" in google and it does a good job | 18:07 |
| clarkb | now is 1800 UTC fwiw. In google calendar there is no proper UTC timezone but using reykjavic works instead (as they do not do DST and are +0) | 18:07 |
| JayF | Uh, there is, I thought? It's just listed as "Coordinated Universal Time" so it's weirdly abc'd | 18:08 |
| dking | I used to do that, but I thought it would be convenient to just put it in Calendar. | 18:08 |
| JayF | I used to do calendar invites for folks | 18:08 |
| JayF | just didn't this year, maybe will restart the habit | 18:08 |
| clarkb | JayF: I just checked on my phone and there is no UTC timezone that I can find | 18:09 |
| JayF | clarkb: https://usercontent.irccloud-cdn.com/file/JftSqY5I/image.png | 18:09 |
| clarkb | neat the web client and mobiel client differ then | 18:10 |
| JayF | I'm shocked that google would have inconsistent user experiences across platforms /s | 18:11 |
| dking | That's what I used. If I put in 1300 in that time zone, it gives me a few minutes ago. | 18:13 |
| JayF | That is strange behavior I've personally not experienced. | 18:13 |
| clarkb | local clock may be set as UTC but is off by 5 hours? | 18:14 |
| clarkb | I want to say this can happen if you dual boot linxu and windows | 18:14 |
| clarkb | because linux assumes the hardare clock is utc and windows set it to local time | 18:14 |
| dking | I'm not sure. I just put it into the Google Calendar website, so my local clock shouldn't affect that. | 18:14 |
| dking | Does anybody with a working Google want to make a calendar invite? | 18:15 |
| JayF | clarkb basically hit what I'd expect, and your local clock is 100% going to rule the roost there | 18:15 |
| JayF | if you're on something unix like, paste the output of `date` compared to `date -u` | 18:16 |
| TheJulia | I greatly dislike having to set something to UTC on Google calendar | 18:17 |
| dking | It worked great for when we were in Vancouver, and when I go other places. I'm not sure why UTC is different. | 18:18 |
| dking | I'm changing it to reykjavic. That seems to give me 8AM EST, so I'll try then. | 18:22 |
| dking | I'll probably be a few minutes late, though, as we have staff devotions right at 8am. | 18:23 |
| iurygregory | ikoliveira, the diskimage-create looks correct to me, but think I only used 2 or 3 times a while back, but I'm not an expert in DIB =) | 18:26 |
| TheJulia | so, crazy idea, for httpboot, any thoughts if we had a specific httpboot bootloader override to point to an ESP-ified ISO image... maybe that is silly because grub needs to be built to have networking | 23:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
