| opendevreview | Steve Baker proposed openstack/ironic master: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 00:48 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic master: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 01:50 |
| opendevreview | Steve Baker proposed openstack/ironic master: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 02:13 |
| *** hroy_ is now known as hroy | 05:58 | |
| *** jlejeune1 is now known as jlejeune | 07:00 | |
| rpittau | good morning ironic! o/ | 07:10 |
| iurygregory | good morning ironic o/ | 11:21 |
| opendevreview | Merged openstack/networking-generic-switch master: etcd: Discover api version automatically https://review.opendev.org/c/openstack/networking-generic-switch/+/954331 | 11:23 |
| opendevreview | nidhi proposed openstack/sushy master: Add comprehensive PCIe function support to Sushy https://review.opendev.org/c/openstack/sushy/+/962680 | 12:01 |
| opendevreview | Verification of a change to openstack/ironic master failed: ci: remove tinyipa job alias usage https://review.opendev.org/c/openstack/ironic/+/963001 | 13:05 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 13:24 |
| opendevreview | Marek Skrobacki proposed openstack/ironic-python-agent master: fix: docs(troubleshooting) update deprecated password hash algorithm https://review.opendev.org/c/openstack/ironic-python-agent/+/963286 | 13:39 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: remove gitignore change https://review.opendev.org/c/openstack/sushy/+/963289 | 13:48 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 13:58 |
| opendevreview | Merged openstack/ironic-python-agent stable/2025.1: Fix software RAID creation on different physical devices https://review.opendev.org/c/openstack/ironic-python-agent/+/962594 | 14:13 |
| cardoe | morning/afternoon ironic | 14:39 |
| cardoe | rpittau/TheJulia: longer term Aykut Bulgu's patch for parsing the error message is something I'd want to bring into that quirks code. Cause we'll be able to intelligently know what behavior pattern is needed from parsing that error msg. | 14:41 |
| rpittau | cardoe: yep, we got that from a bug opened downstream, maybe we should link it to something in launchpad too | 14:42 |
| cardoe | yeah makes sense | 14:43 |
| TheJulia | I would just caution us from setting ourselves up for a situation where we are not actually addressing the problems. | 14:46 |
| TheJulia | or making our ability to do so so slow that we are perceived as such. | 14:47 |
| alegacy | Sorry... forgot to send this earlier, but the adhoc meeting to discuss the import issue from yesterday is happening in a few minutes, here: meet.google.com/mzm-xtky-jbr | 14:57 |
| TheJulia | dtantsur: ^^^ | 15:00 |
| cardoe | cid: I just got to looking at your changes for sushy to be async... Did you have any plans on how to provide both going forward? | 15:01 |
| opendevreview | Merged openstack/ironic-python-agent master: fix: docs(troubleshooting) update deprecated password hash algorithm https://review.opendev.org/c/openstack/ironic-python-agent/+/963286 | 15:01 |
| cardoe | TheJulia: absolutely so we parse the error messages in multiple places differently... all I'm saying is we should have a shared function. | 15:02 |
| dtantsur | cardoe, cid, I looked into a hybrid sync/async approach at some point.. and while the session bits are easy with httpx, authentication and everything else is a total mess | 15:02 |
| dtantsur | we have a lot of places which can implicitly do a network call | 15:02 |
| cardoe | There's another spot that we just json.dumps() the entire return and look for the string in the whole output. | 15:02 |
| cardoe | dtantsur: so might rough thought was to convert everything to async and for the sync side we just create a thread and run asyncio in the thread and makes the sync calls just await the async calls in that thread. | 15:03 |
| dtantsur | cardoe: that's a realistic approach, I'm wondering what the overhead is | 15:04 |
| cardoe | That way we don't have to fix bugs in two different places. | 15:04 |
| cardoe | I dunno. I was just aiming for ease of maintenance. | 15:04 |
| * dtantsur nods | 15:04 | |
| dtantsur | In this case, we may just as well fork sushy and leave the old version alone | 15:04 |
| dtantsur | (this ties into my PTG question: should we maintain sushy as a separate project long-term) | 15:05 |
| opendevreview | Nahian Pathan proposed openstack/sushy master: Support expanded Chassis and Storage for redfish https://review.opendev.org/c/openstack/sushy/+/955211 | 15:05 |
| cid | cardoe, no solid plans really. I just had the draft that I thought is good enough to upstream to get the convo going (towards the PTG) | 15:05 |
| opendevreview | Morten Stephansen proposed openstack/ironic-python-agent master: WIP: Implement functionality for the is_root_volume RAID config https://review.opendev.org/c/openstack/ironic-python-agent/+/963200 | 15:06 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 15:11 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 15:14 |
| kubajj | dtantsur: TheJulia: we were thinking that since we are implementing is_root_volume=True, we should have a look at the other case - would it be sensible to prevent such a RAID from being picked as a root device (similar to what the skip list does)? | 15:16 |
| kubajj | (by skip list, I mean line https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/hardware.py#L1854 - where we just check skip list when listing block devices) | 15:21 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 15:21 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 15:38 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 15:39 |
| priteau | Anyone else around who would give another +2 to https://review.opendev.org/c/openstack/bifrost/+/962926? | 15:49 |
| JayF | priteau: that patch is strangely V+1 and also has lint issues | 15:50 |
| JayF | priteau: I'm not super knowledgable on bifrost so I'm not going to approve it b/c of that but it lgtm otherwise | 15:51 |
| priteau | Zuul shows these comments on every Bifrost patch, I haven't looked into it but it's not related to this change | 15:51 |
| priteau | Probably just some linter warnings | 15:52 |
| JayF | ack | 15:52 |
| JayF | I trust ya then and I don't see any compelling reason this could be dangerous | 15:52 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: remove hard requirement on neutron https://review.opendev.org/c/openstack/networking-generic-switch/+/963302 | 15:56 |
| TheJulia | dtantsur: alegacy: ^^^ | 15:57 |
| TheJulia | anyone else exceptionally concerned about dependencies, feel free to chime in. | 15:58 |
| dtantsur | thx | 15:58 |
| TheJulia | fwiw, the same basic logic might apply to neutron-lib as well, but that can also be broken out separately | 15:59 |
| dtantsur | neutron-lib is currently a hard requirement, not just for ML2 bits | 16:02 |
| TheJulia | so something to address then as time goes on | 16:10 |
| JayF | I mean, why? It's a library and an implementation detail. Not like taking on a hard dep to a service. | 16:12 |
| TheJulia | well, to soften the hard requirement so there is a dual use case which can exist | 16:18 |
| TheJulia | so I think ovn stuffs are broken for Ironic | 16:18 |
| TheJulia | I just raised a question in #openstack-neutron to see if anyone was aware of any changes on or after october 5th which would have caused stuff to break | 16:19 |
| JayF | I'm saying, I don't think a neutron-lib dep impedes a dual-use case | 16:24 |
| TheJulia | oh, it likely doesn't but the rest of the code in ngs might need to be double chcked anyhow | 16:25 |
| TheJulia | So looks like Neutron ended up landing 5 changes in the time window where it appears to have started braking | 16:44 |
| TheJulia | 4 of them are in the OVN code, so 4 suspects to eliminate | 16:45 |
| opendevreview | Merged openstack/bifrost stable/2025.2: Add support for Rocky Linux 10 image download https://review.opendev.org/c/openstack/bifrost/+/962926 | 17:35 |
| TheJulia | I'm trying to standup a local stack to start reverting out neutron patches to see if I can pin it down exactly | 17:45 |
| * TheJulia tries to convince devstack to stack | 17:49 | |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: devstack: Drop explicit etcd api version https://review.opendev.org/c/openstack/networking-generic-switch/+/957207 | 18:22 |
| opendevreview | Steve Baker proposed openstack/ironic master: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 19:43 |
| opendevreview | Steve Baker proposed openstack/ironic master: Ensure a stack user session is running for console containers https://review.opendev.org/c/openstack/ironic/+/963323 | 19:43 |
| opendevreview | Steve Baker proposed openstack/ironic master: Improve logging when console unit doesn't start https://review.opendev.org/c/openstack/ironic/+/963324 | 19:43 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 19:44 |
| TheJulia | https://bugs.launchpad.net/ironic/+bug/2127038 <-- Looking at this | 20:46 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Check _by_arch values with redfish vmedia https://review.opendev.org/c/openstack/ironic/+/963335 | 21:00 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Check _by_arch values with redfish vmedia https://review.opendev.org/c/openstack/ironic/+/963335 | 21:03 |
| opendevreview | Steve Baker proposed openstack/ironic master: Fixes for console containers functional testing https://review.opendev.org/c/openstack/ironic/+/963323 | 21:47 |
| opendevreview | Steve Baker proposed openstack/ironic master: Improve logging when console unit doesn't start https://review.opendev.org/c/openstack/ironic/+/963324 | 21:47 |
| denver_ba | Hi everyone, I'm Denver Baraka, an Outreachy applicant interested in contributing to the Ironic project - node deployment history api. Still setting up my environment and exploring. Looking forward to learn from the community. | 22:11 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 22:26 |
| JayF | welcome denver_ba | 22:34 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM/Test: Try turning on cleaning on pxe to see if behavior changes https://review.opendev.org/c/openstack/ironic/+/963342 | 22:55 |
| TheJulia | so, locally, cleaning works, I guess the question is if the job itself breaks sooner or later based upon cleaning. | 22:57 |
| opendevreview | Steve Baker proposed openstack/ironic master: Improve logging when console unit doesn't start https://review.opendev.org/c/openstack/ironic/+/963324 | 23:19 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 23:20 |
| tony__k | Hi everyone 👋 | 23:59 |
| tony__k | My name is Tony Muriuki, and I’m interested in learning about and contributing to OpenStack Ironic. | 23:59 |
| tony__k | Looking forward to collaborating and would love to connect with y’all too! | 23:59 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!