[02:15] <lifeless> question - is the seed directory needed post install? there's 250MB in there that appears to be one-off detritus
[05:13] <mborzecki> morning
[05:48] <mborzecki> wow /r/linux is melting down
[06:08] <zyga> good morning
[06:08] <zyga> mborzecki: i386?
[06:08] <mborzecki> zyga: hey, yes
[06:09] <zyga> https://discourse.ubuntu.com/t/i386-architecture-will-be-dropped-starting-with-eoan-ubuntu-19-10/11263/77 is pretty long too
[06:09] <zyga> fun times
[06:09] <zyga> how are you?
[06:10] <zyga> we're driving south from Lux
[06:10] <zyga> we're aiming for the coast of France
[06:10] <zyga> we stayed longer with our family in Lux because Gosia was not feeling very well
[06:11] <zyga> so we are one day behind
[06:11] <zyga> but now things are looking positive
[06:11] <zyga> mvo and pedronis are on a sprint this week?
[06:11] <mborzecki> zyga: kids are on vacations with their grandparents and i caught cold :/ before the weekend i hoped to be able to to 100km+ on bike, but did none of that
[06:11] <zyga> cold!? how did you do that
[06:12] <zyga> we're running south, trying to outrun the heat wave
[06:12] <mborzecki> zyga: heh, to much ice cream and cold drinks :)
[06:12] <mborzecki> s/to/too/
[06:12] <zyga> aha, well, the heat can be very hard
[06:13] <mborzecki> hope i'll be able to land some of the prs
[06:13] <zyga> mee too
[06:13] <mborzecki> wanted to do that on tuesday, but builds kept on being red
[06:13] <zyga> I'm going to review what happened in the last few days
[06:13] <zyga> and then look at most effective use of the time
[06:14] <zyga> ooo
[06:14] <mborzecki> zyga: https://github.com/snapcore/snapd/pull/6922 your comments suggest you did not finish your last pass on this one ?
[06:14] <zyga> raspi 4
[06:14] <zyga> https://www.raspberrypi.org/blog/raspberry-pi-4-on-sale-now-from-35/
[06:14] <zyga> wow
[06:14] <mborzecki> surprise surprise :)
[06:14] <zyga> woooow
[06:14] <zyga> cha ching
[06:14] <zyga> though I wished for risc v
[06:15] <jamesh> is there anything more that needs to be done to land https://github.com/snapcore/snapd/pull/6954? (skeleton of desktop session agent)
[06:15] <zyga> jamesh: hey, I'll review that after 6922
[06:16] <jamesh> zyga: thanks. https://github.com/snapcore/snapd/pull/6959 is also waiting on a review from you (a helper split out from the icon theme support PR)
[06:16] <jamesh> the session agent one is probably higher priority
[06:16] <zyga> I'll focus on helping others in the morning then
[06:16] <zyga> I've queued all three now
[06:21] <zyga> small gap in coverage while crossing  countries
[06:21] <zyga> back now
[07:06] <pstolowski> morning
[07:13] <zyga> hello pawel
[07:20] <mborzecki> pstolowski: hey
[07:33] <zyga> brb
[07:33] <zyga> need to stretch my legs
[07:35] <zyga> jamesh: https://github.com/snapcore/snapd/pull/6959#pullrequestreview-252711139
[07:35] <zyga> I'll do rest shortly
[07:40] <zyga> omg it is so hot
[07:49] <zyga> back
[07:49] <zyga> looking at 6954 now
[09:18] <Chipaca> ondra: you around?
[09:18] <ondra> Chipaca sort of uc20 sprint
[09:19] <ondra> Chipaca what's up?
[09:19] <Chipaca> ondra: spidev devices are always spidev< a bunch of numbers> < an actual period > < a bunch of more numbers >?
[09:19] <Chipaca> like spidev12345.12345 ?
[09:19] <Chipaca> device nodes*
[09:23] <ondra> Chipaca so only saw  spidev< a bunch of numbers> < an actual period > < single digit number>
[09:24] <ondra> Chipaca not sure of num after period can be also multi digit
[09:24] <Chipaca> ondra: ok
[09:24] <Chipaca> ondra: i'll comment on your PR
[09:24] <ondra> Chipaca probably best to ask some real kernel expert :)
[09:24] <ondra> Chipaca thanks :)
[09:24] <ondra> Chipaca PR is based on actual hw experience
[09:24] <Chipaca> ondra: it's about that pesky period
[09:25] <Chipaca> ondra: which lets any character through right now
[09:26] <ondra> Chipaca ah, can we lock that? I will double check PR
[09:27] <zyga> mborzecki: hey
[09:27] <zyga> mborzecki: I will need to fix 1819875 today
[09:27] <zyga> mborzecki: wanna talk about it briefly
[09:27] <zyga> mborzecki: I had some ideas before and those got stuck somehow
[09:28] <Chipaca> ondra: package main
[09:28] <Chipaca> import (
[09:28] <Chipaca> 	"fmt"
[09:28] <Chipaca> 	"regexp"
[09:28] <Chipaca> )
[09:28] <Chipaca> var rx = regexp.MustCompile("^/dev/spidev[0-9]+.[0-9]+$")
[09:28] <Chipaca> func main() {
[09:28] <Chipaca> 	for _, s := range []string{
[09:28] <Chipaca> 		"/dev/spidev1.0",
[09:28] <Chipaca> 		"/dev/spidev1💩0",
[09:28] <zyga> Chipaca: get a pastebin ;)
[09:28] <Chipaca> 	} {
[09:28] <Chipaca> 		fmt.Println(s, rx.FindString(s))
[09:28] <Chipaca> 	}
[09:28] <Chipaca> }
[09:28] <Chipaca> ugh
[09:28] <Chipaca> sorry
[09:28] <zyga> hey Chipaca :)
[09:28] <Chipaca> that was meant to be a url
[09:28] <Chipaca> https://play.golang.org/p/ZzpJfEhHC66
[09:28] <Chipaca> ^ that one
[09:28] <Chipaca> zyga: hiya!
[09:28] <zyga> long time no see, how are you?
[09:28] <Chipaca> zyga: apparently, bad at copy-and-paste
[09:29]  * Chipaca unsubscribes from stack overflow
[09:29] <Chipaca> ondra: in case I bamboozled you too much, here's the url again: https://play.golang.org/p/ZzpJfEhHC66
[09:29] <zyga> Chipaca: 386 is the number of diffefent of opinions on 32bit libraries today
[09:29] <Chipaca> ondra: i.e. the current code has a bug, plz fix :-D (note the bug is there already, but, while you're there...)
[09:30] <Chipaca> zyga: I have Opinions.
[09:30] <mborzecki> Chipaca: apparently the whole of internet has opinions
[09:32] <Chipaca> mborzecki: https://picon.ngfiles.com/599000/flash_599131_largest_crop.jpg
[09:34] <ondra> Chipaca yeah agree, I will update PR
[09:39] <Chipaca> important to remember that go's regexpses are always unicode-y
[09:57] <ondra> Chipaca when I use your example I get actually error
[09:57] <Chipaca> ondra: what error?
[09:57] <ondra> Chipaca ./prog.go:8:49: unknown escape sequence
[09:58] <Chipaca> ondra: can you show me the regexp?
[09:58] <ondra> Chipaca ah wait,  I might need to changes brackets
[10:00] <ondra> Chipaca OK this one works var rx = regexp.MustCompile(`^/dev/spidev[0-9]+\.[0-9]+$`)
[10:00] <Chipaca> ondra: by brackets you mean changing "s to `s?
[10:01]  * pstolowski early lunch
[10:02] <ondra> Chipaca yeap
[10:02] <Chipaca> ondra: k
[10:02] <ondra> Chipaca that fixed it I will update PR
[10:03] <zyga> looking at https://github.com/snapcore/snapd/pull/6922
[10:22] <mborzecki> ondra: left some questions under https://github.com/snapcore/snapd/pull/7022
[10:22] <mborzecki> that mediatek driver looks fishy :/
[10:26] <mborzecki> ondra: typo in the regex https://github.com/snapcore/snapd/compare/4a8f6caec4705fd950cdd15ded97b7e972f13076..703e8fc307c499659bbd21e36882bee18a39be7d#discussion_r296654422 ?
[10:27] <Chipaca> #7024 is a nice simple mostly-mechanical PR if you're feeling like it
[10:49] <ondra> mborzecki what is that diff for? I think you are diffing too many commits, PR is just one small commit
[10:51] <mborzecki> ondra: wrong link last time, try this one: https://github.com/snapcore/snapd/pull/7023#discussion_r296654422
[11:40] <pstolowski> mborzecki: 6997 can land
[11:41] <mborzecki> pstolowski: great, landing now ;)
[12:34]  * zyga is feeling so so due to driving and working
[12:48] <Chipaca> zyga: https://www.youtube.com/watch?v=52ogQS6QKxc
[12:51] <zyga> Chipaca: if that scales to 13" panels I'm happy
[12:51] <Chipaca> now trying to find dobey's twitter handle to point him at it :)
[13:00] <zyga> I will be joining shortly
[13:02] <mborzecki> cachio: standup?
[13:02] <ogra> zyga, https://i.ebayimg.com/images/g/IwQAAOSwvUlWsxE6/s-l640.jpg scales it to 13"
[13:04] <ogra> like in https://www.youtube.com/watch?v=rXW32WB3Xm4
[13:07] <ogra> ppisati, sniff ... found a kernel bug with "sudo modprobe bcm2835-v4l2" ... https://paste.ubuntu.com/p/xB8JcJXGx9/ ... the pi camera works fine when using it via the python lib
[13:07] <ogra> (just not when trying to get a v4l device for it)
[13:08] <ppisati> ogra: open an LP bug, how to reproduce it, etc
[13:10] <ogra> yeah, will do
[13:11] <ogra> havent tried core18 yet, perhaps 4.15 behaves better (but UC18 is a pain to use for development, missing all ools i like to use)
[13:11] <ogra> *tools
[13:37] <ogra> ppisati, https://bugs.launchpad.net/ubuntu/+source/linux-raspi2/+bug/1834039 is yours
[13:50] <ppisati> ogra: so, just loading the kmod oops the kernel?
[13:58] <ackk> jdstrand, hi, around?
[14:03] <pstolowski> yay, travis finally believed me when i say i signed CLA
[14:07] <jdstrand> ackk: I am, though I am preparing for/about to attend a meeting, so feel free to ask and I can either answer quickly or circle back
[14:10] <ackk> jdstrand, just to confirm, with the patched snapd for the system users it should be possible to call setgroups(0, NULL), correct?
[14:12] <jdstrand> ackk: yes, but your application needs to be modified to do it if it is calling setgroups some other way.
[14:13] <jdstrand> (eg, patch or LD_PRELOAD)
[14:13] <jdstrand> snap run --strace ... will show you how it is being called
[14:14] <jdstrand> ackk: also, fyi, I will be picked up that PR this week. there will be some minor snap.yaml changes but otherwise it will work as you've been testing it
[14:14] <jdstrand> picking*
[14:14] <ackk> jdstrand, yes, that's what I have, the preload ends up calling it with 0, NULL
[14:15] <ackk> jdstrand, awesome, any ETA on when it will be available in a release?
[14:16] <jdstrand> ackk: considering the focus on uc20 this week and the people who would review it are sprinting this week, there probably isn't going to be much movement for a week or two (I have a lot to do and will do it, but then need reviews, etc)
[14:17] <ackk> jdstrand, ok, cool
[14:17] <ackk> thanks
[14:17] <jdstrand> ackk: I say that cause I don't know the timing of 2.40. I guess it is possible to be in that, but more likely 2.41
[14:18] <jdstrand> ackk: they want it, so it is possible if I get to everything this week while they are sprinting (my plan), then they'll make it required for 2.40
[14:19] <jdstrand> we'll see
[14:19] <ackk> jdstrand, ok, thanks for the info
[14:20] <marcustomlinson> hey snappy brains. Have you ever thought about the possibility of allowing snaps to perform additional install time operations?
[14:21] <ogra> ppisati, yep, exactly
[14:23] <marcustomlinson> I was thinking of the slow first launch situation we face with some snaps. Would be cool (maybe? I think so at least) if we could offload some of that required initial config to the install
[14:24] <seb128> marcustomlinson, https://docs.snapcraft.io/supported-snap-hooks ?
[14:25] <ogra> marcustomlinson, the prob here is that many bits that cause slowness are related to putting bits and pieces into SNAP_USER_DATA ... at install time you dont really have that around (apart from the ability to forcefully put pre-cached files into every users home)
[14:25] <marcustomlinson> ogra: ah
[14:26] <marcustomlinson> seb128: thanks, that is also useful info
[14:26] <seb128> np
[14:39] <zyga> re
[14:40] <zyga> 39C
[14:40] <zyga> halp
[15:21] <ogra> jdstrand, hmm ... so i'm playing with a Pi camera attached via the CSI interface of the Pi, not usb ... using the picamera python lib i can access the camera via /dev/vchiq ... for access to that device node i need to use the opengl interface instead of the camera interface which feels a bit unlogic (but such is the Pi in itself :P ) ...
[15:21] <niemeyer> mup: Hello.. are you alive and kicking?
[15:21] <pstolowski> sergiusens: hey, can take a look at https://github.com/snapcore/snapcraft/pull/2609 ?
[15:21] <mup> niemeyer: I apologize, but I'm pretty strict about only responding to known commands.
[15:21] <mup> PR snapcraft#2609: schema: allow 'snapd' snap type <Created by stolowski> <https://github.com/snapcore/snapcraft/pull/2609>
[15:21] <niemeyer> mup: o/
[15:21] <mup> niemeyer: In-com-pre-hen-si-ble-ness.
[15:21] <ogra> jdstrand, i wonder if we could have vchiq added to the camera interface too so i could use the camera interface here
[15:23] <sergiusens> pstolowski: sure. Right after I finish picking up my son from school
[15:23] <pstolowski> sergiusens: ty!
[15:38] <mup> PR snapd#6995 closed: gadget: fallback device lookup <Gadget update> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6995>
[15:38] <mborzecki> yay, mup is back
[15:41] <niemeyer> Yeah, sorry about that
[15:41] <niemeyer> I'll patch it so this manual action isn't necessary as soon as I can stop traveling for a few days
[15:55] <zyga> mborzecki: https://github.com/snapcore/snapd/pull/7026
[15:55] <mup> PR #7026:  cmd,tests: forcibly discard mount namespace when bases change <Created by zyga> <https://github.com/snapcore/snapd/pull/7026>
[15:55] <zyga> mborzecki: draft, I will likely split the patch into several but I _think_ this is mostly it
[15:56] <mborzecki> niemeyer: thank you!
[15:56] <mup> PR snapd#7026 opened:  cmd,tests: forcibly discard mount namespace when bases change <Created by zyga> <https://github.com/snapcore/snapd/pull/7026>
[15:56] <mborzecki> zyga: ok, looking
[15:58] <zyga> thank you
[15:58] <zyga> I'm in a tunnel, spread running
[15:58] <zyga> will see
[16:03] <zyga> mborzecki: it passed locally
[16:04] <zyga> trying on core16 next
[16:04] <zyga> as there are usually dragons there
[16:07] <mup> Bug #1834061 opened: qt apps in hidpi looks tiny <Snappy:New> <https://launchpad.net/bugs/1834061>
[16:30] <ogra> bah ... kyrofa, sergiusens, did https://github.com/snapcore/snapcraft/pull/2322 not land in a stable snapcraft yet ? or can you not use it with the python plugin ?
[16:30] <mup> PR snapcraft#2322: project_loader: add build-environment part property <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2322>
[16:30] <ogra> i'm getting: "Failed to load plugin: properties failed to load for picamera: Additional properties are not allowed ('build-environment' was unexpected)"
[16:32] <kyrofa> ogra, it landed ages ago, but are you specifying a base?
[16:32] <ogra> nope, building for core16 so no base
[16:32] <kyrofa> ogra, try `base: core`
[16:32] <kyrofa> That will opt into the "new" snapcraft
[16:33] <kyrofa> But still build for core16
[16:33] <ogra> ok
[16:33] <kyrofa> If you're not specifying a base, you're essentially still using snapcraft v2
[16:34] <ogra> well, lets see what build.s.io thinks now
[16:36] <ogra> the big sillyness of the week ... all raspberry pi pip modules force-run a test if they are getting built on a pi ... if not they fail ... to override it you need to export "READTHEDOCS=True" ... so intuitive !!!
[16:37] <jdstrand> ogra: I've taken a note to look at that
[16:38] <ogra> jdstrand, thanks ... it isnt a blocker or anything ... just feels more natural to use camera for cameras :)
[16:38]  * jdstrand nods
[16:51] <ogra> Issues while validating snapcraft.yaml: The 'parts/picamera/build-environment[0]/READTHEDOCS' property does not match the required schema: True is not of type 'string'
[16:51] <ogra> GRRR !!!
[17:06] <mborzecki_> zyga: left some comments
[17:06] <mborzecki_> zyga: best if jdstrand takes a look as well
[17:07] <zyga> mborzecki_: thank you, looking
[17:19] <mborzecki> zyga: i noticed that selinux raised something in your PR
[18:04]  * Chipaca EODs
[19:13] <cjwatson> ogra: READTHEDOCS: 'True'   if you haven't figured it out already
[19:29] <sergiusens> kyrofa: do you have time this week to work with cachio on why the catkin-pull tests fail on the gce images?
[19:34] <cachio> sergiusens, kyrofa do you have logs?
[19:35] <sergiusens> cachio: here's one failure https://travis-ci.org/snapcore/snapcraft/jobs/548801440
[19:37] <cachio> sergiusens, the error is related to the repo (Failed to update the package cache)
[19:37] <sergiusens> kyrofa: I guess a quick trick we can do is just add a PR that prints out apt's configuration
[19:38] <cachio> sergiusens, kyrofa it would be nice if you add debug: to the test
[19:38] <cachio> then it will print all the debug info you want to see in case the test fails
[19:38] <sergiusens> cachio: yes, because `  The following signatures couldn't be verified because the public key is not available: NO_PUBKEY F42ED6FBAB17C654`, but xenial doesn't enforce that by default
[19:38] <sergiusens> so the theory is that the images were modified to enforce it
[19:39] <sergiusens> cachio: do you have an example of that? I suppose it does not require calling `spread` with `-debug`
[19:40] <cachio> just add debug:
[19:40] <cachio> to the test
[19:40] <cachio> and then the scirpt you want to run to get debug info in case of failure
[19:41] <cachio> https://github.com/snapcore/snapd/blob/master/tests/main/appstream-id/task.yaml#L13
[19:41] <cachio> sergiusens, this is anexample
[19:41] <cachio> the first I found
[19:45] <sergiusens> cachio: `debug` is only triggered on errors and before `restore` is run, correct?
[19:45] <cachio> sergiusens, yes
[19:52] <sergiusens> cachio: kyrofa if so, then https://github.com/snapcore/snapcraft/pull/2610
[19:57] <cachio> sergiusens, debug: |
[20:04] <sergiusens> cachio: even if it is a one liner? I'll fix it regardless in case it becomes a N liner
[20:23] <cachio> sergiusens, Failed to fetch http://us-east-1.ec2.archive.ubuntu.com/ubuntu/pool/main/a/apt/libapt-pkg5.0_1.2.32_amd64.deb
[20:23] <cachio> sergiusens, which images are you using?
[20:25] <sergiusens> cachio: https://github.com/snapcore/snapcraft/blob/master/spread.yaml#L35
[20:25] <sergiusens> cachio: fortunately, those network errors are transient ones from Travis, we have not reached the spread stage yet
[20:28] <cachio> sergiusens, yes, reading the https://github.com/snapcore/snapcraft/blob/master/.travis.yml
[20:29] <cachio> the alternative to skip this errors could be to create a spread tests which runs the unit tests
[20:29] <cachio> so you just need to install spread and then you run everything in gce
[20:29] <cachio> and you acn run units tests together with the rest of the spread tests
[20:29] <sergiusens> oh, it is transient and moving that to spread has been in our thoughts
[20:30] <cachio> or you can keep it in deferent stages if you want
[21:42]  * cachio afk