Tuesday, 2024-11-19

opendevreviewVerification of a change to openstack/ironic stable/2024.2 failed: Gracefully handle bad request exception  https://review.opendev.org/c/openstack/ironic/+/93303600:10
opendevreviewMerged openstack/ironic stable/2024.2: Gracefully handle bad request exception  https://review.opendev.org/c/openstack/ironic/+/93303603:10
rpittaugood morning ironic! o/07:57
opendevreviewPierre Riteau proposed openstack/tenks stable/2.0: CI: Use 2024.1 requirements on stable/2.0 branch  https://review.opendev.org/c/openstack/tenks/+/93556708:29
opendevreviewPierre Riteau proposed openstack/tenks stable/2.0: CI: Use 2024.1 requirements for Python 3.8 job  https://review.opendev.org/c/openstack/tenks/+/93556708:50
opendevreviewVerification of a change to openstack/tenks stable/2.0 failed: CI: Use 2024.1 requirements for Python 3.8 job  https://review.opendev.org/c/openstack/tenks/+/93556709:19
opendevreviewPierre Riteau proposed openstack/tenks stable/2.0: CI: Use 2024.1 requirements for Python 3.8 job  https://review.opendev.org/c/openstack/tenks/+/93556709:27
opendevreviewMerged openstack/tenks stable/2.0: CI: Use 2024.1 requirements for Python 3.8 job  https://review.opendev.org/c/openstack/tenks/+/93556710:11
opendevreviewOpenStack Release Bot proposed openstack/tenks stable/2.0: Update .gitreview for stable/2.0  https://review.opendev.org/c/openstack/tenks/+/92715910:16
opendevreviewPierre Riteau proposed openstack/tenks stable/2.0: Remove deprecated use of include  https://review.opendev.org/c/openstack/tenks/+/93562410:40
opendevreviewRiccardo Pittau proposed openstack/ironic master: Use OVN and OVS from OS packages in CI  https://review.opendev.org/c/openstack/ironic/+/93562811:06
opendevreviewMerged openstack/ironic stable/2024.2: Actually ignore [inspector]power_off with fast track  https://review.opendev.org/c/openstack/ironic/+/93442411:31
opendevreviewMerged openstack/ironic stable/2024.2: Fix policy checks added with runbooks  https://review.opendev.org/c/openstack/ironic/+/93553011:55
opendevreviewcid proposed openstack/ironic-specs master: Add a Kea DHCP backend  https://review.opendev.org/c/openstack/ironic-specs/+/93102512:20
opendevreviewcid proposed openstack/ironic master: Fix double JSON encoding of error message  https://review.opendev.org/c/openstack/ironic/+/93179512:21
opendevreviewcid proposed openstack/ironic master: Use microversion handler  https://review.opendev.org/c/openstack/ironic/+/93564412:22
opendevreviewcid proposed openstack/ironic master: [WIP] Agentless Kea DHCP backend.  https://review.opendev.org/c/openstack/ironic/+/93564512:22
opendevreviewcid proposed openstack/ironic master: Fix double JSON encoding of error message  https://review.opendev.org/c/openstack/ironic/+/93179512:26
cardoeMorning ironic.12:59
TheJuliagood morning14:01
rpittaugood night! o/16:51
stephenfinadamcarthur5: Sorry, I forgot to open Hexchat yesterday (and most of today, apparently). I have the WIP comment addressed since in nova. Let me respin with my attempt at a resolution and you can take it and rework as you see fit?17:19
JayFstephenfin: I think we were both confused about what that comment meant; it looked like e.g. that shards change does what needs to be done17:32
JayFso might be helpful to restate the issue in more specific detail if it's not 100% resolved or you want us to be able to tell what's going on17:32
stephenfinYeah, that was poorly worded. I should have said "we don't use all the code yet"17:51
JayFjfyi our CI just got switched to noble, apparently18:16
JayFso be very aware when rechecking, I had a change get bombed just now :/18:16
opendevreviewStephen Finucane proposed openstack/ironic master: api: Introduce new mechanism for API versioning  https://review.opendev.org/c/openstack/ironic/+/92891918:24
opendevreviewStephen Finucane proposed openstack/ironic master: api: Add schema validation framework  https://review.opendev.org/c/openstack/ironic/+/92892018:24
opendevreviewStephen Finucane proposed openstack/ironic master: WIP: api: Add schema for allocations API  https://review.opendev.org/c/openstack/ironic/+/92892118:24
opendevreviewStephen Finucane proposed openstack/ironic master: api: Don't use private attributes  https://review.opendev.org/c/openstack/ironic/+/93569718:24
opendevreviewJay Faulkner proposed openstack/ironic master: Use specific fix-commit from dnsmasq  https://review.opendev.org/c/openstack/ironic/+/93569918:39
JayF^ is the new version of the dnsmasq workaround. We're kinda lucky it didn't land in the old version of it, bceause it would've blocked all CI when noble was switched to18:40
JayFalthough TBH I don't know if CI is happy right now or not :) was going to let that patch double as my clean test18:40
TheJuliaI guess time will tell18:51
JayFI did email dnsmasq-discuss and update the ubuntu bug18:53
JayFasking for a release and a backport to noble 18:53
JayFso hopefully we can get outta the dnsmasq building business one way or another, but at least now we can build a *shiny new version* instead of downgrading to a less-broken one18:53
TheJuliawheeeeee19:18
TheJuliaJayF: hey, what was the feedback regarding out docs and networking details? I've got a downstream bug of sorts I'm looking at which basically points to upstream docs and I'm looking at them with a differen that on and thinking "wow, we did a bad job at setting the wrong context in the networking docs"20:17
JayFI don't remember specifically, it'd be in the full document -- but that sounds like part of the thread through everything: talking about what things are theoretically possible instead of giving opinionated use cases and how to implement them20:18
JayFI can search for the full doc20:18
TheJuliaThis seems like a lot of "focusing on how you can" and failing to build the base context20:19
JayFhttps://docs.google.com/document/d/1e9URuPHKNTx5QXdkCFAzsS0EAAxPJ-xQC4BEpDwESp4/edit?tab=t.0 my recollection is accurate20:19
JayFyep, exactly20:19
TheJuliabasically, the question got raised "how do I do other non-provisioning" network port configuration20:20
TheJuliaand some of that *is* defeinitely theory we seem to leave out20:20
TheJuliaWe're like... missing the "here is the 101 level class on how this works20:20
JayFextremely yes20:21
TheJuliaAnd as much as I think every vendor would prefer customers just focusing on their downstream docs, people go look upstream  and even downstream docs folks go look at upstream20:21
JayFyeah20:22
JayFsimilarly I have an ask from my (GR-OSS, not GR) downstream to help with like ... a reference architecture for running OpenStack + Kubernetes20:22
JayFnot a "how do you arrange your k8s sandwich" doc, but literally a  "if you don't know what to do; do it this way" kinda thing20:22
JayFand I think advanced (physical) networking is one of the big holes in the pieces that exist regardingt hat already20:23
cardoeOpenStack + Kubernetes in what sense?20:29
cardoeOpenStack on top of Kubernetes?20:29
cardoeLook at you guys caring about docs. Meanwhile I'm working on my fork of the keystone docs.20:34
adamcarthur5stephenfin Thank you, much appreciated 20:34
opendevreviewMerged openstack/ironic-tempest-plugin master: Fix failure cases around test_ports  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/93438720:36
JayFcardoe: the interesting thing is the answer can be either thing in either palce; which is part of why we want a good example20:44
cardoehttps://github.com/rackerlabs/genestack is OpenStack on top of Kubernetes20:44
cardoeThat's Kevin Carter's baby.20:45
cardoehttps://review.opendev.org/c/openstack/tenks/+/907112 if you wanna +W that. That'll land Adam's chain.20:45
cardoeMy baby is https://github.com/rackerlabs/understack which is very rough but it's Ironic (Nova, Neutron, Glance, Keystone) + Nautobot on top of Kubernetes.20:45
JayFThe fact you, at a single comapny, just listed two examples is emblematical of the problem20:46
JayFwhen so many things are possible it leads to decision paralysis20:46
JayF(while asking for a review on a third)20:47
JayFcardoe: you didn't +2 907112, you only +1'd20:47
cardoeI can't +2 it20:47
JayFthat is a bug in that ACL then, tenks is an ironic project and should be following the rest of the stuff20:48
JayFmight wanna have a look at project-config, I'm not going to rabbithole on  it now20:48
cardoeSo different things for different folks AFAIK Kubernetes. So genestack uses kube-ovn so that the services are all plugged in together and magical with OVN/OVS Neutron or something.20:58
cardoeI dunno. I defer to jamesdenton on magical unicorn network bits.20:59
cardoeunderstack is baremetal kubernetes so that we can do DHCP and stuff.20:59
JayFYeah, but I think you're nibbling around the entire point: there are a thousand knobs21:06
cardoeCorrect21:07
JayFand when we are trying to pitch people on OpenStack, or more generally, and upstream-focused cloud stack including OpenStack/Ironic elements and Kubernetes, it'd be nice to have one that's relatively proscriptive for most people.21:07
cardoe2^1000 possibilities21:07
JayFwe need the sales engineer ppt for reference architecture :D 21:07
cardoeWell that's the thing. That's not OpenStack.21:07
cardoeOpenStack is like the big box of multi colored Legos that's instruction books are a bunch of pictures of stuff you can build without the actual steps.21:08
cardoeSome projects *cough* keystone *cough* resent if you try to provide instructions. While others (Ironic) embrace having details in the docs.21:09
JayFI'm gonna be honest, I don't think it's productive to talk about other projects' flaws in Ironic's IRC channel 21:09
cardoeI'm just grumpy right now. I'll stop.21:09
JayFI know you got issues with some of the stuff over there, but if we can keep positive and goal oriented we can get more done21:10
cardoeBut ultimately OpenStack embraces the possibilities of what you can do.21:10
cardoeI dunno if you're going to come up with one answer. I know some folks are working on the OpenStack operators on top of OpenShift. There's OpenStack Helm.21:12
JayFwell the thing is, especially for software like OpenStack -- and Ironic moreso given we even have more modes of operation (bifrost, metal3, standalone, integrated) -- that even though almost infinite things are possible; it's valuable for us to identify one or two use cases and have an opinion about how to use them21:13
JayFif nothing else to help people get started with their own architecture with something other than a blank sheet of paper21:13
JayFI think I just cracked the nut on getting CI working for Gentoo DIB again \o/21:14
JayFwhich means I can pick up the ramdisk building21:14
cardoeSo I do agree there and that's what my goal is.21:14
cardoeMy downstream is a configurable deployment within my narrowly defined use case.21:15
JayFhttps://review.opendev.org/c/openstack/ironic/+/935699 is passing CI with the new version of the dnsmasq fix21:15
JayFif someone wants to land it I suspect it'll have a positive impact on CI randomness21:15
cardoeYou also get a +1 from me on your Gentoo dev test by using make PREFIX=/usr install instead of the sed.21:17
JayFyeah tbh that original sed drove me a little bonkers but I'm wise enough not to try and shove a style fix through :)21:18
JayFI also tested to ensure it worked lol21:18
JayF#1 thing you learn working on Gentoo: never ever ever trust a build system21:18
JayFOMG cardoe I forgot21:18
JayFcardoe: I voluntell you to review this and help me maintain it: https://review.opendev.org/c/openstack/ironic/+/9356921:19
JayFlol21:19
JayF(is it that easy?)21:19
JayFthat link is wrong 21:20
cardoeI already +'d it21:20
JayFhttps://review.opendev.org/c/openstack/diskimage-builder/+/92398521:20
JayFI meant this one21:20
cardoeOH NO.21:20
cardoeEven before I clicked it I'm scared.21:20
JayFI've done all the hard work21:21
cardoeYou know prometheanfire should also help review and maintain.21:22
cardoeI was his dev mentor.21:23
JayFhe wrote the original stuff21:23
JayFbasically we wanna fix this21:23
JayFmove tinyipa ramdisk to a gentoo+dib ramdisk (since gentoo should let us slim it down dramatically)21:23
JayFthen I wanna go over on gentoo wiki side and document this as a useful tool because it's the most gentoo thing I've ever seen outside of gentoo21:23
JayFI think Sam James from over there is interested in using DIB to build gentoo images once I get it working :D 21:24
cardoeYou're changes are actually definitely positive fixes and you've got good comments.21:24
cardoeThat makes sense.21:25
JayFwith a cached stage321:26
JayFusing binary packages by default21:26
JayFthe build is BLAZING FAST21:26
cardoeheh until you change a USE flag.21:29
JayFportage is pretty smart these days about handling that; basically if you hook up the binhost, it just works. You get binaries when the use flags match (and they are building multiple profiles; so you have more than one shot at matching even on a custom system) and you build em when they don't21:30
JayFwe don't default to them because some of our test clouds are too old, but they even have x86-64-v3 binaries available21:30
JayFI think Eli also even has that build machine up to build random packages with random use flags as a test and as a bonus (you might win the lottery and have a version you need to upgrade to built for you)21:31
cardoedon't you talk like that about OSPC 21:31
JayFOSPC?21:32
cardoeYour old thing at Rackspace.21:33
JayFooh21:33

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