[00:15]  * ToyKeeper o/ a bit late...
[00:17] <ToyKeeper> n
[00:18] <ToyKeeper> ... would mean [n]ext song, if I hadn't moved the cursor right before pressing enter.
[00:18] <knome> well it created nice legs for your smiley.
[00:21] <ToyKeeper> :)
[00:52] <elopio> o/
[00:52] <elopio> am I too late?
[00:53] <veebers> elopio: afraid so, your ubuntu\-qa badge has been revoked :-(
[00:53] <knome> and all the cookies and pizza are gone
[00:54] <knome> there's only one salty stick left but the dog licked it
[00:54] <elopio> :'(
[00:54] <knome> well cheer up, at least it has some natural flavors to it from... nevermind.
[00:54]  * elopio goes away to cry on the bed, and enjoy the extended nap :)
[00:54] <knome> sleep tight elopio
[00:59] <veebers> elopio: I'll be asking you for a review shortly if you'll be around
[00:59] <elopio> veebers: I was joking.
[00:59] <elopio> I'll be around.
[01:00] <elopio> I can't even go to the gym, today I can't raise my arms higher than the keyboard.
[01:00] <veebers> elopio: heh, you feeling the burn?
[01:01] <elopio> yes, I took the wrong class
[01:01] <veebers> I should have gone today as my gym buddy won't be there tomorrow and the gym is a scary place for a scrawny skinny guy :-|
[01:01] <veebers> elopio: what did you take?
[01:03] <elopio> vveebers: it was a kettelbells class. We learned how to hit our forearms and shoulders with iron balls.
[01:07] <veebers> elopio: holy crap, that does sound intense :-P
[01:08] <elopio> veebers: when you have some time, can you take a look to the packaging card? I've tested it and +1. It would be nice to get the second approval.
[01:13] <veebers> elopio: ack, can do. That's actually next on my things to do :-)
[01:20] <elopio> :)
[01:35] <elopio> I think I got the dbus upgrade working!
[01:41] <veebers> elopio: awesome!
[01:42] <veebers> elopio: hey, I've reviewed the packaging branch, Needs Info, due to the lack of -B in the readme (works for me with -B, I get apt 404s on the device otherwise)
[01:42] <elopio> veebers: did you phablet-network?
[01:44] <elopio> veebers: for some reason, -B works for me but fails for barry. without -B it fails for me with a missing fakeroot error, but works for barry.
[01:45] <elopio> now you are throwing a new error to the mix :(
[01:45] <veebers> elopio: really odd. Yeah the network is setup, it's that the image is out of date and needs an apt-get update. The problem is that even w/ a fresh image we have seen the apt cache out of date
[01:48] <elopio> right. I'm all for -B, as that's how we got the sanity working. But without understanding barry's error, I'm not sure which one to recommend.
[01:52] <elopio> veebers: do you know how to check the version of a py package if it has no __version__ attribute?
[01:58] <veebers> elopio: no I do not, sorry
[02:06] <elopio> veebers: that last card for xlib update just requires a top-approval on the autopilot branch, right?
[02:09] <veebers> elopio: if the packging has been acked, yes. We'll be doing a release tomorrow
[02:10] <elopio> ok, top-approving then.
[02:36]  * elopio goes for food
[02:36] <elopio> bbs.
[03:33] <veebers> elopio: if you're still around can you review the reflash MP for me please?
[03:33] <elopio> veebers: sure.
[03:38] <veebers> thanks
[03:38] <elopio> veebers: I think find_adt_test_file_names is complex. What do you think of this:
[03:39] <elopio> if we put all the upgrade tests in  ubuntu_ota_tests.tests, then we can use discover or something like autopilot list to find all of them.
[03:39] <elopio> then, we can define the Test-Command like python3 -m unittest $TEST_NAME
[03:40] <veebers> elopio: if we want to use --testname to run a test then flash the device they need to be named in a Tests: stanza, using Test-Command: doesn't work (doesn't have a name)
[03:40] <elopio> and we could use something like: adt-run --testname command1 --setup-commands "export $TEST_NAME=ubuntu_ota_tests.tests.test_basic_upgrade" ...
[03:41] <veebers> also, I think it only _looks_ complex, it's not really it's just parsing a text file
[03:41] <elopio> veebers: Test-Command gets the name command{index}
[03:41] <veebers> ah right
[03:41] <elopio> maybe what I'm proposing is more complex. I'm just thinking here.
[03:42] <veebers> elopio: I like the spitballing. where would we get the name to export in TESTNAME?
[03:43] <elopio> veebers: we need something like autopilot does when listing the tests.
[03:43] <elopio> we do a loop over that list, running adt-run once per test.
[03:43] <veebers> well, find_adt_test_file_names does that kind of, assuming the test is defined in d/t/control :-)
[03:44] <veebers> sounds like what you're proposing is pretty much what we have there, just actually running them is slightly different
[03:44] <veebers> i.e. parse what tests are available, and run them individually.
[03:44] <veebers> As it is currently the tests can be shell scripts or python scripts etc. no restrictions there
[03:44] <elopio> right.
[03:45] <veebers> elopio: oh on that note, brendand tells me that the check accounts test needs to be a shellscript (not python) due to some permissions issues he came across
[03:45] <veebers> so we can't say it _has_ to be a python unittest file
[03:45] <elopio> ugh, I hate that.
[03:46] <veebers> yeah, I wonder if a little more investigation we could make it a python script, but for now . . .
[03:46] <elopio> veebers: so, on your branch we will need to move the basic test out of ubuntu_sanity_tests.tests and put it into debian/tests, right?
[03:47] <veebers> elopio: hmm, we should be able to keep it there and have a test script that calls python3 -m unittest ub. . . .
[03:47] <veebers> that sounds like a bit of a work around though
[03:48] <veebers> hmm, although I'm not to worried about that, it's moved a command from the control file into it's own standalone file which isn't too much of a bad thing
[03:48] <elopio> veebers: I won't mind about moving it out. I think your branch is right, just not what I had in mind.
[03:48] <elopio> but it should work, lets give it a try.
[03:49] <veebers> elopio: aye, fair enough. If you think it should be (and can be) improved lets discuss it. Perhaps its a case of (and I say this a lot ;-)) that we iterate on it
[03:49] <elopio> veebers: I don't understand this comment: Hmm, apparently the revision argument is no more :-\
[03:49] <elopio> I actually like when you say "lets iterate on it" :)
[03:50] <elopio> veebers: I have latest vivid, and the revision argument is in there.
[03:50] <veebers> elopio: heh, that's a stale comment. The argument does work
[03:50] <elopio> cool.
[03:50] <veebers> its just that it needs to be before 'touch'
[03:50] <veebers> So that's the 2nd time in as many days that I've been caught out by an agument being position dependant'
[03:51] <elopio> veebers: so I like the branch, I would like it even better with tests.  Like putting it in ubuntu_ota_tests/command_line and make some selftests, like we did for sanity.
[03:52] <elopio> but the branch is already too big. I would suggest to split it: a first branch which only does one flash and runs adt-run ...
[03:52] <elopio> this one will work on the current branch, where we have only one test for upgrade.
[03:52] <elopio> then a second branch which does the test discovery, and puts the flashing in a loop.
[03:59] <veebers> elopio: I agree that its stupid that it lacks tests. I got a little carried away getting POC up on that branch before actually testing.
[04:00] <veebers> I disagree with the size, it's not that big and will take a while to take apart to make smaller branches that are all related anyway.
[04:00] <elopio> veebers: ok. I can help adding the missing tests, I still have a couple of hours before going to bed.
[04:01] <veebers> if you're really keen on that I can revert to an earlier rev where it's really simple and propose that, but then I would be fixing things in that branch that are already fixed anyway. I guess I'm just being difficult thought :-)
[04:02] <veebers> elopio: coolio, I'm confident that it's written in a a pretty testable way. Let me move things around a little so we can test it etc.
[04:02] <veebers> elopio: rats, actually I want to rebase it on barrys branch, but that hasn't landed yet. I don't want merge conflicts wasting our time either
[04:02] <elopio> veebers: splitting the branch was just a suggestion. If you think there will be no problem adding tests with the branch as is, I'm ok with that.
[04:03] <elopio> veebers: about barry's branch, I suggest to land it.
[04:03] <elopio> there's a card about updating the readme, where we can investigate about -B or not -B
[04:03] <veebers> so on that note I've bottom approved.
[04:03] <veebers> do you mind if I top  . . . you've already suggested that
[04:04] <elopio> veebers: please, go aheaad.
[04:04] <veebers> elopio: done. I'm going to grab trunk + barrys branch, merge that all in and move things around to get testing sorted
[04:04] <elopio> what we can do for now is to pass the -B from your script to adt-run.
[04:06] <elopio> one more crazy error in here. With v2.5, check_upgrade returns that there is no upgrade available when there is one.
[04:09] <veebers> ugh, any idea why that is ?
[04:12] <elopio> veebers: no clue. For now I'll just skip the assertion.
[04:12] <elopio> actually, I'll let it fail.
[04:13] <elopio> that's what the suite is for. There is a bug, it must ffail.
[04:13] <veebers> ah I see, you're saying there is an actual bug, not with the test
[04:14] <elopio> yes.
[04:15] <veebers> Well thats good I guess :-) Already being useful
[05:18] <veebers> elopio: you'll be happy to see* that I've added another 100+ lines to that already big diff  :-)
[05:18] <veebers> (*note you may not actually be happy)
[05:19] <veebers> elopio: tests added (should have been there from the start, I know) and I'm going to go have tea. I'll be back on later tonight though to clear up anything
[05:20] <elopio> veebers: I'm happy.
[05:20] <elopio> I'll take a look.
[05:21] <elopio> veebers: I found that revision is actually not working for me.
[05:21] <elopio> I tell it to flash 152, and it flashes 153.
[05:29] <veebers> oh? ugh hmm
[05:29] <elopio> veebers: http://paste.ubuntu.com/10625388/
[05:30] <elopio> I'm not doing anything stupid, right?
[05:30] <elopio> might be that it's late.
[05:31] <elopio> oh, I was doing something stupid.
[05:31] <elopio> :)
[05:31] <elopio> I forgot the --.
[05:31] <elopio> lets blame the hour.
[05:32] <veebers> elopio: heh, you need to get some rest :-)
[05:33] <elopio> I really do. I spent like an hour wondering why the fuck it was saying that there were no updates.
[05:35] <veebers> and it turned out to be a bug?
[05:35] <elopio> veebers: no, it turned out to be me being stupid for an hour.
[05:36] <elopio> veebers: your branch looks nice, and I like the tests. But I will leave the detailed review to the EU team.
[05:36] <elopio> I better go to bed. I'll just try to leave my branch in a decent state, in case one of them can finish it.
[05:37] <veebers> elopio: sounds good, thanks for your help tonight. See you tomorrow o/
[05:37] <veebers> elopio: let me know if there is anything I might be able to do with your branch, I may have some spare cycles this evening
[05:40] <elopio> veebers: you can give it a review, and give it a try.
[05:40] <elopio> https://code.launchpad.net/~canonical-platform-qa/ubuntu-ota-tests/dbus-upgrade/+merge/253423
[14:42] <elopio> no more meetings for... almost one hour.
[14:42] <elopio> brendand: is there something useful I can do before the meeting?
[14:43] <brendand> elopio, i wanted to know if you tested barrys packaging changes?
[14:43] <elopio> brendand: I did.
[14:43] <brendand> elopio, if you leave out -B it tries to build the package but it fails because of lack of fakeroot on the device
[14:44] <elopio> brendand: yes. And if you add -B, it fails for barry but works for the rest.
[14:44] <elopio> brendand: have you tried with -B ?
[14:45] <brendand> elopio, -B means not to build anything
[14:45] <elopio> brendand: yes, but the ubuntu_ota_tests do not need to be build
[14:45] <barry> i have apparently bricked my phone :/
[14:45] <elopio> they are on the current working directory.
[14:46] <brendand> elopio, hmmm. yes but only for the phablet user
[14:47] <brendand> elopio, it fails for commands run with sudo (the import)
[14:47] <brendand> elopio, at least when using Tests: (i was using Test-Command:)
[14:49] <elopio> brendand: this is the command I'm currently using:
[14:49] <elopio> Test-Command: PYTHONPATH=$PYTHONPATH:$(pwd) python3 -m unittest ubuntu_ota_test\
[14:49] <elopio> s.tests.test_basic_upgrade
[14:49] <elopio> once we figure out how to make it work for everybody without -B, exporting patching the python path won't be needed.
[14:49] <elopio> brendand: for your case, can you patch the pythonpath ?
[14:50] <barry> right, we really should not need to hack sys.path
[14:50] <barry> that's kind of the whole point ;)
[14:51] <brendand> barry, the build doesn't work for me unless i install fakeroot on the device (and specify --gpg-dir)
[14:51] <elopio> barry: we need to figure out what was special about your device that didn't fail with fakeroot. But now you bricked it :)
[14:51] <elopio> barry: were you running adt-run while the partition was r/w?
[14:51] <barry> yeah ;)
[14:51] <barry> no, i flashed it to -1 on ubuntu-touch/devel-proposed :(
[14:52] <barry> now i'm trying to find a working revision
[14:52] <elopio> I'm using 150. Newer ones are bombing in weird ways.
[14:53] <barry> heh, i skipped over 150, let me try that one
[14:54] <brendand> elopio, barry - don't go near 153
[14:55] <barry> spinny ubuntu symbol isn't spinny
[14:58] <barry> yeah, even 150 asks for my passphrase but gives me no digits to push
[15:00] <zsombi> ping ubuntu-qa
[15:01] <brendand> zsombi, hey
[15:01] <zsombi> I need some help in creating an autopilot test case for a sample test app which supposed to be an UriHandler
[15:01] <zsombi> brendand: hello :)
[15:02] <brendand> zsombi, ok, what problem do you have exactly?
[15:02] <zsombi> brendand: so, UITK autopilot test, must launch a test app with a given set of URIs
[15:02] <zsombi> brendand: the test must be an app which url_dispatch can call
[15:02] <brendand> elopio, i can't rely on PYTHONPATH because i need to use sudo to call the upgrade commands and it's not passing the environment through (even if i use -E)
[15:03] <zsombi> brendand: the instructions I see here https://developer.ubuntu.com/en/apps/platform/guides/url-dispatcher-guide/ state that I'd need to get 2 JSON files, but these are for clicks, and we all have fake desktop files, without any proper executable line...
[15:03] <brendand> zsombi, so you need to make a minimal app which can handle url-dispatcher requests?
[15:04] <brendand> zsombi, ok
[15:04] <zsombi> brendand: that's the smallest pain, I need to get that app to be registered as URL handler
[15:04] <zsombi> brendand: the app is fine, we have the QML elements to do the job
[15:05] <brendand> zsombi, ok. i'm in a bit of a tangle at the moment so i'll get back to you
[15:05] <brendand> zsombi, but i understand your problem and acknowledge your request
[15:05] <zsombi> brendand: ok, thx, you might get me on mail eventually, my day is pretty much done :(
[15:07] <brendand> balloons, issue with this page: https://developer.ubuntu.com/en/apps/platform/guides/url-dispatcher-guide/
[15:07] <brendand> balloons, the section on 'temporarily hardcoded urls' is false
[15:09] <balloons> brendand, looking
[15:09] <brendand> elopio, thoughts on zsombi's request?
[15:09] <brendand> balloons, that was reverted i distinctly remember, but confirm with ted
[15:09] <balloons> brendand, ahh.. so there are no special urls?
[15:10] <brendand> balloons, support for file:// urls was completely removed
[15:11] <brendand> balloons, again i'd confirm with ted though
[15:11] <balloons> right, will do.. thanks
[15:12] <elopio> brendand: zsombi: we need ted involved in this discussion. How to register a click app as url handler without doing the click install ?
[15:12] <brendand> elopio, yeah
[15:13] <tedg> Howdy
[15:13] <tedg> There is still a hardcoded URL for apps, but not the music or video ones.
[15:13] <tedg> brendand, ^
[15:13] <brendand> tedg, so that page is wrong?
[15:13] <tedg> Out of date, yes.
[15:14] <jfunk> ubuntu-qa -- new QA Engineer role is now up
[15:14] <jfunk> so tell your friends
[15:14] <brendand> tedg, we also had a question, how would one register an app as a url handler without actually doing the click install?
[15:14] <balloons> tedg, I'd be happy to help get the page updated. Is there a source for up to date information?
[15:14] <davmor2> jfunk: you think we have friends
[15:14] <brendand> tedg, for test purposes for example
[15:14] <om26er> davmor2, talk for yourself :p
[15:15] <brendand> jfunk, all my friends are already QA engineer :)
[15:15] <brendand> jfunk, or should that be :(
[15:15] <rvr> davmor2: Silos are very friendly to me ;)
[15:15] <tedg> brendand, You drop a url-dispatcher file in ~/.config/url-dispatcher/urls/
[15:15] <tedg> brendand, The name of the file is $(appid).url-dispatcher
[15:15] <om26er> davmor2, I have 200 friends on facebook
[15:15] <tedg> brendand, It should then detect the file and update the DB.
[15:16] <tedg> brendand, BTW, that feature has a unit test, but I don't think anyone has used it regularly, so there might be bugs.
[15:16] <davmor2> om26er: now remove family and people that work at canonical and how many then ;)
[15:16] <tedg> balloons, No, just delete the "Temporary" section.
[15:16] <brendand> tedg, where does the appid come from? considering this wouldn't actually be an installed click
[15:17] <tedg> brendand, From the desktop file for the application. Which would be $(appid).desktop
[15:17] <brendand> tedg, so as long as it matches the .desktop file that's ok?
[15:17] <tedg> brendand, That would probably be in ~/.local/share/applications
[15:17] <elopio> brendand: we are using a temp file for that, so it's like tmp123p1masd.desktop, but it's something we can get.
[15:17] <elopio> barry, brendand:
[15:17] <elopio> $ bzr bd
[15:17] <elopio> $ adt-run --built-tree=. --- ssh -s adb
[15:17] <elopio> That should work for read-only.
[15:17] <tedg> brendand, Correct, basically URL dispatcher will pass that to UAL.
[15:18] <tedg> brendand, So if UAL can launch it, you're good.
[15:18] <brendand> zsombi, does tedg's comments help?
[15:19] <barry> elopio: that makes sense i think
[15:19] <barry> fwiw, i'll need to do a quick reboot in a few minutes
[15:20] <elopio> tedg: I have doubts about this: https://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.UriHandler/
[15:20] <elopio> how do you pass those URIs to the app through ual ?
[15:20] <tedg> elopio, Magic :-)
[15:20] <tedg> elopio, They replace the %u on the Exec line in the desktop file
[15:21] <tedg> elopio, Or if the app is running it is sent via dbus.
[15:21] <elopio> ahhh
[15:21] <balloons> tedg, brendand section removed ty
[15:22] <tedg> Awesome, thanks balloons
[15:22] <elopio> rhuddie: did you work on the noop test, or was it federico? I think it needs to be skipped when there is an update.
[15:22] <elopio> my phone has just upgraded during selftests.
[15:22] <brendand> elopio, that didn't work for me
[15:22] <brendand> elopio, i still get the fakeroot error
[15:23] <elopio> brendand: with --built-tree? That's not right, it shouldn't build anything.
[15:23] <rhuddie> elopio, that one was federico's, I saw it upgrader earlier too
[15:24] <elopio> rhuddie: do you think it's right to skip it? Just like on your test but the opposite? :)
[15:24] <brendand> elopio, see i look at buil-tree and automatically read unbuilt-tree!
[15:24] <brendand> which could have been the problem all along...
[15:24] <rhuddie> elopio, well, given that its a self test it shouldn't be doing the actual update... let me have quick look
[15:25] <elopio> brendand: the readme currently says unbuilt-tree. So you were right before.
[15:25] <brendand> elopio, maybe that was a typo
[15:25] <brendand> elopio, so we update the README too?
[15:25] <elopio> brendand: no, for some reason it was working for barry.
[15:26] <elopio> brendand: there's a card for updating the README. But the instructions depend on whether we land veebers branch or don't.
[15:26] <elopio> at this point, I'm guessing we won't land it.
[15:26] <brendand> elopio, isn't that card for sanity tests?
[15:26] <brendand> elopio, who added it?
[15:26] <elopio> brendand: no, for ota tests. I did, when I noticed the README made no sense with what we have now.
[15:27] <brendand> elopio, it shouldn't really be extra then
[15:27] <brendand> elopio, it's related to a story
[15:27] <elopio> rhuddie: I would like to have fake upgrades for these tests, but that will take some time to write.
[15:27] <elopio> brendand: it's related to all the cards, but we forgot to edit it in all the cards.
[15:28] <elopio> so I made a new one so we didn't forget at the end. I will update it.
[15:29] <rhuddie> elopio, yes, I agree, some local server would be much better.
[15:30] <elopio> rhuddie: maybe even a dbus mock. If you agree, I will make the skip on this README test, so it makes sense to say that we should start by flashing an old version.
[15:31] <elopio> s/README test/README branch
[15:32] <rhuddie> elopio, yes that sounds good to me.
[15:32] <brendand> elopio, so with that command the existing tests work but that seems to be a coincidence
[15:32] <brendand> elopio, since i don't think it actually installs python3-ubuntu-ota-tests
[15:32] <elopio> brendand: it does. At least, that's what I understand from the docs.
[15:33] <brendand> elopio, unless of course it's specified as a dependency in a test
[15:33] <brendand> elopio, and when i do that it it says 'E: Unable to locate package python3-ubuntu-ota-tests'
[15:34] <elopio> I will try that.
[15:36] <brendand> 'Note that all test dependencies are then satis‐
[15:36] <brendand>               fied by archive packages, unless you explicitly specify  locally
[15:36] <brendand>               built .debs with --binary.'
[15:37] <elopio> brendand: where are you getting that from?
[15:37] <brendand> elopio, from the adt-run manpage
[15:37] <elopio> I see. brendand and does that work for you?
[15:38] <brendand> elopio, i'm trying it
[15:41] <brendand> elopio, it doesn't seem to do anything though
[15:49] <elopio> brendand: did you pass the --binary before the build-tree?
[15:50] <brendand> elopio, strange that makes a difference
[15:51] <brendand> elopio, now it fails because of read-only
[15:51] <elopio> brendand: well, it gives a different error :)
[15:51] <elopio> I will report a bug to adt-run, because it should install the binaries in the tmp.
[15:53] <elopio> brendand: so for your branch, what about cd into the source directory after switching the user?
[15:54] <brendand> elopio, i don't actually switch the user
[15:54] <brendand> elopio, that actually breaks adt-run
[15:55] <brendand> doing sudo -u in a test script causes it to hang
[15:55] <elopio> brendand: I know, but like: sudo cd /tmp/blah; sh debian/test/upgrade
[15:56] <brendand> elopio, i could try that
[15:56] <elopio> brendand: there's also the option to install fakeroot with a setup command, which I presume is the problem without -B.
[15:56] <brendand> elopio, yeah
[15:57] <brendand> elopio, it's a 'pity' pitti is not here
[15:57] <elopio> and anyway, all this would be temp solutions until we get the bug solved.
[15:58] <elopio> brendand: also, I think you should report a bug for not being able to switch the user.
[16:50] <elopio> barry: brendand: https://bugs.launchpad.net/ubuntu/+source/autopkgtest/+bug/1434163
[17:56] <barry> elopio: i wonder if i wasn't hitting that bug because i made my root fs writable?
[17:56] <balloons> la_juyis, fyi, the help app is in the store if you want to check it out on your device. I'm curious again about any video content you might have
[18:06] <elopio> barry: yes, the problem is that with r/o the fakeroot is not properly installed.
[19:41] <Guest10803> I submitted my first merge request!
[19:41] <Guest10803> https://code.launchpad.net/~darran-kelinske/music-app/lp_bug_1428454/+merge/253566
[19:42] <dazza5000> thank you to Nicholas Skaggs for the documentation on how to write autopilot tests!
[19:44] <alesage> balloons ^^ woo!
[19:44] <dazza5000> me happy
[20:12] <dazza5000> Is anyone running the ubuntu-calendar-app on vivid?
[20:14] <dazza5000> When I try to add a calendar I get this in syslog: Mar 19 15:12:36 localhost gnome-session[1833]: gvfs-open: settings:///online-accounts: error opening location: The specified location is not supported
[20:24] <dobey> dazza5000: url-dispatcher and all that stuff only works on the phone images with mir currently; tried doing it under unity8 preview session instead?
[20:25] <dazza5000> not yet
[20:25] <dazza5000> haven't run a unity8 preview session before
[20:49] <dazza5000> if you have to make changes to a merge request
[20:59] <balloons> dazza5000, glad you liked the docs
[21:09] <Letozaf_> balloons, hey