== Meeting information == * #ubuntu-touch-meeting: Calendar app meeting, 20 Mar at 14:00 — 14:49 UTC * Full logs at [[http://ubottu.com/meetingology/logs/ubuntu-touch-meeting/2014/ubuntu-touch-meeting.2014-03-20-14.00.log.html]] == Meeting summary == ''LINK:'' https://blueprints.launchpad.net/ubuntu-phone-commons/+spec/coreapps-1404-calendar-dev ''LINK:'' https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/WeekViewFix/+merge/211544 ''LINK:'' https://code.launchpad.net/~mihirsoni-123/ubuntu-calendar-app/limitFunctionality/+merge/211536 ''LINK:'' https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/EventBubble-MinHeight/+merge/210947 ''LINK:'' https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1294681 ''LINK:'' https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/revert-212/+merge/211813 == Vote results == == Done items == * (none) == People present (lines said) == * pkunal-parmar (51) * popey (48) * balloons (42) * mihir (24) * oSoMoN (14) * michelr (9) * meetingology (3) == Full Log == 14:00 #startmeeting Calendar app meeting 14:00 Meeting started Thu Mar 20 14:00:18 2014 UTC. The chair is popey. Information about MeetBot at http://wiki.ubuntu.com/meetingology. 14:00 14:00 Available commands: action commands idea info link nick 14:00 Hi popey 14:01 Calendar App links:- 14:01 Bugs: https://bugs.launchpad.net/ubuntu-calendar-app/+bugs 14:01 Reviews: https://code.launchpad.net/ubuntu-calendar-app/+activereviews 14:01 Blueprint: https://blueprints.launchpad.net/ubuntu-phone-commons/+spec/coreapps-1404-calendar-dev 14:01 General Links:- 14:01 Milestones: https://launchpad.net/ubuntu-phone-coreapps/+milestones 14:01 Burndown: http://status.ubuntu.com/coreapps-14.04/ 14:01 Blockers: https://wiki.ubuntu.com/Touch/CoreApps/Blockers 14:01 How are you guys? 14:02 I am good, How are you ? 14:02 Fine. ☻ 14:03 i am fine :) 14:03 ok, a couple of things. 14:03 Next week we're planning on having another set of hack days 14:04 Wednesday will be Clock & Calendar. 14:04 Okay 14:04 I'll be blogging about it tomorrow. 14:04 The main goal is to get some of the final bits and pieces fixed. 14:04 hi michelr 14:05 hi ! 14:05 ok 14:05 Hi pkunal-parmar 14:05 i was just telling the guys we're going to have a "hack day" on wednesday for calendar and clock 14:05 hi michelr 14:05 blog post coming tomorrow 14:05 So keep an eye out for bug reports 14:05 hi michelr 14:05 I'll also be asking people to test the app over the weekend on devices and on desktop 14:05 hi mihir 14:06 we are going to target some bite-sized bugs to a milestone called "hackdays-1403" 14:06 ok 14:06 so we can track the bugs people are working on next week, and target some for new developers 14:06 of course we also want to encourage people to review existing merge proposals 14:07 popey, sounds great :) 14:07 As always we are trying to recruit new people too! 14:07 it'll on wednesday 26th ? 14:08 But we're also wanting to get some testing of the apps done. 14:08 yes. 14:08 Tuesday is Music and reminders, Thursday is weather & calculator 14:08 ok 14:10 ok, lets look at the blueprint.. 14:10 https://blueprints.launchpad.net/ubuntu-phone-commons/+spec/coreapps-1404-calendar-dev 14:10 so last week was week 11, any updates to those items guys? 14:10 i chased up the agenda view but no reply yet, will ping again 14:11 I started working on calendar management 14:11 popey, I am done with Limit functionality, i will start working on change date on event 14:12 I started initially with Key Navigation in NewEvent 14:12 ok, please update statuses, to INPROGRESS or DONE as appropriate pls ☻ 14:12 but then dropped it and started with calendar management 14:12 popey, we are facing many issues with Jenkins we have to resolve them first 14:13 oh? 14:13 I don't know what is happening , it is failing each time on each MR 14:13 MR : Merge Request ? 14:14 yes 14:14 michelr, there are few new bugs registered, where design input is needed 14:14 can you drop some comments there 14:14 ok 14:15 balloons: ping? 14:15 i tried to ping him , but couldn't catch him 14:15 https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/WeekViewFix/+merge/211544 14:15 https://code.launchpad.net/~mihirsoni-123/ubuntu-calendar-app/limitFunctionality/+merge/211536 14:15 https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/EventBubble-MinHeight/+merge/210947 14:16 as far as i know, balloons was working on fixing those failure 14:16 pkunal-parmar: where/which are new bugs ? 14:16 michelr, https://bugs.launchpad.net/ubuntu-calendar-app/+bugs?search=Search&field.status=New 14:17 qt 5.2 is also seems reason, and he found out that if we revert cal rev 242 test case are passing 14:17 but dont know latest status 14:18 I'll ping balloons when he's around 14:18 pkunal-parmar, mihir let's chat 14:18 popey, balloons just replied me 14:18 oh, hello 14:18 ok 14:19 michelr, bug like https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1294672 14:19 https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1294681 14:19 it would be good, if you can provide your suggestion 14:19 same for others as well 14:20 balloons, so now we decided to revert changes 14:20 so it seems folks have narrowed down some of the issues 14:20 so you would like to land https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/revert-212/+merge/211813? 14:20 I was not able to follow up, does that help ? 14:21 so if we revert, it pass all test cases? 14:21 or we are still facing crash ? 14:21 pkunal-parmar, yes, if we revert trunk is unblocked and everything passes 14:21 I have no issue, if reverting helps 14:22 I made the changes twice.. https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/standalone-ap-for-1293489/+merge/211854 14:22 that merge is only the test fixes and still fails. But if we add in the revert for r212, everything passes again 14:22 yes, I seen there are many MR with revering code 14:22 which one is final > 14:22 https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/revert-212/+merge/211813 14:22 sorry : urgent phone call : afk for few minutes 14:22 i imagine it got a little confusing, but :-) https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/revert-212/+merge/211813 14:23 ok 14:24 ok, then do I need to do anything ? 14:24 if you think it's the best way forward, let's merge it 14:25 I can propose those changes again, but olivier had put lot of effort in reviewing those changes 14:26 lets ask him as well 14:26 but I am fine for reverting change 14:27 can we ping him? 14:27 it seems they did work to discover it was indeed something in qtorganizer 14:28 they assigned it as such.. I don't know exactly :-) 14:28 I do know the changes cause the bug to occur 14:29 yes, according to last backtrace, issue seems to be originating from QtOrganizer 14:29 but it also seems related to Qt Js engine 14:29 its looks confusing 14:30 But when I proposed those changes, test case got passed fine 14:30 right.. the only difference between then and now is perhaps qt5.2? or a new qtorganizer release? 14:30 I'm not sure.. again, it's unclear why trunk got broken 14:31 I'd like to dig in and find out after we solve the issue 14:31 yes, I guess, we can resolve this issue by reverting 14:32 and debug it later, I can propose MR again with same changes 14:32 yes I think that's the best way at this point. Ok, pkunal-parmar would you approve the mp? 14:32 lets confirm from Olivier once, as he might need to review it again 14:33 i pinged olivier. nothing yet 14:33 I think that MR is already approved, do you want me to TopApprove it ? 14:33 mihir, so if we revert the code your merges should pass tests and things should work again 14:34 balloons, most probably 14:34 mihir, meaning, trunk won't be broken and you'll be able to merge again 14:34 i'm back. looking bugs 14:34 pkunal-parmar, yes I'd like top approval 14:34 ok 14:34 balloons, ideally it shouldn't 14:37 mihir, what do you mean it shouldn;t? 14:37 balloons, it should pass my MR , and other pending MRs as well 14:37 i just have one MR but rest of them are from pkunal-parmar 14:37 mihir, its not jenkins issue 14:38 and people are working hard to find out actual cause of issue, as of now we are not sure of issue, 14:38 but we found out one alternative 14:38 olivier is coming now 14:38 ok 14:38 hey folks 14:38 hey :-) 14:39 hey oSoMoN 14:39 so the question is to merge https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/revert-212/+merge/211813 or not 14:39 I’d say the answer depends on the urgency 14:39 we can merge it, which fixes the tests and reverts r212. We'd then need to re-propose it 14:39 I’d rather not revert so quickly, if we can affort a bit of time to really investigate and understand the issue 14:39 the trouble is nothing can merge, as mihir is finding out, as jenkins fails every attempted branch 14:39 I’ve asked Renato to look into the bug, seems to be in QtOrganizer 14:40 the other issue is the landing team wants to land a stable image, and poor xnox needs calendar updated as well 14:40 I’m concerned that if we revert we’ll get the app seemingly working again, but in fact we’ll still have that bug lurking and ready to bite us when we least expect it 14:40 oSoMoN, how can we get backtrace for QML app ? 14:40 oSoMoN, yes I don't want to hide the bug 14:41 pkunal-parmar, just run qmlscene in gdb 14:41 pkunal-parmar, a complete backtrace is attached to the bug report already 14:41 ok 14:41 we could revert and then immediately merge it back after we land what we need 14:41 I used qtcreator and ptrace/strace 14:41 balloons, if the landing team is desperate for a stable image without further delays, then I say let’s revert 14:42 but let’s also make it the top priority to understand and fix the issue 14:42 actually, I am getting some crash, I think issue is with EDS plugin, but bt shows issue with qtpim 14:42 ok, great.. I want to know how it broke, as it worked when you landed 14:42 so I was wondering how to debug EDS plugin 14:42 so that's the next step after we fix it 14:42 balloons, it broke when we switched to Qt 5.2 14:42 balloons, IIRC when we landed rev 212 we were still on Qt 5.1 14:43 oSoMoN, I agree more or less, but I don't want to point at qt 5.2 14:43 timo didn't like it last time I did :-) 14:43 balloons, I’m not saying it’s a bug/regression in Qt 14:43 balloons, it’s more likely a bug in calendar/qtorganizer code that got uncovered by the transition to Qt 5.2 14:44 ok so I top approved the merge it will land. mihir and pkunal-parmar I'm going to push the new version to the store as well 14:44 ok 14:44 thanks balloons 14:44 i'd advise landing all your branches.. Can we make sure we get a MP that adds it back so we can keep track of the bug itself 14:44 ok 14:44 +1 14:45 I will propose MR soon, for reverted changes 14:45 I just have one change , i will re-propose again 14:45 I just talked to Renato, he said he would take a look at the issue later today 14:45 great. I hope this helps you guys as well since jenkins will stop rejecting all your branches :-) 14:46 popey, is there a new image coming we could meet with calendar? 14:46 eh? 14:46 balloons, thanks a lot :) 14:46 i dont understand the question 14:47 sorry, it's a landing team question 14:47 i'll just ping them 14:47 k 14:47 Ok. Anything else we need to discuss? 14:47 thanks everyone 14:47 popey, nothing from my side we are working on bugs :) 14:47 Great! 14:47 thanks balloons for your help !! 14:47 Music to my ears! 14:48 nothing from my side as well 14:49 Super! Thanks everyone! 14:49 have a great weekend! 14:49 #endmeeting Generated by MeetBot 0.1.5 (http://wiki.ubuntu.com/meetingology)