=== nyx_ is now known as kjar
dholbachgood morning08:02
noizergood morning08:50
JamesTaitGood morning all!  Happy Tuesday, and happy Pancake Day! 🙌09:32
diwicJamesTait, good morning! What do I need to install to see that unicode character? It is not visible on OOTB Ubuntu 14.04, at least.09:33
JamesTaitdiwic, you're not the first person to ask, but I still don't know the answer.  I might have had to install a specific font for it, not sure.09:34
dholbachdiwic, fonts-symbola?09:34
dholbachI searched for it in gucharmap and right-clicked on it09:34
JamesTaitdholbach, that does sound familiar, actually.09:35
dholbachmh... then I don't know :)09:35
diwicdholbach, thanks, but I can't find fonts-symbola in the archive...?09:35
dholbachdiwic, I'm on xenial09:36
JamesTaitIn fact I think someone on IRC told me to install that, let me check my logs.09:36
dholbachDescription: symbolic font providing emoji characters from Unicode 7.009:36
diwicinteresting, it appears on vivid and nothing below that09:37
dholbachthat's what it provides in 16.04, yes09:37
dholbach14.04 is pre-emoji times09:37
diwicmaybe I can just install the .deb from vivid and hope for the best :p09:38
diwicit worked! \o/ thanks dholbach and JamesTait09:41
JamesTaitWoohoo!  🙌09:42
diwicor should I say 🙌09:42
diwicoh well, now on to figuring out how to make snapcraft generate correct security profiles09:43
opnyHello! After a bit of hassle, I've been able to get running Snapcraft 2.1 on Xenial. I've noticed however the jdk plugin fails now, is this a known issue?09:46
dholbachopny, fails how?09:49
opnydholbach, http://paste.ubuntu.com/15000971/09:51
dholbachopny, I can't find any bug report for it at https://bugs.launchpad.net/snapcraft09:52
opnydholbach, is it an implicit request?09:53
dholbachopny, you asked if it was a known issue :)09:53
dholbachI'm just playing around with the java related examples to see if I can reproduce anything09:53
dholbachso the java-hello-world example works for me too09:54
dholbachtrying the tomcat example now09:54
opnydholbach, :) ok.. If you mind I have a github repository for that snap I'm looking to build09:54
dholbachcool - have a link?09:55
opnydholbach, https://github.com/muka/kura.snap (Let me a moment to push current status)09:55
dholbachcool, let me know once you're done09:57
opnydholbach, ok done09:58
dholbachtaking a look now10:00
dholbachopny, I can't reproduce the problem10:02
opnydholbach, uhm ok thank you for checking. May be the case I'm using a mount in virtualbox, will try to move it out10:04
dholbachmaybe you can run  snapcraft clean  and start over? it looks like an apt problem10:04
dholbachmaybe... *shrug*10:04
dholbachfirst I thought "maybe a part can't have the name of a snapcraft plugin", but that seemed irrelevant10:04
dholbachthen I saw that build-packages should probably be stage-packages10:05
dholbach(if you want to bundle things in your snap, as opposed to making sure that certain packages are installed on the machine where you're building the snap)10:05
opnydholbach, build vs stage packages is a recurring topic for me :) Yes, stage is the appropriate one!10:06
opnydholbach, moving out of shared folder worked.. thanks10:11
dholbachexcellent :-D10:12
asacelopio: running unit snapcraft tests on master cant find the storeapi etc... i use ./runtests.sh unit10:18
asacok seems thats xenial only10:26
* zyga pushed https://github.com/ubuntu-core/snappy/pull/46810:54
opnyHi, in snapcraft can I specify a git branch to pull?11:03
dholbachopny, yes - take a look at examples/godd/snapcraft.yaml11:04
opnydholbach, thank you11:05
opnydholbach, I do not see any branch ref https://github.com/ubuntu-core/snapcraft/blob/master/examples/godd/snapcraft.yaml#L1511:10
opnydholbach, Is it like url#branch ?11:10
dholbachopny, use source-branch11:41
dholbach(type: snapcraft help sources)11:48
* zyga pushed big security part of skills for review11:50
zygaelopio: do you know why some integration tests fail on all the branches (3 out of 58)11:51
fgimenezzyga, that was fixed with https://github.com/ubuntu-core/snappy/pull/467, if you rebase to the tip of master all should be fine12:06
zygafgimenez: ah, thanks12:21
zygafgimenez: I should stop being lazy and use my VPN :)12:21
=== vrruiz_ is now known as rvr
kyrofaGood morning13:17
didrockshey kyrofa!13:17
kyrofadidrocks, hey there! Good day so far?13:17
didrockskyrofa: so far, yeah :-)13:18
didrockshow are you?13:18
kyrofaI'm doing pretty well-- my son has been sleeping kinda bad, which means I have been too :P13:18
didrocksargh, yeah, coffee time I guess then! :-)13:19
kyrofaOh yes ;)13:20
ogra_ppisati, hmm, is the "linux-snapdragon" you uploaded this morning usable on the dragonboard already ?13:21
ppisatiogra_: nope13:22
ogra_k, thx13:22
opnyhi, snapcraft fails while doing its own apt-get update with a  "Hash Sum mismatch". I've edited the sources.list to point to other archives but snapcraft still tries to use the old one14:05
kyrofaopny, that's usually a temporary error with the archives14:06
kyrofaopny, just try again in a few minutes14:06
opnykyrofa, thanks, however, this is making me go crazy!14:07
kyrofaopny, yeah it kills our CI tests every once in a while too14:07
opnykyrofa, I have a sed thing to switch around countries and have it working. This work for apt-get but not for snapcraft14:08
kyrofaYou know you're OCD when you realize you have a thousand tabs open and just close the browser completely to start over14:12
ogra_the bad thing is that over time you end up with all these "session restore" tabs that pile up :P14:16
kyrofaogra_, hahaha14:17
kyrofaogra_, but those only come up if it exits inappropriately, though14:17
ogra_well, depends if you actually want that (i have occasions where i just want to archive the tabs and start over to save ram... and kill -9 ...)14:18
kyrofaogra_, ha!14:20
JamesTaitSo glad to know I'm not the only one who has that problem.14:22
kyrofaJamesTait, which one, slaughtering firefox to archive your tabs, or OCD?14:23
JamesTaitkyrofa, just the "ZOMG, I have 7 Firefox windows with 54 tabs in each, how did this happen?" problem.14:24
kyrofaJamesTait, ah, yes exactly. "Kill them all! Go away! I don't care what I was in the middle of doing, I can't keep track of all this!"14:25
JamesTaitAnd they're killing tab groups!14:25
JamesTaitSo instead I made an attempt to organise things into bookmark folders, and then "open all in tabs" when I want to work on something specific.14:26
JamesTaitWith varying degrees of success....14:26
kyrofaInteresting idea14:26
JamesTaitI'm still trialling it.  It's taking some time to get used to and to be disciplined about.14:27
kyrofaelopio, I'm assuming you can't hear me :P14:33
jdstranddholbach: hi! would you mind looking at my response to your comment to my MP?14:35
kyrofaelopio, if you have headphones in, and aren't hearing me... what you are listening to? Are you muted and just have headphones on?14:35
dholbachjdstrand, I'm in calls for the next hour, but could take a look later on14:36
kyrofaelopio, so this is creepy. I'll get out so I don't feel like I'm spying on you :P . Ping me when you're ready?14:36
kyrofaPing timeout... that might explain some things14:37
kyrofaelopio, you there now?14:40
JamesTaitjdstrand, I just re-merged that branch locally and get the same, but I didn't see it yesterday.  Not sure what changed since then.14:41
elopiokyrofa: I can't see you.14:41
elopionor hear you.14:41
kyrofaelopio, yeah that was several minutes ago now. Let me hop back in14:41
JamesTaitjdstrand, are there any plans (is it even possible) to split the MP into smaller chunks?  10k lines is a lot to review. ☺14:42
elopioseveral minutes ago it also appeared that I was alone.14:42
zygajdstrand: hey :)14:45
plarselopio: for the testconfig.json needed by the integration tests, what's the bare minimum we need to make it run?14:48
plarselopio: also, do the values in it really affect the tests that run?14:48
jdstrandJamesTait: well, I could remove some of the sr_ bits14:49
jdstrandJamesTait: but the problem is that it will be large because it is a refactor14:50
jdstrandwell, in part14:50
jdstrandzyga: hi :)14:50
jdstrandI did write a ton of new tests for snap.yaml which are adding to the length14:51
JamesTaitjdstrand, yeah, I noticed that, and I'm thankful. ☺14:52
JamesTaitjdstrand, I've been looking at individual commits to try and make the diff smaller.  I just wondered if there was a neat way to split "this factors out the common stuff but maintains feature parity" from "this starts adding snappy 2.0 specific functionality".14:53
elopioplars: I think now it will just affect the update tests.14:54
elopiooh well, after a couple of my branches land, that simplify things.14:54
elopioI need to check in detail if we still need it.14:54
JamesTaitjdstrand, fwiw I've been throwing lots of files at the refactored version and haven't noticed any obvious problems.14:55
kyrofaelopio, ... it's an invalid python syntax thing?!14:55
jdstrandJamesTait: well, I can probably do that if is do the refactor, then pull out the 16.04 tests from cr_*, then add the 16.04 sr_* tests. the problem is, that is going to take me a while to make those 3 commits committable since the refactor happened rather organically since I didn't know what needed refactoring until I got into it14:56
jdstrandJamesTait: but, if that is going to help with the review, I can take the time to do that14:56
plarselopio: these branches will make it so that all the tests don't try to need that json file?14:56
elopioplars: no, I think we'll still need it.14:57
jdstrandJamesTait: re lots of files-- yeah, as part of the testing I plan to run all the snaps through the tools in the old and new versions and do a compare (like I've done with other big changes)14:57
elopioI'm just not 100% sure why. It had like three functions. I removed one.14:57
elopiokyrofa: that's what it says, but it's valid syntax.14:58
jdstrandbut as I've been going I've been doing various spot checks14:58
jdstrandJamesTait: thanks for the testing14:58
elopiokyrofa: ah, it could be missing the quotes.14:59
plarselopio: ok, we're trying to run these tests under checkbox, and it's complaining about not having that file. I'm trying to see if there's a way to just avoid it, or if we should just put a stub file there to make it happy, or if we need to generate the real thing14:59
kyrofaelopio, yeah running that command here doesn't work, though it exits with a shell error, not a python error14:59
elopioplars: you need to generate the real thing.14:59
JamesTaitjdstrand, yeah, I understand. ☺   I've been there enough times myself!  On the one hand I'd like to do a thorough review of all the changes, on the other hand we'd like to get this landed sooner rather than later.14:59
elopioplars: I'm with you all morning today. Just give me a second to get ready.15:00
plarselopio: no rush, I am off to a standup anyway15:00
jdstrandJamesTait: ok, I will focus on finishing the TODO for this so at the very least, reviewers can run this branch locally on snaps that are flagged for manual review. that will make sure there isn't any more refactoring (it's possible I'll have more for the security bits). then I can look into breaking this up into review parts15:02
jdstranddholbach, JamesTait: weird, if I bzr branch anew and the run-tests, it passes15:03
jdstranddholbach, JamesTait: the new check_snappy_readme_md() just doesn't use is_squashfs anymore... not sure why it is that way for you guys15:03
JamesTaitjdstrand, interesting - I just did that and it worked for me as well.15:04
jdstranddholbach, JamesTait: do you guys need to do 'make clean'?15:04
* JamesTait does that just to be sure.15:04
JamesTaitNope, that still failed. :-/15:05
jdstranddiff -Naur -x .bzr ./existing ./newcheckout15:06
jdstrandmight be interesting15:06
dholbachjdstrand, still fails here - I checked 'bzr {diff,ignored,unknowns}' to have no output... not sure what else to do15:06
jdstrandit is a mystery, but if I am breaking this up into 3 different parts, then perhaps looking into why it is a problem now is probably not the best use of our time15:08
JamesTaitWeird, this is the same branch that was passing for me last week.15:08
jdstranddholbach, JamesTait: you you both on xenial? I am, though I haven't updated in a few days15:08
JamesTaitI'm not - still on Wily.15:09
dholbachyes, on xenial, up to date15:09
dholbachhah, a new checkout works15:09
jdstrandso, probably not xenial specific15:09
dholbachthat'S bizarre15:09
JamesTaitYeah, odd.15:09
jdstrandok, I'm not going to worry about this. I'll keep pushing to the big branch, then I'll break up into smaller branches later15:10
jdstrandif there is a problem then, we'll dive in15:10
JamesTaitThanks, jdstrand!15:11
dholbachjdstrand, I think I know what the problem is - maybe we have a backed-out change in lp:click-reviewers-tools?15:11
dholbachjdstrand, http://paste.ubuntu.com/15002261/15:12
dholbachor rather: http://paste.ubuntu.com/15002273/15:14
=== nyx__ is now known as kjar
JamesTaitdholbach, I see http://paste.ubuntu.com/15002276/15:15
JamesTaitHeh. ☺15:16
diwicjdstrand, hi, want to chat about PulseAudio on snappy?15:16
* tedg watches, he wants to define new skills too15:17
* JamesTait defines the new skill Time Travel.15:17
kyrofaJamesTait, I didn't know Apple was bring Time Machine to Ubuntu Core!15:18
jdstranddholbach, JamesTait: are your branches up to date with trunk?15:19
JamesTaitkyrofa, you should see it - it's magical! 😝15:19
dholbachjdstrand, yes15:19
jdstranddiwic: give me just a minute15:19
diwicthe "kill daemons" skill sounds like it could fit in an RPG style game15:19
JamesTaitSeems to be.15:19
tedgHonestly, that would be a sweet snap. Have it come on a device with a HDD or put it on your server.15:19
diwicjdstrand, sure15:19
jdstrandcause, I'm at r572 in my trunk15:20
dholbachjdstrand, are there any previously backed-out changes that might confuse bzr during the merge?15:20
jdstrandI think I didn't merge 569 and later into the cleanup branch15:21
* JamesTait wonders what on earth he did to have this working locally.15:21
JamesTaitProbably not a make clean. :-/15:21
JamesTaitBah, school run time.15:22
jdstrandbut I just tried to merge with trunk and it came back clean15:22
jdstrandok, pulling in as cherrypicks15:22
jdstrandand I can reproduce15:23
jdstrandlet me fix that and push15:23
jdstranddholbach, JamesTait: ^. thanks for helping me figure that out15:23
dholbachglad it's going to be fixed soon :-)15:24
jdstranddholbach, JamesTait: ok, pushed15:28
jdstranddiwic: ok15:28
diwicjdstrand, ok, so what's currently blocking me is access to /dev/snd/*15:29
dholbachjdstrand, confirming the fix :)15:29
jdstranddiwic: are you on 15.04 or 16.04?15:29
diwicjdstrand, 16.04, just upgraded to the latest skill-based images15:30
jdstranddiwic: ok, things may be a bit rough atm as the skills work is mid-implementation15:30
jdstranddiwic: let me check something15:30
diwicjdstrand, to the extent that you think I should defer the snappy work and go do something else and come back in a week or two?15:31
kyrofaelopio, I can't duplicate that ROS error locally15:31
jdstranddiwic: alright, since /dev/snd devices aren't implemented as skills yet, hw-assign is what you need to use, and it is still available on 16.0415:31
kyrofaelopio, I'm really not sure what's happening there15:31
jdstranddiwic: sudo snappy hw-assign foo.sideload /dev/snd/*15:31
diwicjdstrand, ok, let me see if that works15:32
elopiokyrofa: ok, I can try to debug in a docker that's the same as the slave.15:32
jdstranddiwic: well, we're going to need to understand what pulse needs in order to create the skills for it. I'm confident we can get you unblocked such that we have enough information for developing those skills15:33
kyrofaelopio, alright. That call happens via a subprocess.check_output, so there should be no problems with the quotes15:33
kyrofaelopio, I suspect the syntax error is a red herring15:33
diwicjdstrand, cool. Feel free to ping me if you have further questions about what pulse needs - the hwassign thing does seem to work.15:33
diwicjdstrand, how does hw-assign work internally, btw?15:34
diwicjdstrand, is it related to acls somehow?15:34
jdstranddiwic: great. really, if you have a working snap, I just need it or the link to the branch15:34
jdstranddiwic: yes. if the specified files are in /sys, then you get apparmor rules, if in /dev, the device gets added to an app-specific cgroup15:35
jdstranddiwic: https://wiki.ubuntu.com/SecurityTeam/Specifications/SnappyConfinement15:35
diwicjdstrand, cool, thanks. I'll need to work some more on the snap - now that I know how to deal with /dev/snd/ - to make sure the sound card loads correctly15:37
* diwic unblocked15:37
jdstranddholbach: did it work for you?15:49
dholbachjdstrand, yes15:50
dholbach<dholbach> jdstrand, confirming the fix :)15:50
tedgjdstrand: But won't diwic need to provide a skill for applications to use? I don't understand how he does that.15:50
tedgjdstrand: Seems he'll need to put some apparmor configuration inside his snap.15:50
jdstranddholbach: ah, I took 'confirming' as you were in the process of confirming it and would get back to me. thanks!15:50
dholbachah, sorry :)15:50
dholbachso, yes - fixed, all good! :-D15:51
jdstrandtedg: that part of the discussion has been deferred since he is unblocked15:51
jdstrandtedg: but the general idea is this15:51
jdstrandtedg: people have something that needs to use skills that aren't yet defined. in this case, he needed /dev/snd and mayve some udev stuff15:52
jdstrandtedg: comes to snappy team, decide if the skill is worthwhile. if it is, define a skill type and the security policy, and add to snappy15:52
jdstrandtedg: then the app can 'uses' the skill15:52
tedgjdstrand: That clearly doesn't scale.15:52
JamesTaitjdstrand, dholbach - thanks!15:53
diwicpulseaudio clients don't need /dev/snd, only the pulseaudio server does.15:53
tedgjdstrand: And what you're saying is that snappy hardcodes the path to the pulse socket. When in reality, it should be pulse defining that. So if it is "foo" today and "foo1" tomorrow, that's defined in the relationship between the provider and the slot.15:54
jdstrandtedg: then if the app is going to provide skills, they use existing types to declare what it provides. so, in this case, a socket and a dbus interface. snappy understands these skill types (eventually) and will dynamically create security policy for apps that 'uses' them15:54
tedgjdstrand: It shouldn't be dependent on a specific version of snappy.15:54
jdstrandtedg: no, not what I'm saying15:54
jdstrandtedg: there were two parts15:54
jdstrandthere is what pulse needs from the system that is different from what is already there15:55
jdstrandand there is what pulse can provide to other snaps15:55
tedgSure, I'm worried about the second.15:55
jdstrandthe first part will be a smallish set of skills15:55
jdstrandthat is what I described15:55
jdstrandthe second part is a smallish set of skill *types* that are configurable by the providing snap15:55
jdstrandeg, lets say for simplicity there is a dbus skill type15:56
jdstrandsnappy knows about this skill type15:56
jdstrandpulse snap fills in attributes for this skills type so that snappy can do stuff on the fly15:56
jdstrandpulse provides a pulse skill to the system15:57
jdstrandapps 'uses' this pulse skill15:57
tedgOkay, I'm good with that, but then what does the app specify? All the parameters for dbus?15:57
jdstrandtedg: what attributes pulse can set in its snap skill declaration for dbus is TBD15:58
jdstrandtedg: bluez, pulse and nm are three snaps we can look at to help define what a dbus skill or set of skills will look like15:59
tedgSeems like a big TBD, as that's kinda the core of how they integrate with each other.15:59
diwicside note: pulseaudio uses unix sockets as primary IPC mechanism15:59
tedgUnfortunately none of those deal with things like AppIDs being part of the dbus path, which is important for us on the phone.16:00
jdstranddiwic: yes, we have 'socket' right now for that, but there will be some sort of transition to skills for that aiui (which is TBD)16:00
tedgIt would be nice to add a case of that as well.16:00
jdstranddiwic: and the 'socket' options support both file and abstract sockets16:01
diwicjdstrand, in snapcraft terms, where should I add 'socket' ?16:01
diwici e where in snapcraft.yaml16:01
jdstrandtedg: today on snappy there is only a singular 'bus-name' (eg, org.freedesktop.pulseaudio)16:02
* ogra_ curses 16:02
jdstranddiwic: alongside 'command' under 'apps'16:02
ogra_root@anubis:/# snappy build --squashfs snap/16:02
ogra_2016/02/09 16:02:18.472075 main.go:40: WARNING: can not create syslog logger16:02
ogra_open snap/meta/snap.yaml: no such file or directory16:02
jdstranddiwic: see docs/meta.md in the snappy git tree16:02
tedgjdstrand: So, in general, I agree that this overall approach makes sense. But when I said that on the list, zyga said that is incorrect.16:02
jdstrandtedg: but this was far too limiting on two fronts: bluez listens on more than one bus-name and there was no way to define dbus bus policy16:03
tedgjdstrand: The problem is that is the "easy case" — where there is less interaction and negotiation that is needed. We need to tackle the hard cases too.16:03
diwicjdstrand, in 16.04 too?16:04
jdstrandtedg: so the skills work will need to accommodate that. note, I said for the simple case there is a skills type of 'dbus'-- I don't know if that will be more specific or not16:04
jdstranddiwic: yes16:04
tedgjdstrand: Before feature freeze next week? ;-)16:05
jdstrandtedg: now, in terms of APP_ID, that shouldn't be a hard problem. only 'trusted helpers' need to know about APP_ID's. app templated policy won't really have to change here and the trusted helper will only need security policy to allow looking at its own path, etc. it should basically work, but there might be some fine-tuning (possibly an additional skill for trusted helpers, but I doubt it). I don't see a big issue there16:06
jdstrandtedg: this is snappy, feature freeze isn't until May :P16:06
noizerhi, short question. is ASCII not support by a snap?16:07
noizerbecause i get the following error with my uwsgi LookupError: no codec search functions registered: can't find encoding16:07
tedgjdstrand: Sure, there could be another simple type, but we're startting to add a lot of simple types then.16:08
tedgjdstrand: I'm worried we're only covering the trivial cases and not specing out the complex ones early enough. The format doesn't seem like it can handle the complex cases and is becoming more and more difficult to chagne.16:09
jdstrandtedg: I personally worry about the number types exploding. on touch we tried to keep that low. in snappy 15.04 we did the same. in snappy 16.04 and skills, people said this wasn't a problem and that is the path forward16:10
jdstrandtedg: types are actually supposed to be pretty easy to add16:10
tedgjdstrand: I *think* the hooks are going to need richer communication with the security frameworks, which is harder to add. It's hard to see with the current state of things though.16:11
jdstrandand 16.04 is only supposed to have a smallish number of easily understood types so that we can learn from them to do more complex ones16:11
jdstrandtedg: mapping click hooks to snappy is not on anyone's 16.04 todo list afaik16:12
tedgI'm fine as long as "smallish" is "everything we need for phone" :-)16:12
tedgjdstrand: It is on my todo list, but I can't figure out enough info to make anything other than a shot in the dark.16:12
jdstrandtedg: I though 'Personal' work was not for 16.04. did that change?16:13
jdstrandalso, while I am speaking rather authoritatively here, skills are Gustavo's and zyga's baby-- I'm only providing input for the security bits and bringing up other things as needed (ie, I'm thinking about touch too)16:14
tedgjdstrand: Getting snaps so they could be installed on desktop or phone is a desirable feature, and we'd want those snaps to work into the future. So "personal" is not, but getting apps that will work for personal is.16:15
fgimenezelopio, sorry, don't know how i ended in the internal channel :D16:18
jdstrandtedg: I think getting leaf desktop snaps will be fine. that is different from a trusted helper and different than a click with lots of hooks, both I interpreted in the realms of 'Personal'16:20
tedgjdstrand: Certainly desktop is easier than phone, but while no one has decided anything, phone apps are on the table still as well.16:23
zygajdstrand: I refreshed bool-file skill, would love if you can have a 2nd look at the security side16:24
elopiofgimenez: I think this is ready: https://github.com/ubuntu-core/snappy-jenkins/pull/7216:25
fgimenezelopio, thx let's merge after travis finishes16:27
jdstrandzyga: ack-- saw it, haven't gotten out of irc questions yet :)16:27
zygajdstrand: thanks, if you have any questions about it we can also talk on irc16:31
JamesTaitjdstrand, what gotchas would I need to look out for if I wanted to use click-reviewers-tools as a library in the Store, rather than shelling out to a subprocess?  Is there a stable public API?16:37
jdstrandJamesTait: I think to do that well, we'd want to break what is in click-review out into a library and have that be the api. then click-review and you could use that16:43
* JamesTait nods16:43
jdstrandJamesTait: click-review itself has been very stable and not changed much (until this cleanup branch)16:43
jdstrandso I think breaking out those bits would provide a good api16:44
JamesTaitIt's not something I'm looking to do immediately, unless it turns out to be so trivial it'd just fit into existing work, but as a future path I think it'd be preferable.16:45
plarselopio: I think it looks like we won't need much (anything?) in that json file if we aren't using the update/rollback tests, but it will still error if it can't read the file. So we can stub one, but it seems to read from a hardcoded relative path, that picks up its base path from the GOPATH on the build system16:47
plarselopio: would a parameter for this testconfig path be ok to add?16:48
plarselopio: or do you have a better suggestion16:48
JamesTaitjdstrand, also, this is totally work that I'd expect the Store team to take on - just to be clear. 😉16:48
elopioplars: it's only used for update, rollback and the info_test.16:49
elopioI'm not sure if you want to run the info test.16:49
elopioif you don't want anything to do with it, I can catch the file not existent error, and put a default config.16:49
JamesTaitWe've got a Trello card open for it, I'm just going to add some thoughts to it for now and we'll see where it fits in.16:50
plarselopio: but all tests try to read it: Error reading config: open integration-tests/data/output/testconfig.json: no such file or directory16:50
jdstrandmakes sense16:50
elopioplars: yes, because all tests check if they are in the middle of an update or rollback.16:50
elopioI can put a default false in there.16:51
plarselopio: or we could probably just generate a json file with "{}" in it right?16:51
elopioplars: I'm not sure what will happen in that case. Maybe json.Unmarshal sets default values, or maybe it fails because of the missing fields.16:52
elopioplars: but there's something more important to solve here.16:52
elopiothat integration-tests/data/ path is important, not just for the config. All the test snaps are in there.16:53
elopioso it's not just the binary what you need. You also need the test data.16:53
plarselopio: oh, so it's not enough just to ship the test binary, there's other data there too it needs?16:53
plarsgot it16:53
elopioyes. The config is not too important now. But the others are.16:54
elopioplars: can you try creating that testconfig.json to see if the binary finds it?16:54
plarselopio: I see that now... elopio but those don't get built when I build the integration tests with go test -d16:54
plarserr.. -c16:55
elopionop. Just the code goes into the binary, no data files.16:55
plarselopio: yeah, we'll give that a try. What will I need to force it to build all those other snaps? and will they need to be in that exact directory structure?16:56
elopioplars: you don't need to build the snaps, they are build during the tests. You can't include them in the binary, afaik. You'll have to copy them to the testbed, as we do here with adt-run.16:57
olli_I love xchat and how it moves keyboard input between channels when starting up16:57
fgimenezelopio, this is done https://hub.docker.com/r/ubuntucore/snappy-jenkins-slave-xenial/builds/ we can build the new base image now16:57
elopiofgimenez: let me first re-run the tests on my branch and merge it.16:57
fgimenezelopio, the error from travis should be fixed with the latest changes too, ok16:58
kyrofaolli_, so you don't actually know where you pinged unless you look through them all? Yeah I love that too17:09
=== retrack is now known as Guest58434
jerryGChipaca: have you done a coredump ?17:49
jdstrandbeuno, matiasb: I just uploaded snappy-debug to the store for rolling only, and 'snap find snappy-debug' doesn't see it. is this because the device is not authenticated with the store?18:24
jdstrandbeuno, matiasb: it is published afaics18:25
plarselopio: getting a log of errors, even if I put the testconfig.json and snaps data there: http://paste.ubuntu.com/15003786/18:25
jdstrandbeuno, matiasb: in the logs: snapd[3217]: 2016/02/09 18:20:36.827803 response.go:118: method "GET" not allowed18:25
jdstrandbeuno, matiasb: /usr/bin/snapd[3217]: daemon.go:153: DEBUG: uid=0;@ GET /2.0/snaps?q=snappy-debug&sources=store 630.821587ms 20018:26
elopioplars: is that an allsnaps image generated with mvo's latest udf?18:27
matiasbjdstrand, let me check18:27
plarselopio: no18:28
plarselopio: it needs to be in order to work?18:28
elopioplars: if you have a 1504 image, you have to run the tests from the 1504 branch. If you want to run tests in rollilng, you need to use mvo's allsnaps image.18:28
jdstrandmatiasb: thanks18:29
elopiopindonga: how can I upload a snap to the edge channel?18:30
plarselopio: so, the test binary I built was from ubuntu-core/snappy trunk, with your patches to enable the parameters cherry-picked on top of it18:30
pindongaelopio, setting channels is not yet supported in the api, so I think you need to upload then go to the web ui and set the channels for the new verison18:30
elopiopindonga: do you know if that will be ready for 1604?18:31
elopioplars: yes, trunk needs allsnaps.18:31
pindongaelopio, depends on how strong you can make your case for beuno  :)18:31
plarselopio: ack, thanks18:31
pindongaelopio, I'd start by filing a bug for it18:31
matiasbjdstrand, afaict, the revno is published and it looks ok; we would need to confirm that the rolling snappy client is not sending a framework when hitting CPI <- mvo could you confirm?18:31
pindongaelopio, and possibly listing all bugs related to the upload api18:32
jdstrandmvo is out sick18:32
pindongaso that we can see what's really missing and consider the effort18:32
elopiopindonga: well, I need squashfs first autoreview first, so I won't push to much with the channel.18:32
pindongaelopio, as said, autoreview depends on click-reviewers tools18:32
pindongaso, not something we can change in the store itself18:32
jdstrandmatiasb: perhaps Chipaca could help in mvo's absense?18:32
elopiopindonga: right, but is that beuno's team too?18:32
pindonganot really18:32
pindongait's snappy team I think? (jdstrand, mvo, ...?)18:33
matiasbjdstrand, did you get any results when searching? I mean, are you getting an old revision instead?18:33
elopiooh good. So I can make the two requests in parallel to two different teams :D18:33
jdstrandmatiasb: no18:33
jdstrandpindonga: I wasn't following the conversation. I can say that snap.yaml changes for the review tools are in progress and squashfs in the store is after that18:34
elopiojdstrand: awesome!18:34
jdstrandmatiasb: $ snap find snappy-debug18:34
jdstranderror: no snaps found for "snappy-debug"18:34
jdstrand$ snap find snappy-debug.canonical18:35
jdstranderror: no snaps found for "snappy-debug.canonical"18:35
nessitajdstrand, you need to be running this commit of snap command (or newer) https://github.com/ubuntu-core/snappy/commit/d18e35ef719a418332cd14eb61ec3841ff2a535f18:35
matiasbjdstrand, odd, everything looks ok, and hitting the index directly, the search seems to return the expected result18:35
jdstrandnessita: I'm on latest all snaps images from mvo...18:36
jdstrandlet me see if I can determine if that is in it18:36
nessitajdstrand, if you could grab the raw http request is being done, it would be helpful18:37
nessitathe headers being sent is key18:37
jdstrandok... tcpdump not on the device...18:38
nessitajdstrand, as a summary, if your snap search is sending the framework ubuntu-core-15.04-dev1 only revnos with that framework will be returned, and revno 12 opf snappy-debug does not have any framework (which is correct)18:39
jdstrandnessita: hrmm, this is https so I don't seem to be able to see the headers18:42
* jdstrand goes back at looking at changes in the ppa18:43
nessitajdstrand, I guess source inspection with go is not possible?18:44
jdstrandnessita: that is what I am doing. the current os snap has an earlier version than what is in the ppa so I'm just tracking it all down18:45
jdstrandnessita: based on the diff for 1.7.3ubuntu1-1+1108~ubuntu16.04.1 , which is what is on the image, the change should be on the image18:47
jdstrandnessita: could this have something to do with it not authenticating with the store, ala, that other thread?18:48
nessitajdstrand, nopes, since search is anonymous: https://search.apps.ubuntu.com/api/v1/package/snappy-debug.canonical18:48
nessitajdstrand, not authenticating could be a 401 when downloading the package, but this is not the case18:48
jdstrandthis is what I see in the log:18:49
nessitajdstrand, from those package index URLs, you can see a search (with no headers) returns your package18:49
jdstrand/usr/bin/snapd[3217]: response.go:118: method "GET" not allowed18:49
jdstrand/usr/bin/snapd[3217]: daemon.go:153: DEBUG: uid=1000;@ GET /2.0/snaps?q=snappy-debug&sources=store 1.059970664s 20018:49
nessitajdstrand, that URL is not store "server" side, may be something internal18:49
jdstrandI guess I'll file a bug18:50
nessitathe URL for searching is https://search.apps.ubuntu.com/api/v1/search?<params>18:50
nessitaplus headers18:50
=== retrack is now known as Guest94879
jdstrandChipaca, matiasb, nessita: fyi, https://bugs.launchpad.net/snappy/+bug/154373118:57
ubottuLaunchpad bug 1543731 in Snappy "'snap find snappy-debug' cannot find snappy-debug" [Undecided,New]18:57
elopioChipaca: do you have the credentials of the canonical account in the store? I need it to get the spi tests working.19:00
jerryGany1 been able to create coredumps successfully?19:00
wigglewormI am getting a java error when I try to start my snap - "nohup: failed to run java: no such file or directory20:18
wigglewormany thoughts on how to fix this?20:18
wigglewormOr better yet, Am i not dofing somthing correct in snapcraft?20:19
jerryGkgunn: are you there?23:02
kgunnjerryG: i'm literally needing to step away just now...feel free to leave query23:11
jerryGkgunn: kk lol.  ill wait till tomorrow :}23:23

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