mupBug #1758465 opened: snapd doesn't upgrade on bionic when a local installed snap mount is failing <Snappy:New> <https://launchpad.net/bugs/1758465>00:09
mupPR snapd#4858 closed: strutil, cmd/snap: drop strutil.WordWrap, first pass at replacement <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/4858>00:59
mupBug #1741486 changed: failed snap try leaves snap symlink around <Snappy:Expired> <https://launchpad.net/bugs/1741486>04:20
mupPR snapd#4882 closed: snap: make `snap run` look at the system-key for security profiles <Critical> <Squash-merge> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4882>07:04
mupPR snapd#4924 opened: snap: make `snap run` look at the system-key for security profiles (#… <Created by mvo5> <https://github.com/snapcore/snapd/pull/4924>07:12
mupPR core#86 opened: hooks: create compat copy of /lib/udev/snappy-app-dev <Created by mvo5> <https://github.com/snapcore/core/pull/86>07:44
mupPR core#86 closed: hooks: create compat copy of /lib/udev/snappy-app-dev <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core/pull/86>07:45
pedronismvo: zyga: I'm a bit confused, it seems  account-control expects files to be already present in /var/lib/extrausers  but in our tests they are just because of a hack08:28
mupPR snapd#4924 closed: snap: make `snap run` look at the system-key for security profiles (2.32) <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4924>08:32
pedronisif I tweak the hack I get:    useradd: cannot open /var/lib/extrausers/passwd08:33
pedronisit's not a confinement issue btw,     useradd --extrausers alice  as root also gives the same08:37
mupPR snapd#4925 opened: tests: fix snap-run tests when snapd is not running <Created by mvo5> <https://github.com/snapcore/snapd/pull/4925>08:59
mupPR snapd#4926 opened: tests: fix snap-run tests when snapd is not running <Created by mvo5> <https://github.com/snapcore/snapd/pull/4926>09:00
threshhello.  how do I debug why stage-packages: pulls in packages that I don't see installed when I do apt-get install for that list?10:10
threshI see some packages that should not be there10:11
threshfound --debug, let me see if that helps...10:12
mborzeckizyga: hi, you around?10:16
mborzeckizyga: there's a hackathon at GOG office in May, want to show them how make snaps? https://www.gog.com/hackathon10:17
mupPR snapd#4925 closed: tests: fix snap-run tests when snapd is not running (2.32) <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4925>10:19
zygaHi mborzecki10:28
zygaWanna come?10:29
mborzeckizyga: i'm interested for sure, don't know yet if i will be able to go, probably will know more mid-april, registration closes on may 7, so we still have some time10:36
zygaWe could snap gog games10:37
bulldam building my app with qt5.7.1 and later am going to snap pack it, how can i use my local qt5.7.1 toolkit to build in snapcraft file10:37
zygaI’ll register us back home10:37
zygabulld: hey, I’m not sure but it’s possible for sure.10:38
zygaDid you check on the forum? Maybe someone did this already10:38
bulldi build qt5.7.1's webengine myself with codecs support so i need it to build/ship the binary am using in snap package10:39
threshso I've ran two snapcraft --debug prime runs, and it seems like the behaviour is inconsistent: https://thre.sh/stuff/snapcraft-different-packages-staged.diff.txt10:39
bulldokay @zyga i will check10:39
threshany idea how to make sure snapcraft works the same between different runs?10:39
* zyga is a snapd hacker and builds snaps less frequently than others10:40
threshe.g. you can see it fetches libvlc5 in one case and not in the other10:40
zygaAnd when I do I build weird snaps that use new features10:40
bulld@zyga :)10:40
zygaSpeaking of which, I have some cool snap demo to publish10:41
zygaBut first family time with my daughter :-)10:41
bulldhmm :)10:42
bulldmy snap installation is broken somehow cannot install any snap package local or from store. getting some apparmor errors10:55
bulldam using ubuntu 16.0410:55
mupPR snapcraft#2024 closed: errors: improve the UX for sending error data <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2024>10:57
bulldi already posted details of errors am getting in snapcraft forums https://forum.snapcraft.io/t/unable-to-install-snap-from-store-or-loallay/462911:02
zygabulld: can you give me more details11:08
zygaSnap version11:08
zygaAnd the errors you are seeing11:08
bulldi provided everything in the forum link in my previous msg11:09
zygaI’ll check that later today11:22
bulldokay ty11:27
mupPR snapd#4927 opened: release 2.32 <Created by mvo5> <https://github.com/snapcore/snapd/pull/4927>11:55
pedroniszyga: I think  I know how to disentangle the etc mess,  was confused for a bit by writable-paths (surprise)12:02
zygaI sent a PR that looks for the damage12:02
zygaThere are some tests that need fixes12:02
pedroniszyga: well, it's no the tests12:03
pedronisis the prepare stuff that does very strange stuff12:04
pedronis(in hindsight)12:04
pedroniszyga: we basically run the tests with /etc/passwd  being a bind mount of /var/lib/extrausers/passwd12:04
pedronisthat makes little sense conceptually12:04
zygaWhy do we do that’s12:05
zygaNote that would not break the mimic code12:05
pedroniszyga: well we do need to bind mount something on /etc/passwd12:05
zygaIt is this with combination of some unfortunate rm -rf12:05
pedronisbut making it /var/lib/extrausers/passwd12:06
pedronisadds to the confusion12:06
pedronisbecause then some other code will modify it12:06
pedroniszyga: it doesn't seem to be a rm -rf , more some cases of sed -i12:06
pedronisbut I bit unclear exactly how (I don't get that effect playing locally with sed -i)12:06
zygaI think sed would not cause that12:07
zygaKernel code indicates that dentry must be removed12:07
zygaSo more like the parent directory12:07
pedroniszyga: anyway  having etc/passwd === extrausers/passwd doesn't make sense12:07
zygaI agree12:07
pedroniszyga: I think we need to bind mount /etc/passwd   because we need to put there the spread root user12:08
zygaCannot we use extra users for that?12:08
pedroniszyga: no, that's the problem12:08
pedronisroot cannot be in extrausers12:08
pedronisbecause the one in /etc/passwed wins12:08
zygaAh, I see12:09
pedronisso we need a bind mount12:09
pedronisbut no reason to do this bind mount12:09
pedronis(is super confusing in various ways)12:09
pedronisalso it means that when we test extrausers stuff12:09
pedronis(which is important)12:09
pedroniswe really test nonsense12:09
zygaAt the very least I know how to improve mimic code12:09
zygaBut our test environment is in a very broken state12:10
pedroniszyga: anywy I can propose a change that doesn't have any /etc  //deleted mounts at least12:10
pedronisand also untangle a bit that confusion12:10
pedronisI don't know if it's enough to not confused layout12:10
pedronisbut is a step12:10
zygaBecause all those //deleted mount points are broken12:10
zygaI will check kernel code12:10
pedronisno more of those12:11
pedroniswith my change12:11
pedronisI mean for etc12:11
zygaBut I’m worried that such deleted mount points cannot be the target of a mount call12:11
pedroniswe have some for try snaps afaict12:11
zygaBecause they no longer exist conceptually12:11
pedroniszyga: ?12:11
pedroniszyga: so?12:11
zygaIt’s a detached inode12:11
pedroniszyga: as I said I fixed the //deleted bit12:11
zygaSo mount with that as a target will always fail12:11
zygaSo what can we12:11
pedronisis that a problem?12:12
zygaThe mimic is broker12:12
pedronisI mean do you think it will happen in practice?12:12
zygaBut we must handle that in tests12:12
zygaIn reality it will not happen12:12
pedronisif there are no  //deleted12:12
pedronisthere's nothing to handle12:12
pedronisI made changes such that there are no more //deleted12:13
pedronisI think I said that a couple fo times12:13
zygaBut I don’t want to fail tests 30% of the time because of that12:13
zygaCan you merge my patch from a later pr that detects dangling //deleted please?12:14
pedroniszyga: I used your patch12:14
pedronisas a baseline12:14
pedronisotherwise how would I know12:14
pedronisthat there no //deleted12:14
zygaAh perfect12:15
pedronisto be clear I'm talking about /etc here12:15
pedronisthere are some tests using try12:15
pedronisthat live //deleted around12:15
pedronisnot sure it's an issue or not12:15
zygaI see12:15
zygaI can review that next week and try to eliminate them12:15
* zyga is very much afk12:17
mborzeckiupdated snapd to 2.32 in AUR13:12
mupPR snapd#4922 closed: overlord/configstate: change how ssh is stopped/started (#4912) <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4922>15:07
pedronisok, my change seems to do what I wanted (there was a further subtle detail, but I think I have a fix)15:27
mupPR snapd#4928 opened: tests: disentangle etc vs extrausers <Created by pedronis> <https://github.com/snapcore/snapd/pull/4928>15:54
pedroniszyga: ^15:56
zygaThank you. I will return home soon and review it15:56
zygaDid you find this by reading the strace from last night?15:57
pedroniszyga: no, I just assumed  that /etc/group  being //deleted would be the source of the problem?15:59
pedroniszyga: anyway I tried to run those account tets and layout together and layout passes15:59
pedroniswith this change15:59
zygaThank you so much!16:03
zygaMy question about the strace was motivated by wanting to see if there was anything else fishy there16:04
zygaI will review it in the evening16:04
pedroniszyga: anyway as I explain in the PR, this is a saner change because what we did would make our /var/lib//extrausers not very realistic16:05
pedronisbecause /etc/passwd would also contain the stuff16:06
pedroniszyga: rechecked, on master if one runs ubuntu-core-create-user just before layout  the latter fails with the /etc issue16:16
zygaGreat find! But I also managed to fail it by running just layout with -repeat 216:17
pedronisthat is stranger16:18
zygaBut that was against core 1616:19
zygaSo I thought the suite restore is buggy16:19
pedronisthis is a core only problem btw16:19
pedroniswe don't do this strange things on classic16:19
zygaIt would fail on 2nd try16:19
zygaI hope so. I think we saw layout failures in classic too16:20
zygaBut perhaps older codebase16:20
pedroniswell it might be that layout breaks layout16:20
pedronisin some cases16:20
pedronisare the  threspassing fixes merged?16:20
zygaNo but they cannot affect this16:21
pedronisalso linode and google are different because reuse vs not16:22
zygaTrespassing bug means we leave empty regular file, directory or symlink16:22
pedronisthat also changed16:22
zygaAh, good point16:22

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