| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Create service plugin to configure security groups https://review.opendev.org/c/openstack/networking-generic-switch/+/949834 | 00:16 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Support security groups in netmiko base class https://review.opendev.org/c/openstack/networking-generic-switch/+/949835 | 00:16 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Add security group support to netmiko_cisco_nxos https://review.opendev.org/c/openstack/networking-generic-switch/+/952314 | 00:16 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Add security group support to netmiko_sonic https://review.opendev.org/c/openstack/networking-generic-switch/+/955252 | 00:16 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: WIP Add security group support to ovs https://review.opendev.org/c/openstack/networking-generic-switch/+/956519 | 00:16 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Minor doc generation fixes https://review.opendev.org/c/openstack/networking-generic-switch/+/958498 | 00:16 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Add security group support to netmiko_dell_os10 https://review.opendev.org/c/openstack/networking-generic-switch/+/958499 | 00:16 |
| opendevreview | cid proposed openstack/ironic master: [WIP] Add NFS virtual media support https://review.opendev.org/c/openstack/ironic/+/958505 | 00:49 |
| TheJulia | cardoe: huh? | 02:58 |
| opendevreview | Ghanshyam proposed openstack/ironic master: DNM: Testign Nova service role https://review.opendev.org/c/openstack/ironic/+/958510 | 03:04 |
| cardoe | The redfish inspection series is down to 1 line change. | 04:22 |
| cardoe | Which failed in the snmp driver and metal3 installing centos9 | 04:23 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix insufficient mocking in rpc_service tests https://review.opendev.org/c/openstack/ironic/+/958517 | 07:56 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix insufficient mocking in rpc_service tests https://review.opendev.org/c/openstack/ironic/+/958517 | 07:59 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Launch API in the same process as conductor for singleprocess Ironic https://review.opendev.org/c/openstack/ironic/+/958462 | 07:59 |
| abongale | good morning ironic o/ | 08:01 |
| opendevreview | Kaifeng Wang proposed openstack/ironic master: Follow up to I9a5a2bfd9f4b538cc7217aefb7333df9ccdb9095 https://review.opendev.org/c/openstack/ironic/+/958522 | 08:24 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Launch API in the same process as conductor for singleprocess Ironic https://review.opendev.org/c/openstack/ironic/+/958462 | 10:01 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Trivial: remove meaningless logging https://review.opendev.org/c/openstack/ironic/+/958525 | 10:04 |
| dtantsur | TheJulia: https://review.opendev.org/c/openstack/ironic/+/958462 brings the 100 nodes test in the IrSO CI to around 6 seconds except for the TLS case (20 seconds). | 10:48 |
| cardoe | Metal3 is consistently failing in centos9 messing with the shim. | 11:50 |
| cardoe | https://zuul.opendev.org/t/openstack/build/5dba07d3f30f4856818574464b5b48d6 | 11:53 |
| opendevreview | Merged openstack/ironic master: Fix improper HTTP status code usage (RFC 7231) https://review.opendev.org/c/openstack/ironic/+/958454 | 12:27 |
| TheJulia | dtantsur: nice | 13:07 |
| TheJulia | iurygregory: do you have some time today to sync up w/r/t idrac10 ? | 13:47 |
| TheJulia | cardoe: dtantsur: looks like the job tries to pull in the aarch64 shim as well ?! | 14:07 |
| iurygregory | TheJulia, sure | 14:07 |
| TheJulia | iurygregory: now, later? | 14:08 |
| iurygregory | now it's ok, no meetings =) | 14:08 |
| TheJulia | would a meet work, just thinking quick high bandwidth for efficiency | 14:08 |
| iurygregory | yup | 14:09 |
| cardoe | What's wrong with iDRAC10? | 14:09 |
| iurygregory | we can't provision it :D | 14:09 |
| TheJulia | https://meet.google.com/tjg-nqkh-fcr?authuser=0 | 14:09 |
| iurygregory | if you were able to provision it please share how you did cardoe | 14:09 |
| TheJulia | cardoe: feel free to join ^ | 14:09 |
| cardoe | I don't know. My first pile of that gear is suppose to hit the DC floor later this week or early next week. | 14:10 |
| dtantsur | TheJulia: the aarch64 shim problem was temporary, rpittau already solved it in ironic-image. We just got an unlucky moment. | 14:14 |
| TheJulia | ok | 14:14 |
| dtantsur | cardoe: do you have any recent (the last few hours) failures with shim? | 14:15 |
| dtantsur | https://github.com/metal3-io/ironic-image/pull/734 merged 5 hours ago | 14:15 |
| iurygregory | Lol i can hear you anymore | 14:29 |
| cardoe | dtantsur: not since I linked it. But running another here... https://review.opendev.org/c/openstack/ironic/+/957609 | 14:39 |
| dtantsur | my patches passed 3 hours ago, so unless we have another problem, you should be good | 14:40 |
| cardoe | iurygregory: TheJulia: https://www.dmtf.org/sites/default/files/standards/documents/DSP0268_2024.2.html#payload-annotations | 14:43 |
| cardoe | Is their ComputerSystem annotated? Cause they can move that stuff to Settings maybe? | 14:43 |
| TheJulia | cardoe: computersystem, at least historically is annotated for the base fields, sub-structures I have no idea about | 14:45 |
| TheJulia | iurygregory: looks like your way to detect may be the model field under the Manager object, '15G Monolithic' in one of the logs you pointed me out. I mean, its not a great way but its better than trying to map the actual chassis hardware models | 14:47 |
| * TheJulia wonders if there are other fields but ones we don't directly model into our internal representation of the manager | 14:49 | |
| cardoe | There are. | 14:52 |
| cardoe | There's fields that come and go in versions as well. | 14:52 |
| cardoe | Well I dunno about manager. | 14:52 |
| iurygregory | TheJulia, yeah =) | 14:59 |
| iurygregory | cardoe, tks I will check after lunch | 15:00 |
| iurygregory | funny enough the SystemConfiguration is now different for the Slot in the Server https://github.com/dell/iDRAC-Redfish-Scripting/commit/f211e600cdfbd843984fed7598b1d04017a722de#diff-16af5fb78d1e5125515e3b9d46efa36674bccecbd08065dc6cf5ecb3b21afc42L91-R94 <this is fine gif> | 15:01 |
| JayF | iurygregory: cardoe: I too am invested in getting iDRAC 10 working, although I don't personally have access to hardware. If there's anything I can review, or if you want me to get a scouting report from any gear we might have remote hands on lmk. | 15:05 |
| JayF | RFR https://review.opendev.org/c/openstack/releases/+/958489 [ironic] Cycle highlights for Flamingo/2025.2 <-- please take a look when you get a chance | 15:08 |
| dtantsur | wow, we're so close now? | 15:10 |
| TheJulia | to release? | 15:10 |
| dtantsur | yeah | 15:10 |
| TheJulia | yeah, Yesterday I noted we likely need to release for the cycle in ~2 weeks | 15:10 |
| dtantsur | Then I'd appreciate pretty urgent review of https://review.opendev.org/c/openstack/ironic/+/958226 and https://review.opendev.org/c/openstack/ironic/+/958462 | 15:11 |
| dtantsur | I'd like them in earlier so that we can the performance signal before the final release | 15:11 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/957609 if that lands, then the release note for Redfish can also include inspector rules. | 15:11 |
| TheJulia | dtantsur: they were already on my plate for today | 15:13 |
| TheJulia | Just trying to use some overlap to begin to figure out idrac10 stuffs on my end. | 15:13 |
| TheJulia | o/ johnthetubaguy | 15:13 |
| dtantsur | thx! | 15:13 |
| TheJulia | cardoe: back to your queestion, it looks like they moved one of the annotations to only exist in their oemey field. https://etherpad.opendev.org/p/ironic-idrac10-issues#L30 (BootSourceOverrideTarget@Redfish.AllowableValues...) | 15:15 |
| * TheJulia will go see how horrible that may make things in a little bit | 15:16 | |
| cardoe | So where I'm getting at is that you can have that stuff under the Boot object AND require the /Settings | 15:21 |
| cardoe | e.g. if the ComputerSystem object from /redfish/v1/Systems/System.Embedded.1 has "@Redfish.Settings": { "SettingsObject": { "@odata.id": "/redfish/v1/Systems/System.Embedded.1/Settings" } } | 15:25 |
| cardoe | Then iDRAC 10 is standards compliant AFAIK | 15:25 |
| cardoe | iurygregory: TheJulia ^ | 15:25 |
| TheJulia | cardoe: I don't think it has any linkage, but I don't have a idrac10 machine to look at | 15:30 |
| TheJulia | so, we may be okay with some of the duplication, they do appear to set the annotation and we record it in _settings on our internal object in sushy | 15:50 |
| TheJulia | https://redfish.dmtf.org/schemas/DSP0266_1.7.0.html#settings-resource | 15:50 |
| TheJulia | err, not with the way we're patterned and the way its framed | 15:52 |
| TheJulia | its supposed to be future settings, but we only look on the base object it looks like because we're not expecting the future state to even be part of that | 15:53 |
| TheJulia | Has anyone seen if the centos image build related issues are still around today or resolved? They seem to think things are better today after the release they cut yesterday | 15:55 |
| TheJulia | iurygregory: I've made some updates to https://etherpad.opendev.org/p/ironic-idrac10-issues including what I expect the next issue to be, line 13. Line 17 is the basic issue, we need to get the hardware closer to upstream engagement to resolve these issues since we don't log the raw json payloads back and forth with the bmc | 15:57 |
| iurygregory | TheJulia, ack will take a look | 16:16 |
| cardoe | TheJulia: so TL;DR, Dell is compliant with the spec for a feature we've not supported in sushy? | 17:31 |
| TheJulia | cardoe: give me a few, I've got a 7715 on one screen and the spec in another | 17:31 |
| cardoe | TheJulia: the CentOS 9 image build is fixed... https://review.opendev.org/c/openstack/ironic/+/957609 just passed if ya wanna +2/+W that and we can then state redfish supports both hooks and rules. :D | 17:31 |
| TheJulia | cardoe: so reading the schema definition strictly, I don't think it is in compliance because your supposed to have the fields on the main property, and when the settings field exists, your supposed to have a subset of the existing values on the main object. What is "weird" is they also put BootSourceOverrideTarget@Redfish.AllowableValues value exclusively under the settings resource structure and it itself is a read only | 17:39 |
| TheJulia | value you cannot change | 17:39 |
| TheJulia | so its like.. in a rush to move some stuff to settings, they just moved the values and the representations in the api | 17:39 |
| TheJulia | I think the thing we could likely do is teach sushy to explicitly look for a sub setting resource and to see if the value is hiding there instead. | 17:40 |
| cardoe | So I know that's how NVIDIA (or whats the right marketing capitalization?) does their redfish for their hardware. | 17:40 |
| cardoe | They've got a settings reference and you've got the current state at the top level and the pending application state in the settings field. You then patch/post the settings. | 17:41 |
| TheJulia | Yeah, Settings is future state and you can change it | 17:43 |
| TheJulia | I'm going to try and connect sushy to this machine and see if it magically maps it together correctly, but I don't think (offhand) it will based upon a glance at the code | 17:44 |
| TheJulia | My concern is largely, dmtf says subset of values which can be changed, and an AllowedValues field inside of that is just... sort of breaking the meaning of the spirit but at the same times I could see "maybe you'd want it there?!" | 17:45 |
| TheJulia | then again, maybe you can change it and none of us have been crazy enough to try?! | 17:45 |
| JayF | I will note that the dell folks I talk to are *wildly insistent* that iDRAC 10 is DMTF compliant | 17:55 |
| JayF | if we find hard evidence it isn't, I can have folks talk to folks | 17:55 |
| dtantsur | It's not hard to be redfish compatible and still be useless.. | 18:14 |
| JayF | that is more-or-less what I told my downstream | 18:19 |
| JayF | surprise surprise that hardware vendors write a spec so loose that they can implement it and still be a PITA | 18:20 |
| JayF | lol | 18:20 |
| TheJulia | https://redfish.dmtf.org/schemas/DSP0266_1.7.0.html#settings-resource FYI | 18:22 |
| TheJulia | so, the tl;dr is the method to set a boot device should still work, but we're not going to pre-flight check the value because the allowed value list is in the wrong place | 18:26 |
| TheJulia | As long as it is a standardized value we're aware of | 18:27 |
| TheJulia | I think the question is, should we peek into a settings resource to hydrate our current representation of fields | 18:27 |
| TheJulia | which is sort of a double edged sword | 18:27 |
| TheJulia | we have a little bit (uefi http boot) reliance upon sushy's get_system result object boot.allowed_values being populated, fwiw | 18:31 |
| TheJulia | iurygregory: I think fixing that should semi-resolve most of the issues with the boot resource but it likely just quietly works in the non-vmedia path. I guess the thing I'm wondering is about more advanced code paths, and if someone like janders has done any digging there? | 18:37 |
| opendevreview | Merged openstack/ironic master: redfish: process inspection rules during inspection https://review.opendev.org/c/openstack/ironic/+/957609 | 18:42 |
| opendevreview | Julia Kreger proposed openstack/sushy master: WIP: handle idrac10 moving some boot items https://review.opendev.org/c/openstack/sushy/+/958578 | 19:35 |
| TheJulia | thoughts? | 19:35 |
| TheJulia | FWIW, that works, but I just tossed it together with claude code | 19:37 |
| cardoe | Probably okay | 19:40 |
| TheJulia | I *feel* like _load in general might want to make such a check, but boot is ... a weird middleground of sorts | 19:43 |
| TheJulia | and...yeah | 19:43 |
| TheJulia | might be best to have it housed in the cases we know where we might need to | 19:43 |
| cardoe | So that's what I was trying to go with https://review.opendev.org/c/openstack/sushy/+/955701 | 19:52 |
| TheJulia | I'm not sure its really a quirk given its sort of in the standard albeit all about interpretation | 19:53 |
| TheJulia | Standard says subset of values, not your values elsewhere | 19:53 |
| TheJulia | of course | 19:53 |
| TheJulia | If they moved them to /redfish/v1/Systems/System.Embedded.1/Oem/foo/bar/Settings, then yeah, it would totally be that way | 19:55 |
| cardoe | So a quirk of the standard? ;) | 19:55 |
| cardoe | But I understand its a safe fallback. | 19:56 |
| TheJulia | nah, it looks like you can "do" it anywhere, but quirks would be more that Oem is where they put it | 19:56 |
| TheJulia | (why do you hate us $vendor) | 19:56 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add node.instance_name https://review.opendev.org/c/openstack/ironic/+/952790 | 20:22 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Drop wsgi script, docs around mod_wsgi https://review.opendev.org/c/openstack/ironic/+/951055 | 20:25 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Revert "Make ironic-tempest-uefi-redfish-vmedia-4k non-voting" https://review.opendev.org/c/openstack/ironic/+/958580 | 20:29 |
| opendevreview | Merged openstack/ironic master: Trivial: remove meaningless logging https://review.opendev.org/c/openstack/ironic/+/958525 | 20:55 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: disable the novnc service on multinode https://review.opendev.org/c/openstack/ironic/+/958583 | 20:55 |
| TheJulia | stevebaker[m]: ^ I think that might result in some more happiness for the stack of ngs changes you have | 20:56 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Drop wsgi script, docs around mod_wsgi https://review.opendev.org/c/openstack/ironic/+/951055 | 21:03 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Drop wsgi script, docs around mod_wsgi https://review.opendev.org/c/openstack/ironic/+/951055 | 21:10 |
| opendevreview | Merged openstack/ironic master: Follow up to I9a5a2bfd9f4b538cc7217aefb7333df9ccdb9095 https://review.opendev.org/c/openstack/ironic/+/958522 | 21:11 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Change Portgroup minimum microversion to 1.26 https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/955799 | 21:29 |
| JayF | TheJulia: ^ that failure was caused by a bad image. | 21:43 |
| JayF | [ 422.080317] NetworkManager[1096]: <info> [1756239768.2987] exiting (success)[ 422.068623] EXT4-fs error (device dm-0): __ext4_get_inode_loc_noinmem:4702: inode #133939: block 524483: comm plymouthd: unable to read itable block | 21:43 |
| JayF | from [ 121.312895] dracut-initqueue[1172]: anaconda: fetching stage2 from https://mirror.stream.centos.org/9-stream/BaseOS/x86_64/os/ | 21:43 |
| JayF | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_9ad/openstack/9ad91f178bc2452cbab04fa33f07e7b1/controller/logs/ironic-bm-logs/node-1_console_2025-08-26-21%3A15%3A29_log.txt | 21:43 |
| JayF | hmmm I guess there's a small chance it's out of ram? | 21:43 |
| opendevreview | Jay Faulkner proposed openstack/ironic-tempest-plugin master: Anaconda science -- more ram? https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/958588 | 21:46 |
| JayF | that's bumping from 3.1ish -> 4 gig | 21:46 |
| JayF | just as science | 21:46 |
| TheJulia | quite possibly, the anaconda stuff is just ram hungry | 22:12 |
| TheJulia | since it builds the system in ram :( | 22:12 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!