| === chihchun_afk is now known as chihchun | ||
| === chihchun is now known as chihchun_afk | ||
| mvo | Chipaca: \o/ for your excellent code review | 05:57 |
|---|---|---|
| clobrano | morning o/ | 06:41 |
| Chipaca | mvo: if you s/FIMXE/FIXME/, it'll get picked up by some tools better | 06:51 |
| mvo | hey clobrano and Chipaca | 06:52 |
| mvo | Chipaca: heh, thanks. sounds like I need more tea :) | 06:52 |
| Chipaca | mvo: how was this not obvious when I clearly said “bä bä svarta får” | 06:52 |
| mvo | :) | 06:54 |
| mvo | fixed | 06:55 |
| Chipaca | ouh | 06:57 |
| Chipaca | one thing | 06:57 |
| Chipaca | mvo: you're using %v with the output of cmd.Output() | 06:57 |
| Chipaca | mvo: that might not be what you want | 06:57 |
| Chipaca | cmd.Output() gives you a []byte | 06:57 |
| Chipaca | and []byte looks like [20 75 124 94 ...] | 06:58 |
| Chipaca | with %v i mean | 06:58 |
| * Chipaca can only handle short sentences this early | 06:58 | |
| Chipaca | mvo: suggest using %s instead, or string(output) (although that creates a third copy) | 06:58 |
| Chipaca | although | 07:00 |
| Chipaca | that'd break your error message | 07:00 |
| Chipaca | because output will probably have \n's in it | 07:01 |
| Chipaca | mwhudson: congrats | 07:02 |
| Chipaca | mvo: not the first time i'm thinking we should hide all this stuff inside helpers somewhere :-/ | 07:02 |
| mvo | Chipaca: uh, good point, let me fix this as well | 07:07 |
| Chipaca | mvo: bottom line would be: either remove the ()s, or %#q and string(output) | 07:07 |
| Chipaca | um | 07:07 |
| Chipaca | %q, not %#q | 07:07 |
| Chipaca | that is: either accept the \n's you'll get with %s, or quote the thing at the expense of some memory | 07:08 |
| Chipaca | it's build anyway, not a long-lived thing :) | 07:08 |
| mvo | Chipaca: indeed, let me add %q | 07:11 |
| Chipaca | and now i'm off to take the boys to school | 07:12 |
| mvo | Chipaca: see you | 07:12 |
| Chipaca | bbi~1h | 07:12 |
| mwhudson | Chipaca: \o. | 07:14 |
| fgimenez | good morning | 07:17 |
| fgimenez | hi mvo, how are you? | 07:25 |
| mvo | hey fgimenez, good morning. I'm good, how are you? | 07:26 |
| fgimenez | mvo, fine, a little cold this morning but anyways :) | 07:27 |
| fgimenez | mvo, i've been working on a logging package for the integration tests, can you have a look when you have the time https://code.launchpad.net/~fgimenez/snappy/integration-tests-verbosity-flag/+merge/273670 ? | 07:27 |
| fgimenez | mvo, no rush at all :) | 07:27 |
| mvo | fgimenez: sure, happy to do that | 07:28 |
| fgimenez | mvo, thanks a lot, specifically if you could address elopio's question it would be great | 07:28 |
| === chihchun_afk is now known as chihchun | ||
| * mvo nods | 07:30 | |
| dholbach | good morning | 07:41 |
| davidcalle | Morning o/ | 07:45 |
| mvo | fgimenez: I commented in the MP, hope it makes sense, please let me know if there are any quesitons | 07:58 |
| fgimenez | mvo, ok thx :) | 07:58 |
| Chipaca | mvo: i have good news and bad news | 08:34 |
| Chipaca | oh, wait | 08:35 |
| * Chipaca checks something | 08:35 | |
| * mvo waits | 08:35 | |
| Chipaca | mvo: i have no bad news | 08:35 |
| mvo | *puhhh* | 08:36 |
| mvo | what was the bad news? | 08:36 |
| Chipaca | mvo: unless "i didn't know %q worked that way with []byte" is bad news, nothing :) | 08:36 |
| Chipaca | so the good news is, i learned that (wonder how many times i learned it before) | 08:36 |
| Chipaca | mvo: now, can we talk about husks? :) | 08:37 |
| mvo | Chipaca: you mean about leightweights ;) | 08:37 |
| mvo | Chipaca: is the other stuff addressed? if so and we can not find a better name husks win | 08:38 |
| Chipaca | mvo: i meant exuvia, sorry | 08:38 |
| Chipaca | "other stuff"? | 08:38 |
| * Chipaca looks | 08:38 | |
| Chipaca | i thought that was all there was | 08:38 |
| mvo | Chipaca: I can't remember if so, then great, let me double check | 08:38 |
| mvo | Chipaca: some inline comments about tiny bits (comments, the one panic) | 08:47 |
| Chipaca | dude, i'd missed those entirely | 08:47 |
| * Chipaca reads | 08:47 | |
| mvo | Chipaca: I like lightweights, so if its not too bothersome and makes stuff not too ugly I'm slightly in favour of changing (but not strongly) | 08:48 |
| davmor2 | mvo: is that because you can't lift the heavy one ;) | 08:53 |
| === chihchun is now known as chihchun_afk | ||
| mvo | davmor2: lol | 09:02 |
| === chihchun_afk is now known as chihchun | ||
| === chihchun is now known as chihchun_afk | ||
| JamesTait | Good morning all; happy Thursday, and happy World Sight Day! 😃 👓 👁 | 09:09 |
| === chihchun_afk is now known as chihchun | ||
| Chipaca | mvo: question for you | 09:27 |
| Chipaca | mvo: was going with lightweight.PartList | 09:27 |
| Chipaca | mvo: but then i thought, does that make you think you can range over it? should i call it a PartBag instead? | 09:27 |
| Chipaca | mvo: or did you mean just call it lightweight.Lightweight? | 09:28 |
| mvo | Chipaca: hm, PartBag sounds ok, PartList is probably ok as well, but you are right, there is this association with a real list then | 09:33 |
| * Chipaca nods | 09:34 | |
| Chipaca | mvo: hmmmmmm | 09:47 |
| Chipaca | mvo: LoadActive() is “return h.Load(h.ActiveIndex())” | 09:47 |
| Chipaca | mvo: not sure it's worth it :) | 09:47 |
| mvo | Chipaca: indeed, wasn't there error checking on the index before? or am I misrembmering? | 09:49 |
| mvo | definitely mistyping ;) | 09:49 |
| * Chipaca looks at merged-list | 09:49 | |
| Chipaca | mvo: there is error checking | 09:50 |
| Chipaca | hmm | 09:50 |
| Chipaca | mvo: so LoadActive could return nil,nil if activeIndex<0, or nil, ErrNoActiveIndex or sth | 09:51 |
| Chipaca | either way the caller would have to handle it differently | 09:51 |
| mvo | Chipaca: aha, ok, in this case its not worth it I agree | 09:52 |
| Chipaca | that is, today: idx:=h.ActiveIndex(); if idx<0{not found}; if h.load(idx) errors { internal error } | 09:52 |
| mvo | Chipaca: I just noticed in the MP what looked like repeating code that could be done in a single place, but if it actually can't … | 09:52 |
| Chipaca | with active index: part,err:=h.LoadActive(); if err=NoActive{not found} else if err != nil {internal} | 09:52 |
| Chipaca | mvo: promise i'll revisit it in a month to see how we're actually using it and add convenience methods :) | 09:55 |
| * Chipaca adds a calendar thing | 09:55 | |
| mvo | Chipaca: deal! | 09:55 |
| Chipaca | mvo: i think that addresses all your issues, please take a look when you cna | 10:12 |
| Chipaca | can* | 10:12 |
| sergiusens | morning | 10:12 |
| Chipaca | uia! un sergio! | 10:15 |
| Chipaca | sergiusens: morning | 10:15 |
| mvo | Chipaca: thanks, will do | 10:16 |
| mvo | sergiusens: good morning | 10:16 |
| Chipaca | mvo: ah, one question about the squashfs build branch | 10:17 |
| Chipaca | mvo: i heard something about moving build to snapcraft | 10:17 |
| Chipaca | mvo: is that accurate? | 10:17 |
| mvo | Chipaca: yes, eventually | 10:18 |
| mvo | Chipaca: we still need something for our tests in snappy I think | 10:18 |
| Chipaca | ok | 10:18 |
| sergiusens | Chipaca, mvo the code base for lp:snappy looks so different 5 days past | 10:19 |
| Chipaca | mvo: yes, but once snapcraft does build, let's not have two implementations of it :) | 10:19 |
| mvo | Chipaca: yeah, I agree, at the same time I don't want to be distracted by this just now :) | 10:19 |
| * Chipaca nods | 10:20 | |
| Chipaca | speaking of changes to lp:snappy, how do you remove tags from it? | 10:20 |
| Chipaca | i added tags when playing with them, removed them, but they have appeared upstream somehow, and no amount of removing them makes them go away :-( | 10:21 |
| sergiusens | Chipaca, as in bzr tags? | 10:21 |
| Chipaca | yes | 10:21 |
| sergiusens | Chipaca, --overwrite I think, it is extremely complicated | 10:21 |
| mvo | does bzr tag --delete work? | 10:21 |
| Chipaca | it works locally | 10:21 |
| Chipaca | but --overwrite-tags says it's got nothing to do | 10:21 |
| mvo | but not with a lp prefix? | 10:21 |
| mvo | Chipaca: what tag should go? | 10:22 |
| Chipaca | mvo: " " and "\t" | 10:22 |
| sergiusens | Chipaca, interesting tags, those should stay :-P | 10:22 |
| Chipaca | sergiusens: um. Ok. Don't --print-ids then. | 10:22 |
| Chipaca | --show-ids i mean | 10:23 |
| Chipaca | whaaaat | 10:48 |
| Chipaca | no LastIndexByte before 1.5 | 10:49 |
| Chipaca | sigh | 10:49 |
| * Chipaca fixes | 10:49 | |
| === chihchun is now known as chihchun_afk | ||
| Chipaca | hmm | 11:17 |
| Chipaca | tests pass here, with go 1.3. fail in tarmac. map output for failed tests is identical for obtained/expected. | 11:18 |
| Chipaca | wait, they're not all identical | 11:20 |
| Chipaca | differing keys: | 11:20 |
| Chipaca | 'installed_size': expected '208', got '8260' | 11:20 |
| Chipaca | wtf | 11:20 |
| * guest42315 EWWWW mailing lists | 11:34 | |
| Chipaca | guest42315: ? | 11:35 |
| guest42315 | Chipaca, oh sorry /me /ame | 11:36 |
| Chipaca | sergiusens: ping | 11:38 |
| sergiusens | Chipaca, pong | 11:39 |
| Chipaca | sergiusens: any ideas wrt tests failing in tarmac in weird ways? | 11:39 |
| sergiusens | Chipaca, weird ways, no; what are you running? | 11:41 |
| Chipaca | sergiusens: in snappy, run-checks locally passes fine, every time, with go 1.5 and 1.3, starting from a pristine directory (just like tarmac does it) | 11:42 |
| Chipaca | sergiusens: in tarmac, https://code.launchpad.net/~chipaca/snappy/husk/+merge/273482/comments/690882 | 11:43 |
| Chipaca | sergiusens: 4 failed tests, of which 1 the obtained/expected mappings are apparently identical | 11:44 |
| Chipaca | sergiusens: and the others differ in "installed_size", with 8k extra on disc | 11:44 |
| Chipaca | oh, wait, 8k extra | 11:44 |
| Chipaca | hmm | 11:45 |
| sergiusens | Chipaca, 'snappy unpack' and snappy installed on tarmac | 11:45 |
| sergiusens | Chipaca, could it be that? | 11:45 |
| sergiusens | Chipaca, not sure how unit the unit test is :-) | 11:45 |
| Chipaca | the 8k might be because i use tmpfs | 11:45 |
| Chipaca | snappy unpack not involved | 11:45 |
| sergiusens | Chipaca, don't use ue tmpfs locally and why would it be different? | 11:46 |
| sergiusens | if both places use tmpfs | 11:46 |
| Chipaca | sergiusens: in tmpfs a directory's du is 0, an empty file's du is 0 | 11:47 |
| Chipaca | sergiusens: outside of tmpfs, both of those are 4k | 11:47 |
| Chipaca | or 2k or 16k | 11:47 |
| Chipaca | block size | 11:47 |
| Chipaca | dangit :( | 11:47 |
| Chipaca | sergiusens: thank you! | 11:48 |
| sergiusens | Chipaca, you did it all by yourself ;-) | 11:49 |
| Chipaca | sergiusens: you're a good sounding board | 11:50 |
| biezpal | Hi all! Is it hard to make our oem snap packages signed? :) | 11:50 |
| Chipaca | biezpal: put them in the store? | 11:53 |
| biezpal | Chipaca, is it enough to make it signed? actually, we just don't like "sideload" prefix :) | 11:55 |
| biezpal | and random string in version field.. | 11:55 |
| biezpal | because of https://bugs.launchpad.net/snappy/+bug/1498396 | 11:56 |
| ubottu | Launchpad bug 1498396 in Snappy "Random string in data path breaks application" [Undecided,New] | 11:56 |
| Chipaca | oh, bug 1498396 | 11:57 |
| ubottu | bug 1498396 in Snappy "Random string in data path breaks application" [Undecided,Fix released] https://launchpad.net/bugs/1498396 | 11:57 |
| Chipaca | i knew there was a bug about it but didn't find it when making the branch. fixed, anyway. | 11:58 |
| Chipaca | biezpal: data dir now has a 'current' link | 11:58 |
| biezpal | Chipaca, in current edge this bug is still exists | 11:58 |
| Chipaca | biezpal: anyway, getting it from the store is (for now) the only way to get it auth'ed | 11:58 |
| Chipaca | biezpal: really? | 11:59 |
| Chipaca | maybe i should've checked first :-/ | 11:59 |
| * Chipaca checks now | 11:59 | |
| soffokl | Chipaca, we are build it from edge channel | 12:00 |
| soffokl | sudo ubuntu-device-flash core 15.04 --channel edge --oem parallella_4.0_all.snap --device-part=device.tar.xz -o parallella1.img | 12:00 |
| Chipaca | soffokl: biezpal: 15.04? | 12:00 |
| Chipaca | yes, 15.04 | 12:00 |
| Chipaca | sigh | 12:00 |
| Chipaca | needs backporting i guess | 12:00 |
| Chipaca | i was sure that was included in the latest 15.04 | 12:01 |
| Chipaca | ah. no. it's for the next one. i remember now. | 12:01 |
| biezpal | for the next one edge?) | 12:02 |
| Chipaca | i mean: it's in rolling/edge | 12:04 |
| Chipaca | i guess it's not on stable | 12:04 |
| Chipaca | gah | 12:04 |
| Chipaca | i mean not on 15.04 | 12:04 |
| Chipaca | biezpal: if youc reate the symlink by hand for now, things'll sort themselves out | 12:05 |
| biezpal | Chipaca, sigh | 12:08 |
| biezpal | thanks | 12:08 |
| Chipaca | $ ls -l /var/lib/apps/hello-world.canonical/ | 12:08 |
| Chipaca | total 4 | 12:08 |
| Chipaca | drwxr-xr-x 2 root root 4096 Oct 8 12:03 1.0.18 | 12:08 |
| Chipaca | lrwxrwxrwx 1 root root 6 Oct 8 12:04 current -> 1.0.18 | 12:08 |
| Chipaca | biezpal: ^ :) | 12:09 |
| biezpal | Chipaca, handmade link? :) | 12:09 |
| Chipaca | biezpal: no, rolling edge | 12:09 |
| Chipaca | biezpal: s/15.04/rolling/ in your u-d-f command above and you get that | 12:10 |
| Chipaca | but rolling is the one we *actively* break :) | 12:10 |
| Chipaca | so beware | 12:10 |
| ogra_ | we should really default to do development in 15.04 and forward port instead of the other way round | 12:11 |
| biezpal | Chipaca, ok, i guess we can use rolling for development purposes | 12:11 |
| sergiusens | Chipaca, can I get your opinion on https://code.launchpad.net/~sergiusens/snapcraft/1500505/+merge/273826 ? | 12:12 |
| Chipaca | sergiusens: you can try :) | 12:12 |
| sergiusens | Chipaca, I don't like the MP, but I don't know what else to do | 12:13 |
| Chipaca | sergiusens: i guess the removing the dangling one if unresolved is the bit you don't like? | 12:14 |
| Chipaca | sergiusens: BTW, $ dpkg -S /usr/lib/jvm/java-7-openjdk-amd64/jre/lib/security/cacerts | 12:15 |
| Chipaca | openjdk-7-jre-headless:amd64: /usr/lib/jvm/java-7-openjdk-amd64/jre/lib/security/cacerts | 12:15 |
| sergiusens | Chipaca, oh, then that is fine and covered, what about libc6? | 12:17 |
| sergiusens | Chipaca, I might have been having issues with ':' | 12:17 |
| sergiusens | Chipaca, and wait | 12:19 |
| sergiusens | Chipaca, you just confused me, the thing it links to is what I can't figure out where it comes from | 12:19 |
| sergiusens | Chipaca, as in /etc/ssl/certs/java/cacerts | 12:19 |
| Chipaca | ca-certificates-java i'd assume | 12:19 |
| Chipaca | oh, oh, oh | 12:20 |
| Chipaca | i know what's missing here | 12:20 |
| sergiusens | Chipaca, nope | 12:20 |
| sergiusens | Chipaca, not provided by ca-certificates-java | 12:21 |
| Chipaca | ok, need to dig a bit more | 12:21 |
| Chipaca | but | 12:21 |
| Chipaca | you need to loop | 12:21 |
| sergiusens | to loop? | 12:21 |
| Chipaca | or does realpath do that? | 12:22 |
| Chipaca | if it's a symlink-to-a-symlink | 12:22 |
| sergiusens | Chipaca, oh, I can try (s/realpath/readlink/ I suppose) | 12:23 |
| Chipaca | realpath might resolve it; check (or read docs) | 12:25 |
| Chipaca | so | 12:25 |
| Chipaca | that file | 12:25 |
| Chipaca | is *probably* created by ca-certificates-java | 12:25 |
| Chipaca | via /etc/ca-certificates/update.d/jks-keystore | 12:25 |
| Chipaca | sergiusens: by running /usr/share/ca-certificates-java/ca-certificates-java.jar | 12:26 |
| Chipaca | sergiusens: realpath loops | 12:28 |
| Chipaca | sergiusens: you're set | 12:28 |
| sergiusens | Chipaca, this MP won't work :-) | 12:28 |
| Chipaca | still trying to understand it tbh :) | 12:28 |
| Chipaca | ah! | 12:29 |
| sergiusens | Chipaca, look at the bug | 12:29 |
| Chipaca | sergiusens: so, yes, change the first readlink to realpath | 12:29 |
| Chipaca | ... or something! i dunno | 12:29 |
| Chipaca | programming is complicated, i'm going to have a cuppa tea | 12:29 |
| sergiusens | Chipaca, so we don't run postinst for 'stage-packages' | 12:29 |
| sergiusens | Chipaca, and jdstrand is not using any of the java plugins | 12:30 |
| sergiusens | Chipaca, just downloading debs, so he doesn't have the certs; and I can't just copy because a bare bones system won't have them either | 12:30 |
| Chipaca | right | 12:30 |
| sergiusens | Chipaca, then there is libc6; do we want those libs in the snap or should they be allowed as dangling symlinks? | 12:31 |
| Chipaca | sergiusens: libc6 is problematic, because nss needs to be exactly the same | 12:35 |
| Chipaca | sergiusens: so copying it in won't work | 12:35 |
| Chipaca | sergiusens: that's how you get segfaults or, worse, weird behaviour | 12:35 |
| sergiusens | Chipaca, right, so I need to circle back with jdstrand to get more smarts in the click-review tool and also make sure that using 'stage-packages' requires hand holding in some cases | 12:37 |
| biezpal | Chipaca, cat /etc/lsb-release | 12:39 |
| biezpal | DISTRIB_ID=Ubuntu | 12:39 |
| biezpal | DISTRIB_RELEASE=15.10 | 12:39 |
| biezpal | DISTRIB_CODENAME=wily | 12:39 |
| biezpal | DISTRIB_DESCRIPTION="Ubuntu Wily Werewolf (development branch)" | 12:39 |
| biezpal | (Parallella)ubuntu@localhost:~$ ls -l /var/lib/apps/parallella/ | 12:39 |
| biezpal | total 4 | 12:39 |
| biezpal | drwxr-xr-x 2 root ubuntu 4096 Oct 8 2015 ICaVaRAVXWYS | 12:39 |
| ubottu | Ubuntu bug 4096 in meld (Ubuntu) "meld: merge new debian version" [Medium,Fix released] https://launchpad.net/bugs/4096 | 12:39 |
| soffokl | sudo ubuntu-device-flash core rolling --channel edge --oem parallella_4.0_all.snap --developer-mode --device-part=device.tar.xz -o parallella2.img | 12:40 |
| Chipaca | ... | 12:40 |
| Chipaca | interesting :-/ | 12:40 |
| Chipaca | wait | 12:41 |
| Chipaca | that parallela package | 12:41 |
| Chipaca | that was installed by u-d-f | 12:41 |
| Chipaca | and that u-d-f is probably not recent enough to include these changes? | 12:41 |
| Chipaca | soffokl: apt-cache policy ubuntu-device-flash if you please | 12:41 |
| soffokl | ubuntu-device-flash: | 12:42 |
| soffokl | Installed: 0.31-0ubuntu1 | 12:42 |
| soffokl | Candidate: 0.31-0ubuntu1 | 12:42 |
| soffokl | Version table: | 12:42 |
| soffokl | *** 0.31-0ubuntu1 0 | 12:42 |
| soffokl | 500 http://ppa.launchpad.net/snappy-dev/tools/ubuntu/ vivid/main amd64 Packages | 12:42 |
| soffokl | 100 /var/lib/dpkg/status | 12:42 |
| soffokl | 0.20-0ubuntu1 0 | 12:42 |
| soffokl | 500 http://kg.archive.ubuntu.com/ubuntu/ vivid/universe amd64 Packages | 12:42 |
| sergiusens | Chipaca, meh, ca-certificates-java.jar harcodes to /etc :/ | 12:46 |
| Chipaca | sergiusens: when/how does u-d-f get rebuilt? | 12:47 |
| Chipaca | mvo: ok to top-approve merged-list? | 12:54 |
| mvo | yes, done so | 12:57 |
| Chipaca | mvo: no files were sneaked in to that branch last minute* | 13:00 |
| mvo | :P | 13:00 |
| Chipaca | * or so you think | 13:01 |
| Chipaca | snuck, not sneaked. sigh. | 13:01 |
| * ogra_ hands Chipaca a pair of sneakers | 13:02 | |
| * Chipaca snickers | 13:02 | |
| ogra_ | yummy peanuts and caramel | 13:03 |
| * ogra_ wishes he could eat that :( | 13:03 | |
| jdstrand | sergiusens: I guess you are saying some things should symlink out (libc6) but some things shouldn't (cacerts) | 13:03 |
| jdstrand | sergiusens: if you file a bug against the click-reviewers-tools project, I'll fix it. I have a couple of other things to fix this week | 13:04 |
| jdstrand | sergiusens: just let me know what is a legitimate symlink | 13:04 |
| Chipaca | ogra_: you want one of these: http://pickupthefork.com/wp-content/uploads/2014/10/IMG_8754.jpg | 13:05 |
| ogra_ | mean ! | 13:05 |
| ogra_ | (no sugar for me for another week .... ) | 13:06 |
| Chipaca | ogra_: oh! i thought it was the nuts you couldn't have | 13:07 |
| ogra_ | well, i cant bite nuts :) | 13:07 |
| ogra_ | (thats another 4 weeks) | 13:08 |
| Chipaca | hence why i thought one of those would give you the nuts+caramel thing without having to bite nuts | 13:08 |
| ogra_ | heh | 13:08 |
| Chipaca | see, i'm so thoughtful it hurts | 13:08 |
| sergiusens | jdstrand, ok, it's mostly libc6 things as per what we discussed with Chipaca; for cacerts, I'm solving it in the jdk plugin, the error for it is fine, but I can't fix it in a generic way for stage-packages uses because some of these are postinst hook created. | 13:11 |
| ogra_ | yay, maintainer scripts | 13:11 |
| ogra_ | :P | 13:11 |
| jdstrand | sergiusens: won't the jdk plugin pull in a bunch of stuff I don't need? | 13:12 |
| jdstrand | sergiusens: why can't you examine the symlinks in the directory? perhaps you could prompt to include them from the system? | 13:13 |
| sergiusens | jdstrand, I have as a first approach https://code.launchpad.net/~sergiusens/snapcraft/1500505/+merge/273826 | 13:14 |
| sergiusens | jdstrand, I just feel it would be weird | 13:14 |
| jdstrand | sergiusens: at the very least, you should detect them and report them with suggestions on how to fix them, because if you don't then snapcraft will generate a package that fails review. the review tools could be the place to detect them and make a suggestion if you'd prefer | 13:15 |
| jdstrand | but it seems like snapcraft is in a position to fix things up | 13:15 |
| mvo | Chipaca: there are more branches ftw, just in case you are bored ;) | 13:15 |
| Chipaca | i saw nothing | 13:16 |
| mvo | fgimenez, elopio: could one of you install squashfs-tools on the tarmac server please? | 13:16 |
| jdstrand | eg, "oh, cacerts is pointing to something in /etc. let me copy what is in /etc to ./snap/etc and adjust the symlink | 13:16 |
| jdstrand | " | 13:16 |
| jdstrand | sergiusens: ^ | 13:16 |
| sergiusens | jdstrand, yeah, I'll go with that branch and polish a bit to not include libc6 links | 13:16 |
| sergiusens | Chipaca, u-d-f, when you dput it | 13:17 |
| jdstrand | sergiusens: cool. btw, I don't care if you copy in place (your mp) or copy to the intended place and adjust the link (what I suggested). however, what I suggested is probably more robust if you have multiple symlinks to the same thing | 13:18 |
| sergiusens | Chipaca, ideally the ubuntu-snappy stuff goes to the archive and then gets rebuilt and it does so prior to a release | 13:18 |
| sergiusens | jdstrand, right, it could save more space, not sure if it would when everything is squashfs | 13:19 |
| fgimenez | mvo, sure, give me a minute | 13:25 |
| fgimenez | mvo, done http://paste.ubuntu.com/12714696/ | 13:28 |
| Chipaca | ogra_: did you get too track down the ipv6-only thing? | 13:33 |
| Chipaca | (were you [going to be] doing that?) | 13:33 |
| ogra_ | ipv6-only ? | 13:34 |
| * ogra_ hasnt heard of it | 13:34 | |
| Chipaca | fgimenez: elopio: you might want to tell ogra_ about it :) | 13:37 |
| Chipaca | ogra_: rpi2 was coming up with only an ipv6 address (and i think fgimenez got it in kvm also once, but not consistent) | 13:41 |
| Chipaca | ogra_: but i don't know the details | 13:41 |
| ogra_ | weird | 13:43 |
| fgimenez | ogra_, this is the bug https://bugs.launchpad.net/snappy/+bug/1503329 | 13:48 |
| ubottu | Launchpad bug 1503329 in Snappy "not getting an ipv4 address on the first boot" [Undecided,Confirmed] | 13:48 |
| ogra_ | fgimenez, well, there was a kernel update between 186 and 187 i think | 13:51 |
| ogra_ | (no idea if related or not) | 13:54 |
| tedg | sergiusens: So I know we wanted to get rid of the situation where plugins subclass other plugins. | 14:21 |
| tedg | sergiusens: Did we also want to get rid of 'requires' in the plugin yaml? | 14:21 |
| sergiusens | tedg, hmm, wasn't aware of subclassing | 14:24 |
| sergiusens | tedg, as we discussed this last week https://code.launchpad.net/~sergiusens/snapcraft/1500902/+merge/273444 | 14:24 |
| sergiusens | still needs a round of testing for local plugins, but works fine otherwise | 14:25 |
| tedg | sergiusens: So I think that's fine, but the specific case here is a plugin that needs Python3, including PIP support, so it doesn't make sense to copy-and-past. | 14:26 |
| tedg | paste (should have cut-and-pasted that from somewhere) | 14:26 |
| tedg | sergiusens: So it'd like to basically have the python3 plugin run first for everything, which is kinda what requires did. | 14:26 |
| sergiusens | tedg, we have plugins already that base out of python3 | 14:26 |
| sergiusens | tedg, oh, the requires problem and pull phases problem is a problem of the past; it is fine if a pull phase for every part can't be completed due to an 'after' | 14:27 |
| sergiusens | tedg, in the end, lp should not be a blocker (words out of beuno's mouth ;-) ) | 14:28 |
| sergiusens | Chipaca, mind taking another look at https://code.launchpad.net/~sergiusens/snapcraft/1500505/+merge/273826 ? | 14:29 |
| sergiusens | Chipaca, this time it is better | 14:31 |
| sergiusens | ogra_, why did you break my 2fa? :-P | 14:31 |
| ogra_ | sergiusens, well, i was bored y'know | 14:32 |
| ogra_ | (what do you mean ? ) | 14:32 |
| ogra_ | sergiusens, do you mean the authenticator app breakage ? | 14:34 |
| * ogra_ fully blames bzoltan_ for that :P | 14:34 | |
| * bzoltan_ takes the blame ... do you want to punish me ;P | 14:35 | |
| ogra_ | i guess he will throw argentinian steaks at you :) | 14:35 |
| sergiusens | ogra_, hah, it is my 2fa device so now I am sort of in limbo :-P | 14:36 |
| matiasb | Chipaca, jdstrand, o/ good news, I managed to work-around transmission issues with mount/quotactl by using statvfs, and it works! I'll be uploading the fixed version later today :) | 14:40 |
| jdstrand | matiasb: great! :) | 14:40 |
| ogra_ | yay | 14:41 |
| matiasb | jdstrand, btw, re random/uuid, everything seems to work ok even if that is denied by apparmor; I was thinking if I should I upload the package without any profile customization then? once the perm is added to the default template, future installs will get access? (it seems the profile is generated from the default template on install, right?) | 14:43 |
| jdstrand | matiasb: yes on all counts. if it works fine, but remove security-policy. it will use the default template with network-client. you'll have the denial until I do the upload | 14:44 |
| jdstrand | s/but remove/then remove/ | 14:44 |
| matiasb | jdstrand, great, that's my plan then; that should go through review without any issues and without requiring manual intervention | 14:45 |
| jdstrand | matiasb: with that change and using 'architectures' in your package.yaml, it will pass automated review | 14:45 |
| jdstrand | yep | 14:45 |
| matiasb | correct | 14:45 |
| Chipaca | matiasb: you're building it multi-arch, yes? pretty please? | 14:51 |
| matiasb | Chipaca, yeap :) | 14:51 |
| Chipaca | sweet | 14:52 |
| Chipaca | pitti: you around? question about removing a service from systemd's status | 14:53 |
| pitti | hey Chipaca | 14:53 |
| Chipaca | pitti: hi! | 14:53 |
| Chipaca | pitti: right now, if you have a snap with a service, and upgrade it, the old service never goes away from systemctl | 14:54 |
| Chipaca | pitti: are we doing something wrong? | 14:54 |
| Chipaca | we stop the service, disable it, reload-daemon | 14:54 |
| pitti | Chipaca: you mean you don't stop the old service before upgrading? | 14:54 |
| Chipaca | we stop it | 14:54 |
| pitti | ah, then it should be stopped, no? | 14:54 |
| Chipaca | we disable it | 14:54 |
| Chipaca | we remove the service file | 14:55 |
| Chipaca | we do a reload-daemon or daemon-reload or whatever that was | 14:55 |
| pitti | daemon-reload | 14:55 |
| Chipaca | but it's still there in the systemctl output | 14:55 |
| pitti | Chipaca: with --all? yes, that's right | 14:55 |
| pitti | shoudl be inactive/dead then, or is it still running? | 14:55 |
| Chipaca | inactive/dead/not found | 14:56 |
| Chipaca | not with --all though | 14:56 |
| pitti | this happens a lot, particularly with instantiated units (foo@.service) | 14:56 |
| pitti | Chipaca: hm, by default "systemctl" only lists active units -- if it's listed there as inactive that's a bug indeed | 14:56 |
| Chipaca | systemctl show, not status, sorry | 14:56 |
| pitti | ah | 14:56 |
| pitti | you can still query e. g. the journal from the old units, or their status | 14:57 |
| sergiusens | Chipaca, another smaller one when you feel like dropping a finger on the approve button https://code.launchpad.net/~sergiusens/snapcraft/1504174/+merge/273853 | 14:57 |
| Chipaca | pitti: oh, wait, i spotted the bug. and it's mine. sorry to bug you :-/ | 14:58 |
| Chipaca | pitti: wouldn't've spotted it without you though, so thanks ! :) | 14:58 |
| pitti | heh | 14:59 |
| pitti | Chipaca: so, this might just be a cosmetical thing -- systemctl status anything.service will say "not found" | 14:59 |
| pitti | (and exit with 3) | 14:59 |
| Chipaca | pitti: i was getting a list of all installed snaps | 15:00 |
| pitti | Chipaca: except that for services which did exist in the past you still get the old journal | 15:00 |
| Chipaca | pitti: not filtering by "active" | 15:00 |
| Chipaca | so i'm *asking systemctl for it myself* | 15:00 |
| pitti | with completely bogus names you just get not-found | 15:00 |
| * Chipaca puts the brown paper bag back on | 15:00 | |
| * pitti pats Chipaca, no worries :) | 15:00 | |
| ogra_ | sergiusens, do we have a "null" plugin now ? | 15:36 |
| ogra_ | or do i still have to abuse the copy plugin (and copy an empty README file) to just get some deb binary content | 15:36 |
| ogra_ | (i thinnk someone said that was planned) | 15:37 |
| elopio | fgimenez: would it be too hard to add levels to the snappy/log ? | 15:40 |
| fgimenez | elopio, not sure, i can try in a branch | 15:42 |
| elopio | fgimenez: it would be awesome if we could make this an independent library. But just if the effort required is not too much. | 15:42 |
| elopio | fgimenez: also, there's https://github.com/golang/glog | 15:44 |
| fgimenez | elopio, ok thanks, i'll take a look | 15:47 |
| sergiusens | ogra_, it was an idea, but is there any case where that would produce a good snap? | 16:26 |
| ogra_ | http://bazaar.launchpad.net/~ogra/+junk/htop-unconfined/view/head:/snapcraft.yaml | 16:26 |
| sergiusens | jdstrand, this should solve minecraft (except for libc6 libs) https://code.launchpad.net/~sergiusens/snapcraft/1500505/+merge/273826 | 16:26 |
| ogra_ | sergiusens, ^^^ thats what i use now to quickly pack a tool if i need it ... | 16:27 |
| sergiusens | ogra_, heh, the 'nop' plugin I need to propose | 16:27 |
| ogra_ | its just silly to have an empty README file to copy around just to get the staging packag | 16:27 |
| ogra_ | e | 16:27 |
| ogra_ | i mean it doesnt bother me to do it like that, it just feels a little wrong inside :) | 16:28 |
| elopio | this is cool https://github.com/bluele/factory-go | 16:30 |
| mvo | elopio: nice! | 17:36 |
| jdstrand | sergiusens: thanks! I assigned the crt bug to me. I'll fix it tomorrowish | 18:47 |
| sergiusens | elopio, I hope you like this http://bazaar.launchpad.net/~sergiusens/snapcraft/1500902/revision/235 (I am finally getting rid of most of the getattr's in there too) | 20:27 |
| * sergiusens heads out for some errands | 20:27 | |
| === mwenning is now known as mwenning-appt | ||
| tedg | sergiusens: I'm having a heck of an issue getting pip to install the aws cli binary | 21:19 |
| tedg | sergiusens: Any ideas there. It installs the library, but not the actual thing in /usr/bin | 21:20 |
| tedg | Considering just working around it at this point, but seems like defeat. | 21:20 |
| sergiusens | tedg, does the setup.py declare any scripts? | 22:01 |
| sergiusens | tedg, can you point be to the package name? | 22:01 |
| tedg | sergiusens: It's the awscli from pypi | 22:05 |
| tedg | Let me look for the setup.py | 22:05 |
| tedg | Oh, oops. I did a snapcraft clean. Give me a sec to have that :-) | 22:07 |
| tedg | sergiusens: Doesn't seem to have a setup.py | 22:12 |
| sergiusens | tedg, it works fine in a pyvenv | 22:27 |
| tedg | Hmm, bother. | 22:27 |
| sergiusens | tedg, it has scripts in setup.py, it should just work | 22:30 |
| tedg | sergiusens: I gonna have to look more tomorrow, seems so simple, sure it's a one character change. | 22:30 |
| sergiusens | tedg, which also makes me wonder, have you tried making the python plugins work with pyvenv? | 22:30 |
| tedg | sergiusens: we had wanted to avoid that, apparently it changes a lot, mostly stuff that the python interpreter should do on it's own. But apparently makes things weird for other cases. | 22:31 |
| tedg | sergiusens: So, no, but I was asked to avoid it. | 22:31 |
| sergiusens | tedg, ok; seems like we should use it from what others have said | 22:32 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
