| mup | PR snapd#10707 opened: tests/regression: add regression test for LP #1942266 <Bug> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10707> | 01:28 |
|---|---|---|
| mup | PR snapd#10708 opened: tests/lib/prepare.sh: use core20 from beta channel temporarily <⚠ Critical> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10708> | 01:43 |
| mborzecki | morning | 05:20 |
| zyga-mbp | good morning school chaos :) | 05:22 |
| mborzecki | zyga-mbp: heh, damn right | 05:45 |
| mborzecki | need to run an errand, bb around 9 | 05:45 |
| pstolowski | morning | 06:27 |
| mborzecki | re | 06:44 |
| zyga-mbp | pstolowski hey :) | 06:44 |
| mborzecki | pstolowski: hey | 06:44 |
| mardy | hi all! Yes, I also went to attend to the school chaos :-) | 07:18 |
| === alan_g_ is now known as alan_g | ||
| mborzecki | zyga-mbp: meh, snapd on fedora is broken https://bugzilla.redhat.com/show_bug.cgi?id=1999998 https://forum.snapcraft.io/t/cannot-install-snap-file-snap-is-unusable-due-to-missing-files/25719/15 | 07:22 |
| zyga-mbp | oh | 07:23 |
| mborzecki | not quite sure how that squashfs-tools update went in with only 3 days in testing | 07:23 |
| mborzecki | eh and i hate fedora packaging process, so many manual steps | 07:23 |
| zyga-mbp | mborzecki break fast and often | 07:23 |
| zyga-mbp | my fedora laptop is in the Huawei office today | 07:23 |
| zyga-mbp | mborzecki what's the fix strategy? change snapd to cope with the new output? | 07:25 |
| zyga-mbp | perhaps snap-unsquashfs is needed, it could link to cgo libsquashfs? | 07:26 |
| mborzecki | zyga-mbp: i've already fixed it, but did not update the package in fedora yet because i did not suspect they would actually update to new squashfs-tools in a fedora version that was released a while back | 07:26 |
| zyga-mbp | fedora updates are like that | 07:27 |
| mborzecki | yeah, fun and unpredictable | 07:27 |
| zyga-mbp | would a zool test prevent that issue? | 07:28 |
| zyga-mbp | IIRC zool is like autopkgtest? | 07:28 |
| mborzecki | anyways, the fix was cherry-picked for 2.51.7 so i can drop the patch in arch too and probably update tw pacakge as well | 07:28 |
| mborzecki | so half a day lost to packaging | 07:29 |
| zyga-mbp | mborzecki good luck! lots of people depend on that | 07:29 |
| mborzecki | zyga-mbp: still, i'm happy that i don't have to deal *deb beaurocracy | 07:31 |
| zyga-mbp | see :) | 07:32 |
| zyga-mbp | some upsides ;D | 07:32 |
| mborzecki | hmmm why do we have this tag in the repo: https://github.com/snapcore/snapd/tree/3.9.9-0sdhd5 ? | 07:49 |
| zyga-mbp | mborzecki looks like a mistake? | 08:02 |
| pstolowski | hmm i'm very confused by snap-preseed test failures on 21.10 on master; on our PRs it fails in prepare on /dev/nbd0p1 device check for qemu, but when I run this test manually with spread on google:ubuntu-21.10-64 I'm always getting a build failure on "# build squashfuse and rename to snapfuse" | 09:00 |
| pstolowski | mvo: ^ is there anything magical now happening wrt to snapfuse (v3?)? | 09:00 |
| mvo | pstolowski: oh, there should not be, can you show me the full output? maybe the script is buggy or something | 09:02 |
| mvo | pstolowski: what is needed to reproduce? | 09:02 |
| pstolowski | mvo: I run spread -debug google:ubuntu-21.10-64:tests/main/preseed | 09:03 |
| mvo | pstolowski: thanks, let me try this | 09:05 |
| pstolowski | mvo: a bit more context: https://pastebin.ubuntu.com/p/WgvdZK6Nyb/ | 09:05 |
| pstolowski | mvo: in our tests it fails on nbd check though https://pipelines.actions.githubusercontent.com/xS8oSnypZkPEQZqiZgDaRp2kdvQJKbOY08TesHp7E8vn7g4hYR/_apis/pipelines/1/runs/34095/signedlogcontent/101?urlExpires=2021-09-01T08%3A25%3A11.5093188Z&urlSigningMethod=HMACV1&urlSignature=IfSWYWEXjw9%2F0MFg%2Bu3E3F3noQ%2BO1o7x%2BD8tFdqxq60%3D | 09:05 |
| mvo | pstolowski: oh, this looks like the c-vendor file is not there | 09:05 |
| pstolowski | so i wonder what's different when I trigger it manually | 09:06 |
| mvo | pstolowski: probably, just rm -rf ./c-vendor/squashfuse and try again, maybe it's outdated | 09:06 |
| mvo | pstolowski: actually I think that is it :/ the script may need to be made smarter for situations like this | 09:07 |
| mvo | pstolowski: rm -rf c-vendor/squashfuse/ and then try again, I will also run it hree to double check | 09:07 |
| pstolowski | mvo: thanks, re-trying | 09:07 |
| pstolowski | mvo: yup, that solved it, i'm now seeing the same failure as we have on the PRs, thanks! | 09:20 |
| mup | PR snapd#10709 opened: spread: add 21.10 to qemu, remove 20.10 (EOL) <Simple 😃> <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10709> | 09:29 |
| sil2100 | mvo, pstolowski: oh no, snapd FTBFS again: https://launchpad.net/ubuntu/+source/snapd/2.51.1+20.04ubuntu1/+build/22030863 | 09:46 |
| mvo | sil2100: did you try one rebuild already? maybe just bad luck? | 09:48 |
| mvo | sil2100: I can also trigger the rebuild of course (don't want to dump this on you) | 09:50 |
| mborzecki | zyga-mbp: can you take a look at https://build.opensuse.org/request/show/915439 ? | 10:05 |
| zyga-mbp | sure | 10:06 |
| zyga-mbp | boo? | 10:06 |
| zyga-mbp | is boo#nnn a suse bug scheme? | 10:06 |
| zyga-mbp | mborzecki "Do you really want to approve this request, despite of open review requests?" | 10:08 |
| zyga-mbp | do you know where I can send a review so that this pop up does not show up? | 10:08 |
| mborzecki | zyga-mbp: iirc it's bugzilla.opensuse.org | 10:08 |
| mborzecki | zyga-mbp: sorry, forgot to drop a patch and had to supersede the request with a new one https://build.opensuse.org/request/show/915446 | 10:10 |
| zyga-mbp | done | 10:11 |
| mup | PR snapd#10709 closed: spread: add 21.10 to qemu, remove 20.10 (EOL) <Simple 😃> <Skip spread> <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10709> | 10:19 |
| mborzecki | mvo: do we have anyone who can look into fixing https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=993233 ? | 10:20 |
| zyga-mbp | mborzecki is it a dput that's required? | 10:21 |
| mvo | mborzecki, zyga-mbp indeed, seems like we need to update snapd in debian to fix this | 10:23 |
| zyga-mbp | debian is open now | 10:23 |
| mborzecki | mv | 10:25 |
| mborzecki | mvo: fwiw the squashfs-tools patch was cherry-picked to 2.51.7 so a simple update is all we need | 10:25 |
| mvo | yeah, I think so, I will try to get to it today | 10:26 |
| pstolowski | sil2100: oh.. and these are different failures | 10:31 |
| pstolowski | sil2100: did anything change with these builders? are they slower than before? | 10:32 |
| mvo | 10634 needs a second review, looks like go modules are finally ready | 10:32 |
| mup | PR snapd#10710 opened: tests: add more space on ubuntu xenial <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10710> | 10:34 |
| mup | PR snapd#10711 opened: tests: bump the number of retries when waiting for /dev/nbd0p1 <Simple 😃> <Run nested> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10711> | 10:45 |
| sil2100 | pstolowski: no idea, I think those are standard ones as before | 11:00 |
| sil2100 | Interesting, hirsute was fine? | 11:00 |
| ijohnson[m] | Morning folks is UC20 in gce spread still broken? | 11:09 |
| mvo | ijohnson[m]: yeah, it's confusing, I tried it in qemu and it's fine | 11:13 |
| mvo | ijohnson[m]: but there was a change https://people.canonical.com/~mvo/core20-changes/html/edge/'20210826'r1090_'20210831'r1097.html so | 11:14 |
| zyga-mbp | good morning ijohnson[m] | 11:14 |
| ijohnson[m] | Hey zyga-mbp | 11:22 |
| ijohnson[m] | mvo :-/ yeah I was trying to debug it with Sergio last night didn't make it far enough, I'll resume looking at the GCE stuff, Sergio showed me how to view the serial console on GCE myself so that will aid debugging a bit | 11:23 |
| mvo | ijohnson[m]: do you have a log output that you can paste? | 11:24 |
| popey | ijohnson[m] morning. thanks for the help with etrace yesterday | 11:24 |
| mvo | ijohnson[m]: I'm very puzzled by the fact that it's only affecting google nad not qemu | 11:24 |
| mvo | popey: hey, nice to see you | 11:24 |
| popey | hey mvo o/ | 11:24 |
| mup | PR snapd#10712 opened: tests: ensure the `libvirt-daemon-system` pacakge is installed <⚠ Critical> <Test Robustness> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10712> | 11:25 |
| popey | ijohnson[m] I filed a few bugs for you on etrace :D | 11:25 |
| mvo | pstolowski: thanks so much for 10711 | 11:25 |
| mvo | pstolowski: I think it's the last thing that fails on 21.10, then we can make it required | 11:25 |
| mvo | ijohnson[m]: in case you want a break from debugging that gce, 10643 (moving to go modules) needs a :+1: :) | 11:26 |
| ijohnson[m] | Whoops I step away for a minute and have like 11 pings haha | 11:30 |
| ijohnson[m] | mvo i finished a review on the go modules or yesterday but I'm looking into the golang-evdev thing still as I'm still puzzled | 11:30 |
| ijohnson[m] | @mvo I put log output in the SU doc | 11:31 |
| ijohnson[m] | popey thanks for testing it out and helping debug yesterday :-) | 11:31 |
| mvo | ijohnson[m]: thanks, I think there are some small things to address in the go mod still so no real rush, just really would like to get it out :) | 11:33 |
| ijohnson[m] | Sure | 11:34 |
| popey | ijohnson[m] happy to test further, I have a spare machine i can kick off tests on and leave running for long periods. I want desktop snaps to be faster, so am motivated to help. | 11:34 |
| ijohnson[m] | That's great thanks for the offer, I'll be sure to take you up on it, I'll take a look at the snaps you filed issues for first though it might be a simple/silly thing that doesn't require 20 minutes to run through just to get a single error message :-) | 11:36 |
| popey | I do think the ability to run non-snap packages in the same run would be important. | 11:37 |
| popey | so you can compare /snap/bin/vlc with /usr/bin/vlc and see what the difference is. Because I don't believe some people believe there's any issue beyond compression | 11:38 |
| pstolowski | mvo: let's see if it's happy, i only run it twice manually | 11:38 |
| pstolowski | sil2100: is the riscv64 build failure reproducible on every run? | 11:41 |
| ijohnson[m] | popey yeah I think I'm response to your issue the thing to do is create a matrix for each startup type so you can compare the different versions since for example I added some really basic support for running flatpaks too | 11:44 |
| popey | good call | 11:44 |
| mup | PR snapd#10710 closed: tests: add more space on ubuntu xenial <Test Robustness> <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10710> | 11:50 |
| mborzecki | ijohnson: hi, do you have a log or sth related to https://github.com/snapcore/snapd/pull/10708 ? i'm trying to investigate why uc20 fails to boot all with my remodel pr, maybe it's related | 12:08 |
| mup | PR #10708: tests/lib/prepare.sh: use core20 from beta channel temporarily <⚠ Critical> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10708> | 12:08 |
| sergiusens | > removing (with --purge) the lxd snap and reinstalling it helped | 12:23 |
| sergiusens | Aside from a potential bug, next time, try and remove the dataset | 12:23 |
| ijohnson[m] | bboozzoo hey there's a log in the SU doc | 12:31 |
| ijohnson[m] | bboozzoo all UC20 systems in gce are failing to boot AIUI | 12:31 |
| ijohnson[m] | On master that is | 12:31 |
| mborzecki | ijohnson: heh, i merged master to my remodel pr yesterday and it was getting stuck on install, although seemingly in tests that required resealing | 12:32 |
| pstolowski | sil2100: i'll see if I can find something about this new failures later today | 12:44 |
| mup | PR core#121 closed: Generate the dpkg.yaml file <Created by ilasc> <Merged by mvo5> <https://github.com/snapcore/core/pull/121> | 13:21 |
| sil2100 | pstolowski: thanks! | 13:36 |
| mup | PR snapd#10712 closed: tests: ensure the `libvirt-daemon-system` pacakge is installed <⚠ Critical> <Test Robustness> <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/10712> | 13:45 |
| mup | PR snapd#10713 opened: tests: use host-scaled settle timeout for hookstate tests <Skip spread> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10713> | 13:50 |
| mborzecki | heh, so the snapd snap from that remodel PR works in qemu | 13:51 |
| pstolowski | sil2100: ^ #10713 should help with one of the failures | 13:51 |
| mup | Bug #10713: Broken context-sensitive spell check in Evolution (Greek, Hebrew) <Evolution:Won't Fix> <gtkhtml3.14 (Ubuntu):Fix Released by desktop-bugs> <https://launchpad.net/bugs/10713> | 13:51 |
| mup | PR #10713: tests: use host-scaled settle timeout for hookstate tests <Skip spread> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10713> | 13:51 |
| pstolowski | sil2100: but the second failure is unclear; i think mvo wants to re-run this build | 13:51 |
| mup | PR snapd#10714 opened: tests: move interfaces-libvirt test back to 16.04 <⚠ Critical> <Simple 😃> <Test Robustness> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10714> | 13:55 |
| sil2100 | pstolowski: there is a re-run running right now, let's see how it goes! It takes around 3-4 hours though | 13:57 |
| pstolowski | sil2100: whaaat?!!! | 13:57 |
| mup | PR snapd#10711 closed: tests: bump the number of retries when waiting for /dev/nbd0p1 <Simple 😃> <Run nested> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10711> | 14:00 |
| sil2100 | It's riscv! | 14:03 |
| ijohnson[m] | fun, so while I can see serial output from gcloud it has a buffer of like 150K or something silly like this and so I lost all of the previous serial console output due to the immense spam, seems I need to stream the output continuously while spread is running to get the full log for some reason :-/ | 14:26 |
| ijohnson[m] | and the only way to know the next iteration is to parse stderr, but we also want to keep stdout for the log itself | 14:29 |
| ijohnson[m] | look at this grossness https://gist.github.com/anonymouse64/90b30502ffa3093f77d9a48d9f1d5e8a | 14:29 |
| mup | PR core20#108 closed: Generate dpkg.yaml file <Created by ilasc> <Merged by sil2100> <https://github.com/snapcore/core20/pull/108> | 14:30 |
| mup | PR core18#180 closed: Generate dpkg.yaml <Created by ilasc> <Merged by sil2100> <https://github.com/snapcore/core18/pull/180> | 14:30 |
| mborzecki | ijohnson: mvo ok, i can reproduce the failure from my remodel PR with core20 from edge, the system does not boot | 14:36 |
| ijohnson[m] | bboozzoo: I finally have a system running with full serial console logs on gce | 14:36 |
| mborzecki | the exact revision is 1097 | 14:37 |
| ijohnson[m] | took a bit of finnicky nonsense with the gcloud command to stream the console output | 14:37 |
| mborzecki | ijohnson: i've repacked the pc gadgetg with cmdline.extra now | 14:40 |
| ijohnson[m] | yeah that's what I'm doing too | 14:40 |
| mvo | ijohnson[m]: anything interessting so far on the console :) ? I'm very curious about this one | 14:41 |
| ijohnson[m] | this is the patch to master I'm booting with https://pastebin.ubuntu.com/p/WmtvTVmq63/ | 14:41 |
| ijohnson[m] | mvo: just booted now, I'm watching the output | 14:42 |
| ijohnson[m] | hmm it's just stuck in install mode | 14:46 |
| ijohnson[m] | it's not stuck in the initrd, I think that may have been a red herring that it seemed like it was stuck there | 14:46 |
| ijohnson[m] | and actually it made it through install mode successfully, it is just stuck in run mode, I mispoke earlier | 14:47 |
| ijohnson[m] | of course I forgot to include snapd.debug=1 in the kernel command line so probably need to run again with that, but it does seem to have booted successfully into run mode | 14:47 |
| ijohnson[m] | this is the current log https://pastebin.ubuntu.com/p/PQKq6xNDVd/ | 14:49 |
| ijohnson[m] | and it just stays stuck there indefinitely | 14:50 |
| ijohnson[m] | alright I'm trying a new run with more debug info | 14:50 |
| mborzecki | ijohnson: hmm, at least what I observe is that the snapd-seeding service starts, but then it takes very long for actual snapd output to appear and the system seems to be stuck | 14:59 |
| mborzecki | ijohnson: and it's happening at random | 15:00 |
| ijohnson[m] | hmm so you see it sometimes works? | 15:04 |
| ijohnson[m] | bboozzoo: also interestingly, @ondra reported the same thing in MM this morning | 15:05 |
| mvo | mborzecki: can you reproduce anything there outside of google? or only google too? | 15:05 |
| ijohnson[m] | mvo: it kinda sounds like ondra may have reproduced it on whatever physical device he was working on | 15:05 |
| mborzecki | ijohnson: yeah, it kind of gets stuck, happens at random | 15:08 |
| mvo | ijohnson[m]: yeah, was just reading this, strange | 15:08 |
| mborzecki | ijohnson: it looks like this: https://paste.ubuntu.com/p/JmtvzVgwMH/ | 15:09 |
| mborzecki | then if you wait for like 30s or more it proceeds further | 15:10 |
| mvo | what is confusing to me is that the PR that switches core20 back to beta does not help | 15:10 |
| ijohnson[m] | bboozzoo: looking at your logs, but for me using core20 from edge it never proceeds even after waiting 5 minutes | 15:10 |
| ijohnson[m] | mvo: well what I see is that we get further with using core20 to beta | 15:11 |
| mborzecki | ijohnson: it was like that in gce, even when i restarted the vm service | 15:11 |
| mvo | ijohnson[m]: yeah, one thing is that it seems there is a new 20/beta,edge kernel from the 24th too | 15:11 |
| ijohnson[m] | wait bboozzoo are you talking about booting uc20 "natively" in GCE or for nested tests ? | 15:11 |
| mborzecki | ijohnson: no, nested in gce | 15:12 |
| ijohnson[m] | I have only concerned myself with native uc20 runs in GCE, hasn't looked at nested tests at all | 15:12 |
| ijohnson[m] | ah okay, so we are looking at different things, but could be different manifestations of the same problem | 15:12 |
| mborzecki | ijohnson: heh, not it got stuck locally again, look at this jump from 11 to 39 seconds, https://paste.ubuntu.com/p/Y7qKWG5WBT/ maybe there's also some networking issues in gce? | 15:12 |
| ijohnson[m] | oh interesting that is weird | 15:15 |
| ijohnson[m] | okay well I have gotten as much as I can out of this run, need to start another run | 15:15 |
| ijohnson[m] | mvo: this was the next run I did, also with master snapd + edge core20 | 15:15 |
| mvo | so in "packaging: add libfuse3-dev build-dep" (2 days ago) ubuntu-core-20-64 had no failure, let me try to bisect from our previous runs if I can pinpoint this more | 15:15 |
| ijohnson[m] | I'm going to try beta core20 again to see how far it gets | 15:15 |
| ijohnson[m] | https://pastebin.ubuntu.com/p/QnKdX4bgMf/ | 15:15 |
| mvo | ijohnson[m]: \o/ thanks | 15:16 |
| mvo | late Monday it seems to have started | 15:22 |
| ijohnson[m] | I see https://github.com/snapcore/snapd/pull/10685 was happy | 15:30 |
| mup | PR #10685: many: fix run-checks gofmt check <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10685> | 15:30 |
| ijohnson[m] | oh but that started running at 2021-08-30 07:35:59 | 15:33 |
| ijohnson[m] | I assume UTC? | 15:33 |
| ijohnson[m] | aha, so beta channel core20 snap does work, but it is quirky in that we seeded the beta channel revision, but we use --channel=edge in our ubuntu-image command, so when the system boots up with the beta channel, it immediately refreshes to the edge channel and becomes broken again | 15:39 |
| ijohnson[m] | mvo: ^ | 15:39 |
| ijohnson[m] | the reason my PR failed is because spread doesn't expect the system to be rebooting so it fails trying to run commands on the VM in GCE, and when it sees that the system isn't responding tries to SSH in via debug and gets EOF since the machine is rebooting at that exact moment | 15:40 |
| ijohnson[m] | so if we do a spread run with the beta channel of the core20 snap, and we make sure that the core20 snap is not refreshed, things should be happy | 15:40 |
| ijohnson[m] | out of curiosity I am seeing if after refreshing from beta channel core20 to edge if we can still boot or if even that is broken too | 15:41 |
| * cachio_ afk | 15:43 | |
| ijohnson[m] | mvo: here's the log from using beta core20 snap, you can see it proceeds all the way to refreshing core20 from beta back to edge and it reboots https://pastebin.ubuntu.com/p/zShWjWCfwm/ | 15:52 |
| mvo | ijohnson[m]: ohhhh, ok | 15:52 |
| ijohnson[m] | mvo: still no smoking gun on why core20 is broken | 15:52 |
| ijohnson[m] | but now I know how to unbreak our spread systems for PR's | 15:52 |
| ijohnson[m] | just testing my patch locally and then I will update https://github.com/snapcore/snapd/pull/10708 | 15:53 |
| mup | PR #10708: tests/lib/prepare.sh: use core20 from beta channel temporarily <⚠ Critical> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10708> | 15:53 |
| mup | PR snapd#10715 opened: Bump secboot <Created by xnox> <https://github.com/snapcore/snapd/pull/10715> | 16:06 |
| mvo | ijohnson[m]: I'm looking over the PRs that recently got merged for core20 and there are some potential culprits - we merge the systemd-time-wait-sync service, that sounds like something that may break things | 16:08 |
| mvo | ijohnson[m]: we changed some bits in writeable (/etc/issue,motd) | 16:09 |
| * ijohnson[m] looks for some goats to sacrifice to prevent this from being a writable-paths problem | 16:11 | |
| mvo | ijohnson[m]: is there a way to run this easily against a branch of the core20 snap? I could prepare something | 16:11 |
| mvo | ijohnson[m]: I suspect the enable-time-wait-sync even though it feels odd but it's the usual messing with systemd dependencies that might have got us into this mess | 16:12 |
| mvo | ijohnson[m]: I prepared a PR with the revert but would love to test first | 16:12 |
| ijohnson[m] | mvo: sure, if you can wait like 5-10 minutes I should be done verifying my fix which should show how to use a different version of core20 to test this easily | 16:12 |
| mvo | ijohnson[m]: sure, will wait and prepare the snap in the meantime | 16:13 |
| ijohnson[m] | mvo: ok it worked \o/ | 16:16 |
| ijohnson[m] | let me push it | 16:16 |
| mvo | a simple pr: 10714 needs a review, another thing that will unbreak tests | 16:22 |
| mvo | ijohnson[m]: excllent, I'm preparing the revert and testing it | 16:24 |
| ijohnson[m] | mvo: alright https://github.com/snapcore/snapd/pull/10708 is ready again | 16:26 |
| mup | PR #10708: tests/lib/prepare.sh: use core20 from beta channel temporarily <⚠ Critical> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10708> | 16:26 |
| ijohnson[m] | mvo: specifically, just change the channel of snap download core20 in prepare.sh there | 16:26 |
| ijohnson[m] | mvo: I hope it's okay I added a change to my PR there to also patch the gadget so we get more debugging by default | 16:27 |
| ijohnson[m] | though I suppose that change may break some output on QEMU | 16:27 |
| ijohnson[m] | mmm, let me back that out of that PR and we can propose that separately I suppose | 16:27 |
| mvo | ijohnson[m]: +1 | 16:27 |
| mup | PR snapd#10714 closed: tests: move interfaces-libvirt test back to 16.04 <⚠ Critical> <Simple 😃> <Test Robustness> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10714> | 16:31 |
| ijohnson[m] | ok, I put the extra gadget debug stuff into 10716 instead, so 10708 is just about switching which channel of the core20 snap we use | 16:33 |
| ijohnson[m] | one thing I'm unsure about with 10708 is whether there are any spread tests which rely on the core20 / base snap being asserted from the store, since that will no longer be true with my branch/workaround, I had a look and couldn't see any but I could be wrong | 16:33 |
| ijohnson[m] | and I did change the patch in 10716 to just make the console changes for GCE specifically, so it shouldn't detract from the qemu experience | 16:34 |
| mup | PR snapd#10716 opened: tests/lib/prepare.sh: add debug kernel command line params via gadget on UC20 <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10716> | 16:36 |
| mvo | ijohnson[m]: thanks! I opened pr#110 for core20 to undo the timesync target, but the spread test is still running | 16:38 |
| ijohnson[m] | ack | 16:39 |
| mup | PR core20#110 opened: hooks: revert PR#105 - it seems to <Created by mvo5> <https://github.com/snapcore/core20/pull/110> | 16:40 |
| mvo | ijohnson[m]: fwiw, this is what is running right now https://paste.ubuntu.com/p/kMY3bVz7kY/ | 16:41 |
| mvo | 16:41 | |
| ijohnson[m] | mvo: ack that looks good, if it works then we have our smoking gun as it were | 16:42 |
| ijohnson[m] | also I'm not sure should we be using CORE_CHANNEL or BASE_CHANNEL ? | 16:42 |
| ijohnson[m] | also do we want all our CI for snapd to be running on edge ? | 16:42 |
| mvo | ijohnson[m]: well, I think we need better QA for core20, i.e. build an image and run a test smoke test :/ | 16:49 |
| mvo | ijohnson[m]: in general I think running edge is fine, because the alternative is that we run beta and get the same issue a bit later when edge goes to beta | 16:50 |
| ijohnson[m] | true | 16:50 |
| ijohnson[m] | mvo: but yeah having spread tests for core20 would be great, would be a bit of work to enable though | 16:50 |
| mvo | well, if we have gating that buids a core20 in gce and does the edge->beta core20 then that's fine of coure, then beta would be better for us. but I don't think we have this :/ | 16:50 |
| ijohnson[m] | mvo: also I think we were bitten by the fact that the TPE lab was moving so the tests that normally do run on edge channel of core20 snap did not run | 16:51 |
| ijohnson[m] | mvo: because AIUI cachio_ has tests that run against the edge channel of the core20 snap | 16:51 |
| mvo | ijohnson[m]: could be, I don't know to what extend this is tested, if it is we should probably move to beta indeed | 16:51 |
| ijohnson[m] | but they run on devices in the lab, and the entire lab is down right now | 16:51 |
| ijohnson[m] | though actually if the tests run in the lab they wouldn't have caught this particular issue in GCE 🤔 | 16:52 |
| mvo | ijohnson[m]: probably not :/ | 16:52 |
| * mvo goes for dinner and checks the spread run with the updated core20 with #110 in a bit | 16:53 | |
| mup | PR #110: introduce AccountKey, support decoding it and validation <Created by pedronis> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/110> | 16:53 |
| mup | PR core20#110 closed: hooks: revert PR#105 - it seems to <Created by mvo5> <Merged by xnox> <https://github.com/snapcore/core20/pull/110> | 16:55 |
| mvo | ijohnson[m]: indeed with this reverted things seem to be unbroken | 17:28 |
| ijohnson[m] | mvo great, and it seems xnox has already merged it too | 17:31 |
| mvo | I triggred an edge build | 17:34 |
| om26er | Is there a way for a snap that runs as a background service to read environment variables from the HOST ? | 17:47 |
| om26er | we have two snaps that run as background service on our hardware. We want developers to export "credentials" on their Ubuntu system and then just restart the two snap daemons | 17:48 |
| mvo | ijohnson[m]: and it's ready, I triggered a new test run in #10634 | 17:53 |
| mup | Bug #10634: gnome-mag: SONAME change breaks existing gnopernicus package <gnome-mag (Ubuntu):Fix Released> <gnome-mag (Debian):Fix Released> <https://launchpad.net/bugs/10634> | 17:53 |
| mup | PR #10634: many: move to go modules <Created by mvo5> <https://github.com/snapcore/snapd/pull/10634> | 17:53 |
| mup | PR snapd#10717 opened: tests: fix interfaces-libvirt test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10717> | 18:11 |
| ijohnson[m] | om26er: can't you just use `snap set foo creds=secrets` ? | 18:15 |
| om26er | ijohnson[m] will it persist reboots though ? | 18:15 |
| ijohnson[m] | om26er: yes snap config set with `snap set ...` is persistent | 18:16 |
| om26er | I just tried, it seems I need a config hook for this to work ? | 18:17 |
| ijohnson[m] | yes you do, it can be empty though | 18:17 |
| om26er | ijohnson[m] I just updated my snap and created a `snap/hooks/configure` file with shebang only and made it executable. The snap build was successfully and I can call `snap set ...`, however the exported variable does not show | 18:30 |
| om26er | ...when I run `snap run --shell <snap_name>` | 18:30 |
| sergiusens | shells don't persist, how is this being exported? | 18:31 |
| ijohnson[m] | om26er: I mean you can do `snap set foo thing=other-thing` and then in that snap do `snapctl get thing` | 18:32 |
| ijohnson[m] | om26er: I made an example of converting snapctl settings to environment variables easily here: https://forum.snapcraft.io/t/neat-trick-for-over-writing-default-environment-variable-values-for-daemons/13404/3 | 18:32 |
| om26er | Is there a reason why `snap set foo` doesn't support underscores ? | 19:21 |
| om26er | You can't set things like `snap set foo CLOUD_USER_ENDPOINT=ws://localhost:9000/ws` | 19:22 |
| ijohnson[m] | just the original design I guess, why is it a problem for you ? | 19:22 |
| om26er | The environment variables are generated by a mobile app, which are also going to be used by other clients. So it would have been "simpler" to ask people to just run `snap set foo ENV_VAR=VALUE` | 19:24 |
| om26er | We might have to do some special casing for snaps as those keys would have to be redefined | 19:25 |
| om26er | i.e. CLOUD_USER_ENDPOINT --> CLOUD-USER-ENDPOINT | 19:26 |
| ijohnson[m] | yeah that would be the easiest thing to do, but probably also worth filing a bug against snapd for allowing _ in key names, the key names just get serialized as json strings in an object so _ is not a problem serialization | 19:27 |
| mup | PR core20#109 closed: hooks: add bootchart configuration <Created by alfonsosanchezbeato> <Merged by xnox> <https://github.com/snapcore/core20/pull/109> | 19:30 |
| om26er | launchpad or forum ? | 19:30 |
| ijohnson[m] | launchpad | 19:31 |
| mup | Issue core20#111 opened: Convert hooks/024-configure-bootchart.chroot to static files <Created by xnox> <https://github.com/snapcore/core20/issues/111> | 19:35 |
| om26er | Reported https://bugs.launchpad.net/snapd/+bug/1942367 | 19:36 |
| mup | Bug #1942367: Support underscore in 'snap set` keys <snapd:New> <https://launchpad.net/bugs/1942367> | 19:36 |
| * cachio_ afk -> doc app | 19:40 | |
| cachio_ | ijohnson[m], hey | 19:41 |
| cachio_ | ijohnson[m], about the #10708 which fails google:ubuntu-20.04-64:tests/main/interfaces-libvirt | 19:41 |
| mup | Bug #10708: syck_0.42-3(ia64/unstable): FTBFS: test failures <syck (Ubuntu):Fix Released by lamont> <syck (Debian):Fix Released> <https://launchpad.net/bugs/10708> | 19:41 |
| mup | PR #10708: tests/lib/prepare.sh: use core20 from beta channel temporarily <⚠ Critical> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10708> | 19:41 |
| ijohnson[m] | hey cachio_ | 19:41 |
| om26er | Can a snap read values from `/etc/environment` if run as a daemon ? | 19:41 |
| cachio_ | I created a pr to fix that but stillnot working | 19:42 |
| ijohnson[m] | om26er: it should automatically inherit from /etc/environment if it is a daemon | 19:42 |
| cachio_ | to workaround that we need to revert the test interfaces-libvirt to be executed on xenial | 19:42 |
| ijohnson[m] | cachio_: what do you mean ? | 19:42 |
| cachio_ | ijohnson[m], google:ubuntu-20.04-64:tests/main/interfaces-libvirt | 19:42 |
| cachio_ | that test was updated | 19:43 |
| cachio_ | initially was running on xenial | 19:43 |
| cachio_ | but few days ago I moved that to focal | 19:43 |
| ijohnson[m] | cachio_: right did you see my comment ? mvo reverted that test to xenial so master is unblocked | 19:43 |
| cachio_ | in a PR | 19:43 |
| cachio_ | ah, nice | 19:43 |
| ijohnson[m] | cachio_: still no idea what's wrong with running the test on focal though, I haven't looked at that test very much | 19:44 |
| cachio_ | didn't see that comment | 19:44 |
| cachio_ | ijohnson[m], I left a comment here https://github.com/snapcore/snapd/pull/10717#issuecomment-910529226 | 19:44 |
| mup | PR #10717: tests: fix interfaces-libvirt test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10717> | 19:44 |
| cachio_ | with the error I see | 19:45 |
| cachio_ | I need to go to the doctor now, I'll continue with that one later | 19:45 |
| ijohnson[m] | ack | 19:47 |
| mup | PR snapd#10708 closed: tests/lib/prepare.sh: use core20 from beta channel temporarily <⚠ Critical> <Simple 😃> <Created by anonymouse64> <Closed by anonymouse64> <https://github.com/snapcore/snapd/pull/10708> | 21:06 |
| mup | PR snapd#10718 opened: tests/lib/prepare.sh: download core20 for UC20 runs via BASE_CHANNEL <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10718> | 21:06 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!