Wednesday, 2024-10-02

opendevreviewJulia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM  https://review.opendev.org/c/openstack/ironic/+/93106102:57
opendevreviewOpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/ironic/+/93112104:09
rpittaugood morning ironic! o/06:46
opendevreviewMerged openstack/sushy master: contributor: links to ironic dev docs, launchpad  https://review.opendev.org/c/openstack/sushy/+/93094607:17
opendevreviewMerged openstack/ironic master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/ironic/+/93112107:41
iurygregorygood morning ironic10:56
TheJuliagood morning13:09
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM  https://review.opendev.org/c/openstack/ironic/+/93106113:37
* TheJulia puts on a sign which says "zombie"13:44
TheJuliajamesdenton: Thank you for replying to that email to the mailing list, I haven't had spoons to really read much of the content yet, but really appreciate the engagement13:45
jamesdentonof course. i don't get an opportunity as much as i'd like13:46
TheJuliajamesdenton: do you feel there is a need for my engagement on the thread in the next day or so?13:48
jamesdentonhard to say - more details about their particular setup might provide some insight on how to handle their request. But based on what i read, it seems like some kind of driver to interact with the switches would be the approach to go with. We'll see if they follow up and we can start going down that rabbit hole13:51
TheJuliaclarkb: what was that etherpad for zuul feedback? I'm looking at the UI thinking "it would be awesome if I could save a preference to narrow/shrink the progress status bar so the job names don't wrap14:17
clarkbTheJulia: I think we're trying to use that document as a synthesis of the feedback we're receiving rather than a direct feedback/complaints doc. Helps keep it manageable and allows for deduplication etc. If we can get the feedback in a channel that zuul devs frequent (zuul matrix room, zuul-discuss mailing list, or even #opendev) we can make sure it gets collected14:44
clarkbin the meantime you might be able to `ctrl + -` in the browser and get things on one line that way?14:45
TheJuliaclarkb: I'll try that, but persistent zooming means I would be altering the browser while hunting fo what is going on and then digging in because I likely wouldn't be able to easily read the actual logs if I zoom out too much.14:51
clarkbya and I'm getting mixed results testing that myself14:52
clarkbthe mysteries of html rendering14:52
TheJuliaapparently my first attempt at sending that message didn't go through14:53
opendevreviewTakashi Kajinami proposed openstack/ironic-python-agent master: Get rid of pkg_resources  https://review.opendev.org/c/openstack/ironic-python-agent/+/93118815:10
jamesdentonTIL that specifying DIB_INIT_SYSTEM really helps unlock some useful behaviors when building images :D15:21
TheJuliajamesdenton: building what?15:21
jamesdentonironic-python-agent-builder - was having to revisit the patch to remove glean-early.service. I was finding that the kernel/ramdisk i was building didn't trigger that until I set DIB_INIT_SYSTEM. https://github.com/openstack/ironic-python-agent-builder/blob/a23d3d03286165f22e77b60abebf5e4507188156/dib/ironic-python-agent-ramdisk/post-install.d/80-ironic-python-agent-ramdisk#L6515:24
jamesdentonThis was my prior build command: DIB_RELEASE=9-stream ironic-python-agent-builder -o /tmp/ipa-ramdisk-stream centos -e simple-init -e devuser15:26
TheJuliayou shouldn't need DIB_RELEASE on the latest release of DIB, fwiw15:26
jamesdentongood to know15:26
TheJuliabut that might switch later on when 10 drops and it becomes reasonable for the default to change15:27
TheJuliain other words, the centos element no longer focuses on centos715:27
TheJulia:)15:27
TheJuliaThat sort of highlighted why we likely just need to make reasonable default tracking a thing15:27
jamesdentoni was also able to build centos from ubuntu, but needed to add the ca path: REQUESTS_CA_BUNDLE="/etc/pki/tls/cert.pem" DIB_RELEASE=9-stream ironic-python-agent-builder -o /tmp/ipa-ramdisk-stream centos -e simple-init -e devuser15:27
TheJuliaawesome15:28
jamesdentonis there a set of DIB_* defaults that maybe i'm failing to set?15:28
TheJuliayou *really* shouldn't need to imho, except in weird situations like you need to support evil 4k block devices15:29
TheJuliaQueue Julia emulating an unhappy cat while glaring at servers with specific raid controllers15:30
rpittaugood night! o/15:40
jamesdentonTheJulia so, turns out i was wrong about needing to specify DIB_INIT_SYSTEM and it was simply a matter of me using the wrong branch (2024.1) vs 2024.2 or master and making a false correlation. Too much coffee and not enough sleep. 16:13
TheJuliajamesdenton: for diskimage-builder related stuffs, you might be best just chasing master, but for building a specific version of IPA, I could see you wanting to do that16:15
jamesdentoni'm not sure what got me into the pickle but it's all good now16:16
TheJuliaGood! </mr. burns>16:16
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM  https://review.opendev.org/c/openstack/ironic/+/93106116:33
opendevreviewTakashi Kajinami proposed openstack/ironic-python-agent master: Get rid of pkg_resources  https://review.opendev.org/c/openstack/ironic-python-agent/+/93118816:48
JayFtkajinam: thank you for these changes, btw, I appreciate that you're helping clean some of this up16:50
JayFtkajinam: if there's ever anything like this you identify but don't have time to fix, if you file a bug we can point someone at it :)16:51
tkajinamJayF,  thanks ! yeah I'll do so when needed :-D16:53
opendevreviewTakashi Kajinami proposed openstack/ironic-python-agent master: Get rid of pkg_resources  https://review.opendev.org/c/openstack/ironic-python-agent/+/93118817:21
TheJuliaiurygregory: I have bad news and semi-good news. The I have not located the stickers where I thought they would be. The semi-good news is I know where 5they are not at present18:02
iurygregoryTheJulia, no worries =) tks for the update :D18:13
JayFI'm just curious: is "discovery" and "unmanaged inspection" the same thing?18:19
JayFJust making sure they are, as I assume so18:19
JayFalso an interesting code review question here, https://review.opendev.org/c/openstack/ironic/+/930944/2/ironic/api/controllers/v1/port.py#569 18:31
iurygregorygreat question in the review, now my head is spinning to find an answer =)18:38
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM  https://review.opendev.org/c/openstack/ironic/+/93106119:00
TheJuliathere it goes19:00
opendevreviewJulia Kreger proposed openstack/ironic master: DNM/Ignore: Science! trim out cirros partition...  https://review.opendev.org/c/openstack/ironic/+/93122219:53
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM  https://review.opendev.org/c/openstack/ironic/+/93106120:42
opendevreviewJulia Kreger proposed openstack/ironic master: CI: Add a 4k disk CI job  https://review.opendev.org/c/openstack/ironic/+/93065521:25
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM  https://review.opendev.org/c/openstack/ironic/+/93106121:25
JayFcardoe: https://review.opendev.org/c/openstack/ironic-python-agent/+/912949/2#message-25ebb161e04b395aa1a41969dccf7c61aa717117 tl;dr we should kill CI with fire on ipa um/wallaby21:47
JayFimho21:47
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM  https://review.opendev.org/c/openstack/ironic/+/93106121:57
TheJuliaI'm good with dialing most of the CI back there22:00
cardoejamesdenton: what shenanigans are you up to?22:00
cardoeJayF: sure. I was just trying to clear up patches.22:00
JayFI'm just saying, as a UM and Ironic core, I'd merge such a change.22:01
JayFIf I made such a change, I couldn't merge it.22:01
JayF;)22:01
TheJuliaI could...22:02
JayFbe vewy quiet I'm hunting cardoe^W wabbits22:03
JayF:D 22:03
cardoeWell only so much beach time22:04
cardoeMy monitor tan can handle. So it might happen this week or next.22:04
JayFoh, you're on vacation22:08
JayFwhy are you poking at patches?!22:08
JayFnow I feel guilt by association22:08
cardoeCause I would melt in the sun!22:09
JayFhave you ever done a fun thing on your computer before? you should try :P 22:10
JayFhttps://review.opendev.org/c/openstack/ironic-python-agent/+/912949/ pushed with ci removals22:21
JayFalso proposed https://review.opendev.org/c/openstack/project-config/+/931240 which would put UM patch notifications in here22:26
JayF(for ironic projects)22:26
jamesdentoncardoe just circling back on outstanding projects and tests22:33
TheJulia... okay conductor: ironic-conductor[123723]: DEBUG ironic.drivers.modules.agent_base [-] Deploy step still running for node 4575ff10-31df-4308-97ed-a1090b1f6250: None {{(23:11
* TheJulia lays down23:12
opendevreviewMerged openstack/ironic stable/2023.2: [CI] Fix job parent name  https://review.opendev.org/c/openstack/ironic/+/92597623:22
cardoeSo I can’t touch hashtags on sushy but can on the rest.23:27
cardoeJayF: patches aren’t fun?!?23:27
JayFnot as fun as other things :D 23:58

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