=== cjwatson_ is now known as cjwatson
mupPR snapd#7755 closed: cmd/snap-failure: fallback to snapd from core, extend tests <Remodel ๐Ÿš‹> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7755>07:33
mupPR snapd#7769 opened: cmd/snap-failure: passthrough snapd logs, add informational logging <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7769>07:51
=== pstolowski|afk is now known as pstolowski
mborzeckipstolowski: hey08:17
mborzeckipstolowski: pekkari: can you take a look at https://github.com/snapcore/snapd/pull/7769 ? super simple09:46
mborzeckipedronis: ^^09:46
mupPR #7769: cmd/snap-failure: passthrough snapd logs, add informational logging <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7769>09:46
pedronisa 2nd review of #7764 would be great, it's largely new tests09:46
mupPR #7764: many: test various kinds of overriding for the snapd snap in Core 20 <Created by pedronis> <https://github.com/snapcore/snapd/pull/7764>09:46
mborzeckipedronis: sure, will do09:47
=== pedronis_ is now known as pedronis
zygamborzecki: I have something that I wanted to show you10:21
mborzeckizyga: meet?10:22
zygasure, hold on a moment10:22
pedronismborzecki: I have a question for you in 776210:25
zygamborzecki: https://github.com/zyga/snapd/commits/wip/dont-reuse-mounts and https://github.com/zyga/snapd/commits/fix/lp-1852361 are the branches of interest10:27
mborzeckipedronis: we have some code that looks up devices based on structure definition, but it might not be exported atm, need to double check10:28
pedronismborzecki: ok, thx10:29
zygamborzecki: can you join the standup meet please10:30
zygamborzecki: google:ubuntu-18.04-64:tests/regression/lp-185236110:40
mupPR snapd#7770 opened: testutil, many: make MockCommand() create prefix of absolute paths <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7770>10:47
joci have a amd64 device running a UC18 image on which i'm trying to install the docker snap, however if fails with:10:53
jocerror: cannot perform the following tasks:10:53
joc- Mount snap "docker" (423) (snap "docker" assumes unsupported features: snapd2.40 (try to refresh the core snap))10:53
ogradid you try to refresh the core snap yet ?10:54
jocaccording to snap list these are installed:10:55
joccore           16-2.42.1     8039  stable    canonicalโœ“  core10:55
joccore18         20191030      1265  stable    canonicalโœ“  base10:55
ograand what does "snap version" output (all lines please) ?10:56
joc$ snap version10:57
jocsnap    2.36.310:57
jocsnapd   2.36.310:57
jocseries  1610:57
jockernel  4.15.0-70-generic10:57
ograhmm, very interesting10:57
mupPR snapd#7771 opened: o/hookstate/ctlcmd: snapctl is-connected command <Created by stolowski> <https://github.com/snapcore/snapd/pull/7771>10:58
ograif you are on core18 snapd comes from the snapd snap ... try refreshing that10:58
ogra(though it should totally auto-refresh... not sure how it can be so behind while the core snaps seem to be nicely up to date)10:58
zygamborzecki: I pushed the comment changes again10:59
zygamborzecki: to both branches10:59
mborzeckizyga: ok10:59
zygamborzecki: I tg'd mvo to know about the situation10:59
zygamborzecki: the most important comment is https://github.com/snapcore/snapd/compare/master...zyga:wip/dont-reuse-mounts?expand=1#diff-6cdce42a784f927f710f0f5241a8c68dR41810:59
zygamborzecki: I also adjusted https://github.com/snapcore/snapd/compare/master...zyga:wip/dont-reuse-mounts?expand=1#diff-6cdce42a784f927f710f0f5241a8c68dR493 as mentioned during the call11:00
zygaI'm wrapping up for today11:00
zygagood luck guys!11:00
mborzeckizyga: cool, thanks11:00
mborzeckirunning spread with both regression tests11:02
jocsnapd did indeed update, thanks for that hint - now indeed i wonder how it managed to be behind given a `sudo snap refresh` was run before11:03
zygaondra: ^^^11:04
zygaondra: this is the stuff that fixes things blocking you11:04
zygaondra: if you take the diff from https://github.com/snapcore/snapd/compare/master...zyga:fix/lp-1852361?expand=111:05
zygaondra: and build snapd with it11:05
zygaondra: and confirm it works on your devices11:05
zygaondra: that would be a useful data point11:05
zygaondra: we need to discuss and properly package that for release (package as in do more code changes where this can safely land behind a feature flag)11:05
zygaondra: but the essence is the same11:06
mupPR snapd#7772 opened: wrappers: write and undo snapd services on core <Remodel ๐Ÿš‹> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7772>11:18
ondrazyga on it :)11:47
* pstolowski lunch11:50
mupPR snapd#7773 opened: cmd/snap-update-ns: fix overlapping, nested writable mimic handling <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7773>12:13
mborzeckizyga: pedronis: ^^ opened the branch, added some description to explain the problem, both spread tests are passing, i've looked the mount ns after the tests and things seemed fine12:14
mupPR snapcraft#2817 opened: snapcraft/plugins: Add gomod plugin <Created by mhilton> <https://github.com/snapcore/snapcraft/pull/2817>12:21
mupPR snapd#7769 closed: cmd/snap-failure: passthrough snapd logs, add informational logging <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7769>12:28
mborzeckipedronis: do you want to have a quick chat about 7769?12:34
pedronismborzecki: you mean 7773 ?12:35
mborzeckipedronis: right, 777312:36
pedronismborzecki: let's have a chat12:36
pedronismborzecki: I'm in the standup12:37
=== ricab is now known as ricab|lunch
ackkhi, with snappy-debug I see the following message from the sna maas snap: https://paste.ubuntu.com/p/X7pKKCqRCc/ is there a way to tell what's that mknod about?13:38
mupPR snapcraft#2814 closed: Remove gsettings from comment in kde extension <Created by hellsworth> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2814>13:51
=== ricab|lunch is now known as ricab
mupPR snapcraft#2802 closed: uprev devel python package requirements <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2802>13:54
mupPR snapcraft#2804 closed: conda plugin: simplify source url/checksum handling <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2804>13:57
mupPR snapcraft#2808 closed: repo: fix fetch_binary()'s return type for deb repo <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2808>14:09
mborzeckiondra: did you manage to run zyga's branch?14:09
ondramborzecki yep, just did it14:09
ondramborzecki still failing14:09
mborzeckiondra: duh, logs? :)14:09
mborzeckiondra: are you sure you're using the right snapd?14:10
ondramborzecki https://pastebin.canonical.com/p/TBgSJ8QPcN/14:10
ondramborzecki this is build I'm using https://code.launchpad.net/~ondrak/+snap/snapd-fix-lp-185236114:11
mborzeckiondra: ha, that's a new thing then, before it failed with EBUSY when trying to roll back some mount changes14:11
mborzeckizyga: ^^14:11
ondramborzecki yep, error is different14:11
ondramborzecki and i can confirm snap    2.42.2+git1279.g9c608d914:11
ondramborzecki which is commit from that branch I used for build14:12
mupPR snapcraft#2806 closed: cli: address type errors for mypy uprev  <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2806>14:12
ondramborzecki happy to test more fixes :)14:12
mborzeckiondra: can you set SNAPD_DEBUG in snapd, restart snapd and try to install the snap again?14:19
ondramborzecki you mean refresh?14:19
mborzeckiondra: yes14:19
mborzeckiondra: you can drop an ovverride in /etc/systemd/system/snap.service.d/local.conf with [Service]\nEnvironment=SNAPD_DEBUG=1\n14:22
ondramborzecki it's fine I updated snapd.service file14:23
ondramborzecki https://pastebin.canonical.com/p/YKKKKjV32G/14:25
mupPR snapd#7764 closed: many: test various kinds of overriding for the snapd snap in Core 20 <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7764>14:29
mupPR snapcraft#2807 closed: meta: address errors from mypy uprev <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2807>14:30
mborzeckiondra: interesting, is this the same device that failed the refresh before? was it restarted since then?14:45
ondramborzecki yeah same devce14:57
ondramborzecki before test I first tested with stable snapd to make sure it's still broken14:57
ondramborzecki BTW when I manually stop services I was able to refresh ( without need to reboot)14:58
mborzeckiondra: ok, can you try this, stop whatever service is there, /usr/lib/snapd/snap-discard-ns <snap-name>; start the service, refresh the snap14:58
mborzeckiondra: services from that snap ofc14:59
ondramborzecki I cannot restart services they are failing with "cannot update snap namespace: permission denied"15:01
mborzeckiondra: you should be able to stop them15:02
mborzeckiondra: as in snap stop <snap> or via systemctl15:02
ondramborzecki I stopped them and discarded ns15:02
ondrabut I cannot restart them anymore15:02
ondramborzecki and I know refresh worked before when services are stopped15:03
ondraso running refresh with stopped services will not not help much15:03
mborzeckiondra: i suspected that the previous version of snap-update-ns left the mount ns in some half assed state, so the new one would fail but differently15:04
mborzeckiondra: can you add SNAPD_DEBUG=1 to the service environment in the unit file? (or via an override)15:13
* cachio lunch15:17
mupPR snapd#7774 opened: seed: proper support for optional snaps for Core 20 models <Created by pedronis> <https://github.com/snapcore/snapd/pull/7774>15:21
* zyga acks the results from ondra, asked follow up questions in private and resumes off-work activity 15:33
mupPR snapcraft#2818 opened: project: pivot `info` from ProjectInfo to Snap <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2818>15:42
ackkhi, does anyone know if it's possible to run sshd in a confined snap?16:15
ackkI get this error when trying to ssh in: "fatal: permanently_set_uid: was able to restore old [e]gid [preauth]"16:15
ograackk, during build ?16:53
ackkogra, no, when I try to ssh16:53
ograah, sorry ... i'm blinf16:53
ackkogra, I have ssh running (with some LD_PRELOAD) but I can't log in16:53
ograi think your prob isnt ssh but login/pam16:53
ograwhen trying to spawn a terminal for you16:54
ograhave you tried something like "ssh <host> uptime" (that doesnt spawn a shell but directly execs the command)16:55
ogratheoretically all you shoudl need for snapping ssh is network and network-bind plugs ... but there is more to a full login than just ssh ...16:58
=== pstolowski is now known as pstolowski|afk
cjwatsonNot sure you can coherently say that the problem is login/pam when the error message above comes from sshd17:09
cjwatsonProbably need to LD_PRELOAD somewhat harder to do a better job of mimicking the uid swap (i.e. it needs to be impossible to swap back)17:10
mupPR snapd#7775 opened: seed: support extra snaps on top of Core 20 dangerous models <Created by pedronis> <https://github.com/snapcore/snapd/pull/7775>17:32
mupPR snapcraft#2805 closed: plugins: address type errors for mypy uprev <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2805>18:12
mupPR snapcraft#2811 closed: extractors: address errors from mypy uprev <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2811>18:12
mupPR snapcraft#2812 closed: store: address errors from mypy uprev <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2812>18:18
diddledancjwatson: LD_PRELOAD hard. no harder!18:21
* cachio afk18:32
ackkogra, yeah I tried a non-interactive command, same error18:58
ackkcjwatson, ah, I see18:59
ackkcjwatson, although, why does it try to change uid if I try to login as root?19:03
mupPR snapd#7776 opened: interfaces: add login-session-observe for who, {fail,last}log and loginctl <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7776>19:03
mupPR snapcraft#2819 opened: isort: automatic formatting/sorting of imports <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2819>19:07
* ijohnson unbroke all his snaps (including irccloud) yay19:07
mupPR pc-amd64-gadget#11 closed: Add mmx64.efi (MokManager) to support mokutil <Created by tsunghanliu> <Merged by vorlonofportland> <https://github.com/snapcore/pc-amd64-gadget/pull/11>19:12
mupPR snapcraft#2813 closed: pluginhandler: address errors from mypy uprev <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2813>19:22
mupPR snapd#7777 opened: snap-confine: suppress noisy classic snap file_inherit denials <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7777>19:27
mupPR snapcraft#2810 closed: build-providers: address errors from mypy uprev <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2810>19:46
mupPR snapd#7778 opened: seccomp: also allow 'chown snap_daemon:root' <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7778>19:49
jdstrandogra: hey, I have a note to add this: /sys/firmware/devicetree/base/system/linux,revision and /sys/firmware/devicetree/base/soc/ranges read access to hardware-observe (needs investigating; these are binary)19:54
jdstrandogra: but reads on /sys/firmware/** is already allowed via hardware-observe back in 78505e088b79e5ee68e01f23ba5d954dac97c520 (2016). do you recall what this is about?19:55
mupPR snapd#7779 opened: interfaces: misc updates for u2f-devices, browser-support, hardware-observe, et al <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7779>20:04
mupPR snapcraft#2816 closed: assorted: address errors from mypy uprev <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2816>20:25
mupPR snapcraft#2809 closed: project-loader: address errors from mypy uprev <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2809>20:28
mupPR snapcraft#2820 opened: uprev mypy to 0.740! (and address remaining errors) <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2820>20:49
ograjdstrand, various adafruit libs that are shipped with HW sensors try to access it for whatever reason21:33

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!