| mborzecki | morning | 05:43 |
|---|---|---|
| _moep_ | good morning mborzecki | 05:49 |
| mborzecki | _moep_: morning | 05:50 |
| pstolowski | morning | 06:04 |
| mborzecki | pstolowski: hey | 06:44 |
| zyga | good morning | 06:58 |
| mborzecki | mardy is on vacation right? | 07:32 |
| mborzecki | meh, i need to run some errands today :/ | 07:42 |
| pstolowski | mborzecki: yeah | 08:17 |
| mborzecki | re | 08:38 |
| _moep_ | wb | 08:42 |
| pstolowski | damn, services are complicated | 09:19 |
| mborzecki | pstolowski: haha damn right :) | 09:31 |
| mborzecki | zyga: btw. i've updated the bpf program to support matching any minor number in device cgroup (similar to eg. `c 1:*` to allow all char devs with major 1): https://github.com/bboozzoo/snapd/commit/ad3ad5c76cf1535d9554dc88abc1f60377b73539 turned out to be quite easy actually once the verifier was happy | 09:32 |
| zyga | mborzecki, nice :) | 09:44 |
| zyga | mborzecki, how close is full v2 support? | 09:44 |
| mborzecki | zyga: a branch with some refactoring is up | 09:45 |
| pstolowski | hmm error: cannot refresh "lxd": unexpectedly empty response from the server , it's the 2nd time i see this today on github | 10:42 |
| mborzecki | errand, back in 1h | 10:53 |
| ijohnson[m] | morning folks | 11:23 |
| ijohnson[m] | pstolowski: hey thanks for #10551, so is that PR sufficient to fix the failing spread test I proposed ? | 11:23 |
| ijohnson[m] | or is that PR a pre-req to be able to fix the failing spread test | 11:23 |
| pstolowski | ijohnson[m]: hey | 11:24 |
| pstolowski | ijohnson[m]: i think it needs snapctl side fix, but need to think a bit more. if it passes then it may be by luck | 11:26 |
| ijohnson[m] | interesting, I would have thought that since snapctl just creates service-control tasks that it would fix it | 11:27 |
| pstolowski | ijohnson[m]: it gets tricky wrt to explicit services vs disabled services | 11:28 |
| ijohnson[m] | ah I see | 11:28 |
| pstolowski | ijohnson[m]: if snapctl passes foo (as opposed to snapname.foo) it won't get onto explicit services list afaiu. but i need to double check, this stuff is tricky | 11:29 |
| ijohnson[m] | indeed it is tricky | 11:29 |
| pstolowski | and everything being strings makes it harder | 11:29 |
| ijohnson[m] | I'll give this PR a review now and then I'll try to review your auto-refresh one today I started a bit yesterday | 11:29 |
| pstolowski | thanks | 11:30 |
| * pstolowski lunch | 11:30 | |
| pstolowski | ijohnson[m]: see also my comments under mardy's pr | 11:30 |
| ijohnson[m] | ack | 11:30 |
| mborzecki | re | 11:44 |
| mborzecki | pstolowski: quick thought, maybe we should have separate types for those strings? | 13:10 |
| pstolowski | mborzecki: yeah, i was moaning about that in the su notes | 13:11 |
| pstolowski | mborzecki: i think that's not for now, but would be nice when we do further changes wrt to pedronis' plan | 13:12 |
| * cachio lunch | 15:13 | |
| === futuretim_ is now known as admin | ||
| === admin is now known as futuretim_ | ||
| === futuretim_ is now known as futuretim | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!