=== jamesh_ is now known as jamesh
mborzeckizyga: hey07:02
zygahey :)07:02
zygahttps://listed.zygoon.pl/20686/raspberry-pi-4b-and-4k-display-at-60hz battle from last evening07:02
mborzeckizyga: btw. this is interesting: https://gist.github.com/dmp1ce/8a321608fbcf4c3ea61fce134e1f0b0d there's apparmor enabled, but /proc/self/attr/apparmor/current does not exist07:17
zygamborzecki how about /proc/self/attr/current07:17
zygathe apparmor sub-directory is relatively recent addition07:17
mborzeckithe user has that, maybe it's the lts kernel they are booting, suggested using the non lts one to see07:18
mborzeckifwiw, this code is supposed to guess the right location: https://gitlab.com/apparmor/apparmor/-/blob/master/libraries/libapparmor/src/kernel.c#L23507:18
mborzeckizyga: it eventually gets called from aa_getcon()07:19
zygawhat arch? do you have uname?07:20
zygamaybe wonky kernel from some cloud07:20
zygaor arm gizmo kernel07:20
mborzeckizyga: they said it's an lts server, so probably 5.4.82-107:20
mborzeckihm maybe i should ask them to build and run a snippet like this: https://gist.github.com/dmp1ce/8a321608fbcf4c3ea61fce134e1f0b0d07:21
* zyga changes hosts07:26
mupPR snapcraft#3391 opened: plugins v2: port the qmake plugin <Created by jhenstridge> <https://github.com/snapcore/snapcraft/pull/3391>07:38
zygamborzecki, pi4 desktop is quite nice07:44
zygasuch a long way from "it barely works" to "4K 60Hz"07:44
zygagood morning pawel08:03
zygaI managed to sort out my alt-desktop for now :)08:03
zygahello mvo08:22
mvogood morning zyga! how are you doing?08:22
zygahey :-)08:22
zygaI'm working on a pi desktop today08:22
zygafeeling good, trying to build a pipeline for debian packaging now08:23
pstolowskihey mvo, zyga!08:26
mvozyga: nice!08:30
mvopstolowski: good morning08:30
mborzeckipstolowski: mvo: hey08:36
mborzeckimvo: we should land it when the spread jobs finishes: https://github.com/snapcore/snapd/pull/978508:37
mupPR #9785: tests/main/fake-netplan-apply: disable test on xenial for now <Simple πŸ˜ƒ> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9785>08:37
mvomborzecki: ineed08:51
mvomborzecki: we need this for 2.48 too, right?08:52
mvoa review for 9758 would be really good08:53
mborzeckimvo: yes, i think so, unless we can get #9784 (or both?)08:57
mupPR #9784: interfaces/builtin/network-setup-{control,observe}: allow using netplan directly <Needs security review> <Simple πŸ˜ƒ> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9784>08:57
pedronisI don't understand the last change in #9784 fwiw08:59
mupPR #9784: interfaces/builtin/network-setup-{control,observe}: allow using netplan directly <Needs security review> <Simple πŸ˜ƒ> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9784>08:59
pedronisalso it really needs security reviews08:59
mupPR snapd#9785 closed: tests/main/fake-netplan-apply: disable test on xenial for now <Simple πŸ˜ƒ> <Squash-merge> <⚠ Critical> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9785>09:05
mupPR snapd#9786 opened: tests: backport netplan workarounds from #9785 (2.48) <Run nested> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9786>09:10
pedronismvo: I re-reviewed 9783, it's now failing on classic/hotplug :/09:31
pstolowskiSergio mentioned it, i'll take a look at this test09:35
mvopedronis: thanks for this!09:37
mborzeckimvo: we need this now? https://github.com/snapcore/snapd/pull/978309:56
mupPR #9783: gadget: use "sealed-keys" to determine what method to use for reseal <Run nested> <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9783>09:56
mvomborzecki: yeah, this pr will unblock master09:58
pedronismborzecki: I reviewed 9781, it conflicts with some of my incoming PRs but not too badly fwiw10:03
pedronisyes, we should really try to land 9783 as quickly as possible, it also unblocks my PRs10:04
pedronismvo: in theory we could just force land it, and then have a follow with the small changes suggested by Ian and me?10:07
pedronismvo: is it blocked by something else?10:08
mvopedronis: that sounds good, no10:08
mvopedronis: not blocked10:08
mvopedronis: I can squash merge it now10:08
mborzeckipedronis: thanks, we can land it before of after, wahtever is easier to do10:09
pedronismborzecki: I tried after10:13
mvopedronis: merged and cherry-picked, I will do the tweaks in a small followup10:13
pedronismvo: thanks10:13
pedronisI will merge master in my gadget PR10:13
pedronisI have follow ups but that one needs to land first10:14
mupPR snapd#9783 closed: gadget: use "sealed-keys" to determine what method to use for reseal <Run nested> <Squash-merge> <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9783>10:15
mborzeckipedronis: so let's merge #9775, then #978110:22
mupPR #9775: gadget,o/devicestate,tests: drop EffectiveFilesystemLabel and instead set the implicit labels when loading the yaml <Cleanup :broom:> <Run nested> <UC20> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9775>10:22
mupPR #9781: gadget/quantity: introduce Offset, start using it for offset related fields in the gadget <Run nested> <Simple πŸ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9781>10:22
mborzeckiin the meantime i'll look at updating #977110:22
mupPR #9771: boot: boot config update & reseal <Run nested> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9771>10:22
mvomborzecki: should I do this?10:25
mvomborzecki: I will force-merge 978110:28
mupPR snapd#9781 closed: gadget/quantity: introduce Offset, start using it for offset related fields in the gadget <Run nested> <Simple πŸ˜ƒ> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9781>10:30
pedronismvo: that's not the order mborzecki suggested though10:40
mupPR snapd#9758 closed: secboot: add new LockSealedKeys() that uses either TPM/fde-reveal-key <Squash-merge> <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9758>10:40
mupPR snapd#9787 opened: boot: tweak resealing with fde-setup hooks <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9787>10:40
pedronismvo: now 9775 has conflicts :/10:41
pedronisso it will need a new test run10:41
mvopedronis: oh no! I'm sorry!10:44
mvosorry, should not have interefreed10:45
mborzeckipedronis: i can help you and fix conflicts in 977511:00
pedronismborzecki: I'm fixing them but we need anyway to wait the current test run11:00
pedronisif we hope to avoid two nested runs11:01
paul424heh is it fair to duplicate the questions from #snapcraft here ?11:01
paul424and don't get banned ?11:01
pstolowskipaul424: you won't get banned, but forum.snapcraft.io is the best place for questions with most visibility and chances for answers11:06
paul424Heh witam jakie miasto ? :D11:07
paul424mogę dobrze napisać po polsku w czym rzecz :D11:08
paul424oki no more polish11:08
pstolowskipaul424: sorry, please use our forums11:09
pstolowskithe nested hotplug test failure seems to be caused by a change in qemu, the usb serial port doesn't show up on the bus11:35
ograoSoMoN, do we not update the chromium snap installer version alongside a new stable release ? (chromium-browser is at 85...)11:48
ograif we dont, we should probably not try to match the chromium version string in the deb package11:49
ogra(there is a user asking about it in #ubuntu)11:49
oSoMoNogra, we don't, unless there are other changes to the packaging11:49
oSoMoNI agree it's misleading, but it's also harmless11:50
ograright ... so it should probably just use a native debian version instead of suggesting to the users they get an old version11:50
oSoMoNbecause the user will always get the latest stable update through the snap11:50
oSoMoNogra, do you have a suggestion for a version number that would not be misleading?11:51
ograor 1-ubuntu1 if you want11:52
ograjust not the chromium version string11:53
ackkogra, that's pretty much what we did for the maas package in focal, which installs/transitions to the snap11:58
ackk(plus epoch)11:58
oSoMoNwe already have an epoch, so it would need to be bumped to ensure a greater version number than the current 1:85.0.4183.83-0ubuntu212:12
oSoMoNyeah, something like that IΒ guess12:13
oSoMoNackk, so for maas, you changed the version number to 0.x and added an epoch in focal, and removed the package altogether in groovy onwards?12:17
ackkoSoMoN, correct12:31
* cachio afk12:47
mborzeckimvo: https://github.com/snapcore/snapd/pull/9786/ mostly passed, there's a failure in tests/core20/basic but unrelated to the PR13:09
mupPR #9786: tests: backport netplan workarounds from #9785 (2.48) <Run nested> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9786>13:09
mborzeckithe failure is intersting though, become-operational does not complete13:10
mupPR snapd#9775 closed: gadget,o/devicestate,tests: drop EffectiveFilesystemLabel and instead set the implicit labels when loading the yaml <Cleanup :broom:> <UC20> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9775>13:16
pedronismborzecki: I'm getting internal errors from shellcheck :/13:17
mborzeckipedronis: got log? which test?13:17
mborzeckipedronis: or do you mean actual shellcheck is failing?13:17
pedronisI'm getting this13:18
pedronisrunning unit tests in wrappers13:18
pedronismborzecki: https://paste.ubuntu.com/p/QXBgGQxf6j/13:19
pedronismborzecki: it's https://github.com/snapcore/snapd/pull/9779 perhaps ?13:21
mupPR #9779: testutil: make DBusTest use a custom bus configuration file <Created by jhenstridge> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9779>13:21
pedronisI got it only now after merging master13:22
pedronisah, mess13:23
pedronisI have shellcheck installed as a snap13:23
pedronisbut now we can't run snaps from inside our tests I suppose13:24
pedronisit's snap run trying to talk to systemd and failing13:24
mborzeckiheh, as in real system apparently13:25
mborzeckiso why dont i see it too?13:26
pedronisdo you have shellcheck as a snap?13:26
pedronisI'm not even sure why that code gets involved13:27
mborzeckihm i have, maybe it isn't used13:27
mborzeckipedronis: dbustest.go sets DBUS_SESSION_BUS_ADDRESS13:28
mborzeckiin env13:29
mborzeckithat's probably why snap run connects to the session bus (although is the one for tests) and attempts to chat with systemd user session manager13:29
pedronisso, yes, if I got back before that was merged things don't explode13:32
pedronisfor me13:32
pedronisbit of a mess13:35
mborzeckipedronis: https://paste.ubuntu.com/p/33RHztGMJd/ ?13:37
mborzeckii can propose taht13:38
pedronisI suppose, but there should be something easier13:38
pedronisno, let me try something else13:38
mborzeckiwith the diff the tests work for me if they involve shellcheck from a snap13:38
mborzecki(actually had to remove the shellcheck package, somehow it was picked first in the go workspace i have)13:39
pedronismborzecki: I have a shorter change13:39
mborzeckioh, nice, can you propose it?13:40
pedronismborzecki: https://github.com/snapcore/snapd/pull/978813:43
mupPR #9788: testutil: use the original environment when calling shellcheck <Created by pedronis> <https://github.com/snapcore/snapd/pull/9788>13:43
mupPR snapd#9788 opened: testutil: use the original environment when calling shellcheck <Created by pedronis> <https://github.com/snapcore/snapd/pull/9788>13:46
mupPR snapd#9789 opened: many: separate consistency/content validation into gadget.Validate|Content <Cleanup :broom:> <Run nested> <UC20> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9789>13:46
mupPR snapd#9790 opened: gadget: move BuildPartitionList to install and make it unexported <Cleanup :broom:> <UC20> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9790>13:51
mborzeckipedronis: thanks, lgtm and works locally here too13:54
ijohnsonpedronis: mvo: the cloud-init pr is ready to merge, I reviewed all the spread failures they are all unrelated or already fixed in master, see https://github.com/snapcore/snapd/pull/9770#issuecomment-74323136614:44
mupPR #9770: sysconfig/cloudinit.go: add "manual_cache_clean: true" to cloud-init restrict file <Bug> <Run nested> <Simple πŸ˜ƒ> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9770>14:44
zygaafter a day of work on aarch64 desktop the only thing that I find annoying is ports.ubuntu.com vs archive.ubuntu.com, it's surprisingly smooth otherwise14:45
ijohnsonhey zyga14:59
zygahey :)14:59
ijohnsonI noticed you mentioned in your blog post the whole switching between bootloaders on your rpi14:59
ijohnsondid you try dave's new tool for doing just that ?14:59
zygaijohnson, no14:59
ijohnsonI think it's like pibootctl or something14:59
zygaI just dsit-upgraded, I wasn't aware14:59
zygaI know about pibootctl14:59
zygaI think here the problem was that 20.04 desktop and 20.10 desktop have totally different boot setup15:00
ijohnsonit's pretty snazzy I used it for my pi that I upgraded from focal to groovy and eliminated uboot that way15:00
zygaand regular package updates just don't handle the transition15:00
ijohnsonzyga: see https://waldorf.waveform.org.uk/2020/introducing-pibootctl.html15:00
zygaright, he shared that with me15:00
ijohnsonyou can even backup/restore changes to config.txt and such15:00
zygamy point was for simple guide, to avoid debugging, just reinstall15:00
ijohnsonanyways I thought it was a cool tool that worked really well when I used it15:01
ijohnsonbut yes agreed reinstall is much simpler15:01
zygaif you copy the right dbts and config yeah, it will wrok15:01
zygaare you using your groovy pi?15:01
ijohnsonwell I use it headlessly15:01
mupPR snapd#9791 opened: devicestate: implement checkFDEFeatures() <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9791>15:02
ijohnsonI don't use desktop just server15:03
zygaijohnson, I was mainly after a working desktop with 60Hz in 4K15:03
zyganow it works but I had my doubts along the way :)15:03
mvoijohnson: merged and cherry-picked15:04
mupPR snapd#9770 closed: sysconfig/cloudinit.go: add "manual_cache_clean: true" to cloud-init restrict file <Bug> <Run nested> <Simple πŸ˜ƒ> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9770>15:07
mborzeckineed to wrap it up, enjoy your holidays15:25
mvoreviews for 9769 and 9791 would be amazing, it's the last fde-hook bits15:49
* cachio lunch15:52
pstolowskiok, i think i know what's wrong with hotplug tests... it's shockingly simple15:58
pstolowskiand embarassing15:58
zygapstolowski, what is ti?16:04
pstolowskizyga: apt install -y "linux-modules-extra-$(uname -r)"' || true16:05
zyga|| true :D16:05
pstolowskiit seems to be linux-modules-extra-...-generic now16:06
zygakernel vs headers mismatch?16:06
zyganice :)16:06
pstolowskiand a multitude of other variants16:06
zygaI'll gladly review that :)16:06
pstolowskithe above is still fine on old releases16:06
pstolowskineeds to support both i think, testing now... nested is terribly sloooow16:07
ijohnsonwow look at that a green pr16:24
ijohnsona thing of beauty that is16:24
ijohnsonnice work pstolowski !16:24
mupPR snapd#9788 closed: testutil: use the original environment when calling shellcheck <Created by pedronis> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9788>16:27
pstolowskihmm there is more to it but i'm confused16:46
pstolowskiapart from package name change, it doesn't provide ftdi_sio and usbserial  modules for 5.8.0-31-generic kernel; in fact no other package has them16:49
pstolowskithat's 20.1016:49
zygapstolowski, I see this on my pi16:50
zygalinux-modules-5.8.0-1006-raspi: /lib/modules/5.8.0-1006-raspi/kernel/drivers/usb/serial/usbserial.ko16:50
pstolowskizyga: i'm talking about -31- specifically. it see it for older versions too16:51
pstolowskizyga: i mean this: https://pastebin.ubuntu.com/p/WnmkdMmZ7y/16:53
zygayeah, that's weird for sure16:54
zygano idea16:54
pstolowskihmm no fix for nested today i guess :/16:58
pstolowskicachio: ping17:06
pstolowskicachio: the reason nested/classic/hostplug is failing is we are missing ftdi_sio and usbserial modules; we try to install modules-extra in the test (and ignore errors if it's missing), but this package was renamed with -31- kernel. but oddly, these modules are no longer provided by any package for that kernel. i'm stopping investigating this for today; if you have some time or any ideas then feel free to look at it; if not then17:20
pstolowski i'll continue on Monday. in any case, happy holidays!17:20
* pstolowski eow17:20
=== ijohnson is now known as ijohnson|lunch
mupPR snapd#9792 opened: tests: enable ubuntu 21.04 for spread tests <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9792>18:23
=== ijohnson|lunch is now known as ijohnson
mupPR snapd#9737 closed: tests: use os.query tool instead of comparing the system var <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9737>22:29
mupPR snapd#9793 opened: tests: add os query commands for subsystems and architectures <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9793>22:29
=== King_InuYasha is now known as Conan_Kudo
=== Conan_Kudo is now known as King_InuYasha

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