Friday, 2025-09-05

opendevreviewVerification of a change to openstack/ironic master failed: Log how long power sync and sensor collections take  https://review.opendev.org/c/openstack/ironic/+/95576208:11
opendevreviewVerification of a change to openstack/ironic master failed: Add a suggestive warning around power and sensor syncs  https://review.opendev.org/c/openstack/ironic/+/95582108:12
opendevreviewPierre Riteau proposed openstack/bifrost master: Add support for CentOS Stream 10  https://review.opendev.org/c/openstack/bifrost/+/95885308:13
opendevreviewPierre Riteau proposed openstack/bifrost master: CI: Add testing for CentOS Stream 10  https://review.opendev.org/c/openstack/bifrost/+/95968408:13
abongalehappy friday ironic o/08:48
mnasiadkaGood morning Ironic09:36
mnasiadkaIs it me - or is pxe-filter broken after switch to threading? https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_0ad/openstack/0ad0132c187b40d2b0ac57c811df224a/primary/logs/kolla/ironic/ironic-pxe-filter.txt09:36
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969010:11
dtantsurmnasiadka: looks pretty broken indeed..10:12
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969010:13
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969010:24
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969010:40
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969010:43
opendevreviewMerged openstack/ironic master: Log how long power sync and sensor collections take  https://review.opendev.org/c/openstack/ironic/+/95576210:47
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969010:48
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969010:55
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969011:04
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969011:15
opendevreviewMerged openstack/ironic master: Add a suggestive warning around power and sensor syncs  https://review.opendev.org/c/openstack/ironic/+/95582111:21
opendevreviewPierre Riteau proposed openstack/bifrost master: Add support for CentOS Stream 10  https://review.opendev.org/c/openstack/bifrost/+/95885311:26
opendevreviewOpenStack Release Bot proposed openstack/metalsmith stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/metalsmith/+/95970212:14
opendevreviewOpenStack Release Bot proposed openstack/metalsmith stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/metalsmith/+/95970312:14
opendevreviewOpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/2025.2  https://review.opendev.org/c/openstack/metalsmith/+/95970412:14
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/95970512:14
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/95970612:14
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update master for stable/2025.2  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/95970712:14
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/python-ironicclient/+/95970812:14
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/python-ironicclient/+/95970912:14
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient master: Update master for stable/2025.2  https://review.opendev.org/c/openstack/python-ironicclient/+/95971012:14
opendevreviewOpenStack Release Bot proposed openstack/sushy stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/sushy/+/95971112:14
opendevreviewOpenStack Release Bot proposed openstack/sushy stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/sushy/+/95971212:14
opendevreviewOpenStack Release Bot proposed openstack/sushy master: Update master for stable/2025.2  https://review.opendev.org/c/openstack/sushy/+/95971312:14
opendevreviewMatt Crees proposed openstack/bifrost master: Support IPA download with authenticated requests  https://review.opendev.org/c/openstack/bifrost/+/94532412:53
TheJuliagood morning12:58
TheJuliamnasiadka: doh12:59
TheJuliaWell, I guess I know what I'll be doing today12:59
TheJuliaI guess we also don't run it in any single devstack job13:00
opendevreviewMerged openstack/python-ironic-inspector-client stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/95970513:07
opendevreviewMerged openstack/python-ironicclient stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/python-ironicclient/+/95970813:09
opendevreviewMerged openstack/python-ironicclient stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/python-ironicclient/+/95970913:09
opendevreviewMerged openstack/python-ironic-inspector-client stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/95970613:09
opendevreviewMerged openstack/sushy stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/sushy/+/95971113:09
opendevreviewMerged openstack/sushy stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/sushy/+/95971213:09
opendevreviewMerged openstack/metalsmith stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/metalsmith/+/95970213:10
opendevreviewMerged openstack/metalsmith stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/metalsmith/+/95970313:10
opendevreviewJulia Kreger proposed openstack/ironic master: fix: remove wait logic in pxe filter  https://review.opendev.org/c/openstack/ironic/+/95985013:12
TheJuliaI *think* that would do it, mnasiadka is kolla ci able to pull in ^^^ and give it a spin?13:12
opendevreviewJulia Kreger proposed openstack/ironic master: fix: remove wait logic in pxe filter  https://review.opendev.org/c/openstack/ironic/+/95985013:22
mnasiadkaYup, let me add depends-on13:32
mnasiadkaTesting in https://review.opendev.org/c/openstack/kolla-ansible/+/939231 - will let you know TheJulia how it goes13:36
TheJuliait... looks... good on my end, it starts, and errors as I would expect with a lack of local configuration13:37
TheJuliaso I guess that is a somewhat good sign13:37
opendevreviewStephen Finucane proposed openstack/ironic master: api: Add schema for inspection rules API (requests)  https://review.opendev.org/c/openstack/ironic/+/95403914:27
cardoeTheJulia: I had a chat with ralonsoh about the VXLAN bits. He should come your way about some questions about your spec.15:13
TheJuliaack, I'm out next week, but this is going to take us some time to hammer out overall15:15
opendevreviewStephen Finucane proposed openstack/ironic master: pre-commit: Run ruff before hacking  https://review.opendev.org/c/openstack/ironic/+/95986915:23
TheJuliamnasiadka: I suspect you need to re-stack the changes, fwiw15:23
opendevreviewArnaud Morin proposed openstack/ironic master: Avoid the wsgi application to start with eventlet  https://review.opendev.org/c/openstack/ironic/+/95987015:32
amorinhey ironic team, I propose the above, but note that I havn't tested it locally, I discovered the issue in mistral initialy (as I reuse the wsgi application code as you do)15:33
amorinI discoverd it by trying to uninstall eventlet inside my mistral venv, and found out that uwsgi was not able to start the app anymore, so feel free to discard if you think it's not applying to ironic!15:34
amorinand enjoy the week-end :)15:34
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969015:41
opendevreviewStephen Finucane proposed openstack/ironic master: api: Add schema for inspection rules API (requests)  https://review.opendev.org/c/openstack/ironic/+/95403915:53
opendevreviewStephen Finucane proposed openstack/ironic master: api: Add schema for events API  https://review.opendev.org/c/openstack/ironic/+/95988015:53
opendevreviewStephen Finucane proposed openstack/ironic master: WIP: api: Add schema for portgroups API  https://review.opendev.org/c/openstack/ironic/+/95988115:53
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969015:58
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969016:09
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969016:20
dtantsurI already hate ansible and deeply regret not writing ^^ as one bash script16:20
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969016:23
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969016:34
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job  https://review.opendev.org/c/openstack/ironic/+/95969016:47
dtantsurI think I need to call it a day before the number of revisions exceed one of the patch introducing IPA support back in the days :D16:52
JayFhttps://review.opendev.org/c/openstack/ironic/+/959870 makes a hell of a lot of sense to me17:32
JayFbut I'm confused as to how we would've missed it17:32
JayFso I wonder if there's something I'm missing as to the workflow here17:32
TheJulianumerous launch omdes17:38
TheJuliamodes17:38
TheJuliaunfortunately, I don't think that one is obvious when we're in our command launch context17:38
JayFthis is how we run in devstack though17:46
JayFso does this ... invalidate some of our testing?17:46
TheJuliaI don't think eveything in devstack runs wsgi, because I know some of our jobs run ironic as a standalone api service17:47
TheJuliaI don't think it invalidates our testing, its largely pluggable at that level of interaction17:47
JayFfair, just sad we caught something at the last minute after declaring victory :)17:53
JayFI'll make sure it gets backported to flamingo17:53
TheJuliait happens17:56
TheJuliaas long as we have dependencies which can invoke it, like oslo.service, this is a risk17:56

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