[05:55] <mardy> 'morning!
[05:59] <mborzecki> morning
[06:27] <pstolowski> morning
[06:59] <zyga-mbp> good morning :)
[07:18] <mvo> good morning zyga-mbp 
[07:31] <mborzecki> mvo: mardy: can you take a look at https://github.com/snapcore/snapd/pull/10744 ?
[07:31] <mup> PR #10744: overlord/ifacestate: fix arguments in unit tests <⚠ Critical> <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10744>
[07:33] <mup> PR snapd#10744 opened: overlord/ifacestate: fix arguments in unit tests <⚠ Critical> <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10744>
[07:34] <zyga-mbp> hey mvo  :)
[07:34] <mardy> mborzecki: +1
[07:34] <mardy> hi zyga-mbp, pstolowski, mvo :-)
[07:34] <mborzecki> mardy: thanks!
[07:35]  * zyga-mbp declares this a hot-tea season
[07:35] <zyga-mbp> cold coffee is out
[07:37] <mborzecki> zyga-mbp: haha you just let your coffee cool down for too long 🙂
[07:38] <zyga-mbp> I prefer my coffee cold but given that it's <18 in the office and I'm fully dressed my preference is secondary
[07:41] <mvo> good morning mardy mborzecki pstolowski 
[07:41] <mborzecki> yeah, the mornings are cold
[07:42] <mborzecki> mvo: hey
[07:42] <mvo> mborzecki: sure, looking - fwiw it looks like debian-sid is broken right now because apparmor 3 hit unstable
[07:42] <mvo> looks like we need our snapd-apparmor-service there now too :)
[07:42] <pstolowski> hey
[07:43] <mborzecki> uhh
[07:43] <mborzecki> fun
[07:50] <mardy> is there any way to get regexps to work on multiline strings? (assuming that this is the reason for this failure: https://paste.ubuntu.com/p/d4gXhNGbBM/)
[07:51] <mborzecki> mardy: you want `(?m)` in the pattern then
[07:52] <mborzecki> quick errand, back in 30
[08:09] <mborzecki> re
[08:13] <mup> PR snapd#10744 closed: overlord/ifacestate: fix arguments in unit tests <⚠ Critical> <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10744>
[08:33] <mup> PR snapd#10745 opened: packaging: ship the `snapd.apparmor.service` unit in debian <Created by mvo5> <https://github.com/snapcore/snapd/pull/10745>
[09:17] <mborzecki> mvo: hmm related to sid pulling in apparmor 3? https://forum.snapcraft.io/t/snap-apps-not-opening-after-update/26446/2 iirc kali was pulling in some packages from sid
[09:23] <mup> PR snapd#10746 opened: spread: bump delta ref to 2.52 <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10746>
[09:24] <mborzecki> trivial one ^^
[10:01] <mborzecki> errand, back in 30
[10:21] <mborzecki> re
[10:42] <mborzecki> hehe, so the MemoryMax in systemd service overrides goes out of sync with what's really set in memory controller in 16.04
[10:43] <mborzecki> override file was removed, service restarted, systems claims no limit, but /sys/fs/cgroup/memory shows otherwise 😕 https://paste.ubuntu.com/p/KCPKvxRQdR/
[10:44] <mborzecki> could this also affect our snap quotas?
[10:44] <mborzecki> mvo: ^^
[10:46] <mvo> mborzecki: uh, maybe/probably, where do you see this?
[10:48] <mborzecki> mvo: on 16.04 in my memory limit PR https://github.com/snapcore/snapd/pull/10241 which consistently keeps failing there although the limit it supposed to be disabled in those specific tests
[10:48] <mup> PR #10241: tests: set memory limit for snapd <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10241>
[10:55] <mvo> ta
[10:58] <mup> PR snapd#10747 opened: packaging: remove duplicated `golang-go` build-dependency <Simple 😃> <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10747>
[11:28] <mup> PR snapd#10748 opened: systemd: add AtLeast() method, add mocking in systemdtest <Created by mardy> <https://github.com/snapcore/snapd/pull/10748>
[12:19] <mup> PR snapd#10749 opened: tests: remove travis leftovers <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10749>
[12:49] <mup> PR snapd#10750 opened: cmd/snap: don't translate service statuses in 'snap services' output <Simple 😃> <Bug> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10750>
[12:59] <mup> PR snapd#10736 closed: tests: skip overlord tests on riscv64 due to timeouts <Simple 😃> <Skip spread> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/10736>
[13:06] <mborzecki> mvo: can you land https://github.com/snapcore/snapd/pull/10545 ?
[13:06] <mup> PR #10545: boot: record recovery capable systems in recovery bootenv <Needs Samuele review> <Remodel 🚋> <Run nested> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10545>
[13:44] <mvo> mborzecki: sure
[13:45] <mborzecki> mvo: thanks
[13:49] <mup> PR snapd#10545 closed: boot: record recovery capable systems in recovery bootenv <Needs Samuele review> <Remodel 🚋> <Run nested> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10545>
[14:34] <mup> PR snapd#10745 closed: packaging: ship the `snapd.apparmor.service` unit in debian <cherry-picked> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10745>
[14:34] <mup> PR snapd#10747 closed: packaging: remove duplicated `golang-go` build-dependency <Simple 😃> <Skip spread> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10747>
[14:51] <pstolowski> mvo: is https://bugs.launchpad.net/snapd/+bug/1940553 fixed and released? or just fixed?
[14:51] <mup> Bug #1940553: Wrong dtb loaded on rpi <snapd:In Progress> <https://launchpad.net/bugs/1940553>
[14:53] <ijohnson[m]> pstolowski: technically the fix is "released" in 2.51.7, the release is awaiting testing before released to stable however
[14:54] <ijohnson[m]> pstolowski: I would leave it as committed until 2.51.7 is actually released to stable
[14:54] <pstolowski> ijohnson[m]: ack, thanks
[14:56] <pstolowski> ijohnson[m]: that was released right https://bugs.launchpad.net/snapd/+bug/1910298 ?
[14:56] <mup> Bug #1910298: ~/snap directory should be o0700 <Snapcraft:Triaged by sergiusens> <snapd:In Progress by anonymouse67> <https://launchpad.net/bugs/1910298>
[14:57]  * mvo switches network
[15:39] <mup> PR snapd#10751 opened: client: turn the Client type into an interface <Created by mardy> <https://github.com/snapcore/snapd/pull/10751>
[15:59] <mup> PR snapd#10752 opened: configcore: implement netplan write support via dbus <Created by mvo5> <https://github.com/snapcore/snapd/pull/10752>
[16:09] <cachio> ijohnson[m], hi, I answered your comment https://github.com/snapcore/snapd/pull/10743#discussion_r703576248
[16:09] <mup> PR #10743: tests: fix fakedevicesvc service already exists <Simple 😃> <Run nested> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10743>
[16:24] <mup> PR snapd#10746 closed: spread: bump delta ref to 2.52 <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10746>
[16:24] <cachio> ijohnson[m], I already improved the solution to allow nested tests to use tests.cleanup
[16:25] <cachio> because I saw that other tests are using it as well
[16:25] <cachio> so it is better to allow to use that tool
[17:35]  * cachio afk
[17:56] <ijohnson[m]> thanks cachio 
[19:45] <mup> PR snapd#10749 closed: tests: remove travis leftovers <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10749>
=== clobrano8 is now known as clobrano