| === wgrant_ is now known as wgrant | ||
| mlw | Anyone 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-ubuntu | good morning | 06:39 |
| mvo | hey zyga-ubuntu , good morning | 06:41 |
| zyga-ubuntu | mvo: hey, what a autumn-ish monday :) | 06:46 |
| zyga-ubuntu | mvo: how are you doing? | 06:46 |
| zyga-ubuntu | mvo: I'll get to reviews in a moment, just making tea | 06:46 |
| mvo | zyga-ubuntu: things are going well, I start 2.29~rc1 today | 06:51 |
| mup | PR 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-ubuntu | mvo: do you think we will make it with all the PRs marked as a part of 2.29? | 06:52 |
| mvo | zyga-ubuntu: not for ~rc1, but we can cherry-pick them all for ~rc2 | 06:53 |
| zyga-ubuntu | mvo: all right then | 06: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 | ||
| kalikiana | o/ | 07:30 |
| zyga-ubuntu | jamesh: hello | 07:32 |
| mvo | Chipaca: 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 |
| Chipaca | mvo: there's no business logic | 08:13 |
| mvo | Chipaca: aha, and we need that client side? | 08:14 |
| jamesh | zyga-ubuntu: hi | 08:14 |
| Chipaca | mvo: as i understand it the client needs to sanity-check epochs as part of validation, yes | 08:14 |
| mvo | Chipaca: ok, thats fine then. was wondering if it should go into the 2.29 release notes draft or not :) | 08:15 |
| zyga-ubuntu | jamesh: hey! :) | 08:15 |
| Chipaca | mvo: i'd keep mum about it until both sides are done, in any case | 08:16 |
| kalikiana | elopio: still awake? mind reviewing snapcraft#1627 ? | 08:17 |
| mup | PR snapcraft#1627: lxd: split container classes into different files <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1627> | 08:17 |
| zyga-ubuntu | jamesh: how are you doing | 08:17 |
| zyga-ubuntu | jamesh: I'd like to sync on our work around mount features | 08:17 |
| mvo | zyga-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-ubuntu | mvo: I don't know of any workaround | 08:20 |
| jamesh | zyga-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-ubuntu | jamesh: I'm working on overlay support and I think it could now be used for improved content interface | 08:20 |
| zyga-ubuntu | jamesh: I'll probably look at a quick attempt to implement the new content interface ideas | 08:21 |
| zyga-ubuntu | jamesh: (the one with saner one-to-many connections) | 08:22 |
| zyga-ubuntu | jamesh: 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 work | 08:22 |
| pedronis | hi | 08:23 |
| jamesh | zyga-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 itself | 08:24 |
| jamesh | zyga-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 APIs | 08:25 |
| zyga-ubuntu | jamesh: 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 |
| jamesh | zyga-ubuntu: let's say that /foo is a fuse file system, and I want to do "mount --bind /foo/bar /mountpoint" | 08:31 |
| jamesh | zyga-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 |
| kalikiana | c-lobrano: Did you see snapcraft#1636 ? | 08:32 |
| mup | PR snapcraft#1636: internal: more gracefully determine host OS <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1636> | 08:32 |
| jamesh | the kernel will only let the user that mounted the fuse file system see files and directories within | 08:32 |
| zyga-ubuntu | jamesh: I see, how are they mounted by bubblewrap then? through capabilities? | 08:32 |
| jamesh | zyga-ubuntu: yeah. add CAP_SYS_ADMIN to permitted set, setuid(real_uid), reacquire CAP_SYS_ADMIN, mount | 08:33 |
| c-lobrano | Hey kalikiana, yes I saw it. Unfortunately I was to slow :D | 08:33 |
| zyga-ubuntu | jamesh: could we do that with the C preamble? | 08:33 |
| jamesh | zyga-ubuntu: using cgo might be the best option, yeah. | 08:34 |
| zyga-ubuntu | jamesh: snap-update-ns has a non-trivial C preamble to do various pre-go manipulations | 08:34 |
| ackk | niemeyer, hi, could you have another look at https://github.com/snapcore/snapd/pull/3916 ? | 09:55 |
| mup | PR #3916: snap,wrappers: add support for socket activation <Created by albertodonato> <https://github.com/snapcore/snapd/pull/3916> | 09:55 |
| Chipaca | pedronis: you're alive! \o/ | 09:56 |
| * Chipaca was starting to worry | 09:56 | |
| Chipaca | pedronis: how're you doing? | 09:56 |
| pedronis | Chipaca: better | 09:58 |
| Chipaca | pedronis: does that mean 'not 100% yet'? | 09:59 |
| pedronis | not 100%, but back to work, still haven't finished my treatments (some more days of pills) | 10:00 |
| Chipaca | pedronis: blagh :-/ | 10:01 |
| Chipaca | pedronis: 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 specially | 10:03 |
| === chihchun is now known as chihchun_afk | ||
| pedronis | Chipaca: I'm probably missing context, why doing tab completetion would break tests? | 10:05 |
| pedronis | and what are aliases from foo to foo | 10:05 |
| Chipaca | pedronis: aliases := []*backend.Alias{{"x", "x"}} | 10:05 |
| Chipaca | that's an alias for something that already exists, right? | 10:07 |
| pedronis | I suppose | 10:07 |
| pedronis | I still don't understand why that then breaks when you add completion | 10:08 |
| Chipaca | pedronis: i mis-spoke, the aliases don't break; matchingAliases et al don't return that as an alias, which means it's not returned | 10:09 |
| Chipaca | i mean | 10:09 |
| Chipaca | pedronis: should i push the branch and you read code, or should i continue trying to explain? | 10:10 |
| Chipaca | pedronis: i tweaked the helpers to return the matching aliased commands and completions | 10:10 |
| Chipaca | pedronis: commands is what was there before, those still work fine | 10:10 |
| pedronis | which helpers? | 10:11 |
| Chipaca | pedronis: completions will all be missing these auto-entries (the {"x", "x"} ones) | 10:11 |
| Chipaca | pedronis: matchingAliases, missingAliases | 10:11 |
| pedronis | matchingAliases is a test only helper | 10:11 |
| Chipaca | yes | 10:11 |
| Chipaca | as i say, it works, my doubt was with tests | 10:11 |
| Chipaca | so i wanted to check with you about the intent of those {"x", "x"} ones before just removing them from the completion checks | 10:11 |
| pedronis | can you give a test file name ? (there's a lot of tests) | 10:12 |
| Chipaca | pedronis: https://github.com/snapcore/snapd/compare/master...chipaca:complete-aliased#diff-66a739a11ce504d9bc94df4adb1e8bcdR377 | 10:13 |
| Chipaca | pedronis: the last two lines on that test | 10:13 |
| Chipaca | one is for the command aliases, which is as it was before | 10:13 |
| Chipaca | ie c.Check(matchingAs, DeepEquals, []*backend.Alias{{"baz", "y.baz"}, {"y", "y"}}) | 10:14 |
| pedronis | why completers need tests in backend? | 10:14 |
| Chipaca | pedronis: because they're created there? | 10:15 |
| pedronis | ah, this is not about completeing aliases | 10:15 |
| pedronis | this as about completing aliases args ? | 10:15 |
| pedronis | we were on completely different thoughts | 10:15 |
| Chipaca | this is about creating the completion snippet symlinks for aliased commands | 10:15 |
| Chipaca | that is, when you alias foo.bar to bar, you want bar<tab> to run the completer for foo.bar | 10:16 |
| pedronis | will not Alias{x,x} create a circular symlink ? | 10:17 |
| pedronis | you are probably reading too much in those tests | 10:18 |
| mup | PR snapcraft#1412 closed: lxd: snapcraft refresh in containers <Created by kalikiana> <Closed by kalikiana> <https://github.com/snapcore/snapcraft/pull/1412> | 10:18 |
| Chipaca | pedronis: Alias{x,x} wouldn't work because it either doesn't exist, or already exists, so in either case it won't get touched | 10:18 |
| Chipaca | pedronis: 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 back | 10:19 |
| pedronis | I have as much clue as you on this | 10:19 |
| Chipaca | hah | 10:20 |
| Chipaca | pedronis: i thought you were mr aliases :-) but ok | 10: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 |
| pedronis | Chipaca: wrong abotu what? | 10:21 |
| Chipaca | pedronis: about you being the person who knew about this bit | 10:21 |
| pedronis | I wrote that code | 10:21 |
| pedronis | it doesn't mean I remember why a unit tests is some way or another | 10:21 |
| pedronis | after months | 10:21 |
| pedronis | afaict Alias{x,x} will create a circular symlink | 10:22 |
| mup | PR snapd#4065 opened: servicestate: use taskset <Created by stolowski> <https://github.com/snapcore/snapd/pull/4065> | 10:23 |
| Chipaca | pedronis: fair enough. I'll go ahead and fix the tests to pass then, as the errors make sense | 10:24 |
| Chipaca | pedronis: thank you | 10:24 |
| Chipaca | pedronis: (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 |
| pedronis | Chipaca: well it might be that we don't defend about somebody trying to set up such an alias, I don't remember | 10:25 |
| * kalikiana break | 10:26 | |
| pedronis | Chipaca: so it might be worth checking what happens | 10:26 |
| pedronis | in that case | 10:26 |
| Chipaca | ok | 10:29 |
| pedronis | ah, 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 |
| pedronis | so we don't get there because of that | 10:31 |
| pedronis | fwiw | 10:32 |
| pedronis | Chipaca: ^ | 10:49 |
| Chipaca | pedronis: ack | 10:49 |
| jamesh | zyga-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-ubuntu | no, not really | 10:59 |
| zyga-ubuntu | I mostly use make hack | 10:59 |
| jamesh | that doesn't really help for snapd changes though | 11:00 |
| zyga-ubuntu | snapd is fine to run just from shell | 11:00 |
| zyga-ubuntu | go build && sudo ./snapd | 11:00 |
| Chipaca | zyga-ubuntu: no, because it panics if it's not running from /usr/lib/snapd/ | 11:01 |
| jamesh | I was getting this error: https://github.com/zyga/devtools/issues/25 | 11:01 |
| zyga-ubuntu | Chipaca: what? I never got that | 11:01 |
| zyga-ubuntu | Chipaca: maybe I had no-reexec key set | 11:01 |
| Chipaca | zyga-ubuntu: then you're not running it like that enough | 11:01 |
| Chipaca | zyga-ubuntu: i get this with no-reexec set | 11:01 |
| zyga-ubuntu | hmmm | 11:02 |
| Chipaca | zyga-ubuntu: when trying to install something | 11:02 |
| zyga-ubuntu | is that new? | 11:02 |
| Chipaca | zyga-ubuntu: a month? two? | 11:02 |
| Chipaca | super old | 11:02 |
| zyga-ubuntu | I really think it's a bit shooting ourselves in the foot | 11:02 |
| zyga-ubuntu | Chipaca: weird | 11:02 |
| zyga-ubuntu | maybe my braches are just equally old | 11:02 |
| zyga-ubuntu | maybe make hack should do more? | 11:02 |
| zyga-ubuntu | (make hack == ~~ make install) | 11:02 |
| mup | PR snapd#4066 opened: overlord/snapstate: support completion for command aliases <Created by chipaca> <https://github.com/snapcore/snapd/pull/4066> | 11:19 |
| Chipaca | ok, i think i should go for a run | 11:20 |
| === facundo__ is now known as Facu | ||
| * Chipaca goes | 11:39 | |
| niemeyer | ackk: Will do, thanks for the ping | 11:50 |
| ackk | thanks niemeyer | 11:51 |
| mvo | mwhudson: 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 |
| mvo | mwhudson: aha, nevermind, found the forum topic about it | 12:17 |
| mup | PR 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 |
| cachio | jdstrand, hey I am doing a test for the gsttings interface | 12:40 |
| leocadio_ | Is this possible? | 12:41 |
| cachio | jdstrand, I see that the gsettings command is not able to access the schemas | 12:41 |
| cachio | jdstrand, any idea about the configuration needed for that? is it ok to use the gsettings too to test that interface? | 12:42 |
| cachio | jdstrand, I already tried with a python script but had a lot of issues with dependencies and showed similar problems with the schemas | 12:42 |
| Chipaca | booo, something changed in travis and the "all good" thing is now even more garbled | 12:48 |
| * Chipaca will fix, sometime | 12:48 | |
| Chipaca | mvo: how are we for adding things to 2.29? | 12:48 |
| cachio | mvo, Chipaca are we still adding things to 2.29? | 12:49 |
| Chipaca | cachio: that's what i asked, more or less :-) | 12:49 |
| cachio | I already started beta testing on this | 12:49 |
| mvo | Chipaca, cachio first rc1 is out but there will most certainly a rc2 so things are still open | 12:50 |
| Chipaca | although 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 |
| mvo | but we already catching the first issues (ftbfs on ppc64el/zesty for example) | 12:50 |
| Chipaca | ok, if #4066 is as low drama as i think it is, i'll suggest it for point inclusion | 12:51 |
| mup | PR #4066: overlord/snapstate: support completion for command aliases <Created by chipaca> <https://github.com/snapcore/snapd/pull/4066> | 12:51 |
| Chipaca | otherwise .30 is fine | 12:51 |
| pedronis | mvo: 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 too | 13:33 |
| pedronis | if we redo the auto-import.assert one we probably need to redo some of the models too | 13:34 |
| pedronis | mvo: is this related to the fact that there's a limit on how long a system-user can last ? | 13:35 |
| mvo | pedronis: I think its unrelated but I have not looked in detail | 13:38 |
| pedronis | mmh, no, the limit is only if no models | 13:43 |
| pedronis | not sure why federico set it so | 13:43 |
| pedronis | but maybe we changed that in between | 13:43 |
| mvo | I suspect its and old test, that was one of the early ones | 13:44 |
| mup | PR snapd#4068 opened: interfaces/builtin: add support for content "source" section <Created by zyga> <https://github.com/snapcore/snapd/pull/4068> | 13:46 |
| jdstrand | cachio: 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 |
| cachio | jdstrand, good, thanks | 13:47 |
| cachio | mvo, 2.28.5 in stable | 13:50 |
| cachio | \o/ | 13:51 |
| zyga-ubuntu | I need to pick up my daughter | 13:52 |
| zyga-ubuntu | cachio: woot, thank you | 13:52 |
| ogra_ | heh, looking at snap info core looks cnfising with beta and edge now | 13:52 |
| zyga-ubuntu | jdstrand: o/ | 13:53 |
| jdstrand | zyga-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 edge | 13:54 |
| zyga-ubuntu | jdstrand: thank you! :-) | 13:54 |
| zyga-ubuntu | jdstrand: 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 4008 | 13:54 |
| mvo | ogra_: yeah, I think its confusing, edge is actually also new but the version does not reflect it. I will fix in a wee bit | 13:57 |
| mvo | cachio: yay, thank you so much! | 13:57 |
| cachio | mvo, np | 14:00 |
| cachio | niemeyer, this week could you take a look to the PRs in spread-cron? | 14:04 |
| cachio | niemeyer, taht would be grat if we can merge it soon | 14:04 |
| * kalikiana need.. more.. coffee.. | 14:19 | |
| kalikiana | kyrofa: elopio Are we having our weekly between the three of us then? | 14:46 |
| kalikiana | in 15min | 14:46 |
| kyrofa | kalikiana, I was assuming so after our discussion on Friday, but we honestly didn't make much more progress after you had to jet | 14:46 |
| kalikiana | kyrofa: Ah. Should we maybe schedule another one for that? I can work late tomorrow so we have more time. | 14:48 |
| mup | PR snapd#4069 opened: debian: do not build static snap-exec on powerpc <Created by mvo5> <https://github.com/snapcore/snapd/pull/4069> | 14:48 |
| zyga-ubuntu | re | 14:49 |
| kalikiana | kyrofa: 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 |
| kyrofa | kalikiana, fine by me-- I'm always happy to meet regardless | 14:52 |
| kyrofa | let's do it | 14:52 |
| kalikiana | Okay | 14:52 |
| kyrofa | I mean the one today. As for tomorrow, let's ask sergio when he's back | 14:52 |
| kyrofa | I do think we need to continue breaking it down, but not sure when | 14:53 |
| sergiusens | kyrofa what's up? | 14:54 |
| sergiusens | kyrofa I added comments to the OsRelease stuff too | 14:54 |
| sergiusens | kyrofa is snapcraft#1632 ready to go? this would be my final gripe to go to the stable channel :-) | 14:55 |
| mup | PR snapcraft#1632: libraries: exclude the full set of libc6 <bug> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1632> | 14:55 |
| kyrofa | sergiusens, indeed it is | 14:55 |
| kyrofa | sergiusens, I think we can improve that lib method, but it causes no harm in that PR | 14:56 |
| kyrofa | And I've tested it pretty extensively at this point | 14:56 |
| kyrofa | sergiusens, I'll go +1 it | 14:56 |
| kyrofa | sergiusens, you realize though that those other two PRs are required for trusty support, right? | 14:57 |
| sergiusens | kyrofa I have a PR refactor to change that module from `libraries` to `elf` and do some neat things in there | 14:57 |
| kyrofa | sergiusens, excellent | 14:57 |
| sergiusens | but I'd rather do minimal changes here so the test surface is not so broad to get a stable release out first | 14:57 |
| kyrofa | sergiusens, fair enough. +1d | 14:58 |
| zyga-ubuntu | niemeyer: I realized that due to a way apparmor is behaving we may need to have apparmor and mount kernels coordinate on the rename | 14:59 |
| zyga-ubuntu | niemeyer: I'll make a test case to examine this | 14:59 |
| mup | PR 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-ubuntu | niemeyer: I also think that we must be careful in mixing slots that use source and those that don't | 15:01 |
| kyrofa | sergiusens, 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 this | 15:01 |
| zyga-ubuntu | niemeyer: as this will cripple plug side | 15:02 |
| zyga-ubuntu | niemeyer: I was thinking about an attribute that would let the plug side define and control one-to-many expectations: only-one, allow-many | 15:02 |
| niemeyer | zyga-ubuntu: Which renames? | 15:02 |
| sergiusens | kyrofa I think it should be possible with these changes; non classic confinement at least | 15:02 |
| kyrofa | sergiusens, yeah that's a whole different ballgame | 15:02 |
| niemeyer | zyga-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 priorities | 15:03 |
| zyga-ubuntu | niemeyer: https://forum.snapcraft.io/t/improvements-in-the-content-interface/2387 | 15:03 |
| kyrofa | cjwatson, 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-ubuntu | niemeyer: ack | 15:03 |
| zyga-ubuntu | niemeyer: on that page look for "renamed" | 15:03 |
| kyrofa | cjwatson, the snapcraft in the archive is v1, targeting 15.04 | 15:03 |
| kyrofa | But there are people who actually need to build series 16 snaps on trusty | 15:04 |
| kyrofa | Oh sorry kalikiana got caught up in coffee | 15:05 |
| kyrofa | On my way | 15:05 |
| kalikiana | haha | 15:05 |
| kalikiana | no worries | 15:05 |
| niemeyer | zyga-ubuntu: By "kernels" you mean "backend" I suppose? | 15:08 |
| cjwatson | kyrofa: 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 selectable | 15:10 |
| kyrofa | cjwatson, well, the builders expose a lot more capability-- different archs, etc | 15:10 |
| kyrofa | cjwatson, curious to see if anyone is using it for v1 | 15:10 |
| kyrofa | (anyone using them today is either using v1 or expecting it to be v2 and giving up) | 15:11 |
| cjwatson | kyrofa: right, but BSI is strictly more limited, it doesn't have its own special build code or anything over and above what's on LP | 15:11 |
| kyrofa | cjwatson, right, which is why I specified them both | 15:11 |
| cjwatson | kyrofa: actually BSI would be out of scope here anyway since it's xenial-only | 15:11 |
| kyrofa | Ah | 15:11 |
| zyga-ubuntu | niemeyer: yes, not sure why I said "kernels" :) | 15:11 |
| cjwatson | kyrofa: we do have the notion of permitted combinations of Ubuntu series and snappy series in LP | 15:12 |
| niemeyer | zyga-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 needs | 15:12 |
| cjwatson | kyrofa: right now, the snappy series (e.g. 16) isn't passed through to the builder, but it could be | 15:12 |
| cjwatson | kyrofa: that's probably how we'd do that kind of thing if we needed to | 15:13 |
| zyga-ubuntu | niemeyer: 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 end | 15:13 |
| cjwatson | kyrofa: (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 |
| kyrofa | cjwatson, actually yeah, that doesn't sound like a bad idea | 15:15 |
| cjwatson | it would involve carefully-sequenced changes to both launchpad-buildd and LP itself, but apart from that it's pretty isolated | 15:15 |
| cjwatson | actually maybe not much sequencing | 15:16 |
| cjwatson | basically 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 involved | 15:19 |
| cjwatson | and then we could add the trusty/16 combination to SnappyDistroSeries via an API POST request once we've verified that it works | 15:20 |
| kyrofa | cjwatson, good deal, once a few of our PRs make it into a release, I'll look into that | 15:20 |
| kyrofa | (right now the snap doesn't run) | 15:21 |
| cjwatson | cool - let me know | 15:21 |
| cjwatson | I expect we'll want the builders to know the snappy series eventually for other reasons too | 15:21 |
| zyga-ubuntu | man | 15:22 |
| zyga-ubuntu | I wish to thank everyone for introducing the automatic and smart "night mode" | 15:23 |
| zyga-ubuntu | with almost no direct sun nowadays, it's always "dark and gloomy" | 15:23 |
| zyga-ubuntu | but at least my eyes hurt less | 15:23 |
| Chipaca | zyga-ubuntu: the redshift developers disagree :-) | 15:23 |
| zyga-ubuntu | Chipaca: the coverflow developers you say? ;-) | 15:24 |
| Chipaca | zyga-ubuntu: (because gnome wrote their own and made them redundant) | 15:24 |
| cachio | mvo, I am taking a look to the snap-confine issue that seems to be failing on autopkgtest:ubuntu-* | 15:24 |
| zyga-ubuntu | Chipaca: all software falls into the back hole of "built-in feature" | 15:24 |
| zyga-ubuntu | cachio: hmm, which issue is that? | 15:24 |
| cachio | https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-zesty-snappy-dev-image/zesty/amd64/s/snapd/20171023_140008_52dda@/log.gz | 15:24 |
| zyga-ubuntu | aha that one | 15:25 |
| zyga-ubuntu | I suspect it's a very very slow VM | 15:25 |
| zyga-ubuntu | unless you can reproduce it | 15:25 |
| cachio | something related to module:nvidia_modeset | 15:25 |
| zyga-ubuntu | cachio: does it only fail in that place? | 15:26 |
| zyga-ubuntu | cachio: that error is a very indirect way of saying "3 seconds elapsed but we took longer to finish stuff" | 15:26 |
| zyga-ubuntu | cachio: did you see it in any interactive work? | 15:28 |
| mup | PR 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-ubuntu | hmm | 15:33 |
| zyga-ubuntu | but something is indeed fishy | 15:33 |
| kyrofa | cjwatson, agreed, that seems generally useful | 15:33 |
| zyga-ubuntu | lots of tests are failing on this | 15:33 |
| zyga-ubuntu | mvo: maybe we should increase that timeout to 6 seconds | 15:34 |
| === JoshStrobl|AFK is now known as JoshStrobl | ||
| cachio | zyga-ubuntu, sorry, I was having lunch | 15:46 |
| cachio | zyga-ubuntu, well, that could be a reason, I'll try to reproduce it on a vm here | 15:46 |
| niemeyer | cachio: Reviewed both PRs there | 16:15 |
| cachio | niemeyer, tx | 16:15 |
| ogra_ | niemeyer, hrm ... when we switched to the forum were all the old mailing lists archives simply wiped ? | 16:35 |
| niemeyer | ogra_: 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.com | 16: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 ... though | 16:37 |
| cachio | niemeyer, please tell me if the doc that I added in the PR help to understand the change | 16:45 |
| niemeyer | cachio: Thanks, looking | 16:46 |
| ogra_ | cachio, sorry that you have to suffer from my shell-like pyton stuff :) | 16:53 |
| cachio | ogra_, np :) | 16:56 |
| ogra_ | (even though sergiusens claims that my python looks like perl ... :P ) | 16:57 |
| kyrofa | ogra_, hahaha | 16:59 |
| niemeyer | cachio: Replied | 17:01 |
| niemeyer | ogra_: Are you looking for this: https://lists.ubuntu.com/mailman/listinfo/snapcraft | 17:02 |
| sergiusens | ogra_ perl 6 even | 17: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 it | 17:02 |
| ogra_ | thanks ! | 17:03 |
| niemeyer | ogra_: That's the actual mailing list we used before the move to the forum | 17:03 |
| ogra_ | yeah | 17:04 |
| niemeyer | ogra_: It got renamed 20 years ago.. shortly after Linus came up with the Linux kernel | 17:04 |
| ogra_ | hahaha | 17:04 |
| kyrofa | ogra_, I need to see an example then. I can't imagine python that looks like an AES-encrypted message | 17:06 |
| sergiusens | cjwatson 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 too | 17:06 |
| ogra_ | kyrofa, well, i hate curly brackets ... so it wont be perl ever again :) | 17:07 |
| kyrofa | sergiusens, that's a great point, I wonder how hard it would be | 17:07 |
| sergiusens | kyrofa we wrote it down on a forum post already, somewhere at some point in time ;-) | 17:08 |
| kyrofa | sergiusens, haha, good luck finding it | 17:08 |
| ogra_ | sudo snap install forum-crawler ... | 17:10 |
| ogra_ | :P | 17:10 |
| kyrofa | $ snap find forum-crawler | 17:11 |
| kyrofa | The search "forum-crawler" returned 0 snaps | 17:11 |
| kyrofa | ogra_, you got my hopes up | 17:11 |
| ogra_ | haha | 17:11 |
| ogra_ | sorry | 17:11 |
| niemeyer | kyrofa: I find the forum search surprisingly effective actually | 17:11 |
| kyrofa | niemeyer, I've had pretty poor experiences with it, but I could also blame that on human error | 17:12 |
| ogra_ | which human exactly though ? | 17:13 |
| kyrofa | Yes I will leave that nebulous | 17:15 |
| ogra_ | :) | 17:16 |
| ogra_ | only 4 billion choices... | 17:16 |
| cachio | niemeyer, 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 tiredness | 17:47 | |
| === JanC_ is now known as JanC | ||
| kyrofa | snappy-m-o, autopkgtest 1583 xenial:amd64 xenial:armhf artful:amd64 | 17:50 |
| snappy-m-o | kyrofa: I've just triggered your test. | 17:50 |
| niemeyer | zyga-ubuntu: ping | 18:39 |
| niemeyer | jdstrand: ping.. maybe you might know the answer | 18:46 |
| zyga-ubuntu | niemeyer: yes? | 18:46 |
| zyga-ubuntu | (not here full time anymore) | 18:46 |
| niemeyer | zyga-ubuntu: Oh, hay | 18:46 |
| zyga-ubuntu | how can I help? :) | 18:46 |
| niemeyer | zyga-ubuntu: Sorry, I realize it's a bit late | 18:46 |
| zyga-ubuntu | that's ok, I'm trying to solve a problem and I came to give it another try | 18:47 |
| niemeyer | zyga-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-confine | 18:47 |
| mup | PR #3958: many: add support for /home on NFS <Created by zyga> <https://github.com/snapcore/snapd/pull/3958> | 18:47 |
| niemeyer | zyga-ubuntu: But I cannot see any code related to this | 18:47 |
| niemeyer | zyga-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-ubuntu | niemeyer: right, thank you for checking | 18:48 |
| zyga-ubuntu | niemeyer: 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 NFS | 18:49 |
| niemeyer | zyga-ubuntu: Aha, ok.. that makes sense | 18:49 |
| zyga-ubuntu | niemeyer: some of those, similar to NFS, leak through the abstractions apparmor gives us | 18:49 |
| niemeyer | zyga-ubuntu: Merging then, thanks | 18:49 |
| zyga-ubuntu | niemeyer: we have some workarounds but AFAIK what doesn't work very well is "snap try" on top of one | 18:49 |
| zyga-ubuntu | thank you! | 18:49 |
| niemeyer | zyga-ubuntu: Do you have anything on top of this, or can I squash? | 18:50 |
| zyga-ubuntu | niemeyer: no, nothing on top | 18:50 |
| zyga-ubuntu | feel free to squash/not squash :) | 18:50 |
| niemeyer | Ok, here we go | 18:50 |
| zyga-ubuntu | woot, thank you | 18:50 |
| zyga-ubuntu | I'm sure CE will like that aspect of 2.29 :) | 18:51 |
| mup | PR 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 |
| pstolowski | niemeyer, hey, can you take a look at my 2 interfaces-related PRs before your vacation? | 18:56 |
| niemeyer | pstolowski: Yeah, will do, thanks for the ping | 18:56 |
| pstolowski | cool | 18:57 |
| zyga-ubuntu | woot | 19:14 |
| mup | PR snapd#4070 opened: hooks/configure: queue service restarts <Created by stolowski> <https://github.com/snapcore/snapd/pull/4070> | 19:15 |
| pstolowski | niemeyer, ^ I decided to simplify the approach with the above (compared to what I was trying to explain in the standup) | 19:17 |
| pstolowski | niemeyer, happy to discuss/explain if needed | 19:18 |
| niemeyer | pstolowski: Sounds good, thanks! | 19:20 |
| niemeyer | pstolowski: Reviewing now | 19:21 |
| zyga-ubuntu | niemeyer: woot, no need to sync anything, I got one thing wrong; apparmor needs the original path :) | 19:31 |
| zyga-ubuntu | niemeyer: and de-clashing of plugins should now be oK | 19:31 |
| zyga-ubuntu | pushed and EODing | 19:35 |
| zyga-ubuntu | well, | 19:36 |
| zyga-ubuntu | one more small patch | 19:36 |
| zyga-ubuntu | then EOD | 19:36 |
| niemeyer | zyga-ubuntu: Which original path? | 19:37 |
| zyga-ubuntu | niemeyer: let me show you an example | 19:39 |
| zyga-ubuntu | https://github.com/snapcore/snapd/pull/4068/files#diff-7d64932a9164eaac515f310a2759e820R506 | 19:39 |
| mup | PR #4068: interfaces/builtin: add support for content "source" section <Created by zyga> <https://github.com/snapcore/snapd/pull/4068> | 19:39 |
| zyga-ubuntu | here, this is exactly what I was trying to solve | 19:40 |
| zyga-ubuntu | one app, two plugins (via content) | 19:40 |
| zyga-ubuntu | in the mount entry they are renamed to unclash | 19:40 |
| skjensen | Evening 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 range | 19:40 |
| zyga-ubuntu | in the apparmor profile they are not renamed because apparmor needs a rule for the source (which is the original path) because of AF_UNIX shortcomings | 19:40 |
| zyga-ubuntu | and I just saw a typo :) | 19:40 |
| skjensen | Full error description can be found here: https://pastebin.com/5aUPQXcB | 19:41 |
| zyga-ubuntu | there | 19:41 |
| zyga-ubuntu | (I wrote "indented" instead of "intended") | 19:42 |
| zyga-ubuntu | jdstrand: I think 4068 is also interesting for you, in the same vein as other PRs lately | 19:46 |
| * zyga-ubuntu EOds | 19:51 | |
| cachio | niemeyer, I got disconnected, did you see what I wrote? | 21:10 |
| kyrofa | elopio, 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.gz | 22:24 |
| kyrofa | Ruby failures on artful autopkgtests | 22:24 |
| elopio1 | segmentation 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!