[01:30] <mup> PR snapd#5385 opened: Update SELinux policy <Created by kevinanderson1> <https://github.com/snapcore/snapd/pull/5385>
[05:10] <mborzecki> morning
[05:43] <taaperotassu> Is there any way to see snapcraft stores most downloaded packages? Or see all the packages that are available?
[06:15] <zyga> Good morning
[06:17] <mborzecki> zyga: mvo: hey
[06:22] <mvo> hey mborzecki and zyga !
[06:25] <mup> PR snapd#5385 closed: Update SELinux policy <Created by kevinanderson1> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5385>
[06:33] <zyga> hey mvo
[07:11] <mup> PR core18#37 opened: static: add systemd environment generator to ensure PATH contains /snap/bin <Created by mvo5> <https://github.com/snapcore/core18/pull/37>
[07:15] <zyga> re
[07:15] <zyga> ok, kids are handled
[07:15] <zyga> mvo: re, so about those issues you saw last week
[07:15] <zyga> I'd like to look at the one with layouts later today
=== pstolowski|afk is now known as pstolowski
[07:15] <pstolowski> morning
[07:15] <zyga> I need to work on some leftovers from Friday
[07:15] <zyga> and I need to plan a rework of security profiles after the call with gustavo
[07:18] <abeato> mvo, hey, do you have any comments on https://forum.snapcraft.io/t/pulling-network-online-target-as-prerequisite-target-slows-down-starting-services/6063 ? Would be nice to have the opinion of some core team member there :)
[07:20] <mborzecki> pstolowski: hey
[07:25] <mvo> zyga: sure
[07:26] <mvo> abeato: let me look
[07:26] <mvo> zyga: a quick look at core18#37 would be great, I hope this unblocks some tests
[07:26] <mup> PR core18#37: static: add systemd environment generator to ensure PATH contains /snap/bin <Created by mvo5> <https://github.com/snapcore/core18/pull/37>
[07:27] <abeato> mvo, thanks
[07:30] <mup> PR core18#37 closed: static: add systemd environment generator to ensure PATH contains /snap/bin <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/37>
[07:31] <mborzecki> abeato: wondering, aside from longer times reported by system-analyze, does this dependency have any negative impact?
[07:33] <abeato> mborzecki, it does, it implies that starting a service can take 2 minutes more than it should. Think for instance in a system with eth0 and wlan0 configured but SSID not found: you get those additional 2 minutes
[07:33] <abeato> or eth0 not connected
[07:35] <mborzecki> abeato: right, but this is just snapd depending on the network (which it could not depend on because the failures are already handled in the code), but this should be fairly transparent for installed snaps and other services, i.e. you don't depend on snapd or network-online -> there's no penalty
[07:36] <mborzecki> so it's either a service depends on snaps (when it shouldn't) or it's part of graphical target (which iirc depends on mult-user, which in turn snapd is WantedBy)
[07:36] <abeato> mborzecki, nope, it is that the unit created by snapd when installing the snap includes the network-online.target dependency. The impact is on the service defined by the snap, not in snapd
[07:37] <mborzecki> abeato: aah ok, missed that! thanks for claryfing
[07:37] <abeato> np
[07:39] <mborzecki> mvo: didn't we have some proposals for expressing dependencies on external services/targets?
[07:39] <mvo> mborzecki: we have a request but no design for this so far :/
[07:40] <mborzecki> still, we don't have support for Requires at this point
[07:42] <mvo> yep
[08:00] <Chipaca> moin moin
[08:01] <pedronis> mvo: hi, did we add back code to wait for restarts for core/snapd ? we probably need to before 2.34
[08:04] <mvo> pedronis: no code for this yet
[08:04] <mvo> pedronis: I will look into it later today
[08:05] <pedronis> thx
[08:06] <mup> PR core18#38 opened: hooks: add libpam-systemd <Created by mvo5> <https://github.com/snapcore/core18/pull/38>
[08:10] <mup> PR core18#38 closed: hooks: add libpam-systemd <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/38>
[08:16] <mvo> sil2100: hey, good morning! I did (as an experiment) mergehttps://github.com/snapcore/core18/commit/a9c4ea6ff0732911a47fa7a1b035eb10c1e68cd4  and https://github.com/snapcore/core18/commit/28b28abd1968b75d3d201d57321a746b6912c7b1 - the background is that "su -l -c env user" on core18 has a PATH without /snap/bin and I suspect its the missing libpam-systemd. if you have more ideas they are very welcome, its not entirely clea
[08:16] <mvo> r to me if its that or not
[08:32] <mborzecki> anyoen feels like reviewing #5363 or #5366?
[08:32] <mup> PR #5363: snap: introduce the instance key field <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5363>
[08:32] <mup> PR #5366: snap: helper for validating snap instance names <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5366>
[08:39] <pedronis> mvo: is the plan to add "netcat" to core18 or not ?
[08:39] <mvo> pedronis: I pushed a pr to core18 with it, its tiny, I think its worth adding it
[08:39] <mvo> pedronis: given that its useful in tests and on its own (for admins for debugging)
[08:40] <mborzecki> thanks for the reviews guys!
[08:41] <Chipaca> you ask for reviews, you get reviews
[08:41] <Chipaca> 20 reviews per pr, approximately
[08:41] <mvo> sil2100: hm, libpam-systemd did not cut it, I will need to dig into this a bit more it seems, very strange
[08:42] <pedronis> mvo: yes, not against, just confused by the PR that remove tests that needs it
[08:43] <pedronis> s/remove/disable/
=== chihchun_afk is now known as chihchun
[08:44] <mvo> pedronis: yeah, my initial thinking was to disable those but then I realized just how many that are and I think now that the better approach is to just add it
[08:45] <mvo> pedronis: also it was to cut down noise from the tests, I ran the main suite against core18 over the weekend and tried to classify the failures
[08:45] <pedronis> it's ok
[08:45] <pedronis> also they are all about network interfaces that are quite fundamental
[08:46] <pedronis> and they might have classic vs core differences
[08:46] <pedronis> Chipaca: I added some questions to your snapshotstate PR,  not sure that's a way to unblock that doesn't involve doing the right thing though :/
[08:47] <pedronis> s/that's/there's/
[08:48] <Chipaca> pedronis: I just see one about "snapshot" -> "snapshotSetup"
[08:48] <pedronis> Chipaca: yes
[08:48] <Chipaca> ah the others are under some of gustavo's hidden ones
[08:49] <Chipaca> pedronis: I'll look at it later today (i probably meant IsActive instead of IsInstalled there)
[08:49] <Chipaca> need to do paperwork now
[08:50] <pedronis> Chipaca: yes, IsInstalled is just needed for the pattern I show
[08:50] <pedronis> (I think we have one place still in the code base that has a silly one)
[08:51] <pedronis> Chipaca: anyway I need to finish the improved error stuff before I can help unblock you
[08:51] <pedronis> Chipaca: sounds like snapshot will 2.35 though :(
[08:51] <pedronis> *will be
[08:52] <mvo> pedronis: thank you. I will update the PR once I finished digging into a strange PATH issue on core18
[08:53] <pedronis> mvo: which reminds me,  not urgent,  there's an extra ":"  in the entry  ': Interface hooks' in the snapd 2.34 roadmap  https://forum.snapcraft.io/t/the-snapd-roadmap/1973
[08:54] <mvo> pedronis: thank you, nice catch
=== chihchun is now known as chihchun_afk
[09:12] <abeato> mvo, a possible optimization: https://forum.snapcraft.io/t/are-systemctl-daemon-reload-calls-before-enabling-a-service-needed/6086
=== chihchun_afk is now known as chihchun
[09:22] <mup> PR snapd#5386 opened: snap: introduce a struct StoreChannel to represent store channels, and helpers to work with it <Created by pedronis> <https://github.com/snapcore/snapd/pull/5386>
[09:22] <pedronis> Chipaca: ^  what we discussed
[10:08] <mup> PR snapd#5363 closed: snap: introduce the instance key field <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5363>
[10:25]  * zyga struggles with apparmor profiles
[10:45] <mup> PR snapd#5387 opened: snap{/snaptest}: set instance key based on snap name <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5387>
[10:45] <mborzecki> another little piece of parallel installs is up for review
[10:45] <mborzecki> pedronis: ^^
[11:05] <mup> PR core18#39 opened: hooks: ensure /etc/login.defs PATHs contain /snap/bin <Created by mvo5> <https://github.com/snapcore/core18/pull/39>
[11:20]  * Son_Goku groans to life
[11:38] <mborzeck1> wtf, no power, obviously the electricity company allegedly has this info posted 'somewhere', bunch of lunatics
=== mborzeck1 is now known as mborzecki
[11:45] <Chipaca> mborzecki: did you electricity company not pay their bills
[11:49] <mborzecki> Chipaca: no it's just run by a bunch of a*holes, because it's easier to collect the bills rather than keep the infra in decent shape
[11:50] <Chipaca> mborzecki: in my current font, * looks just right for use in that word
[11:50] <mborzecki> Chipaca: what's the font?
[11:50] <Chipaca> mborzecki: the go one
[11:50] <Chipaca> https://blog.golang.org/go-fonts
[11:50] <Chipaca> reminds me of the old solaris console font
[11:51] <mborzecki> glad i pushed everything minutes before the power was gone
[12:00] <mup> PR snapd#5329 opened: DON'T REVIEW: tests: Adding debug information to know why econnreset is failing <Blocked> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5329>
[12:08]  * zyga runs another round of layout tests
[12:12]  * Chipaca ~> lunch
=== pstolowski is now known as pstolowski|lunch
[12:15] <mvo> a review of core18#39 would be great, this should unblock some more tests
[12:15] <mup> PR core18#39: hooks: ensure /etc/login.defs PATHs contain /snap/bin <Created by mvo5> <https://github.com/snapcore/core18/pull/39>
[12:23] <zyga> mvo: done
[12:23] <niemeyer> Heya
[12:23] <niemeyer> I've just did an update on Spread to fix the issue reported by Google.. please let me know if you see any hiccups
[12:23] <niemeyer> I've just done
=== pstolowski|lunch is now known as pstolowski
[12:45] <mup> PR snapd#5376 closed: tests: skip security-udev-input-subsystem without /dev/input/by-path <Created by mvo5> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5376>
[12:54] <Chipaca> niemeyer: how did google report an issue on spread?
[12:54] <niemeyer> Chipaca: I got an email
[12:55] <Chipaca> niemeyer: "you're the person responsible for 98% of all api calls"?
[12:55] <Chipaca> flashbacks of linode
[12:56] <niemeyer> Chipaca: Yeah, it did bring me memories, but this time it was more like "You are the person responsible for 98% of the API calls that start with a double dash (//)"
[12:56] <niemeyer> s/dash/slash
[12:56] <Chipaca> niemeyer: so causing a lot of redirects?
[12:56] <mborzeck1> let me give you this bag og 500 Internal Server Error
[12:56] <Chipaca> ah :)
=== mborzeck1 is now known as mborzecki
[12:57] <niemeyer> Or something.. we did get 500s
[12:57] <niemeyer> They didn't specify what the outcome was on their end, just politely asked whether we could get it fixed
[12:58] <Chipaca> and you didn't turn around and spam double-slashed endpoints with all the bandwidth you could muster?
[12:58] <Chipaca> you're too kind
[12:58] <mup> PR core18#39 closed: hooks: ensure /etc/login.defs PATHs contain /snap/bin <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/39>
[13:21] <mup> PR snapd#5388 opened: tests: fix tests when no keyboad input detected <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5388>
=== chihchun is now known as chihchun_afk
[13:26] <mup> PR snapd#5382 closed: tests: add halt-timeout to google backend <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5382>
[13:31] <zyga> cachio: which PR is that?
[13:35] <mvo> mborzecki: ha! I found *why* su -l is not working anymore in core18: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390 - anyway, I will work with foundations now to see if I can get it upstream
[13:35] <mborzecki> zyga: i see 4.14 kernel in for raspberrypi in yocto
[13:35] <mup> Bug #984390: $PATH is taken from login.defs not /etc/environment <patch> <shadow (Ubuntu):Triaged by canonical-foundations> <shadow (Ubuntu Precise):Triaged by canonical-foundations> <https://launchpad.net/bugs/984390>
[13:35] <mborzecki> mvo: thanks, intersting
[13:36] <zyga> mborzecki: current "production" is 4.9 for older models
[13:37] <mborzecki> heh 'production' for raspberry pi :P
[13:37] <mvo> mborzecki: its quite terrible, I debugged fixed this ~2y ago already but it did not land in the distro
[13:37]  * zyga had a full successful run on his layout patch, whee
[13:37] <zyga> mborzecki: I mean the one they ship with by default
[13:38] <mborzecki> wonder what's holding them back aside from gpu
[13:38] <zyga> probably the question is different
[13:38] <zyga> no need to go forward so why would they
[13:38] <mborzecki> (and totally crazy boot process, unless they changed it already)
[13:38] <zyga> the boot spec is part ot the hardware
[13:39] <zyga> mvo, niemeyer: one new interesting feature of systemd is the specification for new boot process
[13:39] <zyga> that's worth reading
[13:39] <mvo> zyga: do you have a link?
[13:39] <zyga> yes, one sec
[13:39] <mborzecki> mvo: https://lwn.net/Articles/758128/
[13:39] <mvo> mborzecki: ta
[13:39] <zyga> https://github.com/systemd/systemd/blob/master/doc/BOOT_LOADER_SPECIFICATION.md
[13:43] <mborzecki> zyga: NoNewPrivileges looks interesting
[13:44] <zyga> yes but it is unlikely to work soon
[13:44] <zyga> (as stated in the docs)
[13:45] <zyga> mborzecki, mvo: I personally really like the idea of that single-efi file that bundles linux, initrd and os-release file
[13:45] <zyga> that means you can boot a single "linux.efi" file
[13:45] <zyga> that's self contained
[13:45] <zyga> that's neat IMO
[13:46] <mvo> zyga: sounds a bit like a snap
[13:46] <zyga> mvo: yes, we perhaps could adopt it
[13:47] <zyga> it would mean we are an easier support target down the line
[13:47] <zyga> well, at least an idea
[13:50] <zyga> brb, need to resize my partitions
[13:50] <mborzecki> heh, the hangouts call pulled 1.2GB of data on my modem
[13:55] <zyga> revising from break=bottom ;)
[13:55] <zyga> resizing*
[13:56] <zyga> ok, quick lunch break
[13:59] <sitter> when I run snapcraft should it automatically detect common-id's or do I have to manually specify them in my snapcraft.yaml?
[14:07] <cachio> zyga, sorry for the delay, #5343
[14:07] <mup> PR #5343: tests: adding extra check to validate journalctl is showing current test data <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5343>
[14:10] <zyga> sitter: I don't know if it is implemented but there are specs for a system where appstream meta-data can be automatically ingested
[14:10] <zyga> sitter: I don't know if that's implemented yet though, perhaps kyrofa knows
[14:20] <mup> PR core18#40 opened: hooks: add FIXME for /etc/login.defs changes <Created by mvo5> <https://github.com/snapcore/core18/pull/40>
[14:22]  * zyga sincerely hopes for the weather to improve and some pressure to increase
[14:22] <zyga> today feels like mid winter when it's dark, gloomly and wet outside
[14:22] <zyga> not like first week of summer
[14:28] <mup> PR core18#40 closed: hooks: add FIXME for /etc/login.defs changes <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/40>
[14:33] <sitter> seems https://forum.snapcraft.io/t/adopt-info-from-other-metadata-sources/4370 has the information, assuming that is the latest and greatest way of doing it anyway
[14:33]  * zyga has IRL interrupt
=== chihchun_afk is now known as chihchun
[14:39] <mup> PR snapd#5380 closed: tests: blacklist more main tests for core18 <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/5380>
[14:42] <pedronis> pstolowski: disconnect branch seems to need now a merge from master (only test failing is spellecheck)
[14:43] <pstolowski> pedronis: thanks. it still needs some work re undo on disconnects
[14:56] <mup> PR snapcraft#2168 opened: build_providers: add ssh key managemet support to the qemu build prov… <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2168>
[15:12] <zyga> jdstrand: good day, are you back by any chance?
[15:13] <jdstrand> zyga: hi! yes. lots of catching up. what's up?
[15:13] <zyga> jdstrand: I will have some follow up from layouts but nothing major, just wanted to say hi :)
[15:13] <mup> PR snapd#5226 closed: data: add systemd user environment generator <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/5226>
[15:13] <mup> PR snapd#5389 opened: snap: account for parallel installs when dealing with broken snaps <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5389>
[15:13] <zyga> I can imagine you have a lot of catchup to do
[15:14] <jdstrand> zyga: :)
[15:18]  * zyga thinks about a 3rd coffee
[15:18] <zyga> rainly gloomly summer
[15:23]  * cachio lunch
[15:49] <mup> PR core18#35 closed: hooks: add netcat package <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/35>
=== Sir_Gallantmon is now known as Son_Goku
[16:14] <mup> PR snapd#5390 opened: data: add systemd environment configuration <Created by mvo5> <https://github.com/snapcore/snapd/pull/5390>
=== pstolowski is now known as pstolowski|afk
[16:37] <ogra_> zyga, i'm a but surprised about that snap version output in the forum, doesnt the snap command itself pull the info our of os-release and uname ? (i thought snap version does not require snapd to run to get at least the needed info from the os)
[16:38] <zyga> ogra_: no, it's all server side
[16:38] <ogra_> hmm
[16:38] <zyga> ogra_: (which makes sense if you think about it being, one day, able to talk to remote servers)
[16:38] <ogra_> (that makes debugging  a server outage indeed a bit tricky :) )
[16:39] <ogra_> true ... but perhaps it should then have a --no-remote flag so it can still collect the system info for debugging
[16:39] <zyga> well, it's not much of use if the server is down
[16:40] <zyga> and the set of collected things is tricky to compute locally really
[16:40] <ogra_> haha, systemd is always so helpful with its messages ...
[16:41] <ogra_> "he start-up result is RESULT"
[16:41] <ogra_> *the
[16:41] <ogra_> shouting doesnt help ! ... not even in logs :P
[16:42] <cwayne> It just wanted to make sure you heard it ogra_
[16:43] <ogra_> yeah, obviously :)
[16:43] <ogra_> thats like "command failed with: SUCCESS"
[16:45] <ogra_> zyga, i bet one of these php thingies he installed is some third party installer or tarball that mangles the install bad enough to break the world
[16:45] <ogra_> ot that "magento CMS platform" or whatnot
[16:46] <ogra_> *or
[16:48] <zyga> yeah, pretty horrible people make php
[16:48] <zyga> (as in make use of it)
[16:48] <zyga> also make the thing
[16:49] <ogra_> WOW ... that log now !
[16:49] <ogra_> curious that *anything* works on that install
[16:50] <mvo> hm, hm, can someone help with "error: File must begin with "/": %{_environmnentdir}/990-snapd.conf" - I get this from rpm packaging in https://github.com/snapcore/snapd/pull/5390 but the macro should be defined inhttps://github.com/systemd/systemd/blob/master/src/core/macros.systemd.in
[16:50] <mup> PR #5390: data: add systemd environment configuration <Created by mvo5> <https://github.com/snapcore/snapd/pull/5390>
[16:52] <Son_Goku> mvo, which target?
[16:52] <zyga> mvo: I was reviewing
[16:53] <mvo> Son_Goku: opensuse and fedora
[16:53] <mvo> Son_Goku: is this stuff just too new? git blame tells me it was added only one month ago?
[16:53] <mvo> zyga: ups, sorry
[16:53] <zyga> no worries :)
[16:53] <Son_Goku> while the functionality existed for a while, the macro didn't exist until systemd 239
[16:53] <Son_Goku> we just need a compat macro ;)
[16:53] <mvo> also "_environmnentdir" looks like a typo
[16:54] <mvo> Son_Goku: aha!
[16:54] <Son_Goku> and yes, someone screwed up
[16:54] <mvo> Son_Goku: I will push a fix
[16:54] <Son_Goku> %{?!_environmentdir: %global _environmentdir %{_prefix}/lib/environment.d}
[16:54] <mvo> Son_Goku: \o/
=== jkridner|pd is now known as jkridner
[16:57] <Son_Goku> mvo, zyga: https://github.com/systemd/systemd/pull/9417
[16:57] <mup> PR systemd/systemd#9417: rpm: Fix typo in %_environmentdir <Created by Conan-Kudo> <https://github.com/systemd/systemd/pull/9417>
[16:58] <mvo> Son_Goku: heh, was about to do this, thank you!
=== Sixth_Doctor is now known as Son_Goku
[17:14] <Son_Goku> mvo, I would have liked to see you send a PR for rpm things :P
[17:19] <pedronis> niemeyer: btw, if I remember you said you wanted to chime in on (there is related work going on):  https://forum.snapcraft.io/t/url-contact-fields-in-snap-metadata/3067/18
[17:28] <zyga> Thank you Son_Goku
[17:28] <zyga> my son has smalled a huge sliding window on his toe
[17:28] <zyga> some tears and blood later, I am back
[17:28] <Son_Goku> ...
[17:38] <cachio> mvo, hey, do you know why the econnreset test blocks the download once it is started?
[17:39] <cachio> mvo, it is any reason to don't block it before it starts?
[17:40] <cachio> because I could reproduce the error and found the download finishes after we compare the size of the partial file and before iptables is applied
[17:40] <cachio> mvo, this is the log https://paste.ubuntu.com/p/2yFqrXT7rd/
[17:40] <cachio> it has extra debug info
[17:47] <niemeyer> pedronis: Looking
[17:56] <niemeyer> pedronis: Provided some feedback there
[17:57] <niemeyer> zyga: Ouch
[17:57] <mvo> Son_Goku: hehe, exactly :)
[17:57] <mvo> cachio: iirc in the old days it would not retry on econnectionrefused
[17:58] <mvo> cachio: but the retry logic has changed quite a bit so maybe now thats fine?
[17:59] <Son_Goku> mvo, one day, I'll win you over to the awesome side :)
[18:00] <cachio> mvo, ok, I'll change the test and see what's the result
[18:00] <mvo> Son_Goku: :-D
[18:01] <mvo> cachio: cool, keep me updated
[18:02] <cachio> mvo, running
[18:03] <mup> PR core18#36 closed: hooks: set timezone to Etc/UTC <Created by mvo5> <Merged by sil2100> <https://github.com/snapcore/core18/pull/36>
[18:06] <sil2100> mvo: hmm, let me also experiment with that PAT
[18:07] <sil2100> *PATH missing /snap/bin
[18:08] <mvo> sil2100: I found the issue
[18:08] <mvo> sil2100: https://github.com/shadow-maint/shadow/pull/119
[18:08] <mup> PR shadow-maint/shadow#119: su.c: run pam_getenvlist() after setup_env <Created by mvo5> <https://github.com/shadow-maint/shadow/pull/119>
[18:08] <mvo> sil2100: this also links to a bug (from 2012 :)
[18:09] <mvo> sil2100: so we have a workaround for now, I will revert the systemd generator
[18:10] <mvo> sil2100: and once the fix for su.c is approved I/we can SRU the fix
[18:10] <mvo> sil2100: and undo the workaround
[18:14] <sil2100> wow
[18:16] <sil2100> ;)
[18:36]  * zyga moves to another room
[19:18] <mup> PR snapd#5391 opened: tests: simplify econnreset test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5391>
[19:34]  * cachio afk
[19:42] <mup> PR snapcraft#2169 opened: Rust plugin improvements <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2169>
[19:45] <mup> PR snapcraft#2158 closed: rust plugin: fix cargo builds and run tests <Created by tismith> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2158>
[19:45] <mup> PR snapcraft#2170 opened: Rust plugin env <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2170>
[19:48] <mup> PR snapcraft#2169 closed: Rust plugin improvements <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2169>
[20:19]  * zyga has a bad test day
[20:19] <zyga> or bad network day maybe
[20:22] <tomreyn> hi there. i just read https://github.com/canonical-websites/snapcraft.io/issues/651 and https://blog.ubuntu.com/2018/05/15/trust-and-security-in-the-snap-store and am wondinerg whether embedding (privacy impacting) trackers is currently (1) considered acceptable (b) tested for and (c) how it's being handled, if at all.
[20:23] <tomreyn> from my POV this is a major issue on android, and mostly unhandled / unsolved (or rather accepted) there. so i'm wondering whether what the situation with snaps is.
[20:26] <tomreyn> s/on android/on the google play store/
[21:46]  * zyga sees the same error mvo saw
[21:46] <zyga> that took a while, let's fix it
[23:57] <mup> PR snapcraft#2170 closed: rust plugin: fix cargo builds and run tests <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2170>