| Chipaca | mwhudson: o/ | 00:51 |
|---|---|---|
| mwhudson | Chipaca: hello | 00:51 |
| 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:52 |
| 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:53 |
| Chipaca | in any case it's not something critical :-) | 00:54 |
| mwhudson | ha this chroot doesn't have man | 00:54 |
| Chipaca | I'd make fun of your chroot, but UC doesn't have man either | 00:55 |
| 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:56 |
| 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 | 00:57 |
| 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:00 |
| mwhudson | bah not in latest release | 01:01 |
| 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:02 |
| Chipaca | yeah what he said | 01:03 |
| * Chipaca ⇝ zzzz | 01:03 | |
| mup | PR snapd#6398 opened: tests: update systems for google sru backend <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6398> | 02:02 |
| mborzecki | morning | 06:20 |
| mvo | hey mborzecki | 06:20 |
| mborzecki | wow amdgpu-pro driver is a mess, no wonder nobody uses it | 06:23 |
| mvo | mborzecki: meh! on fridays I only want good news ;) | 06:24 |
| mborzecki | mvo: good news: today is friday :P | 06:25 |
| mvo | mborzecki: heh :) | 06:29 |
| 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 :/ | 06: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:55 |
| zyga | hey ho | 07:59 |
| zyga | mvo: I'm back to work from yesterday | 08:00 |
| zyga | my two multipass machines lost their disks, I fortunately pushed everything out | 08:01 |
| mborzecki | zyga: mvo: left some comments in the debian packaging pr | 08:06 |
| zyga | mborzecki: thank you | 08:06 |
| 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:07 |
| mvo | mborzecki: thanks! | 08:08 |
| 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:10 |
| 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:14 |
| 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:15 |
| mup | PR snapcraft#2443 closed: schema: allow before and after <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2443> | 08:16 |
| 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:23 |
| 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:25 |
| 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:26 |
| zyga | and if we do a .1 I would have an easier life for suse release too | 08:27 |
| === pstolowski|afk is now known as pstolowski | ||
| pstolowski | mornings | 08:28 |
| mborzecki | pstolowski: hey | 08:28 |
| zyga | hey pawel | 08:28 |
| zyga | oh, btw, I will be off on monday | 08:28 |
| mborzecki | zyga: i can later look into bcond_with thing, hopefully my opensuse installation is still working | 08:30 |
| zyga | mborzecki: thank you | 08:31 |
| zyga | actually | 08:31 |
| 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:33 |
| 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:34 |
| 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:35 |
| zyga | and not stay in the current split model where we pay the price for each distro/release | 08:36 |
| mup | PR snapcraft#2444 opened: snap: move to core18 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2444> | 08:37 |
| zyga | mvo: any luck with sbuild and tests? | 08:52 |
| zyga | hey Chipaca :) | 09:01 |
| 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:02 |
| 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:03 |
| zyga | thank you sir :) | 09:07 |
| 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:09 |
| 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:10 |
| 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:11 |
| mborzecki | one interesting spread failure: https://paste.ubuntu.com/p/Tz4JCptgPb/ pstolowski something you may want to look at | 09:26 |
| mborzecki | pstolowski: on a second look, doesn't seem to be related to hotplug | 09:27 |
| pstolowski | mborzecki: hmm something went terribly wrong with reboot? | 09:28 |
| pstolowski | localhost.localdomain systemd[1]: etc-shadow.mount: Failed to check directory /etc/shadow: Not a directory | 09:29 |
| pstolowski | etc | 09:29 |
| mborzecki | pstolowski: ah, watchdog timeout | 09:30 |
| mvo | zyga: re sbuild> running but slow, hopefully finishing soon | 09:31 |
| mborzecki | pstolowski: actually itnersting, snapd was waiting for system reboot, and got killed by systemd because of watchdog timeout | 09:32 |
| mvo | zyga: hrm, I'm confused, it fails in sbuild early but works in pbuilder, need to look closer | 09:37 |
| 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:51 |
| mvo | zyga: TestSetupSnapConfineGeneratedPolicyError bits | 09:52 |
| 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:54 |
| zyga | mvo: both ubuntu and debian-10 | 09:55 |
| mvo | zyga: interessting, my debian-sid sbuild is still building | 09:56 |
| zyga | I had to spend a moment to add caching to my setup as otherwise iteration was mostly network bound | 09:58 |
| zyga | now it should be quite a lot faster | 09:59 |
| * Chipaca ⇝ break | 10:02 | |
| zyga | mvo: FYI: https://salsa.debian.org/debian/snapd/merge_requests/1 | 10:07 |
| zyga | mvo: that's just last evening state | 10:07 |
| mvo | zyga: nice | 10:08 |
| mvo | zyga: yeah, same errors in debian too | 10:18 |
| 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:20 |
| zyga | unless you see a point I think we should not both spend time on it today | 10:21 |
| mvo | zyga: its fine, I try to minimize the time on this | 10:22 |
| mvo | zyga: working mostly on remodel now | 10:22 |
| 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:35 |
| 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:36 |
| mvo | mborzecki: nice catch (6317) | 10:42 |
| 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:49 |
| mvo | Chipaca: in the state | 10:53 |
| Chipaca | mvo: what would you GET? | 10:54 |
| 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:56 |
| mvo | Chipaca: the important use case for this endpoint is to post a new model :) | 10:57 |
| Chipaca | mvo: I think /v2/model would be fine | 11:13 |
| Chipaca | mvo: how is it used? | 11:13 |
| 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:14 |
| mvo | Chipaca: we have not talked much about the cli yet | 11:15 |
| 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:29 |
| 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:33 |
| zyga | mvo: I like that | 11:42 |
| mborzecki | amdgpu-pro is so much fun, i'm typing something in the terminal, suddengly gnome-session fails and the whole Xorg goes down :/ | 11:56 |
| mborzecki | and it's mind boggling qt wants to compile shaders to display a piece of text from qml | 12:04 |
| 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:20 |
| pstolowski | mborzecki: i don't think it's unique to amdgpu ;) | 12:44 |
| mborzecki | pstolowski: frankly the windows drivers aren't much better, i've deliverately skipped the last 3 updates :P | 12:45 |
| pstolowski | :) | 12:50 |
| 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:41 |
| mborzecki | greyback: interesting, sounds like i should search qt blog archives for the details :P | 13:47 |
| cachio | mvo, my internet connection is breaking up | 14:05 |
| cachio | can't connect ot the hangouts | 14:05 |
| 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:58 |
| 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> | 14:59 |
| 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:00 |
| mvo | mborzecki: thanks, merged and I cherry picked it into 2.37 in case we do anohter release of that | 15:01 |
| 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:16 |
| Chipaca | booo, multiple progress bars doesn't work :-( | 15:26 |
| * Chipaca takes a breaks | 15:26 | |
| === phoenix_firebrd is now known as murthy | ||
| === murthy is now known as phoenix_firebrd_ | ||
| === phoenix_firebrd_ is now known as murthy | ||
| * cachio lunch | 16:37 | |
| 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:43 |
| 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! | 16:44 |
| Chipaca | I'm off to the gym. Will bbl, but if I don't see y'all, have a good weekend! | 17:00 |
| 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:01 |
| pstolowski | mvo: ty! | 17:02 |
| === pstolowski is now known as pstolowski|afk | ||
| === murthy is now known as phoenix_firebrd_ | ||
| === phoenix_firebrd_ is now known as murthy | ||
| === murthy is now known as phoenix_firebrd_ | ||
| === phoenix_firebrd_ is now known as murthy | ||
| === murthy is now known as phoenix_firebrd_ | ||
| === phoenix_firebrd_ is now known as murthy | ||
| === murthy is now known as phoenix_firebrd_ | ||
| === phoenix_firebrd_ is now known as murthy | ||
| === Luke is now known as bitnomial | ||
| === bitnomial is now known as Luke | ||
| 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? | 23:00 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!