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