[00:00] if it's a known issue, is there a plan to, or has it been, solve(d)? [00:03] snapcraft seems to have trouble with non-ascii === cwayne_ is now known as cwayne [00:04] yeah. that's crazy considering that 90% of the population aren't English [00:05] of course 87% of all statistics are made up on the spot, but you get my point [00:05] lol [00:31] 502 proxy error? [00:35] hmm? [00:35] probably https://bugs.launchpad.net/launchpad/+bug/1683819 [00:35] Bug #1683819: Continuous 502 proxy error on snap upload to store [00:58] diddledan: https://bugs.launchpad.net/snapcraft/+bug/1662456 [00:58] Bug #1662456: snapcraft fails if desktop file included "ascii" cannot decode byte [00:58] ah, thankyou [00:58] * diddledan clicks that it affects me [00:59] so now i get: desktop interfaces (x11) specified without meta/gui/*.desktop. Please provide a desktop file via setup/gui/*.desktop if using snapcraft or meta/gui/*.desktop otherwise. It should reference one of the 'apps' from your snapcraft/snap.yaml. [00:59] i have no idea what this means [00:59] ali1234: does your yaml refer to the desktop interface? [00:59] no? [01:00] it refers to x11 [01:00] ali1234: right, that's what the above messages says is "desktop interfaces" [01:00] okay but what is setup/gui/*.desktop? [01:00] ali1234: a .desktop file for yoru application [01:00] what is the setup/gui part? [01:01] ali1234: i'm guessing a path in your snap's source [01:01] its nothing to do with anything i am trying to build [01:02] ali1234: hrm? you're tyring to use an interface that apparently requires .desktop files [01:02] ali1234: so ... absolutely has to do with wha tyou're building (a snap) [01:02] it doesn't require them, the snap works fine when i build and install it locally [01:02] setup is a snapcraft-specific thing that if desired needs to be a folder along-side your yaml file [01:03] ali1234: could be a disagreement about the version of snapcraft you're using and the one the builder is using? [01:03] so it is telling me to create setup/gui/pulseview.desktop, and then put it in the snap somehow? [01:03] ali1234: you put it in the repository containing your yaml [01:03] it'll put it in there automatically [01:03] ali1234: at that path [01:03] ali1234: and it will dtrt [01:04] so i just create it? [01:04] how does it know what filename to look for? [01:04] let me screenie my tree [01:04] https://usercontent.irccloud-cdn.com/file/I0ALdl5B/ [01:04] so i do have to make the directories? [01:04] ali1234: presumably via the application name that is using the interface? [01:04] and what is your app called? [01:04] ali1234: yes, you ahve to make the directories [01:05] ali1234: it's telling you, you need a file at setup/gui/.desktop [01:05] the app is called pulseview but it shows up as sigrok.pulseview when installed, because the snap has two apps [01:05] ali1234: so of course you need the directgories [01:05] ali1234: how else would you create the file in the directories? [01:05] well, i would use dump plugin with a line like organize: pulseview.desktop: setup/gui/pulseview.desktop [01:06] that's how you normally get a file into the snap [01:06] ali1234: this is not into the snap [01:06] ali1234: this is in the 'source' [01:06] the error comes from store review of the built snap [01:06] ali1234: the same place your yaml lives [01:06] ali1234: right, so you put in the source and snapcraft will put it in the right place [01:06] so i would expect it means it wants the file in the snap [01:06] that's my reading [01:07] it's an automatic thing [01:07] you don't do anything with it other than create it [01:07] ali1234: i don't think you do anything with teh built snaps [01:07] that's all snapcraft's job [01:07] you build them [01:07] i.e. don't try to put it in the build process [01:07] except maybe organize them [01:08] snapcraft handles the setup/ folder automatically [01:08] so what did you put in the .desktop file? [01:08] ahi found an example [01:09] diddledan: right that makes sense, it's like custom parts [01:21] I'm getting closer to finally achieving a custom gtk build [01:21] corebird 1.5 (released a few days ago) requires more recent gtk deps than xenial provides so my snap needs custojobby [01:22] I have managed a complete compile but I'm still working out the right stuff to force into the snap === chihchun_afk is now known as chihchun [07:12] ogra_, it looks like --extrausers is not working for chfn command in latest core snap (for armhf at least), which prevents "snap create-user" to work, is that a known issue? [07:41] abeato: we found that yesterday and ogra_ created a new core snap which should fix that problem [07:41] abeato: that should be fixed in edge [07:43] morphis, great... the perils of using edge ;) [07:43] abeato: yes :-) [07:55] PR snapd#3281 opened: tests: add test for empty snap name on revert [08:13] re [08:13] good morning [08:19] morning zyga ! [08:23] good morning peeps! [08:24] morning [08:25] all that stuff last night about meta gui? turns out its just a warning, not an error [08:25] so now i have 6 versions of the same snap pending review in the store [08:25] Chipaca: hey hey :) [08:25] pstolowski: good morning! [08:25] morning! [08:25] * zyga coundn't help it and had to take a walk in the morning [08:26] morphis: o/ [08:26] zyga: can you give https://github.com/snapcore/snapd/pull/3274 another look? I've added more test cases and printing a warning whenver a confinement check is ignored [08:26] PR snapd#3274: cmd/snap,tests/main: add force-devmode switch instead of spread system blacklisting [08:26] morphis: I already am, I'm mid way through the new commit [08:26] great [08:40] PR snapcraft#1295 closed: recording: record build packages installed in the snap [08:44] fgimenez: do we use travis oauth tokens? [08:45] fgimenez: (good morning, also :-) ) [08:51] Chipaca: hey :) if you mean github oauth tokens from travis nope we let travis interact with github [08:52] fgimenez: ah alright [08:52] asking because of https://blog.travis-ci.com/2017-05-08-security-advisory [08:52] but if we don't, great :-) [08:52] “Various GitHub OAuth tokens and secure environment variables that users included in their builds were accidentally exposed via inclusion in build logs on Travis CI.” [08:52] Chipaca: yep, no problem for us, we push to gh, but from our side of the vpn [08:57] morphis: quick suggestion, perhaps the skip part could be a function that we source so that we don't have to copy-paste the same line for consistency in behavior [08:58] zyga: you mean the snap forced-devmode check? [08:58] morphis: otherwise looks ok, I was doing some other things but I'll +1 it (again) in a moment [08:58] morphis: yeah, something about that, it's tricky because it's simetimes if ... then exit; but sometimes not [08:58] zyga: I was thinking about that before but actually like this to be really explicit and as it might be used in different combinations [08:59] morphis: yeah, I understand [09:00] lets leave it lets see how complex the logic gets into the future [09:00] I will have multiple changes for fedora too which makes things more complicate and might refactor a few of these tests again [09:00] zyga, hey, wrt to the problem of hanging configure hook on debian; I found out (by examining and comparing strace for that hook on both debian ubuntu) that a posible suspect is: "bind(3, {sa_family=AF_INET6, sin6_port=htons(0), inet_pton(AF_INET6, "::1", &sin6_addr), sin6_flowinfo=htonl(0), sin6_scope_id=0}, 28) = ? " [09:01] zyga, i'm not familiar with ipv6 peculiarities, any idea what may be different in that aspect on debian? [09:01] pstolowski: hey [09:02] pstolowski: isn't that what jdstrand found when initially looking at the bug? [09:04] morphis, oh, where did he mention that? [09:04] https://bugs.launchpad.net/snappy/+bug/1674193/comments/16 and https://bugs.launchpad.net/snappy/+bug/1644573 [09:04] Bug #1674193: core snap's configuration hangs on debian | openSUSE | mainline kernel [09:04] Bug #1644573: snapctl causes hooks to attempt to open ip/ipv6 tcp connection [09:04] pstolowski: the thing is that there was a fix merged in 2.23.6 merged which broke with 2.24 again [09:05] pstolowski: verified once ago that this problem was fixed with 2.23.6: https://bugs.launchpad.net/snappy/+bug/1674193/comments/25 [09:07] morphis, oh, wonderful! thanks for these links [09:07] pstolowski: they* are on your forum post already :-) https://forum.snapcraft.io/t/tests-broken-in-master/457/8 [09:08] morphis, oh indeed.. [09:08] * pstolowski embarassed [09:08] happens :-) [09:08] so i finally have an approved snap in the store. do i just click release now? [09:17] ali1234: as you wish :-) [09:18] what does the release button do? [09:18] what is the difference between releasing and publishing? [09:19] ali1234: if my memory serves, the release button takes you to a page where you choose which channels to publish it to [09:20] oh [09:20] so it has a terrible name [09:21] ali1234: the button? [09:21] and it should just say "edit channels" [09:21] ali1234: you don't edit channels with it though [09:21] really? [09:21] ali1234: editing a channel means changing the channel itself [09:21] why does it show a form labelled channels: which i can edit then? [09:21] "Select the channels to release current revision." [09:22] yep [09:22] no, you mean publish [09:22] PR snapcraft#1297 closed: sources: validate unknown source-type in yaml [09:22] "Select the channels to [publish] current revision [in]." [09:22] ali1234: i'd suggest you open a forum topic about this [09:22] it's possible the store uses publish and release inconsistently [09:23] but i don't know; there might be nuance [09:23] if you have a button with a verb on it i expect clicking it will do that verb [09:23] but it's definitely not "edit a channel"; it's editing a channel assignment, which seems to be called releasing [09:23] ali1234: as i say, please open a forum topic [09:23] unless you're just ranting, that is :-) [09:23] ranting is fine btw [09:24] i can rant on the forum if you want [09:24] ali1234: forum is for constructive, long-term engagement [09:24] well, long-term in internet time :-p [09:24] rants are fine here [09:24] okay. where do i go to just report a list of problems? [09:25] i am not interested in long term engagement. i will just come back in another 9 months and see if you have fixed all the problems [09:25] ali1234: the forum is fine to just report a list of problems [09:25] ali1234: (the 'long-term' side of that kind of post comes from the people going through the list and addressing them) [09:26] maybe a better term would be 'asynchronous' actually [09:27] vs irc that is synchronous [09:27] should i create one topic for each issue? [09:27] or just make one long rant post? [09:27] ali1234: just one long one is probably easier for you [09:27] ali1234: i believe moderators can then split it up [09:28] yeah but we both know they won't [09:28] dude [09:28] someone will come along and address one of the problems (probably by saying "that isn't in scope of the project") and then the rest will be ignored [09:28] that's what happened yesterday [09:28] ali1234: where? [09:29] https://forum.snapcraft.io/t/trying-to-build-sigrok-snap/503 [09:30] ali1234: that looks like the opposite [09:31] really? i was told to use launchpad, which addresses exactly one of the four problems i raised [09:31] it's not being ignored; you even got ev following up a day later [09:31] which was less than an hour ago [09:32] Chipaca, mind giving a second thumbs up on https://github.com/snapcore/core/pull/37 ? (trivial changes) [09:32] PR core#37: install keyring (LP: #1670475), update pkglists [09:33] * ogra_ hugs Chipaca [09:34] PR core#37 closed: install keyring (LP: #1670475), update pkglists [09:35] ali1234: about the snapcraft.yaml in the root of the repo, I *think* (but am not sure) that you can also make it .snapcraft.yaml, or snap/snapcraft.yaml [09:35] you can [09:35] upstream will put it at cross-compile/snap/snapcraft.yaml because that is how their repository is organized [09:36] Hi, I'm trying to create a snap of my app (https://github.com/tizonia/tizonia-openmax-il) and I'm currently facing problems with pulseaudio. The snap I've created installs correctly in Fedora 25 but it can't access pulseaudio (Access Denied)...is there any good reference or tuturial on how to create snaps that need access to pulseaudio? [09:36] juanrubio: is it using the pulseaudio interface? [09:36] Chipaca: we don't ahve any confinement on Fedora so the interface is useless unless its a seccomp issue [09:37] juanrubio: can you give us the exact error message? [09:37] yes, plugs section contains 'pulseaudio' [09:37] juanrubio: is the plug connected? [09:37] morphis: point [09:37] the error message is coming from my app [09:37] * Chipaca goes back to work [09:38] juanrubio: is there an error in dmesg or journalctl? [09:38] "Access denied" (which comes from pulseaudio's api) [09:38] i can't seen any errors in journalctl's output [09:39] juanrubio: and you're running your app with the same user you're logged into the system [09:39] morphis: i thought the plug would be automatically connected? [09:40] yes, same user [09:40] juanrubio: is it connected or not? [09:41] if I connect the plug with 'sudo snap connect tizonia:pulseaudio :pulseaudio' i get the same result [09:41] juanrubio: can you share the .yaml? [09:42] oh it's there in tools/snapcraft.yaml [09:42] yes, in tools [09:44] I created my snap in Ubuntu 16.04 and Fedora 25 runs on a Virtualbox VM [09:45] juanrubio: and does it work in ubuntu 16.04? [09:45] actually, I have not tried it in ubuntu [09:45] please do [09:46] ok, let me give it a try in an 16.04 VM i have over here [09:48] pstolowski: re [09:48] pstolowski: sorry, I was in a call [09:48] PR snapd#3282 opened: hooks: default timeout [09:49] pstolowski: so the problem is snapctl and golang's network stack that checks if you have ipv6 connectivity/support by creating that socket [09:49] pstolowski: as you know on ubuntu & linode we disable ipv6 (as there was some problem with routing, historically) [09:49] pstolowski: but I don't think we disable that on debian [09:49] pstolowski: we may thus hit a confinement issue where seccomp kills part of snapctl [09:50] pstolowski: I think in addition the actual hook, before it tries snapctl, checks something else which on ubuntu is blocked by apparmor [09:50] pstolowski: so when the plug (core-support-plug) is disconnected, the hook exits gracefully [09:52] juanrubio: hey [09:53] hi zyga, i've filed this bug https://bugs.launchpad.net/snappy/+bug/1689332 wrt the issue with the 2.25 ubuntu-core snap and the hardcoded snap-confine path [09:53] Bug #1689332: internal error with any command using ubuntu-core snap on classic [09:53] juanrubio: intersting, what is your snap doing with audio? just playback or something more? [09:53] fgimenez: thank you! I'll assign it to myself [09:54] zyga, my understanding - after looking at comments from jdstrand and later morphis - is that yes, it's ipv6 reatlated, Jamie recommended a specific policy rule. the problem was fixed but by mistake not included in the 2.24 release, but according to morhpis it's in the new release (https://forum.snapcraft.io/t/tests-broken-in-master/457/8?u=pstolowski) [09:54] morphis, https://github.com/snapcore/pi3-gadget/pull/6 [09:54] PR pi3-gadget#6: make pi-bluetooth name more generic [09:54] zyga: great thanks! :) do you think this issue should block the 2.25 sru? [09:54] pstolowski: I'm not sure I understand, what do you aim to achieve in the end? [09:54] Bug #1689332 changed: internal error with any command using ubuntu-core snap on classic [09:55] fgimenez: well, hard to say, it only affects people that are on ubuntu-core (not just core) [09:55] zyga, and i've just proposed a PR to have a default timeout in case we have a task created by old snapd with no timeout set [09:55] PR snapcraft#1288 closed: store: collaboration UI for snaps [09:55] zyga: audio playback [09:55] fgimenez: I think it is not so critical but perhaps my view is just too optimistic [09:56] zyga: ok thanks, we can talk later in the meeting [09:56] juanrubio: so fedora 25 may have a more recent version of pulseaudio than ubuntu 16.04 but I bet that pulseaudio still supports the older protocol [09:56] juanrubio: and even on fedora 26 I saw snaps play audio [09:57] juanrubio: could you please open a forum topic for this (on forum.snapcraft.io), perhaps other people are facing the same issue and even if not we can try to help you there [09:57] juanrubio: if your snap is public I could try it on my f26 machine [09:57] juanrubio: and try to debug it there [09:57] zyga: these are my first steps with snaps, so I'm sure I'm missing something [09:57] juanrubio: does it work on ubuntu? if so, it should work on fedora too [09:57] zyga: i think there's another issue with https://github.com/snapcore/snapd/pull/3218/files, i'm getting "Bad system call" trying to use systemd-cat from a snap, will file a bug about this too [09:57] PR snapd#3218: interfaces: allow writing to /run/systemd/journal/stdout by default [09:58] zyga: I'm currently updating a 16.04 VM that I have here. I will try it out in 16.04 in a few minutes [09:58] morphis, thx [09:58] ogra_: np [10:04] fgimenez: in lxc? [10:04] zyga: nope, on a regular classic vm [10:05] I'm opening a forum topic for this, which category 'snap' or 'snapcraft'? [10:08] hmm [10:08] fgimenez: what was the system call? do you have the error message? [10:08] juanrubio: snap please [10:08] zyga: sure, [ 910.074185] audit: type=1326 audit(1494320944.576:31): auid=1000 uid=12345 gid=12345 ses=1 pid=30704 comm="systemd-cat" exe="/usr/bin/systemd-cat" sig=31 arch=c000003e syscall=48 compat=0 ip=0x7fcdee899987 code=0x0 [10:09] fgimenez: 48 is "shutdown" [10:09] fgimenez: hmm, probably closing some socket [10:09] hm, did anyone run into this one with spread already: in test-snapd-tools.channels: extra keys: {'latest/candidate', 'latest/beta'} [10:09] fgimenez: where did that fail? was it a specifi test? [10:09] that is for tests/main/snap-info [10:09] morphis: yep on it [10:10] fgimenez: great! [10:10] morphis: nope, but Chipaca wrote that so I bet he groks he messages [10:10] where what? [10:10] * Chipaca reads up [10:10] Chipaca: https://travis-ci.org/snapcore/snapd/builds/230270742?utm_source=github_status&utm_medium=notification [10:11] zyga: not yet, i was adding it while reviewing the 2.25 changelog https://github.com/snapcore/snapd/compare/master...fgimenez:spread-interfaces-default?expand=1#diff-e2f8bc979b0f080a4b32db7b71057b46R9 [10:11] morphis: that looks like a snap/snapd version mismatch [10:12] as in, snapd sending track/channel, snap expecting channel [10:12] but let me look at it a moment [10:13] morphis: Chipaca probably related, i just uploaded an updated version of test-snapd-tools and the snap-info test started failing, reverted back to the previous version and keep failing [10:13] fgimenez: it might be that the test expects it to be only in certain channels [10:13] and you published it to more of them [10:13] it'd look like this, also [10:14] i.e. it used to only be in stable and edge, and now it's also in candidate and beta [10:14] Chipaca: i tried to keep the same channels, rev 3 was in stable, candidate and beta (as it is now) [10:15] Chipaca: aha, let me try to remove it from candidate and beta [10:16] so, yes, the snap-info test expects it only in stable and edge [10:16] morphis: found the problem :-) [10:16] ("channels", check, [10:16] ("latest/stable", matches, verRevNotesRx("-")), [10:16] ("latest/edge", matches, verRevNotesRx("-")), [10:16] ), [10:17] that will fail if the channels key has more entries than those two [10:18] actually, I said before that journalctl didn't show any messages, but actually i didn't realise that the older messages were at the top.. so yes I see a bunch of errors, not sure I understand much... what should I look for? [10:19] juanrubio: anything with 'audit' in it [10:19] juanrubio: can you paste those somewhere? [10:22] I've pasted some of it here: https://gist.github.com/juanrubio/d7ed44344ef2e7a659dede5f788d07f5 [10:22] PR snapd#3280 closed: configstate: return error if patch is invalid [10:23] permissive=0 is curious, [10:23] ogra_: remind me, how long does the prepare-device hook take on the bbb, worst case? [10:24] Chipaca, i havent tested in a while but it should not take long anymore [10:25] it used to be 10-15min in the wrst times [10:25] *worst [10:25] and this are the steps I'm following to install the snap on Fedora 25 : https://gist.github.com/juanrubio/4ca10f5a6468fc12bcbed6bf01a7d8f7 [10:25] Chipaca: now i begin to rant about the release page too :) do you know if i can unpublish a released revision from a channel somehow? [10:26] fgimenez: yes, you click on the channels and edit the map as you please [10:26] fgimenez, i dont think you can "unrelease" but you can release a former version [10:26] fgimenez: i think i should be able to do it [10:26] * Chipaca looks [10:26] PR snapd#3276 closed: tests: additional setup in docker test for core systems [10:26] juanrubio: why do you had to do $ mkdir -p ~/snap/tizonia/x1/.config/tizonia ? [10:27] iirc the store spills an error if you just try to unrelease [10:28] juanrubio: was that just because of ~/snap/tizonia/x1/.config missing or did even ~/snap/tizonia/x1/ not exist? [10:28] Chipaca: releasing a former version leaves the candidate and beta channels in snap-info's output http://paste.ubuntu.com/24542209/ not sure if the test would pass with this? [10:30] that looks like a bug [10:30] noise][: are you around? [10:30] morphis: my app needs this 'tizonia.conf' located in $HOME/.config/tizonia [10:30] fgimenez: i mean, what looks like a bug is the not being able to remove channels [10:30] juanrubio: so ~/snap/tizonia/x1/ was automatically created? [10:30] maybe it's always been this way but i thought not [10:31] so I currently copy this manually [10:31] ~/snap/tizonia/x1 is automatically create [10:31] ~/snap/tizonia/x1 is automatically created [10:31] ok [10:31] the idea of opening a channel, having a snap in it, and then closing it has existed forever [10:33] Chipaca: yep i think so, the message from the release page is clear though "Released channels cannot be removed." [10:35] Chipaca: this fixes the tests http://paste.ubuntu.com/24542214/ should i propose it? [10:35] right, thats the typüical message [10:37] fgimenez: if it can wait a bit, i'd rather first determine whether the store is broken in this way [10:37] or if it's how it should be, in which case yes let's fix the test [10:38] Chipaca: ok [10:38] Chipaca, it has always been like this (at least with the web UI) [10:38] fgimenez: if determining this takes more than an hour, let's fix the tests [10:39] the above is the message you get when unticking the channel checkbox ... but ticking it on an older version will override and release that version instead [10:39] Chipaca: ok sounds good [10:45] hmm something wrong with my snap. i now have both meta/gui and setup/gui [10:45] My 16.04 is taking a bit long to update. I've uploaded the snap package to this location, in case someone might want to give it a try on Fedora 25 or 26: http://www.juanrubio.me/tizonia/tizonia_0.7.0_amd64.snap [10:45] actually it looks like it dumped everything from the source repo into the snap [10:46] pstolowski: hey, I read https://github.com/snapcore/snapd/pull/3282/files and it looks OK but I recall you said it didn't work, do I remember that correctly? [10:46] PR snapd#3282: hooks: default timeout [10:50] zyga, I said that, yes, but later I figured out (correct me if I'm wrong) that the test I was using in qemu re-execs into new snapd from core image, so I wasn't really testing my change at that time [10:51] pstolowski: I think you were, because the "new" snapd was the one from the package [10:51] pstolowski: and the package is built [10:51] pstolowski: I mean we re-write the core snap [10:51] juanrubio: getting a 404 [10:52] juanrubio: ah now it worked [10:53] Chipaca: anyway, something strange has happened with test-snapd-tools, it seems that beta and candidate have been used since rev 1 http://paste.ubuntu.com/24542284/ (i actually checked the channels used by rev 3 before releasing rev 6, and those included candidate and beta), how was snap-info test passing before? [10:53] I've created a forum topic: https://forum.snapcraft.io/t/pulseaudio-access-denied-accessing-pulseaudio-for-audio-playback-in-fedora-25/519 [10:54] juanrubio: what do i do to run it? [10:54] * Chipaca knows nothing about tizonia [10:54] fgimenez: ¯\_(ツ)_/¯ [10:55] fgimenez: let's see what facu says [10:56] fgimenez, the pi2-kernel in edge is pretty outdated, i'll sync the newer one from beta into it (runs fine here, just FYI in case any tests start to misbehave in edge) [10:59] ogra_: ok thanks! i'll give it i try [11:00] zyga, hmm.. ok, i'm confused. i understand what you're saying, just not sure about my findings anymore. I could manually kill -9 the hanging configure script and this is what we do in hookmgr. [11:02] * Chipaca hugs Facu [11:02] juanrubio: the help output for tizonia talks about ~/.config/tizonia/tizonia.conf, which is wrong [11:02] pstolowski: I could try your branch and experiment [11:03] morphis, hmm, who owns the wifi-ap snap ? seems i cant run the setup wizard anymore ... http://paste.ubuntu.com/24542320/ [11:03] Chipaca: I was using it with my Spotify account, but the pulseaudio problem should be seen while trying to play a local mp3 file [11:03] Chipaca: why do you say the help output is wrong? [11:04] ogra_: we do [11:04] juanrubio: because a snap won't be able to read ~/.config/tizonia/tizonia.conf [11:04] /find nikc "we" [11:04] *nick [11:04] :P [11:04] juanrubio: apps usually set XDG_CONFIG_DIR (or whatever it was) to something inside ~/snap/ [11:04] ogra_: you ping'ed the right one :-) [11:04] zyga, feel free to if you have a few moments [11:04] ogra_: which syscalls are denied? [11:04] morphis, werll, seems "socket" [11:05] Chipaca: right now I am only concerned about making it work on Fedora as a test, will deal with those other things later [11:05] ogra_: are all plugs connected? [11:05] XDG_CONFIG_HOME, actually [11:05] juanrubio: ok fair enough [11:05] that worked before i reinstalled the image ... all auto-connect ones are [11:05] juanrubio: fedora doesn't have apparmor confinement so it's less picky :-) [11:05] ogra_: can you paste $ snap interfaces wifi-ap [11:05] juanrubio: another question, why do you use the alsa plug? [11:06] morphis, http://paste.ubuntu.com/24542331/ [11:06] juanrubio: and yet another question, how do i play a local mp3 :-) [11:06] Chipaca: tizonia can render audio with alsa and pulseaudio [11:06] morphis, and i'm only calling "sudo wifi-ap.setup-wizard" [11:06] nothing fancy [11:07] but that will talk with the service over a local socket [11:07] you could play an mp3 like : tizonia my-mp3.mp3 [11:07] juanrubio: ok. Just note that pulseaudio is connected by default, but alsa isn't (because it lets you do nasty things) [11:07] ogra_: hm [11:07] morphis, well, whatever it does ... it worked 100 times for me before ... [11:07] juanrubio: all it ever does is complain about a missing tizonia.conf [11:07] (probably only 87 ... i'm exaggerating ;) ) [11:08] ogra_: can you check if the socket syscall is in the seccomp profile? [11:08] Chipaca: yes, please go to my post on snapcraft.io.. I've posted the files tizonia.conf and .log4crc [11:09] https://forum.snapcraft.io/t/pulseaudio-access-denied-accessing-pulseaudio-for-audio-playback-in-fedora-25/519 [11:09] juanrubio: wait, it doesn't work in --devmode either? [11:09] * zyga spends a moment on improving some tests [11:10] pstolowski: I'll check your branch just after that [11:10] morphis, looks fine to me http://paste.ubuntu.com/24542345/ [11:10] juanrubio: anyway, thanks for those files, looking [11:11] Chipaca: thanks for your time [11:11] ogra_: yeah .. [11:11] ogra_: which snapd version are you running? [11:12] ogra@pi3:~$ snap version [11:12] snap 2.25+201705082317.git.3ff5a56~ubuntu16.04.1 [11:12] snapd 2.25+201705082317.git.3ff5a56~ubuntu16.04.1 [11:12] series 16 [11:12] kernel 4.4.0-1051-raspi2 [11:12] so edge [11:12] can you try with beta? [11:12] edge ... from 2h ago or so [11:12] refreshing ... [11:12] I fear that the seccomp arg filtering jdstrand landed recently might be the problem here [11:13] well, see line 525 of the profile paste ... [11:13] "# FIXME: remove this after LP: #1446748 is implemented" [11:13] offtopic, I'd *love* to merge https://github.com/snapcore/snapd/pull/3026 [11:13] Bug #1446748: implement seccomp filtering by argument [11:13] PR snapd#3026: cmd/snap-confine: use defensive argument parser [11:13] juanrubio: tizonia dies with "Unable to verify the component list." (and leaves the terminal in a bad state) [11:13] ogra_: hm [11:14] ogra_: lets verify first if this a regression [11:14] fgimenez: btw did 'snapcraft close' work? [11:14] i wonder if all snaps in the store that use this workaround have to be updated in the store [11:14] morphis, yeah ... rollback to beta works [11:15] ogra_: ok, then we have a regression, can you open a bug about that? [11:15] yep [11:15] ogra_: that is then a blocker for 2.25 [11:17] pstolowski, Chipaca: https://github.com/snapcore/snapd/pull/3283 [11:17] PR snapd#3283: overlord/hookstate: remove unused Context.timeout [11:17] PR snapd#3283 opened: overlord/hookstate: remove unused Context.timeout [11:17] yuss [11:18] morphis, well ... [11:18] ogra@pi3:~$ snap version [11:18] snap 2.25 [11:18] snapd 2.25 [11:18] series 16 [11:18] kernel 4.4.0-1051-raspi2 [11:18] Chipaca: you need to install tizonia.conf as posted in the forum topic [11:18] juanrubio: done that [11:18] morphis, i'd rather say it blocks 2.26 :) [11:18] juanrubio: until I did, all tizonia did was complain about that missing file [11:19] Chipaca: ok, then could you please check that you have this in the config file? 'component-paths = /var/lib/snapd/snap/tizonia/x1/lib/tizonia0-plugins12;' [11:19] uhhh [11:19] PR snapcraft#1302 opened: lxd: Mount project folder via sshfs in case of a remote [11:19] juanrubio: that's not right [11:19] aha? [11:20] juanrubio: that's the path from "outside" the snap [11:20] juanrubio: (and only when that is where the snaps are installed) [11:20] in ubuntu, snaps are installed in /snap/ [11:20] and not /var/lib/snapd/snap [11:20] Chipaca: looks like it went fine http://paste.ubuntu.com/24542381/ running the test now [11:20] juanrubio: but in any case, inside the snap, it's always /snap/ [11:20] Chipaca: I see [11:20] editing the file now [11:21] ogra_: ah right, getting confused ... [11:21] juanrubio: also, you have a /home/joni in that file [11:21] Chipaca: with that path, tizonia can locate the plugins (decoders, sources, renderers etc) and the proceed to do some playback [11:21] juanrubio: do you know if tizonia supports using environment variables in its config? [11:22] juanrubio: what is the 'resource manager database'? [11:23] Bug #1689536 [11:23] Bug #1689536: snapd 2.25+201705082317.git.3ff5a56~ubuntu16.04.1 breaks seccomp handling [11:23] jdstrand, morphis ^^^ [11:24] Chipaca: uhm.. let me see, it is a db used to figure out what plugins can be accessed by tizonia in some cases, but not relevant here [11:24] yeah, noticed it's got enabled=false [11:24] correct [11:24] juanrubio: from a quick check it looks like tizonia doesn't support using environment variables in its config (but i haven't read the code to make sure) [11:24] juanrubio: so you're going to have to generate the file (e.g. from a wrapper) [11:25] Chipaca: that is correct, env vars are not supported right now [11:25] juanrubio: but, on ubuntu, when not installed with devmode, you get [11:25] juanrubio: [196128.136517] audit: type=1400 audit(1494329031.902:417): apparmor="DENIED" operation="connect" profile="snap.tizonia.tizonia" pid=22296 comm="audio_renderer" family="unix" sock_type="stream" protocol=0 requested_mask="send receive connect" denied_mask="send connect" addr=none peer_addr="@/tmp/.X11-unix/X0" peer="unconfined" [11:26] why is a music app trying to talk to X? [11:26] (a music app that doesn't ask for the x11 interface, as well) [11:27] Chipaca: it may talk to X to get the pulseaudio auth cookie [11:27] Chipaca: (beacause) [11:27] Chipaca: in my Fedora VM, I've got to the point where I can load the spotify plugin, and the pulseaudio plugin. Spotify works but audio rendering fails with pulseaudio access denied error [11:27] Chipaca: ah [11:27] juanrubio: which desktop environment are you using? [11:27] I think that the pulse cookie is in /tmp [11:27] awesome wm [11:27] morphis: we may need [11:27] juanrubio: here you go [11:28] this is why it doesn't work, [11:28] morphis: we may need someting quite like xauthority support for puslseaudio cookie [11:28] zyga: are you saying it would work on Gnome desktop? [11:28] juanrubio: yes [11:28] juanrubio: because it depends on how you initialize pulseaudio in the system [11:28] juanrubio: and some peculiar ways snapd works [11:29] zyga: let me log into gnome [11:30] zyga: heyy... it works! [11:30] really nice! [11:31] juanrubio: yes, it's really annoying as well but at least we know what we're standing on :) [11:31] zyga: so, is there anything I can do to workaround the cookie issue? [11:31] juanrubio: and it's a bug in snapd [11:31] juanrubio: just one that doesn't affect the default configuration for many [11:31] juanrubio: so we didn't bump into it yet [11:32] juanrubio: but it's just a bug, it'll get fixed [11:32] juanrubio: no, it's nothing you can fix [11:32] juanrubio: (in a snap) [11:32] zyga: is there a launchpad issue? [11:32] juanrubio: it's something that needs to be fixed in snapd/snap-confine pair [11:32] juanrubio: I don't think so, please file a bug about it [11:32] morphis: can you look at that bug please? [11:32] zyga: really? [11:32] morphis: ? [11:32] why are those things in /tmp .. [11:33] morphis: there are different ways to auth with pulse [11:33] Chipaca: yes, the snap-info test is passing now [11:33] morphis: ^ [11:33] morphis: and I think on ubuntu we use the cookie attached to the root window [11:33] zyga: I know but /run exists to store those things [11:33] morphis: but then if you for whatever reason cannot get that cookie [11:33] morphis: auth looks for the next thing [11:33] morphis: and I think that's somewhere inaccesislbe, maybe run but maybe not (perhaps $HOME somewhere) [11:34] morphis: I don't know [11:34] morphis: the x11 socket is in /tmp, not in /run, for instance [11:34] yeah, I know what pulse does pretty well, caused my quite some headache to get that right for U-Core and the pulseaudio snap [11:34] and the xim socket [11:34] Chipaca: however this is superconfusing, from the test-snapd-tools page in the store i can see that rev 6 is released to beta, candidate and stable [11:34] oh drat [11:34] my wife is killing me with irresistible smell downstairs :) [11:34] * zyga runs, bbl :) [11:35] zyga: sometimes i wish i had your problems [11:35] :-) [11:36] Chipaca: I don't mind creating the defect, although you might be able to record the details more accurately than me... [11:37] zyga, Chipaca: how are we sharing the X11 socket from /tmp with the snaps today? [11:39] zyga: normal location of the cookie on a desktop is $HOME/.config/pulse/cookie [11:40] morphis: I think all we do is copy .Xauthority to /run/user/etc [11:40] Chipaca: right, that is what I added recently [11:40] ah :-) [11:41] however for the pulse cookie that gets harder [11:42] juanrubio: can you file a bug about this on https://bugs.launchpad.net/snapd/+filebug , I will take it from there [11:44] morphis: filed it here -> https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1689539 [11:44] Bug #1689539: Pulseaudio "Access Denied" on Fedora 25 with Awesome WM as desktop environment [11:44] juanrubio: thanks [11:45] thanks guys, that was very helpful... my wife also has some smells dowstairs! [11:55] morphis, erm ... isnt the pulse cookie in $HOME usually ? [11:56] ogra_: yes [11:56] .config/pulse/cookie [11:56] or its set via X11 [11:56] right [11:56] and the pulse interface should give you access to ~/.config/pulse afaik [11:57] yeah ... owner @{HOME}/.config/pulse/cookie rk, [11:58] ogra_: right, but in this case it seems to be different, will look into that [12:02] morphis: ogra_: some apps may just go to the x11 root window by default [12:02] dunno [12:03] well, i see libboost in the error logs [12:04] and looking at the snapcraft.yaml it seems to pull in a good bunnch of other stuff https://github.com/tizonia/tizonia-openmax-il/blob/master/tools/snapcraft.yaml [12:04] dbus-x11 [12:04] there you go .... [12:05] that pulls in athe whole x11 stack through deps [12:05] (at least the libs) [12:06] and the app uses mpris as well [12:07] so it is actually deeply hooked into X11 and should use the x11 interface [12:14] mpris and dbus probably means it needs the unity7 one (for lack of a better match) [12:19] Chipaca: I always joke that my wife and I should switch roles; we'd get thinner instantly :) [12:20] morphis: no idea [12:20] morphis: there must be another way to connect [12:21] zyga: to connect to pulse you need the socket and the cookie [12:21] morphis: I must say that the modern desktop stack is pretty complex [12:21] and the cookie comes either from the fs or from X11 [12:21] it is [12:21] zyga: I will look into this problem [12:21] morphis: thank you! [12:21] morphis: I have no idea but I'd love to help if you figure out what needs doing [12:30] zyga: will ping you [12:30] ogra_: don't worry about line 525. it is referring to socketcall. that is something different [12:30] ogra_: I also answered in the bug [12:30] jdstrand, ok ... well, it dies on "socket" [12:31] ok [12:31] * jdstrand is wondering if there is an AF_ that this kernel has that isn't in generic [12:32] jdstrand: you mean custom address family? [12:32] if it was netlink, I would expect needing a 'network netlink' apparmor rule and I checked all those (I could've missed something) [12:32] jdstrand, well, the kernel doesnt seem to matter, it is the core that breaks it [12:32] zyga: yes [12:33] ogra_: I understand that [12:33] ah, k [12:33] ogra_: oh, for the strace. add 'socket' to the seccomp policy, then strace and show me all the socket calls [12:34] if you don't allow it in the policy there won't be anything to trace :) [12:34] do we have strace in the toolbelt snap ? [12:34] ogra_: no [12:34] bah [12:34] ogra_: it's just busybox [12:34] ogra_: but I could add it [12:35] yeah, that might make sense given how often we use it [12:35] ogra_: https://code.launchpad.net/~jdstrand/+git/debug-tools [12:39] ogra_: if using that snap. install it then run 'debug-tools' and it will tell you how to invoke [12:39] ogra_: though with strace, you literally can scp the binary and it will work [12:40] these days maybe not so well though [12:40] because of go being hard to strace [12:40] /path/to/strace -o ./trace -D -f -vv -e '!select [12:41] whoops [12:41] /path/to/strace -o ./trace -D -f -vv -e '!select' [12:41] that works pretty well [12:41] pstolowski: https://travis-ci.org/snapcore/snapd/builds/230295130#L3172 [12:42] jdstrand: i'll grep for this the next time i need it, thanks :-) [12:42] pstolowski: is that something we should worry about? [12:43] looking [12:43] pstolowski: hi, is snapd#3282 something we want soon/for 2.26? I answered something there [12:43] PR snapd#3282: hooks: default timeout [12:46] PR snapd#3283 closed: overlord/hookstate: remove unused Context.timeout [12:46] pedronis, I think it would be nice to have but not critical [12:46] Chipaca, econnreset test failure, is that what you mean? [12:47] pstolowski: yep [12:47] pstolowski: the link should've taken you to the line, but maybe it fails with the slow async loading of the log [12:47] Chipaca, it showed me the last summary line, yes, just wasn't sure [12:48] Chipaca, this test had some when Michael introduced it and I remember he fixed it... or so he thought ;) [12:48] s/some/some flakiness/ [12:58] pstolowski: so i should just re-run it? [12:59] pstolowski: actually rather than re-run it, as it seems i was wrong about device-hook, could you revert that edit? [13:00] Chipaca, re-run, yes, I'd say so. you mean you were wrong in the light of what pedronis said? [13:00] yeh [13:01] pstolowski: I have something for you to review [13:01] https://github.com/snapcore/snapd/pull/3284 [13:01] PR snapd#3284: interfaces: ensure that legacy interface methods are unused [13:01] PR snapd#3284 opened: interfaces: ensure that legacy interface methods are unused [13:01] Chipaca: ^^ you too as you have better go insight than I did [13:01] Chipaca: (I wanted to do this without introducing reflection in non-test code) [13:01] Chipaca: and without making definers public [13:01] fgimenez: do you got the CI fixed? [13:01] s/did/do/ ;) [13:01] zyga, stop slacking ... standup ;) [13:01] morphis: yes, should be working now [13:01] this is an old branch that I wanted to resurrect [13:01] ogra_: oh [13:01] sorry [13:02] zyga, fgimenez : stand up? [13:02] :) [13:02] coming [13:03] ogra_: let me try on my pi2 [13:03] fgimenez: thanks [13:07] jdstrand, not sure that works since you dont have a wlan device on it [13:07] fgimenez: re /dev/meminfo> it is allowed by the base abstraction, so, yes, it will be allowed everywhere [13:07] jdstrand, i'll collect the info after our team standup ... [13:07] I actually have a usb wlan I can plug in [13:08] the device isn't rebooting into 1876 though... [13:08] you could try (if that can work in ap mode) [13:08] 'Setup snap "core" (1876) security profiles (phase 2)' [13:08] * jdstrand taps fingers [13:08] ctrl-C if you dont want to wait for the auto reboot [13:08] and just sudo reboot [13:09] ah [13:09] (it wont return) [13:09] ok [13:09] that seems odd, but... unblocked [13:09] yes, is odd [13:10] jdstrand: great thanks, then do we need this rule https://github.com/snapcore/snapd/blob/master/interfaces/builtin/system_observe.go#L53 ? [13:10] fgimenez: no, we don't, ar it should at least be commented [13:10] or* [13:10] fgimenez: I took a note to do one of those things [13:11] jdstrand: ok thank you! :) [13:14] ogra_: fyi, not having the wlan is enough to trigger it [13:14] * jdstrand takes over [13:16] ah, nice [13:25] jdstrand: yes, that waiting command was the result of a quick bug fix, we know we have to improve the UX, it's on my todo list for soon [13:26] hmm [13:27] Chipaca: hmm? [13:27] niemeyer, fgimenez: i do still see the right reply from the store for snap info [13:27] so maybe we broke something? [13:27] Chipaca: we switched to use it afaik (2.24 was still using the hack) [13:27] this is wrt the "^" channels [13:27] so maybe we are using it wrong [13:27] oh wait let me make sure i'm using the right snap + snapd first :-) [13:28] and didn't notice [13:29] Chipaca: not sure, this is from 2.25 http://paste.ubuntu.com/24542694/ and this from 2.24.1 http://paste.ubuntu.com/24542709/ 2.25 doesn't show the closed channels [13:29] fgimenez: right, that means we've got a bug wrt closed channels [13:30] pedronis: nice! [13:30] fgimenez: which we've actually written into the tests [13:30] i wonder how this slipped past me :-) [13:30] * Chipaca knows how this could slip by [13:31] anyway, i'll push a branch for this [13:31] Chipaca: :) is it on snapd side then? Facu suggested to file it under https://bugs.launchpad.net/snapstore/ [13:31] Chipaca, reverted prepare-device timeout change [13:36] Chipaca, no, wait, which bug you mean? === om26er is now known as om26er1 === om26er1 is now known as om26er [13:37] fgimenez: yes, snapd side [13:37] Facu: what [13:38] Facu: we've got a bug in the way 'snap info' formats its output [13:38] Facu: this is not a launchpad bug [13:38] Chipaca, it should say (for the fgimenez example) "candidate: closed" (same for beta), right? [13:39] (at first I thought something else was broken, then I understood, and I just want you to NOT work twice) [13:42] Facu: candidate: ^ [13:42] socket(PF_NETLINK, SOCK_RAW, NETLINK_ROUTE) [13:42] ogra_: ^ [13:42] jdstrand: la que te recontra por las dudas [13:43] * Chipaca runs [13:43] Chipaca, ogra_: fyi, updated strace command for armhf (for grepping backscroll :) -- sudo ./strace -o /tmp/trace -D -f -vv -e '!_newselect,select,clock_gettime' -- /usr/bin/snap run --shell wifi-ap.automatic-setup [13:44] nice [13:44] on x86 you need to !select, but on armhf need to also not trace _newselect and clock_gettime otherwise it hangs [13:44] Chipaca, no, it should say "candidate: closed" and "beta: closed" (that's why CPI returns "null" for those channels, not "tracking" or something, see Mark's answer to my specific question about this in https://bugs.launchpad.net/snappy/+bug/1628640 [13:44] Bug #1628640: Add 'snap info' command showing publisher, channel map [13:44] pstolowski: i thought you'd just 'git reset --hard HEAD^ && git push --force', but yours is much more polite [13:45] Chipaca, yeah, I wasn't sure if github likes rewritten history [13:46] Facu: oh. Hm. There might be conflicting info here. [13:47] Chipaca, HO? [13:47] ogra_: the simple instructions are that automatic-setup needs to 'plugs: network-bind', but don't do that yet. let me review my notes on NETLINK_ROUTE, maybe I should just add it to network [13:47] Facu: nah, let me find things [13:47] ogra_: (which it already plugs) [13:47] Facu: either my memory plays tricks, or different people have different opinions on this and they need to settle it between themselves before we go off and implement it :-) [13:48] Chipaca, no problem! note we modeled server behaviour around that specific Mark's answer, we may need to change stuff if that changes [13:50] Chipaca: is the snap-info failure that's affecting PRs something you are on top of now? [13:50] ogra_: what you could do is add to /var/lib/snapd/seccomp/profiles/snap.wifi-ap.automatic-setup at the end: socket AF_NETLINK - NETLINK_ROUTE [13:50] zyga: the snap-info failure is resolved [13:50] ogra_: and let me know if that fixes it for you [13:51] zyga: it is no longer affecting PRs, since a while before the standup [13:51] ogra_: you'll have to service snap.wifi-ap.automatic-setup stop, then start [13:51] if I do that, I don't see the seccomp denial [13:52] but I don't have the wlan plugged in [13:52] * jdstrand goes to find it [13:53] Chipaca: aha [13:53] Chipaca: thanks! [13:54] pstolowski: why are you doing the get/set thing? https://github.com/snapcore/snapd/pull/3282/files#diff-3a26c6cd9ed243bb3b102f2d9589ba30R300 [13:54] PR snapd#3282: hooks: default timeout [13:54] pstolowski: specifically the set? [13:55] fgimenez: do we have any test-snapd- snap that is not in stable? [13:55] ogra_: huh, seems like it worked [13:55] zyga, ah, good catch, it's unneeded, copy/paste from other test [13:55] Chipaca: i think so, lately we have been releasing only to edge, let me check [13:55] ogra_: $ sudo wifi-ap.status [13:55] ap.active: true [13:55] Facu: in the "snap and snapcraft ux" doc there's a ^ in 'snapcraft status' (with a comment elsewhere of 'snap info' having the same layout as that) [13:56] ogra_: ok, so, yes, please test that, and I'll do a little research [13:56] Chipaca: test-snapd-openvswitch-consumer [13:56] Facu: where a closed channel that doesn't have a tracking thing shows "-", but closed-but-tracking is "^" [13:57] fgimenez: thanks! [13:57] zyga, updated [13:58] niemeyer: ping, if you're around [13:58] Chipaca: pongus [13:58] Chipaca, can you give me the doc url, please? [13:58] if you're not around, also ping, but ¯\_(ツ)_/¯ [13:59] :) [13:59] pstolowski: thank you [14:00] niemeyer: so i have conflicting specs for 'snap info', one from "make the channel map like 'snapcraft status'", which uses ^ or - to indicate closed channels (depending on whether they're tracking something or not), and a later comment in bug#1628640 that says they should show : closed [14:00] bug #1628640 [14:00] Bug #1628640: Add 'snap info' command showing publisher, channel map [14:00] ogra_: where is the source code for bin/client? [14:00] mup, you so confusing with "snapd#foo" and "bug #foo" :-) [14:00] Chipaca: I really wish I understood what you're trying to do. [14:01] * zyga goes to take the garbage out [14:01] morphis: maybe you know. where is the source code for bin/client in the wifi-ap snap? [14:01] jdstrand, lp:~snappy-hwe-team/snappy-hwe-snaps/+git/wifi-ap [14:01] ah, thanks [14:01] morphis: nm [14:01] niemeyer: personally I think the ^/- communicates more than just "closed" [14:02] and I think there is value in it being similar to 'snapcraft status' [14:02] niemeyer: (right now we don't do either, because of a bug, which is why i'm bothering you with this :) ) [14:03] Chipaca: Can we have a forum topic for this? [14:04] niemeyer: forum topics don't just grow on trees, you know [14:04] * Chipaca creates [14:04] Chipaca, niemeyer, I personally think that "^/-" is better, that's why I asked Mark specifically about that, and Mark said ": closed" (in that bug ↑) [14:04] Chipaca: I know.. that's why I bother people to create them :) [14:05] Facu: yeah, i'm suspecting that was a miscommunication / thinko [14:05] Facu: but let's settle it one way or t'other [14:05] That's exactly the sort of topic we should keep in the forum [14:05] Chipaca, for me it's settled, in the bug :) [14:05] Facu: either way you're good [14:05] Facu: bug is all my end [14:05] Let's please move the thread there so we have track record and rationale well kept [14:06] yep yep [14:06] on it [14:06] Chipaca, not really, the store should not answer "null" for those tracks, otherwise you'd need to start doing algorithms in the client to see if the channels is tracking or nothing is really released into it [14:06] Facu: Even if the bug isn't in your end, please join the thread with the details you have [14:06] niemeyer: in our defense all this wandering and wondering comes from before the forum was a thing :-) [14:07] Chipaca, the store should say "tracking" for you to present "^" and "null" for "-" [14:07] Facu: you and your nullable strings [14:07] Chipaca, this is what we're doing for snapcraft, anyway, so logic for channel interactions are defined in one place only (server side) [14:07] niemeyer, of course [14:07] now hush, i'm creating a forum topic :-p [14:07] Chipaca: Yeah, no worries or blame assigned.. just trying to get ourselves (certainly including myself) into the practice of organizing these exchanges [14:07] niemeyer: none implied either, just explaining how we got here [14:08] and yes, agreed [14:08] 👍 [14:08] Facu: now that you said it ↑ looks better than ^ [14:09] zyga, :) [14:12] jdstrand: question about https://github.com/snapcore/snapd/pull/3051 -- did you have a chance to look at the kernel to see if this interface can land; I'm asking to know if I should spend effort trying to get it to be mergable today or should I just look at another PR [14:12] PR snapd#3051: interfaces: add consoles interface [14:13] zyga: please look at another PR [14:13] zyga: this requires extensive investigation to make sure we dtrt [14:13] and it is behind a lot of other work atm [14:13] zyga: in fact, please add the blocked tag [14:14] * jdstrand commented in the pr already that it is in the queue, but behind other things [14:15] niemeyer: the more I write in the description of this, the more it feels like a 1-minute conversation with mark would resolve it :-) [14:15] anyhow, nearly done [14:17] Chipaca: Mark is the forum too.. [14:17] good point! [14:17] * Chipaca calls him out [14:22] hey everyone, how can I list the available config params for a snap? [14:22] Facu: niemeyer: https://forum.snapcraft.io/t/snap-info-channel-map-format/521 [14:22] Thanks! [14:23] bdx: I don't think you can [14:23] well, you can dig into the hook script usually [14:23] bdx: not sure if that's intentional or not-gotten-around-to-it yet [14:23] Chipaca, gracias [14:23] Facu: niemeyer: de nada caez's [14:23] (unless the package provides some binary config hook) [14:23] caez'ns* [14:24] zyga: ↑ vs ^ TBD (when we have something that knows whether it's writing to something UTF-8 friendly or not [14:24] jdstrand: OK [14:24] ) [14:24] will do [14:25] niemeyer: is it intentional that 'snap get' doesn't let you list all config keys? [14:25] PR snapcraft#1303 opened: kernel plugin: slightly improve the messaging of check_config() [14:25] niemeyer: e.g. 'snap get foo' dumping the whole config (vs 'snap get foo bar' just getting bar) [14:26] also why does snap get return json instead of yaml /o\ [14:26] so many questions :-) [14:34] Chipaca: that would be part of having a schema, at the moment we could only list the currently set ones [14:34] pedronis: https://forum.snapcraft.io/t/snap-get-config-dump/522/1 [14:34] :-) [14:34] that's not an official answer, you need niemeyer for that [14:35] pedronis: 🐔 [14:35] :-) [14:35] ogra@pi3:~$ snap get core system [14:35] error: snap "core" has no "system" configuration option [14:35] ogra@pi3:~$ snap set core system.power-key-action=halt [14:35] ogra@pi3:~$ snap get core system [14:35] { [14:35] "power-key-action": "halt" [14:35] } [14:36] the error is also definitely a lie [14:36] (i guess adding "set" at the end of the error text would be enough though) [14:39] pedronis: i think getting the set ones would be fine, at least until we have schemas (if a snap cares about this, all they need to do is set the defaults on first config call) [14:45] Chipaca: I feel like a broken record, but can you open a forum topic to discuss those points? [14:46] write an IRC macro! [14:46] niemeyer: way ahead of you dude [14:46] ;) [14:46] niemeyer: in fact you already replied to one of them [14:46] niemeyer, I tried to subscribe to snapcraft mailing list a few days ago, it said to be waiting for moderation from you. Can you please add me ? [14:46] Chipaca: So are you just trying to confuse me? :P [14:47] niemeyer: which makes me think you've been using the wrong flask to top up your mate [14:47] om26er: Heya [14:47] niemeyer: :) [14:47] om26er: Per the note at the front of the subscription page, we've integrated the mailing list into the forum [14:47] niemeyer: i said those things on irc, then created the topics; i think you just read them here later [14:47] Chipaca: You win [14:47] :) [14:48] * Chipaca takes the rest of the day off to celebrate [14:48] om26er: So all conversations previously in the list are now landing directly into the forum [14:48] om26er: You can still use it via email, if that's your preference [14:48] om26er: https://forum.snapcraft.io [14:48] niemeyer, interesting, will make finding old conversations easier. [14:49] om26er: Yeah, quite a few things get nicer [14:49] * om26er creates a new account. [14:50] and if you insist in email, you can actually use the email frontend to the forum ;) [14:54] niemeyer: http://pastebin.ubuntu.com/24543228/ OK? [14:54] zyga: ^ [14:54] or should i say zyga: 👆 [14:55] (i'm only kidding about the 👆; changed it back to ^ already) [14:55] LOL [14:55] are the release announcements also made on the forum now ? [14:55] @Chipaca To be honest, it does look more clear than ^ :) [14:55] niemeyer: No such command! [14:55] 1 month and it all looks different already ;) [14:55] mostly because 👆 is very hard to tell apart from 🖕 [14:55] @nothal You should ignore it then [14:55] niemeyer: No such command! [14:55] Chipaca: ROTFL [14:56] Chipaca: Depending on the situation the second one might be more appropriate [14:56] (but also, technically, because our output path does not know whether it's writing to something that likes UTF-8) [14:56] Chipaca: Like.. "No stable, 🖕" [14:56] oooh [14:56] so one instead of -, the other instead of ^ [14:56] makes sense [14:56] super easy to read :-p [14:58] http://pastebin.ubuntu.com/24543243/ [14:59] Chipaca: looking [15:00] Chipaca: haha [15:00] Chipaca: well.. [15:00] Chipaca: if it gracefully degrades in ASCII-only terminals then +1 [15:00] TODO on this front: [15:01] lol [15:01] 1. patch yaml to not revert to binary mode when it finds printable non-ascii characters [15:01] should we display 💩 instead of edge? [15:01] [...lots more points...] [15:01] 42. profit! [15:02] 43. detect UTF-8 console [15:02] Chipaca: that's spelled … [15:03] Facu: wrt having "tracking" in the json from the store, that wouldn't really help us at the moment [15:04] Facu: ¯\_(ツ)_/¯ [15:04] Facu: changing subjects: can a snap have a track with no open channels? [15:08] Chipaca: I believe that would be the default state when we first create a track [15:08] noise][: do you know of any snap with such a track in the store right now? [15:08] wanting to see what it looks like in 'snap info' [15:08] i don't think so but i could easily add a track for a test snap [15:09] noise][: otherwise, if it's easy for you to do, can you create an 'ingest' track in the http snap? [15:09] heh [15:09] yep [15:09] noise][: yes please [15:09] just to have a concrete example for discussion [15:12] Chipaca: created [15:12] Lunch! [15:13] noise][: looks like a track with no channels does not appear in channels_maps_list [15:13] noise][: (and that's fine) [15:14] yeah, there's no there there [15:14] ? [15:14] seems reasonable since nothing has ever been released into any channels in the track, there's no channel map or history [15:15] yup, as i say, that's fine [15:15] if it had appeared we would've had to decide whether to show it at all on the client, or just hide it [15:15] so huzzah, less stuff to decide :-) [15:16] \o/ [15:20] niemeyer: i wonder if someone from the snap team can update the open bot code to respond to !snap like it does to !info? The latter does a apt query (iirc) or it might look at the archive. It seems like we would like to be able to query the store just as easily from IRC for status. [15:20] niemeyer: let me find the linnk [15:20] https://launchpad.net/ubuntu-bots [15:21] niemeyer: we are starting to see some #ubuntu questions re: snaps and what's available periodically and i have to go run `snap find` myself :) [15:23] nacc, add a !snapforum command to that just prints "Please add this as a topic to the forum" ... [15:23] s/to/too/ [15:23] ogra_: heh :) [15:27] Bug #1689578 opened: Lack of shell completion installation support for command line apps [15:27] ogra_: yeah, i was going to take a stab at it myself, but was hoping someone with more snap api knowledge could do it if not :) [15:30] Chipaca: hey, tests/main/completion often fails on zesty [15:31] Chipaca: are you running zesty by any chance? [15:31] zyga: I am not. When it fails, can I see *how* it fails? [15:31] zyga: spread, or autosmackagetest? [15:31] Chipaca: just run it in qemu in a loop [15:31] Chipaca: I think autosmacketest [15:32] zyga: i'll give it a look later tonight [15:32] Chipaca: thanks! [15:32] meanwhile if you come across it in the wild, point me at the logs [15:32] Chipaca: yep [15:32] https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-zesty-snappy-dev-image/zesty/amd64/s/snapd/20170509_140006_d459a@/log.gz [15:32] times out [15:33] what's the trick these days with uploading something to the canonical account? [15:33] s/to the/with the/ [15:33] jdstrand: not sure :/ [15:33] zyga: there are some improvements in tests/completion/lib.exp0 that are not in tests/main/completion/lib.exp0 which might make it better, if it's just timeouts [15:33] jdstrand: the aha [15:33] I meant [15:33] jdstrand: i think you get given access per snap [15:33] Chipaca: aha [15:33] ah [15:34] jdstrand: which snap? [15:34] ogra_: do you have access to the classic snap? [15:34] jdstrand: FYI: I added something that will make sure we're not happily running with old unused interface APIs https://github.com/snapcore/snapd/pull/3284/files#diff-634051b94f9aae657f75a08bc250a139L113 [15:34] PR snapd#3284: interfaces: ensure that legacy interface methods are unused [15:34] Chipaca: classic [15:34] jdstrand: +1 welcome :) [15:34] jdstrand: if some of us, like me - being non canonical employee, knew the trick then we might be labelled haxx0rs ;-) [15:34] zyga: that test failure looks like an actual timeout from the store [15:34] yeah, I forgot it was per snap [15:34] Chipaca: another bump in your graph/ [15:35] I've lost count of how many times I've compiled my corebird snap trying to get a custom gtk build to work :-) [15:35] jdstrand: i've got access to classic [15:36] Chipaca: this will greatly improve the experience: http://paste.ubuntu.com/24543356/ [15:36] jdstrand: what's your lp email? [15:36] Chipaca: jamie at canonical.com [15:36] jdstrand: you've got mail [15:36] I'm happy to be added and to do the upload, etc [15:37] cool thanks [15:38] jdstrand: shouldn't that have an 'assumes' also? [15:39] * Chipaca doesn't know [15:47] Chipaca: I've seen comments that assumes isn't well tested. I can add it since I'm going to leave the one in beta alone [15:47] jdstrand: I don't really know :-) [15:47] jdstrand: ev seems to really like it though [15:47] the interfaces code has always handled it ok [15:47] * jdstrand shrugs [15:48] Bug #1689578 changed: Lack of shell completion installation support for command line apps [15:50] sergiusens: hiya [15:50] sergiusens: dunno if you saw about the 'completer' thing [15:50] oh that reminds me [15:50] Chipaca: yeah, I saw things here and there [15:50] jdstrand: i haven't tried to push something to the store that has a 'completer' key; will it break? [15:50] validation i mean === chihchun is now known as chihchun_afk [15:50] Chipaca: I wish you just create a PR against snapcraft though :-P [15:51] sergiusens: do you need anything from me to be able to do support it? [15:51] sergiusens: i know, but that'll not really be a good use of our time [15:51] I don't think so, if all we need is to point into a file inside `prime` (aka, the snap), then it is all good [15:51] when is this released? [15:52] sergiusens: it's in edge already [15:52] sergiusens: it'll move to beta tomorrow [15:52] Chipaca: when to us normals? [15:52] sergiusens: and it'll take a couple of weeks to reach the normies :-p [15:52] sergiusens: (modulo QA passing) [15:53] Chipaca: I can create a PR but not merge to see how it goes, but not going to make it in as we have bad precedent on introducing assumes and environment before it reached users [15:53] jdstrand: if the validator needs work for this, a bug task on #1689578 might be appropriate [15:53] Bug #1689578: Lack of shell completion installation support for command line apps [15:53] sergiusens: oh, totally [15:54] if `assumes` were functional, we could use that to proceed in an elegant way [15:54] I thought that assumes are functional [15:54] so did i :-) [15:55] sergiusens: in any case, I added a snapcraft bug task to the above bug, to track this [15:55] sergiusens: and i think not implementing it in snapcraft until it's fix-released in snapd is fine [15:56] pstolowski: can you review that branch (no-funny-interfaces) and once it lands, merge it into your PR [15:56] pstolowski: I'd like to detect the attribute-unaware methods this wayt [15:56] Chipaca: task added. classic updated. PR for classic submitted [15:56] zyga: Chipaca functional internally maybe, but where is the documentation and where are the allowed keywords? [15:56] jdstrand: huzzah [15:56] zyga, ok [15:57] Chipaca, just arrived, was AFK; may I help you with anything or it's solved? [15:57] now people's logs won't fill up when using classic :) [15:57] Facu: i don't think there's anything to do at this point in time [15:57] sergiusens: we have had a few but I think the idea is to use "assumes snapd25" or similar so that you can know you are using at least certain snapd version [15:58] jdstrand, i do [15:58] (i think) [15:58] ooh, getting closer. corebird now fires-up into a gui [15:58] I needed to hack the desktop-gtk3 definition [15:59] ogra_: ah, thanks, but nm, got it sorted [15:59] jdstrand, source is at https://github.com/snapcore/classic-snap [15:59] (in case you didnt know that yet) [15:59] * jdstrand ods [15:59] nods even [15:59] I already have a PR [16:00] cool [16:02] zyga: did you see that we've got kernel debs to try? https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1672819 [16:02] Bug #1672819: exec'ing a setuid binary from a threaded program sometimes fails to setuid [16:03] Chipaca: nope, not yet [16:07] zyga, +1 with one comment [16:07] pstolowski: thank you, looking [16:09] pstolowski: thanks! applied :) [16:27] zyga: great, now I need a map of snapd versions and features provided ;-) [16:30] sergiusens: {i: awesomeness*i for i in range(...)} [16:31] PR snapd#3285 opened: interfaces/network: workaround Go's need for NETLINK_ROUTE on ARM with 'net'. LP: #1689536 [16:33] sergiusens: just set to current ^_^ [16:35] PR snapd#3286 opened: cmd/snap: tweak info channels output [16:50] Chipaca: can you do a 2nd review on trivial, important: https://github.com/snapcore/snapd/pull/3285 [16:50] PR snapd#3285: interfaces/network: workaround Go's need for NETLINK_ROUTE on ARM with 'net'. LP: #1689536 === kay is now known as Guest11889 [16:52] zyga: ew [16:54] Chipaca: another timeout on tab completion https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-yakkety-snappy-dev-image/yakkety/amd64/s/snapd/20170509_083625_159d3@/log.gz [16:54] PR snapd#3281 closed: tests: add test for empty snap name on revert [16:55] niemeyer: can you please +1 https://github.com/snapcore/snapd/pull/3026 [16:55] PR snapd#3026: cmd/snap-confine: use defensive argument parser [16:55] zyga: and in the same place [16:56] May 09 08:26:32 autopkgtest snapd[611]: 2017/05/09 08:26:32.144671 daemon.go:176: DEBUG: uid=0;@ GET /v2/find?name=test%2A 5.073657694s 200 [16:56] zyga: the store takes 5 seconds to reply [16:56] zyga: expect's timeout is 5 seconds [16:56] ¯\_(ツ)_/¯ [16:57] Chipaca: thanks! [16:57] * Chipaca is wearing out his :shrug: today [16:57] Chipaca: I wish that was easier to grok :D [16:57] Chipaca: like if >5.0 printf "WTF is so slow?" [16:57] zyga: suggest bumping expect timeout to, say, 7 seconds :-) [16:57] zyga: +1 [16:57] niemeyer: thank you! [16:58] niemeyer: can you also look at https://github.com/snapcore/snapd/pull/3284 perhaps, I'd appreciate advice on how to make that better [16:58] PR snapd#3284: interfaces: ensure that legacy interface methods are unused [16:58] I wanted to avoid using reflection in non-test code [16:58] zyga: I really want to finish my expense reporting.. it's way to tempting not to [16:58] I could also make the tinye "definer" interfaces public [16:58] too [16:58] niemeyer: understood :) [16:59] we are two PRs away from being one-page-long [16:59] PR snapd#3026 closed: cmd/snap-confine: use defensive argument parser [16:59] * zyga goes after another interface [16:59] niemeyer: dunno if you saw but in the 'snap info' output i've had to change the - to a -- (for the 'channel is closed, and no less risky channel is open' case) [17:00] because, “stable: -” is not valid yaml [17:00] :-) [17:00] “stable: --” is, though, so i'm going with that [17:01] Ouch [17:01] :-) [17:01] Chipaca: How about an ndash? [17:01] Too confusing? [17:01] niemeyer: not confusing, but it'll look nasty in some cases [17:01] what is an ndash? [17:01] niemeyer: (non-utf8 aware cases, which will already look nasty for other reasons i'm sure) [17:02] zyga: a dash the width of an n [17:02] aah [17:02] zyga: as opposed to the mdash which is a dash the width of an m [17:02] This: foo: – [17:02] zyga: also an ndash is 1 en wide, and an mdash is one em wide [17:02] also an x is one en tall [17:02] I think I will hug my 1-cell sized monospace fonts : [17:03] ... and onf of those can be false in weird fonts [17:03] Chipaca: about our discussions earlier [17:03] Chipaca: double-cell glyphs! [17:03] Chipaca: back to regular broadcast [17:03] Chipaca: It does look nicer than -- though [17:03] Bug #1673763 changed: Help for 'version' [17:03] Bug #1680097 changed: core:network-bind plug is not connected after ubuntu-core -> core transition [17:03] zyga: hah. what about characters that are wide or thin depending [17:03] depending on what? who knows. depending. [17:03] (but people get upset) [17:04] * niemeyer wonders how unreasonable that is [17:04] niemeyer: it does [17:04] Chipaca: there is a block like that [17:04] Chipaca: they are so-called ambiguous characters [17:04] Chipaca: and may be half-width or regular size [17:04] i know [17:04] Chipaca: where half width is really "regular" and regular is "double cell" [17:04] anyway [17:04] insane stuff [17:05] niemeyer: so... should I just do it? [17:05] Chipaca: If we also used hands it'd be more clear that it's UTF-8 :) [17:05] * niemeyer teases [17:05] niemeyer: i was about to suggest using an actual arrow [17:06] actual arrows are really ... plentiful in unicode [17:06] Chipaca: Can you make an example for us to ponder on? [17:06] Chipaca: An arrow seems nicer than ^ as well, and not a big departure [17:08] niemeyer: ↑ ⇧ ⇪ ⇫ ⟰ ⤒ ⥉ ⥜ ⥣ ⥾ [17:08] First one seems nice [17:09] Although a bit too invisible [17:09] Chipaca: Do we have a larger one that looks like it [17:09] ? [17:09] Chipaca: I'm sure unicode must have hundreds of arrows that look almost exactly like that :) [17:09] yeah [17:09] that last one looks like a fountain [17:09] there's a lot more arrows pointing right than up though [17:09] http://www.fileformat.info/info/unicode/block/arrows/utf8test.htm [17:10] niemeyer: let me put it in code so you can see it in context [17:10] http://www.fileformat.info/info/unicode/block/arrows/list.htm [17:10] I see fields of ... arrows [17:10] I myself am quite fond of ⇡ [17:10] though ↯ is funky, just goes the wrong way [17:11] niemeyer: http://pastebin.ubuntu.com/24543901/ [17:11] niemeyer: looks like a tree [17:11] niemeyer: if anything it's _too_ big [17:12] Chipaca: I'd add a "legend" line at the bottom saying what those arrows and funkiness all mean [17:13] i think if we need that, we're doing it wrong [17:14] ah, ↑ is a lot less bold in the ubuntu mono font [17:14] bah. not in the ubuntu font. it's coming from DejaVu Sans [17:14] Chipaca: look at a map [17:14] Chipaca: maps have legend, they are not doing it wrong, it's just everything is unfamiliar at first [17:15] Chipaca: you may skip the legend later but it's useful to have one [17:15] * Chipaca likes mapmaking, but doesn't feel the terminal is an adequate place for it [17:15] anyway, i must EOD [17:15] I'm working on converting some systemd services to snaps. Is there a snappy way to edit variables that would be included in a environment file? [17:16] niemeyer: zyga: leave me a comment on the PR if you decide anything wrt this [17:16] * Chipaca off [17:16] cratliff: in the environment of everything else in the system? [17:19] that could probably work for me, but the way it is working right now is the services start with an EnvironmentFile parameter that sets a few variables used in the ROS launch file and other places. [17:22] cratliff: aha [17:23] cratliff: can you use the environment section in the snapcraft yaml? are those constants ore variables? [17:24] I must have missed the environment section details. They are used to reuse packages on multiple devices so at runtime they are constants. [17:24] cratliff: give that a try! [17:28] can you send me a link to a doc? I'm still having trouble finding it. [17:33] cratliff: I don't know where this is documented, perhaps sergiusens or kyrofa does [17:36] cratliff, zyga I'm not sure where it's documented either, but here's an example: https://github.com/snapcore/snapcraft/blob/master/demos/git/snap/snapcraft.yaml [17:36] niemeyer: can you please re-review https://github.com/snapcore/snapd/pull/2869; it has jdstrand's +1 and I added one missing bit and reviewed it myself [17:36] PR snapd#2869: interfaces/builtin: add online-accounts-service interface [17:36] kyrofa: thank you! [17:36] zyga: Still doing expenses [17:36] niemeyer: sure [17:38] Cool, thanks for pointing me in the right direction. [18:06] hey [18:09] * zyga_ breaks for evening walk [20:52] PR snapd#3285 closed: interfaces/network: workaround Go's need for NETLINK_ROUTE with 'net'. LP: #1689536 [20:59] PR snapd#3279 closed: tests: extend kernel-module-control interface test [21:00] PR snapd#3251 closed: packaging: cleanup how built-using is generated [21:14] zyga: you around? [21:14] zyga: have you done the thing about the expect timeout? === JanC is now known as Guest59098 === JanC_ is now known as JanC