| cardoe | Don't you know we evaluate everyone on LOC? | 00:01 |
|---|---|---|
| JayF | That's how they did it 5 years ago. Now it's all based on how many plates^W AI agents you can keep spinning | 00:01 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: Test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 00:34 |
| opendevreview | Steve Baker proposed openstack/ironic master: Add an autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/973187 | 01:35 |
| opendevreview | Steve Baker proposed openstack/ironic master: Replace ironic-tempest-bios-ipmi-direct with -autodetect https://review.opendev.org/c/openstack/ironic/+/974260 | 01:35 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Change default value of some redfish firmware update config https://review.opendev.org/c/openstack/ironic/+/974800 | 04:09 |
| rpittau | good morning ironic! o/ | 07:27 |
| kubajj | good morning rpittau, and ironic! o/ | 07:31 |
| kubajj | is anything wrong with the metal3-integration job in Ironic? I keep getting an Internal Server Error when getting collection version metadata for kubernetes.core:6.2.0 | 07:33 |
| rpittau | kubajj: it's a problem with the ansible galaxy site, it's been fixed yesterday though | 07:42 |
| rpittau | recheck should work | 07:42 |
| kubajj | rpittau: will do, thanks (didn't want to waste recheck again) | 07:42 |
| janders | good morning Ironic o/ | 08:26 |
| abongale | good morning o/ | 09:05 |
| opendevreview | Abhishek Bongale proposed openstack/python-ironicclient master: Fix field names for selection and json https://review.opendev.org/c/openstack/python-ironicclient/+/973948 | 09:14 |
| opendevreview | Merged openstack/python-ironicclient master: Filter portgroups by shard https://review.opendev.org/c/openstack/python-ironicclient/+/970562 | 10:40 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Change default value of some redfish firmware update config https://review.opendev.org/c/openstack/ironic/+/974800 | 11:48 |
| dtantsur | cardoe: I don't understand why you say that the schema approach won't work.. | 12:53 |
| dtantsur | we literally do it for RAID | 12:53 |
| dtantsur | no objects to using something more sophisticated, but then we may need to fix other usages | 12:55 |
| dtantsur | * objections | 12:57 |
| cardoe | Opening a data file like that in the source tree won’t work correctly for many packaging | 13:12 |
| cardoe | Methods. That’s why pkg_resources was created in the first place. | 13:13 |
| cardoe | It definitely won’t work when that path is not called out in the packaging metadata. Now pbr does some magical things but I wouldn’t count on that forever. | 13:14 |
| dtantsur | cardoe: then how does it work for RAID? | 13:14 |
| dtantsur | I think if your packaging arbitrary omits files, it may be a problem with packaging | 13:15 |
| cardoe | I’m speaking to the Python packaging spec. | 13:15 |
| dtantsur | Well, that part evidently works unless I'm missing a subtle difference to the existing cases | 13:16 |
| * dtantsur still does not quite get why it should not work | 13:17 | |
| cardoe | https://importlib-resources.readthedocs.io/en/stable/using.html?utm_source=chatgpt.com | 13:18 |
| dtantsur | I've just built a wheel out of alegacy's patch, and it does contain the schemas | 13:18 |
| cardoe | https://docs.python.org/3/library/pkgutil.html | 13:18 |
| dtantsur | I don't mind us switching to a better way, I just don't understand why block the patch | 13:18 |
| cardoe | You’ve not -1’d patches cause there’s something technically incorrect that happens to pass tests? | 13:19 |
| dtantsur | define technically incorrect please | 13:20 |
| cardoe | You cannot open a file along side and installed Python file the way it is being done. | 13:20 |
| cardoe | It’s a violation of the Python packaging spec which is why there are a multitude of helpers over the years for this. | 13:20 |
| dtantsur | Citation needed | 13:21 |
| cardoe | https://docs.python.org/3/library/importlib.resources.html | 13:22 |
| cardoe | I’ve literally just sent you now 3 places in the Python docs that say this. | 13:22 |
| dtantsur | No, you did not | 13:23 |
| dtantsur | >>> from ironic.drivers.modules.network import ironic_networking | 13:23 |
| dtantsur | >>> len(ironic_networking._LAG_SCHEMA) | 13:23 |
| dtantsur | 8 | 13:23 |
| dtantsur | It literally works, installed in a venv from a wheel | 13:23 |
| cardoe | Read any of the 3 links. | 13:23 |
| dtantsur | Start with proving that RAID has never worked in Ironci pleas | 13:24 |
| dtantsur | Maybe, just maybe, nobody ever imports Ironic from a zip file | 13:25 |
| dtantsur | But since they're already badly broken (even iPXE uses local files), I'm not sure why we need to block a patch o nit | 13:25 |
| dtantsur | I mean all this stuff that also does not work wip zips https://opendev.org/openstack/ironic/src/branch/master/ironic/conf/pxe.py#L143-L147 | 13:26 |
| dtantsur | cardoe: my problem with this request is that we're tasking alegacy with fixing something that we've done for ages, and where some pondering is required about the path forward (i.e. how to handle operator-provided paths that are not packaged) | 13:30 |
| dtantsur | JayF: since you also put -1 ^^^ | 13:31 |
| cardoe | You’ve always been the one advocating for the code base to | 13:32 |
| cardoe | Improve and be technically correct. I’m surprised to hear to push for what’s clearly wrong because it works now. | 13:33 |
| cardoe | Override my vote and merge it anyway. | 13:33 |
| dtantsur | Heh, if I was pushing for everything to be perfect, you'd not be complaining about broken inspection rules :) | 13:33 |
| dtantsur | Look, whatever beef you have with me, could you please not apply it to alegacy? While we work in the same company, he's not even my teammate. | 13:34 |
| dtantsur | And he's doing a feature that nobody has dared taking for many years of taking about it. | 13:34 |
| cardoe | I have no beef or issue. I’m just saying I’m surprised. You’ve done a good job holding the project to a high standard. | 13:34 |
| dtantsur | Also, my point is not to keep this behavior forever, my point is deciding on how to fix it within the core team and fixing it then. If you want me to do it - happy to. | 13:35 |
| dtantsur | (I also don't mind alegacy just moving the schemas to constants as JayF suggested) | 13:35 |
| cardoe | I was trying to ask yesterday if the schemas should be constants or files. | 13:35 |
| dtantsur | We don't have a policy on that, to my best knowledge | 13:36 |
| cardoe | I even offered to convert them one way or the other if there was a preferred approach. | 13:36 |
| dtantsur | Some schemas can be overriden via configuration (network data, for instance), these make sense as files (and it's not trivial to use importlib for them because they can be operator-provided) | 13:37 |
| cardoe | Well when they are operator provided that path is different | 13:37 |
| dtantsur | I think RAID is the only one that we hardcode, maybe we could just move it to a constant | 13:38 |
| dtantsur | I think using constants for alegacy's work is a fair call (I'm personally happy with a follow-up). | 13:39 |
| cardoe | I did the -1 for awareness since it was gonna get merged in otherwise. | 13:40 |
| cardoe | iurygregory: just wondering if you're able to set Dell BIOS settings without booting into IPA? | 13:41 |
| iurygregory | cardoe, hey o/ | 13:41 |
| iurygregory | I haven't touched BIOS setting yet | 13:42 |
| dtantsur | IIRC we use IPA heartbeats as a sign that the machine is back | 13:42 |
| iurygregory | we are focused on the firmware updates first, trying to address the issues we see in our testing.. | 13:42 |
| cardoe | dtantsur: yeah. And Dell has some weird thing that SOMETHING must call ExitBootServices() which we solve by booting IPA since the Linux kernel does that for us. | 13:43 |
| dtantsur | yep, correct | 13:44 |
| cardoe | I'm just looking at an email where apparently we got a bunch of stuff from Dell and the only enabled boot device is the hard disk. So I need to enable something first. | 13:44 |
| * dtantsur is speechless | 13:45 | |
| cardoe | And I was just thinking if I could have Ironic enroll the node and then ensure a boot device is enabled before moving to inspection or any other state. | 13:45 |
| dtantsur | Sounds like a verify step to me | 13:46 |
| cardoe | Yeah. | 13:47 |
| dtantsur | (in metal3, we wipe idrac jobs as a verify step to make sure virtual media works - a somewhat similar case) | 13:48 |
| cardoe | So booting into IPA *AFTER* the BIOS change wouldn't be problematic. But right now it seems like Ironic wants it to boot into IPA prior to the BIOS change and then come back into IPA. | 13:48 |
| dtantsur | hmm, I think janders handled that for servicing, maybe we need to adapt his patch to cleaning | 13:49 |
| dtantsur | cardoe: I mean https://opendev.org/openstack/ironic/commit/1b2c01185c7907a1d17c4d90cf3bcd73fe345658 | 13:50 |
| cardoe | Yeah that makes sense. | 13:50 |
| cardoe | Curious to see how you add a verify step in Ironic | 13:50 |
| dtantsur | Quite similar to other steps https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/drac/management.py#L421 | 13:51 |
| dtantsur | but enabled in ironic.conf | 13:51 |
| dtantsur | https://opendev.org/openstack/ironic/src/branch/master/ironic/conf/conductor.py#L438-L447 | 13:52 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2025.1: [stable only] Ensure firewalld is unmasked before enabling it https://review.opendev.org/c/openstack/bifrost/+/974855 | 13:53 |
| dtantsur | cardoe: but note that verify steps are designed to be rather fast, I'm not sure about 15 minutes of rebooting the machine.. I also don't think we have any framework for async steps (no VERIFYWAIT) | 13:54 |
| cardoe | I could at least check that the right boot device is enabled. | 14:03 |
| JayF | dtantsur: cardoe: I would prefer as constant but my -1 was based on the (unsure if flawed) understanding we needed to switch it or add something to the project toml. If that's not true feel free to land over the -1 | 14:09 |
| JayF | I won't be at a desk to update that vote for a bit | 14:09 |
| cardoe | That's what I was saying that pbr might be overriding things for us. | 14:09 |
| dtantsur | it probably does | 14:10 |
| dtantsur | I've chatted with alegacy, he'll create a constant. No need to update the vote. | 14:10 |
| JayF | alegacy: My downstream got back to me about that log and said that not only was it okay from a security standpoint that it would be super duper useful | 14:39 |
| alegacy | JayF... ok, good enough for me, thanks. | 14:47 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Migrate database charset from UTF8MB3 to UTF8MB4 https://review.opendev.org/c/openstack/ironic/+/974430 | 15:15 |
| cardoe | rpittau: /home/zuul/src/opendev.org/openstack/bifrost/scripts/collect-test-info.sh: line 102: /home/zuul/openrc: No such file or directory | 15:23 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-specs master: Asynchronous sensor data collection https://review.opendev.org/c/openstack/ironic-specs/+/972754 | 15:34 |
| dtantsur | Claude things this specification ^^^ is "well-written with good technical detail, clear structure, and logical flow. The code examples are helpful and the reasoning is sound." You have no excuses not to review it :D | 15:36 |
| cardoe | I'll give it a look. Claude also says you can review some times in https://review.opendev.org/q/hashtag:ironic-week-prio+status:open ;) | 15:52 |
| opendevreview | cid proposed openstack/ironic-specs master: A specification for a node power lock feature https://review.opendev.org/c/openstack/ironic-specs/+/974894 | 16:11 |
| TheJulia | dtantsur: if that was a bar for reviews, the VXLAN spec would have landed long ago ;) | 16:24 |
| TheJulia | I’ll try to review it tomorrow, fwiw | 16:24 |
| dtantsur | well, yeah, there you also have the "not being entirely clueless" aspect :D | 16:25 |
| TheJulia | I have had several people remark a statement along the lines of: this vxlan stuff breaks my brain” | 16:26 |
| TheJulia | cardoe: oh, sounds like you might need a manageable -> cleaning -> manageable flow which then allows normal provide flow to proceed | 16:28 |
| cardoe | yep. but I've got the catch-22 of not being able to go into IPA | 16:29 |
| cardoe | TheJulia: did you see that person from ib systems asking about vxlan and bridging to physical hardware. | 16:29 |
| cardoe | I read their initial "this is what I was thinking" and I was like "oh my sweet summer child... I remember when I thought this too. then the hell of vxlan unleashed its face on me and I aged 20 years." | 16:30 |
| TheJulia | Heh, where was this?! | 16:30 |
| TheJulia | cardoe:yeah, the catch-22 is sort of provide *expects* to be able to boot and yeah | 16:32 |
| TheJulia | It… may be as simple as some state machine changes to enable a manual cleaning on that level | 16:33 |
| TheJulia | *may* | 16:33 |
| TheJulia | although, on aging 20 years, I don’t think anyone aged 20 years that I’ve talked to | 16:33 |
| cardoe | https://review.opendev.org/c/openstack/neutron/+/965053 | 16:34 |
| cardoe | Timeout reached while cleaning the node. Please check if the ramdisk responsible for the cleaning is running on the node. Failed on step {}. ORLY? | 16:35 |
| TheJulia | lol | 16:35 |
| TheJulia | cardoe:yeah, that be automated cleaning at it’s finest! | 16:35 |
| TheJulia | Boot the agent, find out what to do | 16:35 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Drop redundant basepython https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/974431 | 16:41 |
| TheJulia | FWIW, I also commented on there. | 16:43 |
| cardoe | Much better said than me. | 16:46 |
| cardoe | I also pinged ya on https://review.opendev.org/c/openstack/neutron/+/965415 | 16:47 |
| Continuity | Hey ironic'er | 16:55 |
| Continuity | Is anyone else being bitten by | 16:55 |
| Continuity | https://bugs.launchpad.net/neutron/+bug/1995078 | 16:55 |
| TheJulia | cardoe: yeah, I’d move it at this point. They are clearly disinterested. | 16:55 |
| cardoe | Continuity: yes | 16:56 |
| TheJulia | Could this be an opportunity for more ironic-neutron-agent to be… evil ?! | 16:56 |
| * TheJulia jumps on the next meeting | 16:56 | |
| Continuity | cardoe: any Perls of wisdom on a work around, one of my engineers had a pretty good work around for 2022.2 but a load of code changed since then | 16:57 |
| Continuity | Short of shouting at and manually changing it on the occasions when it goes wrong | 16:57 |
| Continuity | It seems very... brittle. | 16:57 |
| opendevreview | Verification of a change to openstack/ironic master failed: Set columns in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/968348 | 16:58 |
| cardoe | Working through this with the baremetal-l2vni and how to fix it fully. | 16:58 |
| Continuity | Just to confirm we are talking about the same thing, as sometimes these are a little obtuse | 17:00 |
| Continuity | Creating a new vlan backed provider network sometimes results in ha chassis and the ovn gateway chassis being out of sync | 17:00 |
| Continuity | Which then causes north south traffic form the baremetal node to be lost | 17:01 |
| cardoe | I'm using vxlan -> vlan but yes. | 17:05 |
| cardoe | It's in older OpenStacks. I've not run multiple in newer versions to confirm. | 17:06 |
| cardoe | But that's cause I'm trying to get us updated to the baremetal-l2vni setup first and then work there. | 17:06 |
| * TheJulia re-reads the bug | 17:24 | |
| Continuity | TheJulia: cadre: if we can work out a fix, or a work around I will personally buy you a case of your fav tipple | 17:24 |
| opendevreview | Merged openstack/ironic master: Populate the conductor group field from config https://review.opendev.org/c/openstack/ironic/+/974694 | 17:30 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Add project badge https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/972357 | 17:30 |
| TheJulia | Hmmmmm | 17:36 |
| TheJulia | So, I’m not an OVN expert, but I feel like I’ve been on a crash course as of recent | 17:36 |
| TheJulia | Seems like the “right” solution, is to evaluate the list of routers, and ensure the local ports are appropriately tagged out to the ha_chassis_group so they are appropriately scheduled out. I wonder if we can update post deploy and just handle it in some reconciliation logic | 17:40 |
| TheJulia | That at-least begins to anchor the routers into a specific chassis group which the networks would as well | 17:41 |
| TheJulia | Continuity: More of a pale ale person, but the brewery which makes my favorite is now like… a bazillion miles away and only makes a batch of their best every few years which makes me super sad. *cries* | 17:42 |
| Continuity | Yeah I was looking at the code for that whole area, its a little twisty and I keep getting lost :D | 17:42 |
| Continuity | It also seems to update EVERY time something changes | 17:43 |
| Continuity | Which can lead to flip flopping | 17:43 |
| TheJulia | I think what Gabriel posited works for bare metal which is just vlans, but I think that will compound as we support connecting Geneve/VXLAN to VLAN switches | 17:45 |
| TheJulia | Yeah, because every time it recalculates, without the anchor, i could see it trying to land everything elsewhere and things just breaking | 17:46 |
| TheJulia | It’s like the agent needs to go and say “for every lrp attached external port, regardless of type, where a vlan or vlan segment is attached, ensure we have the chassis group anchoring it | 17:47 |
| TheJulia | Since those are in groups of up to 5 hosts, it can continue to figure out where and let everything map | 17:47 |
| JayF | "I'm [snip] an OVN expert" - Julia ;) | 17:52 |
| TheJulia | Retro | 17:54 |
| TheJulia | There is likely also an opportunity to actually look and log “broken” cases | 17:55 |
| TheJulia | Like, if your router exists, it has bound ports, but not everything is online because it’s own external port is not online because it can’t be brought online, ovn localnet ports as it relates to the router never engage | 17:56 |
| TheJulia | And then never forward traffic. Your silently broken but don’t really know it because it is not blinking in your face that there is a misconfigured or broken state | 17:56 |
| TheJulia | Rutro, I might be becoming an OVN expert. Please send help. | 17:57 |
| Continuity | Yeah im in the same place.. its scary and I dont like it | 17:57 |
| TheJulia | I’d like my self hugging jacket in purple. | 17:58 |
| TheJulia | Continuity: I’m fairly certain this is something we can address in networking-baremetal because this whole thing keeps going around and around, and we can at least take enough of the information to fix or highlight the issues | 17:59 |
| TheJulia | cardoe: Do you have any thoughts? | 18:00 |
| cardoe | JayF: "I'm [snip] OVN [snip]." - Julia | 18:01 |
| JayF | Now I know that's a lie, because I like Julia :D | 18:01 |
| JayF | she makes sense most of the time | 18:02 |
| cardoe | TheJulia: uhh bagels are tasty? | 18:02 |
| cardoe | So nothing that's necessarily backportable. | 18:02 |
| TheJulia | JayF: Summer says “Can confirm… some of the time” | 18:02 |
| TheJulia | cardoe: moved on to the doctor’s office | 18:03 |
| cardoe | Neutron has recently acknowledged that the way the chassis IDs were being done by Neutron is all wrong. | 18:03 |
| TheJulia | The bagel was before this. | 18:03 |
| cardoe | And recently is like ~1 year. | 18:04 |
| TheJulia | cardoe: Is that the whole bug where they were like “the two chassis levels are bonkers” | 18:04 |
| cardoe | Maybe? | 18:04 |
| cardoe | So my rough understanding is that they should allocate the port to the HA Chassis Group and then OVN will schedule it to the Chassis and then two fields are different and unique values. | 18:05 |
| cardoe | And Helen might know more here. But that's how I've understood the behavior. | 18:05 |
| cardoe | The port can even get scheduled to multiple chassis (chassi? chassises?) but that's all recorded. | 18:05 |
| TheJulia | Yeah, except it gets created on the front end without the ha chassis group (afaik), and then it just gets scheduled where it thinks it can | 18:06 |
| cardoe | Right. Which is wrong. | 18:06 |
| TheJulia | Plural of chassis is chassis. I dislike the word so very much as a result. | 18:06 |
| TheJulia | I mean, its not a bad model if the creation knows where it needs to land or something reconciles it back in | 18:06 |
| TheJulia | It is one of those, it doesn’t know what it can’t possibly entirely know upfront | 18:07 |
| TheJulia | Things | 18:07 |
| cardoe | Well you can do that but then that's two different fields. | 18:07 |
| cardoe | So there were some patches to neutron to fix this a while ago in gerrit. There wasn't a lot of tests to the patches. | 18:09 |
| cardoe | But they got ignored and abandoned cause they auto-abandon old stuff. | 18:10 |
| cardoe | But they fix the issue. | 18:10 |
| cardoe | It's being addressed in neutron proper. | 18:11 |
| cardoe | For the network node getting the proper VLAN plugged up, we need to get the right event off the right chassis | 18:14 |
| cardoe | And have the port with the right device ID | 18:14 |
| TheJulia | I think the event matters a bit less as long as the ports get enough data for the mapping to occur | 18:20 |
| TheJulia | But, I guess sort of a different approach model, I always lean towards fix upfront if we can | 18:21 |
| opendevreview | Nahian Pathan proposed openstack/ironic master: Reduce API calls when collecting sensor data with redfish https://review.opendev.org/c/openstack/ironic/+/955484 | 18:40 |
| Continuity | When I looked at the code, it feels like it Guesses | 18:43 |
| Continuity | As to how to sync them up | 18:44 |
| Continuity | There used to be a periodic that ran to try and sync stuff up, but that seems to ahve gone away | 18:44 |
| Continuity | The strange thing, is that when a new node is added to the network, it should try and reconcile then, and sometimes it does, | 18:44 |
| Continuity | Its just very flakey | 18:44 |
| TheJulia | Continuity: do you need it to be very static, or would you be okay with any appropriate member of the matching physical networks being okay? | 19:09 |
| JayF | dtantsur: rpittau: cardoe: iurygregory: https://review.opendev.org/c/openstack/governance/+/974930 Please -1 this to indicate your willinness to continue as a DPL liason | 19:24 |
| cardoe | iurygregory: you still want https://review.opendev.org/c/openstack/ironic/+/972421 ? | 19:29 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Migrate setup configuration to pyproject.toml format https://review.opendev.org/c/openstack/networking-generic-switch/+/973526 | 19:54 |
| cardoe | ^ I don't even see what failed? | 20:17 |
| opendevreview | Merged openstack/ironic master: make inspection doc link better in the side menu https://review.opendev.org/c/openstack/ironic/+/974570 | 20:38 |
| Continuity | TheJulia: static is fine as long as it is consistent and works first time | 20:53 |
| opendevreview | Merged openstack/ironic bugfix/33.0: Fix introspection failure handling https://review.opendev.org/c/openstack/ironic/+/974070 | 21:01 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Change default value of some redfish firmware update config https://review.opendev.org/c/openstack/ironic/+/974800 | 21:08 |
| TheJulia | Continuity: so funny thing is that static is actually harder, but it would require almost managing placement of ha chassis groups | 21:11 |
| TheJulia | But I could almost see it desirable, as in if your underlying fabric prefers/needs MACs to stay pinned | 21:12 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Update TBN simulator for vif_attach planning https://review.opendev.org/c/openstack/ironic/+/973691 | 22:00 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add default trait behavior to _vif_attach_tbn https://review.opendev.org/c/openstack/ironic/+/974960 | 22:00 |
| opendevreview | Steve Baker proposed openstack/ironic master: Add an autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/973187 | 22:22 |
| opendevreview | Steve Baker proposed openstack/ironic master: Replace ironic-tempest-bios-ipmi-direct with -autodetect https://review.opendev.org/c/openstack/ironic/+/974260 | 22:22 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove old -ipmitool job definitions https://review.opendev.org/c/openstack/ironic/+/974789 | 23:08 |
| opendevreview | Nahian Pathan proposed openstack/ironic master: Reduce API calls when collecting sensor data with redfish https://review.opendev.org/c/openstack/ironic/+/955484 | 23:09 |
| cardoe | Should we https://opendev.org/openstack/ironic/src/commit/cd20dbc87e3b135954367820cb231e8bfdcd8a4c/ironic/drivers/modules/redfish/inspect.py#L184 create ports in redfish inspection or remove that and defer it to the hook? | 23:20 |
| opendevreview | Merged openstack/ironic bugfix/33.0: fix: port endpoints did not return vendor and category and fix docs https://review.opendev.org/c/openstack/ironic/+/973533 | 23:39 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!