/srv/irclogs.ubuntu.com/2019/10/25/#snappy.txt

mupPR snapcraft#2764 closed: Snapd doesn't actually honor the slot specified in the default-provider <Created by kenvandine> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2764>00:31
mvohey mbo05:24
mborzeckimorning05:36
mvohey mborzecki05:38
mborzeckimvo: PRs still red?05:39
mvomborzecki: looks better today05:39
mvomborzecki: at least the one from zyga 7h ago is green05:39
mborzeckiwth https://github.com/snapcore/snapd/pull/7647 has pending build status, but when you go to the details page it's actually green05:40
mupPR #7647: gadget: skip structures with MBR role during remodel <Remodel 🚋> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7647>05:40
mvomborzecki: oh no05:41
mborzeckimvo: i've restated the cla check job, maybe it's enough to switch the status :/05:41
mvomborzecki: the cla job is not finished05:42
mvomborzecki: which is slightly strange05:42
mvomborzecki: I can cancel and restart?05:42
mborzeckimvo: restarted it just now intentionally05:42
mvomborzecki: aha, ok05:42
mborzeckimvo: all the other jobs were green, and cla runs are shortest, hope that when it completes travis will report the status back to gh05:43
mvomborzecki: right, makes sense. if not, I will just merge and override the missing check05:43
mupPR snapd#7647 closed: gadget: skip structures with MBR role during remodel <Remodel 🚋> <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7647>06:02
mupPR snapd#7657 closed: snap: fix default-provider in seed validation <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7657>06:16
zygahey06:20
zygamborzecki: the secret for green spread PRs is to open them at 1AM06:21
zygaspread has logic for that06:21
zygareally ;)06:21
zygajuts look at 1AM and you will see it06:21
mborzeckizyga: i.e. switch timezones then06:21
mborzeckizyga: and hey :)06:21
* zyga had a horrible horrible night06:21
zygaJanek decided to sleep in our bed and we couldn't convince him otherwise06:22
zygaand that's just not enough space06:22
zygamy whole body hurts06:22
zygamvo: I found a curious bug06:22
zygaI discussed it with jamie a little06:22
zygabut I couldn't convince him at the time06:22
zygamvo: I opened a thread about it to discuss it https://forum.snapcraft.io/t/classic-confinement-breaks-high-dpi-support/1386806:23
zygamvo: I think the bug is more serious for non-x-wayland sockets because it is likely that classic software is broken and unable to open files it expects06:23
jameshIdeally we wouldn't change XDG_RUNTIME_DIR for confined snaps either...06:24
zygaby applying the attached fix all my snap apps are non-blurry now06:24
zygajamesh: I tend to agree, after all we have apparmor for confinement06:24
zygajamesh: I think jamie wanted to do this (set it to a custom value) to support things like instances06:25
zygabut I don't know if that is sensible, how are apps expected to talk to each other?06:25
zygahey jamesh :-)06:25
jameshzyga: by that I mean mounting something over /run/user/$UID for strict confinement, with bind mounts of wayland, dbus, pulse sockets06:25
zygajamesh: mhm06:26
zygajamesh: for strict apps that's sensible IMO06:26
zygajamesh: especially that we can "recover" those socktes via hostfs06:26
mvozyga: is this 7659 ?06:26
zygajamesh: so even regular mount backend code would make it work06:26
zygamvo: yes06:26
mvozyga: aha, I see06:27
zygaactually using wayland, snaps on the desktop06:27
zygabrave new world06:27
zygaI need to figure out the horrid suspend issue where x input breaks06:27
zyga(but works in console)06:27
mvozyga: this looks fine to me but I don't know enough06:27
zygaI only can reboot to recover from that06:27
zyganeed to learn how xinput works06:27
mvozyga: do you think could have a look at 7640? this would allow us to ship this in the initramfs06:28
zygayes, I think you pinged me about that yesterday06:28
zygalet me do so now06:28
zygamvo: question about the path, we have /run/snapd/* already, could that path be /run/snapd/recovery or something like that?06:29
mvozyga: sure, I can change that06:29
mvozyga: this is in initramfs06:29
mvozyga: so we don't have /run/snapd yet but its fine to add it06:30
zygayeah, just for consistency,06:30
zygait's not a blocker in any way06:30
zygamvo: total lack of docstrings makes this harder to review06:30
zygamvo: I'm not following core20 work as closely06:31
zygamvo: what kind of filesystems are going to be mounted?06:31
mvozyga: vfat and ext4 currently06:31
zygamvo: by deployFilesystemContent06:31
zygaok06:31
mborzeckizyga: i'm looking for the place where we creat that per snap XDG_RUNTIME_DIR but i don't see it06:38
mborzeckizyga: there's some dead code in s-c06:38
mborzeckizyga: and nothing in snap run?06:38
zygamborzecki: snap run06:39
zygamborzecki: cmd/snap/cmd_run.go:49506:39
mborzeckizyga: btw. wonder if that also breaks the thing with xdg-open & classic snaps that cause new browser window to pop up06:39
zygamborzecki: probably06:39
zygayeah, it's just silly we set it06:39
zygafor classic I mean06:39
mupPR snapd#7660 opened: snap-recovery: rename to "snap-bootstrap" <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7660>06:39
mborzeckizyga: hmm that code in snap run doesn't create /run/user/<uid>/snap.*06:43
jameshthere's code in snapcraft-desktop-helpers that creates it, with a comment referencing https://bugs.launchpad.net/snappy/+bug/165634006:45
mupBug #1656340: XDG_RUNTIME_DIR is not created on app startup <bionic> <cosmic> <eco-team> <xenial> <Snappy:Triaged by zyga> <snapd (Ubuntu):Confirmed> <https://launchpad.net/bugs/1656340>06:45
zygamvo: https://github.com/snapcore/snapd/pull/7640#pullrequestreview-30700767906:49
mupPR #7640: snap-recovery: deploy gadget content when creating partitions <Created by mvo5> <https://github.com/snapcore/snapd/pull/7640>06:49
mvozyga: thank you! about the ShiftStructureTo - thats something for mborzecki06:50
mvozyga: I will follow the second suggestion in a followup, I need to make it transactional and then its good for now06:51
mupPR snapd#7640 closed: snap-recovery: deploy gadget content when creating partitions <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7640>06:51
zyga+106:52
mvothanks zyga !06:56
zygamvo: is https://github.com/snapcore/snapd/pulls/zyga something for 2.42.1 or 2.43?06:59
zygaI'll get breakfast and then work on follow ups07:02
=== pstolowski|afk is now known as pstolowski
pstolowskimorning07:05
zygahey palasso07:08
zygapawel :)07:08
palassohello zyga07:08
mupPR snapd#7630 closed: overlord/devicestate: check snap handler for gadget remodel compatibility <Remodel 🚋> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7630>07:11
mvohey pstolowski ! good morning07:17
mborzeckizyga: reading the comments from ian about docker snap, wtf is docker doing there?07:19
mborzeckibtw. wonder if we could snap podman and use the same docker interface07:20
zygai have ideas07:23
zygabreakfasting07:23
mborzeckizyga: #7632 is green07:28
mupPR #7632: interfaces: de-duplicate emitted update-ns profiles <Bug> <Created by zyga> <https://github.com/snapcore/snapd/pull/7632>07:28
zygayep07:29
zygare07:36
zygaokay07:36
zygamborzecki: we had an issue with docker in the past07:36
zygamborzecki: it was trying to use /sys from ... /var/lib/snapd/hostfs/sys07:37
zygamborzecki: I think it's a bit "too" smart07:37
zygamborzecki: I'll start by hiding the cgroup from snap view07:37
zygamborzecki: let's see if that is enough to fix it07:37
mborzeckizyga: so smart that the smartness overflowed07:37
mborzeckizyga: won't the cgroup always appear via /proc/$$/cgroup?07:38
zygamborzecki: nuclear gandhi07:39
zygamborzecki: yes but if it doesn't show up mounted maybe it will ignore it07:40
zygamborzecki: we need to check docker logic07:40
zygaafk again07:40
mborzeckizyga: iirc when i looked at libcontainer it tried to parse /proc/self/cgroup and /proc/mountinfo07:41
zygamborzecki: we can remove it from mountinfo07:43
zygaif it parses cgroup then we're SOL07:43
zygabut then again, it's silly to do :P07:43
mupPR snapd#7661 opened: packaging: tweak handling of usr.lib.snapd.snap-confine <Created by mvo5> <https://github.com/snapcore/snapd/pull/7661>08:03
zygamvo: thanks08:06
zygamvo: is it working?08:06
zygais that tested?08:06
zygawe did that before and it was a hit-and-miss08:06
zygamborzecki: I wrote a small patch - checking if that makes docker work08:06
mvozyga: we test it in one spread test but I could not reproduce08:06
zygaaha08:07
zygawell08:07
zygaI wrote that patch for moving that out of /etc08:07
zygabut it's not finished (tests need adjusting)08:07
zygaI think it's fine08:07
* mvo nods08:07
zygamborzecki: it works08:12
zygamborzecki, ijohnson: this fixes docker08:13
zygahttps://github.com/snapcore/snapd/pull/7547/commits/1c60b84df1faf7ca2e80bf30d50850e6322da87b08:13
mupPR #7547: many: use a dedicated named cgroup hierarchy for tracking <⛔ Blocked> <Created by zyga> <https://github.com/snapcore/snapd/pull/7547>08:13
zygaso yay08:13
pedronismvo: should #7642 be merged?  #7626 and #7645 need a master merge I think08:44
mupPR #7642: travis.yml: add 'sh -x' to run-checks <Created by mvo5> <https://github.com/snapcore/snapd/pull/7642>08:44
mupPR #7626: managers: add remodel undo test for new required snaps case <Created by mvo5> <https://github.com/snapcore/snapd/pull/7626>08:44
mupPR #7645: client: add xerrors and wrap errors coming from "client" <Created by mvo5> <https://github.com/snapcore/snapd/pull/7645>08:44
pedronismborzecki: hi, maybe you can do a 2nd review of 762608:45
mupPR snapd#7632 closed: interfaces: de-duplicate emitted update-ns profiles <Bug> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/7632>08:48
pedronislots of repeated red :/09:22
* dot-tobias says hi09:29
zygapstolowski: https://github.com/snapcore/snapd/pull/766209:31
zygahey dot-tobias09:31
mupPR snapd#7662 opened: interfaces: simplify AddUpdateNS and emit <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/7662>09:31
mupPR #7662: interfaces: simplify AddUpdateNS and emit <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/7662>09:31
pstolowskizyga: ty09:33
pedronismvo: did you see my ping here about some of your PRs ?09:37
mvopedronis: re 7642> if we don't see the mysterious error 6 anymore I think we can omit it09:41
mvopedronis: I suspect something with travis was wrong that triggered these errors09:41
pedronisok09:44
mvopedronis: feel free to close, we can always reopen if these mysterious errors  come ack09:45
mupPR snapd#7663 opened: tests: add 14.04 canonical-livepatch test <Created by mvo5> <https://github.com/snapcore/snapd/pull/7663>09:51
mupPR snapd#7664 opened: cmd/cmdutil: version helper <Prebaking> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7664>09:52
mupPR snapd#7642 closed: travis.yml: add 'sh -x' to run-checks <Created by mvo5> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/7642>10:03
mupPR snapd#7665 opened:  devicestate: add support for gadget->gadget remodel  <Remodel 🚋> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7665>10:16
mborzeckimvo: pedronis: ^^ decided against splitting it further for now10:17
mborzeckistill 12 files changed, 820 insertions(+), 97 deletions(-) :/ meh10:18
ijohnsonzyga nice10:22
zygahey :)10:22
zygayeah10:22
ijohnsonMuch better than trying to patch docker :-)10:25
pedronismborzecki: sounds ok size wise, I imagine a bunch is the managers_test.go test ?10:33
mborzeckipedronis: yup, managers_test, devicestate tests, spread test ;)10:33
pedronisyea, sounds alright :)10:34
mvothanks mborzecki10:37
mborzeckimvo: got some reviews for zyga and then i'll look into the copr repo with xerrrors to unblock you10:37
mvomborzecki: \o/10:38
* Chipaca gets confuzzled and takes a break10:50
* pstolowski lunch10:51
Wimpressmvo Chipaca We (popey and I) have things to discuss with you for todays catch up :-)11:25
mupPR snapd#7648 closed: interfaces/policy: expand cstrs/cstrs1 to altConstraints/constraints <Simple 😃> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7648>11:27
zygaWimpress: sounds interesting11:28
mupPR snapd#7655 closed: interfaces: allow introspecting network-manager on core <Bug> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/7655>11:29
mvoWimpress: I can make it if we keep it short11:30
* zyga afk12:10
=== ricab is now known as ricab|lunch
mupPR snapcraft#2707 closed: manifest: sort package and snap lists for consistency <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2707>12:35
mupPR snapcraft#2755 closed: extensions: kde-neon: add icon and sound themes <Created by galgalesh> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2755>12:35
ijohnsonmorning12:50
pedronispstolowski: I did a pass on #765812:50
mupPR #7658: cmd/snap-start-preseed: add snap-start-preseed executable (1/N) <Prebaking> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7658>12:50
pstolowskipedronis: yes just saw it, thank you12:59
ijohnsonpstolowski: thanks for the review, will address your suggestions today12:59
mvomborzecki: about this copr repo - do I need to add something to the xerrors pr under review?13:08
mborzeckimvo: i'll open a PR adding that to the early setup we do on fedora13:09
mvomborzecki: \o/13:09
mupPR snapd#7662 closed: interfaces: simplify AddUpdateNS and emit <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/7662>13:10
zygajdstrand: can you re-review https://github.com/snapcore/snapd/pull/754713:18
mupPR #7547: many: use a dedicated named cgroup hierarchy for tracking <Created by zyga> <https://github.com/snapcore/snapd/pull/7547>13:18
zygajdstrand: mainly to check the comments about v1 vs v2 mode13:18
zygahuh13:23
zygahttps://www.irccloud.com/pastebin/eonsNHRx/13:23
zygamborzecki: did you see this when changing delta code in spread recently?13:23
zyganot sure why this didn't flag before13:23
zygaperhaps just 19.10 -> amazon is the problem13:24
mborzeckizyga: nope? this an opensuse host?13:24
zygano, ubuntu 19.1013:24
mborzeckiah 19.10 hm, maybe some defaults changed?13:24
zygamhm13:24
sergiusenshow good is "snap set core experimental.refresh-app-awareness=true"?13:28
zygasergiusens: it's partially useful, it's not handling classic confinement which will be fixed shortly (code ready, waiting on reviews)13:31
zygasergiusens: there's no UX but that's under development13:31
mupPR snapd#7666 opened: tests: don't depend on GNU time <Created by zyga> <https://github.com/snapcore/snapd/pull/7666>13:31
zygasergiusens: there will be some in the next two weeks13:31
sergiusenszyga: I just want to prevent updates for when running confined snaps like the browser, which updates frequently and too many weird things start to happen :-)13:32
* sergiusens sets the flag13:32
zygasergiusens: yes, please set it13:32
zygasergiusens: and report all weird stuff13:32
zygasergiusens: I'm working on making it much better13:32
zygamborzecki: small python helper if you want to look13:33
zygahttps://github.com/snapcore/snapd/pull/7666/files13:33
mupPR #7666: tests: don't depend on GNU time <Created by zyga> <https://github.com/snapcore/snapd/pull/7666>13:33
ijohnsonpedronis: I will look at your seedwriter PR now13:37
zygaInstalling updates....13:38
=== ricab|lunch is now known as ricab
pedronisijohnson: thx13:43
mupPR snapd#7667 opened: spread: enable bboozzoo/snapd-devel-deps COPR repo for getting golang-x-xerrors <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7667>13:46
mborzeckimvo: ^^13:46
ijohnsonpedronis: reviewed one question about the tests13:57
* ijohnson goes to get coffee before meeting13:57
jdstrandzyga: it is on the list, yes. pedronis, I commented (only) on pr 7659. I am not explicitly blocking the PR, but wanted to make sure my concerns were considered14:02
mupPR #7659: snap/snapenv: preserve XDG_RUNTIME_DIR for classic confinement <Created by zyga> <https://github.com/snapcore/snapd/pull/7659>14:02
zygajdstrand: ack14:02
jdstrandwell, I did vote -1 for 2.42.x14:03
jdstrandbut anyhoo, it is all in the PR14:03
jdstrandzyga: also, I don't think I'm needed for 7666, but I glanced at it and the approach looks good. thanks for the followup :)14:04
zygajdstrand: thanks! :)14:05
abeatojdstrand, hey, is https://github.com/snapcore/snapd/pull/7603 on your radar? the MP contains a couple of new interfaces for dpdk14:20
mupPR #7603: interfaces: add dpdk and hugepages interfaces <Created by alfonsosanchezbeato> <https://github.com/snapcore/snapd/pull/7603>14:20
jdstrandabeato: it is. I need to dig in a bit on that one14:22
abeatogreat14:22
hellsworthi see two content snaps listed when i do snap connections on my snap so can someone point me to where connections grabs its info from?14:27
hellsworthi'm building my snap in a normal way, only pointing to one extension. that extension only points to one platform snap.14:27
hellsworthdoes the core18 base know about extensions?14:33
ijohnsonhellsworth: look at `sudo cat /var/lib/snapd/state.json | jq .data.conns`.14:36
ijohnsonhellsworth: probs you are suffering from the bug where your snap you are working on was previously installed and had that connection to the other content interface and so now `snap connections` shows that14:37
hellsworthhow can i clear that history though14:37
hellsworthyes i did previously have this snap installed with the old extension14:37
hellsworthwhich connected to the old content snap14:38
ograOct 25 16:19:28 ... systemd[1]: [/etc/systemd/system/snap-pi2\x2dkernel-102.mount:10] Unknown lvalue 'LazyUnmount' in section 'Mount'14:38
ograOct 25 16:19:28 ... systemd[1]: [/etc/systemd/system/snap-core18-1226.mount:10] Unknown lvalue 'LazyUnmount' in section 'Mount'14:38
ograhmm, thats a new one14:38
ogra(my log is full of these)14:38
* zyga goes for lunch14:44
=== JanC_ is now known as JanC
oSoMoNjdstrand, have you seen my last message on https://forum.snapcraft.io/t/auto-connecting-the-personal-files-interface-for-the-chromium-snap-part-ii/13705/8 ?14:49
oSoMoNauto-connection of personal-files for chromium is refused, I thought you had allowed it?14:49
jdstrandoSoMoN: I hadn't, but I'll look at it14:50
jdstrandzyga: ok, reviewed pr 7547, please see both the review comments and the additional comment I added after14:50
mupPR #7547: many: use a dedicated named cgroup hierarchy for tracking <Created by zyga> <https://github.com/snapcore/snapd/pull/7547>14:51
oSoMoNjdstrand, you wrote "Granting use of the updated path. This is now live.", I assumed this was good to go14:51
jdstrandoSoMoN: it should be. that is why I need to look at it :)14:52
oSoMoNok, thanks :)14:52
* cachio lunch14:53
zygajdstrand: ack14:55
jdstrandoSoMoN: btw, the memory usage PR as merged. it should be in edge whenever the core snap is rebuilt14:56
jdstrandzyga: it probably wouldn't be terrible to get that ^ in 2.4214:56
jdstrandI'll let others decide14:56
zygajdstrand: the apparmor memory fix? I think that's up to mvo to cherry pick but I agree it's not likely to be hard to do or breaking stuff14:57
zygaso yeah =114:57
zyga+114:57
jdstrandzyga: yeah14:57
mvozyga: if you and jdstrand are good I can cherry pick the appamror de-dup fix to 2.42.1  (unless pedronis  objects of course)14:59
oSoMoNjdstrand, I subscribed to the PR so I got a notification when it was merged, that's very nice, thanks!14:59
oSoMoNmy opinion doesn't count, but I'm totally +1 for cherry-picking :)15:00
jdstrandmvo: +115:00
zygamvo: yes, just remember it was split into three PRs15:00
zygamvo: 1) OrderedSet, 2) emit 3) the actual fix with spread test15:00
oSoMoNbtw, thanks zyga for working on this15:01
jdstrandmvo: regression potential should be shallow. if snap-update-ns isn't working right, it should be broken in many places. asking QA for explicitly testing the preinstalled gnome snaps is likely not a bad idea to build confidence15:02
mvozyga, jdstrand lets quickly catchup with pedronis  (he is in a meeting right now) and if he is +1 I will cherry-pick15:02
jdstrandoSoMoN: typo in snap decl. I thought it was chromium-browser-init. it is chromium-browser.init15:05
jdstrandoSoMoN: shall I fix the snap decl or you fix the snap?15:05
jdstrandoSoMoN: it is easy to fix the snap decl15:05
jdstrandoSoMoN: I see the original request was for chromium-browser.init, so I'll fix the snap decl15:07
jdstrandoSoMoN: fixed: https://forum.snapcraft.io/t/auto-connecting-the-personal-files-interface-for-the-chromium-snap-part-ii/13705/815:10
jdstrandoSoMoN: be sure to 'snap refresh' before install if using the same system15:11
* zyga made coffee15:13
ograzyga, did you see my error above ? seems all my machines are spamming their logs with that15:13
zygaogra: no, I did not15:13
ograOkt 24 20:52:26 anubis systemd[1]: [/etc/systemd/system/snap-core-7917.mount:10] Unknown lvalue 'LazyUnmount' in section 'Mount'15:13
zygaI see that now15:13
ograogra@anubis:~$ journalctl |grep -c lvalue15:13
ogra148315:13
jdstrandogra: oh, I've seen that. I kept meaning to mention it somewhere15:13
zygaolder versions of systemd don't understand lazy unmount15:13
zygaogra: which version is that?15:13
zygaof the distribution15:14
jdstrandzyga: for me it was uc1615:14
ograi just noticed it on a pi here ... and checked the other machines ... seems all of them show it15:14
zygaI see15:14
ograthats 16.04 and a Core 16 Pi15:14
zygait's a part of a fix15:14
zygathat doesn't work on old systems15:14
ograso both, core and desktop15:14
pedronismvo: what's the question?15:14
ograyeah, i was guessing you use some new systemd feature there :)15:14
jdstrandzyga: perhaps use of the conditional needs to be conditional on new enough systemd? not sure if that is system-key material...15:15
jdstrandmeh15:15
zygait is https://github.com/snapcore/snapd/commit/5f16e6b0738410d07995b85bc516d9bbb50263c815:15
jdstranduse of the feature*15:15
mvopedronis: if we can backport the de-duplication for apparmor lines into 2.42.115:16
zygajdstrand: we don't re-generate mount units today, system key won't help15:16
zygajdstrand: I agree it would be nicer to know if it is supported15:16
zygaI need a moment to finish the coffee15:17
zygathen jumping into a customer bug15:17
jdstrandogra: I'm thinking there should be a bug. I suspect we'll get customers wondering about it15:17
jdstrandogra: and that will help the snapd team to prioritize15:18
pedronismvo: it seems ok, if jdstrand is also +115:18
mvozyga: has the 3rd part landed already?15:20
zygamvo: yes, all parts are inn15:20
zyga*in15:20
jdstrandpedronis (cc mvo): I am, though asking for some light additional QA for say running gnome snaps and verifying no new denials would not be unreasonable. I know there are nice tests for that in the PRs, so, all of your call15:20
ograzyga, jdstrand https://bugs.launchpad.net/snapd/+bug/184986115:21
mupBug #1849861: Unknown lvalue 'LazyUnmount' message in logs with latest snapd on 16.04 based systems <snapd:New> <https://launchpad.net/bugs/1849861>15:22
jdstrandogra: thanks!15:22
pedronismvo: as jdstrand says a bit of extra QA would be good? what was your plan, do 2.42.1 straight? do a pre?15:25
mvopedronis: probably no pre, should be very controlled but we can keep it longer in candidate for example15:30
pedronisok15:30
mvozyga: what was the pr number for the 3rd part15:30
zygamvo: just install a snap before and after and check sha1 of the kernel image of the profile15:30
zygamvo: one moment15:30
zygamvo: 7632 7633 763915:31
zygamvo: please make sure to cherry pick the test as well, so that you can see it pass15:32
mvozyga: thank you! 763215:34
mvozyga: 7632 tripped me over15:35
zygamvo: do you need help?15:35
mvozyga: in a meeting right now, I will probably do this on monday, but you are welcome to prepare a PR against 2.42 - but no worries, I will get to it eventually15:42
zygamvo: not today, I'll just inspect that bug and EOD15:43
zygamvo: I -1 one of your branches, sorry, just a  question but need to make sure it does not land unanswered15:43
mvozyga: thats fine, its a valid question, we do it in other tests too but I will have a look later15:51
* zyga is under invasion from a particularly lovely toddler :D15:52
mvozyga: eod> totally fine!15:55
=== pstolowski is now known as pstolowskiafk
* zyga breaks to run an errand16:01
ijohnsonhellsworth: sorry I missed your response, probably what you can do is first remove the snap, then make a backup of your state.json, stop snapd with `systemctl stop snapd.socket && systemctl stop snapd` and then delete the conns for the snap in question in state.json and then re-enable snapd with `systemctl start snapd.socket && systemctl start snapd`and finally try installing your snap again16:36
ijohnsonnote that afaik the bug is just about the connections showing up in `snap connections`, I don't think the security policy still actually has the stuff that those connections implies16:36
hellsworthok thanks. should snap-discard-ns also cleare this cached history too?16:39
ijohnsonhellsworth: snap-discard-ns will clean up the security policy stuff (well specifically the mount ns), but it won't clean up the connections themselves17:10
hellsworthok thanks for clearing that up ijohnson17:11
ijohnsonnp, sorry you are encountering that bug17:11
hellsworthbugs happen :)17:11
ijohnsonhellsworth: that bug fwiw is https://bugs.launchpad.net/snapd/+bug/184851617:12
mupBug #1848516: snap connections/interfaces shows dropped interfaces as connected after refresh <snapd:Confirmed for stolowski> <https://launchpad.net/bugs/1848516>17:12
hellsworthah thanks17:12
jdstrandzyga: oh, I meant to ask, what was up with all the .keep files in PR 7632?18:11
mupPR #7632: interfaces: de-duplicate emitted update-ns profiles <Bug> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/7632>18:11
zygajdstrand: those are so that we have a18:23
zygaDirectory structure18:23
zygaIt is like gnome system monitor snap18:23
zygaJust frozen in time18:23
zygaAnd empty18:23
jdstrandzyga: oh right, git18:24
zygaYeah :-)18:24
jdstrandzyga: it was weird cause github said 'No changes' in the PR18:24
jdstrandso it threw me18:24
zygaYeah, all empty18:24
jdstrandok, thanks :)18:24
zyga:-)18:24
mupPR snapcraft#2768 opened: remote-build: autorecovery and improved resiliency <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2768>19:00
* cachio afk19:08
kenvandinesergiusens: https://paste.ubuntu.com/p/HTgWjC9tJV/19:29
kenvandinethat is me building with the WIP gnome-3-34 extension which uses gnome-3-34-1804-sdk build snap19:29
kenvandinelibffi.so.7 is in the same directory as libgobject-2.0.so19:30
mupPR snapcraft#2761 closed: Core20 base <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2761>19:30
kenvandinesergiusens: any suggestions appreciated19:31
mupPR snapd#7667 closed: spread: enable bboozzoo/snapd-devel-deps COPR repo for getting golang-x-xerrors <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7667>19:31
mupPR snapd#7664 closed: cmd/cmdutil: version helper <Prebaking> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7664>19:35
mupPR snapd#7654 closed: cmd/snap, store: include snapcraft.io page URL in snap info output <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7654>19:37
mupPR snapd#7605 closed: tests: configure the journald service for core systems <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7605>19:38
mupPR snapd#7656 closed: tests: verify host is not affected by mount-ns tests <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/7656>19:53

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