=== AmarokNelg_ is now known as AmarokNelg | ||
mborzecki | morning | 06:06 |
---|---|---|
zyga | good morning! :) | 07:26 |
* zyga looks at a mountain of email | 07:27 | |
mborzecki | zyga: hey | 07:30 |
zyga | hey :) | 07:31 |
zyga | happy new year | 07:31 |
mborzecki | likewise :) | 07:32 |
mborzecki | zyga: did you enjoy your break? | 07:33 |
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:35 |
kalikiana | good morning | 07:46 |
kalikiana | happy 2018 | 07:46 |
zyga | good morning kalikiana! | 07:52 |
pstolowski | mornings! | 08:10 |
mborzecki | pstolowski: hey | 08:11 |
zyga | good morning koza | 08:19 |
* zyga reboots for some updates | 08:31 | |
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:44 |
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:47 |
mborzecki | i'm poking around yocto support, slow progress however | 08:48 |
* mvo nods | 08:48 | |
mvo | mborzecki: yocto> nice to hear | 08:48 |
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:49 | |
mup | Issue snapcraft#1834 closed: Feature Request: NGINX plugin <Created by jamesbeedy> <Closed by kalikiana> <https://github.com/snapcore/snapcraft/issue/1834> | 08:50 |
mup | PR snapd#4427 opened: snap: fix snap find " " output <Created by mvo5> <https://github.com/snapcore/snapd/pull/4427> | 09:03 |
mborzecki | zyga not around? | 09:22 |
mvo | mborzecki: heh | 09:22 |
zyga | hey mvo :) long time no see | 09:24 |
zyga | how are you doing? | 09:24 |
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:29 |
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: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 break | 09:36 | |
=== __chip__ is now known as Chipaca | ||
zyga | hey Chipaca | 09:44 |
Chipaca | zyga! | 09:44 |
Chipaca | how's things? | 09:44 |
zyga | Chipaca: snappy :) | 09:44 |
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:45 |
zyga | Chipaca: I'm doing great, disappointed for not having much snow this xmas | 09:46 |
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:47 |
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:48 |
Chipaca | mvo: i'll preted not to know half of your hacking today is from an unconventional seat | 09:49 |
mvo | lol | 09:49 |
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:50 |
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:51 |
Chipaca | mvo: is the client side of the bad query kind coming, or not needed? | 09:54 |
mvo | Chipaca: I can add it, I think it makes sense, I was mostly doing it for roberts glib based client implementation | 09:58 |
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 | 09:59 |
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:00 |
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:01 |
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:02 |
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:03 | |
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:04 |
zyga | it's from _november_ and has one review | 10:05 |
* zyga boots subsequent VM for the round of updates | 10:05 | |
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:06 |
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:09 |
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:10 |
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:11 |
Chipaca | zyga: around … and this is from memory … the security backend setup loop? something like that | 10:12 |
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:13 |
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:14 |
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:15 |
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:16 |
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:17 |
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:49 |
mup | PR snapd#4405 closed: taskrunner/many: KnownTaskKinds helper <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/4405> | 10:50 |
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:51 |
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:55 |
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> | 10:57 |
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:02 |
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:04 |
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:05 |
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:06 |
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:07 |
Chipaca | bit of paper | 11:08 |
* Chipaca has no idea where he picked up that word | 11:08 | |
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:12 |
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:13 |
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:14 |
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:15 |
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:17 | |
zyga | mvo: looking | 11:19 |
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:20 |
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:22 |
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:23 |
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:25 |
mvo | classic | 11:26 |
zyga | mvo: aha, let me see master quickly now, just adding some logging | 11:26 |
mvo | ok | 11:27 |
zyga | mvo: logging in tests says it's not broken, may be broken later, trying with real snaps now | 11:29 |
mvo | zyga: ta | 11:30 |
zyga | mvo: hmm, mir-kiosk is not in the store? | 11:31 |
zyga | ah, edge | 11:31 |
mvo | yes | 11:32 |
zyga | ha | 11:33 |
zyga | indeed | 11:33 |
zyga | the reason is simple, autoconnect | 11:33 |
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:34 |
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:35 | |
mvo | zyga: iirc it was still showing as connected when I tried | 11:37 |
zyga | mvo: yes, there's something buggy going on | 11:38 |
zyga | mvo: it should say it got disconnected but didn't | 11:38 |
* mvo nods | 11:42 | |
zyga | mvo: oddly repo.DisconnectSnap returns [] so something is very fishy there | 11:42 |
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:50 |
pstolowski | zyga, is this re 1708703? | 11:53 |
zyga | yes | 11:53 |
zyga | I see the bug now | 11:53 |
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 :/ | 11:55 |
zyga | fixed :) | 12:01 |
zyga | now for some tests | 12:02 |
pstolowski | zyga, thank you | 12:04 |
* pstolowski lunch | 12:15 | |
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:16 |
* mvo hugs zyga | 12:21 | |
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:30 |
sergiusens | morning | 12:51 |
zyga | hey sergiusens :) | 12:54 |
kalikiana | morning sergiusens | 13:06 |
kalikiana | and happy new year | 13:06 |
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:08 |
* kalikiana going to take a lunch break in ~15 | 13:19 | |
Chipaca | mborzecki: in https://paste.ubuntu.com/26306252/, what is the 1st column? | 13:29 |
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:30 |
mborzecki | Chipaca: same dataaset, ubuntu-16.04-64 only this time: https://paste.ubuntu.com/26306338/ | 13:32 |
Chipaca | mborzecki: hmmm | 13:32 |
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:33 |
Chipaca | tests/main/completion vs tests/completion :-) | 13:34 |
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:35 |
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:36 |
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:37 |
zyga | thank you for reminding me about it | 13:38 |
* kalikiana bbl | 13:38 | |
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:39 |
Chipaca | man, apt needs a 'find' alias now :-( | 13:40 |
mborzecki | zyga: about teleconsole on fedora, `snap run --shell teleconsole` should not fail with 'execv failed: No such file or directory', right? | 13:41 |
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:42 |
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:43 |
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:44 |
=== __chip__ is now known as Chipaca | ||
mborzecki | zyga: right, works now | 13:46 |
zyga | mborzecki: cool, that should be as simple fix | 13:46 |
zyga | *a | 13:46 |
* zyga removes 34,028 messages from launchpad from his inbox | 13:51 | |
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:53 |
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:55 |
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:58 |
zyga | mborzecki: but this looks ok for fedora | 13:59 |
mborzecki | zyga: https://paste.ubuntu.com/26306521/ | 14:04 |
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:05 |
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:06 |
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:07 |
mborzecki | mhm | 14:08 |
mborzecki | looks like we have a way forward, i'll open a PR later | 14:08 |
zyga | thank you :) | 14:09 |
mborzecki | been fun chasing it | 14:09 |
mborzecki | still now idea why strace hangs occasionally | 14:09 |
mborzecki | anyways, i'm off to pick up the kids | 14:10 |
Chipaca | so, the completion tests do take 50s per | 14:19 |
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:20 |
zyga | Chipaca: worth a try, I wonder if we could do _something_ to become reliable | 14:29 |
kalikiana | re | 14:32 |
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:33 |
mup | PR 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 paperwork | 14:46 | |
elopio | good morning team! | 14:47 |
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:58 |
jdstrand | kyrofa: and happy new year :) | 14:59 |
sergiusens | elopio good morning | 15:02 |
kyrofa | Happy New Year everyone! | 15:04 |
kyrofa | Morning jdstrand :) | 15:04 |
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:05 |
kyrofa | Hey there sergiusens, nice break? | 15:06 |
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:08 |
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:09 |
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:10 |
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:11 |
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:12 |
kyrofa | Hmm, that sounds odd | 15:13 |
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:14 |
zyga | how are you guys doing? I heard US has move to ice age | 15:15 |
zyga | *moved | 15:16 |
sergiusens | kyrofa good, yourself? | 15:16 |
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:17 |
sergiusens | zyga just the week in between the two big holidays | 15:19 |
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:22 |
kalikiana | kyrofa: Happy New Year, Franz ;-) | 15:29 |
kyrofa | Hey there kalikiana! Back at you. How was your break? | 15:30 |
kalikiana | Very necessary, haha | 15:30 |
kyrofa | No kidding | 15:35 |
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:45 |
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> | 15:48 |
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:06 |
ppisati | ogra_: ta | 16:07 |
=== ahasenac` is now known as ahasenack | ||
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 |
=== ahasenack is now known as Guest12330 | ||
mvo | ppisati: fwiw, this was requested to be a symlink instead of a copy to help the store with diff generation | 16:10 |
ogra_ | yeah, looks more like an issue with the kernel plugin not respecting links | 16:13 |
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:17 |
ogra_ | yeah, looks good to me | 16:18 |
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:22 |
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:24 |
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:25 |
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:26 |
mvo | zyga: thanks | 16:27 |
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:43 |
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? | 16:58 |
kyrofa | leftyfb, it's just a runtime for other snaps | 17:00 |
leftyfb | ok, good to know | 17:00 |
leftyfb | hey kyrofa! | 17:00 |
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:01 |
leftyfb | kyrofa: snappy is something we'll definitely be looking at this year. We first need to get all the customers running Xenial | 17:02 |
kyrofa | leftyfb, understood! You know where to find me, always happy to help :) | 17:03 |
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:04 |
* kalikiana going to call it a day - see you guys tomorrow | 17:05 | |
mup | PR snapcraft#1840 opened: docker: instructions to build from the snap <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1840> | 17:31 |
kyrofa | Hey zyga, are you still working on that lxd snaps not updating bug? | 18:07 |
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:08 |
kyrofa | zyga, okay | 18:10 |
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:11 |
sergiusens | kyrofa tutorials, to match the travis one | 18:12 |
kyrofa | Oh riiight, forgot about the travis one | 18:12 |
zyga | mvo: so, who can we ask from the store? | 18:20 |
kyrofa | sergiusens, 1-1 today? I don't have anything particularly new to share, so we can skip if you like | 18:32 |
kyrofa | I'll assume that's a skip | 18:41 |
sergiusens | kyrofa yeah | 18:57 |
sergiusens | elopio let's skip unless you have something | 18:57 |
elopio | sergiusens: next week I will have something :) | 18:58 |
mup | PR snapcraft#1841 opened: Update test_export_login.py <Created by heesen3> <https://github.com/snapcore/snapcraft/pull/1841> | 19:37 |
mup | PR 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!