=== razer_ is now known as razer2 [02:29] I tried to install Spotify with snap and it would not launch. Is this a known problem? [02:29] I ment to mention I'm using Debian. [02:30] I believe the .deb repository/package does work though. I'm run it in the past. [06:59] Mornings === pstolowski|afk is now known as pstolowski [07:54] Hi [07:55] what is the difference between a kernel snap and a gadget snap [08:11] moin moin [08:11] mvo: thank you for the reviews! [08:11] mvo: also, welcome back, I hope you had a lovely holiday [08:13] Chipaca: hey, good morning! my pleasure, hope my ramblings were useful. I had a lovely vacation indeed. also happy to be back, hacking is also a lot of fun :) [08:15] mvo: most of your ramblings are spot on [08:21] mvo: which of course means I'll make fun of you over the only one you got wrong [08:21] * Chipaca won't, tho [08:22] Chipaca: heh :) [08:22] Chipaca: thank you [08:22] Chipaca: (for not making (too much) fun of me) [08:23] mvo: and in your defence, it's not at all clear why the test function that tests fooPublisher would be called `TestGetEscapes` [08:24] * mvo nods [08:26] mvo: welcome back, I was a bit confused I thought you would still be on vacation [08:26] Chipaca: hi [08:27] pedronis: let's all go on vacation and avoid these confusions [08:27] pedronis: hey, thank you! maybe confused because Maciej is off for another week? [08:28] pedronis: at least currently I can't be away longer, we have some deadlines to make (18.04.1 and core18) :/ [08:29] mvo: what part about "dead" in deadline is unclear [08:29] mvo: I think for a while I thought you would be at the sprint, but then I saw it was not the case so I thought you were doing vacation, and maybe somebody said you were having 2 weeks [08:29] anyway [08:31] pedronis: yeah, we decided I should skip the sprint to focus on core18 during the week. but indeed … anyway :) [08:31] pedronis: happy to be back, did I miss anything exciting? [08:31] pstolowski: and hello to you as well :) [08:32] mvo: zyga's remapping branches went through some iterations but are not landed yet, I need to re-review the last version [08:33] pedronis: ok, I will check it out and can help with it if needed [08:33] pedronis: I'm getting myself an overview about the open PR right now to see where we stand [08:34] mvo: I proposed one about conflicts, the first pass of streamlining about those [08:35] pedronis: yeah, thanks for this! [08:36] I'm working on a follow up atm [08:37] seems we are bad at returning proper error kinds for anything that is not install/refresh etc [08:38] pedronis: yeah, that sounds likely :/ [08:40] pedronis: kinds are added when requested by clients though [08:40] Chipaca: ? [08:40] pedronis: error kinds are not always needed, and have been added by client request [08:40] client == snap and gnome software, usuallly [08:41] pedronis: OTOH we have an outstanding request from gnome software to add error kinds to async errors, which we haven't done yet [08:41] Chipaca: we don't return the various install related error kinds for install-path and try for example [08:42] Chipaca: and now it makes sense to return the conflict kind for anything that creates a change [08:42] pedronis: that's a good example: the text error is (or was) good enough for 'snap', and gnome software doesn't let you do those things [08:42] the kind is needed when the client needs to do something other than print the message to the user [08:42] Chipaca: are you saying I should not do this? [08:43] that logic doesn't make sense to me for a documented api [08:43] pedronis: I'm just as much a completionist (completist?) as you :-) i'm just explaining why things are as they stand [08:43] the documentation for `kind` is: Unique code for the error, to enable a snapd client to display appropriate behaviour (optional). [08:43] Chipaca: ok, but afaict we reached a point where that approach doesn't make sense [08:44] pedronis: while you're working on this, give async ones a think too :-) [08:44] I'm still not sure what that means [08:45] pedronis: if a change fails after it's being accepted, we don't return an error kind anywhere [08:45] s/being/been/ [08:45] well but definition we don't have kinds for those sort of errors [08:46] they are mostly fmt.Errorf anyway [08:46] pedronis: and gnome software would like something like kinds to know how to present the error to the user [08:46] at the same time, we don't have multi-errors for multi-snap actions, which might (or might not) be related [08:47] pedronis: sorry if this is very braindump-y [08:47] "tell pedronis all your gripes about errors" [08:47] dunno what triggered that :-) [08:53] PR snapd#4940 opened: WIP: added UDevMonitor for future hotplug support [08:57] PR snapd#5401 closed: asserts: add (optional) kernel-track to model assertion [08:59] mvo: I'm wondering what to do about fooPublisher and filler [09:00] filler should probably be called fillerPublisher or something [09:00] as it's the filler for (short)publisher [09:01] (or for long publisher, but long publisher isn't in a table where it matters, yet) [09:11] Chipaca: have you looked at how Maintenance is set in responses? I'm wondering if it's applicable looking at #5509 and warnings [09:12] PR #5509: daemon: move to use a constructor for Meta [09:13] hey guys, if I'm following kyrofa's blog on publishing snaps, would I be spamming the store/repo? Afterall, they are tests. Is there an alternative store for all test snaps? [09:13] pedronis: hmm, I hadn't noticed that, no [09:14] eraserpencil: there is, but it's a pain to set up at this point (including needing a recompile of snapd) [09:15] anyway to unregister or remove it from the store? [09:18] eraserpencil: not without manual admin intervention [09:18] eraserpencil: you could just make it private [09:21] kk thanks alot! [09:22] mvo: Chipaca: working on one of my PRs I noticed we have prepare code like this: https://github.com/snapcore/snapd/blob/master/tests/main/snap-run-symlink/task.yaml#L8 it sounds like we wouldn't need it anymore? [09:23] pedronis: correct [09:24] PR snapd#5509 closed: daemon: move to use a constructor for Meta [09:24] PR snapd#5510 closed: daemon, overlord/state: warnings pipeline [09:31] pedronis: yeah, I will clean that up in a sec, thanks for noticing [09:38] PR snapd#5511 opened: tests: remove unneeded setup code in snap-run-symlink [09:44] mvo: thx, snap-run-alias is the same [09:44] PR snapd#5512 opened: interfaces/builtin: create can-bus interface (2.34) [09:44] pedronis: thanks, will work on this too [09:45] Chipaca: fillerPUblisher sounds not bad, it clearly communicates what it does [09:46] Chipaca: glad that pattern can work warnings, I didn't look into details [09:46] *work for [09:46] Hey ho [09:47] pedronis: at some point I'm sure Response vs resp made sense, but these days it seems to be silly [09:47] pedronis: in any case, yes it works [09:47] pedronis: just need to find a name for the interface that provides both transmitMaintenance and addWarningsToMeta (or pay 2x interface lookup) [09:49] PR snapd#5512 closed: interfaces/builtin: create can-bus interface (2.34) [09:50] PR snapd#5513 opened: interfaces/builtin: create can-bus interface (2.34) [09:56] zyga: hello! how are you? all well? [10:00] PR snapd#5508 closed: cmd/snap: print unset license as "unset", instead of "unknown [10:06] pedronis: _very_ happy with how this has turned out [10:07] PR snapd#5514 opened: daemon, overlord/state: warnings pipeline [10:07] and i'm off to the gym for a bit [10:19] Mvo: all is good :-) [10:21] Once you catch up with everything I would love to know if we are green [10:34] zyga: green as in "the integration PR passes"? === BlueT_ is now known as BlueT === BlueT is now known as BlueTraveler === BlueTraveler is now known as BlueT_Lien === BlueT_Lien is now known as BlueT === pstolowski is now known as pstolowski|lunch [11:09] Bug #1781906 opened: Content provider interfaces introduced to snaps aren't correctly set up [12:00] mvo: this week the boys are on a different timetable, meaning they leave school at standup o'clock (2pm local), so i'm going to be missing (or at least very late) to the standups, probably [12:01] pedronis: when you can, take a look at what I ended up doing wrt maintenanceTransmitter: https://github.com/snapcore/snapd/pull/5514/files#diff-1a1f3e7ad9b1d7584e2d3e7d0c4c3db9 [12:02] PR #5514: daemon, overlord/state: warnings pipeline === pstolowski|lunch is now known as pstolowski| === pstolowski| is now known as pstolowski [12:06] Chipaca: trying to finish this branch and then I will look at various reviews [12:07] pedronis: no no no, I demand nothing less than you dropping everything and obeying my every whim [12:08] mvo: yes, that integration passes on master + the remapping branch + the implicit snapd patch [12:08] * zyga waves from the morning plenary [12:09] hmpf ... refreshing core to edge in a lvm instance gets me a hanging snapd shutdown process [12:09] ("A stop job is running for snappy daemon" (0:00/1:30)) [12:10] s/lvm/kvm/ [12:11] ogra_: journalctl -u snapd ? [12:13] Chipaca, to late, it has shut down now so the journal is gone [12:13] ogra_: you were holding it wrong anyway [12:13] but it should be easy to repro ofr anyone running kvm and switching from stable to edge [12:13] its a plain kvm with no additional snaps installed yet [12:14] and just "snap refresh --edge core" [12:14] (and reboot) [12:14] ogra_: on a core device image, or classic with core? [12:17] Chipaca, kvm UbuntuCore image ... latest from cdimage [12:17] ogra_: okey cokey [12:17] okie cokie? [12:17] wookie cookie? [12:18] sounds like a childrens TV show [12:18] Chipaca: ok, thanks for letting me know [12:18] zyga: sure, checking [12:39] mvo: hi there, is there a plan to add base information to `snap info `? [12:51] sergiusens: we added it , in verbose mode [12:55] ah, thanks [13:00] cjwatson: the launchpad builders seem a bit clogged with one recipe. Causing a small backlog. [13:28] * Chipaca returns [13:30] "the second part of the movie series" [13:30] pstolowski: I looked at my net-bpf stuff and I was indeed using unsafe.Pointer for the endianess detection, so I can offer no better approach unfortunately [13:31] mvo: fair enough, thanks for checking. i think i convinced upstream it's the reasonable approach [13:31] PR snapd#5515 opened: daemon: make sure most change generating handlers can produce errors with kinds [13:34] Chipaca: mvo: ^ based on #5502 [13:34] PR #5502: many: streamline the generic conflict check mechanisms [13:56] pedronis: thank you [14:17] mvo: #5491 bits about config seems wrong [14:17] PR #5491: overlord,daemon,cmd: re-map snap names around the edges of snapd [14:19] pedronis: yeah, I think so too [14:19] pedronis: makes we wonder if we should grow a configstate.Remap* helper ? [14:19] it's an approach, I don't have strong feelings either way, doing that, keeping the bits in snap [14:21] pedronis: "keeping the bits in snap" you mean having snap.Remap* for all the remappings? [14:21] mvo: no, I mean keeping the current helper and use them only for config [14:21] pedronis: aha, I see [14:22] pedronis: my gut-feeling is that moving them into configstate is slightly cleaner as it will make clear that the mapping is related to config (only). but no super strong opinion [14:23] mvo: I think it's fine as long as we know we don't need in the client, client importing *state is strange [14:23] /problematic [14:23] pedronis: +100 [14:25] pedronis: what I like about the PR from zyga is that it drops the client side (re)mapping and that its all done on the server side, it seems a nice property [14:26] mvo: yes, that was the agreement [14:27] * mvo nods [14:33] pstolowski: did you work on stopping snap connect to do something if the connection already exist? [14:33] popey: I was out, but I don't see a significant problem now. Recipe builds are usually pretty short but happen in batches, so if you're just looking at /builders then they often look more significant than they are, FWIW [14:33] popey: thanks for letting me know though [14:34] ah okay, thanks. [14:35] pedronis: no, that's not started yet [14:49] cjwatson: another for you. possible proxy error. "fatal: unable to access 'https://git.xiph.org/celt.git/': Received HTTP code 503 from proxy after CONNECT" in lp builder. [14:49] https://launchpadlibrarian.net/378715573/buildlog_snap_ubuntu_xenial_armhf_96015d27dc6daea80ecdd7c5bac64b6c-xenial_BUILDING.txt.gz [14:51] pedronis, mvo: I would suggest to solve the config remapping story within this PR so we cannot have fractured view of how things are mapped. I think that we should map core(state)-snapd(memory)-system(api) consistently and that this model is good. We just need to map the storage of the config using the existing function for this to work end to end [14:53] zyga: we don't store config on snapd, so not sure what that sentence mean [14:54] pedronis: that we should map the actual config store with the remapper for state so that it is "core.foo" as we want [14:54] pedronis: I want to avoid a special-cased mapper for config vs other cases [14:54] zyga: it doesn't make sense for the remapper to live ifacestate for that tough [14:54] what's the relation between config and interfaces [14:55] pedronis: ish, I agree but the decision on which remapper to use was is made there for convenience. I agree this could all live in snapmgr [14:55] also using a remapper for state in api is also strange [14:55] what do you mean about that? [14:55] s/about/by/ [14:56] I don't understand what you are proposing [14:56] it sounds very roundabout [14:56] it's unclear to me that what we do for config and interfaces is the same thing [14:56] it is just similar [14:57] I think we want a consistent view [14:57] if we have "system" thing that is mapped to something [14:57] we should not have two opinions as to what that is mapped to [14:57] ? [14:57] but we have two options [14:57] config: system -> core [14:57] interfaces: system -> core|snapd [14:58] popey: could you file that sort of thing as a bug against launchpad-buildd, please? [14:58] not really, interfaces are system->core (via snapd because it matters in memory) [14:58] will take some investigation [14:58] sure thing [14:58] I just want to avoid the possibility of having two pieces of code that remap and get out of sync [14:58] zyga: you are proposing to be very roundabout to avoid a very theoritecial problem [14:58] I'm not sure who that helps [14:58] not the reader of the code [14:59] anyway I'm happy to leave mvo pick a path forward [14:59] pedronis: I don't think that's the case, I'm saying something very simple: remap the config store from system to core using one of the proposed functions, that's all (and not have a yet-another function for config explicitly). [15:00] I also trust mvo to handle this gracefully [15:03] pedronis: reviewed your branch, looks great, just one change equest [15:03] *request [15:05] pstolowski: that doesn't work with the tests [15:06] unless we add some way to clear them but then it gets a bit too verbose vs the potential issue [15:07] and function cannot be compared in go [15:08] pedronis: right, i see [15:08] pedronis, mvo: I commented on the PR https://github.com/snapcore/snapd/pull/5491#discussion_r202718439 but I won't have time to explore this right now [15:08] PR #5491: overlord,daemon,cmd: re-map snap names around the edges of snapd [15:08] i just looked at task handlers, it's similiar [15:09] fair enough [15:10] +1 [15:11] zyga: that is roundabout :) but I let mvo think about it [15:11] thanks mvo :) [15:33] oh wow, git subtrees are awesome [15:34] git fetch go-udev; git merge -s subtree go-udev/master merges latest go-udev changes into our osutil/udev [15:35] magic [15:37] PR snapd#5516 opened: tests: add test that ensures we do not boot any system in degraded state [15:38] pstolowski: nice [15:40] PR snapd#5517 opened: osutil/udev: sync with upstream === pstolowski is now known as pstolowski|afk [16:04] PR snapd#5518 opened: systemd: fix snapd.apparmor.service.in dependencies [16:18] pedronis, zyga thanks, I am working on the degraded boot regression right now, once that is fixed I look at the mapping (should be soon :) [16:21] PR snapd#5519 opened: debian: do not ship snapd.apparmor.service on ubuntu [16:32] zyga, i playing with layouts here ... is it not possible to use single files (binaries) in layouts ? (i get a plethora of errors when trying to re-map a single binary from $SNAP/usr/bin) [16:33] ogra_: note zyga is sprinting [16:33] ogra_: telegram if urgent [16:33] yeah, i didnt expect an immediate answer [16:34] i assume he reads backlog :) [16:38] heh [16:39] /usr/lib/snapd/snap-device-helper: 18: /usr/lib/snapd/snap-device-helper: tr: not found [16:40] * ogra_ tries a symlink instead of binding all of /usr/bin ... [16:54] * Chipaca off [17:11] zyga, pedronis I pushed some ideas to 5491, I am happy to revert and push as its own PR if that is more convenient. please let me know how it feels, spread tests are running right now [17:12] fwiw, a review of 5410 would be great [17:15] mvo: ack, I'll check soon [17:16] mvo: what are the ConfigIfaceRemap... functions ? [17:17] mvo: I don't think snap is a good place for this , some of its logic is very snapd internal state dependent [17:31] PR # closed: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:31] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:34] PR # closed: snapcraft#1649, snapcraft#1720, snapcraft#1875, snapcraft#1905, snapcraft#2020, snapcraft#2040, snapcraft#2078, snapcraft#2105, snapcraft#2110, snapcraft#2121, snapcraft#2128, snapcraft#2135, snapcraft#2173, snapcraft#2176, snapcraft#2177, snapcraft#2180 [17:42] PR # opened: snapcraft#1649, snapcraft#1720, snapcraft#1875, snapcraft#1905, snapcraft#2020, snapcraft#2040, snapcraft#2078, snapcraft#2105, snapcraft#2110, snapcraft#2121, snapcraft#2128, snapcraft#2135, snapcraft#2173, snapcraft#2176, snapcraft#2177, snapcraft#2180 [17:42] PR # opened: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:42] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:43] PR # closed: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:43] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:46] PR # closed: snapcraft#1649, snapcraft#1720, snapcraft#1875, snapcraft#1905, snapcraft#2020, snapcraft#2040, snapcraft#2078, snapcraft#2105, snapcraft#2110, snapcraft#2121, snapcraft#2128, snapcraft#2135, snapcraft#2173, snapcraft#2176, snapcraft#2177, snapcraft#2180 [17:48] PR core18#41 opened: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" [17:48] PR core18#43 opened: Remove manpages and other documentation, cleanup empty doc dirs [17:48] PR core-build#11 opened: remove cruft from the writable-paths [17:48] PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build [17:48] PR core-build#22 opened: unit testing for sync_dir() [17:49] PR # opened: snapcraft#1649, snapcraft#1720, snapcraft#1875, snapcraft#1905, snapcraft#2020, snapcraft#2040, snapcraft#2078, snapcraft#2105, snapcraft#2110, snapcraft#2121, snapcraft#2128, snapcraft#2135, snapcraft#2173, snapcraft#2176, snapcraft#2177, snapcraft#2180 [17:49] PR # opened: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:49] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:49] PR core-build#22 closed: unit testing for sync_dir() [17:49] PR core-build#26 closed: move most of the customization into the core snap build [17:50] PR # closed: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:50] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:50] PR core-build#11 opened: remove cruft from the writable-paths [17:50] PR core-build#22 opened: unit testing for sync_dir() [17:51] PR core#38 closed: Add another pi-config option [17:51] PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build [17:51] PR # opened: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:51] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:52] PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build [17:53] Issue core18#4 closed: Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD)) [17:53] PR core18#43 closed: Remove manpages and other documentation, cleanup empty doc dirs [17:53] PR # closed: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:53] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:54] PR # opened: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:54] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:55] pedronis: snap not a good place> ack - any suggestions? overlord/mapper as its own package seems a bit extreme [17:55] PR core18#41 opened: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" [17:55] PR # closed: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:55] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [17:56] pedronis: the ConfigIface* functions are a copy/paste error, sorry for that [17:58] Issue core18#4 closed: Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD)) [17:58] PR core18#41 closed: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" [17:59] Issue core18#4 opened: Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD)) [17:59] PR core18#41 opened: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" [17:59] PR # opened: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [17:59] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [18:00] PR core18#43 closed: Remove manpages and other documentation, cleanup empty doc dirs [18:00] PR core-build#26 closed: move most of the customization into the core snap build [18:00] PR # closed: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [18:00] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [18:01] Issue core18#4 opened: Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD)) [18:01] PR core18#41 opened: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" [18:01] PR core18#43 opened: Remove manpages and other documentation, cleanup empty doc dirs [18:01] PR core-build#11 opened: remove cruft from the writable-paths [18:01] PR core-build#22 opened: unit testing for sync_dir() [18:01] PR core-build#26 opened: move most of the customization into the core snap build [18:01] PR # opened: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [18:01] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [18:02] PR core#38 closed: Add another pi-config option [18:02] PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build [18:02] PR # closed: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [18:02] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [18:03] PR core#38 opened: Add another pi-config option [18:03] PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build [18:03] PR # opened: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [18:03] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [18:06] Issue core18#4 closed: Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD)) [18:06] PR core18#41 closed: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" [18:06] PR core18#43 closed: Remove manpages and other documentation, cleanup empty doc dirs [18:06] PR # closed: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [18:06] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [18:07] Issue core18#4 opened: Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD)) [18:07] PR # opened: snapd#4416, snapd#4767, snapd#4940, snapd#4951, snapd#5122, snapd#5170, snapd#5187, snapd#5234, snapd#5271, snapd#5307, snapd#5318, snapd#5340, snapd#5341, snapd#5346, snapd#5395, snapd#5410, snapd#5413, snapd#5421, snapd#5433, snapd#5434, snapd#5446, snapd#5450, snapd#5451, [18:07] snapd#5452, snapd#5456, snapd#5469, snapd#5473, snapd#5474, snapd#5475, snapd#5477, snapd#5486, snapd#5491, snapd#5495, snapd#5497, snapd#5502, snapd#5503, snapd#5505, snapd#5506, snapd#5507, snapd#5511, snapd#5513, snapd#5514, snapd#5515, snapd#5516, snapd#5517, snapd#5518, snapd#5519 [18:07] It's getting 500s from GitHub.. will disable it [18:08] https://status.github.com/messages indicates github is experiencing issues and are fixing it [18:13] Done.. the relevant plugins have been disabled. Will try enabling them again later today. [18:14] mup: You ok for now? [18:14] niemeyer: I apologize, but I'm pretty strict about only responding to known commands. [18:14] Cool [18:14] o/ [18:29] mvo: I don't know, my feeling in the end is that putting some bits in ifacestate and some in configstate would have been fine. if we don't think so then I fear we indeed need mapper or to stick them in snapstate because that's the one that everything can import [19:04] mvo: mapper is not a great name tough, too vague [19:45] pedronis: ok, having bits in configstate and bits in ifstate is fine with me, I was mostly thinking it would be good to have a single place but probably overkill, I can (trivially) revert, my first commit was actually just doing this [19:45] pedronis: thanks for your thoughts [19:49] pedronis: I pushed a (much) simpler version of my previous PR to the same place, lets talk tomorrow :)