| TheJulia | cardoe: one thing to think of is dnsmasq has issues with configuration refreshes. Movement away from it is advisable. | 01:06 |
|---|---|---|
| cardoe | Hmm. That does make sense. | 02:06 |
| rpittau | good morning ironic! o/ | 06:21 |
| rpittau | Thanks JayF :) | 06:22 |
| rpittau | JayF: please go ahead for permission change in LP for cid if no one else has anything against it | 06:22 |
| rpittau | Now I need to remember that I'm bug deputy this week :) | 06:23 |
| opendevreview | cid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype] https://review.opendev.org/c/openstack/ironic/+/922142 | 06:37 |
| opendevreview | cid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype] https://review.opendev.org/c/openstack/ironic/+/922142 | 06:47 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 07:39 |
| rpittau | if 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 |
| cid | There's this change too, that I added patches to, though it's not mine (so I couldn't add it for review). | 08:33 |
| cid | https://review.opendev.org/c/openstack/networking-generic-switch/+/888047 | 08:33 |
| rpittau | cid: thanks, I think we can wait for ngs and networking-baremetal | 08:36 |
| cid | ++ | 08:37 |
| rpittau | and we will need this https://review.opendev.org/c/openstack/virtualbmc/+/922589 for virtualbmc | 08:38 |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Update to match latest development cycle https://review.opendev.org/c/openstack/networking-generic-switch/+/923202 | 08:45 |
| rpittau | also this ^ | 08:45 |
| opendevreview | Riccardo Pittau proposed openstack/networking-baremetal master: Update to match latest development cycle https://review.opendev.org/c/openstack/networking-baremetal/+/923204 | 08:51 |
| opendevreview | Riccardo Pittau proposed openstack/networking-baremetal master: Update to match latest development cycle https://review.opendev.org/c/openstack/networking-baremetal/+/923204 | 08:52 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-lib master: Update to match latest development cycle https://review.opendev.org/c/openstack/ironic-lib/+/923205 | 08:56 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Log the response when unable to parse JSON https://review.opendev.org/c/openstack/sushy/+/923210 | 10:44 |
| iurygregory | good morning ironic | 11:16 |
| iurygregory | rpittau, looking now at the patches you mentioned | 11:16 |
| opendevreview | Merged openstack/python-ironicclient master: Fixes the description filter https://review.opendev.org/c/openstack/python-ironicclient/+/923097 | 12:13 |
| dtantsur | rpittau: you know that bumping to tox 4.4.0 makes the development harder on CS9? | 13:34 |
| rpittau | dtantsur: no, why's that? | 13:44 |
| dtantsur | rpittau: only 3.18 available there | 13:45 |
| rpittau | oh ok, I thought it was some kind of issue with python | 13:46 |
| rpittau | we changed tox min version in ironic 5 months ago | 13:46 |
| rpittau | actually we should use an even more recent version as 4.4.0 is bugged | 13:46 |
| dtantsur | I know... | 13:46 |
| dtantsur | hold on, bugs are never a reason to bump a dependency | 13:46 |
| rpittau | no, of course, we bumped the version because of the constrain_package_deps feature | 13:48 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-lib master: Update to match latest development cycle https://review.opendev.org/c/openstack/ironic-lib/+/923205 | 13:50 |
| opendevreview | Mohammed 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/+/923111 | 14:56 |
| opendevreview | Mohammed 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/+/923111 | 15:01 |
| JayF | dtantsur: [not arguing the theoretical ' | 16:07 |
| JayF | ** 'is it okay] but could pip install --user tox work? | 16:07 |
| rpittau | JayF: that's the default (--user) now, I'd expect that too, but I see the point of having an older version by default | 16:09 |
| clarkb | fwiw 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 cases | 16:09 |
| rpittau | clarkb: yep, that's why I'm updating all the tox.ini :) | 16:09 |
| rpittau | wow it's super late! see a tomorrow! o/ | 16:12 |
| opendevreview | cid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype] https://review.opendev.org/c/openstack/ironic/+/922142 | 20:40 |
| JayF | I 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->low | 21:09 |
| JayF | If there's any objection to this plan, and creating those bugs, please make loud noises :) | 21:09 |
| JayF | I 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 |
| JayF | fixing this bug should be prioritized if anyone has any spare cycles to point at it | 21:43 |
| JayF | From 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 |
| JayF | Please let me know if you have any questions or want additional context. | 21:49 |
| cid | I 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 | |
| cid | I will be nipping out now. | 22:00 |
| cid | o// | 22:00 |
| fungi | thanks JayF! | 22:02 |
| JayF | cid: lets talk before you pick up 2071740. It might be deceptively broadly scoped. | 22:02 |
| JayF | I'm doing a quick implementation of 2071741, the hard part will be the removal in a year | 22:10 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Fixup: Move release note to proper directory https://review.opendev.org/c/openstack/ironic/+/923327 | 22:16 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Deprecate [agent]/manage_agent_boot https://review.opendev.org/c/openstack/ironic/+/923328 | 22:23 |
| JayF | 923327 is ... wildly wtf | 22:24 |
| JayF | unsure how that even happened, but it's kinda hilarious | 22:24 |
| ashinclouds[m] | That happens | 22:32 |
| JayF | it's 100% the kinda thing I'd miss in a review tbh | 22:57 |
| JayF | just gave me a little WTF when I was making my releasenote for the manage_agent_boot deprecation | 22:57 |
| JayF | I'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 doc | 23:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!