[01:49] thomi: hey, what's you opinion on this approach for restarting maliit-server for the ubuntu keyboard tests? https://code.launchpad.net/~veebers/ubuntu-keyboard/restarting-maliit-for-tests/+merge/190005 === _salem is now known as salem_ === salem_ is now known as _salem === plars_ is now known as plars === chihchun_afk is now known as chihchun [04:57] Good morning [05:00] bonjour jibel, ça va? [05:03] pitti, ça va et toi? [05:03] jibel: je vais bien, merci ! [05:03] yet another build :/ I'm happy the release is approaching, I couldn't do that forever [05:16] good morning all === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun [06:16] hey DanChapman [06:30] hey Noskcaj :-) === wxl_ is now known as wxl === iahmad is now known as iahmad|afk [09:11] morning all === iahmad|afk is now known as iahmad [09:37] elfy: ping [09:57] Morning all [09:58] slickymaster: pong [09:58] davmor2: morning [09:59] elfy: hi. good morning. Whenever you'll have 5 minutes to spare, https://code.launchpad.net/~slickymaster/ubuntu-manual-tests/TomcatJavaServer_Install/+merge/189444 is waiting for your review [09:59] ok [09:59] thanks [10:06] slickymaster: thanks - all done [10:07] elfy: np, it's me who thanks you [10:07] :) [12:46] davmor2: ping === chihchun is now known as chihchun_afk [12:59] nuclearbob: pong === chihchun_afk is now known as chihchun [15:57] hello! [15:57] I'm sorry for the late start. There was no power. [16:01] ping DanChapman [16:29] elopio: excuses excuses ;) It's a good one though so we'll let you off.....Oh and morning dude :) [16:30] good morning davmor2 :) [16:38] could anyone verify bug 1237501 ? [16:38] bug 1237501 in unity8 (Ubuntu) "[Regression] Expanding Installed category does not show all the installed apps at once" [Medium,New] https://launchpad.net/bugs/1237501 [16:38] balloons, does ubuntu-ui-toolkit or SDK Team have an irc channel? my sphinx documentation branch is failing CI. I want to make sure I didn't break anything in the qdoc build... so looking for someone to chat with [16:39] you can talk in #ubuntu-touch [16:39] or #sdk internally [16:39] balloons, k thanks [16:44] balloons, confirming bugs much ? :) [16:44] om26er, this week, not so much [16:45] I'm full on with autopilot tests, mir, and sdk [16:45] busy balloons is busy [16:45] om26er, lol :-) I'm happy to confirm a bug for you if needed however [16:45] balloons, if you are using image 89 bug 1237501 [16:46] bug 1237501 in unity8 (Ubuntu) "[Regression] Expanding Installed category does not show all the installed apps at once" [High,New] https://launchpad.net/bugs/1237501 [16:46] om26er, I'm not yet, but I can be.. :-) [16:46] a more easier way to verify is specified in my first comment. Make sure you are connected to wifi so that 'more suggestions' appears [16:47] balloons, ok, its not urgent. If you get around to flash the device later in the day, that's fine [16:47] om26er, certainly. I need to update my device today so I'll confirm when I do [16:47] balloons, ack. thanks [17:09] om26er, are you missing datetime in the indicators? [17:09] balloons, its not shown on some reboots [17:09] known bug. [17:09] om26er, ty [17:10] you are confirmed [17:12] bug 1227519 [17:12] bug 1227519 in indicator-datetime (Ubuntu) "no time indicator on boot" [Medium,Confirmed] https://launchpad.net/bugs/1227519 [17:30] balloons: pong! sorry for the slow response :-) [17:32] DanChapman, no worries mate! [17:32] always lots going on this last weeks. So I wanted to see how we were doing in regards to ubiquity is all.. And yourself, hanging in there? [17:38] balloons: The tests themselves are coming along nice. Just adding the finishing touches to the custom install. Ended up taking a different approach, it now selects a random partition config from a selection set out in a config file. Rather than seperate tests for different configurations :-) still waiting to see them running on jenkins though [17:39] hows your week going? [17:42] DanChapman, not too bad at all. Today is much saner than the first couple of days. We're not quite 1 week to release [17:45] balloons: eek it's coming round quickly. [17:48] roll on Friday week [17:50] evening elfy :-) [17:51] hi DanChapman [18:52] buonsera letozaf. come stai? [18:53] balloons, buonasera :D bene e tu ? [18:53] bene [18:53] balloons, I see your Italian is improving a lot :) [18:54] letozaf, lol.. you've just seen all of it :-) [18:54] balloons, :D [18:55] so rssreader is the last app to get squared away.. how are we looking? :-) [18:55] good morning Noskcaj [18:56] hey balloons [18:58] balloons, got to fix an error, hope quickly :p [19:02] letozaf, let me know.. just finished up music app I think, so I can dive in.. I'll probably grab your branch now ;-) [19:02] balloons, wait let me push the last changes [19:09] morning [19:11] balloons, if I stop bumping into errors I will push it shortly :P [19:13] hehe [19:15] balloons, I have an error that maybe needs your help [19:16] balloons, TypeError: 'tuple' object is not callable [19:16] letozaf, yes, is everything pushed? [19:16] balloons, yes [19:16] me syncs [19:16] https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/edit-topic-test [19:16] balloons, it happens in a "strange" place [19:17] balloons, that is the remove feed part, that part has been called before and did not have problems [19:17] balloons, it returns an error in the new test [19:17] ubuntu_rssreader_app.tests.test_rssreader.TestMainWindow.test_edit_topic? [19:18] ok got the error [19:18] * balloons looks @ code [19:20] letozaf, essentially what that error is telling you is you can't use eventually on a non-function [19:20] self.assertThat(feedlistpage.visible, Eventually(Equals(True))) [19:20] TypeError: 'tuple' object is not callable [19:20] feedlist.visible will never change, so eventually doesn't make sense [19:20] balloons, yes but if you run the ubuntu_rssreader_app.tests.test_rssreader.TestMainWindow.test_view_feeds [19:20] letozaf, does it have the same thing, lol? [19:20] balloons, test the same routine is called and no error [19:21] balloons, I'm a bit "confused" :D [19:22] letozaf, well let's fix it to be correct regardless [19:22] it should fail.. it doesn't make sense [19:22] likely what is happening is it is true right away, so it's not triggering perhaps [19:22] balloons, yes [19:23] letozaf, feedlistpage = lambda: self.main_view.get_feed_management_page().visible [19:23] self.assertThat(feedlistpage, Eventually(Equals(True))) [19:23] letozaf, got a link to the stacktrace? sounds like you are passing a tuple where it is expecting a callable (funtion name) essentially) [19:24] letozaf, I see more like this.. [19:24] * balloons edits [19:25] cgoldberg, if you want the code is here https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/edit-topic-test [19:25] letozaf, OHH [19:25] cgoldberg, thanks [19:25] balloons, what ? [19:26] letozaf, can you paste me the error you are getting so I dont have to run it? :) i just wanna look over the code quick and see if I can identify something [19:26] full stracktrace [19:26] ignore my idea is all letozaf :-) [19:27] cgoldberg, is this enough ? http://paste.ubuntu.com/6215135/ [19:27] should do [19:28] letozaf, because it's running through select_single_retry [19:28] balloons, ooh!! [19:29] I was thinking perhaps it was sending an error or something [19:29] but that doesn't look correct at all [19:29] can somebody confirm if bug 1237594 is affecting konsole? [19:29] bug 1237594 in xfce4-terminal (Ubuntu) "The U+0001 control character is output in terminal" [Undecided,Confirmed] https://launchpad.net/bugs/1237594 [19:29] so as i said, ignore me ;-) [19:29] (hai) [19:29] balloons, lol [19:30] knome, cool bug [19:30] I like the colors! [19:30] * knome facepalms [19:30] it affects gnome and xfce terms that I can tes [19:31] i know, i marked those as affected ;)= [19:31] probably related to libvte [19:31] xterm works fine [19:31] yep, that's noted too in the bug report ;) [19:32] and that's not using libvte [19:32] I would agree with your synopsis [19:33] letozaf, so for some reason you are getting back something weird.. the asserts and everything all makes sense now [19:34] balloons, yes I'm stuck [19:34] a tuple is an immutable list more or less.. cgoldberg and/or thomi are more up on python than me [19:34] balloons, do not know how to solve it [19:34] yo [19:34] balloons, yes I read about that [19:34] why you are getting something like that is odd, but I'll still call out the _retry function :-) [19:34] bbl [19:35] hey thomi :-) [19:35] * thomi hears his name [19:35] sup? [19:35] letozaf, i'm looking.. somewhere you are passing a tuple when it's excpecting aq callable object [19:35] thomi, a select_single is returning a tuple [19:35] umwut? [19:35] got a code example? that's odd [19:35] cgoldberg, thanks for your help [19:35] thomi, however, it's passing through a select_single_retry function, so I'm blaming that [19:36] letozaf, it would also be nice to find an idiomatic way to remove those lambda's if possible... me hates lambdas.. and they are sorta sprinkled intermittently in that code [19:36] thomi, https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/edit-topic-test [19:36] morning thomi [19:36] cgoldberg, it used to be much worse :-) [19:36] hi cgoldberg [19:36] letozaf: what file/line? [19:36] thomi, the test_edit_topic [19:37] line 117 [19:37] thomi http://paste.ubuntu.com/6215135/ [19:37] thomi, if you run it [19:37] line 1116 is an attempt to get the object [19:37] thomi, you get the error [19:37] cgoldberg, I also do not fancy lambda's :p [19:38] * thomi looks [19:38] the poor rssreader testcase is abused.. it's quite beat [19:38] brb [19:38] I've been on both sides of it [19:38] thomi, the strange fact is that if you run the test_view_feeds test it works no error [19:39] found the problem [19:40] line 392 in that file: [19:40] self.assertThat=(topicValueselectorValue.text, Eventually(Equals("news"))) [19:40] you're assigning a tuple to the 'assertThat' method - I guess a typo that happened to be valid python :) [19:40] ohh nice! [19:41] and line 420, same thing [19:41] copy paste error I guess [19:41] thomi, :( [19:41] :) [19:42] thomi, excellent work :-) [19:42] why thank you sir. Time for my second coffee I think [19:42] little errors like that are so hard to spot.. we knew select_single wasn't giving a tuple, haha [19:42] thomi, thank you [19:42] no worries [19:42] tests pass as-is with that fix [19:42] but I'm guessing you've more cleanup to do? [19:45] balloons, I'm quite new to python, but how do I check that tuple value now ? [19:45] balloons, cannot use asserthat [19:45] letozaf, no, the issue was you were assigning assertthat to a value [19:46] it's really weird.. the end result is just remove the accidental equals sign [19:46] everything works [19:46] balloons, ah ok, let me try [19:47] you can drop the sleep also if you add a retry to the get_feedlist_feed [19:48] then just an clean up work you need to do.. and merge proposal time.. [19:50] balloons, argh! forgot that sleep there :( [19:51] no worries.. glad it's working.. push it up and let's see what the mp looks like ;- [19:52] balloons, ooooh!!! that equals sign !!!! didn't see it lol [19:52] * letozaf slaps her head [19:52] it's the little things that get you === chihchun is now known as chihchun_afk [19:57] balloons, thomi thanks the test works now :p [19:57] sweet [20:00] elopio, ready to rock? [20:00] jfunk: yes sir. [20:01] jfunk: I think elopio more of a soul man thank a rocker ;) [20:01] davmor2, you got him all wrong, Leo is a rocker at heart [20:01] davmor2: I'm psicotropical. [20:01] hahah [20:01] elopio, can't hear you [20:02] jfunk: I just reinstalled my machine. Give me a second to figure it out [20:02] elopio, np [20:03] elopio: pulseaudio god always has to be to your east and in the rain for audio to work correctly ;) [20:03] and with that I bid you all a good night [20:06] balloons, proposed merge [20:12] letozaf, got it [20:12] balloons, :D [20:18] after seeing elopio's Google+ photo, I'm convinced he's a son of Frank Zappa.. so a rocker [20:22] letozaf, jenkins build: http://91.189.93.70:8080/job/ubuntu-rssreader-app-ci/89/ [20:24] balloons, it green :) [20:25] balloons, for now [20:26] balloons, red :( === salem_ is now known as _salem [20:35] letozaf, I see [20:35] a StateNotFoundError error [20:50] letozaf, so it failed the same way [20:50] well.. [20:50] cgoldberg: I was ashamed of going outside with my Zappa's mustache and took it off, so not a real rocker :( [20:51] no need to be ashamed.. wear with pride :) [20:51] balloons, something I can do ? [20:51] letozaf, well it's this line.. It's kind of odd [20:51] cgoldberg: next halloween. [20:52] balloons, did you have time to look at filemanager tests for click? [20:52] sergiusens, sure [20:52] I need 5 mins [20:52] balloons, oh, well the code init needs refactoring I think, but the test setup is a little big; doing the same with the music app myself [20:53] letozaf, maybe thomi has some insight here again [20:53] hmmm? [20:53] sergiusens, we just landed quite a bit of changes to music app, fair warning [20:53] sergiusens, what refactoring specifically? for click? for ro? [20:53] thomi, http://91.189.93.70:8080/job/generic-mediumtests/861/testReport/ubuntu_rssreader_app.tests.test_rssreader/TestMainWindow/test_edit_topic_with_mouse_/? [20:54] thomi, it's a StateNotFoundError [20:54] trying to land the code you were looking at earlier; https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/edit-topic-test/+merge/190234 [20:55] balloons, for click; when was this landed? [20:56] balloons, the music app has a home stub, but when launching the app with upstart, that's a bit messy [20:56] sergiusens, music app? sorry I should be specific, we merged code to trunk within the last 3 hours [20:56] balloons: it'd be good to see the test log as well [20:56] balloons: can you change it to run with -v? [20:57] thomi, they should all be running like that.. [20:58] fginther, can we ensure we are running autopilot with -v in the core apps vm's if we're not for some reason? ;-) [20:58] thomi, since it doesn't happen here, I can't give anything more [20:58] I like running with -vv actually.. I like logs :-) [20:59] balloons, ack [20:59] yeah, you need the logs before you can tell anything really [20:59] fginther, specifically we're looking at rssreader app atm [20:59] ty! [20:59] balloons, done [21:00] thomi, I'll get a log in a few :-) [21:01] balloons, so just to be aware, you know about the swipe to delete missing emulator? [21:01] balloons, and I can't seem to fix these either: http://reports.qa.ubuntu.com/smokeng/saucy/touch_ro/maguro/89:20131009.1:20131009.1/4640/ubuntu-rssreader-app-autopilot/470106/ [21:02] sergiusens, we're trying to fix rssreader right now actuall [21:02] sergiusens, swipe to delete? you mean the trash can change? [21:04] balloons: when you get a spare couple of minutes can you give me a ping for a quick PM. Thanks. [21:04] balloons, first I've heard of trash can change :-) [21:04] balloons, I mean this http://reports.qa.ubuntu.com/smokeng/saucy/touch_ro/maguro/89:20131009.1:20131009.1/4640/ubuntu-calculator-app-autopilot/469697/ [21:04] sergiusens, there was a swipe to delete change that would force you to tap a trash can icon after swiping [21:04] good night all [21:05] sergiusens, I'm not sure of the status atm on it.. don't have the bug handy [21:05] balloons, yeah, that; didn't think it would be referred to as the 'trash can change' :-) [21:05] sergiusens, hahahahah [21:05] sergiusens, I'm a slow typist my friend.. short and simple [21:05] balloons, no problem, bfiller told me that the sdk wouldn't do it by default but the feature hadn't landed yet [21:06] but it would be nice to have an emulator [21:06] sergiusens, right.. the not by default action made me go "whew" [21:07] balloons, need me to wait or can I go to sleep ? [21:08] letozaf, no reason to wait. arrivederci [21:09] balloons, buona notte! hope you fix rssreader :D [21:09] I'm sure sergiusens will make it work or remove it :-) [21:12] balloons, heh :-) [21:12] balloons, so back to music, is the refactor making it work with click as well? [21:12] fginther, ty, looks better now :-). thomi have a look: http://91.189.93.70:8080/job/generic-mediumtests/862/testReport/ubuntu_rssreader_app.tests.test_rssreader/TestMainWindow/test_edit_topic_with_mouse_/? [21:13] sergiusens, the primary purpose was keeping the tests inline with the huge UI changes that were landing [21:14] sergiusens, the rssreader was the last one on my list of stuff to fix before diving more into MIR. Are you finding issues with them running as click? [21:15] balloons, yes; but just the tests; everything runs until it gets an exception (qml) that you can see in the upstart log for the click app [21:19] sergiusens, and this is across the board with all the core apps/ [21:19] ? [21:19] balloons, no, only reader [21:20] sergiusens, ahh.. so it's probably just the tests not running successfully due to UI changes atm [21:20] so this merge should fix that issue [21:20] balloons, the error is in the dashboard [21:20] balloons, great [21:21] sergiusens, ahh you are basically looking at these eh? http://reports.qa.ubuntu.com/smokeng/saucy/touch_ro/maguro/89:20131009.1:20131009.1/4640/ [21:22] yup [21:22] darn, music app merged incorrectly, missing files in trunk :-/ [21:24] yes, so rssreader is the only one on that list I consider having a true issue. The others are a bit perplexing.. random failures [21:24] still an issue, as any failure is a potential sign tests aren't robust [21:28] balloons, not sure about dropping letters, feels the device just failed and calculator just needs that swipte to delete fix from the sdk [21:29] sergiusens, yes, dropping letters ran fine on the other device, and I've never seen a failure from it [21:30] sergiusens, ohh, if calc didn't apply the fix.. hmm, you know I'm not sure it did. Clock, weather and rss did I remember, but I don't remember for calc [21:31] balloons, I fixed calc and clock this morning with the devs [21:31] balloons, also, https://code.launchpad.net/~andrew-hayzen/music-app/toolbar-rewrite-1/+merge/188917 [21:31] balloons, I see you happroved [21:32] balloons, there's no reasoning for the removal of manifest.json in there [21:33] I top approved that yes, it had been sitting out there forever waiting on an sdk fix [21:33] balloons, so autolanding woes? [21:33] there is no reason for the removal [21:34] balloons, let me re add it [21:34] go for it, I'll approve and get it put back [21:34] andrew probably did that inadvertently during one of his rebases [21:40] balloons, seems so; I'm being slow at finding it though [21:43] balloons, the file only gets removed after the merge though; might be looking at it wrong [21:43] balloons, https://code.launchpad.net/~sergiusens/music-app/manifest/+merge/190250 [21:48] done [22:02] so when I'm adb-shelled into the phone I get exactly 80 characters and it's kind-of cramping my style, esp. b/c a newline begins on present line [22:02] I'm sure I just lack shell-settings-fu, can anyone suggest a remedy? [22:13] alesage, stty [22:13] you can manually resize it that way, not ideal, but [22:15] balloons, thanks that's a start