mup | PR snapd#8774 closed: tests: disable test of nfs v3 with udp proto on debian-sid <⚠ Critical> <Created by stolowski> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/8774> | 01:24 |
---|---|---|
=== KindTwo is now known as KindOne | ||
mborzecki | morning | 05:21 |
mborzecki | hmm https://forum.snapcraft.io/t/linux-mint-20-will-forbid-snapd-from-installing-and-snaps-growing-external-adoption-problem/17913 | 06:36 |
mborzecki | and https://blog.linuxmint.com/?p=3906 | 06:37 |
mborzecki | errand, back in 30 or so | 06:50 |
pstolowski | morning | 07:01 |
zyga | o/ | 07:07 |
mborzecki | re | 07:36 |
mborzecki | zyga: pstolowski: hey | 07:36 |
zyga | o/ | 07:37 |
mborzecki | mvo wrote he's going to start later today | 07:37 |
zyga | yeah, I noticed | 07:37 |
mborzecki | zyga: seen the bad press? :) | 07:37 |
zyga | mborzecki: it's not bad press | 07:48 |
zyga | (when a competitor makes a press release, it's not press, it's just a statement) | 07:48 |
zyga | I wish them good luck on maintaining chromium as a deb | 07:49 |
mborzecki | zyga: from what i understand they won't maintain a deb | 07:49 |
mborzecki | zyga: but rather complain about snaps each time one tries to install chromium? | 07:49 |
zyga | heh | 07:50 |
pedronis | interesting | 07:50 |
mborzecki | i guess a blog post is less work than packaging chromium | 07:50 |
zyga | the blog post clearly states commercial inteterests | 07:52 |
zyga | "it becomes a threat" | 07:52 |
pedronis | pstolowski: unit tests failing on #8567 | 07:56 |
mup | PR #8567: o/devicestate: core20 early config from gadget defaults <UC20> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8567> | 07:56 |
pstolowski | pedronis: ah, thanks, weird, go test ./... didn't run those | 07:57 |
mborzecki | in the context of https://forum.snapcraft.io/t/snapd-cannot-run-daemon-cannot-read-state-unexpected-eof/17908/2 should we keep a backup of state.json? | 08:59 |
mborzecki | last known good state is likely better than scrubbing all of it | 09:00 |
mborzecki | we'd need to carefully select which scenarios restoring from backup could be useful | 09:02 |
pstolowski | yes, would be good to avoid a single point of failure like this | 09:05 |
mup | PR snapd#8776 opened: bootloader: rename test helpers to reflect we are mocking EFI boot locations <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8776> | 09:06 |
pstolowski | i think we scrapped any plans to migrate from json to a db? | 09:06 |
mborzecki | trivial PR ^^ | 09:06 |
mborzecki | pstolowski: i don't see it in trello, so probably longer term goal? | 09:07 |
mborzecki | pedronis: do you remember whether we dropped json -> db? | 09:08 |
pedronis | I don't thinkd dropped it the right way to put it, we don't have any immediate plan, for sure not this cycle | 09:09 |
pedronis | it's also not clear we have a db choice | 09:09 |
mborzecki | diddledan: hi, pushed a little fix for the openttd snap https://github.com/diddlesnaps/openttd/pull/2 | 09:36 |
mup | PR diddlesnaps/openttd#2: snap/gui: fix desktop file Exec <Created by bboozzoo> <https://github.com/diddlesnaps/openttd/pull/2> | 09:36 |
zyga | drat | 10:00 |
zyga | my thinkpad keeps shutting down all of a sudden | 10:00 |
zyga | stuff like that always happens when you look at new thinkpads | 10:00 |
mborzecki | zyga: anything in journal? | 10:01 |
zyga | 4 times today | 10:01 |
mborzecki | or dmesg? maybe a hardware problem? | 10:01 |
zyga | mborzecki: it doesn't even reset, it just goes "off" | 10:01 |
zyga | https://www.irccloud.com/pastebin/sHPeZkTd/ | 10:02 |
zyga | but that's not even the last thing | 10:02 |
zyga | just stood out of the oridinary | 10:02 |
zyga | hmmm | 10:15 |
zyga | I'm seeing more and more failures from restarting journald | 10:15 |
zyga | https://www.irccloud.com/pastebin/W6kTcLLE/ | 10:15 |
mup | PR snapd#8776 closed: bootloader: rename test helpers to reflect we are mocking EFI boot locations <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/8776> | 10:16 |
ijohnson | morning folks | 10:16 |
mborzecki | ijohnson: hey, thanks for merging the PR | 10:17 |
ijohnson | hey mborzecki | 10:17 |
zyga | ijohnson: good morning! | 10:23 |
ijohnson | hey zyga | 10:23 |
ijohnson | how are you doing this morning | 10:23 |
zyga | heh, I was about to tell mvo about my health | 10:23 |
zyga | better, not well, painkillers work reliably now | 10:23 |
ijohnson | well that's good you're feeling better than yesterday | 10:24 |
zyga | I really need to see a doctor but I want to stay clear of any hospital for the next few weeks, at least | 10:24 |
zyga | it's so annoying, my left leg is heavily impacted by this | 10:24 |
zyga | I mostly work from bed, apart from standups and other calls | 10:24 |
zyga | I had re-lapses of this a few times but none that would be this long | 10:24 |
ijohnson | sorry to hear that, I know others who are in a similar situation, not wanting to go to any hospitals right now and it's quite unfortunate :-( | 10:25 |
mvo | zyga: ok, thank you! | 10:25 |
zyga | I think our government is not really doing a good job with this, and prefers to pretend the situation is good by not testing | 10:26 |
ijohnson | heh sounds familiar | 10:26 |
zyga | if it would deteriorate more I would definitely go despite the risk | 10:26 |
zyga | last time that saved me from a wheelchair :| | 10:26 |
mborzecki | zyga: do you work seated? | 10:28 |
zyga | mborzecki: now? | 10:28 |
zyga | mborzecki: no, I'm on the floor | 10:28 |
mborzecki | zyga: i mean in general, i recall you bought a standing desk | 10:29 |
zyga | yes, I use it | 10:29 |
ijohnson | I'm pretty strongly considering buying a standing desk or at least retro fitting my current desk to work standing | 10:30 |
mborzecki | i remember when i started working from home full time i had some backpain, but then with the standing desk it's no longer an issue | 10:30 |
mup | PR snapd#8777 opened: Enable riscv64 builds in the edge PPA <Created by xnox> <https://github.com/snapcore/snapd/pull/8777> | 10:31 |
zyga | mborzecki: but now standing up is not great either, I keep getting those phantom pains in my left leg | 10:31 |
zyga | that only go away when I'm curled up or at least not standing straight | 10:31 |
mborzecki | i work seated maybe 1-1.5h a day, partially caused by my super uncomfortable chair | 10:31 |
zyga | it was much worse before but I wonder if I will need some real help later on | 10:32 |
zyga | oh, my main chair broke :| | 10:32 |
zyga | we bought it looong time ago, when I met my wife | 10:32 |
zyga | so it's ~15-18 years old now | 10:32 |
mborzecki | zyga: yeah, better check it, sciatica is no joke if untreated | 10:33 |
zyga | we've ordered parts but the company making them has some issues | 10:33 |
zyga | mborzecki: I know | 10:33 |
zyga | mborzecki: spent half a year in bed crawling to the bathroom | 10:36 |
zyga | mborzecki: fun days | 10:36 |
zyga | mborzecki: how do you work for the reminder of the day? | 10:43 |
mborzecki | zyga: i'm available now, i need to go pick up some stuff right after the standup and be back for a bit ~5pm | 10:45 |
zyga | mborzecki: no no, I mean | 10:45 |
zyga | you said that you work seted for a fraction of the day | 10:45 |
mborzecki | ah, though you wanted to discuss something on HO ;) | 10:45 |
zyga | do you use a standing desk otherwise? | 10:45 |
zyga | (please forgive my terrible english phrases) | 10:46 |
mborzecki | zyga: yeah standing, sprints are probably the only time when i work seated (hence enjoying walking around later in the day) | 10:47 |
zyga | I see :) | 10:47 |
zyga | yeah, standing desk is really worth it :) | 10:47 |
zyga | I like biking so much because the position on a bike is really close to curled up "neutral" for me, so I can just go and go and go without any pain | 10:49 |
zyga | mborzecki: btw, I'm still using fish, I grew to like it more than zsh | 10:50 |
mborzecki | zyga: haha, maybe you can fix that problem then ;) | 10:55 |
mborzecki | zyga: the biking thing probably depends on the bike type/frame geometry, i can totally see how most people would find a road bike uncomfortable | 11:00 |
mup | PR snapd#8778 opened: tests: modernize and use snapd.tool <Created by zyga> <https://github.com/snapcore/snapd/pull/8778> | 11:01 |
zyga | mborzecki: not being a biking expert I'm not sure how to call my bike :) | 11:01 |
mborzecki | zyga: not an expert either, i just enjoy riding bikes :P | 11:03 |
zyga | I think it's a mountain bike | 11:04 |
zyga | though now with all the bike lanes that grow like grass after rain, I think a city bike would be good too | 11:05 |
zyga | mborzecki: could you look at https://github.com/snapcore/snapd/pull/8733 | 11:08 |
mup | PR #8733: tests: port document-portal-activation to session-tool <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8733> | 11:08 |
zyga | I addressed your comments there | 11:08 |
mup | PR snapcraft#3153 opened: review-tools: add --allow-classic flag for local review <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3153> | 11:15 |
zyga | more journal failures | 11:59 |
zyga | I will look after the branch I'm working on is odne | 11:59 |
zyga | *done | 11:59 |
mup | PR snapd#8733 closed: tests: port document-portal-activation to session-tool <Test Robustness> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8733> | 12:16 |
mup | PR snapd#8779 opened: osutil: Enable riscv64 build <Created by xnox> <https://github.com/snapcore/snapd/pull/8779> | 12:16 |
cachio | mvo, hey | 12:31 |
cachio | mvo, any idea why test-snapd-rsync-core20 is not published for arm64? | 12:33 |
ijohnson | cachio: wasn't that the one I built locally for you and sent to you ? | 12:41 |
cachio | ijohnson, yes, but then mvo fixed that and published for all the archs | 12:43 |
cachio | but arm64 is missing | 12:43 |
ijohnson | ah ok | 12:44 |
cachio | don't know if mvo forgot to upload or there is a problem | 12:44 |
cachio | not sure | 12:44 |
mup | PR snapd#8780 opened: tests: core20 early defaults spread test <Created by stolowski> <https://github.com/snapcore/snapd/pull/8780> | 12:46 |
mup | PR snapd#8781 opened: tests: modernize tests.session and port everything using it <Created by zyga> <https://github.com/snapcore/snapd/pull/8781> | 12:56 |
mvo | cachio: let me check, maybe something failed somewhere | 12:59 |
cachio | mvo, thanks | 13:01 |
mup | PR snapd#8770 closed: snap/naming: add ParseSecurityTag and friends <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8770> | 13:01 |
zyga | this is ready now https://github.com/snapcore/snapd/pull/8778 | 13:41 |
mup | PR #8778: tests: modernize and use snapd.tool <Created by zyga> <https://github.com/snapcore/snapd/pull/8778> | 13:41 |
mup | PR snapd#8779 closed: osutil: enable riscv64 build <Created by xnox> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/8779> | 13:47 |
ijohnson | cachio: pstolowski: thoughts on https://github.com/snapcore/snapd/pull/8780/files#r433896786 ? | 14:00 |
mup | PR #8780: tests: core20 early defaults spread test <Created by stolowski> <https://github.com/snapcore/snapd/pull/8780> | 14:00 |
cachio | ijohnson, thanks | 14:05 |
* cachio afk | 14:10 | |
ijohnson | degville: when you have a chance could you review the wording in my message here for `snap remove --help`? | 14:12 |
ijohnson | https://github.com/snapcore/snapd/pull/8782 | 14:12 |
mup | PR #8782: cmd/snap/remove: mention snap restore/automatic snapshots <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8782> | 14:12 |
mup | PR snapd#8782 opened: cmd/snap/remove: mention snap restore/automatic snapshots <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8782> | 14:12 |
degville | ijohnson: yes, of course. Looking now. | 14:17 |
ijohnson | thank you! | 14:17 |
=== King_InuYasha is now known as Conan_Kudo | ||
=== Conan_Kudo is now known as King_InuYasha | ||
zyga | pedronis: could you please look at https://github.com/snapcore/snapd/pull/8778 | 14:39 |
mup | PR #8778: tests: modernize and use snapd.tool <Created by zyga> <https://github.com/snapcore/snapd/pull/8778> | 14:39 |
zyga | just to make sure the snapd.tool exec <TOOL> thing is what you expected | 14:39 |
ijohnson | gosh I still just can't believe how cool it is that when I restart a spread run all the successful runs are cached | 14:42 |
zyga | yeah :D | 14:42 |
zyga | move from travis was worth it | 14:42 |
diddledan | zyga, it looks like the crash in openttd is because pulseaudio isn't being used on the pi default desktop? | 14:48 |
zyga | diddledan: I installed ubuntu-desktop (note: this was not raspbian) | 14:49 |
diddledan | oh | 14:49 |
diddledan | hmm | 14:49 |
zyga | diddledan: I also installed the deb and that did work | 14:49 |
diddledan | no idea then | 14:49 |
zyga | though there was no sound as my speaker was connected over usb and probably not default | 14:49 |
diddledan | it looks to be crashing immediately after trying to access pulse on my raspberry pi os install | 14:49 |
diddledan | s/pulse/alsa/ | 14:49 |
mvo | zyga: what's the current way to format our c code? clang-format? | 14:50 |
zyga | mvo: we use a hybrid approach to avoid flag days, make fmt will use both clang format and indent depending on the file name | 14:50 |
zyga | mvo: I strongly prefer clang format but I don not want to enforce it as the format also shifts from time to time and it's just not worth it | 14:51 |
mvo | zyga: that's fine, it's for a new file | 14:51 |
zyga | mvo: ideally new files should be clang, existing files should be using whatever was used | 14:51 |
pstolowski | ijohnson: thanks, replied | 14:56 |
zyga | pstolowski: https://bugs.launchpad.net/bugs/1881350 is interesting | 14:57 |
mup | Bug #1881350: snap seeding fails with libc6-lse <glibc (Ubuntu):New> <snapd (Ubuntu):New> <https://launchpad.net/bugs/1881350> | 14:57 |
zyga | does anyone know what libc6-lse is? | 14:58 |
pstolowski | zyga: indeed | 14:58 |
diddledan | zyga: not existing in the archive? | 14:59 |
diddledan | .. at least I can't find it | 14:59 |
zyga | diddledan: the report mentions 20.10 | 15:00 |
diddledan | oh. must be new then | 15:00 |
ijohnson | lse is a new arm64 feature AFAIK | 15:00 |
ijohnson | "Large System Extensions" | 15:01 |
ijohnson | see also https://launchpad.net/ubuntu/+source/glibc/2.31-0ubuntu9 | 15:01 |
zyga | heh | 15:01 |
ijohnson | also https://en.wikichip.org/wiki/arm/armv8.1 | 15:01 |
zyga | pstolowski: new custom weird stuff :) | 15:01 |
diddledan | aah | 15:02 |
zyga | https://github.com/snapcore/snapd/pull/8781 is ready for review | 15:06 |
mup | PR #8781: tests: modernize tests.session and port everything using it <Created by zyga> <https://github.com/snapcore/snapd/pull/8781> | 15:06 |
mup | PR snapd#8784 opened: snap: add new `snap run --gdbserver` option <Created by mvo5> <https://github.com/snapcore/snapd/pull/8784> | 15:07 |
mup | PR snapd#8640 closed: wrappers: pass 'disable' flag to StopServices wrapper (2/N) <Services ⚙️> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/8640> | 15:12 |
zyga | https://github.com/snapcore/snapd/pull/8778 needs a 2nd review though perhaps I can merge it as-is as it's really boring and should not linger | 15:16 |
mup | PR #8778: tests: modernize and use snapd.tool <Created by zyga> <https://github.com/snapcore/snapd/pull/8778> | 15:16 |
mborzecki | mvo: did you get gdbserver working for a regular user then? | 15:26 |
mborzecki | if it works it'd be a great improvement | 15:27 |
mborzecki | then i can polish some of the manual solib-search-path/sysroot tweaks and turn those into a script you could load from gdb | 15:29 |
mvo | mborzecki: ohhh, that would be nice | 15:29 |
mvo | mborzecki: well, gdbserver runs as root as it needs to attach via ptrace :( | 15:30 |
mvo | mborzecki: but the app runs as user, the gdb that attaches is run as user | 15:30 |
mborzecki | mvo: i'll play with the branch a bit | 15:31 |
pstolowski | zyga: see last comments https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1881350 | 15:32 |
mup | Bug #1881350: snap seeding fails with libc6-lse <glibc (Ubuntu):New> <snapd (Ubuntu):New> <https://launchpad.net/bugs/1881350> | 15:32 |
zyga | looking | 15:33 |
zyga | ooohh | 15:33 |
zyga | interesting :) | 15:33 |
zyga | onse sec | 15:33 |
zyga | *one sec | 15:33 |
mvo | mborzecki: cool, keep me updated | 15:33 |
zyga | btw, are there more complete logs in that bug report? | 15:33 |
mborzecki | mvo: btw. we need gdbserver in core20 | 15:33 |
mvo | mborzecki: it will use the gdbserver from the outside | 15:34 |
mvo | mborzecki: to avoid the ptrace problem inside the confinement | 15:34 |
mborzecki | ah cool | 15:34 |
zyga | pstolowski: snap-confine has this permission | 15:34 |
zyga | /{,usr/}lib{,32,64,x32}/{,@{multiarch}/}libpthread{,-[0-9]*}.so* mr, | 15:34 |
zyga | pstolowski: I commented on the bug but it looks like something is more seriously broken | 15:48 |
zyga | I suspect we need a different apparmor permission for the different binary | 15:48 |
zyga | and it just manifests itself as that pthread thing | 15:48 |
zyga | the fix is likely a one liner in the apparmor profile | 15:48 |
zyga | I will EOD soon | 15:50 |
zyga | need to go for a bike ride or I will go insane | 15:50 |
pstolowski | zyga: i see. thanks | 15:54 |
zyga | pstolowski: thank you! | 16:03 |
mup | PR snapd#8769 closed: dbusutil: move all D-Bus helpers and D-Bus test helpers <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8769> | 16:07 |
* cachio lunch | 16:10 | |
pstolowski | cachio: +1 for your 20.10 PR | 16:13 |
zyga | one last PR today: https://github.com/snapcore/snapd/pull/8785 | 16:21 |
mup | PR #8785: sandbox/cgroup: move FreezerCgroupDir from dirs.go <Created by zyga> <https://github.com/snapcore/snapd/pull/8785> | 16:21 |
zyga | breaking parts of the big sandbox/cgroup patch from the backend branch | 16:21 |
zyga | it's super small and straightforward | 16:21 |
mup | PR snapd#8785 opened: sandbox/cgroup: move FreezerCgroupDir from dirs.go <Created by zyga> <https://github.com/snapcore/snapd/pull/8785> | 16:22 |
* zyga EODs with a warm encouragement to merge https://github.com/snapcore/snapd/pull/8781 and avoid further suffering during the transition | 16:30 | |
mup | PR #8781: tests: modernize tests.session and port everything using it <Created by zyga> <https://github.com/snapcore/snapd/pull/8781> | 16:30 |
cachio | Psi-Jack, tx | 16:38 |
Psi-Jack | Huh? | 16:39 |
diddledan | Psi-Jack, is that a "huh? I didn't do nuffin. you can't prove I did it. wasn't me." | 16:41 |
mup | PR snapd#8786 opened: arch: add riscv64 <Created by xnox> <https://github.com/snapcore/snapd/pull/8786> | 16:42 |
mvo | cachio: test-snapd-rsync-core20 should be available for arm64 now | 16:47 |
cachio | mvo, yes I saw that | 16:47 |
cachio | thanks | 16:47 |
mvo | yw | 16:47 |
cachio | mvo, I created #8787 | 16:49 |
mup | PR #8787: tests: install test-snapd-rsync snap from edge channel <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8787> | 16:49 |
mvo | cachio: ta | 16:50 |
cachio | which is needed for uc20 | 16:50 |
=== mbriza is now known as mbriza2 | ||
Psi-Jack | More like a "huh? Why are you telling me to transmit randomly?" | 16:50 |
mup | PR snapd#8787 opened: tests: install test-snapd-rsync snap from edge channel <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8787> | 16:52 |
* Psi-Jack glares to cachio. | 16:52 | |
cachio | :), sorry about thatpstolowski -> Psi-Jack | 16:55 |
cachio | pstolowski left and I didn't noticed | 16:55 |
Psi-Jack | Hehe. It's all cool. :) | 16:56 |
cachio | ijohnson, zyga? could yo please take a quick look to #8787 ? | 16:58 |
mup | PR #8787: tests: install test-snapd-rsync snap from edge channel <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8787> | 16:58 |
zyga | yes | 16:58 |
cachio | I need it for rpi4 validation on uc20 | 16:58 |
cachio | it is super simple | 16:58 |
zyga | cachio: I don't understand the "idea is to use edge" | 16:58 |
zyga | is edge different from stable here? | 16:58 |
cachio | it is the same | 16:58 |
cachio | but we are publishing just on edge for the new snaps | 16:59 |
zyga | I see | 16:59 |
cachio | I'll remove them from stable | 16:59 |
zyga | ok | 16:59 |
cachio | the test snapd should be just published on edge | 16:59 |
cachio | when possible | 16:59 |
zyga | cachio: reviewed | 17:00 |
cachio | zyga, ijohnson thanks!! | 17:00 |
ijohnson | cachio: yaw | 17:02 |
mup | PR snapcraft#3153 closed: review-tools: add --allow-classic flag for local review <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3153> | 17:06 |
* zyga EODs for real this time | 17:17 | |
mup | PR snapd#8788 opened: cmd/snap-confine: add support for libc6-lse <Bug> <Needs security review> <Created by zyga> <https://github.com/snapcore/snapd/pull/8788> | 17:18 |
cmatsuoka | cachio: where is MATCH defined in our tests? | 17:20 |
cmatsuoka | cachio: ah, just found it | 17:22 |
cachio | cmatsuoka, there are 2 MATCH | 17:27 |
cachio | 1 in testslib/bin | 17:27 |
cachio | another defined inside spread | 17:28 |
cachio | also we have NOMATCH | 17:28 |
cachio | defined inside spread | 17:28 |
cmatsuoka | cachio: yes, I now see that in spread.yaml we overwrite the dummy definition | 17:28 |
ijohnson | mvo: if you have time yet today, could you sudo git merge https://github.com/snapcore/snapd/pull/8782 ? it is stuck waiting for travis, I could close and re-open if you wanted instead but seems wasteful | 17:31 |
mup | PR #8782: cmd/snap/remove: mention snap restore/automatic snapshots <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8782> | 17:31 |
* cachio -> cofee | 17:54 | |
mvo | ijohnson: sure | 18:20 |
=== facundo__ is now known as facubatista | ||
mup | PR snapd#8782 closed: cmd/snap/remove: mention snap restore/automatic snapshots <Simple 😃> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8782> | 18:23 |
mup | PR snapd#8764 closed: tests: add ubuntu 20.10 to spread tests <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8764> | 18:28 |
mup | PR snapd#8781 closed: tests: modernize tests.session and port everything using it <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8781> | 18:33 |
mup | PR snapd#8789 opened: interfaces/docker: use implicitOnClassic: true <Needs Samuele review> <⛔ Blocked> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8789> | 18:58 |
=== KindTwo is now known as KindOne | ||
mup | PR snapcraft#3150 closed: cli: error/warn when using sudo <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3150> | 19:16 |
=== KindTwo is now known as KindOne | ||
mup | PR snapcraft#3151 closed: cli: disable --target-arch support on core20 <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3151> | 19:46 |
mup | PR snapd#8787 closed: tests: install test-snapd-rsync snap from edge channel <Simple 😃> <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8787> | 20:13 |
cachio | ijohnson, hey | 20:27 |
ijohnson | hi cachio | 20:27 |
cachio | about the comment you left in the review | 20:27 |
ijohnson | on stolowski's PR? | 20:28 |
cachio | I dont get the first part | 20:28 |
cachio | yes | 20:28 |
ijohnson | yes sorry I was confused | 20:28 |
ijohnson | I read the wrong section in the spread.yaml | 20:28 |
cachio | ah, ok | 20:28 |
cachio | because in the past I created some images in gce with nested vms precreated and running | 20:29 |
cachio | I dont know if you were suggesting something like this | 20:29 |
ijohnson | no sorry for the confusion | 20:29 |
cachio | ijohnson, np | 20:29 |
cachio | ijohnson, I have a question | 20:30 |
ijohnson | sure | 20:30 |
cachio | in pi4 I see this error https://paste.ubuntu.com/p/rsGWsBWJ2r/ | 20:30 |
cachio | preparing the main suite | 20:30 |
cachio | any idea? | 20:31 |
ijohnson | huhu | 20:31 |
ijohnson | huh | 20:31 |
ijohnson | no I don't let me look in the prepare for that suite | 20:31 |
ijohnson | perhaps it's checking for a particular dtb | 20:31 |
cachio | dindt found nothing about dtb in prepare | 20:32 |
cachio | first time I see that issue | 20:32 |
ijohnson | cachio: do you have more logs ? | 20:32 |
ijohnson | yes it is very odd indeed I've never seen it either | 20:33 |
ijohnson | cachio: is it maybe from get_boot_path ? | 20:34 |
cachio | ijohnson, this is the full log https://paste.ubuntu.com/p/RP3wNqJvvC/ | 20:34 |
cachio | it is the rpi with 8gb | 20:34 |
cachio | ijohnson, running now with the 4gb version | 20:36 |
cachio | ijohnson, using this image http://cdimage.ubuntu.com/ubuntu-core/20/dangerous-beta/current/ubuntu-core-20-arm64+raspi.img.xz | 20:36 |
ijohnson | cachio: I'm inclined to think it's failing with the get_boot_path but for some reason it clips the output | 20:36 |
ijohnson | cachio: I see similar end of output when I run that same command on my uc pi, i.e. `ls -alR /boot` shows similar output of | 20:38 |
ijohnson | /boot/uboot/pi-kernel_162.snap/dtbs/overlays: | 20:38 |
ijohnson | total 385 | 20:38 |
ijohnson | drwxr-xr-x 2 root root 15360 May 29 07:11 . | 20:38 |
ijohnson | drwxr-xr-x 4 root root 512 May 29 07:10 .. | 20:38 |
ijohnson | -rwxr-xr-x 1 root root 569 May 27 13:45 act-led.dtbo | 20:38 |
cachio | ijohnson, in restore I see this boot_path='Cannot determine boot path | 20:41 |
ijohnson | cachio: right that makes sense that would cause the prepare error you see | 20:41 |
ijohnson | let me look at the logs more | 20:41 |
ijohnson | not sure what the issue is yet | 20:42 |
ijohnson | cachio: I don't see that `Cannot determine boot path` message in the full log you sent, where do you see that? | 20:43 |
cachio | ijohnson, hehe, I read the previous error log | 20:43 |
cachio | 1 sec | 20:43 |
cachio | ijohnson, https://paste.ubuntu.com/p/KCdkWyt4ND/ | 20:44 |
ijohnson | cachio: do you have a shell to this system ? | 20:44 |
cachio | ijohnson, no | 20:44 |
ijohnson | cachio: but you are using the released image? | 20:45 |
cachio | yes | 20:45 |
ijohnson | err rather the images published on cdimage | 20:45 |
ijohnson | ok | 20:45 |
cachio | the one from beta | 20:45 |
cachio | http://cdimage.ubuntu.com/ubuntu-core/20/dangerous-beta/current/ubuntu-core-20-arm64+raspi.img.xz | 20:45 |
ijohnson | let me download it and try to flash it, it will take me a while though | 20:45 |
cachio | well | 20:46 |
cachio | I know | 20:46 |
cachio | total 5 | 20:46 |
cachio | drwxr-xr-x 3 root root 512 Jun 2 20:16 . | 20:46 |
cachio | drwxr-xr-x 6 root root 70 May 27 15:05 .. | 20:46 |
cachio | -rwxr-xr-x 1 root root 4096 Jun 2 20:16 boot.sel | 20:46 |
cachio | drwxr-xr-x 3 root root 512 May 15 21:06 pi-kernel_155.snap | 20:46 |
cachio | this is what we have | 20:46 |
ijohnson | hmmmm | 20:46 |
cachio | if [ -f /boot/uboot/uboot.env ] || [ -f /boot/uboot/uboot.sel ]; then | 20:46 |
cachio | and we check that | 20:47 |
ijohnson | cachio: is this shell maybe being run in install mode ? | 20:47 |
cachio | boot.sel vs uboot.sel | 20:47 |
ijohnson | ahhhh I see | 20:47 |
ijohnson | cachio: yes it should be boot.sel | 20:47 |
cachio | I'll create a quick pr for that | 20:48 |
ijohnson | sure, let me know and I can approve it | 20:48 |
cachio | ijohnson, thanks | 20:49 |
cachio | ijohnson, I don't know why it is not failing in pi3 | 20:49 |
cachio | ijohnson, any idea if in pi3 we have uboot.sel? | 20:50 |
cachio | or boot.sel? | 20:50 |
cachio | I can't get any pi3 on the lab | 20:51 |
ijohnson | cachio could be a typo in the gadget maybe? | 20:59 |
ijohnson | Let me look | 20:59 |
ijohnson | cachio no it should not be uboot.sel on pi3 | 21:01 |
cachio | ok, thnanks | 21:01 |
cachio | ijohnson, https://github.com/snapcore/snapd/pull/8790 | 21:04 |
mup | PR #8790: tests: update the file used to detect the boot path on uc20 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8790> | 21:04 |
mup | PR snapd#8790 opened: tests: update the file used to detect the boot path on uc20 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8790> | 21:08 |
* cachio afk | 21:13 | |
mup | PR snapd#8791 opened: snap/prepare_image: Add support for append/remove <Created by xnox> <https://github.com/snapcore/snapd/pull/8791> | 21:44 |
mup | PR snapd#8792 opened: interfaces: miscellanious policy update xlv <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8792> | 21:49 |
mup | PR snapd#8793 opened: interfaces: miscellanious policy update xlv - 2.45 <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8793> | 21:54 |
mup | PR snapd#8794 opened: boot/bootstate16.go: clean snap_try_* vars when not in Trying status too <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8794> | 23:19 |
mup | PR snapd#8795 opened: cmd/snap-bootstrap/initramfs-mounts: also copy systemd clock + netplan files <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8795> | 23:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!