| ricolin | Hi team, since feature freeze is less then a month, please kindly help to review on https://review.opendev.org/c/openstack/ironic/+/968348 this is a really useful fix when come to deploy new gen. GPU server IMO. Thank you! | 00:21 |
|---|---|---|
| JayF | +2A | 00:38 |
| ricolin | JayF: thanks! | 00:54 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove old -ipmitool job definitions https://review.opendev.org/c/openstack/ironic/+/974789 | 01:46 |
| opendevreview | Merged openstack/ironic master: Set columns in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/968348 | 01:58 |
| opendevreview | Steve Baker proposed openstack/ironic master: Add an autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/973187 | 02:24 |
| opendevreview | Steve Baker proposed openstack/ironic master: Replace ironic-tempest-bios-ipmi-direct with -autodetect https://review.opendev.org/c/openstack/ironic/+/974260 | 02:24 |
| opendevreview | cid proposed openstack/ironic master: Redfish VirtualMedia NFS and SMB/CIFS Support https://review.opendev.org/c/openstack/ironic/+/958505 | 03:43 |
| opendevreview | cid proposed openstack/ironic master: Redfish VirtualMedia NFS and SMB/CIFS Support https://review.opendev.org/c/openstack/ironic/+/958505 | 03:51 |
| opendevreview | Merged openstack/ironic master: Remove old -ipmitool job definitions https://review.opendev.org/c/openstack/ironic/+/974789 | 05:05 |
| dtantsur | JayF: yeah, I think the noop deploy is truly noop | 07:57 |
| rpittau | good morning ironic! o/ | 08:11 |
| abongale | good morning o/ | 09:53 |
| kubajj | Morning Ironic! o/ | 12:59 |
| kubajj | oops, lost a bit of irc | 12:59 |
| kubajj | janders: we only use Ironic with the rest of openstack for now, no metal3 | 12:59 |
| janders | kubajj ACK+THX | 13:22 |
| kubajj | dtantsur: about the default_inspect_interface, maybe I am wrong, but I still haven't figured out how it would be used. | 13:32 |
| rpittau | clif: I've answered your comments in https://review.opendev.org/c/openstack/ironic/+/974430 I wonder if dynamically generating the lists you pointed out actually makes sense, for the complexity I mean | 13:43 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Migrate database charset from UTF8MB3 to UTF8MB4 https://review.opendev.org/c/openstack/ironic/+/974430 | 13:46 |
| iurygregory | good morning ironic | 14:16 |
| TheJulia | stevebaker[m]: I'm leaving a quick comment on https://review.opendev.org/c/openstack/ironic/+/973187, I guess it might just be confusion or a slight oversight, dunno. Just let me know | 14:43 |
| TheJulia | Good morning! | 14:43 |
| clif | rpittau: yea, the other thought that occurs to me is if there's a way to add a double-check at the end of the migration that all tables/columns are at the new encoding | 14:44 |
| rpittau | clif: good idea, I'll add that | 14:52 |
| clif | awesome, thank you | 14:53 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Migrate database charset from UTF8MB3 to UTF8MB4 https://review.opendev.org/c/openstack/ironic/+/974430 | 14:58 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2024.2: [stable only][2024.1] Ensure firewalld is unmasked before enabling it https://review.opendev.org/c/openstack/bifrost/+/975169 | 15:12 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2024.2: [stable only][2024.2] Ensure firewalld is unmasked before enabling it https://review.opendev.org/c/openstack/bifrost/+/975169 | 15:13 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2025.1: [stable only][2025.1] Ensure firewalld is unmasked before enabling it https://review.opendev.org/c/openstack/bifrost/+/974855 | 15:14 |
| TheJulia | cardoe: adding some more tests now, but the agent is getting there. There might be some logic issues to address. my local environment is also seemingly broken... I think my bridge mappings are wrong at the moment which impacts things | 15:50 |
| JayF | dtantsur: it feels weird to me that a *deploy* interface being noop impacts the state of the network, but I guess that can make sense given how much of our network-swaps are driven by deploy flows | 15:54 |
| TheJulia | This is that whole meta bit that the deploy interface really is the business logic holder | 15:55 |
| JayF | yeah | 15:56 |
| dtantsur | true | 15:57 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: Trunk port reconciliation for L2VNI attachments https://review.opendev.org/c/openstack/networking-baremetal/+/974619 | 15:59 |
| opendevreview | Merged openstack/ironic master: feat: skip invalid mac addr interfaces in redfish inspect add speed https://review.opendev.org/c/openstack/ironic/+/973283 | 16:21 |
| opendevreview | Merged openstack/ironic master: devstack: Fix OVN init for hardware deployments https://review.opendev.org/c/openstack/ironic/+/974303 | 16:21 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2024.2: [stable only][2024.2] Ensure firewalld is unmasked before enabling it https://review.opendev.org/c/openstack/bifrost/+/975169 | 16:32 |
| TheJulia | cardoe: how is your mental modeling of identifying chassies and resources cross-host in OVN? | 16:42 |
| opendevreview | Merged openstack/ironic master: Drop encoding magic comment https://review.opendev.org/c/openstack/ironic/+/974341 | 16:46 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Migrate database charset from UTF8MB3 to UTF8MB4 https://review.opendev.org/c/openstack/ironic/+/974430 | 16:59 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Migrate database charset from UTF8MB3 to UTF8MB4 https://review.opendev.org/c/openstack/ironic/+/974430 | 17:02 |
| cardoe | TheJulia: evolving... you have something? | 17:11 |
| TheJulia | eh, the code is logically broken but I think its more confusion about field naming and value presence or not | 17:14 |
| TheJulia | I think I just sort of figured it out and presented it to claude on a platter | 17:14 |
| opendevreview | Clif Houck proposed openstack/ironic master: Prevent multiple attach actions being generated for the same port https://review.opendev.org/c/openstack/ironic/+/974520 | 17:58 |
| opendevreview | Clif Houck proposed openstack/ironic master: Filter out NoMatch actions in _vif_attach_tbn https://review.opendev.org/c/openstack/ironic/+/974569 | 17:58 |
| opendevreview | Clif Houck proposed openstack/ironic master: Update TBN config file to improve trait structure https://review.opendev.org/c/openstack/ironic/+/974776 | 17:58 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add an ordering method for TBN traits https://review.opendev.org/c/openstack/ironic/+/974519 | 17:58 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add default trait behavior to _vif_attach_tbn https://review.opendev.org/c/openstack/ironic/+/974960 | 17:58 |
| opendevreview | Clif Houck proposed openstack/ironic master: Filter out already attached portlikes in plan_vif_attach https://review.opendev.org/c/openstack/ironic/+/974521 | 17:58 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Update TBN simulator for vif_attach planning https://review.opendev.org/c/openstack/ironic/+/973691 | 17:58 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add max_concurrent_per_conductor_group config opt https://review.opendev.org/c/openstack/ironic/+/975195 | 18:29 |
| TheJulia | dtantsur: w/r/t https://review.opendev.org/c/openstack/ironic/+/955484 I left a comment concurring with your logging. If we can get a comment/committment/confirmation of a follow-up I'd be okay with +2+A'ing it, but as it stands now its a bit much logging wise, espescially since I know you have folks who want super low level data resolution | 19:07 |
| TheJulia | o/ jamesdenton | 19:34 |
| jamesdenton | this is the ghost of jamesdenton | 19:34 |
| jamesdenton | :D | 19:34 |
| jamesdenton | hi TheJulia! | 19:34 |
| TheJulia | ghosts! sweet! | 19:34 |
| gouthamr | iurygregory: rpittau: need your eyes on https://review.opendev.org/c/openstack/governance/+/974930 when you get a chance, please | 20:06 |
| iurygregory | gouthamr, done | 20:06 |
| gouthamr | ty iurygregory | 20:06 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: Trunk port reconciliation for L2VNI attachments https://review.opendev.org/c/openstack/networking-baremetal/+/974619 | 21:56 |
| TheJulia | cardoe: getting there, event driven responses is clearly brokenn | 22:05 |
| clif | anything we can do about redfish node_failure? | 22:36 |
| TheJulia | clif: what do you mean? | 22:36 |
| clif | I see node_failure on all my changes | 22:36 |
| TheJulia | oh, out of zuul | 22:37 |
| TheJulia | ummm hmmm | 22:38 |
| TheJulia | This is the tl;dr: https://opendev.org/opendev/zuul-providers/src/branch/master/zuul.d/providers.yaml#L643 | 22:43 |
| TheJulia | only one provider has it, so likely that is why | 22:45 |
| clif | just there are no nodes with the requested specs available? | 22:48 |
| TheJulia | at this moment, apparently so | 22:54 |
| TheJulia | fungi: you might have more context here, I'm not in front of matrix so dunno | 22:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!