[00:51] <Chipaca> mwhudson: o/
[00:51] <mwhudson> Chipaca: hello
[00:52] <Chipaca> mwhudson: question about snapd on debian
[00:52] <Chipaca> mwhudson: in 'man snap', if you look for the entry for 'info', is it indented too much compared to the other commands?
[00:53] <mwhudson> Chipaca: let's see
[00:53] <Chipaca> if so, that's the main bug we revved go-flags for
[00:53] <Chipaca> there are others around it using too much space needlessly, but those are less ugly
[00:54] <Chipaca> in any case it's not something critical :-)
[00:54] <mwhudson> ha this chroot doesn't have man
[00:55] <Chipaca> I'd make fun of your chroot, but UC doesn't have man either
[00:56] <mwhudson> Chipaca: man page looks like this https://paste.ubuntu.com/p/Qq7KPtZrQq/
[00:56] <mwhudson> which is pretty bad (what are those Help Options about?) but doesn't sound like what you are saying
[00:56] <Chipaca> whoa
[00:56] <Chipaca> mwhudson: well, maybe
[00:56] <Chipaca> mwhudson: we "hid" the help options
[00:57] <Chipaca> mwhudson: which starts showing off the other bug, where it doesn't de-dent if a command only has hidden options
[00:57] <mwhudson> this is 2.30
[00:57] <mwhudson> ah ok
[00:57] <Chipaca> so then we fixed go-flags
[00:57] <mwhudson> is the fix upstream?
[00:57] <Chipaca> yes
[00:57] <mwhudson> i guess i can rev it in debian and see what falls out there
[01:00] <Chipaca> the fix is https://github.com/jessevdk/go-flags/commit/14b8957836290f46f473f39b104761889b2f0af0
[01:00] <Chipaca> but there are a bunch more fixes that are nice to have so if you can rev it, all the better
[01:01] <mwhudson> bah not in latest release
[01:02] <Chipaca> psh, "releases", who uses those
[01:02] <Chipaca> anyway, I should go do that "sleep" thing that seems popular around here
[01:02] <Chipaca> just out of FOMO
[01:02] <mwhudson> Chipaca: good night
[01:02] <Chipaca> \o
[01:02] <mwhudson> heh https://github.com/jessevdk/go-flags/issues/288
[01:03] <Chipaca> yeah what he said
[01:03]  * Chipaca ⇝ zzzz
[02:02] <mup> PR snapd#6398 opened: tests: update systems for google sru backend <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6398>
[06:20] <mborzecki> morning
[06:20] <mvo> hey mborzecki
[06:23] <mborzecki> wow amdgpu-pro driver is a mess, no wonder nobody uses it
[06:24] <mvo> mborzecki: meh! on fridays I only want good news ;)
[06:25] <mborzecki> mvo: good news: today is friday :P
[06:29] <mvo> mborzecki: heh :)
[06:55] <mborzecki> heh no way that driver is going to work on arch, it's super outdated and i'd have to downgrade xorg and the kernel :/
[07:55] <mup> PR snapd#6398 closed: tests: update systems for google sru backend <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6398>
[07:59] <zyga> hey ho
[08:00] <zyga> mvo: I'm back to work from yesterday
[08:01] <zyga> my two multipass machines lost their disks, I fortunately pushed everything out
[08:06] <mborzecki> zyga: mvo: left some comments in the debian packaging pr
[08:06] <zyga> mborzecki: thank you
[08:07] <zyga> mborzecki: I think my focus today is to do the downstream package release
[08:07] <zyga> it still fails to build correctly
[08:07] <zyga> unit tests fail in sbuild
[08:07] <zyga> but pass in the same machine running debian 10 normally
[08:07] <zyga> some a bit mysteriously
[08:07] <mborzecki> oh, sbuild is just chrot?
[08:07] <mborzecki> chroot
[08:08] <mvo> mborzecki: thanks!
[08:10] <mvo> zyga: I had the "smoke" suite passing and the main ones are running but of course that is not inside sbuild
[08:10] <mvo> zyga: also its still a snapshot, would love to try a 2.37.1 package build from a git tag too
[08:10] <zyga> mvo: yeah,
[08:10] <zyga> ideas on why it would fail in sbuild appreciated
[08:10] <zyga> I'm rebuilding my environment
[08:14] <mvo> zyga: let me try to run the import-debian-salsa-no-history .dsc in sbuild
[08:14] <mvo> (this will take a bit to setup the env)
[08:15] <zyga> import-debian-salsa-no-history?
[08:15] <zyga> that's a branch?
[08:15] <zyga> I haven't look at git much yet
[08:15] <zyga> ah
[08:15] <zyga> mvo: do let me know if you have any unit test failures
[08:16] <mup> PR snapcraft#2443 closed: schema: allow before and after <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2443>
[08:23] <mup> PR snapd#6399 opened: packaging: make sure that /var/lib/snapd/lib/glvnd is accounted for <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6399>
[08:25] <mborzecki> zyga: ^^ no rush, you can look at it when you get back
[08:25] <zyga> mborzecki, mvo: shall we merge the opensuse packaging branch
[08:25] <zyga> and then separately have someone look at bcond/with part of it
[08:26] <zyga> because I think we are paying the price for the divergence way too often
[08:26] <zyga> and that is managed there
[08:26] <mborzecki> zyga: sounds good to me
[08:26] <zyga> mborzecki: +1 on your branch but I'd rather work on reusing that for arch/debian than just keep whack-a-mole
[08:26] <zyga> mvo: if you agree then let's please land the opensuse PR
[08:27] <zyga> and if we do a .1 I would have an easier life for suse release too
[08:28] <pstolowski> mornings
[08:28] <mborzecki> pstolowski: hey
[08:28] <zyga> hey pawel
[08:28] <zyga> oh, btw, I will be off on monday
[08:30] <mborzecki> zyga: i can later look into bcond_with thing, hopefully my opensuse installation is still working
[08:31] <zyga> mborzecki: thank you
[08:31] <zyga> actually
[08:33] <mvo> zyga: I need to look at this again, I'm slightly wary that it might lead us to a tangent that makes the debian release slower, but I really need to look first
[08:34] <zyga> mvo: it's not affecting debian
[08:34] <zyga> mvo: it's an opt-in
[08:34] <zyga> only suse uses it now
[08:34] <zyga> any work to enable it for more distros is explicit
[08:34] <zyga> but
[08:34] <zyga> it's a reference list to look at
[08:35] <mvo> zyga: ok, I check it out in a wee bit, just need to do one more remodel thing
[08:35] <zyga> thank you!
[08:35] <zyga> it's not urgent but I'd love to improve upon that
[08:36] <zyga> and not stay in the current split model where we pay the price for each distro/release
[08:37] <mup> PR snapcraft#2444 opened: snap: move to core18 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2444>
[08:52] <zyga> mvo: any luck with sbuild and tests?
[09:01] <zyga> hey Chipaca :)
[09:02] <Chipaca> moin moin
[09:02] <zyga> Chipaca: https://github.com/snapcore/snapd/pull/6395#pullrequestreview-193983092 :)
[09:02] <mup> PR #6395: cmd/snap: use a fake user for 'run' tests <Simple 😃> <Created by chipaca> <https://github.com/snapcore/snapd/pull/6395>
[09:03] <Chipaca> :-)
[09:03] <mup> PR snapd#6395 closed: cmd/snap: use a fake user for 'run' tests <Simple 😃> <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6395>
[09:07] <zyga> thank you sir :)
[09:09] <zyga> btw, I know this is a bit big but I'd love have some quick pass on the design in https://github.com/snapcore/snapd/pull/6360
[09:09] <mup> PR #6360: cmd/snap-update-ns: refactor of profile application <Per-user mount ns  🐎> <Created by zyga> <https://github.com/snapcore/snapd/pull/6360>
[09:09] <zyga> I think it's sufficient if one reads one function there
[09:09] <zyga> the rest is plumbing into that idea
[09:10] <zyga> specifically this one https://github.com/snapcore/snapd/pull/6360/files#diff-d5f861571a1e368f7c7f933b29316750R37
[09:10] <zyga> the whole PR is about allowing that function to exist
[09:10] <zyga> and the fuction is a generalization of the existing algorithm (that part is unchanged)
[09:11] <zyga> but the details of system vs user profile are abstracted behind the interface
[09:11] <zyga> so reuse, easier testing and less code (the tests cover more code now so there's more +es than -es)
[09:26] <mborzecki> one interesting spread failure: https://paste.ubuntu.com/p/Tz4JCptgPb/ pstolowski something you may want to look at
[09:27] <mborzecki> pstolowski: on a second look, doesn't seem to be related to hotplug
[09:28] <pstolowski> mborzecki: hmm something went terribly wrong with reboot?
[09:29] <pstolowski> localhost.localdomain systemd[1]: etc-shadow.mount: Failed to check directory /etc/shadow: Not a directory
[09:29] <pstolowski> etc
[09:30] <mborzecki> pstolowski: ah, watchdog timeout
[09:31] <mvo> zyga: re sbuild> running but slow, hopefully finishing soon
[09:32] <mborzecki> pstolowski: actually itnersting, snapd was waiting for system reboot, and got killed by systemd because of watchdog timeout
[09:37] <mvo> zyga: hrm, I'm confused, it fails in sbuild early but works in pbuilder, need to look closer
[09:51] <mvo> zyga: I get errors in sbuild using an ubuntu kernel I think you had this too, trying on pure debian noe
[09:51] <mvo> now
[09:52] <mvo> zyga: TestSetupSnapConfineGeneratedPolicyError bits
[09:54] <zyga> mvo: iterating on sbuild
[09:54] <zyga> mvo: oh!
[09:54] <zyga> that's interesting
[09:54] <zyga> mvo: I'll pastebin my errors
[09:54] <zyga> we can compare
[09:54] <zyga> mvo: note, I got errors on both kernels
[09:55] <zyga> mvo: both ubuntu and debian-10
[09:56] <mvo> zyga: interessting, my debian-sid sbuild is still building
[09:58] <zyga> I had to spend a moment to add caching to my setup as otherwise iteration was mostly network bound
[09:59] <zyga> now it should be quite a lot faster
[10:02]  * Chipaca ⇝ break
[10:07] <zyga> mvo: FYI: https://salsa.debian.org/debian/snapd/merge_requests/1
[10:07] <zyga> mvo: that's just last evening state
[10:08] <mvo> zyga: nice
[10:18] <mvo> zyga: yeah, same errors in debian too
[10:20] <mvo> zyga: http://paste.ubuntu.com/p/5D3dggjbvM/ full log
[10:20] <zyga> that's the same that I see
[10:20] <zyga> ok
[10:20] <zyga> I think you can put this on hold now
[10:20] <zyga> let's attack it tomorrow in master
[10:20] <zyga> I'll work on packaging only
[10:21] <zyga> unless you see a point I think we should not both spend time on it today
[10:22] <mvo> zyga: its fine, I try to minimize the time on this
[10:22] <mvo> zyga: working mostly on remodel now
[10:35] <Chipaca> pedronis: if-and-when you have a chance, a short glance at #6400 would be good
[10:35] <mup> PR #6400: overlord/snapstate: use an ad-hoc error when no results <Created by chipaca> <https://github.com/snapcore/snapd/pull/6400>
[10:35] <mup> PR snapd#6400 opened: overlord/snapstate: use an ad-hoc error when no results <Created by chipaca> <https://github.com/snapcore/snapd/pull/6400>
[10:35] <Chipaca> pedronis: it's +30-7, so should be quick
[10:36] <Chipaca> this is to block the "yeah sure switch to non-existent channel"
[10:36] <Chipaca> but, it has more consequences, because of course it does :-)
[10:42] <mvo> mborzecki: nice catch (6317)
[10:49] <mvo> Chipaca: do you have a moment for me? I was wondering where the api of remodel should life: /v2/remodel? or /v2/model/ and a GET to get the current and a POST to post a new one?
[10:49] <Chipaca> mvo: where is the model kept?
[10:53] <mvo> Chipaca: in the state
[10:54] <Chipaca> mvo: what would you GET?
[10:56] <mvo> Chipaca: the currently get model
[10:56] <mvo> Chipaca: I implemented this for testing already as part of debug but was wondering if it should be more official
[10:56] <mvo> Chipaca: hence my wondering
[10:57] <mvo> Chipaca: the important use case for this endpoint is to post a new model :)
[11:13] <Chipaca> mvo: I think /v2/model would be fine
[11:13] <Chipaca> mvo: how is it used?
[11:14] <mvo> Chipaca: thanks, I think that makes sense
[11:14] <mvo> Chipaca: we will provide it via an api endpoint mostly
[11:14] <mvo> Chipaca: but I think there will also be a "snap remodel" command
[11:14] <Chipaca> mvo: ah, no 'snap model --re
[11:14] <Chipaca> '
[11:14] <Chipaca> oh
[11:14] <mvo> Chipaca: interessting, maybe `snap model {get,set}`?
[11:15] <mvo> Chipaca: we have not talked much about the cli yet
[11:29] <zyga> mvo: I like snap model get
[11:29] <zyga> though I very much like "snap remodel"
[11:29] <zyga> it spells out explicitly this is an operation that can change things more than "assign a value"
[11:33] <mvo> zyga: yeah
[11:33] <mvo> zyga: I did "snap debug get-model" for now, not sure if we need something on the cli, will talk with samuele
[11:42] <zyga> mvo: I like that
[11:56] <mborzecki> amdgpu-pro is so much fun, i'm typing something in the terminal, suddengly gnome-session fails and the whole Xorg goes down :/
[12:04] <mborzecki> and it's mind boggling qt wants to compile shaders to display a piece of text from qml
[12:20] <mup> PR snapd#6401 opened: many: add /v2/model API, `snap remodel` CLI and spread test <Created by mvo5> <https://github.com/snapcore/snapd/pull/6401>
[12:44] <pstolowski> mborzecki: i don't think it's unique to amdgpu ;)
[12:45] <mborzecki> pstolowski: frankly the windows drivers aren't much better, i've deliverately skipped the last 3 updates :P
[12:50] <pstolowski> :)
[13:41] <greyback> mborzecki: qml's text renderer uses distance field rendering technique, good for UIs where text can change size a lot (especially animated) to avoid lots of cpu-bound rasterisation. Yep it needs shaders, but why not? :)
[13:41]  * greyback puts https://github.com/snapcore/snapd/pull/6361 out there
[13:41] <mup> PR #6361: kvm: load required kernel modules if necessary <Created by gerboland> <https://github.com/snapcore/snapd/pull/6361>
[13:47] <mborzecki> greyback: interesting, sounds like i should search qt blog archives for the details :P
[14:05] <cachio> mvo, my internet connection is breaking up
[14:05] <cachio> can't connect ot the hangouts
[14:58] <mup> PR snapd#6361 closed: kvm: load required kernel modules if necessary <Created by gerboland> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6361>
[14:58] <mborzecki> mvo: https://github.com/snapcore/snapd/pull/6399 super simple review
[14:58] <mup> PR #6399: packaging: make sure that /var/lib/snapd/lib/glvnd is accounted for <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6399>
[14:59] <mup> PR snapd#6317 closed: overlord/snapstate/backend: call fontconfig helpers from the new 'current' <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6317>
[15:00] <mborzecki> mvo: thanks!
[15:00] <mup> PR snapd#6399 closed: packaging: make sure that /var/lib/snapd/lib/glvnd is accounted for <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6399>
[15:01] <mvo> mborzecki: thanks, merged and I cherry picked it into 2.37 in case we do anohter release of that
[15:16] <mup> PR snapd#6397 closed: tests: Use backend: [autopkgtest] for smoke test suite <Created by sergiocazzolato> <Closed by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6397>
[15:26] <Chipaca> booo, multiple progress bars doesn't work :-(
[15:26]  * Chipaca takes a breaks
[16:37]  * cachio lunch
[16:43] <pstolowski> mvo: hey, do you have a moment for https://github.com/snapcore/snapd/pull/6379 ?
[16:43] <mup> PR #6379: ifacestate/tests: extra test for hotplug-connect handler <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6379>
[16:44] <mvo> pstolowski: do you need a review?
[16:44] <pstolowski> mvo: yes, 2nd review
[16:44] <pstolowski> it's just a test
[16:44] <mvo> pstolowski: sure, looking
[16:44] <pstolowski> thanks!
[17:00] <Chipaca> I'm off to the gym. Will bbl, but if I don't see y'all, have a good weekend!
[17:01] <mup> PR snapd#6379 closed: ifacestate/tests: extra test for hotplug-connect handler <Hotplug 🔌> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6379>
[17:02] <pstolowski> mvo: ty!
[23:00] <Croepha> so, I have added wget in a stage-packages list, but that command isnt found when I try to run it from a shell script inside the snap... am I missing something?