=== 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!