| === JoshStrobl is now known as JoshStrobl|zzz | ||
| === chihchun_afk is now known as chihchun | ||
| === JanC is now known as Guest58454 | ||
| === JanC_ is now known as JanC | ||
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| pstolowski | mornings | 07:14 |
|---|---|---|
| davidcalle | Good morning | 07:42 |
| zyga-ubuntu | good morning | 07:48 |
| kalikiana_ | o/ | 08:34 |
| === JoshStrobl|zzz is now known as JoshStrobl | ||
| Son_Goku | zyga-ubuntu, can we merge this now? https://github.com/snapcore/snapd/pull/3984 | 09:44 |
| mup | PR #3984: release,cmd,dirs: Redo the distro checks to take into account distribution families <Created by Conan-Kudo> <https://github.com/snapcore/snapd/pull/3984> | 09:44 |
| zyga-ubuntu | Son_Goku: hello | 09:45 |
| zyga-ubuntu | Son_Goku: looking | 09:45 |
| Son_Goku | morning | 09:45 |
| zyga-ubuntu | Son_Goku: +1'd, I'll ack with mvo and merge | 09:47 |
| Son_Goku | mvo is probably not going to be alive for some time... | 09:47 |
| zyga-ubuntu | Son_Goku: I'm a bit absent minded today with the news about catalan news | 09:47 |
| zyga-ubuntu | Son_Goku: how was the event? | 09:48 |
| zyga-ubuntu | Son_Goku: at least you didn't have to fly far :) | 09:48 |
| Son_Goku | I didn't fly at all | 09:48 |
| Son_Goku | it's too close to fly ;) | 09:48 |
| Son_Goku | it was pretty good | 09:48 |
| Son_Goku | though now I'm sick :( | 09:48 |
| zyga-ubuntu | Son_Goku: me too :) | 09:49 |
| zyga-ubuntu | Son_Goku: it was 3C in the morning today | 09:49 |
| zyga-ubuntu | Son_Goku: and the temperature jumps around to 15 and back all the time | 09:49 |
| Son_Goku | it's 8C right now | 09:50 |
| mup | PR snapd#3966 closed: cmd/snap-seccomp,osutil: make user/group lookup functions public <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3966> | 10:05 |
| mup | PR snapd#3988 closed: Added note in HACKING file <Created by rmescandon> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3988> | 10:05 |
| pstolowski | zyga-ubuntu, hey, pondering about your comment https://github.com/snapcore/snapd/pull/3972/files/46c79aed60e0b8a3cfe7d827a32af297f2d4950e#r141618790 | 10:10 |
| mup | PR #3972: repo: sanitize plugs and slots early in ReadInfo <Created by stolowski> <https://github.com/snapcore/snapd/pull/3972> | 10:10 |
| pstolowski | zyga-ubuntu, ie. the check if snap.SanitizePlugsSlots is not set. it's tricky considering the tests | 10:10 |
| pstolowski | zyga-ubuntu, would you expect it to panic if already set? or just print a warning? | 10:11 |
| zyga-ubuntu | pstolowski: hey | 10:13 |
| zyga-ubuntu | pstolowski: I was thinking about it myself | 10:13 |
| zyga-ubuntu | pstolowski: it'd be easier if it would allow a nil value and just handle that elsewhere | 10:13 |
| zyga-ubuntu | pstolowski: then any non-nil value could panic | 10:13 |
| zyga-ubuntu | pstolowski: as for tests they should be fine as they can skip that check and assign the variable directly | 10:14 |
| pstolowski | zyga-ubuntu, I did with a global bool for a quick test, but the problem is tests seem to be creating more than one instance when they e.g instantiate overlord | 10:14 |
| zyga-ubuntu | aha | 10:15 |
| zyga-ubuntu | pstolowski: well, think about it | 10:16 |
| zyga-ubuntu | pstolowski: maybe you can come up with something | 10:16 |
| pstolowski | zyga-ubuntu, sure, will keep investigating | 10:17 |
| === chihchun is now known as chihchun_afk | ||
| * zyga-ubuntu finishes NFS spread test and picks up another topic | 10:42 | |
| zyga-ubuntu | hmm | 10:54 |
| zyga-ubuntu | so I have 8GB of ram now and I still see swap | 10:54 |
| zyga-ubuntu | :/ | 10:54 |
| kalikiana_ | zyga-ubuntu: Same here, I can't find a way to avoid it even if to my best knowledge memory never exceeds what I have | 11:00 |
| zyga-ubuntu | kalikiana_: I know linux swaps out unused memory just to use the RAM better but I really want to avoid that | 11:01 |
| zyga-ubuntu | meanwhile, I'm in shell quoting hello | 11:01 |
| zyga-ubuntu | *hell | 11:01 |
| kalikiana_ | I thought that's what "shell" stood for if you spelt it out ;-) | 11:03 |
| zyga-ubuntu | shell hell :) | 11:03 |
| kalikiana_ | I'm already feeling the reverse jetlag... this week is not gonna be any easier to get through | 11:10 |
| zyga-ubuntu | so I have quoting fixed, now for the other half of the test | 11:12 |
| === ShalokShalom is now known as ShalokShalom2 | ||
| Son_Goku | kalikiana_: well, at least you're not sick | 11:51 |
| zyga-ubuntu | ok, more progress, expanded the NFS test, now just to ensure it restores and I can push that | 11:51 |
| zyga-ubuntu | woot, passed | 12:03 |
| zyga-ubuntu | ok | 12:03 |
| zyga-ubuntu | ok, now 14.04 to check it works too | 12:11 |
| kalikiana_ | Son_Goku: now you're making me wonder... but I sure hope it's just the jetlag, at least now I can take power naps during the day | 12:13 |
| kalikiana_ | on the sprint that was difficult, with it being a constant hallway conversation and such | 12:14 |
| Son_Goku | I hope I can get over this cold quickly... | 12:16 |
| zyga-ubuntu | pstolowski: hey, can you please have a look at https://github.com/snapcore/snapd/pull/3958 | 12:17 |
| mup | PR #3958: many: add support for /home on NFS <Created by zyga> <https://github.com/snapcore/snapd/pull/3958> | 12:17 |
| pstolowski | yep | 12:17 |
| zyga-ubuntu | thanks | 12:18 |
| zyga-ubuntu | I need some tea and medicine, back soon | 12:20 |
| zyga-ubuntu | (the joys of working from home is that you can work while ill) | 12:20 |
| ogra_ | hmm, are all snaopcraft.io URLs (including the store) dead or is it me ? | 13:00 |
| Son_Goku | ogra_: well, https://snapcraft.io/ works for me | 13:00 |
| zyga-ubuntu | pstolowski: and https://github.com/snapcore/snapd/pull/3965 is also easy to review | 13:00 |
| mup | PR #3965: interfaces/mount: add support for parsing x-snapd-mkdir-{mode,uid,gid}= <Created by zyga> <https://github.com/snapcore/snapd/pull/3965> | 13:00 |
| niemeyer | Running late, but will be in the standup in a couple of minutes | 13:00 |
| * zyga-ubuntu joins | 13:00 | |
| ogra_ | weird | 13:06 |
| ogra_ | Son_Goku, thanks ... seems it is related to the canonical VPN (my desktop got stuck) | 13:08 |
| === JoshStrobl is now known as JoshStrobl|AFK | ||
| * zyga-ubuntu is back but needs to do another errand, I'll be online but off irc as connection will be hit/miss | 14:46 | |
| kyrofa | flexiondotorg, did that alsa fix actually end up working for you on that snap you were working on? | 15:40 |
| flexiondotorg | kyrofa: It did. | 15:44 |
| flexiondotorg | I want to make it a remote part because anything needing just ALSA will require that bespoke set of config files. | 15:45 |
| kyrofa | Yes. Definitely | 15:45 |
| kyrofa | flexiondotorg, I _think_ this is the case but want to verify: those */card/* files are generic, right? They're the entire set of config files for every card supported by alsa, not somehow specific to the hardware on which alsa is installed. Agreed? | 15:46 |
| sergiusens | flexiondotorg wait, you got things working without the need to snapcraft-preload? | 15:46 |
| kyrofa | i.e. it actually does makes sense to make it a remote part? | 15:46 |
| kalikiana_ | kyrofa: I'm guessing you said something like 'have a nice day' but it was slow motion robot voice :-P | 15:47 |
| kyrofa | sergiusens, yeah baby | 15:47 |
| kyrofa | kalikiana_, hahaha | 15:47 |
| flexiondotorg | kyrofa: The card configuration is generic but requires the paths are modififed in each one. | 15:47 |
| kyrofa | sergiusens, this is what happens when you go get coffee | 15:47 |
| kyrofa | flexiondotorg, of course. Excellent | 15:47 |
| sergiusens | I was thinking of just biting the bullet and creating snapcraft quirks logic for stage-packages | 15:47 |
| flexiondotorg | sergiusens: Not quite. It works in classic but still needed to ALSA config work around. | 15:47 |
| kyrofa | flexiondotorg, can I help you with the remote part? I want to make sure we don't drop that | 15:48 |
| sergiusens | we already do it implicitly for other things, we should probably just make it an official thing | 15:48 |
| flexiondotorg | To work in strict I have to manually add `/dev/shm/* rwl,` to the AppArmor profile. | 15:48 |
| sergiusens | a remote part for now should be ok | 15:48 |
| kalikiana_ | flexiondotorg: that's what preload is fixing ;-) | 15:48 |
| sergiusens | flexiondotorg that is a bit broad though | 15:48 |
| flexiondotorg | Sadly, we couldn't get snapcraft-preload to work. | 15:49 |
| flexiondotorg | I dicussed with sergiusens and he has some ideas about what can be added to snapcraft-preload. | 15:49 |
| sergiusens | flexiondotorg I want to make a minimal "configurable" version of it so you do not unnecessarily have to preload everything you may not want to | 15:50 |
| kalikiana_ | sergiusens: so are my fixes still blocking on the rewrite? | 16:15 |
| kalikiana_ | or should the user of preload define stuff like /dev/shm or /tmp manually instead? | 16:16 |
| kalikiana_ | Might be worth opening a forum topic | 16:17 |
| * kalikiana_ wrapping for today | 16:21 | |
| zyga-ubuntu | kalikiana_: o/ | 16:24 |
| zyga-ubuntu | jdstrand: thank you for the review, have you had a good trip home? | 17:31 |
| === JoshStrobl|AFK is now known as JoshStrobl | ||
| mup | PR snapcraft#1581 opened: catkin plugin: support rosdep pip dependencies <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1581> | 17:33 |
| kyrofa | sergiusens, that one is the last bit for one roadmap item ^^ | 17:35 |
| sergiusens | kyrofa anything pending before that needs to go in? | 17:36 |
| mup | PR snapcraft#1420 closed: add new "no-wrapper" property to apps <enhancement> <Created by mvo5> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1420> | 17:36 |
| sergiusens | kalikiana_ if your preload fixes are already converted to c++ you should be fine and just requires a review | 17:37 |
| kyrofa | sergiusens, that has no dependencies | 17:37 |
| jdstrand | zyga-ubuntu: yes, it was uneventful. direct flight, home friday night. couldn't have gone better. thanks for asking :) | 17:37 |
| Son_Goku | jdstrand: and you didn't even wind up sick :) | 18:00 |
| jdstrand | well, knock on wood :) | 18:00 |
| zyga-ubuntu | Son_Goku: my weirdest patch to snapd *ever*: https://github.com/snapcore/snapd/pull/3958/commits/56aaca91a5d7e7aed076703186c729a3283d4c2c | 18:04 |
| mup | PR #3958: many: add support for /home on NFS <Created by zyga> <https://github.com/snapcore/snapd/pull/3958> | 18:04 |
| Son_Goku | wtf | 18:05 |
| Son_Goku | that... literally makes no sense as a patch | 18:05 |
| zyga-ubuntu | it makes the PR green :) | 18:05 |
| Son_Goku | also, you're really not going to like this... | 18:05 |
| zyga-ubuntu | the previous patch failed because "exportfs" is obviously a mis-spelling of "exports" | 18:05 |
| Son_Goku | rot13 isn't in /usr/games in Fedora | 18:05 |
| * Son_Goku grumbles | 18:05 | |
| zyga-ubuntu | Son_Goku: I know, I know, I just made this test to run on 16.04 | 18:05 |
| zyga-ubuntu | Son_Goku: I will expand it to do better and use python to do the rot13 | 18:06 |
| zyga-ubuntu | Son_Goku: I just wanted to push something quickly not and not re-design it today | 18:06 |
| Son_Goku | this is such a stupid PR | 18:06 |
| Son_Goku | for such a dumb reason | 18:06 |
| zyga-ubuntu | Son_Goku: no, I mean, the PR is nice | 18:06 |
| zyga-ubuntu | Son_Goku: but the patch that made it green is silly | 18:06 |
| zyga-ubuntu | (read the rest) | 18:06 |
| Son_Goku | err, the patch I mean | 18:06 |
| zyga-ubuntu | right, no disagreement tere | 18:07 |
| zyga-ubuntu | *there | 18:07 |
| Son_Goku | fixing /home on NFS is absolutely good | 18:07 |
| Son_Goku | and necessary | 18:07 |
| Son_Goku | zyga-ubuntu: https://github.com/SELinuxProject/cil | 18:08 |
| Son_Goku | this looks like it'd be useful for snapd | 18:08 |
| Son_Goku | as it has its own HLL | 18:08 |
| zyga-ubuntu | Son_Goku: if I have a moment next week I'll update this to work with cifs | 18:08 |
| zyga-ubuntu | thanks, I'll check it out | 18:08 |
| Son_Goku | err: https://github.com/SELinuxProject/cil/wiki | 18:09 |
| zyga-ubuntu | wow, a documented project | 18:10 |
| zyga-ubuntu | I wonder how that happens | 18:10 |
| zyga-ubuntu | are all developers a great tech writers? | 18:10 |
| zyga-ubuntu | are the docs written ahead of code? | 18:10 |
| Son_Goku | CIL docs were written before code | 18:11 |
| Son_Goku | the whole thing had a huge design process ahead of time | 18:11 |
| Son_Goku | it was explicitly designed to make writing SELinux policies and policy modules easier | 18:11 |
| Son_Goku | and allow developing custom languages on top | 18:11 |
| Son_Goku | and since snapd already has its own custom format, it would make sense to use CIL rather than the normal language (HLL) | 18:12 |
| zyga-ubuntu | that does look sensible, yes | 18:12 |
| Son_Goku | CIL stuff can be directly loaded by libselinux | 18:12 |
| Son_Goku | and semodule | 18:12 |
| * zyga-ubuntu sees lisp-like language | 18:12 | |
| Son_Goku | haha | 18:13 |
| Son_Goku | intermediate languages tend to look very LISPy | 18:13 |
| * zyga-ubuntu needs to run (not because of lisp, lisp/scheme are great) | 18:13 | |
| zyga-ubuntu | yep :) | 18:13 |
| zyga-ubuntu | I'll be back later | 18:13 |
| sergiusens | kyrofa heh, you changed the PR while I was looking it seemed ;-) | 18:25 |
| kyrofa | sergiusens, just the commit, no content. Forgot to ref the bug | 18:25 |
| mup | PR snapcraft#1582 opened: plugins: add ros2 boostrapper <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1582> | 18:40 |
| kyrofa | sergiusens, that one is ready as well ^^ | 18:45 |
| sergiusens | kyrofa do you have a 14.04 environment handy? | 19:00 |
| sergiusens | kyrofa almost done that review, got distracted by someone wrong on the internet | 19:00 |
| kyrofa | sergiusens, I'm afraid not, but I can make one. Who's wrong, let me at 'em | 19:00 |
| sergiusens | kyrofa lol, I am sort of joking fwiw ;-) | 19:01 |
| kyrofa | :P | 19:01 |
| nacc | (cough relatively trivial to setup a 14.04 env with either `lxc launch ubuntu:t` or `uvt-kvm sync release=trusty arch=amd64; uvt-kvm create trusty-vm release=trusty arch=amd64`, but I suspect you knew those :) | 19:04 |
| sergiusens | nacc it is a matter of having the correct bandwidth :-) | 19:14 |
| nacc | sergiusens: heh | 19:14 |
| sergiusens | diddledan does your corebird currently support wayland? | 19:16 |
| sergiusens | kyrofa added comments to the review, looks good, tell me what you think and will do another quick round | 19:41 |
| kyrofa | jdstrand, I'm getting "snap-confine has elevated permissions and is not confined but should be" on 14.04, using the hwe kernel. Am I missing something? | 19:43 |
| Son_Goku | kyrofa: that's probably because it's setuid | 19:44 |
| kyrofa | Son_Goku, well, that doesn't explain why it's not confined | 19:45 |
| Son_Goku | probably the AppArmor profile didn't apply | 19:45 |
| jdstrand | kyrofa: did you reboot into that kernel? | 19:46 |
| jdstrand | it is like Son_Goku said. | 19:46 |
| kyrofa | jdstrand, Yeah | 19:47 |
| Son_Goku | just because I don't use AppArmor doesn't mean I don't know how it works ;) | 19:47 |
| jdstrand | what does 'sudo aa-status|grep snap-confine' have to say? | 19:47 |
| jdstrand | kyrofa: ^ | 19:48 |
| kyrofa | jdstrand, https://pastebin.ubuntu.com/25662590/ | 19:51 |
| jdstrand | kyrofa: hmm, what about just 'sudo aa-status' | 19:52 |
| kyrofa | jdstrand, https://pastebin.ubuntu.com/25662599/ | 19:53 |
| jdstrand | kyrofa: and: snap version ; snap list ; cat /proc/version_signature | 19:54 |
| kyrofa | jdstrand, https://pastebin.ubuntu.com/25662608/ | 19:55 |
| jdstrand | kyrofa: did you set SNAP_REEXEC=0 somewhere? | 19:56 |
| kyrofa | jdstrand, no, this is a fresh install, nothing weird here | 19:57 |
| jdstrand | let me try in a vm | 19:58 |
| jdstrand | kyrofa: it worked here. just a simple 'sudo apt-get install snapd ; sudo reboot ; sudo snap install hello-world ; hello-world' | 20:04 |
| jdstrand | kyrofa: what is the output of 'dpkg -l|grep apparmor' | 20:05 |
| kyrofa | jdstrand, wait... I've rebooted several times now in order to play with VM networking settings, and all of a sudden things are working | 20:05 |
| kyrofa | As rebooted after I installed the core snap, so I have no explanation for this | 20:07 |
| jdstrand | kyrofa: so, that is only supposed to happen if the profile fails to load on a distro that supports apparmor | 20:07 |
| kyrofa | Hmm | 20:07 |
| jdstrand | so, the deb is 2.27.5~14.04 | 20:07 |
| jdstrand | but the core is 2.27.6 | 20:07 |
| jdstrand | idk | 20:08 |
| jdstrand | hard to say what it was since it is now working. it worked right away here | 20:08 |
| jdstrand | kyrofa: did you try to install the snap before rebooting? | 20:11 |
| kyrofa | jdstrand, yeah I did. But I installed from the netboot image with the hwe kernel | 20:11 |
| jdstrand | kyrofa: I don't know. I used a desktop install. maybe snapd is missing a dependency? iirc, apparmor_parser isn't in netinst (which uses ubuntu-minimal) | 20:18 |
| kyrofa | jdstrand, that's possible | 20:18 |
| jdstrand | I should've phrased that differently. apparmor isn't in netinst, iirc netinst uses that instead of ubuntu-standard | 20:19 |
| kyrofa | Uh. sergiusens have you ever seen pip-installed packages get chmod'd 700? | 20:59 |
| sergiusens | kyrofa nope | 21:00 |
| kyrofa | It's suddenly happening to me and I have no explanation as to why | 21:00 |
| kyrofa | I just tested this a few days ago... it's like pip updated from underneath me, but it hasn't been updated for a while | 21:06 |
| kyrofa | sergiusens, it looks like it's expected behavior from pip install --user | 21:09 |
| sergiusens | kyrofa new behavior? | 21:10 |
| * sergiusens needs to run and pick up his kid from day care | 21:10 | |
| kyrofa | No, I'm seeing stuff from 2013 about this. I'm so confused | 21:10 |
| sergiusens | kyrofa wait, check if we have any postprocessing in the python plugin itself | 21:11 |
| kyrofa | Haha, _fix_permissions, dangit | 21:12 |
| kyrofa | Well, easy fix at least | 21:13 |
| kyrofa | I still can't explain why my testing ever passed, but I'll let it slide | 21:17 |
| kyrofa | Maybe I tested as root for some odd reason | 21:28 |
| kyrofa | In a container, perhaps | 21:28 |
| kyrofa | sergiusens, alright, snapcraft#1581 is ready for another look | 21:35 |
| mup | PR snapcraft#1581: catkin plugin: support rosdep pip dependencies <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1581> | 21:35 |
| kyrofa | That was annoying | 21:36 |
| nacc | kyrofa: yeah, containers as effective root have made of my testing in the past invalid too | 21:52 |
| nacc | kyrofa: might be worth adding a local cloud-config snippet to ensure the ubuntu user has your ssh key and go that route | 21:52 |
| kyrofa | nacc, yeah I actually do that. The only way I would have tested like that would have been unusual and a mistake. But it's the only way I can explain it :P | 21:53 |
| nacc | kyrofa: ah ok | 21:53 |
| kyrofa | sergiusens, +1 from OSRF on snapcraft#1582 | 22:02 |
| mup | PR snapcraft#1582: plugins: add ros2 boostrapper <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1582> | 22:02 |
| kyrofa | niemeyer, https://forum.snapcraft.io/t/error-when-updating-snap-and-cleaning-old-revisions makes for a very sad experience in lxd :( | 23:15 |
| kyrofa | But only after one has time to get invested in it | 23:16 |
| niemeyer | kyrofa: Thanks for the ping.. let's find someone to have a look at this so zyga can focus on layouts | 23:18 |
| kyrofa | niemeyer, no problem, I wasn't sure if anyone else was familiar with snap-confine other than zyga or jdstrand | 23:19 |
| niemeyer | kyrofa: zyga worked a lot on it, but we can find someone else to have a look | 23:20 |
| kyrofa | niemeyer, excellent, thank you | 23:20 |
| niemeyer | kyrofa: np! | 23:37 |
| * niemeyer looks for dinner | 23:37 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!