/srv/irclogs.ubuntu.com/2020/08/13/#snappy.txt

ElladanI installed a snap named "arduino" on Mint 19.3, and when I start it it just crashes with: "java.lang.UnsatisfiedLinkError: /snap/arduino/41/usr/lib/jvm/java-11-openjdk-amd64/lib/libsplashscreen.so: libX11.so.6: cannot open shared object file: No such file or directory"00:16
ElladanAnother snap I have installed using X11 works fine.00:17
jameshElladan: The "snap info" output for that snap suggests reporting problems at https://github.com/snapcrafters/arduino/issues03:31
jameshI haven't used that snap, so don't have much more to suggest.03:33
oerhekshowcome you could install snaps on mint?03:34
oerheksthey blocked that in a funny way03:35
jameshoerheks: they replaced some "deb to snap" transition packages in their archive overlay.  I don't think they've done anything to actively break snap support if you install it03:36
jameshunless there's something else new.03:36
oerheksThey introduced this etc/apt/preferences.d/nosnap.pref file03:37
jameshAh.  I missed that.03:40
jameshPresumably if the user managed to install a snap, then they would have got past that setting though.03:41
oerheksfirst remove that blob, then you could install snapd,03:41
=== JanC is now known as Guest56745
=== JanC_ is now known as JanC
mborzeckimorning06:19
mupPR snapd#9144 closed: bootloader: add helper for creating a bootloader based on gadget <Squash-merge> <UC20> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9144>06:37
mborzeckimvo: hey06:43
mborzeckiomg, forgot about squash merge label i added myself06:44
mvomborzecki: good morning06:44
mborzeckimvo: we're not doing $kernel after all?06:46
mupPR snapd#9130 closed: [RFC] Support for gadget.yaml "$kernel:" style references (1/N) <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/9130>06:47
mupPR snapd#9142 closed: sandbox/cgroup: detect dangling v2 cgroup <Bug> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9142>06:47
mupPR snapd#9147 closed: [RFC] Support for gadget.yaml "$kernel:" style references (2/N) <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/9147>06:47
mupPR snapd#9148 opened: gadget: add new Kernel{Info,Asset} struct and helpers <Created by mvo5> <https://github.com/snapcore/snapd/pull/9148>06:57
mborzeckipstolowski: hey07:04
mvogood morning pstolowski07:04
pstolowskimorning o/07:04
mborzeckipedronis: hey, thanks for the feedback07:57
mborzeckipedronis: i've updated #914508:59
mupPR #9145: boot: track trusted assets during initial install, assets cache  <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9145>08:59
pedronismborzecki: thx, I will look in a little bit (was in meetings)09:01
mborzeckipedronis: sure, let me know if we should have a chat as well09:01
* mborzecki goes looking for a copy-tree helper09:01
mborzeckizyga is coming back and should be online in a bit09:18
zyga-mbpwhat a day09:35
zyga-mbpmvo I will file half day off,09:35
zyga-mbpthere was a leak at my parent's flat, someone upstairs from them had a broken pipe or something09:35
zyga-mbpI need to spend a moment to file the insurance claim09:36
zyga-mbpI'll be back around noon09:36
mvozyga-mbp: ok09:37
zyga-mbphow is master? are we green?09:39
mborzeckizyga-mbp: yeah, the PRs i restarted today were green09:41
mborzecki(as in all green)09:42
zyga-mbpsuper-green :)09:42
mupPR snapd#9129 closed: sandbox/cgroup: remove temporary workaround for multiple cgroup writers <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/9129>09:42
mborzeckizyga-mbp: #28a745 green09:43
mupPR #28: Updated import path, closes #27 <Created by fgimenez> <Merged by elopio> <https://github.com/snapcore/snapd/pull/28>09:43
zyga-mbp"super green" is a quote, from, I think, the 5th element09:43
mborzeckihm now that you reminded me, i'd love to watch it again09:44
zyga-mbpok , paperwork ready09:50
zyga-mbpeh09:50
zyga-mbplet's focus on work now09:50
mupPR snapd#9118 closed: tests: detect unexpected xenial kernels <Test Robustness> <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/9118>09:52
pedronismborzecki: do we need to discuss something?09:58
mborzeckipedronis: idk, do you want to go over the PR maybe?09:59
mborzeckipedronis: otherwise we're good i suppose10:00
mborzeckipedronis: btw. we haven't discussed it, but we should probably add boot.SealWithModeenv() or similar at some point10:01
pedronismborzecki: I'm sure I have mentioned it many times10:02
pedroniswithout a name yet10:02
mupPR snapd#9149 opened: [RFC] gadget: provide new gadget.ResolveContentPaths() (2/N) <Created by mvo5> <https://github.com/snapcore/snapd/pull/9149>10:02
pedronisIsaid we need functions that seal/reseal based on a modeenv (or at least to start a function that takes a modeenv and produces sealing params)10:03
mborzeckipedronis: ok, i'll add a stub when i open the bits with content update observer10:03
pedronismborzecki: please coord with Claudio on this10:03
mborzeckipedronis: yup, will do10:04
pedronismborzecki: a few more comments10:12
mvomborzecki, pedronis I'm moving kernel.yaml reading into the kernel package now, but one open question is the edition, it's private to gadget right now, so I would either need to copy or extract it10:13
mvomborzecki, pedronis (the edition datastructure) - any recommendations for this?10:13
pedronismvo: gadget will to import kernel, right?10:15
mvopedronis: yes10:15
pedronis*will have to10:15
pedronismvo: it's annoying, but I recommend making a tiny gadget/edition package10:16
pedroniswith edition.Number10:16
mvopedronis: works for me, thanks10:16
mborzeckior maybe we could move editionNumber to snap?10:16
pedronisnot unless we use for a 3rd thing10:16
mborzeckifair enough10:16
pedronisI see the appeal, but snap is already full of things10:17
pedronisand snap/edition is not better than gadget/edition10:17
mborzeckimissing a kitchen sink ;)10:17
zyga-mbpmborzecki snap/mime and we're close to sending email10:28
mupPR snapd#9148 closed: gadget: add new Kernel{Info,Asset} struct and helpers <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/9148>10:38
mvopedronis, mborzecki updated the kernel yaml stuff in 9150, sorry I got confused by GH10:52
mupPR snapd#9150 opened: gadget,kernel: add new kernel.{Info,Asset} struct and helpers <Created by mvo5> <https://github.com/snapcore/snapd/pull/9150>10:53
zygabrb11:11
mborzeckipedronis: hm, i think we've forgotten to support a forced bootloader in ForGadget, anyways we can remedy that later11:13
pedronismborzecki: I'm sure it will be obvious when you need it :)11:20
zygamost tests are red, are we in another wave of problems or are those just old?11:21
pedronisI don't know if people merged master in their PR, some of michael stuff has vet/fmt problems11:26
pedronisit seems some uc20 prepare are timing out11:27
pedronismborzecki: uc20 prepare is timing out here: https://github.com/snapcore/snapd/pull/9145, real problem?11:28
mupPR #9145: boot: track trusted assets during initial install, assets cache  <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9145>11:28
mborzeckihmmm11:29
mborzeckiinteresting, let me check locally in a vm11:30
pedroniszyga: can you point to an interesting bit of history in 7285 that you think we don't have in master atm11:35
zygapedronis: I think I can but it will take me a moment, there's a lot of discussion there and resulting patches11:36
pedroniszyga: well the discussion is in the PR, not the commits11:36
pedronisthere's a lot of back and forth that has no footprint in master11:36
pedronisat this point11:36
zygayes but the comments contain responses in code commits11:37
pedronisbut I'm not sure that's a bug11:37
mborzeckipedronis: well, pushed mocking of for gadget bootloader too to 914511:49
mborzeckihm the core-20 spread tests don't use a tpm & secboot, do they?11:51
pedronismborzecki: not those ones11:52
pedroniszyga: I found only https://github.com/snapcore/snapd/pull/7825#discussion_r41126950411:52
mupPR #7825: many: use transient scope for tracking apps and hooks <Needs security review> <Security-High> <⛔ Blocked> <Created by zyga> <https://github.com/snapcore/snapd/pull/7825>11:52
mborzeckiwierd, anyways snap prepare-image is downlaoding all snaps, we'll know what's wrong in a bit11:52
* zyga debugs something as well11:59
mborzeckipedronis: hm looks like Observe is called with a nil structure12:11
pedronisinteresting12:16
pedronisseems some preexisting bug in gadget though?12:17
mupPR snapd#9151 opened: vendor: update secboot to fix key data validation <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/9151>12:18
mborzeckipedronis: unclear, gadget unit tests assert that structure is non-nil12:19
pedronissome weird case of := vs =12:19
pedronisthat's a way to get nil where you don't expect one staring at the code12:20
cmatsuokamborzecki: did an impossible situation happen in tests?12:21
pedronismborzecki: the other option is something wrapping a nil into an interface, and the thing going past != nil12:22
mborzeckihm actually it's more intersting12:23
mborzeckiobserver is nil12:23
pedronisah12:23
mborzeckifunc (o *TrustedAssetsInstallObserver) Observe(op gadget.ContentOperation, affectedStruct *gadget.LaidOutStructure, root, realSource, relativeTarget string) (bool, error) <- so o is nil12:23
mborzeckibut it's called from a place where it's not nil12:24
pedroniswell, see my comment about wrapping nils into interfaces12:24
pedronisthat might be one reason12:24
pedronismborzecki: ah, yes12:25
pedronisfun12:25
ijohnsonmorning folks12:25
cmatsuokahey ijohnson12:25
mborzeckiijohnson: hey12:25
ijohnsonhey cmatsuoka mborzecki12:25
pedronismborzecki: the issue is probably the signature of TrustedAssetsInstallObserverForModel12:25
pedronisyou probably take that and cast implicitly to an interface12:25
pedronisthat doesn't give you a nil for the interface12:26
pedronismborzecki: does this make sense?12:26
ijohnsonpedronis: regarding making recover -> run mode automatic via a reboot, I got the initramfs method working easily ... if I base the changes on top of my other 2 initramfs PR's (systemd-mount and cross-check) the test diff is small, otherwise there are a number of conflicts I'd rather not resolve (but could if needed)12:29
pedronisijohnson: yes, I saw your notes, let's chat at the standup12:30
ijohnsonpedronis: I am going to take a quick stab at the ensure devicemgr method we discussed now to see how much work that is12:30
ijohnsonpedronis: ack12:30
mborzeckipedronis: yes, most likely, https://play.golang.org/p/UWyducWpRj712:31
mborzeckimissing test case in handlers_install too12:32
pedronisyes12:33
pedronisfunction returning nil for structs are dangerous12:33
pedronismborzecki: I'm happy to chat about how to clean this up, I think both me and mvo made some comments about those functions originally12:34
mborzeckipedronis: and the unit tests check for NotNil with an interface12:34
zygahuh12:38
mborzeckipedronis: i'm open to suggestions, a chat?12:40
pedronismborzecki: yes12:40
mupPR snapd#9152 opened: cmd/snapd-generator: generate drop-in to use fuse in container <Needs Samuele review> <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9152>13:18
cachiozyga, to reproduce the error you can use this image https://storage.googleapis.com/spread-snapd-tests/images/pc-amd64-16-beta/pc.img.xz13:37
zyga-mbpcachio sorry, which error?13:37
cachiozyga-mbp, the reloading with --user13:38
zyga-mbpI see13:38
zyga-mbpdo you know which commit corresponds to that?13:38
zyga-mbpI don't need to look at the image, more at the history13:38
zyga-mbpwe can look at the image once we know where it is anchored13:39
cachioI am running eith the branch for beta13:39
cachio2.4613:39
zyga-mbpok, I'll check13:40
zyga-mbpit probably doesn't have the fixes then13:40
zyga-mbpso unlucky fork13:40
* pstolowski lunch13:55
pedroniszyga-mbp: if you can update the description of #7825 with something that hints at the journey and then ping me13:57
mupPR #7825: many: use transient scope for tracking apps and hooks <Needs security review> <Security-High> <⛔ Blocked> <Created by zyga> <https://github.com/snapcore/snapd/pull/7825>13:57
zygapedronis: sure13:57
ijohnsonzyga: so what I was trying to say in the SU is that I don't think the extra info from mount helps because if the fsck fails or the mount fails, it shows up the same to us when we run `systemd-mount`13:58
zygapedronis: I need to prepare for another call so likely in the evening or tomorrow13:58
zygaijohnson: I see13:58
ijohnsoni.e. systemd-mount doesn't tell us how things fail it just says that the unit failed13:58
ijohnsonand then we have to go on digging to find out what failed13:58
mborzeckiijohnson: what if we start the fsck unit ourselves?13:59
ijohnsonmborzecki: yes we could do this13:59
mborzeckipedronis: https://github.com/snapcore/snapd/pull/915314:02
mupPR #9153: overlord/devicestate: workaround non-nil interface with nil struct <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9153>14:02
mupPR snapd#9153 opened: overlord/devicestate: workaround non-nil interface with nil struct <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9153>14:03
* zyga breaks for lunch14:05
mborzeckihmm should have skipped spread there14:08
* zyga waits for food to heat so https://github.com/snapcore/snapd/pull/915414:16
mupPR #9154: tests: unmount FUSE file-systems from XDG runtime dir <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/9154>14:17
mupPR snapd#9154 opened: tests: unmount FUSE file-systems from XDG runtime dir <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/9154>14:19
mborzeckiijohnson: can you take a look at https://github.com/snapcore/snapd/pull/9153 ?14:21
mupPR #9153: overlord/devicestate: workaround non-nil interface with nil struct <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9153>14:21
ijohnsonmborzecki: yes14:22
mborzeckiijohnson: thanks!14:22
mvoijohnson: I just did14:50
mvomborzecki: just had a look14:50
mborzeckimvo: ijohnson: thanks!14:50
mvomborzecki: I will merge once spread is done14:50
mborzeckicmatsuoka: thanks too!14:51
ijohnsonhaha wow 3 reviews in under a minute14:51
mborzeckihaha14:51
ijohnsonmborzecki's code is real popular14:51
mvowoah14:51
cmatsuoka:)14:51
mvowhat a shinny PR :)14:51
ijohnson#8931 could use a review too and is a test-only fully green 31 line change :-)14:53
mupPR #8931: tests/main/install-fontconfig-cache-gen: enhance test by verifying, add fonts to test <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8931>14:53
mvoijohnson: sure, looking14:53
mvoijohnson: sorry, somehow missed this one (or rather missed that I can review it again)14:54
ijohnsonno worries14:54
* cachio lunch14:56
ijohnsonthanks mvo14:59
mborzeckihmm all jobs are queued?15:03
mborzeckinothing in progress here https://github.com/snapcore/snapd/actions?query=is%3Ain_progress15:03
zyga-mbpmborzecki looking15:15
zyga-mbpmborzecki we're executing 32 spread runs now15:16
zyga-mbpit's a message queue, perhaps under load and out of date15:16
zyga-mbpwe're definitely not idle15:16
mborzeckihm maybe15:16
zyga-mbpactually some more as I run 4 more locally for extra safety in case one node goes down15:18
zyga-mbpoh15:18
zyga-mbpcmatsuoka I found that doc about actions15:18
cmatsuokazyga-mbp: ah nice, please share if you can15:22
zyga-mbpdone15:23
cmatsuokathanks!15:23
zyga-mbpI'm happy to talk if you want to set it up15:26
mupPR snapd#9017 closed: o/snapstate: disk space check in Ensure (5/N) <Disk space awareness> <Needs Samuele review> <⛔ Blocked> <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/9017>15:49
zygaErr:125 http://us-east1.gce.archive.ubuntu.com/ubuntu groovy/main amd64 libio-pty-perl amd64 1:1.12-115:56
zyga  Temporary failure resolving 'us-east1.gce.archive.ubuntu.com'15:56
mupPR snapcraft#3247 opened: spread tests: fix classic patchelf linker regex to match all arches <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3247>16:04
mvozyga: if you could look at 9152 too that would be great, I think system() is fine here but I would love someone that is not me to come to the same conclusion :)16:06
zygasure16:06
zygalooking16:06
mvopstolowski: some interssting points in the generator PR from dimitri16:06
mvozyga: thank you!16:06
pstolowskimvo: yes, just looking16:06
mvopstolowski: \o/16:07
mborzeckimeh, i'll grow more grey hair waiting for 9153 spread jobs to finish16:10
mborzeckiit's looking good though, so bbiab16:11
zygareviewed https://github.com/snapcore/snapd/pull/9152#pullrequestreview-46691366316:17
mupPR #9152: cmd/snapd-generator: generate drop-in to use fuse in container <Bug> <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9152>16:17
mupPR snapd#9153 closed: overlord/devicestate: workaround non-nil interface with nil struct <UC20> <Created by bboozzoo> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/9153>16:59
mupPR snapd#9154 closed: tests: unmount FUSE file-systems from XDG runtime dir <Test Robustness> <Created by zyga> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/9154>16:59
mborzeckiyay, cmatsuoka thanks!17:13
ijohnsonughhhhhh I spent like all week trying to understand why the user wasn't added to this core-initrd nested spread test, turns out the code was expecting the user1 user, but I was creating the ubuntu user17:20
ijohnson🤦17:20
mborzecki#9145 is now unblocked17:23
mupPR #9145: boot: track trusted assets during initial install, assets cache  <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9145>17:23
mborzeckiijohnson: uhh, kind of reminds me of snapstate tests, fakestore and assumptions about snap names that aren't obvious17:25
ijohnsonmborzecki: yeah lots of copying and surgical extraction of code which has assumptions that aren't obvious makes simple things more frustrating, just glad I finally figured it out17:39
pedronisijohnson: thanks for the review, no I think adding bootState in that interface is fine18:33
ijohnsonpedronis: ack I can push that change up for you and add that comment18:33
pedronisijohnson: thx, that would be nice18:33
ijohnsonsounds good18:33
ijohnsonpedronis: need anything else from me before you EOD/EOW ?18:34
pedronisdon't think so, I'm trying to wrap (as I said) the next PR for validation sets18:34
ijohnsonI chatted with mvo and we will work on landing the open uc20 PR's I have then I will work on cloud-init for uc20 after those have landed as my next big uc20 thing18:34
pedronisijohnson: ok, do you need to rediscuss quickly what's the plan for that?18:35
ijohnsonthat's not a bad idea, give me a minute to look at the doc/matrix again to refresh my memory18:35
pedronisyea, I'm not sure it completely reflects what we agreed after we started fixing the bug18:36
* cachio afk18:36
ijohnsonpedronis: ok, ready when you are I can join the SU HO18:39
pedronislet me join it18:40
mupPR snapd#9155 opened: asserts/snapasserts: introduce ValidationSets <validation-sets :white_check_mark:> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9155>19:10
mupPR snapd#9151 closed: vendor: update secboot to fix key data validation <Simple 😃> <UC20> <Created by cmatsuoka> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/9151>19:15
mupPR snapcraft#3247 closed: spread tests: fix classic patchelf linker regex to match all arches <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3247>19:49
mupBug #1891564 opened: Fonts are not displayed correctly <Snappy:New> <https://launchpad.net/bugs/1891564>22:39
mupBug #1891564 changed: Fonts are not displayed correctly <Snappy:New> <https://launchpad.net/bugs/1891564>22:42
mupBug #1891564 opened: Fonts are not displayed correctly <Snappy:New> <https://launchpad.net/bugs/1891564>22:51
mupPR snapcraft#3244 closed: file utils: introduce get_host_tool_path() to find commands on host <enhancement> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3244>23:40

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