Universal Blue•3y ago•

42 replies

bsherman looks like some issue with

@bsherman looks like some issue with glibc

bsherman•9/28/23, 1:34 PM

is it?

EyeCantCUOP•9/28/23, 1:34 PM

It can't pull in the proper version

EyeCantCUOP•9/28/23, 1:34 PM

At least over in Nvidia

EyeCantCUOP•9/28/23, 1:34 PM

I think the images are mismatched with what's in the Fedora repos rn

bsherman•9/28/23, 1:34 PM

well nvidia fails are expected by me, since main is failing too

bsherman•9/28/23, 1:34 PM

https://github.com/ublue-os/main/actions/runs/6335640464/job/17207163548#step:9:621

GitHub

build-ublue · ublue-os/main@b9b3095

OCI base images of Fedora with batteries included. Contribute to ublue-os/main development by creating an account on GitHub.

bsherman•9/28/23, 1:35 PM

like why is kernel-uki-virt 6.4.15 here, but we should be on 6.5.5

bsherman•9/28/23, 1:35 PM

same with xpadneo

bsherman•9/28/23, 1:35 PM

it's like it started running before our akmods images published?

EyeCantCUOP•9/28/23, 1:35 PM

This probably explains why I've been having so many build issues the last two days. That would explain it

bsherman•9/28/23, 1:36 PM

yeah, i let it go yesterday because 1) i was busy 2) it can happen

bsherman•9/28/23, 1:36 PM

but 2 days in a row is more concerning

EyeCantCUOP•9/28/23, 1:36 PM

We may need to push back mains cron job

bsherman•9/28/23, 1:36 PM

hmmm no

bsherman•9/28/23, 1:37 PM

akmods finished at 6:15am GMT... and main didn't start until 7:05am GMT or so

bsherman•9/28/23, 1:39 PM

OH!

bsherman•9/28/23, 1:39 PM

i know

EyeCantCUOP•9/28/23, 1:39 PM

Yeah, I don't think akmods is holding it back after looking more into it

bsherman•9/28/23, 1:40 PM

i changed the tagging in akmods and we forgot to make main use the new tagging

EyeCantCUOP•9/28/23, 1:40 PM

Oooooof

EyeCantCUOP•9/28/23, 1:40 PM

That explains it

bsherman•9/28/23, 1:40 PM

instead of simple ":38" it's now ":main-38" becasue of the kernel flavor

bsherman•9/28/23, 1:41 PM

tell me now if that's horrible

bsherman•9/28/23, 1:42 PM

i did it on purpose, and it was approved

but it was one of those things I expected to have a discussion about, but it was approved without discussion

EyeCantCUOP•9/28/23, 1:43 PM

I like that it's using the base name. I don't like that it breaks downstream. We should check for main as the base and use the old tagging for main akmods

bsherman•9/28/23, 1:43 PM

my reasoning was to get away from the idea that akmods was one-size-fits-all ... we'd not have a "default" that was just a version

beekabop•9/28/23, 1:43 PM

If that means you wont get updates when your system is on silverblue-main:38 then its bad. otherwise ignore me

EyeCantCUOP•9/28/23, 1:43 PM

That's true..

EEyeCantCU I like that it's using the base name. I don't like that it breaks downstream. We...

bsherman•9/28/23, 1:43 PM

well it won't break downstream

if we make a simple change in main

Bbeekabop If that means you wont get updates when your system is on silverblue-main:38 the...

EyeCantCUOP•9/28/23, 1:43 PM

It should not impact that

Bbeekabop If that means you wont get updates when your system is on silverblue-main:38 the...

bsherman•9/28/23, 1:44 PM

it'll be fine

this is exactly what we are trying to fix now

Bbsherman well it won't break downstream 🙂 if we make a simple change in main

EyeCantCUOP•9/28/23, 1:44 PM

That's true. Either way I'm fine

bsherman•9/28/23, 1:44 PM

i don't like the word "main" it's so overloaded

EyeCantCUOP•9/28/23, 1:44 PM

It also looks like we have unanimous approval to remove kmods from 39 so I'll work on that

EyeCantCUOP•9/28/23, 1:44 PM

It definitely is

Bbsherman i don't like the word "main" it's so overloaded

j0rge•9/28/23, 1:46 PM

remmember when we made a base-basebase-base?

bsherman•9/28/23, 1:47 PM

ok, so either in akmodsakmods repo we have a little annoying logic to set the tag name to ":RELEASE" intead of ":KERNEL_FLAVOR-RELEASE" when KERNEL_FLAVOR=main

or... we change the tag

main

main

consumes...
AND update the akmods README (which desperately needs it, i've got 3 open issues to update parts of it)

I think as long as it's documented in README we can made the change

j0rge•9/28/23, 1:47 PM

just turn-up-the base-basejust turn-up-the base-base

bsherman•9/28/23, 1:49 PM

my reason for explicitly using :main-38:main-38 instead of just :38:38 is to make it more clear for anyone that they can't just assume the kernel by tags

bsherman•9/28/23, 1:49 PM

so i think i'm going to stick to my guns there

bsherman•9/28/23, 1:49 PM

fix incoming on

main

main

repo to use new tag...

then i'll update some akmodsakmods README stuff

Bbsherman ok, so either in `akmods` repo we have a little annoying logic to set the tag na...

EyeCantCUOP•9/28/23, 1:51 PM

If it's as simple as the second, let's do that

bsherman•9/28/23, 1:52 PM

there's actually a PR needed for nvidia, too... doing that, but it shouldn't build until after main does

bsherman looks like some issue with

Similar Threads

Similar Threads

Similar Threads