[06:06] <mborzecki> morning
[07:26] <zyga> good morning! :)
[07:27]  * zyga looks at a mountain of email
[07:30] <mborzecki> zyga: hey
[07:31] <zyga> hey :)
[07:31] <zyga> happy new year
[07:32] <mborzecki> likewise :)
[07:33] <mborzecki> zyga: did you enjoy your break?
[07:35] <zyga> mborzecki: yes, it almost feels weird to go back after so long
[07:35] <mborzecki> i would call it a shock, rather than just being weird :)
[07:46] <kalikiana> good morning
[07:46] <kalikiana> happy 2018
[07:52] <zyga> good morning kalikiana!
[08:10] <pstolowski> mornings!
[08:11] <mborzecki> pstolowski: hey
[08:19] <zyga> good morning koza
[08:31]  * zyga reboots for some updates
[08:44] <mborzecki> mvo: hey
[08:44] <mup> Issue 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:47] <mvo> hey mborzecki! how are you?
[08:47] <mvo> mborzecki: did I miss anything interessting this morning :) ?
[08:47] <mborzecki> (unsurprisingly) the morning has been rather slow
[08:48] <mborzecki> i'm poking around yocto support, slow progress however
[08:48]  * mvo nods
[08:48] <mvo> mborzecki: yocto> nice to hear
[08:49] <mborzecki> promised 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 nods
[08:50] <mup> Issue snapcraft#1834 closed: Feature Request: NGINX plugin <Created by jamesbeedy> <Closed by kalikiana> <https://github.com/snapcore/snapcraft/issue/1834>
[09:03] <mup> PR snapd#4427 opened: snap: fix snap find " " output <Created by mvo5> <https://github.com/snapcore/snapd/pull/4427>
[09:22] <mborzecki> zyga not around?
[09:22] <mvo> mborzecki: heh
[09:24] <zyga> hey mvo :) long time no see
[09:24] <zyga> how are you doing?
[09:29] <mborzecki> zyga: https://github.com/snapcore/snapd/blob/master/cmd/snap-confine/mount-support.c#L589 afaiu /root is assumed to always exist then?
[09:29] <zyga> mborzecki: yes, there's a lot of directories we just assume
[09:29] <zyga> mborzecki: for /root we could skip it if absent
[09:30] <mborzecki> i'm asking as it's not here in poky (yocto build), at least with rocko brach (the last release)
[09:30] <mvo> hey zyga ! good to see you, happy new year
[09:30] <mborzecki> frankly, i don't recall if it used to be there or now, never needed to do/keep anything around in /root
[09:34] <clienthax__226> ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ ANOTHER EXPLOIT HAS BEEN FOUND IN WEECHAT ONE THAT SENDS YOUR LOGS TO THE DEVS IP ADDRESS!!jmnaxjlu: ubot9 hurricanehrndz verterok ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
[09:36]  * kalikiana coffee break
[09:44] <zyga> hey Chipaca
[09:44] <Chipaca> zyga!
[09:44] <Chipaca> how's things?
[09:44] <zyga> Chipaca: snappy :)
[09:45] <zyga> Chipaca: still waking up/getting into coding spirit (code reviews now)
[09:45] <zyga> how are you doing? are you feeling better?
[09:45] <mvo> hey Chipaca! welcome back, great to "see" you
[09:45] <Chipaca> mvo: o/
[09:45] <Chipaca> zyga: i caught a lot of sun, and a little head cold, but overall feeling good. you?
[09:46] <zyga> Chipaca: I'm doing great, disappointed for not having much snow this xmas
[09:47] <Chipaca> mvo: likewise, how're you doing?
[09:47] <mup> PR snapd#4428 opened: daemon: return "bad-query" error kind for store.ErrBadQuery <Created by mvo5> <https://github.com/snapcore/snapd/pull/4428>
[09:47] <Chipaca> zyga: snow is nice but i'm happy with my choice :-)
[09:48] <zyga> Chipaca: did you travel somewhere warmer for your holidays/
[09:48] <mvo> Chipaca: 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:49] <Chipaca> mvo: i'll preted not to know half of your hacking today is from an unconventional seat
[09:49] <mvo> lol
[09:50] <Chipaca> zyga: visiting friends on the costa d'azur
[09:50] <zyga> Chipaca: stdout flushes manually, stderr flushes on output ;-)
[09:50]  * zyga stops thinking about IO jokes
[09:50] <zyga> Chipaca: ah, lovely
[09:51] <zyga> Chipaca: I've visited some friends from Catalonia but we chose to go to the capital of coal in poland (katowice) instead :)
[09:51] <Chipaca> oooh
[09:51] <Chipaca> i've only run trains through the places you visit
[09:54] <Chipaca> mvo: is the client side of the bad query kind coming, or not needed?
[09:58] <mvo> Chipaca: I can add it,  I think it makes sense, I was mostly doing it for roberts glib based client implementation
[09:59] <mvo> (where it is not needed AIUI) - but for symmetry it does make sense
[09:59] <Chipaca> mvo: if we're not going to use it, it's just bitrot waiting to happen
[10:00] <Chipaca> mvo: (otoh all the kinds could use a reshuffle to not be defined twice...)
[10:00] <Chipaca> dunno
[10:00]  * Chipaca does not bring new wisdom into the new year
[10:00] <mvo> Chipaca: lol
[10:00] <mvo> Chipaca: all good, I will push the client side and make a nice(r) error message for `snap find "()!§$§%§%*"`
[10:01] <mvo> Chipaca: something like "error: cannot list snaps: bad query, please use less special chars"
[10:01] <zyga> mvo: "please use latin letters or digits" maybe
[10:02] <Chipaca> oh you just reminded me i probably need to change this error message before submitting it as a PR
[10:02] <mvo> zyga: yeah, thats better
[10:02]  * zyga looks at the intense fog outside
[10:02] <Chipaca> 	ErrAppEscapes   = errors.New("snap is unusable due to fuckery")
[10:02] <mvo> Chipaca: *cough* this is a family channel!
[10:02] <zyga> mvo: or arabic digits ;-)
[10:03] <Chipaca> mvo: /o\ sorry
[10:03] <mvo> zyga: maybe ascii letters/digests
[10:03]  * mvo hugs Chipaca 
[10:03] <zyga> Chipaca: go and stand in the corner now ;;-)
[10:03] <zyga> mvo: yeah
[10:03]  * Chipaca hugs mvo from the corner
[10:04] <zyga> we have a lot of PRs
[10:04] <mvo> zyga: my fault
[10:04] <zyga> mvo: how is that a fault? :)
[10:04] <zyga> mvo: thank you for all the nice fixes
[10:04] <mvo> zyga: I was doing bug triage and there is soooo many low hanging fruits
[10:04] <zyga> can I get a review on https://github.com/snapcore/snapd/pull/4315
[10:04] <mup> PR #4315:  cmd/snap-update-ns: add execWritableMimic <Created by zyga> <https://github.com/snapcore/snapd/pull/4315>
[10:05] <zyga> it's from _november_ and has one review
[10:05]  * zyga boots subsequent VM for the round of updates
[10:06] <Chipaca> how's travis/linode/spread behaving?
[10:06] <zyga> I haven't tried anything new yet, I see lots of red PRs though
[10:06] <zyga> so maybe it's still having a hangover
[10:09] <Chipaca> zyga: one of the things we realised towards the year end was that the interfaces-many was taking a relatively long time to run
[10:10] <Chipaca> as in 10 minutes
[10:10] <zyga> that's pretty slow, is it CPU bound on apparmor compiler?
[10:10] <Chipaca> zyga: probably not
[10:11] <Chipaca> zyga: because dropping the unlock/lock around things dropped the time down a lot
[10:11] <Chipaca> iirc down to 4 minutes? somehting like that
[10:11] <zyga> Chipaca: oh? that's curiou
[10:11] <zyga> Chipaca: around what specifically?
[10:12] <Chipaca> zyga: around … and this is from memory … the security backend setup loop? something like that
[10:13] <Chipaca> zyga: basically it means moving to an actual database for state should not be postponed much longer
[10:13] <zyga> Chipaca: and that locking serializes state and this is expensive?
[10:13] <Chipaca> zyga: yep
[10:14] <Chipaca> zyga: when done in a loop like in that test, yes
[10:14] <Chipaca> keep in mind because of what that test does, it's almost a worst case
[10:14] <Chipaca> lots of changes to serialize
[10:15] <zyga> Chipaca: lots as in "KBs of text"
[10:15] <zyga> Chipaca: I think the real DB will take some time to arrive
[10:15] <mborzecki> Chipaca: 'actual database' do you have anything particular in mind?
[10:15] <zyga> Chipaca: it's a major change
[10:16] <Chipaca> zyga: yes i know
[10:16] <pedronis> I told the same, given everything it's a post 18.04 afai would guess
[10:16] <Chipaca> mborzecki: gustavo had thoughts on this, there was somehting ike an in-memory all-go db
[10:16] <zyga> Chipaca: is it possible that the cost is in fsync ops and not in anything else?
[10:16] <Chipaca> pedronis: hello :-)
[10:16] <pedronis> BoltDB I think
[10:16] <Chipaca> pedronis: just trying to make sure this isn't lost
[10:16] <pedronis> is what Gustavo mentioned
[10:17] <Chipaca> zyga: go's profiler pointed to json encode
[10:17] <pedronis> Chipaca: hello,  me is slowly rebooting :)
[10:17] <mborzecki> mhm, i'm asking cause we moved to lmdb at some point in mender
[10:17] <Chipaca> zyga: but it _might_ be io disguised as that
[10:49] <mup> PR 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_Goku> I still have one more week of vacation :)
[10:50] <mup> PR snapd#4405 closed: taskrunner/many: KnownTaskKinds helper <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/4405>
[10:51] <mup> PR 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:55] <mup> PR 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:57] <mup> PR 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] <mup> PR snapd#4422 closed: packaging/arch: disable services when removing <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/4422>
[11:02] <Chipaca> Son_Goku: showoff
[11:02] <Son_Goku> eh, I was sick most of my vacation
[11:02] <Son_Goku> this might be my only "enjoyment" week
[11:04] <Chipaca> Son_Goku: I think everywhere I've lived, if you're sick during a vacation it counts as sick leave and not vacation
[11:04] <Son_Goku> I filed it as vacation, then became sick after
[11:04] <Son_Goku> at that point, it doesn't matter
[11:05] <Son_Goku> I got sick during my trip to my parents' home
[11:05] <Chipaca> Son_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] <Chipaca> this is, in argentina and the eu it works
[11:05] <Chipaca> i've not lived elsewhere
[11:05] <Son_Goku> it's true here too
[11:06] <Chipaca> good to know
[11:06] <Son_Goku> but as I have "unlimited PTO" and only 5 sick days, there's no point
[11:06] <Chipaca> Son_Goku: i understand if it's academic because of … yeah, those limits
[11:06] <Chipaca> Son_Goku: especially if sick days are not paid
[11:06] <Son_Goku> sorry, five paid sick days
[11:06] <Chipaca> Son_Goku: and is the unlimited PTO also paid?
[11:07] <Son_Goku> yes
[11:07] <Son_Goku> if I take more than two weeks of sick leave, I'd probably be fired
[11:07] <Chipaca> ah fair enough :-)
[11:07] <Son_Goku> not without some kind of doctor's notice, anyway
[11:07] <Chipaca> ah yeah the "claim it back" thing involves a doctors scrip
[11:07] <Chipaca> strip?
[11:08] <Chipaca> bit of paper
[11:08]  * Chipaca has no idea where he picked up that word
[11:12] <mvo> zyga: quick question about lp: 1708703
[11:12] <kalikiana> Chipaca: 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] <Son_Goku> heh
[11:13] <Chipaca> kalikiana: pretty sure it's 'scrip'
[11:13] <Chipaca> kalikiana: but as i say, dunno where i picked it up
[11:13] <Chipaca> my english has [*] reference needed all over the place
[11:13] <Chipaca> citation*
[11:14] <Son_Goku> scrip is technically correct
[11:14] <Son_Goku> though I've not heard that word in common use
[11:14] <Son_Goku> it's short for prescription
[11:15] <Chipaca> the sources of my english include, but are not limited to, north scottish fishermen, south american british hospital staff, and welsh [censored]
[11:15] <Chipaca> ¯\_(ツ)_/¯ family
[11:17] <pedronis> interestingly enough in that use is a contraction of script itself there a contraction of prescription
[11:17]  * pedronis (is perusing a dictionary)
[11:19] <zyga> mvo: looking
[11:20] <mvo> zyga: mostly curious if you know more about it as you assigned it to yourself
[11:20] <zyga> mvo: yes, I remember now
[11:22] <mvo> zyga: tell me more please
[11:22] <zyga> mvo: looking at the code, when we enable / disable we setup security
[11:22] <zyga> mvo: then we look at snaps that were affected and also setup those
[11:23] <zyga> mvo: then that can fail "silently" when we cannot get snap state for any of those snaps
[11:23] <zyga> mvo: now I'm not sure if this is happening here
[11:23] <zyga> mvo: let me try
[11:23] <zyga> mvo: it looks ok but ...
[11:25] <mvo> zyga: I tried it today and can still reproduce it (with the mir snaps)
[11:25] <zyga> mvo: did you try that in a VM with core snap?
[11:25] <zyga> mvo: or on classic?
[11:26] <mvo> classic
[11:26] <zyga> mvo: aha, let me see master quickly now, just adding some logging
[11:27] <mvo> ok
[11:29] <zyga> mvo: logging in tests says it's not broken, may be broken later, trying with real snaps now
[11:30] <mvo> zyga: ta
[11:31] <zyga> mvo: hmm, mir-kiosk is not in the store?
[11:31] <zyga> ah, edge
[11:32] <mvo> yes
[11:33] <zyga> ha
[11:33] <zyga> indeed
[11:33] <zyga> the reason is simple, autoconnect
[11:34] <Chipaca> for some reason i thought we were interpolating variables in app commands
[11:34] <zyga> mvo: when the mir-kiosk-apps is disabled it gets disconnected
[11:34] <Chipaca> turns out we aren't
[11:35] <Chipaca> weren't we going to do that?
[11:35] <zyga> mvo: then when it is enabled it thinks that nothing happened because of autoconnect
[11:35]  * zyga looks
[11:37] <mvo> zyga: iirc it was still showing as connected when I tried
[11:38] <zyga> mvo: yes, there's something buggy going on
[11:38] <zyga> mvo: it should say it got disconnected but didn't
[11:42]  * mvo nods
[11:42] <zyga> mvo: oddly repo.DisconnectSnap returns [] so something is very fishy there
[11:50] <zyga> mvo: so on enable r.plugs[] and r.slots for that snap is empty
[11:50] <zyga> mvo: so something is wrong in the repo, digging deeper
[11:50] <mvo> ok
[11:50] <mvo> thanks
[11:50] <zyga> pstolowski: ^ FYI
[11:53] <pstolowski> zyga, is this re 1708703?
[11:53] <zyga> yes
[11:53] <zyga> I see the bug now
[11:55] <pstolowski> ok, for a moment I thought I broke something in recent refactorings, but I guess that's an older issue
[11:55] <zyga> yes, it looks like an older issue
[11:55] <pstolowski> interesting how it survived all the changes :/
[12:01] <zyga> fixed :)
[12:02] <zyga> now for some tests
[12:04] <pstolowski> zyga, thank you
[12:15]  * pstolowski lunch
[12:16] <mup> PR snapd#4430 opened: overlord/ifacestate: fix disable/enable cycle to setup security <Created by zyga> <https://github.com/snapcore/snapd/pull/4430>
[12:16] <zyga> pstolowski, mvo: ^
[12:21]  * mvo hugs zyga 
[12:30] <Son_Goku> zyga, when are you going to give me snapd selinux backend? :)
[12:30] <zyga> Son_Goku: no idea, but if I start now maybe in December?
[12:30] <Son_Goku> better than never, so worksforme
[12:30] <zyga> Son_Goku: I haven't discussed what our todo list is yet
[12:51] <sergiusens> morning
[12:54] <zyga> hey sergiusens :)
[13:06] <kalikiana> morning sergiusens
[13:06] <kalikiana> and happy new year
[13:08] <mup> PR 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:19]  * kalikiana going to take a lunch break in ~15
[13:29] <Chipaca> mborzecki: in https://paste.ubuntu.com/26306252/, what  is the 1st column?
[13:30] <mborzecki> Chipaca: just an index in the data set
[13:30] <mup> PR snapcraft#1838 opened: tests: fix broken rust test snap <bug> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1838>
[13:32] <mborzecki> Chipaca: same dataaset, ubuntu-16.04-64 only this time: https://paste.ubuntu.com/26306338/
[13:32] <Chipaca> mborzecki: hmmm
[13:33] <Chipaca> mborzecki: what does the duration of the suite mean?
[13:33] <Chipaca> mborzecki: because the suite takes 58s and in the suite a bunch of tests take 50s, it sounds like an accouting error
[13:33] <Chipaca> oh wait
[13:33] <zyga> Chipaca: is it measuring wall time or accumulated time from each node?
[13:33] <mborzecki> Chipaca: it's taken from travis folds
[13:33] <Chipaca> mborzecki: my mistake sorry
[13:34] <Chipaca> tests/main/completion vs tests/completion :-)
[13:35] <zyga> meh, it's getting dark already
[13:35] <zyga> my monitors are too bright
[13:35] <zyga> and have poor controls for adjusting
[13:36] <Chipaca> zyga: redshift?
[13:36] <zyga> Chipaca: i think it's built into gnome now
[13:36]  * zyga looks for that
[13:36] <Chipaca> zyga: yes (but don't you need to enable it?)
[13:37] <zyga> Chipaca: I'm also complaining about short daylight cycle here :( software cannot fix that
[13:37] <Chipaca> zyga: i'm pretending not to notice
[13:37] <zyga> enabled :)
[13:38] <zyga> thank you for reminding me about it
[13:38]  * kalikiana bbl
[13:39] <zyga> Chipaca: on the other hand I moved my systems to a darker green tinted wallpapers/color schemes and I like that a lot
[13:40] <Chipaca> man, apt needs a 'find' alias now :-(
[13:41] <mborzecki> zyga: about teleconsole on fedora, `snap run --shell teleconsole` should not fail with 'execv failed: No such file or directory', right?
[13:42] <zyga> mborzecki: right
[13:42] <zyga> mborzecki: it looks like it's trying to exec /usr/lib/snapd/snap-exec
[13:42] <zyga> vs libexec
[13:42] <zyga> (just a guess)
[13:42] <zyga> I haven't rebooted F27 after updates
[13:42] <zyga> 47% installing
[13:42] <mborzecki> but it works for 'hello' snap
[13:43] <zyga> classic doesn't do redirection
[13:43] <zyga> I think I know what's going on
[13:43] <zyga> we disabled classic on fedora
[13:43] <zyga> so you did the symlink
[13:43] <zyga> but the code that runs classic stuff had hardcoded assumptions on dir layout
[13:43] <zyga> if you print execed programs you will see (probably) it running the wrong directory
[13:44] <pedronis> Chipaca: #4389 and #4392 are probably PRs you could review
[13:44] <mup> PR #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] <mup> PR #4392: many: refresh with appropriate creds <Created by pedronis> <https://github.com/snapcore/snapd/pull/4392>
[13:44] <zyga> mborzecki: as a quick test add a symlink /usr/lib/snapd -> /usr/libexec/snapd
[13:46] <mborzecki> zyga: right, works now
[13:46] <zyga> mborzecki: cool, that should be as simple fix
[13:46] <zyga> *a
[13:51]  * zyga removes 34,028 messages from launchpad from his inbox
[13:53] <mborzecki> zyga: 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 exits
[13:55] <zyga> no no, the symlink is just a test
[13:55] <zyga> we don't need it
[13:55] <zyga> but we now know exactly what is wrong
[13:55] <zyga> it's a simple bug in cmd.go
[13:55] <zyga> or cmd_run.go
[13:58] <mborzecki> zyga: https://paste.ubuntu.com/26306484/ like this? :)
[13:58] <zyga> yes
[13:58] <zyga> looks excellent
[13:58] <zyga> thank you!
[13:58] <zyga> mborzecki: look at how this behaves when classic + reexec is in use
[13:59] <zyga> mborzecki: but this looks ok for fedora
[14:04] <mborzecki> zyga: https://paste.ubuntu.com/26306521/
[14:05] <mborzecki> and teleconsole works now, yay
[14:05] <zyga> I think the test is wrong
[14:05] <zyga> if you are reexeed you want:
[14:05] <zyga> path to core snap in the distro + core libexec dir
[14:06] <zyga> e.g. /var/lib/snapd/snap/core/current/usr/lib/snapd/snap-exec
[14:06] <zyga> which combines a fedora-specific snap mountdir + core layout that contains snap-exec
[14:06] <zyga> you can perhaps simplify
[14:06] <mborzecki> hmm right
[14:06] <zyga> by using the same trick that cmd.go does
[14:06] <zyga> find your own executable
[14:06] <zyga> and then run snap-exec as your peer
[14:07] <zyga> though because snap is not in the same directory as snap-exec or snap-confine, this may be more cloudy
[14:07] <zyga> I'd vote for a direct test + explicit join
[14:08] <mborzecki> mhm
[14:08] <mborzecki> looks like we have a way forward, i'll open a PR later
[14:09] <zyga> thank you :)
[14:09] <mborzecki> been fun chasing it
[14:09] <mborzecki> still now idea why strace hangs occasionally
[14:10] <mborzecki> anyways, i'm off to pick up the kids
[14:19] <Chipaca> so, the completion tests do take 50s per
[14:20] <Chipaca> they can be made significantly faster by dropping a delay and/or by bumping a send rate
[14:20] <Chipaca> these two things exist because without them the tests were flaky
[14:20] <Chipaca> should I try dropping them to see if they continue being flaky?
[14:29] <zyga> Chipaca: worth a try, I wonder if we could do _something_ to become reliable
[14:32] <kalikiana> re
[14:33] <mup> PR 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:34] <mup> PR snapd#4427 closed: snap: fix snap find " " output <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4427>
[14:46]  * zyga break for small taxes and paperwork
[14:47] <elopio> good morning team!
[14:58] <jdstrand> kyrofa: 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 videos
[14:59] <jdstrand> kyrofa: and happy new year :)
[15:02] <sergiusens> elopio good morning
[15:04] <kyrofa> Happy New Year everyone!
[15:04] <kyrofa> Morning jdstrand :)
[15:05] <sergiusens> kyrofa hey
[15:05] <kyrofa> jdstrand, 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 useful
[15:06] <kyrofa> Hey there sergiusens, nice break?
[15:08] <jdstrand> kyrofa: with the nextcloud box, it stores its data not in /media by default since it formats the drive and installs itself on it
[15:08] <popey> Saviq: is there an easy way to make a multipass disk bigger, 2.1GB seems awfully small for the initial size
[15:08] <kyrofa> jdstrand, yeah, by default it uses $SNAP_COMMON
[15:09] <Saviq> popey: on launch, pass `--disk 5G` or so
[15:09] <jdstrand> kyrofa: 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 people
[15:09] <popey> Saviq: but not after I made it?
[15:09] <Saviq> popey: 2G is what the default size is of the cloud images
[15:09] <jdstrand> kyrofa: anyway, food for thought
[15:09] <Saviq> popey: I think someone wrote on the forums how to do it after launch
[15:09] <Saviq> it's not trivial since you have to resize the FS
[15:10] <popey> found it, thanks
[15:10] <Saviq> so 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] <popey> I'll destroy and start again :(
[15:11] <jdstrand> kyrofa: also, is the nextcloud-client expected to be fully functional? (excepting auto-start)
[15:11] <Saviq> popey: we might bump the default anyway, it's qcow so it's not stealing your storage straight away
[15:12] <kyrofa> jdstrand, 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] <jdstrand> kyrofa: (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] <jdstrand> )
[15:13] <kyrofa> Hmm, that sounds odd
[15:14] <zyga> jdstrand, kyrofa: hey :)
[15:14] <jdstrand> happy new year zyga :)
[15:14] <kyrofa> Happy new year, zyga! How was break?
[15:14] <zyga> kyrofa: exhausting :) too long I would say
[15:14] <kyrofa> Hahaha
[15:14] <zyga> kyrofa: I'm happy to be back :)
[15:15] <zyga> how are you guys doing? I heard US has move to ice age
[15:16] <zyga> *moved
[15:16] <sergiusens> kyrofa good, yourself?
[15:17] <sergiusens> zyga I want to be on a break for longer!
[15:17] <kyrofa> Yeah me too
[15:17] <kyrofa> sergiusens, short, but good
[15:17] <zyga> sergiusens: for how long were you away?
[15:19] <sergiusens> zyga just the week in between the two big holidays
[15:22] <ppisati> ogra_: i think some files were moved inside os.snap, without updating the kernel plugin
[15:22] <ppisati> ogra_: can you give your opinion on this?
[15:22] <ppisati> ogra_: https://bugs.launchpad.net/snapcraft/+bug/1740882
[15:22] <mup> Bug #1740882: Missing initrd.img-core in os.snap <Snapcraft:New> <https://launchpad.net/bugs/1740882>
[15:29] <kalikiana> kyrofa: Happy New Year, Franz ;-)
[15:30] <kyrofa> Hey there kalikiana! Back at you. How was your break?
[15:30] <kalikiana> Very necessary, haha
[15:35] <kyrofa> No kidding
[15:45] <mup> PR 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:48] <Chipaca> mborzecki: mvo: anything further blocking #4394?
[15:48] <mup> PR #4394: snap: give the snap.Container interface a Walk method <Created by chipaca> <https://github.com/snapcore/snapd/pull/4394>
[16:06] <ogra_> ppisati, i didnt touch anything (still on vacation) ... if anyone moved anything around it would likely be mvo though
[16:06] <ppisati> mvo: ^
[16:07] <ppisati> ogra_: ta
[16:10] <mvo> ppisati: probably https://github.com/snapcore/core/pull/63 but its slightly puzzling, there is a symlink there - is this symlink broken?
[16:10] <mup> PR 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] <mvo> ppisati: fwiw, this was requested to be a symlink instead of a copy to help the store with diff generation
[16:13] <ogra_> yeah, looks more like an issue with the kernel plugin not respecting links
[16:17] <ppisati> nope, 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 there
[16:17] <ppisati> anyhow, thnaks for the confirmation, i alread attached a patch to the LP that fixes it
[16:18] <ogra_> yeah, looks good to me
[16:22] <mup> PR snapd#4431 opened: snap: make `snap info invalid-snap` output more user friendly <Created by mvo5> <https://github.com/snapcore/snapd/pull/4431>
[16:24] <mvo> zyga: iirc we discussed lp: #1705549 a while ago, do you remember the details? I would like to update the bugreport
[16:24] <mup> Bug #1705549: Snaps in lxc can't refresh if old revisions need to be cleaned <snapd:New> <https://launchpad.net/bugs/1705549>
[16:24] <zyga> mvo: yes
[16:24] <zyga> mvo: 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 it
[16:25] <mup> PR 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] <mvo> zyga: anything I could point to in the bug?
[16:25] <zyga> mvo: prior attempts either worked but were too open for security or didn't work
[16:25] <zyga> mvo: I can look for several PRs but those are all closed now
[16:25] <zyga> one has some interesting discussion
[16:25] <zyga> let me find it
[16:26] <zyga> mvo: this is one https://github.com/snapcore/snapd/pull/4258
[16:26] <mup> PR #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:27] <mvo> zyga: thanks
[16:43] <mvo> jdstrand: 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:58] <leftyfb> If 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?
[17:00] <kyrofa> leftyfb, it's just a runtime for other snaps
[17:00] <leftyfb> ok, good to know
[17:00] <leftyfb> hey kyrofa!
[17:01] <leftyfb> kyrofa: sorry we haven't got back to you
[17:01] <mup> PR 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] <leftyfb> kyrofa: things got kinda busy with the holidays/retail
[17:01] <kyrofa> Haha, same here! No worries
[17:01] <jdstrand> mvo: I'll take a look and comment
[17:02] <leftyfb> kyrofa: snappy is something we'll definitely be looking at this year. We first need to get all the customers running Xenial
[17:03] <kyrofa> leftyfb, understood! You know where to find me, always happy to help :)
[17:04] <mvo> jdstrand: also lp: #1689577 would be appreciated. hopefully quick and easy (both of them :)
[17:04] <mup> Bug #1689577: bad system call "shutdown" trying to use systemd-cat from a snap <snapd:Incomplete> <https://launchpad.net/bugs/1689577>
[17:04] <kyrofa> leftyfb, and I'm glad to see you around!
[17:05]  * kalikiana going to call it a day - see you guys tomorrow
[17:31] <mup> PR snapcraft#1840 opened: docker: instructions to build from the snap <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1840>
[18:07] <kyrofa> Hey zyga, are you still working on that lxd snaps not updating bug?
[18:08] <zyga> hey
[18:08] <zyga> not at present but I will likely come back to it soon
[18:08] <zyga> (this week)
[18:08] <zyga> I'll focus on landing https://github.com/snapcore/snapd/pull/4329
[18:08] <mup> PR #4329:  cmd/snap-confine: discard stale mount namespaces (v2) <Created by zyga> <https://github.com/snapcore/snapd/pull/4329>
[18:10] <kyrofa> zyga, okay
[18:11] <kyrofa> sergiusens, the circle ci tutorial, would you like that to be a snappy-docs document, or a tutorials.ubuntu.com document?
[18:11] <zyga> HEADS UP EVERYONE: 2.30 is now in stable
[18:12] <sergiusens> kyrofa tutorials, to match the travis one
[18:12] <kyrofa> Oh riiight, forgot about the travis one
[18:20] <zyga> mvo: so, who can we ask from the store?
[18:32] <kyrofa> sergiusens, 1-1 today? I don't have anything particularly new to share, so we can skip if you like
[18:41] <kyrofa> I'll assume that's a skip
[18:57] <sergiusens> kyrofa yeah
[18:57] <sergiusens> elopio let's skip unless you have something
[18:58] <elopio> sergiusens: next week I will have something :)
[19:37] <mup> PR snapcraft#1841 opened: Update test_export_login.py <Created by heesen3> <https://github.com/snapcore/snapcraft/pull/1841>
[20:43] <mup> PR snapcraft#1833 closed: cli: humanize push message <Created by Sheogorath2> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1833>