lifeless | question - is the seed directory needed post install? there's 250MB in there that appears to be one-off detritus | 02:15 |
---|---|---|
mborzecki | morning | 05:13 |
mborzecki | wow /r/linux is melting down | 05:48 |
zyga | good morning | 06:08 |
zyga | mborzecki: i386? | 06:08 |
mborzecki | zyga: hey, yes | 06:08 |
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:09 |
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:10 |
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:11 |
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:12 |
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:13 |
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:14 |
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:15 |
=== 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 | ||
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:16 |
zyga | small gap in coverage while crossing countries | 06:21 |
zyga | back now | 06:21 |
=== pstolowski|afk is now known as pstolowski | ||
pstolowski | morning | 07:06 |
zyga | hello pawel | 07:13 |
mborzecki | pstolowski: hey | 07:20 |
zyga | brb | 07:33 |
zyga | need to stretch my legs | 07:33 |
zyga | jamesh: https://github.com/snapcore/snapd/pull/6959#pullrequestreview-252711139 | 07:35 |
zyga | I'll do rest shortly | 07:35 |
zyga | omg it is so hot | 07:40 |
zyga | back | 07:49 |
zyga | looking at 6954 now | 07:49 |
Chipaca | ondra: you around? | 09:18 |
ondra | Chipaca sort of uc20 sprint | 09:18 |
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:19 |
ondra | Chipaca so only saw spidev< a bunch of numbers> < an actual period > < single digit number> | 09:23 |
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:24 |
Chipaca | ondra: which lets any character through right now | 09:25 |
ondra | Chipaca ah, can we lock that? I will double check PR | 09:26 |
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:27 |
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:28 |
* 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:29 |
Chipaca | zyga: I have Opinions. | 09:30 |
mborzecki | Chipaca: apparently the whole of internet has opinions | 09:30 |
Chipaca | mborzecki: https://picon.ngfiles.com/599000/flash_599131_largest_crop.jpg | 09:32 |
ondra | Chipaca yeah agree, I will update PR | 09:34 |
Chipaca | important to remember that go's regexpses are always unicode-y | 09:39 |
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:57 |
Chipaca | ondra: can you show me the regexp? | 09:58 |
ondra | Chipaca ah wait, I might need to changes brackets | 09:58 |
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:00 |
* pstolowski early lunch | 10:01 | |
ondra | Chipaca yeap | 10:02 |
Chipaca | ondra: k | 10:02 |
ondra | Chipaca that fixed it I will update PR | 10:02 |
zyga | looking at https://github.com/snapcore/snapd/pull/6922 | 10:03 |
mborzecki | ondra: left some questions under https://github.com/snapcore/snapd/pull/7022 | 10:22 |
mborzecki | that mediatek driver looks fishy :/ | 10:22 |
mborzecki | ondra: typo in the regex https://github.com/snapcore/snapd/compare/4a8f6caec4705fd950cdd15ded97b7e972f13076..703e8fc307c499659bbd21e36882bee18a39be7d#discussion_r296654422 ? | 10:26 |
Chipaca | #7024 is a nice simple mostly-mechanical PR if you're feeling like it | 10:27 |
ondra | mborzecki what is that diff for? I think you are diffing too many commits, PR is just one small commit | 10:49 |
mborzecki | ondra: wrong link last time, try this one: https://github.com/snapcore/snapd/pull/7023#discussion_r296654422 | 10:51 |
pstolowski | mborzecki: 6997 can land | 11:40 |
mborzecki | pstolowski: great, landing now ;) | 11:41 |
* zyga is feeling so so due to driving and working | 12:34 | |
Chipaca | zyga: https://www.youtube.com/watch?v=52ogQS6QKxc | 12:48 |
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 :) | 12:51 |
zyga | I will be joining shortly | 13:00 |
mborzecki | cachio: standup? | 13:02 |
ogra | zyga, https://i.ebayimg.com/images/g/IwQAAOSwvUlWsxE6/s-l640.jpg scales it to 13" | 13:02 |
ogra | like in https://www.youtube.com/watch?v=rXW32WB3Xm4 | 13:04 |
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:07 |
ppisati | ogra: open an LP bug, how to reproduce it, etc | 13:08 |
ogra | yeah, will do | 13:10 |
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:11 |
ogra | ppisati, https://bugs.launchpad.net/ubuntu/+source/linux-raspi2/+bug/1834039 is yours | 13:37 |
ppisati | ogra: so, just loading the kmod oops the kernel? | 13:50 |
ackk | jdstrand, hi, around? | 13:58 |
pstolowski | yay, travis finally believed me when i say i signed CLA | 14:03 |
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:07 |
ackk | jdstrand, just to confirm, with the patched snapd for the system users it should be possible to call setgroups(0, NULL), correct? | 14:10 |
jdstrand | ackk: yes, but your application needs to be modified to do it if it is calling setgroups some other way. | 14:12 |
jdstrand | (eg, patch or LD_PRELOAD) | 14:13 |
jdstrand | snap run --strace ... will show you how it is being called | 14:13 |
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:14 |
ackk | jdstrand, awesome, any ETA on when it will be available in a release? | 14:15 |
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:16 |
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:17 |
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:18 |
jdstrand | we'll see | 14:19 |
ackk | jdstrand, ok, thanks for the info | 14:19 |
marcustomlinson | hey snappy brains. Have you ever thought about the possibility of allowing snaps to perform additional install time operations? | 14:20 |
ogra | ppisati, yep, exactly | 14:21 |
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:23 |
seb128 | marcustomlinson, https://docs.snapcraft.io/supported-snap-hooks ? | 14:24 |
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:25 |
marcustomlinson | seb128: thanks, that is also useful info | 14:26 |
seb128 | np | 14:26 |
zyga | re | 14:39 |
zyga | 39C | 14:40 |
zyga | halp | 14:40 |
=== tinwood_ is now known as tinwood | ||
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:21 |
sergiusens | pstolowski: sure. Right after I finish picking up my son from school | 15:23 |
pstolowski | sergiusens: ty! | 15:23 |
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:38 |
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:41 |
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:55 |
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:56 |
zyga | thank you | 15:58 |
zyga | I'm in a tunnel, spread running | 15:58 |
zyga | will see | 15:58 |
zyga | mborzecki: it passed locally | 16:03 |
zyga | trying on core16 next | 16:04 |
zyga | as there are usually dragons there | 16:04 |
=== pstolowski is now known as pstolowski|afk | ||
mup | Bug #1834061 opened: qt apps in hidpi looks tiny <Snappy:New> <https://launchpad.net/bugs/1834061> | 16:07 |
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:30 |
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:32 |
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:33 |
ogra | well, lets see what build.s.io thinks now | 16:34 |
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:36 |
jdstrand | ogra: I've taken a note to look at that | 16:37 |
ogra | jdstrand, thanks ... it isnt a blocker or anything ... just feels more natural to use camera for cameras :) | 16:38 |
* jdstrand nods | 16:38 | |
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 !!! | 16:51 |
mborzecki_ | zyga: left some comments | 17:06 |
mborzecki_ | zyga: best if jdstrand takes a look as well | 17:06 |
zyga | mborzecki_: thank you, looking | 17:07 |
=== mborzecki_ is now known as mborzecki | ||
mborzecki | zyga: i noticed that selinux raised something in your PR | 17:19 |
* Chipaca EODs | 18:04 | |
cjwatson | ogra: READTHEDOCS: 'True' if you haven't figured it out already | 19:13 |
sergiusens | kyrofa: do you have time this week to work with cachio on why the catkin-pull tests fail on the gce images? | 19:29 |
cachio | sergiusens, kyrofa do you have logs? | 19:34 |
sergiusens | cachio: here's one failure https://travis-ci.org/snapcore/snapcraft/jobs/548801440 | 19:35 |
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:37 |
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:38 |
sergiusens | cachio: do you have an example of that? I suppose it does not require calling `spread` with `-debug` | 19:39 |
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:40 |
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:41 |
sergiusens | cachio: `debug` is only triggered on errors and before `restore` is run, correct? | 19:45 |
cachio | sergiusens, yes | 19:45 |
sergiusens | cachio: kyrofa if so, then https://github.com/snapcore/snapcraft/pull/2610 | 19:52 |
cachio | sergiusens, debug: | | 19:57 |
sergiusens | cachio: even if it is a one liner? I'll fix it regardless in case it becomes a N liner | 20:04 |
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:23 |
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:25 |
cachio | sergiusens, yes, reading the https://github.com/snapcore/snapcraft/blob/master/.travis.yml | 20:28 |
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:29 |
cachio | or you can keep it in deferent stages if you want | 20:30 |
* cachio afk | 21:42 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!