| mup | PR snapd#8927 closed: tests: avoid exit 1 when nested type var is not defined <Created by sergiocazzolato> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/8927> | 00:11 |
|---|---|---|
| === jamesh_ is now known as jamesh | ||
| mup | PR snapd#8748 closed: overlord: refuse to install snaps providing user daemons on Ubuntu 14.04 <Created by jhenstridge> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/8748> | 01:36 |
| jamesh | ijohnson: thanks! | 01:40 |
| ijohnson | jamesh: np | 01:40 |
| mborzecki | morning | 05:20 |
| zyga | Hey | 06:06 |
| zyga | I’ll start at nine | 06:07 |
| mborzecki | zyga: heya, last day of 'school' | 06:07 |
| zyga | Yes. Kids are very happy | 06:07 |
| zyga | Too bad the circumstances are not better | 06:07 |
| mvo | hey zyga and mborzecki | 06:22 |
| zyga | Hey | 06:30 |
| zyga | The | 06:30 |
| seb128 | hey mvo, thx for the fontconfig and freetype fix ... do you have any hint on the easiest way to test the change? | 06:30 |
| zyga | Took meds now | 06:30 |
| zyga | Will be able to work in 30 | 06:30 |
| mvo | seb128: I can build a custom snapd snap for trying it out, will take some minutes | 06:31 |
| seb128 | mvo, that would be nice if you could, I'm happy to give it a try today | 06:32 |
| mvo | seb128: sure, building now and once done will ping you | 06:35 |
| seb128 | thx | 06:39 |
| pstolowski | morning | 07:05 |
| mvo | good morning pstolowski | 07:11 |
| mborzecki | pstolowski: heya | 07:14 |
| mborzecki | need coffee | 07:14 |
| * zyga starts with reviews | 07:25 | |
| zyga | mborzecki: btw, for evening reading: https://lwn.net/Articles/824380/ | 07:26 |
| zyga | oh and https://github.com/snapcore/snapd/pull/8728 is a low hanging fruit | 07:26 |
| mup | PR #8728: tests: detect stray dbus-daemon <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8728> | 07:26 |
| mborzecki | zyga: oh, nice | 07:26 |
| mup | PR snapd#8928 opened: tests: add spread test for disconnect undo caused by failing disconnect hook <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8928> | 07:28 |
| pstolowski | zyga: ^ something easy for a warmup ;) | 07:28 |
| jamesh | zyga: if you've got time, could you give this a review? https://github.com/snapcore/snapd/pull/8861 | 07:30 |
| mup | PR #8861: data,packaging,wrappers: extend D-Bus service activation search path <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8861> | 07:30 |
| zyga | jamesh: I'll do my best to help | 07:31 |
| jamesh | zyga: thanks. You gave it a once over earlier, and now I just need a second sign off | 07:31 |
| zyga | jamesh: ah, I really like how github shows what I reviewed | 07:34 |
| zyga | and shows only changes | 07:34 |
| pstolowski | zyga: nb, snap remove --force needs ack from Samuele | 07:37 |
| zyga | sure | 07:37 |
| zyga | I was thinking if it makes sense to fail on interface hooks when removing a snap | 07:37 |
| pstolowski | zyga: indeed | 07:41 |
| mvo | seb128: sorry, took a while, had some unclean checkout and a bug in the cleanup: https://people.canonical.com/~mvo/tmp/snapd_2.45.1+git1648.gf492c71_amd64.snap is the test snaps | 08:09 |
| zyga | mborzecki: https://github.com/snapcore/snapd/pull/8925#pullrequestreview-438094666 | 08:13 |
| mup | PR #8925: bootloader: allow managed bootloader to update its boot config <Needs Samuele review> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8925> | 08:13 |
| zyga | brb | 08:13 |
| mborzecki | zyga: thx | 08:19 |
| mborzecki | mvo: can you cherry pick https://github.com/snapcore/snapd/commit/3192141bc8481f58639e71a099b17a608366b7a0#diff-ccde24f4fb0b0adce138f9b26cea79ed to 2.45 in case we do another point release? | 08:21 |
| mvo | mborzecki: sure thing | 08:21 |
| mborzecki | mvo: thanks | 08:22 |
| mvo | mborzecki: thanks, done | 08:22 |
| mborzecki | yay | 08:22 |
| mvo | mborzecki: nice one! | 08:22 |
| zyga | mborzecki: what does NoSlashBoot do? | 08:29 |
| mborzecki | zyga: haha, so it means you're lookin at the root of the boot fs | 08:29 |
| zyga | hmm? | 08:29 |
| mborzecki | zyga: instead of the actual rootfs where you'd normally find /boot/.. | 08:30 |
| zyga | so /foo/bar/froz becomes just /? | 08:30 |
| mborzecki | iow no /boot | 08:30 |
| * zyga looks at the source | 08:30 | |
| mborzecki | zyga: not necessarily, we ahve /boot/grub/.. which is a bind mount of /EFI/ubuntu from system-boot iirc | 08:30 |
| seb128 | mvo, thx, I will give a try a bit later and let you know if it improves things | 08:50 |
| mvo | seb128: thanks | 08:53 |
| zyga | mborzecki: https://github.com/snapcore/snapd/pull/8924#pullrequestreview-438131587 | 09:20 |
| mup | PR #8924: gadget, bootloader: preserve managed boot assets during gadget updates <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8924> | 09:20 |
| zyga | more reviews | 09:32 |
| zyga | thank you for looking at the tracking PR | 09:32 |
| zyga | I will iterate on it soon | 09:32 |
| mborzecki | zyga: thanks for the reviews too | 09:37 |
| zyga | pstolowski: https://github.com/snapcore/snapd/pull/8923#pullrequestreview-438156283 | 09:42 |
| mup | PR #8923: wrappers: helper for enabling services (8/9) <Services ⚙️> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8923> | 09:42 |
| zyga | jdstrand: is https://github.com/snapcore/snapd/pull/8920 something that I should review now or is it better to wait for the final form? | 09:42 |
| mup | PR #8920: interfaces: update cups-control and add cups for providing snaps <⛔ Blocked> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8920> | 09:42 |
| zyga | jdstrand: also note that opening a draft vs a blocked PR is probably more idiomatic for GH | 09:43 |
| pstolowski | zyga: thanks! | 09:43 |
| mup | PR snapd#8929 opened: [RFC] many: add new "daemon-startup: inhibit" option <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8929> | 09:48 |
| zyga | mvo: ^ quick comment | 09:53 |
| mvo | zyga: thanks, interessting idea | 09:58 |
| zyga | mvo: it doesn't have to be complex | 09:58 |
| zyga | perhaps just | 09:58 |
| zyga | startup-presets: default: app-name: disabled | 09:58 |
| zyga | if you catch my "language" | 09:58 |
| zyga | default would be the only supported value now | 09:58 |
| zyga | presets are somewhat different, as they come from the distro | 09:59 |
| zyga | but we could convey the same idea | 09:59 |
| zyga | e.g. apache is not started automatically | 09:59 |
| zyga | but the server preset makes that so | 09:59 |
| zyga | while desktop does not | 09:59 |
| zyga | (just a made up example) | 10:00 |
| zyga | perhaps an app could even have a snapctl level influence over the preset to use | 10:00 |
| zyga | anyway, just an idea | 10:00 |
| zyga | https://www.freedesktop.org/software/systemd/man/systemd.preset.html has some docs about tihs | 10:00 |
| zyga | *this | 10:00 |
| pstolowski | #8928 needs 2nd review and is a low hanging 🍎 | 10:02 |
| mup | PR #8928: tests: add spread test for disconnect undo caused by failing disconnect hook <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8928> | 10:02 |
| zyga | pstolowski: I see what you did there :D | 10:03 |
| mup | PR snapd#8930 opened: many: managed boot config during run mode setup <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8930> | 10:03 |
| pstolowski | :) | 10:03 |
| mvo | zyga: yeah, the idea of having hooks that would enable/disable under certain conditions is part of the idea for this | 10:05 |
| zyga | ijohnson: https://github.com/snapcore/snapd/pull/8918#pullrequestreview-438161318 | 10:07 |
| mup | PR #8918: many: make nested spread tests more reliable <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8918> | 10:07 |
| zyga | review days are good | 10:08 |
| zyga | ok, only dbus to review | 10:55 |
| zyga | and then back to iteration | 10:55 |
| zyga | or maybe small lunch break and then iteratoin | 10:56 |
| zyga | *iteration | 10:56 |
| mup | PR snapd#8931 opened: tests/main/install-fontconfig-cache-gen: enhance test by verifying, add fonts to test <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8931> | 11:18 |
| zyga | jamesh: https://github.com/snapcore/snapd/pull/8861#pullrequestreview-438214407 | 11:30 |
| mup | PR #8861: data,packaging,wrappers: extend D-Bus service activation search path <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8861> | 11:30 |
| zyga | cc mvo ^ | 11:30 |
| zyga | not +1, not sure what to do about this | 11:30 |
| zyga | I think we need to revisit our reexecution strategy with regards to how it should be structured internally | 11:30 |
| zyga | what's okay and not and if we need any belts and suspenders to make it sane over time | 11:30 |
| zyga | 9% of battery, time for a trip downstairs for the charger | 11:33 |
| zyga | oh boy | 11:33 |
| jdstrand | zyga: probably best to wait for cups-control. I wasn't expected it to be blocked for so long but I got pulled away. if you're looking to review things, I suggest https://github.com/snapcore/snapd/pull/8699 and https://github.com/snapcore/snapd/pull/8301 | 11:39 |
| mup | PR #8699: interfaces/desktop-launch: support confined snaps launching other snaps <Needs Samuele review> <Needs security review> <Created by AlanGriffiths> <https://github.com/snapcore/snapd/pull/8699> | 11:39 |
| mup | PR #8301: interfaces/many: deny arbitrary desktop files and misc from /usr/share <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8301> | 11:39 |
| jdstrand | expecting* | 11:39 |
| zyga | jdstrand: ack, thanks! | 11:39 |
| jdstrand | but I am getting back to cups-control today (want to right a spread test) | 11:39 |
| zyga | I'll do both, just need to charge | 11:39 |
| jdstrand | thanks! | 11:40 |
| zyga | in the office | 11:54 |
| zyga | stairs feel wobbly | 11:54 |
| zyga | https://github.com/snapcore/snapd/pull/8728 needs a 2nd review\ | 12:17 |
| mup | PR #8728: tests: detect stray dbus-daemon <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8728> | 12:17 |
| * zyga takes a break to rest | 12:25 | |
| mup | PR snapd#8932 opened: Update security profiles after disconnecting plug-slot in connect und… <Bug> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8932> | 12:44 |
| mup | PR snapd#8933 opened: tests/core/uc20-recovery: apply hack to get gopath in recover mode w/ external backend <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8933> | 12:59 |
| mborzecki | cachio: downloaded the image, trying now | 13:33 |
| cachio | mborzecki, tx | 13:34 |
| zyga | I will resume work in 30 minutes | 13:37 |
| zyga | the schedule for painkillers is somewhat annoying | 13:38 |
| zyga | btw, typing on the faceless keyboard is really much easier than I anticipated; I think the only thing I stil struggle with are arrow keys | 13:38 |
| mborzecki | cachio: in what scenarios does the test fail? | 13:41 |
| cachio | so | 13:41 |
| cachio | just need to run the test like this | 13:41 |
| cachio | first prepare ssh | 13:42 |
| cachio | ./tests/lib/external/prepare-ssh.sh localhost 8022 sergio-j-cazzolato | 13:42 |
| cachio | with your lp id | 13:42 |
| cachio | export SPREAD_EXTERNAL_ADDRESS=localhost:8022 | 13:42 |
| cachio | and run spread -debug external:ubuntu-core-20-64:tests/core/snap-set-core-config | 13:42 |
| cachio | mborzecki, to start the vm I do | 13:43 |
| cachio | sudo qemu-system-x86_64 -smp 2 -m 2048 -snapshot -net nic,model=virtio -net user,hostfwd=tcp::8022-:22 -drive file=/usr/share/OVMF/OVMF_CODE.fd,if=pflash,format=raw,unit=0,readonly=on -drive file=~/workspace/validator/images/output/pc-amd64-20-edge/pc.img,cache=none,format=raw,id=disk1,if=none -device virtio-blk-pci,drive=disk1,bootindex=1 -machine accel=kvm | 13:43 |
| mborzecki | cachio: hmm, i've built a pc20 image and the test would fail there too from what i can see | 13:43 |
| mborzecki | wonder why it does not fail when we run it on gce | 13:43 |
| cachio | mborzecki, that's my concern | 13:43 |
| cachio | not sure where the difference is | 13:44 |
| mup | PR snapd#8728 closed: tests: detect stray dbus-daemon <Test Robustness> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8728> | 13:44 |
| cachio | I didn't see any special config for console conf in the code | 13:44 |
| ijohnson | cachio: what's the test that doesn't work ? | 13:45 |
| ijohnson | sorry I had to drop from SU before you could share details | 13:45 |
| cachio | ijohnson, snap-set-core-config | 13:45 |
| ijohnson | I'm all setup to run external spread tests easily now though :-) | 13:45 |
| ijohnson | cachio: ack I will give it a try right now | 13:45 |
| ijohnson | cachio: same pc.img.xz as yesterday ? | 13:45 |
| cachio | yes | 13:46 |
| ijohnson | cachio: much like the other test, if this is being run with an external backend, it functions differently | 13:52 |
| ijohnson | cachio: the issue is that this test tries to re-enable console-conf after disabling it, but that only works if console-conf was never run manually by the user | 13:53 |
| mborzecki | heh, 2nd thunderstorm today | 13:53 |
| ijohnson | cachio: in our case, we manually run console-conf in order to be able to create the test / ubuntu users | 13:53 |
| mborzecki | hah | 13:54 |
| ijohnson | cachio: which means that snapd does the right thing and when we unset the system config, snapd doesn't undo console-conf disabling itself | 13:54 |
| ijohnson | cachio: the test needs to be adjust to special case external backend, I will prep a PR for oyu | 13:54 |
| mborzecki | ijohnson: thanks for investigating! | 13:55 |
| cachio | ijohnson, thanks | 13:59 |
| cachio | ijohnson, everything is realted to all the magic we did to prepare snapd to be executed on google | 14:00 |
| mup | PR snapd#8934 opened: overlord: mock timings.DurationThreshold in TestNewWithGoodState <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8934> | 14:04 |
| * zyga finally had lunch and meds and can get back to work :) | 14:20 | |
| ijohnson | cachio: yes essentially all these bugs are related to the magic we do for snapd + UC on google | 14:57 |
| zyga | ijohnson: o/ | 15:03 |
| ijohnson | hey zyga | 15:04 |
| zyga | ah, you're around - great | 15:04 |
| zyga | I'll push tweaks to comments in 8829 in a moment, wanted to ask if you could look at my explanations there | 15:04 |
| zyga | I'll let you know when they are up | 15:04 |
| ijohnson | zyga: yes I'll take a look just let me know when they're up | 15:05 |
| zyga | thank you | 15:05 |
| * cachio lunch | 15:11 | |
| zyga | ijohnson: pushed https://github.com/snapcore/snapd/pull/8829 | 15:23 |
| mup | PR #8829: sandbox/cgroup: add tracking helpers <Created by zyga> <https://github.com/snapcore/snapd/pull/8829> | 15:23 |
| zyga | it's broken down in commits so you can see it more easily | 15:23 |
| zyga | https://github.com/snapcore/snapd/pull/8829/commits/6430d396deb9ad5daaced3b119b9bb1d64b413a9 | 15:23 |
| zyga | https://github.com/snapcore/snapd/pull/8829/commits/d20c0bfb6d48ce1cd650229c8d8b718ae9fe8640 | 15:23 |
| zyga | https://github.com/snapcore/snapd/pull/8829/commits/89d8a1a34426dfc94adc33147c186a2a9caa5f0a | 15:23 |
| zyga | https://github.com/snapcore/snapd/pull/8829/commits/105748396066ddd45c455b177f6accad5e670871 | 15:23 |
| zyga | that's all | 15:24 |
| zyga | if you have cycles today I'd love a review of the counterpart https://github.com/snapcore/snapd/pull/8863 | 15:24 |
| mup | PR #8863: sandbox/cgroup: allow discovering PIDs of given snap <Created by zyga> <https://github.com/snapcore/snapd/pull/8863> | 15:24 |
| ijohnson | zyga: thanks, one small comment then LGTM | 15:26 |
| ijohnson | zyga: I'll take a look at the other one today too | 15:26 |
| ijohnson | xnox: what is the canonical way to know if console-conf ran? I thought it was the presence of the /var/lib/console-conf/complete file, but it doesn't appear that file is ever written by console-conf itself ? that file simply seems to be used to _disable_ console-conf, not to tell whether console-conf ran or not | 15:45 |
| zyga | ijohnson: applied now | 15:52 |
| xnox | ijohnson: pass. I am not sure if it is possible to tell if it ran, or did anything, or if it's state was cleared, or like who/what created the managed user.... | 15:53 |
| ijohnson | xnox: I'm wondering about for our tests in snapd about disable console-conf | 15:53 |
| ijohnson | xnox: we can hack some things into place, but our test was checking for that file, but it seems that file is not actually created when we expected it to be | 15:54 |
| pstolowski | ijohnson: hey, would you take a look at #8928? it's test only PR | 15:54 |
| mup | PR #8928: tests: add spread test for disconnect undo caused by failing disconnect hook <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8928> | 15:54 |
| ijohnson | xnox: if there's not an "official way" that's fine I can replace it with a working hack to see that the test is checking the right thing | 15:54 |
| ijohnson | pstolowski: sure, will be my afternoon though at this point | 15:54 |
| pstolowski | thanks! | 15:55 |
| xnox | ijohnson: maybe something is buggy? It should be leaving that stamp file at the end... When ran successfully. At least that's what I thought. | 15:56 |
| mvo | seb128: did my updated snapd snap help with the fc-cache issue? no rush, just curious if I can update anything in the PR | 15:56 |
| ijohnson | xnox: it doesn't leave a stamp file on uc20 or uc18 afaict | 15:56 |
| ijohnson | xnox: shall I file a bug ? | 15:56 |
| ijohnson | zyga: thanks for adding that Error(), I'll keep an eye on the PR and merge it for you tonight | 15:57 |
| seb128 | mvo, ah, sorry, I meant to comment, I just tested half an hour ago, I wiped the cache, installed your snap and the font was correctly listed in the newly generated cache, so seems to work from that testing | 15:58 |
| zyga | thanks | 15:58 |
| mvo | seb128: \o/ yay | 15:58 |
| zyga | once it lands I need to merge it back into to big helper | 15:58 |
| seb128 | mvo, thanks for the quick fix :) | 15:58 |
| mvo | seb128: my pleasure, thanks for all your help with this! (and to jamesh) | 15:58 |
| ijohnson | mvo: regarding that bug, how will the spread tests get the fix for that? do we build the fc-cache helpers in the snapd deb the way we do for the snapd snap? just wondering how we could properly fix my spread test showing that it works | 16:01 |
| seb128 | mvo, I've added a comment on the bug and the pr now | 16:01 |
| ijohnson | cachio: do you have time today to verify #8933 ? | 16:02 |
| mup | PR #8933: tests/core/uc20-recovery: apply hack to get gopath in recover mode w/ external backend <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8933> | 16:02 |
| mvo | ijohnson: I think we just merge the fix into edge, rebuild the snap and then re-run your test (that now no longer needed to rebuild the snapd snap). it's not as nice as you test but we can always revert again if needed | 16:03 |
| mvo | ijohnson: sorry, it's very low-tech, a real friday evening suggestion :) | 16:03 |
| ijohnson | mvo: so the test will use the snapd snap ? | 16:03 |
| ijohnson | mvo: that's fine, just wondering since it's a little annoying in the test to have to revert from the snapd snap to the deb, certainly doable though I think | 16:03 |
| mvo | ijohnson: yeah, once fc-cache-builder is in git and the snapd snap is rebuild we should have the right version | 16:03 |
| ijohnson | mvo: ah actually all the font tests are run on classic, so it's not a problem, the magic spread restore should take care of it | 16:04 |
| ijohnson | nvm me, thanks mvo | 16:04 |
| mvo | ijohnson: aha, ok | 16:04 |
| mvo | ijohnson: sorry, I think I was not even grasping the depth of the question :) | 16:04 |
| mvo | ijohnson: but happy to help! | 16:04 |
| ijohnson | mvo: haha no worries it's probably EOW time for you anyways :-) | 16:04 |
| mvo | ijohnson: it is - plus it's hot here :) | 16:05 |
| ijohnson | yah I hear that | 16:05 |
| * mvo is good at making excuses | 16:05 | |
| * mvo will just write one or two more mails and then call it a day | 16:05 | |
| ijohnson | xnox: seems that /run/console-conf/login-details* is the best indicator if console-conf ran successfully / normally on the system | 16:10 |
| xnox | ijohnson: but that could also be generated if the system got provisioned elsehow too I think | 16:13 |
| ijohnson | xnox: I don't see it, I tried both with cloud-init and with system user assertions and don't see that dir | 16:13 |
| xnox | Interesting. | 16:13 |
| ijohnson | xnox: maybe I checked in bad ways | 16:13 |
| xnox | And after reboot it's still there? | 16:13 |
| ijohnson | xnox: but I can make my test pass now, so that's all that's important :-) | 16:13 |
| ijohnson | xnox: afaict yes | 16:13 |
| xnox | Cool. Roll with it. | 16:14 |
| xnox | Next time I touch consoleconf I will try to understand how it works. | 16:14 |
| ijohnson | haha sounds good | 16:14 |
| === benfrancis1 is now known as benfrancis | ||
| cachio | ijohnson, sure | 16:45 |
| cachio | ijohnson, https://paste.ubuntu.com/p/RvVKN4978X/ | 17:50 |
| cachio | https://paste.ubuntu.com/p/JfCz4D3Rfz/ | 17:51 |
| ijohnson | cachio: very interesting, was this when the system was in recover mode or run mode ? | 17:51 |
| cachio | https://paste.ubuntu.com/p/hsNtwZNYsw/ | 17:51 |
| cachio | no space error | 17:51 |
| ijohnson | cachio: ah I see that is in recover mode | 17:51 |
| === benfrancis0 is now known as benfrancis | ||
| === benfrancis4 is now known as benfrancis | ||
| ijohnson | cachio: also re: the snap-set-core-config, the test exposed a real bug, so I will need to fix that bug before we can fix the test | 18:16 |
| cachio | ijohnson, hehe | 18:33 |
| cachio | good news in that case | 18:34 |
| ijohnson | cachio: yeah :-) | 18:34 |
| ijohnson | cachio: also for your uc20-recovery is that snapd panic reproducible for you ? | 18:34 |
| ijohnson | or did it just happen once, as I don't see that with my VM | 18:34 |
| cachio | yes | 18:36 |
| cachio | the once I sent before? | 18:36 |
| ijohnson | cachio: the one from https://paste.ubuntu.com/p/hsNtwZNYsw/ | 18:36 |
| ijohnson | cachio: what arguments are you running the external VM with ? | 18:37 |
| cachio | yes | 18:37 |
| cachio | I have a debug session opened | 18:37 |
| ijohnson | cachio: I use this: | 18:37 |
| ijohnson | https://www.irccloud.com/pastebin/DDPDyNii/ | 18:37 |
| ijohnson | maybe I need to provide less RAM to the VM | 18:38 |
| cachio | ijohnson, I use https://paste.ubuntu.com/p/mCyynrpCpQ/ | 18:38 |
| cachio | 2048 | 18:38 |
| cachio | big diff | 18:38 |
| ijohnson | cachio: ack I'll try with 2048 to see if I can reproduce it | 18:39 |
| ijohnson | cause yeah we might have run out of space, but we shouldn't panic in that case I'm pretty sure | 18:39 |
| cachio | I'll try with 3gb | 18:39 |
| mup | PR snapd#8829 closed: sandbox/cgroup: add tracking helpers <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8829> | 18:40 |
| cachio | ijohnson, with 3GB didn't work | 19:25 |
| cachio | trying with 4GB now | 19:25 |
| cachio | got stuck after reboot | 19:25 |
| ijohnson | cachio: you mean with 3 GB it didn't panic but it did get stuck after reboot? | 19:28 |
| cachio | dont know because after 20 minutes I killed spread | 19:28 |
| cachio | https://paste.ubuntu.com/p/JC4kDVGCWV/ | 19:28 |
| cachio | ijohnson, this is the output | 19:29 |
| ijohnson | ah cachio that seems to be the bug that I thought I fixed by deleting the ~/.ssh/rc file | 19:32 |
| ijohnson | cachio: interesting, I need to step out for a while now, but I will try again to see if I can reproduce that issue | 19:32 |
| * ijohnson -> afk for bit | 19:32 | |
| cachio | ijohnson, np, see you then | 19:34 |
| cachio | ijohnson, with 4GB workerd well | 19:38 |
| cachio | ijohnson, https://paste.ubuntu.com/p/PDt2jk3W2f/ | 19:39 |
| cachio | the problem is that in the lab devices we don't have 4gb | 19:53 |
| mup | PR snapcraft#3190 opened: extensions: export content snap egl vendor dir <maintenance> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3190> | 19:59 |
| ijohnson | cachio: yes I will see what to do about using 2G, also won't this test be run on devices like rpi that might only have 1G of RAM? | 20:01 |
| cachio | ijohnson, yes | 20:25 |
| cachio | it runs in pi3 with 1 or 2 gb | 20:25 |
| cachio | and pi4 with 4/8gb | 20:26 |
| cachio | which is fine | 20:26 |
| mup | PR snapcraft#3174 closed: link_or_copy: do not try to create hardlinks to symlinks <bug> <Created by hpoul> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3174> | 21:09 |
| mup | PR snapcraft#3190 closed: extensions: export content snap egl vendor dir <maintenance> <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3190> | 21:09 |
| mup | PR snapcraft#3191 opened: plugins: introduce v1.FlutterPlugin <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3191> | 22:04 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!