| opendevreview | Ivan Anfimov proposed openstack/ironic-ui master: Remove unicode prefix from code https://review.opendev.org/c/openstack/ironic-ui/+/956615 | 00:37 |
|---|---|---|
| opendevreview | Ivan Anfimov proposed openstack/ironic-ui master: wip https://review.opendev.org/c/openstack/ironic-ui/+/956615 | 00:40 |
| opendevreview | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-ui/+/956624 | 03:31 |
| rpittau | good morning ironic! o/ | 06:33 |
| queensly[m] | Good morning o/ | 07:11 |
| opendevreview | Tatiana Kholkina proposed openstack/ironic master: Initialize variable to prevent an error https://review.opendev.org/c/openstack/ironic/+/956629 | 07:57 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic bugfix/31.0: Update .gitreview for bugfix/31.0 https://review.opendev.org/c/openstack/ironic/+/956635 | 08:40 |
| opendevreview | Dmitriy Chubinidze proposed openstack/ironic-ui master: Capitalize RAID https://review.opendev.org/c/openstack/ironic-ui/+/956647 | 09:57 |
| opendevreview | Dmitriy Chubinidze proposed openstack/ironic-ui master: Capitalize RAID https://review.opendev.org/c/openstack/ironic-ui/+/956647 | 09:58 |
| opendevreview | Merged openstack/ironic bugfix/31.0: Update .gitreview for bugfix/31.0 https://review.opendev.org/c/openstack/ironic/+/956635 | 11:02 |
| rpittau | Python 3.9 support removal patches are ready to be merged | 11:07 |
| rpittau | plase notice that in Bifrost CI will be broken until we merge https://review.opendev.org/c/openstack/bifrost/+/955181 and https://review.opendev.org/c/openstack/bifrost/+/955660 | 11:07 |
| dtantsur | rpittau: a comment on each | 11:17 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent stable/2025.1: Hint at sector sizes when reporting an invalid written image https://review.opendev.org/c/openstack/ironic-python-agent/+/956662 | 12:09 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Updated pinned upper-constraints for Python 3.9 https://review.opendev.org/c/openstack/bifrost/+/955181 | 12:41 |
| TheJulia | good morning | 13:04 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Move all CS9 jobs to non-voting https://review.opendev.org/c/openstack/bifrost/+/955660 | 13:11 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Move all CS9 jobs to non-voting https://review.opendev.org/c/openstack/bifrost/+/955660 | 13:12 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Move all CS9 jobs to non-voting https://review.opendev.org/c/openstack/bifrost/+/955660 | 13:13 |
| cardoe | morning Ironic | 13:18 |
| cardoe | TheJulia: did you find your brain yet? or still brains hunting? :-D | 13:19 |
| cardoe | JayF: I also realized the source of all my frustrations relates to you. It's also only fitting that you're now a Gentoo dev after I've retired. | 13:19 |
| TheJulia | cardoe: om nom nom brraaaainnsss | 13:21 |
| cardoe | JayF: it's all in jest btw... you suggested something over 10 years ago as a review comment but said you're fine without it or as a follow on... you were only a +1 on that project but none of the +2's echoed your comment... and turns out you were right. | 13:23 |
| cardoe | free brains for a +2 on https://review.opendev.org/c/openstack/ironic/+/955536 :D | 13:26 |
| TheJulia | now, are these brain cookies... or what? :) | 13:48 |
| opendevreview | Merged openstack/ironic master: fix up redfish inspection mock ethernet interface data https://review.opendev.org/c/openstack/ironic/+/955536 | 13:48 |
| cardoe | brain cookies seem like power food | 13:49 |
| cardoe | sorry that one was just bugging me that we were testing entirely the wrong thing and just making the tests pass. That's ACTUALLY testing the behavior and I can actually confirm the behavior in the code now. | 13:49 |
| dtantsur | JayF: you might be not the only one with weird IPv6 setup: a part of Metal3's CI also fails.. | 14:01 |
| opendevreview | cid proposed openstack/ironic master: Handle race conditions in power state changes https://review.opendev.org/c/openstack/ironic/+/956674 | 14:08 |
| TheJulia | :( | 14:31 |
| JayF | cardoe: what are you referencing? | 14:43 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/31.0: Comment out metal3 integration job https://review.opendev.org/c/openstack/ironic/+/956685 | 14:50 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix local RPC IPv6 detection to use socket binding instead of file checks https://review.opendev.org/c/openstack/ironic/+/956687 | 14:57 |
| dtantsur | JayF: can you still reproduce your odd setup? it would be great if you could try ^^ | 14:57 |
| JayF | just echo 0 > /path/to/lo/ipv6_disable | 14:58 |
| JayF | I forget the exact path, but the one in /sys where you can disable per interface; I was disabled on lo | 14:58 |
| dtantsur | okie | 15:00 |
| dtantsur | (if the commit message seems unusual for me, see the generated-by tag) | 15:01 |
| cardoe | JayF: speaking about vlan stuff in network-data but I sent ya a longer /query | 15:06 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Remove ancient Python 3.6 tox test https://review.opendev.org/c/openstack/bifrost/+/956694 | 15:16 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/31.0: Fix local RPC IPv6 detection to use socket binding instead of file checks https://review.opendev.org/c/openstack/ironic/+/956701 | 15:42 |
| dtantsur | rpittau: in addition to your bifrost patches, we need to consider changing the bifrost job on ironic (there is one cs9 job now) | 16:50 |
| rpittau | dtantsur: it's non-voting already, unless we want to remove it | 17:26 |
| rpittau | or wait | 17:27 |
| rpittau | change to ubuntu-noble maybe | 17:27 |
| opendevreview | Merged openstack/ironic master: Fix local RPC IPv6 detection to use socket binding instead of file checks https://review.opendev.org/c/openstack/ironic/+/956687 | 17:46 |
| cardoe | how would folks feel if I made a DEFAULT.state_dir that defaults to /var/lib/ironic and then made everything else that does path stuff use that as the base instead of hard coding out /var/lib/ironic? | 18:03 |
| cardoe | For some reason OpenStack Helm forces the usage of /var/lib/openstack-helm instead but they've missed a few config knobs. | 18:06 |
| opendevreview | cid proposed openstack/ironic master: Add node.instance_name as API microversion 1.100 https://review.opendev.org/c/openstack/ironic/+/952790 | 18:19 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add node.instance_name as API microversion 1.100 https://review.opendev.org/c/openstack/ironic/+/952790 | 18:20 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Optional indirection API use https://review.opendev.org/c/openstack/ironic/+/956504 | 19:09 |
| TheJulia | cardoe: its not an awful idea, really | 19:11 |
| * TheJulia hopes CI lets new futurist in before tomorrow | 19:14 | |
| guilhermesp | hi team! its me again sharing my saga with lenovo systems :P this time its another deployment which uses almalinux 9 and have lenovos using VROC. So anyone with experiencie with metal3/ironic and raids coming from intel VROC? | 19:16 |
| guilhermesp | inspection is showing me a /dev/md126 but if i set this as the root device, thats what i get: | 19:17 |
| guilhermesp | https://www.irccloud.com/pastebin/zTzOx5la/ | 19:17 |
| TheJulia | ... vrocs, really | 19:18 |
| * TheJulia sighs | 19:18 | |
| * guilhermesp shares the same feeling | 19:18 | |
| * TheJulia feels like she needs a Bar | 19:18 | |
| guilhermesp | and even though, if i just use regular software raid, almalinux will just enter emergency mode :P | 19:18 |
| * TheJulia wonders if maybe that should be dinner, for it opens at 4 PM. | 19:18 | |
| TheJulia | so... hmm | 19:20 |
| guilhermesp | its been a loooot of fun lately | 19:20 |
| guilhermesp | but yeah, i think at the end i will have to deploy the system with a single disk and covert to software raid somehow :P | 19:21 |
| TheJulia | I'm thinking because it triggers a software raid device on driver load, and presents the entire disk as it... maybe we need to detect "Could not get holder disks of /dev/md.*" and keep the code out of the software RAID path | 19:21 |
| TheJulia | it will take code being added to the agent | 19:23 |
| TheJulia | guilhermesp: do you have logs which match the ironic-python-agent execution? | 19:25 |
| guilhermesp | i can try to figure how to store the logs when using metal3 and standalone ironic here, i was trying to get this and enable ipa debug to get more details | 19:26 |
| TheJulia | I think it is going to require changes to the efi_utils.py manage_uefi method | 19:27 |
| TheJulia | since we treat all md devices as software raid and head down that path then, we need to try to get holders earlier I guess | 19:27 |
| guilhermesp | that makes sense and i figure could be something like that | 19:29 |
| guilhermesp | and the story of no being able to just https://book.metal3.io/bmo/raid?search=#software-raid is a almalinux thing | 19:29 |
| guilhermesp | since i was able to do this running ubuntu | 19:29 |
| TheJulia | Yeah, I guess I'm not really sure because there are a couple places where that can sort of begin to bubble up | 19:31 |
| TheJulia | but mange_uefi is sort of the most likely case/position | 19:31 |
| guilhermesp | yeah i might try seeing how legacy works here in this case | 19:32 |
| TheJulia | that might be how you avoided it otherwise on that hardware | 19:33 |
| TheJulia | but I'm sorry you have a VROC card | 19:33 |
| TheJulia | Clearly we need a bar tender in channel! | 19:33 |
| guilhermesp | lol | 19:34 |
| guilhermesp | thanks for the insights though TheJulia -- hope you enjoy a bar today | 19:35 |
| TheJulia | I might not today, I might cook in tonight instead. | 19:41 |
| TheJulia | Anyway, I need to go do things and stuff in workday. \o/ | 19:41 |
| JayF | Is VROC rebranded RSTe? Or at least a similar idea? | 19:44 |
| TheJulia | JayF: yup | 19:45 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [trivial] Fix rendering of 1.98 header https://review.opendev.org/c/openstack/ironic/+/956733 | 20:48 |
| TheJulia | JayF: fwiw, Intel's webpage says "formerly known as"... so yeah. | 21:44 |
| opendevreview | Merged openstack/ironic master: [trivial] Fix rendering of 1.98 header https://review.opendev.org/c/openstack/ironic/+/956733 | 21:47 |
| JayF | in which case cardoe has some 12-ish year old notes on it somewhere buried in a wiki :P | 21:48 |
| JayF | the real useful bit of which was the mdadm create command that would avoid doing a full raid reassembly if you knew the disks were blank | 21:49 |
| TheJulia | but we've added a lot of code since then, and it is recognized as a software raid device | 21:49 |
| JayF | yeah, that was back when all the raid stuff was just done in onmetal cleaning | 21:50 |
| JayF | but rste mostly looked like software raid except with lots of undocumented restrictions which caused failures :) | 21:50 |
| JayF | (unless you knew the right flags for mdadm) | 21:51 |
| janders | good morning Ironic o/ | 22:13 |
| janders | TheJulia w/r/t the issue we discussed yesterday, would you like me to file the bug 1) as a defect in hold/wait steps, or 2) lack of easy way to back out of service failed? | 22:14 |
| opendevreview | Merged openstack/ironic-ui master: Remove Python 3.9 support https://review.opendev.org/c/openstack/ironic-ui/+/953362 | 22:43 |
| opendevreview | Merged openstack/python-ironicclient master: Remove Python 3.9 support https://review.opendev.org/c/openstack/python-ironicclient/+/953363 | 22:45 |
| opendevreview | Merged openstack/python-ironic-inspector-client master: Remove Python 3.9 support https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/953364 | 22:46 |
| opendevreview | Merged openstack/ironic-inspector master: Remove Python 3.9 support https://review.opendev.org/c/openstack/ironic-inspector/+/953361 | 22:47 |
| opendevreview | Merged openstack/networking-generic-switch master: Remove Python 3.9 support https://review.opendev.org/c/openstack/networking-generic-switch/+/953365 | 22:49 |
| opendevreview | Merged openstack/ironic master: Remove Python 3.9 support https://review.opendev.org/c/openstack/ironic/+/953360 | 22:55 |
| opendevreview | Merged openstack/ironic master: Follow-up: Clear `last_error` on power match/sync https://review.opendev.org/c/openstack/ironic/+/956511 | 23:15 |
| opendevreview | Merged openstack/bifrost master: Updated pinned upper-constraints for Python 3.9 https://review.opendev.org/c/openstack/bifrost/+/955181 | 23:33 |
| opendevreview | Merged openstack/networking-baremetal master: Remove Python 3.9 support https://review.opendev.org/c/openstack/networking-baremetal/+/953366 | 23:39 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!