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