opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 02:57 |
---|---|---|
opendevreview | OpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/931121 | 04:09 |
rpittau | good morning ironic! o/ | 06:46 |
opendevreview | Merged openstack/sushy master: contributor: links to ironic dev docs, launchpad https://review.opendev.org/c/openstack/sushy/+/930946 | 07:17 |
opendevreview | Merged openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/931121 | 07:41 |
iurygregory | good morning ironic | 10:56 |
TheJulia | good morning | 13:09 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 13:37 |
* TheJulia puts on a sign which says "zombie" | 13:44 | |
TheJulia | jamesdenton: 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 engagement | 13:45 |
jamesdenton | of course. i don't get an opportunity as much as i'd like | 13:46 |
TheJulia | jamesdenton: do you feel there is a need for my engagement on the thread in the next day or so? | 13:48 |
jamesdenton | hard 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 hole | 13:51 |
TheJulia | clarkb: 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 wrap | 14:17 |
clarkb | TheJulia: 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 collected | 14:44 |
clarkb | in the meantime you might be able to `ctrl + -` in the browser and get things on one line that way? | 14:45 |
TheJulia | clarkb: 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 |
clarkb | ya and I'm getting mixed results testing that myself | 14:52 |
clarkb | the mysteries of html rendering | 14:52 |
TheJulia | apparently my first attempt at sending that message didn't go through | 14:53 |
opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Get rid of pkg_resources https://review.opendev.org/c/openstack/ironic-python-agent/+/931188 | 15:10 |
jamesdenton | TIL that specifying DIB_INIT_SYSTEM really helps unlock some useful behaviors when building images :D | 15:21 |
TheJulia | jamesdenton: building what? | 15:21 |
jamesdenton | ironic-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#L65 | 15:24 |
jamesdenton | This was my prior build command: DIB_RELEASE=9-stream ironic-python-agent-builder -o /tmp/ipa-ramdisk-stream centos -e simple-init -e devuser | 15:26 |
TheJulia | you shouldn't need DIB_RELEASE on the latest release of DIB, fwiw | 15:26 |
jamesdenton | good to know | 15:26 |
TheJulia | but that might switch later on when 10 drops and it becomes reasonable for the default to change | 15:27 |
TheJulia | in other words, the centos element no longer focuses on centos7 | 15:27 |
TheJulia | :) | 15:27 |
TheJulia | That sort of highlighted why we likely just need to make reasonable default tracking a thing | 15:27 |
jamesdenton | i 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 devuser | 15:27 |
TheJulia | awesome | 15:28 |
jamesdenton | is there a set of DIB_* defaults that maybe i'm failing to set? | 15:28 |
TheJulia | you *really* shouldn't need to imho, except in weird situations like you need to support evil 4k block devices | 15:29 |
TheJulia | Queue Julia emulating an unhappy cat while glaring at servers with specific raid controllers | 15:30 |
rpittau | good night! o/ | 15:40 |
jamesdenton | TheJulia 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 |
TheJulia | jamesdenton: 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 that | 16:15 |
jamesdenton | i'm not sure what got me into the pickle but it's all good now | 16:16 |
TheJulia | Good! </mr. burns> | 16:16 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 16:33 |
opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Get rid of pkg_resources https://review.opendev.org/c/openstack/ironic-python-agent/+/931188 | 16:48 |
JayF | tkajinam: thank you for these changes, btw, I appreciate that you're helping clean some of this up | 16:50 |
JayF | tkajinam: 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 |
tkajinam | JayF, thanks ! yeah I'll do so when needed :-D | 16:53 |
opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Get rid of pkg_resources https://review.opendev.org/c/openstack/ironic-python-agent/+/931188 | 17:21 |
TheJulia | iurygregory: 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 present | 18:02 |
iurygregory | TheJulia, no worries =) tks for the update :D | 18:13 |
JayF | I'm just curious: is "discovery" and "unmanaged inspection" the same thing? | 18:19 |
JayF | Just making sure they are, as I assume so | 18:19 |
JayF | also an interesting code review question here, https://review.opendev.org/c/openstack/ironic/+/930944/2/ironic/api/controllers/v1/port.py#569 | 18:31 |
iurygregory | great question in the review, now my head is spinning to find an answer =) | 18:38 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 19:00 |
TheJulia | there it goes | 19:00 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM/Ignore: Science! trim out cirros partition... https://review.opendev.org/c/openstack/ironic/+/931222 | 19:53 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 20:42 |
opendevreview | Julia Kreger proposed openstack/ironic master: CI: Add a 4k disk CI job https://review.opendev.org/c/openstack/ironic/+/930655 | 21:25 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 21:25 |
JayF | cardoe: https://review.opendev.org/c/openstack/ironic-python-agent/+/912949/2#message-25ebb161e04b395aa1a41969dccf7c61aa717117 tl;dr we should kill CI with fire on ipa um/wallaby | 21:47 |
JayF | imho | 21:47 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 21:57 |
TheJulia | I'm good with dialing most of the CI back there | 22:00 |
cardoe | jamesdenton: what shenanigans are you up to? | 22:00 |
cardoe | JayF: sure. I was just trying to clear up patches. | 22:00 |
JayF | I'm just saying, as a UM and Ironic core, I'd merge such a change. | 22:01 |
JayF | If I made such a change, I couldn't merge it. | 22:01 |
JayF | ;) | 22:01 |
TheJulia | I could... | 22:02 |
JayF | be vewy quiet I'm hunting cardoe^W wabbits | 22:03 |
JayF | :D | 22:03 |
cardoe | Well only so much beach time | 22:04 |
cardoe | My monitor tan can handle. So it might happen this week or next. | 22:04 |
JayF | oh, you're on vacation | 22:08 |
JayF | why are you poking at patches?! | 22:08 |
JayF | now I feel guilt by association | 22:08 |
cardoe | Cause I would melt in the sun! | 22:09 |
JayF | have you ever done a fun thing on your computer before? you should try :P | 22:10 |
JayF | https://review.opendev.org/c/openstack/ironic-python-agent/+/912949/ pushed with ci removals | 22:21 |
JayF | also proposed https://review.opendev.org/c/openstack/project-config/+/931240 which would put UM patch notifications in here | 22:26 |
JayF | (for ironic projects) | 22:26 |
jamesdenton | cardoe just circling back on outstanding projects and tests | 22: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 down | 23:12 | |
opendevreview | Merged openstack/ironic stable/2023.2: [CI] Fix job parent name https://review.opendev.org/c/openstack/ironic/+/925976 | 23:22 |
cardoe | So I can’t touch hashtags on sushy but can on the rest. | 23:27 |
cardoe | JayF: patches aren’t fun?!? | 23:27 |
JayF | not 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/!