| mborzecki | morning | 05:35 |
|---|---|---|
| pstolowski | morning | 07:07 |
| zyga | good morning | 07:08 |
| mvo | good morning pstolowski and zyga | 07:12 |
| zyga | hey, how is the sprint progressing? | 07:13 |
| mvo | zyga: good so far | 07:17 |
| 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:18 |
| mborzecki | hello guys | 07:22 |
| === tomwardill_ is now known as tomwardill | ||
| 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:29 |
| 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:31 |
| mvo | good morning mborzecki | 07:32 |
| 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:33 |
| zyga | mvo: are you the only one who can update strace static? | 07:34 |
| 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:38 |
| mborzecki | wish there was something in go runtime/stdlib to tell you that | 07:39 |
| zyga | meh, I see | 07:39 |
| 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 | 07:47 |
| pedronis | jamesh: yes, I'll try to look at it today | 08:01 |
| jamesh | pedronis: cheers. | 08:02 |
| zyga | hmm | 08:19 |
| zyga | brb, small break | 09:35 |
| zyga | re | 09:51 |
| 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:09 |
| 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:30 |
| 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:33 |
| 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:34 |
| ijohnson | probably they should be added to core20, are they in core18? | 11:35 |
| * 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:36 |
| mborzecki | heh, was suppsoed to do bug triage today | 11:40 |
| zyga | mborzecki: https://forum.snapcraft.io/t/injecting-snapd-tools-into-base-snaps-and-keeping-them-up-to-date/12139/8 | 11:43 |
| zyga | can you tell me if this makes sense to you? | 11:45 |
| 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:46 |
| zyga | small break and then back to coding | 11:50 |
| zyga | and maybe time for meds | 11:50 |
| zyga | eh | 11:50 |
| === Trevinho_ is now known as Trevinho | ||
| zyga | ok, 50C now, no longer burning my legs | 13:19 |
| * pstolowski lunch | 13:24 | |
| zyga_ | mborzecki do you have time to review snap run changes today? | 13:29 |
| === zyga_ is now known as zyga-mbp | ||
| mborzecki | zyga-mbp: i can certainly try | 13:37 |
| zyga-mbp | please :) | 13:37 |
| zyga-mbp | I would move a lot with that | 13:37 |
| 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> | 14:35 |
| * zyga-mbp reboots to tweak the kernel | 15:58 | |
| zyga-mbp | brb | 15:58 |
| zyga-mbp | well... the vm that is | 15:59 |
| 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:01 |
| 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:06 |
| zyga-mbp | done | 16:07 |
| * zyga-mbp -> coffee | 16:25 | |
| 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> | 20:47 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!