| NickZ | kyrofa: is there any documentation on the changes to the catkin plugin syntax when you specify a base core? | 01:36 |
|---|---|---|
| `dw | is there some way to get around the forum's "new users may only include (1 pictures|2 links)" restriction on the forum? | 01:57 |
| mup | PR snapd#6377 opened: tests: auto-clean the test directory <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6377> | 02:45 |
| kyrofa | NickZ, sorry, we're not timezone aligned this week at all! When you specify a base, the catkin plugin will use the ROS release that corresponds to it. core/core16 will use kinetic, core18 will use melodic | 05:16 |
| kyrofa | We've moved away from being able to specify the rosdistro so that we can ensure we're running the supported ROS distro on the supported ubuntu distro | 05:16 |
| mborzecki | morning | 06:11 |
| mvo | hey mborzecki ! | 06:11 |
| mborzecki | mvo: hey hey | 06:11 |
| mborzecki | #6370 is finally green | 06:12 |
| mup | PR #6370: interfaces/builtin/opengl: allow access to NVIDIA VDPAU library <Created by cgutman> <https://github.com/snapcore/snapd/pull/6370> | 06:12 |
| mborzecki | mvo: can you take a look ^^ ? | 06:12 |
| mvo | mborzecki: I just have it open - the author has signed our CLA? | 06:12 |
| mborzecki | mvo: the cla check is green, hope that's the same as confirmation he's signed it :) | 06:13 |
| mvo | mborzecki: yeah, let me do a quick check | 06:14 |
| zyga | Good morning | 06:15 |
| mvo | mborzecki: yeah, looks good | 06:15 |
| mvo | hey zyga ! good morning | 06:15 |
| mborzecki | #6326 sound like 2.37 material too, jdstrand requested changes there, but it's fixed already | 06:15 |
| mup | PR #6326: interface: raw-usb: Adding ttyACM ttyACA permissions <Created by kubiko> <https://github.com/snapcore/snapd/pull/6326> | 06:15 |
| mborzecki | zyga: hey | 06:15 |
| zyga | how was yesterday? | 06:16 |
| zyga | I saw some failures on master | 06:16 |
| mvo | zyga: things are mostly ok | 06:21 |
| mvo | zyga: very few failures | 06:21 |
| mvo | zyga: afaict at least | 06:21 |
| mvo | mborzecki: I taged it 2.37 but will consider it a target of opportunity. I would really like to have the +1 from jdstrand on pr#6326 before merging it | 06:22 |
| zyga | mvo: that's good, must have been a fluke on one branch then | 06:26 |
| === pstolowski|afk is now known as pstolowski | ||
| pstolowski | morning | 08:02 |
| mvo | hey pstolowski - good morning | 08:04 |
| mvo | mborzecki: meh, this configure hook with network-control forum topic is a bit of a can of worms, looks like we have a bug in snap-confine for hooks, the cgroup name is incorrect so no devices are part of the cgroup :/ | 08:14 |
| pstolowski | oh, what's that? | 08:16 |
| mvo | pstolowski: there is this forum topic "stracing a configure hook" | 08:17 |
| mvo | pstolowski: and it points to a real bug it seems in snap-confine | 08:18 |
| pstolowski | interesting, thanks | 08:20 |
| mborzecki | mvo: which topic is that? | 08:24 |
| mborzecki | mvo: ah the strace topic | 08:24 |
| mvo | mborzecki: yeah, I think I'm close, its quite twisted though | 08:24 |
| mvo | mborzecki: anyway, just wanted to let you all know that I look into it (to avoid duplication of effort) | 08:25 |
| mborzecki | mvo: no worries, i was looking into https://forum.snapcraft.io/t/restart-a-snap-background-service-on-resume-from-suspend/9182/5, power mgmt can of worms | 08:26 |
| mborzecki | i don't understand why it's so convoluted | 08:28 |
| mvo | mborzecki: yeah :/ | 08:29 |
| mvo | mborzecki: thanks for looking into this one | 08:29 |
| mborzecki | mvo: about cgroup, iirc we add a bunch of /dev/* by default | 08:33 |
| mborzecki | mvo: including /dev/null | 08:33 |
| mborzecki | mvo: https://github.com/snapcore/snapd/blob/master/cmd/snap-confine/udev-support.c#L168 | 08:34 |
| mvo | mborzecki: yeah, its all good | 08:36 |
| mvo | mborzecki: I mean, its not :) | 08:36 |
| mvo | mborzecki: but the code is mostly DTRT, it seems there is a bug in the cgroup name generation | 08:36 |
| zyga | re | 08:37 |
| mvo | mborzecki: so we do DTRT just to the wrong cgroup (test-snapd-with-configure-nc_hook.hook_configure vs test-snapd-with-configure_nc_hook_configure | 08:37 |
| mvo | hey zyga | 08:37 |
| zyga | mmm | 08:38 |
| mborzecki | mvo: interesting | 08:38 |
| zyga | cgroup name for hooks | 08:38 |
| zyga | interesting | 08:38 |
| zyga | I'll let you handle that but one word of advice | 08:38 |
| zyga | device cgroups sticks around | 08:38 |
| mvo | zyga: :) ok | 08:38 |
| zyga | you can run something and then, at your leisure, look at it later | 08:38 |
| zyga | and you can move a shell there for full hands on | 08:38 |
| zyga | let me know if you need help with any of that | 08:39 |
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| mborzecki | pstolowski: hi, can you take a look at https://github.com/snapcore/snapd/pull/6373 ? | 09:31 |
| mup | PR #6373: overlord/ifacestate: helper API to obtain the state of connections <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6373> | 09:31 |
| mborzecki | guys, if you have any thoughts about presenting disconnected plugs/slots or undesired connections in the output of snap connections, feel free to share them ehre https://forum.snapcraft.io/t/rfc-snap-connections-command/4296/12 | 09:34 |
| Chipaca | mborzecki: o/ | 09:39 |
| mborzecki | Chipaca: hey | 09:39 |
| Chipaca | mborzecki: I'll reply there | 09:39 |
| mborzecki | Chipaca: sure | 09:39 |
| mborzecki | pstolowski: yeah, maybe the endpoint should only return connections, connected or undesired, but no plugs/slots list like we get from /v2/interfaces | 09:48 |
| pstolowski | mborzecki: ah, going even further. might be a good idea | 09:50 |
| mup | PR snapd#6378 opened: cmd: fix snap-device-helper to deal correctly with hooks <Created by mvo5> <https://github.com/snapcore/snapd/pull/6378> | 09:52 |
| Chipaca | mborzecki: there ya go | 09:53 |
| Chipaca | hah | 09:53 |
| mborzecki | Chipaca: thx, reading | 09:53 |
| Chipaca | pstolowski: agreed on -d :-) | 09:53 |
| Chipaca | woops typo | 09:54 |
| * Chipaca fixes | 09:54 | |
| Chipaca | mborzecki: wrt my last comment, I can do that in a followup if you'd rather not bash your head against tabwriter and escape sequences :-) | 09:54 |
| mborzecki | Chipaca: tis fine, go ahead | 09:55 |
| mup | PR snapd#6379 opened: ifacestate/tests: extra test for hotplug-connect handler <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6379> | 10:23 |
| Chipaca | degville: o/ | 10:26 |
| degville | Chipaca: hello! | 10:26 |
| Chipaca | degville: PM | 10:27 |
| Chipaca | degville: should I create the topic for #6369, or would you rather do it? (if I do it it'll be a placeholder for now) | 10:33 |
| mup | PR #6369: Add check for snap binaries dir not being in path <Simple 😃> <Squash-merge> <Created by liamg> <https://github.com/snapcore/snapd/pull/6369> | 10:33 |
| degville | Chipaca: feel free to create the topic. I guess you'd want it to be as short as possible. I can always add the text. | 10:34 |
| Chipaca | degville: I'll leave it unlisted for now :-) | 10:35 |
| === chihchun is now known as chihchun_afk | ||
| === cpaelzer_ is now known as cpaelzer | ||
| mup | PR snapcraft#2439 closed: snap: add xslt dependencies for lxml <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2439> | 11:26 |
| === chihchun_afk is now known as chihchun | ||
| mup | PR snapd#6380 opened: Nm iface add perms <Created by alfonsosanchezbeato> <https://github.com/snapcore/snapd/pull/6380> | 11:46 |
| === chihchun is now known as chihchun_afk | ||
| mborzecki | Chipaca: https://paste.ubuntu.com/p/VVNpnmJ6QY/ | 12:31 |
| mborzecki | still have to add the count of disconnected/unconnected ifaces | 12:31 |
| === chihchun_afk is now known as chihchun | ||
| pstolowski | hey marcustomlinson :) | 12:47 |
| marcustomlinson | hey pstolowski ! | 12:48 |
| mborzecki | off to pick up the kids | 13:02 |
| pstolowski | mborzecki: can you take a look at https://github.com/snapcore/snapd/pull/6379 ? | 13:08 |
| mup | PR #6379: ifacestate/tests: extra test for hotplug-connect handler <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6379> | 13:08 |
| Gargoyle | Hey all. | 13:40 |
| Gargoyle | Is there any fix for broken electron snaps on 18.10 yet. I now have no text in atom's menu bar! :/ | 13:41 |
| Chipaca | Gargoyle: I wasn't aware they were broken :-) | 13:50 |
| Chipaca | let me start an 18.10 vm to have a look | 13:51 |
| Chipaca | Gargoyle: is there a bug or forum post talking about the issue? | 13:51 |
| Chipaca | Gargoyle: I have text in atom's menu bar here | 13:53 |
| Chipaca | Gargoyle: https://i.imgur.com/56r236E.png | 13:53 |
| Chipaca | mborzecki: Conflicts=sleep.target WFM AFAICT FWIW | 14:16 |
| Chipaca | but maybe I should test it a bit more :-) | 14:17 |
| mborzecki | Chipaca: interesting, i'll play a bit more with these on the laptop | 14:18 |
| jdstrand | mvo: hi! miss you in sunny cape town :) | 14:23 |
| jdstrand | mvo: it seems like you are looking into this, so fyi, https://forum.snapcraft.io/t/stracing-a-configure-hook/9452/7 | 14:23 |
| jdstrand | mvo: also, I'll look at PR 6326 again | 14:23 |
| mup | PR #6326: interface: raw-usb: Adding ttyACM ttyACA permissions <Created by kubiko> <https://github.com/snapcore/snapd/pull/6326> | 14:23 |
| Chipaca | mborzecki: to be clear, Conflicts=sleep.target works to stop the service before suspending | 14:26 |
| Chipaca | mborzecki: you need something else to wake it after resuming | 14:27 |
| jdstrand | mvo: re 6326, just need a one character fix then feel free to commit | 14:27 |
| Chipaca | mborzecki: you'd think there'd be a resume.targt, but lennart says "Why would a service need something like this? This sounds systematically flawed" | 14:28 |
| mborzecki | Chipaca: hahah | 14:29 |
| mborzecki | Chipaca: hard to disagree with him | 14:30 |
| Chipaca | mborzecki: that's https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=744753#30 fwiw | 14:30 |
| mborzecki | Chipaca: on a side note, the use case from the forum should probably hook to nelink and act on interface state changes | 14:32 |
| * Chipaca covers his ears and goes LALALALA | 14:33 | |
| zyga | https://github.com/snapcore/snapd/pull/6380 | 14:39 |
| mup | PR #6380: Nm iface add perms <Created by alfonsosanchezbeato> <https://github.com/snapcore/snapd/pull/6380> | 14:39 |
| zyga | this is one terse commit message | 14:39 |
| zyga | I re-wrote the PR title | 14:40 |
| mvo | jdstrand: cool, will look at it | 14:42 |
| mvo | jdstrand: heh - I miss the sun as well! next time I can make it again. I am looking into the strace right now, I think I have a fix | 14:43 |
| mvo | jdstrand: not strace, the device cgroup with configure hook support | 14:43 |
| mborzecki | Chipaca: left a note in the forum, imo the guy should really be using avahi | 14:58 |
| mup | PR core18#114 opened: Set the C.UTF-8 locale for the 001-extra-packages.test <Created by sil2100> <https://github.com/snapcore/core18/pull/114> | 15:04 |
| mup | PR snapd#6381 opened: devicestate: add initial Remodel support <Created by mvo5> <https://github.com/snapcore/snapd/pull/6381> | 15:13 |
| mborzecki | Chipaca: hm that interface count is quite interesting, the API actually does some filtering and returns only connected interfaces by default, unless the client asked for all | 15:14 |
| Chipaca | mborzecki: isn't this the api that you're adding in the same PR? | 15:23 |
| Chipaca | #6373 | 15:24 |
| mup | PR #6373: overlord/ifacestate: helper API to obtain the state of connections <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6373> | 15:24 |
| Chipaca | ah no | 15:24 |
| * Chipaca looks more | 15:24 | |
| Chipaca | #6333 | 15:24 |
| mup | PR #6333: daemon: introduce /v2/connections snapd API endpoint <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6333> | 15:24 |
| mborzecki | Chipaca: not the same PR, but yes, that's the API i'm adding, that's why it's helpful to get some feedback on the functionality people expect | 15:25 |
| Chipaca | oh and it's 'changes requested' by me and I never looked again | 15:25 |
| Chipaca | I'm a horrible person | 15:25 |
| * Chipaca wanders off | 15:25 | |
| mborzecki | haha :P | 15:25 |
| Chipaca | :-) | 15:25 |
| mborzecki | Chipaca: so the API actually takes ?select=all to include disconnect plugs/slots and undesired conections, otherwise it's just connected | 15:26 |
| mborzecki | i could push everything and filter in the client | 15:26 |
| Chipaca | mborzecki: or, add counts to the interfaceJSON (if I'm reading the code right) | 15:27 |
| mborzecki | Chipaca: hm some itneresting bits are still missing in that pr | 15:27 |
| Chipaca | mborzecki: how is gnome software going to use v2/connections? | 15:28 |
| mborzecki | Chipaca: that's what I discussed with pedronis but haven't pushed out yet: https://paste.ubuntu.com/p/G4h3q4n2Zz/ | 15:29 |
| Chipaca | aha | 15:30 |
| mborzecki | Chipaca: Established is what's currently connected, so one does not have to dig inside Plugs/Slots, Undesired is what's manually disconnected | 15:31 |
| Chipaca | mborzecki: what about things that aren't undesired and aren't connected? | 15:33 |
| Chipaca | i mean established | 15:33 |
| Chipaca | i mean: established means connected right now, undesired means manually disconnected? | 15:33 |
| mborzecki | Chipaca: yes, established === connected, undesired === manually disconnected | 15:34 |
| Chipaca | mborzecki: so what about the other ones? | 15:34 |
| Chipaca | where do they appear? | 15:35 |
| mborzecki | Chipaca: that's where you dig into Plugs/Slots | 15:37 |
| mborzecki | Chipaca: iow, pairs that could be connected, but aren't right? | 15:37 |
| Chipaca | yeah | 15:37 |
| mborzecki | Chipaca: yup, so that's in Plugs/Slots, unconnected ones have len(Plug.Connections) == 0 | 15:38 |
| mborzecki | Chipaca: but there is no indication what given plug/slot could be connected to | 15:38 |
| mvo | fun! core18 fails to boot in spread :( spread qemu:ubuntu-core-18-64 | 15:39 |
| mborzecki | Chipaca: to be fair, you could connnect any pair that uses matching interface | 15:40 |
| Chipaca | mvo: excessive collisions & not enough packet ambulances ? | 15:41 |
| Chipaca | mborzecki: I'm never fair | 15:41 |
| mborzecki | hah :) | 15:41 |
| mvo | mounting /dev/sda3 on /tmpmnt_writable failed: device or resource busy | 15:44 |
| mvo | cachio: ^- could you please have a look? maybe a new kernel or a new core18 that breaks this | 15:47 |
| mvo | cachio: I need to leave for ~2h for a kids appointment unfortunately so can't look right now | 15:47 |
| === chihchun is now known as chihchun_afk | ||
| cachio | mvo, in 10 minutes after lunch :) | 16:20 |
| cachio | mvo, Ill leave the tests running | 16:21 |
| mup | PR snapd#6377 closed: tests: auto-clean the test directory <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6377> | 17:02 |
| === pstolowski is now known as pstolowski|afk | ||
| mup | PR snapd#6382 opened: tests: use pc-kernel from stable instead of edge to build core-16 and core-18 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6382> | 17:58 |
| Chipaca | ouch, EOD | 18:21 |
| * Chipaca runs | 18:21 | |
| cachio | mvo, hey | 18:49 |
| cachio | mvo, same issue I see when I start a core-18 vm locally | 18:50 |
| cachio | related to lack of entropy | 18:50 |
| cachio | we are currently faking the entropy source on google instnaces, perhaps we could do the same on this core instance before booting it | 18:51 |
| mvo | cachio: I was getting a busybox shell when running spread on ubuntu-core-18 here locally | 19:01 |
| mvo | cachio: you can run SPREAD_QEMU_GUI=1 spread ... | 19:02 |
| mvo | cachio: and then you get the qemu window and can get around the entropy problem | 19:02 |
| mvo | cachio: and thanks for looking into it! if reverting to the stable kernel fixes all issues we should probably let the kernel guys know :) | 19:03 |
| cachio | mvo, it fixes the problem | 19:09 |
| cachio | but now I see other issues becuase it refreshes the kernel snap | 19:10 |
| cachio | during the execution | 19:10 |
| === grumble is now known as AMENDMENT_EFFFFF | ||
| === AMENDMENT_EFFFFF is now known as grumble | ||
| cachio | mvo, I'll need to omve the IMAGE_CHANNEL to stable too | 19:12 |
| cachio | mvo, not sure if it if gonna work | 19:12 |
| mup | PR core18#114 closed: Set the C.UTF-8 locale for the 001-extra-packages.test <Created by sil2100> <Merged by sil2100> <https://github.com/snapcore/core18/pull/114> | 19:21 |
| mvo | cachio: ok | 19:34 |
| mvo | cachio: not sure what is going on, please keep me updated! I can start digging into that in my morning | 19:34 |
| cachio | using the image and gadget snap = stable it wokrs | 19:35 |
| cachio | it is a workaround until the issue is fixed | 19:38 |
| mvo | cachio: yeah, interessting! | 19:38 |
| cachio | there is a PR with thnat changge | 19:38 |
| mvo | cachio: I wonder if we can figure out what happend, i.e. what the root cause is | 19:38 |
| cachio | well, the pc-kernel snap has changed | 19:39 |
| cachio | from edge to candidate | 19:39 |
| cachio | no idea yet which change on the kernel is producing the error | 19:40 |
| cachio | still researching | 19:40 |
| mvo | cachio: thank you | 19:42 |
| mvo | cachio: interessting, there was a new pc-kernel upload 8h ago and a new one 2h ago | 19:44 |
| mvo | cachio: maybe that helped with things? | 19:45 |
| mvo | klebers: hey, I saw two kernel uploads recently. one 8h ago and one 2h ago. are there any known issues in the prev kernel or just an update? | 19:45 |
| mvo | cachio: do we have all must-have 2.37 PR (that fix the tests etc) tagged in github? | 20:18 |
| cachio | I think there are nice to have | 20:23 |
| cachio | mvo, | 20:23 |
| cachio | It is not needed to add them to 2.37 | 20:24 |
| mvo | cachio: great, thanks | 20:24 |
| cachio | mvo, yaw | 20:25 |
| mvo | cachio: I go over the PRs in the morning, I hope we can finalize 2.37 then | 20:26 |
| cachio | mvo, great | 20:29 |
| cachio | I'll check the status tomorrow morning | 20:29 |
| cachio | thanks!! | 20:29 |
| === chihchun_afk is now known as chihchun | ||
| mup | PR snapd#6383 opened: tests: provide a fake random device to the core images <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6383> | 21:46 |
| === chihchun is now known as chihchun_afk | ||
| kravietz | hi everyone, I build a snap with PhantomJS that works well in classic mode, but when in full confinement it cannot load libfreetype and libfontconfig libraries, even though they are packaged (in $SNAP/usr/lib...) | 22:12 |
| kravietz | I'm a bit confused - from where the confined binary should load the libraries? from system-wide /lib & /usr/lib or the $SNAP/lib $SNAP/usr/lib ? in such case why cannot it see them by default? | 22:14 |
| kravietz | LD_LIBRARY_PATH="/snap/webcookies/x23/usr/lib/x86_64-linux-gnu" seems to help but why isn't it set by default if libraries are snapped? | 22:15 |
| Gargoyle | In answer to my own question from earlier - In synaptic you can select a package and choose "lock version". So I was able to pick my way through and change from PPA version to official cosmic version. | 22:47 |
| Gargoyle | Chipaca: https://i.imgur.com/YVeUqls.png | 22:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
