/srv/irclogs.ubuntu.com/2019/04/02/#snappy.txt

jameshpopey: the gtk-2-themes content interface should connect to the gtk-common-themes snap: not gtk2-common-themes01:26
=== chihchun_afk is now known as chihchun
mborzeckimorning05:05
mupPR snapd#6602 closed: cmd,interfaces: replace local helpers with cmd.InternalToolPath <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6602>05:26
zygagood morning05:56
zygamborzecki: I sent some goodie last night05:56
mborzeckizyga: hey, morning05:56
zygaobviously red05:57
zygaeh05:57
zygahttps://github.com/snapcore/snapd/pull/667405:57
mupPR #6674: tests: use apt via eatmydata <Created by zyga> <https://github.com/snapcore/snapd/pull/6674>05:57
zygahmm, I will need to look05:58
zygaI install eatmydata but ... somehow not?05:58
zyganot sure05:58
zygamborzecki: offtopic, I need a review for https://github.com/snapcore/snapd/pull/6675 - 2 lines of changes in code05:58
mupPR #6675: cmd/snap-confine: allow using tools from snapd snap <Created by zyga> <https://github.com/snapcore/snapd/pull/6675>05:59
mborzeckiwonder why it doesn't work on debian06:01
mborzeckieatmydata should end up in $PATH, not sure why the tests are getting eatmydata: command not found06:04
mborzeckimeanwhile, one of selinux branches is hitting this: https://paste.ubuntu.com/p/kZkFvkGzyp/ somehow there's a file named 'foo' that ended up as unlabeled_t06:06
mborzeckianyone https://github.com/snapcore/snapd/pull/6654 ?06:18
mupPR #6654: packaging/fedora, tests/upgrade/basic: patch existing mount units with SELinux context on upgrade <SELinux> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6654>06:18
zygaI will review shortly06:20
zygadoing now06:30
mupPR snapd#6675 closed: cmd/snap-confine: allow using tools from snapd snap <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6675>06:31
=== mvo is now known as Guest61543
=== Guest61543 is now known as mvo_
mvo_zyga: hey, good morning - nice job on the eatmydata branch06:40
zygahey06:40
zygait doesn't pass yet06:40
zygasomething wonky on debian06:40
mvo_zyga: unrelated I suppose?06:41
zygaI'm reviewing maciej's branch and will get back to it06:41
zygamvo_: related!, eatmydata is not on path somehow?06:41
zygaI will debug shortly06:41
mvo_oh, interessting06:41
zygamvo_: I used qemu all day yesterday, I have several more speed-up patches06:41
zygamvo_: but providing quantitative data takes time so it will be a while for the next round06:41
mvo_zyga: ok06:42
zygamborzecki: reviewed06:44
zygamvo_: can you please look at https://github.com/snapcore/snapd/pull/667306:44
mupPR #6673: cmd,tests: forcibly discard mount namespace when bases change <Created by zyga> <https://github.com/snapcore/snapd/pull/6673>06:45
zygamvo_: mainly to decide if the algorithm makes sense06:45
zygamvo_: or if we should do something entirely different06:45
mvo_zyga: yeah, I look in a wee bit, just looking at a bugreport about a potential mem leak06:45
zygaooo06:45
mborzeckizyga: thanks06:47
mborzeckimvo_: morning06:47
mvo_hey mborzecki06:50
zygamvo_: https://github.com/snapcore/snapd/pull/6667 is +2 and green06:50
mupPR #6667: tests: enable tests that write /etc/{hostname,timezone} on core18 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6667>06:50
mvo_zyga: I will comment, we probably want to wait with this until the sru is officially released06:52
=== pstolowski|afk is now known as pstolowski
pstolowskimornings07:00
zygagood morning pawel07:01
mvo_pstolowski: hey, good morning07:03
zygahttps://travis-ci.org/snapcore/snapd/jobs/51452587407:39
zygaWAT?07:39
zyga../../../golang.org/x/sys/unix/zsyscall_darwin_amd64.1_11.go:687:22: undefined: SYS_CLOCK_GETTIME07:39
zygathat's a darwin build07:39
zygadid go just broke on us?07:40
pstolowskizyga: i hit something similiar with golang on Mac recently; some x/sys/unix stuff needed `go get`07:46
zygahmm?07:46
zygaare you saying you had to update the version of something or ... ?07:47
zygacan you rephrase please07:47
pstolowskizyga: i had to manually pull some x/sys packages on mac as apparently they were not shipped with default installation of go; that was a few months ago though when i first installed go on mac, don't remember the details07:48
mborzeckianyone up for a simple review https://github.com/snapcore/snapd/pull/6672 ?07:50
mupPR #6672: metautil, snap: extract yaml value normalization to a helper package <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6672>07:50
mborzeckipedronis: btw. ^^^ use metautil pacakge name you suggested07:50
pedronismborzecki: thx, will look in  aibt07:52
pedronis*a bit07:52
mborzeckipedronis: thanks!07:52
* zyga -> coffee, sorry still sleepy somehow08:00
mvo_mborzecki: hey, if you have some spare cycles it would be great if you could have a look athttps://bugs.launchpad.net/ubuntu/+source/snapd/+bug/182273808:08
mupBug #1822738: memleak in 2.38+ ? <snapd (Ubuntu):New> <https://launchpad.net/bugs/1822738>08:08
mborzecki../../../golang.org/x/sys/unix/zsyscall_darwin_amd64.1_11.go:687:22: undefined: SYS_CLOCK_GETTIME heh, what zyga pasted before08:15
mborzeckianything we can do about that?08:16
pedronismborzecki: stubbing usually, but what new code bring that in?08:18
mborzeckipedronis: looks like it's from golang.org/x/sys/unix08:18
pedronisbut it's new?08:19
mborzeckiand comes up when go getting it08:19
mborzeckiwe didn't see it yday08:19
pedronisyea, so it's new08:19
pedronisanyway Chipaca was the last one untangling that stuff08:19
pedronishe might be able to help08:19
* pedronis quick errands08:19
mborzeckihm govendor lists /x/sys/unix as external dep, we don't pin it to a specific revision08:21
mborzeckihaha the last commit there is titled: 'unix: add SysctlClockinfo on darwin'08:22
mvo_:P08:22
mupPR snapd#6676 opened: travis: use Go 1.11 os OSX <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6676>08:24
dot-tobias hello everyone08:25
mborzeckilet's see if 6676 helps08:25
zygamborzecki: there is no SYS_CLOCK_GETTIME on macos08:26
zygaunless it is coming in a unreleased beta08:26
mborzeckisounds like we should pin /x/sys/unix08:27
mborzeckiand it doesn't work08:28
mborzeckizyga: that's the change in /x/sys/unix https://go-review.googlesource.com/c/sys/+/17029708:29
zygalooking08:30
zygatoo bad it's not really explained08:32
zygathere's no SYS_CLOCK_GETTIME on any file in my macos install08:32
zyga(as spotlight can tell me in a fraction of a second)08:32
mborzeckiheh, the change landed just today morning08:33
zygaI bet it must be coming from a test system with macos 10.1508:33
zygahttps://go-review.googlesource.com/c/sys/+/17029708:34
zygaI left a comment08:35
=== chihchun is now known as chihchun_afk
Chipacamvo_: pedronis: http://paste.ubuntu.com/p/yZkPWJ3FmH/ ← add this to snapd to test the memleak?08:43
mvo_Chipaca: nice one08:43
Chipacaer08:44
Chipacanil, not nul, but you get the idea08:44
mupPR snapd#6677 opened: vendor: pin golang.org/x/sys/unix to a revision before SYS_CLOCK_GETTIME on OSX <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6677>08:45
mborzeckiok, maybe 6677 will get us somewhere08:46
mupPR snapd#6676 closed: travis: use Go 1.11 on OSX <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/6676>08:46
zygamborzecki: ha08:48
zygahttps://go-review.googlesource.com/c/sys/+/17029708:48
mborzeckizyga: hm?08:48
zygamborzecki: ^08:48
zyga^_^08:48
mborzeckinice08:48
zygaI really like this about software08:49
zygathis would take a week conf call to untangle otherwise08:49
zygahttps://go-review.googlesource.com/c/sys/+/17029708:50
zygahttps://go-review.googlesource.com/c/sys/+/170299/ sorry08:50
Chipacazyga: mborzecki: what broke that's needing the pin?08:51
Chipacaah08:51
mborzeckiChipaca: golang.org/x/sys/unix ^^08:51
zygait's almost solved now08:51
Chipacasomebody broke darwin i see :-)08:51
zygamborzecki: I approved the fix in go now08:52
zygaperhaps it will land in a moment and we can just rebuild08:52
zygait's funny that ..08:52
zygayou know08:52
zygathere's no CI that failed on this before08:52
zygajust saying08:52
zygasnapd ci is better than go's apparently ;)08:52
mborzeckimaybe the don't care about darwin that much08:53
mborzeckicrowdsourcing the CI effectively08:53
Chipacabefore you get your hubris wound up, remember these interactions are tricky to get right08:53
zygahmm?08:53
zygago test on the tree picks this up08:54
Chipacaon which tree?08:54
zygasys/unix08:54
zygait just doesn't compile now08:54
=== chihchun_afk is now known as chihchun
Chipacapedronis: what's a good message/code combo for 'hook present, did not fail, but did not call snapctl set-health'?09:06
pedronisChipaca: I'm not entirely sure we should set those09:06
Chipacapedronis: just leave it at "unknown"?09:06
pedronisfor a start09:06
ograogra@anubis:~$ snap install freecad09:07
ograerror: snap "freecad" not found09:07
ograHMMMM ...09:07
* ogra looks at https://snapcraft.io/freecad ... 09:07
ogra(i can install other snaps ... why not this one ?)09:07
ograinterestingly snap info freecad doesnt work either for me ...09:09
zygamy laptop got fixed09:10
zygaI will go and fetch it soon09:10
ograhmm, interesting ... doesnt work on my laptop either09:11
pedronisChipaca: the other option is to set the code to something like snapd-hook-run or snapd-hook-ran09:13
Chipacapedronis: snapd-hook-is-full-of-it09:14
Chipacapedronis: snapd-hook-was-silly-banana09:14
Chipaca:-)09:14
Chipacapedronis: I'd rather set it to something so we can tell the cases apart, yes09:14
Chipacapedronis: but i'm not too bothered if you'd rather wait on that09:14
mupPR snapd#6678 opened: cmd/snap, api: use DebugGet for timings <Created by stolowski> <https://github.com/snapcore/snapd/pull/6678>09:15
mborzeckiso if the host is short on memory to start with, snapd takes around snapd-binary-size + whatever runtime alloces, running 'snap foo' is making the situation even worse, because that'll load up another snap-binary-size blob into memory09:15
mborzeckisnapd is ~20M here, snap ~15M that's 35M+ to run snap refresh, not mentioning the runtime stuff09:16
pstolowskiChipaca: addressed your suggestion from timings review in #667809:16
mupPR #6678: cmd/snap, api: use DebugGet for timings <Created by stolowski> <https://github.com/snapcore/snapd/pull/6678>09:16
Chipacapstolowski: why 24 commits in that?09:16
pstolowskiChipaca: because it depends on the other PR09:17
pedronismborzecki: 6672 looks good, I would add a package doc comment though, made a suggestion.09:18
pstolowskiChipaca: it's really this: https://github.com/snapcore/snapd/pull/6678/commits/ef29d5563f677e0943088369fe92ad8b73bad86709:18
mupPR #6678: cmd/snap, api: use DebugGet for timings <Created by stolowski> <https://github.com/snapcore/snapd/pull/6678>09:18
mborzeckipedronis: thanks09:18
Chipacapstolowski: my last comment was because you don't really need the url key to have the right name09:29
Chipacapstolowski: GET /v2/debug?aspect=change-timings&arg=<chg id> works just as well09:30
Chipacapstolowski: in fact I'm starting to prefer it to the other more magical one :-)09:30
mborzeckiChipaca: do you happen to have the package size breakdown graph for 'snap' too?09:31
Chipacamborzecki: yes09:31
Chipacamborzecki: read the message again09:31
mborzeckiuhh ;) i'm getting blind09:31
Chipacamborzecki: 's ok, we'll get you a dog09:32
Chipacanot a seeing eye dog, those are expensive09:32
pstolowskiChipaca: ty, +109:32
mborzeckiChipaca: haha :)09:32
Chipacamborzecki: I'd like something like this but for the running thing09:33
Chipacanot holding my breath for it though09:33
mborzeckiChipaca: pprof maybe? though it'd be more live allocations09:34
mborzeckihad a PR that would expose a pprof endpoint over the api09:34
ChipacaI just want to know how much RSS is added by just importing, say, godb09:35
mborzeckiah, there was a thing for this09:35
Chipacas/godb/bolt/ i meant09:36
* Chipaca needs more coffee09:36
=== mvo is now known as Guest18430
=== Guest18430 is now known as mvo_
* zyga wonders why a particular test happens to behave in a particular way09:39
zygadebugging galore09:39
mborzeckiChipaca: https://paste.ubuntu.com/p/FFbYCRk68N/ though i had trouble with newer versions of go as they changed where *.a are placed and the naming, and the sizes don't add up correctly09:40
Chipacamborzecki: go clean -cache, and/or GODEBUG=gocacheverify=1 help with that?09:42
Chipacamborzecki: also the -a flag to go build09:43
mborzeckiChipaca: i'd prefer to see a tool that's part of the go suite, which souds like an interesting little project itself09:52
mborzeckiChipaca: there was a tool like this in busybox tree, but it used objdump and friends to analyze and produce a per symbol breakdown09:53
mborzeckiand it assumed one is using C obviously09:54
popeyjamesh: tried that, still get old school looking stuff. Is your forum post in need of updating. As it doesn'r work here.09:58
pedronismborzecki: Chipaca: put some notes of things you mentioned or I chatted about with mvo in the bug10:08
mborzeckipedronis: i'm checking out some curl queries to put there10:09
=== chihchun is now known as chihchun_afk
pedronismborzecki: added one more obvious thing there10:10
=== chihchun_afk is now known as chihchun
uebera||Hi. Should I just ignore #snappy, #snapframework or are they reserved for specific topics?11:03
uebera||So, what's the easiest way (if any) to use /snap/core/current/usr/bin/gpg, presuming it's newer than the version shipped with Ubuntu Xenial?11:04
ograsince the core snap is built from the ubuntu xenial archive, it wont be newer11:07
uebera||ogra: I see. Thanks!11:08
pedroniszyga: for clarity I commented on 6502 that keeping the two functions (Soft,Hard) matches how we discussed things and should be kept11:09
zygapedronis: I agree, thank you for clarifying that11:10
=== chihchun is now known as chihchun_afk
zygaApr 02 13:14:29 fyke kernel: snapcraftctl[7576]: segfault at 2d ip 00007fd0131e1681 sp 00007ffdc8e9fe20 error 4 in ld-2.28.so[7fd0131d8000+20000]11:15
zygaApr 02 13:14:29 fyke kernel: Code: 54 24 18 e8 71 72 00 00 4c 8b 54 24 18 85 c0 0f 85 d4 0a 00 00 0f 1f 40 00 48 83 c5 01 49 39 ea 0f 86 23 06 00 00 49 8b 04 ec <48> 8b 58 28 48 3b 9c 24 e8 00 00 00 74 e1 8b 34 24 85 f6 74 09 f611:15
zyganot my luckiest day11:15
Chipacawait, where has the morning gone11:21
Chipacastaahp11:21
zygaChipaca: what? :)11:21
zygaoh, making progress :)11:22
Chipacazyga: my morning vanished in a flurry of unproductive nonsense11:24
Chipacathat's what :-/11:24
zygaChipaca: you didn't see my morning :)11:30
zygaChipaca: I'm stumbling blind around parts I'm clueless to11:31
zygatrying to make some progress but slow at it11:31
mborzeckidamn, #6677 travis job failed, desktop-portal-filechooser11:33
mupPR #6677: vendor: pin golang.org/x/sys/unix to a revision before SYS_CLOCK_GETTIME on OSX <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6677>11:33
pstolowskican #6659 land?11:35
mupPR #6659: snapcraft: build static fontconfig in the snapd snap <Created by mvo5> <https://github.com/snapcore/snapd/pull/6659>11:35
=== ricab is now known as ricab|lunch
pedronispstolowski: as such yes, but it needs work somewhere else, I'm not sure how mvo is tracking that11:53
pedronispstolowski: I would let mvo deal with it11:54
pstolowskipedronis: ack.. i was trying to find something that can land.. our queue keeps growing11:54
mborzeckirevived my branch to expose pprof over snapd api11:55
=== chihchun_afk is now known as chihchun
* cmatsuoka wonders what "duplicate the fake tar sleeps" means11:58
* Chipaca reads the news and considers sending zyga a big box of harry potter books12:01
pedronismborzecki: commented in the bug that master has timings too, which 2.38 didn't have12:02
zygaChipaca: no need, I have plenty at home12:02
zygaChipaca: in both languages12:02
Chipacazyga: I'll send you an empty box (cheaper!), that says harry potter in big letters on the outside then12:03
zygaChipaca: it will be a magic box :)12:03
Chipacaobvs12:03
zygaChipaca: it's a terrible world we live in but I think what happened is not representative of my broken country12:03
zygaluckily12:03
Chipacazyga: I assumed as much12:03
Chipacazyga: if it were, it wouldn't be news i guess?12:04
Chipacasmells like my lunch is ready, ttfn12:04
zygaunfortunately wit the current government we cannot see a strong signal either12:04
zygaI read that we should condemn that act, separate church and state more strongly and invite ms Rowing12:04
zygabut alas, that's a dream in current days12:04
mborzeckiheap profile after running some install/remove cycles: https://gist.githubusercontent.com/bboozzoo/ea6dc44bae2816fbb07776aa9cd9f5fb/raw/616b1b7f888f4dfac1ab1c5c6953b2c9080865b4/pprof001.svg12:05
mborzeckiyou'12:05
mborzeckiyou'll probably need to open that with eog or sth similar12:05
mborzeckiidk why, but it doesn't load in ff as svg12:05
mborzeckioff to pick up the kids12:06
Chipacapprof001.svg:1930: parser error : Opening and ending tag mismatch: script line 7 and svg12:11
Chipacapprof001.svg:1930: parser error : Premature end of data in tag svg line 612:11
Chipacamborzecki: ^12:11
Chipacamborzecki: that's from inkscape; google chrome says "error on line 1930 at column 12: Opening and ending tag mismatch: script line 0 and svg"12:12
Chipacamborzecki: (surprisingly similar errors … :-) )12:12
Chipacamborzecki: emacs hates it and crahsed12:13
=== chihchun_afk is now known as chihchun
zygaChipaca: at least vim can open it and see text ;) (/me hides now)12:13
Chipacazyga: emacs is doing what all the cool people do, and brexiting12:14
zygahahahaha12:14
Chipacaeog says12:14
Chipaca(eog:22352): GLib-WARNING **: GError set over the top of a previous GError or uninitialized memory.12:14
ChipacaThis indicates a bug in someone's code. You must ensure an error is NULL before it's set.12:14
ChipacaThe overwriting error message was: Error domain 1 code 76 on line 1930 column 12 of file:///tmp/pprof001.svg: Opening and ending tag mismatch: script line 0 and svg12:14
zygaChipaca: brexit the frog, waiting till the kettle boils12:15
mupPR snapd#6679 opened: many: implement user removal <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/6679>12:26
jameshpopey: sorry, was out earlier for an event.  Try the gtk2-demo snap from the store (edge channel): it is themed correctly for me, and relies on gtk-common-themes/gtk2-common-themes for the theme data and engines12:28
mupPR snapcraft#2509 closed: build providers: initial support for LXD <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2509>12:35
mborzeckiChipaca: hah, so the crash is reproducible then :)12:39
Chipacamborzecki: also, also, memory use is probably arch-dependent (esp wrt 32 vs 64 bits)12:42
Chipacagosh the ubuntu-core image from http://cdimage.ubuntu.com/ubuntu-core/16/stable/current/ is ancient12:45
Chipacamvo_: do you know what's up with that? ^12:46
Chipacamvo_: it comes with 2.33.112:47
mvo_Chipaca: probably best to check with foundations, iirc they have a different cadance now and only update core images on point releases of the distro. but given that these images are 8 month old maybe that is not a good policy anymore12:49
Chipacamvo_: also that site's a mess, there are images in the directory above 'current'12:49
mborzeckihm should have dumped the svg directly from pprof12:51
mvo_Chipaca: I put the update on the agenda for the meeting we have with foundations today12:52
Chipacaok12:52
mborzeckiChipaca: idk if it's much better now https://gist.githubusercontent.com/bboozzoo/ea6dc44bae2816fbb07776aa9cd9f5fb/raw/c51c7ba5970dc25888832fea251178ab56dd2ba0/profile001.svg12:53
zygacmatsuoka: I reviewed https://github.com/snapcore/snapd/pull/667912:53
mupPR #6679: many: implement user removal <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/6679>12:53
Chipacamborzecki: much12:53
cmatsuokazyga: thanks, I'm reading it right now12:54
Chipacamborzecki: yeah the read state thing is yuge12:54
ChipacaI could have some fun in there12:55
Chipacamvo_: I'm not seeing the OOMs on a i386 kvm with 256MB and no swap, fwiw12:55
* Chipaca tries to install nextcloud on it12:56
mvo_Chipaca: oh, nice12:56
mvo_mvo_: that is a great data point12:56
mvo_Chipaca: -^12:57
Chipacanextcloud installed as well12:57
Chipacamvo_: i asked a few q's in the bug, about this12:57
Chipacanow i need to go make tea before the standup12:57
* Chipaca runs12:57
zygauh13:00
zygaso13:00
zygais there a video call URL today?13:00
zygaI cannot seem to find one13:00
cmatsuokait seems that we don't have one13:01
degvilleI can't find one either.13:01
zygalet's use the one from last week13:01
zygahttps://hangouts.google.com/hangouts/_/canonical.com/snappy-devel13:02
=== ricab|lunch is now known as ricab
degvillethanks zyga!13:02
diddledandoes that work for non canonicalites? just wondering whether sharing allows randoms to join your meeting :-p13:02
zygadiddledan: it doesn't work, you need to be allowed to enter13:03
zygaand we'd notice ;)13:03
diddledanaha13:03
diddledanteehee13:03
* diddledan covers the camera13:03
ograhmm, how do i use snap refresh ... --stable with a ansp using tracks ?13:17
ogra*with a snap13:17
ograogra@pocketbeagle:~$ snap list pc-kernel13:17
ograName       Version       Rev  Tracking  Publisher   Notes13:17
ograpc-kernel  4.15.0-47.50  199  18/edge   canonical✓  kernel13:17
ograogra@pocketbeagle:~$ snap refresh pc-kernel --stable13:17
ograerror: cannot refresh "pc-kernel": cannot switch from kernel track "18" as specified for the13:17
ogra       (device) model to "stable"13:17
ograogra@pocketbeagle:~$13:17
diddledan"I'm typing all the right letters. not necessarily in the right order"13:17
ograyeah, i'm slowly becoming the king of dsylxeia :P13:18
diddledanand yet, I read that correctly until I look closer13:19
ograhaha13:19
diddledanit's weird that you can often read words that are misspelt without thinking about it13:20
ograaand ... to answer myself ...13:20
ograsnap switch pc-kernel --channel=18/stable && snap refresh pc-kernel13:20
diddledandoes `snap refresh --channel` work too?13:22
ograno idea, i refreshed to stable now ...13:26
ograbah ... and i notice i shouldnt have ... forgot that avahi has issues with that kernel13:27
ograogra@pocketbeagle:~$ snap refresh --channel=18/edge pc-kernel13:31
ograSetup snap "pc-kernel" (199) security profiles13:31
ograseems to work13:31
__chip__pedronis: would it be worth looking into always loading state on demand (as opposed to keeping it all in memory)? (all of it/ some of it (which?)/etc)13:32
__chip__pedronis: we can use mmap + json stream decoding to bring down memory usage a lot for that, if we want13:33
__chip__granted "a lot" is not going to be much bigger than state.json, but ¯\_(ツ)_/¯13:34
solkkuhello! I was wondering if there's a way to run a snap in the background on ubuntu server 18.04? now when I run it, it keeps giving me logs and I'm unable to use the terminal window for anything else14:01
mupPR snapd#6680 opened: [RFC] daemon: expose pprof endpoints <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6680>14:01
mborzeckimvo_: Chipaca: ^^ if you want to play with it14:02
mborzeckii'll add some notes on how to set it up14:02
mborzeckithe pprof tools don't know about unix sockets, so a proxy is needed14:02
=== mvo_ is now known as mvo
=== mvo is now known as mvo_
mvo_mborzecki: nice one14:03
=== chihchun is now known as chihchun_afk
mborzeckimvo_: fairly self contained, we could have it behind an env flag14:06
sil2100Hey guys, I started noticing that the core18 CI started failing due to some snapd errors: https://travis-ci.org/snapcore/core18/jobs/512567447 <- for example here14:21
sil2100"error: too early for operation, device not yet seeded or device model not acknowledged"14:21
sil2100When travis is trying to do a snap install14:22
sil2100We're using trusty for CI there, maybe I should try switching to xenial again?14:22
mvo_sil2100: we might need to add an extra command to .travis.yml, something like "snap wait system seed.loaded"14:34
mvo_sil2100: before snap install14:34
sil2100mvo_: oh, sounds fancy14:36
sil2100Will try that14:36
zygamborzecki: https://github.com/snapcore/snapd/pull/667714:39
mupPR #6677: vendor: pin golang.org/x/sys/unix to a revision before SYS_CLOCK_GETTIME on OSX <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6677>14:40
mvo_sil2100: good luck14:40
mborzeckizyga: hmm?14:41
mupPR core18#124 opened: Switch travis CI to xenial, add a snap wait after installing snapd to… <Created by sil2100> <https://github.com/snapcore/core18/pull/124>14:41
zygamborzecki: the fix was merged upstream now14:41
zygamborzecki: I wanted to let you know14:41
mborzeckiah, so maybe we can close it14:41
mborzeckior keep it14:41
mborzeckizyga: either way 6677 was timing out when running tests :/14:43
zygamborzecki: not our lucky day14:43
Chipacasolkku: o/14:44
Chipacasolkku: what do you mean?14:44
solkkuChipaca: I'm trying to run the snap as a daemon/service, so it does it's thing in the background and I can use the terminal for other things. Also the snap only stays alive as long as the terminal window is open14:50
Chipacasolkku: when you say 'the snap', you mean the app in the snap?14:52
Chipacasolkku: what happens if you add 'daemon: simple' to your app in your snap?14:52
Chipacasolkku: (with 'daemon: simple', it will be started automatically when you install the snap -- you can see it in 'snap services')14:54
solkkuChipaca: I did "snap install sickgear", when I run it with "sudo sickgear" I get all kinds of logs: https://pastebin.com/61CgtWA914:54
Chipacasolkku: is sickgear your snap?14:54
mvo_can we land 6660 btw? has two +1 afaict and everyone seems to be happy with the new UX14:54
Chipacasolkku: or is it somebody else's?14:54
solkkuI can use "sudo sickgear &", but still get logs on screen once in a while :)14:55
solkkuChipaca: it's a snap I found at snapcraft.io14:56
=== chihchun_afk is now known as chihchun
Chipacasolkku: and is sickgear something that's meant to (a) run as root, and (b) run in the background ?14:56
mvo_pedronis: was 6660 something you wanted to double check before it goes in? iirc yes but I'm not sure anymore14:58
Chipacasolkku: looking at the website it seems to be a GUI app14:58
Chipacasolkku: why would you run this as root? and why would you start it from the terminal?14:58
pedronismvo_: yes14:58
solkkuChipaca: you can run it in the background if you install it the "old fashioned" way, but couldn't tell you about a or b14:58
pedronismvo_: I will get to those PRs by pstolowski today or tomorrow morning14:58
solkkuChipaca: it starts a webserver and I access it via the web browser14:59
Chipacasolkku: ohhh14:59
Chipacasolkku: then maybe reach out to the sickgear people (file an issue?), tell them to make it a daemon14:59
Chipacasolkku: if they need help, we're here (or forum.snapcraft.io)15:00
mupPR core18#125 opened: hooks: create snapd directory skeleton <Created by zyga> <https://github.com/snapcore/core18/pull/125>15:00
mvo_pstolowski: thanks, I added a tag15:00
solkkuI said to the maker of sickrage that I'm trying to run it as a daemon, but he just said that "no where in the documentation does it say to run like that" and "if you want to run like that, then you must take apart the /snap folder and reconstruct yourself".. so I guess I'm on my own and came here for help next :)15:01
zygasolkku: you can create a systemd service that just runs the snap command15:02
solkkuChipaca: when I try to run it as a daemon "sudo snap start --enable sickgear.daemon", I get: "error: snap "sickgear" has no service "daemon""15:03
zygasolkku: you don't have to repackage the snap15:03
zygasolkku: just install it and create a systemd unit that runs it as a service15:03
solkkuzyga: oh, is there a noob-friendly guide on this?15:03
zyganot sure :) just systemd docs15:04
Chipacaer15:04
Chipacasolkku: yeah, if the developer doesn't think you should run it like that … ¯\_(ツ)_/¯ maybe you shouldn't?15:05
solkkuChipaca: maybe, but I'm going to try it my way first ;)15:06
MattJsolkku, https://www.shellhacks.com/systemd-service-file-example/15:07
solkkuMattJ: thanks, seems to have worked!15:15
solkkuyou learn something new every day15:20
cwaynemvo_: sil2100: heya, should we setup a call re: beta gating for core18?15:37
=== pstolowski is now known as pstolowski|afk
mupPR core18#124 closed: Switch travis CI to xenial, add a snap wait after installing snapd to… <Created by sil2100> <Merged by sil2100> <https://github.com/snapcore/core18/pull/124>16:18
zygasil2100: hi17:12
zygaCould you please review https://github.com/snapcore/core18/pull/12517:12
mupPR core18#125: hooks: create snapd directory skeleton <Created by zyga> <https://github.com/snapcore/core18/pull/125>17:12
=== Erich is now known as Eickmeyer
sil2100zyga: sure o/ It's a bit troublesome right now, since I can't do any test builds of the snap right now due to the archive slowness17:14
sil2100zyga: so I guess it'll have to wait till tomorrow17:14
* sil2100 just tried building his own core18 snap but the archive was super slow and actually mismatching sizes17:14
sil2100zyga: it's on my plate anyway o/17:15
zygasil2100: thank you17:30
mupPR snapd#6681 opened: misc: support system-global-ids for 'daemon' user <Created by jdstrand> <https://github.com/snapcore/snapd/pull/6681>17:40
* zyga EODs18:56
zygajdstrand: super nice, I will read in detail tomorrow18:56
zygajdstrand: there are some interactions with snap-confine changes I have in the pipe18:56
zygajdstrand: I will try to de-conflict it as things that are in progress are merged into master18:56
zygajdstrand: gentle ping about mointinfo fix https://github.com/snapcore/snapd/pull/660518:58
mupPR #6605: cmd/libsnap,osutil: fix parsing of mountinfo <Created by zyga> <https://github.com/snapcore/snapd/pull/6605>18:59
zygaif you want I can merge as-is, you can always review post-factum later18:59
zygait has two reviews and is green already18:59
zygawow, today many things can get merged19:00
jdstrandzyga: 6605 is on my list. now that the daemon user PR low-level technical details are worked through, I'm going through my inbox/todo/etc again, which that pr is in19:17
* jdstrand wonders if anyone uses the non-raw travis log in the snapd travis ci tests19:22
=== pfsmorigo is now known as Guest7369
mupIssue # closed: core18#56, core18#86, core18#89, core18#11723:50
mupPR # closed: core18#43, core18#72, core18#90, core18#98, core18#120, core18#121, core18#122, core18#123, core18#12523:50
mupIssue # opened: core18#56, core18#86, core18#89, core18#11723:51
mupPR # opened: core18#43, core18#72, core18#90, core18#98, core18#120, core18#121, core18#122, core18#123, core18#12523:51

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