[05:35] <mborzecki> morning
[07:07] <pstolowski> morning
[07:08] <zyga> good morning
[07:12] <mvo> good morning pstolowski and zyga
[07:13] <zyga> hey, how is the sprint progressing?
[07:17] <mvo> zyga: good so far
[07:18] <mup> PR snapd#8990 closed: systemd/escape: fix issues with "" and "\t" handling <Bug> <Simple 😃> <Test Robustness> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8990>
[07:22] <mborzecki> hello guys
[07:29] <mup> PR snapd#8988 closed: many: use more specific check for unit test mocking <Bug> <Simple 😃> <Test Robustness> <Created by anonymouse64> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8988>
[07:29] <pstolowski> o/
[07:31] <mborzecki> it'd be so nice if go had a way to check whether the binary is runing as part of go test invocation
[07:32] <mvo> good morning mborzecki
[07:33] <zyga> snap run --strace does not support snap aliases
[07:33] <zyga> aliased snaps are just ignored
[07:33] <zyga> and strace static is out of date and needs updates
[07:34] <zyga> mvo: are you the only one who can update strace static?
[07:38] <zyga> mborzecki: we have that
[07:38] <mborzecki> zyga: see #8988
[07:38] <mup> PR #8988: many: use more specific check for unit test mocking <Bug> <Simple 😃> <Test Robustness> <Created by anonymouse64> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8988>
[07:38] <zyga> ah
[07:38] <mborzecki> we have something, but it's not necessarily 100% correct ;)
[07:39] <mborzecki> wish there was something in go runtime/stdlib to tell you that
[07:39] <zyga> meh, I see
[07:47] <jamesh> pedronis: I know you're busy at the sprint, but I'd appreciate some comment on https://forum.snapcraft.io/t/proposal-new-rest-api-for-theme-installation/18732 -- particularly the authentication section.
[07:47] <jamesh> if you have time at some point, any feedback would be welcome
[08:01] <pedronis> jamesh: yes, I'll try to look at it today
[08:02] <jamesh> pedronis: cheers.
[08:19] <zyga> hmm
[09:35] <zyga> brb, small break
[09:51] <zyga> re
[11:09] <mup> PR snapd#8992 opened: boot: better naming of helpers for obtaining kernel command line <Simple 😃> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8992>
[11:30] <mup> PR snapd#8993 opened: bootloader, boot: compose kernel command line passed by grub bootloader <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8993>
[11:33] <mborzecki> meh, no grub-editenv means i need to reboot, go to grub cmdline, load_env myself, edit, save_env, reboot
[11:33] <ijohnson> mborzecki: I thought there was an issue for adding those to core20 snap ?
[11:34] <ijohnson> mborzecki: did they not get added to the core20 snap?
[11:34] <mborzecki> ijohnson: i don't see it on my $PATH
[11:34] <mborzecki> actually not in the snap either
[11:34] <ijohnson> ah yeah that's unfortunate
[11:35] <ijohnson> probably they should be added to core20, are they in core18?
[11:36]  * mborzecki checks quickly
[11:36] <mborzecki> ijohnson: nope, only in core
[11:36] <ijohnson> ah interesting
[11:36] <ijohnson> perhaps it was intentionally left out of core18 then
[11:40] <mborzecki> heh, was suppsoed to do bug triage today
[11:43] <zyga> mborzecki: https://forum.snapcraft.io/t/injecting-snapd-tools-into-base-snaps-and-keeping-them-up-to-date/12139/8
[11:45] <zyga> can you tell me if this makes sense to you?
[11:46] <zyga> hmm, some problems with parallel instances https://forum.snapcraft.io/t/parallel-instances-cant-remove-cant-connect-interfaces-cant-use-layouts-env-vars-are-wrong/18696
[11:50] <zyga> small break and then back to coding
[11:50] <zyga> and maybe time for meds
[11:50] <zyga> eh
[13:19] <zyga> ok, 50C now, no longer burning my legs
[13:24]  * pstolowski lunch
[13:29] <zyga_> mborzecki do you have time to review snap run changes today?
[13:37] <mborzecki> zyga-mbp: i can certainly try
[13:37] <zyga-mbp> please :)
[13:37] <zyga-mbp> I would move a lot with that
[14:35] <mup> PR snapd#8992 closed: boot: better naming of helpers for obtaining kernel command line <Simple 😃> <UC20> <Created by bboozzoo> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/8992>
[15:58]  * zyga-mbp reboots to tweak the kernel
[15:58] <zyga-mbp> brb
[15:59] <zyga-mbp> well... the vm that is
[16:01] <mup> PR snapcraft#3208 closed: snap: set PATH for snapcraft command <bug> <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3208>
[16:06] <mup> PR snapd#8994 opened: tests: fix some snapstate tests to use pointers for snapmgrTestSuite <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8994>
[16:06] <pstolowski> zyga-mbp: ^ can you tak a look, it's trivial
[16:06] <pstolowski> ?
[16:06] <zyga-mbp> sure
[16:06] <zyga-mbp> looking now
[16:06] <pstolowski> ty
[16:07] <zyga-mbp> done
[16:25]  * zyga-mbp -> coffee
[20:47] <mup> PR snapd#8994 closed: tests: fix some snapstate tests to use pointers for snapmgrTestSuite <Simple 😃> <Created by stolowski> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/8994>