Wednesday, 2020-12-16

*** tosky has quit IRC00:04
stevebakerTheJulia: hmm those jobs are still failing00:26
openstackgerritVerification of a change to openstack/ironic failed: Prepare release 16.1  https://review.opendev.org/c/openstack/ironic/+/76693800:56
*** rloo has quit IRC00:57
*** rcernin_ has joined #openstack-ironic01:09
*** rcernin has quit IRC01:10
*** k_mouza has joined #openstack-ironic01:36
*** JamesBenson has quit IRC01:39
*** k_mouza has quit IRC01:41
*** hoonetorg has quit IRC01:52
*** hoonetorg has joined #openstack-ironic02:03
openstackgerritArun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface  https://review.opendev.org/c/openstack/ironic-specs/+/74850302:25
*** benj_- has joined #openstack-ironic02:35
*** benj_ has quit IRC02:35
*** benj_- is now known as benj_02:35
openstackgerritArun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface  https://review.opendev.org/c/openstack/ironic-specs/+/74850302:43
*** rcernin_ has quit IRC02:46
*** mgoddard has quit IRC02:58
openstackgerritSteve Baker proposed openstack/ironic-lib stable/train: Use python3 for stable/train testing  https://review.opendev.org/c/openstack/ironic-lib/+/76722103:19
*** jamesden_ has quit IRC03:29
*** gyee has quit IRC03:43
*** ociuhandu has joined #openstack-ironic04:13
*** ociuhandu has quit IRC04:17
*** JamesBenson has joined #openstack-ironic04:24
*** JayF has quit IRC04:27
*** JayF has joined #openstack-ironic04:27
*** JamesBenson has quit IRC04:28
*** JamesBenson has joined #openstack-ironic04:29
openstackgerritMerged openstack/ironic-python-agent master: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic-python-agent/+/76708804:35
*** k_mouza has joined #openstack-ironic04:48
*** k_mouza has quit IRC04:53
*** JamesBenson has quit IRC05:29
*** JamesBenson has joined #openstack-ironic05:30
*** JamesBenson has quit IRC05:30
openstackgerritVishal Manchanda proposed openstack/ironic-ui master: Remove Chrome browser from karma.conf.js  https://review.opendev.org/c/openstack/ironic-ui/+/76688005:31
*** JamesBenson has joined #openstack-ironic05:35
*** tzumainn has quit IRC05:37
*** zzzeek has quit IRC05:39
*** JamesBenson has quit IRC05:39
*** zzzeek has joined #openstack-ironic05:39
openstackgerritankit proposed openstack/ironic master: Fixes issue of redfish firmware update  https://review.opendev.org/c/openstack/ironic/+/76491405:46
*** zzzeek has quit IRC05:46
openstackgerritankit proposed openstack/sushy master: Fixes issue of redfish firmware update  https://review.opendev.org/c/openstack/sushy/+/76491505:47
*** zzzeek has joined #openstack-ironic05:48
*** ricolin has joined #openstack-ironic05:59
*** zzzeek has quit IRC06:02
*** zzzeek has joined #openstack-ironic06:06
openstackgerritankit proposed openstack/sushy master: Fixes issue of redfish firmware update  https://review.opendev.org/c/openstack/sushy/+/76491506:44
*** abdysn has joined #openstack-ironic06:46
*** Abdallahyas has joined #openstack-ironic07:04
*** abdysn has quit IRC07:09
*** Abdallahyas has quit IRC07:34
*** Abdallahyas has joined #openstack-ironic07:34
*** zzzeek has quit IRC07:40
*** zzzeek has joined #openstack-ironic07:41
*** ociuhandu has joined #openstack-ironic07:50
*** zzzeek has quit IRC07:53
*** ociuhandu has quit IRC07:54
*** zzzeek has joined #openstack-ironic07:55
*** Qianbiao has joined #openstack-ironic07:56
arne_wiebalck Good morning, ironic!08:03
*** Abdallahyas has quit IRC08:10
*** abdysn has joined #openstack-ironic08:14
*** rpittau|afk is now known as rpittau08:18
rpittaugood morning ironic! o/08:18
*** tosin has joined #openstack-ironic08:23
*** ociuhandu has joined #openstack-ironic08:31
*** zzzeek has quit IRC08:31
*** tosky has joined #openstack-ironic08:33
*** zzzeek has joined #openstack-ironic08:34
*** zzzeek has quit IRC08:39
*** zzzeek has joined #openstack-ironic08:41
*** jawad_axd has joined #openstack-ironic08:43
*** dougsz has joined #openstack-ironic08:44
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/train: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic-python-agent/+/76728408:46
*** mgoddard has joined #openstack-ironic08:46
*** dougsz has quit IRC08:49
*** ociuhandu has quit IRC08:51
openstackgerritRiccardo Pittau proposed openstack/sushy-tools master: Adding basic support for processors schema  https://review.opendev.org/c/openstack/sushy-tools/+/76190908:58
*** zzzeek has quit IRC08:59
*** zzzeek has joined #openstack-ironic09:00
openstackgerritRiccardo Pittau proposed openstack/sushy-tools master: Adding basic support for processors schema  https://review.opendev.org/c/openstack/sushy-tools/+/76190909:01
*** zzzeek has quit IRC09:05
*** mgoddard has quit IRC09:06
*** zzzeek has joined #openstack-ironic09:06
*** mgoddard has joined #openstack-ironic09:07
*** zzzeek has quit IRC09:11
*** zzzeek has joined #openstack-ironic09:14
*** derekh has joined #openstack-ironic09:18
*** zzzeek has quit IRC09:23
*** abdysn has quit IRC09:25
*** zzzeek has joined #openstack-ironic09:25
*** jawad_axd has quit IRC09:30
*** jawad_axd has joined #openstack-ironic09:30
*** zzzeek has quit IRC09:41
*** zzzeek has joined #openstack-ironic09:42
*** tosky_ has joined #openstack-ironic09:47
*** tosky is now known as Guest2437209:49
*** tosky_ is now known as tosky09:49
*** Guest24372 has quit IRC09:50
openstackgerritMerged openstack/ironic master: Prepare release 16.1  https://review.opendev.org/c/openstack/ironic/+/76693809:54
*** zzzeek has quit IRC09:57
*** zzzeek has joined #openstack-ironic09:58
*** dtantsur|afk is now known as dtantsur09:59
dtantsurmorning ironic10:00
openstackgerritMerged openstack/ironic-python-agent stable/victoria: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic-python-agent/+/76721110:00
dtantsurTheJulia: it probably means dnf has a direct dependency on some cockpit components10:00
rpittaugood morning dtantsur :)10:00
*** abdysn has joined #openstack-ironic10:11
*** zzzeek has quit IRC10:13
*** zzzeek has joined #openstack-ironic10:14
openstackgerritRiccardo Pittau proposed openstack/python-ironicclient master: Remove lower-constraints job  https://review.opendev.org/c/openstack/python-ironicclient/+/76698610:15
rpittaulol in virtualbmc we had lower-constraints twice10:16
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Remove lower-constraints job  https://review.opendev.org/c/openstack/virtualbmc/+/76708910:16
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Make the debian job voting and start publishing debian images  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76715810:17
dtantsurrpittau: lol indeed10:18
dtantsurcould you check https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/762816 please?10:18
rpittausure :)10:18
dtantsuralso arne_wiebalck if you have a minute ^^^10:18
rpittaudtantsur: if you have time today please check the l-c removal patches10:19
dtantsurI will10:19
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent stable/victoria: Generate TLS certificates with validity time in the past  https://review.opendev.org/c/openstack/ironic-python-agent/+/76731010:19
arne_wiebalckdtantsur: checking ...10:19
dtantsurthere are also a few outstanding IPA patches that could be merged before the release10:21
rpittaumozjs60 ? :D10:21
dtantsuryeah :)10:21
rpittauI'm not even going to ask :D10:22
dtantsurall these came from an actual inspection of the ramdisk's contents, I'm not to blame10:22
rpittauI know, just amazed :)10:23
arne_wiebalckjust checked what we remove downstream: mozjs6010:24
arne_wiebalck:)10:24
arne_wiebalckwe also remove postfix10:24
dtantsurI can follow-up with postfix removal. Since it didn't get on my radar, it's probably not huge.10:25
arne_wiebalck(I think I looked at the 10 or 20 largest packages)10:25
arne_wiebalckhmm, not sure how/why I though this is a sensible thing to remove10:25
arne_wiebalck*thought10:25
rpittaudon't we want to implement an ipa-powered mail relay server ?10:26
dtantsurit's a lower priority than a bitcoin miner10:27
rpittautrue10:28
arne_wiebalckthe pkg-map changes do not remove packages, correct?10:28
* arne_wiebalck thinks we rely on curl in the IPA10:28
dtantsurjust imagine how many bitcoins you can mine on that 6k servers of arne_wiebalck!10:28
dtantsurarne_wiebalck: no, pkg-map is just a mapping of generic names to distro-specific10:28
dtantsurso "curl": "curl" makes no sense10:28
arne_wiebalckright :)10:28
dtantsuralso IIRC curl is required by DIB itself10:29
*** lbragstad has quit IRC10:31
*** lbragstad has joined #openstack-ironic10:31
rpittauthis should pass now, just need +W -> https://review.opendev.org/c/openstack/ironic-python-agent/+/76529410:34
dtantsurchecking10:36
dtantsurrpittau: I'd hear from TheJulia if she's fine with the release note change10:37
rpittaudtantsur: oh you're right, didn't see the comment10:41
*** ociuhandu has joined #openstack-ironic10:41
*** JamesBenson has joined #openstack-ironic10:42
*** ociuhandu_ has joined #openstack-ironic10:44
*** zzzeek has quit IRC10:44
*** ociuhandu_ has quit IRC10:46
*** JamesBenson has quit IRC10:47
*** zzzeek has joined #openstack-ironic10:47
*** ociuhandu_ has joined #openstack-ironic10:47
*** ociuhandu has quit IRC10:47
*** ociuhandu_ has quit IRC10:52
*** ociuhandu has joined #openstack-ironic10:52
*** ociuhandu has quit IRC10:53
*** tkajinam has quit IRC10:54
*** zzzeek has quit IRC11:07
*** k_mouza has joined #openstack-ironic11:08
*** zzzeek has joined #openstack-ironic11:09
*** mkowalski has joined #openstack-ironic11:18
*** JamesBenson has joined #openstack-ironic11:20
*** JamesBenson has quit IRC11:25
*** anuradha1904 has joined #openstack-ironic11:40
*** zzzeek has quit IRC11:41
*** zzzeek has joined #openstack-ironic11:44
*** ociuhandu has joined #openstack-ironic11:55
*** zzzeek has quit IRC11:55
*** zzzeek has joined #openstack-ironic11:57
*** ociuhandu has quit IRC11:59
*** JamesBenson has joined #openstack-ironic12:00
*** JamesBenson has quit IRC12:03
*** JamesBenson has joined #openstack-ironic12:04
*** zzzeek has quit IRC12:11
*** k_mouza has quit IRC12:13
*** k_mouza has joined #openstack-ironic12:13
*** zzzeek has joined #openstack-ironic12:13
*** k_mouza has quit IRC12:17
*** zzzeek has quit IRC12:18
*** zzzeek has joined #openstack-ironic12:20
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Fix default disk label with partition images  https://review.opendev.org/c/openstack/ironic-python-agent/+/76245312:29
*** k_mouza has joined #openstack-ironic12:29
*** zzzeek has quit IRC12:31
*** zzzeek has joined #openstack-ironic12:32
* rpittau needs coffee12:35
openstackgerritVerification of a change to openstack/ironic-python-agent-builder failed: Clean up package installation  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76281612:38
*** ociuhandu has joined #openstack-ironic12:58
*** paras333 has quit IRC13:08
Qianbiaodtantsur curl is required by almost all base image to fetch qcow image.13:12
*** ociuhandu has quit IRC13:12
*** ociuhandu has joined #openstack-ironic13:13
dtantsuryep13:16
openstackgerritVladyslav Drok proposed openstack/networking-baremetal master: Fix exception handling when querying ironic ports  https://review.opendev.org/c/openstack/networking-baremetal/+/76585113:16
*** ociuhandu has quit IRC13:18
*** ociuhandu has joined #openstack-ironic13:21
*** ociuhandu has quit IRC13:24
*** ociuhandu has joined #openstack-ironic13:24
*** k_mouza has quit IRC13:31
*** zzzeek has quit IRC13:34
*** zzzeek has joined #openstack-ironic13:36
*** k_mouza has joined #openstack-ironic13:37
openstackgerritRiccardo Pittau proposed openstack/python-ironicclient master: Remove lower-constraints job  https://review.opendev.org/c/openstack/python-ironicclient/+/76698613:42
openstackgerritMerged openstack/ironic-python-agent master: Correctly decode error messages from ironic API  https://review.opendev.org/c/openstack/ironic-python-agent/+/76583813:43
*** zzzeek has quit IRC13:45
*** Abdallahyas has joined #openstack-ironic13:46
*** zzzeek has joined #openstack-ironic13:49
*** abdysn has quit IRC13:49
*** abdysn has joined #openstack-ironic13:49
*** Abdallahyas has quit IRC13:51
*** ociuhandu has quit IRC13:54
*** lbragstad has quit IRC13:54
*** lbragstad has joined #openstack-ironic13:57
*** lbragstad has quit IRC13:57
*** lbragstad has joined #openstack-ironic13:58
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/ussuri: Run dib ipa src jobs on ubuntu focal  https://review.opendev.org/c/openstack/ironic-python-agent/+/76735513:59
mgoddardDuring sync_power_state, max retries exceeded for node e81f00f4-1532-41a5-b8e9-e70465d05860, node state None does not match expected state 'None'. Updating DB state to 'None' Switching node to maintenance mode. Error: An exclusive lock is required, but the current context has a shared lock.: ironic.common.exception.ExclusiveLockRequired: An exclusive lock is required, but the current context has a14:01
mgoddardshared lock.14:01
mgoddard^ is that a known issue?14:01
*** lbragstad_ has joined #openstack-ironic14:03
*** ociuhandu has joined #openstack-ironic14:04
TheJuliagood morning14:04
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/ussuri: Run dib ipa src jobs on ubuntu focal  https://review.opendev.org/c/openstack/ironic-python-agent/+/76735514:04
*** ociuhandu has quit IRC14:05
*** ociuhandu has joined #openstack-ironic14:05
openstackgerritMerged openstack/ironic-python-agent stable/victoria: Generate TLS certificates with validity time in the past  https://review.opendev.org/c/openstack/ironic-python-agent/+/76731014:05
TheJuliamgoddard: could we get more logging context for that node?14:05
*** lbragstad has quit IRC14:05
mgoddardTheJulia: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_550/766203/12/check/kayobe-overcloud-centos8/5502141/primary/kolla/ironic/ironic-conductor.txt14:06
openstackgerritVerification of a change to openstack/ironic-python-agent-builder failed: Add additional firmware images to be excluded  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76716914:07
*** rloo has joined #openstack-ironic14:08
*** ociuhandu has quit IRC14:08
*** ociuhandu has joined #openstack-ironic14:09
TheJuliahmm14:09
TheJuliamgoddard: hmm....14:09
mgoddardTheJulia: I reproduced it locally14:09
dtantsurmorning TheJulia14:10
mgoddardalthough it is based on unmerged changes to tenks & kayobe14:10
TheJuliamgoddard: what does your db reservation field show14:10
dtantsurmgoddard: sounds like a programming error14:10
dtantsurthe exclusive lock check happens in a decorator14:10
mgoddardhowever, these changes did work a couple days ago14:10
TheJulialooks like node update never released the lock in this, but we need to know the state of the node in the db first14:11
mgoddardI will do a little digging. Just wondered if it was a known issue14:11
TheJuliafirst I'm hearing of this specifically14:13
TheJuliaIt seems very odd, to be honest14:13
* TheJulia needs coffee14:15
*** abdysn has quit IRC14:22
*** ociuhandu has quit IRC14:29
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent stable/victoria: Correctly decode error messages from ironic API  https://review.opendev.org/c/openstack/ironic-python-agent/+/76732214:38
*** ociuhandu has joined #openstack-ironic14:40
*** Qianbiao has quit IRC14:41
*** tzumainn has joined #openstack-ironic14:46
dtantsurTheJulia: once you coffeeinate, your input is required on https://review.opendev.org/c/openstack/ironic-python-agent/+/76529414:47
openstackgerritRichard G. Pioso proposed openstack/ironic master: WIP: Create 'redfish' driver Redfish Interop Profile  https://review.opendev.org/c/openstack/ironic/+/75406114:50
*** ociuhandu has quit IRC14:51
*** belmoreira has joined #openstack-ironic15:01
*** zzzeek has quit IRC15:06
*** zzzeek has joined #openstack-ironic15:08
*** ricolin_ has joined #openstack-ironic15:15
TheJuliarutro15:33
* TheJulia goes and grabs more coffee15:33
*** ociuhandu has joined #openstack-ironic15:41
derekhHi all, I've been looking into virtualmedia on a system (HPE ProLiant DL380, UEFI boot w/ redfish), the vmedia gets attached and BootSourceOverrideTarget is set to "Cd" but during boot gets ignored, has anybody hit this before ?15:43
derekhI can manualy change the boot order to have "PciRoot(0x0)/Pci(0x1C,0x4)/Pci(0x0,0x4)/USB(0x1,0x0)" at the top and the host boots from vmedia but there is no way for ironic to do it via one time boot15:44
TheJuliadtantsur: commented15:45
openstackgerritMerged openstack/ironic master: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic/+/76699515:46
openstackgerritRiccardo Pittau proposed openstack/ironic-lib stable/victoria: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic-lib/+/76720315:49
NobodyCamGood Morning Ironic folks15:49
rpittauTheJulia: I disabled dstat here ^ focal still has the issue15:49
rpittauhey NobodyCam :)15:49
NobodyCamhey hey rpittau good morning!15:50
openstackgerritRiccardo Pittau proposed openstack/ironic stable/victoria: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic/+/76732315:51
*** ociuhandu has quit IRC15:53
openstackgerritRiccardo Pittau proposed openstack/ironic stable/victoria: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic/+/76732315:53
*** k_mouza has quit IRC15:53
dtantsurmorning NobodyCam! how's pre-xmas life?15:54
*** k_mouza has joined #openstack-ironic15:54
rpittaummmm we might need to lock version of ipa-builder for ussuri and train15:54
openstackgerritRiccardo Pittau proposed openstack/ironic stable/ussuri: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic/+/76732415:54
openstackgerritRiccardo Pittau proposed openstack/ironic stable/ussuri: Remove lower-constraints job  https://review.opendev.org/c/openstack/ironic/+/76732415:55
dtantsurrpittau: why?15:55
rpittaudtantsur: because ipa-builder is not branched, and we've migrated to tinyipa 11.x that can be built only on focal or later, because of recent kernel15:55
dtantsurahhh, because of that15:56
TheJuliagood morning NobodyCam15:57
TheJuliarpittau: ack15:57
rpittauyeah, I hope to find a very low-pain way to do it15:57
TheJuliaderekh: what version of hte ilo firmware is that15:58
TheJulia?15:58
rpittauussuri really does not want to run on focal "[ERROR] ./stack.sh:227 If you wish to run this script anyway run with FORCE=yes"16:01
dtantsuryeah, devstack hardcodes supported distros16:02
dtantsurI don't see a huge problem with capping IPA-builder, we already do it for the tempest plugin16:02
rpittauright16:02
rpittauI'll copy-paste from there :D16:02
derekhTheJulia: it had been one from January but we have now upgraded too "iLO Firmware Version 2.31 Oct 13 2020" and having the same problem16:02
TheJuliaderekh: what is the override field set to in terms of continious or onetime?16:03
derekh'BootSourceOverrideEnabled': 'Once', 'BootSourceOverrideMode': 'UEFI', 'BootSourceOverrideTarget': 'Cd', 'BootSourceOverrideTarget@Redfish.AllowableValues': ['None', 'Cd', 'Hdd', 'Usb', 'SDCard', 'Utilities', 'Diags', 'BiosSetup', 'Pxe', 'UefiShell', 'UefiHttp', 'UefiTarget'], 'UefiTargetBootSourceOverride': 'None'16:04
TheJuliaugh16:05
TheJuliaUGH16:05
TheJuliathey changed the behavior16:05
TheJuliathey added the Uefi override targets16:05
derekhTheJulia: Ya, I can successfully boot from virtmedia by doing curl -k -X PATCH  -u XXX:XXX -H "Content-Type: application/json" -H 'OData-Version: 4.0' https://10.46.61.16/redfish/v1/Systems/1 -d '{"Boot": {"UefiTargetBootSourceOverride": "PciRoot(0x0)/Pci(0x1C,0x4)/Pci(0x0,0x4)/USB(0x1,0x0)"}}'16:07
TheJuliaderekh: does it support the aliased name fields for targets in that version?16:08
derekhBut without knowing that PciRoot... address I don't see how ironic could do it16:08
*** ociuhandu has joined #openstack-ironic16:08
derekhTheJulia: don't know tbh, do you know how I could find out? If not I'll find out16:10
TheJuliauhh... can you grab the entire boot field contents, I'd recognize. I think I started a patch somewhere...16:11
TheJuliaderekh: https://review.opendev.org/c/openstack/sushy/+/718275/1/sushy/resources/system/system.py16:12
dtantsurso, the time has come. sigh.16:13
TheJuliaFor a very very very strong beverage?16:14
TheJuliaor three?16:14
TheJuliaor six?16:14
derekhTheJulia: this? https://goodsquishy.com/upload/551e5d43c62112acdb1216:14
dtantsurI have a birthday tomorrow, 3 bottles of craft beer and a bottle of japanese whiskey are waiting!16:14
TheJuliadtantsur: yum!16:14
derekhdtantsur: enjoy, your birthday? happy birthday16:15
dtantsurthank you :)16:15
TheJuliayes, happy birthday dtantsur !16:15
rpittauhappy birthday dtantsur :)16:16
TheJuliaderekh: so for hp gear, your having to set set it to boot from usb for virtual media ? with the media detached, does the usb device show up in the list16:16
dtantsurthanks folks!16:17
dtantsur"UsbClass(0xFFFF,0xFFFF,0xFF,0xFF,0xFF)" - it?16:17
bfourniedtantsur: Happy Birthday \o/16:17
rlooHappy birthday dtantsur! No doubt that you will be enjoying yourself tomorrow! Whee.16:18
dtantsur\o/16:18
TheJuliadtantsur: your taking tomorrow off right?16:18
dtantsurand it will be a round number - 32 :D16:18
dtantsurTheJulia: yep, and Friday too16:18
TheJuliaawesome!16:18
TheJuliaI'm off Friday, but Thursday I'm hoping to also go ship my old work laptop back so that will be an interesting errand into a pandemic zone if I can't figure out a easy/low/no risk way of getting it shipped16:19
dtantsurTheJulia: why hurry with that? I think everyone will understand16:19
dtantsurI was supposed to ship mine more than a year ago16:20
TheJuliadtantsur: statistically thursdays are the most idle time at the local shipping place.16:20
TheJulialol16:20
TheJuliawow16:20
TheJuliaokay16:20
dtantsurI don't think anyone in their right mind will blame you for avoiding a post office amid pandemic16:20
* TheJulia wonders what usbclass is....16:21
dtantsurI confused Usb and Cd, sorry (and I'm still sober!)16:21
dtantsurbut it looks like some sort of a wildcard for USB devices16:21
TheJuliaWell, luckily it is a pre-paid label. I just pack it up, tape it, hand it over and ask for a receipt16:21
TheJuliawell, while we're requesting cd for general override, i bet in uefi mode it is being viewed as a usb device16:22
*** jawad_axd has quit IRC16:23
dtantsurderekh: will it work if you use UsbClass(0xFFFF,0xFFFF,0xFF,0xFF,0xFF)"?16:24
derekhTheJulia: iirc the USB device is always there but I'll double check once I can get back on the system16:24
derekhdtantsur: I haven't tried it but can do that next16:24
* TheJulia wonders if PciRoot(0x0)/Pci(0x1C,0x4)/Pci(0x0,0x4)/USB(0x1,0x0) is a special device.16:24
TheJuliait is the same on a dl380g1016:24
dtantsurand there is also "PciRoot(0x0)/Pci(0x1C,0x4)/Pci(0x0,0x4)/USB(0x1,0x0)/CDROM(0x1)/\\EFI\\redhat\\shimx64.efi"16:25
dtantsurworst comes to worst, we have to hardcode these.. but I'd rather not.16:25
TheJuliai suspect it is auto-adding that16:25
dtantsuryeah, the value that works for derekh is not even on the list16:26
dtantsurI guess you can trim any part from it, and it's still allowed?16:26
TheJuliaoh...16:26
TheJuliathe virtual cd is a virtual usb device16:27
TheJuliaomg16:27
dtantsuron the other hand, we can finally choose which NIC to boot from!16:28
rpittaumaybe it's on the same bus16:28
dtantsurI wonder if we can get the PCI path from the virtual media device properties16:29
dtantsurderekh: could you check ^^ as well?16:30
TheJuliaIt is looking like that is consistent across proliant models16:30
derekhdtantsur: TheJulia getting access back onto the system, I'll grab the full json both with and without the vmedia attched and get back to you16:30
dtantsurderekh: virtual media is a separate resource under Managers16:30
*** ricolin_ has quit IRC16:31
TheJuliadtantsur: bad news on this field, it is up to the vendor what they want to put into it :\16:32
dtantsurit = UefiTargetBootSourceOverride?16:32
TheJuliayeah16:33
dtantsuromg16:33
dtantsurthe redfish standard keeps being automation-hostile16:33
TheJuliaOn one vendor it is "0000", another vendor it is the hex string for the uefi order entry16:33
TheJuliadtantsur: this was all proposed a while back16:33
dtantsurokay, "has always been" :)16:33
TheJuliadtantsur: and there are new fields to make this easy, just they don't support them yet :(16:33
dtantsurokay, "has always been, but may stop soon if we're lucky" :))16:34
TheJuliaso16:35
TheJuliathe UsbClass entry is "any usb device in the system" per HPE docs16:35
dtantsurmay do the trick if nothing else helps..16:36
TheJuliastill digging, since the PDF only printed as images *grumbles*16:36
TheJuliabut it looks like it may be reliable for the bmc16:36
TheJuliait being the special pciroot path16:36
dtantsurbtw virtual media supports username/password for images, we should support them too16:37
TheJulia++16:37
dtantsurbut now.. we're finally in the field of hardcoding vendor differences16:38
dtantsurI wonder if should just merge sushy-oem-idrac in sushy and stop pretending we can maintain any distance...16:38
TheJuliadtantsur: wondering the same thing16:40
TheJuliamental note while I skim docs: vmedia in ilos is == CD.Virtual.2.1 for iso images or Usb device HD.Virtual.1.116:41
dtantsuroh, I need to test attaching configdrives16:41
TheJuliaerr, conflicting docs yay16:42
TheJulia2.1 is apparently the VID too16:42
dtantsurVID?16:42
TheJuliavirutal install device16:42
* TheJulia loads up the official docs and digs16:42
TheJuliafor the api surface16:42
*** juanoterocas has joined #openstack-ironic16:43
juanoterocasis there an easy way  to setup Ironic ?, tutorial16:43
dtantsurjuanoterocas: try https://docs.openstack.org/bifrost/latest/ maybe?16:44
dtantsuror do you need it as part of openstack?16:44
juanoterocasyes, integrating with openstack should be easy16:45
dtantsurjuanoterocas: do you use any tool to install openstack itself? maybe it supports ironic already?16:45
juanoterocasI-m using microstack to test locally, after that I'm planning to use juju charm16:46
dtantsurjuanoterocas: I think you should check if there is a charm for ironic then16:46
TheJuliaI'm thinkign the pci references are bus address locations, not device IDs16:47
dtantsurjuanoterocas: there seem to be https://opendev.org/openstack/charm-ironic16:47
*** ociuhandu_ has joined #openstack-ironic16:49
juanoterocasIt is ok later when using full openstack charm but at this very moment I'm doing some local test using microstack.16:49
juanoterocasand I want to test integration with Virtual machines and baremetal (physical machine)16:51
juanoterocasusing neutron network.16:51
*** ociuhandu has quit IRC16:52
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Add ironic-ramdisk-base to build general-purpose ramdisks  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76737616:52
dtantsurjuanoterocas: you can start following our docs for manual installation, but it's not a trivial task (as we most openstack services). I assume microstack does not support ironic?16:53
dtantsurmaybe you should start with bifrost, then modify it for Neutron/Glance/Nova integration16:53
*** ociuhandu has joined #openstack-ironic16:53
juanoterocassome files are in differet places since is a snap installation but I managed to found where I need to do some things16:54
juanoterocasI'll check bifrost, I haven't work with ansible before, but i'll give a try16:56
dtantsurjuanoterocas: you don't need to know ansible to get a simple installation started. then you can hack it manually to work with your microstack installation.16:56
dtantsurIt's not going to be trivial, unfortunately. https://docs.openstack.org/ironic/latest/ is your help16:57
*** ociuhandu_ has quit IRC16:57
juanoterocasthanks dtantsur. Yes in fact I have followed that documentation and I have some things working now16:59
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Fix default disk label with partition images  https://review.opendev.org/c/openstack/ironic-python-agent/+/76245317:00
*** ociuhandu_ has joined #openstack-ironic17:02
*** tosin has quit IRC17:02
TheJuliaPciRoot(0x0)/Pci(0x1C,0x4)/Pci(0x0,0x4)/USB(0x1,0x0) is PCI.Slot.4.1 or -> pci(0x1c,0x4) <-- main bus or bridge it looks like. Pci(0x0,0x4) <- slot 4 USB(0x1,0x0) <-- device 1. Apparently, we MAY be abel to shoren the path. Say boot to slot 4. but that may also not work since there is a CD nested below this. So I suspect this likely varies by hardware, I suspect we're going to need insight from the HPE ilo17:04
TheJuliafolks17:04
*** JayF has quit IRC17:04
TheJuliathen again, I'd eexpect a dl360 and 380 to be similar. An edgeline is likely different17:05
*** ociuhandu has quit IRC17:05
TheJuliaAnyone got an edgeline handy? :)17:05
openstackgerritRiccardo Pittau proposed openstack/ironic stable/ussuri: Cap version of ipa-builder to 2.2.0  https://review.opendev.org/c/openstack/ironic/+/76737917:06
openstackgerritMerged openstack/ironic-python-agent-builder master: Clean up package installation  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76281617:06
*** ociuhandu_ has quit IRC17:07
*** gyee has joined #openstack-ironic17:07
rpittauif that works I guess we need to combine it with the l-c removal17:07
*** juanoterocas has quit IRC17:08
*** JayF has joined #openstack-ironic17:08
rpittaugood rest of the day everyone! o/17:11
*** rpittau is now known as rpittau|afk17:11
TheJuliao/17:11
* TheJulia makes her desktop cry by trying to load multiple super dense dynamic documentation pages17:11
TheJuliaI can literally hear my fans at full power17:11
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Handle situation when a configdrive is already mounted  https://review.opendev.org/c/openstack/ironic-python-agent/+/76738117:18
dtantsura fix for my recent patch ^^^17:18
*** ociuhandu has joined #openstack-ironic17:18
dtantsurmaybe we shouldn't fail at all if we cannot copy the configuration..17:19
TheJuliashoot, didn't shiv say they were in shutdown starting on the 16th or something?17:21
dtantsurI don't remember17:21
TheJuliaderekh: maybe send shiv an email and try to obtain insight. That being said I'm suspecting you won't hear back until next year17:22
*** ociuhandu has quit IRC17:23
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Enable ramdisk deploy by default, disable iscsi  https://review.opendev.org/c/openstack/bifrost/+/76738417:26
*** belmoreira has quit IRC17:34
*** k_mouza has quit IRC17:35
mgoddardTheJulia, dtantsur: found the issue. ipmitool.get_power_state now calls ipmitool.detect_vendor, which requires an exclusive lock17:45
TheJuliaugh17:46
mgoddardTheJulia: https://review.opendev.org/c/openstack/ironic/+/75719817:46
mgoddard(but I guess you knew that :) )17:46
TheJuliaI guess we could only upgrade the lock if we have something to save?!17:46
mgoddarddetect_vendor itself does not seem to save17:47
mgoddardI wonder why ironic CI did not catch this. Perhaps vendor is set on nodes17:48
mgoddardTheJulia: anyway, home duties call. Let me know if you do anything with this, otherwise I'll pick it up tomorrow17:49
TheJuliamgoddard: good question, working on a fix17:51
derekhIt'll be the morning before I get to attach vmedia to that hardware again to check the json as its doing something at the moment, I'll pick it up then17:51
TheJuliaderekh: ok17:52
TheJuliaderekh: I'd look on an edgeline machine as well17:52
mgoddardTheJulia: verified that removing the decorator allows the power sync to work17:52
TheJuliamgoddard: ack17:52
derekhTheJulia: will see if I can get a look on one17:53
* TheJulia ponders for a moment if there should be additional testing17:53
openstackgerritLeo McGann proposed openstack/ironic-specs master: Add security interface spec  https://review.opendev.org/c/openstack/ironic-specs/+/57671817:55
openstackgerritJulia Kreger proposed openstack/ironic master: Remove detect_vendor decorator  https://review.opendev.org/c/openstack/ironic/+/76739017:55
openstackgerritDmitry Tantsur proposed openstack/ironic master: Documenting using ramdisks with the ramdisk deploy interface  https://review.opendev.org/c/openstack/ironic/+/76739117:57
dtantsurfinally ^^17:57
TheJuliadtantsur: and that element has merged?17:58
dtantsurnot yet, see depends-on17:59
dtantsurGot heartbeat message from node 4e41df61-84b1-5856-bfb6-6b5f2cd3dd11, but the driver redfish does not support heartbeating17:59
dtantsurWAIT WUT17:59
dtantsurI think cleaning does not like the ramdisk deploy..17:59
TheJuliadidn't it get decoupled from the agent base?18:00
dtantsurwell, not in the right way, apparently.. lemme try something18:00
* TheJulia has removed stickers from old lapto18:02
* TheJulia is sad now and needs all the stickers18:02
openstackgerritDmitry Tantsur proposed openstack/ironic master: Include HeartbeatMixin in the ramdisk deploy  https://review.opendev.org/c/openstack/ironic/+/76739318:07
dtantsura simple fix ^^18:07
TheJuliaYeah, that should do it18:08
TheJuliaI went ahead and +2+A'ed it18:09
TheJuliasince it is trivial18:09
TheJuliahttps://review.opendev.org/c/openstack/ironic/+/767390 <-- also really trivial and I think mgoddard would appreciate it :)18:09
TheJuliadidn't add a release note since on backporting that will likely need to be squashed in18:10
openstackgerritDmitry Tantsur proposed openstack/ironic master: Document using ramdisks with the ramdisk deploy interface  https://review.opendev.org/c/openstack/ironic/+/76739118:13
dtantsurokay, this actually works (forgot about DHCP)18:13
* dtantsur wonders if debian-minimal would actually work for DHCP-less18:18
openstackgerritLeo McGann proposed openstack/ironic-specs master: Add security interface spec  https://review.opendev.org/c/openstack/ironic-specs/+/57671818:18
TheJuliamaybe18:23
TheJuliaif there is no networkmanager18:23
dtantsurI built a bare ramdisk, and nothing DHCP'ed there18:23
TheJuliahmmm18:23
*** derekh has quit IRC18:26
dtantsurIT WORKS!18:27
dtantsuromg, I finally found something that works with DHCP-less!18:27
dtantsur28 minutes past the times I should have left for PTO :D18:28
dtantsurnow the problem is that we always depend on dhcp-all-interfaces18:28
dtantsur(I manually removed it)18:28
dtantsurmm, bifrost is broken?18:33
dtantsurah!18:34
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Enable ramdisk deploy by default, disable iscsi  https://review.opendev.org/c/openstack/bifrost/+/76738418:35
openstackgerritMerged openstack/ironic-python-agent stable/victoria: Correctly decode error messages from ironic API  https://review.opendev.org/c/openstack/ironic-python-agent/+/76732218:38
openstackgerritDmitry Tantsur proposed openstack/ironic master: Document that DHCP-less deploy does work with debian-minimal  https://review.opendev.org/c/openstack/ironic/+/76739918:45
dtantsurTheJulia:  some good news here ^^^18:45
*** MentalSiege has joined #openstack-ironic18:49
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Make the debian job voting and start publishing debian images  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76715818:49
dtantsurthings suspiciously work today, I better leave before they stop :)18:50
dtantsurhave a nice rest of the week, folks, see you on Monday or in the near year18:50
*** dtantsur is now known as dtantsur|afk18:50
TheJuliadtantsur|afk: \o/19:04
TheJuliahave a wonderful pto!19:04
*** stevebaker has quit IRC19:35
*** zzzeek has quit IRC19:36
*** stevebaker has joined #openstack-ironic19:36
*** brtknr has quit IRC19:37
*** zzzeek has joined #openstack-ironic19:38
dkingI"m not sure if this is the right place to ask this, but I'm hoping somebody has more experience. I have an environment variable that I would like to bake into my IPA image for use in my custom hardware manager. Where would be the best place to set that? I'm using the IPA Builder script (so, DIB) to build the image. Would I have my install script place it into /etc/profile, or would there be some better place to put it?19:40
*** brtknr has joined #openstack-ironic19:40
dkingI tried placing it in a .env and loading it setup.py with dotenv, but it doesn't seem to be finding it.19:42
* TheJulia rips out extra['vif_port_id')19:42
stevebakermorning19:45
TheJuliaGood morning19:47
stevebakerTheJulia: I made the stable/train ironic-lib jobs align closer to stable/ussuri, which I don't know if it is appropriate. Either way they're still broken https://review.opendev.org/c/openstack/ironic-lib/+/76722119:51
*** lbragstad_ is now known as lbragstad19:54
TheJuliadking: you can't really edit in place the env vars upon launch, I think you'd need to have it set in the systemd unit in the ramdisk19:55
dkingSo, in the IPA service file?19:57
*** zzzeek has quit IRC20:03
TheJuliai think so yeah20:03
TheJuliastevebaker: fyi, I'm ripping out ye olde extra['vif_port_id'] support for the rbac stuffs20:04
TheJuliaI didn't realize how much testing had to be changed though :\20:04
TheJuliastevebaker: ugh, looks like we have to change the distro out20:05
TheJuliaor define it :\20:05
TheJulia"kernel too old" :(20:05
stevebakerTheJulia:  FATAL: kernel too old20:05
stevebakerja20:05
TheJuliaI think that means we need to be on focal20:05
*** zzzeek has joined #openstack-ironic20:07
*** juanoterocas has joined #openstack-ironic20:08
TheJuliastevebaker: oh, you know this means ironic is broken. I'd go ahead and just pull the patch downstream. It is going to take some more work to fix upstream ci20:08
TheJuliaand not work that is likely to merge "soon"20:08
TheJuliasadly20:08
stevebakerTheJulia: already did that :)20:08
TheJuliaack20:09
stevebakerTheJulia: so maybe try nodeset: openstack-single-node-focal on the ironic-lib-base job definition?20:09
* TheJulia goes and removes a pizza from the oven20:09
TheJuliawe could try that20:09
stevebakerwait, ironic-lib-base inherits ironic-base, I wonder what that is on stable/train20:10
TheJulialikely not focal20:11
stevebakerbionic20:12
TheJulia:(20:12
TheJuliamaybe change ironic on that branch?20:12
TheJuliaerr, ironic-base20:12
TheJuliasince the in-branch builds likely don't work either due to this20:13
openstackgerritSteve Baker proposed openstack/ironic-lib stable/train: Use python3 for stable/train testing  https://review.opendev.org/c/openstack/ironic-lib/+/76722120:14
*** zzzeek has quit IRC20:14
stevebakerI'll try this first20:14
stevebakervictoria is the first ironic-base job to use focal20:15
*** zzzeek has joined #openstack-ironic20:17
openstackgerritSteve Baker proposed openstack/ironic stable/train: Switch from bionic to focal nodeset  https://review.opendev.org/c/openstack/ironic/+/76741620:21
*** zzzeek has quit IRC20:25
*** zzzeek has joined #openstack-ironic20:27
TheJuliaI wonder if something else changed :\20:29
*** ociuhandu has joined #openstack-ironic20:35
*** ociuhandu has quit IRC20:40
stevebakerblarg https://zuul.opendev.org/t/openstack/build/3f8858185fc348d8a3ad17e2da084225/log/job-output.txt#197320:51
TheJuliaugh20:52
*** SpamapS has quit IRC20:53
TheJuliai fix things and more tests break  \o/21:27
stevebakermaybe we should just stop for the year21:31
TheJuliaI'm liking that idea21:31
TheJuliaback down to 24 tests21:33
openstackgerritSteve Baker proposed openstack/ironic-lib stable/train: Use python3 for stable/train testing  https://review.opendev.org/c/openstack/ironic-lib/+/76722121:33
stevebakerTheJulia: btw regarding https://review.opendev.org/c/openstack/placement/+/760240 I've always wanted to use gabbi for something. I might stick with ddt for now but align the format so a possible switchover is easier21:35
TheJuliaokay21:35
TheJuliaI think that is reasonable21:36
TheJulia12 more tests fixed \o/21:40
openstackgerritArun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface  https://review.opendev.org/c/openstack/ironic-specs/+/74850321:47
TheJuliaone... test.22:01
*** sdanni has quit IRC22:03
*** rcernin has joined #openstack-ironic22:17
*** derekh has joined #openstack-ironic22:58
*** tkajinam has joined #openstack-ironic22:59
*** juanoterocas has quit IRC23:05
NobodyCammorning TheJulia and dtantsur|afk23:05
NobodyCam;p been one of those days23:05
openstackgerritJulia Kreger proposed openstack/ironic master: WIP: remove vif_port_id  https://review.opendev.org/c/openstack/ironic/+/76743223:07
TheJuliastevebaker: ^^^ not... that horrible... I likely need to do more on the api surface23:08
stevebakerTheJulia: I'll check it out, maybe after lunch23:08
TheJuliaoh, no rush, I just wanted to get that out of my head23:08
openstackgerritSteve Baker proposed openstack/ironic master: Consistently use utils functions for policy auth  https://review.opendev.org/c/openstack/ironic/+/76686923:11
openstackgerritSteve Baker proposed openstack/ironic master: Rewrite existing ACL tests with ddt, yaml  https://review.opendev.org/c/openstack/ironic/+/76743423:11
stevebakerTheJulia: here is a first crack^^. The one thing I like about this approach rather than gabbi is that the asserts can have a very clean representation, we can avoid assert patterns looking like jsonpath garble23:13
TheJuliaI'll try to take a look tomorrow23:13
TheJuliastevebaker: that is super read-able23:21
TheJuliais there any way to composite or would we be doomed for hashing out the entire matrix?23:21
TheJulia(not that would be that painful, just lots of heads down work)23:21
*** rloo has quit IRC23:25
*** paras333_ has joined #openstack-ironic23:40

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