[06:00] <zyga-ubuntu> good morning
[06:01] <zyga-ubuntu> hey mvo :)
[06:01] <mvo> pedronis: fwiw, the errors during build are "settle is not converging" so your theory about the time being too low seems to be correct (for the armhf build failure)
[06:03] <mvo> hey zyga-ubuntu
[06:05] <mup> PR snapd#3909 opened: daemon: remove unused installSnap var <Created by mvo5> <https://github.com/snapcore/snapd/pull/3909>
[06:08] <mup> PR snapd#3910 opened: snap-repair: fix test failure in TestRepairHitsTimeout <Created by mvo5> <https://github.com/snapcore/snapd/pull/3910>
[06:50] <mup> PR snapd#3911 opened: many: add logger.MockLogger() and use it in the tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/3911>
[07:09]  * mwhudson o/
[07:10] <zyga-ubuntu> hey mwhudson
[07:17]  * zyga-ubuntu has a sad face at ext4 corruption on his artful box
[07:19] <zyga-ubuntu> in other news seagate disks are still seagate
[07:22] <zyga-ubuntu> hey pstolowski
[07:23] <pstolowski> zyga-ubuntu, morning!
[07:46] <mup> PR snapd#3900 closed: snap-seccomp: manually resolve socket() call in tests <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3900>
[07:48] <mup> PR snapd#3891 closed: daemon: reach for Overlord.Loop less thanks to overlord.Mock <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3891>
[08:15]  * zyga-ubuntu -> quick walk
[08:42] <pstolowski> pedronis, zyga-ubuntu hey guys, do you have a moment for quick HO?
[08:43] <om26er> popey: Hi! Mind taking a look at https://dashboard.snapcraft.io/dev/snaps/8349/rev/1/ ?
[08:43] <om26er> it just uses the snapd-control interface
[08:44] <pedronis> pstolowski: yes
[08:49] <pstolowski> let's wait for zyga-ubuntu
[08:50] <zyga-ubuntu> pstolowski: yes
[08:51] <pstolowski> pedronis, zyga-ubuntu https://hangouts.google.com/call/IDjZPuXPNW1dCKRuMO8sAAkI
[08:54] <mup> PR core#59 opened: fix object path used by usr/bin/xdg-open in core <Created by mvo5> <https://github.com/snapcore/core/pull/59>
[09:01]  * Son_Goku gurgles awake
[09:02] <Son_Goku> zyga-ubuntu, can you *please* test the new snapd in Fedora?
[09:02] <Son_Goku> https://bodhi.fedoraproject.org/updates/FEDORA-2017-4546b9dd38 (F27)
[09:02] <Son_Goku> https://bodhi.fedoraproject.org/updates/FEDORA-2017-8ef8c9e6c2 (F26)
[09:03] <Son_Goku> https://bodhi.fedoraproject.org/updates/FEDORA-2017-d699df2619 (F25)
[09:07] <Son_Goku> zyga-ubuntu: I've got the docs update waiting for it: https://github.com/CanonicalLtd/snappy-docs/pull/103
[09:07] <mup> PR CanonicalLtd/snappy-docs#103: Update Fedora snapd to 2.27.6 <Created by Conan-Kudo> <https://github.com/CanonicalLtd/snappy-docs/pull/103>
[09:12] <zyga-ubuntu> Son_Goku: hey
[09:13] <Son_Goku> zyga-ubuntu: I've also got hughsie waiting for it because GNOME Software things
[09:13] <zyga-ubuntu> Son_Goku: yes, I'll test them immediately
[09:13] <zyga-ubuntu> Son_Goku: I also saw the golang import was changed in the repos
[09:13] <zyga-ubuntu> Son_Goku: so we might be able to test that and land your other branch
[09:13] <Son_Goku> it's in updates-testing, yes
[09:13] <Son_Goku> well, my current stuff is landed
[09:13] <Son_Goku> because the CI was going to fail going forward anyway :)
[09:21] <Chipaca> niemeyer: thank you for the review on #3866! good stuff
[09:21] <mup> PR #3866: many: implement fetching sections and package names periodically <Blocked> <Created by chipaca> <https://github.com/snapcore/snapd/pull/3866>
[09:31] <zyga-ubuntu> Son_Goku: updating F26 now, I'll test it when it is ready
[09:33] <mup> PR snapd#3912 opened: asserts: add empty values check in HeadersFromPrimaryKey <Created by pedronis> <https://github.com/snapcore/snapd/pull/3912>
[09:52] <ackk> mvo, hi, any chance your PR can be merged today? :)
[09:52] <zyga-ubuntu> Son_Goku: ok, testing F26 now
[09:57] <mvo> ackk: #3908 has one review already, if e.g. Chipaca does another one it can go in. there is an open question about a small detail raised by niemeyer but I think the overall direction is good so hopefully it lands today
[09:57] <mup> PR #3908: snap-seccomp, osutil: use osutil.AtomicFile in snap-seccomp <Created by mvo5> <https://github.com/snapcore/snapd/pull/3908>
[09:58] <ackk> nice, thanks
[10:07] <pedronis> Chipaca: hi, #3908 seems something you could review
[10:07] <mup> PR #3908: snap-seccomp, osutil: use osutil.AtomicFile in snap-seccomp <Created by mvo5> <https://github.com/snapcore/snapd/pull/3908>
[10:07] <Chipaca> pedronis: sorry, can't review things twice :-p
[10:07] <Chipaca> (i _just_ did it)
[10:09] <mup> PR snapd#3909 closed: daemon: remove unused installSnap var <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3909>
[10:09] <pedronis> mvo: are we sure in #3910 that 100ms is enough ?
[10:09] <mup> PR #3910: snap-repair: fix test failure in TestRepairHitsTimeout <Created by mvo5> <https://github.com/snapcore/snapd/pull/3910>
[10:10] <mvo> pedronis: no, this is just a guestimate, I figured 10x than before is hopefullyok
[10:10] <mvo> hopefully ok even
[10:15] <zyga-ubuntu> hmm, gnome shell crashes sometimes :/
[10:16] <mup> PR snapd#3912 closed: asserts: add empty values check in HeadersFromPrimaryKey <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3912>
[10:25]  * ogra_ gllares at https://github.com/snapcore/core/pull/59 and wonders why it works for him even without that change
[10:25] <mup> PR core#59: fix object path used by usr/bin/xdg-open in core <Created by mvo5> <https://github.com/snapcore/core/pull/59>
[10:28] <ogra_> mvo, seeing that, it shouldnt work at all for me, should it ?
[10:29]  * ogra_ is confused
[10:30] <mvo> ogra_: yes, it should not work if you have the 2.28 debs
[10:30] <mvo> ogra_: does it work with those?
[10:30] <ogra_> it wrks with the 2.27 debs and yoour test core from candidate
[10:30] <mvo> ogra_: the fallback (snapd-xdg-open) launcher will work
[10:31] <ogra_> ah !
[10:31] <ogra_> ok
[10:31] <mvo> ogra_: yeah, that is ok, then you are using the fallback
[10:31] <ogra_> thats clears my cooonfusion :)
[10:31] <mvo> ogra_: tests ftw :)
[10:31]  * mvo will write some more today
[10:31] <ogra_> :)
[10:37] <zyga-ubuntu> wrz 13 12:37:26 fyke gnome-shell[1591]: Failed to apply DRM plane transform 0: Brak dostępu
[10:37] <zyga-ubuntu> hmmm
[10:37] <zyga-ubuntu> Brak dostępu = no access/permission denied
[10:45] <zyga-ubuntu> is repowerd nuts?
[10:45] <zyga-ubuntu> wrz 13 12:44:21 fyke sudo[24161]: pam_unix(sudo:session): session opened for user root by (uid=0)
[10:45] <zyga-ubuntu> wrz 13 12:44:38 fyke repowerd[940]: LogindSessionTracker: change_seat_properties(/org/freedesktop/login1/seat/seat0), ActiveSession=(c1,/org/freedesktop/login1/session/c1)
[10:45] <zyga-ubuntu> wrz 13 12:44:38 fyke repowerd[940]: LogindSessionTracker: activate_session(c1)
[10:45] <zyga-ubuntu> wrz 13 12:44:38 fyke sudo[24161]: pam_unix(sudo:session): session closed for user root
[10:45] <zyga-ubuntu> wrz 13 12:44:38 fyke repowerd[940]: LogindSessionTracker: change_seat_properties(/org/freedesktop/login1/seat/seat0), ActiveSession=(13,/org/freedesktop/login1/session/_313)
[10:45] <ackk> uhm, I'm getting "AttributeError: /home/ack/virtualenv/snapcraft/bin/python: undefined symbol: archive_errno" in snapcraft master
[10:45] <zyga-ubuntu> wrz 13 12:44:38 fyke repowerd[940]: LogindSessionTracker: activate_session(13)
[10:45] <zyga-ubuntu> every time I close my root shell it does this
[10:57] <zyga-ubuntu> Son_Goku: send F26 feedback
[11:16] <mup> PR snapd#3910 closed: snap-repair: fix test failure in TestRepairHitsTimeout <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3910>
[11:18] <morphis_> mvo, popey: can you have a look at https://forum.snapcraft.io/t/auto-connecting-interfaces-for-easy-openvpn-snap/2029/3 and give +1 if you're fine? saw you're listed as store reviewer
[11:18]  * popey looks
[11:24] <zyga-ubuntu> mvo: on zesty with wayland I don't get gnome shell to find any snap desktop files
[11:25] <zyga-ubuntu> mvo: not sure if regression but annoying
[11:25] <zyga-ubuntu> mvo: /home/zyga/.local/share/flatpak/exports/share/:/var/lib/flatpak/exports/share/:/usr/local/share/:/usr/share/
[11:25] <zyga-ubuntu> this is XDG_DATA_DIRS
[11:25] <pedronis> zyga-ubuntu: is it this, or related?   https://forum.snapcraft.io/t/desktop-file-names/2096/2
[11:25] <ogra_> zyga-ubuntu, isnt the PR fr that still open ?
[11:25] <zyga-ubuntu> how come flatpak is there but snapd is not
[11:26] <ogra_> hmm https://github.com/snapcore/snapd/pull/3398 says merged
[11:26] <mup> PR #3398: env: set XDG_DATA_DIRS for wayland et.al <Created by sergiusens> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3398>
[11:26] <zyga-ubuntu> pedronis: also annoying but does not seem related
[11:27] <pedronis> ah, zesty though?  not artful?
[11:27] <pedronis> does it work on artful?
[11:27] <pedronis> do we even support walyand officially on zesty?
[11:28] <zyga-ubuntu> pedronis: yes
[11:28] <zyga-ubuntu> pedronis: it works but I'm on xorg there due to GPU
[11:28] <zyga-ubuntu> pedronis: so s/yes/maybe/
[11:28] <zyga-ubuntu> it's in the environment for sure but I'm running xorg, not wayland
[11:28] <ogra_> well, the above PR fixes it ... but i'm not sure it is anywhere else but master yet
[11:29] <zyga-ubuntu> ogra_: I'm running master
[11:29] <zyga-ubuntu> ogra_: maybe I need to tweak something
[11:29]  * zyga-ubuntu looks
[11:29] <zyga-ubuntu> I probably didn't install data files
[11:29] <ogra_> well, you should have a snapd.sh in /etc/profile.d
[11:30] <ogra_> (are you running the master *deb* ?)
[11:30] <zyga-ubuntu> I have apps-bin-path.sh
[11:30] <zyga-ubuntu> ogra_: no, that's the problem
[11:30] <ogra_> :)
[11:30] <ogra_> just create the .sh file from the PR
[11:30] <ogra_> (and re-login indeed)
[11:31] <zyga-ubuntu> ok, tweaked
[11:31] <zyga-ubuntu> yep
[11:31] <zyga-ubuntu> brb
[11:31] <zyga-ubuntu> my open terminals :'((
[11:31] <ogra_> yeah ... we should really have a "save session per terminal" feature :)
[11:31] <zyga-ubuntu> I used :mks in vim
[11:31] <zyga-ubuntu> but not other places
[11:31] <zyga-ubuntu> gnome used to have this but then nobody did it correctly so it was removed
[11:31] <ogra_> that still only gives you one bash histroy :)
[11:32] <zyga-ubuntu> (and ironically then apple did it)
[11:32] <zyga-ubuntu> so maybe gnome will do it now again
[11:32] <zyga-ubuntu> well
[11:32] <zyga-ubuntu> brb
[11:32] <ogra_> i want each term pop up where it closed ... with the same histry it had
[11:33] <zyga-ubuntu> well, that worked
[11:33] <ogra_> :)
[11:35] <ogra_> heh ... https://massdrop-s3.imgix.net/product-images/massdrop-x-oblotzky-sa-oblivion-custom-keycap-set/sa_oblivion_kit_04_git_modifiers_20170828131112.png?auto=format&fm=jpg&fit=crop&w=924&dpr=1
[11:35] <zyga-ubuntu> Chipaca: what did I do wrong: "snap refresh -> all snaps are up to date"; "snap refresh core -> working on core refresh"
[11:35] <zyga-ubuntu> Chipaca: bug? feature?
[11:36] <ogra_> stop using core in devmode ? :P
[11:36] <Chipaca> what ogra_ said
[11:36] <zyga-ubuntu> Chipaca: and core was in devmode for ? reasons
[11:36] <zyga-ubuntu> how is that possible
[11:36] <ogra_> (i was joking ! )
[11:36] <Chipaca> zyga-ubuntu: wasn't it?
[11:36] <zyga-ubuntu> (this is on debian)
[11:36] <Chipaca> zyga-ubuntu: without seeing what we told the store and what the store replied i have no idea
[11:37] <zyga-ubuntu> Chipaca: where is devmode listed
[11:37] <ogra_> snap list
[11:37] <zyga-ubuntu> Chipaca: I bet this is local and the fact that core was in devmode
[11:37] <Chipaca> zyga-ubuntu: you have debug logs on, right?
[11:37] <Chipaca> zyga-ubuntu: if core is in devmode it's because you installed it in devmode
[11:37] <zyga-ubuntu> core was auto-pulled on 1st snap
[11:38] <Chipaca> then it shouldn't be in devmode
[11:38] <Chipaca> zyga-ubuntu: show me the logs
[11:38] <zyga> Chipaca: https://paste.gnome.org/paaya4wqy
[11:38] <ogra_> (does core in devmode even make any sense ?)
[11:38] <zyga-ubuntu> I haven't got more than this so far
[11:38] <Chipaca> zyga-ubuntu: journalctl -u snapd ?
[11:39] <zyga> https://paste.gnome.org/psr27grjj
[11:40] <Chipaca> zyga: zyga-ubuntu: why aren't you running with debug :-(
[11:40] <zyga> hold on, https://paste.gnome.org/pjorsj5aj
[11:40] <zyga> Chipaca: because I just booted that, sorry,
[11:40] <Chipaca> zyga: turn on debug and try to reproduce?
[11:41] <Chipaca> zyga: in particular i need the SNAPD_DEBUG_HTTP=7 as well as SNAPD_DEBUG=1
[11:41] <Chipaca> while you do that, i'm going to lunch
[11:42] <zyga-ubuntu> Chipaca: reproduced
[11:42] <zyga> https://paste.gnome.org/pbsnaw5lx
[11:42] <zyga> I reverted, set the environment, restarted snapd, did refresh (all) again
[11:46] <Chipaca> zyga: so that's just for 'snap refresh', right?
[11:48] <ackk> does network-bind also allow to create unix sockets?
[11:50] <Chipaca> zyga: what revision of core do you have?
[11:50] <Chipaca> zyga: nm, i see it in the other paste
[11:50] <Chipaca> zyga: so, the store is very clearly telling us to refresh core
[11:51] <Chipaca> zyga: we're saying we have rev 2774, it's saying there's 2844 available, and here is the delta for going from 2774 to 2844
[11:52] <Chipaca> now for real, i need to go make lunch
[11:52] <Chipaca> zyga: (just in case i wasn't clear: BUG IN SNAPD)
[11:52] <Chipaca> :-)
[11:52]  * Chipaca lunch
[11:55] <zyga-ubuntu> Chipaca: thank you!
[11:57]  * zyga-ubuntu -> lunch
[12:32] <mup> PR snapd#3913 opened: tests: add test that ensures that all core services are working <Created by mvo5> <https://github.com/snapcore/snapd/pull/3913>
[12:37] <zyga-ubuntu> Pharaoh_Atem: https://taskotron.fedoraproject.org/artifacts/all/f04e1f84-9873-11e7-89a8-525400817a8f/task_output/snapd-2.27.6-1.fc26.log ?
[12:37] <jdstrand_> exit
[12:38] <jdstrand_> meh
[12:38] <zyga-ubuntu> jdstrand: :)
[12:58] <ogra_> jdstrand, have you seen https://forum.snapcraft.io/t/hdmi-audio-on-db410c-not-working/2106 ? (i wonder if the alsa interface needs to support sockets or if alsa-utils needs network interface support ... though i'm not sure if that has any influence on the functionality)
[13:00] <Son_Goku> zyga-ubuntu: the f27 update needs karma: https://bodhi.fedoraproject.org/updates/FEDORA-2017-4546b9dd38
[13:00] <zyga> aha
[13:00] <zyga> I'll try to test it too
[13:03] <Son_Goku> f25 has the same problem: https://bodhi.fedoraproject.org/updates/FEDORA-2017-d699df2619 :)
[13:03] <zyga> yep, I have that too
[13:08] <glitsj16> Hi all, after noticing an error in systemd journal about snap-repair, I looked at https://git.launchpad.net/snapd/tree/data/systemd/snapd.snap-repair.service.in and noticed that is already fixed in git by adding ConditionKernelCommandLine=snap_core to the unit file. So all good in that respect. Just wanted to report a possible typo in the same file. Last line (10) mentions "Environment=SNAP_REAPIR_FROM_TIMER=1", which I assume is meant t
[13:08] <glitsj16> o be "Environment=SNAP_REPAIR_FROM_TIMER=1" instead?
[13:08] <zyga> oh
[13:09] <zyga> glimcoil: yes that looks like a typo
[13:09] <zyga> mvo, pedronis: ^
[13:09] <glitsj16> seemed a bit silly to file a bug report on a typo, thought I would mention it here first
[13:10] <ikey> unless its a super hidden acronym
[13:10] <ikey> like.. Rest Easy As Paddy Invokes Republicanism
[13:11]  * ikey had little to go on. :P
[13:11] <glitsj16> ikey: possibly, I'm not at all familiar with the code
[13:11] <ikey> looks typo-y
[13:12] <glitsj16> nor with Republicanism :p .. but that's a minefield I'm not getting into
[13:12] <ikey> xD
[13:20] <ogra_> we'll just fix the oxford dictionary and add an alias !
[13:39] <mvo> cachio: if you have a working aarch64 sysstem, could you please paste me the output of "SNAP_SECCOMP_DEBUG=1 /usr/lib/snapd/snap-seccomp compile <(echo mknod) /tmp/xxx" (or anyone else with an arm64 system)
[13:40] <cachio> mvo, one minute
[13:40] <mvo> cachio: no rush, thanks a lot!
[13:45] <pedronis> pstolowski: btw, I was thinking that maybe it's easier to split switching from PlugInfo to PlugData, in two parts, first part Plug/SlotData has the same definition as Plug/SlotInfo (only difference is we make a deep copy of attrs when we make PlugData),  so in a lot of places is just a rename, 2nd phase we give PlugData its real interface
[13:48] <cachio> mvo, https://paste.ubuntu.com/25527599/
[13:48] <mvo> cachio: \o/ thank you so much
[13:48] <cachio> mvo,  np
[13:49] <mvo> cachio: oh, this is amd64, right? I need it for arm - aarch64, the dragonboard
[13:50] <cachio> ahh, one minute
[13:51] <niemeyer> mvo: I thought we had fixed the issue of quoting on "snap info"
[13:51] <mvo> niemeyer: I thought so too, let me look
[13:51] <niemeyer> mvo: But it seems that 2.27.6 is still quoting all summaries..
[13:52] <mvo> niemeyer: that was pr 3610
[13:52] <mup> PR #3610: snap: do not always quote the snap info summary <Created by mvo5> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/3610>
[13:53] <mvo> niemeyer: let me check what is going on
[13:53] <roadmr> mvo: hi! hey so - check an api response to https://api.snapcraft.io/api/v1/snaps/details/$SOME_SNAP. We're now exposing "contact" - the field was renamed in the web UI to "Contact URL" (can be either http{,s} or mailto url)
[13:54] <mvo> roadmr: nice!
[13:54] <roadmr> mvo: so for now and for backward-compatiblity it has the same info as support_url, but that'll eventually be deprecated
[13:54] <mvo> roadmr: sounds good, thanks for adding this
[13:54] <roadmr> note though the API response's field is called simply "contact", not "contact_url"
[13:54] <roadmr> mvo: thank matiasb_, he did it all :)
[13:54]  * mvo thanks matiasb_
[13:56] <niemeyer> mvo: Yeah, clearly recall discussing this in the sprint and looking at that code
[13:56] <niemeyer> It's even tested.. wtf
[13:56] <mvo> niemeyer: I have exactly the same feeling
[13:57] <mvo> niemeyer: looks like 2.27 does not have it yet
[13:57] <mvo> niemeyer: if you switch to edge it seems to be ok
[13:58] <niemeyer> mvo: Wow.. wat
[13:58] <mvo> niemeyer: yeah, huge latency :(
[13:59] <mvo> ogra_: do you know if we have a git branch for initramfs-tools-ubuntu-core?
[13:59] <niemeyer> mvo: Yeah, I hadn't realized (or internalized) that the dots were not really catching up
[13:59] <ogra_> mvo, under core-build
[14:00] <mvo> niemeyer: I am surprised about the delay too, we need to get to 2.28 to be vaguely back on track
[14:00] <mvo> ogra_: ta
[14:01] <cachio> mvo, https://paste.ubuntu.com/25527679/
[14:01] <cachio> mvo, on my db
[14:02] <mvo> cachio: thank you, that is very interessting
[14:02] <cachio> mvo, yaw
[14:07] <zyga-ubuntu> re
[14:07] <zyga-ubuntu> Chipaca: FYI the debug logs above were not useful because we had reverted, earlier change where the core did not refresh is real but we don't have a case for reproducing that yet
[14:10] <zyga-ubuntu> jdstrand: hey, just a quick note, I'm still going through the review, using code from libsnap-confine-private in snap-update-ns bootstrap.go is rather painful because of how linking in go build works
[14:14] <zyga-ubuntu> jdstrand: any code sharing between is in general a global pain
[14:15] <mup> PR core-build#20 opened: ubuntu-core-rootfs: deal with (broken) symlink in sync_dirs <Created by mvo5> <https://github.com/snapcore/core-build/pull/20>
[14:20] <ogra_> mvo, approved ... (needs a manual deb build and PPA upload)
[14:20] <ogra_> oh ... and a kernel snap rebuild too indeed
[14:21] <zyga-ubuntu> mvo: for testing, you _might_ explore the initramfs test tools I wrote
[14:21] <zyga-ubuntu> mvo: but it could be time consuming to learn how to use it initially
[14:22]  * ogra_ looks forward to split-initrd ... that will save so much timt
[14:22] <ogra_> *time
[14:22] <ogra_> (just waiting to get my splash PRs all approved before landing that one)
[14:24] <zyga-ubuntu> mvo: let me know if you want to try that and need a hand
[14:24] <pstolowski> pedronis, that could make things a little bit easier, but I'm not too keen on that. anyway, thanks for the suggestion
[14:27]  * zyga-ubuntu is hungry-ish and considers breaking now, doing some office manintenance and then sitting with jdstrand for a quick chat about bootstrap.c
[14:27] <zyga-ubuntu> jdstrand: would you have some time to discuss those points?
[14:28] <jdstrand> zyga-ubuntu: I have an appt and about to step out
[14:28] <zyga-ubuntu> jdstrand: are you going to be back today?
[14:28] <jdstrand> zyga-ubuntu: yes, in ~3 hours
[14:28] <mup> PR snapd#3914 opened: snap-seccomp: skip mknod syscall on arm64 <Created by mvo5> <https://github.com/snapcore/snapd/pull/3914>
[14:28] <jdstrand> zyga-ubuntu: was there a contentious point in my feedback?
[14:33] <zyga-ubuntu> jdstrand: not sure, I think I just want to chat about each and explain what happened
[14:33] <zyga-ubuntu> jdstrand: and discuss possible approaches
[14:33] <zyga-ubuntu> jdstrand: mainly that reusing code from other parts is hard
[14:33] <zyga-ubuntu> jdstrand: but anyway
[14:33] <zyga-ubuntu> jdstrand: my wife just called me and I need to break for ~3 hours too
[14:33] <zyga-ubuntu> so let's talk later :)
[14:36] <jdstrand> zyga-ubuntu: note, I wasn't blocking on that
[14:36] <jdstrand> but sure, 3 hours
[14:41] <zyga-ubuntu> jdstrand: if those are not blocking can we land and iterate?
[14:42] <jdstrand> there was a lot more to my comments than code factoring
[14:42] <jdstrand> but if you do those things, sure
[14:42]  * jdstrand -> appt
[14:42] <zyga-ubuntu> k
[14:45] <kyrofa> niemeyer, it seems ubuntu.rocket.com has an expired cert
[14:45] <kyrofa> Err, other way... you get the idea. No coffee yet
[14:46] <niemeyer> kyrofa: :)
[14:46] <niemeyer> kyrofa: I can't fix that one I think.. but let me look
[14:48] <kyrofa> Since we use HSTS, I can't even add an exception for it
[14:48] <ogra_> i just clicked on "yes" when the question about the cert popped up
[14:49] <ogra_> didnt you get that popup ?
[14:49] <ogra_> (that should add the exception client side)
[14:50] <ogra_> (/me notes that kyrofa perhaps doesnt use the rocket client snap though)
[14:51] <kyrofa> ogra_, no, firefox
[14:51] <ogra_> ah
[14:51] <ogra_> yeah, that might treat it different indeed
[14:51] <kyrofa> The fact that the client doesn't obey HSTS probably isn't a good thing
[14:52] <ogra_> yeah
[14:55] <niemeyer> kyrofa: Yeah, I can't do much about it
[14:55] <kyrofa> niemeyer, is that IS?
[14:56] <niemeyer> kyrofa: I have only admin access to the software, but not to the system, and I think the cert is done via nginx
[14:56] <niemeyer> kyrofa: yeah, I would try there first..
[14:56] <niemeyer> kyrofa: They will at least know how runs it
[14:56] <niemeyer> who
[14:56] <niemeyer> It was sabdfl originally, but I'd be surprised if it was still there
[15:04] <ogra_> oh, actually, the rocket desktop client only works half now after restarting it
[15:05] <ogra_> no graphics at all
[15:05] <kyrofa> Interesting
[15:05] <ogra_> hmm, a second restart fixed it
[15:05] <ogra_> weird
[15:45] <sergiusens> ogra_ retrying enough made it work for me
[15:45] <sergiusens> and yes, the cert expired at ~noon utc today
[15:45] <ogra_> yeah, it works here as well again
[16:08] <sborovkov_> ogra_, in the gadget snap would not it make sense to pin down specific revisions for the components that are built from source?
[16:09] <ogra_> sborovkov_, we do that ...
[16:10] <ogra_>  git clone --depth=1 https://github.com/raspberrypi/firmware.git -b "1.20170515" ....
[16:10] <ogra_>     source: git://git.denx.de/u-boot.git
[16:10] <ogra_> +    source-branch: v2017.05
[16:10] <ogra_> in https://github.com/snapcore/cm3-gadget/pull/1/files
[16:10] <mup> PR cm3-gadget#1: build uboot from source, pull blobs from upstream, use dtbs from archive <Created by ogra1> <https://github.com/snapcore/cm3-gadget/pull/1>
[16:11] <sborovkov_> ogra_, not for psplash though?
[16:11] <ogra_> well, psplash doesnt really change in the tree we pull it from (and i'm not sure they even use tags or branches)
[16:11] <ogra_> but yeah, one could do that
[16:14] <ogra_> the prob is that psplash doesnt really have any upstream anymore ... poky itself is dead and got merged into yocto
[16:15] <ogra_> we could as well, just fork it into a GH tree
[16:15] <sborovkov_> Ah ok. Just to make sure it does not actually point to something else
[16:15] <sborovkov_> as it uses master
[16:17] <ogra_> http://git.yoctoproject.org/cgit/cgit.cgi/psplash
[16:17] <ogra_> there are hHEAD and master ... but they are identical
[16:18] <sborovkov_> ogra_ yeah but repo is alive, last commit from 12 days ago. So something might get broken at one point
[16:18] <ogra_> long term it probably makes sense to carry our own tree but i dont think it is super important
[16:19] <ogra_> did you have any issues building ?
[16:19] <sborovkov_> no :-) I am just being bothered by people on code review who want to pin every possible thing down to specific version :-(
[16:22] <Chipaca> zyga-suse: how does arch define what files get removed when you remove a package?
[16:25] <ogra_> sborovkov_, https://github.com/ogra1/psplash ... i promise i wont change it :P
[16:26] <ogra_> sborovkov_, you can use the 09.2017 tag in your gadget now
[16:30] <Chipaca> zyga-suse: also, is %dir documented anywhere? i guess it does what i want but i'm just guessing
[16:30] <sborovkov_> ogra_ ok thanks
[16:59] <Chipaca> zyga-suse: o/
[16:59] <Chipaca> zyga-ubuntu: or you :-) o/
[17:00] <mvo> 2.28~rc3 in the beta channel  now and rc3 builds in xenial,zesty,artful,trusty everywhere so yay progress
[17:01] <niemeyer> \o/
[17:01] <niemeyer> mvo: Thanks!
[17:01] <Pharaoh_Atem> zyga-suse: karma for Fedora updates?
[17:01] <Pharaoh_Atem> I only see F26
[17:02] <Pharaoh_Atem> I need some for F25 and F27
[17:03] <cachio> mvo, did you add the fixes for failover and install-store tests?
[17:04] <mvo> cachio: if there were tagged with 2.28 then yes, if not then no :) if there were not tagged, could you please give me the PR numbers and I can look at this in my morning
[17:04] <cachio> sure
[17:05] <cachio> mvo, let me search forthem
[17:07] <zyga-ubuntu> Chipaca: hey
[17:07] <Chipaca> zyga-ubuntu: dunno if you saw my questions to your alter ego
[17:07] <zyga-ubuntu> Pharaoh_Atem: soon, I had to go out
[17:07] <Chipaca> zyga-ubuntu: does arch have a way of saying what needs to be done to remove the package? I can't see it
[17:07] <zyga-ubuntu> Chipaca: no, but if you sent them to zyga-suse then I can see them in 2 minutes
[17:07] <zyga-ubuntu> Chipaca: just pulling over now
[17:07] <Chipaca> zyga-ubuntu: and the second question was whether opensuse's %dir was documented anywhere
[17:08] <zyga-ubuntu> Chipaca: I suspect so
[17:08] <zyga-ubuntu> one sec
[17:08] <Chipaca> i'm guessing at opensuse needing a "%dir /var/cache/snapd"
[17:08] <zyga-ubuntu> Chipaca: https://en.opensuse.org/openSUSE:Packaging_guidelines
[17:08] <zyga-ubuntu> look for %dir
[17:10] <Chipaca> zyga-ubuntu: yeah, i'd gotten to this page on my own, but it doens't really tell me if %dir means the directory will be cleaned up on remove
[17:10] <zyga-suse> Chipaca: I think so but let me do an experiment
[17:11] <zyga-suse> Chipaca: I just need to put the groceries where they belong
[17:11] <Chipaca> zyga-suse: and i need to get the fish to hug the broccoli
[17:11] <Chipaca> zyga-suse: but i'll leave this on and read it when i get back
[17:12] <Pharaoh_Atem> if the directory is created by snapd package, it'll be removed by it, if there's no contents inside
[17:12] <Pharaoh_Atem> http://ftp.rpm.org/max-rpm/s1-rpm-inside-files-list-directives.html#S3-RPM-INSIDE-DIR-DIRECTIVE
[17:13] <Chipaca> Pharaoh_Atem: and if it has contents?
[17:14] <zyga-ubuntu> Chipaca: fine
[17:15] <mup> PR snapd#3911 closed: many: add logger.MockLogger() and use it in the tests <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3911>
[17:17] <Pharaoh_Atem> Chipaca: I think it will not remove it
[17:18] <Pharaoh_Atem> though I'm not sure
[17:18] <Pharaoh_Atem> it may choose to delete it since it doesn't know what's in it
[17:29] <mup> PR snapd#3915 opened: cmd/snap: return empty document if snap has no configuration <Created by stolowski> <https://github.com/snapcore/snapd/pull/3915>
[18:11] <kwmonroe> hey snappers, should i be able to share $SNAP_USER_foo with the content interface?  my slot can define "read: $SNAP" and a plug works as expected.. but when i use "read: $SNAP_USER_COMMON", i get this, as if $SNAP_USER_COMMON is being taken literally:
[18:11] <kwmonroe> $ snap run --shell pig
[18:11] <kwmonroe> cannot perform operation: mount --bind -o ro,nosuid,nodev /snap/hadoop/x1/$SNAP_USER_COMMON /snap/pig/x1/hadoop: No such file or directory
[18:13] <ogra_> just use the home interface, it should have access to all non dot dirs in your home already
[18:14] <ogra_> (SNAP_USER_COMMON in the above case is ~/snap/pig/common )
[18:16] <kwmonroe> oooh!  gotcha ogra_.  /snap/hadoop/x1/$SNAP_USER_COMMON  threw me off, but it makes sense that it's USER_blah in the context of the pig snap.
[18:16] <kwmonroe> thanks!
[18:18] <ogra_> if you have a hadoop snap and want to access the user specific data, the home interface should give you access to ~/snap/hadoop/current (or ~/snap/hadoop/common) as long as no hidden dir is involved
[18:19] <kwmonroe> yup, +1
[18:31] <mup> PR snapcraft#1540 closed: plugins: extract python finder functions <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1540>
[18:39] <mup> PR snapd#3913 closed: tests: add test that ensures that all core services are working <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3913>
[18:46] <mup> PR snapcraft#1547 opened: plugins: extract sitecustomize logic from python <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1547>
[18:55] <om26er> does Ubuntu Core have dbus ?
[19:04] <roadmr> jdstrand: howdy, click-reviewer-tools r930 is now live
[19:09] <kyrofa> niemeyer, in the configure hook, is there any way to compare new values to old? Or is it still always new?
[19:10] <niemeyer> kyrofa: Still always new..
[19:10] <niemeyer> kyrofa: Really want to do something about that
[19:10] <niemeyer> Ideally with deltas rather than just what's new and what's old
[19:10] <niemeyer> Config has a lot of love to receive still
[19:10] <kyrofa> Indeed
[19:10] <kyrofa> Alright, thanks!
[19:13] <jdstrand> roadmr: woo, thanks!
[19:14] <jdstrand> roadmr: kenvandine fyi, that has the desktop and desktop-legacy interfaces ^
[19:14] <roadmr> jdstrand: hehe you lucked out as usual, not many changes to deploy
[19:14] <mup> PR snapd#3880 closed: tests: add trivial canonical-livepatch test to ensure we do not break it <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3880>
[19:14] <jdstrand> roadmr: I'm curious now what we'll see with those occasional unexpected outputs
[19:14] <roadmr> \o/
[19:15] <roadmr> jdstrand: yeah... not sure :( if it doesn't have the thing in the json that causes the store to consider it failed, it'll likely just be noted as another test and stored in the output
[19:16] <kenvandine> jdstrand, woot
[19:30] <jdstrand> roadmr: it will show up as an error, which will flag manual review, so I'll see it
[19:31] <jdstrand> roadmr: that is for anything that calls error(). if we see 'unexpected output' after this, there are only a couple of things it could be
[19:33] <roadmr> jdstrand: ah! I get it now.
[19:48] <tvansteenburgh> kyrofa: anything i can do to help move along https://bugs.launchpad.net/snapcraft/+bug/1686481 ?
[19:48] <mup> Bug #1686481: stage-packages doesn't respect architectures <kubernetes> <Snapcraft:Confirmed> <https://launchpad.net/bugs/1686481>
[19:49] <tvansteenburgh> this bug prevents us from getting snap builds in upstream kubernetes
[19:51] <kyrofa> Not directly, we've been revamping that part of the codebase, I need to test again with the new changes
[19:51] <cachio> om26er, you have a dbus interface, see https://snapcraft.io/docs/reference/interfaces
[19:51] <cachio> om26er, does it work for you?
[20:01] <kyrofa> tvansteenburgh, it would be helpful if you're able to test with the newest snapcraft, if you're able, v2.34. It's in artful, and -proposed for z and x
[20:01] <kyrofa> Huh, I don't think I could have written that sentence worse
[20:01] <kyrofa> Changed gears partway through apparently
[20:08] <tvansteenburgh> kyrofa: okay thanks
[20:15] <om26er> cachio: yes that interface does work. My need was only related to getting the machine-id
[20:15] <om26er> and it seems I can just read /etc/machine-id even in a confined snap.
[20:22] <mup> PR snapcraft#1541 closed: tests: fix the TEST_STORE environment variable  <Created by elopio> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1541>