[08:31]  * tsdgeos does happy dance
[08:31] <tsdgeos> i've been able to reproduce the missing icons
[08:31] <tsdgeos> once
[08:31] <tsdgeos> let's see if i can reproduce it more
[08:31] <tsdgeos> but seems it should
[08:37] <Saviq> yay
[08:39] <tsdgeos> yeah reproduced
[08:39] <tsdgeos> now i need to compile qt with some debug symbols
[08:39] <tsdgeos> s/symbols/extra output
[08:42] <Mirv> tsdgeos: hey. I ran all AP:s once again during weekend against silo 018, no regressions. and I rebuilt it to fix the symbol problem it had. but I can't still start konsole etc.
[08:43] <Mirv> and he dropped
[08:44] <Mirv> and he's back
[08:44] <Mirv> [repeat] tsdgeos: hey. I ran all AP:s once again during weekend against silo 018, no regressions. and I rebuilt it to fix the symbol problem it had. but I can't still start konsole etc.
[08:45] <tsdgeos> :/
[08:48] <Mirv> tsdgeos: if for a moment thinking from our side only, could you give 018 a try on your arale or krillin? I'd especially like you to add Ubuntu One account and try installing applications. I'm not sure if it's a vivid problem in general but I sometimes have trouble with that so that the account adding just endlessly spins or alternatively clicking Install without account yet doesn't bring up the account adding dialog.
[08:49] <tsdgeos> Mirv: i found a way to reproduce https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1430828 going to try to get it fixed
[08:49] <tsdgeos> or actually
[08:49] <tsdgeos> i guess i should try to make it happen in vivid now that i made it happen on rtm
[08:49] <tsdgeos> since we're not releasing "old rtm" anymore, right?
[08:49] <tsdgeos> next rtm will be vivid afaiu?
[08:50] <Mirv> tsdgeos: correct, no more old rtm. I just started landing https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1441822 , I wonder if I should wait to have both?
[08:51] <tsdgeos> Mirv: both == the fix for https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1430828 ?
[08:51] <tsdgeos> or?
[08:51] <Mirv> tsdgeos: both, so include 1430828 fix also in the same landing so one round of testing
[08:53] <Mirv> tsdgeos: well, I'll start with 1441822 and if you find a fix before I finish testing I may restart testing the silo so that QA only needs one round of testing
[08:53] <Saviq> Mirv, let me know what testing you need, let's let tsdgeos focus on the image bit
[08:53] <Saviq> tsdgeos, FWIW, we did not backport your "let other requests go while http stuck" change to the image loading code did we?
[08:54] <Mirv> Saviq: ok. so 018 krillin or arale, generic smoke test with special emphasis on installing apps / adding ubuntu account (start with just going to app instllation without first adding an account, and see if account dialog correctly pops up and you can successfully install the app.. if there's problem, try to see if it's anything DBus related or not)
[08:55] <Mirv> Saviq: at least bug #1439263 is still open
[08:56] <Saviq> Mirv, yeah, there's a few levels of what we were able to do
[08:58] <Saviq> Mirv, btw, re: new landing process, so now even if we wanted, we can't land to vivid through citrain, 'cause all silos there have the PPA added as dependency, so there's chance of the PPA having an unwanted impact?
[09:01] <Mirv> Saviq: right, in theory that's the case. there are currently just two landings though in the overlay, qtbase (no effect on anything) and net-cpp (maybe effect on something)
[09:01] <Mirv> Saviq: but we can also switch the dependencies on request
[09:02] <Saviq> Mirv, right, I just wanted to raise awareness, same as when I wrote to Steve that we need those PPAs enabled as deps :)
[09:03] <Mirv> yes, it's a very good point and didn't occur to me (that you need to add those deps to all PPA:s like now has been done)
[09:03] <Saviq> *that PPA
[09:04] <Saviq> tsdgeos, network woes?
[09:04] <Saviq> tsdgeos, FWIW, we did not backport your "let other requests go while http stuck" change to the image loading code did we?
[09:04] <tsdgeos> Saviq: well i need to play with network dhcp to make the thing fail/work/fail/work
[09:04] <tsdgeos> so i'll be unstable today
[09:04] <Saviq> (and I'll help Mirv with the testing)
[09:04] <tsdgeos> Saviq: not afaik
[09:05] <Saviq> tsdgeos, right, that my other suspicion :)
[09:06] <Mirv> Saviq: adding that PPA (overylay) dep to all PPA:s (silos), that is
[09:06] <Saviq> yup
[09:07] <Saviq> tsdgeos, can you point to the right commit? we want to backport it regardless of the async provider bits, right/
[09:07] <tsdgeos> Saviq: so there's two commits
[09:07] <tsdgeos> one with tha async provider bits
[09:07] <tsdgeos> and another without it
[09:08] <Saviq> oh right, so we rather want the full monty
[09:09] <tsdgeos> so https://codereview.qt-project.org/#/c/109756/3//ALL,unified is just the fix for images not stalled by http
[09:09] <tsdgeos> the other one is https://codereview.qt-project.org/#/c/108540/
[09:09] <Saviq> tsdgeos, thanksies
[09:29] <pstolowski> tsdgeos, hey, i'm looking at solving https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1335761 in the shell-plugin, using gotoScope(scopeid) signal
[09:29] <pstolowski> tsdgeos, looks like it could work, but shell is complaining
[09:29] <pstolowski> tsdgeos, "qml: No match for scope with id: mediascanner-video"
[09:30] <pstolowski> tsdgeos, ah. i think this is because it's a temporary scope i'm currently in
[09:30] <tsdgeos> let me see
[09:31] <pstolowski> tsdgeos, so I'm in My Videos, which is not favorited. I open a preview there. In the preview i've a button that executes new search in My Videos.
[09:31] <tsdgeos> pstolowski: are you sending a gotoScope?
[09:31] <pstolowski> tsdgeos, well, in my modified branch only
[09:32] <tsdgeos> it's the only way that warning is triggered
[09:32] <tsdgeos> and gotoscope is not ok for non favorite scopes, no?
[09:32] <pstolowski> tsdgeos, yes, it's not going to work with temp scopes
[09:32] <pstolowski> tsdgeos, we need a new signal then?
[09:33] <pstolowski> tsdgeos, e.g. showResults()
[09:34] <tsdgeos> pstolowski: give me  a sec
[09:35] <pstolowski> tsdgeos, i don't like abusing any other signal for that
[09:37] <tsdgeos> pstolowski: so what we are doing here is "close previews"
[09:37] <tsdgeos> basically?
[09:40] <pstolowski> tsdgeos, yes
[09:41] <tsdgeos> pstolowski: same question i made the other day, can't we just hide previews if the scope search string changes?
[09:41] <tsdgeos> sorry don't remember the answer :D
[09:41] <pstolowski> tsdgeos, but this is a common piece of code inside the plugin where i don't know if i'm in the preview, therefore i'd prefer to just signal the need to show results rather than call it 'hidePreview'
[09:42] <pstolowski> tsdgeos, no, the query may be the same
[09:42] <pstolowski> tsdgeos, the scope may want to just refresh the results
[09:42] <tsdgeos> k
[09:43] <tsdgeos> then yes, we're going to need a new signal from the scope i guess
[09:44] <tsdgeos> there's a showDash signal
[09:44] <pstolowski> tsdgeos, there is, yes. would it work?
[09:44] <tsdgeos> pstolowski: actaully if you emit that one, it would work already i think
[09:44] <pstolowski> tsdgeos, isn't preview the dash as well
[09:45] <pstolowski> ok i can try
[09:45] <tsdgeos> pstolowski: it is, but we do
[09:45] <tsdgeos> ./Dash/GenericScopeView.qml:145:        onShowDash: subPageLoader.closeSubPage()
[09:45] <pstolowski> tsdgeos, nice
[09:45] <tsdgeos> and that will hide the preview
[09:45] <tsdgeos> i think :D
[09:45] <tsdgeos> since we have
[09:45] <tsdgeos>     function closePreview() {
[09:45] <tsdgeos>         subPageLoader.closeSubPage()
[09:45] <tsdgeos> :D
[09:46] <pstolowski> tsdgeos, ok, thanks, checking
[09:56] <Saviq> pstolowski, yeah, showDash should work for that (if it doesn't - our bug), showDash was always meant as "go to dash in response to a preview action"
[09:57] <pstolowski> Saviq, yes and no. i found a case where it doesn't do the right thing, but that's the problem of our navigation
[09:59] <Saviq> pstolowski, right, it probably doesn't do the right thing when you have a temp scope and you want to go back to that temp scope
[09:59] <pstolowski> Saviq, it gets me back to video aggregator view when called from the preview of an aggregated result. i'm now looking if it's a plugin issue
[10:00] <Saviq> pstolowski, we're likely closing the temp scope on showDash actually
[10:00] <pstolowski> Saviq, ah, this could be the case
[10:02] <pstolowski> Saviq, hmm no, it actually breaks if both Videos and MyVideos are favorited.. investigating the plugin
[10:09] <Mirv> Saviq: is it a known bug that when one clicks Install and it gives you Ubuntu One dialog, unity8-dash consumes 150% CPU? this is now without PPA since I just upgraded to latest image
[10:16] <Mirv> Saviq: bug #1446150 feel free to mark as duplicate
[10:18] <Mirv> Saviq: meanwhile, unping related to the account adding problems being 018 specific since I seem to have the same spinning animation on stock image too
[10:21] <pstolowski> Saviq, tsdgeos it works! I made a little mistake in the plugin, just fixed it and it works, I've tried all the combinations of temp/favorite with aggregator and a child
[10:22] <tsdgeos> :)
[10:24] <tsdgeos> pstolowski: so no change needed on our side?
[10:32] <pstolowski> tsdgeos, correct
[10:33] <tsdgeos> \o/ :D
[10:35] <tsdgeos> pstolowski: so i assigned https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1335761 to you
[10:35] <pstolowski> tsdgeos, yup, thanks
[10:36] <Saviq> Mirv, it feels bug #1441822 could be related
[10:38] <Saviq> Mirv, can you confirm CPU goes high without clicking "Install"?
[10:42] <tsdgeos> Saviq: soooo, i can reproduce the images bug in rtm but not in ubuntu-touch/devel-proposed/krillin.en-proposed
[10:42] <tsdgeos> Saviq: question is, should i try to spend some time in rtm trying to fix it? or just wait for something vivid-based to be released and see if others can still reproduce it?
[10:43] <Saviq> tsdgeos, can you comment on the bug with steps you used to reproduce
[10:43] <tsdgeos> yeah
[10:43] <Saviq> tsdgeos, and no, no point in fixing in RTM since we're not releasing that any more
[10:43] <tsdgeos> it's what comment #29 said basically
[10:43] <tsdgeos> disabled dhcp on my router
[10:44] <tsdgeos> but yeah i'll give a full comment info in there
[10:45] <Mirv> Saviq: please ask on the bug report so I can check later. I just started AP suites run and don't want to abort.
[10:45] <Saviq> Mirv, will do
[10:49] <Saviq> /food
[13:12] <dandrader> Saviq, so, what's next for shellRotation?
[13:12] <Saviq> dandrader, reviews
[13:14] <dandrader> mzanetti, ^^
[13:14] <mzanetti> dandrader, is the silo up to date?
[13:14] <Saviq> MacSlow, was there any AP failure you were looking at, still?
[13:15] <Saviq> mzanetti, not silo, PPA
[13:15] <Saviq> dandrader, mzanetti, u8 failed to build on Friday apparently
[13:16] <Saviq> dandrader, http://pastebin.ubuntu.com/10856056/
[13:18] <MacSlow> Saviq, ok... was battling with systemd all morning... but will take a look now
[13:18] <dandrader> Saviq, crap, bad merge
[13:18] <dandrader> will fix it
[13:19] <Saviq> MacSlow, I was asking, not telling ;)
[13:20] <MacSlow> Saviq, well... I've not mcuh I can do on the unity8-launcher feature-parity front atm (no feedback from Design yet) so working on the shellRotation-MP comments seems a logical task-switch atm
[13:21] <MacSlow> Saviq, we've (you, kgunn) to split the card still
[13:22] <Saviq> MacSlow, right, but we've a stable set of tests atm, it's just that review process need to continue?
[13:23] <MacSlow> Saviq, I'll do what I can wrt to the MP-comments/requests
[13:23] <kgunn> MacSlow: but the question is, is there a known aspect of the test that fails intermittently
[13:23] <kgunn> (unrelated to the crashing)
[13:24] <kgunn> ?
[13:24] <MacSlow> kgunn, no ... the test is fine... from all the MP-comments I see, they are all "this would be nicer to do in such a way"
[13:24] <kgunn> got it
[13:28] <MacSlow> kgunn, regarding the unity8-launcher feature-parity card... splitting up the card into the six remaining checklist-points I leave to you/saviq
[13:28] <kgunn> mk
[13:29] <kgunn> MacSlow: sounds like it's all effectively blocked ?
[13:29] <MacSlow> kgunn, yup
[13:29] <MacSlow> kgunn, maybe some manager-level poking will help :)
[13:29] <tsdgeos> mterry: you do https://code.launchpad.net/~josharenson/unity8/greeter_mode/+merge/256385 ?
[13:30] <kgunn> design is swamped...no amount of poking will help i think
[13:30] <mterry> tsdgeos, sure
[13:33] <tsdgeos> Saviq: so i'll mark the loading images bug as fixed for vivid as Pat suggests and hope that the one case i see fixed also fixes the cases victor was seeing
[13:33] <MacSlow> kgunn, expected somehow
[13:56] <Saviq> tsdgeos, yup, I really feel it all came down to network going awry
[14:02] <tsdgeos> Saviq: maybe
[14:35] <dandrader> josharenson, the new https://code.launchpad.net/~dandrader/unity8/homeKey/+merge/256658 is waiting for your review :)
[14:36] <josharenson> dandrader: ack, I'll do it today
[14:37] <josharenson> mterry:  While we're at it, can you review this today if you have time? https://code.launchpad.net/~josharenson/unity8/greeter_mode
[14:38] <mterry> josharenson, maybe?  I did claim the review but am not sure of my schedule today -- trying to get a handle on several bugs that have accrued  :)
[14:38] <josharenson> mterry: no problem, I know the features aren't critical and that you just got back to real life
[16:06] <Saviq> elopio, hey, I updated https://code.launchpad.net/~saviq/unity8/fix-flake8/+merge/256510
[16:06] <elopio> Saviq: thanks. I'll take a look soon.
[23:27] <mterry> What handles the normal power press for screen on/off?  I see PhysicalKeysMapper which handles long presses.  But I don't see the short-press code
[23:39] <greyback_> mterry: usc handles it