=== maclin_ is now known as maclin [05:24] jtaylor: yes, it is; but jenkins doesn't export it as an artifact unfortunately, we need to fix that in the job config [06:57] Good morning [08:01] good morning [08:26] Hi, I have an old bugreport which is not fixed correctly because I can't test the newer versions of ModemManager. Now trusty will be shipped with ModemManager 1.* and the issue effects trusty too. Now my questions, should I report this issue in a testing-tracker? When yes, which (link) ? Should I "reject" this bug to the ModemManager Team and when yes, how? [08:27] Here is the Link: https://bugs.launchpad.net/ubuntu/+source/modemmanager/+bug/1198283 I hope my english is good enough that everbody can understand me. [08:27] Ubuntu bug 1198283 in modemmanager (Ubuntu) "Huawei E3276 will not recognized correctly" [Undecided,Confirmed] === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === psivaa is now known as psivaa-lunch [12:45] elopio: (or someone else) do we have any AP experts who can help with https://bugs.launchpad.net/music-app/+bug/1300450 ? ahayzen (music app dev) has taken a look, but is stuck, and it's blocking us being able to get music updated in the store.. [12:45] Ubuntu bug 1300450 in Ubuntu Music App "Autopilot failing on mako #269 Object not found with name '*' and properties {'objectName': 'playshape'}." [Undecided,New] === _salem is now known as salem_ === fader is now known as fader_ === fader_ is now known as fader === psivaa-lunch is now known as psivaa [14:40] jibel: the rmtree issue you pinged me about yesterday (or this morning?), is that bug 1300474 ? [14:40] bug 1300474 in apport (Ubuntu) "tmp directory cleanup fails" [Undecided,New] https://launchpad.net/bugs/1300474 [14:45] pitti, yes, it is the same issue. It is because the exit handler is called twice [14:45] jibel: ack, thanks; fixing now [16:31] alesage: about https://code.launchpad.net/~allanlesage/unity8/autopilot-indicator-page-title-matches-widget/+merge/196991 [16:31] did you agree with my suggestions from yesterday, or would you like to merge it as it is now? [16:32] popey: oh, sorry, I missed your ping [16:32] I'll take a look in some minutes. [16:32] elopio: np, i asked robotfuel to take a look, dunno if he has had a chance yet [16:32] elopio if you don't mind I'd like to proceed now, and then I'll file a bug for the suggestion you made [16:33] popey: elopio I just started looking in to it. [16:33] robotfuel: ok, thanks. [16:34] alesage: ok. I'm taking a last look. I don't get this: [16:34] 181 + self.wait_select_single('Indicators', fullyOpened=True) [16:34] why on the close indicator method you are waiting for it to be fully opened? [16:34] elopio egads you're right [16:35] elopio, one minute to fix pls [16:35] alesage: oh, the thing is that you are never calling close indicator. [16:36] I would remove that method. Code that's not used nor tested can become a headache. [16:37] elopio you make a good point, removing [16:38] elopio, pushed that, thanks :/ [16:39] alesage: last detail while you are there, please turn get_indicator_widget and get_indicator_page into _get_indicator_widget and _get_indicator_page [16:39] so nobody outside of unity is tempted to use them. [16:40] I'll approve the branch. [16:41] elopio just verifying by running again thx [16:47] elopio, pushed that again thanks again [16:57] alesage: one last, sorry [16:57] 137 +from autopilot.input._common import get_center_point [16:57] yessir [16:57] you can do from autopilot import input. And then input.get_center_point() [16:57] no need to import _common. === bfiller is now known as bfiller_afk [16:59] elopio, done [17:00] alesage: ok, approved. Now you need to do this to get a review from the unity team [17:00] https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8 [17:01] I'll do the review checklist. [17:02] ok elopio thanks, I may need some coaching here :) [17:05] alesage: I'm looking at your other branch for the visual ordering. [17:05] with the new scopes branch they changed the behaviour and I didn't notice. [17:05] instead of returning autopilot elements, this method should return a list of the names of the apps. [17:07] elopio, o hmm, doesn't that revert an earlier change to returning the autopilot elements? === roadmr is now known as roadmr_afk [17:08] alesage: I'm not sure I'm getting your question. [17:09] elopio, I noticed that in order to get this code working I had to shift from using names to using autopilot objects, is it true that they've reversed this change? [17:09] elopio, I'm surprised because I liked the 'returning autopilot objects' decision [17:10] alesage: if we follow the page object pattern, we should never return a UI element that the user can't see. [17:10] when the user is looking at the UI, what he sees are icons and application names. That's the only thing we should return. [17:11] elopio that's fair, although the 'tile' would be a kind of object for the user--maybe the idea would be just to hide all of the autopilot guts from them [17:11] alesage: that's precisely the idea. [17:12] the name of the test is test_get_applications_should_return_list_with_names [17:12] and they change it to return a list of autopilot proxy objects :( [17:12] doorbell brb [17:14] elopio ok hmm, well I'll adapt my test--do you know if that reversion is released? [17:14] alesage: no, I'm making a branch to revert that. It's a small one. [17:14] give me a couple of minutes. [17:15] elopio ok no hurry, I'll wait for your ping [17:22] alesage: https://code.launchpad.net/~elopio/unity8/revert_get_applications/+merge/213693 [17:24] elopio, I wonder if this has effects elsewhere [17:24] alesage: just in the unity-click-scope tests. [17:24] without my branch, they are broken. [17:25] I need to put them on CI as soon as possible, so they can't get broken anymore. [17:26] elopio, ok weird, ok I'll test in a bit, must do an errand, then my branch will depend on this [17:26] alesage: yes, please add this as a prerequisite for yours. [17:27] elopio, ok more news a bit later [17:27] I'm going to leave in about two hours. So I might not have a chance to check it before tomorrow. === maclin__ is now known as maclin [17:54] somebody feels like working on a generic wiki page for r/zsync instructions for bug 1004558? [17:54] bug 1004558 in Ubuntu QA Website "In download links, link to instructions on using r/zsync" [Wishlist,Triaged] https://launchpad.net/bugs/1004558 === bfiller_afk is now known as bfiller === roadmr_afk is now known as roadmr [19:16] balloons, hello [19:22] knome: I can do that [19:24] though I might wait till september so it's two years old :) [20:00] elfy, hah [20:06] Letozaf_, howdy [20:08] balloons, I checked again the rssreader app error I had, and it's still there even if I branched rsssreader-app from trunk and put my tests back in, I commented the merge proposal [20:11] Letozaf_, you should push your code up :-) [20:18] balloons, If I push from the new branched reminders-app with my tests in I get a diverged branch error, so should I push on a new mp ? [20:18] Letozaf_, ohh really? you can supersede your mp if needed, so the comments stay in there [20:19] balloons, oh! how do you do that ? [20:19] https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/tests-refactoring/+merge/213335/+resubmit [20:19] Letozaf_, when you do it, change the source branch to your new branch :-) [20:19] balloons, ok let me try, thanks [20:23] balloons, lp:~carla-sella/ubuntu-rssreader-app/test-view-mode [20:26] Letozaf_, cool, yep looks good [20:27] balloons, :P not really the test fails [20:34] running now [20:38] Letozaf_, do you always get the tab index error? [20:39] balloons, yes when I run that test on my desktop, never tried on device [20:39] Letozaf_, I see it only on the one test [20:39] shorts_app.tests.test_rssreader.TestMainWindow.test_remove_feed_and_topic [20:39] so I would reckon we should have a look at the test [20:40] balloons, no I get it only on this test: test_view_mode the others run fine [20:41] balloons, I mean if I run the other tests they do not fail, only the test_view_mode does and with the tab index error [20:41] Letozaf_, right.. I can confirm [20:41] so let's look at it quickly [20:41] balloons, ok [20:41] fails on and_topic [20:41] so I would reckon we sh [20:42] shortsListModePage = self.main_view.get_tab_ListModePage() [20:42] balloons, yep [20:43] hmmm [20:44] get_current_tab() is an emulator bit [20:44] popey: I can't get the music-app to run in the emulator, I don't have an n4 to debug further. [20:44] balloons, yes [20:44] elopio, you about? [20:44] robotfuel: oh ☹ [20:45] balloons: if you have time, are you able to help robotfuel ? [20:45] trying to get this music AP problem fixed [20:45] the one I can't reproduce? [20:45] I have a few moments sure [20:45] well, its changed now [20:45] andrew posted a patch which fixes one thing but now it fails differently [20:46] ok, how can I help? [20:46] Letozaf_, I'm playing around with working around the issue in the emulator [20:46] balloons, ok [20:46] I think we should file a bug about it, and change the test if needed [20:46] balloons: https://bugs.launchpad.net/music-app/+bug/1300450 is the bug [20:46] Ubuntu bug 1300450 in Ubuntu Music App "Autopilot failing on mako #269 Object not found with name '*' and properties {'objectName': 'playshape'}." [Undecided,New] [20:47] otherwise your tweaks work, and we should land them when the bug you reported is fixed [20:47] as right now, jenkins fails everything [20:47] popey, oO! right! did you hear back from Andrew on that? [20:47] yeah, small patch, but it still fails, differently [20:47] he wasn't supposed to implement it that way [20:48] as in, there is a bug veebers has been working on to land support for properly selecting his custom object [20:48] how did this pass in jenkins I wonder? [20:52] Letozaf_, for your sanity. http://paste.ubuntu.com/7191549/ [20:52] Now, this is bad because it assumes the current Page is the first object returned. [20:53] But it's proof you can just use the parent to make the select work. Can you specify the tab any better? [20:54] balloons, haven't tried if you want I will try to see if I can find a way [20:54] Letozaf_, that will help you to land it if needed [20:54] anyways, everything else looks good. [20:55] Letozaf_, can you file a bug against the emulator for the issue though? Mention the branch and the test, so Leo can recreate the issue [20:55] Letozaf_, https://bugs.launchpad.net/ubuntu-ui-toolkit/+filebug [20:55] just mention it's the autopilot sdk helper [20:56] balloons, ok was just doing that (reporting the bug) [20:56] popey, so looking at the report quickly it seems like Victor is thinking it's a timing issue [20:56] that is inline with the error being reported [20:57] I'll pull trunk and confirm/deny [20:57] balloons: its possible [21:00] balloons, fyi bug 1301012 [21:00] bug 1301012 in Ubuntu UI Toolkit "Getting current tab using Ubuntu Ui Toolkit fails" [Undecided,New] https://launchpad.net/bugs/1301012 [21:01] Letozaf_, perfect. [21:01] balloons, I am going to bed now I will try to fix the tests with a workaround tomorrow [21:01] Letozaf_, I think you can hold on the workaround [21:02] Letozaf_, until trunk for rssreader builds a proper deb package it can't land :-) [21:02] balloons, ok so I will wait for the bug to be fixed [21:02] Letozaf_, and elopio should have a look [21:02] Letozaf_, but if he can't fix, then yes certainly workaround it and land it [21:02] balloons, ok [21:03] balloons, buona notte [21:03] Letozaf_, buona notte! [21:10] popey, it's interesting to note this test is very very similar to the play_pause_now_playing test [21:14] popey, I fixed it [21:15] ♥ [21:18] you can have a vacation now ☻ [21:24] popey, left an mp and comments.. cheers [21:25] ok === salem_ is now known as _salem === popey_ is now known as popey [22:00] hey balloons [22:02] hey knome [22:02] you might be interested in some bugs... [22:02] bug 1300576 [22:02] bug 1300576 in Ubuntu QA Website "Tooltip text appears on top of the bug link" [Medium,Fix committed] https://launchpad.net/bugs/1300576 [22:02] bug 1035503 [22:02] bug 1035503 in Ubuntu QA Website "Bug description yellow boxes exceed browser window dimensions." [Low,Fix committed] https://launchpad.net/bugs/1035503 [22:02] O:) === slickyma2ter is now known as slickymaster [22:22] knome, you fixed the yellow box bug?! [22:22] <3! [22:22] haha, yep... [22:22] two of those