Friday, 2022-11-04

opendevreviewSONG SHUKUN proposed openstack/ironic bugfix/20.2: Add support auth protocols for iRMC  https://review.opendev.org/c/openstack/ironic/+/86357301:23
opendevreviewMerged openstack/bifrost master: Remove unused iniparse python system dependency  https://review.opendev.org/c/openstack/bifrost/+/86239104:13
opendevreviewMerged openstack/bifrost master: Evaluate truthiness of inspector.discovery.enabled with bool filter  https://review.opendev.org/c/openstack/bifrost/+/84530504:13
ajyadtantsur: deja vu, we talked about this before and I was not able to reproduce it in Ironic at that time. I see there are new messages added. Maybe Ironic logs could be retrieved to replicate the issue without OCP?07:20
ajyaNot ware of any changes around this. Reading the last message it seems that BIOS job does not get executed or after it system reboots and resets any other pending configuration around boot.07:21
arne_wiebalckGood morning ajya and Ironic!07:22
ajyaBIOS job is async and it will need for the system to reboot, but what should happen is that after it is completed as part of booting system will continue. Maybe something goes wrong here and my assumption is not correct.07:23
ajyaHi arne_wiebalck 07:23
rpittauhappy friday ironic! o/07:59
opendevreviewEbbex proposed openstack/bifrost master: Remove pymysql from system dependencies  https://review.opendev.org/c/openstack/bifrost/+/86153708:07
opendevreviewEbbex proposed openstack/bifrost master: Install passlib to venv (htpasswd)  https://review.opendev.org/c/openstack/bifrost/+/86153608:07
opendevreviewEbbex proposed openstack/bifrost master: Install firewall to venv (redhat)  https://review.opendev.org/c/openstack/bifrost/+/86153808:07
rpittauwe should really merge this to fix py310 testing https://review.opendev.org/c/openstack/ironic/+/86302908:18
jandersajya dtantsur w/r/t secureboot issue - I wonder, is it possible LC clear is somehow a factor? Only mentioning it cause we'd LC clear prior to the initial deploy but not prior to the subsequent ones... Not particularly confident about this but just throwing the idea out there for discussion, pls disregard if it makes no sense at all.08:26
ajyajanders: verification steps? As long as they don't clear pending tasks, shouldn't matter, but who knows. That's why Ironic logs would be useful to see what exactly is the workflow here.08:41
kubajjGood morning everybody08:54
rpittaulooks like secureboot issue is the hot topic right now, I'm also dealing with one :/08:58
dtantsurajya: okay, so if changing secure boot is a bios job, then it indeed won't be executed without automated cleaning (no reboot happens there).09:02
dtantsurthat's the minimum issue we need to fix in Ironic09:02
kubajjdtantsur: Any other suggestions on what to try to do with the db change?09:04
ajyadtantsur: ah, I was testing this as part of deploy09:04
dtantsurkubajj: not yet, need to get back to it (customer escalations, sorry)09:04
kubajjdtantsur: no worries, thanks09:04
dtantsurajya: yep, the cleaning problem is quite clear. the deploy problem is what I find confusing.09:04
dtantsurajya: what they're doing is essentially ISO ramdisk boot with force_persistent_boot_device=Never (i.e. one-shot boot).09:05
ajyadtantsur: hm, maybe Never is the key here, that after completing the BIOS job the previous settings were cleared, but not sure, will try to check09:44
dtantsurajya: if that's the case, it's a problem. we don't know when the job is over to repair the settings.10:06
ajyadtantsur: yes, sushy just sets the secure boot, have to check if Redfish gives back task monitor uri, if so, it could be used 10:07
dtantsur++10:08
opendevreviewMerged openstack/bifrost master: Enable epel repository for more than CentOS  https://review.opendev.org/c/openstack/bifrost/+/86108210:24
iurygregorygood morning Ironic10:49
TheJuliagood morning13:32
JayFhttps://review.opendev.org/c/openstack/ironic/+/860820 is failing on a couple of glance imageservice tests in py31014:19
JayFI thought we had fixed that?14:19
TheJuliadunno, not in current brain memory working set14:20
rpittauJayF: we did, but we missed one here https://review.opendev.org/c/openstack/ironic/+/86302914:24
JayFbam, nice14:25
opendevreviewMerged openstack/ironic master: Fix the invalid glance client test  https://review.opendev.org/c/openstack/ironic/+/86302914:40
rpittaubye! have a great weekend! o/15:26
*** rcastillo is now known as rcastillo|rover15:28
opendevreviewSven Kieske proposed openstack/ironic-inspector stable/ussuri: Fix port id vs uuid in ValidateInterfacesHook  https://review.opendev.org/c/openstack/ironic-inspector/+/86362615:45
opendevreviewSven Kieske proposed openstack/ironic-inspector stable/ussuri: Use port.id instead of port.uuid  https://review.opendev.org/c/openstack/ironic-inspector/+/86362715:48
dtantsurhave a nice weekend folks o/17:20
opendevreviewMerged openstack/ironic master: Cross test sushy with python 3.10  https://review.opendev.org/c/openstack/ironic/+/86214017:54
*** dtantsur_ is now known as dtantsur17:56
TheJuliao/18:14
*** dtantsur_ is now known as dtantsur18:29

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