[02:29] <razer2> I tried to install Spotify with snap and it would not launch. Is this a known problem?
[02:29] <razer2> I ment to mention I'm using Debian.
[02:30] <razer2> I believe the .deb repository/package does work though. I'm run it in the past.
[06:59] <pstolowski|afk> Mornings
[07:54] <eraserpencil> Hi
[07:55] <eraserpencil> what is the difference between a kernel snap and a gadget snap
[08:11] <Chipaca> moin moin
[08:11] <Chipaca> mvo: thank you for the reviews!
[08:11] <Chipaca> mvo: also, welcome back, I hope you had a lovely holiday
[08:13] <mvo> 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] <Chipaca> mvo: most of your ramblings are spot on
[08:21] <Chipaca> 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] <mvo> Chipaca: heh :)
[08:22] <mvo> Chipaca: thank you
[08:22] <mvo> Chipaca: (for not making (too much) fun of me)
[08:23] <Chipaca> 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] <pedronis> mvo: welcome back,  I was a bit confused I thought you would still be on vacation
[08:26] <pedronis> Chipaca: hi
[08:27] <Chipaca> pedronis: let's all go on vacation and avoid these confusions
[08:27] <mvo> pedronis: hey, thank you! maybe confused because Maciej is off for another week?
[08:28] <mvo> pedronis: at least currently I can't be away longer, we have some deadlines to make (18.04.1 and core18) :/
[08:29] <Chipaca> mvo: what part about "dead" in deadline is unclear
[08:29] <pedronis> 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] <pedronis> anyway
[08:31] <mvo> pedronis: yeah, we decided I should skip the sprint to focus on core18 during the week. but indeed … anyway :)
[08:31] <mvo> pedronis: happy to be back, did I miss anything exciting?
[08:31] <mvo> pstolowski: and hello to you as well :)
[08:32] <pedronis> mvo: zyga's remapping branches went through some iterations but are not landed yet,  I need to re-review the last version
[08:33] <mvo> pedronis: ok, I will check it out and can help with it if needed
[08:33] <mvo> pedronis: I'm getting myself an overview about the open PR right now to see where we stand
[08:34] <pedronis> mvo: I proposed one about conflicts, the first pass of streamlining about those
[08:35] <mvo> pedronis: yeah, thanks for this!
[08:36] <pedronis> I'm working on a follow up atm
[08:37] <pedronis> seems we are bad at returning proper error kinds for anything that is not install/refresh etc
[08:38] <mvo> pedronis: yeah, that sounds likely :/
[08:40] <Chipaca> pedronis: kinds are added when requested by clients though
[08:40] <pedronis> Chipaca: ?
[08:40] <Chipaca> pedronis: error kinds are not always needed, and have been added by client request
[08:40] <Chipaca> client == snap and gnome software, usuallly
[08:41] <Chipaca> 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] <pedronis> Chipaca: we don't return the various install related error kinds for install-path and try for example
[08:42] <pedronis> Chipaca: and now it makes sense to return  the conflict kind for anything  that creates a change
[08:42] <Chipaca> 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] <Chipaca> the kind is needed when the client needs to do something other than print the message to the user
[08:42] <pedronis> Chipaca: are you saying I should not do this?
[08:43] <pedronis> that logic doesn't make sense to me for a documented api
[08:43] <Chipaca> pedronis: I'm just as much a completionist (completist?) as you :-) i'm just explaining why things are as they stand
[08:43] <Chipaca> the documentation for `kind` is: Unique code for the error, to enable a snapd client to display appropriate behaviour (optional).
[08:43] <pedronis> Chipaca: ok, but afaict we reached a point where that approach doesn't make sense
[08:44] <Chipaca> pedronis: while you're working on this, give async ones a think too :-)
[08:44] <pedronis> I'm still not sure what that means
[08:45] <Chipaca> pedronis: if a change fails after it's being accepted, we don't return an error kind anywhere
[08:45] <Chipaca> s/being/been/
[08:45] <pedronis> well but definition we don't have kinds for those sort of errors
[08:46] <pedronis> they are mostly fmt.Errorf anyway
[08:46] <Chipaca> pedronis: and gnome software would like something like kinds to know how to present the error to the user
[08:46] <Chipaca> at the same time, we don't have multi-errors for multi-snap actions, which might (or might not) be related
[08:47] <Chipaca> pedronis: sorry if this is very braindump-y
[08:47] <Chipaca> "tell pedronis all your gripes about errors"
[08:47] <Chipaca> dunno what triggered that :-)
[08:53] <mup> PR snapd#4940 opened: WIP: added UDevMonitor for future hotplug support <Created by stolowski> <https://github.com/snapcore/snapd/pull/4940>
[08:57] <mup> PR snapd#5401 closed: asserts: add (optional) kernel-track to model assertion <Core18> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5401>
[08:59] <Chipaca> mvo: I'm wondering what to do about fooPublisher and filler
[09:00] <Chipaca> filler should probably be called fillerPublisher or something
[09:00] <Chipaca> as it's the filler for (short)publisher
[09:01] <Chipaca> (or for long publisher, but long publisher isn't in a table where it matters, yet)
[09:11] <pedronis> 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] <mup> PR #5509: daemon: move to use a constructor for Meta <Created by chipaca> <https://github.com/snapcore/snapd/pull/5509>
[09:13] <eraserpencil> 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] <Chipaca> pedronis: hmm, I hadn't noticed that, no
[09:14] <Chipaca> eraserpencil: there is, but it's a pain to set up at this point (including needing a recompile of snapd)
[09:15] <eraserpencil> anyway to unregister or remove it from the store?
[09:18] <Chipaca> eraserpencil: not without manual admin intervention
[09:18] <Chipaca> eraserpencil: you  could just make it private
[09:21] <eraserpencil> kk thanks alot!
[09:22] <pedronis> 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] <Chipaca> pedronis: correct
[09:24] <mup> PR snapd#5509 closed: daemon: move to use a constructor for Meta <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/5509>
[09:24] <mup> PR snapd#5510 closed: daemon, overlord/state: warnings pipeline <Blocked> <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/5510>
[09:31] <mvo> pedronis: yeah, I will clean that up in a sec, thanks for noticing
[09:38] <mup> PR snapd#5511 opened: tests: remove unneeded setup code in snap-run-symlink <Simple> <Created by mvo5> <https://github.com/snapcore/snapd/pull/5511>
[09:44] <pedronis> mvo: thx, snap-run-alias is the same
[09:44] <mup> PR snapd#5512 opened: interfaces/builtin: create can-bus interface (2.34) <Created by mvo5> <https://github.com/snapcore/snapd/pull/5512>
[09:44] <mvo> pedronis: thanks, will work on this too
[09:45] <mvo> Chipaca: fillerPUblisher sounds not bad, it clearly communicates what it does
[09:46] <pedronis> Chipaca: glad that pattern can work warnings, I didn't look into details
[09:46] <pedronis> *work for
[09:46] <zyga> Hey ho
[09:47] <Chipaca> pedronis: at some point I'm sure Response vs resp made sense, but these days it seems to be silly
[09:47] <Chipaca> pedronis: in any case, yes it works
[09:47] <Chipaca> pedronis: just need to find a name for the interface that provides both transmitMaintenance and addWarningsToMeta (or pay 2x interface lookup)
[09:49] <mup> PR snapd#5512 closed: interfaces/builtin: create can-bus interface (2.34) <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/5512>
[09:50] <mup> PR snapd#5513 opened: interfaces/builtin: create can-bus interface (2.34) <Created by mvo5> <https://github.com/snapcore/snapd/pull/5513>
[09:56] <mvo> zyga: hello! how are you? all well?
[10:00] <mup> PR snapd#5508 closed: cmd/snap: print unset license as "unset", instead of "unknown <Simple> <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5508>
[10:06] <Chipaca> pedronis: _very_ happy with how this has turned out
[10:07] <mup> PR snapd#5514 opened: daemon, overlord/state: warnings pipeline <Created by chipaca> <https://github.com/snapcore/snapd/pull/5514>
[10:07] <Chipaca> and i'm off to the gym for a bit
[10:19] <zyga> Mvo: all is good :-)
[10:21] <zyga> Once you catch up with everything I would love to know if we are green
[10:34] <mvo> zyga: green as in "the integration PR passes"?
[11:09] <mup> Bug #1781906 opened: Content provider interfaces introduced to snaps aren't correctly set up <Snappy:New> <https://launchpad.net/bugs/1781906>
[12:00] <Chipaca> 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] <Chipaca> 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] <mup> PR #5514: daemon, overlord/state: warnings pipeline <Created by chipaca> <https://github.com/snapcore/snapd/pull/5514>
[12:06] <pedronis> Chipaca: trying to finish this branch and then I will look at various reviews
[12:07] <Chipaca> pedronis: no no no, I demand nothing less than you dropping everything and obeying my every whim
[12:08] <zyga> mvo: yes, that integration passes on master + the remapping branch + the implicit snapd patch
[12:08]  * zyga waves from the morning plenary
[12:09] <ogra_> hmpf ... refreshing core to edge in a lvm instance gets me a hanging snapd shutdown process
[12:09] <ogra_> ("A stop job is running for snappy daemon" (0:00/1:30))
[12:10] <ogra_> s/lvm/kvm/
[12:11] <Chipaca> ogra_: journalctl -u snapd ?
[12:13] <ogra_> Chipaca, to late, it has shut down now so the journal is gone
[12:13] <Chipaca> ogra_: you were holding it wrong anyway
[12:13] <ogra_> but it should be easy to repro ofr anyone running kvm and switching from stable to edge
[12:13] <ogra_> its a plain kvm with no additional snaps installed yet
[12:14] <ogra_> and just "snap refresh --edge core"
[12:14] <ogra_> (and reboot)
[12:14] <Chipaca> ogra_: on a core device image, or classic with core?
[12:17] <ogra_> Chipaca, kvm UbuntuCore image ... latest from cdimage
[12:17] <Chipaca> ogra_: okey cokey
[12:17] <Chipaca> okie cokie?
[12:17] <Chipaca> wookie cookie?
[12:18] <ogra_> sounds like a childrens TV show
[12:18] <mvo> Chipaca: ok, thanks for letting me know
[12:18] <mvo> zyga: sure, checking
[12:39] <sergiusens> mvo: hi there, is there a plan to add base information to `snap info <snap-name>`?
[12:51] <pedronis> sergiusens: we added it , in verbose mode
[12:55] <sergiusens> ah, thanks
[13:00] <popey> cjwatson: the launchpad builders seem a bit clogged with one recipe. Causing a small backlog.
[13:28]  * Chipaca returns
[13:30] <ogra_> "the second part of the movie series"
[13:30] <mvo> 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] <pstolowski> mvo: fair enough, thanks for checking. i think i convinced upstream it's the reasonable approach
[13:31] <mup> PR snapd#5515 opened: daemon:  make sure most change generating handlers can produce errors with kinds <Created by pedronis> <https://github.com/snapcore/snapd/pull/5515>
[13:34] <pedronis> Chipaca: mvo: ^   based on #5502
[13:34] <mup> PR #5502: many: streamline the generic conflict check mechanisms <Created by pedronis> <https://github.com/snapcore/snapd/pull/5502>
[13:56] <Chipaca> pedronis: thank you
[14:17] <pedronis> mvo: #5491  bits about config seems wrong
[14:17] <mup> PR #5491: overlord,daemon,cmd: re-map snap names around the edges of snapd <Core18> <Created by zyga> <https://github.com/snapcore/snapd/pull/5491>
[14:19] <mvo> pedronis: yeah, I think so too
[14:19] <mvo> pedronis: makes we wonder if we should grow a configstate.Remap* helper ?
[14:19] <pedronis> it's an approach, I don't have strong feelings either way, doing that, keeping the bits in snap
[14:21] <mvo> pedronis: "keeping the bits in snap" you mean having snap.Remap* for all the remappings?
[14:21] <pedronis> mvo: no, I mean keeping the current helper and use them only for config
[14:21] <mvo> pedronis: aha, I see
[14:22] <mvo> 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] <pedronis> 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] <pedronis>  /problematic
[14:23] <mvo> pedronis: +100
[14:25] <mvo> 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] <pedronis> mvo: yes, that was the agreement
[14:27]  * mvo nods
[14:33] <pedronis> pstolowski: did you work on stopping snap connect to do something  if the connection already exist?
[14:33] <cjwatson> 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] <cjwatson> popey: thanks for letting me know though
[14:34] <popey> ah okay, thanks.
[14:35] <pstolowski> pedronis: no, that's not started yet
[14:49] <popey> 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] <popey> https://launchpadlibrarian.net/378715573/buildlog_snap_ubuntu_xenial_armhf_96015d27dc6daea80ecdd7c5bac64b6c-xenial_BUILDING.txt.gz
[14:51] <zyga> 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] <pedronis> zyga: we don't store config on snapd, so not sure what that sentence mean
[14:54] <zyga> 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] <zyga> pedronis: I want to avoid a special-cased mapper for config vs other cases
[14:54] <pedronis> zyga: it doesn't make sense for the remapper to live ifacestate for that tough
[14:54] <pedronis> what's the relation between config and interfaces
[14:55] <zyga> 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] <pedronis> also using a remapper for state in api is also strange
[14:55] <zyga> what do you mean about that?
[14:55] <zyga> s/about/by/
[14:56] <pedronis> I don't understand what you are proposing
[14:56] <pedronis> it sounds very roundabout
[14:56] <pedronis> it's unclear to me that what we do for config and interfaces is the same thing
[14:56] <pedronis> it is just similar
[14:57] <zyga> I think we want a consistent view
[14:57] <zyga> if we have "system" thing that is mapped to something
[14:57] <zyga> we should not have two opinions as to what that is mapped to
[14:57] <pedronis> ?
[14:57] <pedronis> but we have two options
[14:57] <pedronis> config: system -> core
[14:57] <pedronis> interfaces:  system -> core|snapd
[14:58] <cjwatson> popey: could you file that sort of thing as a bug against launchpad-buildd, please?
[14:58] <zyga> not really, interfaces are system->core (via snapd because it matters in memory)
[14:58] <cjwatson> will take some investigation
[14:58] <popey> sure thing
[14:58] <zyga> I just want to avoid the possibility of having two pieces of code that remap and get out of sync
[14:58] <pedronis> zyga: you are proposing to be very roundabout to avoid a very theoritecial problem
[14:58] <pedronis> I'm not sure who that helps
[14:58] <pedronis> not the reader of the code
[14:59] <pedronis> anyway I'm happy to leave mvo pick a path forward
[14:59] <zyga> 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] <zyga> I also trust mvo to handle this gracefully
[15:03] <pstolowski> pedronis: reviewed your branch, looks great, just one change equest
[15:03] <pstolowski> *request
[15:05] <pedronis> pstolowski: that doesn't work with the tests
[15:06] <pedronis> unless we add some way to clear them but then it gets a bit too verbose vs the potential issue
[15:07] <pedronis> and function cannot be compared in go
[15:08] <pstolowski> pedronis: right, i see
[15:08] <zyga> 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] <mup> PR #5491: overlord,daemon,cmd: re-map snap names around the edges of snapd <Core18> <Created by zyga> <https://github.com/snapcore/snapd/pull/5491>
[15:08] <pstolowski> i just looked at task handlers, it's similiar
[15:09] <pstolowski> fair enough
[15:10] <pstolowski> +1
[15:11] <pedronis> zyga: that is roundabout :) but I let mvo think about it
[15:11] <zyga> thanks mvo :)
[15:33] <pstolowski> oh wow, git subtrees are awesome
[15:34] <pstolowski> git fetch go-udev; git merge -s subtree go-udev/master merges latest go-udev changes into our osutil/udev
[15:35] <pstolowski> magic
[15:37] <mup> PR snapd#5516 opened: tests: add test that ensures we do not boot any system in degraded state <Created by mvo5> <https://github.com/snapcore/snapd/pull/5516>
[15:38] <Chipaca> pstolowski: nice
[15:40] <mup> PR snapd#5517 opened: osutil/udev: sync with upstream <Simple> <Created by stolowski> <https://github.com/snapcore/snapd/pull/5517>
[16:04] <mup> PR snapd#5518 opened: systemd: fix snapd.apparmor.service.in dependencies <Created by mvo5> <https://github.com/snapcore/snapd/pull/5518>
[16:18] <mvo> 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] <mup> PR snapd#5519 opened: debian: do not ship snapd.apparmor.service on ubuntu <Created by mvo5> <https://github.com/snapcore/snapd/pull/5519>
[16:32] <ogra_> 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] <Chipaca> ogra_: note zyga is sprinting
[16:33] <Chipaca> ogra_: telegram if urgent
[16:33] <ogra_> yeah, i didnt expect an immediate answer
[16:34] <ogra_> i assume he reads backlog :)
[16:38] <ogra_> heh
[16:39] <ogra_> /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] <mvo> 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] <mvo> fwiw, a review of 5410 would be great
[17:15] <zyga> mvo: ack, I'll check soon
[17:16] <pedronis> mvo: what are the   ConfigIfaceRemap...  functions ?
[17:17] <pedronis> mvo: I don't think snap is a good place for this , some of its logic is very snapd internal state dependent
[17:31] <mup> 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] <mup> 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] <mup> 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] <mup> 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] <mup> 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] <mup> 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] <mup> 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] <mup> 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] <mup> 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] <mup> PR core18#41 opened: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" <Created by mvo5> <https://github.com/snapcore/core18/pull/41>
[17:48] <mup> PR core18#43 opened: Remove manpages and other documentation, cleanup empty doc dirs <Created by sil2100> <https://github.com/snapcore/core18/pull/43>
[17:48] <mup> PR core-build#11 opened: remove cruft from the writable-paths <Created by mvo5> <https://github.com/snapcore/core-build/pull/11>
[17:48] <mup> PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>
[17:48] <mup> PR core-build#22 opened: unit testing for sync_dir() <Created by mvo5> <https://github.com/snapcore/core-build/pull/22>
[17:49] <mup> 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] <mup> 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] <mup> 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] <mup> PR core-build#22 closed: unit testing for sync_dir() <Created by mvo5> <https://github.com/snapcore/core-build/pull/22>
[17:49] <mup> PR core-build#26 closed: move most of the customization into the core snap build <Created by mvo5> <https://github.com/snapcore/core-build/pull/26>
[17:50] <mup> 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] <mup> 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] <mup> PR core-build#11 opened: remove cruft from the writable-paths <Created by mvo5> <https://github.com/snapcore/core-build/pull/11>
[17:50] <mup> PR core-build#22 opened: unit testing for sync_dir() <Created by mvo5> <https://github.com/snapcore/core-build/pull/22>
[17:51] <mup> PR core#38 closed: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>
[17:51] <mup> PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>
[17:51] <mup> 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] <mup> 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] <mup> PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>
[17:53] <mup> Issue core18#4 closed:  Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD))  <Created by EndrII> <https://github.com/snapcore/core18/issue/4>
[17:53] <mup> PR core18#43 closed: Remove manpages and other documentation, cleanup empty doc dirs <Created by sil2100> <https://github.com/snapcore/core18/pull/43>
[17:53] <mup> 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] <mup> 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] <mup> 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] <mup> 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] <mvo> pedronis: snap not a good place> ack - any suggestions? overlord/mapper as its own package seems a bit extreme
[17:55] <mup> PR core18#41 opened: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" <Created by mvo5> <https://github.com/snapcore/core18/pull/41>
[17:55] <mup> 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] <mup> 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] <mvo> pedronis: the ConfigIface* functions are a copy/paste error, sorry for that
[17:58] <mup> Issue core18#4 closed:  Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD))  <Created by EndrII> <https://github.com/snapcore/core18/issue/4>
[17:58] <mup> PR core18#41 closed: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" <Created by mvo5> <https://github.com/snapcore/core18/pull/41>
[17:59] <mup> Issue core18#4 opened:  Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD))  <Created by EndrII> <https://github.com/snapcore/core18/issue/4>
[17:59] <mup> PR core18#41 opened: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" <Created by mvo5> <https://github.com/snapcore/core18/pull/41>
[17:59] <mup> 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] <mup> 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] <mup> PR core18#43 closed: Remove manpages and other documentation, cleanup empty doc dirs <Created by sil2100> <https://github.com/snapcore/core18/pull/43>
[18:00] <mup> PR core-build#26 closed: move most of the customization into the core snap build <Created by mvo5> <https://github.com/snapcore/core-build/pull/26>
[18:00] <mup> 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] <mup> 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] <mup> Issue core18#4 opened:  Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD))  <Created by EndrII> <https://github.com/snapcore/core18/issue/4>
[18:01] <mup> PR core18#41 opened: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" <Created by mvo5> <https://github.com/snapcore/core18/pull/41>
[18:01] <mup> PR core18#43 opened: Remove manpages and other documentation, cleanup empty doc dirs <Created by sil2100> <https://github.com/snapcore/core18/pull/43>
[18:01] <mup> PR core-build#11 opened: remove cruft from the writable-paths <Created by mvo5> <https://github.com/snapcore/core-build/pull/11>
[18:01] <mup> PR core-build#22 opened: unit testing for sync_dir() <Created by mvo5> <https://github.com/snapcore/core-build/pull/22>
[18:01] <mup> PR core-build#26 opened: move most of the customization into the core snap build <Created by mvo5> <https://github.com/snapcore/core-build/pull/26>
[18:01] <mup> 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] <mup> 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] <mup> PR core#38 closed: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>
[18:02] <mup> PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>
[18:02] <mup> 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] <mup> 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] <mup> PR core#38 opened: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>
[18:03] <mup> PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>
[18:03] <mup> 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] <mup> 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] <mup> Issue core18#4 closed:  Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD))  <Created by EndrII> <https://github.com/snapcore/core18/issue/4>
[18:06] <mup> PR core18#41 closed: Revert "static: add systemd environment generator to ensure PATH contains /snap/bin" <Created by mvo5> <https://github.com/snapcore/core18/pull/41>
[18:06] <mup> PR core18#43 closed: Remove manpages and other documentation, cleanup empty doc dirs <Created by sil2100> <https://github.com/snapcore/core18/pull/43>
[18:06] <mup> 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] <mup> 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] <mup> Issue core18#4 opened:  Crash qt gui aplication (build with core 18 on ubuntu 18.04(LXD))  <Created by EndrII> <https://github.com/snapcore/core18/issue/4>
[18:07] <mup> 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] <mup> 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] <niemeyer> It's getting 500s from GitHub.. will disable it
[18:08] <zyga> https://status.github.com/messages indicates github is experiencing issues and are fixing it
[18:13] <niemeyer> Done.. the relevant plugins have been disabled. Will try enabling them again later today.
[18:14] <niemeyer> mup: You ok for now?
[18:14] <mup> niemeyer: I apologize, but I'm pretty strict about only responding to known commands.
[18:14] <niemeyer> Cool
[18:14] <niemeyer> o/
[18:29] <pedronis> 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] <pedronis> mvo: mapper is not a great name tough, too vague
[19:45] <mvo> 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] <mvo> pedronis: thanks for your thoughts
[19:49] <mvo> pedronis: I pushed a (much) simpler version of my previous PR to the same place, lets talk tomorrow :)