[00:07] <mup> PR snapcraft#2568 closed: docs: consolidate on a simple HACKING.md <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2568>
[05:06] <mborzecki> morning
[06:00] <zyga> Hey mborzecki
[06:00] <mborzecki> zyga: hey
[06:00] <mborzecki> zyga: how's your day off coming? :)
[06:00] <zyga> Haha
[06:00] <zyga> I was just wondering the same thing
[06:00] <zyga> Should I take today off for real
[06:00] <zyga> Or work some more to get stuff done
[06:01] <zyga> So many things to work on
[06:05] <zyga> mborzecki: I sent a small patch for spread-shellcheck
[06:05] <zyga> And had an idea on how to fix not MATCH
[06:07] <zyga> We can save MATCH to lib/bin on startup
[06:07] <zyga> Like we do now to match.sh include file
[06:07] <zyga> Since it is on path it will be more flexible
[06:08] <zyga> Then MATCH is no longer a function
[06:08] <zyga> Then we can put “not” on path and fix all tests
[06:15] <mborzecki> zyga: in other news, got an image built by snap-image (strawman) to boot up to a point where i got console conf :P
[06:16] <zyga> Wooooot
[06:16] <zyga> That is great
[06:17] <zyga> It still feels that landing your changes will take a month
[06:17] <mborzecki> yeah, and tests and all that
[06:18] <mborzecki> zyga: if you guys have a system-rescue role that you'd like me to add or something, this coudl be done really easily
[06:19] <zyga> Role as in partition type?
[06:19] <zyga> I think we should talk about how the recovery system is designed, so far, to keep you in the loop
[06:19] <mborzecki> zyga: role as role in gadget.yaml terms, and whatever that means practically, eg. like system-boot is a bit special
[06:19] <mborzecki> zyga: and so is system-data
[06:20] <zyga> I think some things there will change
[06:20] <zyga> Though I think that is best for next week
[06:37] <zyga> re, back in the office
[06:38] <zyga> mborzecki: can you do a quick review of https://github.com/snapcore/snapd/pull/6906
[06:38] <zyga> just a variable rename
[06:38] <mup> PR #6906: cmd/snap-update-ns: rename ctx to upCtx <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6906>
[06:38] <zyga> though more importantly I need a review for https://github.com/snapcore/snapd/pull/6903 to get 2.39.1 fix in place
[06:38] <mup> PR #6903: spread-shellcheck: add support for variants and environment <Created by zyga> <https://github.com/snapcore/snapd/pull/6903>
[06:38] <zyga> the shellcheck change is not perfect
[06:38] <zyga> but it's also not totally incorrect
[06:38] <zyga> and allows my tests to not get stuck on that
[07:01]  * zyga quick coffee
[07:01] <zyga> mborzecki: if you can review the spread-shellcheck change I will be able to propose the propagation fixes
[07:03] <zyga> I will do some branch gardening soon
[07:03] <zyga> but first coffee
[07:04] <pstolowski|afk> morning
[07:09] <zyga> Hey :-)
[07:19] <zyga> Interesting https://forum.snapcraft.io/t/snapshots-can-expose-sensitive-data/11493
[07:27] <pstolowski> zyga: hmm interesting indeed
[07:27] <mup> PR snapd#6906 closed: cmd/snap-update-ns: rename ctx to upCtx <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6906>
[07:27] <zyga> makes me feel that snapshots should be saved back to $HOME
[07:27] <zyga> as anything else is not correct wrt NFS and encryption
[07:28] <zyga> pstolowski: quick trivial https://github.com/snapcore/snapd/pull/6907
[07:28] <mup> PR #6907: cmd/snap-update-ns: add several TODO comments <Created by zyga> <https://github.com/snapcore/snapd/pull/6907>
[07:28] <zyga> just comments
[07:28] <zyga> but that's *all* of the refactoring :D
[07:28] <zyga> wooooot
[07:28] <mup> PR snapd#6907 opened: cmd/snap-update-ns: add several TODO comments <Created by zyga> <https://github.com/snapcore/snapd/pull/6907>
[07:29] <pstolowski> +1§
[07:30] <mborzecki> pstolowski: hey
[07:30] <pstolowski> o/
[07:40] <mup> PR snapd#6894 closed: tests: add "not" command as replacement for "!" in tests <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/6894>
[07:44] <mup> PR snapd#6908 opened: tests: add "not" command <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6908>
[08:05] <zyga> mborzecki, pstolowski: how do you feel about ^ not
[08:05] <zyga> I want to build this on top  https://github.com/snapcore/snapd/pull/6909
[08:05] <mup> PR #6909: spread.yaml,tests: change MATCH and REBOOT to cmds <Created by zyga> <https://github.com/snapcore/snapd/pull/6909>
[08:05] <zyga> and then fix our scripts to use not rather than !
[08:06] <mup> PR snapd#6909 opened: spread.yaml,tests: change MATCH and REBOOT to cmds <Created by zyga> <https://github.com/snapcore/snapd/pull/6909>
[08:11] <pstolowski> zyga: interesting; don't we still need to make it a compound expressionm e.g. { ! "$@" } ?
[08:11] <zyga> no,  why?
[08:12] <zyga> ! functions correctly
[08:12] <zyga> note that this executable is not sourced, it's a program that runs without set -e
[08:12] <pstolowski> zyga: ok, so -e is the key here. i was reading also https://stackoverflow.com/questions/39581150/why-do-i-need-parenthesis-in-bash-set-e-and-negated-return-code/39582012
[08:22] <zyga> I wonder if snapd is going to get any sponsors: https://techcrunch.com/2019/05/23/github-launches-sponsors-lets-you-pay-your-favorite-open-source-contributors/
[08:40] <zyga> mborzecki: https://github.com/google/pytype
[08:40] <zyga> mborzecki: from that video you linkedto
[08:51] <zyga> mborzecki: updated https://github.com/snapcore/snapd/pull/6903
[08:51] <mup> PR #6903: spread-shellcheck: add support for variants and environment <Created by zyga> <https://github.com/snapcore/snapd/pull/6903>
[08:55] <mborzecki> zyga: nhm, lgtm
[08:56] <zyga> nhm?
[09:00] <zyga> mborzecki: something for you :) https://github.com/snapcore/snapd/pull/6910
[09:00] <mup> PR #6910: spread.yaml: use "snap connections" in debug <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6910>
[09:00] <mup> PR snapd#6910 opened: spread.yaml: use "snap connections" in debug <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6910>
[09:03]  * zyga really gets that coffee now
[09:03] <zyga> after the break I will try to land "not" and "MATCH" as commands and see if I can fix ! tree-wide
[09:03] <zyga> and revert shellcheck patch so that we track edge again
[09:17] <zyga> back now :)
[09:17] <zyga> aaaand, "not" is red
[09:20] <mup> PR snapd#6907 closed: cmd/snap-update-ns: add several TODO comments <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6907>
[09:35] <pstolowski> and i just had google:ubuntu-14.04-64:tests/main/parallel-install-aliases failure on one of my PRs
[09:35] <pstolowski> never seen it failing
[09:38] <zyga> pstolowski: how did it fail?
[09:40] <pstolowski> zyga: it was expecting the change to fail with 'cannot enable alias "alias1" for "aliases_foo", already enabled for "aliases"', but instead got 'Done    today at 15:27 UTC  today at 15:27 UTC  Setup manual alias "alias1" => "cmd1" for snap "aliases"'
[09:40] <pstolowski> zyga: i wonder it snap change --last=alias check there isn't racy
[09:41] <zyga> huh, I saw that fail once as well
[09:41] <zyga> yesterday
[09:41] <pstolowski> because just above we test happy execution
[09:41] <pstolowski> zyga: i'll kick it again on travis and in the meantime execute and investigate it locally
[09:41] <zyga> k
[09:42] <pstolowski> and if that's it, will fix it
[09:42] <zyga> pstolowski: can you look at https://github.com/snapcore/snapd/pull/6903
[09:42] <mup> PR #6903: spread-shellcheck: add support for variants and environment <Created by zyga> <https://github.com/snapcore/snapd/pull/6903>
[09:42] <pstolowski> k
[09:42] <zyga> I need a 2nd review to open the fix for the mount propagation PR
[09:52] <zyga> mborzecki: curious, permissions package seems to be updated with some snapd bits
[09:52] <zyga> as a simple build in obs failed on that
[09:52]  * zyga looks
[09:53] <zyga> settle not converging
[09:53]  * zyga loves when unit tests fail
[09:54] <zyga> pstolowski: does this sound familiar?
[09:54] <zyga> https://www.irccloud.com/pastebin/02IEvvjO/
[09:55] <zyga> ah, this is device registration, not hotplug
[09:55] <zyga> sorry
[09:55] <zyga> bah
[09:56] <zyga> this happens each time
[09:56]  * zyga debugs
[10:01] <mup> PR snapd#6893 closed: gadget: helper for shifting structure start position <Gadget update> <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6893>
[10:01] <zyga> hrm
[10:01] <zyga> go test passes on my tree with 2.39
[10:01] <zyga> fails in the tarball
[10:01] <zyga> whhyyyyy
[10:05] <zyga> hmmm
[10:14] <zyga> I disabled unit tests and installed the resulting package
[10:14] <zyga> let's see how it operates
[10:14] <zyga> then I can see why they fail again
[10:20] <zyga> eh
[10:20] <zyga> in x11 mode I don't have /snap/bin on path
[10:20] <zyga> in wayland mode some apps just crash
[10:20] <zyga> lovely
[10:23] <zyga> after reboot things seem to be have
[10:23] <zyga> behave even
[10:35] <pstolowski> zyga: hmm somewhat, seen failures like this from time to time while working on these tests, usually was a race. you're saying it fails each time like this?
[10:35] <zyga> do you remember https://github.com/snapcore/snapd/pull/6360 <- I just merged master into it :)
[10:35] <mup> PR #6360: cmd/snap-update-ns: refactor of profile application <Per-user mount ns  🐎> <Created by zyga> <https://github.com/snapcore/snapd/pull/6360>
[10:35] <zyga> pstolowski: yeah
[10:35] <zyga> but only in packaging build
[10:35] <zyga> not in a checkout
[10:35] <zyga> I didn't look deeper yet
[10:36] <zyga> "not" failed again
[10:37] <pstolowski> zyga: on which test?
[10:37] <zyga> service watchdog
[10:51] <pstolowski> zyga: +1 on 6903 with a question
[10:51] <zyga> looking
[10:55] <pstolowski> of course parallel-install-aliases is passing when run locally... will keep it running during lunch
[10:56] <zyga> pstolowski: replied there
[11:00] <pstolowski> ty
[11:00]  * pstolowski lunch
[11:03] <mup> PR snapd#6903 closed: spread-shellcheck: add support for variants and environment <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6903>
[11:04] <zyga> I've opened https://github.com/snapcore/snapd/pull/6891 for real now
[11:04] <mup> PR #6891: many: make per-snap mount namespace MS_SHARED <Created by zyga> <https://github.com/snapcore/snapd/pull/6891>
[11:04] <zyga> tests *should* pass now
[11:04] <zyga> so fingers crossed
[11:05] <zyga> I think I need a break
[11:05] <zyga> but it's moving along
[11:05] <zyga> what do you guys think about https://github.com/snapcore/snapd/pull/6909 ?
[11:05] <mup> PR #6909: spread.yaml,tests: change MATCH and REBOOT to cmds <Created by zyga> <https://github.com/snapcore/snapd/pull/6909>
[11:05] <zyga> I will work on mountinfo-tool to support earlier versions of python so that we can use it in 16.04
[11:09] <mup> PR snapd#6911 opened: cmd/snap-update-ns: rename leftover ctx to upCtx <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6911>
[11:11]  * zyga breaks
[11:14] <mup> PR snapd#6912 opened: gadget: keep track of the index where structure content was defined <Gadget update> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6912>
[11:15] <mup> PR snapd#6883 closed: tests: fix how strings are matched on auto-refresh-retry test <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6883>
[11:15] <mup> PR snapd#6910 closed: spread.yaml: use "snap connections" in debug <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6910>
[11:17] <zyga> mborzecki: is https://github.com/snapcore/snapd/pull/6890 really for review?
[11:17] <mup> PR #6890: gadget: mounted filesystem writer & updater <Gadget update> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6890>
[11:17] <mborzecki> zyga: heh, yes
[11:17] <zyga> *gulp*
[11:17] <mborzecki> zyga: want to dive in? :)
[11:18] <zyga> they told me to split a 1K diff
[11:18] <zyga> took several moths to review
[11:18] <zyga> *months
[11:18] <zyga> (i'd be faster with moths :D
[11:18] <mborzecki> zyga: 1.6k is tests :P
[11:18] <zyga> for me 80% was tests
[11:18] <zyga> good luck :-)
[11:18] <zyga> no way to chunk that?
[11:19] <mborzecki> idk, maybe i could sample, method by method :/
[11:19] <zyga> at the end of the day I can review and approve but you need extra reviews from samuele
[11:19] <mborzecki> hm maybe i could pull the writer thing apart, that's ~10% of the thing
[11:20] <zyga> ok, really time to break now
[11:20] <zyga> I'll merge "not" when green
[11:20] <zyga> and if you guys can review https://github.com/snapcore/snapd/pull/6909 we could fix all of ! foo bugs
[11:20] <mup> PR #6909: spread.yaml,tests: change MATCH and REBOOT to cmds <Created by zyga> <https://github.com/snapcore/snapd/pull/6909>
[11:20] <mup> PR snapd#6913 opened: tests: force snap-reexec test to fail to reproduce the error on ubuntu 14.04 <⛔ Blocked> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6913>
[11:29] <mborzecki> zyga: should we have NOT instead of lowercase not to match MATCH and REBOOT?
[11:36] <popey_> chipaca not working today?
[11:46] <zyga> mborzecki: MATCH and REBOOT come from spread
[11:46] <zyga> popey_: I think he's off today
[11:46] <zyga> mborzecki: I think lowercase "not" looks nicer
[11:46] <popey_> ta
[11:47] <zyga> mborzecki: but to answer your question, I kept REBOOT and MATCH uppercase only to match spread names and avoid code changes
[11:47] <zyga> not failed again :D
[11:47] <zyga> heh
[11:47] <zyga> landing a trivial unused file
[11:47] <mborzecki> pstolowski: a look at https://github.com/snapcore/snapd/pull/6912 ?
[11:47] <mup> PR #6912: gadget: keep track of the index where structure content was defined <Gadget update> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6912>
[11:48] <zyga> this time on refresh-delta-from-core
[11:48] <zyga> meh
[11:48] <zyga> I'll grab some food
[11:48] <zyga> ttyl
[12:19] <zyga> tests still churning ...
[12:35] <mborzecki> zyga: can you do another pass of #6750? you did one there already, and I pushed some changes in the meantime
[12:35] <mup> PR #6750: overlord/devicestate: update-gadget task handler with stubbed gadget callbacks <Gadget update> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6750>
[12:35] <zyga> yep
[12:35] <mup> PR snapd#6908 closed: tests: add "not" command <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6908>
[12:35] <zyga> nice!
[12:36] <zyga> mborzecki: can you discuss that portal test change PR with cachio please?
[12:37] <zyga> mborzecki: https://github.com/snapcore/snapd/pull/6912 needs formatting fixes
[12:37] <mup> PR #6912: gadget: keep track of the index where structure content was defined <Gadget update> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6912>
[12:37] <cachio> zyga, sure
[12:37] <zyga> probably need to reformat with ancient go
[12:37] <zyga> (yuck)
[12:38] <zyga> settle not converging (unit tests) https://www.irccloud.com/pastebin/KgUHJ07T/
[12:38] <cachio> mborzecki, did you see the comments in the PR?
[12:38] <zyga> loads of those
[12:42] <mborzecki> omg, i hate they made the formatting change in 1.11
[12:42] <mborzecki> super annoying
[12:51] <jdstrand> zyga: fyi, I added a sprinking of emojis, a comment on your response to /etc and then also https://github.com/snapcore/snapd/pull/6891#issuecomment-495609846
[12:51] <mup> PR #6891: many: make per-snap mount namespace MS_SHARED <Created by zyga> <https://github.com/snapcore/snapd/pull/6891>
[12:51] <zyga> :-)
[12:51] <zyga> thanks!
[12:51] <zyga> I noticed it failed for real in one specific test
[12:51] <zyga> looking now
[13:01] <zyga> pstolowski: standup
[13:01] <pstolowski> yep
[13:01] <Wimpress> Snapcraft Live is just starting - https://www.youtube.com/watch?v=yAnRM4m63MY
[13:02] <zyga> Wimpress: nice!
[13:02] <mborzecki> Wimpress: nice, shirt, tie and shorts right? :)
[13:03] <zyga> mborzecki: *hopefully* there are shorts
[13:03] <mborzecki> the upsides of working remotely :)
[13:09] <mup> PR snapd#6911 closed: cmd/snap-update-ns: rename leftover ctx to upCtx <Simple 😃> <Created by zyga> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6911>
[13:39] <zyga> jdstrand: I consistently get a denial for a permission I hold
[13:39] <zyga> perhaps something quirky with mount options and 14.04 (4.4) kernel
[13:39] <jdstrand> zyga: does it go away with a bare mount rule?
[13:39] <zyga> jdstrand: just "mount," ?
[13:40] <jdstrand> yes
[13:40] <zyga> jdstrand: curiously, it does not
[13:40] <zyga> ahhh
[13:40] <zyga> I'm a tool
[13:41] <zyga> sorryt
[13:41] <zyga> reexec bit me
[13:41] <jdstrand> heh. it always does at least once ;)
[13:42] <zyga> jdstrand: bare mount rule is ok
[13:42] <zyga> let's see if without it it also works :D
[13:43] <zyga> so it works but I don't know why ... let's restart and see
[13:44] <zyga> jdstrand: I think something is still fishy but I cannot point to anything apart from the failure in spread
[13:46] <zyga> jdstrand: I tweaked things a little os that they read:
[13:46] <zyga> relevant fragment of apparmor permissions https://www.irccloud.com/pastebin/v267dsbt/
[13:46] <pstolowski> zyga: do you have anything needing 2nd review that i can take a look at?
[13:46] <zyga> the denial I got is associated with permission on line 6
[13:46] <zyga> pstolowski: https://github.com/snapcore/snapd/pull/6891
[13:47] <zyga> it's not 2nd but it needs review
[13:47] <mup> PR #6891: many: make per-snap mount namespace MS_SHARED <Created by zyga> <https://github.com/snapcore/snapd/pull/6891>
[13:47] <zyga> that's the 2.39.1 patch
[13:48] <jdstrand> zyga: so, you are saying there is no apparmor denial, but an eperm, but with a bare mount rule, it all works?
[13:48] <zyga> there's a denial
[13:48] <zyga> bare mount fixes it
[13:48] <jdstrand> what is the denial?
[13:49] <zyga> the denial is
[13:49] <zyga> May 24 13:41:11 may241325-532686 kernel: audit: type=1400 audit(1558705271.095:74): apparmor="DENIED" operation="mount" info="failed flags match" error=-13 profile="/snap/snapd/x1/usr/lib/snapd/snap-confine" name="/tmp/snap.rootfs_dPNxms/" pid=18844 comm="snap-confine" flags="rw, rshared"
[13:49] <jdstrand> you're missing rw
[13:49] <zyga> but wait, rw I had
[13:49] <zyga> I removed it
[13:50] <zyga> (also rw makes no sense for propagation changes)
[13:50] <jdstrand> I can only comment on what you have shown me
[13:50] <jdstrand> :)
[13:50] <zyga> yeah I know :)
[13:51] <jdstrand> zyga: I'm not sure why rw would pop out. that may be a bug or it may be some subtlety in the kernel that needs it cause it is shared
[13:51] <jdstrand> seeing the strace of that might help...
[13:51] <zyga> yeah
[13:53] <roadmr> jdstrand: hey! tools r... wait... checking the actual version... "20190522-1231UTC click-reviewers-tools version " is now in production \o/ I think you can now get rid of your bzr branch
[13:53] <jdstrand> zyga: I mean, looking at the strace I have here, the rbinds don't specify rw, but there it is on line 4 of your paste
[13:53] <jdstrand> roadmr: yay!!
[13:54] <jdstrand>  \o\
[13:54] <jdstrand>  /o/
[13:54] <jdstrand>  \o\
[13:54] <jdstrand>  \o/
[13:54]  * jdstrand hugs roadmr :)
[13:54] <roadmr> 🙌  hehe
[13:55] <jdstrand> zyga: I suspect if you add the rw and account for reexec, you will be on your way
[13:55] <zyga> there's rw there already
[13:55] <zyga> it's not that
[13:55] <zyga> I'm debugging
[13:56]  * jdstrand is confused
[13:57] <zyga> jdstrand: there *was* rw, I removed it thinking it is the issue
[13:59] <jdstrand> zyga: but reexec?
[13:59] <zyga> I changed the source and re-ran spread
[14:00] <zyga> gah, I'm tired
[14:00] <zyga> jdstrand: I merged master into the ancient huge PRs
[14:01] <zyga> jdstrand: https://github.com/snapcore/snapd/pull/6341 is interesting (diff wise small)
[14:01] <mup> PR #6341: WIP: persist per-user mount namespace profile <Created by zyga> <https://github.com/snapcore/snapd/pull/6341>
[14:02] <mup> Bug #1830382 opened: Have a progress indicator for snap pack (like with snapcraft pack) <Snappy:New> <https://launchpad.net/bugs/1830382>
[14:03] <zyga> jdstrand: with that branch I should be able to answer your questions about the set of required capabilities
[14:05] <jdstrand> these PRs are like rabbits
[14:05] <mup> Bug #1830382 changed: Have a progress indicator for snap pack (like with snapcraft pack) <snapd:Triaged> <https://launchpad.net/bugs/1830382>
[14:05] <jdstrand> I review a bunch and then end up with more than I started with
[14:05] <zyga> yeah
[14:07] <sparkiegeek> mvo, zyga: btw, shouldn't https://bugs.launchpad.net/snapd/+bug/1819318 be at least Fix Committed? and arguably Fix Released?
[14:07] <mup> Bug #1819318: no interfaces when installing only core18 <snapd:In Progress> <https://launchpad.net/bugs/1819318>
[14:08] <zyga> sparkiegeek: in a call
[14:08] <zyga> mvo worked on that
[14:15] <zyga> jdstrand: https://github.com/snapcore/snapd/pull/6891#issuecomment-495643768 I'm giving up now
[14:15] <mup> PR #6891: many: make per-snap mount namespace MS_SHARED <Created by zyga> <https://github.com/snapcore/snapd/pull/6891>
[14:15] <zyga> need to take a walk and think
[14:22] <pstolowski> a few hours later and... parallel-install-aliases doesn't want to fail
[14:22] <mup> PR snapd#6912 closed: gadget: keep track of the index where structure content was defined <Gadget update> <Simple 😃> <Created by bboozzoo> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6912>
[15:34] <mup> PR snapd#6914 opened: tests: change strace parameters on snap-run test to avoid the test gets stuck <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6914>
[15:57]  * cachio lunch
[16:55]  * zyga feels sick somehow, EODing now
[17:10] <mup> Bug #1830412 opened: QT5 classic snaps segmentation fault <Snappy:New> <https://launchpad.net/bugs/1830412>
[18:59]  * cachio afk
[20:53] <mup> PR pc-amd64-gadget#17 closed: add .travis.yml to enable basic smoke testing (20) <Created by mvo5> <Merged by vorlonofportland> <https://github.com/snapcore/pc-amd64-gadget/pull/17>
[20:55] <mup> PR pc-amd64-gadget#16 closed: add .travis.yml to enable basic smoke testing (18) <Created by mvo5> <Merged by vorlonofportland> <https://github.com/snapcore/pc-amd64-gadget/pull/16>
[21:00] <mup> PR pc-amd64-gadget#15 closed: add .travis.yml to enable basic smoke testing <Created by mvo5> <Merged by vorlonofportland> <https://github.com/snapcore/pc-amd64-gadget/pull/15>