| mup | PR snapd#9139 opened: interfaces/many: miscellaneous updates for strict microk8s <Created by jdstrand> <https://github.com/snapcore/snapd/pull/9139> | 00:39 |
|---|---|---|
| mup | PR snapd#9140 opened: vendor: update github.com/kr/pretty to fix diffs of values with pointer cycles <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/9140> | 04:00 |
| mborzecki | morning | 05:59 |
| mborzecki | mvo: hey | 06:27 |
| mvo | good morning mborzecki | 06:29 |
| mborzecki | hm the forum gained a canonical branded bar at the top of the page | 06:38 |
| oerheks | and a roll down 'products' button, nice | 06:40 |
| === oerheks is now known as DT_oerheks | ||
| === DT_oerheks is now known as oerheks | ||
| * mvo hugs jamesh for pr9140 | 06:48 | |
| jamesh | :-) Having tests that don't hang when they fail is nice | 06:49 |
| mvo | jamesh: +100 for that | 06:49 |
| mup | PR snapd#9140 closed: vendor: update github.com/kr/pretty to fix diffs of values with pointer cycles <Simple 😃> <Created by jhenstridge> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9140> | 07:00 |
| pstolowski | morning | 07:03 |
| mvo | good morning pstolowski | 07:06 |
| mvo | pstolowski: great news, the PR for kr/pretty that prevents circular data structure hangs got merged upstream and james did a PR to update our vendor.json | 07:07 |
| pstolowski | mvo: whaaaaat? unbelievable | 07:07 |
| pstolowski | that's great | 07:07 |
| mvo | pstolowski: yeah :) | 07:08 |
| mvo | pstolowski: https://github.com/kr/pretty/pull/64 | 07:08 |
| mup | PR kr/pretty#64: diff: detect cycles in the values being compared <Created by jhenstridge> <Merged by kr> <https://github.com/kr/pretty/pull/64> | 07:08 |
| pstolowski | kinda annoying my PR was hanging that for 1,5y with no traction, oh well | 07:12 |
| jamesh | pstolowski: I hadn't seen yours when I wrote mine | 07:12 |
| pstolowski | jamesh: no worries, good it's fixed | 07:13 |
| jamesh | I noticed the maintainer had merged something at end of July, so pinged him in my PR and got lucky. | 07:14 |
| mvo | slightly sad that we had to duplicate some work but the outcome is really good, this bug was super annoying in the past. I'm very happy | 07:14 |
| pstolowski | yes | 07:14 |
| jamesh | my version also detects if you have different structure to the cycles in the two values being compared too | 07:15 |
| jamesh | e.g. A->A compares different to A->B->A | 07:15 |
| pstolowski | jamesh: ah, nice | 07:25 |
| zyga | good morning | 07:48 |
| zyga | mvo could you please advise on https://github.com/snapcore/snapd/pull/9125 | 07:48 |
| mup | PR #9125: tests: fix issues related to restarting systemd-logind.service <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9125> | 07:48 |
| zyga | it makes master much more gree | 07:48 |
| zyga | *green | 07:48 |
| zyga | and it is green itself | 07:48 |
| zyga | there are two separate fixes there | 07:48 |
| zyga | I can open two new PRs that won't be green but we can review separately and force-merge | 07:49 |
| zyga | or I can do something else, up to you | 07:49 |
| mvo | zyga: new ones are fine | 07:51 |
| zyga | mvo ack, I'll make that happen | 07:51 |
| zyga-mbp | the first fix is https://github.com/snapcore/snapd/pull/9141 | 08:07 |
| mup | PR #9141: tests: cope with ghost cgroupv2 <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9141> | 08:07 |
| zyga-mbp | mborzecki ^ mvo ^ (it will require force merging) | 08:07 |
| zyga-mbp | this is a low-priority fix https://github.com/snapcore/snapd/pull/9142 | 08:10 |
| mup | PR #9142: sandbox/cgroup: detect dangling v2 cgroup <Bug> <Created by zyga> <https://github.com/snapcore/snapd/pull/9142> | 08:10 |
| mup | PR snapd#9141 opened: tests: cope with ghost cgroupv2 <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9141> | 08:11 |
| mup | PR snapd#9142 opened: sandbox/cgroup: detect dangling v2 cgroup <Bug> <Created by zyga> <https://github.com/snapcore/snapd/pull/9142> | 08:11 |
| mvo | zyga-mbp: looking now | 08:13 |
| mvo | zyga-mbp: 9125 is still marked as draft, should that get promoted | 08:14 |
| zyga-mbp | not yet, I'm removing those commits and adding one more there | 08:14 |
| zyga-mbp | I'll open it in a moment | 08:14 |
| zyga-mbp | mvo https://github.com/snapcore/snapd/pull/9125 is now force-pushed to remove the commits split out to the two new branches | 08:16 |
| mup | PR #9125: tests: fix issues related to restarting systemd-logind.service <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9125> | 08:16 |
| zyga-mbp | and is now open | 08:16 |
| mvo | zyga-mbp: one comment, thanks for this one | 08:20 |
| mvo | zyga-mbp: also remarkable complicated :( | 08:20 |
| mvo | zyga-mbp: again, thanks for diving so deep | 08:22 |
| zyga-mbp | looking | 08:26 |
| zyga-mbp | mvo which comment should I look like? | 08:27 |
| zyga-mbp | look at | 08:27 |
| zyga-mbp | sorry | 08:27 |
| zyga-mbp | I EODd late last night, should make 2nd coffee | 08:27 |
| zyga-mbp | I will make coffee and review https://github.com/snapcore/snapd/pull/8960 | 08:29 |
| mup | PR #8960: o/snapstate,servicestate: use service-control task for service actions (9/9) <Needs Samuele review> <Services ⚙️> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8960> | 08:29 |
| zyga-mbp | back with freshly ground coffee :) | 08:39 |
| zyga-mbp | and into the review from pawel | 08:39 |
| shuduo | hi, i'm working on to package a cmake-based project to snap format. I refer mosquitto since they are similar since need both daemon and utils. mosquitto works well. but mine does not. I see snapcraft do not put file cmake built to stage and prime. I guess the CMakeLists.txt are something non-standard style. but I wonder if there is a way to workaround. thanks. | 09:25 |
| zyga | shuduo I'd ask this question in #snapcraft in a few hours | 09:32 |
| shuduo | zyga, or i ask there? | 09:33 |
| zyga | yeah, ask there but wait for a few hours till the developers are all there | 09:33 |
| shuduo | OK | 09:33 |
| shuduo | thanks | 09:33 |
| pedronis | zyga: do we still need #9125 with #9141 ? or do we need both? | 09:49 |
| mup | PR #9125: tests: fix issues related to restarting systemd-logind.service <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9125> | 09:49 |
| mup | PR #9141: tests: cope with ghost cgroupv2 <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9141> | 09:49 |
| zyga | pedronis we need both, I separated them for review per mvo's advice | 09:50 |
| pedronis | zyga: I'm not talking about #9142 | 09:50 |
| mup | PR #9142: sandbox/cgroup: detect dangling v2 cgroup <Bug> <Created by zyga> <https://github.com/snapcore/snapd/pull/9142> | 09:50 |
| zyga | yes, we should get all three but 9142 is less important - we are immune by lucky coincidence | 09:51 |
| mup | PR snapd#9136 closed: bootstate20: rm bootState20Modeenv, pass around modeenv directly <Cleanup :broom:> <⛔ Blocked> <Created by anonymouse64> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/9136> | 09:51 |
| zyga | so 9142 can land later | 09:51 |
| pedronis | ok | 09:51 |
| mup | PR snapd#9134 closed: boot, o/devicestate: TrustedAssetUpdateObserver stubs, hook up to gadget updates <UC20> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9134> | 09:56 |
| zyga | https://github.com/snapcore/snapd/pull/9141 needs a 2nd review and force-merge | 09:58 |
| mup | PR #9141: tests: cope with ghost cgroupv2 <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9141> | 09:58 |
| zyga | so does https://github.com/snapcore/snapd/pull/9125 | 09:58 |
| mup | PR #9125: tests: fix issues related to restarting systemd-logind.service <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9125> | 09:58 |
| zyga | mborzecki, pstolowski: ^ if you have a moment | 09:58 |
| ijohnson | morning folks | 10:04 |
| pedronis | mvo: a remark about "snap recovery", it seems to produce errors like this: error: cannot list recovery systems: cannot list recovery systems: access denied | 10:07 |
| pedronis | we should find out what's the nesting cause, and remove it | 10:07 |
| mvo | pedronis: uh, I will add a test, thanks for noticing | 10:09 |
| pedronis | I don't know if one comes from snapd and one from the client or something else | 10:10 |
| zyga | brb | 10:10 |
| mvo | zyga: hey, with your systemd experience, how do you feel about changing our written mount units to have "[Install]\nWantedBy=local-fs.target" ? | 10:10 |
| zyga | hmmmm, I don't know yet | 10:11 |
| zyga | let me think and check and get back to you | 10:11 |
| mvo | no worries | 10:11 |
| pedronis | mvo: also a comment about "snap reboot" and related: https://github.com/snapcore/snapd/pull/9021#issuecomment-672783319 | 10:13 |
| mup | PR #9021: snap: implement new `snap reboot` command <Created by mvo5> <https://github.com/snapcore/snapd/pull/9021> | 10:13 |
| mvo | pedronis: thanks | 10:14 |
| mup | PR snapd#9143 opened: snap: fix repeated "cannot list recovery system" and add test <Created by mvo5> <https://github.com/snapcore/snapd/pull/9143> | 10:26 |
| pedronis | mvo: should we close #9110 ? | 10:29 |
| mup | PR #9110: preseed: cherry-pick #8704, #8709, #9088 (2.45) <Created by mvo5> <https://github.com/snapcore/snapd/pull/9110> | 10:29 |
| mvo | pedronis: yes, let me do that | 10:33 |
| mup | PR snapd#9110 closed: preseed: cherry-pick #8704, #8709, #9088 (2.45) <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/9110> | 10:36 |
| ijohnson | zyga: did you take a look at the arch failure on #9125 ? very odd | 10:44 |
| mup | PR #9125: tests: fix issues related to restarting systemd-logind.service <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9125> | 10:44 |
| zyga | ijohnson I did not yet | 10:44 |
| zyga | looking | 10:44 |
| ijohnson | but I just re+1'd that PR for you | 10:44 |
| ijohnson | would be great to sudo land that | 10:44 |
| zyga | yeah | 10:44 |
| zyga | checking log now | 10:44 |
| ijohnson | I saved it here anyways https://pastebin.ubuntu.com/p/QZBFr9QshT/ | 10:45 |
| ijohnson | I can't reproduce on ubuntu, but I've also never seen this test fail before | 10:45 |
| zyga | hmm | 10:45 |
| zyga | my first hunch was enoent -> it didn't mount | 10:45 |
| zyga | it's not the dynamic linker for sure | 10:46 |
| zyga | but it's mounted | 10:46 |
| zyga | there are a few other things we run that would fail this way | 10:46 |
| zyga | too bad it doesn't say _which_ exec fail | 10:46 |
| zyga | *failed | 10:46 |
| zyga | let me look at the source quickly | 10:46 |
| zyga | god this 16" is so much better than the x240 | 10:47 |
| zyga | almost comfortable in bed | 10:47 |
| zyga | damn, that's snap-exec | 10:47 |
| zyga | snap-confine failed to execute snap-exec | 10:48 |
| ijohnson | oh weird | 10:48 |
| ijohnson | how could that happen | 10:48 |
| zyga | no idea yet | 10:54 |
| zyga | actually | 10:54 |
| zyga | all snaps are mounted | 10:54 |
| zyga | so no, no idea | 10:54 |
| zyga | mvo could you please force-merge https://github.com/snapcore/snapd/pull/9125 | 11:19 |
| mup | PR #9125: tests: fix issues related to restarting systemd-logind.service <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9125> | 11:19 |
| zyga | https://github.com/snapcore/snapd/pull/9141 needs a 2nd review | 11:19 |
| mup | PR #9141: tests: cope with ghost cgroupv2 <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9141> | 11:19 |
| mvo | zyga: sure | 11:19 |
| zyga | and we should be back on our feet | 11:19 |
| zyga | thank you | 11:20 |
| mup | PR snapd#9125 closed: tests: fix issues related to restarting systemd-logind.service <Test Robustness> <⚠ Critical> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9125> | 11:22 |
| cachio | zyga, hi | 11:53 |
| zyga | hey | 11:53 |
| mup | PR snapcraft#3245 opened: specifications: minor cleanup for package-repositories <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3245> | 11:55 |
| cachio | zyga, did you see the message I left in the standaup doc= | 12:01 |
| zyga | yes, but I did not investigate yet | 12:01 |
| cachio | nice, thnaks | 12:01 |
| cachio | I'll investigate a bit now | 12:02 |
| zyga | cachio it may go away with the fixes that are incoming | 12:02 |
| zyga | the one that just landed | 12:02 |
| zyga | I don't have spread2 around, can you pull master and see if it happens now? | 12:02 |
| cachio | zyga, sure | 12:03 |
| zyga | thank you | 12:03 |
| zyga | I need a moment, back in 30 | 12:03 |
| cachio | zyga, it passed now | 12:16 |
| mup | PR snapd#9144 opened: bootloader: add helper for creating a bootloader based on gadget <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9144> | 12:22 |
| mborzecki | pedronis: ^^ | 12:22 |
| zyga | cachio I think the fix for linger affected it | 12:31 |
| zyga | as that test - the failure cgroup one, otherwise nuked systemd --user for root | 12:31 |
| cachio | zyga, good to know, thanks for that!! | 12:31 |
| ijohnson | cachio: sorry I don't think I followed up but do we have a problem with the uc20-recovery test ? | 12:31 |
| cachio | ijohnson, sorry, yes, it failed yesterday on the pi3 when I did beta validation for 2.46 | 12:32 |
| ijohnson | cachio: do you have logs? | 12:32 |
| cachio | and then the debug is showing the same problem than before | 12:32 |
| cachio | ijohnson, yes, 1 sec | 12:32 |
| ijohnson | thanks | 12:32 |
| zyga | mvo could you please merge https://github.com/snapcore/snapd/pull/9141 | 12:33 |
| mup | PR #9141: tests: cope with ghost cgroupv2 <Test Robustness> <⚠ Critical> <Created by zyga> <https://github.com/snapcore/snapd/pull/9141> | 12:33 |
| zyga | we should be good now | 12:33 |
| ijohnson | nice good work zyga! | 12:33 |
| zyga | we can start restarting tests, just a few at first, to see where we are | 12:33 |
| zyga | nothing like bad red master to spoil monday | 12:33 |
| zyga | it's Wed so let's just keep pushing now | 12:33 |
| pedronis | mborzecki: reviewed #9144 | 12:34 |
| mup | PR #9144: bootloader: add helper for creating a bootloader based on gadget <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9144> | 12:34 |
| mup | PR snapcraft#3242 closed: tests: migrate legacy classic patchelf tests to spread <enhancement> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3242> | 12:35 |
| cachio | ijohnson, https://paste.ubuntu.com/p/kXWV6tQDcN/ | 12:37 |
| ijohnson | thanks cachio I'll have a look | 12:37 |
| cachio | tx | 12:37 |
| mborzecki | pedronis: btw. if you want to look, i've made the changes for install boot config right here: https://github.com/bboozzoo/snapd/commit/9b684dfadba34afd7199891ba8bfe0d6ec56b1d6 mostly works, there's one case for uboot that ijohnson probably knows more about | 12:39 |
| * ijohnson goes to go hide from uboot bootloader details | 12:39 | |
| ijohnson | cachio: ah very interesting | 12:41 |
| ijohnson | cachio: so part of what happened is that we failed to finish seeding in recover mode there | 12:41 |
| ijohnson | cachio: then we hit that rather weird/interesting bug with the recursive bind mount output from findmnt which broke cwayne's server | 12:42 |
| pedronis | mborzecki: if I remember that should be fine, is some test failing? | 12:43 |
| pedronis | the logic checks for .conf in any case, there is just differen behavior if .conf is empty | 12:43 |
| pedronis | vs not | 12:43 |
| pedronis | for uboot | 12:43 |
| cachio | ijohnson, yes | 12:45 |
| cachio | ijohnson, I think it is not breaking the server anymore | 12:46 |
| ijohnson | that's nice | 12:46 |
| cachio | because now I see that the output at some point is cut | 12:46 |
| cachio | so they implemented something in their side for that | 12:46 |
| mup | PR snapd#9141 closed: tests: cope with ghost cgroupv2 <Test Robustness> <⚠ Critical> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9141> | 12:52 |
| zyga | thank you | 12:52 |
| mup | PR snapcraft#3246 opened: plugins v2: quote python packages argument for pip <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3246> | 13:31 |
| mup | PR snapcraft#3245 closed: specifications: minor cleanup for package-repositories <specification> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3245> | 13:46 |
| zyga | hmm | 13:58 |
| zyga | + test-snapd-busybox-static.busybox-static echo hello | 13:58 |
| zyga | execv failed: No such file or directory | 13:58 |
| zyga | grep error: pattern not found, got: | 13:58 |
| zyga | this failed again | 13:58 |
| zyga | mborzecki does test-snapd-busybox-static work for you on arch? | 13:59 |
| mborzecki | zyga: i have a fix, just a sec | 13:59 |
| zyga | oh even better | 13:59 |
| zyga | thanks | 13:59 |
| ijohnson | ah so that is a real issue | 14:05 |
| mup | PR snapd#9145 opened: boot: track trusted assets during initial install, assets cache <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9145> | 14:07 |
| mup | PR snapd#9146 opened: packaging/arch: use external linker when building statically <Simple 😃> <Test Robustness> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9146> | 14:12 |
| ijohnson | mborzecki: ^ is needed to make master all green again right ? | 14:13 |
| ijohnson | arch is required and that test is always failing right ? | 14:13 |
| mborzecki | ijohnson: yes | 14:15 |
| zyga | thanks | 14:16 |
| mborzecki | got to go for now | 14:17 |
| zyga | not fun | 14:17 |
| zyga | thank you | 14:17 |
| pedronis | pstolowski: some comments in #9084 | 14:45 |
| mup | PR #9084: o/snapstate: check disk space before creating automatic snapshot on remove (3/N) <Disk space awareness> <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9084> | 14:45 |
| pstolowski | pedronis: ty | 14:45 |
| * cachio lunch | 14:59 | |
| ijohnson | cachio: does that uc20-recovery issue happen reliably such that I could reproduce it with a live rpi3 ? | 15:25 |
| mup | PR core#117 opened: extrafiles/writable-paths: make /etc/default/crda writable <Created by anonymouse64> <https://github.com/snapcore/core/pull/117> | 15:49 |
| mup | PR snapd#9146 closed: packaging/arch: use external linker when building statically <Simple 😃> <Test Robustness> <⚠ Critical> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9146> | 15:53 |
| ijohnson | pedronis: so I'm looking right now at transitioning from recover to run automatically via any reboot, and we currently detect in devicestate Manager that we are in recover mode via the modeenv, but regardless of how we set that up, at a minimum we need to set bootenv, but we will also need to set the modeenv too | 15:54 |
| ijohnson | the issue is that we use the modeenv to determine what mode we are in from devicestate | 15:54 |
| ijohnson | so I'm thinking that a pre-req to this is to have devicestate decide what mode it is in by 1) just presence of a modeenv file, and 2) decide what mode via kernel command line | 15:55 |
| pedronis | ijohnson: why that? for ephemeral mode the modeenv is ephemeral | 15:55 |
| ijohnson | oh wait | 15:55 |
| pedronis | I'm probably missing something | 15:55 |
| pedronis | the run mode modeenv has always run in it | 15:55 |
| pedronis | if that's not the case we have a bug | 15:56 |
| ijohnson | ok, but even if it's ephemeral/tmpfs modeenv, we still write it as "mode=recover" from the initramfs there | 15:56 |
| ijohnson | ah but nevermind the modeenv doesn't matter | 15:56 |
| ijohnson | nvm me the bootenv is all we care about | 15:56 |
| pedronis | yes, but those modeenv go away each time | 15:56 |
| ijohnson | yes yes yes I had just confused myself | 15:56 |
| ijohnson | all makes sense now | 15:56 |
| pedronis | it's ok, I was worried we had a bug | 15:56 |
| ijohnson | well fear not we do have a bug ... points at bugs.launchpad.net/snapd :-D | 15:57 |
| ijohnson | just not in this particular area | 15:57 |
| cachio | ijohnson, I could reproduce it twice | 16:01 |
| ijohnson | cachio: ack I will try to reproduce it myself with a debug shell I can poke around with | 16:02 |
| cachio | not sure if 100% of the times | 16:02 |
| cachio | nice | 16:02 |
| cachio | ijohnson, this is theimage | 16:02 |
| cachio | ijohnson, https://storage.googleapis.com/spread-snapd-tests/images/pi3-20-beta/pi.img.xz | 16:03 |
| ijohnson | cachio: nice thanks, do you use tf or do you flash it to your local pi3 ? | 16:03 |
| cachio | ijohnson, sudo dd if=<PAT_TO_IMG> of=/dev/mmcblk0 bs=4M oflag=sync status=noxfer | 16:04 |
| ijohnson | thanks cachio | 16:05 |
| cachio | ijohnson, yaw | 16:05 |
| pedronis | ijohnson: you should try to look #9145 if possible, I looked at it, I think it's mostly good, but there are some implicit assumptions that I find a bit confusing (especially they way they might or not affect the update case) | 16:13 |
| mup | PR #9145: boot: track trusted assets during initial install, assets cache <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9145> | 16:13 |
| ijohnson | pedronis: sure | 16:14 |
| pedronis | thx | 16:14 |
| ijohnson | pedronis: have you seen https://forum.snapcraft.io/t/module-blacklisting-interface/19171 or is it on your queue ? | 16:17 |
| pedronis | I can put it on my queue, but I will not get to it for a while | 16:22 |
| ijohnson | pedronis: that's fine, just wondering if you had it on your queue already | 16:29 |
| mup | PR core18#155 closed: hooks: fix broken symlink /etc/sysctl.conf.d/99-sysctl.conf <Created by mvo5> <Merged by sil2100> <https://github.com/snapcore/core18/pull/155> | 16:38 |
| mup | PR snapcraft#3240 closed: cli: only issue warning when checking for usage of sudo <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3240> | 17:51 |
| mup | PR snapd#9086 closed: many: reorg cmd/snapinfo.go into snap and new client/clientutil <Cleanup :broom:> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9086> | 17:53 |
| zyga | re | 17:57 |
| * zyga jumps into customer bug analysis | 18:00 | |
| mvo | mountedfilesystemwriter tests are happy with the new resolvedSrc stuff I'm working on, yay | 18:06 |
| zyga | mvo I won't make the morning desktop call tomorrow, I had to move my rehab from Friday to Thursday 9:00 AM | 18:07 |
| mup | PR snapcraft#3232 closed: build providers: install apt-transport-https <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3232> | 18:11 |
| mvo | zyga: ok | 18:14 |
| mvo | zyga: I think you mean physio? | 18:14 |
| zyga | I really don't know how it is translated | 18:14 |
| zyga | :D | 18:15 |
| zyga | I think 'rehabilitation' is also valid but I could be wrong | 18:15 |
| zyga | it's not the kind of rehab where rich people try not to kill themselves with alcohol and drugs | 18:15 |
| mvo | zyga: I'm probably wrong (no native speaker etc) but when I hear rehab I usually think of alcohl or something like this | 18:15 |
| mvo | zyga: maybe some native speaker can enlighten us :) | 18:16 |
| mvo | zyga: anyway I know what you mean | 18:16 |
| zyga | it's the kind where they teach you to slowly make progress and get back to shape | 18:16 |
| zyga | plus a lot of the time is actually spent on the scar itself, weird but supposedly important so that it doesn't hurt later | 18:16 |
| zyga | aaaanyw | 18:16 |
| zyga | I can show you my scar but it's in the same spot like before, just colored differently, guess that's a poor luck in body scar trophy ;) | 18:17 |
| zyga | hmm | 18:17 |
| zyga | Aug 12 14:32:53 localhost snapd[1507]: udevmon.go:149: udev event error: Unable to parse uevent, err: cannot parse libudev event: invalid env data | 18:17 |
| zyga | we may need to update our udev go code | 18:17 |
| zyga | and improve error handling so that we can at least get a dump of the event in hex | 18:18 |
| zyga | ijohnson ^ how would you describe what we were talking about above? | 18:18 |
| zyga | (which word would you use?) | 18:18 |
| mup | PR snapd#9147 opened: [RFC] Support for gadget.yaml "$kernel:" style references (2/N) <Created by mvo5> <https://github.com/snapcore/snapd/pull/9147> | 18:18 |
| === lfaraone_ is now known as lfaraone | ||
| ijohnson | zyga: sorry I lost some context | 18:22 |
| zyga | ijohnson no worries, not important, just wondering how to say something in english | 18:22 |
| ijohnson | Oh you mean the appointment you have? | 18:22 |
| zyga | is "rehabilitation" understood as more than avoiding booze/drugs? | 18:22 |
| zyga | yes | 18:22 |
| zyga | like special exercises to get back to basic mobility | 18:22 |
| ijohnson | In th US we would call it "physical therapy" or just PT | 18:23 |
| zyga | aaah | 18:23 |
| zyga | thanks, | 18:23 |
| zyga | PT it is | 18:23 |
| zyga | though I'm sure Graham would call it something else ;-) | 18:23 |
| ijohnson | But that's just for recovering back to where you were if it's like something from birth or a long standing thing you do for a really long time then we call it "occupational therapy" | 18:23 |
| ijohnson | zyga haha yeah that's why I prefaced with "in the US" | 18:23 |
| zyga | ah, more terms I never used before | 18:24 |
| zyga | I wonder how brits call that actually | 18:24 |
| zyga | it's such a cool aspect of our work that we have all those people from different places in one team :) | 18:24 |
| mvo | zyga: we call it physio | 18:26 |
| mvo | ijohnson: I was wondering if "going to rehab" would be somethign that is more about drugs or more about physio | 18:26 |
| mvo | ijohnson: this is how this started | 18:26 |
| mvo | ijohnson: anyway, not *really* important :) | 18:27 |
| zyga | back to musl | 18:28 |
| ijohnson | mvo yeah rehab usually has that connotation in casual usage but I've heard it used in medical setting as well but it's less common than calling it physical therapy | 18:29 |
| mvo | thanks! | 18:29 |
| * mvo feels smarter now :) | 18:29 | |
| ijohnson | np I agree with zyga it's really interesting to see how these things are called around the world | 18:30 |
| mvo | ijohnson: totally! | 18:30 |
| mvo | I think I call it a day, one more small victor on kernel-dtb, now I quickly need to call it a day before I see spread failures | 18:31 |
| * mvo waves | 18:31 | |
| zyga | o/ | 18:35 |
| * cachio -> kinesiologist | 18:46 | |
| ijohnson | cachio: aroud ? | 20:23 |
| ijohnson | *around | 20:24 |
| cachio | ijohnson, yes | 20:24 |
| ijohnson | cachio: what is the command to setup an external system so I can use it with spread ? | 20:24 |
| ijohnson | something from nested.sh I seem to remember... | 20:24 |
| cachio | you need to create the users? | 20:24 |
| cachio | or the env var to set ? | 20:25 |
| ijohnson | I have a rpi3 I want to run the external uc20 recovery test with | 20:25 |
| cachio | ok, in that case | 20:25 |
| cachio | you need | 20:26 |
| ijohnson | (and I have a locally created user) | 20:26 |
| cachio | ./tests/lib/external/prepare-ssh.sh <ip> 22 <your-lp-user> | 20:26 |
| cachio | this will create the users | 20:26 |
| cachio | then | 20:27 |
| cachio | export SPREAD_EXTERNAL_ADDRESS=<IP> | 20:27 |
| ijohnson | awesome thanks | 20:28 |
| cachio | then spread external:ubuntu-core-20-arm-32:tests/core/uc20-recovery | 20:28 |
| cachio | ijohnson, yaw | 20:28 |
| ijohnson | cachio: btw whatever happened to the tf backend for spread ? | 20:30 |
| ijohnson | was that completed but never merged ? | 20:30 |
| cachio | ijohnson, needs reviews | 20:30 |
| cachio | and approval | 20:30 |
| ijohnson | cachio: I see, would be good if we could get that in sometime soon | 20:31 |
| cachio | I already asked for reviewes and approval | 20:31 |
| mup | PR snapcraft#3246 closed: plugins v2: quote python packages argument for pip <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3246> | 20:36 |
| ijohnson | cachio: I reproduced the failure with the rpi3 I am using, investigating now | 20:51 |
| ijohnson | interestingly, our new `snap debug seeding` returns `seed-completion: 3195h26m10.26s` on this system hehe | 20:51 |
| cachio | ijohnson, nice | 20:51 |
| cachio | hehe | 20:51 |
| cachio | the customers wont be happy waiting that time | 20:52 |
| ijohnson | haha no I don't think so | 20:57 |
| ijohnson | I think I have an idea what's going on here | 20:57 |
| ijohnson | this happens specifically on the pi where we don't have a real time clock | 20:57 |
| cachio | ijohnson, in pi4 didn't fail | 21:23 |
| ijohnson | cachio: right because pi3 runs slower than pi4 | 21:23 |
| cachio | is it the same case? | 21:23 |
| cachio | ijohnson, ahh | 21:24 |
| ijohnson | cachio: no, pi4 is faster | 21:24 |
| cachio | makes sense | 21:24 |
| mup | PR snapd#8942 closed: tests: support different images on nested execution <Run nested> <Squash-merge> <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8942> | 22:19 |
| cachio | vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv88888888888888888 | 23:47 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!