| iurygregory | TheJulia, any other patches for the httpboot you would like some attention? | 00:28 |
|---|---|---|
| iurygregory | If anyone has some time for review https://review.opendev.org/c/openstack/ironic/+/894918 | 00:29 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/900964 but I'll need to cut a sushy release first | 00:29 |
| iurygregory | ack | 00:29 |
| TheJulia | I'm hoping to do code reviews, and hopefully do the sushy thing tomorr | 00:30 |
| TheJulia | *hoping* | 00:30 |
| iurygregory | if the patch merges I can cut a release for you also :D | 00:32 |
| opendevreview | Merged openstack/ironic-python-agent master: fix multipathd error handling release notes https://review.opendev.org/c/openstack/ironic-python-agent/+/900575 | 01:45 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: WIP expose EFI boot entries in inspection boot info https://review.opendev.org/c/openstack/ironic-python-agent/+/901645 | 04:17 |
| opendevreview | Merged openstack/sushy master: Adds an option for setting the http boot uri https://review.opendev.org/c/openstack/sushy/+/718276 | 05:20 |
| rpittau | good morning ironic! o/ | 07:34 |
| rpittau | TheJulia, stevebaker[m], when you got a chance can you please have another look at https://review.opendev.org/c/openstack/ironic/+/894918 ? thank you! | 07:44 |
| opendevreview | Merged openstack/ironic-inspector master: SQLAlchemy 2.0 - Fix items lost in migration https://review.opendev.org/c/openstack/ironic-inspector/+/900413 | 09:46 |
| opendevreview | Merged openstack/ironic-inspector master: Drop before_first_request usage of Flask app https://review.opendev.org/c/openstack/ironic-inspector/+/901615 | 09:59 |
| iurygregory | good morning Ironic | 11:25 |
| iurygregory | ok questions about bug triaging: 1- old bugs from like 2017, do we just mark as invalid? 2- Newer bugs (July this year) https://bugs.launchpad.net/bifrost/+bug/2028390 - to me this worked but wasn't the same version the person used, so we ask the reporter for more details and mark status Incomplete on it? | 12:34 |
| iurygregory | maybe we should create doc describing what we do in some scenarios when we are doing the triage, wdyt? | 12:39 |
| iurygregory | while looking at the bugs I saw https://bugs.launchpad.net/ironic/+bug/2021995, this probably explain some of the behavior I saw when executing Firmware updates in Gen10 machines .-. | 13:09 |
| iurygregory | I'm wondering if we can mark https://bugs.launchpad.net/ironic/+bug/2023316 as fixed released since in the end we have fixed (but I don't think it was tracked in this bug) | 13:16 |
| rpittau | iurygregory: I think we marked very old bugs as invalid or won't fix | 13:44 |
| rpittau | I mean, it's ok to mark a 6 years old bug as invalid :) | 13:49 |
| rpittau | for the bifrost bug probably good to ask a retest, I think it was fixed at some point, maybe by me :P | 13:49 |
| rpittau | iurygregory: good idea on adding some triage guidelines, maybe add a comment here https://review.opendev.org/c/openstack/ironic/+/900449 | 13:59 |
| iurygregory | rpittau, ack, tks! yeah I will add to it | 13:59 |
| iurygregory | add a comment =) | 13:59 |
| TheJulia | good morning | 14:13 |
| TheJulia | brraaaains | 14:25 |
| rpittau | good morning TheJulia :) | 14:26 |
| opendevreview | Julia Kreger proposed openstack/sushy master: Add a boot progress indicator https://review.opendev.org/c/openstack/sushy/+/896835 | 14:27 |
| TheJulia | just a rebase since it was pending with +2+A but the parent which merged (httpbooturi support) merged | 14:27 |
| TheJulia | iurygregory: btw, could you be awesome and trigger a sushy release if your still up for it? :) | 14:28 |
| iurygregory | TheJulia, sure! | 14:29 |
| iurygregory | TheJulia, do you need https://review.opendev.org/c/openstack/sushy/+/896835 ? | 14:30 |
| TheJulia | no, that is just a nice to have | 14:30 |
| TheJulia | we can also always just merge it and wait an hour or so :) | 14:30 |
| TheJulia | a nice to have as in we can leverage it later | 14:31 |
| TheJulia | I have no active plans to, and Dmitry put the bug on wishlist status | 14:31 |
| TheJulia | well, RFE overall for ironic using it to provide further error message clarity | 14:31 |
| iurygregory | ack | 14:32 |
| iurygregory | TheJulia, https://review.opendev.org/c/openstack/releases/+/901676 | 14:39 |
| dtantsur | To be clear: RFEs are Wishlist by default, unless we have strong reasons to give them a particular priority | 14:44 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Redfish UefiHttp boot support https://review.opendev.org/c/openstack/ironic/+/900964 | 14:49 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Add redfish https CI job https://review.opendev.org/c/openstack/ironic/+/901090 | 14:49 |
| TheJulia | dtantsur: I thought you were going to likely jump on possibly doing some error message improvement based upon the PTG, fwiw | 14:50 |
| TheJulia | which is why I even commented, because I was slightly surprised | 14:50 |
| TheJulia | those patches just polishing, I'm thinking instead of doing a big scenario job, we just do a big specialized boot interface job in tempest as motivation to move away from the each job is a specific configuration, maybe a nice december task, which is why the CI change is still DNM | 14:52 |
| dtantsur | TheJulia: it's possible that I planned on, but now I don't even recall what you're talking about, sorry | 14:55 |
| dtantsur | absolutely crazy time downstream | 14:55 |
| TheJulia | sorry :( | 14:56 |
| TheJulia | but I get it. Think: If has_an_os_booted(task): error_message='fix your networking' else: error_message='Something weird happened and the agent ramdisk never booted.' | 14:57 |
| TheJulia | hmm, we need to revise pxe booting docs | 14:59 |
| dtantsur | ahhh, this one. I would love to do that, but it's up in the air when I have time for that. | 15:00 |
| TheJulia | understand completely | 15:00 |
| dtantsur | My current priority is to survive until the vacation. | 15:01 |
| TheJulia | this is always a good plan | 15:01 |
| TheJulia | Speaking of survival until vacation, and the inevitable post-vacation survivals! | 15:02 |
| opendevreview | Merged openstack/sushy master: Add a boot progress indicator https://review.opendev.org/c/openstack/sushy/+/896835 | 16:29 |
| TheJulia | iurygregory: You mentioned the redfish resource move for virtual media at one point, do you think you'll have time to look at it in the near term? | 16:42 |
| iurygregory | TheJulia, yes | 16:49 |
| iurygregory | but it won't be me =) but my mentee | 16:49 |
| iurygregory | we need to update sushy to support Vmedia trough Systems, update Ironic and probably update sushy-tools also | 16:51 |
| TheJulia | yeah | 17:02 |
| TheJulia | all three | 17:02 |
| rpittau | good night! o/ | 17:06 |
| iurygregory | yeah, I think they are all low-hang-fruit (kinda) =) | 17:15 |
| iurygregory | bye rpittau o/ | 17:15 |
| iurygregory | We will be tracking in https://bugs.launchpad.net/sushy-tools/+bug/2039458 | 17:17 |
| TheJulia | iurygregory: well, since vendors are breaking stuff, maybe not quite low hanging fruit ;) | 17:29 |
| TheJulia | a branch or two higher ;) | 17:29 |
| iurygregory | the change seems simple at least initially =) | 17:30 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add HTTP versions of network boot interfaces https://review.opendev.org/c/openstack/ironic/+/900965 | 18:02 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI test for httpboot jobs https://review.opendev.org/c/openstack/ironic/+/901182 | 18:02 |
| TheJulia | Okay, polished, just won't pass until the base redfish change is sorted | 18:02 |
| TheJulia | and a recheck *should* tell us if sadness exists, or not | 18:04 |
| iurygregory | yeah, the change has a +2 from Elod already | 18:11 |
| TheJulia | sweet | 18:25 |
| TheJulia | writing email to the mailing list regarding driver deprecations | 18:25 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Always generate checksum files on image builds https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/878297 | 20:04 |
| TheJulia | dtantsur: when you have a chance in the next few days, please take another glance at https://review.opendev.org/c/openstack/ironic/+/890411 | 20:12 |
| opendevreview | Merged openstack/networking-generic-switch master: Fix batching error due to outdated etcd3gw https://review.opendev.org/c/openstack/networking-generic-switch/+/886404 | 20:30 |
| TheJulia | brraaaaaiiins | 20:57 |
| opendevreview | Merged openstack/ironic-inspector master: [CI] Add sqlalchemy 2.0 unit testing https://review.opendev.org/c/openstack/ironic-inspector/+/900414 | 21:19 |
| opendevreview | Merged openstack/ironic-python-agent master: Test coverage for efi_utils.get_boot_record https://review.opendev.org/c/openstack/ironic-python-agent/+/899774 | 21:36 |
| opendevreview | Merged openstack/ironic-python-agent master: Handle different device outputs for multipath https://review.opendev.org/c/openstack/ironic-python-agent/+/901514 | 21:36 |
| opendevreview | Merged openstack/ironic master: Implement drain shutdown support https://review.opendev.org/c/openstack/ironic/+/897496 | 21:46 |
| opendevreview | Merged openstack/ironic-python-agent stable/2023.1: Conditional creation of RAIDed ESP for UEFI Software RAID https://review.opendev.org/c/openstack/ironic-python-agent/+/899859 | 22:06 |
| opendevreview | Verification of a change to openstack/networking-generic-switch stable/2023.1 failed: Honor ngs_save_configuration setting when using batch commands https://review.opendev.org/c/openstack/networking-generic-switch/+/900884 | 22:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!