Tuesday, 2024-07-02

TheJuliacardoe: one thing to think of is dnsmasq has issues with configuration refreshes. Movement away from it is advisable.01:06
cardoeHmm. That does make sense.02:06
rpittaugood morning ironic! o/06:21
rpittauThanks JayF :)06:22
rpittauJayF: please go ahead for permission change in LP for cid if no one else has anything against it06:22
rpittauNow I need to remember that I'm bug deputy this week :)06:23
opendevreviewcid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype]  https://review.opendev.org/c/openstack/ironic/+/92214206:37
opendevreviewcid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype]  https://review.opendev.org/c/openstack/ironic/+/92214206:47
opendevreviewMichal Nasiadka proposed openstack/networking-generic-switch master: Cumulus NVUE support  https://review.opendev.org/c/openstack/networking-generic-switch/+/90755107:39
rpittauif any core has a moment would love a review for https://review.opendev.org/c/openstack/python-ironicclient/+/923097 so we can release ironicclient afterwards, thanks!08:10
cidThere's this change too, that I added patches to, though it's not mine (so I couldn't add it for review).08:33
cidhttps://review.opendev.org/c/openstack/networking-generic-switch/+/88804708:33
rpittaucid: thanks, I think we can wait for ngs and networking-baremetal08:36
cid++08:37
rpittauand we will need this https://review.opendev.org/c/openstack/virtualbmc/+/922589 for virtualbmc08:38
opendevreviewRiccardo Pittau proposed openstack/networking-generic-switch master: Update to match latest development cycle  https://review.opendev.org/c/openstack/networking-generic-switch/+/92320208:45
rpittaualso this ^08:45
opendevreviewRiccardo Pittau proposed openstack/networking-baremetal master: Update to match latest development cycle  https://review.opendev.org/c/openstack/networking-baremetal/+/92320408:51
opendevreviewRiccardo Pittau proposed openstack/networking-baremetal master: Update to match latest development cycle  https://review.opendev.org/c/openstack/networking-baremetal/+/92320408:52
opendevreviewRiccardo Pittau proposed openstack/ironic-lib master: Update to match latest development cycle  https://review.opendev.org/c/openstack/ironic-lib/+/92320508:56
opendevreviewDmitry Tantsur proposed openstack/sushy master: Log the response when unable to parse JSON  https://review.opendev.org/c/openstack/sushy/+/92321010:44
iurygregorygood morning ironic11:16
iurygregoryrpittau, looking now at the patches you mentioned11:16
opendevreviewMerged openstack/python-ironicclient master: Fixes the description filter  https://review.opendev.org/c/openstack/python-ironicclient/+/92309712:13
dtantsurrpittau: you know that bumping to tox 4.4.0 makes the development harder on CS9?13:34
rpittaudtantsur: no, why's that?13:44
dtantsurrpittau: only 3.18 available there13:45
rpittauoh ok, I thought it was some kind of issue with python13:46
rpittauwe changed tox min version in ironic 5 months ago13:46
rpittauactually we should use an even more recent version as 4.4.0 is bugged13:46
dtantsurI know...13:46
dtantsurhold on, bugs are never a reason to bump a dependency13:46
rpittauno, of course, we bumped the version because of the constrain_package_deps feature13:48
opendevreviewRiccardo Pittau proposed openstack/ironic-lib master: Update to match latest development cycle  https://review.opendev.org/c/openstack/ironic-lib/+/92320513:50
opendevreviewMohammed Boukhalfa proposed openstack/sushy-tools master: Extend Fake System Driver for Interaction with Fake IPA via REST API  https://review.opendev.org/c/openstack/sushy-tools/+/92311114:56
opendevreviewMohammed Boukhalfa proposed openstack/sushy-tools master: Extend Fake System Driver for Interaction with Fake IPA via REST API  https://review.opendev.org/c/openstack/sushy-tools/+/92311115:01
JayFdtantsur: [not arguing the theoretical '16:07
JayF** 'is it okay] but could pip install --user tox work?16:07
rpittauJayF: that's the default (--user) now, I'd expect that too, but I see the point of having an older version by default16:09
clarkbfwiw I would recommend against trying to support both tox 3 and tox 4. The changes between the two were quite large and it is probably best to just focus on 4 at this point otherwise you end up dealing with a bunch of corner cases16:09
rpittauclarkb: yep, that's why I'm updating all the tox.ini :)16:09
rpittauwow it's super late! see a tomorrow! o/16:12
opendevreviewcid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype]  https://review.opendev.org/c/openstack/ironic/+/92214220:40
JayFI have all the suggested enhancements Reverbverbverb's report had in it in JSON format. I was going to put this into a quick script to file them all as bugs against ironic, specifically tagging as "docs audit" and marking them as triaged->low21:09
JayFIf there's any objection to this plan, and creating those bugs, please make loud noises :)21:09
JayFI filed https://bugs.launchpad.net/ironic-lib/+bug/2071740 and triaged it as critical based on a conversation I had today with TheJulia and fungi 21:43
JayFfixing this bug should be prioritized if anyone has any spare cycles to point at it21:43
JayFFrom that same conversation, I added https://bugs.launchpad.net/ironic/+bug/2071741 -- which is an RFE to deprecate [agent]manage_agent_boot option.21:48
JayFPlease let me know if you have any questions or want additional context.21:49
cidI will look at the bugs tomorrow, especially the critical one.21:58
* cid currently foucsing more on runbooks, we will discuss my progress tomorrow :(22:00
cidI will be nipping out now.22:00
cido//22:00
fungithanks JayF!22:02
JayFcid: lets talk before you pick up 2071740. It might be deceptively broadly scoped.22:02
JayFI'm doing a quick implementation of 2071741, the hard part will be the removal in a year22:10
opendevreviewJay Faulkner proposed openstack/ironic master: Fixup: Move release note to proper directory  https://review.opendev.org/c/openstack/ironic/+/92332722:16
opendevreviewJay Faulkner proposed openstack/ironic master: Deprecate [agent]/manage_agent_boot  https://review.opendev.org/c/openstack/ironic/+/92332822:23
JayF923327 is ... wildly wtf22:24
JayFunsure how that even happened, but it's kinda hilarious 22:24
ashinclouds[m]That happens22:32
JayFit's 100% the kinda thing I'd miss in a review tbh22:57
JayFjust gave me a little WTF when I was making my releasenote for the manage_agent_boot deprecation22:57
JayFI'd appreciate if someone could help me land my local-ironic-dev change => https://review.opendev.org/c/openstack/ironic/+/922348 -- it's really small, a straightforward improvement ... and the reason I want it done soon is I have a video in production talking about using this to run Ironic; and I really can't/shouldn't publish it until I can link to the (now existing) upstream doc23:00

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!