| === jamesh_ is now known as jamesh | ||
| mborzecki | morning | 07:12 |
|---|---|---|
| mup | PR snapd#9921 closed: boot: helper for setting up a try recover system <UC20> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9921> | 07:46 |
| zyga | good morning | 07:49 |
| pstolowski | morning | 08:04 |
| mborzecki | pstolowski: mvo: hey | 08:05 |
| mvo | good morning pstolowski and mborzecki ! happy monday | 08:08 |
| pstolowski | o/ | 08:11 |
| zyga | good morning mvo :) | 08:18 |
| mvo | good morning zyga, happy monday to you as well! | 08:29 |
| zyga | going downstairs to set up some recording, I'll be back in the afternoon | 08:36 |
| pstolowski | oh well, snapshot tests mock too much... | 09:18 |
| mborzecki | anyone seen this? https://paste.ubuntu.com/p/Vb9jzgR5Zx/ | 09:57 |
| mborzecki | mvo: ^^ looks like core-initrd was changed and our kernel repacking is broken now | 10:16 |
| mvo | mborzecki: it was changed, yes | 10:16 |
| mvo | mborzecki: check https://github.com/snapcore/core-initrd/commits/main | 10:17 |
| mborzecki | that's probably the changes for having more modules in the server builds | 10:17 |
| mborzecki | mvo: yeah, i suspect it's this commit: https://github.com/snapcore/core-initrd/commit/9ca7436f821f0e2d37874fd217407275f1e24e3c | 10:17 |
| mvo | mborzecki: yeah | 10:18 |
| mup | PR snapd#10008 opened: tests/lib/prepare: fix repacking of the UC20 kernel snap for with ubuntu-core-initramfs 40 <Run nested> <Simple 😃> <⚠Critical> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10008> | 10:37 |
| mborzecki | mvo: ^^ | 10:39 |
| mvo | mborzecki: thanks, checking | 10:45 |
| mvo | mborzecki: I cced you on a question about appamror confinement partil vs full, I think I got the right answer but if it looks strange let me know (no need to deep dive) | 10:46 |
| mvo | mborzecki: \o/ thanks for the fix! | 10:47 |
| mborzecki | mvo: it looks correct | 10:59 |
| mup | PR snapcraft#3464 closed: meta: add support for `kernel.yaml` for kernel snaps <Created by mvo5> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3464> | 11:13 |
| mup | PR snapd#9994 closed: tests: improve tests documentation - part 2 <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9994> | 11:37 |
| mborzecki | mvo: pushed a couple more fixes to #10008 | 11:49 |
| mup | Bug #10008: Automatic configuration of wireless pcmcia card does not work <pcmcia-cs (Ubuntu):Invalid> <https://launchpad.net/bugs/10008> | 11:49 |
| mup | PR #10008: tests/lib/prepare: fix repacking of the UC20 kernel snap for with ubuntu-core-initramfs 40 <Run nested> <Simple 😃> <⚠Critical> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10008> | 11:49 |
| mup | PR snapcraft#3465 opened: [backport] store: improve platform detection (#2931) <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3465> | 12:13 |
| pstolowski | mvo: #10007 is ready for reviews | 12:19 |
| mup | Bug #10007: HD drive very slow when no cd inserted whith kernel 2.6 and Gnome 2.6 or 2.8 <linux-source-2.6.15 (Ubuntu):Invalid> <https://launchpad.net/bugs/10007> | 12:19 |
| mup | PR #10007: o/configstate, o/snapshotstate: fix handling of nil snap config on snapshot restore <Bug> <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10007> | 12:19 |
| ijohnson | morning folks | 12:35 |
| diddledan | o/ | 12:36 |
| ijohnson | hey diddledan | 12:36 |
| diddledan | the snap store is being a pain (has been for days) https://usercontent.irccloud-cdn.com/file/Jf25WrzK/image.png | 12:37 |
| ijohnson | yeah entirely possible, we had to push out a new snapd/core snap which has a tendency to slow down things, hopefully this week it is better though | 12:38 |
| diddledan | I'm sure my snap being 2.5 GB isn't helping | 12:38 |
| diddledan | (CUDA libraries are huge!) | 12:39 |
| ijohnson | oh yeah that definitely is a problem, I seem to recall that snaps over 1 GB are very finnicky to upload | 12:39 |
| diddledan | tis a good job I don't have a data cap with the number of times I've got to 99% | 12:46 |
| ijohnson | haha yeah that would really sting if you had a data upload cap too | 12:47 |
| ijohnson | mvo: not sure when I mentioned it last week, but for example today I see we only have one single PR running via github actions workers: https://github.com/snapcore/snapd/actions?query=is%3Ain_progress but many, many queued jobs | 12:47 |
| ijohnson | I think we have some github actions self hosted runners offline | 12:47 |
| diddledan | self-hosted runners for the win though (I have one here at home ;-)) | 12:49 |
| ijohnson | yeah they are great except when you realize you now are doing devops :-| | 12:49 |
| diddledan | true | 12:50 |
| diddledan | <insert yo dawg, I heard you like devops meme here> | 12:50 |
| ijohnson | haha | 12:51 |
| mborzecki | ijohnson: yeah, not sure how to even see how many runners are up at this point | 13:13 |
| mborzecki | mvo: can you see it? | 13:13 |
| ogra | diddledan, you should try to find that norwegian guy (peer) that resets your connection and tell him to stop that ! | 13:13 |
| ijohnson | mborzecki: yeah real annoying with that u-c-i bug you found this morning too | 13:13 |
| mborzecki | ijohnson: and it's stuck in the actions queue ;) | 13:13 |
| diddledan | peer is an ass | 13:13 |
| ogra | totally ! | 13:13 |
| cachio_ | ijohnson, hi | 13:18 |
| cachio_ | ijohnson, did you see this error= https://github.com/snapcore/snapd/pull/9997/checks?check_run_id=2055837552#step:5:547 | 13:19 |
| mup | PR #9997: tests: update details on task.yaml files <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9997> | 13:19 |
| ijohnson | hey cachio_ let me have a look | 13:19 |
| cachio_ | it started on friday I think | 13:20 |
| ijohnson | cachio_: that is the issue that mborzecki is fixing, but we are stuck waiting for github actions workers to run this PR: https://github.com/snapcore/snapd/pull/10008 | 13:20 |
| mup | PR #10008: tests/lib/prepare: fix repacking of the UC20 kernel snap for with ubuntu-core-initramfs 40 <Run nested> <Simple 😃> <⚠Critical> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10008> | 13:20 |
| cachio_ | I already fixed the github actions agents | 13:20 |
| cachio_ | mvo, hi, is any of the agents offline? | 13:20 |
| cachio_ | ijohnson, ah, nice | 13:21 |
| cachio_ | I'll check again the workers | 13:22 |
| cachio_ | I fixed that on friday | 13:22 |
| ijohnson | thanks cachio | 13:23 |
| mvo | cachio_: I see a lot of prodstack offline, eg prodstack-a-45 to 76 (some isolted idle ones in their though) | 13:32 |
| cachio_ | mvo, thnaks | 13:33 |
| cachio_ | qc | 13:33 |
| mvo | ijohnson, mborzecki I can see some stat about the runners but not super many, I see a good bunch of stuff online but also quite a few offline | 13:34 |
| ijohnson | mmm that's unfortunate | 13:34 |
| zyga | hey guys | 13:36 |
| zyga | oh, workers down? | 13:37 |
| ijohnson | something like that | 13:37 |
| zyga | I learned one interesting thing the other day | 13:37 |
| zyga | gitlab can run CI for projects that are hosted on githu | 13:38 |
| zyga | *github | 13:38 |
| zyga | so you can use the somewhat more powerful pipeline system and easily host your workers (and scale them) | 13:38 |
| zyga | as the provisioning process is a bit better there | 13:38 |
| zyga | you could even auto-scale with juju | 13:38 |
| ijohnson | yeah maybe gitlab is nice, but I doubt we have any time to switch | 13:55 |
| diddledan | a downside of gitlab-ci is there is no inbuilt concept of a `matrix` | 13:56 |
| zyga | diddledan that's true but I don't see that as a major flaw or limitation | 14:00 |
| zyga | you can easily create that with .extends | 14:00 |
| zyga | I think that having a native matrix would be nice to unclutter more complex pipelines | 14:00 |
| zyga | but I'm still learning that, and there are parts I did not explore | 14:01 |
| zyga | like pipeline triggering and subscription, that allow for even more composition | 14:01 |
| diddledan | I like the ability to include/import pipeline stuff from external files/repos | 14:02 |
| diddledan | that really helps when you have multiple projects that all need the same or similar pipelines | 14:02 |
| zyga | yeah, that's true | 14:04 |
| zyga | salsa has a very nice and featureful pipeline for packaging | 14:04 |
| zyga | in my current work I'm setting up reusable pipelines that are easy to include into various repos | 14:04 |
| zyga | it's not far off from github actions | 14:05 |
| zyga | but actions are not so composable as you can only define one step, and not a larger chunk | 14:05 |
| === cachio_ is now known as cachio | ||
| * cachio lunch | 16:10 | |
| cachio | ijohnson, hey, about the PR #9960 do you have any update? | 17:55 |
| mup | PR #9960: tests: update permission denied message for test-snapd-event on ubuntu 2104 <â›” Blocked> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9960> | 17:55 |
| cachio | should we skip that test? | 17:55 |
| ijohnson | cachio: no not yet, but we should not skip that test on 21.04 | 17:55 |
| ijohnson | cachio: is that test blocking beta certification of 2.49.1 ? | 17:55 |
| cachio | ijohnson, no | 17:56 |
| ijohnson | yeah let's leave the test as-is for now, I will check in on it today after lunch | 17:56 |
| cachio | as 21.04 is in red | 17:56 |
| cachio | I want to know what to do with that one | 17:56 |
| cachio | ijohnson, I'll wait in that case, thanks | 17:56 |
| diddledan | popey: if I register cuda-runtime-11-2-1804, would it be ok to organise to add it to snapcrafters with a view to making it a content-snap for anybody to use? | 18:39 |
| diddledan | (I'm trying to work around the store bailing out because my snap is huge) | 18:40 |
| diddledan | well I registered it anywho :-p | 18:42 |
| cachio | ijohnson, hi | 19:49 |
| cachio | quick question | 19:49 |
| ijohnson | hey cachio | 19:49 |
| cachio | I install the snap test-snapd-sh and then I remove it | 19:49 |
| cachio | it is ok if: | 19:50 |
| cachio | ls /sys/fs/cgroup/devices/snap.test-snapd-sh.sh/ | 19:50 |
| cachio | cgroup.clone_children cgroup.procs devices.allow devices.deny devices.list notify_on_release tasks | 19:50 |
| cachio | it remains there? | 19:50 |
| ijohnson | cachio: that's a bug where we don't remove/cleanup the device cgroup for a snap after it's removed | 19:51 |
| ijohnson | I filed that a while ago, let me see if I can find the bug | 19:51 |
| ijohnson | cachio: see https://bugs.launchpad.net/snapd/+bug/1803210 | 19:52 |
| mup | Bug #1803210: snap's device cgroup is not discarded upon uninstall <snapd:Confirmed for zyga> <https://launchpad.net/bugs/1803210> | 19:52 |
| cachio | ijohnson, ok, because I have a test failing and not sure if it could be related to that | 19:52 |
| ijohnson | cachio: how is the test failing? | 19:53 |
| cachio | ijohnson, https://paste.ubuntu.com/p/nWQS6DnZFd/ | 19:53 |
| cachio | don't know if the test clean up is bad | 19:54 |
| ijohnson | cachio: yeah we probably need that test to clean itself up | 19:54 |
| cachio | or something is not working properly when we remove a snap | 19:54 |
| mup | PR snapcraft#3466 opened: store: do not unnecessarily catch/rewrite exceptions <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3466> | 21:58 |
| mup | PR snapd#10009 opened: boot: export bootAssetsMap as AssetsMap <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10009> | 22:55 |
| mup | PR snapd#10010 opened: boot/assets.go: add InstallObserverOptions to allow providing previous assets <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10010> | 22:55 |
| mup | PR snapcraft#3467 opened: gitignore: sort list <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3467> | 23:58 |
| diddledan | well fooey | 23:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
