opendevreview | likui proposed openstack/python-ironicclient master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/python-ironicclient/+/906077 | 01:23 |
---|---|---|
opendevreview | Kyrylo Romanenko proposed openstack/ironic-tempest-plugin master: [WiP] Add negative tests for VIF attach/detach operations https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/906078 | 01:24 |
opendevreview | likui proposed openstack/python-ironic-inspector-client master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/906079 | 01:27 |
opendevreview | likui proposed openstack/ironic master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/ironic/+/906080 | 01:28 |
rpittau | good morning ironic! happy friday! o/ | 08:00 |
opendevreview | Verification of a change to openstack/ironic master failed: Don't create a hardlink to a symlink when handling file:// URLs https://review.opendev.org/c/openstack/ironic/+/905626 | 08:11 |
tkajinam | dtantsur, hi. I'm facing debian jobs failure in my ironic-python-agent-builder patch for stable/wallaby. I'm wondering if https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/808194 could be related to the current problem | 08:37 |
tkajinam | tbvh I'm not very familiar with the job and still need to dig into it to find out the cause, but this is one debian-related changes (according to the commit messages) which presents in stable/xena but does not in stable/wallaby | 08:38 |
tkajinam | these job pass in stable/xena so I guess something have to be backported to stable/wallaby | 08:38 |
tkajinam | we may want to make these jobs non-voting considering the fact wallaby is quite old, but in case you think that is a valid fix then I'll attempt to backport it. | 08:38 |
tkajinam | you can find the current error in https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/905835 | 08:39 |
iurygregory | good morning Ironic | 11:23 |
dtantsur | tkajinam: yeah, it's quite possible that you need to backport my patch, otherwise a much newer debian is used | 11:27 |
opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent-builder stable/wallaby: CI: specify Debian version explicitly https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/906121 | 11:58 |
tkajinam | dtantsur, ok, thanks. let me try ^^^ then | 12:00 |
iurygregory | Nisha_Agarwal, hey o/ I'm wondering about the status of https://bugs.launchpad.net/ironic/+bug/1526382 when you have a chance can you check? Tks! | 12:08 |
iurygregory | https://bugs.launchpad.net/ironic-inspector/+bug/1525218 I think we can close this one, right? | 12:47 |
iurygregory | and also https://bugs.launchpad.net/ironic-inspector/+bug/1525231 | 12:47 |
*** tosky_ is now known as tosky | 12:50 | |
rpittau | iurygregory: yes and yes thanks! :) | 13:34 |
iurygregory | rpittau, tks for checking =) | 13:35 |
dtantsur | oh, FUUUUUUN https://forums.rockylinux.org/t/cant-find-command-linuxefi-and-initrdefi-while-booting-from-uefi/11300 | 13:42 |
iurygregory | "linuxefi and initrdefi, despite their generic names, are specific to x86/64 and its (deprecated) " | 13:43 |
iurygregory | WOW | 13:43 |
opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent-builder stable/wallaby: Drop TripleO job https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/905835 | 13:44 |
tkajinam | dtantsur, backport worked and resolved the debian job failures in wallaby https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/906121/ \o/ | 13:45 |
iurygregory | \o/ +2 | 13:45 |
* tkajinam rebased that tripleo job removal patch on it for quicker merge. | 13:45 | |
tkajinam | iurygregory, thanks :-D | 13:46 |
iurygregory | yw =) | 13:46 |
dtantsur | It looks like linuxefi should no longer be used, fun | 13:56 |
iurygregory | yup =( | 13:57 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: DNM try s/linuxefi/linux/ https://review.opendev.org/c/openstack/ironic/+/906113 | 14:01 |
dtantsur | Or even: it looks like linuxefi is required for secure boot. So we can probably modify the templates based on that. | 14:04 |
dtantsur | sigh, that's a mess... | 14:04 |
iurygregory | jesus .-. | 14:06 |
iurygregory | why keep it simple right? | 14:06 |
rpittau | probably better leave linuxefi for now and modify that for ARM | 14:06 |
dtantsur | ohhh, linuxefi is a non-standard extension that is not in upstream grub | 14:08 |
dtantsur | the more I read about it, the more cursed it becomes | 14:08 |
TheJulia | Grub is generally cursed | 14:09 |
TheJulia | Since that is actually kind of… common | 14:10 |
opendevreview | Julia Kreger proposed openstack/ironic-inspector stable/2023.2: Fix RBAC access for service/admin user access https://review.opendev.org/c/openstack/ironic-inspector/+/906122 | 14:33 |
* dtantsur opens an empty document to write what may become his most far-reaching RFE this year | 14:33 | |
dtantsur | kaboom https://bugs.launchpad.net/ironic/+bug/2049913 | 14:56 |
* iurygregory updates the summary of bug triage this week | 15:06 | |
JayF | dtantsur: someone, somewhere, without a bmc, is very sad at you lol | 15:06 |
dtantsur | I know, I'm also without a bmc and sometimes very sad! | 15:07 |
dtantsur | (maybe it's not clear, but all this stuff is optional) | 15:07 |
JayF | dtantsur: fyi, there is going to be an mlh fellow that I'm going to have pointed at the ironic bug dashboard. Are you okay with us using the GitHub accoutrement attached to that for issue tracking and such? | 15:07 |
JayF | dtantsur: that was more a joking reference to a specific conversation this week | 15:07 |
dtantsur | yeah, I recognized it :) | 15:08 |
dtantsur | Assuming I googled "accoutrement" correctly, I'm okay with it | 15:08 |
JayF | Yeah I'm just saying the issue tracker pull requests etc | 15:09 |
JayF | I'm going to ensure this contributor gets at least one commit in openstack proper, but my hope is that they can significantly improve the bug dashboard | 15:09 |
dtantsur | Well, unless you want to move it under opendev and our governannce.. | 15:09 |
JayF | I see it as a feature and not a bug that I will have to spend less time onboarding an mlh fellow to Gerrit | 15:10 |
dtantsur | haha, okay, we can do it later | 15:10 |
JayF | So even if we do that, I would not be opposed to doing it after the mlh semester | 15:10 |
rpittau | dtantsur: I'll have to read it again when I can give proper focus, but at a glance I guess it makes sense :) | 15:39 |
rpittau | bye everyone, have a great weekend! o/ | 15:51 |
TheJulia | Have a wonderful weekend | 15:52 |
opendevreview | Merged openstack/ironic-python-agent master: [Codespell] Adding tox target to CI (Non-Voting) https://review.opendev.org/c/openstack/ironic-python-agent/+/905980 | 16:16 |
opendevreview | Merged openstack/ironic-python-agent stable/2023.2: Add missing headers to the inspection callback https://review.opendev.org/c/openstack/ironic-python-agent/+/905739 | 16:16 |
TheJulia | JayF: iurygregory: I added you both to https://review.opendev.org/c/openstack/ironic-inspector/+/906122 as a backport. If your for it cool, if not just let me know and I'll carry it downstream only. | 16:54 |
JayF | TheJulia: if anything, I could see an argument that not only it should be back ported, but we should potentially consider an OSS* around it. | 18:18 |
JayF | I don't actually think we should go that far, but I think it's pretty easy to make the argument it should be back ported | 18:18 |
TheJulia | eh, an oss would be misplaced since we effectively just end up with authenticaiton being impossible without highly specialized configuration | 18:20 |
TheJulia | The fix just aligns us really, which is why I'd like to take it to 2023.1, but realizing it is a change, always important to see what others think because at worst case I'll just backport it downstream and move on. | 18:21 |
TheJulia | I think the lesson is, folks haven't tried scope enforcement, or are running scope enforced configs like we originally intended similar to cern who we know was a very early adopter to the new rbac policies | 18:26 |
JayF | TheJulia: The other perspective is: "Red Hat Ironic deployments" and "other Ironic deployments" having different APIs is also nonzero harm to Ironic; so I'd like to avoid that when possible as well :D | 19:49 |
JayF | all arrows point to "JFDI the backport" | 19:49 |
TheJulia | that is only if you inherently tie user role in with everything as part of the API, but that can be independently forced via custom policy | 19:51 |
JayF | I always view "default policy" as a weak part of our API contract | 19:51 |
TheJulia | and I'm not saying custom policy is the solution I'm working, I'm just trying to get things to where they are supposed to be, and if I can backport it upstream than \o/ | 19:52 |
JayF | I already +2a btw | 19:52 |
TheJulia | And also what users folks use too is a whole other variable | 19:52 |
TheJulia | and likely the bigger variable here too | 19:52 |
TheJulia | cool cool, thanks | 19:52 |
TheJulia | One less thing for me to worry about :)\ | 19:52 |
opendevreview | Julia Kreger proposed openstack/ironic-inspector stable/2023.1: Fix RBAC access for service/admin user access https://review.opendev.org/c/openstack/ironic-inspector/+/906123 | 19:54 |
opendevreview | Merged openstack/ironic-inspector stable/2023.2: Fix RBAC access for service/admin user access https://review.opendev.org/c/openstack/ironic-inspector/+/906122 | 20:57 |
JayF | Curious what folks think about https://review.opendev.org/c/openstack/ironic/+/900568 -- I emailed the original RFE filer to review with zero response. I have no hardware to validate this, but have read the Neutron code and I have reasonable confidence we're sending across the right metadata ... but essentially this is 100% untested. | 21:23 |
JayF | I think it's worthwhile to land in that state -- especially given reports in the original RFE bug that it works with compatible hardware (with an contributed, incomplete patch), but I'm not sure how/if to document it. | 21:24 |
TheJulia | hmmmm it really all boils down to how we frame it, I think | 21:24 |
opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Add support for reporting CPU socket number https://review.opendev.org/c/openstack/ironic-python-agent/+/900642 | 21:24 |
JayF | Yeah, I'm tempted to add a section to your OVN doc and label it "alpha" support for OVN switches | 21:27 |
JayF | and invite people to use our support to test their hardware and provide feedback | 21:28 |
TheJulia | +++ | 21:28 |
JayF | let me do that right now | 21:28 |
* JayF trying to get to the bottom of a list and get out a little early | 21:28 | |
JayF | migraine and neckache pretty much all day \o/ my body demands a weekend | 21:28 |
TheJulia | Enjoy! | 21:31 |
TheJulia | I've been working on a doc and it has been going super slow :( | 21:32 |
JayF | Yeah, I had to write up the fellowship doc for the incoming MLH fellow today. We're going to have them polish up the ironic bug dashboard. | 21:38 |
JayF | Also will still owe some doc writing this weekend if I head out early ... mainly notes for upcoming podcast (we are going to be doing the first taping of a GR-OSS OUTreach podcast sometime in the next 2-4 weeks ... hosted by me :D) | 21:38 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Basic support for OVN VTEP switches https://review.opendev.org/c/openstack/ironic/+/900568 | 21:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!