[00:22] \o/ released [07:43] thomi, now [07:43] thomi, I saw your ap 1.5 change [09:13] Saviq: cool - any chance you can get it reviewed, merged, and landed? :) [09:14] thomi, some, yes ;) [09:14] Saviq: for the price of a beer in Malta? [09:14] thomi, ;) [11:42] elopio, balloons: Do you happen to know if the new SDK Headers are support by the emulator helper functions? Just curious since I am getting a branch ready in the clock app to make the transition but wondering the AP side of things. [11:42] s/support/supported [13:33] nik90, elopio is the best person to answer that [13:34] however, I believe work was being done in conjunction. I'd have to dig to find the mp [13:47] balloons: ok. I sort of remember t1mp fixing the AP while implementing the new transitions but wanted to confirm with you guys. [13:48] s/transitions/headers [14:20] nik90: yes, you should have helpers for the new features of the header. [14:25] elopio: awesome [16:26] autopilot gurus, please help =) [16:26] http://paste.ubuntu.com/7468524/ says in the end 4 failures, yet, i can only find one FAIL [16:27] FAIL: gallery_app.tests.test_album_view.TestAlbumView.test_add_photo [16:27] what are the other failures? [16:32] nuclearbob, cgoldberg: do you know what could cause this? [16:32] AttributeError: 'module' object has no attribute 'get_classname_from_path' [16:32] http://91.189.93.70:8080/job/generic-mediumtests-utopic/190/testReport/junit/reminders.tests.test_reminders/RemindersTestCaseWithoutAccount/test_go_to_account_settings_with_mouse_/ [16:36] elopio, this looks suspicious in the log: [16:36] file:///usr/share/reminders/qml/ui/NotesPage.qml:83: TypeError: Cannot read property 'reminder' of null [16:36] file:///usr/share/reminders/qml/ui/NotesPage.qml:79: TypeError: Cannot read property 'reminder' of null [16:36] file:///usr/share/reminders/qml/reminders.qml:287:30: Unable to assign null to QQuickAnchorLine [16:37] cgoldberg: I think that's safe to ignore, as the videos show the application being properly displayed. [16:40] elopio, this is from https://code.launchpad.net/~dpm/reminders-app/fix-1317683-update-template/+merge/219669? basically showing that trusty works, while utopic doesn't right/ [16:40] elopio, can I see reminders/__init__.py ? [16:41] cgoldberg: http://bazaar.launchpad.net/~reminders-app-dev/reminders-app/trunk/view/head:/tests/autopilot/reminders/__init__.py [16:42] elopio, dbus.get_classname_from_path ? [16:44] it overrides validate_dbus_object and calls that ^^ but what is that? [16:50] that's how nuclearbob told me to use a python class that's different from the qml class. [16:55] hey, the method was removed! [16:55] weird.. i tried from command line and I can see that attribute fine [16:55] elopio, but it's not in API docs [16:57] elopio, in changelog for 1.5: Autopilot introspection package now has a cleaner API [16:59] balloons: on trusty we have autopilot 1.5 and on utopic 1.4? or something like that? [17:01] elopio, locally... i have 1.4 on trusty.. but 1.5 landed in utopic afaik [17:02] elopio: sorry about that, I was in a meeting, I'm reading the scrollback now, did you get something resolved? [17:04] elopio, so did 1.5 break tests? [17:05] cgoldberg: yes [17:05] doh! is it isolated to that one? [17:05] nuclearbob: well, now I know the reason. The new introspection api breaks our reminder tests. [17:05] should I put a depends autopilot (= 1.4) ? [17:06] I'll echo cgoldberg's doh [17:06] elopio: which tests was it breaking? I'd like to look back through the testing we did and see how we could have caught that [17:07] http://91.189.93.70:8080/job/generic-mediumtests-utopic/190/testReport/junit/reminders.tests.test_reminders/RemindersTestCaseWithoutAccount/test_go_to_account_settings_with_mouse_/ [17:08] nuclearbob: can you put a deprecation warning instead of removing that function? [17:12] elopio: I'll propose that in the meeting today [17:27] elopio, nuclearbob ohh.. I'll bet you didn't test reminders did you? since it's not in the image [17:28] I should go look [18:09] balloons: let me know what you find out, I'd like to make sure it's tested in the future [18:14] nuclearbob, I couldn't find the doc thomi shared with me highlighting everything [18:14] but yea, just make sure it's on the list :-) [19:06] balloons, elopio hi [19:06] Letozaf_, hello [19:08] balloons, I saw elopio's branch for evernote sdk [19:08] Letozaf_, yes, essentially he wants to use the python3 version of the sdk which hasn't been packaged yet [19:09] balloons, ok so this means that we will have to wait I suppose [19:10] Letozaf_: but we can work on your test on the mean time. [19:10] elopio, ok :) [19:10] you can just set the PYTHONPATH to point to the right thrift module. [19:11] Letozaf_: do you have some time now? [19:11] elopio, yes sure [19:11] Letozaf_: ok, first thing, merge your branch with mine. [19:11] bzr merge lp:~elopio/reminders-app/evernote-sdk [19:12] elopio, ok just a second [19:14] elopio, done, got 4 conflicts... [19:15] * Letozaf_ is looking at the conflicts [19:29] elopio, should be ok now, no more conflicts [19:30] Letozaf_: great. so, next, github clone https://github.com/evernote/evernote-sdk-python3.git [19:30] anywhere on your workspace. [19:33] elopio, haven't got the github command, in what package is it ? still looking for it... [19:33] Letozaf_: sudo apt-get install github [19:33] oh, no, sorry. [19:33] it's not github, just git [19:33] sudo apt-get install git. [19:33] elopio,ok thanks [19:35] elopio, ok done [19:36] Letozaf_: now, when you run the tests, you will have to do PYTHONPATH=path/to/your/workspace/evernote-sdk-python3/lib autopilot3 run reminders [19:37] elopio, ok let me try [19:41] elopio, got some errors :( trying to figure out what they are [19:41] Letozaf_: paste them here, I might be able to help. [19:41] elopio, ok [19:42] elopio, http://paste.ubuntu.com/7469435/ [19:43] Letozaf_: do you have the applications ppa on your machine? [19:44] elopio, I have the ubuntu-touch-coreapps-drives one [19:44] elopio, I am running utopic on my desktop [19:45] Letozaf_: yes, then just install evernote-sdk-python3 [19:45] that will solve your missing dependencies. [19:45] elopio, ok [19:48] elopio, now I got this: http://paste.ubuntu.com/7469469/ [19:49] agh, you updated autopilot [19:49] it has a regression, they changed the api breaking our tests. [19:49] elopio, :( [19:49] Letozaf_, you are on utopic yes? [19:49] balloons, yes sir [19:49] Letozaf_, I'll give you the old package to install [19:50] balloons, ok thanks [19:52] https://launchpad.net/ubuntu/+archive/primary/+files/python3-autopilot_1.4%2B14.04.20140416-0ubuntu1_all.deb [19:52] https://launchpad.net/ubuntu/+archive/primary/+files/python-autopilot_1.4%2B14.04.20140416-0ubuntu1_all.deb [19:52] start with those.. might need more packages [19:53] * Letozaf_ is installing [19:55] balloons, done [19:56] Letozaf_, if that doesn't fix it we'll need to backdate libs too [19:56] give it a try [19:56] and yea, don't update ;-) [19:58] * Letozaf_ is running the tests [20:03] hey thomi.. morning to you [20:03] balloons, elopio got this: http://paste.ubuntu.com/7469563/ [20:04] morning [20:04] barry: did you get time to look at the packaging stuff? [20:05] thomi: somehow, i knew that was coming. :) no, but i am finishing up my Big Project today and you're tops on my list [20:05] OK :) [20:05] I'll keep nagging, although my next working day will be the first day in Malta [20:06] thomi: safe travels! i'll follow up to the email [20:06] awesome, thanks - and sorry for the nagging [20:06] thomi, can you share the paste you had showing the the tests you run before ap 1.5? [20:06] no, it's cool. i *want* you to nag me 'til its done [20:07] Letozaf_: try this before running the tests from the root of the branch [20:07] cmake . [20:07] make [20:07] elopio, ah yes! I had forgot about this :P [20:18] elopio, well it's better now but still got some failures: http://paste.ubuntu.com/7469632/ [20:19] Letozaf_: yes, your online accounts is crazy [20:19] May 15 22:15:44 letozaf-PC signond[19627]: ../../../../src/signond/signondaemon.cpp 388 init Failed to SUID root. Secure storage will not be available.}}} [20:19] process-return-code: {{{-15}}} [20:19] May 15 22:16:06 letozaf-PC kernel: [ 9309.074638] reminders[19638]: segfault at 10 ip 00007f63f0d63950 sp 00007fff8f1f2110 error 4 in libsignon-qt5.so.1.0.0[7f63f0d57000+34000]}}} [20:19] process-return-code: {{{-11}}} [20:20] Letozaf_: can you remove all your evernote accounts? That might help. [20:20] elopio, :) I was looking but I have no Evernote accounts in online accounts :O [20:20] elopio, must be somewhere else [20:22] Letozaf_: I don't know what to do with that. mardy is the only one that knows about online accounts. [20:22] elopio, could it be I have to remove the ~/.local/share/com.ubuntu.reminders [20:22] elopio, directory ? [20:22] unlikely as we are using a tmp home. [20:23] but remove it if you are not going to miss it. [20:23] elopio, ok [20:26] elopio, this did not solve the problem, not that I thought it would but I tried all the same [20:26] Letozaf_: try pkill -f signon* [20:27] elopio, done, let me run the tests again [20:30] elopio, http://paste.ubuntu.com/7469680/ [20:31] elopio, looks the same [20:34] Letozaf_: I don't know what else to look for. balloons: any idea? [20:34] * balloons is busy editing docs [20:34] let me have a look [20:34] my eyes need a break heh [20:35] ok, so Letozaf_ it might be worth doing a fresh build if you haven't done one recently [20:35] I like to clean up all the cmake stuff first [20:36] balloons, ok [20:36] find -iname '*cmake*' -not -name CMakeLists.txt -exec rm -rf {} \+ [20:36] make clean [20:36] meanwhile, I'll try. [20:36] balloons, done [20:37] balloons, form reminders-app main dir [20:37] from [20:37] Letozaf_, yep.. top level of reminders-app [20:37] balloons, ok so done [20:38] what's the branch [20:38] Letozaf_, then rebuild it using cmake . [20:38] make [20:38] balloons, ok [20:38] balloons, witch branch mine ? [20:39] Letozaf_, whichever branch I need to reproduce what you and elopio are doing [20:39] I'm lost [20:39] balloons, let me push this to a new one, just in case [20:40] Letozaf_: resubmit the merge proposal with my branch as a prerequisite, please [20:41] elopio, you mean push lp:~elopio/reminders-app/xxx ? [20:42] Letozaf_: no, I mean, go to the website https://code.launchpad.net/~carla-sella/reminders-app/download-list-of-notebooks-and-notes/+merge/218325 [20:42] you'll see a button on the top-right that says resubmit proposal [20:42] on the for that will be opened, there's a field for the prerequisite branch. [20:43] elopio, yes got it [20:43] put lp:~elopio/reminders-app/autopilot-add_notebook there [20:43] no, that's not the one. [20:43] lp:~elopio/reminders-app/evernote-sdk [20:44] elopio, ok done, so now I should push what I've got here ? [20:45] Letozaf_: yes. [20:46] balloons, ok so lp:~carla-sella/reminders-app/download-list-of-notebooks-and-notes [20:46] balloons, is the branch [20:47] elopio, looks like I made a mess I have conflicts [20:48] elopio, let me see if I can fix [20:48] Letozaf_: I see it fine here: https://code.launchpad.net/~carla-sella/reminders-app/download-list-of-notebooks-and-notes/+merge/218325 [20:49] elopio, oh! wonder what I was lookig at :P [20:50] * Letozaf_ is rebuilding with cmake [20:52] I'm sorry, I need to go and have lunch now. [20:52] I'll be back later. [20:53] elopio, I will be back tomorrow, it's late here [20:54] ok, have a good night. [20:54] elopio, thank you and enjoy your lunch [20:58] balloons, this is what I got after rebuilding with cmake http://paste.ubuntu.com/7469814/ [20:58] balloons: http://etherpad.ubuntu.com/B2Wt9rUiHX [20:58] sorry for the delay [20:59] balloons, I think I will go to bed now it's quite late, I will be back tomorrow ok ? [21:19] thomi, ok so I wanted the pad link to confirm one thing. And indeed,it seems you did not test reminders [21:20] Which is fine, just if you could add that to the list. I've been trying to get it into the image, but they are waiting for it to use the production evernote servers. It should happen soon. [21:20] anyways, those tests were broken by ap 1.5 [21:26] balloons: we're just talking about it now [21:26] balloons: care to join our hangout? [21:26] elopio: are you still there? [21:26] balloons: or elopio: https://plus.google.com/hangouts/_/calendar/Y2Fub25pY2FsLmNvbV91bnZxZzdzamhoMW11YzJxZ2FxZjA5a2p2NEBncm91cC5jYWxlbmRhci5nb29nbGUuY29t.fk2gi9fumct9rei9q2fatpq7u0?authuser=1 [21:32] elopio: balloons: too slow! [21:32] sorry, didn't see ;-( [21:33] balloons: so the conclusion is that we messed up, and didn't include that function in the API. I'll propose an MP that fixes that toda [21:34] y [21:34] balloons: However, we're reluctant to start running test suites that aren't in the dashboard list [21:34] thomi, I do get that.. but reminders is a bit special. Still, it's fine if you don't want to add it now. It will be in the list soon enough [21:35] yeah - when it's in the list we'll start running those tests magically [21:35] sorry about missing that function from our public API. We're trying to get better this cycle at separating our public from our private API [21:36] I'm ok with the mistake.. glad we caught it. It's just annoying to have this stuff come up when I've been pushing for reminders to be on the dash.. [21:37] stuff like this is the reason why.. if the tests aren't on the dash, they aren't being exercised. And they could break.. and in general they serve no purpose unless they are being run [21:37] yup [22:00] veebers: any chance you could glance at https://code.launchpad.net/~thomir/autopilot/temp-dev-add-get_classname_from_path-to-public-api/+merge/219765 please? [22:01] thomi: sure thing [22:01] veebers: also, I realise I forgot to scope that task [22:01] thomi: have you run it over reminders app to make sure it fixes the issue? [22:01] thomi: heh, d'oh :-P [22:01] no [22:02] but this is the only place I'm willing/able to expose that function, so if it's not right, they'll need to import it from here [22:09] thomi: ok [22:11] thomi, hey.. reminders was calling get_classname_from_path on autopilot.introspection.dbus [22:16] http://bazaar.launchpad.net/~reminders-app-dev/reminders-app/trunk/view/head:/tests/autopilot/reminders/__init__.py#L68 [22:21] cgoldberg: yeah, but that's not going to be public for much longer [22:21] balloons: who's working on that app? [22:22] thomi, Leo notified us today.. so he will need to change his import to use your fix [22:22] thomi, elopio and carla atm.. I've not done much but comment and help :-) [22:22] balloons: OK, well, I guess their approach is to import it from the private module today [22:23] and when we do the new release y'all will have to import it from the autopilot.introspection package instead [22:23] but that can be done at your leisure [22:23] :-) [22:24] thomi: oh, approved btw