=== razer_ is now known as razer2 | ||
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:29 |
razer2 | I believe the .deb repository/package does work though. I'm run it in the past. | 02:30 |
pstolowski|afk | Mornings | 06:59 |
=== pstolowski|afk is now known as pstolowski | ||
eraserpencil | Hi | 07:54 |
eraserpencil | what is the difference between a kernel snap and a gadget snap | 07:55 |
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:11 |
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:13 |
Chipaca | mvo: most of your ramblings are spot on | 08:15 |
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:21 | |
mvo | Chipaca: heh :) | 08:22 |
mvo | Chipaca: thank you | 08:22 |
mvo | Chipaca: (for not making (too much) fun of me) | 08:22 |
Chipaca | mvo: and in your defence, it's not at all clear why the test function that tests fooPublisher would be called `TestGetEscapes` | 08:23 |
* mvo nods | 08:24 | |
pedronis | mvo: welcome back, I was a bit confused I thought you would still be on vacation | 08:26 |
pedronis | Chipaca: hi | 08:26 |
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:27 |
mvo | pedronis: at least currently I can't be away longer, we have some deadlines to make (18.04.1 and core18) :/ | 08:28 |
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:29 |
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:31 |
pedronis | mvo: zyga's remapping branches went through some iterations but are not landed yet, I need to re-review the last version | 08:32 |
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:33 |
pedronis | mvo: I proposed one about conflicts, the first pass of streamlining about those | 08:34 |
mvo | pedronis: yeah, thanks for this! | 08:35 |
pedronis | I'm working on a follow up atm | 08:36 |
pedronis | seems we are bad at returning proper error kinds for anything that is not install/refresh etc | 08:37 |
mvo | pedronis: yeah, that sounds likely :/ | 08:38 |
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:40 |
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:41 |
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:42 |
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:43 |
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:44 |
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:45 |
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:46 |
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:47 |
mup | PR snapd#4940 opened: WIP: added UDevMonitor for future hotplug support <Created by stolowski> <https://github.com/snapcore/snapd/pull/4940> | 08:53 |
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:57 |
Chipaca | mvo: I'm wondering what to do about fooPublisher and filler | 08:59 |
Chipaca | filler should probably be called fillerPublisher or something | 09:00 |
Chipaca | as it's the filler for (short)publisher | 09:00 |
Chipaca | (or for long publisher, but long publisher isn't in a table where it matters, yet) | 09:01 |
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:11 |
mup | PR #5509: daemon: move to use a constructor for Meta <Created by chipaca> <https://github.com/snapcore/snapd/pull/5509> | 09:12 |
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:13 |
Chipaca | eraserpencil: there is, but it's a pain to set up at this point (including needing a recompile of snapd) | 09:14 |
eraserpencil | anyway to unregister or remove it from the store? | 09:15 |
Chipaca | eraserpencil: not without manual admin intervention | 09:18 |
Chipaca | eraserpencil: you could just make it private | 09:18 |
eraserpencil | kk thanks alot! | 09:21 |
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:22 |
Chipaca | pedronis: correct | 09:23 |
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:24 |
mvo | pedronis: yeah, I will clean that up in a sec, thanks for noticing | 09:31 |
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:38 |
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:44 |
mvo | Chipaca: fillerPUblisher sounds not bad, it clearly communicates what it does | 09:45 |
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:46 |
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:47 |
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:49 |
mup | PR snapd#5513 opened: interfaces/builtin: create can-bus interface (2.34) <Created by mvo5> <https://github.com/snapcore/snapd/pull/5513> | 09:50 |
mvo | zyga: hello! how are you? all well? | 09:56 |
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:00 |
Chipaca | pedronis: _very_ happy with how this has turned out | 10:06 |
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:07 |
zyga | Mvo: all is good :-) | 10:19 |
zyga | Once you catch up with everything I would love to know if we are green | 10:21 |
mvo | zyga: green as in "the integration PR passes"? | 10:34 |
=== 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 | ||
mup | Bug #1781906 opened: Content provider interfaces introduced to snaps aren't correctly set up <Snappy:New> <https://launchpad.net/bugs/1781906> | 11:09 |
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:00 |
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:01 |
mup | PR #5514: daemon, overlord/state: warnings pipeline <Created by chipaca> <https://github.com/snapcore/snapd/pull/5514> | 12:02 |
=== pstolowski|lunch is now known as pstolowski| | ||
=== pstolowski| is now known as pstolowski | ||
pedronis | Chipaca: trying to finish this branch and then I will look at various reviews | 12:06 |
Chipaca | pedronis: no no no, I demand nothing less than you dropping everything and obeying my every whim | 12:07 |
zyga | mvo: yes, that integration passes on master + the remapping branch + the implicit snapd patch | 12:08 |
* zyga waves from the morning plenary | 12:08 | |
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:09 |
ogra_ | s/lvm/kvm/ | 12:10 |
Chipaca | ogra_: journalctl -u snapd ? | 12:11 |
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:13 |
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:14 |
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:17 |
ogra_ | sounds like a childrens TV show | 12:18 |
mvo | Chipaca: ok, thanks for letting me know | 12:18 |
mvo | zyga: sure, checking | 12:18 |
sergiusens | mvo: hi there, is there a plan to add base information to `snap info <snap-name>`? | 12:39 |
pedronis | sergiusens: we added it , in verbose mode | 12:51 |
sergiusens | ah, thanks | 12:55 |
popey | cjwatson: the launchpad builders seem a bit clogged with one recipe. Causing a small backlog. | 13:00 |
* Chipaca returns | 13:28 | |
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:30 |
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:31 |
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:34 |
Chipaca | pedronis: thank you | 13:56 |
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:17 |
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:19 |
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:21 |
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:22 |
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:23 |
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:25 |
pedronis | mvo: yes, that was the agreement | 14:26 |
* mvo nods | 14:27 | |
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:33 |
popey | ah okay, thanks. | 14:34 |
pstolowski | pedronis: no, that's not started yet | 14:35 |
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:49 |
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:51 |
pedronis | zyga: we don't store config on snapd, so not sure what that sentence mean | 14:53 |
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:54 |
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:55 |
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:56 |
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:57 |
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:58 |
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). | 14:59 |
zyga | I also trust mvo to handle this gracefully | 15:00 |
pstolowski | pedronis: reviewed your branch, looks great, just one change equest | 15:03 |
pstolowski | *request | 15:03 |
pedronis | pstolowski: that doesn't work with the tests | 15:05 |
pedronis | unless we add some way to clear them but then it gets a bit too verbose vs the potential issue | 15:06 |
pedronis | and function cannot be compared in go | 15:07 |
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:08 |
pstolowski | fair enough | 15:09 |
pstolowski | +1 | 15:10 |
pedronis | zyga: that is roundabout :) but I let mvo think about it | 15:11 |
zyga | thanks mvo :) | 15:11 |
pstolowski | oh wow, git subtrees are awesome | 15:33 |
pstolowski | git fetch go-udev; git merge -s subtree go-udev/master merges latest go-udev changes into our osutil/udev | 15:34 |
pstolowski | magic | 15:35 |
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:37 |
Chipaca | pstolowski: nice | 15:38 |
mup | PR snapd#5517 opened: osutil/udev: sync with upstream <Simple> <Created by stolowski> <https://github.com/snapcore/snapd/pull/5517> | 15:40 |
=== pstolowski is now known as pstolowski|afk | ||
mup | PR snapd#5518 opened: systemd: fix snapd.apparmor.service.in dependencies <Created by mvo5> <https://github.com/snapcore/snapd/pull/5518> | 16:04 |
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:18 |
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:21 |
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:32 |
Chipaca | ogra_: note zyga is sprinting | 16:33 |
Chipaca | ogra_: telegram if urgent | 16:33 |
ogra_ | yeah, i didnt expect an immediate answer | 16:33 |
ogra_ | i assume he reads backlog :) | 16:34 |
ogra_ | heh | 16:38 |
ogra_ | /usr/lib/snapd/snap-device-helper: 18: /usr/lib/snapd/snap-device-helper: tr: not found | 16:39 |
* ogra_ tries a symlink instead of binding all of /usr/bin ... | 16:40 | |
* Chipaca off | 16:54 | |
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:11 |
mvo | fwiw, a review of 5410 would be great | 17:12 |
zyga | mvo: ack, I'll check soon | 17:15 |
pedronis | mvo: what are the ConfigIfaceRemap... functions ? | 17:16 |
pedronis | mvo: I don't think snap is a good place for this , some of its logic is very snapd internal state dependent | 17:17 |
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:31 |
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:34 |
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:42 |
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:43 |
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:46 |
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:48 |
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:49 |
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:50 |
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:51 |
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:52 |
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:53 |
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:54 |
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:55 |
mvo | pedronis: the ConfigIface* functions are a copy/paste error, sorry for that | 17:56 |
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:58 |
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 | 17:59 |
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:00 |
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:01 |
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:02 |
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:03 |
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:06 |
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:07 |
zyga | https://status.github.com/messages indicates github is experiencing issues and are fixing it | 18:08 |
niemeyer | Done.. the relevant plugins have been disabled. Will try enabling them again later today. | 18:13 |
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:14 |
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 | 18:29 |
pedronis | mvo: mapper is not a great name tough, too vague | 19:04 |
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:45 |
mvo | pedronis: I pushed a (much) simpler version of my previous PR to the same place, lets talk tomorrow :) | 19:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!