/srv/irclogs.ubuntu.com/2020/09/18/#snappy.txt

ijohnsoncachio: around?01:01
cachioyes01:01
ijohnsonhey, so I figured out why the refresh fundamentals test has been broken all day01:02
ijohnsonit got broken by the 2.47 snapd in beta channel today01:02
ijohnsonand to fix it we need a new kernel snap01:02
ijohnsonso until that lands, I would like to disable the test01:02
ijohnsoncachio: so could you quick approve https://github.com/snapcore/snapd/pull/9368?01:02
mupPR #9368: tests/nested/manual/refresh-revert-fundamentals: disable temporarily <Skip spread> <UC20> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9368>01:02
cachioijohnson, sure01:02
ijohnsonthanks01:02
cachio+101:03
cachiothanks for that01:03
ijohnsonthanks, I'll merge it in a few minutes and open a PR undoing the change we can use to monitor until the test is passing again with a new kernel01:04
ijohnsonhave a good night!01:04
cachioijohnson, nice, thanks01:04
=== benfrancis4 is now known as benfrancis
mupPR snapd#9368 opened: tests/nested/manual/refresh-revert-fundamentals: disable temporarily <Skip spread> <UC20> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9368>01:04
mupPR snapd#9369 opened: tests/nested/manual/refresh-revert-fundamentals: re-enable test <Run nested> <UC20> <⛔ Blocked> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9369>01:09
mupPR snapd#9368 closed: tests/nested/manual/refresh-revert-fundamentals: disable temporarily <Skip spread> <UC20> <⚠ Critical> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9368>01:19
=== chesty_ is now known as chesty
* cachio EOD01:21
=== benfrancis6 is now known as benfrancis
mupPR snapd#9370 opened: usersession/userd: separate bus name ownership from defining interfaces <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/9370>05:20
zyga-x240good morning05:49
zyga-x240hop hop...05:58
mupBug #1896182 opened: Snapd is waking up my screen from sleep regularly <Snappy:New> <https://launchpad.net/bugs/1896182>06:14
mborzeckimorning06:17
zyga-x240mborzecki: hi06:18
zyga-x240mborzecki: curious bug06:18
mborzeckizyga-x240: hey06:18
mborzeckihm, what bug?06:18
zyga-x240https://bugs.launchpad.net/snappy/+bug/189618206:18
mupBug #1896182: Snapd is waking up my screen from sleep regularly <Snappy:New> <https://launchpad.net/bugs/1896182>06:18
mborzeckizyga-x240: heh, i was reading https://bugzilla.redhat.com/show_bug.cgi?id=188026106:18
mborzeckisame thing?06:18
zyga-x240haha06:19
zyga-x240ye06:19
zyga-x240any ideas?06:19
zyga-x240could that be snapd hotplug logic?06:19
mborzeckizyga-x240: https://bugzilla.redhat.com/show_bug.cgi?id=1880261#c306:33
mborzeckihm long shot but maybe there's something else at play here06:33
zyga-x240I also replied to the launchpad bug06:33
mborzeckihahah06:34
mborzeckizyga-x240: noone can say we're not responding to bug reports now06:34
zyga-x240heh :)06:34
zyga-x240I pushed https://github.com/snapcore/snapd/pull/9345#pullrequestreview-49116795006:34
mupPR #9345: overlord: introduce the export manager, export snapd tools <Created by zyga> <https://github.com/snapcore/snapd/pull/9345>06:34
mborzeckilet my grab my wife's laptop and i'll try it there06:34
zyga-x240I'll squash history and remove some typos and stuff from this later today06:35
mborzeckizyga-x240: what's this functionality? sorry, i was uc20 only for the last few weeks06:35
zyga-x240mborzecki: it's a solution to a very old bug06:40
zyga-x240mborzecki: introduced pretty much since day one06:40
zyga-x240mborzecki: when snapd or core refreshes and we don't reboot06:40
zyga-x240mborzecki: certain tools, like snap-exec and others, as seen by snaps from their mount namespace, become stale06:41
zyga-x240mborzecki: this fixes that problem by changing how those tools are exposed to snaps06:41
mborzeckiah06:41
zyga-x240mborzecki: snapd allows snaps to export content in a specific way06:41
zyga-x240mborzecki: the content shows up in /var/lib/snapd/export/<snap>/<revision>/<set-name>06:41
zyga-x240mborzecki: there's also a new current symlink at /var/lib/snapd/export/<snap>/current -> <revision>06:42
zyga-x240there's a special case for core and host06:42
zyga-x240but in general snapd tools are all available from /usr/lib/snapd/<tool> as symlinks to the export three, through the new current symlink there06:43
zyga-x240and snapd can update the current symlink atomically06:43
zyga-x240this also supports using host tools06:43
zyga-x240so for the first time ever, SNAP_REEXEC=0 uses host's snap-exec and other tools now06:43
zyga-x240and since this is not related to mount changes, it does not require mount namespace re-configuration when core or snapd change06:44
zyga-x240there's a new export manager that manages putting content on disk06:44
zyga-x240and a callback to the snap manager to update the new current symlink when we also update the primary current symlink06:44
zyga-x240that's pretty much it06:45
zyga-x240it is relevant to core20 as snapd refreshes separately from boot base06:45
zyga-x240so we reboot less often06:45
zyga-x240and we see this bug more and more over time06:45
mborzeckimhm, maybe i should take a look at the pull requests then06:45
zyga-x240heh06:46
zyga-x240it's not ready for proper review but if you want to look at this, please do06:46
zyga-x240alternatively06:46
zyga-x240in udev theme06:46
zyga-x240please look at06:46
zyga-x240https://github.com/snapcore/snapd/pull/761406:46
mupPR #7614: cmd/snap-confine: implement snap-device-helper internally <Needs security review> <Created by zyga> <https://github.com/snapcore/snapd/pull/7614>06:46
zyga-x240you reviewed it06:46
zyga-x240but maybe we can land it somehow06:46
zyga-x240and there were changes after your review06:46
zyga-x240lucy is sleeping but I will go AFK when she wakes up06:49
zyga-x240I'm so tired today06:49
zyga-x2401M -> 6AM06:49
zyga-x240ENOSLEEP06:49
pstolowskimorning06:58
zyga-x240hello06:59
mborzeckipstolowski: mvo: morning07:08
zyga-x240mvo: hello07:09
mvohey mborzecki, pstolowski and zyga-x240 ! good morning07:09
zyga-x240mvo: I will be semi absent in the morning, lucy is still sleeping next to me but she will wake up soon07:09
zyga-x240mvo: and I was up till midnight and need some rest / break from screens07:09
mvozyga-x240: ok07:09
zyga-x240I pushed https://github.com/snapcore/snapd/pull/9345 but I need to squash, clean up some typos and amend the commit message with a better description07:09
mupPR #9345: overlord: introduce the export manager, export snapd tools <Created by zyga> <https://github.com/snapcore/snapd/pull/9345>07:09
zyga-x240also doing a small self-review showed a few extra changes to make07:10
zyga-x240I wrote a very useful test that shows many things about how snap execution environment works07:10
zyga-x240and found a few surprises myself07:10
zyga-x240the test is here: https://github.com/snapcore/snapd/pull/9345/files#diff-e96db38f8622850cdf91281e5344f76207:11
mupPR #9345: overlord: introduce the export manager, export snapd tools <Created by zyga> <https://github.com/snapcore/snapd/pull/9345>07:11
mvozyga-x240: oh, nice07:13
mvozyga-x240: anything I can help you with? I mean that does not requires samuele reviews and could still land :)07:13
zyga-x240mvo: not yet, I will open it properly later today, then it needs reviews07:14
zyga-x240mvo: I mainly wanted to share what I have now and see what fails07:14
zyga-x240did anyone look at centos / opensuse failures?07:15
mborzeckisomeone pushed wip/export-manager-patch-set branch to snapd upstream?07:15
zyga-x240ah07:15
mborzeckizyga-x240: ?07:15
zyga-x240sorry07:15
zyga-x240yeah07:16
zyga-x240done07:16
zyga-x240sorry07:16
mvozyga-x240: sure thing07:16
* zyga-x240 goes to take a shower07:20
zyga-x240mborzecki: more selinux denials07:20
zyga-x240from export manager07:21
zyga-x2402020-09-18T06:52:34.1787196Z type=AVC msg=audit(1600411883.733:7774): avc:  denied  { mount } for  pid=69477 comm="snap-confine" name="/" dev="tmpfs" ino=227354 scontext=system_u:system_r:snappy_confine_t:s0 tcontext=system_u:object_r:tmpfs_t:s0 tclass=filesystem permissive=107:21
zyga-x2402020-09-18T06:52:34.1789352Z type=AVC msg=audit(1600411883.733:7775): avc:  denied  { write } for  pid=69477 comm="snap-confine" name="/" dev="tmpfs" ino=227354 scontext=system_u:system_r:snappy_confine_t:s0 tcontext=system_u:object_r:tmpfs_t:s0 tclass=dir permissive=107:21
zyga-x2402020-09-18T06:52:34.1793007Z type=AVC msg=audit(1600411883.734:7776): avc:  denied  { add_name } for  pid=69477 comm="snap-confine" name="etelpmoc.sh" scontext=system_u:system_r:snappy_confine_t:s0 tcontext=system_u:object_r:tmpfs_t:s0 tclass=dir permissive=107:21
zyga-x2402020-09-18T06:52:34.1796343Z type=AVC msg=audit(1600411883.734:7777): avc:  denied  { create } for  pid=69477 comm="snap-confine" name="etelpmoc.sh" scontext=system_u:system_r:snappy_confine_t:s0 tcontext=system_u:object_r:tmpfs_t:s0 tclass=lnk_file permissive=107:21
zyga-x240and more07:21
zyga-x240ehh07:21
mborzeckitmpfs_t ?07:21
mborzecki/var/lib/snapd/export is tmpfs?07:21
zyga-x240yes07:21
zyga-x240we07:21
zyga-x240no07:21
zyga-x240sorry07:21
zyga-x240 /usr/lib/snapd is a tmpfs07:22
zyga-x240 /var/lib/snapd/export is not07:22
zyga-x240also some real failures on 14.0407:22
zyga-x240but first shower07:22
mborzeckizyga-x240: when mounting /usr/libexec/snapd, it'd be nice to set some label for that, preferrably the same as the default one for that location07:29
mupPR snapd#9371 opened: o/snapstate: improve snapshot iteration <Created by stolowski> <https://github.com/snapcore/snapd/pull/9371>08:26
mupPR snapd#9372 opened: tests: make nested signing helpers less confusing <Run nested> <Simple 😃> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9372>08:46
mupPR snapd#9373 opened: snap: add new `snap debug show-recovery-key` command <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9373>08:46
mupPR snapd#9367 closed: tests: misc nested changes <Run nested> <Simple 😃> <Test Robustness> <Created by anonymouse64> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9367>08:56
mvopstolowski: I reviewed your PR about snapshots, nice job09:02
mvopstolowski: each step to untangle this a bit is good :)09:03
pstolowskimvo: thanks, yeah ;)09:04
ijohnsonmorning folks09:56
pstolowskihey ijohnson !10:00
ijohnsono/ pstolowski10:00
mborzeckipstolowski: added some comments to #937110:07
mupPR #9371: o/snapstate: improve snapshot iteration <Created by stolowski> <https://github.com/snapcore/snapd/pull/9371>10:07
pstolowskity10:08
mborzeckianyoen looked at #9370?10:08
mupPR #9370: usersession/userd: separate bus name ownership from defining interfaces <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/9370>10:08
zygare10:15
zygaman I was sleepy10:15
zygaso why does livepatch test faik10:37
zyga*fail10:37
zygahmm10:38
zygait passed10:38
* zyga looks at logs10:38
zygaha, reproduced10:48
zyganow to lok10:48
zygalook10:48
mborzeckiso bug or no bug?10:48
zygayeah10:49
zygafiguring out why10:49
zygacontent was not exported10:49
zygaok10:50
zygaI understand10:50
zygahmmm10:50
zygaso10:50
zygare exec from old snapd10:50
zygathe init process of the manager is correct10:50
zygabut needs to handle exporting core/snapd, not just host10:50
zygagood thing we have those tests!10:50
zygaok testing a fix now11:10
zygaaa11:15
zygaI switched to dvorak by accident11:15
zygafelt uber weird in vim11:15
zygaspread running, now to write an unit test11:15
ijohnsonzyga: mind quickly reviewing https://github.com/snapcore/snapd/pull/9374 ?11:15
mupPR #9374: tests/main/lxd: disable cgroup combination for 16.04 that is failing a lot <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9374>11:15
mupPR snapd#9374 opened: tests/main/lxd: disable cgroup combination for 16.04 that is failing a lot <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9374>11:17
* pstolowski lunch11:26
zygaand now spread passes11:27
zygaijohnson: looking11:27
ijohnsonthanks11:27
zygaijohnson: thanks for that!11:27
zygaijohnson: https://github.com/snapcore/snapd/pull/9374#pullrequestreview-49138453311:33
mupPR #9374: tests/main/lxd: disable cgroup combination for 16.04 that is failing a lot <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9374>11:33
ijohnsonah fair enough that makes sense11:33
ijohnsonzyga: fixed11:36
zygalooking11:36
zygadone11:37
ijohnsoncool, yeah this was the combo I've seen fail just today 2 times, but perhaps another combo with xenial also doesn't work11:40
zygawe'll see11:41
zygapawel has network issues11:46
zygamborzecki: this keyboard is so cool, no mouse required11:46
zygamborzecki: no arrow keys required11:46
zygamborzecki: I think I properly got used to it now11:47
ograhmm, all of a sudden the whole documentation links from MOTD are gone on a few of my core18 installs11:49
ijohnsonthere were motd changes to the core* snaps recently11:50
ograyeah, feels a bit regressed11:50
zygawoot11:50
zygamore tests11:50
ograogra@acheron:~$ ssh 192.168.2.8511:50
ograWelcome to Ubuntu Core 18 (GNU/Linux 5.3.0-1033-raspi2 armv7l)11:50
ograLast login: Fri Sep 18 11:12:05 2020 from 192.168.2.4811:50
ograogra@localhost:~$11:50
ograthats really sparse now11:50
ogracompared to how it should look like:11:51
ograhttps://paste.ubuntu.com/p/r9dcmvqnfG/11:51
ijohnsonogra: probably worth a bug to github.com/snapcore/core1811:51
* zyga found snap.R11:51
ograbah ... and bluez doesnt work at all on core16 (and i really dont want to move that specific pi to core1😎11:52
ogragrr ... emoji invasion !11:52
ogras/core1😎/core18 /11:52
zygaok one more unit test and everything is covered11:55
zygaI like this new manager11:56
zygait has potential11:56
zygaperfect12:01
zygaok12:01
zyganow to go through the todos left12:02
zygameh12:17
zygaI broke my git config and this makes git push go to origin12:17
ijohnsonit's a whole 1 degree celsius outside12:35
ijohnsonSU this morning will be a chilly one12:36
mupPR snapd#9372 closed: tests: make nested signing helpers less confusing <Run nested> <Simple 😃> <UC20> <Created by bboozzoo> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/9372>12:37
mvoijohnson: impressive! it's +24°C here12:42
ijohnsonhaha yeah it's supposed to be ~20°C tomorrow12:43
ogradid you move to antarcica ?12:44
ijohnsonhaha nope, still MN12:44
ogracrazy12:44
ijohnsonjust very cold September this year12:44
mvoogra: I think they have +34°C or something12:44
ograbrrr12:44
* ogra sits on the terrace (enjoying my vac) ... at +20°C ... 12:45
ijohnsonweather at the south pole today is -48 C12:45
mvoijohnson: heh12:46
mvoijohnson: I was refering to https://earthobservatory.nasa.gov/images/146322/antarctica-melts-under-its-hottest-days-on-record but I'm apparently a couple of months late12:47
ijohnsonyeah I think it's still winter in antartica, so a little bit colder than the summer but yes indeed that is crazy news12:48
ograyeah, i read that one on german news sites a few months ago12:48
mvocurrent -48°C sounds more like I imagine antartica :)12:49
ijohnsonas I googled that I was very confused why it was like midnight at the south pole right now, but actually you could use whatever TZ you want, but I guess they customarily use New Zealand time12:50
ijohnsonTIL12:51
mupPR snapd#9375 opened: tests/nested/cloud-init-*: simplify tests and unify helpers/seed inputs <Run nested> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9375>12:52
mvoijohnson: haha, good point12:52
ograxnox, motd being broken in core18 ... do i file that to GH or under lp:snappy ?12:54
ogra(well ... s/core18/UC18/)12:56
ijohnsonogra: github pretty sure12:56
ograkay12:57
* ogra files https😕/github.com/snapcore/core18/issues/17013:02
ograGRRR !!!13:02
* ogra files github.com/snapcore/core18/issues/17013:02
mupIssue core18#170 opened: [Regression] all documentation links and explanatory text vanished from MOTD <Created by ogra1> <https://github.com/snapcore/core18/issues/170>13:05
* zyga breaks for errand13:40
zyga1-2h13:41
=== kirkland is now known as Guest82000
mupPR snapd#9374 closed: tests/main/lxd: disable cgroup combination for 16.04 that is failing a lot <Simple 😃> <Test Robustness> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9374>15:03
mupPR snapd#9376 opened: check-pr-title.py * : allow "*" in the first part of the title <Simple 😃> <Skip spread> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9376>15:03
ijohnson#9376 is really simple and would be nice to have :-)15:04
mupPR #9376: check-pr-title.py * : allow "*" in the first part of the title <Simple 😃> <Skip spread> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9376>15:04
cachiomvo, hey15:12
cachiothe delay that I mentioned was caused by this15:12
cachio[   14.613707] systemd[1]: Condition check resulted in Create System Users being skipped.15:12
cachioerror: cannot obtain system details: cannot communicate with server: Get http://localhost/v2/system-info: dial unix /run/snapd.socket: connect: connection refused15:12
cachioPress enter to configure.15:12
cachiothis is the full log15:12
cachiohttps://paste.ubuntu.com/p/tn2sK4cDSC/15:13
cachioI created the image from edge15:13
ijohnsoncachio: edge is broken15:13
ijohnsoncachio: this is the issue I was talking about in SU, we need a new kernel snap15:13
cachioijohnson, I should try with beta?15:13
ijohnsoncachio: no beta is broken too15:13
cachiohehehe15:13
ijohnsoncachio: the only thing to do is to rebuild the kernel snap15:13
ijohnson(or wait for that to be done and published)15:13
cachioijohnson, so, I could make the focal image work with this old kernel15:14
ijohnsoncachio: yeah that was great news15:14
cachioworks with kvm15:14
cachioyes15:14
ijohnsoncachio: I mean really weird15:14
ijohnsonbut still great that you got it working15:14
ijohnsonhow difficult is it to create and update that image in gce ?15:14
cachioI tried with another kernel, an older one and didn't work15:14
cachioijohnson, I have an image ready15:15
ijohnsoncachio: great15:15
cachioijohnson, test-virt-115:15
ijohnsoncachio: might also be worth asking the server team if they expect any issues with a xenial kernel on focal15:15
cachiothis is the name15:15
ijohnsoncachio: nice15:15
cachioit uses the last qemu and ovmf for focal15:16
ijohnsonright15:16
mupPR snapcraft#3291 opened: build providers: make mount_project() interface private <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3291>15:16
cachioI¿ll create a new pr to enable kvm and test nested15:16
ijohnsonnice, let me know when it's up and I'll review it15:16
cachioijohnson, sure, I'll make it after lunch15:17
ijohnson+115:17
* cachio lunch15:27
mvocachio: thanks, in meetings. but thanks for letting me know15:32
zygare15:46
zygaback from store15:46
* zyga reviews failures on 934515:47
zygaaha15:51
zygaI guess SNAP_REEXEC=0 is not supported on core :D15:51
* zyga adjusts 15:51
zygaok now to dive into selinux permissions15:55
mupPR snapd#9377 opened: [RFC] snap-repair: minimal uc20 support <Created by mvo5> <https://github.com/snapcore/snapd/pull/9377>15:58
zygammm16:12
zygamy wife just brought me some coffee16:12
mupPR snapd#9378 opened: tests/nested/manual: add uc20 grade signed cloud-init test <Run nested> <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9378>16:28
mupPR snapd#9379 opened: [RFC] cmd/s-b/initramfs-mounts: use ConfigureTargetSystem for install, recover modes <Bug> <Needs Samuele review> <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9379>17:58
* ijohnson EODs a bit early today, still around for random fires if needed18:12
cachioijohnson, sorry for the delay18:34
cachio#938018:34
mupPR #9380: tests: update to support nested kvm without reboots on UC20 <Run nested> <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9380>18:34
cachioyou can check that on Monday18:34
cachiono hurry18:35
cachiocmatsuoka, you too #938018:35
zyga-mbpre18:35
* cmatsuoka verifies...18:35
zyga-mbpcan I review anything specific to help?18:35
cachiozyga-mbp, also for you if you have time #938018:35
mupPR #9380: tests: update to support nested kvm without reboots on UC20 <Run nested> <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9380>18:35
zyga-mbpon it18:35
cmatsuokacachio: is the focal-with-old-kernel image already in gce?18:37
zyga-mbpcachio done18:37
mupPR snapd#9380 opened: tests: update to support nested kvm without reboots on UC20 <Run nested> <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9380>18:38
cachiocmatsuoka, yes18:42
zyga-mbpanything else I can review?18:42
cachiozyga-mbp, thanks!!18:42
cachiocmatsuoka, I also updated spread images project and tested the new images18:43
cmatsuokacachio: ah nice thanks!18:43
cachiocmatsuoka, the problem now is that edge build is broken18:44
cachiobecause of the kernel18:44
cmatsuokaah yes, that's true18:44
zyga-mbpohhh18:44
cachiocmatsuoka, so I tested the image works with -smp 218:44
zyga-mbpvmware fusion supports virtual TPM18:44
zyga-mbpcmatsuoka how can I build up-to-date core20 image for amd64?18:45
cachioyou will see that the image boot fails18:45
cachiobet you can login18:45
zyga-mbpone that would be testable with vTPM18:45
cachioto the nested vm18:45
cachiobut snap command fails18:45
cachiozyga-mbp, it is working with vtpm18:45
zyga-mbpyeah, I just never played with that befor18:45
zyga-mbp*before18:45
zyga-mbpand since it's Friday I might as well try18:45
cachionice18:46
* cachio quick break18:46

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