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