/srv/irclogs.ubuntu.com/2023/02/01/#ubuntu-devel.txt

bryceh*wave* MacSlow05:02
ahasenackis there a way to pass DEB_BUILD_OPTIONS to a ppa build? I'm guessing not, but thought I should ask14:07
cjwatsonahasenack: no, you have to actually change the packaging.  (We've discussed it in the past, albeit more about profiles than options I think, but it's never got past the discussion phase.)14:30
ahasenackok, thx. Yeah, profiles would be a way14:30
ahasenackan ubuntu-advantage-tools dep8 test in bionic/arm64 is failing with this:14:39
ahasenackInstalling for arm64-efi platform.14:39
ahasenackgrub-install: error: efibootmgr: not found.14:39
ahasenackdpkg: error processing package grub-efi-arm64-signed (--configure):14:39
ahasenack installed grub-efi-arm64-signed package post-installation script subprocess returned error exit status 114:39
ahasenackProcessing triggers for man-db (2.8.3-2ubuntu0.1) ...14:39
ahasenackErrors were encountered while processing:14:39
ahasenack grub-efi-arm64-signed14:39
ahasenackE: Sub-process /usr/bin/dpkg returned an error code (1)14:39
ahasenackops, sorry, longer past than I thought14:39
ahasenackI know there are grub updates in flight, it this related? Should the test be run with proposed?14:40
ahasenackit's failing like this just in bionic/arm6414:41
jbichajawn-smith: since you touched icu last, will we be syncing icu 72 from Debian (starts a big transition)?15:56
ahasenackjuliank: is this something known in bionic/arm64 related to the recent grub/signed/shim updates? Excerpt above, but also at the end of https://autopkgtest.ubuntu.com/results/autopkgtest-bionic/bionic/arm64/u/ubuntu-advantage-tools/20230201_124925_331ea@/log.gz17:29
juliankyes17:29
ahasenackshould I re-run that with proposed, or just ignore the failure?17:30
julianktemporary mishap as broken 1.52 was released to release17:30
juliankit doesn't work until a new image with 1.54 has been built17:30
juliankor well a new image with the old shim now in updates :)17:30
juliankah sorry17:30
juliankI read the wrong log17:31
juliankthe bionic images are broken somehow because they have UEFI but not efibootmgr installed which is needed to actually install grub, so how did they install grub in the first palce17:31
ahasenack¯\_(ツ)_/¯17:32
julianklike there's no fix for that in proposed, that's not a regression17:32
juliankbionic images also can't install kernel upgrades on arm6417:33
ahasenackit says it's upgrading grub-efi-*, so it must exist in the image already17:33
juliankyes we accidentally dropped the dependency on efibootmgr in 2020 I think17:33
ahasenackbut the test did pass in 2023-01-28 01:49:44 UTC17:34
ahasenackhttps://autopkgtest.ubuntu.com/packages/u/ubuntu-advantage-tools/bionic/arm6417:34
ahasenackjust a few days ago17:34
juliankyes but it never passes when it needs to upgrade a grub17:34
ahasenacklet me check some other recent bionic sru17:34
juliank:D17:34
cjwatsonI deliberately patched grub-install in 2019 to use libefiboot/libefivar directly rather than efibootmgr17:35
cjwatsonBut possibly that didn't apply to arm64, I don't remember17:35
juliankwe didn't pick that back to bionic17:35
cjwatsonThat's also post-bionic, so ...17:35
cjwatsonYeah17:35
juliankI think I'll just add a Depends: efibootmgr to grub-common tbh17:35
juliankI need to add support for gzipped kernels to grub-common too17:36
juliankSo like that's the lowest impact fix17:36
juliankPeople without grub-efi stuff installed will get efibootmgr pulled in but meh17:36
ahasenackhttps://autopkgtest.ubuntu.com/packages/s/shim-signed is also failing, slightly different error17:36
ahasenack /bin/sh: 1: grub-efi-arm64-signed: not found17:37
juliankThat's expected17:37
juliankThe error is weird17:37
juliankBut anyhow it depends on a newer fwupd than we have in bionic yet17:37
cjwatsonThere's no reason to put the dependency in grub-common rather than in grub-efi-*-bin17:38
ahasenackhttps://autopkgtest.ubuntu.com/packages/c/containerd/bionic/arm64 last run on the 26th, green, but a few days before the breakage started17:38
cjwatsonIt certainly used to be in grub-efi-arm64-bin among others17:38
juliankcjwatson: yes, but I don't want to respin all the grub2-unsigned grub2-signed17:38
juliankcjwatson: I only build them in kinetic17:38
juliankcjwatson: They are binary copied to bionic (the unsinged)17:38
juliankSo you gotta update all release series when making a change17:39
cjwatsonHow does adding a dependency to grub-common rather than grub-efi-*-bin help you there?  They're both from the same source package, right?17:39
juliankgrub-common is in src:grub2, not src:grub2-unsigned17:39
julianksrc:grub2 is not shared between releases17:39
cjwatsonAh I see, that got moved.  Ugh.  I think that's a bad reason for a spurious dep, but not my call17:40
juliankWe split that up back in 2020 to have a single grub version for security supported platforms :)17:40
juliankcjwatson: Right we could backport libefivars support too but then it ends up in grub-common too, so the difference isn't large.17:40
juliankvorlon: What's your opinion? Add efibootmgr Depends to grub-common/bionic on arm64 amd64 (i386?), or respin the whole grub2-unsigned batch with a Depends: efibootmgr | grub2-common (>= 2.04-something?), or backport libefivars support to grub-common?17:43
juliankLike if users upgrade from bionic to focal their grub-common depends on libefivars anyhow, having bionic depend on efibootmgr there seems fair17:44
juliankThat avoids the annoying dependency in the shared package just for bionic :)17:45
juliankBasically I lean towards grub-common as I try to converge on whatever we do in later series, so there's less surprises when backporting grub2-unsigned17:45
juliankAnd we can fast track the src:grub2/bionic SRU17:46
juliankvorlon: have a look here, that SRU I think we can fast track, dannf could verify and we could just release it same-day https://code.launchpad.net/~juliank/grub/+git/bionic/+merge/43670517:50
juliankwith the patch, grub-check-signatures is identical to all other series17:50
vorlonjuliank: so, I need to step out for food, back in ~1.5h.  I haven't had a chance to digest this and think about it seriously.  But I think my default position is that we should not work around having to do a separate build of grub2-unsigned for bionic with ugly packaging tricks17:53
juliankWe absolutely should not do separate builds, and we'd still need to respin all series to accomodate17:54
juliankso the versions don't go backward :)17:55
juliankWe tried that once to make a change in bionic only and it was lost again in the tumult of the next security update so that's just risk17:55
juliank(that became https://bugs.launchpad.net/ubuntu/+source/grub2/+bug/1995751)17:57
-ubottu:#ubuntu-devel- Launchpad bug 1995751 in grub2-unsigned (Ubuntu Bionic) "update to 2.04-1ubuntu47.4 drops zz-update-grub" [Undecided, Confirmed]17:57
juliankxenial ESM too17:59
vorlonwe'd have to do respins, yes; but I don't particularly agree that the greater risk is from having deltas across series, we should just have a policy of not clobbering them in updates like we do for everything else!  trying to make a single package compatible across all series is a worse risk because of things like this17:59
vorlonanyway, food, argue more later :)17:59
* juliank uploaded that branch to the queue just in case :)18:02
ahasenackoh, you found an existing bug about this18:04
ahasenackhttps://bugs.launchpad.net/ubuntu/+source/grub2-signed/+bug/193685718:04
-ubottu:#ubuntu-devel- Launchpad bug 1936857 in grub2-signed (Ubuntu Bionic) "grub-install: error: efibootmgr: not found." [Undecided, In Progress]18:04
ahasenackalso https://bugs.launchpad.net/ubuntu/+source/linux-hwe-5.4/+bug/1991676, more specific to autopkgtests18:05
-ubottu:#ubuntu-devel- Launchpad bug 1991676 in grub2-signed (Ubuntu Bionic) "Package grub-efi-arm64-signed 1.173.2~18.04.1+2.04-1ubuntu47.4 from bionic-proposed fails to install/upgrade (grub-install: error: efibootmgr: not found.)" [High, Triaged]18:05
juliankahasenack: yeah dann reported it again this morning :)18:05
juliankdannf: fixes for the bugs you complained about yesterday: https://code.launchpad.net/~juliank/grub/+git/bionic/+merge/43670518:06
ahasenackjuliank: lunar is also failing in a related area in arm64, with proposed already it seems: https://autopkgtest.ubuntu.com/results/autopkgtest-lunar/lunar/arm64/u/update-motd/20230201_130652_2066a@/log.gz18:27
ahasenack update-alternatives: error: alternative link /usr/lib/shim/shimaa64.efi.signed is already managed by shimx64.efi.signed18:27
juliankahasenack: yes we know, see my first reply to you, that was supposed to be about this18:34
ahasenackok, cool18:34
juliank🥺19:41
arrayboltCBook[]19:43
arrayboltCBookHmm... that emoji renders in Element but not at all in WeeChat on Lubuntu...19:44
arrayboltCBook(the one above, not my [], that was just silliness)19:44
ograi only see two brackets here19:44
ograah 🙂19:44
ogralol19:44
arrayboltCBookI saw what looked like a block and was like "ok so we're sending blocks to the screen, ok, []" :P19:45
jbicha🫠21:25

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!