[01:05] <mup> PR snapd#8398 opened: usersession/userd: add "slack" to the white list of URL schemes handled by xdg-open <Created by troyready> <https://github.com/snapcore/snapd/pull/8398>
[05:45] <mborzecki> morning
[06:47] <mborzecki> mvo: hey
[06:48] <mborzecki> mvo: the console_conf PR in subiquity has landed
[06:49] <mborzecki> mvo: i'm assembling the image to try everything out
[06:54] <mvo> hey mborzecki
[06:54] <mvo> mborzecki: niiiiiiiiiiiice
[07:01] <pstolowski> morning
[07:10] <mvo> hey pstolowski
[07:37] <mborzecki> pstolowski: hey
[07:39] <mup> PR snapd#8397 closed: cmd/snap-confine/mount-support-nvidia.c: add libnvoptix as nvidia library <Simple 😃> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8397>
[07:43] <zyga> Hey
[07:43] <zyga> Had a very fractured night
[07:43] <zyga> Barely awake now
[07:43] <zyga> Sorry, I will be late today
[07:44] <zyga> I will not be at the desktop sync
[07:50] <mup> PR snapd#8382 closed: packaging: detect/disable broken seed in the postinst <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8382>
[07:56] <mvo> pedronis_, zyga, pstolowski, mborzecki I will release 2.44.2 this morning, please let me know if there is anything last-minute you want to have in it
[07:57] <mup> PR snapd#8226 closed: interfaces/policy/policy_test.go: add more tests <Created by jdstrand> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8226>
[07:57] <zyga> ack
[07:57] <zyga> mvo: I would really like to have https://github.com/snapcore/snapd/pull/8089
[07:58] <mup> PR #8089: features: enable robust mount ns updates <Created by zyga> <https://github.com/snapcore/snapd/pull/8089>
[07:58] <zyga> mvo: but it's up to you
[07:59] <mvo> zyga: hm, it's not on master yet, we should at least get it there
[08:00] <zyga> mvo: yeah but it's pending for a long time
[08:00] <zyga> just shinging some light on it
[08:00] <mvo> zyga: ta
[08:00] <zyga> if it helps our customers enable it via gadget
[08:00] <zyga> and it's well-known to improve
[08:01] <zyga> er, cause improvements
[08:01]  * zyga is trying to say awake
[08:01] <mvo> zyga: I'm not against it, just saying it needs to land in master first and get reviews etc
[08:01] <mvo> zyga: haha - did you had coffee already :) ?
[08:01] <zyga> +3/-2 - let's review it :)
[08:01] <zyga> mvo: yeah, my wife just brought me a cup
[08:02] <zyga> 2-3 AM, 4AM, 6AM wake-ups (for me) last night, not great
[08:03] <mborzecki> meh, when i repack cor20 snap the system no longer boots, it stops at initrd-switch-root and complains it cannot resolve /sbin/init even though /sysroot/sbin/init is there (though it points to /lib/systemd/systemd but it's there too)
[08:03] <pstolowski> mvo: thanks, i don't have anything
[08:04] <zyga> mborzecki: is there any write-up on how to try core20 locally?
[08:04] <zyga> mborzecki: can you please review the tiny branch referenced above (8089)
[08:04] <mborzecki> zyga: try or build?
[08:04] <zyga> mborzecki: be able to run from source
[08:04] <zyga> mborzecki: boot and see what happens, etc
[08:04] <zyga> mborzecki: (in qemu)
[08:05] <mborzecki> zyga: i don't think so, but it's the usual thing, grab the model, build image with u-i, run qemu but adding UEFI
[08:05] <pstolowski> grr during the transition to new laptop i forgot to set git author initially and managed to commit with defaults to all my PRs, and have to fight with CLA checks (need rebasing to fix as it's for past commits, not just latest)
[08:05] <zyga> mmm
[08:06] <zyga> pstolowski: ouch, there's a way to rewrite the author and committer documented in git rebase
[08:06] <zyga> pstolowski: you didn't migrate your HOME?
[08:17] <pstolowski> zyga: i know of git commit --amend --reset-author which works for last commit; i'm not sure rebase has anything specific like that, but simply rebasing fixes this
[08:17] <abeato> hm, CI fails on 'apt update'
[08:18] <pstolowski> zyga: yeah i didn't migrate home, i wanted to have clean slate (instead of removing cruft after migration)
[08:18]  * zyga nods
[08:21] <mborzecki> aah
[08:21] <mborzecki> w8, how's that even possible?
[08:24] <zyga> mborzecki: 2 st phase of debugging
[08:24] <zyga> 2st is a new number, between 1st and 2nd
[08:24]  * zyga should sleep
[08:26]  * mborzecki trying a differnt rsync incantation
[08:28] <mborzecki> rsync --wingardium-leviosa --dwim
[08:28] <zyga> it works on 2st try
[08:31] <mborzecki> yup, much better now
[08:47] <abeato> could somebody retrigger tests for https://github.com/snapcore/snapd/pull/8320 ?
[08:47] <mup> PR #8320: interfaces: updates to login-session-observe, network-manager and modem-manager interfaces <Created by alfonsosanchezbeato> <https://github.com/snapcore/snapd/pull/8320>
[08:48] <mup> PR snapd#8399 opened: interfaces/policy: fix comment in recent new test <Created by pedronis> <https://github.com/snapcore/snapd/pull/8399>
[08:49] <mborzecki> btw. snap debug state is super slow when there's many tasks
[08:55] <mup> PR snapd#8400 opened: osutil: make the TestWriter() test less racy <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8400>
[09:07] <zyga> mvo: let's merge it but I left a kind of -1 comment
[09:07] <mvo> zyga: tell me how to do it non-racy :)
[09:07] <mvo> zyga: I thought aobut it but couldn't come up with an idea
[09:07] <zyga> that's much more towards a solution than what we usually do :)
[09:07] <zyga> mvo: no that I disagree it is hard
[09:08] <mvo> zyga: I mean, we test that during the time until the deadline is triggered we get data
[09:08] <mvo> zyga: *if* the system is that slow that there is just no thread scheduled during $time we lost
[09:08] <zyga> mvo: but just doing 0.1 seconds * number_of_times_we_had_to_change_it really feels iffy
[09:09] <zyga> mvo: I get that, but in general a test for more than one concurrent activity must not be based on wall clock time
[09:09] <mvo> zyga: well, $time is a fundamental part of the test, so I don't really know how to do it non-racy, I'm open for ideas though
[09:09] <zyga> mvo: here it seems that way but really what you want is to show a certain sequence of events happens in two threads
[09:09] <zyga> mvo: not that it took this amount of time or that amount of time
[09:09] <mvo> zyga: yes, in general I agree, but this test is precisely about testing wall-clock behavior
[09:09] <zyga> mvo: anyway, I will look
[09:10] <zyga> mvo: I think at some point the tipping point makes us research how to deal with sutuff
[09:10] <zyga> *stuff
[09:10] <zyga> mvo: I can only re-iterate that this is not about that wall clock time _at all_ but about the sequence
[09:10] <zyga> mvo: in a way if we find a way to do this we will get correct *and* faster tests
[09:10] <mvo> zyga: did you look at the test? it's about hitting a deadline and what happens until this deadline is hit
[09:11] <zyga> as arbitrary sleep durations go away
[09:11] <zyga> mvo: I did, perhaps my point is not expressed clearly enough though
[09:11] <zyga> mvo: anyway, I will look
[09:11] <mvo> zyga: don't get me wrong, I agree 100% in general, but I don't see how to do this for this specific case
[09:12] <mborzecki> hmmm soo, the chooser run nicely in a local terminal, but is stul reading from stdin when running in a uc20 shell, same command line but different effect
[09:12] <mborzecki> even simple echo '{}' | python3 -m console_conf.cmd.tui --recovery-chooser-mode is stuck
[09:13] <mvo> mborzecki: oh no!
[09:13] <pedronis> mvo: mmh
[09:13] <zyga> mborzecki: can you debug log the mode of stdin/stdout
[09:14] <zyga> mborzecki: are you in binary or text mode
[09:14] <zyga> mborzecki: what's the encoding?
[09:14] <mborzecki> ah w8, maybe that
[09:14] <pedronis> mvo: those tests are really testing context.WithTimeout, not our code
[09:14] <zyga> mborzecki: cheers :)
[09:14] <pedronis> mvo: so a bit unclear if need all of them, or the cancel one is enough
[09:14] <pedronis> or we just need our own base context
[09:15] <mvo> pedronis: fair enough, we could remove it entirely
[09:17] <mvo> pedronis: happy to update the PR to remove it, wdyt?
[09:17] <pedronis> mvo: I think the Cancel one is enough, but maybe I'm missing something ?
[09:17] <pedronis> we lose the happy case ?
[09:18] <pedronis> maybe the happy case is just with a cancelable that we cancel after and not before
[09:18] <mvo> pedronis: I guess so, I don't know more about this particular case then you :) it was a drive-by from my bug triage
[09:18] <mup> PR snapd#8401 opened: snap: improve TestWaitRecovers test <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8401>
[09:23] <zyga> mborzecki: the chooser was merged but I will continue with my review
[09:23] <zyga> mborzecki: there are some things that probably ought to be followed upon
[09:24] <mwhudson> can i make stage-packages: architecture dependent?
[09:25] <mvo> mwhudson: iirc "- on amd64\n  - "pkg"" works
[09:25] <mwhudson> mvo: just found a forum post with a syntax like that, yeah
[09:26] <mwhudson> mvo: is it documented at all?
[09:26] <mvo> mwhudson:
[09:26] <mvo> mwhudson: https://forum.snapcraft.io/t/architectures/4972
[09:26] <mvo> mwhudson: I thnk that might be the docs
[09:26] <mvo> mwhudson: actually no
[09:26] <mvo> mwhudson: it's just about the architecture :/
[09:27] <mwhudson> like can i say - on amd64, arm64\n  - efibootmgr
[09:27] <mborzecki> damn, it works over ssh, but not in a console
[09:27] <mborzecki> zyga: ^^
[09:27] <zyga> mborzecki: what do you get in the console
[09:27] <mvo> mwhudson: yeah, this should work
[09:27] <zyga> mborzecki: binary/text?
[09:27] <zyga> mborzecki: and what encoding if text
[09:27] <mborzecki> even though i added LANG, TERM and whatnot in the console
[09:28] <zyga> mborzecki: did you add the debugging?
[09:28] <mvo> mwhudson: I couldn't find docs for this, might be worthwhile to check with degville and/or the snapcraft guys if "stage-package:\n- on $arch:\n  - pkg1" is documented somewhere and just hard to find
[09:29] <mwhudson> mvo: i shall whine on the forum!
[09:29] <mvo> mwhudson: :) +1
[09:33] <abeato> mvo, could you please retrigger tests for https://github.com/snapcore/snapd/pull/8320 and https://github.com/snapcore/snapd/pull/8220 ?
[09:33] <mup> PR #8320: interfaces: updates to login-session-observe, network-manager and modem-manager interfaces <Created by alfonsosanchezbeato> <https://github.com/snapcore/snapd/pull/8320>
[09:33] <mup> PR #8220: interfaces/seccomp: allow passing an address to setgroups <Created by alfonsosanchezbeato> <https://github.com/snapcore/snapd/pull/8220>
[09:41] <mwhudson> mvo, degville: what's "try" about in this context?
[09:42] <mvo> abeato: sure, looking
[09:43] <mwhudson> aaah i found https://snapcraft.io/docs/snapcraft-advanced-grammar
[09:49] <mup> PR snapd#8320 closed: interfaces: updates to login-session-observe, network-manager and modem-manager interfaces <Created by alfonsosanchezbeato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8320>
[09:50] <pedronis> mborzecki: mvo: what's the status of #8253 ?
[09:50] <mup> PR #8253: snap-bootstrap: expand data partition on install <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8253>
[09:50] <pedronis> mvo: do we need it for beta?
[09:51] <mborzecki> pedronis: thought i +1'ed it already, let me check
[09:52] <mborzecki> pedronis: there's an open question about explicitly identifying expand/noexpand for partitions at the gadget yaml level, but i don't think we can resolve it now
[09:58] <mvo> mborzecki: I would say let's always expand and if we have a use-case *not* to do that we can add an extension to the gadget.yml (cc pedronis) - irrc this is what I suggested in there too, no?
[10:15] <pedronis> mvo: it sounded you suggested the reverse, to have an explicit way to expand
[10:15] <mvo> pedronis: that was my first thinking but that's silly, it will be the common case I think, let me clarify
[10:20] <mvo> pedronis: I updated the PR with my thinking, hope it makes sense
[10:29] <mup> PR snapd#8378 closed: o/configcore: introduce core config handlers (3/N) <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/8378>
[10:38] <mup> PR snapd#8402 opened: o/configstate: add backlight option for core config <Created by stolowski> <https://github.com/snapcore/snapd/pull/8402>
[10:40] <mup> PR snapd#8160 closed: overlord/configstate: add backlight option <Created by EthanHsieh> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/8160>
[10:47] <mup> PR snapd#8220 closed: interfaces/seccomp: allow passing an address to setgroups <Created by alfonsosanchezbeato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8220>
[10:57] <mborzecki> mvo: soo it works :P
[10:57] <mborzecki> mvo: need to patch subiquity though
[11:00] <ijohnson> morning folks
[11:00] <ijohnson> hey mborzecki seems I missed the deadline on your subiquity PR, but I still would like to do a review
[11:01] <zyga> ijohnson: I'm doing a review as well
[11:01] <mborzecki> ijohnson: hey, no problem
[11:01] <zyga> I think that's good
[11:01] <zyga> and yes, you should too
[11:01] <mborzecki> btw. so with patched subiquity, it works, i can trigger the chooser, the UI runs and all, reboot is requested
[11:01] <mborzecki> but then i get this: https://i.imgur.com/IYb0mgs.png
[11:02] <mborzecki> it'd noble that it's trying fd0 but i doubt it's gonna succeed anytime soon
[11:02] <zyga> fd0?
[11:02] <zyga> whaat :)
[11:02] <zyga> insert assertion in disk A and pres return to continue ;D ?
[11:02] <zyga> *press
[11:02] <mborzecki> haha right? :)
[11:03] <ijohnson> haha wow
[11:04] <zyga> mborzecki: https://github.com/CanonicalLtd/subiquity/pull/655#pullrequestreview-385648364
[11:04] <mup> PR CanonicalLtd/subiquity#655: console_conf: implement UC20 recovery chooser <Created by bboozzoo> <Merged by xnox> <https://github.com/CanonicalLtd/subiquity/pull/655>
[11:05] <pstolowski> pedronis: is this https://forum.snapcraft.io/t/how-to-install-and-enable-syslog/15012 the general idea wrt persistent storage config flag?
[11:07] <zyga> post merge review for https://github.com/snapcore/snapd/pull/8370#pullrequestreview-385530483
[11:07] <mup> PR #8370: snap-bootstrap: fix disk layout sanity check <UC20> <Created by cmatsuoka> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8370>
[11:07] <pedronis> pstolowski: yes,  something like that, not sure the directory needs to be created or not
[11:07] <pedronis> it needs to be created for auto
[11:07] <pedronis> but not sure about persistent
[11:10] <zyga> ijohnson: can you review https://github.com/snapcore/snapd/pull/8389 as well, it would help me make progress
[11:10] <mup> PR #8389: tests: make session tool way more robust <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8389>
[11:10] <ijohnson> sure I was meaning to look at it
[11:11] <zyga> it's not the end of that road
[11:11] <zyga> but it's a step along the way
[11:11] <pstolowski> pedronis: ok. so we want to support actual systemd storage type as a config option (auto|persistent), and not simplify it/hide it behind e.g. persistent=true|false?
[11:16] <pedronis> pstolowski: just persistent=true|false, I don't think auto makes a lot of sense for use on core
[11:17] <pedronis> pstolowski: it might be that we implement false as auto
[11:17] <pedronis> but not sure , we can discuss in the PR
[11:17] <mborzecki> so should the auto-import udev rule have KERNEL!="fd*" as well?
[11:18] <pstolowski> pedronis: ok, thanks
[11:18] <zyga> pstolowski: https://github.com/snapcore/snapd/pull/8402#pullrequestreview-386327449
[11:18] <mup> PR #8402: o/configstate: add backlight option for core config <Created by stolowski> <https://github.com/snapcore/snapd/pull/8402>
[11:19] <pstolowski> zyga: thanks
[11:24] <ijohnson> zyga: approved with a question about busctl
[11:24] <zyga> ijohnson: looking
[11:25] <zyga> FYI, the contributor needs to go through the CLA process
[11:25] <zyga> I didn't find any documentation on that in our HACKING
[11:25] <zyga> should we have some?
[11:25] <zyga> https://github.com/snapcore/snapd/pull/8398
[11:25] <mup> PR #8398: usersession/userd: add "slack" to the white list of URL schemes handled by xdg-open <Created by troyready> <https://github.com/snapcore/snapd/pull/8398>
[11:27] <zyga> ijohnson: replied
[11:28] <ijohnson> thanks
[11:32] <zyga> ijohnson: I added one more comment at the bottom of session-tool spread test
[11:32] <zyga> ijohnson: I wanted to take a break from this monstrosity and look at other things for a while (reviews) but I need to address this in a follow-up
[11:32] <zyga> ijohnson: if you are happy with my comments I will merge this now
[11:32] <ijohnson> sure no problem
[11:32] <ijohnson> go for it
[11:33] <zyga> thank you!
[11:33] <zyga> I'll include -- and the comments you mentioned in a follow up
[11:33] <zyga> and now reality will verify my oh-so-alredy-regretful commit message
[11:33] <zyga> "make session tool way more robust"
[11:33] <mup> PR snapd#8389 closed: tests: make session tool way more robust <Test Robustness> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8389>
[11:34] <ijohnson> next PR: "make session tool way way way more robust for realz this time (I promise)"
[11:34] <zyga> ijohnson: I will probably use "tests/session-tool: you cannot wait to see how this will break next"
[11:35] <ijohnson> zyga: "tests/session-tool: this one weird trick makes old linux distros so mad"
[11:37] <zyga> ijohnson: "tests/session-tool: mark as manual and move to tests/masochism"
[11:41] <zyga> I'll go make coffee and return to prepare the big PR with things that have recently landed
[11:49] <mvo> mborzecki: nice, is the patch complicated?
[11:50] <mborzecki> mvo: no, it's quite trivial, got merged already
[12:14] <zyga> ogra: https://github.com/snapcore/snapd/pull/8329#pullrequestreview-385876868
[12:14] <mup> PR #8329: interfaces: allow raw access to USB printers <Created by ogra1> <https://github.com/snapcore/snapd/pull/8329>
[12:26] <zyga> mborzecki: is your comment on https://github.com/snapcore/snapd/pull/8377 a +1?
[12:26] <mup> PR #8377: sandbox/cgroup: add ProcessPathInTrackingCgroup <Created by zyga> <https://github.com/snapcore/snapd/pull/8377>
[12:29] <mup> PR snapd#8399 closed: interfaces/policy: fix comment in recent new test <Skip spread> <Created by pedronis> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/8399>
[12:39] <sdhd-sascha> hi, all :-)
[12:42] <mup> PR snapd#8377 closed: sandbox/cgroup: add ProcessPathInTrackingCgroup <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8377>
[12:43] <sdhd-sascha> I didn't write C++ yet. I read a lot about it, around 12 years ago. Now, i help a very very litle bit with the backend of #WirVsVirus. What DB layer is scalable, and NoSQL. What DB should is a choice for this task ?
[12:43] <sdhd-sascha> Can anybody give a hint ?
[12:44] <zyga> sdhd-sascha: don't use json with manual serialization
[12:45] <sdhd-sascha> zyga: ?
[12:45] <zyga> sdhd-sascha: we don't have experience with nosql beyond that I think
[12:46] <sdhd-sascha> i found mongo, with this: https://github.com/datastax/cpp-driver
[12:46] <sdhd-sascha> But, why use in 2020 a language where i have to free memory manually ?.... Oooh
[12:46] <sdhd-sascha> Don't know why C++ was the choice... ;-)
[12:48] <ogra> zyga, commented
[12:48] <mup> PR snapd#8401 closed: snap: improve TestWaitRecovers test <Simple 😃> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8401>
[12:49]  * ogra curses ... why did nvidia *completely* change with the nvidia-4XX series .. 
[12:49] <zyga> ogra: to punish us for not pushing ahead with nvidia-runtime snap
[12:49] <ogra> yeah apparently ...
[12:50] <zyga> mvo: master is over 50 minutes in travis now
[12:50] <zyga> mvo: we're going to see failures soon
[12:51] <ogra> sadly i have a snap where i need to force-preload the nvidia libs (it completely unsets LD_LIBRARY_PATH and is proprietary) ... works just fine with nvidia-3XX and before ... but with 440 ... i get
[12:51] <ogra> ERROR: ld.so: object '/var/lib/snapd/lib/gl/libOpenGL.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
[12:51]  * sdhd-sascha ogra: ooh, nvidia ;-)
[12:52] <ogra> the lib is there with find and ls and the ldd dependencies dont seem to have changed vs 3XX either ... i dont get why i cant preload it
[12:52] <zyga> ogra: it's classic?
[12:52] <ogra> nope
[12:52] <ogra> strict
[12:52] <zyga> hmm
[12:52] <ogra> zoom-client
[12:52] <zyga> can we ldconfig to look at /var/lib/snapd/gl
[12:53] <zyga> and drop that silly SNAP_LIBRARY_PATH thing?
[12:53] <ogra> well, all these vars get unset by the zoom binary ... i cant make any use of them anyway
[12:53] <zyga> ogra: hence >  can we ldconfig to look at /var/lib/snapd/gl
[12:53] <zyga> ogra: so no variables at all but it _still looks_
[12:54] <zyga> ogra: we could even be more ... say, creative
[12:54] <zyga> and have a new mount point
[12:54] <zyga>  /app
[12:54] <ogra> hmm
[12:54] <zyga> and mount current revision of the app there
[12:54] <zyga> and have a fixed ldconfig
[12:54] <ogra> does ldconfig actually load anything ? i thought it only resolves deps ...  i need them in ram before the app starts
[12:55] <zyga> so we take libs from [/var/lib/snapd/lib/*, /app/lib/$triplet, regular locations]
[12:55] <ogra> or do you mean combine ldconfig and LD_PRELOAD ?
[12:55] <zyga> ogra: it finds them
[12:55] <zyga> e.g. copy a .so to /usr/local/lib
[12:55] <zyga> and run a binary that needs it
[12:55] <zyga> then ldconfig
[12:55] <zyga> then run it again
[12:55] <zyga> we could essentially tell the linker, here they are
[12:55] <ogra> yes, but zoom doesnt if it isnt loaded already
[12:55] <zyga> ?
[12:55] <zyga> this is entirely unrelated to zoom
[12:56] <ogra> well, i can try ...
[12:56] <zyga> it would be some work
[12:56] <zyga> but as a spike to see if it works
[12:56] <zyga> I think it could
[12:56] <zyga> repackage core snap to have different ldconfig
[12:59] <abeato> ijohnson, fyi I have UC20 running yesterday, the crash happened because I was greedy and was no giving enough memory to kvm - I've found I need at least 2GB.
[12:59] <ijohnson> ah yes that's another thing I forgot to tell you :-)
[13:00] <ogra> 2GB ... nobody thinks of the netbooks !
[13:00] <abeato> ijohnson, and the problem with the eth was related to the image, the one I built locally had some problem, maybe I need edge ubuntu-image
[13:00] <abeato> the cdimage one worked in the end
[13:19] <ijohnson> abeato: ah that's great news
[13:21] <abeato> ijohnson, yeah, really happy to start testing uc20 :) - thanks for your help
[13:27] <zyga> https://github.com/google/thread-weaver
[13:29] <ogra> moving snapd to java now ?
[13:29] <zyga> core 22j
[13:29] <ogra> hah
[13:31] <zyga> we can finally support arm7tdmi
[13:32] <ogra> yay, finally going for ARMv4 :)
[13:33]  * ogra digs in the box for old nokia phones to run core on
[13:43] <mup> PR snapd#8219 closed: interfaces: use udev backend if udev socket exists <Security-High> <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/8219>
[13:46] <mup> PR snapd#7963 closed: xdgopenproxy: forward requests to the desktop portal <⛔ Blocked> <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/7963>
[13:52] <zyga> mborzecki: follow up https://github.com/snapcore/snapd/pull/8403
[13:52] <mup> PR #8403: sandbox/cgroup: avoid making arrays we don't use <Simple 😃> <Skip spread> <Created by zyga> <https://github.com/snapcore/snapd/pull/8403>
[13:52] <mup> PR snapd#8403 opened: sandbox/cgroup: avoid making arrays we don't use <Simple 😃> <Skip spread> <Created by zyga> <https://github.com/snapcore/snapd/pull/8403>
[14:18] <ijohnson> hey I fixed the usb audio thing, yet again my usb hub seems to be causing problems :-/ plugging the device directly into my desktop makes it work fine
[14:18]  * ijohnson is happy with high fidelity audio output now
[14:35] <mup> PR snapd#8404 opened: client: increase timeout in client tests to 100ms <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8404>
[14:54] <mvo> zyga: you will love this test failure https://paste.ubuntu.com/p/yKy6G2x59x/ (in a PPA)
[15:01] <mup> PR snapd#8089 closed: features: enable robust mount ns updates <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8089>
[15:06] <zyga> mvo: looking
[15:06] <zyga> Mvo: heh
[15:06] <mvo> zyga: I will look later
[15:06] <mvo> zyga: in a meeting right now
[15:07]  * zyga hugs mvo
[15:07] <mvo> zyga: but yeah, lots of fun, the PPA is super slow right now
[15:07] <zyga> I’m sorry, I know we are in tough times
[15:19] <zyga> back now
[15:19]  * zyga took a walk
[15:20] <zyga> I guess people are renting dogs to walk now
[15:20] <zyga> I see more people with dogs than I ever did here
[15:28]  * zyga stares at tig output
[15:28] <zyga> time to rebase that WIP thing away
[15:43] <zyga> hmmm
[15:43] <zyga> import cycle
[15:43] <zyga> perfect :)
[15:43] <zyga> https://www.irccloud.com/pastebin/cBg4Sia0/
[15:43] <zyga> client -> asserts -> release ->sandbox/cgroup -> naming -> snapdenv -> release
[15:43] <zyga> "fun"
[15:43] <zyga> I really dislike this part of go
[15:43] <zyga> as theoretically nice it may be
[15:44]  * zyga wonders what to do
[15:44] <zyga> can snapdenv not import release?
[15:45] <zyga> pedronis: how would you feel if I changed snapdenv.SetUserAgentFromVersion to take arguments instead of consuming release as a package?
[15:48] <pedronis> zyga: to be honest I feel the proble here is more release -> sandbox/cgroup
[15:48] <zyga> ok, let me check why that is there
[15:48] <zyga> ForceDevMode wants to know
[15:49] <zyga> pedronis: how about we move ForceDevMode to sandbox/ ?
[15:49] <pedronis> yes, that would have been my proposal too
[15:49] <pedronis> we can keep some tables or something in release if that make sense
[15:49] <pedronis> or move them
[15:49] <zyga> pedronis: the, funny enough, release will be about os-release again :D
[15:49] <zyga> ok, let me try :)
[15:56] <mup> PR snapd#8405 opened: boot: simplify modeenv mocking to always write a modeenv <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8405>
[15:58] <zyga> fun
[15:58] <zyga> so many cycles
[15:59] <mvo> mborzecki: I merged 8305, there are some tweaks worth doing in a followup. looks great overall, thanks for pushing this!
[15:59] <mup> PR snapd#8305 closed: cmd/snap-recovery-chooser: add recovery chooser <UC20> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8305>
[15:59] <mborzecki> mvo: thanks for merging!
[16:00] <zyga> hmm
[16:00] <zyga> I think that user agent thing is really a problem
[16:00] <zyga> but let me try something
[16:05] <pedronis> zyga: doesn't moving ForceDevMode help?
[16:05] <zyga> no, it's all interconnected :/
[16:05] <pedronis> what is interconnected?
[16:05] <pedronis> master is fine, what's the change?
[16:05] <ijohnson> zyga: github actions debian dependencies failed again: https://pastebin.ubuntu.com/p/jkYFxdGjdK/
[16:06] <zyga> https://www.irccloud.com/pastebin/WVqzY0tr/
[16:06] <zyga> pedronis: I'm trying a few ideas
[16:06] <ijohnson> zyga: seems like an issue  with the repos?
[16:06] <zyga> ijohnson: I asked IS about this
[16:06] <zyga> ijohnson: and it's a spike on a pipe that connects us to them
[16:07] <pedronis> zyga: sorry, why is snapdenv importing sandbox
[16:07] <zyga> ijohnson: it's really flaky in the sense that we run out of bandwidth
[16:07] <ijohnson> zyga: :-/ oh well
[16:07] <zyga> pedronis: snapdenv needs to know if it's in devmode for the user agent
[16:07] <zyga> pedronis: ForcedDevMode thing
[16:07] <zyga> ijohnson: it shows up as an alert on our side (we know it's saturated) and we drop connections
[16:08] <pedronis> zyga: I see
[16:08] <ijohnson> cool
[16:08] <pedronis> I forgot that
[16:08] <pedronis> let me think a bit
[16:08] <zyga> pedronis: yeah, I'm looking at this for possibilities
[16:12] <zyga> pedronis: I'll share what I have soon
[16:20] <mup> PR snapd#8406 opened: usersession: extend timerange in TestExitOnIdle <Created by mvo5> <https://github.com/snapcore/snapd/pull/8406>
[16:20] <mvo> woah, down to 52(!) prs
[16:20] <mvo> yay
[16:31] <zyga> mvo: yeah, that close button sure works well ;D
[16:31] <zyga> (I'm in a silly humor mode)
[16:31] <pedronis> zyga: progress? I think I have some ideas of what I would do
[16:31] <mvo> zyga: uh  - I did not check how we got to this number :(
[16:31] <zyga> yeah
[16:32] <zyga> pedronis: yeah, let me open a PR
[16:32] <zyga> mvo: by merging, i was just silly
[16:32] <mup> PR snapd#8402 closed: o/configstate: add backlight option for core config <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8402>
[16:35] <mup> PR snapd#8253 closed: snap-bootstrap: expand data partition on install <UC20> <Created by cmatsuoka> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8253>
[16:36] <zyga> pedronis: ok
[16:36] <zyga> pedronis: https://github.com/snapcore/snapd/pull/8407
[16:36] <mup> PR #8407: many: move IsForcedDevMode to sandbox/misc, move UA to snapdenv/usera… <Created by zyga> <https://github.com/snapcore/snapd/pull/8407>
[16:36] <mup> PR snapd#8407 opened: many: move IsForcedDevMode to sandbox/misc, move UA to snapdenv/usera… <Created by zyga> <https://github.com/snapcore/snapd/pull/8407>
[16:37] <zyga> pedronis: I verified this really works for me to be able to import cgroup bits in cmd/run
[16:37] <zyga> there are two patches here, squashed because I reset the tree for simplicity
[16:37] <zyga> I think sandbox/misc can just become sandbox now though
[16:38] <zyga> pedronis: I'll clean my other desk and wait for a ping, I'm just a meter away :)
[16:39] <pedronis> zyga: thank you, I'll probably play a bit with it and push some slightly different org, at least you are unblocked
[16:40] <zyga> pedronis: I need one to land so if you feel strongly about it please propose yours and lets land it
[16:40] <zyga> otherwise I cannot merge master and make progress
[16:40] <pedronis> ?
[16:41] <zyga> pedronis: I need _any_ variant of this to land to progress
[16:42] <zyga> (or I must bundle an unapproved variant in my branch temporarily)
[16:42] <pedronis> that sounds fine
[16:42] <pedronis> I'm not entirely sure what's the problem with that
[16:43] <pedronis> anyway my goal here is to reduce the diff, reduce a bit need to add more packages
[16:43] <zyga> my original problem was inability to import sandbox/cgroup from cmd/snap IIRC
[16:43] <zyga> or maybe I'm forgetting bits, I'm tired a bit today (ENOSLEEP)
[16:44] <pedronis> it's ok, I'm just not sure why you think proposing a PR with the intermediate solution is a problem
[16:44] <pedronis> except for size
[16:45] <zyga> no, it's not
[16:45] <zyga> it's ok :)
[16:46] <kenvandine> jdstrand: i think i need a minor tweak to the appstream-metadata interface
[16:46] <kenvandine> https://github.com/snapcore/snapd/blob/master/interfaces/builtin/appstream_metadata.go#L49
[16:47] <kenvandine> that allow read access to the contents of those dirs
[16:47] <kenvandine> but not read on the dir itself
[16:47] <kenvandine> jdstrand: right?
[16:52] <kenvandine> jdstrand: snap-store is checking for read access on the dirs as it iterates them and fails
[16:58] <jdstrand> kenvandine: are you saying that snap-store is trying to figure out what files are in /usr/share/metainfo and is failing to enumerate them?
[16:58] <kenvandine> i think it's just checking if it can read the dir
[16:58] <jdstrand> kenvandine: oh, were you asking what those rules do? yes, not a read on the dir, just everything under the dir
[16:59] <jdstrand> kenvandine: sounds like you need:
[16:59] <kenvandine> if i manually tweak the profile, how do i recompile it to test my theory?
[16:59] <jdstrand> /usr/share/metainfo/{,**} r,
[16:59] <kenvandine> yeah, i've tweaked the profile locally to test
[16:59] <jdstrand> kenvandine: sudo apparmor_parser -r /var/lib/snapd/apparmor/profiles/snap.snap-store.snap-store (or whatever file you modified)
[17:00] <kenvandine> bingo
[17:01] <zyga> jdstrand: o/
[17:01] <zyga> jdstrand: do you have 3minutes?
[17:01] <jdstrand> kenvandine: I'm planning on doing a PR for some things in the next few days. if I slid this in there, would that be good enough for you?
[17:01] <zyga> jdstrand: https://github.com/snapcore/snapd/pull/8408 is verbatim from the refresh-app-awareness-v2 branch
[17:01] <mup> PR #8408: snap/naming: add validator for snap security tag <Created by zyga> <https://github.com/snapcore/snapd/pull/8408>
[17:01] <zyga> cherry picked to reduce the diff
[17:02] <kenvandine>  /usr/share/metainfo/{,**} r,
[17:02] <kenvandine>  /usr/share/appdata/{,**} r,
[17:02] <kenvandine> jdstrand: that's what i need
[17:02] <mup> PR snapd#8408 opened: snap/naming: add validator for snap security tag <Created by zyga> <https://github.com/snapcore/snapd/pull/8408>
[17:02] <kenvandine> jdstrand: that's fine
[17:02] <kenvandine> we need it in 20.04 ;)
[17:02] <jdstrand> kenvandine: ack, trello'd
[17:03] <jdstrand> "to trello: the act of adding or modifying a trello card"
[17:03] <kenvandine> lol
[17:03] <kenvandine> jdstrand: thanks!
[17:03] <jdstrand> kenvandine: np
[17:04] <zyga> hmm, we may need to discuss deps with IS
[17:04] <zyga> and I need to check why we don't cache them
[17:04] <zyga> (deps, repos)
[17:04] <jdstrand> zyga: what do you need wrt PR 8408?
[17:04] <mup> PR #8408: snap/naming: add validator for snap security tag <Created by zyga> <https://github.com/snapcore/snapd/pull/8408>
[17:04] <jdstrand> zyga: a review now?
[17:04] <jdstrand> zyga: or something else?
[17:04] <zyga> jdstrand: please either look if you want to review it and if so enqueue it, or just defer the review to others
[17:06] <jdstrand> zyga: enqueued, thanks!
[17:06] <zyga> thanks!
[17:06]  * zyga EODs and returns to cleaning stuff from the other desk
[17:14] <jdstrand> bye zyga :)
[17:14] <zyga> bye, have a good evening :)
[17:19] <mup> PR snapd#8409 opened: snap-bootstrap: seal and unseal encryption key using tpm <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8409>
[17:25] <mup> PR snapd#8095 closed: snap-bootstrap: add tpm support (draft) <UC20> <Created by cmatsuoka> <Closed by cmatsuoka> <https://github.com/snapcore/snapd/pull/8095>
[17:51] <mup> PR snapcraft#2999 closed: requirements: uprev python-apt to 1.6.0 (bionic package) <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2999>
[17:53] <mup> PR snapd#8405 closed: boot: simplify modeenv mocking to always write a modeenv <Simple 😃> <UC20> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/8405>
[18:44] <mup> PR snapd#8410 opened: many: disentagle release and snapdenv from sandbox/* <Created by pedronis> <https://github.com/snapcore/snapd/pull/8410>
[18:47] <mup> PR snapd#8407 closed: many: move IsForcedDevMode to sandbox/misc, move UA to snapdenv/usera… <Created by zyga> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/8407>
[19:39] <mup> PR snapcraft#3007 closed: go: support projects with multiple binaries when using go.mod <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3007>
[20:34] <mup> PR # closed: core#38, core#107, core#108, core#110, core#111
[20:41] <mup> PR snapd#8411 opened: boot: cleanup more things, simplify code <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8411>
[21:01] <mup> PR # closed: core-build#11, core-build#22, core-build#26, core-build#37, core-build#51, core-build#58
[21:07] <mup> PR snapcraft#3000 closed: repo: use python-apt's fetch_binary implementation <Created by cjp256> <Closed by cjp256> <https://github.com/snapcore/snapcraft/pull/3000>
[21:13] <mup> PR snapcraft#3009 opened: repo: use python-apt's fetch_binary implementation <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3009>
[21:35] <mup> PR snapcraft#3008 closed: cli: use the channel-map api for status <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3008>
[21:40] <zyga> snapd is blocked in debian
[21:40] <zyga> dh_missing: warning: usr/bin/snap-preseed exists in debian/tmp but is not installed to anywhere
[22:53] <mup> PR snapcraft#3009 closed: repo: use python-apt's fetch_binary implementation <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3009>
[22:56] <mup> PR snapcraft#3010 opened: cli: add progressive releases support to the release command <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3010>