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

mborzeckimorning05:25
mborzeckischool run, back in 3005:36
mvohey mborzecki - good morning05:58
mborzeckire06:11
mborzeckimvo: hey06:12
mborzeckihmm https://www.reddit.com/r/openSUSE/comments/j0zcs1/snap_cannot_open_display_0/06:17
mborzeckisimilar thing was reported in the forums, but fixed apparently? https://forum.snapcraft.io/t/opensuse-tumbleweed-snap-application-cannot-open-display-0/1909306:18
zygagood morning06:34
mborzeckimvo: can you take a look at https://github.com/snapcore/snapd/pull/9401 ?06:39
mupPR #9401:  gadget: allow content observer to have opinions about a change  <Simple 😃> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9401>06:39
zygamvo: some question for you wrt nested https://github.com/snapcore/snapd/pull/9414#pullrequestreview-49722082906:39
mupPR #9414: tests: new nested tool <Run nested> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9414>06:39
zygasince you looked at nested recently06:39
zygabut feel free to defer to ian if you are too busy06:39
zygagroovy's handling of intel graphics is terrible06:44
zygaI have screen artefacts, tearing and triangles while I scroll anything06:44
pstolowskimorning07:00
mvopstolowski: good morning07:03
pstolowskio/07:03
mvozyga, mborzecki sorry, was distracted but will look at your questions now07:04
mborzeckipstolowski: hey07:04
mupPR snapd#9401 closed:  gadget: allow content observer to have opinions about a change  <Simple 😃> <UC20> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9401>07:17
mvo9383 needs a second review but maybe something for ian07:18
* mvo hugs zyga for all his code reviews this morning07:19
mvopstolowski: is the "Thank you" in 9404 a "Approve" ?07:33
pstolowskimvo: ah, not really, i though i made that comment under the original PR (where I asked about this line). i'll review this one in full07:35
pstolowski*though07:35
pstolowski*thought07:35
pstolowskigrr07:35
mborzeckimvo: uc20 does not boot with 256mb of ram08:28
mborzeckimvo: https://i.imgur.com/ZGUrsi4.png08:29
mupPR snapd#9420 opened: tests/nested/manual/minimal-smoke: revert to using 256MB for nested UC20 <Run nested> <UC20> <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9420>08:32
mupPR snapd#9404 closed: asserts: deserialize grouping only once in Pool.AddBatch if needed <Bulk assert refresh :scroll::scroll::scroll:> <Created by pedronis> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9404>08:42
mborzeckimvo: i've filed https://bugs.launchpad.net/snapd/+bug/1897533 please adjust the priority08:56
mupBug #1897533: pc-kernel does not boot on a system with 256MB of RAM <snapd:New> <https://launchpad.net/bugs/1897533>08:56
zygamborzecki: https://github.com/snapcore/snapd/pull/9420 will this run nested after you added the label?09:14
mupPR #9420: tests/nested/manual/minimal-smoke: revert to using 256MB for nested UC20 <Run nested> <UC20> <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9420>09:14
mborzeckizyga: yes, some of the nested runs already finished09:15
zygaor is the UI confusing and it was done initially?09:15
zygaok09:15
mborzeckizyga: yup, i was added when i opened the issue09:15
mborzeckizyga: anyways, uc20 will fail there :/09:16
mvomborzecki: nice, thanks for this one09:18
zygaack09:23
mupPR snapd#9421 opened: Added Ledger to U2F devices list in interfaces/builtin <Created by TamtamHero> <https://github.com/snapcore/snapd/pull/9421>09:27
zygasigh09:29
zyga20.10 broke or hid cellular connection control09:30
mupPR snapd#9041 closed: osutil/group.go: treat all non-nil errs from user.Lookup{Group,} as Unknown* <Bug> <Needs security review> <Preseeding 🍞> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9041>09:47
pedronisdegville: mvo: I commented in #939909:52
mupPR #9399: snap: snap help output refresh <Needs Samuele review> <Created by degville> <https://github.com/snapcore/snapd/pull/9399>09:52
mvopedronis: ta09:52
mvopedronis: sounds good09:53
mvo(just read it)09:53
degvillepedronis: mvo: thank you! I've left a few comments after your comments.10:20
* zyga splits first branch of the export manager 10:32
pedronisdegville: thanks, I will try to look at it again later today. What do you think of the idea of organizing/splitting Other into real categories for snap help --all ?11:28
degvillepedronis: np - hopefully the latest update is easier to look over. I think splitting Other into real categories for snap help --all is a good idea!11:36
mupPR snapd#9411 closed: cmd/snap/auto-import: stop importing system user assertions from initramfs mnts <Bug> <Simple 😃> <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9411>11:38
mupPR snapd#9422 opened: overlord: add link participant for linkage transitions <Created by zyga> <https://github.com/snapcore/snapd/pull/9422>11:58
zygaFYI, on groovy I have errors in master related to user lookup12:13
zygauser_test.go:552:12:13
zyga    c.Assert(err, check.ErrorMatches, "useradd failed with: some error")12:13
zyga... error string = "user: lookup username lakatos: no such file or directory"12:13
zyga... regex string = "useradd failed with: some error"12:13
zygahas anyone else seen this?12:14
zygaI didn't debug it yet12:14
zygago 1.14.712:14
* zyga goes for a walk12:23
mvozyga: one PR from ian landed recently, let me try this12:26
pedronismvo: that PR was very old, it probably needed a master merge and test run before landing12:30
mvopedronis: oh no12:30
pedronisI mean, it was open and untouched for months12:30
mvopedronis: I will fix12:30
mvopedronis: yeah12:30
mvopedronis: but probably after the meeting12:31
micahhi, i've just installed the grafana snap, and I'm trying to find where the typical config.ini would be located, I see there is a ~/snap/grafana/current, but there isn't anything in there13:14
mborzeckimvo: pushed a little fix to https://github.com/snapcore/snapd/pull/938313:36
mupPR #9383: snap-repair: add uc20 support <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9383>13:36
mvomborzecki: aha, nice13:36
mvomborzecki: thanks!13:36
ijohnsonmborzecki: pedronis: should I add #8947 back to be reviewed again?13:48
mupPR #8947: many: update managed boot config when refreshing snapd <UC20> <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8947>13:48
ijohnsonpedronis: mvo: I thought I recently pushed a master merge to that gid/uid branch, perhaps I didn't13:49
ijohnsonyeah I merged master 10 days13:49
ijohnsonago13:49
ijohnsonjdstrand: do we still need security review for pr's like #9421, which are just adding additional devices to the U2F interface? We've gotten a few of these from external contributors the past few months, and while simple perhaps we still should get security review for them13:50
mupPR #9421: interfaces/utf: Add Ledger to U2F devices <Created by TamtamHero> <https://github.com/snapcore/snapd/pull/9421>13:50
jdstrandijohnson: yes, anything touching interface policy should get an ack from our team. you can ping me and I can try to find someone to do it13:55
jdstrandijohnson: for that on, I just now commented: https://github.com/snapcore/snapd/pull/9421/files#r49595791913:55
mupPR #9421: interfaces/utf: Add Ledger to U2F devices <Created by TamtamHero> <https://github.com/snapcore/snapd/pull/9421>13:55
ijohnsonjdstrand: ok, sounds good I will add "needs security review" to that pr and any new ones that come in too13:55
jdstrandone*13:55
ijohnsonawesome thanks jdstrand13:56
ijohnsonin that case I don't even need to add "needs security review" to that pr :-)13:56
jdstrandI don't think so :)13:56
ijohnson:-D14:05
zygamborzecki: can you do a review for https://github.com/snapcore/snapd/pull/7614#pullrequestreview-49717634814:15
mupPR #7614: cmd/snap-confine: implement snap-device-helper internally <Needs security review> <Created by zyga> <https://github.com/snapcore/snapd/pull/7614>14:15
zygait got a security review last night so we are good to have it in, finally :)14:15
zygaI know you +1 already but having a new review would be good14:15
zygathanks!14:16
* zyga is hungry, time for dinner 14:20
micahwhere does data get saved when I use a snap and save something?14:54
zygamicah: I think /var/lib/snapd/snapshots14:56
micahzyga: huh, ok... thanks14:57
ijohnsonmicah: do you mean you can't find a file you meant to save in $HOME? it could be in $HOME/snap/<snap-name>/current15:12
micahijohnson: i saved a grafana dashboard, and I am wondering where it 'saved' to15:32
ijohnsonmicah: probably somewhere in /var/snap/grafana/current15:34
micahijohnson: unfortunately, that directory is just empty :o15:35
ijohnsonmicah: what about /var/snap/grafana/common ?15:35
ijohnsonmicah: also to confirm what is the name of the snap that you used when you are running grafana? I am just assuming it's "grafana" verbatim15:36
mupPR snapd#9395 closed:  o/snapstate/check_snap_test.go: mock osutil.Find{U,G}id to avoid bug in test <Bug> <Test Robustness> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9395>15:39
mupPR snapd#9423 opened: tests/nested/manual/preseed: include a system-usernames snap when preseeding <Preseeding 🍞> <Run nested> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9423>15:39
* cachio lunch15:47
mupPR snapd#9348 closed: tests: print all the serial logs for the nested test <Run nested> <Simple 😃> <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9348>17:09
pedronismmh, you can't leave out commands apparently: SnapSuite.TestHelpCategories17:28
mupPR snapd#9424 opened: cmd/snap: allow snap help vs --all to diverge purposefully <Created by pedronis> <https://github.com/snapcore/snapd/pull/9424>18:19
pedronisdegville: ^  the things I discussed this morning about snap help18:24
degvillepedronis: just looking now - thanks!18:25
pedronisdegville: thx, I answered your comments. Any input on the special purpose section?18:56
degvillepedronis: sorry Samuele - just about to record a podcast. I couldn't think of anything quickly, but I'll give it some thought tonight (if that's not too late). I think it's fundamentally fine as it is though.19:04
=== JanC is now known as Guest49463
=== JanC_ is now known as JanC
mupPR snapd#9425 opened: tests: new tests.backup tool <Run nested> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9425>19:30
mupPR snapcraft#2673 closed: docker: parametrize risk and OS for image builds (core18 etc) <Created by abitrolly> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2673>20:36
mupPR snapcraft#2871 closed: snaps: account for forwarded (effective) installation channels <Created by cjp256> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2871>20:41
* cachio afk20:53
mupPR # closed: snapcraft#2971, snapcraft#3062, snapcraft#3127, snapcraft#328720:56
mupPR snapcraft#3234 closed: project loader: refactor errors to conform to SnapcraftException <Created by cjp256> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/3234>21:06
mupPR snapcraft#3221 closed: repo: install requested build-package versions <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3221>21:11
mupPR snapcraft#3256 closed: [wip] repo: consolidate BaseRepo and Ubuntu & reorganize package <enhancement> <Created by cjp256> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/3256>21:16
mupPR snapcraft#3297 closed: cli: support snap --output <directory> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3297>21:16

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