[00:13] <tianon> ijohnson: fwiw, I think needing to do all of https://github.com/snapcore/snapcraft/blob/4ac0faaf436a6d3e31fe06cd154c552b32b9bffd/snapcraft/internal/build_providers/_lxd/_lxd.py#L412-L470 myself is the biggest hurdle with creating the lxc container myself /o\
[00:14] <ijohnson> Oof yeah that seems like a lot, I don't remember needing to do all of that when I tried but that was also a release or two ago so maybe things have changed
[00:16] <tianon> error: cannot communicate with server: Put http://localhost/v2/snaps/system/conf: dial unix /run/snapd.socket: connect: no such file or directory
[00:16] <tianon> 27
[00:17] <tianon> (that's the error I get without doing those setup steps, and what led me to dig for them)
[00:29] <tianon> I'm in way over my head (as usual with these things, heh), but I *think* https://github.com/snapcore/snapcraft/blob/4ac0faaf436a6d3e31fe06cd154c552b32b9bffd/snapcraft/internal/build_providers/_lxd/_lxd.py#L197 needs an "architecture" field to enable what I'm trying "officially"
=== benfrancis2 is now known as benfrancis
[01:03] <tianon> got further down the rabbit hole by doing manual lxc bits: error: cannot install snap file: snap "snapd" supported architectures (amd64) are incompatible with this system (i386)
[01:04] <tianon> so I guess I need a more "holistic" i386 environment to make this work
[01:27] <mup> PR snapd#9952 opened: interfaces: allow reading the Xauthority file KDE Plasma writes for Wayland sessions <Simple 😃> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/9952>
[07:46] <mborzecki> morning
[08:03] <pstolowski> morning
[08:07] <mborzecki> pstolowski: hey
[08:15] <mvo> good morning mborzecki and pstolowski
[08:30] <mborzecki> mvo: hey, https://github.com/snapcore/snapd/pull/9952 looks like an easy fix, and we don't need a security review for it, do we?
[08:30] <mup> PR #9952: interfaces: allow reading the Xauthority file KDE Plasma writes for Wayland sessions <Simple 😃> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/9952>
[08:48] <mup> PR snapd#9941 closed: overlord: save fails with sockets in folder <Bug> <⛔ Blocked> <Created by mapero> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/9941>
[09:09] <mvo> mborzecki: looking
[09:20] <mvo> mborzecki: let's bring it up with alex, looks trivial and similar permissons are there a few lines above
[09:21] <mborzecki> mvo: yeah, and it matches the source code linked in the description
[09:39] <mup> PR snapd#9953 opened: overlord/snapshotstate: include the last message printed by tar in the error <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9953>
[10:00] <pedronis> mborzecki: do you have anything that needs my input atm?
[10:00] <mborzecki> pedronis: no, thanks
[10:49] <mup> PR snapd#9954 opened: snap: fix unit tests on Go 1.16 <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9954>
[10:53] <zyga> mborzecki +1
[10:56] <mvo> mborzecki: I added "skip-spread" there too, let's see if I'm not too late
[10:57] <mborzecki> mvo: too late probably, from what we've seen the label must be part of the event that triggers the action
[10:59] <mborzecki> closed, canceled all pending workflows and reopened, should be good now
[11:00] <mborzecki> weird that this didn't come up in the `latest/stable` go channel in the test matrix
[11:04] <dot-tobias> Is there a way to (manually) get a core dump for a single snap service command? I'm trying to debug why my WPE WebKit kiosk snap crashes on certain pages, have a hunch it has something to do with missing/misconfigured graphics stack … G_DEBUG_MESSAGES=all doesn't provide details, and WPE developers would then check the core dump.
[11:04] <zyga> yeah, mborzecki is right about the label
[11:22] <mborzecki> dot-tobias: you can try snap run --gdb (or --gdbserver), core dumps were likely intercepted by systemd, so i'd check coredumpctl
[11:23] <mborzecki> some time rounding error in asserts|tests|daemon https://paste.ubuntu.com/p/47gXWB3rwt/ ?
[11:37] <dot-tobias> mborzecki: Thanks! I read about coredumpctl in a snapcraft forum thread, but that's only on classic systems AFAIK?
[11:37] <dot-tobias> (need to debug on a Core device)
[11:49] <zyga> mborzecki https://github.com/snapcore/snapd/pull/9954#pullrequestreview-596206395 is green
[11:49] <mup> PR #9954: snap: fix unit tests on Go 1.16 <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9954>
[11:49] <zyga> can I merge it? :)
[11:49] <mborzecki> zyga: go ahead
[11:49] <zyga> woot
[11:49] <zyga> some progress :)
[11:54] <mup> PR snapd#9954 closed: snap: fix unit tests on Go 1.16 <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by zyga> <https://github.com/snapcore/snapd/pull/9954>
[11:54] <mborzecki> thx
[12:16] <mborzecki> dot-tobias: hm coredumpctl may be missing from core*, but unless coredumps are disabled completely, there should be something under /var/lib/systemd/coredump iirc
[12:21] <pedronis> mborzecki: when you have time could you look at  https://github.com/snapcore/snapd/pull/9895
[12:21] <mup> PR #9895: gadget/many: rm, delay sector size + structure size checks to runtime <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9895>
=== msalvatore_ is now known as msalvatore
[12:35] <mup> PR snapcraft#3445 opened: Fix snap <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3445>
[12:52] <pstolowski> pedronis: when you commented about AddSequenceToUpdate & fetcher, you mean the same logic that exists in AddToUpdate right?
[12:52] <pedronis> pstolowski: yes
[12:52] <pedronis> pstolowski: as I said it can be done as a follow-up, the PR is already biggish
[12:52] <pstolowski> pedronis: thanks. I remember removing that initially due to confusion
[12:53] <pedronis> it avoids round trips with the store
[12:54] <pstolowski> pedronis: ok, will do in a followup. also, i pushed many changes to monitor PR but it also has pool changes so is big
[12:54] <pedronis> pstolowski: yea, I think we should try to get the pool one in first
[12:55] <pedronis> pstolowski: you can leave a TODO I suppose inside AddSequenceToUpdate
[12:57] <pstolowski> ok
[15:15] <mup> PR snapcraft#3445 closed: Fix snap <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3445>
[15:56]  * cachio lunch
[16:50] <mup> PR snapcraft#3446 opened: extensions: suppress realpath noise on headless systems <Created by MrCarroll> <https://github.com/snapcore/snapcraft/pull/3446>
[16:52] <bandali> hey folks, could someone from the snapcraft review team please look my request from a few days ago? https://forum.snapcraft.io/t/request-for-auto-connection-of-jami-network-manager/22833
[17:05] <mup> PR snapcraft#3444 closed: snap: explicitly add setuptools-rust package to python-packages <Created by cjp256> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/3444>
=== pedronis_ is now known as pedronis
[17:25] <mup> PR snapcraft#3447 opened: snap: pin cryptography to 3.4 <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3447>
[17:28] <ijohnson> amurray: see bandali's question ^
[18:20] <mup> PR snapcraft#3447 closed: snap: pin cryptography to 3.4 <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3447>
=== ijohnson is now known as ijohnson|lunch
[20:10] <mup> PR snapcraft#3443 closed: extensions: add conditional for GNOME Makefile using bindtextdomain <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3443>
=== ijohnson|lunch is now known as ijohnson
[20:50] <mup> PR snapcraft#3442 closed: storeapi: reduce the amount of constants and their meaning <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3442>
=== genii_ is now known as genii
[21:26] <cachio> ijohnson, hey
[21:27] <ijohnson> hey cachio
[21:27] <cachio> don't merge preseed PR please
[21:27] <cachio> I already made a nicer fix
[21:27] <cachio> I am testing it
[21:27] <ijohnson> cachio: ok, I wasn't going to merge it, I was going to wait for a 2nd +1 anyways
[21:28] <cachio> ijohnson, ah, perfect
[21:28] <cachio> because I'll include other dbus dependencies to fix more tests
[21:31] <ijohnson> cachio: I'd prefer if you only fix the preseed tests there, and fix the dbus things in a different pr
[21:31] <ijohnson> smaller prs take more time to land, but are easier to review
[22:06] <mup> PR snapd#9955 opened: .github/workflows/labeler.yaml: try work-around to not sync labels <Run nested> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9955>
[22:33] <amurray> bandali: yep it's in the queue for the team to get to - I hope to take a look at it later today
[22:35] <bandali> amurray, many thanks!
[22:36] <cachio> ijohnson, I did it together because both just need dependencies
[22:36] <ijohnson> cachio: ok
[22:36] <cachio> ijohnson, so to avoid manual merge
[22:36] <cachio> I did it togeter
[22:36] <cachio> the change is super simple