mupPR snapcraft#2897 opened: meta: include environment in hook wrappers <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2897>00:08
mupPR snapcraft#2893 closed: [legacy] meta: include environment in hook wrappers <Created by kyrofa> <Closed by cjp256> <https://github.com/snapcore/snapcraft/pull/2893>00:14
mupPR snapcraft#2898 opened: meta: remove dead code from snap packaging <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2898>01:17
mvohey mborzecki ! good morning06:03
mborzeckimvo: hey!06:04
mupPR snapd#8049 closed: tests: fix revisions leaking from snapd-refresh test <Test Robustness> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8049>06:04
mupPR snapd#8032 closed: boot, cmd/snap, cmd/snap-bootstrap: move run mode and system label detection to boot <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8032>06:19
mupPR snapd#8040 closed: gadget: skip update when raw structure content is unchanged <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8040>06:23
mupPR snapd#7993 closed: devicestate: enable encryption based on model grade <UC20> <Created by cmatsuoka> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/7993>06:48
zygaHey guys07:26
zygaI’m going to check my wrist07:27
zygaI have a visit at 9:3007:27
zygaI’ll be back later07:27
mvozyga: good luck07:36
mborzeckizyga: hey07:38
mborzeckibtw. played a little with gnome-builder yesterday, it'd be nice to have some snap integration there, looks like it supports building flatpaks ootb07:49
jameshmborzecki: I did some work on that ages ago.  I set it aside, as I couldn't get things to work the way upstream wanted without changes to Snapcraft, and couldn't get buy-in to get those Snapcraft features done.07:57
mborzeckijamesh: interesting, thanks for the insight!08:00
jameshmborzecki: my patch essentially introduced "Snapcraft" as a new build system.  What they wanted was to keep the underlying build system visible (i.e. automake, meson, etc), and allow Builder to invoke it08:01
jameshThe idea is that build system plugins could support operations like "add newfile.c to library X", and they would update the appropriate makefile or equivalent08:03
jameshyou also want to be able to determine compile flags for each source file in order to properly parse them for completion08:05
jameshso if Snapcraft was build system, it would need to reimplement all of this for each of its supported build systems08:05
mborzeckijamesh: yeah, makes sense when you're working on your piece of software and snapcraft is really just a packaging method08:06
mborzeckijamesh: btw. have you worked with builtstream too?08:07
jameshmborzecki: IIRC, the Flatpak plugin adds an early build step to download dependencies, and provides the appropriate environment to run the underlying build system.  There's also an extra step at the end to package up the result08:09
jameshThere is the risk that this could produce different results to a standard build, but it means you have incremental builds and all the IDE features08:09
jameshI haven't looked at Buildstream08:09
mupPR snapd#8054 opened: snap: add `snap pack --compression=<comp>` options <Performance 🚀> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8054>08:23
Chipacamo'in, all09:05
mvohey Chipaca and pedronis ! good morning09:07
Chipacamvo: had an adequate amount of rest? :-)09:07
Chipacapedronis: I set up a short meeting to go over some things about manpages, either to clear the path for me to push a PR with it this week, or to spread the knowledge around of what needs doing later09:09
pedronisChipaca: yes, saw it09:09
Chipacamvo: pedronis: it just struck me that postUserCreateData's ForceManaged flag might be deprecated as well, I'm not clear what the use case was for it (ie should I leave it behind in the to-be-deprecated endpoint or carry it forwards as is currently done)09:10
pedronisChipaca: it's a bit of an orthogonal concern, I think we do want to carry it forward09:11
pedroniseven it's hopefully low usage09:11
mvoChipaca: people seems to use the flag to create additional users on devices which seems to be a valid use-case. the name is a bit confusing though :/09:14
mvoChipaca: thanks for setting up that meeting09:14
Chipacamvo: should I add you to it? so far it's me, zyga, pedronis09:14
Chipacamvo: working on the assumption that you don't love meetings _that_ much :-p09:15
mvoChipaca: I'm fine not being part of it09:16
zygaI’ll be back soon09:31
zygaHey pedronis, welcome back :-)09:31
mborzeckiChipaca: pedronis: hey09:31
zygaHey mborzecki :-)09:32
pedronismvo: zyga: shouldn't the disabling in 5512861442cbf containt TODO:UC20 ?09:33
zygaIf that is the patch I think it is there was a follow up that adds the todo core 20 marker09:35
zyga(I’m still afk for a short while)09:36
pedronismborzecki: in the doc comment for ModeAndRecoverySystemFromKernelCommandLine "returns the current run mode", I think we need to s/run/operating/09:38
mvopedronis: I had the same comment about the todo:uc20 and it was added in pr 805009:39
mupPR #8050: tests: add marker tag for core 20 test failure <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8050>09:39
pedronisah ok09:39
mborzeckipedronis: ack09:41
mupPR snapd#8055 opened: boot: tweak kernel cmdline helper docstring <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8055>09:45
mborzeckihm, it'd be cool if we could indicate that full spread run is unnecessary for PRs like that09:45
pedronismborzecki: maybe there's a way to read via the api if a given label is applied to the PR09:57
mborzeckipedronis: that or just scrape the page, like we did with the PR title, as the API had limitations and required api key09:59
mupPR snapd#8056 opened: client: move user-related things to their own files <Simple 😃> <Created by chipaca> <https://github.com/snapcore/snapd/pull/8056>10:03
Chipacamborzecki: pedronis: FWIW, https://gist.github.com/chipaca/911a8a4905b091c10caa9854bf7ea4b4 looks at labels on PRs10:06
Chipacathey're reasonably straightforward to get10:06
Chipaca(but yes without an api key we'd hit rate limits very fast)10:06
pedronismvo: did you see this: https://forum.snapcraft.io/t/auto-import-assert-fails/1518110:10
mvopedronis: have not seen it, will followup10:11
mupPR snapd#8057 opened: overlord/auth: add RemoveUserByName <Simple 😃> <Created by chipaca> <https://github.com/snapcore/snapd/pull/8057>11:27
zygaChipaca: reviewed11:32
popeyHm, anyone built an image with series:: "18"? ubuntu-image doesn't seem to like it11:32
zygapopey: series 18 is not a thing11:33
zygaseries is always 1611:33
zygapopey: and hi :)11:33
popeythats odd :)11:33
Chipacapopey: series is the restart-the-world generation switch11:33
Chipacapopey: if we change the series it's because we're starting over11:34
zygawe should have called it multiverse11:34
Chipacapopey: as in, nothing from series X works on series Y11:34
Chipacapopey: granted in retrospect we should've made it more internal-only, but we hadn't learned how to make things co-habitate back then11:34
Chipacawe thought we'd have to bump the series every lts11:34
Chipacaor something like that11:34
popeyIs it possible to seed something to stop me having to do the U1 SSO dance on first boot of a core device?12:05
mupPR snapd#8058 opened: run-checks, travis: allow skippng spread jobs by adding a label <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8058>12:07
mborzeckianyone remember when cachio is coming back?12:10
zyganext week12:10
mborzeckiah, ok12:10
zygafocal with zfs12:13
zygaoh mhy12:13
mupPR snapd#8053 closed: osutil: implement deluser <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/8053>12:14
mborzeckizyga: is the oracle police already banging at your door trying to extort a fee? :)12:16
mupPR snapd#8055 closed: boot: tweak kernel cmdline helper docstring <Simple 😃> <Created by bboozzoo> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/8055>12:16
zygapopey: you might know - nvidia experimental drivers for ubuntu?12:18
zygapopey: how do I get some12:18
mupPR snapd#8056 closed: client: move user-related things to their own files <Simple 😃> <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8056>12:19
popeyPossibly https://launchpad.net/~graphics-drivers/+archive/ubuntu/ppa  zyga12:20
zygapopey: thank you! :)12:26
Chipacahmm, shouldn't ubuntu-20.04 no longer be unstable?12:31
mupPR snapd#8057 closed: overlord/auth: add RemoveUserByName <Simple 😃> <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/8057>12:35
mupPR snapd#8059 opened: daemon: implement user removal <Created by chipaca> <https://github.com/snapcore/snapd/pull/8059>12:36
* Chipaca goes out for a while12:36
zyganeed to make tea for Iza13:03
mupPR snapd#8060 opened: gadget: skip update when mounted filesystem content is identical <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8060>13:08
mborzeckioff to pick up the kids from school13:11
=== ricab is now known as ricab|lunch
zygaijohnson: without the window reflection https://twitter.com/zygoon/status/1177917609783808001/photo/114:16
=== ricab|lunch is now known as ricab
ijohnsonzyga: very nice, it's so cleanly organized14:32
ijohnsonmy desktop has cables running everywhere and looks very messy generally14:32
Chipacazyga: https://i.redd.it/4c9udy915qyy.jpg14:34
mborzeckiijohnson: had to address a similar problem, otherwise my vega 56 would run really hot (still runs quite warm under load, but UV keeps the temps in check)14:35
Chipacazyga: (via https://www.reddit.com/r/cablefail/comments/6ccwaq/looked_tidy_until_the_side_fell_off_in_the_wind/ )14:35
zygaChipaca: just add fire :D14:35
ijohnsonChipaca: how did you get a picture of the inside of my desktop ?!?! :-)14:36
zygaijohnson: the other PC is also pretty tidy but not to that level14:36
Chipacaijohnson: i had to resverse the polarity on your webcam, and the rest was easy14:36
Chipacatachion flux*14:37
ijohnsonChipaca: ah I knew I shouldn't have installed that 1.21 gigawatts power generator on my webcam14:37
zygahttps://twitter.com/zygoon/status/1222166974802878464 :)14:38
zygaugly cables going everywhere14:38
ijohnsonzyga: do you have a noctua cpu cooler on that? looks very similar to mine14:42
zygawith that carpet you could get great results buy just having feet and a dust filter14:42
zygait's a 10+ year old PC14:42
zygawith a new case14:42
zyga1st gen core i714:42
zygafor the xmas refresh I will get a mobo/ram/gpu and go with something newer14:43
zygabut keep the outside the same14:43
ijohnsonyeah mine could do better, but it works well enough, just need to blow out the dust every once and a while14:43
ijohnsonthere's a big fan on the top and on the side panel that aren't in the picture that keep the whole system quite cool14:44
zygayeah, cooling in PCs is really hard to get wrong :)14:44
zygalots of space14:44
ijohnsonI wanted to do a water cooled system but I was a bit lazy14:45
ijohnsonpedronis: do you have a minute to chat about snap-bootstrap re-execing into the snapd snap on first-boot? I agree that it is complicated but I think it would be very good to have14:48
ijohnsonpedronis: I tried to make it work quickly, the most difficult thing I ran into was picking which snapd snap to mount snap-bootstrap from14:48
zygadegville: you didn't tell me you are making the corsair link snap!14:49
degvillezyga: aha, yeah. I use it personally. I think there may be a permission issue I've not sorted out with the one I've pushed though, so it's not made it to stable (I've not looked at it in a while).14:51
degvillezyga: I use it set my own fan profile for silent operation :)14:51
zygadegville: what are you controlling with it?14:56
zygadegville: the PSU?14:56
degvillezyga: HX750i PSU and H100i GT v2 CPU water cooler.14:58
zygaI have the fan controller only14:58
zygaactually, no fan and RGB controller14:59
zygaand fan controller has more rgb and temp14:59
degvillezyga: it does use a poorly documented syntax. I don't set the RGB (my box is under the table), but the syntax for my cooling profile is --device 1 --fan channel=1,mode=6,temps=0:35:45:50:55:60,speeds=0:0:55:60:75:100.15:01
pedronisijohnson: that's why it's hard, we would need to put a snapd copy into ubuntu-boot15:22
pedroniswith all that entails in terms of state and checks15:22
ijohnsonpedronis: oh also I just realized that in the fde case ubuntu-data is encrypted too15:22
pedronisijohnson: yes, ubuntu-data is encrypted15:22
ijohnsonpedronis: okay I see why you would want snapd in ubuntu-boot too15:23
ijohnsonpedronis: maybe this is too much, but maybe for the snapd snap specifically we could have a "extract snapd assets" function which would copy things like snap-bootstrap to ubuntu-boot when a snapd snap is installed?15:23
pedronisijohnson: how do we check that that binary is good15:24
pedroniswe don't sign single binaries that way15:24
pedronisijohnson: I understand the appeal, I don't see a way to keep the complexity cost vs benefit under control15:25
ijohnsonpedronis: hmm I suppose currently snap-bootstrap is part of the initramfs which is part of the kernel.efi and so thus is measured, but if it was just ran directly then it wouldn't be measured15:25
ijohnsonyeah I guess I don't see a way to easily do it then without possibly waiting to re-exec until after ubuntu-data is decrypted perhaps, but that kind of defeats the purpose15:27
pedronisijohnson: it needs to get stable and stay relatively simple15:32
ijohnsonpedronis: yes another thing I just thought about is perhaps for our tests, we could re-build the initramfs for spread tests that way we can test changes in the PR without needing to wait for a new kernel snap?15:32
pedronisijohnson: that should be possible in some way15:33
pedronismvo: ^  thoughts on on that?15:33
pedronispatching initramfs in tests15:33
zygapedronis: initramfs may need to be signed - is that a problem?15:33
* zyga needs a coffee15:33
pedroniszyga: well , we'll special certs that we use in tests15:34
ijohnsonzyga: not a problem for tests15:34
pedronisso that's should be the main problem for tests15:34
pedronisshouldn't be15:34
zygain that case it feels okay15:34
zygajust like rebuilding core/snapd snaps15:34
ijohnsonwell it would be a problem if we do try to test FDE via spread15:34
ijohnsonbut I think we have other, related problems with testing that anyways15:35
pedronisijohnson: we actually discussed some to have some kind unpack/repack mechanism for it15:35
pedronisnot sure there was progress on that15:35
ijohnsonyes I remember mvo mentioning that in SU a few weeks/months ago15:35
* zyga goes upstairs for a while15:37
ijohnsonwell I think it's a reasonable suggestion so I'm gonna prototype out building a kernel snap for uc20 in the spread test with the right initramfs15:37
* Chipaca rages a little15:41
Chipacawhy do we have an endpoint that returns things or lists of things depending on the phase of the moon15:41
* Chipaca spends a PR to fix it15:42
pedronisChipaca: which one?15:49
Chipacapedronis: /v2/create-user15:49
pedronisI wasn't aware of that15:49
Chipacapedronis: it's got two modes, create a user, or create all known users15:50
Chipacapedronis: so i'm changing users to return a list always, and hacking create-users to preserve behaviour15:50
pedronisChipaca: +115:50
Chipacai'll also change remove from returning true to returning nil as the result, for the same reason15:51
Chipacapedronis: at some later point you might want to consider login also being an action on users15:52
mvoijohnson, pedronis rebuilding the initramfs sounds appealing to me. if it's not too much hassle I would love to have this. we have snakeoil signing keys so we can even make this work in the tpm case (sorry for the slow reply, was in a ad-hoc meeting)16:15
ijohnsonmvo: ack I'll work on this then, not entirely sure how to use the snake oil keys for tpm, but I can just do the non-fde case right now easily enough16:17
mvoijohnson: yeah, one step at a time :)16:18
mupPR snapd#8061 opened: daemon: make users result more consistent <Created by chipaca> <https://github.com/snapcore/snapd/pull/8061>16:20
Chipacamuch nicer now :)16:21
mvozyga: the release/2.43 is a bit of a nightmare because of all the conflicts in "mount-ns" PER-SNAP-* files. any hints for me, it looks like this changed quite a bit between 2.43 and master and the revert of the /writable view that needs to be undone in the merge again makes this all a bit messy16:47
mvozyga: nevermind, I will just revert manually16:51
zygamvo: hmm16:53
zygamvo: re16:53
zygamvo: I'm confused - I merged the 2.43 revert16:54
zygamvo: in master it's good and we should not bring that patch to master16:54
mvozyga: yes, that's my point :) but we merge the release/* branches into master so that was a bit of pain16:55
mvozyga: it's all fine again16:55
zygaresolve with --theirs or --ours or whatever was the option16:55
zyga(don't take changes to that directory)16:55
mupPR snapd#8062 opened: release: 2.43.2 <Created by mvo5> <https://github.com/snapcore/snapd/pull/8062>16:55
zygaor just cp it back from master :)16:56
zygamvo: thank you for doing this; I'm sorry for the annoying merge16:56
mvozyga: no worries16:56
zygaand that I didn't respond faster, I was having dinner16:56
mvozyga: also no worries :)16:56
zygathere's a changelog conflict16:56
mvozyga: oh, silly me16:56
mvozyga: thank you!16:57
Chipacacurrent status of user work is https://github.com/snapcore/snapd/compare/master...chipaca:client-deluser … one more commit to go (in cmd/snap!) and it's Done(tm)17:01
Chipaca(if it all comes together without needing to go back and tweak things I'll be giving myself a putty medal)17:01
Chipacanow going to take a break17:02
mvoChipaca: nice!17:02
Chipacamvo: each of those commits becomes a PR, but I don't want to push them stacked (although i did do that for the last one)17:03
Chipacaactually, i lie, the first two commits are a PR, the others are a pr each17:03
Chipacaanyway, break time17:03
Chipacatravis is all kinds of sad17:19
Chipacalots of 503s from the store17:20
noise][yeah, we are troubleshooting CDN issues17:20
* Chipaca hugs noise][ 17:20
Chipacahmm, hmm, hm17:47
* Chipaca broke things17:47
* Chipaca fixes things17:47
mupPR snapd#8062 closed: release: 2.43.2 <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8062>20:27
mupPR snapcraft#2893 opened: [legacy] meta: include environment in hook wrappers <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2893>20:29
mupPR snapcraft#2899 opened: ci: publish the CI built snap to the Snap Store <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2899>20:29
Chipacachipaca@jan282010-194967:~$ sudo snap remove-user chipaca20:31
Chipacaerror: cannot delete user "chipaca": userdel: user chipaca is currently used by process 486520:31
* Chipaca counts that as winning20:31
zygaChipaca: woot20:49
* zyga installs a round of mac updates20:53
mupPR snapd#8063 opened: Remove users [DRAFT] <Created by chipaca> <https://github.com/snapcore/snapd/pull/8063>22:48

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