/srv/irclogs.ubuntu.com/2017/06/13/#snappy.txt

ssbashhttp://paste.ubuntu.com/24845115/00:00
cachiothe error is the same?00:01
ssbashyea, "ImportError: No module named 'wraticus.lib'; 'wraticus' is not a package"00:02
cachioand the snapcraft.yaml?00:06
ssbashhttp://paste.ubuntu.com/24845159/00:08
ssbashpretty much unchaged aside from the script00:08
ssbashI tried it both with and without the environment specifications00:08
cachiossbash, https://paste.ubuntu.com/24845181/00:11
cachiohttps://paste.ubuntu.com/24845185/00:11
cachiothere you have an example00:11
cachiousing a wrapper00:11
ssbashok should I move the environment block to the parts section?00:14
cachioyou create a part with the wrapper00:33
ssbashhttp://paste.ubuntu.com/24845323/ this is my lastest version, I'm still getting the 'wraticus is not a package' error00:35
cachiossbash, does it exist?00:45
cachio$SNAP/bin/wraticus.py00:45
cachioand it calls to the lib?00:46
=== chihchun_afk is now known as chihchun
=== pbek_ is now known as pbek
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
francis[m]Any recipes to make a snap from a deb?06:13
zygagood morning06:14
zygafrancis[m]: you can use snapcraft and stage-packages06:14
zygafrancis[m]: look at the array of snapcraft examples on the web06:15
francis[m]zyga (IRC): i looked around06:17
francis[m]But apart from some no longer maintained projects i didnt find06:17
francis[m]Will look at snapcraft06:18
francis[m]Thx!06:18
luk3yxIf I want to include a binary from /usr/bin in a snap, what's the best way to do it?06:20
zygafrancis[m]: https://github.com/search?utf8=%E2%9C%93&q=stage-packages+extension%3A.yaml&type=Code&ref=advsearch&l=&l=06:20
zygaluk3yx: stage-packages as this binary is typically packaged itself06:21
luk3yxzyga: Okay, I'll try that.06:24
zyga good luck06:27
zyganote that stage packages are not always the best way to make a snap, but they are the quickest for sure06:27
luk3yxIt works, thanks.06:30
=== elfgoh_ is now known as elfgoh
sborovkovHello, stupid question but I never used uboot before, if I add some binary to the boot-assets in the gadget snap from what path would it be available in uboot when booting?09:24
morphisogra: ^^09:30
morphiszyga: you will love https://github.com/morphis/spread-images/commit/628d428c655f1004dcadb91bfd3d5eebd44ce80409:31
zygalooking09:31
zyganice :-)09:31
morphiszyga: if we can't fix pacman in the archive lets import packaging bits and make sure they are taken care of, then whoever maintains the pkg can pull these into the main archive09:33
zygayes, I think that's the natrual way to handle this09:33
zygathen anyone can propose improvements to the snapd tree (no need to be a distro maintainer)09:34
zygaand we can work with timothy or anyone else to synchronize with upstream packaging09:34
=== chihchun is now known as chihchun_afk
ograsborovkov, usually from / in the system-boot partition ... unless you push it to a raw place via the gadget snap like i do in https://ograblog.wordpress.com/2017/05/30/building-u-boot-gadget-snap-packages-from-source/09:45
ograsborovkov, if you mean from the uboot shell, then "fatls mmc <devicenumber> <partition>" is your friend to find it ...09:48
ograi.e. if system-boot is yuor first partition on the first SD: "fatls mmc 0 1"09:49
ogra(or "...mmc 0 0")09:49
* zyga needs a coffee09:51
sborovkovogra: ah, alright, yes I am building the gadget snap. Trying to add splash screen10:18
ograsborovkov, heh, i have that half done here for the default gadgets ... :)10:18
ogra(but it is more of a pet project so i only work on it every now and then)10:18
sborovkovogra: is there some standard mechanism (or it's going to be developed) to hide login promt on the first boot? First boot takes quite awhile and that would look bad for user to see login promt at that time...10:20
sborovkovalso, is I understood you correctly when adding bmp file to the boot-assets it would be available from /? so I can add splashscreen=/boot-assets/Screenly.bmp?10:21
ograsborovkov, well ... depends ... in IoT i would expect most boards to only have a serial console and operate completely headless ...10:21
ograsame goes for core based cloud images10:22
ograno, you wouldnt use boot-assets/ ...10:22
sborovkovAlright, understood thanks10:23
ograif it sits directly on your system-boot partition you'd use splashscreen=Screenly.bmp10:23
ograif i were ou i'd just add it to the gadget, build an image with it, stop the boot on the u-boot console and develop the scripting for it in there ... (then later add it to uboot.env.in)10:24
ograso that you can check for paths and the like with fatls/fatload etc10:25
ograhttps://www.denx.de/wiki/DULG/UBootSplashScreen and https://www.denx.de/wiki/DULG/UBootBitmapSupport should be helpful10:25
sborovkovthanks!10:26
mupPR snapd#3457 closed: tests: add refresh --time output check <Created by fgimenez> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3457>11:20
mupPR snapd#3474 opened: tests: fix ipv6 disable for ubuntu-core <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3474>11:22
ograkoza, hmm ... i only actually use the hummingboard with HDMI and kbd attached, did you try that ? (i only opened the case once but didnt touch any jumpers, the board is as it got it from madper)11:29
ogra*as *I* got it11:29
ograkoza, it should work fine from microSD11:30
=== zyga_ is now known as zyga-fedora
=== zyga is now known as zyga-suse
=== om26er_ is now known as om26er
morphiszyga-suse, zyga-fedora: two of you now? :_)12:20
zyga-susemorphis: yes, on each system, ubuntu too but I think I can manage with just two IRC accounts :)12:21
morphishehe12:21
zyga-susesadly irc doesn't do multi-client very well12:21
morphiszyga-suse: https://paste.ubuntu.com/24848692/ , that is what we get now on the Pi 1 or Pi Zero with that patch I did12:21
zyga-susemorphis: that's nice12:22
zyga-susemorphis: btw, one idea12:22
morphisnot optimal but that is something we can improve later12:22
zyga-susemorphis: if the core snap for the architecture cannot be found, link to a forum thread12:22
zyga-susemorphis: and encode the architecture in the URL12:22
morphiswhy not12:22
zyga-suseonce it gets found we do nothing more12:22
zyga-suseand meanwhile people will have a place to go to and discuss12:22
* ogra finds it funny that hexchat actually turns zyga from violet to "SuSE green" when he renames himmself to zyga-suse 12:24
ogra-m12:24
niemeyero/12:25
zyga-suseoh really? :D12:25
zyga-susenice :)12:25
cachioChipaca, hey12:52
cachioChipaca, I have seen some tests failing because of an error 50212:53
cachioChipaca, downloading apps12:54
cachioChipaca, have ouy seen that?12:55
pstolowskizyga-suse, hey, re snap-confine and security tag vs snap name matching - do we have strong incentive to get rid of regexp in verify_security_tag? because if not, the fix is going to be very simple with captured text12:56
Chipacacachio: yes; was that yesterday, or today?12:56
cachioyesterday12:56
cachioI saw it twice12:56
Chipacaright12:56
Chipacayesterday the store had some issues12:57
zyga-susepstolowski: I'd say we fix the immediate issue and then iterate on improvements (regexp removal)12:57
Chipacacachio: https://r.chipaca.com/sauc.cgi?maxDays=212:57
pstolowskizyga-suse, sounds good to me12:57
cachioChipaca, are those store metrics?12:58
Chipacacachio: from the perspective of a very cheap shared hosting12:59
cachioChipaca, linode provided that?12:59
Chipacacachio: no, that's me12:59
Chipacacachio: in any case it's back to the usual ~1 error in a 24 hour period13:00
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
kozaogra, lunch this time, ah you are lucky with the board. could you take a photo of the board for me. i could verify if all the jumpers are in correct positions. the dos that I could find over the net are not quite detailed.13:16
ograkoza, that will take a bit, i need to open the case again :) ...13:16
ograkoza, just to make sure we have the same thing, you have a hummingboard GATE with case ?13:16
kozaogra, I have hummingboard gate w/o case (but I have been told this is the same pcb, case is an optional accessory) https://www.solid-run.com/product/hummingboard-gate/#configuration13:18
ograkoza, https://wiki.solid-run.com/doku.php?id=products:imx6:hummingboard has a table for the jumper13:20
ograbridging 3+4 and 5+6 should do it13:21
ogra(at the very bottom)13:21
kozaogra, this is what I bridged :-) thts is why I want to super verify everything13:21
ogragimme a bit ... going to the office and grabbing my board and a screwdriver :)13:22
kozaogra, sure, thanks13:23
mupPR snapd#3453 closed: tests/main/manpages: install missing man package <Created by morphis> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3453>13:23
morphisniemeyer: thanks!13:23
ograkoza, oh, interesting13:29
ograkoza, no jumpers at all !13:29
ograwhich would mean OTG boot ... but i guess it falls back to SD if no OTG is connected or some such13:31
kozaogra, interesting13:33
ograwell, try it ... perhaps it boots then for you too13:33
ograbtw, i use a 12V 1A PSU13:33
ogra(an old thing from some broken netgear switch that happened to have the right plug size)13:34
ograso you should be fine with your 12V2A13:34
kozaogra, trying now, exactly 2A should be fine; one thing how your serial is connected? mine is to the 3-pin uart connector which is on the edge half way between microSD slot and the boot-mode pins13:35
ograthe default cmdline has "console=ttymxc0,115200 console=tty0 " so you should see something on serial if it is wired correctly13:35
ograi have never used serial on it13:35
ograHDMI worked just fine ... i actually didnt open the case when working on the image13:36
ograonly after i had that up ... to take a look how it looks inside13:36
kozaogra, got it13:36
ograbut there is that thing labeled UART on the board with three pins13:36
kozaogra, yeah but there is another one but just holes w/o pins soldered which is a bit confusing which one to use13:38
ograwell, surely the one with pins :)13:39
ograi doubt they want you to solder pins onto the one without before you can boot ;)13:39
kozaogra, hopefully :-)13:39
ogragetting the order right might be a bit of trial and error though13:40
kozaogra, at least the jumpers are clear now :-)13:41
ogradont you have a TV or HDMI monitor you could try ?13:42
kozaogra, I have, no signal though13:43
ogragive it a bit13:43
ograthe u-boot stuff might not be displayed ... only once the kernel loads13:44
ograaha13:44
ograthere is another UART on the GPIO pins it seems13:44
ograhttps://wiki.solid-run.com/doku.php?id=products:imx6:hummingboard:hardware:hbpbgpio13:44
ograscroll to the bottom13:44
ograpin 6, 8 and 1013:45
ogra6 being GND13:45
Chipacaniemeyer: one argument for not having oddjob just implement execute-command is that I won't be able to reuse the code from the systemd package afaict13:46
kozayeah but the gate version has the gpio ports laid out differently, like four pins in each row; there are though three responsible for UART but we just have been discussing them - these do not have pins13:46
Chipacabah, it'll be a rather big refactor13:46
Chipacaniemeyer: but i can probably make it shallow enough that you'll still like it :-)13:47
kozaogra, like here https://www.solid-run.com/wp-content/uploads/2015/09/hummingboard-gate-03.jpg13:47
ograwell, i guess it is just split into two rows ... the prob is to find where pin1 is on that :P13:48
kozaogra, left top corner 2nd set of three holes (labeled J35) is a UART too13:48
ograah13:48
niemeyerChipaca: It feels like the commands to be run there are trivial enough, but I definitely have a much vaguer understanding of the issue than you do at this point13:48
ograyep, i see that on my board13:48
kozahere they say it is uart http://download.solid-run.com/pub/solidrun/HummingBoard%202/HummingBoard2%20Schematics%20rev%201.2.pdf (bottom of page 6)13:49
Chipacaniemeyer: this is true; most of the code in systemd is around things other than these direct commands13:49
Chipacastatus and logs and such13:49
Chipacaniemeyer: I'll take a stab at it :-)13:49
niemeyer👍13:49
ChipacaOTOH maybe this evolvs to be the other way around, with start-snap-services using oddjob directly13:50
Chipacaanyhow, to the code13:50
* zyga-fedora goes for lunch13:54
* zyga-suse goes for lunch too13:57
ograare you lunching together ?13:57
ograkoza, just to make sure, the SD was definitely written correctly ?13:59
ogra(if you plug it into the PC it shows the two partitions ?)13:59
kozaogra, just been checking this; yes two14:02
ograok14:02
kozaogra, perhaps Im using too big card, mine is 32G14:02
ograthat shouldnt matter to find the kernel and the system-boot partition14:03
kozaindeed14:03
ogramine only shows the u-boot stuff occasionally on HDMI though ...14:03
ogranot on every boot14:03
ograif it doesnt show it, the screen sits at "no signal" for like 20-30sec, then flashes to black screen and after another few sec i ger kernel messages14:04
ogra*get14:04
kozanothing here; screen has switched off before it printed anything; I'll try with 2nd card just to ruleut issue with card14:05
ogra+114:05
ograif we have the same board there is no reason that yours shouldnt boot14:05
kozaexactly14:06
ograperhaps my monitor is special :P14:06
Chipacapedronis: do you remember if the tomb that's the second arg of state.HandlerFunc is used to abort?14:08
Chipacatrying to track down how abort happens and it's a little elusive14:08
kozamonitor and board, the extra feature is "it works" ;-)14:09
mupPR snapd#3473 closed: tests: pull from urandom when real entropy is not enough <Created by sergiocazzolato> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3473>14:09
niemeyercachio: ^^^ This is in as a step forward, but let's please do the changes we discussed and get random fixed in a more reliable way14:10
cachioniemeyer, yes, already working on that14:11
niemeyerThanks!14:11
niemeyermorphis: Ok, so which one do you want sanpshotted?14:11
cachioogra, could you please point me to see how you are feeding the entropy pool for ubuntu core?14:12
ogracachio, we dont ... as i explained in the forum, we just force the kernel to do it proper and dont use any userspace tools14:12
morphisniemeyer: spread-54 as archlinux-6414:12
cachioogra, ok14:13
ogracachio, and for the kernel the cmdline option is needed14:13
cachioogra, ok, I'll try to do that14:14
niemeyermorphis: spread-54 is being cleaned by spread, and the arch disks are not even there14:14
morphisniemeyer: hm.. sounds like I have to redo that one again14:20
niemeyermorphis: :(14:29
niemeyermorphis: #3222 is almost ready.. just needs to drop that extra function and it's good to go14:29
pedronisChipaca: yes14:32
morphisniemeyer: wasn't that much work14:34
ograkoza, ok, i bit the bullet andf wired up mine with FTDI ... i'm on J25 wired black,white green when the SD card slot is pointing at me ... i get serial output just fine14:43
mupPR snapd#3410 closed: cmd/hotfix: add hotfix tool <Created by zyga> <Closed by niemeyer> <https://github.com/snapcore/snapd/pull/3410>14:46
cachioniemeyer, so, I can do what is being done for ubuntu core, and add a boot command line option for the entropy14:46
niemeyermorphis: Is there something ready to be looked at?14:46
cachioniemeyer, but, it requires a reboot14:46
niemeyercachio: So we can't do it.. we don't want to add another reboot on that pipeline14:47
morphisniemeyer: not yet, being in a meeting but will do that afterwards14:47
ograkoza, http://people.canonical.com/~ogra/image20170613_164358179.jpg ... in my terminal i use "screen /dev/ttyUSB0 115200"14:47
niemeyercachio: We want something closer to what we discussed today in the call14:47
cachioniemeyer, ok14:47
ograkoza, also ... which image exactly are you using ... daily or daily-stable (i only ever use daily here)14:48
* zyga-suse returns to work14:52
ograzyga-suse, did you bring zyga-fedora along with you ? :)14:53
kozaogra, i think stable, will go for daily then14:54
kozalol :-)14:55
ograkoza, well, the gadget is the same on both so you should definitely see serial output frrom uboot14:55
* ogra tries the different images 14:55
* koza will try once is done with meeting14:58
ograso daily-stable seems fine here, i'm in console-conf14:59
kozaogra, i see, must be sth with my board then14:59
ograhow did you write the SD ? perhaps something went wrong there15:00
kozasudo dd if=/home/konrad/Downloads/ubuntu-core-16-hummingboard.img of=/dev/sdb bs=32M15:00
kozaogra, ^^15:00
ograsounds about right ...15:00
ograxzcat /datengrab/images/snappy/daily-stable/current/ubuntu-core-16-hummingboard.img.xz | sudo dd of=/dev/sdc bs=32M15:00
ograthats what i just used (got a local copy of the all-snaps fiolder here)15:01
ograkoza, oh,.... did you make sure that the card was unmounted before dd'ing to it ?15:03
ogramine gets automounted by nautilus and i have to sudo umount /dev/sdc1 and 2 first15:03
ograif you dont do that you get a corrupt card15:04
ogra(or like US people might call it a "trump card" :P )15:05
psftwI should NOT be running snapcraft as root, right?15:05
ograright15:05
psftw <3 ogra15:05
psftwso I have a package which depends on a PPA ;-)15:05
ograif it needs root permissions it is clever enough to ask for it during the process15:05
psftwhttps://github.com/docker/docker-snap/blob/master/snap/snapcraft.yaml#L6815:06
psftw^ installs the PPA to host to support the build process15:06
psftwshould I just prepend "sudo" to appropriate commands ?  wonder if that would work on the launchpad build machines.15:07
ograi think in LP it simply runs as root anyway15:07
ograthere are no users in the buildd setups15:07
psftwthank you, will give it a shot15:08
ograkoza, oh, interesting fact (not relevant for your prob though) is that serial doesnt accept any input15:11
kozaogra, could not unmount indeed (slow re still in a mtg)15:15
ograkoza, yeah, no worries i'm noot impatient :)15:16
ogra*not15:16
ograhmm, i really wonder why my serial only has output ... but no input15:18
morphisniemeyer: spread-13 it is now15:19
niemeyermorphis: On it15:19
morphisniemeyer: thanks!15:19
niemeyermorphis: The image is 1.7G15:23
morphisniemeyer: 1.2G is it for me, again block-level thing but nothing more I can cleanup15:24
niemeyermorphis: Just for comparison15:24
morphisah ok15:24
niemeyerubuntu 16.04, is 608MB15:24
niemeyer32 bits is 87215:25
niemeyer16.10 is 61015:25
niemeyerfedora is 1.4GB, opensuse is 1.5, and now arch is 1.7GB15:26
ogracrazy ... why are they so excessively big ?15:26
niemeyermorphis: What might be going on there?  Block or not those images are growing and growing15:26
niemeyermorphis: We have a limit in how much we can take in space, not to mention those bits need to move around15:26
morphisniemeyer: I don't really know, I got the image with 1.9G and reduced it to 1.2G15:27
niemeyermorphis: All of the sizes above are block wise15:27
morphisok15:28
niemeyermorphis: My guess is that either there's data being left around, or that there's a ton of software that is not relevant for our tests in place15:28
ograkoza, and just FYI http://paste.ubuntu.com/24849651/ is the serial log of my most recent boot15:28
morphisniemeyer: https://paste.ubuntu.com/24849664/15:29
morphisthat is on the file level15:29
mupPR snapcraft#1364 opened: lxd: Inject snapcraft and core snaps into the container <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1364>15:29
niemeyermorphis: Yeah, so 1.2GB on usr.. what is there?15:29
morphisniemeyer: usr/lib15:29
morphisusr/share is 289M and usr/lib 789M15:30
niemeyermorphis: Yeah, so there's likely a *ton* of libraries in there, right? :)15:30
morphisthe difference to debian/ubuntu is that the packages are not really split into sub packages like -docs etc15:30
morphisniemeyer: sure but removing more than I did break the system15:30
niemeyermorphis: Yeah, I can understand that removing certain things might break the system.. we don't want to remove those15:31
niemeyermorphis: But are you saying that every single library in there is a strict requirement?15:31
niemeyermorphis: I'd look for large trees of packages which are not relevant for us15:32
niemeyerThat's what I did with the Ubuntu images, and why they are small15:32
niemeyerI wasn't the one cooking 14.04.. it's 1.45G as well15:32
mvojdstrand: hey, I am looking into porting the sh based tests for snap-confine current, I put a bunch of things into the unit tests, I wonder if you have suggestions/opinions about the spread based part of this, one way would be a direct port like https://github.com/snapcore/snapd/pull/3431/commits/5dafe3f80c0afc1378be975f5a14d7c27c03d3b2#diff-c0e64403c5c43941d93df36f87c53bb5R6 but that feels slightly overkill to do for all the old sh-based tests,15:35
mvohow do you feel about it?15:35
mupPR snapd#3431: interfaces: simplify snap-confine by just loading pre-generated bpf code <Created by mvo5> <https://github.com/snapcore/snapd/pull/3431>15:35
morphisniemeyer: https://paste.ubuntu.com/24849689/15:35
mvojdstrand: maybe can can have a short HO or something later to discuss how to best test this15:36
morphisniemeyer: that are the packages which are left15:36
fgimenezhey jdstrand, i needed to add some changes to the test-snapd-autopilot-consumer snap, could you please take a look (no rush at all)? https://dashboard.staging.snapcraft.io/dev/snaps/8405/rev/5/ if you can take a look to prod too (same snap) that would be great https://dashboard.snapcraft.io/dev/snaps/7807/rev/2/15:37
niemeyermorphis: Pretty small.. still, have you tried to take stuff out of there?15:38
morphisniemeyer: sure15:38
niemeyermorphis: licenses is likely just the text.. reiserfsprogs would be surprising to be a requirement, etc15:39
niemeyermorphis: tzdata, we don't care15:39
morphisbut that one is really just 577kb15:39
morphistzdata breaks glibc15:39
morphishm, why was make still in there but 1.47M just15:40
niemeyermorphis: Yes, the difference between size bit-wise and size block-wise is that every block is 4kb..15:40
niemeyermorphis: make isn't a big deal.. we'll need to get it in anyway15:41
mupPR snapd#3475 opened: Change main store host to api.snapcraft.io <Created by cjwatson> <https://github.com/snapcore/snapd/pull/3475>15:41
niemeyermorphis: The 320MB in var.. what are they mostly about?15:45
mupPR snapd#3476 opened: snap-confine: validate SNAP_NAME against security tag <Created by stolowski> <https://github.com/snapcore/snapd/pull/3476>15:45
pstolowskizyga-suse, ^ when you have a moment..15:45
mvojdstrand: I get some lunch now, lets catchup later15:45
zyga-fedorayep, looking15:45
zyga-fedorapstolowski: make fmt15:46
pstolowskizyga-suse, note, I deliberately removed two "paranoid" checks, not sure if they were adding any value at that point.. just repeated cheks15:46
pstolowskizyga-fedora, ah, sorry15:47
zyga-fedorapstolowski: 2 comments15:51
pstolowskithx15:51
pstolowskizyga-fedora, done15:55
zyga-fedorapstolowski: can you add some tests that show that a valid snap name and valid security tag that don't match are not passing verification please?16:00
zyga-fedorapstolowski: perhaps at the end of the test function you modified16:00
zyga-fedorapstolowski: // Good name and security tags that don't match each other16:00
zyga-fedorapstolowski: and then some tests16:00
pstolowskizyga-fedora, I added one test16:01
pstolowskizyga-fedora, look for "nonmatchingname"16:01
pstolowskiah perhaps it belongs to the "..names we know are good" section16:02
pstolowskior as you say16:02
pstolowskia new one16:02
jdstrandmvo: sorry, still heads down in the bpf review16:02
jdstrandmvo: ping me when you get back16:02
zyga-fedorapstolowski: I added two more comments16:02
morphisniemeyer: that is the pkgdb again which I just have temporarily until I logout16:11
morphisniemeyer: then /var is down to 24M16:11
morphisniemeyer: https://paste.ubuntu.com/24849997/ is the size breakdown per package16:16
niemeyermorphis: Cool, thanks16:17
morphisbut nothing of the top ones can be remove without breaking the system16:17
niemeyermorphis: I went ahead and created the image so you're not blocked for the rest of the week16:17
morphisniemeyer: really? I am still logged in :-)16:17
niemeyermorphis: But this is our largest image yet.. would be nice to drive it down at some point16:17
morphiswondering if Linode uses a different fs block size for these images16:18
niemeyermorphis: Nope, 4kb16:18
morphisright16:18
morphisniemeyer: but I agree that these larger images sizes are not really what we want16:19
morphiswondering how much is this related to the fact that other distros pay less attention to properly splitting source packages into multiple binary packages16:20
morphisniemeyer: found another one, now down to 901M according to df16:21
niemeyermorphis: Ah, nice!16:22
morphisniemeyer: give me a few more minutes before you snapshot again16:22
niemeyermorphis: Sounds good, just ping me.. I'll take the machine off the pool to make sure we're not losing that16:22
morphisthanks16:23
pstolowskizyga-fedora, pushed16:23
zyga-fedorapstolowski: thanks16:23
zyga-fedorapstolowski: +116:24
zyga-fedorapstolowski: I requested a 2nd review from jdstrand16:24
pstolowskizyga-fedora, thanks!16:24
morphisniemeyer: ok, don't get it much smaller, lets take what we have16:35
=== zyga is now known as zyga-fedora
niemeyermorphis: Ready to snapshot 13?16:49
morphisniemeyer: yes!16:50
niemeyerOk, there we go16:50
niemeyermorphis: I must be looking at the wrong machine16:51
niemeyermorphis: Size didn't change at all16:52
niemeyermorphis: It's also suspect that you didn't get logged off earlier16:52
morphisah!16:52
niemeyermorphis: Are you still logged in now?16:52
morphisniemeyer: yes, which one did you look at?16:53
niemeyer:P16:53
niemeyermorphis: Okay, we're definitely not looking at the same machine16:53
niemeyermorphis: 13..16:53
morphis.spread-reuse.yaml tells me 20 .. wtf16:53
niemeyermorphis: Aha!16:53
niemeyerOkay, let me check16:53
morphisniemeyer: don't tell me it's much smaller now :-)16:55
niemeyerWell, I hope it is!16:55
niemeyermorphis: 1.25G, well done!17:02
morphisniemeyer: not well enough :-)17:02
niemeyermorphis: Machine back up and running, snapshot done17:02
morphisthanks!17:02
niemeyermorphis: Thank you!17:02
morphisniemeyer: if we don't talk anymore, enjoy your long weekend!17:02
niemeyermorphis: Thanks a lot17:03
Chipacaniemeyer: 23   Done    2017-06-13T17:04:51Z  2017-06-13T17:04:51Z  [/usr/bin/printf hello]17:05
Chipacaniemeyer: also, http://pastebin.ubuntu.com/17:06
Chipacastill need to look into aborting it17:06
niemeyerChipaca: Suspect there's a number missing on that second line :P17:07
Chipacamaybe17:07
Chipacawho knows17:07
Chipacaniemeyer: http://pastebin.ubuntu.com/24850224/17:07
Chipacasilly "this field is required" error17:07
Chipacaniemeyer: the “INFO hello” thing is noteworthy17:08
niemeyerChipaca: Yeah, I was just thinking about it17:08
Chipacaniemeyer: I'm spawning a couple of goroutines and updating the task's log as stdout/stderr change17:08
niemeyerChipaca: I don't think we do that in hooks except when it errors17:09
niemeyerChipaca: WHich is perhaps a good idea17:09
morphisniemeyer, zyga-suse, mvo: was there anything special you guys did with the loop-devices each snap create to prevent them from showing up in the desktop launcher?17:09
Chipacaniemeyer: probably for the best :-)17:10
Chipacai'll throw away the goroutines and just use a pair of buffers17:10
niemeyerChipaca: +117:10
niemeyerChipaca: and then, I think we want to tune the summary, but that was probably just for demonstration's purpose, right?17:10
Chipacayeah.... demonstration...17:11
Chipaca:-)17:11
niemeyerSuppa, thanks17:11
Chipacaniemeyer: whoever creates the task sets the summary, so they'd have more context to add17:11
Chipacathe task summary itself i think will stay about the same17:11
niemeyerYeah, +117:11
Chipaca(helps debugging)17:11
Chipacathe change summary would be the contextual one, i mean17:12
niemeyerChipaca: About the same as what?17:12
Chipacae.g. on the change summary "restarting services: foo1 foo2"17:12
Chipacaon the task summary, "systemctl restart snap.foo1.service snap.foo2.service"17:12
niemeyerChipaca: The task summary should follow the style we use elsewhere I think17:13
niemeyerChipaca: Same for the change summaries, actually17:13
Chipacaniemeyer: can i have an example?17:13
niemeyerChipaca: We even have a style for when there are too many things to report, IIRC17:13
niemeyer% snap changes17:14
niemeyerID   Status  Spawn                 Ready                 Summary17:14
niemeyer488  Done    2017-06-13T12:24:58Z  2017-06-13T12:25:09Z17:14
niemeyerClearly NOT THAT17:14
niemeyerWhere did the summary go?17:14
Chipacaniemeyer: no i mean for the specific case of this17:14
niemeyerBizarre17:14
ChipacaI can see e.g. Done    2017-06-12T13:04:42Z  2017-06-12T13:04:42Z  Remove aliases for snap "test-snapd-service"17:14
niemeyerChipaca: Yeah.. Restart service "foo.baz" on snap "baz"?17:16
Chipacaniemeyer: so the same as the change summary?17:16
niemeyerYeah, if we have nothing better17:16
niemeyerHere are some example for the ghost change above17:16
Chipacaso just set by the caller17:16
niemeyerhttps://www.irccloud.com/pastebin/1EgvoizI/17:17
Chipacaniemeyer: but a difference is that those tasks are always doing that thing, afaik17:17
niemeyerChipaca: I don't see how that would justify an entirely different style17:18
Chipacawhereas this task is always just doing exec of something it doens't know the significance of17:18
=== JanC_ is now known as JanC
Chipacathat's why i asked you for an example for this case in particular17:18
niemeyerE.g. we literally have a task saying 'Stop snap "lxd" services'  above17:18
Chipacayes, in a task that just does that17:18
niemeyerYeah..?17:19
niemeyerI hope all these tasks just do what they claim :)17:19
Chipacamy point is that this task gets told “execute systemctl restart snap.lxd.service”17:19
Chipacawhich is a long way from "restart lxd services"17:19
niemeyerChipaca: Sorry, I'm trying to be difficult.. I just don't understand the point.. in my mind we have a pretty clear pattern for task summaries, and should follow suit for this one task as well.. all these tasks we see above have different implementation details across each other17:20
niemeyerEven then, we have a one-liner summary that is user oriented and worded in similar terms17:21
Chipacaniemeyer: can you, please, for the third time, give me a concrete example of what you want this task summary to look like?17:21
niemeyerChipaca: I already provided an example above (17:16).. but I think something even simpler, like "Restart service foo.bar" when it's a single one, or "Restart service foo.bar and bar.baz" when two, or something similar to what we do when aggregating snap names in the API otherwise, should do.. WDYT?17:24
niemeyerChipaca: and again, yes, I think it's fine to have change and task with the same summary if they are indeed matching each other17:25
Chipacathank you17:25
Chipacai needed to be clear on whether the summary was going to get passed in or not17:25
Chipacathis does mean we lose track of exactly what was executed, although I guess I could log it17:26
niemeyerChipaca: I don't think we need to log what's executed.. that's an implementation detail, same as for other tasks17:26
Chipacaok17:27
=== elfgoh_ is now known as elfgoh
jdstrandmvo: hey, since I didn't hear from you, this should make it easier: https://github.com/snapcore/snapd/pull/3431/commits/5dafe3f80c0afc1378be975f5a14d7c27c03d3b2#r12174402517:28
mupPR snapd#3431: interfaces: simplify snap-confine by just loading pre-generated bpf code <Created by mvo5> <https://github.com/snapcore/snapd/pull/3431>17:28
pedronisChipaca: HookTask gets the summary passed in, I suppose you want the same17:29
mvojdstrand: thanks a bunch, I have some obligations here for at least one more hour, thanks a lot for your comments and also for the verification code!17:36
mvojdstrand: and agreed that it is valuable to have one integration test with bad syntac17:38
jdstrandmvo: whenever you get back to it, I gave input for read/stat as well as the path checks. I also responded to Gustavo's comment. basically, check your mail :)17:45
jdstrandmvo: I'm continuing to look at this PR17:45
Chipacapedronis: yeap17:47
jdstrandmvo: (not saying you haven't checked your mail, just saying more is coming)17:47
cachioogra, do you know another way to increase entropy, I can't reboot the machine so I cant add that command line17:48
ogracachio, not really, but perhaps the security guys do ...17:49
ogratyhicks, jdstrand ^^17:49
cachiotx17:49
tyhickscachio: do you have a hwrng device available?17:51
cachiotyhicks, I need it for a vm17:51
mupPR snapd#3447 closed: debian: unify built_using between the 14.04 and 16.04 packaging branch <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3447>17:51
cachioand I need to increase the entropy17:52
tyhickscachio: is the VM using pollinate?17:52
cachioI have used pollinate and rngd-tools but we are seeing a segfault which is breaking our tests17:52
cachiotyhicks, yes, we are currently using pollinate but we want to change it17:53
cachiotyhicks, https://forum.snapcraft.io/t/error-creating-key/96417:53
cachiotyhicks, some tests are failing because the entropy is too low and snapd cand generate a key17:55
cachiotyhicks, it happens when there is a seg faul as it is described in the forum17:55
tyhickscachio: have you looked into fixing the rngd segfault?17:55
cachiotyhicks, no17:56
cachiotyhicks, no idea how to do it17:56
mupPR snapd#3429 closed: interfaces/snapd-control: also allow use of /usr/bin/snap <Created by jdstrand> <Closed by niemeyer> <https://github.com/snapcore/snapd/pull/3429>17:56
tyhickscachio: I need some time to read through that forum thread17:57
tyhickscachio: this is not a trivial problem17:57
cachiotyhicks, ok, any advice about what I could start doing?17:59
tyhickscachio: not yet - I need to understand your problem18:01
cachiotyhicks, ok18:01
tyhickscachio: you said that the tests are running in a VM - do you have admin rights on the host?18:12
cachiotyhicks, yes18:13
tyhickscachio: you could set up virtio-rng to feed entropy from the host into the guest via a virtual hwrng18:14
cachiotyhicks, ok18:15
cachiotyhicks, I'll try with this one18:17
cachiotyhicks, virtio-rng is just for virtual environments?18:18
tyhickscachio: yes18:19
tyhickscachio: it is only helpful in VMs where the host has set up a virtio-rng device for the VM18:19
tyhickscachio: what's the oldest version of gpg that you'll need to operate on?18:22
cachiotyhicks, I need to run snapd test in real hardware and in vms, that are from different sources, for example we currently used linode in the cloud and qemu18:22
tyhickscachio: then virtio-rng may not work for you18:23
tyhicks(you're less likely to hit lack of entropy issues on real hardware but maybe you still have that problem on real hardware)18:24
cachiotyhicks, is there any way to feed manually the entropy pool18:25
cachiotyhicks, with fake random data?18:25
tyhickscachio: that's what you're doing with rngd18:26
tyhickscachio: fixing the segfault is the easiest solution18:26
cachiotyhicks, ok, I'll report it18:28
tyhickscachio: rngd is community maintained (in universe) so it isn't likely to get attention in normal circumstances18:29
tyhickscachio: how urgent do you need it fixed?18:29
cachiotyhicks, it is for our snapd test suite18:30
cachiothere are several tests failing because of that18:30
tyhickscachio: ok - first step is to report the bug18:30
tyhickscachio: then we'll all need to figure out who is going to work on fixing it18:31
tyhicksrngd is simple so I'm hoping the fix won't be too difficult18:31
cachiotyhicks, well that's something really good18:32
cachioniemeyer, we were discussing a solution to the lack of entropy problem18:33
niemeyercachio: hey18:34
niemeyercachio: Ok, and what's up there?18:34
cachioniemeyer, so, the best approach to deal with this is to fix the issue that it is causing the segfault in rngd18:35
tyhicksto be clear, the rngd hack of using /dev/urandom as the HWRNG device should only be used for testing environments18:36
cachiotyhicks, of course, this is our case18:36
* tyhicks nods18:36
cachioniemeyer, so the change that I did, won't be needed18:37
niemeyercachio: Yeah, that's the most reasonable.. but what's the ETA for the fix?  Do we need to do anything meanwhile to make sure we're not getting tests hanging?18:38
cachioniemeyer, we can leave my patch with the note, and once it is fixed, I'll remove it18:39
cachioniemeyer, I know it is not the best approach18:39
cachioniemeyer, but it is a temporal workaround18:39
niemeyercachio: That has the issues we discussed.. can we cheaply do one of the changes we discussed today?18:40
niemeyercachio: In practice that fix will take a long time to propagate to all the kernels we care about18:41
niemeyerAh, not kernels, it was rngd isn't it?  Either way, same case18:41
tyhicksniemeyer: the fix will be to rngd, not the kernel, but your point still stands18:41
cachioniemeyer, the change is in rngd18:42
niemeyerAnybody else? ;)18:42
tyhicks:)18:42
tyhicksniemeyer: is it possible for you to pre-generate a key that will be used for all test runs?18:43
tyhicks(include it in the test suite and put it in place before each test run)18:43
cachiotyhicks, the issue is that there are tests that are testing the create-key18:45
cachiotyhicks, so, for those cases we can't pregenerate the keys18:46
tyhickscachio: it isn't necessarily correct to pump up the system with fake entropy for those tests, either18:46
tyhicksin the field, create-key will be used on low entropy systems18:47
tyhicks(there's no perfect solution to this problem)18:48
tyhicksthe create-key tests could be skipped when /proc/sys/kernel/random/entropy_avail is below a certain threshold18:49
cachiotyhicks, well that could be a solution18:50
cachioniemeyer, what do you think?18:52
tyhickspossible permanent solutions:18:52
cachioniemeyer, we can't pre-create keys, manually feed the pool is not cheap18:52
tyhicks1) fix rngd (fix has to propagate to all affected distros that you test on)18:53
tyhicks2) write your own version of rngd that knows how to feed fake entropy to /dev/random18:53
tyhicks3) remove /dev/random and mknod it back using the device major/minor of /dev/urandom (I'm not certain that this will work)18:54
cachiotyhicks, well, how much effort should be needed for 2)18:54
cachio?18:54
niemeyercachio: We don't want to skip the test like this.. otherwise this might be completely broken and we'll never know18:54
tyhickscachio: crack open the rngd sources to see - you'd essentially be forking it18:55
niemeyertyhicks: How to feed entropy into the kernel?18:55
niemeyertyhicks: If it's just a matter of piping data somewhere, which I hope it is, it might be much simpler than forking rngd18:56
tyhicksniemeyer: it is an ioctl()18:56
tyhicksniemeyer: the random(4) man page details it18:57
tyhicksthe Configuration section of that man page also touches on my option #3 above (the mknod command)18:58
tyhicksI need to get back to some other work now18:59
tyhicksI hope those suggestions are helpful18:59
cachiotyhicks, sure, tx19:01
niemeyertyhicks: Indeed, thanks19:07
niemeyercachio: You said earlier we already had the /dev/random trick at the pepare level?19:07
cachioniemeyer, yes19:08
niemeyercachio: If that's the case, why are we redoing it in that function?19:09
cachioniemeyer, it is because the entropy pool is cleaned when the segfault happens19:10
cachioniemeyer, I could make something generic, to regenerate entropy when is it under 700 for all the tests19:11
niemeyercachio: Yeah, I get it, but isn't the /dev/random tweak already in place?19:11
cachioniemeyer, it is done at the begening of the test suite19:12
niemeyercachio: Yes, so why do we need to do that every time the function is called?  Why isn't it simply a restart?19:12
cachioniemeyer, you mean just to restart it? not check for the entropy level?19:13
cachioniemeyer, for each test?19:14
niemeyercachio: Why do we have that environment line change every time19:14
cachioniemeyer, echo "HRNGDEVICE=/dev/urandom" > /etc/default/rng-tools =19:14
cachiothis one?19:14
niemeyerYes :)19:15
cachioniemeyer, it can be done just the first time19:15
cachioin the prepare19:15
niemeyercachio: Then, I don't think I get the whole picture.. when it crashes, it restarts, right?19:15
cachiowhen it crashes it does not restart19:16
niemeyercachio: Oh?19:16
cachioniemeyer, that's why I am doing this19:16
niemeyercachio: Why would it not restart?  Isn't it a systemd unit as usual?19:17
cachioyou mean rng-tools?19:17
cachioI'll take a look to see why it is not restarting19:19
niemeyercachio: Thanks19:19
niemeyercachio: Maybe that's the only thing we need to fix19:20
ryebotMy snap has a configure hook that attempts to write to the file $SNAP_DATA/args. Most of the time it works, but on some (identical) lxc containers, I get an apparmor denial: http://bit.ly/2reFFt519:35
ryebotAny idea what might be going wrong, and why it's not going consistently wrong?19:37
kyrofaryebot, that should be non-fatal, it's a snapctl denial, nothing to do with you directly19:38
kyrofaIt's noisy though, and last I heard work was ongoing to fix it19:39
ssbashhey if anyone has a second I'm still having trouble with getting my python path wrapper command to work?19:53
ryebotkyrofa: thanks19:53
ssbashhere is the bash script: http://paste.ubuntu.com/24851223/ snapcraft.yaml: http://paste.ubuntu.com/24851218/ and the error message:http://paste.ubuntu.com/24844781/19:54
mupBug #1697770 opened: can't run juju from snap: "Too many levels of symbolic links" <Snappy:New> <https://launchpad.net/bugs/1697770>20:17
ssbashcachio? do you see any probkems with my bash script or snapcraft.yaml?20:30
ryebotkyrofa: Alright, red herring apparmor logs aside, any reason my snap wouldn't be able to write to $SNAP_DATA? http://bit.ly/2rXVwzm20:34
kyrofaryebot, no, that seems weird indeed. Can I take a look at the configure hook itself?20:36
ryebotkyrofa: certainly, one second20:36
ryebotkyrofa: http://paste.ubuntu.com/24850838/20:36
kyrofaHuh, yeah, I really have no explanation for that. Particularly why it's flaky. I assume you've verified that the only denial you're seeing is the snapctl one?20:38
ryebotkyrofa: let me check. I've also been advised to check the host as well, so let me do that too.20:39
cachioniemeyer, so, basically, rng-tools tools is not configured to restart after a crash20:40
cachioit is stopped when the crash happens20:40
kyrofaryebot, ahhhh, running in lxc?20:42
cachioniemeyer, so I can make some changes to autorestart, but it could have different implementations based on the distro20:42
cachioniemeyer, or I can check in the prepare-each if the service is stopped and start it again20:42
cachioonce I start it the entropy is recovered20:43
ryebotkyrofa: yes20:43
ryebotkyrofa: sorry, second time I haven't made that clear enough in these parts :\20:43
kyrofaryebot, yeah I run snaps on lxc as well, snapd doesn't run quite right there20:44
kyrofaryebot, typically you don't need to. But snapd and lxc share a lot of the same tech, and they don't always get along20:44
ryebotgotcha20:45
ryebotanyway, yeah, only snapctl20:45
ryebotkyrofa ^20:45
kyrofaNothing on the host either?20:47
ryebotcorrect20:47
kyrofaryebot, sorry, I really don't know what's happening. You'll need to wait for someone in snapd20:55
ryebotkyrofa: no worries, thanks for reaching out!20:57
kyrofaIt'd be nice to see snapd add lxc to their test suite20:57
kyrofaWeird breakages will probably continue until that happens20:58
bdxis there a ruby/rails plugin laying around anywhere?21:02
kyrofabdx, ruby is pretty easy, talk to elopio. Rails is a little harder because bundler likes symlinks21:03
kyrofaI just haven't gotten the time to finish one. I'd love to see one though, if you work on it21:04
bdxkyrofa: I've been looking into it pretty heavily ..... I'm having trouble understanding what I need to do in the snap world to even start ....21:15
bdxkyrofa: so, 1) need ruby - it would be best to create a plugin for ruby im guessing21:16
kyrofabdx, yes, that's something elopio is working on. I suppose bundler support could be added there, in which case it would work for rails as well21:17
kyrofaObviously Gemfiles are not unique to rails21:17
bdxthen use the ruby plugin and the git plugin together21:17
bdxyeah21:17
kyrofaThere is no git plugin-- those are sources. You won't need to do anything special there, they're part of core snapcraft21:18
bdxkyrofa: awesome21:18
kyrofabdx, here, let me show you an example, but you really should talk to elopio about the ruby plugin, I know he was wanting to work with someone to do it21:19
bdxkyrofa: great thanks21:19
kyrofabdx, how familiar are you with ruby, by the way?21:19
bdxkyrofa: I can read and write it, but dont prefer it ... I have 30+ prod rails deploys via charms21:20
bdxso my rails charms are on point - i feel like this would set me up to be ready to snap them21:21
kyrofaIndeed21:21
kyrofabdx, here's the cmake plugin as an example: https://github.com/snapcore/snapcraft/blob/master/snapcraft/plugins/cmake.py21:21
kyrofabdx, here's some basic info for how to write plugins and test them in your own tree instead of needing to patch snapcraft: https://snapcraft.io/docs/build-snaps/plugins21:22
bdxexcellent, thanks for these21:23
bdxwould lthe nodejs plugin be more similar you think?21:23
ssbashhttp://paste.ubuntu.com/24851807/21:23
kyrofaYes perhaps so. I do think we should support multiple versions of ruby21:24
bdxit seems like nodejs would be more similar in the sense that there are multiple versions of node that the plugin should be able to handle21:24
kyrofaYep21:24
ssbashhi can someone explain why my snap is deliberly ignoring the python path i've set for it?21:24
naccssbash: if you run snap run --shell and then run your command with your pythonpath set, you should be able to debug it live21:25
bdxkyrofa: I'm also wondering how much the gemfile should matter .... some consider it a best practice to define their ruby version in their gemfile21:25
kyrofabdx, interesting, I've never done that. What tool consumes it? Bundler can't install it, right?21:26
kyrofaI'd lean more toward using .ruby-version like what rvm might place there21:26
bdxelopio21:27
ssbashnacc: I tried running env to seee what the pythonpath was, and it print out "PYTHONPATH=:/snap/wsnap/x1/lib/python3.5/site-packages"21:27
bdxelopio: I would love to connect concerning a ruby snap - jamesbeedy@gmail.com21:28
bdxso21:28
bdxkyrofa: http://imgur.com/a/g44Sb21:28
ssbasheverything seems to be set correctly, I have no idea why the snap command isnt working21:28
bdxI think it acts more like a stop check21:28
bdxif your rubyversion doesnt match, bundler wont install21:28
naccssbash: but now your PYTHONPATH doesn't include the dist-packages? what is the error you get?21:29
ssbash"ImportError: No module named 'wraticus.lib'; 'wraticus' is not a package"21:29
kyrofabdx, interesting, okay. I say keep it simple in the first rev, just accept a `ruby-version` option, fetch the source, and build it21:29
ssbashwraticus is definitely in the site-packages directory21:29
kyrofabdx, then we can start looking into those things once we have a basic plugin working21:30
naccssbash: ok, in your snap shell, run python and try to import wraticus21:30
naccssbash: and then try to import wraticus.lib21:30
bdxkyrofa: entirely ... I'll keep investigating, and try to get in touch with elopio21:31
kyrofabdx, I've given him your contact info in another channel as well, he's probably out to lunch21:31
kyrofaPlease let me know if I can help21:31
ssbashnacc: I'm able to import both wraticus and wraticus.lib21:32
naccssweeny: hrm21:33
bdxkyrofa: your insight has been super helpful, thanks. I'm going to give the cmake and nodjs plugins a glance over this evening and try to get a strwman up21:34
kyrofabdx, any time :)21:34
bdxI'll send you guys a ping tomorrow (hopefully with some progress)21:34
kyrofaGood deal, I'll be here!21:34
ssbashnacc: is there anything I can do? here is my snapcraft.yaml for reference: http://paste.ubuntu.com/24851218/21:38
bdxkyrofa: so you think I should keep away from system ruby, and look to use a ruby version manager instead, due to the ease of changing versions etc etc?21:38
kyrofabdx, I definitely suggest not using system ruby. As for what TO use, well, I'm not sure. First cut I'd just build ruby from source21:39
naccssbash: and what does snapify.sh contain?21:39
kyrofabdx, I doubt any ruby version manager is going to support deploying that ruby somewhere specific, whereas you can always install ruby in a snap21:39
naccssbash: and if you run snapify.sh from your shell session, does it work?21:40
ssbashnacc: http://paste.ubuntu.com/24851223/21:40
kyrofabdx, for example, rvm supports per-user rubies and system-wide rubies, but not "put that ruby <here>" as far as I know21:40
ssbashnacc: running it from the shell session gives me the same "ImportError: No module named 'wraticus.lib'; 'wraticus' is not a package"21:41
naccssbash: line 2, should be PYTHONPATH=$PYTHONPATH:...21:42
naccssbash: you are removing the dist-packages from your path :)21:43
naccssbash: specifically, the wrapper that's calling your wrapper is (i think) setting the snap's dist-patckage in the path and you're then eliding them21:43
bdxkyrofa: yeah ... I was thinking more along the lines of a simple api via the plugin that manages ruby versions through the version manager21:49
ssbashoh ok21:50
bdxbut yeah possibly not ... I'll just try to get something super basic working21:51
kyrofabdx, well, the plugin (and snapcraft in general) is only involved when creating the initial snap. Once the snap it created, it needs to be standalone21:51
bdxI see21:51
ssbashnacc: ill give it a try21:51
kyrofaSo the ruby must be in the snap at the end, regardless of how you get there21:51
bdxso the plugin cant offfer up an api that can be used throughout the lifecycle of the snap?21:52
bdxI see21:52
kyrofaRight, the plugin is gone by the time the snap is running. Snapcraft builds snaps, and has nothing to do with running them (that's snapd)21:52
bdxpossibly the plugin could grab the ruby version from the Gemfile if it exists then, and just download and build that version of ruby21:54
kyrofaRight21:54
kyrofa(though it'd be easier to just start with the user telling the plugin "give me this version of ruby"21:54
kyrofa)21:54
bdxyeah, ok21:54
ssbashnacc: still not working, I'm getting the same error21:57
ssbashcould it be an issue with my setup.py file? where are python packages supposed to be installed by default21:58
kyrofassbash, have you tried just via pip? Ignoring snaps all together? Does it work then?21:58
nacckyrofa: good enough21:58
nacckyrofa: err, good idea! rather :)21:59
ssbashkyrofa: this package is a local. It's not on pip.21:59
kyrofassbash, I mean the typical: make a virtualenv, source it, pip install .22:00
kyrofaOr setup.py install if you prefer22:00
niemeyercachio: Isn't it super trivial to setup systemd units to restart on crash? I think that's actually the default behavior isn't it?22:00
ssbashkyrofa: it works in my virtualenv.22:01
kyrofassbash, is your code public? Can I take a look?22:02
ssbashits not public, the best I can do is share the directory layout http://paste.ubuntu.com/24852053/22:04
kyrofaYeah that doesn't help I'm afraid22:05
ssbashkyrofa: oh ok thank you for your input22:12
cachioniemeyer, it is not a systemd unit22:39
cachioit is a system v service22:39
niemeyerOkay, so maybe that's the fix? Making it one?22:40
cachioniemeyer, well, that should not be difficult but not sure for other distros22:43
cachioniemeyer, I'll give a try22:44
niemeyercachio: Thanks! It might end up nicely as we won't have to worry about it anywhere else22:45
cachioniemeyer, It is a systemd service too, and it is configured to "no restart" :)22:53
cachioniemeyer, I'll change that22:53
niemeyerSo all you need is an override file22:54
cachioniemeyer, yes22:54
cachioor replace Replace=no with Replace=always22:55
elopiobdx: hello! With kyrofa we made this to get started: https://github.com/travis-ci/travis.rb/pull/515/files22:55
elopiobdx: now we want to turn it into a plugin. Your help there would be awesome.22:55
elopiocjwatson: build.snapcraft.io should be able to find the snapcraft.yaml when it's in the root, right?23:06
elopioah, nevermind. The problem is that it's not in the master branch.23:06
cjwatsonelopio: Yeah I must finish the branch to at least allow a different default23:19

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