=== wgrant_ is now known as wgrant
mlwAnyone here running the krita snap?00:33
=== chihchun_afk is now known as chihchun
=== e is now known as deadk
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
zyga-ubuntugood morning06:39
mvohey zyga-ubuntu , good morning06:41
zyga-ubuntumvo: hey, what a autumn-ish monday :)06:46
zyga-ubuntumvo: how are you doing?06:46
zyga-ubuntumvo: I'll get to reviews in a moment, just making tea06:46
mvozyga-ubuntu: things are going well, I start 2.29~rc1 today06:51
mupPR snapcraft#1638 opened:  tests: reorganize unit and integration suites to make them easier to split for travis <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1638>06:51
zyga-ubuntumvo: do you think we will make it with all the PRs marked as a part of 2.29?06:52
mvozyga-ubuntu: not for ~rc1, but we can cherry-pick them all for ~rc206:53
zyga-ubuntumvo: all right then06:54
=== JanC is now known as Guest95859
=== JanC_ is now known as JanC
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
zyga-ubuntujamesh: hello07:32
mvoChipaca: hey, quick question - the structured epochs branch has landed for 2.29 - does that mean our side is complete or will we have to do client side work still?08:09
Chipacamvo: there's no business logic08:13
mvoChipaca: aha, and we need that client side?08:14
jameshzyga-ubuntu: hi08:14
Chipacamvo: as i understand it the client needs to sanity-check epochs as part of validation, yes08:14
mvoChipaca: ok, thats fine then. was wondering if it should go into the 2.29 release notes draft or not :)08:15
zyga-ubuntujamesh: hey! :)08:15
Chipacamvo: i'd keep mum about it until both sides are done, in any case08:16
kalikianaelopio: still awake? mind reviewing snapcraft#1627 ?08:17
mupPR snapcraft#1627: lxd: split container classes into different files <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1627>08:17
zyga-ubuntujamesh: how are you doing08:17
zyga-ubuntujamesh: I'd like to sync on our work around mount features08:17
mvozyga-ubuntu: do you happen to know if there is a workaround for classic snaps not working on artful? the __libc_dl_error_tsd issue?08:18
zyga-ubuntumvo: I don't know of any workaround08:20
jameshzyga-ubuntu: I still need to get my user-mounts branch properly working for fuse mounts.  I got side tracked on some other bugs (there have been a few gnome-software/snap related issues that came up in the last week or so)08:20
zyga-ubuntujamesh: I'm working on overlay support and I think it could now be used for improved content interface08:20
zyga-ubuntujamesh: I'll probably look at a quick attempt to implement the new content interface ideas08:21
zyga-ubuntujamesh: (the one with saner one-to-many connections)08:22
zyga-ubuntujamesh: I could also try the idea where snap-update-ns is called for user mounts, if any exist, and see if that can be made to work08:22
jameshzyga-ubuntu: so, the main issue I can see with using snap-update-ns for the user mounts is that we either (a) need to have snap-confine set up inheritable capabilities so snap-update-ns can do its job as a normal user, or (b) have snap-update-ns do the capability manipulation and setuid itself08:24
jameshzyga-ubuntu: with (a), we'd need to make absolutely sure those capabilities didn't leak out to the confined process.  With (b), we'd need to have Go bindings for the capability APIs08:25
zyga-ubuntujamesh: why does snap-confine need to run as a user the 2nd time? can it run as root and pass some arguments to the mount operation?08:30
jameshzyga-ubuntu: let's say that /foo is a fuse file system, and I want to do "mount --bind /foo/bar /mountpoint"08:31
jameshzyga-ubuntu: if "/foo" was mounted as an ordinary user, running the mount command as root will fail because root can't see "/foo/bar"08:31
kalikianac-lobrano: Did you see snapcraft#1636 ?08:32
mupPR snapcraft#1636: internal: more gracefully determine host OS <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1636>08:32
jameshthe kernel will only let the user that mounted the fuse file system see files and directories within08:32
zyga-ubuntujamesh: I see, how are they mounted by bubblewrap then? through capabilities?08:32
jameshzyga-ubuntu: yeah.  add  CAP_SYS_ADMIN to permitted set, setuid(real_uid), reacquire CAP_SYS_ADMIN, mount08:33
c-lobranoHey kalikiana, yes I saw it. Unfortunately I was to slow :D08:33
zyga-ubuntujamesh: could we do that with the C preamble?08:33
jameshzyga-ubuntu: using cgo might be the best option, yeah.08:34
zyga-ubuntujamesh: snap-update-ns has a non-trivial C preamble to do various pre-go manipulations08:34
ackkniemeyer, hi, could you have another look at https://github.com/snapcore/snapd/pull/3916 ?09:55
mupPR #3916: snap,wrappers: add support for socket activation <Created by albertodonato> <https://github.com/snapcore/snapd/pull/3916>09:55
Chipacapedronis: you're alive! \o/09:56
* Chipaca was starting to worry09:56
Chipacapedronis: how're you doing?09:56
pedronisChipaca: better09:58
Chipacapedronis: does that mean 'not 100% yet'?09:59
pedronisnot 100%,  but back to work,  still haven't finished my treatments (some more days of pills)10:00
Chipacapedronis: blagh :-/10:01
Chipacapedronis: during the NYC sprint I advanced on having tab completion for aliases done. I've got it working, but there are a lot of tests for aliases from foo to foo that, because of the simplistic way i implemented it, don't pass for completion. i wanted to check with you about the rationale for those tests to see if it made sense to skip them for this, or if i needed to fix the implementation to treat it specially10:03
=== chihchun is now known as chihchun_afk
pedronisChipaca: I'm probably missing context,   why doing tab completetion would break tests?10:05
pedronisand what are aliases from foo to foo10:05
Chipacapedronis: aliases := []*backend.Alias{{"x", "x"}}10:05
Chipacathat's an alias for something that already exists, right?10:07
pedronisI suppose10:07
pedronisI still don't understand why that then breaks when you add completion10:08
Chipacapedronis: i mis-spoke, the aliases don't break; matchingAliases et al don't return that as an alias, which means it's not returned10:09
Chipacai mean10:09
Chipacapedronis: should i push the branch and you read code, or should i continue trying to explain?10:10
Chipacapedronis: i tweaked the helpers to return the matching aliased commands and completions10:10
Chipacapedronis: commands is what was there before, those still work fine10:10
pedroniswhich helpers?10:11
Chipacapedronis: completions will all be missing these auto-entries (the {"x", "x"} ones)10:11
Chipacapedronis: matchingAliases, missingAliases10:11
pedronismatchingAliases is a test only helper10:11
Chipacaas i say, it works, my doubt was with tests10:11
Chipacaso i wanted to check with you about the intent of those {"x", "x"} ones before just removing them from the completion checks10:11
pedroniscan you give a  test file name  ?  (there's a lot of tests)10:12
Chipacapedronis: https://github.com/snapcore/snapd/compare/master...chipaca:complete-aliased#diff-66a739a11ce504d9bc94df4adb1e8bcdR37710:13
Chipacapedronis: the last two lines on that test10:13
Chipacaone is for the command aliases, which is as it was before10:13
Chipacaie c.Check(matchingAs, DeepEquals, []*backend.Alias{{"baz", "y.baz"}, {"y", "y"}})10:14
pedroniswhy completers need tests in backend?10:14
Chipacapedronis: because they're created there?10:15
pedronisah,  this is not about completeing aliases10:15
pedronisthis as about completing aliases args ?10:15
pedroniswe were on completely different thoughts10:15
Chipacathis is about creating the completion snippet symlinks for aliased commands10:15
Chipacathat is, when you alias foo.bar to bar, you want bar<tab> to run the completer for foo.bar10:16
pedroniswill not Alias{x,x} create a circular symlink ?10:17
pedronisyou are probably reading too much in those tests10:18
mupPR snapcraft#1412 closed: lxd: snapcraft refresh in containers <Created by kalikiana> <Closed by kalikiana> <https://github.com/snapcore/snapcraft/pull/1412>10:18
Chipacapedronis: Alias{x,x} wouldn't work because it either doesn't exist, or already exists, so in either case it won't get touched10:18
Chipacapedronis: I thought they were probably the trivial case, but I don't know enough to be sure so rather than investigate i paused the work to ask you when you got back10:19
pedronisI have as much clue as you on this10:19
Chipacapedronis: i thought you were mr aliases :-) but ok10:20
Chipaca(it's not worrying that i was wrong -- but everybody else thought so too, and that might be a little worrying)10:20
pedronisChipaca: wrong abotu what?10:21
Chipacapedronis: about you being the person who knew about this bit10:21
pedronisI wrote that code10:21
pedronisit doesn't mean I remember why a unit tests is some way or another10:21
pedronisafter months10:21
pedronisafaict  Alias{x,x} will create a circular symlink10:22
mupPR snapd#4065 opened: servicestate: use taskset <Created by stolowski> <https://github.com/snapcore/snapd/pull/4065>10:23
Chipacapedronis: fair enough. I'll go ahead and fix the tests to pass then, as the errors make sense10:24
Chipacapedronis: thank you10:24
Chipacapedronis: (my doubt was because maybe there were situations where that alias was attempted and something special needed to be done with it, but if it's just an easy test it's fine)10:25
pedronisChipaca: well it might be that we don't defend about somebody trying to set up such an alias, I don't remember10:25
* kalikiana break10:26
pedronisChipaca: so it might be worth checking what happens10:26
pedronisin that case10:26
pedronisah, you get  - Setup manual alias "spread" => "spread" for snap "spread" (cannot enable alias "spread" for "spread", it conflicts with the command namespace of installed snap "spread")10:30
pedronisso we don't get there because of that10:31
pedronisChipaca: ^10:49
Chipacapedronis: ack10:49
jameshzyga-ubuntu: by the way, I noticed that your devtools scripts don't work with the latest snapd.  Is there a newer way to test out changes on my dev system?10:56
zyga-ubuntuno, not really10:59
zyga-ubuntuI mostly use make hack10:59
jameshthat doesn't really help for snapd changes though11:00
zyga-ubuntusnapd is fine to run just from shell11:00
zyga-ubuntugo build && sudo ./snapd11:00
Chipacazyga-ubuntu: no, because it panics if it's not running from /usr/lib/snapd/11:01
jameshI was getting this error: https://github.com/zyga/devtools/issues/2511:01
zyga-ubuntuChipaca: what? I never got that11:01
zyga-ubuntuChipaca: maybe I had no-reexec key set11:01
Chipacazyga-ubuntu: then you're not running it like that enough11:01
Chipacazyga-ubuntu: i get this with no-reexec set11:01
Chipacazyga-ubuntu: when trying to install something11:02
zyga-ubuntuis that new?11:02
Chipacazyga-ubuntu: a month? two?11:02
Chipacasuper old11:02
zyga-ubuntuI really think it's a bit shooting ourselves in the foot11:02
zyga-ubuntuChipaca: weird11:02
zyga-ubuntumaybe my braches are just equally old11:02
zyga-ubuntumaybe make hack should do more?11:02
zyga-ubuntu(make hack == ~~ make install)11:02
mupPR snapd#4066 opened: overlord/snapstate: support completion for command aliases <Created by chipaca> <https://github.com/snapcore/snapd/pull/4066>11:19
Chipacaok, i think i should go for a run11:20
=== facundo__ is now known as Facu
* Chipaca goes11:39
niemeyerackk: Will do, thanks for the ping11:50
ackkthanks niemeyer11:51
mvomwhudson: hey, if you happen to be around, I wonder if you have any idea about a symbol ' out of range error on zesty/ppc64el (go1.7)12:15
mvomwhudson: aha, nevermind, found the forum topic about it12:17
mupPR snapd#4067 opened: packaging,spread: fix and re-enable opensuse builds <Created by zyga> <https://github.com/snapcore/snapd/pull/4067>12:35
leocadio_Hello. I need to make some changes in the mac80211 and ath9k modules. Does anyone know how can I compile a new kernel snap with my changes, to replace the legacy kernel snap available in the Ubuntu Core distro?12:40
cachiojdstrand, hey I am doing a test for the gsttings interface12:40
leocadio_Is this possible?12:41
cachiojdstrand, I see that the gsettings command is not able to access the schemas12:41
cachiojdstrand, any idea about the configuration needed for that? is it ok to use the gsettings too to test that interface?12:42
cachiojdstrand, I already tried with a python script but had a lot of issues with dependencies and showed similar problems with the schemas12:42
Chipacabooo, something changed in travis and the "all good" thing is now even more garbled12:48
* Chipaca will fix, sometime12:48
Chipacamvo: how are we for adding things to 2.29?12:48
cachiomvo, Chipaca are we still adding things to 2.29?12:49
Chipacacachio: that's what i asked, more or less :-)12:49
cachioI already started beta testing on this12:49
mvoChipaca, cachio first rc1 is out but there will most certainly a rc2 so things are still open12:50
Chipacaalthough my question would be "are we still doing that thing where we can sneak branches into the next point release if it happens"12:50
mvobut we already catching the first issues (ftbfs on ppc64el/zesty for example)12:50
Chipacaok, if #4066 is as low drama as i think it is, i'll suggest it for point inclusion12:51
mupPR #4066: overlord/snapstate: support completion for command aliases <Created by chipaca> <https://github.com/snapcore/snapd/pull/4066>12:51
Chipacaotherwise .30 is fine12:51
pedronismvo: cachio: seeems we have a couple of assertions signed by an account/key from federico, not sure how they are used execatly, auto-import.assert is one of them, there's a bunch of model assertions too13:33
pedronisif we redo the auto-import.assert one we probably need to redo some of the models too13:34
pedronismvo: is this related to the fact that there's a limit on how long a system-user can last ?13:35
mvopedronis: I think its unrelated but I have not looked in detail13:38
pedronismmh, no, the limit is only if no models13:43
pedronisnot sure why federico set it so13:43
pedronisbut maybe we changed that in between13:43
mvoI suspect its and old test, that was one of the early ones13:44
mupPR snapd#4068 opened: interfaces/builtin: add support for content "source" section <Created by zyga> <https://github.com/snapcore/snapd/pull/4068>13:46
jdstrandcachio: you need to use the desktop part and its desktop-launch wrapper to generate the schema files aiui. you might talk to the desktop team for more specifics (eg, kenvandine)13:47
cachiojdstrand, good, thanks13:47
cachiomvo, 2.28.5 in stable13:50
zyga-ubuntuI need to pick up my daughter13:52
zyga-ubuntucachio: woot, thank you13:52
ogra_heh, looking at snap info core looks cnfising with beta and edge now13:52
zyga-ubuntujdstrand: o/13:53
jdstrandzyga-ubuntu: hi! fyi, 4008 and other PRs are at top of list today (after one thing I'll finish this morning)13:54
ogra_mvo, should that beta build not also have gone to edge at the same time ? seems weird that beta has a higher versioon (and revision) than edge13:54
zyga-ubuntujdstrand: thank you! :-)13:54
zyga-ubuntujdstrand: I'm still working on some tweaks to the content one but I will post it as soon as 4008 is merged; I'll probably iterate with you today/tomorrow to land 400813:54
mvoogra_: yeah, I think its confusing, edge is actually also new but the version does not reflect it. I will fix in a wee bit13:57
mvocachio: yay, thank you so much!13:57
cachiomvo, np14:00
cachioniemeyer, this week could you take a look to the PRs in spread-cron?14:04
cachioniemeyer, taht would be grat if we can merge it soon14:04
* kalikiana need.. more.. coffee..14:19
kalikianakyrofa: elopio Are we having our weekly between the three of us then?14:46
kalikianain 15min14:46
kyrofakalikiana, I was assuming so after our discussion on Friday, but we honestly didn't make much more progress after you had to jet14:46
kalikianakyrofa: Ah. Should we maybe schedule another one for that? I can work late tomorrow so we have more time.14:48
mupPR snapd#4069 opened: debian: do not build static snap-exec on powerpc <Created by mvo5> <https://github.com/snapcore/snapd/pull/4069>14:48
kalikianakyrofa: The other thing I was thinking, maybe if you want we could chat about the OsRelease class a bit - I made some suggestions in my comments, but maybe we want to check if we're on the same page there?14:49
kyrofakalikiana, fine by me-- I'm always happy to meet regardless14:52
kyrofalet's do it14:52
kyrofaI mean the one today. As for tomorrow, let's ask sergio when he's back14:52
kyrofaI do think we need to continue breaking it down, but not sure when14:53
sergiusenskyrofa what's up?14:54
sergiusenskyrofa I added comments to the OsRelease stuff too14:54
sergiusenskyrofa is snapcraft#1632 ready to go? this would be my final gripe to go to the stable channel :-)14:55
mupPR snapcraft#1632: libraries: exclude the full set of libc6 <bug> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1632>14:55
kyrofasergiusens, indeed it is14:55
kyrofasergiusens, I think we can improve that lib method, but it causes no harm in that PR14:56
kyrofaAnd I've tested it pretty extensively at this point14:56
kyrofasergiusens, I'll go +1 it14:56
kyrofasergiusens, you realize though that those other two PRs are required for trusty support, right?14:57
sergiusenskyrofa I have a PR refactor to change that module from `libraries` to `elf` and do some neat things in there14:57
kyrofasergiusens, excellent14:57
sergiusensbut I'd rather do minimal changes here so the test surface is not so broad to get a stable release out first14:57
kyrofasergiusens, fair enough. +1d14:58
zyga-ubuntuniemeyer: I realized that due to a way apparmor is behaving we may need to have apparmor and mount kernels coordinate on the rename14:59
zyga-ubuntuniemeyer: I'll make a test case to examine this14:59
mupPR snapcraft#1632 closed: libraries: exclude the full set of libc6 <bug> <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1632>15:01
zyga-ubuntuniemeyer: I also think that we must be careful in mixing slots that use source and those that don't15:01
kyrofasergiusens, I of course have some vested interest in trusty support. While I've got you: do you think it's possible to support using the snap (once it works) in the trusty builder? I've been getting some questions about this15:01
zyga-ubuntuniemeyer: as this will cripple plug side15:02
zyga-ubuntuniemeyer: I was thinking about an attribute that would let the plug side define and control one-to-many expectations: only-one, allow-many15:02
niemeyerzyga-ubuntu: Which renames?15:02
sergiusenskyrofa I think it should be possible with these changes; non classic confinement at least15:02
kyrofasergiusens, yeah that's a whole different ballgame15:02
niemeyerzyga-ubuntu: Yeah, we'll need that at some point, but I suggest not doing that now so we don't get lost from the key priorities15:03
zyga-ubuntuniemeyer: https://forum.snapcraft.io/t/improvements-in-the-content-interface/238715:03
kyrofacjwatson, if the snapcraft snap worked on trusty, is that a capability you'd be able to expose in the LP builders, or build.snapcraft.io?15:03
zyga-ubuntuniemeyer: ack15:03
zyga-ubuntuniemeyer: on that page look for "renamed"15:03
kyrofacjwatson, the snapcraft in the archive is v1, targeting 15.0415:03
kyrofaBut there are people who actually need to build series 16 snaps on trusty15:04
kyrofaOh sorry kalikiana got caught up in coffee15:05
kyrofaOn my way15:05
kalikianano worries15:05
niemeyerzyga-ubuntu: By "kernels" you mean "backend" I suppose?15:08
cjwatsonkyrofa: s/, or build.snapcraft.io// (they're the same thing).  maybe?  it'd need some thought; it would be complicated if it had to be selectable15:10
kyrofacjwatson, well, the builders expose a lot more capability-- different archs, etc15:10
kyrofacjwatson, curious to see if anyone is using it for v115:10
kyrofa(anyone using them today is either using v1 or expecting it to be v2 and giving up)15:11
cjwatsonkyrofa: right, but BSI is strictly more limited, it doesn't have its own special build code or anything over and above what's on LP15:11
kyrofacjwatson, right, which is why I specified them both15:11
cjwatsonkyrofa: actually BSI would be out of scope here anyway since it's xenial-only15:11
zyga-ubuntuniemeyer: yes, not sure why I said "kernels" :)15:11
cjwatsonkyrofa: we do have the notion of permitted combinations of Ubuntu series and snappy series in LP15:12
niemeyerzyga-ubuntu: We should be able to do without actual coordination between the backends.. the interface itself is the one that has needs across both, and should be able to inform both of what it needs15:12
cjwatsonkyrofa: right now, the snappy series (e.g. 16) isn't passed through to the builder, but it could be15:12
cjwatsonkyrofa: that's probably how we'd do that kind of thing if we needed to15:13
zyga-ubuntuniemeyer: yes, I was thinking we just need to supply the backends with the same information so that they can come up with the same decision in the end15:13
cjwatsonkyrofa: (I don't suppose I could persuade somebody from the snapcraft side to have a go at contributing the necessary changes?  more than happy to help with mentoring, but it seems that it'd be good for more people on the snapcraft side to know how the moving parts work)15:14
kyrofacjwatson, actually yeah, that doesn't sound like a bad idea15:15
cjwatsonit would involve carefully-sequenced changes to both launchpad-buildd and LP itself, but apart from that it's pretty isolated15:15
cjwatsonactually maybe not much sequencing15:16
cjwatsonbasically just a matter of adding the snap's store_series name to the build args in SnapBuildBehaviour, passing that through to the backend script in launchpad-buildd's SnapBuildManager, and adding the necessary conditional behaviour to launchpad-buildd's BuildSnap.install.  As long as launchpad-buildd tolerates the key being missing from the build args, there's no sequencing involved15:19
cjwatsonand then we could add the trusty/16 combination to SnappyDistroSeries via an API POST request once we've verified that it works15:20
kyrofacjwatson, good deal, once a few of our PRs make it into a release, I'll look into that15:20
kyrofa(right now the snap doesn't run)15:21
cjwatsoncool - let me know15:21
cjwatsonI expect we'll want the builders to know the snappy series eventually for other reasons too15:21
zyga-ubuntuI wish to thank everyone for introducing the automatic and smart "night mode"15:23
zyga-ubuntuwith almost no direct sun nowadays, it's always "dark and gloomy"15:23
zyga-ubuntubut at least my eyes hurt less15:23
Chipacazyga-ubuntu: the redshift developers disagree :-)15:23
zyga-ubuntuChipaca: the coverflow developers you say? ;-)15:24
Chipacazyga-ubuntu: (because gnome wrote their own and made them redundant)15:24
cachiomvo, I am taking a look to the snap-confine issue that seems to be failing on autopkgtest:ubuntu-*15:24
zyga-ubuntuChipaca: all software falls into the back hole of "built-in feature"15:24
zyga-ubuntucachio: hmm, which issue is that?15:24
zyga-ubuntuaha that one15:25
zyga-ubuntuI suspect it's a very very slow VM15:25
zyga-ubuntuunless you can reproduce it15:25
cachiosomething related to module:nvidia_modeset15:25
zyga-ubuntucachio: does it only fail in that place?15:26
zyga-ubuntucachio: that error is a very indirect way of saying "3 seconds elapsed but we took longer to finish stuff"15:26
zyga-ubuntucachio: did you see it in any interactive work?15:28
mupPR snapd#4067 closed: packaging,spread: fix and re-enable opensuse builds <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4067>15:33
zyga-ubuntubut something is indeed fishy15:33
kyrofacjwatson, agreed, that seems generally useful15:33
zyga-ubuntulots of tests are failing on this15:33
zyga-ubuntumvo: maybe we should increase that timeout to 6 seconds15:34
=== JoshStrobl|AFK is now known as JoshStrobl
cachiozyga-ubuntu, sorry, I was having lunch15:46
cachiozyga-ubuntu, well, that could be a reason, I'll try to reproduce it on a vm here15:46
niemeyercachio: Reviewed both PRs there16:15
cachioniemeyer, tx16:15
ogra_niemeyer, hrm ... when we switched to the forum were all the old mailing lists archives simply wiped ?16:35
niemeyerogra_: I don't think anything happened to the ML other than blocking subscriptions and postings.16:36
ogra_well, it seems gone from lists.ubuntu.com16:36
ogra_anyway, no biggie (the stuff i searched for is in my local archives)16:37
ogra_https://lists.ubuntu.com/mailman/listinfo/snappy-devel ... should (theoretically) get me to the archives ... though16:37
cachioniemeyer, please tell me if the doc that I added in the PR help to understand the change16:45
niemeyercachio: Thanks, looking16:46
ogra_cachio, sorry that you have to suffer from my shell-like pyton stuff :)16:53
cachioogra_, np :)16:56
ogra_(even though sergiusens claims that my python looks like perl ... :P )16:57
kyrofaogra_, hahaha16:59
niemeyercachio: Replied17:01
niemeyerogra_: Are you looking for this: https://lists.ubuntu.com/mailman/listinfo/snapcraft17:02
sergiusensogra_ perl 6 even17:02
ogra_niemeyer, oooh, it was originally snappy-devel and got renamed ... (i simply clicked the link at the bottom of an old list mail in my mail app) ... yeah, thats it17:02
ogra_thanks !17:03
niemeyerogra_: That's the actual mailing list we used before the move to the forum17:03
niemeyerogra_: It got renamed 20 years ago.. shortly after Linus came up with the  Linux kernel17:04
kyrofaogra_, I need to see an example then. I can't imagine python that looks like an AES-encrypted message17:06
sergiusenscjwatson kyrofa we would want to take advantage of base declarations in snapcraft.yaml when they come, those would be mapped to builder instances and if we really really want to support building on trusty, I would try to get a trusty base going too17:06
ogra_kyrofa, well, i hate curly brackets ... so it wont be perl ever again :)17:07
kyrofasergiusens, that's a great point, I wonder how hard it would be17:07
sergiusenskyrofa we wrote it down on a forum post already, somewhere at some point in time ;-)17:08
kyrofasergiusens, haha, good luck finding it17:08
ogra_sudo snap install forum-crawler ...17:10
kyrofa$ snap find forum-crawler17:11
kyrofaThe search "forum-crawler" returned 0 snaps17:11
kyrofaogra_, you got my hopes up17:11
niemeyerkyrofa: I find the forum search surprisingly effective actually17:11
kyrofaniemeyer, I've had pretty poor experiences with it, but I could also blame that on human error17:12
ogra_which human exactly though ?17:13
kyrofaYes I will leave that nebulous17:15
ogra_only 4 billion choices...17:16
cachioniemeyer, answers in the forum, now I have a kinesiology appointment but when I am back I'll finish the refactor for the python code.17:47
* Pharaoh_Atem groans in tiredness17:47
=== JanC_ is now known as JanC
kyrofasnappy-m-o, autopkgtest 1583 xenial:amd64 xenial:armhf artful:amd6417:50
snappy-m-okyrofa: I've just triggered your test.17:50
niemeyerzyga-ubuntu: ping18:39
niemeyerjdstrand: ping.. maybe you might know the answer18:46
zyga-ubuntuniemeyer: yes?18:46
zyga-ubuntu(not here full time anymore)18:46
niemeyerzyga-ubuntu: Oh, hay18:46
zyga-ubuntuhow can I help? :)18:46
niemeyerzyga-ubuntu: Sorry, I realize it's a bit late18:46
zyga-ubuntuthat's ok, I'm trying to solve a problem and I came to give it another try18:47
niemeyerzyga-ubuntu: I'm about to merge #3958, and was wondering about the fact we have something about encryption file systems related to that apparmor/snap-confine, according to docs in the template of snap-confine18:47
mupPR #3958: many: add support for /home on NFS <Created by zyga> <https://github.com/snapcore/snapd/pull/3958>18:47
niemeyerzyga-ubuntu: But I cannot see any code related to this18:47
niemeyerzyga-ubuntu: My only concern right now is not breaking something else because e.g. we remove some other file (think * glob + EnsureDir)18:48
zyga-ubuntuniemeyer: right, thank you for checking18:48
zyga-ubuntuniemeyer: right now we don't have any generated include files for encrypted files systems but indeed this feature was planned so that we could add one next to NFS18:49
niemeyerzyga-ubuntu: Aha, ok.. that makes sense18:49
zyga-ubuntuniemeyer: some of those, similar to NFS, leak through the abstractions apparmor gives us18:49
niemeyerzyga-ubuntu: Merging then, thanks18:49
zyga-ubuntuniemeyer: we have some workarounds but AFAIK what doesn't work very well is "snap try" on top of one18:49
zyga-ubuntuthank you!18:49
niemeyerzyga-ubuntu: Do you have anything on top of this, or can I squash?18:50
zyga-ubuntuniemeyer: no, nothing on top18:50
zyga-ubuntufeel free to squash/not squash :)18:50
niemeyerOk, here we go18:50
zyga-ubuntuwoot, thank you18:50
zyga-ubuntuI'm sure CE will like that aspect of 2.29 :)18:51
mupPR snapd#3958 closed: many: add support for /home on NFS <Created by zyga> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3958>18:51
pstolowskiniemeyer, hey, can you take a look at my 2 interfaces-related PRs before your vacation?18:56
niemeyerpstolowski: Yeah, will do, thanks for the ping18:56
mupPR snapd#4070 opened: hooks/configure: queue service restarts <Created by stolowski> <https://github.com/snapcore/snapd/pull/4070>19:15
pstolowskiniemeyer, ^ I decided to simplify the approach with the above (compared to what I was trying to explain in the standup)19:17
pstolowskiniemeyer, happy to discuss/explain if needed19:18
niemeyerpstolowski: Sounds good, thanks!19:20
niemeyerpstolowski: Reviewing now19:21
zyga-ubuntuniemeyer: woot, no need to sync anything, I got one thing wrong; apparmor needs the original path :)19:31
zyga-ubuntuniemeyer: and de-clashing of plugins should now be oK19:31
zyga-ubuntupushed and EODing19:35
zyga-ubuntuone more small patch19:36
zyga-ubuntuthen EOD19:36
niemeyerzyga-ubuntu: Which original path?19:37
zyga-ubuntuniemeyer: let me show you an example19:39
mupPR #4068: interfaces/builtin: add support for content "source" section <Created by zyga> <https://github.com/snapcore/snapd/pull/4068>19:39
zyga-ubuntuhere, this is exactly what I was trying to solve19:40
zyga-ubuntuone app, two plugins (via content)19:40
zyga-ubuntuin the mount entry they are renamed to unclash19:40
skjensenEvening guys, sorry to interrupt but got a bit of a weird error when trying to build my image.. The error is: panic: runtime error: slice bounds out of range19:40
zyga-ubuntuin the apparmor profile they are not renamed because apparmor needs a rule for the source (which is the original path) because of AF_UNIX shortcomings19:40
zyga-ubuntuand I just saw a typo :)19:40
skjensenFull error description can be found here: https://pastebin.com/5aUPQXcB19:41
zyga-ubuntu(I wrote "indented" instead of "intended")19:42
zyga-ubuntujdstrand: I think 4068 is also interesting for you, in the same vein as other PRs lately19:46
* zyga-ubuntu EOds19:51
cachioniemeyer, I got disconnected, did you see what I wrote?21:10
kyrofaelopio, ever seen this? https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-artful-snappy-dev-snapcraft-daily/artful/amd64/s/snapcraft/20171023_200736_fda75@/log.gz22:24
kyrofaRuby failures on artful autopkgtests22:24
elopio1segmentation fault :S kyrofa: that's new to me.22:26

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