=== AmarokNelg_ is now known as AmarokNelg
zygagood morning! :)07:26
* zyga looks at a mountain of email07:27
mborzeckizyga: hey07:30
zygahey :)07:31
zygahappy new year07:31
mborzeckilikewise :)07:32
mborzeckizyga: did you enjoy your break?07:33
zygamborzecki: yes, it almost feels weird to go back after so long07:35
mborzeckii would call it a shock, rather than just being weird :)07:35
kalikianagood morning07:46
kalikianahappy 201807:46
zygagood morning kalikiana!07:52
mborzeckipstolowski: hey08:11
zygagood morning koza08:19
* zyga reboots for some updates08:31
mborzeckimvo: hey08:44
mupIssue snapcraft#1837 closed: Can't run snapd on Codenvy 5.22.0 <Created by 20avva> <Closed by kalikiana> <https://github.com/snapcore/snapcraft/issue/1837>08:44
mvohey mborzecki! how are you?08:47
mvomborzecki: did I miss anything interessting this morning :) ?08:47
mborzecki(unsurprisingly) the morning has been rather slow08:47
mborzeckii'm poking around yocto support, slow progress however08:48
* mvo nods08:48
mvomborzecki: yocto> nice to hear08:48
mborzeckipromised this guy https://forum.snapcraft.io/t/yocto-rocko-core-snap-kernel-panic/3261/11 to look into it once i get back :)08:49
* mvo nods08:49
mupIssue snapcraft#1834 closed: Feature Request: NGINX plugin <Created by jamesbeedy> <Closed by kalikiana> <https://github.com/snapcore/snapcraft/issue/1834>08:50
mupPR snapd#4427 opened: snap: fix snap find " " output <Created by mvo5> <https://github.com/snapcore/snapd/pull/4427>09:03
mborzeckizyga not around?09:22
mvomborzecki: heh09:22
zygahey mvo :) long time no see09:24
zygahow are you doing?09:24
mborzeckizyga: https://github.com/snapcore/snapd/blob/master/cmd/snap-confine/mount-support.c#L589 afaiu /root is assumed to always exist then?09:29
zygamborzecki: yes, there's a lot of directories we just assume09:29
zygamborzecki: for /root we could skip it if absent09:29
mborzeckii'm asking as it's not here in poky (yocto build), at least with rocko brach (the last release)09:30
mvohey zyga ! good to see you, happy new year09:30
mborzeckifrankly, i don't recall if it used to be there or now, never needed to do/keep anything around in /root09:30
clienthax__226▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ ANOTHER EXPLOIT HAS BEEN FOUND IN WEECHAT ONE THAT SENDS YOUR LOGS TO THE DEVS IP ADDRESS!!jmnaxjlu: ubot9 hurricanehrndz verterok ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄09:34
* kalikiana coffee break09:36
=== __chip__ is now known as Chipaca
zygahey Chipaca09:44
Chipacahow's things?09:44
zygaChipaca: snappy :)09:44
zygaChipaca: still waking up/getting into coding spirit (code reviews now)09:45
zygahow are you doing? are you feeling better?09:45
mvohey Chipaca! welcome back, great to "see" you09:45
Chipacamvo: o/09:45
Chipacazyga: i caught a lot of sun, and a little head cold, but overall feeling good. you?09:45
zygaChipaca: I'm doing great, disappointed for not having much snow this xmas09:46
Chipacamvo: likewise, how're you doing?09:47
mupPR snapd#4428 opened: daemon: return "bad-query" error kind for store.ErrBadQuery <Created by mvo5> <https://github.com/snapcore/snapd/pull/4428>09:47
Chipacazyga: snow is nice but i'm happy with my choice :-)09:47
zygaChipaca: did you travel somewhere warmer for your holidays/09:48
mvoChipaca: in general great, today in particular not so great, caught a stomach bug yesterday and still feel weak from that. but hacking beats lying in bed and being bored :)09:48
Chipacamvo: i'll preted not to know half of your hacking today is from an unconventional seat09:49
Chipacazyga: visiting friends on the costa d'azur09:50
zygaChipaca: stdout flushes manually, stderr flushes on output ;-)09:50
* zyga stops thinking about IO jokes09:50
zygaChipaca: ah, lovely09:50
zygaChipaca: I've visited some friends from Catalonia but we chose to go to the capital of coal in poland (katowice) instead :)09:51
Chipacai've only run trains through the places you visit09:51
Chipacamvo: is the client side of the bad query kind coming, or not needed?09:54
mvoChipaca: I can add it,  I think it makes sense, I was mostly doing it for roberts glib based client implementation09:58
mvo(where it is not needed AIUI) - but for symmetry it does make sense09:59
Chipacamvo: if we're not going to use it, it's just bitrot waiting to happen09:59
Chipacamvo: (otoh all the kinds could use a reshuffle to not be defined twice...)10:00
* Chipaca does not bring new wisdom into the new year10:00
mvoChipaca: lol10:00
mvoChipaca: all good, I will push the client side and make a nice(r) error message for `snap find "()!§$§%§%*"`10:00
mvoChipaca: something like "error: cannot list snaps: bad query, please use less special chars"10:01
zygamvo: "please use latin letters or digits" maybe10:01
Chipacaoh you just reminded me i probably need to change this error message before submitting it as a PR10:02
mvozyga: yeah, thats better10:02
* zyga looks at the intense fog outside10:02
ChipacaErrAppEscapes   = errors.New("snap is unusable due to fuckery")10:02
mvoChipaca: *cough* this is a family channel!10:02
zygamvo: or arabic digits ;-)10:02
Chipacamvo: /o\ sorry10:03
mvozyga: maybe ascii letters/digests10:03
* mvo hugs Chipaca 10:03
zygaChipaca: go and stand in the corner now ;;-)10:03
zygamvo: yeah10:03
* Chipaca hugs mvo from the corner10:03
zygawe have a lot of PRs10:04
mvozyga: my fault10:04
zygamvo: how is that a fault? :)10:04
zygamvo: thank you for all the nice fixes10:04
mvozyga: I was doing bug triage and there is soooo many low hanging fruits10:04
zygacan I get a review on https://github.com/snapcore/snapd/pull/431510:04
mupPR #4315:  cmd/snap-update-ns: add execWritableMimic <Created by zyga> <https://github.com/snapcore/snapd/pull/4315>10:04
zygait's from _november_ and has one review10:05
* zyga boots subsequent VM for the round of updates10:05
Chipacahow's travis/linode/spread behaving?10:06
zygaI haven't tried anything new yet, I see lots of red PRs though10:06
zygaso maybe it's still having a hangover10:06
Chipacazyga: one of the things we realised towards the year end was that the interfaces-many was taking a relatively long time to run10:09
Chipacaas in 10 minutes10:10
zygathat's pretty slow, is it CPU bound on apparmor compiler?10:10
Chipacazyga: probably not10:10
Chipacazyga: because dropping the unlock/lock around things dropped the time down a lot10:11
Chipacaiirc down to 4 minutes? somehting like that10:11
zygaChipaca: oh? that's curiou10:11
zygaChipaca: around what specifically?10:11
Chipacazyga: around … and this is from memory … the security backend setup loop? something like that10:12
Chipacazyga: basically it means moving to an actual database for state should not be postponed much longer10:13
zygaChipaca: and that locking serializes state and this is expensive?10:13
Chipacazyga: yep10:13
Chipacazyga: when done in a loop like in that test, yes10:14
Chipacakeep in mind because of what that test does, it's almost a worst case10:14
Chipacalots of changes to serialize10:14
zygaChipaca: lots as in "KBs of text"10:15
zygaChipaca: I think the real DB will take some time to arrive10:15
mborzeckiChipaca: 'actual database' do you have anything particular in mind?10:15
zygaChipaca: it's a major change10:15
Chipacazyga: yes i know10:16
pedronisI told the same, given everything it's a post 18.04 afai would guess10:16
Chipacamborzecki: gustavo had thoughts on this, there was somehting ike an in-memory all-go db10:16
zygaChipaca: is it possible that the cost is in fsync ops and not in anything else?10:16
Chipacapedronis: hello :-)10:16
pedronisBoltDB I think10:16
Chipacapedronis: just trying to make sure this isn't lost10:16
pedronisis what Gustavo mentioned10:16
Chipacazyga: go's profiler pointed to json encode10:17
pedronisChipaca: hello,  me is slowly rebooting :)10:17
mborzeckimhm, i'm asking cause we moved to lmdb at some point in mender10:17
Chipacazyga: but it _might_ be io disguised as that10:17
mupPR snapd#4429 opened: tests: add regression test for LP 1681739 (snap var interpolation) <Created by mvo5> <https://github.com/snapcore/snapd/pull/4429>10:49
Son_GokuI still have one more week of vacation :)10:49
mupPR snapd#4405 closed: taskrunner/many: KnownTaskKinds helper <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/4405>10:50
mupPR snapd#4428 closed: daemon: return "bad-query" error kind for store.ErrBadQuery <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4428>10:51
mupPR snapd#4420 closed: cmd: clarify "This leaves %s tracking %s." message <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/4420>10:55
mupPR snapd#4403 closed: asserts/signtool: support for building tools on top that fill-in/compute some headers <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/4403>10:57
mupPR snapd#4422 closed: packaging/arch: disable services when removing <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/4422>10:57
ChipacaSon_Goku: showoff11:02
Son_Gokueh, I was sick most of my vacation11:02
Son_Gokuthis might be my only "enjoyment" week11:02
ChipacaSon_Goku: I think everywhere I've lived, if you're sick during a vacation it counts as sick leave and not vacation11:04
Son_GokuI filed it as vacation, then became sick after11:04
Son_Gokuat that point, it doesn't matter11:04
Son_GokuI got sick during my trip to my parents' home11:05
ChipacaSon_Goku: i understand you filed it as vacation, but if you get sick during your vacation, you can then claim it back (and take your vacation when you're not sick)11:05
Chipacathis is, in argentina and the eu it works11:05
Chipacai've not lived elsewhere11:05
Son_Gokuit's true here too11:05
Chipacagood to know11:06
Son_Gokubut as I have "unlimited PTO" and only 5 sick days, there's no point11:06
ChipacaSon_Goku: i understand if it's academic because of … yeah, those limits11:06
ChipacaSon_Goku: especially if sick days are not paid11:06
Son_Gokusorry, five paid sick days11:06
ChipacaSon_Goku: and is the unlimited PTO also paid?11:06
Son_Gokuif I take more than two weeks of sick leave, I'd probably be fired11:07
Chipacaah fair enough :-)11:07
Son_Gokunot without some kind of doctor's notice, anyway11:07
Chipacaah yeah the "claim it back" thing involves a doctors scrip11:07
Chipacabit of paper11:08
* Chipaca has no idea where he picked up that word11:08
mvozyga: quick question about lp: 170870311:12
kalikianaChipaca: strips of paper exist. although you might want to take the whole one with you to show to your employer rather than tearing of part of it ;-)11:12
Chipacakalikiana: pretty sure it's 'scrip'11:13
Chipacakalikiana: but as i say, dunno where i picked it up11:13
Chipacamy english has [*] reference needed all over the place11:13
Son_Gokuscrip is technically correct11:14
Son_Gokuthough I've not heard that word in common use11:14
Son_Gokuit's short for prescription11:14
Chipacathe sources of my english include, but are not limited to, north scottish fishermen, south american british hospital staff, and welsh [censored]11:15
Chipaca¯\_(ツ)_/¯ family11:15
pedronisinterestingly enough in that use is a contraction of script itself there a contraction of prescription11:17
* pedronis (is perusing a dictionary)11:17
zygamvo: looking11:19
mvozyga: mostly curious if you know more about it as you assigned it to yourself11:20
zygamvo: yes, I remember now11:20
mvozyga: tell me more please11:22
zygamvo: looking at the code, when we enable / disable we setup security11:22
zygamvo: then we look at snaps that were affected and also setup those11:22
zygamvo: then that can fail "silently" when we cannot get snap state for any of those snaps11:23
zygamvo: now I'm not sure if this is happening here11:23
zygamvo: let me try11:23
zygamvo: it looks ok but ...11:23
mvozyga: I tried it today and can still reproduce it (with the mir snaps)11:25
zygamvo: did you try that in a VM with core snap?11:25
zygamvo: or on classic?11:25
zygamvo: aha, let me see master quickly now, just adding some logging11:26
zygamvo: logging in tests says it's not broken, may be broken later, trying with real snaps now11:29
mvozyga: ta11:30
zygamvo: hmm, mir-kiosk is not in the store?11:31
zygaah, edge11:31
zygathe reason is simple, autoconnect11:33
Chipacafor some reason i thought we were interpolating variables in app commands11:34
zygamvo: when the mir-kiosk-apps is disabled it gets disconnected11:34
Chipacaturns out we aren't11:34
Chipacaweren't we going to do that?11:35
zygamvo: then when it is enabled it thinks that nothing happened because of autoconnect11:35
* zyga looks11:35
mvozyga: iirc it was still showing as connected when I tried11:37
zygamvo: yes, there's something buggy going on11:38
zygamvo: it should say it got disconnected but didn't11:38
* mvo nods11:42
zygamvo: oddly repo.DisconnectSnap returns [] so something is very fishy there11:42
zygamvo: so on enable r.plugs[] and r.slots for that snap is empty11:50
zygamvo: so something is wrong in the repo, digging deeper11:50
zygapstolowski: ^ FYI11:50
pstolowskizyga, is this re 1708703?11:53
zygaI see the bug now11:53
pstolowskiok, for a moment I thought I broke something in recent refactorings, but I guess that's an older issue11:55
zygayes, it looks like an older issue11:55
pstolowskiinteresting how it survived all the changes :/11:55
zygafixed :)12:01
zyganow for some tests12:02
pstolowskizyga, thank you12:04
* pstolowski lunch12:15
mupPR snapd#4430 opened: overlord/ifacestate: fix disable/enable cycle to setup security <Created by zyga> <https://github.com/snapcore/snapd/pull/4430>12:16
zygapstolowski, mvo: ^12:16
* mvo hugs zyga 12:21
Son_Gokuzyga, when are you going to give me snapd selinux backend? :)12:30
zygaSon_Goku: no idea, but if I start now maybe in December?12:30
Son_Gokubetter than never, so worksforme12:30
zygaSon_Goku: I haven't discussed what our todo list is yet12:30
zygahey sergiusens :)12:54
kalikianamorning sergiusens13:06
kalikianaand happy new year13:06
mupPR snapd#4429 closed: tests: add regression test for LP 1681739 (snap var interpolation) <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4429>13:08
* kalikiana going to take a lunch break in ~1513:19
Chipacamborzecki: in https://paste.ubuntu.com/26306252/, what  is the 1st column?13:29
mborzeckiChipaca: just an index in the data set13:30
mupPR snapcraft#1838 opened: tests: fix broken rust test snap <bug> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1838>13:30
mborzeckiChipaca: same dataaset, ubuntu-16.04-64 only this time: https://paste.ubuntu.com/26306338/13:32
Chipacamborzecki: hmmm13:32
Chipacamborzecki: what does the duration of the suite mean?13:33
Chipacamborzecki: because the suite takes 58s and in the suite a bunch of tests take 50s, it sounds like an accouting error13:33
Chipacaoh wait13:33
zygaChipaca: is it measuring wall time or accumulated time from each node?13:33
mborzeckiChipaca: it's taken from travis folds13:33
Chipacamborzecki: my mistake sorry13:33
Chipacatests/main/completion vs tests/completion :-)13:34
zygameh, it's getting dark already13:35
zygamy monitors are too bright13:35
zygaand have poor controls for adjusting13:35
Chipacazyga: redshift?13:36
zygaChipaca: i think it's built into gnome now13:36
* zyga looks for that13:36
Chipacazyga: yes (but don't you need to enable it?)13:36
zygaChipaca: I'm also complaining about short daylight cycle here :( software cannot fix that13:37
Chipacazyga: i'm pretending not to notice13:37
zygaenabled :)13:37
zygathank you for reminding me about it13:38
* kalikiana bbl13:38
zygaChipaca: on the other hand I moved my systems to a darker green tinted wallpapers/color schemes and I like that a lot13:39
Chipacaman, apt needs a 'find' alias now :-(13:40
mborzeckizyga: about teleconsole on fedora, `snap run --shell teleconsole` should not fail with 'execv failed: No such file or directory', right?13:41
zygamborzecki: right13:42
zygamborzecki: it looks like it's trying to exec /usr/lib/snapd/snap-exec13:42
zygavs libexec13:42
zyga(just a guess)13:42
zygaI haven't rebooted F27 after updates13:42
zyga47% installing13:42
mborzeckibut it works for 'hello' snap13:42
zygaclassic doesn't do redirection13:43
zygaI think I know what's going on13:43
zygawe disabled classic on fedora13:43
zygaso you did the symlink13:43
zygabut the code that runs classic stuff had hardcoded assumptions on dir layout13:43
zygaif you print execed programs you will see (probably) it running the wrong directory13:43
pedronisChipaca: #4389 and #4392 are probably PRs you could review13:44
mupPR #4389: overlord/snapstate: override Snapstate.UserID in refresh if the installing user is gone <Created by pedronis> <https://github.com/snapcore/snapd/pull/4389>13:44
mupPR #4392: many: refresh with appropriate creds <Created by pedronis> <https://github.com/snapcore/snapd/pull/4392>13:44
zygamborzecki: as a quick test add a symlink /usr/lib/snapd -> /usr/libexec/snapd13:44
=== __chip__ is now known as Chipaca
mborzeckizyga: right, works now13:46
zygamborzecki: cool, that should be as simple fix13:46
* zyga removes 34,028 messages from launchpad from his inbox13:51
mborzeckizyga: hm we can't exactly mess with rootfs and fedora guys would probably be unhappy if we symlink, the fix would need to be automatically cleaned up when snap exits13:53
zygano no, the symlink is just a test13:55
zygawe don't need it13:55
zygabut we now know exactly what is wrong13:55
zygait's a simple bug in cmd.go13:55
zygaor cmd_run.go13:55
mborzeckizyga: https://paste.ubuntu.com/26306484/ like this? :)13:58
zygalooks excellent13:58
zygathank you!13:58
zygamborzecki: look at how this behaves when classic + reexec is in use13:58
zygamborzecki: but this looks ok for fedora13:59
mborzeckizyga: https://paste.ubuntu.com/26306521/14:04
mborzeckiand teleconsole works now, yay14:05
zygaI think the test is wrong14:05
zygaif you are reexeed you want:14:05
zygapath to core snap in the distro + core libexec dir14:05
zygae.g. /var/lib/snapd/snap/core/current/usr/lib/snapd/snap-exec14:06
zygawhich combines a fedora-specific snap mountdir + core layout that contains snap-exec14:06
zygayou can perhaps simplify14:06
mborzeckihmm right14:06
zygaby using the same trick that cmd.go does14:06
zygafind your own executable14:06
zygaand then run snap-exec as your peer14:06
zygathough because snap is not in the same directory as snap-exec or snap-confine, this may be more cloudy14:07
zygaI'd vote for a direct test + explicit join14:07
mborzeckilooks like we have a way forward, i'll open a PR later14:08
zygathank you :)14:09
mborzeckibeen fun chasing it14:09
mborzeckistill now idea why strace hangs occasionally14:09
mborzeckianyways, i'm off to pick up the kids14:10
Chipacaso, the completion tests do take 50s per14:19
Chipacathey can be made significantly faster by dropping a delay and/or by bumping a send rate14:20
Chipacathese two things exist because without them the tests were flaky14:20
Chipacashould I try dropping them to see if they continue being flaky?14:20
zygaChipaca: worth a try, I wonder if we could do _something_ to become reliable14:29
mupPR snapd#4421 closed: daemon: add new polkit action to manage interfaces <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4421>14:33
mupPR snapd#4427 closed: snap: fix snap find " " output <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4427>14:34
* zyga break for small taxes and paperwork14:46
elopiogood morning team!14:47
jdstrandkyrofa: hi! I played with the nextcloud snap a bit over the holiday. I was wondering if you thought about a content interface (readonly?) for it so that snaps on the system could access the files. eg, a minidlna snap could consume nextcloud content for serving movies, pictures and/or videos14:58
jdstrandkyrofa: and happy new year :)14:59
sergiusenselopio good morning15:02
kyrofaHappy New Year everyone!15:04
kyrofaMorning jdstrand :)15:04
sergiusenskyrofa hey15:05
kyrofajdstrand, I considered it, but it's possible for someone to use the removable-media interface for data, and then the snap isn't sharing anything useful15:05
kyrofaHey there sergiusens, nice break?15:06
jdstrandkyrofa: with the nextcloud box, it stores its data not in /media by default since it formats the drive and installs itself on it15:08
popeySaviq: is there an easy way to make a multipass disk bigger, 2.1GB seems awfully small for the initial size15:08
kyrofajdstrand, yeah, by default it uses $SNAP_COMMON15:08
Saviqpopey: on launch, pass `--disk 5G` or so15:09
jdstrandkyrofa: I was thinking of creating a systemd unit to bind mount stuff into /media and use removable-media for a minidlna snap, but thought maybe things could be easier for people15:09
popeySaviq: but not after I made it?15:09
Saviqpopey: 2G is what the default size is of the cloud images15:09
jdstrandkyrofa: anyway, food for thought15:09
Saviqpopey: I think someone wrote on the forums how to do it after launch15:09
Saviqit's not trivial since you have to resize the FS15:09
popeyfound it, thanks15:10
Saviqso I'm not sure we'll ever built that functionality into multipass (but we'll allow you to change the default initial size)15:10
popeyI'll destroy and start again :(15:10
jdstrandkyrofa: also, is the nextcloud-client expected to be fully functional? (excepting auto-start)15:11
Saviqpopey: we might bump the default anyway, it's qcow so it's not stealing your storage straight away15:11
kyrofajdstrand, indeed, although I'll admit I don't use it due to the autostart thing so I wouldn't be surprised to learn that it has issues. Does it?15:12
jdstrandkyrofa: (nextcloud-client snap that is. I kept ending up with defunct processes and things not syncing (not 100% sure things not syncing was the snap's fault and not mine cause it was still early days in my playing)15:12
kyrofaHmm, that sounds odd15:13
zygajdstrand, kyrofa: hey :)15:14
jdstrandhappy new year zyga :)15:14
kyrofaHappy new year, zyga! How was break?15:14
zygakyrofa: exhausting :) too long I would say15:14
zygakyrofa: I'm happy to be back :)15:14
zygahow are you guys doing? I heard US has move to ice age15:15
sergiusenskyrofa good, yourself?15:16
sergiusenszyga I want to be on a break for longer!15:17
kyrofaYeah me too15:17
kyrofasergiusens, short, but good15:17
zygasergiusens: for how long were you away?15:17
sergiusenszyga just the week in between the two big holidays15:19
ppisatiogra_: i think some files were moved inside os.snap, without updating the kernel plugin15:22
ppisatiogra_: can you give your opinion on this?15:22
ppisatiogra_: https://bugs.launchpad.net/snapcraft/+bug/174088215:22
mupBug #1740882: Missing initrd.img-core in os.snap <Snapcraft:New> <https://launchpad.net/bugs/1740882>15:22
kalikianakyrofa: Happy New Year, Franz ;-)15:29
kyrofaHey there kalikiana! Back at you. How was your break?15:30
kalikianaVery necessary, haha15:30
kyrofaNo kidding15:35
mupPR snapd#4430 closed: overlord/ifacestate: fix disable/enable cycle to setup security <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4430>15:45
Chipacamborzecki: mvo: anything further blocking #4394?15:48
mupPR #4394: snap: give the snap.Container interface a Walk method <Created by chipaca> <https://github.com/snapcore/snapd/pull/4394>15:48
ogra_ppisati, i didnt touch anything (still on vacation) ... if anyone moved anything around it would likely be mvo though16:06
ppisatimvo: ^16:06
ppisatiogra_: ta16:07
=== ahasenac` is now known as ahasenack
mvoppisati: probably https://github.com/snapcore/core/pull/63 but its slightly puzzling, there is a symlink there - is this symlink broken?16:10
mupPR core#63: 25-create-generic-initrd.chroot: use symlink instead of copy <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core/pull/63>16:10
=== ahasenack is now known as Guest12330
mvoppisati: fwiw, this was requested to be a symlink instead of a copy to help the store with diff generation16:10
ogra_yeah, looks more like an issue with the kernel plugin not respecting links16:13
ppisatinope, the kernel plugin expected usr/lib/ubuntu-core-generic-initrd/initrd.img-core to be the real file and not a symlink, that's because at the beginning of _unpack_generic_initrd() only a portion of os.snap was unsquashed - so now instead of the file, we have a dangling symlink to something that is not there16:17
ppisatianyhow, thnaks for the confirmation, i alread attached a patch to the LP that fixes it16:17
ogra_yeah, looks good to me16:18
mupPR snapd#4431 opened: snap: make `snap info invalid-snap` output more user friendly <Created by mvo5> <https://github.com/snapcore/snapd/pull/4431>16:22
mvozyga: iirc we discussed lp: #1705549 a while ago, do you remember the details? I would like to update the bugreport16:24
mupBug #1705549: Snaps in lxc can't refresh if old revisions need to be cleaned <snapd:New> <https://launchpad.net/bugs/1705549>16:24
zygamvo: yes16:24
zygamvo: we went through several rounds of attemps to fix this and have a solution that seems oK both technically and from security POV but haven't implemented it16:24
mupPR snapcraft#1839 opened: kernel plugin: update initrd.img-core path to boot/initrd.img <Created by piso77> <https://github.com/snapcore/snapcraft/pull/1839>16:25
mvozyga: anything I could point to in the bug?16:25
zygamvo: prior attempts either worked but were too open for security or didn't work16:25
zygamvo: I can look for several PRs but those are all closed now16:25
zygaone has some interesting discussion16:25
zygalet me find it16:25
zygamvo: this is one https://github.com/snapcore/snapd/pull/425816:26
mupPR #4258: cmd/snap-confine,tests: fix unmounting on systems without rshared / <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/4258>16:26
zyga(I think the interesting discussion was on IRC acctually :/)16:26
mvozyga: thanks16:27
mvojdstrand: your opinion on lp: 1698412 would be appreciated. the question is if bluetooth-control should be able to read /sys/bus/usb/drivers/btusb/module/16:43
leftyfbIf I install gnome-3-26-1604 on Ubuntu 16.04, will that just allow other snaps which are connected utilize it or does it install/setup my entire DE to use it?16:58
kyrofaleftyfb, it's just a runtime for other snaps17:00
leftyfbok, good to know17:00
leftyfbhey kyrofa!17:00
leftyfbkyrofa: sorry we haven't got back to you17:01
mupPR snapcraft#1838 closed: tests: fix broken rust test snap <bug> <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1838>17:01
leftyfbkyrofa: things got kinda busy with the holidays/retail17:01
kyrofaHaha, same here! No worries17:01
jdstrandmvo: I'll take a look and comment17:01
leftyfbkyrofa: snappy is something we'll definitely be looking at this year. We first need to get all the customers running Xenial17:02
kyrofaleftyfb, understood! You know where to find me, always happy to help :)17:03
mvojdstrand: also lp: #1689577 would be appreciated. hopefully quick and easy (both of them :)17:04
mupBug #1689577: bad system call "shutdown" trying to use systemd-cat from a snap <snapd:Incomplete> <https://launchpad.net/bugs/1689577>17:04
kyrofaleftyfb, and I'm glad to see you around!17:04
* kalikiana going to call it a day - see you guys tomorrow17:05
mupPR snapcraft#1840 opened: docker: instructions to build from the snap <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1840>17:31
kyrofaHey zyga, are you still working on that lxd snaps not updating bug?18:07
zyganot at present but I will likely come back to it soon18:08
zyga(this week)18:08
zygaI'll focus on landing https://github.com/snapcore/snapd/pull/432918:08
mupPR #4329:  cmd/snap-confine: discard stale mount namespaces (v2) <Created by zyga> <https://github.com/snapcore/snapd/pull/4329>18:08
kyrofazyga, okay18:10
kyrofasergiusens, the circle ci tutorial, would you like that to be a snappy-docs document, or a tutorials.ubuntu.com document?18:11
zygaHEADS UP EVERYONE: 2.30 is now in stable18:11
sergiusenskyrofa tutorials, to match the travis one18:12
kyrofaOh riiight, forgot about the travis one18:12
zygamvo: so, who can we ask from the store?18:20
kyrofasergiusens, 1-1 today? I don't have anything particularly new to share, so we can skip if you like18:32
kyrofaI'll assume that's a skip18:41
sergiusenskyrofa yeah18:57
sergiusenselopio let's skip unless you have something18:57
elopiosergiusens: next week I will have something :)18:58
mupPR snapcraft#1841 opened: Update test_export_login.py <Created by heesen3> <https://github.com/snapcore/snapcraft/pull/1841>19:37
mupPR snapcraft#1833 closed: cli: humanize push message <Created by Sheogorath2> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1833>20:43

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