| opendevreview | Merged openstack/ironic stable/2025.2: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988356 | 00:17 |
|---|---|---|
| opendevreview | Merged openstack/ironic-python-agent-builder master: Replace tabs with spaces https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/988397 | 00:41 |
| frickler | fyi ironic unit tests are broken with the latest keystonemiddleware, see https://review.opendev.org/c/openstack/requirements/+/988412 . iiuc the dropped plugin has been deprecated since ... drum roll ... mitaka :-/ | 07:09 |
| *** elodilles is now known as elodilles_pto | 07:49 | |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Drop reference to [keystone_authtoken] auth_uri https://review.opendev.org/c/openstack/ironic/+/988595 | 10:34 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Drop reference to legacy keystonemiddleware options https://review.opendev.org/c/openstack/ironic/+/988595 | 10:35 |
| stephenfin | JayF: jfyi, I don't agree with the solution proposed at https://review.opendev.org/c/openstack/ironic/+/988538/1/doc/source/conf.py#69. type is a builtin. No one would argue/should be arguing that we should qualify all builtins (e.g. builtins.list, builtins.Exception, builtins.staticmethod, ...). Rather, this is a bug/design flaw in autodoc. | 11:03 |
| stephenfin | I'm saying this because I don't want people seeing that comment and proposing we "improve" things in e.g. o.vo or python-ironicclient by qualifying all hints with `builtins` | 11:04 |
| stephenfin | JayF: That's your actual culprit https://github.com/sphinx-doc/sphinx/issues/14223 | 11:42 |
| opendevreview | Stephen Finucane proposed openstack/ironic master: doc: Ignore ironic.wsgi module https://review.opendev.org/c/openstack/ironic/+/988604 | 11:44 |
| *** hroy_ is now known as hroy | 12:08 | |
| opendevreview | Stephen Finucane proposed openstack/ironic master: Update note on autodoc issue https://review.opendev.org/c/openstack/ironic/+/988607 | 12:10 |
| JayF | stephenfin: thanks for the update, I'll review it when I'm at a PC. That was just a really quickly banged out patch to unbreak our CI so I'm not surprised that it was off kilter | 13:07 |
| stephenfin | All good. https://review.opendev.org/c/openstack/ironic/+/988607 does the necessary update | 13:07 |
| TheJulia | I'll be out of pocket for at least half of today. I need to go meet with my Home Owners Association... wish me luck | 14:41 |
| opendevreview | Merged openstack/ironic master: Drop reference to legacy keystonemiddleware options https://review.opendev.org/c/openstack/ironic/+/988595 | 14:41 |
| TheJulia | stephenfin: fyi, responded to your comment on the keystoneauth change I posted yesterday, hopefully that answers the question. If not, just let me know. Happy to dig deeper, but given the context and fallback modeling, may just be... easier, dunno. | 14:42 |
| opendevreview | Merged openstack/ironic master: Update note on autodoc issue https://review.opendev.org/c/openstack/ironic/+/988607 | 14:50 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Fix unit tests errors caused by keystonemiddleware 13.0.0 https://review.opendev.org/c/openstack/ironic/+/988641 | 15:04 |
| opendevreview | cid proposed openstack/ironic master: doc: Remove VirtualPDU from query string https://review.opendev.org/c/openstack/ironic/+/988661 | 17:52 |
| JayF | someone wanna kick along the vpdu reitrement? https://review.opendev.org/c/openstack/virtualpdu/+/984162 | 18:41 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/988468 would be great since it would remove a bunch of deprecation warnings from the logs. | 18:44 |
| cardoe | That change reduced neutron's logs by 16k lines | 18:44 |
| JayF | deprecate this [throws a +2 at you] | 18:44 |
| cardoe | I dunno how much it will do for ironic | 18:44 |
| cardoe | Couldn't load neutron logs on my phone cause of the spam | 18:45 |
| opendevreview | Merged openstack/virtualpdu master: [EOL]: Retire virtualpdu https://review.opendev.org/c/openstack/virtualpdu/+/984162 | 18:49 |
| * TheJulia returns in time for a meeting | 18:52 | |
| opendevreview | cid proposed openstack/tenks master: Removes VirtualPDU from supported emulators list https://review.opendev.org/c/openstack/tenks/+/988681 | 18:59 |
| opendevreview | Merged openstack/ironic master: doc: Remove VirtualPDU from query string https://review.opendev.org/c/openstack/ironic/+/988661 | 19:13 |
| opendevreview | Merged openstack/ironic master: Update packaging for package discovery https://review.opendev.org/c/openstack/ironic/+/988468 | 20:12 |
| opendevreview | Merged openstack/ironic master: Follow-Up: Keep anaconda on provisioning network https://review.opendev.org/c/openstack/ironic/+/988505 | 20:20 |
| *** NobodyCam_ is now known as NobodyCam | 20:55 | |
| opendevreview | Merged openstack/networking-generic-switch master: Exclude .cache from pep8 tests https://review.opendev.org/c/openstack/networking-generic-switch/+/988431 | 22:50 |
| opendevreview | Merged openstack/ironic master: Fix unit tests errors caused by keystonemiddleware 13.0.0 https://review.opendev.org/c/openstack/ironic/+/988641 | 23:29 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!