| mup | PR snapcraft#3064 closed: cmake v2 plugin: rename configflags to cmake-parameters <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3064> | 01:16 |
|---|---|---|
| mup | PR snapcraft#3066 closed: autotools v2 plugin: rename configflags to autotools-configure-parame… <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3066> | 01:16 |
| mup | PR snapcraft#3068 opened: cmake v2 plugin: default to -DCMAKE_BUILD_TYPE=Release <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3068> | 02:16 |
| mborzecki | morning | 05:48 |
| zyga | o/ | 06:08 |
| mvo | hey zyga | 06:09 |
| zyga | hey mvo | 06:09 |
| * zyga needs coffee | 06:14 | |
| mup | PR snapd#8526 closed: image: improve/adjust DownloadSnap doc comment <Simple 😃> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8526> | 06:14 |
| mup | PR snapd#8532 opened: tests: install new snapd deb into preseed image <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8532> | 06:15 |
| mborzecki | zyga: mvo: morning guys | 06:17 |
| mborzecki | zyga: yeah, coffee sounds like a plan | 06:17 |
| * zyga was unable to sleep last night, collapsed after 2AM | 06:18 | |
| mborzecki | coffee iv then ;) | 06:18 |
| mborzecki | s/coffee/caffeine/ | 06:18 |
| mborzecki | in the meantime: https://forum.snapcraft.io/t/centos-8-snapd-install-of-authy-doesnt-show-up-in-applications-folder/16684/5 despite all the tricks we pull, looks like XDG_DATA_DIRS is still not in the env sometimes | 06:19 |
| mborzecki | funnily enough, it's in the terminal, but that's probably a result of /etc/profile.d bits | 06:20 |
| mborzecki | mvo: do you want to take a look at the latest changes in #8487 before i land it? | 06:21 |
| mup | PR #8487: gadget, cmd/snap-bootstrap: MBR schema support <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8487> | 06:21 |
| mvo | mborzecki: let me quickly look | 06:22 |
| mup | PR snapd#8487 closed: gadget, cmd/snap-bootstrap: MBR schema support <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8487> | 06:24 |
| mborzecki | yay! | 06:26 |
| mup | PR snapd#8529 closed: cmd/snap-bootstrap/initramfs-mounts: support EnvRefExtractedKernelBootloader's <UC20> <⚠ Critical> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8529> | 06:33 |
| zyga | re | 06:42 |
| zyga | 0.5 coffee | 06:43 |
| mup | PR core20#46 opened: static: add new handle_writable_defaults() to handle-writable-paths <Created by mvo5> <https://github.com/snapcore/core20/pull/46> | 06:50 |
| pstolowski | morning | 06:59 |
| mvo | pstolowski: good morning - just fyi, i pushed https://github.com/snapcore/core20/pull/46 | 07:03 |
| mup | PR core20#46: static: add new handle_writable_defaults() to handle-writable-paths <Created by mvo5> <https://github.com/snapcore/core20/pull/46> | 07:03 |
| pstolowski | mvo: great, thank you | 07:04 |
| * pstolowski is going to look at the 20.04 preseed test hang issue in a few moments | 07:05 | |
| pedronis | pstolowski: the issue is that the 20.04 cloud images have snapd snap, not the core snap now | 07:09 |
| pedronis | but I'm not even sure we need the hack or not (do they have 2.44 ? now) | 07:10 |
| pstolowski | looking | 07:16 |
| zyga | pstolowski, pedronis: I fixed that locally | 07:21 |
| zyga | I'll send an update to my PR in a moment | 07:21 |
| pstolowski | zyga: yay! | 07:23 |
| zyga | fruit of last night sleeplessness | 07:23 |
| zyga | pstolowski: one interesting thing that changed is that the revisions are no longer unset (in the task log) | 07:26 |
| zyga | is that expected? | 07:26 |
| zyga | or is that a consequence of just using asserted blobs | 07:26 |
| pstolowski | zyga: if the hack is removed and we no longer taint the snap then yes | 07:27 |
| zyga | thanks, I was wondering about that | 07:27 |
| pstolowski | zyga: if the test was failing because of that then i suppose hanging was (again) caused by restore/qemu-nbd stuff | 07:28 |
| zyga | pstolowski: actually I never got to the bottom of why it hanged | 07:28 |
| zyga | pstolowski: I disabled the call to setup_preseeding and adjusted the MATCH rules below to reflect new reality | 07:28 |
| zyga | weird | 07:28 |
| zyga | interactively it would not hang | 07:29 |
| pstolowski | zyga: yes the hang is misterious and i couldn't figure it out before, it's the 3rd time it bites | 07:30 |
| pstolowski | zyga: it is something in the qemu cleanup | 07:31 |
| zyga | hmmm | 07:31 |
| zyga | I think I see what it may have hanged | 07:31 |
| zyga | root 20047 0.7 0.2 466144 8244 ? Ssl 07:31 0:00 qemu-nbd -c /dev/nbd0 cloudimg.img | 07:31 |
| zyga | this process must die | 07:31 |
| pstolowski | zyga: as you say interactively it works | 07:32 |
| zyga | right but this process must die :) | 07:32 |
| pstolowski | zyga: but qemu-nbd -d should do it afaiu? | 07:32 |
| zyga | yes | 07:32 |
| pedronis | mvo: hi, I reviewed Ian's #8512 | 07:32 |
| mup | PR #8512: boot/bootstate20: add EnvRefExtractedKernelBootloader bootstate20 implementation <UC20> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8512> | 07:32 |
| zyga | oh drat, I passed -shell again | 07:32 |
| zyga | pstolowski: I figured it out | 07:34 |
| zyga | pstolowski: such a simple mistake :DDDD | 07:34 |
| pstolowski | \o/ | 07:34 |
| pstolowski | /o\ | 07:34 |
| zyga | look at restore | 07:34 |
| zyga | what is missing? | 07:34 |
| zyga | the test just failed | 07:34 |
| zyga | we are running restore | 07:34 |
| zyga | reset test is equally broken | 07:37 |
| pstolowski | yeah i'm not surprised, they reuse same patterns | 07:38 |
| pstolowski | i probably spent too much on these tests to see the problem :/ | 07:38 |
| pstolowski | what is it? | 07:38 |
| zyga | https://github.com/snapcore/snapd/pull/8528/files#diff-c6e01e719c875585769d5445f3982ecbR21 | 07:40 |
| zyga | restore ran on failure | 07:40 |
| mup | PR #8528: tests: naive fix for preseeding failure <Created by zyga> <https://github.com/snapcore/snapd/pull/8528> | 07:40 |
| pstolowski | zyga: noooooooo | 07:40 |
| zyga | eh sorry | 07:40 |
| zyga | https://github.com/snapcore/snapd/pull/8528/files#diff-c6e01e719c875585769d5445f3982ecbR31 | 07:40 |
| zyga | and ran umount_ubuntu_image without sourcing the definition | 07:41 |
| zyga | I made prepare/restore handle the mounting | 07:41 |
| pstolowski | drat | 07:41 |
| zyga | so it was that process :) | 07:42 |
| zyga | it was the only possibility | 07:42 |
| pstolowski | zyga: thank you! | 07:42 |
| zyga | happy insomnia was productive | 07:42 |
| pstolowski | manual/preseed may need a tweak too | 07:44 |
| zyga | pstolowski: thanks, I'll look | 07:44 |
| zyga | done | 07:46 |
| pedronis | mvo: I did a pass on #8530 as well, some questions there | 07:50 |
| mup | PR #8530: boot: enable makeBootable20RunMode for EnvRefExtractedKernel bootloaders <UC20> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8530> | 07:50 |
| pstolowski | zyga: let me know when i should review | 07:51 |
| zyga | pstolowski: sure | 07:51 |
| zyga | pstolowski: I'll open the draft soon | 07:51 |
| mvo | pedronis: thanks, looking | 07:52 |
| pstolowski | zyga: we need to be careful about 19.10 in these tests, it may not be updated the same way | 07:55 |
| zyga | pstolowski: yeah, I'm running the more complete set to see what fails | 07:56 |
| pedronis | mvo: mmh, I didn't notice #8531, I made a comment there, we also to think how to keep the info around for run mode | 08:18 |
| mup | PR #8531: secboot,cmd/snap-bootstrap: add model to pcr protection profile <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8531> | 08:18 |
| pstolowski | zyga: jamesh proposed a PR for preseed test as well | 08:27 |
| jamesh | pstolowski: mine was mainly just exploring to see if that might fix it | 08:28 |
| jamesh | pstolowski: one of my other PRs changes some packaging (adding some directories), which caused the preseed-reset test to fail since the image had an old snapd in it | 08:29 |
| pstolowski | jamesh: i see. yes, that test is very picky about snapd directories, i updated it recently for 19.10 vs 20.04 | 08:30 |
| jamesh | pstolowski: the comments about certain images containing too old versions made me think it would be better to test the image with the snapd version under test installed. | 08:31 |
| jamesh | pstolowski: although that resulted in some complaints about some files under /var/cache/apparmor | 08:32 |
| pstolowski | jamesh: this is certainly a fair point, otoh it's good to test that we support existing ubuntu images (once they have recent enough snapd but lag a little bit). it's the transition period that's a little annoying but it should stabilize soon | 08:37 |
| jamesh | pstolowski: it seems like the kind of thing that will destabilise any time snapd changes sufficiently though. | 08:39 |
| pstolowski | jamesh: maybe i need to rethink this test; maybe it could look into debian dirs to see what's actually expected atfer a reset | 08:44 |
| jamesh | pstolowski: would that be the debian dir of the snapd under test or the old snapd in the image? | 08:45 |
| zyga | pstolowski: I have one more tweak that addresses the hang | 08:51 |
| pstolowski | jamesh: hmm that's a good question. current reset code is "cheating" and has a simplified view of that and resets for the snapd under test | 08:51 |
| jamesh | pstolowski: that's probably good enough in practice though: a system you reset is eventually going to get the new snapd anyway | 08:52 |
| pstolowski | jamesh: true, and the key point is that we don't leave files around (empty directories should be harmless) | 09:03 |
| zyga | pstolowski: hopefully this will shed some light as to why it was hanging and how to address such problems in general | 09:11 |
| zyga | pstolowski: (earlier I spoke too soon, the small mistake in the restore was not everything) | 09:11 |
| pstolowski | zyga: mhm | 09:11 |
| pstolowski | zyga: thank you for looking into it. a fresh look at these tests helps | 09:12 |
| zyga | pstolowski: it passed now | 09:17 |
| zyga | pstolowski: I'll run the rest | 09:18 |
| pstolowski | nice | 09:18 |
| zyga | I have a small problem at home | 09:22 |
| zyga | my internet quota for the month is almost gone | 09:22 |
| zyga | I have a backup but I need to find the extra modem and connect everything :/ | 09:23 |
| zyga | pstolowski: you may find this interesting https://github.com/snapcore/snapd/pull/8528/files#diff-2a7ec7462b6ff35150bfb39e2b01fc83R18 | 09:28 |
| mup | PR #8528: tests: fix for pre-seeding failures <Created by zyga> <https://github.com/snapcore/snapd/pull/8528> | 09:28 |
| pstolowski | zyga: woot. it affects ssh? | 09:29 |
| pstolowski | ? | 09:29 |
| zyga | the other way around | 09:29 |
| zyga | maybe I should rewrite the comment | 09:29 |
| zyga | and there's a typo there | 09:29 |
| zyga | blargh :) | 09:29 |
| pstolowski | zyga: do you know why is it a problem? | 09:30 |
| zyga | yes | 09:37 |
| zyga | one sec, cannot typpe | 09:38 |
| zyga | pstolowski: because each stuff spread runs is done so in a new ssh session: https://github.com/snapcore/spread/blob/master/spread/client.go#L337 | 09:41 |
| zyga | that session will terminate when the remote side descriptors are closed | 09:42 |
| zyga | but they are not closed because they get inherited into "background" tasks | 09:42 |
| zyga | systemd-run handles descriptors and closes anything that is not explicitly forwarded | 09:42 |
| zyga | pstolowski: please look https://github.com/snapcore/snapd/pull/8528 | 09:50 |
| mup | PR #8528: tests: fix for pre-seeding failures <Created by zyga> <https://github.com/snapcore/snapd/pull/8528> | 09:50 |
| pstolowski | zyga: i see, thank you | 09:51 |
| zyga | jamesh: I'll push the change you requested along with the shellcheck quote fix in a moment, I just want to see it pass for real | 10:01 |
| zyga | jamesh: pushed | 10:10 |
| zyga | mborzecki: there's a failure in selinux lxd test on centos8 | 10:21 |
| zyga | mborzecki: IIRC it is something new | 10:22 |
| zyga | I collected some samples last time I saw this | 10:22 |
| zyga | type=SYSCALL msg=audit(1587118309.956:1881): arch=c000003e syscall=9 success=yes exit=139877378064384 a0=0 a1=4caf68 a2=5 a3=802 items=0 ppid=1371 pid=2743 auid=4294967295 uid=0 gid=0 euid=0 suid=0 fsuid=0 egid=0 sgid=0 fsgid=0 tty=(none) ses=4294967295 comm="snap-update-ns" exe="/usr/libexec/snapd/snap-update-ns" subj=system_u:system_r:snappy_mount_t:s0 key=(null) | 10:22 |
| zyga | type=AVC msg=audit(1587118309.956:1881): avc: denied { execute } for pid=2743 comm="snap-update-ns" path="/usr/lib/x86_64-linux-gnu/libcrypto.so.1.1" dev="loop2" ino=5726 scontext=system_u:system_r:snappy_mount_t:s0 tcontext=system_u:object_r:snappy_snap_t:s0 tclass=file permissive=1 | 10:22 |
| zyga | type=AVC msg=audit(1587118309.956:1881): avc: denied { map } for pid=2743 comm="snap-update-ns" path="/usr/lib/x86_64-linux-gnu/libcrypto.so.1.1" dev="loop2" ino=5726 scontext=system_u:system_r:snappy_mount_t:s0 tcontext=system_u:object_r:snappy_snap_t:s0 tclass=file permissive=1 | 10:22 |
| zyga | mmap? | 10:23 |
| mborzecki | hmm probably wonder why libcrypto got pulled in | 10:27 |
| mborzecki | zyga: iirc snappy_mount_t should have all the permissions to execute/mmap/open from snappy_snap_t | 10:28 |
| mborzecki | zyga: we don't expect it to be statically compiled | 10:28 |
| zyga | mborzecki: no idea, maybe more nsswitch? | 10:28 |
| mborzecki | zyga: do you have shell? can you ldd /usr/libexec/snapd/snap-update-ns? | 10:33 |
| zyga | no I don't -- that was last week | 10:33 |
| zyga | it's not a priority, centos8 is in the unstable group | 10:34 |
| mborzecki | hm so crazy idea abou tthe native endian check | 10:44 |
| mborzecki | zyga: how about we have a dbustest2_amd64.go with `func init() { nativeEndian = binary.LittleEndian } dbustest2_other.go that just sets nativeendian = whatever? | 10:45 |
| zyga | mborzecki: happy to in a follow up | 10:51 |
| zyga | I'm trying to get the big one merged | 10:51 |
| zyga | mborzecki: I was thinking that endianness could be a package somewhere | 10:51 |
| mborzecki | zyga: nothing urgent, you could just leave it as it is now | 10:51 |
| zyga | as its not just this place | 10:51 |
| pstolowski | mvo: one more annoynace... https://github.com/snapcore/core-build/pull/59/files#r412081969 | 10:54 |
| mup | PR core-build#59: initramfs: add new handle_writable_defaults() <Created by mvo5> <https://github.com/snapcore/core-build/pull/59> | 10:55 |
| pstolowski | mvo: it's shocking how easy it is to get things wrong in shell | 10:55 |
| mvo | pstolowski: thnaks! | 10:56 |
| * zyga focuses on review for jamesh | 10:56 | |
| zyga | pstolowski: cp "$srcpath/"* | 10:57 |
| mvo | pstolowski: yeah, plus missing tests :/ I think this needs a test, I will have a look | 10:57 |
| pstolowski | zyga: right | 10:57 |
| mup | PR core20#46 closed: static: add new handle_writable_defaults() to handle-writable-paths <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/core20/pull/46> | 10:57 |
| zyga | pstolowski: the preseed test has different behavior in 19.10 and 20.04 | 11:08 |
| pedronis | mborzecki: not there is really a question whether it should be boot/uboot or just uboot, we don't have boot/uboot in boot or do we? do we want boot/boot/uboot at runtime? | 11:08 |
| zyga | because of core / snapd | 11:08 |
| zyga | pstolowski: I kind of switched gears now so if you want to attack that feel free | 11:09 |
| pstolowski | zyga: yes, this is expected (if you mean the MATCH parts), that's what I tried to convey before | 11:10 |
| zyga | I forgot to run the suite on 19.10 again | 11:10 |
| zyga | oh well :) | 11:10 |
| pstolowski | zyga: sure i can take it from you and fix that | 11:10 |
| zyga | pstolowski: thanks! | 11:10 |
| pstolowski | zyga: ty! | 11:11 |
| zyga | pstolowski: I guess it's just the match and count that has to change | 11:11 |
| pstolowski | zyga: have you pushed everything? | 11:11 |
| zyga | pstolowski: yes | 11:11 |
| pstolowski | zyga: yes, i think we need big if/else there and keep the old matches | 11:11 |
| zyga | indeed | 11:11 |
| pstolowski | zyga: okay, i need lunch soon, will get to it after standup i suppose | 11:11 |
| pstolowski | where do we assign bugs to core, LP? | 11:12 |
| zyga | actually github | 11:13 |
| zyga | because $reasons | 11:13 |
| zyga | but also on lp | 11:13 |
| zyga | so dunno | 11:13 |
| pstolowski | mhm | 11:19 |
| mborzecki | pedronis: so with recovery|noslashboot: true you'd rather use `uboot` as basedir? | 11:22 |
| pedronis | mborzecki: maybe, I wonder what .scr expects and what we do in ubuntu-boot | 11:23 |
| mborzecki | pedronis: if i'm reading this right, ubuntu-seed has boot/uboot, but ubuntu-boot will be mounted at /boot, so we'll ahve /boot/boot/uboot? | 11:23 |
| pedronis | pedronis: yes, exactly, something seems off | 11:23 |
| pedronis | but I don't know what the .scr does exactly | 11:24 |
| mborzecki | pedronis:the bind mount is /run/mnt/ubuntu-boot -> /boot right? | 11:24 |
| pedronis | that's what I would expect yes | 11:24 |
| pedronis | I'm double checking how this looks on amd64 atm | 11:24 |
| mborzecki | pedronis: there's EFI/ubuntu iirc | 11:26 |
| zyga | jamesh: https://github.com/snapcore/snapd/pull/5822#pullrequestreview-395477722 | 11:32 |
| mup | PR #5822: wrappers: allow user mode systemd daemons <:birthday:> <Needs Samuele review> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/5822> | 11:32 |
| mup | PR snapd#8527 closed: seed: clearer errors for missing essential snapd or core snap <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8527> | 11:34 |
| zyga | mborzecki: FYI https://github.com/snapcore/snapd/pull/7825#discussion_r412110567 | 11:42 |
| mup | PR #7825: many: use transient scope for tracking apps and hooks <Security-High> <Created by zyga> <https://github.com/snapcore/snapd/pull/7825> | 11:42 |
| pedronis | mvo: mborzecki: google:ubuntu-core-20-64:tests/core/basic20 from master is getting stuck on me, it's printing Reboot on apr211119-149277 (google:ubuntu-core-20-64:tests/core/) is taking a while... since 20 mins+ | 12:02 |
| mborzecki | pedronis: probably failed in initramfs/grub | 12:03 |
| pedronis | did we break something? | 12:03 |
| mborzecki | it'd be great to ahve a way to store those artifacts | 12:03 |
| zyga | mborzecki: did you say artifacts? :) | 12:03 |
| pedronis | mborzecki: we have a new kernel from today in edge and beta fwiw | 12:09 |
| mborzecki | ayy | 12:10 |
| mborzecki | initramfs changes? :P | 12:10 |
| mborzecki | i'm checking out rhel8/centos for popey, i'll build and image and check what's going on after | 12:11 |
| mup | PR snapd#8533 opened: [WIP] image, tests: core18 early config test <Created by stolowski> <https://github.com/snapcore/snapd/pull/8533> | 12:19 |
| mup | PR snapcraft#2911 closed: [experimental] package-management repository configuration <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2911> | 12:23 |
| mup | PR snapd#8534 opened: tests: 16.04 and 18.04 now have mediating pulseaudio (again) <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8534> | 12:29 |
| zyga | google:ubuntu-16.04-64:tests/main/interfaces-audio-playback-record keeps failing :/ | 12:49 |
| zyga | aha | 12:50 |
| ijohnson | morning folks | 12:50 |
| zyga | I think I see why | 12:50 |
| zyga | https://github.com/snapcore/snapd/pull/8534#pullrequestreview-397296420 | 12:50 |
| pedronis | does #8534 by jdstrand relates? | 12:50 |
| zyga | hey ian :) | 12:50 |
| ijohnson | zyga: hey | 12:50 |
| mup | PR #8534: tests: 16.04 and 18.04 now have mediating pulseaudio (again) <Simple 😃> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8534> | 12:50 |
| zyga | pedronis: indeed! | 12:50 |
| pedronis | anyway everything is terrible | 12:51 |
| zyga | pedronis: I think people are great, despite the fact we are all stressed and the release is looming | 12:51 |
| ijohnson | what a great way to start my day | 12:52 |
| ijohnson | :-D | 12:52 |
| zyga | mvo: https://github.com/snapcore/snapd/pull/8534 is important, please get it into any . releases you may need | 12:52 |
| mup | PR #8534: tests: 16.04 and 18.04 now have mediating pulseaudio (again) <Simple 😃> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8534> | 12:52 |
| pedronis | ijohnson: hi, google:ubuntu-core-20-64:tests/core/basic20 doesn't seem to work on master atm | 12:52 |
| pedronis | there was a new kernel | 12:52 |
| pedronis | I'm trying to run it manually and it doesn't get out of reboot | 12:53 |
| ijohnson | ah | 12:53 |
| ijohnson | pedronis: I'll look into it | 12:53 |
| ijohnson | I don't know what the problem is, but I have a hunch it might be related to the initrd things from yesterday | 12:53 |
| mvo | zyga: oh, ok | 12:58 |
| zyga | cachio: add a cron timer to run tests on master once a day | 13:28 |
| zyga | cachio: it's better than nothing and costs very little | 13:28 |
| cachio | zyga, we can run using spread cron as welll | 13:29 |
| zyga | cachio: actions are better | 13:29 |
| zyga | cachio: they update master status | 13:29 |
| zyga | cachio: they notify everyone automatically | 13:30 |
| zyga | cachio: and they are managed in the repository | 13:30 |
| cachio | zyga, ok | 13:30 |
| zyga | cachio: you can then see it the status alongside each commit that was tested | 13:30 |
| zyga | mvo: can you please override https://github.com/snapcore/snapd/pull/8534 | 13:31 |
| mup | PR #8534: tests: 16.04 and 18.04 now have mediating pulseaudio (again) <Simple 😃> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8534> | 13:31 |
| cachio | zyga, ok | 13:32 |
| cachio | I'll add that so we have runs on master | 13:32 |
| mup | PR snapd#8534 closed: tests: 16.04 and 18.04 now have mediating pulseaudio (again) <Simple 😃> <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8534> | 13:41 |
| zyga | mborzecki: I tweaked the code, it's pretty interesting how everything can fail in various cases. I didn't push yet but I think this is "the" way | 14:26 |
| zyga | mborzecki: the thing I learned is that we may have a session bus | 14:26 |
| zyga | mborzecki: but it may be entirely malfunctioning | 14:27 |
| zyga | mborzecki: because socket activation of dbus is not always going to work | 14:27 |
| zyga | mborzecki: because some systems package that separately | 14:27 |
| zyga | :) | 14:27 |
| zyga | mborzecki: so I tweaked the logic to cope with "late" failure, after we get the connection, it may turn out to be just "bad" | 14:27 |
| zyga | mborzecki: I treat this the same as session bus being unavailable now, for uid == 0 go to system bus | 14:27 |
| zyga | in the end some things got simplified and all error and debug handling is in once spot | 14:28 |
| zyga | mvo: I need to run an errand soon, going to visit my parents to pick up the LTE modem they are not using, I ran out of my 1TB quota /o\ | 14:28 |
| roadmr | I don't know what's more mind-boggling, that you have a 1TB quota on LTE or that you exhausted it 😉 | 14:34 |
| mvo | zyga: ok, thanks for letting me know | 14:43 |
| mup | PR snapcraft#3069 opened: make v2 plugin: make use of make-parameters <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3069> | 14:44 |
| mup | PR snapcraft#3068 closed: cmake v2 plugin: default to -DCMAKE_BUILD_TYPE=Release <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/3068> | 14:59 |
| cmatsuoka | mvo: up and running again | 15:00 |
| ijohnson | cmatsuoka: isn't today supposed to be your day off :-P | 15:01 |
| mvo | cmatsuoka: great! | 15:01 |
| mvo | cmatsuoka: welcome back | 15:01 |
| cmatsuoka | ijohnson: I don't have many places to go actually | 15:01 |
| * cachio lunch | 15:01 | |
| ijohnson | cmatsuoka: haha fair enough | 15:01 |
| cmatsuoka | but it's possible that I could be a bit slower today | 15:01 |
| * zyga is AFK | 15:08 | |
| pedronis | #8512 needs a 2nd review | 15:10 |
| mup | PR #8512: boot/bootstate20: add EnvRefExtractedKernelBootloader bootstate20 implementation <UC20> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8512> | 15:10 |
| ijohnson | anybody else seeing github 500s? | 15:34 |
| roadmr | launchpad.net is perfectly stable ;) | 15:34 |
| cmatsuoka | ijohnson: is it the roadrunner cliff fall error page? | 15:37 |
| ijohnson | cmatsuoka: yes | 15:37 |
| cmatsuoka | ijohnson: saw it a couple of days ago but not today | 15:37 |
| mvo | ijohnson: ssame error, also pushing is very unhappy | 15:38 |
| ijohnson | yes frustrating | 15:39 |
| zyga | github is down? | 15:39 |
| zyga | I was trying to push a patch I wrote for libzt while in the car | 15:39 |
| zyga | and man | 15:39 |
| ijohnson | I dunno if it's down but it's real slow and not working well | 15:39 |
| zyga | interestingly I get new IP addresses for each git push | 15:39 |
| zyga | maybe they are scrambling to deploy | 15:39 |
| zyga | "minor service outage" | 15:40 |
| zyga | oh well | 15:40 |
| * zyga looks through the window instead | 15:40 | |
| cmatsuoka | maybe with the lockdowns everyone got bored and started to use github | 15:40 |
| zyga | maybe they did their planned outage testing but nobody was home ;) | 15:41 |
| pstolowski | i'm getting weird failures with GC ^on spread prepare | 15:42 |
| zyga | the day that azure, gce and aws died ;) | 15:42 |
| roadmr | :( | 15:42 |
| cmatsuoka | ijohnson: unicorn error page for me now | 15:47 |
| ijohnson | yeah I can't get to any real page now, it's just the cliff or a unicorn | 15:50 |
| roadmr | or a clifficorn | 15:51 |
| cmatsuoka | that would be scary! | 15:52 |
| pstolowski | anyone with rhel or centos to try https://bugs.launchpad.net/snapd/+bug/1872368 (but docker may play a role in this)? | 16:00 |
| mup | Bug #1872368: Cannot enable snapd.socket in RHEL 7.8 <redhat> <rhel> <systemd> <snapd:Triaged> <https://launchpad.net/bugs/1872368> | 16:00 |
| ijohnson | pstolowski: if they are running inside docker then this is 100% not our bug to fix | 16:01 |
| pstolowski | ijohnson: thanks for commenting on it! | 16:06 |
| mup | PR # closed: core-build#11, core-build#22, core-build#26, core-build#37, core-build#51, core-build#58, core-build#59 | 16:10 |
| mup | Issue # opened: core18#56, core18#86, core18#89, core18#117, core18#129, core18#137, core18#142, core18#145, core18#151 | 16:20 |
| mup | PR # opened: core18#43, core18#72, core18#90, core18#126, core18#134, core18#144, core18#146, core18#148 | 16:20 |
| mup | PR # closed: snapd#5822, snapd#6258, snapd#7129, snapd#7168, snapd#7205, snapd#7414, snapd#7417, snapd#7436, snapd#7570, snapd#7586, snapd#7590, snapd#7603, snapd#7614, snapd#7700, snapd#7728, snapd#7825, snapd#7869, snapd#7900, snapd#7927, snapd#7982, snapd#8079, snapd#8123, snapd#8143, | 16:20 |
| mup | snapd#8247, snapd#8271, snapd#8301, snapd#8317, snapd#8334, snapd#8340, snapd#8351, snapd#8352, snapd#8366, snapd#8395, snapd#8398, snapd#8400, snapd#8414, snapd#8424, snapd#8436, snapd#8455, snapd#8464, snapd#8468, snapd#8475, snapd#8496, snapd#8499, snapd#8508, snapd#8512, snapd#8513, snapd#8515, | 16:20 |
| mup | snapd#8519, snapd#8520, snapd#8521, snapd#8528, snapd#8530, snapd#8531, snapd#8532, snapd#8533, snapd#8535 | 16:20 |
| pstolowski | mhm | 16:22 |
| cmatsuoka | ijohnson: now it's back, but not reflecting the current repository state | 16:27 |
| ijohnson | cmatsuoka: hmm | 16:27 |
| mup | PR core20#47 opened: hooks: use explicit /sbin/init to run systemd multi-user.target <Created by anonymouse64> <https://github.com/snapcore/core20/pull/47> | 16:28 |
| mvo | pstolowski: fwiw, I updated the core18 initramfs pr and hope to get to zyga pr later tonight | 16:28 |
| ijohnson | mvo do you know if https://github.com/snapcore/snapd/pull/8529 has landed in whatever ppa ubuntu-core-initramfs builds from ? | 16:29 |
| mup | PR #8529: cmd/snap-bootstrap/initramfs-mounts: support EnvRefExtractedKernelBootloader's <UC20> <⚠ Critical> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8529> | 16:29 |
| ijohnson | I thought there was some other ppa that we use when building ubuntu-core-initrmafs | 16:29 |
| ijohnson | like snappy-edge or something? | 16:29 |
| zyga | Mvo thank you | 16:34 |
| zyga | I’m returning home with additional modem to setup my backup network | 16:35 |
| pstolowski | mvo: ty | 16:39 |
| mup | PR snapcraft#3070 opened: plugins v2: update plugins so they have a similar behavior <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3070> | 16:43 |
| === luk3yx` is now known as luk3yx | ||
| mvo | ijohnson: unfortunately I don't know how the initrd gets the snap-bootstrap, I'm a bit in the dark about this | 17:15 |
| mvo | ijohnson: did you figure it out by now? | 17:15 |
| ijohnson | mvo: well the ubuntu-core-initramfs debian package is in the snappy image PPA | 17:16 |
| ijohnson | but I thought that it got snap-bootstrap from the snapd deb, but the snapd deb it got was from some other PPA | 17:16 |
| mvo | ijohnson: :( how does it get to this other ppa? | 17:18 |
| ijohnson | mvo: isn't there some other PPA that builds snapd? | 17:18 |
| * ijohnson doesn't know and doesn't remember | 17:18 | |
| * cmatsuoka will try to eat something, bbl | 17:19 | |
| mvo | ijohnson: there is ubuntu/snappy-dev/edge | 17:20 |
| * mvo tries to find the initramfs ppa | 17:20 | |
| * ijohnson -> lunch | 17:21 | |
| ijohnson | mvo ah that's the ppa I think | 17:21 |
| mvo | that one should be up-todat | 17:21 |
| mvo | e | 17:21 |
| mvo | last build there is 5h old | 17:21 |
| zyga | /me is back home | 17:32 |
| mup | PR core20#46 opened: static: add new handle_writable_defaults() to handle-writable-paths <Created by mvo5> <https://github.com/snapcore/core20/pull/46> | 17:43 |
| mup | PR snapcraft#3071 opened: repo: fix decoding of CalledProcessError output <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3071> | 17:49 |
| mup | PR snapcraft#3072 opened: schema: minor tweaks/fixes for package-repositories <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3072> | 18:01 |
| mup | PR snapcraft#3073 opened: storeapi: remove strict additionalProperties from store responses <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3073> | 18:07 |
| mup | PR core20#47 closed: hooks: use explicit /sbin/init to run systemd multi-user.target <Created by anonymouse64> <Closed by anonymouse64> <https://github.com/snapcore/core20/pull/47> | 18:29 |
| mup | PR snapcraft#3074 opened: specifications: add package-repositories draft spec <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3074> | 18:40 |
| mup | PR snapcraft#3070 closed: plugins v2: update plugins so they have a similar behavior <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3070> | 19:13 |
| mup | PR snapcraft#3072 closed: schema: minor tweaks/fixes for package-repositories <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3072> | 19:19 |
| mup | PR snapcraft#3071 closed: repo: fix decoding of CalledProcessError output <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3071> | 19:31 |
| === bdx1 is now known as bdx | ||
| mup | PR snapcraft#3073 closed: storeapi: remove strict additionalProperties from store responses <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3073> | 20:10 |
| mup | PR snapd#8536 opened: store,asserts,many: support the new action fetch-assertions <Created by pedronis> <https://github.com/snapcore/snapd/pull/8536> | 20:15 |
| mup | PR snapd#8537 opened: store: handle error-list in fetch-assertions results <Created by pedronis> <https://github.com/snapcore/snapd/pull/8537> | 20:17 |
| mup | PR snapcraft#3069 closed: make v2 plugin: make use of make-parameters <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3069> | 20:43 |
| mup | PR snapd#8538 opened: overlord: have a variant of Mock that can take a state.State <Created by pedronis> <https://github.com/snapcore/snapd/pull/8538> | 20:53 |
| === pedronis_ is now known as pedronis | ||
| pedronis | ijohnson: ^ 8538 is a small change I had to do locally to experiment with something, in case you wonder, it touches test code but is not used as such in the code base | 21:04 |
| ijohnson | pedronis: mmm interesting | 21:05 |
| pedronis | ijohnson: I'm going to call it a day | 21:06 |
| ijohnson | ttyl pedronis | 21:06 |
| mup | PR pc-amd64-gadget#43 closed: Clean up cmdline, after initrd&core land <Created by xnox> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/43> | 21:17 |
| mup | Bug #1874156 opened: Outdated docs on "Ubuntu security tips" for seccomp <snap-docs> <Snappy:New> <https://launchpad.net/bugs/1874156> | 23:09 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!