/srv/irclogs.ubuntu.com/2017/03/24/#snappy.txt

mupBug #1650091 changed: console-conf and getty hang if password is not set and press many enters <Snappy:Fix Released> <https://launchpad.net/bugs/1650091>03:08
mupPR snapcraft#1220 closed: Release changelog for 2.28 <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1220>03:48
=== chihchun_afk is now known as chihchun
=== morphis_ is now known as morphis
zygagood morning07:48
mupPR snapd#3076 opened: cmd: disable the re-associate fix as requested by jdstrand <Created by zyga> <https://github.com/snapcore/snapd/pull/3076>07:53
morphisSon_Goku: I have the C part of snapd now building10:33
mupPR snapd#3077 opened: interfaces: convert systemd backend to new APIs <Created by zyga> <https://github.com/snapcore/snapd/pull/3077>10:39
abeatoogra_, where can I find the latest stable rpi3 image? I've flashed the daily one but not sure if it is working10:57
morphisabeato: http://cdimage.ubuntu.com/ubuntu-core/16/stable/current/ubuntu-core-16-pi3.img.xz10:59
abeatomorphis, thanks, seems more modern than last time I took a lookf11:01
morphisSon_Goku: and a few builds later: INFO: Done(/home/simon/rpmbuild/SRPMS/snapd-2.23.1-1.fc25.src.rpm) Config(default) 0 minutes 57 seconds11:11
Son_Gokuunvendored?11:12
morphisyes!11:12
Son_Gokuso you mockchain'd a build and it worked?11:13
morphisI installed a few local packages into the mock env, but yeah11:13
Son_Gokuwhat changes did you need to make to snapd.spec?11:14
morphisa few11:14
morphisI also had to update the tomb package as it was quite outdated11:15
morphisI will push in a bit11:15
morphisthings are still very rought11:15
Son_Gokupush to where?11:15
morphisgithub.com/morphis/snapd11:15
morphisSon_Goku: btw. https://bugzilla.redhat.com/show_bug.cgi?id=143557211:16
morphisSon_Goku: and zyga will be happy that we now see https://bugs.launchpad.net/snappy/+bug/1674193 on fedora too :-)11:17
mupBug #1674193: core snap's configuration hangs on debian|openSUSE <Snappy:In Progress by morphis> <snapd (Debian):New> <openSUSE:Fix Released by morphis> <https://launchpad.net/bugs/1674193>11:17
Son_Gokuwhat I'm concerned about is that I currently need nine patches to make snapd build11:17
Son_Gokuthe rate of churn in snapd git makes it impossible to keep up with out of tree patches easily, unless they are very minor11:18
morphisSon_Goku: yes I agree, but we can do this for now and I will work with zyga to get us drop all these11:18
morphisor just keep one or two11:19
Son_GokuI've already got modified packaging pulling in all the openSUSE patches11:19
morphisah great11:20
morphisSon_Goku: so you have snapd building with the vendorized tree?11:20
Son_Gokuno11:20
Son_Gokuit fails to link vendorized11:20
morphiswith a error that the linker doesn't know about relro?11:20
Son_Gokuyes11:21
morphisok, I've switched from %gobuild to a simple go build for this11:21
morphislooks like LDFLAGS carrys something which the go linker doesn't like11:22
Son_Gokuyeah, we enforce hardening11:22
morphisyeah11:22
morphisSon_Goku: https://github.com/morphis/snapd/tree/f/fedora-packaging11:24
Son_Gokuerrm11:25
Son_Gokuyour systemd units don't work11:25
morphisI know :-)11:25
morphisI said its kind of rough11:26
morphisit also puts things into /snap ..11:26
morphisdoesn't run any tests ..11:26
Son_Gokuwell, I did have a working build, but it wasn't something worth releasing into Fedora11:26
morphissure, but that are the next steps11:27
morphisSon_Goku: so in general, getting something in f25 is possible or only with an exception?11:28
Son_GokuI *could* technically get something released now, with a whole lot of FIXMEs11:29
morphiseven with my two packages not included?11:29
Son_Gokuwell, no11:29
Son_Gokunot without those11:29
Son_Gokuwell11:30
Son_GokuI could, because it's pending their inclusion11:30
morphisok11:30
morphisSon_Goku: then lets do a plan of what next things we take to get those remaining bits fixed so we can submit with a clean and working package11:32
Son_Gokuno hardening flags is a problem, though11:32
Son_Gokuhardened builds should be working11:32
Son_Gokuand I'd be even more concerned with snapd than normal go packages11:32
morphissure, lets collect those bits and get them fixed11:33
morphisSon_Goku: shared a doc with you11:35
morphisSon_Goku: feel free to add things I am missing11:39
Son_Gokumorphis: file a bug in rhbz to request an update for golang-github-go-tomb-tomb11:43
morphisSon_Goku: who has to do the update? we or the maintainer?11:43
Son_Gokuwell, the maintainer usually11:44
Son_Gokuyou should also request comaintainership: https://admin.fedoraproject.org/pkgdb/package/rpms/golang-github-go-tomb-tomb/11:44
Son_Gokuin fact, you should do that for all packages that are snapd deps11:44
Son_Gokuyou and zyga might also want to join the golang sig11:45
Son_Gokuthough apparently no page describing it exists :/11:45
Son_Gokuanyway,  jchaloup is the maintainer of the package, so you could also email him11:46
zygaSon_Goku: +1 good idea11:46
* zyga looks outside at the soaking rain 11:46
zyganot a great way to start spring11:47
Son_Gokuit's cold and rainy here, so spring already sucks11:47
Son_Gokumorphis: you really need to do your introduction in devel@11:48
morphisSon_Goku: yeah, I have that prepared here11:48
morphisSon_Goku, zyga: https://bugzilla.redhat.com/show_bug.cgi?id=143561611:48
mupPR snapd#3078 opened: tests: remove stale apt proxy leftover from cloud-init <Created by zyga> <https://github.com/snapcore/snapd/pull/3078>11:53
morphisSon_Goku: how much time do you have to work on those remaining things?12:08
=== hikiko is now known as hikiko|ln
Son_Gokumorphis: I can try to make as much time as I can, but unlike you guys, I'm doing this in my spare time12:15
Son_Gokuand I juggle a lot of other things too12:15
morphisSon_Goku: yeah, really appreciate this!12:15
Son_Gokuanyway, I approved your two golang package review requests12:15
morphisSon_Goku: so I can take us through all of these next week12:15
Son_Gokuand you're now sponsored into packagers group12:15
morphisawesome!12:16
Son_Gokulimb will approve them later today, most likely12:16
morphislimb?12:16
Son_GokuJon Ciesla12:17
Son_Gokuwell, apparently it's Gwyn now12:19
Son_Gokubut his FAS account name is limb and Gwyn often shows up on IRC as limb, too12:20
morphisah I see12:21
Son_Gokuoh boy, this is going to be rough :/12:21
Son_Gokuremembering to use the right pronouns12:21
mupPR snapd#3079 opened: osutil: add GetBootID <Created by zyga> <https://github.com/snapcore/snapd/pull/3079>12:26
morphisSon_Goku: so any particular item you want to work on or should I take them all for now?12:34
Son_Gokuyou take them all for now12:34
Son_GokuI'm about to push my pending changes to a git repo, one sec12:34
Son_Gokumorphis: would it be possible to release an "overlay tarball" that overlays the vendored stuff on top of the regular snappy sources?12:40
morphisSon_Goku: maybe, need to talk with mvogt about that12:41
morphishe currently generates the release tarballs by hand12:41
Son_Gokuif the "vendorizing" was a separate overlay tarball, that would make this easier for me12:43
morphisSon_Goku: which way?12:43
Son_Gokubecause then I can upload both sources and only use the vendorized code for EL712:43
Son_Gokuand I don't have to condition which tarball to use12:43
Son_Gokuthings like spectool cannot process conditional sources12:44
morphisI see, let me see if that is something we can do12:45
Son_GokuDebian packaging should also be able to leverage such a configuration easily, too12:46
mupPR snapd#3080 opened: interfaces: remove old API <Created by stolowski> <https://github.com/snapcore/snapd/pull/3080>12:51
Son_Gokumorphis: https://gitlab.com/Conan_Kudo/fedorapkgs-snapd/commit/d2cd7591b6f362f4828ed7bbf29fb3cf335b541d12:58
Son_GokuI exported fedora-dist-git and applied my in-progress packaging work there12:58
morphisSon_Goku: great12:58
=== hikiko|ln is now known as hikiko
=== kgunn is now known as Guest24973
Son_Gokumorphis: and here's the "pristine" one: https://github.com/Conan-Kudo/snapd/commit/494f96fa458d6c71d07d2520555e5fabf61a844513:03
zygaSon_Goku: wrt vendor tarball, yes, I'm sure13:04
morphisSon_Goku: thanks, will integrate those bits13:05
Son_Gokuzyga: overlay tarballs would be much better for me, since I can just conditionally install the second set of sources13:05
Son_Gokubut I can always pull them both and upload them in tandem13:05
zygaSon_Goku: we could also split the upstream release to two tarballs, pristine snapd and pristine vendor tree13:07
zygaSon_Goku: it's just a directory after all13:07
Son_Gokuzyga: that's what I meant13:07
Son_Gokuthe vendor tree would be an overlay tarball13:07
zygaSon_Goku: and then packaging for vendor/devendrized becomes easier13:07
zygayes, definitely, when michael is back (he's off resting today) we should propose that13:07
Son_Gokubecause right now, it's a pain to switch back and forth13:08
Son_Gokuespecially when debugging golang issues13:08
=== chihchun is now known as chihchun_afk
Son_Gokuzyga: but I have "building" sources now: https://gitlab.com/Conan_Kudo/fedorapkgs-snapd/blob/snapd-pkg-dev/snapd.spec13:10
Son_Gokubut only when building in vendorized mode13:10
zygaSon_Goku: that's great, I would like to see a vendorized build in CORP13:10
zygaCOPR13:10
zygaSon_Goku: in parallel with the devendorized build in the repository13:10
Son_Gokuwell, I can build vendorized builds now in copr if I wanted to13:11
morphisSon_Goku: lets put a vendorized build into copr now, then we have something people can play with if they want13:14
Son_Gokumorphis, maybe later today13:20
Son_Gokuas it is, I haven't even attempted to use the packages yet :/13:20
zygamorphis: does the package you have built work?13:20
zygamorphis: did you try it with some snaps?13:21
morphiszyga: yes, it works fully13:21
morphisnextcloud etc. comes up without problems13:21
morphiszyga: however its a bit rough, need to fix the systemd packages etc.13:21
morphisSon_Goku, zyga: but we could atleast push these things to https://copr.fedorainfracloud.org/coprs/zyga/snapcore/13:21
morphiszyga: my plan is to merge with Son_Goku's changes next and then we can have something testable and do any further cleanup from there13:22
Son_Gokumy version should also be using /var/lib/snapd/snap correctly, among other things13:22
morphisright13:22
morphisthat is why I need to merge :-)13:23
zygamorphis: rought in which way?13:23
renatujdstrand, could you approve this package? https://myapps.developer.ubuntu.com/dev/click-apps/6453/rev/8/13:23
zygamorphis: we have the presets now so snapd.socket should start ok13:23
zygamorphis: yes, I think we should push something to COPR13:23
jdstrandrenatu: I literally just did :)13:23
renatujdstrand, thanks :D13:23
morphiszyga: yes but the other jobs need patching which Son_Goku did13:23
zygamorphis, Son_Goku: if you guys have a package (vendorized, using /var/lib/snapd/snap) I would like to upload it there13:23
renatujdstrand, can we white list it ?13:23
jdstrandrenatu: I already did13:24
zygaaha, sounds good13:24
morphiszyga: lets target that for monday13:24
renatujdstrand, thanks again :D13:24
zygavery good idea13:24
jdstrandrenatu: check your email :)13:24
jdstrandnp13:24
zygareleases on Friday bring back luck13:24
morphisyeah13:24
Son_Gokuread-only friday!13:24
morphis:-D13:24
Son_Gokuanyway, that also gives us the weekend to get snappy deps updated in Fedora13:25
Son_Gokuwhich allows us to no longer vendorize13:25
morphis+113:25
morphisSon_Goku: but we wont get those into f25, right?13:25
Son_Gokuwhy wouldn't we?13:25
zygamorphis: why not, this is not debian13:26
morphishah!13:26
Son_Gokuwe're not debian or ubuntu13:26
Son_Gokuwe don't make life hard for people ;)13:26
morphisI am starting to like Fedora :-)13:26
Son_GokuFedora master race :)13:26
Son_Gokunew packages can enter all stable repos13:26
zyga:-)13:26
zygalet's try that hangout next week guys13:26
zygaonce this is released we could plan some next steps (CI)13:26
zygaand just enabling unit tests13:26
morphiszyga: +113:26
Son_Gokuwe really need to get this done asap13:27
Son_Gokupreferably before the F26 Alpha goes out13:27
morphisSon_Goku: is there a target date for the alpha?13:27
Son_GokuI'm getting tired of the buildsystem bitching at me about snapd-glib13:27
Son_Gokumorphis: week after next Tuesday13:28
zygaSon_Goku: oh, yes13:28
zygaSon_Goku: btw, I wonder if that is integrated to gnome-software13:28
Son_Gokuright now, no13:28
zygaSon_Goku: once you have snapd installed13:28
Son_Gokubecause gnome-software will FTBFS13:28
zygaaaaha13:28
zygamaybe there's a pkg-config test?13:28
zygaI think this was upstreamed some time ago13:28
Son_Gokugnome software < 3.24 doesn't use snapd-glib, so it works13:28
zygaok13:28
zygawell, litte by little :)13:28
Son_Gokuerr, < 3.2213:28
Son_Gokuso Fedora 25 won't get it13:29
Son_Gokubut we can get it for F2613:29
Son_GokuI just have to ask hughsie very nicely to turn it on once we update everything13:29
morphisSon_Goku: nice!13:29
morphisSon_Goku: I am fully on getting this done and having a target date is even better :-)13:29
Son_Gokuthe GNOME 3.24 megaupdate has landed in Fedora 26: https://bodhi.fedoraproject.org/updates/FEDORA-2017-d1ab69a39513:30
Son_Gokuwell, will land13:30
Son_Gokuit's pending13:30
Son_Gokuit's already in Rawhide13:30
zygaSon_Goku: I'm actually curious, I will try F26, I wanted to see how the new gnome feels like13:31
zygadid you try it?13:31
zygamorphis: how do you feel working with all the distros? :)13:32
Son_Gokuzyga: I actually planned on updating my secondary machine to F26 today13:32
morphiszyga: I am using them all through ssh so I don't see much of the user interface, but yeah, interesting :-)13:32
Son_GokuI always do it around alpha time13:32
zygamorphis: headless VMs?13:33
morphiszyga: not really headless yet but I ignore the vbox windows and have them minized all the time :-)13:34
morphisssh + sshfs integrates much more into my workflo13:34
jdstrandmorphis: hey, can you clarify this comment: "With snapd 2.23 now building on Fedora 25 I can reproduce the same problem without specifying --disable-seccomp for the snap-confine build." - you are saying that when seccomp is enabled on F25, you see the problem but when it isn't, you don't?13:37
morphisjdstrand: correct13:37
jdstrandok, so that is consistent with all other evidence pointing to mainline13:37
morphisjdstrand: so your idea is that something in the kernel part is broken?13:38
jdstrandI don't have any ideas. that is a reflection of Zygmunt's idea13:38
jdstrandI'm going to poke at this and try to provide some extra info for triage13:39
* jdstrand is only just now starting to look at it13:40
morphisjdstrand: ok, I wanted to look into this too but if you do than I can concentrate on getting the packaging stuff done13:40
jdstrandmorphis: yeah, let me poke at it for a bit13:41
zygamorphis: we should have a simple small smoke test13:44
zygamorphis: C program13:44
zygamorphis: using libseccomp13:44
zygamorphis: allowing bind13:44
zygamorphis: doing bind13:44
morphiszyga: +113:45
Son_Gokuzyga: ahh yep, I have some fixing to do13:47
Son_Gokunew snapd is still broken13:48
Son_Gokuthe selinux policy needs to account for the new binaries13:48
zygaSon_Goku: ah, thank you for fixing that13:48
Son_GokuI mean, I haven't done it yet...13:48
zygafixing is a continouous thing ;)13:49
Son_Gokuzyga: https://github.com/snapcore/snapd/pull/308113:53
mupPR snapd#3081: data/selinux: Add contexts for snapctl and ubuntu-core-launcher <Created by Conan-Kudo> <https://github.com/snapcore/snapd/pull/3081>13:53
mupPR snapd#3081 opened: data/selinux: Add contexts for snapctl and ubuntu-core-launcher <Created by Conan-Kudo> <https://github.com/snapcore/snapd/pull/3081>13:54
Son_Gokuzyga, morphis: that should fix issues related to snapctl, since it will have the access rights to snapd13:55
zygaSon_Goku: commented13:55
Son_Gokuzyga: removed ubuntu-core-launcher13:57
zygaSon_Goku: great, I'll merge it when CI goes green (I cannot earlier)13:58
Son_Gokuanyway, now I really need to get to work13:58
pshodother than using node-snapper14:27
pshodcan some1 tell me how do i make a snap application from a script written in node js14:28
pshod?14:28
pshodURGENT! HELP! HELP! HELP!14:28
zygapshod: hey14:33
zygapshod: try snapcraft :-)14:33
zygapshod: snapcraft.io14:33
pshodhey zyga :D14:35
pshodi really do not understand where my .js script goes with the nodejs plugin14:36
pshodso i switched to node-snapper14:36
zygapshod: I'm not a node developer but if you look at snapcraft you may find some examples of how to do stuff14:36
pshodbut would still want to do without it14:36
pshodlooked around a bit14:37
zygapshod: otherwise perhaps ask in https://rocket.ubuntu.com/channel/snapcraft14:37
pshodpeople happily use the node-snapper instead14:37
ogra_people shlouldnt :P14:38
ogra_node-snapper was for 15.04 snaps14:38
ogra_before snapcraft existed at all14:38
ogra_with the creation of snapcraft node-snapper became obsolete ... so i stopped maintaining it14:39
pshodogra :D14:41
pshodwhere do i put my .js script which should run on node14:43
pshodi can put the packages I need with node-packages tag14:43
pshodor do i need to write a package.json14:43
pshod?14:43
ogra_see snapcraft.io there should be examples14:43
pshodif i do14:43
pshodwhere do i put it14:43
pshodthere are not14:44
ogra_i havent packaged a node app since we have the new stuff14:44
pshod"new stuff" ?14:44
zygapshod: quick google: http://blog.bhdouglass.com/nodejs/snap/2016/08/06/packaging-nodejs-projects-as-snaps.html14:44
ogra_https://snapcraft.io/docs/reference/plugins/nodejs ... also has pointers to examples14:44
pshodzyga: saw this one14:44
zygapshod: what's wrong with it?14:44
pshodit has14:44
pshodbut the link doesnt actually have anything14:45
pshodgive me a min14:45
pshodzyga: so in the apps section, if i put my script path in the command tag, it will be automatically executed with node?14:46
zygano, but look at the first line of the script in the hello world example14:46
zygahttps://github.com/bhdouglass/hello-node-snap/blob/master/bin/hello-node-snap#L114:47
pshodhey thanks14:47
pshoddoing tat now14:47
mupPR snapd#3081 closed: data/selinux: add context definition for snapctl <Created by Conan-Kudo> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3081>14:47
pshodhe has listed the snap in the node-packages14:49
pshodand exposes the app in the command14:50
pshodI have 2 .js scripts, one is 'required" in another for using api's14:50
pshodthe .js file acting as an api provider requires other packages which i plan to put in the node-packages tag14:52
pshodsorry if all this sounds pretty noob14:52
pshodI AM :p14:52
pshodnode-packages: -hello-node-snap14:54
pshoddoes this translate to npm install hello-node-snap14:54
pshod?14:54
jdstrandmorphis, zyga, mwhudson: fyi https://bugs.launchpad.net/snappy/+bug/1674193/comments/1615:07
mupBug #1674193: core snap's configuration hangs on debian | openSUSE | mainline kernel <Snappy:In Progress by morphis> <snapd (Debian):New> <snapd (Fedora):In Progress> <snapd (openSUSE):Fix Released by morphis> <https://launchpad.net/bugs/1674193>15:07
morphisjdstrand: what wonders me is that I saw the bind syscall in the seccomp policy for the configure hook, see comment #315:09
jdstrandmorphis: I can try a vanilla kernel too15:10
morphisjdstrand: but do you saw bind in the policy for the configure hook too by default?15:11
jdstrandmorphis: no15:12
morphishm15:12
jdstrandonly mbind15:13
morphisI saw this with 2.23.115:13
jdstrandthis is 2.22.615:13
morphismaybe my fgrep was misleading, let me check15:13
morphishm, checked on fedora with 2.23.1 and no bind there15:14
morphisso looks like you're right15:14
jdstrandmorphis: there was a moment when the core snap plugged network-bind15:19
jdstrandbut it was reverted15:19
jdstrandwell, it isn't in r1441 anyway15:19
jdstrandidk why you saw it15:19
morphisjdstrand: hm, maybe that was what I was seeing yesterday15:20
morphisPharaoh_Atem: can I somehow avoid that I need to do something like this:15:34
morphis[simon@localhost fedorapkgs-snapd]$ cp *.patch ~/rpmbuild/SOURCES/15:34
morphisbefore I can run rpmbuild -bs snapd.spec?15:34
Pharaoh_Atemmorphis: yes, you can --define "_sourcedir </path/to/sources>" on the rpmbuild command line15:47
morphisah great15:47
morphisPharaoh_Atem: also where is this %gobuild macro defined?15:47
Pharaoh_Atemit's defined in /usr/lib/rpm/macros.d/macros.golang-go-compiler15:51
Pharaoh_Atemerr  /usr/lib/rpm/macros.d/macros.golang-compiler15:51
mupPR snapd#3082 opened: many: break the /aliases mutation API with a clean 400 <Blocked> <Created by pedronis> <https://github.com/snapcore/snapd/pull/3082>15:51
morphisPharaoh_Atem: ah, thanks!15:52
Pharaoh_Atemif you don't have it, do "dnf install 'compiler(go-compiler)'"15:52
morphisPharaoh_Atem: ok, looks like I've fixed the linker/relro issue15:58
mupPR snapd#3050 closed: interfaces/mount: compute mount changes required to transition mount profiles <Created by zyga> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3050>15:59
morphisPharaoh_Atem: yeah I did16:00
Pharaoh_Atemwhat was the fix?16:00
mupPR snapd#3067 closed: tests: move docker test to new nightly suite <Created by zyga> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3067>16:01
morphisPharaoh_Atem: switching the order of C and Go build in the spec file16:03
morphisthe C build of snap-confine somehow leaves LDFLAGS set which the go linker can't work with, if you build Go stuff first with %gobuild it isn't set and things build as they should16:04
morphisPharaoh_Atem: https://paste.ubuntu.com/24241472/16:04
Pharaoh_Atemthat's so retarded16:04
morphis:-)16:05
morphisPharaoh_Atem: I will prepare templating for the systemd units then we don't have to carry copies of them16:05
Pharaoh_Atemmy suggestion, use @libexecdir@ and @snapmountdir@ in place of /usr/libexec and /var/lib/snapd/snap, respectively16:06
morphisyeah16:06
morphisPharaoh_Atem: what do you mean with "Retire separate snap-confine package"?16:06
Pharaoh_Atemsnap-confine has been in Fedora since F2316:07
Pharaoh_Atemit's 1.0.43 I think16:07
pshodi am packaging a node js script into a snap app16:07
pshodsome of the packages listed under the node-packages tag cannot be downloaded proly cos of company's proxy16:08
Pharaoh_Atemmorphis: once we have a new snapd ready, then snap-confine needs to be retired16:08
morphisPharaoh_Atem: so the "Obsoletes:" in the spec file isn't enough?16:08
pshodso i first pull the part16:08
morphisPharaoh_Atem: ah there is a separate step we need to do for that16:08
Pharaoh_Atemmorphis: well, it is for users, but we need to get rid of it in the SCM :)16:08
morphisPharaoh_Atem: I see :-)16:08
Pharaoh_Atemso that the build system stops building the old snap-confine package16:08
pshodthen in /parts/part-name/install/lib/node-modules16:08
pshodi copy the packages i needed16:09
morphisPharaoh_Atem: I've reinstalled my package and got https://paste.ubuntu.com/24241487/ during the install16:09
morphislooks like its trying to adjust the existing squashfs mounts16:09
pshodthey also reflect in the prime folder16:09
pshodbut are not being found by the binary inside prime16:09
pshodwhich is exposed as an app16:09
morphispshod: really sounds like necessary env variables or so are not set up properly16:11
pshodummm16:11
pshodin the binary16:11
pshodwhich i have used as a .js script16:11
pshodi removed the extn of .js16:12
Pharaoh_Atemmorphis: that's normal16:12
pshodinstead added #!/usr/bin/env node16:12
pshodis that supposed to work?16:12
morphisPharaoh_Atem: so no way to teach the macros to skip those mounts16:12
Pharaoh_Atemmorphis: it's not the macros16:12
Pharaoh_Atemit's restorecon -Rv16:13
morphispshod: sadly I am not an expert in nodejs things .. if you don't find anyone else here the best is to drop a mail to the snapcraft mailinglist16:13
Pharaoh_Atemmorphis: ideally, it wouldn't matter as snapd would mount them all with that label16:14
pshodhey thank you!16:14
Pharaoh_Atemright now, it does not16:14
pshodmoreover i think if i get the proxy issue resolved this thing also might work16:14
pshodits just too late here for me to call someone at IT to resolve the proxy thing16:15
pshod:(16:15
pshodfuck corporate proxy16:15
morphisPharaoh_Atem: ok16:15
jdstrandmorphis: fyi, https://bugs.launchpad.net/snappy/+bug/1674193/comments/1816:16
mupBug #1674193: core snap's configuration hangs on debian | openSUSE | mainline kernel <Snappy:In Progress by morphis> <snapd (Debian):New> <snapd (Fedora):In Progress> <snapd (openSUSE):Fix Released by morphis> <https://launchpad.net/bugs/1674193>16:16
morphisjdstrand: ok, so things are clear and we need bind for every hook or the snapctl implementation fixed16:17
jdstrandmorphis: yes16:17
jdstrandI would like to be a part of any PR that addresses this16:18
jdstrandmorphis: ^16:18
morphisjdstrand: +116:18
zygapshod: hey, did you check out snapcraft chat room on rocket chat?16:19
pshodzyga: no16:27
pshodwill do16:28
pshodthanks for the info16:28
zygapshod: I think that will work better, there are losts of people making snaps there16:28
Pharaoh_Atemmorphis: pushed revised specs to my git repos16:29
* zyga loves to see morphis and Pharaoh_Atem do everything he wanted to get done :)16:31
* kyrofa too16:31
mupPR snapd#3083 opened: tests: move unity test to nightly suite <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3083>16:33
zygafirst fedora then the world ;D16:33
zygaI'd love to see snapd on osx and windows16:33
zygaI bought the original DOOM on GOG16:34
zygaI'll snap it this weekend16:34
zygaI wonder if GOG folks could offer snaps for linux16:34
zygaor even put them in the store somehow16:34
morphiszyga: :-D16:39
morphisPharaoh_Atem: awesome!16:39
morphisone step closer to a release16:39
morphisPharaoh_Atem: there will be one more patch for us to integrate to get the templating for the systemd units16:39
zygamorphis: we could actually release over weekend, people can check bodhi and try it and vote16:40
zygait's not like it's going live in seconds16:40
zygaPharaoh_Atem: what do you think?16:40
zygamorphis: if you think it is ready16:40
zyga(for wider testing)16:40
morphiszyga: https://docs.google.com/document/d/1l9xS8RqSSjASNEIcHAOanlURNrpmfodf4Fd79QXdLG4/edit16:40
zygaeven if it ends with -2 in the end16:40
zygamorphis: can you share that to make it public16:40
morphisdone16:41
zygaPharaoh_Atem: packaging question, can we recommend/depend on snapd-xdg-open from snapd on a workstation variant but not on a server variant?16:41
zygathanks!16:41
morphisPharaoh_Atem: https://github.com/snapcore/snapd/pull/308417:14
mupPR snapd#3084: packaging: use templates for relevant systemd units <Created by morphis> <https://github.com/snapcore/snapd/pull/3084>17:14
mupPR snapd#3084 opened: packaging: use templates for relevant systemd units <Created by morphis> <https://github.com/snapcore/snapd/pull/3084>17:15
kyrofajdstrand, is there an interface that will cover /dev/input/js0 ?17:38
ogra_kyrofa, most likely the joystick interface ;)17:39
ogra_(no, doesnt exist yet i think)17:39
kyrofaogra_, :P17:39
kyrofajdstrand, assuming it's not covered, think that'd be a pretty easy addition?17:40
Pharaoh_Atemmorphis: see my comments https://github.com/snapcore/snapd/pull/3084#pullrequestreview-2897260718:01
mupPR snapd#3084: packaging: use templates for relevant systemd units <Created by morphis> <https://github.com/snapcore/snapd/pull/3084>18:01
morphisPharaoh_Atem: aye18:02
morphisPharaoh_Atem: I have those bits also included in the rpm packaging now18:03
morphisbut will change to those suggested names18:03
morphisPharaoh_Atem: ok, a problem seems to be still there: $ krita18:08
morphisdropping privs did not work: No such file or directory18:09
morphisbut that is something for next week :-)18:09
jdstrandkyrofa: there isn't, no. I suspect that the interface would need more than just /dev/input/js* (eg, probably /dev/input/event*) and that is where things might get dicey, but, I'd be happy to review a PR18:09
kyrofajdstrand, I assume /dev/input/js* would be safe, but /dev/input/event* sounds a little more general-purpose18:10
jdstrandkyrofa: event* is where you can start doing nasty things18:11
jdstrandkyrofa: I guess you are snapping something that needs it. did you add '/dev/input/js0 rw,' and did it resolve all your denials?18:12
mupPR snapcraft#1221 opened: state: factor state bits out of meta <Created by josepht> <https://github.com/snapcore/snapcraft/pull/1221>18:12
kyrofajdstrand, I haven't tried yet, still putting the snap together18:12
kyrofajdstrand, but I'll try that. If that works, I assume that interface would be a slam dunk?18:13
jdstrandkyrofa: it would, yes18:13
kyrofajdstrand, alright, I'll be in touch18:13
jdstrandkyrofa: if you are writing it, model it after framebuffer (as opposed to say, camera)18:14
kyrofajdstrand, thanks for the hint! I'll make sure I touch bases before I write anything18:14
jdstrandkyrofa: and be aware of https://bugs.launchpad.net/snapd/+bug/1675738. I believe jhodapp's team will be fixing all the historical interfaces that aren't doing the right thing18:15
mupBug #1675738: OpenGL interface should udev tag all /dev/* files <snapd-interface> <snapd:New> <https://launchpad.net/bugs/1675738>18:15
jdstrandkyrofa: cool, np18:15
jhodappjdstrand, yeah, we'll be doing a 2 pass approach to it...first is to do a quick fix so we can get the fix in in time for snapd 2.2418:16
jhodappthen we'll do the comprehensive fix18:16
jdstrandjhodapp: great!18:17
kyrofaHuh, very interesting!18:17
mupPR snapd#3079 closed: osutil: add BootID <Created by zyga> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3079>19:58
NicolinoCurallihi all just a doubt: it seem that a gadget can't be upgraded during the lifetime of a supported board? I don't find anything for making it in a transational manner: do I think wrong?21:04
kyrofaNicolinoCuralli, I'd refer you to ogra_, but I suspect he's off for the day21:06
NicolinoCurallikyrofa: is it better to ask on the mailinglist?21:07
kyrofaNicolinoCuralli, yeah I suspect so21:08
kyrofaNicolinoCuralli, he's very responsive there21:08
NicolinoCurallikyrofa : thanks a lot :D21:08
kyrofaAny time!21:08
kyrofaHey niemeyer, can I not use a relative path for the gadget snap in a model assertion? Does it need to be in the store?21:20
kyrofaIt seems so. jdstrand can I get a review on a gadget snap?21:25
kyrofaI can't even upload a gadget to beta or edge, it seems21:33
kyrofaIt's really hard to make/test a gadget snap!21:34
jdstrandkyrofa: it was uploaded21:37
kyrofajdstrand, heh, release then21:38
kyrofaThanks jdstrand!21:39
jdstrandkyrofa: approved. I'll let you review21:39
jdstranderr21:39
jdstrandrelease21:39
kyrofajdstrand, I'll probably need to iterate on this-- will I need a manual review each time?21:40
jdstrandkyrofa: since we don't have model assertions, I need to add something to the review tools to override this. I'm doing that now, but it won't be in production for a while. until then, yes21:42
kyrofajdstrand, sounds good :)21:42
kyrofaThis might just work!21:42
jdstrandkyrofa: in what capacity is this being supported?21:42
jdstrandkyrofa: you have -kyrofa in the name, so guessing this isn't Canonical supported?21:43
kyrofajdstrand, nah, for a blog post series about the Turtlebot. I need the gadget due to bug #164544521:43
mupBug #1645445: Turtlebot needs /dev/kobuki <snapd-interface> <snapd:Confirmed> <https://launchpad.net/bugs/1645445>21:43
* jdstrand nods21:43
kyrofajdstrand, I'm a little sad no one will be able to follow it though. I was under the impression ubuntu-image would accept file paths21:44
jdstrandhotplugging will be nice21:44
kyrofajdstrand, yes that would be better :)21:44
jdstrandkyrofa: I suggest raising with stakeholders :)21:45
* jdstrand knows this comes up from time to time. squeaky wheel...21:45
kyrofaYeah, I was told to leave it alone and just write the series with a gadget snap21:46
jdstrandroadmr: totally not urgent, but can you please pull r852 whenever is convenient?21:53
jdstrandkyrofa: that is for you ^21:53
roadmrjdstrand: sure thing21:54
kyrofajdstrand, haha, quick work!21:54
kyrofajdstrand, thank you :)21:54
roadmrjdstrand: I should have that deployed early next week, but I'm starting by committing the change21:54
pedroniskyrofa: the model assertion take just a name, you can override it with a local snap using --extra-snaps21:57
kyrofapedronis, ah ha! I tried that, but I still used a path in the assertion21:58
kyrofapedronis, let me give that a shot, thanks for jumping in21:58
pedronis(that option name is very confusing, adding extra-snap is actually the thing we wantt the least, and might remove, but overriding with locall will stay supported)21:58
jdstrandroadmr: thanks :)21:59
kyrofaIndeed21:59
pedroniskyrofa: remember that local snaps will not refresh though , just like snap install .snap22:00
kyrofapedronis, indeed. Although I seem to remember that gadget snaps being refreshed isn't really a thing anyway?22:01
pedronisit gets refreshed22:01
pedroniswe don't do much with the refreshed content though22:01
pedronisthat's a bug, to fix22:01
kyrofaAh, right22:01
kyrofapedronis, there's a mailing list question for you to answer, then22:02
kyrofaSent about an hour ago22:02
pedronismostly that if it's a tutorial you might want to note on the side (I'm using a local snap because it's quick but for the full experience with refresh need a store one or something)22:02
kyrofapedronis, agreed, thanks for the tip22:03
kyrofapedronis, verified: leaving the assertion with just the name and using --extra-snaps works22:06
pedronisI wonder if our page about model assertion/ubuntu-image needs improvement, because I have been explaining this a couple of times22:06
pedronisI think22:06
kyrofapedronis, which page? All I know about is the tutorial22:07
kyrofaI mean on tutorials.ubuntu.com22:07
pedronisyes, it says this:  gadget: name of the gadget snap as published on the store. Note that this snap can be a file on disk.22:08
kyrofapedronis, yeah I read that as "this can be a file path on disk"22:08
pedronisyea, not super clear22:08
pedronisbut it really means what it says literally22:08
pedronishere goes the name (in snap.yaml/store) sense of the snap and the snap can be  a file22:09
pedronisbut then it doesn't says anywhere something about --extra-snaps22:09
kyrofaYeah I would say "name of the gadget snap as published on the store. If you want to use a snap from your disk, you'll need to supply its path via --extra-snaps" or something similar22:10
kyrofaAlthough "as published on the store" begs the question "what if my snap isn't in the store"22:11
pedroniswell name of the snap, is the name of the snap, what you put snapcraft.yaml and goes into snap.yaml22:12
pedronisbut not sure how ingrained is that in general22:13
kyrofapedronis, note that you can edit the snap name in the store though22:13
kyrofapedronis, so there's a few levels of confusion that can happen there22:13
pedronisI don't remember that22:13
pedronisit can be renamed22:14
kyrofapedronis, it doesn't rename it as far as snapd is concerned22:14
pedronisgiven that snap name need to be registered22:14
kyrofapedronis, it's just a presentation layer thing22:14
pedronismight be a left over from clicks22:14
pedronisor something22:14
kyrofaBut the option is "edit name"22:14
pedronissounds strange for snaps22:14
kyrofaYeah maybe22:14
jrwrenIs there any docs on how snaps actually run. I see a snap which runs on xenial, but fails on trusty, but I thought this would be impossible given the ubuntu-core is the same. How does this actually work?22:15
kyrofaHey jdstrand this serial port interface doesn't seem to be doing anything22:39
kyrofajdstrand, this specifically: https://github.com/kyrofa/pc-amd64-turtlebot-gadget/blob/master/snapcraft.yaml22:41
kyrofajdstrand, should the vendor and product actually be strings?22:42
totallyhumanhi22:49
totallyhumani have searched around for this error22:49
totallyhumanand there are several bugs filed22:49
totallyhumanwith no success22:49
totallyhumanerror: cannot list snaps: cannot communicate with server: Get http://localhost/v2/find?q=e: dial unix /run/snapd-snap.socket: connect: no such file or directory22:50

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!