mardymborzecki: hi!05:47
mborzeckimardy: heya05:57
mborzeckihmm hmm06:40
zyga-mbpgood morning06:42
zyga-mbpmborzecki one thing I miss about spain is that at night, it's still 21C06:42
mborzeckimeh, lp builds are unhappy about go module06:54
mupPR snapcraft#3578 opened: Fix: files starting with '.' are not organized and also not caught by filesets <Created by debdutdeb> <https://github.com/snapcore/snapcraft/pull/3578>06:55
mborzeckireproduced the build problem from lp at least :)06:58
mvomborzecki: \o/07:00
mvomborzecki: any more info yet?07:01
mborzeckimvo: hey, so i think we're missing this: https://paste.ubuntu.com/p/BkGRc5QQYW/07:01
mborzeckimvo: also github.com/boltdb/... is listed in go.mod but does not really appear in vendor directory anywhere07:01
mborzeckimvo: do you remember why we have a fork of boltdb?07:03
mvomborzecki: do you know why it started just now?07:03
mvomborzecki: iirc to fix build problems on trusty, let me quickly check07:03
mvomborzecki: build on 32bit systems and riscv07:04
mvomborzecki: https://github.com/snapcore/bolt/commits/master07:04
mborzeckimvo: afaiu lp builders run with disabled network, our go.mod listst github.com/boltdb/.. and thus go build attempts to fetch the module07:04
mvomborzecki: ohhhhh, that makes sense. wonder why it ever worked but maybe it never did inside a buildd07:05
mborzeckimvo: maybe we could just replace that bolt with our fork using https://golang.org/ref/mod#go-mod-file-replace for deb builds?07:05
mvomborzecki: I think so, iirc ian had something like this07:05
mvomborzecki: https://github.com/snapcore/snapd/pull/10559/files#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6R36 <- looks like not for bolt but he looked at it for other things07:06
mupPR #10559: many: move to go modules  <⛔ Blocked> <Created by anonymouse64> <Closed by anonymouse64> <https://github.com/snapcore/snapd/pull/10559>07:06
mvomborzecki: thanks *so* much for finding this07:06
mborzeckimvo: i'll open a pr in a bit07:06
mupPR snapd#10609 closed: interfaces/desktop: mount system fonts via app desktop slot <Created by MarcusTomlinson> <Closed by jhenstridge> <https://github.com/snapcore/snapd/pull/10609>07:17
mupPR snapd#10753 opened: interfaces/desktop: allow snaps to provide a desktop slot, granting access to system fonts <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/10753>07:17
mardyhi zyga-mbp, mvo, pstolowski 07:30
zyga-mbphey :)07:30
zyga-mbpimpish is close to release07:30
zyga-mbpI remember before snaps, everything was very stressful at this time07:30
zyga-mbpI guess now it's more of a mixed set, with some things still having to make it on time07:31
zyga-mbpbut some things having the luxury of releasing on time when they are ready07:31
mardydo you guys see any value in this? https://github.com/snapcore/snapd/pull/10751 I started it yesterday, but today I seem to realize that it's already relatively easy to mock the snapd responses, so maybe I should drop it.07:37
mupPR #10751: client: turn the Client type into an interface <Created by mardy> <https://github.com/snapcore/snapd/pull/10751>07:37
mborzeckimvo: heh so not sure what to make of it, snapd imports snapcore/bolt which has some *_test.go files that improt boltdb/bolt and that makes it an indirect dependency, adding a replace directive confuses go as there are 2 modules doing the same now?07:56
mborzeckimvo: also, we're not setting GO111MDOULE so dh_golang defaults to GO111MODULE=off07:56
mborzeckibut we do pass -mod vendor sometimes?07:56
mvomborzecki: ohhh, maybe we need to just set go111module globally in the debian build rules then? 08:04
mvomborzecki: sorry that I did not explore that, between the different dh-golang versions I got lost a bit but the env might be enough to give us the desired behavior08:04
mvoI wonder if we can somehow make the package build inside spread networkless, if there is a clever "nsenter" to archive that08:06
mborzeckimvo: yeah, that's what im doing right now, added unshare -n around all calls to rpmbuild/debuild/makepkg08:27
mvomborzecki: \o/08:28
mvomborzecki: should I fix the testfiles in boltdb ? or are you on it alredy?08:28
mborzeckimvo: added something like this locally: https://paste.ubuntu.com/p/Csg5JpZTZH/08:29
mborzeckimvo: but i guess the imports need to be fixed anyway as replace are package local iirc08:30
mupPR snapd#10754 opened: tests/lib/prepare-restore: make sure that package build is done without network access <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10754>08:47
mborzeckimvo: ^^08:48
mvomborzecki: nice08:48
sil2100pstolowski, mvo: hey hey! Just poking around to see if there's any progress with the snapd focal riscv64 situation? Is there an upload planned for temporarily skipping the failing tests?08:53
pstolowskisil2100: my PR for disabling the failing suite on riscv landed yesterday in master08:54
mupPR snapd#10717 closed: tests: fix interfaces-libvirt test <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10717>08:57
pstolowskisil2100: it was https://github.com/snapcore/snapd/pull/1073608:58
mupPR #10736: tests: skip overlord tests on riscv64 due to timeouts <Simple 😃> <Skip spread> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/10736>08:58
mardyis it possible for a snap to control (stop, start, restart) its own services, without needing the user to be root? ("snapctl stop", for example, requires root)09:00
zyga-mbpmardy IIRC yes09:24
zyga-mbpmardy snapctl is accessible from the inside of the snap09:24
sil2100pstolowski: do you know when this change will be uploaded to focal?09:29
mborzeckimvo: meh, as expected the deb builds are failing without network09:30
zyga-mbpmborzecki module "fun"?09:32
pstolowskisil2100: dunno, mvo may know09:32
mborzeckizyga-mbp: yeah unforunately09:32
zyga-mbpit's a really hard problem :/09:33
zyga-mbpI have a path to fix that in my yocto builds but nothing fixed yet09:33
zyga-mbpthe complete module system is elaborate and complex09:33
mborzeckizyga-mbp: also dh-* is really getting into the way09:33
mardyzyga-mbp: there's a check there, and the stop, start and restart commands are allowed only for root: https://github.com/snapcore/snapd/blob/master/overlord/hookstate/ctlcmd/ctlcmd.go#L148-L15009:39
mvosil2100: uh, right, sorry, I wanted to do it today/yesterday but 360 :(09:43
zyga-mbppopsulem sobie system10:20
zyga-mbppstolowski uzywasz czegos do kopii zapasowych wirtualek?10:20
pstolowskizyga-mbp: did you mean it for #snappy? ;)10:21
zyga-mbp pstolowski bad channel :)10:22
mupPR snapcraft#3579 opened: snap: move base to core20 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3579>10:51
sil2100mvo: no worries! That reminds me... need to fill in out my 360!11:27
mborzeckiheh  cryptic errors: `devicemgr: cannot update revisions after boot changes: empty snap file name`12:15
mupPR snapd#10743 closed: tests: fix fakedevicesvc service already exists <Simple 😃> <Run nested> <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10743>12:43
mupPR snapd#10755 opened: client: introduce mocking of the Client interface <Created by mardy> <https://github.com/snapcore/snapd/pull/10755>12:58
mvomborzecki: should I commit the boltdb imoport fixes or are you on it alredy?13:32
mborzeckimvo: please do if you can, still fighting managers test suite13:32
mvomborzecki: sure, doing this now13:32
mvomborzecki: fix pushed13:41
mvomborzecki: let's see if it helps :)13:41
mborzeckii'm confused, did old model assertions not require a "base" header?13:41
mupPR snapd#10756 opened: cmd/snap: only log translation warnings in debug/testing <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10756>14:03
mardyI very welcome reviews on https://github.com/snapcore/snapd/pull/10628, if someone has a few minutes to spare14:41
mupPR #10628: usersession/xdgopenproxy: move PortalLauncher class to own package <Squash-merge> <Created by mardy> <https://github.com/snapcore/snapd/pull/10628>14:41
pstolowskicachio: could https://github.com/snapcore/snapd/pull/10561 land now (once reviewed)?14:45
mupPR #10561: tests: revert revert manuall lxd removal <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10561>14:45
pstolowskior are we still waiting for lxd fix?14:45
cachiopstolowski, thanks, the fix is already applied, so it is good time to land this one14:53
pstolowskicachio: ok, +1 from me14:53
pstolowskineed 2nd review for https://github.com/snapcore/snapd/pull/1072515:58
mupPR #10725: o/hookstate: require snap-refresh-control interface for snapctl refresh --proceed <Needs Samuele review> <Refresh control> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10725>15:58
cachioijohnson[m], hi, I see the recover issue again on uc2016:54
ijohnson[m]cachio: did you see it with the debug kernel command line branch or with what's on master?17:05
cachioijohnson[m], I saw that on maste17:08
cachionow I am running tests to see if I can reproduce it and get some extra info17:08
cachioI am using your pr17:08
=== E_Eickmeyer is now known as Eickmeyer
ledhedI have some questions about Full Disk Encryption on Ubuntu Core.  Is this the right place to ask, or is there a more appropriate support channel?18:56
ijohnson[m]ledhed yeah this is the right place 18:57
ijohnson[m]Either here or the snapcraft forum on the channel header 18:57
ledhedthanks,   I have a project coming up and would like to use Ubuntu Core, but my understanding is that FDE only works (at least out of the box) on certified devices.  The hardware I need for my project (CompuLab Fitlet2, AMD64) appears to have all the requirements to support FDE. I'm just not clear if I can build my own image and enable FDE, or if19:00
ledhedFDE is strictly for certified devices and I would need to get the Fitlet2 certified.19:00
ijohnson[m]ledhed FDE for amd64 platforms works out of the box on devices that meet the requirements, i believe the fitlet2 meets the requirements 19:02
ledhedwhen I run: ls /dev/disk/by-partlabel/ | grep enc nothing is returned. Which implies its not encrypted.19:05
ledhedI checked the bios and confirmed that Secure Boot is enabled, as is the TPM19:06
ledhedI was able to get it to encrypt the disk, seems I had to clear the TPM.  The problem I'm running into now is that after it encrypts the disks, on the next boot it asks for the recovery key. Then errors out and says "cannot activate with TPM sealed key (cannot unseal key: invalid data file: cannot complete authorization policy assertions: cannot19:52
ledhedcomplete OR  assertions: current session digest not found in policy data) and activation with recovery key failed (cannot obtain recovery key: /usr/sbin/systemd-ask-password failed: exit status 1)"19:52
* cachio afk20:26
mwhudsonhello, what version of go is the snapd snap built with?22:41
ijohnson[m]mwhudson 1.13 currently22:42
mwhudsonfrom my snap or do we have that in the archive for xenial now?22:43
mwhudson(it is still built on xenial, right? is there a plan to change that?)22:43
mwhudsonah yeah we do have 1.13 on xenial22:43
mwhudsonijohnson[m]: anyway https://bugs.launchpad.net/snapd/+bug/194307722:56
mupBug #1943077: snapd fails to autopkgtest on mksquashfs, which is looking for libgcc_s <update-excuse> <snapd:New> <openssh (Ubuntu):New> <squashfs-tools (Ubuntu):New> <https://launchpad.net/bugs/1943077>22:56
dbungertif you have questions, I filed that monstrosity22:57
dbungertthe summary would be "snapd seems to need libraries outside of itself to pass autopkgtest"22:58
mwhudsondbungert: i think the fix is pretty trivial luckily23:10
mwhudsondbungert, ijohnson[m]: to whit https://github.com/snapcore/snapd/pull/1075723:12
mupPR #10757: build-aux: stage libgcc-s1 library into snapd snap <Created by mwhudson> <https://github.com/snapcore/snapd/pull/10757>23:12
mupPR snapd#10757 opened: build-aux: stage libgcc-s1 library into snapd snap <Created by mwhudson> <https://github.com/snapcore/snapd/pull/10757>23:15
dbungertFailed to fetch stage packages: Error downloading packages for part 'libc6': The package 'libgcc-s1' was not found..23:15
mwhudsonoh really23:17
dbungertmwhudson: libgcc1?23:17
mwhudsonyes probably23:18
mwhudsonok force pushing that23:20
mwhudsonpackage name changed somewhere between bionic and focal it seems23:21

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