Wednesday, 2023-11-15

opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Redfish UefiHttp boot support  https://review.opendev.org/c/openstack/ironic/+/90096400:16
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Add HTTP versions of network boot interfaces  https://review.opendev.org/c/openstack/ironic/+/90096500:16
TheJuliaheh, yeah... nope :)00:17
TheJuliaso the latter one goes a bit further than the spec, and the first one kind of went a little more specific off from the spec when I realized "this would just be better to let sushy handle the details00:17
opendevreviewBoushra Sondos Bettir proposed openstack/ironic master: [WIP] Currently working on adding support for OVN type switches which Neutron already supports.  https://review.opendev.org/c/openstack/ironic/+/90056801:09
opendevreviewVerification of a change to openstack/ironic master failed: [api-ref] Complete port name and shard documentation  https://review.opendev.org/c/openstack/ironic/+/89909701:12
opendevreviewMichal Nasiadka proposed openstack/ironic-python-agent master: docs: improve rootpwd password generation command  https://review.opendev.org/c/openstack/ironic-python-agent/+/89938007:52
rpittaugood morning ironic! o/08:12
opendevreviewMichal Nasiadka proposed openstack/bifrost master: inspector: add console=ttyS0 by default only when testing  https://review.opendev.org/c/openstack/bifrost/+/90102811:52
dtantsurJayF: "just run it locally" is what I used to do fwiw13:27
opendevreviewGraeme Moss proposed openstack/networking-generic-switch stable/zed: Fix regression plugging 802.3ad port group  https://review.opendev.org/c/openstack/networking-generic-switch/+/90098913:27
TheJuliagood morning14:11
opendevreviewGraeme Moss proposed openstack/networking-generic-switch stable/zed: Fix regression plugging 802.3ad port group  https://review.opendev.org/c/openstack/networking-generic-switch/+/90098914:11
opendevreviewRiccardo Pittau proposed openstack/ironic master: Generic API for attaching/detaching virtual media  https://review.opendev.org/c/openstack/ironic/+/89491815:12
opendevreviewRiccardo Pittau proposed openstack/ironic master: Generic API for attaching/detaching virtual media  https://review.opendev.org/c/openstack/ironic/+/89491815:16
opendevreviewRiccardo Pittau proposed openstack/ironic master: Generic API for attaching/detaching virtual media  https://review.opendev.org/c/openstack/ironic/+/89491815:16
opendevreviewRiccardo Pittau proposed openstack/ironic master: Generic API for attaching/detaching virtual media  https://review.opendev.org/c/openstack/ironic/+/89491815:53
opendevreviewRiccardo Pittau proposed openstack/ironic master: Generic API for attaching/detaching virtual media  https://review.opendev.org/c/openstack/ironic/+/89491815:54
TheJuliaDo we know what is going on with the metal3 integration job, looks like the playbook run is failing on a missing package15:57
TheJulialike... trying to pull "https://download.opensuse.org/repositories/devel:/kubic:/libcontainers:/stable/xUbuntu_22.04/amd64/libcap2-bin_2.48-1_amd64.deb"15:57
rpittauJayF, TheJulia, dtantsur, the patch for the bugfix eol is passing CI now, we need to skip some validations though as all the release tools are "stable-branches-centric" https://review.opendev.org/c/openstack/releases/+/90081015:58
rpittaudepending on the support from the release team we could think about adding the missing validations, but it doesn't look trivial15:58
JayFack; thank you15:59
dtantsurTheJulia: interesting, not such failures are knows in metal316:17
dtantsurrpittau: I think the infra tooling will expect - instead of /16:18
rpittaudtantsur: uhmmm where?16:27
dtantsurrpittau: you're creating tag bugfix/X.Y-eol16:27
dtantsurI think it's meant to be bugfix-X.Y-eol16:27
rpittauoh, so bugfix-X.Y-eol ?16:27
rpittauok, I'll double-check that16:28
rpittauthanks16:28
JayFdtantsur: are you coming to doc meeting?16:32
dtantsursorry, yeah16:32
opendevreviewGraeme Moss proposed openstack/networking-generic-switch stable/zed: Add support for Dell OS10  https://review.opendev.org/c/openstack/networking-generic-switch/+/90106117:06
JayFHey; we just had a small chat about docs, as a followup to PTG, notes here: https://etherpad.opendev.org/p/ironic-docs-2023-11-15 -- basically one of my downstream peers at GR-OSS is going to work to find resources to help improve our docs. Only thing we need to do now is if we know of examples of "good" documentation in OSS (closer to our problem set the better -- so maybe17:11
JayFother openstack or bm provisioning projects?), we should add them to the proper place on that etherpad (line 28-ish)17:11
TheJuliaFWIW, it is a perception problem. We've had people also say in the past our docs are extremely verbose and detailed17:17
rpittauJayF: thanks for organizing that :)17:17
TheJulia... possibly to the underlying problem, how to find what your looking for, maybe?17:18
TheJuliadunno17:18
* TheJulia goes back to unit tests17:18
rpittauand time to go for me, see ya tomorrow o/17:19
JayFTheJulia: I think it's a levelling problem. We have a lot of docs at the super duper duper basic level, a lot of docs at the super duper detailed level. The middle is a pretty sizable gap. Plus a pretty bad discovery issue.17:21
TheJuliayup17:21
JayFTheJulia: Given docs is almost a form of marketing; perception issues with docs /are/ docs issues :)17:21
TheJuliaagree completely17:22
JayFThis is the stuff Caterina is good at, I'm hoping she'll help in the right kinda way17:22
JayFgetting someone outside Ironic to look is probably the thing we've needed for a while17:22
opendevreviewJay Faulkner proposed openstack/ironic master: DNM Testing nova change  https://review.opendev.org/c/openstack/ironic/+/90095817:28
*** JasonF is now known as JayF18:47
opendevreviewJulia Kreger proposed openstack/ironic master: Redfish UefiHttp boot support  https://review.opendev.org/c/openstack/ironic/+/90096419:24
opendevreviewJay Faulkner proposed openstack/ironic master: DNM Testing nova change  https://review.opendev.org/c/openstack/ironic/+/90095819:27
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Redfish UefiHttp boot support  https://review.opendev.org/c/openstack/ironic/+/90096419:29
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: Add redfish https CI job  https://review.opendev.org/c/openstack/ironic/+/90109019:29
TheJuliacrazy question, what if we had a tempest test which tried to deploy on *every* boot_interface configured?19:31
JayFwe'd have 'test coverage', I believe, as the kids say :D 19:31
* JayF not sure what you're asking/maybe missing context19:31
TheJuliaheh19:32
TheJuliaso what I'm sort of thinking with https booting... is today say we have "pxe (grub... please use grub), ipxe, redfish-virtual-media, and now adding  redfish-https, and https-ipxe, and https-pxe to that list, *feels* like we should just rotate through them instead of having separate jobs19:34
JayFrotate meaning19:34
JayForder by rand()19:34
JayFor meaning, we do N boots for N interfaces in N jobs19:34
TheJuliawe would likely need to do some test magic maybe19:34
TheJuliabut kind of19:34
JayFor we do N boots in N interfaces in *1* job19:34
TheJulian test classes or n tests inside of a test class19:34
JayFI wonder if like19:34
JayFwe can segment some of this19:34
JayFe.g.19:34
JayFboot interface needs to just boot a thing19:35
JayFthat screams "use ramdisk deploy", right?19:35
JayFif we use ramdisk deploy + standalone, how long can a boot interface test take?19:35
TheJuliapotentially19:35
JayFthen we could have a "test all the boot interfaces job" that tries hard to NOT touch anything else19:35
TheJuliaI'm not sure the PXE interface inherently does ramdisk deploy out of the box, bu it would likely be a quick fix if it doesn't19:35
TheJuliaso the one challenge with only verifying it becomes pingable, is the machine is pingable via EFI firmware19:36
JayFwhich while maybe not working from a "stuff as much coverage into as few jobs as possible" standpoint, would help from a "actually able to tell what breaks when a job fails" standpoint19:36
TheJuliaor iPXE19:36
TheJuliaso we don't *really* know without getting to a steady state19:36
JayFramdisk doesn't preclude ssh, does it?19:36
TheJuliatoday, we don't try to wait that long because we have no key injection means19:37
TheJuliakeep in mind, it has to be completely ready to rock and roll and we're basically hands-off19:37
JayFIn our current devstack model; yes. We have the technology and knowledge to e.g. toss an image/kernel/initrd into an ISO and use it for a ramdisk-based deploy, (with something injected) yes?19:38
TheJuliavmedia we can sort of ping and be sure, we know the bootloader-> OS is causing networking to fire up19:38
TheJuliawe would have to inject some means to know what it is19:38
JayFwell I'm thinking, if we could even get something like a port open inside the ramdisk19:38
JayFjust literally nc -l 9999 or something nuts like that19:39
TheJuliaor verify it, or ssh in, or just fire up cirros and ensure cubsw1n;)19:39
JayFheck, even *look for the ssh port being open*19:39
JayFthere are lots of canaries that we've booted a thing, when we take agent as a possibility out of it19:39
JayFassuming tempest has more than just a menu of "ping or ssh"19:39
JayFbecause you just need to answer "is this running a real OS?"19:39
JayFespecially if it's an isolated job where it's impossible that there would've been another OS on that disk, ever19:40
JayFheck, even provision diskless VMs even19:40
JayF(if we wanna be certain certain)19:40
TheJulialikely best not to over-think it too far beyond the base idea19:53
JayFwe don't have to do any of that stuff, there are just lots of intersting things we could do to go about it19:56
TheJuliaindeed20:02
opendevreviewJay Faulkner proposed openstack/ironic master: DNM Testing nova change  https://review.opendev.org/c/openstack/ironic/+/90095821:27
opendevreviewVerification of a change to openstack/ironic-inspector stable/xena failed: Handle bracketed IPv6 redfish_address  https://review.opendev.org/c/openstack/ironic-inspector/+/89632622:24
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Redfish UefiHttp boot support  https://review.opendev.org/c/openstack/ironic/+/90096423:15
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: Add redfish https CI job  https://review.opendev.org/c/openstack/ironic/+/90109023:15

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