[04:31] <mup> Bug #1906124 changed: Snap applications ignore .hidden files <Snappy:Expired> <https://launchpad.net/bugs/1906124>
[04:34] <mup> Bug #1906124 opened: Snap applications ignore .hidden files <Snappy:Expired> <https://launchpad.net/bugs/1906124>
[04:40] <mup> Bug #1906124 changed: Snap applications ignore .hidden files <Snappy:Expired> <https://launchpad.net/bugs/1906124>
[06:50] <zyga> good morning
[06:57] <amurray> hey zyga :)
[06:57] <zyga> hey amurray :)
[06:57] <zyga> congratulations on your promotion :)
[07:01] <amurray> thanks mate :) how are things for you?
[07:16] <mborzecki> morning
[07:26] <zyga> hey mborzecki, mvo
[07:27] <mvo> good morning zyga and mborzecki !
[07:27] <mborzecki> mvo: zyga: hey guys
[07:27] <mborzecki> mvo: how was your long weekend?
[07:27] <mvo> mborzecki: anything I can help with right away?
[07:27] <mvo> mborzecki: very nice!
[07:27] <zyga> today we have an interesting day at my current job - experience aka dogfood day
[07:27] <zyga> no new code
[07:27] <zyga> just try things out
[07:28] <zyga> have a chance to slow down and see what others did
[07:28] <mvo> mborzecki: really enjoyed it, even did some hockey practise (without actually playing for real as conctact sports are not allowed yet)
[07:28] <zyga> and how it looks like
[07:28] <mborzecki> mvo: nice!
[07:28] <mvo> zyga: sounds like a cool idea, did you came up with it?
[07:29] <zyga> no, not this time :-)
[07:29] <zyga> since there are several reference boards available I plan to use my time to just try them out
[08:00] <pstolowski> morning
[08:01] <zyga> hey pstolowski :)
[08:03] <mvo> good morning pstolowski
[08:08] <mup> PR snapd#10066 opened: overlord/snapstate/backend: do not remove current symlink for the snapd snap <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10066>
[08:19] <zyga> mborzecki ^ quick question posted
[08:58] <mup> PR snapd#10031 closed: o/configstate/configcore: add hdmi_timings to pi-config <Created by sergey-borovkov> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10031>
[08:58] <mup> PR snapd#10042 closed: snapstate: reduce reRefreshRetryTimeout to 1/2 second <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10042>
[09:21] <doko> mvo: do you plan a snapd upload anytime soon? else I'd just like to fix the dh-systemd b-d
[09:48] <mborzecki> mvo: hm related to kernel refs? https://paste.ubuntu.com/p/NSN2QSxhSt/
[09:55] <mvo> mborzecki: yes, where do you see that?
[09:56] <mvo> mborzecki: is that on master?
[10:15] <mborzecki> mvo: this PR https://github.com/snapcore/snapd/pull/10006
[10:15] <mup> PR #10006: cmd/snap-bootstrap: refactor handling of ubuntu-save, do not use secboot's implicit fallback <Run nested> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10006>
[10:20] <mvo> mborzecki: uh, ok. well, I think I fixed it in my "policy" PR because it was failing there too
[11:08] <mborzecki> mvo: can you land https://github.com/snapcore/snapd/pull/10006 ? none of the failures are related to the changes
[11:08] <mup> PR #10006: cmd/snap-bootstrap: refactor handling of ubuntu-save, do not use secboot's implicit fallback <Run nested> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10006>
[11:08] <mvo> mborzecki: sure
[11:08] <mborzecki> thanks!
[11:09] <mup> PR snapd#10006 closed: cmd/snap-bootstrap: refactor handling of ubuntu-save, do not use secboot's implicit fallback <Run nested> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10006>
[12:37] <mup> PR snapcraft#3483 closed: requirements: use PyNaCl 1.3.0 and ensure is compiled on linux <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3483>
[12:42] <mup> PR snapcraft#3485 opened: build(deps): bump lxml from 4.6.2 to 4.6.3 <dependencies> <python> <Created by dependabot[bot]> <https://github.com/snapcore/snapcraft/pull/3485>
[13:04] <mborzecki> pedronis: another take at the current symlink https://github.com/snapcore/snapd/pull/10067
[13:04] <mup> PR #10067: overlord/snapstate: make sure that snapd current symlink is not removed during refresh <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10067>
[13:04] <mup> PR snapd#10067 opened: overlord/snapstate: make sure that snapd current symlink is not removed during refresh <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10067>
[13:22] <mup> PR snapcraft#3482 closed: Support install-mode option for apps <Created by cmatsuoka> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3482>
[13:32] <mup> PR snapcraft#3485 closed: build(deps): bump lxml from 4.6.2 to 4.6.3 <dependencies> <python> <Created by dependabot[bot]> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3485>
[13:54] <mup> PR snapd#10068 opened: [RFC] o/configstate: don't pass --root=/ when masking/unmasking/enabling/disabling services <Run nested> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10068>
[13:55] <pedronis> mborzecki: thx, was in a meeting
[14:44] <mup> PR snapd#10060 closed: boot: extend set try recovery system unit tests <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10060>
[15:33] <pstolowski> pedronis: uh, one more bug around config https://bugs.launchpad.net/snapd/+bug/1920773 ; it's a nice find
[15:37] <mup> PR snapcraft#3486 opened: package-repositories: use last 8 characters of key id for .asc <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3486>
[15:41] <mup> PR core20#100 closed: Add a wrapper for invoking the snap binary <Created by bboozzoo> <Closed by xnox> <https://github.com/snapcore/core20/pull/100>
[15:58] <pedronis> pstolowski: :/
[15:58] <pstolowski> i'm on it
[16:00] <pedronis> pstolowski: and sorry was in a meeting
[16:00] <pstolowski> sure, np
[16:17] <ijohnson> mvo: pedronis: can one of you sudo merge #10045 ? it is super simple and the failing tests there are all store related or other unrelated flakiness
[16:17] <mup> PR #10045: cmd/snap/pack: unhide the compression option <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10045>
[16:17] <mup> Bug #10045: dx: new changes from Debian require merging <Ubuntu:Fix Released by cjwatson> <https://launchpad.net/bugs/10045>
[16:23] <ijohnson> mborzecki: does #10066 also supercede #10054, or do we need both of those things?
[16:23] <mup> PR #10066: overlord/snapstate/backend: do not remove current symlink for the snapd snap <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10066>
[16:23] <mup> Bug #10066: jack-audio-connection-kit: new changes from Debian require merging <jack-audio-connection-kit (Ubuntu):Fix Released by pitti> <https://launchpad.net/bugs/10066>
[16:23] <mup> Bug #10054: gnome-gpg: new changes from Debian require merging <Ubuntu:Fix Released by cjwatson> <https://launchpad.net/bugs/10054>
[16:23] <mup> PR #10054: snapdtool, wrappers: add dependency on usr-lib-snapd.mount for services on core with snapd snap <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10054>
[16:23] <ijohnson> poor mup
[16:24] <ijohnson> mborzecki: ah sorry I meant does #10067 supercede 10054 too ?
[16:24] <mup> PR #10067: overlord/snapstate: make sure that snapd current symlink is not removed during refresh <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10067>
[16:24] <mup> Bug #10067: jigit: new changes from Debian require merging <jigit (Ubuntu):Fix Released by pitti> <https://launchpad.net/bugs/10067>
[16:30] <mup> PR snapd#10069 opened: tests: fix cgroup-tracking test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10069>
[16:33] <pedronis> ijohnson: I marked some of mborzecki PRs as blocked, I think those are superseded, should probably be closed but after we looked a bit at the last one
[16:33] <ijohnson> pedronis: I'm reviewing 10067 right now
[16:34] <ijohnson> was just wondering if I should also look at some of the other ones after I'm done with 10067 or not
[16:34] <pedronis> ijohnson: we still need 10054
[16:35] <ijohnson> ack
[16:36] <mvo> ijohnson: I will sudo merge, sorry, had some meetings but free now
[16:36] <ijohnson> no worries, thanks
[16:40] <mup> PR snapd#10045 closed: cmd/snap/pack: unhide the compression option <Simple 😃> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10045>
[16:55] <mup> PR snapd#10070 opened: tests/core/fsck-on-boot: unmount /run/mnt/snapd directly on uc20 <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10070>
[17:07] <mup> PR snapcraft#3486 closed: package-repositories: use last 8 characters of key id for .asc <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3486>
[17:09] <ijohnson> pedronis: I updated #10035 again
[17:09] <mup> PR #10035: o/devicestate: split off ensuring next boot goes to run mode into new task <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10035>
[17:09] <mup> Bug #10035: camlimages: new changes from Debian require merging <Ubuntu:Fix Released by cjwatson> <https://launchpad.net/bugs/10035>
[17:10] <mvo> a second review for 10044 would be great
[17:14] <pstolowski> mvo: +0.99999
[17:25] <mup> PR snapd#10044 closed: packaging: drop dh-systemd from build-depends on ubuntu-16.04+ <Squash-merge> <⚠ Critical> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10044>
[17:26] <mborzecki> ijohnson: https://github.com/snapcore/snapd/pull/10067 and https://github.com/snapcore/snapd/pull/10054 are hopefully the only ones we need
[17:26] <mup> PR #10067: overlord/snapstate: make sure that snapd current symlink is not removed during refresh <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10067>
[17:26] <mup> PR #10054: snapdtool, wrappers: add dependency on usr-lib-snapd.mount for services on core with snapd snap <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10054>
[17:26] <ijohnson|lunch> ack
[17:28] <pedronis> pstolowski: I reviewed #10068, some questions there
[17:28] <mup> Bug #10068: kbd-chooser: new changes from Debian require merging <kbd-chooser (Ubuntu):Fix Released by cjwatson> <https://launchpad.net/bugs/10068>
[17:28] <mup> PR #10068: [RFC] o/configstate: don't pass --root=/ when masking/unmasking/enabling/disabling services <Run nested> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10068>
[17:29] <pstolowski> pedronis: thanks!
[17:34] <pedronis> mvo: 10063 still needs your review and probably force merge
[17:49] <mvo> pedronis: sorry, crazy day, I commented 8min ago finally with a +1 and some ideas, feedback welcome
[17:50] <pedronis> pstolowski: to be clear I don't think we to change more New in that PR, but is definitely a question for follow-ups
[17:50] <mup> PR snapd#10071 opened:  packaging: drop dh-systemd from build-depends on ubuntu-16.04+ (2.49) <⚠ Critical> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10071>
[17:50] <pedronis> mvo: none of your remark are blockers though we should probably land it and do a follow up? anyway more of a question for ijohnson|lunch
[17:51] <mvo> pedronis: yeah, works for me
[17:51] <pstolowski> pedronis: ah, ok. good
[17:58] <mvo> pedronis, ijohnson|lunch merged 10063 and cherry picked to 2.49. my ideas/suggestion can be followups (I can also propose them)
[18:00] <mup> PR snapd#10063 closed: o/{device,hook}state: encode fde-setup-request key as base64 string  <Bug> <Run nested> <UC20> <⚠ Critical> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10063>
[19:20] <mup> PR snapd#10066 closed: overlord/snapstate/backend: do not remove current symlink for the snapd snap <⛔ Blocked> <Created by bboozzoo> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/10066>
[19:30] <mup> PR snapd#10072 opened: tests/lib/fde-setup-hook: also verify that fde-reveal-key key data is base64 <Run nested> <Simple 😃> <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10072>
[19:45] <mup> PR snapd#10056 closed: cmd/snap: packaging: move snap binary to /usr/lib/snapd/snap, fix invocation of internal tools <⛔ Blocked> <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/10056>
[19:59] <mup> PR core18#167 closed: static/writable-paths: make /etc/default/crda writable <Created by anonymouse64> <Merged by xnox> <https://github.com/snapcore/core18/pull/167>
[20:02] <ijohnson> thanks xnox!
[20:04] <mup> PR core18#169 closed: snapcraft.yaml: use build-base for modern snapcraft, and use lxd in travis to build the snap <Created by anonymouse64> <Merged by xnox> <https://github.com/snapcore/core18/pull/169>
[22:11] <mup> PR snapd#10073 opened: o/devicestate/devicemgr: register install-device hook, run if present in install <Needs Samuele review> <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10073>
[22:48] <mwhudson> how do i get snapcraft to make a new vm for a build?
[22:49] <mwhudson> i had a buggy install step that sprayed files all over / in the build host :)
[22:49] <mwhudson> (also running build steps as root by default is not that great an idea?)
[22:50] <mwhudson> ah clean deletes the vm?