opendevreview | Merged openstack/bifrost master: Drop support for Ubuntu Focal https://review.opendev.org/c/openstack/bifrost/+/903975 | 00:02 |
---|---|---|
JayF | Hey everyone, please give a warm welcome to adamcarthur5 -- Adam is a GR-OSS developer who has been working on armadaproject.io for the last few months and will be shifting focus to OpenStack for the coming months (starting and onboarding with Ironic contributions). | 00:08 |
JayF | frickler: adamcarthur5 is who I intend to be helping us get some of the smaller OpenStack projects migrated off eventlet, as mentioned in #openstack-tc earlier this week. So maybe be extra nice to him :D | 00:09 |
TheJulia | adamcarthur5: welcome! | 00:12 |
opendevreview | Merged openstack/bifrost master: Drop support for rootwrap https://review.opendev.org/c/openstack/bifrost/+/903671 | 00:25 |
opendevreview | Merged openstack/bifrost master: inspector: add console=ttyS0 by default only when testing https://review.opendev.org/c/openstack/bifrost/+/901028 | 00:34 |
opendevreview | Merged openstack/bifrost master: Configure static DHCP before automated cleaning https://review.opendev.org/c/openstack/bifrost/+/903939 | 00:34 |
opendevreview | Merged openstack/bifrost master: Update ansible linters to current version used https://review.opendev.org/c/openstack/bifrost/+/903949 | 00:34 |
opendevreview | Merged openstack/bifrost master: Slightly reduce the disk size when testing DHCP https://review.opendev.org/c/openstack/bifrost/+/903948 | 00:34 |
opendevreview | Merged openstack/ironic master: Fix typo in xclarity docs https://review.opendev.org/c/openstack/ironic/+/904294 | 01:12 |
janders | Good morning Ironic o/ | 01:15 |
janders | Happy New Year 2024! | 01:16 |
janders | I am back from extended leave, will be around for foreseeable future | 01:16 |
opendevreview | Merged openstack/ironic master: Do not log lack of metrics support at WARNING lvl https://review.opendev.org/c/openstack/ironic/+/904455 | 02:08 |
opendevreview | Merged openstack/ironic master: [Docs] Adds changes to 'iLO' docs regarding Gen11 support https://review.opendev.org/c/openstack/ironic/+/904075 | 02:08 |
opendevreview | Merged openstack/ironic master: Remove unnecessary egg_info options https://review.opendev.org/c/openstack/ironic/+/904053 | 02:08 |
opendevreview | Jay Faulkner proposed openstack/ironic stable/2023.2: Do not log lack of metrics support at WARNING lvl https://review.opendev.org/c/openstack/ironic/+/904739 | 02:16 |
dtantsur | welcome adamcarthur5 and welcome back janders! | 08:50 |
dtantsur | JayF: no objections (re bug dashboard) | 08:52 |
adam-metal3 | Hello Ironic, Happy New Year to all of you! | 11:12 |
janders | thank you dtantsur | 11:59 |
janders | hey adam-metal3 o/ | 11:59 |
adam-metal3 | hey janders o/ | 11:59 |
opendevreview | Verification of a change to openstack/ironic master failed: Handle LLDP parse Unicode error https://review.opendev.org/c/openstack/ironic/+/903861 | 12:15 |
dtantsur | folks, could we merge https://review.opendev.org/c/openstack/ironic-lib/+/903815 and then release ironic-lib? | 13:30 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: DNM drop rootwrap support https://review.opendev.org/c/openstack/ironic/+/904836 | 13:35 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Configure the new inspection implementation https://review.opendev.org/c/openstack/bifrost/+/896925 | 14:15 |
dtantsur | has to rebase, also added CI jobs ^^ | 14:15 |
TheJulia | good morning folks! | 14:28 |
opendevreview | Julia Kreger proposed openstack/ironic master: Log upon completion of power sync https://review.opendev.org/c/openstack/ironic/+/891334 | 14:30 |
arne_wiebalck | Good morning, Ironic! | 14:35 |
TheJulia | dtantsur: any thoughts on backporting https://review.opendev.org/c/openstack/ironic/+/902801/ ? | 14:41 |
dtantsur | TheJulia: we haven't even decided on merging it at all :D | 14:41 |
TheJulia | To me, it might make sense to do so | 14:41 |
TheJulia | Well, yes, but a decision to backport can sometimes impact the overall patch, which is why I raise the question | 14:42 |
dtantsur | The opinions split between "don't care" and "meh, odd idea" | 14:42 |
dtantsur | We decided against backporting https://review.opendev.org/c/openstack/ironic/+/897071, so backporting this one is questionable | 14:43 |
TheJulia | While we don't see that exact case, we do have concern of someone malicious with access potentially flooding an ironic deployment with semi-legitimate work and want to prevent any sort of denial of service for essential functions, which seems like that would do | 14:43 |
TheJulia | set as a candidate, but never actually triggered, the overall value change is sort of the issue there I guess, but that is also somewhat also easily set by config too and as a default makes sense to just change in the development branch | 14:45 |
dtantsur | I'm not against backporting if the community sees value in it | 14:46 |
TheJulia | stevebaker[m]: a topic for next week when we have our next team meeting ^^^ | 14:47 |
TheJulia | dtantsur: also, https://review.opendev.org/c/openstack/ironic/+/888297 calls node_history_record which will also populate last_error, so that is user discoverable via the api surface and the state should end up going to maintenance in the end because the node just won't work. Not like the user is going to explicitly change the power state when enrolling a machine into the state machine. | 14:50 |
dtantsur | TheJulia: yep, just commented: last_error is going to be immediately overwritten, no? | 14:50 |
dtantsur | I'm quite sure we have some high-level try..except that will catch IloOperationError and update last_error | 14:51 |
TheJulia | ugh, your right!~ | 14:51 |
* TheJulia fixes | 14:51 | |
* TheJulia raises an eyebrow at some of the tests running oddly slow | 14:59 | |
TheJulia | eh, timeout tests | 15:01 |
opendevreview | Julia Kreger proposed openstack/ironic master: Detect ilo6 and redirect to redfish https://review.opendev.org/c/openstack/ironic/+/888297 | 15:01 |
TheJulia | ugh, the down side of putting something down for a month. No logs | 15:16 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Test multiple boot interfaces as part of one CI job https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/902171 | 15:27 |
opendevreview | Verification of a change to openstack/ironic-lib master failed: Compatibility with zeroconf 0.129.0 https://review.opendev.org/c/openstack/ironic-lib/+/903815 | 15:54 |
jamesdenton___ | good morning, Ironic. Looking at using the Ansible openstack.cloud module, and it seems target_raid_config support isn't fully supported. Is that the case? | 16:29 |
TheJulia | jamesdenton___: Not exactly surprising, most users tend to have pre-configured raid sets or not run raid at all, so entirely through the ansible modules not working is not a shock to me | 16:36 |
TheJulia | o/ Nisha_Agarwal | 16:36 |
jamesdenton___ | i don't mind working on that, but i think it might be deeper than the ansible. Looking at at the openstack client, --target-raid-config isn't available on create | 16:36 |
jamesdenton___ | but it is on set | 16:37 |
TheJulia | not every field is settable on node creation | 16:37 |
TheJulia | and one of those raid fields, I think is read only actually | 16:37 |
TheJulia | if it is CLI settable, it very likely is in openstacksdk, fwiw | 16:40 |
TheJulia | it triggers an rpc call to the conductor, so it is on it's own specific endpoint it looks like | 16:41 |
TheJulia | yeah, the driver has to be fully in place to validate the proposed configuration | 16:42 |
TheJulia | yeah, generally all the same validation logic but drivers can add in their own so they can do sanity/capability checks | 16:43 |
TheJulia | so if your saying "hey, bmc, I want to do this mystical magical raid configuration only this vendor knows about", the driver can do it and not detonate | 16:44 |
jamesdenton___ | Well i kinda assumed the api would intake any target raid configuration without validation, and then bork during the clean if it was wrong. Which is fine. I'm just surprised it's not possible to set the desired raid config on node enrollment/create | 16:47 |
TheJulia | cleaning, or deploy at this point, since you can do raid on deploy as well | 16:48 |
TheJulia | But on deploy also can give a bad taste of the config is wrong because it becomes a super hidden detail | 16:49 |
jamesdenton___ | oh really? nice | 16:49 |
TheJulia | and there is also no way for us to tell nova to go "your raid config was wrong" | 16:49 |
TheJulia | err, tell nova to go tell your api consumer "your raid config was wrong!" | 16:49 |
jamesdenton___ | :D | 16:50 |
jamesdenton___ | we love silent failures | 16:50 |
TheJulia | yeah, the stackhpc folks helped get that working with deploy steps | 16:50 |
TheJulia | so one of those cases where some upfront validation is really... as painful as it can be... kind of better | 16:50 |
TheJulia | Can everyone get silent failures?! | 16:50 |
TheJulia | we can start the "silent failure game show!" | 16:51 |
TheJulia | .... but nothing like "floor is lava" | 16:51 |
TheJulia | "Code is lava! The cooking contest show where developers try to make code food while in the lava is slowly creeping forward!" | 16:52 |
TheJulia | s/code food/code inspired food/ | 16:53 |
* TheJulia may have finally lost it | 16:53 | |
jamesdenton___ | that's like a real life implementation of "this is fine" meme | 16:54 |
* TheJulia awaits "Yes Chef!" as a test | 16:54 | |
TheJulia | We can't actually let everything be on fire. The insurance would be *insane* | 16:55 |
jamesdenton___ | are you aware of any ansible modules to trigger a clean? | 16:56 |
opendevreview | Merged openstack/ironic-lib master: Improve error message for wrong metrics backend https://review.opendev.org/c/openstack/ironic-lib/+/904457 | 16:56 |
TheJulia | originally, you'd use the provision state verb to move it to next state, an explicit module afaik does not exist | 16:57 |
TheJulia | I'm guessing you want a way to pass explicit clean_steps ? | 16:57 |
jamesdenton___ | yes | 16:58 |
TheJulia | I don't think we ever had a specific driver to add it into bifrost in terms of a use case we were running with in CI | 16:58 |
jamesdenton___ | no worries, i'm gonna look at the bifrost playbooks, i'm sure there's a lot to borrow there | 16:59 |
TheJulia | much more geared for specific cases, but was all geared for people to modify/hack and contribute back if they wanted | 17:03 |
jamesdenton___ | *heart* | 17:04 |
TheJulia | I feel like our tempest plugin is trying to gaslight me today | 18:31 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Test multiple boot interfaces as part of one CI job https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/902171 | 18:35 |
TheJulia | gah, it was all me | 18:35 |
opendevreview | Merged openstack/ironic-lib master: Compatibility with zeroconf 0.129.0 https://review.opendev.org/c/openstack/ironic-lib/+/903815 | 19:54 |
TheJulia | So I'm curious if there are any thoughts on https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/902171 specifically because I think the approach would allow us to consolidate ironic-tempest-uefi-redfish-vmedia, ironic-tempest-uefi-redfish-pxe into the ironic-standalone-redfish and cover the impending httpboot jobs as well all together | 20:15 |
TheJulia | we could also likely move towards mixing ipmi/redfish nodes in a same job set at that point, but that might be overkill. It ran on the redfish flavored job over at https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_221/902171/6/check/ironic-standalone-redfish/221ee41/testr_results.html and yes, some of it is intentionally opportunistic so the same job will just run if possible on | 20:19 |
TheJulia | ipmi | 20:19 |
TheJulia | we might be able to roll the ironic-tempest-ipa-partition-uefi-pxe-grub2 job into the ironic-standalone job more or less automatically as well | 20:35 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Change to enforced policy by default https://review.opendev.org/c/openstack/ironic/+/902009 | 21:07 |
opendevreview | Verification of a change to openstack/ironic master failed: Handle LLDP parse Unicode error https://review.opendev.org/c/openstack/ironic/+/903861 | 21:26 |
opendevreview | Merged openstack/ironic stable/2023.2: Do not log lack of metrics support at WARNING lvl https://review.opendev.org/c/openstack/ironic/+/904739 | 22:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!