| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [WIP] iDRAC10 force Slot1 for VirtualMedia https://review.opendev.org/c/openstack/ironic/+/961989 | 04:24 |
|---|---|---|
| rpittau | good morning ironic! o/ | 06:42 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Remove support for Python 3.9 https://review.opendev.org/c/openstack/ironic-python-agent/+/962156 | 07:27 |
| opendevreview | Kaifeng Wang proposed openstack/ironic master: Fix incorrect arg while getting ports by portgroup https://review.opendev.org/c/openstack/ironic/+/962043 | 07:42 |
| opendevreview | Merged openstack/sushy master: Restore flake8-import-order https://review.opendev.org/c/openstack/sushy/+/953124 | 07:45 |
| opendevreview | Merged openstack/ironic master: Fix OCI artifacts pointing to a single manifest https://review.opendev.org/c/openstack/ironic/+/960283 | 08:36 |
| opendevreview | Merged openstack/bifrost stable/2024.2: Fix ansible linters https://review.opendev.org/c/openstack/bifrost/+/962026 | 08:55 |
| opendevreview | Merged openstack/bifrost stable/2024.2: Do not pass empty values to instance_info https://review.opendev.org/c/openstack/bifrost/+/959277 | 08:58 |
| *** dmellado7 is now known as dmellado | 09:23 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2025.2: Fix OCI artifacts pointing to a single manifest https://review.opendev.org/c/openstack/ironic/+/962172 | 10:49 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/31.0: Fix OCI artifacts pointing to a single manifest https://review.opendev.org/c/openstack/ironic/+/962173 | 10:50 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/30.0: Fix OCI artifacts pointing to a single manifest https://review.opendev.org/c/openstack/ironic/+/962174 | 10:51 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2025.1: Fix OCI artifacts pointing to a single manifest https://review.opendev.org/c/openstack/ironic/+/962175 | 10:51 |
| opendevreview | Baptiste Jonglez proposed openstack/networking-generic-switch master: test_devices: Fix test to avoid scary backtrace https://review.opendev.org/c/openstack/networking-generic-switch/+/962178 | 11:38 |
| iurygregory | dtantsur, regarding "idrac11", it shouldn't happen since we will be returning 10 for any generation >= 16 in _get_idrac_version_from_model | 11:57 |
| dtantsur | iurygregory: ahhh, okay. this is a bit confusing, but since these all are internal functions, let's leave them this way | 12:10 |
| iurygregory | dtantsur, working on the changes you required and will add tests and releasenote | 12:11 |
| iurygregory | will also open a bug to have this tracked | 12:11 |
| iurygregory | since we only had https://bugs.launchpad.net/sushy/+bug/2111528 | 12:12 |
| iurygregory | https://bugs.launchpad.net/ironic/+bug/2125571 | 12:19 |
| alegacy_ | Any clue as to why this verify job is failing on this change? https://review.opendev.org/c/openstack/bifrost/+/962038 ... none of the issues flagged appear related to my changes. | 12:21 |
| dtantsur | alegacy_: https://zuul.opendev.org/t/openstack/build/f215398d403b431b8fab49b88bfb5879/log/job-output.txt#2931 may be related | 12:28 |
| dtantsur | alegacy_: while here, would it be possible to isolate the OVS code in a new role? | 12:29 |
| alegacy_ | dtantsur: thanks, I'll take a look at that line. | 12:45 |
| alegacy_ | dtantsur: for the OVS code isolation... are you referring to the install code, maybe drop a comment in the review so I'm clear on which part you mean. | 12:46 |
| TheJulia | good morning | 12:47 |
| dtantsur | alegacy_: ideally, all of it | 12:58 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962185 | 13:10 |
| alegacy_ | dtantsur: ok... i'll give it a go! | 13:51 |
| TheJulia | .... Crazy idea ---> What if we made an audit-esq capabilities spreadsheet. Auditors could *then* also go "audit" deployments. | 14:12 |
| *** diablo_rojo_phone_ is now known as diablo_rojo_phone | 14:19 | |
| *** diablo_rojo_phone is now known as Guest27275 | 14:20 | |
| TheJulia_ | Has anyone proposed meeting with Neutron during the PTG to Neutron? | 14:23 |
| TheJulia_ | It doesn't quite look like it thus far | 14:24 |
| iurygregory_ | TheJulia_, i'm finishing the email to send later today | 14:25 |
| iurygregory_ | got too happy with idrac10 provisioned in ocp and forgot | 14:26 |
| TheJulia_ | Okay, are you proposing any specific topics? I ask because we could start trying to bring clarity someplace | 14:26 |
| *** mnasiadka_ is now known as mnasiadka | 14:26 | |
| TheJulia_ | heh, okay | 14:26 |
| *** johnsom_ is now known as johnsom | 14:26 | |
| *** TheJulia_ is now known as TheJulia | 14:26 | |
| TheJulia | iurygregory_: go ahead and send the email, I can try and push in various areas which make sense from there | 14:27 |
| TheJulia | Thanks! | 14:27 |
| iurygregory_ | TheJulia, only supporting vxlan based terminations that you mentioned and is on the etherpad https://etherpad.opendev.org/p/ironic-ptg-2026.1#L86 | 14:28 |
| TheJulia | iurygregory_: WRT idrac10 beyond the slot, anything else new in all of that? | 14:28 |
| iurygregory_ | nope, things are working end to end | 14:28 |
| TheJulia | There may be some other topics networking related which may interest them | 14:28 |
| TheJulia | I suspect we'll have a solid "networking day" | 14:28 |
| iurygregory_ | the only weird thing I saw in ocp was that it didn't create the node for the bmh, but that is something else to debug | 14:29 |
| TheJulia | fun fun | 14:30 |
| iurygregory__ | jesus, connection today is a pain omg | 14:33 |
| *** johnsom_ is now known as johnsom | 14:39 | |
| *** mnaser_ is now known as mnaser | 14:39 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962185 | 14:49 |
| * TheJulia blinks and things "again?!" | 14:52 | |
| TheJulia | I'm suddenly having deja vu | 14:52 |
| dtantsur | One missing spot.. | 14:53 |
| dtantsur | Meanwhile, a newcomer on the Metal3 meeting mentioned their desire to provision 150k nodes. Will be fun. | 14:53 |
| TheJulia | wheeeeee | 15:15 |
| kubajj | dtantsur: first I've read cores and was like, nice, then read nodes and 👀 | 15:16 |
| TheJulia | rpittau: regarding your last, did you use a code assistant? | 15:20 |
| rpittau | TheJulia: yes, for the unit test part | 15:27 |
| rpittau | did I forget to add the mention in the commit? | 15:27 |
| rpittau | I did.... | 15:27 |
| rpittau | oh gosh I actually changed some stuff there and I forgot to add the parenthesis in L264 and L265 :D | 15:30 |
| *** cardoe_ is now known as cardoe | 15:32 | |
| cardoe | maybe my connection will be stable. | 15:33 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962185 | 15:33 |
| cardoe | So stupid idea if I can throw this out there and see if this makes sense to anyone. I'm still trying to reason about the whole "I have a pile of hardware and I'd like to make it available to people to consume in a fashion that they can get predicable hardware" | 15:34 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Trait Based Networking Filter Expression Parsing and Base Models https://review.opendev.org/c/openstack/ironic/+/961498 | 15:35 |
| cardoe | Today we've got a bunch of out of Ironic hardware categorization / inspection / etc but at the end of the day we set a resource_class so that we can consume it with a Nova flavor. | 15:35 |
| cardoe | How far can I go with traits on a node though... | 15:36 |
| cardoe | Could I in theory make an inspection hook which takes the PCI device data and say for class 02xx devices I add CUSTOM_ETH_14E4_16D7_14E4_4146 as a trait. Would I exceed any reasonable limits in ironic by having too many of those? | 15:38 |
| cardoe | I ask the group at large cause I feel like maybe I can craft something that would be an example use case and I could contribute the code and documentation around it. | 15:40 |
| TheJulia | rpittau: fwiw, I've seen some of the assistants do that before so I was like "huh?!" | 15:40 |
| cardoe | Now if I matched on that trait from my Nova flavor, users could ensure they're always getting that NIC in their machines. | 15:40 |
| JayF | dtantsur: closing loop from yesterday: [inspector]/require_managed_boot=false is what we needed to get the behavior we wanted. Thank you for helping guide me there. | 15:58 |
| dtantsur | nice! | 15:59 |
| JayF | managed inspection with unmanaged boot! so obvious :( | 16:00 |
| JayF | first time in a while *I* got decoder ringed by ironic lol | 16:00 |
| rpittau | TheJulia: the assistant in this case actually didn't put that, it used self.assertFalse(mock_download.called), and did some other shenanigans :D | 16:00 |
| rpittau | the structure was good, but I had to fix stuff, and forgot the parentheses | 16:00 |
| TheJulia | oh, heh | 17:09 |
| TheJulia | yeah, that also won't work, but fun! | 17:09 |
| TheJulia | rpittau: what are you using for an assistant? | 17:09 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for OVS as a virtual switch https://review.opendev.org/c/openstack/bifrost/+/962038 | 17:27 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: Detail availability zone modeling in ironic https://review.opendev.org/c/openstack/ironic/+/962206 | 17:41 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: remove backwards looking notes https://review.opendev.org/c/openstack/ironic/+/962207 | 17:41 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: Add some meta tags to key docs https://review.opendev.org/c/openstack/ironic/+/962208 | 17:41 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: trivial: minor typo fixes on the firmware list https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/962210 | 17:54 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: trivial: minor typo fixes on the firmware list https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/962210 | 17:59 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add force_inspection_dhcp option for managed inspection https://review.opendev.org/c/openstack/ironic/+/960824 | 18:03 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for OVS as a virtual switch https://review.opendev.org/c/openstack/bifrost/+/962038 | 18:24 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/957613/4#message-82c992a9e2a830b41dd90906e468d3d76180802d I suspect cid is looking for other core review opinions before taking aciton here is someone has a sec | 19:31 |
| JayF | also https://review.opendev.org/c/openstack/ironic-python-agent/+/958333 needs comments, it keeps getting passed over despite being -prio | 19:52 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Increase default sync_power_state_interval https://review.opendev.org/c/openstack/ironic/+/961554 | 19:54 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Increase default sync_power_state_interval https://review.opendev.org/c/openstack/ironic/+/961554 | 19:54 |
| TheJulia | uhh, where did my brain go | 20:11 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Force Virtual Media Slot 1 on iDRAC10 https://review.opendev.org/c/openstack/ironic/+/961989 | 20:12 |
| iurygregory_ | ok, should be good now, but I will test on bifrost again | 20:12 |
| TheJulia | I'm going to call it a day a little early. Got started around 5:30 AM. | 20:14 |
| TheJulia | JayF: on my list tomorrow to look at, I just don't have a brain left at the moment | 20:17 |
| JayF | ack, brain leakage is going around a bit | 20:17 |
| JayF | I might need to refill my brain at hte coffee shop lol | 20:18 |
| TheJulia | I think mine is demanding a nap | 20:18 |
| iurygregory_ | TheJulia, ack o/ | 20:34 |
| iurygregory_ | JayF, i'm trying to understand your comment .-., you mean in case a new gen of dell hardware is on the market (idrac11?! or idrac10 with 18G Monolithic) and have the same problems as 17G Monolithic has? | 20:39 |
| JayF | iDRAC 10 with 18G monolithic is exactly what I'm thinking | 20:39 |
| JayF | and today, we don't do a lot of version checking | 20:39 |
| JayF | so old ironic w/new hardware mostly works | 20:40 |
| iurygregory_ | ok, that is why we will consider anything higher than 16G as idrac10 | 20:40 |
| JayF | this would be introducing code that would make potentially otherwise functional machines stop working because a constant in the code didn't get changed | 20:40 |
| JayF | so I misread the code | 20:40 |
| iurygregory_ | https://review.opendev.org/c/openstack/ironic/+/961989/4/ironic/drivers/modules/drac/boot.py L86 | 20:40 |
| iurygregory_ | in case the generation is higher than 16 (17,18...) is idrac10 | 20:41 |
| JayF | vote flipped | 20:41 |
| JayF | I read that exactly reverse somehow | 20:41 |
| JayF | that was a few oz of coffee ago so maybe I've fixed it lol | 20:41 |
| iurygregory_ | I'm happy that I will be able to close this before doing my surgery on friday lol | 20:41 |
| JayF | oh hey hope you are well | 20:41 |
| iurygregory_ | I will be doing a septoplasty, will be back working on Oct 6 | 20:42 |
| iurygregory_ | 1 week off | 20:42 |
| iurygregory_ | going to grab some coffee and re-test in my bifrost env with the new code | 20:43 |
| JayF | ah good luck with that procedure! | 20:44 |
| wf-brodey | Hey everyone! Popping in for a quick question...Curious if it's possible to attach volumes to baremetal instances that were provisioned via nova. Ideally using nova to do the volume attachment dance. If not, I imagine I can do this through the baremetal apis directly? | 20:51 |
| TheJulia | Its slightly different, and ultimately depends on aspects like what your storage backend is | 21:38 |
| TheJulia | And then OS side, you may need to do some magic like ask for volumes to be refreshed if your doing something like iscsi | 21:40 |
| TheJulia | or FC | 21:40 |
| TheJulia | We do have the logic to support storage and transmisison of initiator details, which is sort of a key overall. | 21:40 |
| *** ex_tnode2 is now known as ex_tnode | 22:33 | |
| iurygregory_ | re-tested the deployment of idrac10 with the latest patchset https://review.opendev.org/c/openstack/ironic/+/961989, and it's working \o/ | 23:40 |
| *** iurygregory_ is now known as iurygregory | 23:41 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!