| opendevreview | Yu Zou proposed openstack/ironic master: feat: add verify ca conf support for drivers https://review.opendev.org/c/openstack/ironic/+/947544 | 02:43 |
|---|---|---|
| opendevreview | Yu Zou proposed openstack/ironic master: test: add test cases for parse_driver_verify_ca https://review.opendev.org/c/openstack/ironic/+/947545 | 02:43 |
| opendevreview | Yu Zou proposed openstack/ironic master: feat: add verify ca conf support for drivers https://review.opendev.org/c/openstack/ironic/+/947544 | 04:00 |
| opendevreview | Yu Zou proposed openstack/ironic master: test: add test cases for parse_driver_verify_ca https://review.opendev.org/c/openstack/ironic/+/947545 | 04:00 |
| opendevreview | Yu Zou proposed openstack/ironic master: feat: add verify ca conf support for drivers https://review.opendev.org/c/openstack/ironic/+/947544 | 04:09 |
| opendevreview | Yu Zou proposed openstack/ironic master: test: add test cases for parse_driver_verify_ca https://review.opendev.org/c/openstack/ironic/+/947545 | 04:09 |
| AmarachiOrdor[m] | Good Morning Ironic! | 06:36 |
| opendevreview | 雒海艇 proposed openstack/networking-generic-switch master: Add H3C devices support https://review.opendev.org/c/openstack/networking-generic-switch/+/947554 | 06:49 |
| rpittau | good morning ironic! o/ | 06:54 |
| Ayo[m] | Good morning :) | 07:04 |
| freemanboss[m] | Good morning | 07:06 |
| abongale | Good morning ironic! | 07:07 |
| opendevreview | Adeola Adesina proposed openstack/bifrost master: Add notes to provide more clarity for bifrost installation https://review.opendev.org/c/openstack/bifrost/+/946603 | 10:50 |
| opendevreview | cid proposed openstack/ironic master: Make port binding failure (configurably) fatal https://review.opendev.org/c/openstack/ironic/+/699953 | 12:21 |
| opendevreview | cid proposed openstack/ironic master: Make port binding failure (configurably) fatal https://review.opendev.org/c/openstack/ironic/+/699953 | 13:06 |
| TheJulia | good morning | 13:33 |
| arne_wiebalck | Good morning, Ironic! | 13:46 |
| TheJulia | good morning arne_wiebalck! | 13:46 |
| arne_wiebalck | Hey TheJulia o/ | 13:46 |
| arne_wiebalck | Anyone ever tried to instantiate with Fedora CoreOS on a s/w RAID baremetal node? | 13:47 |
| * dtantsur hasn't | 13:47 | |
| TheJulia | I haven't heard of anyone trying. I would *suspect* there is a distinct possibility that is far outside of intended deploy/use case of FCOS that... maybe nobody has. | 13:49 |
| TheJulia | dtantsur: hey, with metal3, have y'all seen an issue where an agent starting ends up failing on get_deploy_steps, like the very first initial call? | 13:50 |
| dtantsur | TheJulia: I don't recall anything like that. Failing like what? | 13:51 |
| opendevreview | cid proposed openstack/ironic master: Make port binding failure (configurably) fatal https://review.opendev.org/c/openstack/ironic/+/699953 | 13:52 |
| TheJulia | causes provisioning to hang overall, the node deployment ends up in deploy wait continuing to heartbeat. Metal3 says the node is in provisioning but it never proceeds. I've got a downstream bug, hopefully well be able to spend time on it next week. The tl;dr appears to be the agent boots, heartbeats, and the conductor goes to retrieve get_deploy_steps and it seems like network connectivity goes down while that call is in | 13:53 |
| TheJulia | progress. It has been reported as an intermittent issue, and based upon the logs I've got it does appear that for that specific node/cluster, its like a 50% of the time thing for the node which is just bizarre. | 13:53 |
| dtantsur | Even if that happens, Ironic is supposed to retry on the next heartbeat.. | 13:54 |
| TheJulia | yeah, that logic seems to fail in this case. I expect I'll be looking at it next week | 13:55 |
| rpittau | talking about metal3, can I please get an approval here -> https://review.opendev.org/c/openstack/ironic/+/945069 :) | 14:02 |
| arne_wiebalck | TheJulia: The use case is a instantiating a bare metal node via Magnum as a K8s worker (FCOS is the default O/S for our K8s clusters atm) ... is that far outside the FCOS intended use case? | 14:10 |
| dtantsur | arne_wiebalck: ignition has its own basic sw RAID support | 14:10 |
| arne_wiebalck | dtantsur: oh, ok! | 14:10 |
| * arne_wiebalck wasn't aware | 14:10 | |
| dtantsur | arne_wiebalck: https://coreos.github.io/ignition/examples/#create-a-raid-enabled-data-volume | 14:11 |
| dtantsur | may be easier to use that | 14:11 |
| TheJulia | arne_wiebalck: the use case is realistically more modeled on support of a laptop, so raid in general combined with it is likely not explicitly tested, at all. | 14:11 |
| arne_wiebalck | dtantsur: will have a look, thanks! | 14:11 |
| arne_wiebalck | TheJulia: +1, ty | 14:11 |
| dtantsur | Although that won't work for the root volume, I guess.. | 14:12 |
| arne_wiebalck | Hmm ... controlling the root volume is one of the asks, ofc :) | 14:12 |
| arne_wiebalck | Maybe I need to do some partition/fs shrinking. | 14:13 |
| TheJulia | I'm assuming ostree, but I've not seen it's current structure. | 14:13 |
| opendevreview | Merged openstack/python-ironic-inspector-client master: Replace UPPER_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/947481 | 14:25 |
| opendevreview | Merged openstack/ironic master: metal3: Use zuul user for logs, fix remote node ip, reduce timeout https://review.opendev.org/c/openstack/ironic/+/945069 | 15:09 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Mark SNMP driver unsupported for removal https://review.opendev.org/c/openstack/ironic/+/946843 | 15:15 |
| * cardoe looks at all the yaks lined up to be shaved and sighs. | 15:17 | |
| TheJulia | cardoe: I know that feeling very well | 15:19 |
| TheJulia | Today I found Ilya wrote network metadata generation code in ironic already as I was already starting down a similar. On a plus side, its a unique enough case to the internals that it doesn't really make sense to try and merge it all together. | 15:21 |
| opendevreview | Habeeb Babasulaiman proposed openstack/bifrost master: doc: remove note that is not needed https://review.opendev.org/c/openstack/bifrost/+/947600 | 15:23 |
| * TheJulia ponders | 15:24 | |
| JayF | Satoshi's MLH fellowship demo day is today, he's not going to be with us much longer (at least on behalf of MLH/GR-OSS). I wanted to publicly thank him for the good work on container hardware manager and for tackling+shining some light on the image sharing bits. You're always welcome upstream if you wanna kill some time fixing cloudy bugs! :D | 15:26 |
| opendevreview | Merged openstack/ironic stable/2025.1: Fix mismatch in expected loop functionality https://review.opendev.org/c/openstack/ironic/+/947533 | 15:33 |
| cid | Satoshi \o/ | 15:35 |
| TheJulia | Anyone have an exemplar for bonded port metadata I can take a glance at? | 15:39 |
| TheJulia | Thank you satoshi! | 15:39 |
| JayF | TheJulia: https://github.com/canonical/cloud-init/issues/2698#issuecomment-1542397932 it's ugly and unverified, but it matches what I expected | 15:46 |
| JayF | and I saw a lot of those working on onmetal :D | 15:47 |
| TheJulia | ugh, That looks wrong, but might be cloud-init generic correct and not openstack corect | 15:57 |
| satoshi | Thanks a lot to the Ironic team for the support and guidance! I had many interesting developer experiences during the fellowship progarm from implmenting new features, imporove documentation, to barckporting and more. I will continue contributing to open source projects to further my skills. | 16:02 |
| JayF | TheJulia: that is vlan+bond config that looks like what we did at onmetal | 16:08 |
| JayF | TheJulia: maybe the fact it's vlan+bonding is adding too much complexity? | 16:08 |
| dtantsur | Easter holidays here folks, back on Tuesday. Have a nice (potentially extended) weekend! | 16:14 |
| JayF | o/ | 16:14 |
| TheJulia | o/ dtantsur | 16:21 |
| TheJulia | JayF: uhh, possibly, bonding gets a weird data structure, unfortunately | 16:21 |
| opendevreview | Habeeb Babasulaiman proposed openstack/bifrost master: bug: remove outdated release version https://review.opendev.org/c/openstack/bifrost/+/946725 | 16:22 |
| opendevreview | Habeeb Babasulaiman proposed openstack/bifrost master: bug: drop the mention of baremetal introspection list from the doc https://review.opendev.org/c/openstack/bifrost/+/946726 | 16:22 |
| opendevreview | Habeeb Babasulaiman proposed openstack/bifrost master: bug: indentation length fixed https://review.opendev.org/c/openstack/bifrost/+/946728 | 16:22 |
| opendevreview | Habeeb Babasulaiman proposed openstack/bifrost master: commit resolveds https://review.opendev.org/c/openstack/bifrost/+/946984 | 16:22 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic stable/2024.2: Improve is_image_available https://review.opendev.org/c/openstack/ironic/+/947432 | 17:21 |
| JayF | We looked at this ^ I am unsure if we should backport this fix any further, please someone take a look to see if you agree/disagree | 17:24 |
| JayF | IDK if I'm being extra risk averse or not | 17:24 |
| TheJulia | JayF: off hand, did the config default get swapped in th ebackport? | 18:12 |
| JayF | 100% yes | 18:13 |
| TheJulia | okay | 18:13 |
| TheJulia | thanks | 18:13 |
| JayF | my main concern is the is_admin change isn't backportedpast 2025.1 | 18:13 |
| JayF | and I think they should both backport or both not backport | 18:13 |
| JayF | and I lean towards "don't break existing users" | 18:13 |
| TheJulia | okay, that helps frame it | 18:13 |
| JayF | well, it wasn't backported to 2025.1; it got in before the release | 18:14 |
| TheJulia | ahhhh | 18:14 |
| TheJulia | oh | 18:14 |
| TheJulia | do you happen to know the change id off hand? | 18:14 |
| JayF | no, but there's not a lot by satoshi so it's easy to find | 18:14 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/943028 | 18:15 |
| JayF | it's really a question of "how bad is the bug vs how risky the solution is", and I think the answer is "not bad, and kinda risky" but almost because I'm afraid of people who were depending on bad behavior | 18:16 |
| JayF | and maybe makes it unsuitable for stable | 18:16 |
| * JayF notes we should release a patch release for 2025.1 with that patch asap | 18:17 | |
| * JayF looks into doing that | 18:17 | |
| TheJulia | yeah, I'm sort of leaning on maybe we don't | 18:18 |
| JayF | I am THRILLED we got it in 2025.1 | 18:19 |
| JayF | but stopping there seems best | 18:19 |
| TheJulia | the backport on 2024.2 *also* adds community as well | 18:19 |
| TheJulia | so its a feature at that point | 18:19 |
| JayF | #2 question on my mind if you were "nah we should backport it" is if we should change that to public only | 18:20 |
| TheJulia | I think w/r/t 2025.1, timeline wise, it is a behavior fix | 18:20 |
| JayF | https://review.opendev.org/c/openstack/releases/+/947620 Release point release for Ironic | 18:22 |
| TheJulia | sgtm | 18:23 |
| JayF | satoshi: ^ I know you're in the MLH demo day now, but if you get time, you can abandon the backports to stable/2024.2 and earlier; we're going to release 2025.1 with your fix (and cid's inspector rules fix) | 18:23 |
| satoshi | Thanks for having a look at. Two PRs are abondoned. | 18:29 |
| JayF | Sorry you had to write code that got abandoned :( The fix is awesome though | 18:30 |
| satoshi | It was a good practice for backporting. | 18:30 |
| JayF | cardoe: jamesdenton_: btw; I was looking at bitergia stats today, congrats on Rackspace getting #3 in Ironic contribution. I welcome the competition for #2 ;) | 18:30 |
| JayF | (over a 6 month period) | 18:30 |
| opendevreview | cid proposed openstack/ironic master: Allow special characters in patch field keys https://review.opendev.org/c/openstack/ironic/+/933743 | 19:33 |
| cardoe | oh dang. I think skrobul and keekz might have helped out there. | 20:02 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Base standalone job on redfish standalone https://review.opendev.org/c/openstack/ironic-python-agent/+/946580 | 20:13 |
| opendevreview | Habeeb Babasulaiman proposed openstack/bifrost master: bug: remove outdated release version https://review.opendev.org/c/openstack/bifrost/+/946725 | 20:48 |
| opendevreview | Habeeb Babasulaiman proposed openstack/bifrost master: doc: remove note that is not needed https://review.opendev.org/c/openstack/bifrost/+/947600 | 21:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!