| 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:13 |
|---|---|---|
| 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:14 |
| 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:16 |
| tianon | (that's the error I get without doing those setup steps, and what led me to dig for them) | 00:17 |
| 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" | 00:29 |
| === benfrancis2 is now known as benfrancis | ||
| 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:03 |
| tianon | so I guess I need a more "holistic" i386 environment to make this work | 01:04 |
| 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> | 01:27 |
| mborzecki | morning | 07:46 |
| pstolowski | morning | 08:03 |
| mborzecki | pstolowski: hey | 08:07 |
| mvo | good morning mborzecki and pstolowski | 08:15 |
| 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:30 |
| 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> | 08:48 |
| mvo | mborzecki: looking | 09:09 |
| mvo | mborzecki: let's bring it up with alex, looks trivial and similar permissons are there a few lines above | 09:20 |
| mborzecki | mvo: yeah, and it matches the source code linked in the description | 09:21 |
| 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> | 09:39 |
| pedronis | mborzecki: do you have anything that needs my input atm? | 10:00 |
| mborzecki | pedronis: no, thanks | 10:00 |
| 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:49 |
| zyga | mborzecki +1 | 10:53 |
| mvo | mborzecki: I added "skip-spread" there too, let's see if I'm not too late | 10:56 |
| mborzecki | mvo: too late probably, from what we've seen the label must be part of the event that triggers the action | 10:57 |
| mborzecki | closed, canceled all pending workflows and reopened, should be good now | 10:59 |
| mborzecki | weird that this didn't come up in the `latest/stable` go channel in the test matrix | 11:00 |
| 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:04 |
| mborzecki | dot-tobias: you can try snap run --gdb (or --gdbserver), core dumps were likely intercepted by systemd, so i'd check coredumpctl | 11:22 |
| mborzecki | some time rounding error in asserts|tests|daemon https://paste.ubuntu.com/p/47gXWB3rwt/ ? | 11:23 |
| 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:37 |
| 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:49 |
| 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 | 11:54 |
| 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:16 |
| 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> | 12:21 |
| === msalvatore_ is now known as msalvatore | ||
| mup | PR snapcraft#3445 opened: Fix snap <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3445> | 12:35 |
| 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:52 |
| pedronis | it avoids round trips with the store | 12:53 |
| 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:54 |
| pedronis | pstolowski: you can leave a TODO I suppose inside AddSequenceToUpdate | 12:55 |
| pstolowski | ok | 12:57 |
| mup | PR snapcraft#3445 closed: Fix snap <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3445> | 15:15 |
| * cachio lunch | 15:56 | |
| mup | PR snapcraft#3446 opened: extensions: suppress realpath noise on headless systems <Created by MrCarroll> <https://github.com/snapcore/snapcraft/pull/3446> | 16:50 |
| 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 | 16:52 |
| 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> | 17:05 |
| === pedronis_ is now known as pedronis | ||
| mup | PR snapcraft#3447 opened: snap: pin cryptography to 3.4 <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3447> | 17:25 |
| ijohnson | amurray: see bandali's question ^ | 17:28 |
| mup | PR snapcraft#3447 closed: snap: pin cryptography to 3.4 <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3447> | 18:20 |
| === ijohnson is now known as ijohnson|lunch | ||
| 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> | 20:10 |
| === ijohnson|lunch is now known as ijohnson | ||
| 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> | 20:50 |
| === genii_ is now known as genii | ||
| cachio | ijohnson, hey | 21:26 |
| 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:27 |
| cachio | ijohnson, ah, perfect | 21:28 |
| cachio | because I'll include other dbus dependencies to fix more tests | 21:28 |
| 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 | 21:31 |
| 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:06 |
| 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:33 |
| bandali | amurray, many thanks! | 22:35 |
| 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 | 22:36 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!