| mup | PR snapd#5969 closed: apparmor: add unit test for probeAppArmorParser and simplify code <Created by mvo5> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/5969> | 01:04 |
|---|---|---|
| mup | PR snapcraft#2356 closed: nodejs plugin: add support for bases <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2356> | 01:27 |
| === chihchun_afk is now known as chihchun | ||
| mborzecki | morning | 05:00 |
| === chihchun is now known as chihchun_afk | ||
| zyga | Good morning | 06:15 |
| mborzecki | zyga: hey | 06:17 |
| zyga | Cold morning | 06:19 |
| mvo | hey zyga | 06:21 |
| zyga | Hey, how are you? | 06:22 |
| mvo | zyga: good, thank you - how are you? | 06:24 |
| mborzecki | mvo: morning | 06:25 |
| zyga | Good, last evening was fun :-) we loved the movie | 06:26 |
| mvo | hey mborzecki | 06:26 |
| mvo | lots of green tests today? | 06:26 |
| mborzecki | finally :) | 06:31 |
| mup | PR snapd#5948 closed: asserts, image: ensure kernel, gadget, base and required-snaps use valid snap names <Parallel installs ⛓> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5948> | 06:32 |
| mup | PR snapd#6003 closed: cmd/snap: block 'snap help <cmd> --all' <Simple 😃> <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6003> | 06:49 |
| mup | PR snapd#6005 closed: snap/pack, snap/squashfs: remove extra copy before mksquashfs <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6005> | 06:50 |
| mup | PR snapd#5999 closed: tests: ensure that goa-daemon is off <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5999> | 06:51 |
| slvn_ | Hey, my 10 snaps games are published for a while. Only two of them appear in the listing of snapcraft.io when searching in "Games", but appears in "All snaps". It seems Categories Game is not correctly detected. It happens also for many other Snaps. | 06:51 |
| zyga | Hey, slvn_ - I would recommend to ask in the four of you don’t get a reply here. It is still early and store people are in other time zones as well | 06:54 |
| mborzecki | slvn_: s/four/forum/ | 06:54 |
| zyga | But do come back if you don’t get a reply! Thank you for making snaps :-) | 06:55 |
| * zyga responds from a phone | 06:55 | |
| pstolowski | mornings | 06:56 |
| zyga | Dzień dobry | 06:57 |
| mup | PR snapd#5997 closed: ifacestate: simplify task chaining in ifacestate.Connect <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/5997> | 06:59 |
| slvn_ | thanks, ok! I'll repost in a few hours | 07:16 |
| mborzecki | pstolowski: cześć :) | 07:18 |
| zyga | guten tag mvo :) | 07:18 |
| zyga | (not to force one minority language upon others) | 07:18 |
| mborzecki | https://www.reddit.com/r/linux/comments/9ool14/xdgportal_support_landed_on_firefox_nightly/ | 07:20 |
| zyga | that's cool | 07:21 |
| mborzecki | Download (curl) error for 'http://download.opensuse.org/update/leap/42.3/oss/x86_64/libgcrypt20-32bit-1.6.1-45.1.x86_64.rpm': | 07:35 |
| mborzecki | heh | 07:35 |
| slvn_ | zyga, btw, who should be contacted for http://snapcraft.io issue ? | 07:40 |
| === sparkieg` is now known as sparkiegeek | ||
| sparkiegeek | slvn_: if you look in the footer, you'll see links to where to file bugs | 07:49 |
| mup | PR snapd#5979 closed: install: don't start disabled services <Squash-merge> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5979> | 08:01 |
| zyga | mvo: could you please review https://github.com/snapcore/snapd/pull/5727 - this is affecting Firefox in the wild and it would be good to have in 2.36 | 08:03 |
| mup | PR #5727: interfaces/builtin, cmd/snap-seccomp: Allow read-only ptrace, for the Breakpad crash reporter <Created by jld> <https://github.com/snapcore/snapd/pull/5727> | 08:03 |
| mvo | zyga: this is waiting for a security review, no? happy to do a review but it seems we need jamie, no? | 08:05 |
| zyga | yeah, I suppose you are right | 08:05 |
| zyga | https://github.com/snapcore/snapd/pull/6004 is green and I'd love to land it - it is just tweaking debug messages | 08:07 |
| mup | PR #6004: cmd/snap-confine: reduce verbosity of debug and error messages <Created by zyga> <https://github.com/snapcore/snapd/pull/6004> | 08:07 |
| * zyga goes to bring a heater into the office | 08:13 | |
| zyga | I miss the days of Pentium 4 heaters | 08:13 |
| sil2100 | mvo, zyga: hey, could you fill in a bug for the non-working wifi here if you have a moment? https://bugs.launchpad.net/snap-core18 | 08:28 |
| zyga | sure | 08:29 |
| zyga | wow | 08:29 |
| zyga | no bugs | 08:29 |
| zyga | must be some decent software! | 08:29 |
| mborzecki | Chipaca: what if we move static checks to a separate job? | 08:31 |
| sil2100 | (/me deletes all bugs after they pop up) | 08:31 |
| mvo | sil2100: added a bug (cc zyga) | 08:32 |
| mborzecki | Chipaca: unit tests already take ~8 minutes, so the stage is 8+ minutes anyway | 08:32 |
| zyga | oh | 08:32 |
| zyga | do you want both? :) | 08:32 |
| zyga | I just wrote mine but haven't sent it | 08:32 |
| zyga | well, you have two now | 08:32 |
| mvo | :) | 08:32 |
| mvo | sorry! | 08:32 |
| Chipaca | mborzecki: 1/8th is still a serious chunk in the wrong direction | 08:32 |
| Chipaca | mborzecki: static checks in a separate job would work, but I worry about using too many jobs | 08:33 |
| mborzecki | Chipaca: if you take the 1-2m off unit tests & static checks, and put it in a separate job, the total of the 1st stage should not be significantly affected | 08:33 |
| mborzecki | Chipaca: probably only way to know is to arrange it that way :) | 08:34 |
| Chipaca | mborzecki: I'll do it, and then we see :-) | 08:34 |
| mborzecki | Chipaca: ok | 08:34 |
| Chipaca | static checks shouldn't need the build-dep | 08:34 |
| mborzecki | hm we do run some go tools as static checks though | 08:35 |
| mborzecki | misspel, vet? | 08:35 |
| Chipaca | a lot | 08:35 |
| Chipaca | but go is there already | 08:35 |
| Chipaca | we need the build deps for the c libs | 08:35 |
| mborzecki | oh, btw. i had that branch with gometalinter a while ago | 08:35 |
| Chipaca | also we could move the travis-sie unit tests to 1.10 and it might be a little bit faster | 08:36 |
| Chipaca | travis-side* | 08:36 |
| Chipaca | and, yes i can install shellcheck from snap, will need to tweak the python to look there first | 08:37 |
| Chipaca | or fiddle with PATH | 08:37 |
| Chipaca | hm | 08:37 |
| mborzecki | you can set SHELLCHECK=/some/path for spread-shellcheck | 08:38 |
| mborzecki | uh, or not | 08:38 |
| Chipaca | :-) | 08:38 |
| mborzecki | ok, that should be a trivial change :P | 08:38 |
| mvo | sil2100: added one more for you in snap-core18 bugs :) | 08:38 |
| mborzecki | there's SHELLCHECK_SHELL which is kind of useless because spread does bash anywya | 08:39 |
| sil2100 | mvo: thanks! ;) | 08:39 |
| sil2100 | The more bugs the merrier, right?! | 08:39 |
| Chipaca | mborzecki: doesn't spread run with -u on also? | 08:40 |
| mup | PR snapd#6001 closed: interfaces: typo 'allows' for consistency with other ifaces <Simple 😃> <Created by zyga> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6001> | 08:41 |
| sil2100 | mvo, zyga: btw... core18 dailies! http://cdimage.ubuntu.com/ubuntu-core/18/pending/ | 08:41 |
| zyga | thanks pawel! | 08:41 |
| sil2100 | (didn't check if they actually work yet, since still sprinting) | 08:41 |
| zyga | I can check later today | 08:41 |
| mvo | sil2100: nice! build from what channel? | 08:44 |
| mvo | sil2100: good stuff in any case | 08:45 |
| sil2100 | mvo: edge by default as for 16 | 08:45 |
| mborzecki | Chipaca: yup, looks like -u is set | 08:50 |
| mvo | sil2100: interessting, thats a pretty high risk level. I wonder if we should reconsider this as the default (more so if that the default for uc16) | 08:50 |
| mvo | sil2100: but not now :) | 08:50 |
| sil2100 | It's a daily, it's like how we usually build regular images with PROPOSED enabled | 08:51 |
| sil2100 | But yeah, we can have a discussion about that, right now an edge-build is just what we need for testing purposes! | 08:52 |
| mvo | sil2100: ok, if it matches then thats ok I guess. edge is very edgy, probably more so than proposed. but not supper important right now | 08:52 |
| mvo | sil2100: +1 :) | 08:52 |
| sil2100 | Like the edge is edgy idea ;) | 08:52 |
| mvo | sil2100: in the (very) old days the term was "crack of the day" :P but of course for our stuff the CI is a huge and effective filter so that term does no longer make sense (still amusing though) | 08:53 |
| mvo | sil2100: anyway, great progress, thank you! | 08:53 |
| mvo | sil2100: does it also include detection of changes in the core18 packageset? or is that something that will come later? | 08:53 |
| alocer | i m having font problems. no fonts are availble when using colibri or flock-chat in snap. i m on fedora 28 selinux disabled. | 08:54 |
| alocer | tryed to mount the font directories as specified in mount-snap.fstab but not succeeded. | 08:55 |
| alocer | any idea | 08:55 |
| Chipaca | mborzecki: hehe, if I run the checks using apt-provided shellcheck on 16.04, that shellcheck has no -x option, so the lack of a stdout lock makes it look like it's had too many mushrooms | 09:00 |
| Chipaca | mborzecki: e.g., | 09:01 |
| Chipaca | el l =SHELL-NAeM CEO D E1 , C O D E2Sp..e c ify-- edxicalluedce=tC (ODbEa1s,hC,OsDh,Ek2.s.h ) | 09:01 |
| mvo | Chipaca: lol | 09:04 |
| zyga | mborzecki: thanks for the review, updated | 09:05 |
| mborzecki | Chipaca: haha | 09:10 |
| zyga | https://github.com/features/actions <- missing a "push to snap store" | 09:11 |
| niemeyer | Yo | 09:13 |
| Chipaca | Wa | 09:13 |
| niemeyer | zyga: Yeah, I thought the same thing when I saw it.. this opens up some nice possibilities | 09:13 |
| zyga | I don't know if this is powered by or simply integrated with hashicorp tech though | 09:23 |
| zyga | but something to look at for sure | 09:23 |
| zyga | Chipaca: did you see https://bugs.launchpad.net/snapd/+bug/1796125 | 09:24 |
| mup | Bug #1796125: services in a snap with dependencies aren't started in correct order on install <snapd:New> <https://launchpad.net/bugs/1796125> | 09:24 |
| mborzecki | anyone up for a review of #5959? | 09:25 |
| mup | PR #5959: systemd: extend Status() to work for socket and timer units <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5959> | 09:25 |
| zyga | looking | 09:25 |
| zyga | ah | 09:25 |
| zyga | I did :< | 09:25 |
| zyga | sorry | 09:25 |
| zyga | mborzecki: it has a conflict | 09:25 |
| Chipaca | I could really use reviews on #5957 | 09:27 |
| mup | PR #5957: overlord/snapshotstate/backend: fall back on sudo when no runuser <Snapshots 📸> <Created by chipaca> <https://github.com/snapcore/snapd/pull/5957> | 09:27 |
| mborzecki | zyga: didn't notice, thanks | 09:28 |
| mborzecki | and pushed | 09:29 |
| pstolowski | zyga: commented on ipc/helper pr | 09:31 |
| zyga | thanks | 09:31 |
| zyga | pstolowski: replied, thank you | 09:36 |
| niemeyer | zyga: I think it's completely unrelated to HashiCorp.. they just reused the HCL, which is a JSON-like format that is nicer to type by hand | 09:47 |
| zyga | ah, I didn't know that | 09:47 |
| mborzecki | zyga: Chipaca: left a note in #1796125, looks like this is a problem in systemd but i has gone unnoticed for years now, doesn't change the fact we should do something about that | 09:48 |
| niemeyer | It's a nice format.. | 09:48 |
| mup | Bug #1796125: services in a snap with dependencies aren't started in correct order on install <snapd:New> <https://launchpad.net/bugs/1796125> | 09:48 |
| alocer | just trying to figure out what fonts does and snap app use ? | 09:48 |
| mborzecki | zyga: Chipaca: btw. https://github.com/systemd/systemd/pull/7947 is fun to read | 09:49 |
| mup | PR systemd/systemd#7947: man: systemctl doesn't operate multiple units in a single transaction <documentation> <Created by johnlinp> <https://github.com/systemd/systemd/pull/7947> | 09:49 |
| mborzecki | fwiw https://lists.freedesktop.org/archives/systemd-devel/2018-January/040122.html is fun too | 09:52 |
| Chipaca | alocer: sorry, what's that again? | 09:52 |
| Chipaca | mborzecki: sounds like we need to work around the systemd bug by ordering things ourselves as well | 09:52 |
| Chipaca | mborzecki: zyga: either of you have a fedora 28 so you could look into reproducing alocer's issue? | 09:54 |
| mborzecki | Chipaca: either that or adding Requires/RequiredBy | 09:54 |
| Chipaca | mborzecki: requires/requiredby means you can't stop one without t'other | 09:54 |
| zyga | Chipaca: hmm? which issue is taht | 09:54 |
| mborzecki | Chipaca: yeah | 09:54 |
| Chipaca | alocer: what's the output of 'snap version'? | 09:54 |
| Chipaca | zyga: no fonts in colibri | 09:55 |
| mborzecki | zyga: fonts ^^ ~10:53 | 09:55 |
| zyga | mborzecki: wow, I don't see that message, can you paste it again | 09:55 |
| zyga | IRC WTH? | 09:55 |
| Chipaca | zyga: search for alocer? | 09:55 |
| zyga | Chipaca: I looked | 09:55 |
| Chipaca | zyga: IRC is fine, your async client is wacko | 09:56 |
| mborzecki | 10:55:20 alocer | any idea | 09:56 |
| Chipaca | <alocer> i m having font problems. no fonts are availble when using colibri or flock-chat in snap. i m on fedora 28 selinux disabled. | 09:56 |
| Chipaca | <alocer> tryed to mount the font directories as specified in mount-snap.fstab but not succeeded. | 09:56 |
| Chipaca | zyga: ^ | 09:56 |
| zyga | thanks | 09:56 |
| * zyga looks | 09:56 | |
| zyga | alocer: hold on please | 09:56 |
| zyga | alocer: so just install colibri and run it? | 09:56 |
| Chipaca | zyga: seriously it's not the first time i see irccloud mess up basic message delivery on you | 09:57 |
| zyga | alocer quit on ping timeout here | 09:59 |
| zyga | :/ | 09:59 |
| Chipaca | zyga: here as well | 09:59 |
| zyga | hmm https://www.irccloud.com/pastebin/uC4kEeDp/ | 10:05 |
| zyga | I see a gazillion fonts in /usr/share/fonts | 10:06 |
| zyga | when I do snap run --shell colibri I cannot get the app to start | 10:07 |
| zyga | desktophelpers quit | 10:07 |
| zyga | as I cannot reproduce the issue and I suspect the desktop-launch is at fault I would recommend someone that maintains it to look deeper | 10:08 |
| zyga | I can launch colibri by running the real command though | 10:09 |
| * zyga sighs that pastebinit is broken out of the box for several releases on fedora :/ | 10:11 | |
| zyga | willcooke: hey, do you know who maintains desktop-launch? | 10:12 |
| zyga | https://paste.ubuntu.com/p/bCfTD2k65M/ <- shellcheck on the desktop helpers | 10:12 |
| zyga | I didn't investigate what failed | 10:12 |
| willcooke | zyga, kenvandine | 10:12 |
| zyga | thanks! | 10:12 |
| willcooke | zyga, well, desktop team, but Ken is the main man | 10:13 |
| zyga | kenvandine: hey, could you give me a hand debugging app startup issue on fedora | 10:13 |
| zyga | I can run the app just fine manually in snap run --shell | 10:13 |
| zyga | but when invoked through desktop-launch it doesn't start at all | 10:13 |
| zyga | willcooke: super, I'll work with ken on this | 10:13 |
| willcooke | oSoMoN, are you able to offer any advice there too? ^ | 10:13 |
| oSoMoN | zyga, maybe I can help, how does it fail? | 10:16 |
| zyga | oSoMoN: prints nothing at all | 10:16 |
| zyga | let me see if I can massage it to run with set -x | 10:16 |
| zyga | oSoMoN: https://paste.ubuntu.com/p/cjHb2cGhMg/ | 10:19 |
| zyga | this is from snap run --shell | 10:19 |
| zyga | oSoMoN: if I just run ./colibri it does start | 10:19 |
| zyga | but fonts look wrong | 10:19 |
| zyga | and it shows this text: | 10:19 |
| zyga | colibri invoked without desktop helpers on fedora 28 https://www.irccloud.com/pastebin/XP18dwTR/ | 10:20 |
| oSoMoN | zyga, so it looks like it's going through the entire desktop-launch script fine, and then fails on the exec? | 10:23 |
| zyga | dunno | 10:23 |
| mborzecki | zyga: the fonts look a bit odd here too, a bit fuzzy i'd say | 10:24 |
| zyga | I see huge fonts | 10:24 |
| zyga | btw I have a hi-dpi setup | 10:24 |
| mborzecki | and the size is off too | 10:24 |
| oSoMoN | zyga, can you comment out all of the contents of desktop-launch but the last line (exec), and see if that works? | 10:24 |
| zyga | pstolowski: can you look at https://github.com/snapcore/snapd/pull/6004#discussion_r225866841 please | 10:25 |
| mup | PR #6004: cmd/snap-confine: reduce verbosity of debug and error messages <Created by zyga> <https://github.com/snapcore/snapd/pull/6004> | 10:25 |
| zyga | oSoMoN: sure | 10:25 |
| zyga | oSoMoN: done, what did you expect to happen? | 10:26 |
| zyga | I got https://paste.ubuntu.com/p/BbTqzyXmCt/ | 10:26 |
| pstolowski | zyga: replied | 10:26 |
| zyga | ++ cat /home/zyga/snap/colibri/8/.config/user-dirs.dirs.md5sum | 10:27 |
| zyga | + [[ 7b7b9530d5860d479bc2941ec3351c69 - != \7\b\7\b\9\5\3\0\d\5\8\6\0\d\4\7\9\b\c\2\9\4\1\e\c\3\3\5\1\c\6\9\ \ \- ]] | 10:27 |
| zyga | 10:27 | |
| zyga | this looks ... od? | 10:27 |
| zyga | odd | 10:27 |
| pstolowski | sorry for confusion ;) | 10:27 |
| zyga | thanks, will do :) | 10:27 |
| zyga | oSoMoN: I'n not an expert on the desktop stack but since this is a shell script and I am terrible with shell I started by running shellcheck on it, perhaps it would be good to look at the list and see if anything could be a factor | 10:28 |
| oSoMoN | zyga, I'm not a shell script expert either | 10:37 |
| oSoMoN | zyga, I meant to comment out everything except for the last line, looks like you did the opposite | 10:37 |
| zyga | wait, what? | 10:37 |
| zyga | so just exec? | 10:37 |
| zyga | if so then I tried that and then the application does start | 10:38 |
| zyga | so something set along the way makes it not start | 10:38 |
| mvo | niemeyer: could you please add github.com/snapcore/classic-snap to the list of things that mup watches? | 10:38 |
| niemeyer | mvo: Of course | 10:38 |
| mvo | thank you | 10:38 |
| mvo | https://github.com/snapcore/classic-snap/pull/20 is the PR that made me ask about it. reviews welcome :) | 10:39 |
| mup | PR classic-snap#20: port classic to UC18 <Created by mvo5> <https://github.com/snapcore/classic-snap/pull/20> | 10:39 |
| niemeyer | mvo: That's done | 10:41 |
| mvo | ta | 10:41 |
| niemeyer | Woah... | 10:43 |
| niemeyer | https://usercontent.irccloud-cdn.com/file/WHV6Qd8P/image.png | 10:43 |
| niemeyer | That makes the day... | 10:43 |
| mvo | niemeyer: yeah! Chipaca already used it to fix typos :) | 10:43 |
| niemeyer | Super cool | 10:43 |
| Chipaca | :-) | 10:44 |
| * Chipaca wonders if "you will be attributed in the commit" is satisfied by a commit log of "$you is a fool" | 10:45 | |
| zyga | mvo: reviewed | 10:45 |
| zyga | niemeyer: yeah, I saw that too, it's awesome to have that :) | 10:46 |
| zyga | perfect for typo and other things | 10:46 |
| Chipaca | mborzecki: snaps work on travis, but snapd isn't there by default. shellcheck on travis is 0.46, which is sufficiently new i think? | 10:46 |
| Chipaca | mborzecki: it's not 0.50, but 0.50 isn't even out yet :) | 10:46 |
| oSoMoN | zyga, can I easily reproduce in a fedora VM, or does this require some preliminary set-up? | 10:47 |
| zyga | oSoMoN: just install and run | 10:47 |
| oSoMoN | ok, I'll try that | 10:47 |
| zyga | thanks! let me know if I can help | 10:48 |
| oSoMoN | I'm downloading a fedora ISO, I'll let you know when I reproduce the issue and start investigating it | 10:49 |
| zyga | ok | 10:49 |
| mborzecki | Chipaca: any chance of getting a more recent version? iirc 0.5 i had here was more picky about out code than the version in 18.04 which is 0.4.6 | 10:53 |
| Chipaca | mborzecki: grmbl. ok. | 10:56 |
| Chipaca | indeed 0.4.6 not 0.46 | 10:56 |
| mborzecki | hm to do start the after/before services correctly then, we'd need to do the topo sort again | 11:00 |
| mborzecki | s/to do/to/ | 11:00 |
| mborzecki | wonder if we could make that sort generic, iirc zyga will need something like this for mounts | 11:01 |
| zyga | yes please | 11:01 |
| zyga | we shoud | 11:01 |
| zyga | lattice or whatever | 11:01 |
| Chipaca | mborzecki: remind me, do we validate before/afters? | 11:01 |
| Chipaca | for cycles, i mean | 11:01 |
| mborzecki | Chipaca: yes, we do the 'sorting' but we're more interested in finding cycles | 11:02 |
| Chipaca | ah ok | 11:02 |
| mborzecki | Chipaca: but the algo is there | 11:02 |
| mborzecki | iirc we're using kahn | 11:02 |
| Chipaca | hah, validateAppOrderCycles | 11:02 |
| Chipaca | "más claro, echale agua" | 11:02 |
| mborzecki | i added it so i should probably be the one to fix it now :) | 11:03 |
| Chipaca | mborzecki: it's a lovely little algo tho | 11:10 |
| Chipaca | _please_ can i have reviews of #5957 | 11:12 |
| mup | PR #5957: overlord/snapshotstate/backend: fall back on sudo when no runuser <Snapshots 📸> <Created by chipaca> <https://github.com/snapcore/snapd/pull/5957> | 11:12 |
| Chipaca | pedronis: what's #5712 waiting for? | 11:13 |
| mup | PR #5712: overlord: make InstallMany work like UpdateMany, issuing a single request to get candidates <Reviewed> <Created by pedronis> <https://github.com/snapcore/snapd/pull/5712> | 11:13 |
| pedronis | Chipaca: more tests and a refactoring | 11:14 |
| pedronis | Chipaca: related to the Validate*Flags helpers, if you remember I used them more uniformly now | 11:15 |
| Chipaca | pedronis: ah. Should it get marked Blocked? | 11:15 |
| Chipaca | otherwise a trigger-happy me might merge it :) | 11:15 |
| pedronis | Chipaca: well, it wouldn't hurt anything | 11:16 |
| pedronis | it would just need a follow up | 11:16 |
| Chipaca | hmm | 11:16 |
| pedronis | sorry, I always hope to have that 30mins | 11:16 |
| Chipaca | :-D | 11:16 |
| pedronis | to do the thing | 11:16 |
| pedronis | but hasn't happened yet | 11:16 |
| pedronis | if it's annoying I can also temporarely close it | 11:17 |
| zyga | Chipaca: done | 11:17 |
| Chipaca | pedronis: no planning to have 30 minutes around ubuntu release time | 11:17 |
| zyga | brb, coffee time | 11:17 |
| mup | PR classic-snap#21 opened: create: improve user information <Created by mvo5> <https://github.com/snapcore/classic-snap/pull/21> | 11:25 |
| mvo | yay, thanks mup! | 11:28 |
| zyga | mvo: done | 11:28 |
| mvo | ta | 11:29 |
| * mvo hugs zyga | 11:29 | |
| zyga | I didn't get the coffee | 11:30 |
| zyga | ok, really coffee | 11:32 |
| Chipaca | jdstrand: #5727 could use a once-over from you i think | 11:45 |
| mup | PR #5727: interfaces/builtin, cmd/snap-seccomp: Allow read-only ptrace, for the Breakpad crash reporter <Created by jld> <https://github.com/snapcore/snapd/pull/5727> | 11:45 |
| mborzecki | off to pick up the kids | 11:51 |
| zyga | re | 12:08 |
| zyga | coffee expanded into lunch | 12:08 |
| Chipaca | lunch? so early? | 12:09 |
| Chipaca | … oh wait | 12:09 |
| Chipaca | drat | 12:09 |
| Chipaca | i had a morning, once | 12:09 |
| Chipaca | zyga: should I do the rename I mention in #5957? | 12:11 |
| zyga | Chipaca: looking | 12:11 |
| mup | PR #5957: overlord/snapshotstate/backend: fall back on sudo when no runuser <Snapshots 📸> <Created by chipaca> <https://github.com/snapcore/snapd/pull/5957> | 12:11 |
| Chipaca | zyga: the 'unify' comment | 12:12 |
| zyga | thinking | 12:13 |
| zyga | I added an idea but really this is bike shed hue now :) | 12:13 |
| zyga | Chipaca: is 2PM early for lunch? | 12:13 |
| pstolowski | it's not | 12:19 |
| * pstolowski lunches | 12:19 | |
| axino | hi | 12:36 |
| axino | is there a "snap" command knob to tell me if an installed snap has "strict" or "classic" confinement ? | 12:37 |
| ijohnson | axino: `snap info snap-name --verbose` and under "notes" | 12:40 |
| axino | ijohnson: nice, thanks ! | 12:41 |
| niemeyer | Hey folks, we've got a conflict today in our standup, but would be nice to run that exercise for defining the features for the upcoming cycle.. should we/can we postpone the meeting today for an hour? | 12:42 |
| Chipaca | axino: if it's installed, 'snap list snap-name', if it's classic it says 'classic' in the Notes column | 12:48 |
| axino | Chipaca: ack, thanks | 12:49 |
| pstolowski | niemeyer: fine by me | 12:49 |
| mvo | niemeyer: works for me | 12:50 |
| niemeyer | Thanks, I've moved the calendar event | 12:56 |
| jdstrand | Chipaca: ack, yes | 12:57 |
| zyga | uh | 13:01 |
| zyga | did standup move? | 13:01 |
| zyga | ah, I see | 13:01 |
| * zyga gets back to coding | 13:01 | |
| Chipaca | jdstrand: thanks! :-) | 13:01 |
| mup | PR snapd#5959 closed: systemd: extend Status() to work for socket and timer units <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5959> | 13:09 |
| sergiusens | zyga: oSoMoN I wouldn't put too much effort in the shellcheck stuff, you might as well copy over from the extensions work (which has been shellchecked) https://github.com/snapcore/snapcraft/pull/2359/files | 13:10 |
| mup | PR snapcraft#2359: [WIP] extensions: add glib <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2359> | 13:10 |
| zyga | sergiusens: nice, thank you for the tip! | 13:10 |
| mup | PR snapd#6007 opened: tests/lib: add an @mozilla.com exception to the CLA checker <Simple 😃> <⛔ Blocked> <Created by chipaca> <https://github.com/snapcore/snapd/pull/6007> | 13:13 |
| mup | PR snapd#6008 opened: client, daemon, cmd/snap: indicate that services are socket/timer activated <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6008> | 13:24 |
| mborzecki | Chipaca: something you may want to look at ^^ | 13:24 |
| Chipaca | will do | 13:25 |
| mborzecki | it's passing the status of activators to the client, so now we can play with the ui all we want | 13:26 |
| mup | PR snapd#6009 opened: cmd: use relative file names in locking APIs <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6009> | 13:39 |
| mborzecki | i suppose `snap stop` `snap start` should also DTRT wrt after/before | 13:45 |
| Chipaca | mborzecki: as suppositions go, that's a solid one | 13:48 |
| mup | PR snapd#6010 opened: cmd/snap-discard-ns: add support for per-user mount namespaces <Created by zyga> <https://github.com/snapcore/snapd/pull/6010> | 13:48 |
| mborzecki | having 2 separate paths to call sysstemctl is so much fun :) | 13:49 |
| oSoMoN | sergiusens, I'm not planning on applying shellcheck fixes to the desktop helpers, but I want to understand why this doesn't work under fedora | 13:53 |
| sergiusens | oSoMoN: I will not get in your way there 🙂 | 13:54 |
| Chipaca | mborzecki: let's fix that | 13:57 |
| Chipaca | mborzecki: i forget why we have two :-) | 13:57 |
| zyga | new standup time :) | 13:57 |
| Chipaca | mborzecki: once upon a time, everything systemd went via systemd-the-package | 13:57 |
| Chipaca | meaning reimplementing things for runit or whatever was just writing a systemd-the-package-alike | 13:58 |
| zyga | mborzecki: can you please give a final vote on https://github.com/snapcore/snapd/pull/6004 | 13:58 |
| Chipaca | today it's probably less true :-D | 13:58 |
| mup | PR #6004: cmd/snap-confine: reduce verbosity of debug and error messages <Created by zyga> <https://github.com/snapcore/snapd/pull/6004> | 13:58 |
| mup | PR classic-snap#22 opened: make snapcraft.yaml architecture specific <Created by mvo5> <https://github.com/snapcore/classic-snap/pull/22> | 14:11 |
| niemeyer | pedronis: Yeah, that's the code we were just discussing.. some more digging and testing is needed | 16:55 |
| niemeyer | pedronis: Have a good one | 16:55 |
| niemeyer | Chipaca: So, back to media, sorry for the detail | 16:55 |
| niemeyer | derail | 16:55 |
| mup | PR snapcraft#2361 opened: schema: remove deprecated plugin pull and build-properties <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2361> | 16:56 |
| pstolowski | i'll dig in in the morning and see if we're missing normalization somewhere | 16:56 |
| Chipaca | niemeyer: … ? | 16:56 |
| niemeyer | Chipaca: Sorry... I suggest going forward with the plan and preserving the current shape of our API, having "icons" as an independent entry | 16:56 |
| zyga | re | 16:56 |
| zyga | somewhat related https://github.com/snapcore/snapd/pull/6012 | 16:56 |
| mup | PR #6012: interfaces: fix NormalizeInterfaceAttributes, add tests <Created by zyga> <https://github.com/snapcore/snapd/pull/6012> | 16:56 |
| zyga | Chipaca: ^ | 16:56 |
| mup | PR snapd#6012 opened: interfaces: fix NormalizeInterfaceAttributes, add tests <Created by zyga> <https://github.com/snapcore/snapd/pull/6012> | 16:56 |
| cachio | mvo, 2.35.4 is stable now | 16:56 |
| cachio | mvo, smoke test in progress | 16:56 |
| Chipaca | niemeyer: wait, what for? | 16:56 |
| niemeyer | Chipaca: We can also add banners when we have a clear idea of how they look like, but we should clarify the discrepancies that exist between the spec and the dev API spec | 16:56 |
| Chipaca | niemeyer: there's never more than one icon AFAIK | 16:56 |
| Chipaca | niemeyer: is there going to be? | 16:56 |
| niemeyer | Chipaca: Oh my.. I thought we were missing it.. so that's *also* duplciate | 16:56 |
| niemeyer | duplicated | 16:56 |
| niemeyer | Chipaca: We just the banner then.. I wonder if we need more htan one | 16:56 |
| niemeyer | Chipaca: Either way, we should clarify if it's the dev API doc or the spec that is right | 16:56 |
| niemeyer | Chipaca: The dev API doc mentions "key", and "filename" | 16:56 |
| Chipaca | niemeyer: isn't the dev api the one snapcraft talks? | 16:56 |
| niemeyer | Chipaca: The comment is likely just bogus.. it mentions a filename, and provides an example without a URL | 16:56 |
| niemeyer | Chipaca: Well, or maybe not.. it's indeed storing the content, in which case it's the "url" parameter in the API that is unnecessary | 16:56 |
| mup | PR snapd#6013 opened: Revert "snap, client, daemon, store: use and expose "media" more (#5906)" <Created by chipaca> <https://github.com/snapcore/snapd/pull/6013> | 16:56 |
| Chipaca | niemeyer: I'm sure your statements make sense in the context of whatever it is you're looking at, but without that context I'm lost | 16:56 |
| zyga | mvo: if still around please look at 6012 | 16:56 |
| niemeyer | Chipaca: https://dashboard.snapcraft.io/docs/api/snap.html#get--dev-api-snaps-(snap_id)-binary-metadata | 16:56 |
| zyga | I had a look at how Normalize is used as well | 16:56 |
| Chipaca | niemeyer: that's not an api we talk to, is it? | 16:56 |
| niemeyer | Chipaca: Yeah, it's somewhat irrelevant, other than for understanding how we get what we get. On our end the problem is simpler.. we just need to support banners, and banner icon... the one question we need answered still is what's the use for those values.. can we have multiple banners, and why | 16:56 |
| niemeyer | Chipaca: Same thing for banner icon | 16:56 |
| Chipaca | a confirmation over icon being single would be nice | 16:56 |
| Chipaca | as well | 16:56 |
| Chipaca | niemeyer: in any case, #6013 reverts #5906 | 16:56 |
| mup | PR #6013: Revert "snap, client, daemon, store: use and expose "media" more (#5906)" <Created by chipaca> <https://github.com/snapcore/snapd/pull/6013> | 16:56 |
| mup | PR #5906: snap, client, daemon, store: use and expose "media" more <Squash-merge> <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/5906> | 16:56 |
| niemeyer | Chipaca: The obvious reasons for having multiple icons is format and size | 16:56 |
| Chipaca | niemeyer: pending confirmation of these things, a related question: should I make client standalone, and could I make snap use client instead of viceversa | 16:56 |
| Chipaca | and if yes, should I do that after the 1.10 move so we can use aliases | 16:56 |
| niemeyer | Chipaca: I think we had that agreement at some point, to avoid duplication by having the rest of the code importing client, which is okay. I can't remember if we ever discussed reasons for breaking that assumption with the "snap" package.. | 16:56 |
| Chipaca | niemeyer: I think we just never got to it. It's a fair amount of work. | 16:56 |
| niemeyer | Chipaca: There are only two imports of "snap" inside the client, but that doesn't change what you just said | 16:57 |
| Chipaca | heh | 16:57 |
| Chipaca | i mean, it's changing snap.Revision to client.Revision everywhere | 16:58 |
| Chipaca | that sort of thing | 16:58 |
| niemeyer | Chipaca: The alternative would be defining "snap" to be part of the client too | 16:58 |
| niemeyer | Chipaca: Which is reasonable to a degree.. | 16:58 |
| Chipaca | client/snap/... | 16:58 |
| niemeyer | Chipaca: I mean just semantically, not necessarily the location itself | 16:58 |
| Chipaca | niemeyer: i got that, but we can have both | 16:59 |
| Chipaca | bah, i dunno | 16:59 |
| tomwardill | Hi, just to let you know the store is responding slowly / erroring, on both api and dashboard. We're working on it :) | 16:59 |
| Chipaca | and it's too late in the day to go poke | 16:59 |
| niemeyer | Chipaca: We have a bunch of *util packages in a similar category.. wouldn't be nice to have client/<everything> | 16:59 |
| niemeyer | tomwardill: Thanks for the note | 16:59 |
| niemeyer | Chipaca: And agreed, a bit late | 17:00 |
| zyga | tomwardill: thank you for the notice! | 17:00 |
| Chipaca | niemeyer: but there are things in snap that aren't client, so splitting it might be a good exercise | 17:00 |
| mvo | zyga: \o/ you rock | 17:07 |
| zyga | mvo: I don't know if that is the problem though | 17:10 |
| zyga | I did not reproduce it | 17:10 |
| zyga | Chipaca: back now, sorry, the young one wanted food | 17:11 |
| mvo | zyga: it probably is not but still nice | 17:11 |
| zyga | Chipaca: reviewed https://github.com/snapcore/snapd/pull/5957#pullrequestreview-165744410 | 17:20 |
| mup | PR #5957: overlord/snapshotstate/backend: fall back on sudo when no runuser <Snapshots 📸> <Created by chipaca> <https://github.com/snapcore/snapd/pull/5957> | 17:20 |
| jdstrand | roadmr: hi! can you pull r1141 of the review-tools? | 17:21 |
| zyga | hey jdstrand | 17:21 |
| roadmr | jdstrand: sure, will do! | 17:21 |
| zyga | sorry for not doing more reviews | 17:21 |
| jdstrand | roadmr: it would be nice if it was in before SLC, but not mandatory | 17:21 |
| zyga | I'll try to tomorrow, I'm just about to wrap it | 17:21 |
| jdstrand | zyga: hey :) no worries. belive, I know how that can happen | 17:22 |
| zyga | jdstrand: I sent one interesting PR but not security critical | 17:22 |
| jdstrand | believe me* | 17:22 |
| zyga | still I wrote it as tightly as I could | 17:22 |
| zyga | jdstrand: do you remember MNT_DETACH and unmount | 17:24 |
| zyga | jdstrand: and the case that if a dentry is mounted in one mount namespace it is EBUSY to remove it in another | 17:24 |
| zyga | jdstrand: that got fixed in the kernel since | 17:24 |
| zyga | jdstrand: it's not fixed on CentOS or RHEL ... :-( | 17:24 |
| zyga | jdstrand: we very much rely on that so bummer | 17:24 |
| jdstrand | well, still progress | 17:25 |
| roadmr | jdstrand: I'll do my best :) | 17:27 |
| Chipaca | niemeyer: client uses store uses overlord/auth btw | 17:29 |
| * zyga EODs | 17:32 | |
| niemeyer | Chipaca: /o\ | 17:36 |
| niemeyer | zyga: Have a good night | 17:36 |
| jdstrand | roadmr: thanks :) | 17:37 |
| Chipaca | niemeyer: http://r.chipaca.com/snap.png | 17:38 |
| Chipaca | niemeyer: http://r.chipaca.com/client.png | 17:38 |
| Chipaca | niemeyer: osutil is orange just because it's cgo | 17:39 |
| mup | PR snapd#6014 opened: ifstate: fix decoding of json numbers <Created by mvo5> <https://github.com/snapcore/snapd/pull/6014> | 17:40 |
| Chipaca | updated those pngs with splines | 17:40 |
| * zyga sends gentle reminder for someone to look at https://bugs.launchpad.net/snapd/+bug/1796125 | 17:43 | |
| cachio | mvo, https://paste.ubuntu.com/p/Sh4Zf7TyVc/ | 17:43 |
| zyga | and really goes to play | 17:43 |
| mup | Bug #1796125: services in a snap with dependencies aren't started in correct order on install <snapd:New> <https://launchpad.net/bugs/1796125> | 17:43 |
| cachio | mvo, this is the issue what we see snapd.socket failing | 17:44 |
| === leftyfb_ is now known as leftyfb | ||
| cachio | mvo, sorry, this error is caused by systemd | 17:46 |
| zyga | mvo: reviewed your pr | 18:11 |
| zyga | and I'm gone for real | 18:11 |
| mvo | zyga: thank you! and totally agree, we may need to think about this in a deeper level :) enjoy your evening! | 18:21 |
| cachio | mvo, this is the error | 18:32 |
| cachio | https://paste.ubuntu.com/p/KQjX3SYNXz/ | 18:32 |
| cachio | at least one error easy to reproduce | 18:32 |
| cachio | seem to be a race | 18:32 |
| * cachio afk | 19:01 | |
| zyga | kenvandine: hey | 19:15 |
| zyga | kenvandine: just FYI there's an issue with desktop-launcher on Fedora | 19:15 |
| kenvandine | zyga: what kind of issue? | 19:16 |
| kenvandine | i tested the GNOME snaps recently on fedora | 19:16 |
| zyga | kenvandine: calibri doesn't start with it | 19:16 |
| zyga | I spent some time debugging | 19:16 |
| zyga | if the launcher runs and sets up stuff the app doesn't start, just quits (I didn't strace it) | 19:17 |
| zyga | if I run the app directly it runs but some stuff is wonky (fonts) | 19:17 |
| zyga | kenvandine: oSoMoN also looked at it I believe | 19:17 |
| zyga | kenvandine: the script is full of shellcheck warnings, not sure if related | 19:17 |
| kenvandine | zyga: do other snaps work? | 19:17 |
| kenvandine | like gnome-calculator? | 19:17 |
| zyga | kenvandine: I haven't tried many snaps, let me look | 19:17 |
| kenvandine | zyga: those are just places where we could quote variables | 19:18 |
| kenvandine | but shouldn't effect this | 19:18 |
| zyga | kenvandine: perhaps but maybe there are real bugs in the noise | 19:18 |
| kenvandine | perhaps | 19:18 |
| zyga | installing now | 19:19 |
| zyga | kenvandine: it runs | 19:20 |
| kenvandine | ok, interesting | 19:21 |
| zyga | kenvandine: I'll leave the rest to you though, it's far too late for work now :) | 19:21 |
| kenvandine | i can't debug right now, working on release stuff | 19:21 |
| zyga | ack :) | 19:21 |
| kenvandine | zyga: i'll look at it soon though | 19:22 |
| zyga | snapd gets killed on fedora ... | 19:28 |
| zyga | snapd gets killed on fedora by systemd https://www.irccloud.com/pastebin/3VlZmBvX/ | 19:28 |
| oSoMoN | kenvandine : I managed to observe the issue described by zyga in a fedora VM, but I'm afraid I didn't go much further… chromium runs, and I've compared the desktop-launch scripts for chromium and colibri, the one for colibri appears to be a much older version, I wonder if rebuilding with a recent version would make things better? | 19:54 |
| kenvandine | maybe | 19:55 |
| mup | PR snapcraft#2362 opened: python plugin: add support for bases <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2362> | 20:01 |
| oSoMoN | hrmmm, snapcraft is not available in fedora, and installing classic snaps doesn't appear to be supported :/ | 20:04 |
| kenvandine | oSoMoN: nope | 20:10 |
| oSoMoN | kenvandine, I'm rebuilding the snap with a more recent desktop-launch script and will test in my fedora VM, will let you know how that goes | 20:11 |
| oSoMoN | kenvandine, zyga: a more recent desktop-launch script didn't change anything, the snap still won't run | 20:25 |
| oSoMoN | I wonder what's so special about that snap | 20:26 |
| koala_man | are there any snaps written in Haskell that I can look at? I can make my build work locally with 'snapcraft cleanbuild' but not on snapcraft.io | 20:31 |
| oSoMoN | kenvandine, zyga: I'm going offline and won't be around until Monday, if there hasn't been progress on that issue by then I can pick it up and continue investigatingq | 20:54 |
| mup | PR snapcraft#2361 closed: schema: remove deprecated plugin pull and build-properties <Created by sergiusens> <Merged by kyrofa> <https://github.com/snapcore/snapcraft/pull/2361> | 21:50 |
| mup | PR snapcraft#2363 opened: {make,cmake,autotools} plugin: add support for bases <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2363> | 23:14 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
