| mup | PR snapd#9168 opened: o/hookstate/ctlcmd: make is-connected check whether the plug or slot exists <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/9168> | 07:03 |
|---|---|---|
| zyga | mvo hello | 07:56 |
| zyga | mvo I'll be off today, Lucy had fever all night and my wife has some errands so I'm just not available | 07:58 |
| zyga | if there's something you want to talk about we can do a short 1:1 | 07:58 |
| zyga | or talk via messages | 07:58 |
| mvo | zyga: thanks, no worries | 07:59 |
| === seb128_ is now known as seb128 | ||
| mup | PR snapd#9169 opened: secboot,cmd/snap-bootstrap: don't import boot package from secboot <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/9169> | 12:55 |
| jdstrand | mvo: hey, fyi, I'm going to address feedback in PR 8920 and submit 1-2 'misc policy updates' PRs today/tomorrow | 13:49 |
| mup | PR #8920: interfaces: update cups-control and add cups for providing snaps <Needs Samuele review> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8920> | 13:49 |
| mvo | jdstrand: great, thanks you | 13:49 |
| jdstrand | mvo: I'll address the teardown item for PR 9120 now. for PR 9167, I thought about having that in wrappers/desktop.go and having the apparmor policy pull it out, but the more I thought about it, I agreed with James that it should be in snap info. if you want me to put it in wrappers, I can do that though | 13:50 |
| mup | PR #9120: interfaces: add kernel-crypto-api interface <Needs Samuele review> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/9120> | 13:50 |
| mup | PR #9167: many: correctly calculate the desktop file prefix everywhere <Created by jdstrand> <https://github.com/snapcore/snapd/pull/9167> | 13:50 |
| mvo | jdstrand: it's fine, no worries | 13:51 |
| jdstrand | mvo: you probably noticed that for anything going to master that I'd like in 2.46, I'm then creating a 2.46 PR. I'll be doing that with PR 9167 and PR 8301 | 13:51 |
| mup | PR #9167: many: correctly calculate the desktop file prefix everywhere <Created by jdstrand> <https://github.com/snapcore/snapd/pull/9167> | 13:51 |
| mup | PR #8301: interfaces/many: deny arbitrary desktop files and misc from /usr/share <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8301> | 13:51 |
| mvo | jdstrand: I think I agree with james here too plus we could always tweak later if e.g. Samuele feels strongly | 13:51 |
| jdstrand | mvo: assuming that works for you :) | 13:52 |
| mvo | jdstrand: thanks for that, that works | 13:52 |
| mvo | jdstrand: I may actually merge master into 2.46 because since we branched only changes we want there anyway have landed | 13:52 |
| jdstrand | mvo: yeah, I think what convinced me is when I saw some 3 different spots trying to calculate the desktop files | 13:52 |
| mvo | jdstrand: exactly | 13:53 |
| mvo | ijohnson, cmatsuoka a review for 9152 would be great if you have some spare cycles | 13:53 |
| ijohnson | mvo: sure I will look this morning then | 13:53 |
| cmatsuoka | mvo: sure, will check | 13:53 |
| ijohnson | mvo: I just looked at your review of 9010, thanks for the review the TODO there is fine, the TODO is just about moving something from the initrd fs layout units to snap-bootstrap itself, it's not a regression or missing functionality | 13:54 |
| ijohnson | mvo: I will push a patch changing that const name however | 13:54 |
| mvo | ijohnson: nice, thank you! | 13:56 |
| mvo | ijohnson: if there is no regression I will approve it, wasn't totally clear to me | 13:56 |
| ijohnson | thanks | 13:56 |
| mvo | ijohnson: feel free to do as a followup though, given that the PR is green and already long/big etc | 13:57 |
| ijohnson | mvo: sure that would actually be nicer to do as a followup then | 13:58 |
| mvo | ijohnson: and we want followups anyway | 13:58 |
| mvo | ijohnson: +1 | 13:58 |
| ijohnson | yes indeed | 13:58 |
| mvo | ijohnson: let's discuss in the uc20 meeting how to coordinate the landing with the kernel | 13:58 |
| ijohnson | mvo: ack I will wait to hit the big green button then just so we understand that | 13:58 |
| mvo | ijohnson: \o/ | 13:59 |
| mvo | ijohnson: all right, let's merge away I think :) | 14:23 |
| ijohnson | mvo: so from that meeting I am good to squash merge #9010 ? | 14:23 |
| mup | PR #9010: cmd/snap-bootstrap/initramfs-mounts: call systemd-mount instead of the-tool <Run nested> <Squash-merge> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9010> | 14:23 |
| * ijohnson clicks the button | 14:23 | |
| ijohnson | 🎉 | 14:24 |
| * mvo hugs ijohnson | 14:25 | |
| mup | PR snapd#9010 closed: cmd/snap-bootstrap/initramfs-mounts: call systemd-mount instead of the-tool <Run nested> <Squash-merge> <UC20> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9010> | 14:25 |
| mup | PR snapd#9170 opened: cmd/snap-bootstrap/initramfs-mounts: tweak names, add comments from previous PR <Simple 😃> <Skip spread> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9170> | 15:50 |
| mup | PR snapd#9169 closed: secboot,cmd/snap-bootstrap: don't import boot package from secboot <Simple 😃> <UC20> <Created by cmatsuoka> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9169> | 17:56 |
| MikeRL | Anyone know how to restart all snap related services under systemd? I'm running into an error. https://paste.ubuntu.com/p/BXyVY4sk4X/ | 18:31 |
| MikeRL | Hopefully this is the correct channel for snap related support/ | 18:31 |
| MikeRL | Also will apt-purging snapd cause all snaps to be deleted? | 18:32 |
| MikeRL | In the meantime, I will reinstall the snapd package. Via synaptic. Hopefully that stops the error. | 18:34 |
| ijohnson | MikeRL: you can try restarting snapd via `systemctl restart snapd` | 18:35 |
| ijohnson | MikeRL: removing the apt snap package will remove all your snaps, but not your snap data, but purging the package will also remove the snap data as well | 18:35 |
| MikeRL | ijohnson, Thanks. The reinstall fixed it. | 18:35 |
| MikeRL | snap list still shows snaps like chromium as there. | 18:36 |
| ijohnson | great glad to hear that | 18:36 |
| MikeRL | I saw that same error on a Microsoft page for Windows Subsystem for Linux. Weird. I got that error on a Pi 4 on Ubuntu Mate beta 2. | 18:37 |
| MikeRL | Wonder if there's any logs. Should I look for some? Where to look? | 18:37 |
| MikeRL | Maybe it's a bug? | 18:37 |
| ijohnson | MikeRL: if you have `journalctl --no-pager -u snapd` as well as `snap changes` that would be useful to include in a bug report on bugs.launchpad.net/snapd | 18:38 |
| MikeRL | ijohnson, Working on it. | 18:41 |
| MikeRL | Well I've hit an issue. Snapd doesn't allow me to report snaps without contact info. | 18:46 |
| MikeRL | I did this via ubuntu-bug. | 18:46 |
| MikeRL | https://bugs.launchpad.net/snapd/+bug/1891926 | 18:54 |
| mup | Bug #1891926: Snapd won't connect to internet <deb> <network> <snapd> <snapd:New> <https://launchpad.net/bugs/1891926> | 18:54 |
| MikeRL | I would've used a more descriptive title, but I'm uncertain to what a better one would be. | 18:54 |
| ijohnson | thanks MikeRL, could you also include `journalctl --no-pager -u snapd.socket` in that bug report if you're able? | 19:08 |
| MikeRL | On the way. | 19:10 |
| MikeRL | Done. | 19:15 |
| mup | PR snapd#9166 closed: interfaces/many: miscellaneous updates for strict microk8s - 2.46 <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9166> | 19:21 |
| mup | PR snapd#9170 closed: cmd/snap-bootstrap/initramfs-mounts: tweak names, add comments from previous PR <Simple 😃> <Skip spread> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9170> | 19:26 |
| mup | Issue core20#61 closed: /etc/writable is double mounted <Created by cmatsuoka> <Closed by cmatsuoka> <https://github.com/snapcore/core20/issues/61> | 21:44 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!