/srv/irclogs.ubuntu.com/2017/11/01/#snappy.txt

=== chihchun_afk is now known as chihchun
=== ikey is now known as ikey|zzz
elopiosnappy-m-o autopkgtest 1647 xenial:armhf xenial:arm6404:59
snappy-m-oelopio: I've just triggered your test.04:59
mupPR snapcraft#1648 opened: sources: get svn revision with --show-item flag <Created by clobrano> <https://github.com/snapcore/snapcraft/pull/1648>08:04
=== deadk is now known as e
=== JoshStrobl|zzz is now known as JoshStrobl
mupPR snapd#4110 opened: many:  have a timestamp on store assertions <Created by pedronis> <https://github.com/snapcore/snapd/pull/4110>09:55
mupPR snapd#4111 opened: many: make ignore-validation sticky and send the flag with refresh requests <Created by pedronis> <https://github.com/snapcore/snapd/pull/4111>10:10
* ogra_ really likes the new animations for download and install 10:10
* Chipaca is happy that ogra_ likes it10:19
pedronisChipaca: hi10:29
pedronisChipaca: #4111 might be something you could review when you have time10:30
mupPR #4111: many: make ignore-validation sticky and send the flag with refresh requests <Created by pedronis> <https://github.com/snapcore/snapd/pull/4111>10:30
Chipacapedronis: already looking at it yes10:35
Chipacapedronis: we still have no answer for how to turn off these flags :-(10:36
Chipacabeyond reinstalling10:36
pedronisChipaca: snap refresh foo10:36
pedronisthat's the answer Gustavo gave10:36
pedronisChipaca: : https://forum.snapcraft.io/t/make-ignore-validation-sticky-and-send-the-flag-over/2139/210:37
Chipacahmmm10:37
Chipacawe're probably not doing the right thing today in a couple of places then10:38
pedronisthat's also the behavior implemented by the branch10:38
pedronisand tested10:38
pedronisat least for this flag10:38
pedronisChipaca: notice that at least for channel there's special code to do something even if there's no refresh (though for channel we also have switch)10:38
pedronisadding similar code for this is one of the TODOs10:39
pedronisChipaca: I'm talking about this:  https://github.com/snapcore/snapd/pull/4111/files#diff-4f584006240926759b614bfd921c5fc0R93810:40
mupPR #4111: many: make ignore-validation sticky and send the flag with refresh requests <Created by pedronis> <https://github.com/snapcore/snapd/pull/4111>10:40
pedronisChipaca: so I think I'm following the spec afaiu10:41
Chipacapedronis: yeah, that 'do something in refresh when no update' is the missing bit i guess?10:44
pedronisyes10:45
pedronisbut it's annoying enough that I prefer to do it in a separate PR10:45
pedronis(also we leaved without it for channels for a long while)10:45
pedroniss/leaved/lived/10:45
Chipacapedronis: OTOH maybe it's an argument to make switch do this instead of refresh, when no update10:45
pedronisfor channels we do both10:46
pedronisalso I fear switch as designed is too channel specific10:46
pedronisto involve it in this10:46
pedronisanyway I personally don't see any of that as blocker to the main change10:47
Chipacapedronis: why is switch too channel-specific?10:47
pedronissnap switch -h10:47
pedronisswitch switches channel10:48
pedronisif we want it to do other flag toggling I'm not sure the name is great or it is easy to explain10:49
Chipacapedronis: i thought you meant it would feel unnatural to use it for this, not that we'd have to change the documentation10:49
ChipacaOTOH it's true that the "strip devmode" thing doesn't work in switch10:49
Chipacai.e. it'd have to be 'snap switch --disable-devmode thesnap'10:49
Chipacaor somesuch10:49
pedronisyea, it's not an nice UX afaict10:49
pedroniswell, mostly is not a discoverable UX10:50
pedronisanyway I didn't do any of this in this PR because it has huge bikeshedding potential as you see10:51
pedronisChipaca: I'm getting this from prepare in a test run (on 14.04):  + chattr -i /var/cache/snapd/names10:57
pedronischattr: No such file or directory while trying to stat /var/cache/snapd/names10:57
mlwHas anyone here had an issue with cmake "finding" libGLEW.so in the wrong place. Having an issue with snapcraft unable to find it after building. Cmake config says it finds GLEW in /usr/include11:09
mlwCould anyone point me in the right direction?11:09
mlwI have the libglew-dev in my build-packages.11:10
diddledanthe cmake snapcraft plugin currently doesn't add <stage>/usr/local/lib{,/x86_64-linux-gnu} to the CMAKE_LIBRARY_PATH variable, and because the var is set in the environment it isn't possible to override using -DCMAKE_LIBRARY_PATH in configflags:11:48
diddledansame for CMAKE_INCLUDE_PATH and <stage>/usr/local/include11:51
sergiusensdavidcalle hey, not sure how this happened, but the oldest doc in the world was revived here https://docs.snapcraft.io/build-snaps/plugins11:52
diddledanwhy does that page have a blurb at the bottom about yocto being owned by linux foundation? I mean what relevance is there?11:53
diddledansergiusens: ^^^11:54
diddledanlooks like it's on all the snapcraft.io pages11:55
sergiusensdiddledan I don't know, I feel like I woke up in the twilight zone11:55
mlwdiddledan: so I'm stuck?12:31
mlwthe error I get reads12:32
mlwCMake Error at cmake_install.cmake:36 (file):12:32
mlw   file INSTALL cannot find12:32
mlw   "/home/mattias/djv/parts/cmake-build/src/$CMAKE_PREFIX_PATH:/home/mattias/stage/lib/libGLEW.so".12:32
diddledanmlw: my comment was unrelated12:33
diddledanmlw: I was mentioning my own problem12:33
mlwdiddledan: ah, ignore me :)12:34
diddledanmlw: although it does look like your problem might be similar12:35
mupPR snapcraft#1649 opened: add stage/usr/local/lib to cmake library path <Created by diddledan> <https://github.com/snapcore/snapcraft/pull/1649>12:35
sergiusensstgraber jdstrand any ideas about this? "Setup snap "lxd" (4778) security profiles (cannot setup seccomp for snap "lxd": " on a travis run today -> https://pastebin.ubuntu.com/25864977/12:39
* sergiusens goes for a walk12:43
jdstrandfatal error: runtime: address space conflict'13:14
jdstranderr13:14
jdstrandsergiusens: I don't know. I just tried on xenial with edge core and edge lxd and it worked fine. this seems to go a golang thing due to 'runtime: address space conflict: map(0xc820000000) = 0x7fc72ab2400013:14
jdstrandfatal error: runtime: address space conflict'13:14
jdstrandsergiusens: I googled that and it said something about garbage collection and heap memory. did the system oom?13:15
jdstrandsergiusens: looking at the trace, it looks like a heap allocation that failed13:16
pedronisChipaca: I pushed my changes to #4076, it could use  a review13:25
mupPR #4076: many: handle core configuration internally instead of using the core configure hook <Created by mvo5> <https://github.com/snapcore/snapd/pull/4076>13:25
cachiojdstrand, hey, I am creating a test for desktop interface and I can access the /etc/xdg/user-dirs.conf and .defaults13:30
cachiojdstrand, I see this https://paste.ubuntu.com/25865263/13:30
cachiojdstrand, I can access to the rest of the files13:31
cachiojdstrand, not sure if I need to do something else13:31
jdstrandcachio: I'm confused. you can access the files but you have denials?13:34
jdstrandcachio: I can say that we started to allow those files with a recent 2.29 and master PR13:34
cachiojdstrand, no, I getr permission denied13:34
cachiojdstrand, I am able to access to the rest of the files and dirs allowed by the interface13:35
jdstrandcachio: are you on 2.28?13:35
cachioin master13:35
cachiojdstrand, I am creating a new test to validate the desktop interface on master13:36
jdstrandthat doesn't make sense. I'm looking at master now and the files are in there13:36
cachiojdstrand, yes, I did the same13:36
jdstrandcachio: can you paste /var/lib/snapd/apparmor/profiles/snap.test-snapd-desktop.testdesktop?13:36
cachiojdstrand, https://paste.ubuntu.com/25865301/13:37
cachiojdstrand, I have installed 2.28.513:38
cachiois it included on this one?13:38
jdstrandcachio: that file doesn't have the rules13:39
jdstrandcachio: no. it is in 2.29 and higher13:39
cachio jdstrand, well, that's the reason :)13:39
jdstrandindeed :)13:39
cachioI'll update13:39
cachiojdstrand, hanks and sorry13:40
cachiothanks13:40
jdstrandnp13:40
stgrabersergiusens: ouch, never seen that before, that's one very unhappy snapd13:53
sergiusensjdstrand the log is all the info I have, this is travis on trusty14:10
hurricanehrndzHow do you run a search against all the channels?14:10
sergiusensseems to have been a one time thing, luckily14:10
Chipacahurricanehrndz: you can't14:10
Chipacahurricanehrndz: why do you want to do that?14:11
hurricanehrndzChipaca: Why wouldn't I, There is no snapcraft on openSUSE and I would like to use it. I know it is in the candiate channel, but I know that because I ran a search on reddit, I find it very unproductive to have to go to browser in order to find snaps14:13
hurricanehrndzChipaca: so now I'm wondering what other snaps are available on the other channels that I could make use of, but have no way to index14:14
pedronishurricanehrndz:  but snap info snapcraft   tells that14:15
hurricanehrndzOh, didn't know14:15
hurricanehrndzThat's what I was looking for, I think. Let me try14:16
hurricanehrndzpedronis: Info only helps, if I know the snap exists14:17
pedronisthis works if you know the name of the snap,  but in general is not a good idea to use not stable snaps you don't know about14:17
pedronishurricanehrndz: that's a bit by design currently, if a snap is not stable, is not advertized, but can be used  (install, info etc) if one knows about it14:18
hurricanehrndzpedronis: I'm a dev, so I'm okay with being on cutting edge. It just seems that find should have an option to search across all channels14:18
hurricanehrndzpedronis: is there any plans to bring snapcraft to other platforms?14:19
hurricanehrndzOr is a snap of snapcraft the plan?14:20
Chipacahurricanehrndz: that's a question for sergiusens i think14:20
hurricanehrndzsergiusens: Do you know if there are plans to bring snapcraft to other distros14:21
hurricanehrndzThanks for all the guidance guys! :)14:23
sergiusenshurricanehrndz it is a snap, so if snaps work their, it could potentially work, like all debian based distros should be ok; snapcraft with cleanbuild or persistent containers if not14:24
sergiusensthat said, if you want something like Fedora, we are waiting on Son_Goku ;-)14:24
Son_Gokuwait what?14:24
Son_Gokusergiusens: what am I being dragged into now?14:24
sergiusensbtw, 2.35 is going to be the one that makes it to stable14:24
sergiusensSon_Goku dnf?14:25
sergiusensdnf in snapcraft that is14:25
Son_Gokuoh, you mean RPM support for snapcraft14:25
Son_Gokuyeah, I need to make time to hack on it again14:25
Son_GokuI've been bombarded with all kinds of stuff lately14:25
sergiusensSon_Goku yeah, exactly that, and no worries14:28
Son_Gokuhurricanehrndz: I'm working on Fedora/openSUSE support for snapcraft, it's just taking some time14:28
Son_Gokuunlike most folks here, I'm doing this in my free time, so it's not moving as fast as I'd like it to14:28
sergiusenselopio and kyrofa are you around for a quick hangout?14:28
Son_GokuI took a break last weekend to just play video games and do nothing else14:28
Son_GokuSuper Mario Odyssey was awesome14:28
sergiusensSon_Goku heh, some of the things that get done are done in my free time as well but I totally get you as I am not context switching ;-)14:29
=== ikey|zzz is now known as ikey
sergiusensSon_Goku I noticed that SMO took some priority over what is considered free time14:29
Son_Gokuthe last four weekends were spent working14:30
Son_GokuI was exhausted14:30
Son_Gokulike, for $DAYJOB14:30
Son_Gokusergiusens: as for snapcraft cleanbuild, kyrofa has to start packaging lxd for Fedora now ;)14:31
Son_GokuI fulfilled my end of the bargain and made the selinux policy work with lxd, now we just need lxd in Fedora itself14:31
sergiusensSon_Goku just snap install lxd on fedora!14:32
Son_Gokuthe snap doesn't integrate with our selinux policy14:32
Son_Gokuso things break14:32
Son_GokuI don't recall if stgraber fixed it, but it was still broken last I checked14:32
Son_Gokuwhich admittedly, was a few weeks ago14:33
hurricanehrndzsergiusens: Nope not fedora, openSUSE14:36
hurricanehrndzsergiusens: Thank you though14:36
hurricanehrndzSon_Goku: If you need help let me know14:36
hurricanehrndzSon_Goku: Thanks for all your hard work, it means a lot to all of use that are on RPM based distros14:37
hurricanehrndzSon_Goku: Good for you taking a break, I know the feeling of needing a breather. It has been awhile for me, he he. I want to get back into playing games again soon.14:38
hurricanehrndzPs selinux is a pain in the ass14:38
Son_Gokuhurricanehrndz, SELinux isn't bad or difficult, but it's hard to get Ubuntu people to think about the other side so much ;)14:43
Son_Gokutheir only SELinux guy left a long time ago for Google14:43
ogra_if you talk about kees, he didnt leave ubuntu ... only canonical ;)14:44
ogra_(he is actually a member of the ubuntu technical board )14:45
elopiosergiusens finishing my workout. Will be ready in 15.14:45
kyrofasergiusens, elopio sorry, late start today, I'm around now14:53
kyrofaI have coffee roasting though14:57
sergiusenskyrofa elopio ok, let's join when the clock hits 10, on the weekly one15:01
elopioI'm ready15:01
elopioAck15:04
jdstrandsergiusens: it seems something unrelated to snappy. my guess is memory pressure15:05
kyrofaSorry guys, just finished up the coffee, joining now15:11
kyrofaelopio, sergiusens darn, was it that quick? :P15:12
=== cachio is now known as cachio_lunch
elopiokyrofa: where are you?15:15
sergiusenskyrofa we are talking still15:15
elopiowrong hangout?15:15
kyrofaWhat the heck... weekly right? I was the only one there15:16
kyrofaTrying again15:16
om26erHi! re: docker snap, seems when I delete my containers and images the disk space is never freed.15:36
cory_fuAre the build.snapcraft.io builders backed up or something?  I've been waiting all morning for a build of charm-tools so that I can do a release.16:03
om26ercory_fu: I learned yesterday that builders are busy when gates for new Ubuntu release open.16:03
om26ermy build started at like 12AM last night.16:04
cory_fuAh, I see.  That makes sense16:04
mupPR snapcraft#1647 closed: lxd: better surfacing of errors <bug> <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1647>16:08
kyrofaHaha, oh darn, I was testing that PR. Apparently I can only approve OPEN ones16:25
kyrofa+1 anyway16:26
diddledanhas snapd lost it's newly inbuilt xdg-open support again?16:27
diddledansnapd   2.28.5+17.1016:27
mupPR snapd#4112 opened: cmd/snap,client,daemon: show ignore-validation if set in snap info <Created by pedronis> <https://github.com/snapcore/snapd/pull/4112>16:27
diddledanError org.freedesktop.DBus.Error.ServiceUnknown: The name com.canonical.SafeLauncher was not provided by any .service files16:27
diddledanfrom dpkg -l : ii  snapd-xdg-open                                                   2.28.5+17.10                                amd64        Transitional package for snapd-xdg-open16:28
diddledanweird. some apps can launch urls fine but one specific one (djv_view that I've been working with mlw to snap) can't16:30
diddledanoh wait. it's a file:// url16:31
diddledanbah16:31
diddledannot supporting mailto: and file: urls is painful16:32
Chipacadiddledan: raise it in the forum?16:43
diddledanI thought I had. search isn't showing me the thread tho16:45
kyrofaThe forum search does that16:49
diddledanwhy?! seems a bit backwards for a "find the thing you're after" to not show the thing you're after16:49
kyrofadiddledan, haha, I'm just being mean. I just mean to say it doesn't cooperate with me either16:50
diddledanaye, I've noticed it on several different times I've tried finding my previous posts16:50
Chipacadiddledan: i didn't understand it until i tried to read one of the longer topics16:51
Chipacathere, it's exactly what i wanted16:51
Chipacaso ¯\_(ツ)_/¯16:51
=== cachio_lunch is now known as cachio
sergiusensI need to relocate, internet is down18:22
gsilvapthello all19:11
gsilvaptelopio, you around?19:11
mupPR snapcraft#1650 opened: Release changelog for 2.35 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1650>19:45
sergiusenselopio kyrofa ^19:48
kyrofasergiusens, elopio, ament is finally all green! Means autopkgtest is promising20:32
kyrofasergiusens, once we get this release out the door, would you mind giving snapcraft#1636 a priority? I'm seeing more people wanting to run on trusty20:33
mupPR snapcraft#1636: internal: more gracefully determine host OS <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1636>20:33
sergiusenskyrofa hmm, maybe we should get that one in now if it allows trusty, that is a big win. Is it green green?20:39
kyrofasergiusens, no autopkgtests have run there, but I can run them if we can spare the resources20:39
sergiusenskyrofa we will only do trusty as a snap though20:40
kyrofasergiusens, of course20:40
elopiogsilvapt: hello20:40
kyrofasergiusens, I'd also like to write some spread tests for it20:40
kyrofasergiusens, but I was able to build a ROS snap with it20:40
sergiusenskyrofa for which we can just enable a new runner on our travis matrix to run the same suite without lxd20:40
kyrofaOh that's a good idea, assuming you're talking about the integration suite20:41
kyrofaAlthough that will roughly double our PR test time...20:41
kyrofaNightly?20:41
kyrofaIt also assumes the testing deps we have work on trusty20:42
kyrofaI can test that out20:42
kyrofaEh... no that won't work. That will pull all deps from trusty, which I bet won't work for some of our tests20:43
sergiusenskyrofa don't do it now ;-)20:45
sergiusensfor later20:45
kyrofaSounds good20:46
sergiusenskyrofa ugh, you updated the branch, I was about to merge20:46
sergiusensoh well... this gives some time for elopio to review20:46
kyrofasergiusens, you still can if you're comfortable with it, results in the same thing, but yeah that's fine too20:46
kyrofaI'm retesting it here anyway, a few different things have landed that should make this the last one for trusty, just verifying now20:47
sergiusenskyrofa ok20:51
kyrofasergiusens, in the release notes, let's make sure we call it "initial/experimental support" :P20:54
kyrofaThen we can build up our suite of tests to gain confidence20:54
sergiusenskyrofa oh, a first release of anything is that, right?20:54
kyrofaHaha, totally20:54
kyrofasergiusens, built the snap, works great for both the opencv demo as well as the ros-talker-listener integration test (indigo, which is one of the main use-cases for this). Green light on that front21:11
kyrofai.e. this is indeed the final PR for trusty support21:11
sergiusenskyrofa great21:15
kyrofasergiusens, it doesn't look like the nodejs plugin ever runs `yarn install` if the package manager is yarn, right?21:40
sergiusenskyrofa why do you ask?21:41
sergiusensit is very different in the refactor21:41
sergiusensbut yarn install doesn't do the same as npm install21:42
kyrofasergiusens, while I'm waiting for tests to run I'm trying my first nodejs snap, who's build directions are given as yarn install && yarn run start21:42
kyrofaIt's not building without any options, so I've been taking a closer look at the plugin21:43
sergiusenskyrofa oh, maybe let m give you the refactor21:43
sergiusenskyrofa look at my nodejs-refactor branch21:44
kyrofasergiusens, trying it out now, thanks!21:44
sergiusenskyrofa this reminds me about `source: .` ... that needs to leave the world as the default21:44
kyrofasergiusens, that works way better, although trying to use the tarball created by npm pack isn't working, I'll poke at that21:56
kyrofa(it doesn't seem to exist, although pack seemed to succeed)21:56
mwhudsonso uh why do we not have snapd for trusty/{arm64,ppc64el}? is this something you should be talked to your local golang maintainer about?21:56
kyrofamwhudson, probably worth a forum post, I suspect all the snapd folks are long gone for the day22:01
kyrofasergiusens, the tarball doesn't include a 'v' in the version22:04
kyrofaAt least, not this one22:04
kyrofasergiusens, the package.json includes a v, but the tarball doesn't22:05
mwhudsonkyrofa: yeah i guess although i've rolled back my concern another level :)22:05
mupPR snapcraft#1636 closed: internal: more gracefully determine host OS <bug> <Created by kyrofa> <Merged by kyrofa> <https://github.com/snapcore/snapcraft/pull/1636>22:27
mupPR snapd#4113 opened: tests: test for desktop interface <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/4113>22:35
kyrofaelopio, ping on bug #169305822:51
mupBug #1693058: can't install a snap with ca-certificates-java as a build-package <recording> <regression> <Snapcraft:Triaged by elopio> <https://launchpad.net/bugs/1693058>22:51
kyrofasergiusens, when you're able, could you take a look at bug #1724972 ? I swear I remember running into that before, but my memory is so hazy. Any chance you remember?23:00
mupBug #1724972: lib* files in $SNAP/usr/lib/ links to /usr/lib/ <iot> <snap> <Snapcraft:New> <https://launchpad.net/bugs/1724972>23:00
kyrofaI thought we fixed stuff like that in the repo23:01
kyrofaOh interesting-- that file should be filtered out completey23:02
kyrofaOh nevermind. That's just crawling23:03
elopiokyrofa: pong, closed23:07
kyrofaelopio, yay \o/23:07
kyrofasergiusens, elopio you'll never believe it, but snapcraft#1607 is green23:33
mupPR snapcraft#1607: python plugin: use extracted pip <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1607>23:33
elopio^_^23:33
kyrofaNote that I've not completely solved the problem, and while it is still a problem, it was a red herring that wasted a ton of time23:33
kyrofaDuring review, I made a change that happened to cause an integration test to fail. However, instead of showing us that failure, unittest barfed with the exception we all know and love23:34
kyrofaNow that the tests no longer actually fail, unittest isn't dying23:35
kyrofaI'm still investigating why unittest is breaking this way on Travis, but it's no longer blocking this branch23:35
kyrofaUntil I figure out what's happening, if you see this exception, it means the tests are failing and need to be run locally23:36

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