/srv/irclogs.ubuntu.com/2015/10/02/#snappy.txt

plarselopio: yes00:49
elopioplars: could you please check what's the latest error?00:50
elopioI'm not yet catching the good stuff in the payload. Butt when I test it here it works.00:51
plarselopio: sure, just a minute00:55
plarselopio: this seems to be the interesting bit: cat: /results/output/log: No such file or directory\nYou are trying to send an empty document, exiting.\ncat: /results/output/summary: No such file or directory\n./snappy-tests-job.sh: 9: ./snappy-tests-job.sh: cannot create /spi_test_result.json: Permission denied00:59
elopioplars: hum. The docs say that there is an env var $CWD.00:59
elopioI was using that. I'll take it out.00:59
elopiothanks. Another try...01:00
plarselopio: $PWD perhaps?01:00
elopioplars: no, the docs say $CWD. But it's not entirely clear what they meant with that.01:01
plarselopio: which docs?01:01
elopioplars: spi-user-docs-latest.pdf01:02
plarscould be a misprint, but I would think $PWD should work01:02
* elopio needs better bash superpowers.01:45
yashi_regarding to bug #1499429, what's a proper workflow to reflect review comment to my launchpad branch?  force push or another branch?01:54
ubottubug 1499429 in Snapcraft "snapcraft dies with "export: python3.5/dist-packages: bad variable name"" [Critical,Confirmed] https://launchpad.net/bugs/149942901:55
elopioyashi_: hello.02:14
elopioI replied on the merge proposal. Just make the change, bzr commit and bzr push.02:15
yashi_elopio: thanks!02:15
elopioyou use the same branch, but are not forcing anything. It will keep the previous commits.02:15
elopioyashi_: thanks a lot for finding this, and fixing it. Have you signed the contributors agreement?02:16
elopiohttp://www.ubuntu.com/legal/contributors/submit02:16
yashi_elopio: nop, reading....02:18
elopioyashi_: Here's the faq: http://www.ubuntu.com/legal/contributors/licence-agreement-faq02:19
elopioplease let me know if you want to know something else.02:19
yashi_elopio: the submission ask me to put a project contact.  what is it?02:40
yashi_elopio: hmm.  do i need to print the pdf, sign it, and snail back to canonical?02:43
elopioyashi_: the contact is asac. Alexander Sack <alexander.sack@canonical.com>02:44
elopioyashi_: and no, afaik you don't have to send the pdf. Let me confirm.02:44
elopioyashi_: it says that you can send the pdf if you don't have a launchpad account. You are good with filling the form.02:46
elopiosigh, I suck at bash. I can't even capture the error.03:20
elopioplars: again, please, whenever you see this.03:20
yashi_elopio: done03:48
elopioyashi_: thank you!03:49
elopiowelcome to snapcraft. If you ever get bored, we have tons of things funny things you can do ;)03:50
yashi_elopio: will push a fix today03:50
=== chihchun_afk is now known as chihchun
elopiosergiusens: hey, you are up.06:11
elopiosergiusens: these are the problems I had:06:11
elopiottps://bugs.launchpad.net/snapcraft/+bug/149942906:11
elopiohttps://bugs.launchpad.net/snapcraft/+bug/150198006:11
elopiohttps://bugs.launchpad.net/snapcraft/+bug/150197706:11
ubottuLaunchpad bug 1501980 in Snapcraft "the error message when a plugin is not found is not good" [Undecided,New]06:11
ubottuLaunchpad bug 1501977 in Snapcraft "py2-project example fails to snap" [High,New]06:11
elopioother than that, looks good.06:12
elopiohaven't tried the minecraft one yet.06:12
=== bigcat__ is now known as bigcat
clobranomorning :)06:50
fgimenezgood morning07:04
davidcalleasac, thanks for the heads-up, that's an impressive team effort :) Are there any sections you'll still want to keep under wraps or can I assume (after today eod) everything is ready for a first publication?07:25
dholbachgood morning07:32
davidcalleHey dholbach, how are you?07:35
dholbachhey davidcalle07:36
dholbachgood good - how about you?07:36
davidcalledholbach, life is good07:36
davidcalledholbach, nice work with the snappy doc propositions, I agree with everything :)07:38
dholbachyeah, I was trying to find a balance between: more useful for certain demographics and not too much new maintenance work for us :)07:38
dholbach... and leaving IA intact07:39
clobranoChipaca: I'll drop here my hw-assign question (I wrote on the mailing list too), please have a look at it, when you've got time :)08:36
JamesTaitGood morning all; happy Friday, and happy International Day of Non-Violence! 😃08:37
clobranoChipaca: about the possibility to assign a symlink to a device node using hw-assign command (Bug #1496319), I had everything done and ready to be submitted for something like: "snappy hw-assign <device> <symlink>"  but, then, I thought that it's more common to create a symlink only if some conditions are met (like VID/PID, etc.), so I've got the idea to change it in: "hw-assign <device> <properties>" where <properties> would be a string containing part of08:37
clobranothe Udev rule to append to the newly created rule, like "ATTRS{idVendor}==...". This way one can add different rules, not only symlinks. What do you think?08:37
ubottubug 1496319 in Snappy "Could not create symlink to hw device with udev rules" [Undecided,New] https://launchpad.net/bugs/149631908:37
Chipacaclobrano: hmm. first thought is whether udev has been audited enough to have people safely write rules08:53
clobranoChipaca: yep, that was my first idea too08:54
clobranoChipaca: like with great power comes great responsability :)08:54
clobranoChipaca: but at the same time, it's common symlinking to a generic /dev/ttyUSB* and then add conditions to narrow down the list of possible device nodes08:55
loolvmayoral|pc: ssh root@212.47.227.20508:56
clobranoChipaca: however, that things can be done at different times. First try and add the first implementation and then see if/when we can move to the second one08:56
=== vmayoral|pc is now known as vmayoral
Chipacaclobrano: so which would the first implementation be?08:59
clobranoChipaca: just hw-assign <device-node> <symlink>08:59
clobranoChipaca: where <symlink> is optional09:00
asacdavidcalle: not sure yet. lets see how it looks like09:03
Chipacaclobrano: and how would the move to the second phase work?09:04
looltedg, sergiusens: ssh root@212.47.227.205 for the ARM buildd09:05
loolvivid, installing snapcraft there now09:05
clobranoChipaca: that's depend on you all I think :), I can't decide if/when that change will be applicable09:05
Chipacaclobrano: no, i mean09:05
Chipacaclobrano: let's say we implement hw-assign <device> [<symlink>]09:06
Chipacaclobrano: document taht09:06
Chipacaclobrano: people use it09:06
Chipacaclobrano: how do we introduce the new thing without breaking the old?09:06
clobranoChipaca: ooh, right09:06
clobranoChipaca: well, it's not that hard to recognize a symlink (valid) from a string defining a udev rule, just to keep the backward compatibility09:07
clobranoChipaca: but I got your point09:08
loolvmayoral, tedg, sergiusens: please use ubuntu@212.47.227.205 instead, sorry09:08
loolroot will go away on reboots, and we should not use root09:08
loolok, sudo works now, removing your root SSH keys guys09:14
loolyeah, building!09:32
dholbachfgimenez, elopio: does "python3 setup.py test" crash for you in lp:snapcraft too?09:40
fgimenezdholbach, let me try09:41
fgimenezdholbach, yes, pip-requirements is failing http://paste.ubuntu.com/12637380/09:59
dholbachfgimenez, http://pastebin.ubuntu.com/12637392/ is what I'm seeing10:01
dholbachzyga, ^ do you know what's happening here?10:02
zygadholbach: looking10:04
zygadholbach: yes, it's fixed with a branch that didn't land yet10:04
zygadholbach: it's a one-line fix10:04
zygadholbach: but I'm not sure if someone fixed it because the original contributor did the fix in a werid way but didn't respond to feedback10:05
zygasergiusens: ^^10:05
zygadholbach: it happens because wily has two versions of python310:05
dholbachzyga, are http://paste.ubuntu.com/12637380/ and http://pastebin.ubuntu.com/12637392/ the same issue?10:06
zygadholbach: no, that seems different10:08
zygadholbach: the 2nd one looks like a bug elsewhere, how did you get it?10:08
zygadholbach: is that trunk, any released version, something else?10:09
dholbachtrunk10:09
zygadholbach: on wily?10:09
dholbachyes10:09
dholbachI just ran    python3 setup.py test10:09
zygadholbach: one sec10:10
zygadholbach: I'll just fix the bug that the contributor tried10:10
zygadholbach: and see if I get to the 2nd one10:10
dholbach<310:11
zygadholbach: I should have fixed it earlier but I really wanted yashi_ (AFAIR) to reply10:12
dholbachno worries10:12
zygadholbach: https://code.launchpad.net/~yashi/snapcraft/snapcraft/+merge/272337 this is the thing10:12
dpmdholbach, looking at the developer manual, trying to do 'sudo apt-get install snapcraft-examples' does nothing for me. I'm on 15.04, and I've got the ppa:snappy-dev/tools installed10:13
dpmanything I'm missing?10:13
zygayashi_: around?10:13
dholbachdpm, not sure if anyone updated the ppa - let me check10:13
dholbachdpm, https://launchpad.net/~snappy-dev/+archive/ubuntu/tools/+sourcepub/5408454/+listing-archive-extra10:14
dholbachdpm, it should be there10:14
dholbachare you sure you have the ppa enabled and your apt sources updated?10:15
* dpm apt updates to be sure10:15
dpmthat might have been it10:15
dpmyeah, that was it, thanks dholbach10:16
dholbachcool10:17
zygafg10:17
dholbachzyga, what does force-push mean in the MP=10:18
dholbach?10:18
zygadholbach: bzr push --overwrite10:19
zygadholbach: before it lands that's the best way to do it10:19
dholbachah, so you're asking yashi to push --overwrite to his branch?10:20
zygadholbach: lp tracks past in case someone wants to do data mining but for everyone else it gets rid of "fixup stuff" commits that are just noise10:20
zygadholbach: ys10:20
zygayes*10:20
dholbachok10:20
zygadholbach: if you want I can fix and land this myself in 5 minutes10:20
=== chihchun is now known as chihchun_afk
zygadholbach: do you want me to fix that quickly?11:18
dholbachzyga, I don't know - I just noticed that the test was broken11:21
zygadholbach: yes, we know about it, I think I will just fix it now11:26
dholbachok11:27
victorppitti, hey - got home and I am able to ssh over wifi to the pi. Thanks!11:33
pittivictorp: cool!11:33
pittivictorp: wipi!11:34
victorphehe11:34
mvovmayoral: this will work  lp:~mvo/snappy/snappy-classic-mode  but its not the final on-disk layout and the comandline will be slightly diffrent11:42
vmayoralmvo: great, thanks a lot. I'll have a look11:42
mvovmayoral: happy to give you a binary and/or talk more later11:42
vmayoralmvo: that'll be great! let's do that11:43
Chipacamvo: tell me about that branch; does it need reviewing, or is it in progress?11:47
mvoChipaca: it needs to actually go back to WIP11:48
Chipacadone :)11:48
mvoChipaca: because there were some changes during the sprint how its constructed etc11:48
mvoChipaca: thanks!11:48
vmayoralsergiusens: getting https://gist.github.com/vmayoral/b3ea80d53d5fe25e3ed2 when trying to upload the snap to the store11:54
vmayoralsergiusens: the framework doesn't exist so it just rejects it11:54
sergiusensvmayoral, I'll accept it11:56
sergiusensvmayoral, do you have the link?11:56
jdstrandbeuno: hey, pindonga was working on the issues found here: https://myapps.developer.ubuntu.com/dev/click-apps/3560/review/r/2/. it turned out to be that the server isn't setting the proper local and it has nothing to do with my snap. he approved the last one but isn't here now. would you mind approving this one?11:59
vmayoralsergiusens: so i actually don't think that the snap get's even submitted11:59
vmayoralsergiusens: it gets rejected before the review process complaining about the missing framework12:00
* jdstrand -> gone for a few hours12:00
beunovmayoral, I think you can click and ask for a manual review12:01
vmayoralsergiusens, beuno: stucked at this form https://myapps.developer.ubuntu.com/dev/click-apps/upload/?format=snap12:01
vmayorali believe the manual review process is available in a later step, but let me know if i'm not looking at the right place12:02
beunoah12:03
beunoso we need to add that framework first12:03
beunonessita, ^12:03
beunovmayoral, what's the string?12:03
* Chipaca hopes it's vmayoral-the-magnificent12:07
vmayoralbeuno: the framework has been named "raspivid"12:09
nessitabeuno, hello12:10
sergiusensbeuno, fyi, they use a framework, I guess we just need to add it to the framework list12:10
beunovmayoral, done12:10
sergiusensbeuno, I think i can do that12:10
beunosergiusens, indeed12:10
beunodone already12:10
nessitaeven better12:10
vmayoralbeuno: trying again, thanks!12:11
vmayoralbeuno, sergiusens: it got in https://myapps.developer.ubuntu.com/dev/click-apps/359112:14
vmayoralbeuno, sergiusens: remaking the snap to fix the errors12:14
sergiusensvmayoral, awesome, no inlfight installation ftw ;-)12:14
sergiusensvmayoral, oh, run click-review from your host/laptop12:15
vmayoralsergiusens: in the store already, we were missing a "description" field.12:18
sergiusensvmayoral, great12:20
loolvmayoral: I found the issue with ros/cmake/tests12:34
loolvmayoral: in the process, I found it hard to move the directory around, clean didn't clean up enough12:35
loolvmayoral: that is, I copied to a different dir, ran snapcraft clean, and then snapcraft build failed because of a ton of generated files hardcoding the previous path12:35
loolvmayoral: this was really hard to fix, would you know a clean way?12:35
looloh shit, I take it back, the error is still there  :-(12:36
vmayorallool: so generally, "cakin_make clean" does a good job on that. Wanna join us in the room or should i "screen -x"?12:37
=== davidcalle_ is now known as davidcalle
plarselopio: sorry, I had already gone to sleep when you sent that13:08
plarselopio: elasticsearch seems to have blown up on me, so I can't see much right now, let me sort out what happened with that and get you to retry13:08
olliueh14:08
dholbachsergiusens, https://code.launchpad.net/~dholbach/snapcraft/fix-pep8-and-doc-indentation/+merge/273195 updated14:28
plarselopio: when you have a chance, could you try again?14:51
=== chihchun_afk is now known as chihchun
elopioplars: I've launched a new run.14:56
jdstrandpopey: hey, you around? (quick store review)15:22
popeyjdstrand: ya15:22
jdstrandpopey: would you mind taking a look at https://myapps.developer.ubuntu.com/dev/click-apps/3560/r/2/? there are two review failures but it is due to a server side issue that pindonga has the fix for bt that hasn't rolled out yet. the snap is fine15:23
jdstrandpopey: he approved 0.1 and it had the same to review failures15:23
popeyokay15:23
jdstrandtwo*15:23
plarselopio: It looks like it's done, but I didn't see it execute any test commands, last I saw it was trying to boot the test image15:31
plarselopio: perhaps the test image didn't boot?15:32
elopioplars: can you take a look at the script to see if I'm doing something stupid?15:32
plarselopio: I really think the test image didn't boot15:32
elopioplars: if that happens, there will be a ssh timeout and the test will fail.15:33
jdstrandpopey: thanks! :)15:33
elopiobut it always says pass. And I get no payload.15:33
plarselopio: no, it won't even get that far if it can't confirm that the image booted15:33
plarselopio: it won't even try to run the tests15:33
plarselopio: I'm kicking off a test job of mine to see if something else could be causing this15:34
elopiook15:34
plarsgrr... logstash broke again15:36
plarsthis was supposed to work15:36
victorpanyone knows if I have to include a .service file on a snap service or if that is taken care on its own?15:49
Chipacavictorp: you do not have to include a .service file15:50
Chipacavictorp: in fact it will be ignored :)15:50
victorpChipaca, so any idea why when I restart the service I have sideloaded as a snap I get an error ...ICLGcNeLFEPe.service failed to load: No such file or directory15:51
Chipacavictorp: that's a bug15:52
Chipacavictorp: in particular, one we've fixed15:52
victorphehe15:52
Chipacavictorp: but i'm bad at tracking where it is in the pipeline to you15:52
victorpok, I shall ignore then15:52
victorpChipaca, np, I was just thinking I might be doing something wrong, I will ingore it for now and continue15:53
Chipacathe error is real, but if it's not blocking you, sure15:53
victorpis not blocking15:53
Chipacaok :)15:53
victorpservice runs fine from a restart of the pi15:53
* Chipaca nods15:53
Chipacavictorp: and you can probably find the right service and restart it using systemctl, if i remember that bug correctly15:53
Chipacahmm15:57
Chipacavictorp: just reproduced in latest stable, so the fix isn't there15:57
victorpack15:58
Chipaca... this might be a different bug15:58
Chipacavictorp: um. What I'm seeing is that it's not forgetting about the old service15:59
Chipacavictorp: that is, "snappy service status" shows the old one and the new one15:59
victorpoh15:59
victorpyeah I saw that too15:59
Chipacavictorp: but the new one is running alright15:59
fgimenez_have a nice weekend o/16:03
=== chihchun is now known as chihchun_afk
Chipacapitti: you around?16:17
mwenningHi snappy team, I did a sudo snappy update and then rebooted, grub comes up with system-b , goes thru a certain amount of booting, then gets a kernel panic and starts over.  Is there a way to manually switch back to system-a?16:32
Chipacamwenning: it should switch back on its own16:34
Chipacamwenning: what system is this?16:34
Chipacaof it *isn't* switching on its own, i'd love to have a look at the system, if it is image'able16:34
mwenningChipaca, switch to snappy-internal16:36

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