*** tosky has quit IRC | 00:04 | |
stevebaker | TheJulia: hmm those jobs are still failing | 00:26 |
---|---|---|
openstackgerrit | Verification of a change to openstack/ironic failed: Prepare release 16.1 https://review.opendev.org/c/openstack/ironic/+/766938 | 00:56 |
*** rloo has quit IRC | 00:57 | |
*** rcernin_ has joined #openstack-ironic | 01:09 | |
*** rcernin has quit IRC | 01:10 | |
*** k_mouza has joined #openstack-ironic | 01:36 | |
*** JamesBenson has quit IRC | 01:39 | |
*** k_mouza has quit IRC | 01:41 | |
*** hoonetorg has quit IRC | 01:52 | |
*** hoonetorg has joined #openstack-ironic | 02:03 | |
openstackgerrit | Arun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface https://review.opendev.org/c/openstack/ironic-specs/+/748503 | 02:25 |
*** benj_- has joined #openstack-ironic | 02:35 | |
*** benj_ has quit IRC | 02:35 | |
*** benj_- is now known as benj_ | 02:35 | |
openstackgerrit | Arun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface https://review.opendev.org/c/openstack/ironic-specs/+/748503 | 02:43 |
*** rcernin_ has quit IRC | 02:46 | |
*** mgoddard has quit IRC | 02:58 | |
openstackgerrit | Steve Baker proposed openstack/ironic-lib stable/train: Use python3 for stable/train testing https://review.opendev.org/c/openstack/ironic-lib/+/767221 | 03:19 |
*** jamesden_ has quit IRC | 03:29 | |
*** gyee has quit IRC | 03:43 | |
*** ociuhandu has joined #openstack-ironic | 04:13 | |
*** ociuhandu has quit IRC | 04:17 | |
*** JamesBenson has joined #openstack-ironic | 04:24 | |
*** JayF has quit IRC | 04:27 | |
*** JayF has joined #openstack-ironic | 04:27 | |
*** JamesBenson has quit IRC | 04:28 | |
*** JamesBenson has joined #openstack-ironic | 04:29 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-python-agent/+/767088 | 04:35 |
*** k_mouza has joined #openstack-ironic | 04:48 | |
*** k_mouza has quit IRC | 04:53 | |
*** JamesBenson has quit IRC | 05:29 | |
*** JamesBenson has joined #openstack-ironic | 05:30 | |
*** JamesBenson has quit IRC | 05:30 | |
openstackgerrit | Vishal Manchanda proposed openstack/ironic-ui master: Remove Chrome browser from karma.conf.js https://review.opendev.org/c/openstack/ironic-ui/+/766880 | 05:31 |
*** JamesBenson has joined #openstack-ironic | 05:35 | |
*** tzumainn has quit IRC | 05:37 | |
*** zzzeek has quit IRC | 05:39 | |
*** JamesBenson has quit IRC | 05:39 | |
*** zzzeek has joined #openstack-ironic | 05:39 | |
openstackgerrit | ankit proposed openstack/ironic master: Fixes issue of redfish firmware update https://review.opendev.org/c/openstack/ironic/+/764914 | 05:46 |
*** zzzeek has quit IRC | 05:46 | |
openstackgerrit | ankit proposed openstack/sushy master: Fixes issue of redfish firmware update https://review.opendev.org/c/openstack/sushy/+/764915 | 05:47 |
*** zzzeek has joined #openstack-ironic | 05:48 | |
*** ricolin has joined #openstack-ironic | 05:59 | |
*** zzzeek has quit IRC | 06:02 | |
*** zzzeek has joined #openstack-ironic | 06:06 | |
openstackgerrit | ankit proposed openstack/sushy master: Fixes issue of redfish firmware update https://review.opendev.org/c/openstack/sushy/+/764915 | 06:44 |
*** abdysn has joined #openstack-ironic | 06:46 | |
*** Abdallahyas has joined #openstack-ironic | 07:04 | |
*** abdysn has quit IRC | 07:09 | |
*** Abdallahyas has quit IRC | 07:34 | |
*** Abdallahyas has joined #openstack-ironic | 07:34 | |
*** zzzeek has quit IRC | 07:40 | |
*** zzzeek has joined #openstack-ironic | 07:41 | |
*** ociuhandu has joined #openstack-ironic | 07:50 | |
*** zzzeek has quit IRC | 07:53 | |
*** ociuhandu has quit IRC | 07:54 | |
*** zzzeek has joined #openstack-ironic | 07:55 | |
*** Qianbiao has joined #openstack-ironic | 07:56 | |
arne_wiebalck | Good morning, ironic! | 08:03 |
*** Abdallahyas has quit IRC | 08:10 | |
*** abdysn has joined #openstack-ironic | 08:14 | |
*** rpittau|afk is now known as rpittau | 08:18 | |
rpittau | good morning ironic! o/ | 08:18 |
*** tosin has joined #openstack-ironic | 08:23 | |
*** ociuhandu has joined #openstack-ironic | 08:31 | |
*** zzzeek has quit IRC | 08:31 | |
*** tosky has joined #openstack-ironic | 08:33 | |
*** zzzeek has joined #openstack-ironic | 08:34 | |
*** zzzeek has quit IRC | 08:39 | |
*** zzzeek has joined #openstack-ironic | 08:41 | |
*** jawad_axd has joined #openstack-ironic | 08:43 | |
*** dougsz has joined #openstack-ironic | 08:44 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/train: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-python-agent/+/767284 | 08:46 |
*** mgoddard has joined #openstack-ironic | 08:46 | |
*** dougsz has quit IRC | 08:49 | |
*** ociuhandu has quit IRC | 08:51 | |
openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Adding basic support for processors schema https://review.opendev.org/c/openstack/sushy-tools/+/761909 | 08:58 |
*** zzzeek has quit IRC | 08:59 | |
*** zzzeek has joined #openstack-ironic | 09:00 | |
openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Adding basic support for processors schema https://review.opendev.org/c/openstack/sushy-tools/+/761909 | 09:01 |
*** zzzeek has quit IRC | 09:05 | |
*** mgoddard has quit IRC | 09:06 | |
*** zzzeek has joined #openstack-ironic | 09:06 | |
*** mgoddard has joined #openstack-ironic | 09:07 | |
*** zzzeek has quit IRC | 09:11 | |
*** zzzeek has joined #openstack-ironic | 09:14 | |
*** derekh has joined #openstack-ironic | 09:18 | |
*** zzzeek has quit IRC | 09:23 | |
*** abdysn has quit IRC | 09:25 | |
*** zzzeek has joined #openstack-ironic | 09:25 | |
*** jawad_axd has quit IRC | 09:30 | |
*** jawad_axd has joined #openstack-ironic | 09:30 | |
*** zzzeek has quit IRC | 09:41 | |
*** zzzeek has joined #openstack-ironic | 09:42 | |
*** tosky_ has joined #openstack-ironic | 09:47 | |
*** tosky is now known as Guest24372 | 09:49 | |
*** tosky_ is now known as tosky | 09:49 | |
*** Guest24372 has quit IRC | 09:50 | |
openstackgerrit | Merged openstack/ironic master: Prepare release 16.1 https://review.opendev.org/c/openstack/ironic/+/766938 | 09:54 |
*** zzzeek has quit IRC | 09:57 | |
*** zzzeek has joined #openstack-ironic | 09:58 | |
*** dtantsur|afk is now known as dtantsur | 09:59 | |
dtantsur | morning ironic | 10:00 |
openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-python-agent/+/767211 | 10:00 |
dtantsur | TheJulia: it probably means dnf has a direct dependency on some cockpit components | 10:00 |
rpittau | good morning dtantsur :) | 10:00 |
*** abdysn has joined #openstack-ironic | 10:11 | |
*** zzzeek has quit IRC | 10:13 | |
*** zzzeek has joined #openstack-ironic | 10:14 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironicclient master: Remove lower-constraints job https://review.opendev.org/c/openstack/python-ironicclient/+/766986 | 10:15 |
rpittau | lol in virtualbmc we had lower-constraints twice | 10:16 |
openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Remove lower-constraints job https://review.opendev.org/c/openstack/virtualbmc/+/767089 | 10:16 |
openstackgerrit | Dmitry 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/+/767158 | 10:17 |
dtantsur | rpittau: lol indeed | 10:18 |
dtantsur | could you check https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/762816 please? | 10:18 |
rpittau | sure :) | 10:18 |
dtantsur | also arne_wiebalck if you have a minute ^^^ | 10:18 |
rpittau | dtantsur: if you have time today please check the l-c removal patches | 10:19 |
dtantsur | I will | 10:19 |
openstackgerrit | Dmitry 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/+/767310 | 10:19 |
arne_wiebalck | dtantsur: checking ... | 10:19 |
dtantsur | there are also a few outstanding IPA patches that could be merged before the release | 10:21 |
rpittau | mozjs60 ? :D | 10:21 |
dtantsur | yeah :) | 10:21 |
rpittau | I'm not even going to ask :D | 10:22 |
dtantsur | all these came from an actual inspection of the ramdisk's contents, I'm not to blame | 10:22 |
rpittau | I know, just amazed :) | 10:23 |
arne_wiebalck | just checked what we remove downstream: mozjs60 | 10:24 |
arne_wiebalck | :) | 10:24 |
arne_wiebalck | we also remove postfix | 10:24 |
dtantsur | I 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_wiebalck | hmm, not sure how/why I though this is a sensible thing to remove | 10:25 |
arne_wiebalck | *thought | 10:25 |
rpittau | don't we want to implement an ipa-powered mail relay server ? | 10:26 |
dtantsur | it's a lower priority than a bitcoin miner | 10:27 |
rpittau | true | 10:28 |
arne_wiebalck | the pkg-map changes do not remove packages, correct? | 10:28 |
* arne_wiebalck thinks we rely on curl in the IPA | 10:28 | |
dtantsur | just imagine how many bitcoins you can mine on that 6k servers of arne_wiebalck! | 10:28 |
dtantsur | arne_wiebalck: no, pkg-map is just a mapping of generic names to distro-specific | 10:28 |
dtantsur | so "curl": "curl" makes no sense | 10:28 |
arne_wiebalck | right :) | 10:28 |
dtantsur | also IIRC curl is required by DIB itself | 10:29 |
*** lbragstad has quit IRC | 10:31 | |
*** lbragstad has joined #openstack-ironic | 10:31 | |
rpittau | this should pass now, just need +W -> https://review.opendev.org/c/openstack/ironic-python-agent/+/765294 | 10:34 |
dtantsur | checking | 10:36 |
dtantsur | rpittau: I'd hear from TheJulia if she's fine with the release note change | 10:37 |
rpittau | dtantsur: oh you're right, didn't see the comment | 10:41 |
*** ociuhandu has joined #openstack-ironic | 10:41 | |
*** JamesBenson has joined #openstack-ironic | 10:42 | |
*** ociuhandu_ has joined #openstack-ironic | 10:44 | |
*** zzzeek has quit IRC | 10:44 | |
*** ociuhandu_ has quit IRC | 10:46 | |
*** JamesBenson has quit IRC | 10:47 | |
*** zzzeek has joined #openstack-ironic | 10:47 | |
*** ociuhandu_ has joined #openstack-ironic | 10:47 | |
*** ociuhandu has quit IRC | 10:47 | |
*** ociuhandu_ has quit IRC | 10:52 | |
*** ociuhandu has joined #openstack-ironic | 10:52 | |
*** ociuhandu has quit IRC | 10:53 | |
*** tkajinam has quit IRC | 10:54 | |
*** zzzeek has quit IRC | 11:07 | |
*** k_mouza has joined #openstack-ironic | 11:08 | |
*** zzzeek has joined #openstack-ironic | 11:09 | |
*** mkowalski has joined #openstack-ironic | 11:18 | |
*** JamesBenson has joined #openstack-ironic | 11:20 | |
*** JamesBenson has quit IRC | 11:25 | |
*** anuradha1904 has joined #openstack-ironic | 11:40 | |
*** zzzeek has quit IRC | 11:41 | |
*** zzzeek has joined #openstack-ironic | 11:44 | |
*** ociuhandu has joined #openstack-ironic | 11:55 | |
*** zzzeek has quit IRC | 11:55 | |
*** zzzeek has joined #openstack-ironic | 11:57 | |
*** ociuhandu has quit IRC | 11:59 | |
*** JamesBenson has joined #openstack-ironic | 12:00 | |
*** JamesBenson has quit IRC | 12:03 | |
*** JamesBenson has joined #openstack-ironic | 12:04 | |
*** zzzeek has quit IRC | 12:11 | |
*** k_mouza has quit IRC | 12:13 | |
*** k_mouza has joined #openstack-ironic | 12:13 | |
*** zzzeek has joined #openstack-ironic | 12:13 | |
*** k_mouza has quit IRC | 12:17 | |
*** zzzeek has quit IRC | 12:18 | |
*** zzzeek has joined #openstack-ironic | 12:20 | |
openstackgerrit | Verification 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/+/762453 | 12:29 |
*** k_mouza has joined #openstack-ironic | 12:29 | |
*** zzzeek has quit IRC | 12:31 | |
*** zzzeek has joined #openstack-ironic | 12:32 | |
* rpittau needs coffee | 12:35 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent-builder failed: Clean up package installation https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/762816 | 12:38 |
*** ociuhandu has joined #openstack-ironic | 12:58 | |
*** paras333 has quit IRC | 13:08 | |
Qianbiao | dtantsur curl is required by almost all base image to fetch qcow image. | 13:12 |
*** ociuhandu has quit IRC | 13:12 | |
*** ociuhandu has joined #openstack-ironic | 13:13 | |
dtantsur | yep | 13:16 |
openstackgerrit | Vladyslav Drok proposed openstack/networking-baremetal master: Fix exception handling when querying ironic ports https://review.opendev.org/c/openstack/networking-baremetal/+/765851 | 13:16 |
*** ociuhandu has quit IRC | 13:18 | |
*** ociuhandu has joined #openstack-ironic | 13:21 | |
*** ociuhandu has quit IRC | 13:24 | |
*** ociuhandu has joined #openstack-ironic | 13:24 | |
*** k_mouza has quit IRC | 13:31 | |
*** zzzeek has quit IRC | 13:34 | |
*** zzzeek has joined #openstack-ironic | 13:36 | |
*** k_mouza has joined #openstack-ironic | 13:37 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironicclient master: Remove lower-constraints job https://review.opendev.org/c/openstack/python-ironicclient/+/766986 | 13:42 |
openstackgerrit | Merged openstack/ironic-python-agent master: Correctly decode error messages from ironic API https://review.opendev.org/c/openstack/ironic-python-agent/+/765838 | 13:43 |
*** zzzeek has quit IRC | 13:45 | |
*** Abdallahyas has joined #openstack-ironic | 13:46 | |
*** zzzeek has joined #openstack-ironic | 13:49 | |
*** abdysn has quit IRC | 13:49 | |
*** abdysn has joined #openstack-ironic | 13:49 | |
*** Abdallahyas has quit IRC | 13:51 | |
*** ociuhandu has quit IRC | 13:54 | |
*** lbragstad has quit IRC | 13:54 | |
*** lbragstad has joined #openstack-ironic | 13:57 | |
*** lbragstad has quit IRC | 13:57 | |
*** lbragstad has joined #openstack-ironic | 13:58 | |
openstackgerrit | Riccardo 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/+/767355 | 13:59 |
mgoddard | During 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 a | 14:01 |
mgoddard | shared lock. | 14:01 |
mgoddard | ^ is that a known issue? | 14:01 |
*** lbragstad_ has joined #openstack-ironic | 14:03 | |
*** ociuhandu has joined #openstack-ironic | 14:04 | |
TheJulia | good morning | 14:04 |
openstackgerrit | Riccardo 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/+/767355 | 14:04 |
*** ociuhandu has quit IRC | 14:05 | |
*** ociuhandu has joined #openstack-ironic | 14:05 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Generate TLS certificates with validity time in the past https://review.opendev.org/c/openstack/ironic-python-agent/+/767310 | 14:05 |
TheJulia | mgoddard: could we get more logging context for that node? | 14:05 |
*** lbragstad has quit IRC | 14:05 | |
mgoddard | TheJulia: 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.txt | 14:06 |
openstackgerrit | Verification 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/+/767169 | 14:07 |
*** rloo has joined #openstack-ironic | 14:08 | |
*** ociuhandu has quit IRC | 14:08 | |
*** ociuhandu has joined #openstack-ironic | 14:09 | |
TheJulia | hmm | 14:09 |
TheJulia | mgoddard: hmm.... | 14:09 |
mgoddard | TheJulia: I reproduced it locally | 14:09 |
dtantsur | morning TheJulia | 14:10 |
mgoddard | although it is based on unmerged changes to tenks & kayobe | 14:10 |
TheJulia | mgoddard: what does your db reservation field show | 14:10 |
dtantsur | mgoddard: sounds like a programming error | 14:10 |
dtantsur | the exclusive lock check happens in a decorator | 14:10 |
mgoddard | however, these changes did work a couple days ago | 14:10 |
TheJulia | looks like node update never released the lock in this, but we need to know the state of the node in the db first | 14:11 |
mgoddard | I will do a little digging. Just wondered if it was a known issue | 14:11 |
TheJulia | first I'm hearing of this specifically | 14:13 |
TheJulia | It seems very odd, to be honest | 14:13 |
* TheJulia needs coffee | 14:15 | |
*** abdysn has quit IRC | 14:22 | |
*** ociuhandu has quit IRC | 14:29 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/victoria: Correctly decode error messages from ironic API https://review.opendev.org/c/openstack/ironic-python-agent/+/767322 | 14:38 |
*** ociuhandu has joined #openstack-ironic | 14:40 | |
*** Qianbiao has quit IRC | 14:41 | |
*** tzumainn has joined #openstack-ironic | 14:46 | |
dtantsur | TheJulia: once you coffeeinate, your input is required on https://review.opendev.org/c/openstack/ironic-python-agent/+/765294 | 14:47 |
openstackgerrit | Richard G. Pioso proposed openstack/ironic master: WIP: Create 'redfish' driver Redfish Interop Profile https://review.opendev.org/c/openstack/ironic/+/754061 | 14:50 |
*** ociuhandu has quit IRC | 14:51 | |
*** belmoreira has joined #openstack-ironic | 15:01 | |
*** zzzeek has quit IRC | 15:06 | |
*** zzzeek has joined #openstack-ironic | 15:08 | |
*** ricolin_ has joined #openstack-ironic | 15:15 | |
TheJulia | rutro | 15:33 |
* TheJulia goes and grabs more coffee | 15:33 | |
*** ociuhandu has joined #openstack-ironic | 15:41 | |
derekh | Hi 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 |
derekh | I 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 boot | 15:44 |
TheJulia | dtantsur: commented | 15:45 |
openstackgerrit | Merged openstack/ironic master: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic/+/766995 | 15:46 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib stable/victoria: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-lib/+/767203 | 15:49 |
NobodyCam | Good Morning Ironic folks | 15:49 |
rpittau | TheJulia: I disabled dstat here ^ focal still has the issue | 15:49 |
rpittau | hey NobodyCam :) | 15:49 |
NobodyCam | hey hey rpittau good morning! | 15:50 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/victoria: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic/+/767323 | 15:51 |
*** ociuhandu has quit IRC | 15:53 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/victoria: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic/+/767323 | 15:53 |
*** k_mouza has quit IRC | 15:53 | |
dtantsur | morning NobodyCam! how's pre-xmas life? | 15:54 |
*** k_mouza has joined #openstack-ironic | 15:54 | |
rpittau | mmmm we might need to lock version of ipa-builder for ussuri and train | 15:54 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/ussuri: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic/+/767324 | 15:54 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/ussuri: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic/+/767324 | 15:55 |
dtantsur | rpittau: why? | 15:55 |
rpittau | dtantsur: 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 kernel | 15:55 |
dtantsur | ahhh, because of that | 15:56 |
TheJulia | good morning NobodyCam | 15:57 |
TheJulia | rpittau: ack | 15:57 |
rpittau | yeah, I hope to find a very low-pain way to do it | 15:57 |
TheJulia | derekh: what version of hte ilo firmware is that | 15:58 |
TheJulia | ? | 15:58 |
rpittau | ussuri 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 |
dtantsur | yeah, devstack hardcodes supported distros | 16:02 |
dtantsur | I don't see a huge problem with capping IPA-builder, we already do it for the tempest plugin | 16:02 |
rpittau | right | 16:02 |
rpittau | I'll copy-paste from there :D | 16:02 |
derekh | TheJulia: it had been one from January but we have now upgraded too "iLO Firmware Version 2.31 Oct 13 2020" and having the same problem | 16:02 |
TheJulia | derekh: 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 |
TheJulia | ugh | 16:05 |
TheJulia | UGH | 16:05 |
TheJulia | they changed the behavior | 16:05 |
TheJulia | they added the Uefi override targets | 16:05 |
derekh | TheJulia: 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 |
TheJulia | derekh: does it support the aliased name fields for targets in that version? | 16:08 |
derekh | But without knowing that PciRoot... address I don't see how ironic could do it | 16:08 |
*** ociuhandu has joined #openstack-ironic | 16:08 | |
derekh | TheJulia: don't know tbh, do you know how I could find out? If not I'll find out | 16:10 |
TheJulia | uhh... can you grab the entire boot field contents, I'd recognize. I think I started a patch somewhere... | 16:11 |
TheJulia | derekh: https://review.opendev.org/c/openstack/sushy/+/718275/1/sushy/resources/system/system.py | 16:12 |
dtantsur | so, the time has come. sigh. | 16:13 |
TheJulia | For a very very very strong beverage? | 16:14 |
TheJulia | or three? | 16:14 |
TheJulia | or six? | 16:14 |
derekh | TheJulia: this? https://goodsquishy.com/upload/551e5d43c62112acdb12 | 16:14 |
dtantsur | I have a birthday tomorrow, 3 bottles of craft beer and a bottle of japanese whiskey are waiting! | 16:14 |
TheJulia | dtantsur: yum! | 16:14 |
derekh | dtantsur: enjoy, your birthday? happy birthday | 16:15 |
dtantsur | thank you :) | 16:15 |
TheJulia | yes, happy birthday dtantsur ! | 16:15 |
rpittau | happy birthday dtantsur :) | 16:16 |
TheJulia | derekh: 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 list | 16:16 |
dtantsur | thanks folks! | 16:17 |
dtantsur | "UsbClass(0xFFFF,0xFFFF,0xFF,0xFF,0xFF)" - it? | 16:17 |
bfournie | dtantsur: Happy Birthday \o/ | 16:17 |
rloo | Happy birthday dtantsur! No doubt that you will be enjoying yourself tomorrow! Whee. | 16:18 |
dtantsur | \o/ | 16:18 |
TheJulia | dtantsur: your taking tomorrow off right? | 16:18 |
dtantsur | and it will be a round number - 32 :D | 16:18 |
dtantsur | TheJulia: yep, and Friday too | 16:18 |
TheJulia | awesome! | 16:18 |
TheJulia | I'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 shipped | 16:19 |
dtantsur | TheJulia: why hurry with that? I think everyone will understand | 16:19 |
dtantsur | I was supposed to ship mine more than a year ago | 16:20 |
TheJulia | dtantsur: statistically thursdays are the most idle time at the local shipping place. | 16:20 |
TheJulia | lol | 16:20 |
TheJulia | wow | 16:20 |
TheJulia | okay | 16:20 |
dtantsur | I don't think anyone in their right mind will blame you for avoiding a post office amid pandemic | 16:20 |
* TheJulia wonders what usbclass is.... | 16:21 | |
dtantsur | I confused Usb and Cd, sorry (and I'm still sober!) | 16:21 |
dtantsur | but it looks like some sort of a wildcard for USB devices | 16:21 |
TheJulia | Well, luckily it is a pre-paid label. I just pack it up, tape it, hand it over and ask for a receipt | 16:21 |
TheJulia | well, while we're requesting cd for general override, i bet in uefi mode it is being viewed as a usb device | 16:22 |
*** jawad_axd has quit IRC | 16:23 | |
dtantsur | derekh: will it work if you use UsbClass(0xFFFF,0xFFFF,0xFF,0xFF,0xFF)"? | 16:24 |
derekh | TheJulia: iirc the USB device is always there but I'll double check once I can get back on the system | 16:24 |
derekh | dtantsur: I haven't tried it but can do that next | 16:24 |
* TheJulia wonders if PciRoot(0x0)/Pci(0x1C,0x4)/Pci(0x0,0x4)/USB(0x1,0x0) is a special device. | 16:24 | |
TheJulia | it is the same on a dl380g10 | 16:24 |
dtantsur | and there is also "PciRoot(0x0)/Pci(0x1C,0x4)/Pci(0x0,0x4)/USB(0x1,0x0)/CDROM(0x1)/\\EFI\\redhat\\shimx64.efi" | 16:25 |
dtantsur | worst comes to worst, we have to hardcode these.. but I'd rather not. | 16:25 |
TheJulia | i suspect it is auto-adding that | 16:25 |
dtantsur | yeah, the value that works for derekh is not even on the list | 16:26 |
dtantsur | I guess you can trim any part from it, and it's still allowed? | 16:26 |
TheJulia | oh... | 16:26 |
TheJulia | the virtual cd is a virtual usb device | 16:27 |
TheJulia | omg | 16:27 |
dtantsur | on the other hand, we can finally choose which NIC to boot from! | 16:28 |
rpittau | maybe it's on the same bus | 16:28 |
dtantsur | I wonder if we can get the PCI path from the virtual media device properties | 16:29 |
dtantsur | derekh: could you check ^^ as well? | 16:30 |
TheJulia | It is looking like that is consistent across proliant models | 16:30 |
derekh | dtantsur: TheJulia getting access back onto the system, I'll grab the full json both with and without the vmedia attched and get back to you | 16:30 |
dtantsur | derekh: virtual media is a separate resource under Managers | 16:30 |
*** ricolin_ has quit IRC | 16:31 | |
TheJulia | dtantsur: bad news on this field, it is up to the vendor what they want to put into it :\ | 16:32 |
dtantsur | it = UefiTargetBootSourceOverride? | 16:32 |
TheJulia | yeah | 16:33 |
dtantsur | omg | 16:33 |
dtantsur | the redfish standard keeps being automation-hostile | 16:33 |
TheJulia | On one vendor it is "0000", another vendor it is the hex string for the uefi order entry | 16:33 |
TheJulia | dtantsur: this was all proposed a while back | 16:33 |
dtantsur | okay, "has always been" :) | 16:33 |
TheJulia | dtantsur: and there are new fields to make this easy, just they don't support them yet :( | 16:33 |
dtantsur | okay, "has always been, but may stop soon if we're lucky" :)) | 16:34 |
TheJulia | so | 16:35 |
TheJulia | the UsbClass entry is "any usb device in the system" per HPE docs | 16:35 |
dtantsur | may do the trick if nothing else helps.. | 16:36 |
TheJulia | still digging, since the PDF only printed as images *grumbles* | 16:36 |
TheJulia | but it looks like it may be reliable for the bmc | 16:36 |
TheJulia | it being the special pciroot path | 16:36 |
dtantsur | btw virtual media supports username/password for images, we should support them too | 16:37 |
TheJulia | ++ | 16:37 |
dtantsur | but now.. we're finally in the field of hardcoding vendor differences | 16:38 |
dtantsur | I wonder if should just merge sushy-oem-idrac in sushy and stop pretending we can maintain any distance... | 16:38 |
TheJulia | dtantsur: wondering the same thing | 16:40 |
TheJulia | mental note while I skim docs: vmedia in ilos is == CD.Virtual.2.1 for iso images or Usb device HD.Virtual.1.1 | 16:41 |
dtantsur | oh, I need to test attaching configdrives | 16:41 |
TheJulia | err, conflicting docs yay | 16:42 |
TheJulia | 2.1 is apparently the VID too | 16:42 |
dtantsur | VID? | 16:42 |
TheJulia | virutal install device | 16:42 |
* TheJulia loads up the official docs and digs | 16:42 | |
TheJulia | for the api surface | 16:42 |
*** juanoterocas has joined #openstack-ironic | 16:43 | |
juanoterocas | is there an easy way to setup Ironic ?, tutorial | 16:43 |
dtantsur | juanoterocas: try https://docs.openstack.org/bifrost/latest/ maybe? | 16:44 |
dtantsur | or do you need it as part of openstack? | 16:44 |
juanoterocas | yes, integrating with openstack should be easy | 16:45 |
dtantsur | juanoterocas: do you use any tool to install openstack itself? maybe it supports ironic already? | 16:45 |
juanoterocas | I-m using microstack to test locally, after that I'm planning to use juju charm | 16:46 |
dtantsur | juanoterocas: I think you should check if there is a charm for ironic then | 16:46 |
TheJulia | I'm thinkign the pci references are bus address locations, not device IDs | 16:47 |
dtantsur | juanoterocas: there seem to be https://opendev.org/openstack/charm-ironic | 16:47 |
*** ociuhandu_ has joined #openstack-ironic | 16:49 | |
juanoterocas | It is ok later when using full openstack charm but at this very moment I'm doing some local test using microstack. | 16:49 |
juanoterocas | and I want to test integration with Virtual machines and baremetal (physical machine) | 16:51 |
juanoterocas | using neutron network. | 16:51 |
*** ociuhandu has quit IRC | 16:52 | |
openstackgerrit | Dmitry 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/+/767376 | 16:52 |
dtantsur | juanoterocas: 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 |
dtantsur | maybe you should start with bifrost, then modify it for Neutron/Glance/Nova integration | 16:53 |
*** ociuhandu has joined #openstack-ironic | 16:53 | |
juanoterocas | some files are in differet places since is a snap installation but I managed to found where I need to do some things | 16:54 |
juanoterocas | I'll check bifrost, I haven't work with ansible before, but i'll give a try | 16:56 |
dtantsur | juanoterocas: 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 |
dtantsur | It's not going to be trivial, unfortunately. https://docs.openstack.org/ironic/latest/ is your help | 16:57 |
*** ociuhandu_ has quit IRC | 16:57 | |
juanoterocas | thanks dtantsur. Yes in fact I have followed that documentation and I have some things working now | 16:59 |
openstackgerrit | Verification 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/+/762453 | 17:00 |
*** ociuhandu_ has joined #openstack-ironic | 17:02 | |
*** tosin has quit IRC | 17:02 | |
TheJulia | PciRoot(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 ilo | 17:04 |
TheJulia | folks | 17:04 |
*** JayF has quit IRC | 17:04 | |
TheJulia | then again, I'd eexpect a dl360 and 380 to be similar. An edgeline is likely different | 17:05 |
*** ociuhandu has quit IRC | 17:05 | |
TheJulia | Anyone got an edgeline handy? :) | 17:05 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/ussuri: Cap version of ipa-builder to 2.2.0 https://review.opendev.org/c/openstack/ironic/+/767379 | 17:06 |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Clean up package installation https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/762816 | 17:06 |
*** ociuhandu_ has quit IRC | 17:07 | |
*** gyee has joined #openstack-ironic | 17:07 | |
rpittau | if that works I guess we need to combine it with the l-c removal | 17:07 |
*** juanoterocas has quit IRC | 17:08 | |
*** JayF has joined #openstack-ironic | 17:08 | |
rpittau | good rest of the day everyone! o/ | 17:11 |
*** rpittau is now known as rpittau|afk | 17:11 | |
TheJulia | o/ | 17:11 |
* TheJulia makes her desktop cry by trying to load multiple super dense dynamic documentation pages | 17:11 | |
TheJulia | I can literally hear my fans at full power | 17:11 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Handle situation when a configdrive is already mounted https://review.opendev.org/c/openstack/ironic-python-agent/+/767381 | 17:18 |
dtantsur | a fix for my recent patch ^^^ | 17:18 |
*** ociuhandu has joined #openstack-ironic | 17:18 | |
dtantsur | maybe we shouldn't fail at all if we cannot copy the configuration.. | 17:19 |
TheJulia | shoot, didn't shiv say they were in shutdown starting on the 16th or something? | 17:21 |
dtantsur | I don't remember | 17:21 |
TheJulia | derekh: maybe send shiv an email and try to obtain insight. That being said I'm suspecting you won't hear back until next year | 17:22 |
*** ociuhandu has quit IRC | 17:23 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Enable ramdisk deploy by default, disable iscsi https://review.opendev.org/c/openstack/bifrost/+/767384 | 17:26 |
*** belmoreira has quit IRC | 17:34 | |
*** k_mouza has quit IRC | 17:35 | |
mgoddard | TheJulia, dtantsur: found the issue. ipmitool.get_power_state now calls ipmitool.detect_vendor, which requires an exclusive lock | 17:45 |
TheJulia | ugh | 17:46 |
mgoddard | TheJulia: https://review.opendev.org/c/openstack/ironic/+/757198 | 17:46 |
mgoddard | (but I guess you knew that :) ) | 17:46 |
TheJulia | I guess we could only upgrade the lock if we have something to save?! | 17:46 |
mgoddard | detect_vendor itself does not seem to save | 17:47 |
mgoddard | I wonder why ironic CI did not catch this. Perhaps vendor is set on nodes | 17:48 |
mgoddard | TheJulia: anyway, home duties call. Let me know if you do anything with this, otherwise I'll pick it up tomorrow | 17:49 |
TheJulia | mgoddard: good question, working on a fix | 17:51 |
derekh | It'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 then | 17:51 |
TheJulia | derekh: ok | 17:52 |
TheJulia | derekh: I'd look on an edgeline machine as well | 17:52 |
mgoddard | TheJulia: verified that removing the decorator allows the power sync to work | 17:52 |
TheJulia | mgoddard: ack | 17:52 |
derekh | TheJulia: will see if I can get a look on one | 17:53 |
* TheJulia ponders for a moment if there should be additional testing | 17:53 | |
openstackgerrit | Leo McGann proposed openstack/ironic-specs master: Add security interface spec https://review.opendev.org/c/openstack/ironic-specs/+/576718 | 17:55 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Remove detect_vendor decorator https://review.opendev.org/c/openstack/ironic/+/767390 | 17:55 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Documenting using ramdisks with the ramdisk deploy interface https://review.opendev.org/c/openstack/ironic/+/767391 | 17:57 |
dtantsur | finally ^^ | 17:57 |
TheJulia | dtantsur: and that element has merged? | 17:58 |
dtantsur | not yet, see depends-on | 17:59 |
dtantsur | Got heartbeat message from node 4e41df61-84b1-5856-bfb6-6b5f2cd3dd11, but the driver redfish does not support heartbeating | 17:59 |
dtantsur | WAIT WUT | 17:59 |
dtantsur | I think cleaning does not like the ramdisk deploy.. | 17:59 |
TheJulia | didn't it get decoupled from the agent base? | 18:00 |
dtantsur | well, not in the right way, apparently.. lemme try something | 18:00 |
* TheJulia has removed stickers from old lapto | 18:02 | |
* TheJulia is sad now and needs all the stickers | 18:02 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Include HeartbeatMixin in the ramdisk deploy https://review.opendev.org/c/openstack/ironic/+/767393 | 18:07 |
dtantsur | a simple fix ^^ | 18:07 |
TheJulia | Yeah, that should do it | 18:08 |
TheJulia | I went ahead and +2+A'ed it | 18:09 |
TheJulia | since it is trivial | 18:09 |
TheJulia | https://review.opendev.org/c/openstack/ironic/+/767390 <-- also really trivial and I think mgoddard would appreciate it :) | 18:09 |
TheJulia | didn't add a release note since on backporting that will likely need to be squashed in | 18:10 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Document using ramdisks with the ramdisk deploy interface https://review.opendev.org/c/openstack/ironic/+/767391 | 18:13 |
dtantsur | okay, this actually works (forgot about DHCP) | 18:13 |
* dtantsur wonders if debian-minimal would actually work for DHCP-less | 18:18 | |
openstackgerrit | Leo McGann proposed openstack/ironic-specs master: Add security interface spec https://review.opendev.org/c/openstack/ironic-specs/+/576718 | 18:18 |
TheJulia | maybe | 18:23 |
TheJulia | if there is no networkmanager | 18:23 |
dtantsur | I built a bare ramdisk, and nothing DHCP'ed there | 18:23 |
TheJulia | hmmm | 18:23 |
*** derekh has quit IRC | 18:26 | |
dtantsur | IT WORKS! | 18:27 |
dtantsur | omg, I finally found something that works with DHCP-less! | 18:27 |
dtantsur | 28 minutes past the times I should have left for PTO :D | 18:28 |
dtantsur | now the problem is that we always depend on dhcp-all-interfaces | 18:28 |
dtantsur | (I manually removed it) | 18:28 |
dtantsur | mm, bifrost is broken? | 18:33 |
dtantsur | ah! | 18:34 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Enable ramdisk deploy by default, disable iscsi https://review.opendev.org/c/openstack/bifrost/+/767384 | 18:35 |
openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Correctly decode error messages from ironic API https://review.opendev.org/c/openstack/ironic-python-agent/+/767322 | 18:38 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Document that DHCP-less deploy does work with debian-minimal https://review.opendev.org/c/openstack/ironic/+/767399 | 18:45 |
dtantsur | TheJulia: some good news here ^^^ | 18:45 |
*** MentalSiege has joined #openstack-ironic | 18:49 | |
openstackgerrit | Dmitry 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/+/767158 | 18:49 |
dtantsur | things suspiciously work today, I better leave before they stop :) | 18:50 |
dtantsur | have a nice rest of the week, folks, see you on Monday or in the near year | 18:50 |
*** dtantsur is now known as dtantsur|afk | 18:50 | |
TheJulia | dtantsur|afk: \o/ | 19:04 |
TheJulia | have a wonderful pto! | 19:04 |
*** stevebaker has quit IRC | 19:35 | |
*** zzzeek has quit IRC | 19:36 | |
*** stevebaker has joined #openstack-ironic | 19:36 | |
*** brtknr has quit IRC | 19:37 | |
*** zzzeek has joined #openstack-ironic | 19:38 | |
dking | I"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-ironic | 19:40 | |
dking | I 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 | |
stevebaker | morning | 19:45 |
TheJulia | Good morning | 19:47 |
stevebaker | TheJulia: 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/+/767221 | 19:51 |
*** lbragstad_ is now known as lbragstad | 19:54 | |
TheJulia | dking: 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 ramdisk | 19:55 |
dking | So, in the IPA service file? | 19:57 |
*** zzzeek has quit IRC | 20:03 | |
TheJulia | i think so yeah | 20:03 |
TheJulia | stevebaker: fyi, I'm ripping out ye olde extra['vif_port_id'] support for the rbac stuffs | 20:04 |
TheJulia | I didn't realize how much testing had to be changed though :\ | 20:04 |
TheJulia | stevebaker: ugh, looks like we have to change the distro out | 20:05 |
TheJulia | or define it :\ | 20:05 |
TheJulia | "kernel too old" :( | 20:05 |
stevebaker | TheJulia: FATAL: kernel too old | 20:05 |
stevebaker | ja | 20:05 |
TheJulia | I think that means we need to be on focal | 20:05 |
*** zzzeek has joined #openstack-ironic | 20:07 | |
*** juanoterocas has joined #openstack-ironic | 20:08 | |
TheJulia | stevebaker: 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 ci | 20:08 |
TheJulia | and not work that is likely to merge "soon" | 20:08 |
TheJulia | sadly | 20:08 |
stevebaker | TheJulia: already did that :) | 20:08 |
TheJulia | ack | 20:09 |
stevebaker | TheJulia: 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 oven | 20:09 | |
TheJulia | we could try that | 20:09 |
stevebaker | wait, ironic-lib-base inherits ironic-base, I wonder what that is on stable/train | 20:10 |
TheJulia | likely not focal | 20:11 |
stevebaker | bionic | 20:12 |
TheJulia | :( | 20:12 |
TheJulia | maybe change ironic on that branch? | 20:12 |
TheJulia | err, ironic-base | 20:12 |
TheJulia | since the in-branch builds likely don't work either due to this | 20:13 |
openstackgerrit | Steve Baker proposed openstack/ironic-lib stable/train: Use python3 for stable/train testing https://review.opendev.org/c/openstack/ironic-lib/+/767221 | 20:14 |
*** zzzeek has quit IRC | 20:14 | |
stevebaker | I'll try this first | 20:14 |
stevebaker | victoria is the first ironic-base job to use focal | 20:15 |
*** zzzeek has joined #openstack-ironic | 20:17 | |
openstackgerrit | Steve Baker proposed openstack/ironic stable/train: Switch from bionic to focal nodeset https://review.opendev.org/c/openstack/ironic/+/767416 | 20:21 |
*** zzzeek has quit IRC | 20:25 | |
*** zzzeek has joined #openstack-ironic | 20:27 | |
TheJulia | I wonder if something else changed :\ | 20:29 |
*** ociuhandu has joined #openstack-ironic | 20:35 | |
*** ociuhandu has quit IRC | 20:40 | |
stevebaker | blarg https://zuul.opendev.org/t/openstack/build/3f8858185fc348d8a3ad17e2da084225/log/job-output.txt#1973 | 20:51 |
TheJulia | ugh | 20:52 |
*** SpamapS has quit IRC | 20:53 | |
TheJulia | i fix things and more tests break \o/ | 21:27 |
stevebaker | maybe we should just stop for the year | 21:31 |
TheJulia | I'm liking that idea | 21:31 |
TheJulia | back down to 24 tests | 21:33 |
openstackgerrit | Steve Baker proposed openstack/ironic-lib stable/train: Use python3 for stable/train testing https://review.opendev.org/c/openstack/ironic-lib/+/767221 | 21:33 |
stevebaker | TheJulia: 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 easier | 21:35 |
TheJulia | okay | 21:35 |
TheJulia | I think that is reasonable | 21:36 |
TheJulia | 12 more tests fixed \o/ | 21:40 |
openstackgerrit | Arun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface https://review.opendev.org/c/openstack/ironic-specs/+/748503 | 21:47 |
TheJulia | one... test. | 22:01 |
*** sdanni has quit IRC | 22:03 | |
*** rcernin has joined #openstack-ironic | 22:17 | |
*** derekh has joined #openstack-ironic | 22:58 | |
*** tkajinam has joined #openstack-ironic | 22:59 | |
*** juanoterocas has quit IRC | 23:05 | |
NobodyCam | morning TheJulia and dtantsur|afk | 23:05 |
NobodyCam | ;p been one of those days | 23:05 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: remove vif_port_id https://review.opendev.org/c/openstack/ironic/+/767432 | 23:07 |
TheJulia | stevebaker: ^^^ not... that horrible... I likely need to do more on the api surface | 23:08 |
stevebaker | TheJulia: I'll check it out, maybe after lunch | 23:08 |
TheJulia | oh, no rush, I just wanted to get that out of my head | 23:08 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Consistently use utils functions for policy auth https://review.opendev.org/c/openstack/ironic/+/766869 | 23:11 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Rewrite existing ACL tests with ddt, yaml https://review.opendev.org/c/openstack/ironic/+/767434 | 23:11 |
stevebaker | TheJulia: 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 garble | 23:13 |
TheJulia | I'll try to take a look tomorrow | 23:13 |
TheJulia | stevebaker: that is super read-able | 23:21 |
TheJulia | is 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 IRC | 23:25 | |
*** paras333_ has joined #openstack-ironic | 23:40 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!