Saviqmzanetti, ping07:55
mzanettimorning Saviq07:56
Saviqmzanetti, hey, any idea about https://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy/850/console07:56
Saviqmzanetti, ERROR:pbuilderjenkins:H05set_package_version not found in hooks07:56
mzanettiSaviq: not seen yet... let me check existing hooks07:57
Saviqmzanetti, btw dh_install: usr/share/unity8/plugins/mocks/Unity/Launcher/libMockLauncherPlugin.so exists in debian/tmp but is not installed to anywhere07:57
Saviqdh_install: usr/share/unity8/plugins/mocks/Unity/Launcher/qmldir exists in debian/tmp but is not installed to anywhere07:57
mzanettiok. I'll fix that07:57
Saviqmzanetti, I think it might've changed with fginther adding our local repo yesterday07:58
Saviqmzanetti, I mean the hook07:58
mzanettiSaviq: yeah... seems so07:58
mzanettiSaviq: oh crap... thats a mess08:03
Saviqmzanetti, :/08:03
mzanettiSaviq: the jobs pull hooks from 3 different sources08:04
* mzanetti is not going to clean this up now...08:04
mzanettiI'll try to just remove the set_version hook from the mediumtests job08:05
mzanettinot possible :/08:06
Saviqmzanetti, who do we have in our timezone that could help us fix?08:09
mzanettiSaviq: yeah... thats actually a problem... there is no QE in europe timezone any more08:10
Saviqright :/08:10
mzanettiwell, vrruiz is, but he's mostly busy with webapp stuff08:10
Saviqyeah, we're hiring for that, probably, then08:10
mzanettiSaviq: anyways, that missing hook comes from a branch owned by fginther08:11
mzanettiSaviq: and that branch is pulled in the builder jobs but not in the others08:11
mzanettiSaviq: I'm trying to pull the hook from there and add it to where it belongs. the real hooks package08:11
Saviqmzanetti, thanks08:11
Saviqmzanetti, why can't you remove it btw?08:11
mzanettibecause they are inherited from upstream jobs08:12
Saviqmzanetti, which are configured autmagically08:12
mzanettiand if I remove it upstream it won't bump the version number any more08:12
mzanettiwhich would cause the local repository to break I'd assume08:12
Saviqmzanetti, yeah, it's probably for that (it needs to mangle the changelog to actually do a release)08:13
Saviqmzanetti, btw, did qmluitests get the local repo added, too?08:13
* mzanetti checks, but I would assume no08:13
mzanettiSaviq: qmltests does not use the inherited hook list08:15
mzanettiSaviq: because that's a specific job for unity8 so it has the required unity8 hooks hardcoded08:15
Saviqmzanetti, yup, we should add the one that adds the local repo, right?08:16
mzanettiSaviq: guess so, yes... let me first fix the mediumtests one.08:16
mzanettibtw. why is our code coverage dropping like crazy?08:17
mzanettiSaviq: ^08:17
Saviqmzanetti, are you looking at -ci or at -autolanding?08:18
mzanettiboth... right now autolanding08:18
Saviqmzanetti, anyways, indicators-client isn't tested well yet08:18
mzanettioh right... that was my problem child already when I was in the apps team08:19
Saviqmzanetti, yeah, we're taking it over and we'll be improving that08:19
Saviqmzanetti, at least the QML part08:19
Cimimzanetti, how about this? http://paste.ubuntu.com/5857753/08:20
Saviqmzanetti, also, somehow we started gathering coverage for tests again, need to fix that to get some real-ish numbers08:22
mzanettiCimi: what's this?08:24
Cimimzanetti, instead of clearing the model08:24
Cimimzanetti, I push/append08:24
mzanettiCimi: ok. give me a minute for my current task, then I'll re-review08:24
Cimimzanetti, it's a WIP08:25
Cimimzanetti, like, this works but I believe there might be flaws08:25
Cimimzanetti, like if you clear also the buffer and remain with one or two items, will it scroll?08:25
Cimimaybe since onCurrentIndex should populate the sides...08:26
Cimimight have to test08:26
mzanettiCimi: ah ok... yeah, I can give it a closer look in a few. Not sure either how the thing behaves in that situations08:26
Cimimzanetti, it's the corner cases indeed08:26
Cimimzanetti, when something changes currentDate or maximum/minimum08:26
mzanettiSaviq: https://code.launchpad.net/~mzanetti/pbuilderjenkins/add-missing-hooks/+merge/17366808:27
=== 36DAAZP3C is now known as tvoss
mzanettiSaviq: I'm not entirely sure if fginther will be happy about this... but the current state doesn't work either... So I'd say we go for it and if fginther doesn't like it, I'll help him finding a proper solution08:28
Saviqmzanetti, yeah, we need to unblock08:28
mzanetti(I think this is the proper solution though)08:28
Saviqmzanetti, approved08:29
mzanettiSaviq: ok... once this landed, I need to update the pbuilderjenkins package in the mediumtests vm's. Then it should work again08:29
Saviqmzanetti, k, thanks08:30
mzanettiSaviq: as usual, I forgot to release. Could you approve this one too: https://code.launchpad.net/~mzanetti/pbuilderjenkins/relase-new-hooks/+merge/17367008:45
Saviqmzanetti, UNRELEASED ok08:46
mzanettihuh? didn't I release08:46
mzanettione sec08:46
mzanettiSaviq: now08:47
Saviqmzanetti, saucy ok?08:47
Saviqmzanetti, the previous ones are raring08:47
mzanettiSaviq: yeah... there hasn't been a release in a while...08:47
mzanettiCimi: this seems broken:08:51
mzanetti+        while (i < calendarModel.count && calendarModel.get(i).monthStart < minimumDate) {08:51
mzanetti+            calendarModel.remove(0)08:51
mzanetti+            i++08:51
mzanetti+        }08:51
mzanettiCimi: you do a get(i++) and a remove(0).08:51
mzanettiCimi: seems like you're skipping every second one08:51
Cimimzanetti, indeed08:52
mzanettitsdgeos: I can reproduce it now09:07
tsdgeosmzanetti: shoot09:07
mzanettitsdgeos: go to the music lens09:07
mzanettitsdgeos: drag it a little bit up, so that most of the header is hidden09:07
mzanettitsdgeos: but not all of it. the thick sepearator should still be visible09:08
tsdgeosmzanetti: on the phone, right?09:08
mzanettitsdgeos: yes09:08
mzanettitsdgeos: then drag it to the right (moving content to the right, which would get you to the lens on the left, if there would be one)09:08
mzanettitsdgeos: the header will come back09:08
mzanettitsdgeos: release, and drag again09:08
mzanettitsdgeos: the header will come down a bit more09:09
mzanettitsdgeos: and with every drag it goes further down. once it reaches the bottom of the screen the whole lens is messed up and ever interaction breaks it more09:09
tsdgeosblame qreal :D09:09
* mzanetti blames qreal09:09
tsdgeosi'll have a look09:10
tsdgeosi had this before09:10
tsdgeosthought it was fixed09:10
mzanettitsdgeos: its an image from yesterday morning09:10
Saviqtsdgeos, yeah, my header just went far down, it's just GONE! ;)09:14
Saviqtsdgeos, and the content went up, it's GONE, too :D09:15
mzanettiSaviq: yep. thats the one :D09:16
mzanettiSaviq: reproducable with the left/rightmost lens09:16
Saviqmzanetti, made it with Home, too09:17
mzanettiSaviq: I've seen it once there. but way harder to reproduce there09:17
mzanettiSaviq: also, slide the header a little out the screen and tap on search. sometimes it'll come in without moving/clipping the listview09:17
mzanettiit == the search field09:18
tsdgeosyeah can repro the thing09:18
tsdgeoswill work on it09:18
mzanettitsdgeos: well, if you can reproduce it, isn't that half of the win?09:19
tsdgeosdoes pphablet-flash give me a flipped image already?09:19
Saviqmzanetti, yeah, it's the same code - showHeader()09:19
tsdgeosmzanetti: :D09:19
Saviqtsdgeos, yes09:19
mzanettitsdgeos: imagine you couldn't reproduce it and it happens randomly... that would suck09:19
tsdgeosSaviq: do we need any adjustment on run_on_Device or just works?09:19
mhr3_Saviq, the osk doesn't seem to work with latest image, known?09:19
tsdgeosmzanetti: well one could say it happens randomly since on the PC it works :D09:19
mhr3_Saviq, well, in the scope views09:20
Saviqmhr3_, yeah, OSK is broken09:20
mzanettitsdgeos: run_on_device is already adusted for flipped images09:21
mhr3_Saviq, any workaround?09:21
Saviqmhr3_, 'fraid not09:21
tsdgeosmzanetti: greatz09:22
* tsdgeos goes to pay some taxes while the newer image is being downloaded09:22
Saviqmhr3_, bug #119915009:22
ubot5bug 1199150 in touch-preview-images "search lens disappeared when hitting any key on OSK" [High,Confirmed] https://launchpad.net/bugs/119915009:22
=== larsu is now known as Guest60692
nic-doffaySaviq, for the container of the expandable, should the expanding call set a property on the container which is a child? This is specifically for the animations and all.09:32
nic-doffayexpanding signal sorry09:32
nic-doffayAm I making sense?09:32
Saviqnic-doffay, you lost me at "container which is a child" :D09:33
nic-doffayLet me pastebin a quick example.09:33
nic-doffaySaviq, here's an example minus everything else of what I meant: http://pastebin.ubuntu.com/5857906/09:34
Saviqnic-doffay, sure09:34
Saviqnic-doffay, only you don't need (Item expandable)09:34
Saviqnic-doffay, you can use "sender"09:35
nic-doffaySaviq, cool.09:35
SaviqI _think_?09:35
nic-doffayI also obviously have to set the Container in the expandable.09:35
Cimimzanetti, with get(0) should work http://paste.ubuntu.com/5857909/09:35
Saviqnic-doffay, yeah, you need to pass it, I thought sender is available, but apparently it isn't09:36
nic-doffaySaviq, so basically the OptionSelector now works whether there's a scrollable container as a parent or not.09:36
nic-doffaySince this Container is another optional component.09:36
Saviqnic-doffay, yeah, and ideally it still should work without the Container09:37
nic-doffaySaviq, that's what it does atm.09:37
nic-doffaySaviq, however what class should the expandable be passed into that signal as?09:38
nic-doffayItem is a bit vague.09:38
Saviqnic-doffay, ListItem.Expandable, probably, when you introduce it :)09:38
Saviqnic-doffay, don't optimize prematurely09:39
nic-doffaySaviq, what base feature would ListItem.Expandable have that the OptionSelector doesn't though?09:39
nic-doffayI'm assuming you mean for the ListItem.Expandable to be a base class for all of these?09:39
Saviqnic-doffay, it can be inherited by other components09:39
Saviqnic-doffay, hmm09:40
nic-doffaySaviq, just the expanding logic?09:40
Saviqnic-doffay, it should have `real collapsedHeight`, `real expandedHeight`, `bool expanded` at least09:41
Saviqnic-doffay, just think which would be common between the OptionSelector and the expanding Message items you did before09:42
Saviqnic-doffay, it's not like I can tell you from the top of my head what would that be :)09:43
Saviqpaulliu, ping09:49
mzanettiCimi: haven't tested it, but looks better, yes09:54
=== Guest60692 is now known as larsu
nic-doffaySaviq, what's the best way to get more output from a qmlscene?10:07
nic-doffaySomething (nothing) unexplained is happening.10:07
Saviqnic-doffay, onPropertyChanged: console.log("something") usually10:07
Saviqnic-doffay, you can also run the debugger in QtCreator10:07
nic-doffaySaviq, I mean the qmlscene isn't even coming up for weird reasons.10:09
nic-doffayNeed to find out why.10:09
Saviqnic-doffay, does your top-level object have dimensions?10:09
nic-doffaySaviq, yeah.10:09
nic-doffaySaviq, basically what I did was change that Item in the pastebin  I sent you to an OptionSelector, now nothing runs.10:10
nic-doffayObviously an item won't have an expanded property.10:10
Saviqnic-doffay, sorry, quite busy now, we've failed miserably at a review yesterday10:11
nic-doffaySaviq, how so?10:11
Saviqnic-doffay, not you, /me and mzanetti10:11
Saviqbut mostly me10:12
mzanettithe launcher-backend ones?10:12
=== dandrader is now known as dandrader|afk
Saviqmzanetti, no, the dee filtermodel10:18
Saviqmzanetti, it broke _everything_ in the dash10:18
Saviqon the device, that is10:19
mzanettiSaviq: hmm... weird. I ran it on the device tho... seemed to work10:19
Saviqmzanetti, it "works", but there's no data in Home other than apps, and no Installed in Apps10:25
Saviqmzanetti, not to mention icons and titles and such are broken everywhere10:25
nic-doffaymzanetti, have a moment to check something out?10:26
mzanettinic-doffay: hit me10:26
nic-doffayWeird problem where I'm passing something to a signal but as soon as I change the class from Item to the class nothing works.10:27
nic-doffaymzanetti, lp:~nicolas-doffay/+junk/option-selector10:28
nic-doffaymzanetti, if you take a look at ScrollableExpansionContainer.qml in modules/Ubuntu/Components/ListItems10:29
nic-doffayYou'll see an OptionSelector is passed from a signal.10:29
nic-doffayWhen this is an Item everything works properly.10:30
nic-doffayHowever obviously the class needs to be passed in to properly access the variable.10:30
nic-doffayRunning test.qml in modules/Ubuntu/Components/ListItems you'll see what I mean.10:30
nic-doffayNo output, nothing.10:30
mzanettinic-doffay: there is no ScrollableExpansionContainer10:33
nic-doffaymzanetti, whoops one sec.10:34
nic-doffaymzanetti, pushed again.10:38
paulliuSaviq: hi10:40
Saviqpaulliu, hey, thanks for spotting the video breakage... there's actually more of that elsewhere... am fixing now10:40
Saviqpaulliu, it was an epic review fail on my part10:40
paulliuSaviq: ok.. I just fix the video lens to test my branch. :)10:42
mzanettinic-doffay: this looks really scary10:42
mzanettinic-doffay: what do you want to achieve with this?10:43
mzanettinic-doffay: and why is this a signal?10:43
mzanettito me this doesn't make any sense at all10:43
nic-doffaymzanetti, this container will handle scrolling for expandable SDK components.10:49
nic-doffayThe signal is sent from the expandable component.10:49
Saviqpaulliu, can you please include this diff in your MP http://pastebin.ubuntu.com/5858043/10:49
mzanettinic-doffay: yeah... signals are supposed to be used when a component wants to emit something to the outside. not getting something from the outside10:49
mzanettinic-doffay: also, why don't you just use a flickable then where you embed the other component that needs to be scrolled?10:50
mzanettinic-doffay: do you have a spec how things should look like in the end? I'm having a hard time following what you want to do with this10:51
paulliuSaviq: ok10:54
=== hikiko is now known as hikiko|lunch
nic-doffaymzanetti, there aren't any specs I'm aware of.10:58
nic-doffaymzanetti, let me forward you something though...10:58
nic-doffaymzanetti, you have mail.10:58
tsdgeosnow i can repro the header problem on the pc too10:59
tsdgeosbtw do you guys get the images of "Music" in either pc or desktop?11:04
tsdgeosi'm just getting empty ubuntu shapes11:04
tsdgeosSaviq: mzanetti: Cimi: greyback: MacSlow: dednick: all :D↑↑↑11:04
Saviqtsdgeos, yes, we broke it yesterday11:04
Saviqtsdgeos, fixing now11:04
MacSlowtsdgeos, ?11:04
MacSlowtsdgeos, not seen that issue yet11:04
=== mardy_ is now known as mardy
Saviqmzanetti, Cimi, greyback, MacSlow, dednick, all: unping11:05
MacSlowSaviq, :)11:05
* MacSlow notes that term in his memory11:05
mzanettinic-doffay: still don't get it... what is the use case for this? if I have an expanding component, why wouldn't it be expanded to what it needs to be?11:09
nic-doffaymzanetti, the idea is that this container can be used for any expanding component instead of coding it separately for each.11:09
mzanettinic-doffay: ah. so the expanding component itself would always be expanded, but just clipped by the container?11:10
nic-doffaymzanetti, no.11:11
nic-doffaymzanetti, the expanding item would call the container which would in turn set the expansion on the expanding item and handle the animation and movement11:12
nic-doffaydoes that make any sense?11:12
mzanettinic-doffay: not for me... but according to this mail it does make sense for Saviq. So I must be missing something here.11:13
Saviqmzanetti, lol11:13
mzanettiwhat should I say11:14
mzanettithats how it is :D11:14
nic-doffaymzanetti, haha11:14
Saviqnic-doffay, the height animation could happen in the Expandable base component itself11:14
Saviqmzanetti, do you know what the whole deal is about?11:14
mzanettiSaviq: not really, no11:15
Saviqmzanetti, an item in a list11:15
Saviqmzanetti, the item is expandable, i.e. it can change its height11:15
mzanettiyeah. so far I'm good11:15
Saviqmzanetti, but when it expands, it should be limited to the list's height11:15
Saviqmzanetti, and it should be scrolled into view11:15
mzanettiwhat's the "list" ?11:15
=== MacSlow is now known as MacSlow|lunch
Saviqmzanetti, anything, a ListView, say11:16
mzanettioh... I see...11:16
Saviqmzanetti, and an item in that ListView needs to be able to expand11:16
mzanettithe scrolling thing is interesting.11:16
Saviqmzanetti, and when it does, it needs to be constrained to the ListView's height11:16
Saviqmzanetti, and then scroll it into view11:16
Saviqmzanetti, yup11:16
Saviqmzanetti, when the item is smaller than the ListView11:17
Saviqmzanetti, it should be minimal travel11:17
Saviqto show that whole item11:17
mzanettiok... I start to understand...11:17
mzanettithanks Saviq11:17
Saviqmzanetti, so the ListView needs to know the item's expanded and collapsed heights11:17
Saviqmzanetti, and change them accordingly11:17
Saviqmzanetti, I might've overcomplicated things in the email, too :)11:18
mzanettiwhy would the listview need to know both sizes?11:19
mzanettiah no... the listview is the one to scroll11:19
mzanettinic-doffay: ok... so the naming ScrollableExpansionContainer is the first misleading thing. I thought the content in there needs to scroll, not the listview around it11:21
nic-doffaymzanetti, you're right the content within does.11:22
nic-doffaymzanetti, maybe ScrollableContainer is more reasonable?11:22
Saviqmzanetti, how are we re: jenkins?11:22
* mzanetti checks (should work by now)11:22
mzanettiSaviq: I ran a mediumtests job manually and it worked11:23
Saviqmzanetti, k thanks11:23
=== hikiko|lunch is now known as hikiko|
mzanettiSaviq: but wouldn't we need to have our own ListView implementation to support this kind of stuff?11:24
Saviqmzanetti, no, why?11:26
Saviqmzanetti, we just need to have a channel of communication between the Flickable / Scrollable and the Expandables11:27
mzanettibut where is the code that scrolls the flickable?11:27
mzanettiSaviq: the expandable itself?11:28
Saviqmzanetti, no, in the Flickable11:28
Saviqmzanetti, it knows which Expandable it needs to scroll to11:29
Saviqmzanetti, because the Expandable will signal Scrollable.RequestExpansion(self) (pseudo code)11:29
nic-doffaymzanetti, that's what it currently does.11:30
=== dandrader|afk is now known as dandrader
mzanettinic-doffay: ah... so we're not talking about a flickable that uses those components... but another flickable inside the list11:30
Saviqmzanetti, then, the Scrollable will map to its own coordinates and scroll accordingly, animating the height in parallel (that can happen either in Scrollable directly or the Expandable could have  a Behavior, but that could be wasteful when you only ever animate one Expandable at a time)11:31
Saviqmzanetti, hum?11:31
mzanettiwhere would you place such a component? do you have an example?11:32
mzanettiI really dont see the usefulness of such a thing (except maybe the reviews in scopes)11:32
nic-doffaymzanetti, this is the thing.11:32
nic-doffayEveryone is debating this.11:33
nic-doffayDesign doesn't even have a proper mock up yet.11:33
mzanettinic-doffay: anyways... ignoring that... let me write some pseudocode how I would try to approach the problem11:35
mzanetti(besides asking design for clear guidance)11:36
nic-doffaymzanetti, done that bit already.11:36
nic-doffayAny idea about that original issue out of interest?11:36
Saviqmzanetti, mhr3_, https://code.launchpad.net/~saviq/unity8/fix-mock-scopes/+merge/17369811:44
mzanettinic-doffay: http://paste.ubuntu.com/5858161/11:44
mzanettinic-doffay: hope that makes somehow sense11:44
mzanettiSaviq: on it11:44
Saviqmzanetti, re ↑↑↑ http://design.ubuntu.com/apps/building-blocks/option-selector11:44
mzanettiSaviq: for that I don't see why we would need such a construct11:45
mzanettiSaviq: instead of just placing a normal expanding component into a ListView11:45
mzanettiSaviq: and perhaps making the ListView "interactive: contentHeight > height"11:46
Saviqmzanetti, it's not always a ListView, mind you, it can be as simple as a Column11:46
Saviqmzanetti, a Column in a Flickable, that is11:46
Saviqmzanetti, it's actually going to be the more common use, I'd say11:46
mzanettiSaviq: s/ListView/Flickable/11:46
mhr3_Saviq, why is the Qt.binding necessary?11:47
Saviqmhr3_, otherwise it only sets the value once11:47
Saviqmhr3_, whereas with Qt.binding it will update when changed11:47
Saviqmhr3_, so when you send dataChanged(i, i, RoleResults), it will update automagically11:48
Saviqmhr3_, we should be able to test by killing the backends, right?11:48
Saviqmhr3_, as the Scope object itself won't change, nor the Categories will11:48
nic-doffaymzanetti, yeah that makes sense. Cleaner imo11:49
mhr3_ok, looks fine to me, but perhaps it'd be better to add also the s/column_/.../ changes?11:49
mhr3_Saviq, yep11:49
Saviqmhr3_, prereq11:49
mhr3_Saviq, i'd suggest merging it in, one won't work without the other, will it?11:51
Saviqmhr3_, works fine, it fixes two different issues11:52
Saviqmhr3_, with the prerequisite apps and videos will get fixed11:52
Saviqmhr3_, with my MP only the "peeking" into other scopes is being fixed11:52
mhr3_ah, ok, i thought that the raw model is still used by some of the renderers11:54
Saviqmhr3_, we need to re-search after the scope is reconnected after it's killed, right?11:55
mhr3_Saviq, yea11:55
Saviqmhr3_, 'cause now my model is updated, but the results are not11:55
Saviqmhr3_, k, separate MP then11:55
=== greyback is now known as greyback|food
=== alan_g is now known as alan_g|lunch
=== dandrader is now known as dandrader|lunch
Saviqmhr3_, "m_unityScope->Search(m_searchQuery.toStdString());" is missing the callback, right?12:06
mhr3_Saviq, i'm just looking at that, and that should make sure that a re-search happens12:08
Saviqmhr3_, yeah, but there's no callback?12:08
mhr3_not sure why it doesn't12:08
mhr3_Saviq, right, but no biggie12:08
Saviqmhr3_, should work anyway?12:09
mhr3_there'll be one missed searchFinished signal, don't think shell uses it anyway yet12:09
mzanettiSaviq: oh noes! the hooks are there now, but one of them fails to execute12:10
Saviqmzanetti, ;(12:10
mzanettiSaviq: I fear we need fginther for this12:10
mzanettionly he know what this stuff is supposed to do12:10
Saviqit's a fail day if I know one12:11
Saviqmzanetti, nic-doffay now that I think of it again... I'm not sure the "constrain height to Scrollable's height" I described should actually happen12:14
Saviqnic-doffay, can you please confirm with ckpringle what should happen with "overflow"12:15
Saviqnic-doffay, because if we enable scrolling *inside* the Expandable in case of overflow, it kind of conflicts with the fact that you should be able to scroll to the other items12:15
Saviqnic-doffay, and the "expand in place" pattern12:15
mzanettiSaviq: I might have found the issue on jenkins... next test running...12:18
Saviqmzanetti, thanks12:18
nic-doffaySaviq, I'm busy waiting on confirmation about the actual expandable item too.12:19
Saviqnic-doffay, get busy prodding, then! :D12:19
nic-doffaySaviq, that's what I've been doing.12:19
Saviqnic-doffay, k :)12:19
mzanettiSaviq: yeah! thats exactly what I meant12:22
Saviqmzanetti, you mean flicking within another Flickable?12:23
mzanettiI mean... there might be use cases for such a thing. as I said, the DashPreview would be one.12:23
mzanettibut really not for such components as this is intended for12:23
mzanettionly if the content will always expand to fullscreen12:24
=== MacSlow|lunch is now known as MacSlow
sil2100mhr3_: ping!12:31
mhr3_sil2100, pong12:31
sil2100mhr3_: who's the guy I should pester related to Unity dash Previews?12:31
sil2100mhr3_: you guys or dednick? (or someone else even?)12:31
mhr3_sil2100, what is the pestering about?12:32
sil2100mhr3_: failing AP tests of course!12:32
sil2100What else should I pester about? :)12:32
mhr3_sil2100, the same thing as on friday?12:34
mhr3_it wasn't really failing preview but failing dbus12:34
mhr3_probably cause of udev12:34
sil2100Not sure if the same, since I was out on Friday, but there's a lot of preview tests failing12:35
sil2100I see most failures look hm, sane: MismatchError: After 10.0 seconds test on DashView.preview_displaying failed: True != dbus.Boolean(False, variant_level=1)12:36
=== alan_g|lunch is now known as alan_g
=== _salem is now known as salem_
=== dandrader|lunch is now known as dandrader
dandradermzanetti, do you know what this error means? https://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy/872/console13:04
dandraderrandom jenkins failure or do we have to do something?13:04
mzanettidandrader: yes. I'm working on it. I just had a successful build again13:04
mzanettidandrader: with a bit of luck you should be able to autoland again in a few minutes13:04
kgunnSaviq: hey, nic-doffay and i were just chatting....wondering if we might want to pick13:19
kgunnone of the scopes to actually tie up to the13:19
kgunnoption expander as an example to try to drive some feedback13:19
kgunni thought you might have one in mind ?13:20
Saviqkgunn, there's no backends hooked up yet13:20
Saviqkgunn, nic-doffay so just work with any of them13:20
Saviqoff of dummy data13:20
Saviqmzanetti, ↑ see who joined? ;13:21
mzanettifginther: \o/13:21
mzanettiSaviq: but I think I just solved it :D13:21
kgunnnic-doffay: can you just go for it for say "music"13:21
Saviqmzanetti, fginther probably saw that you did :)13:21
Saviqfginther, that's why he joined ;)13:21
fgintheruh oh13:22
fginthermzanetti, Saviq, I'm scared to ask what's going on :-)13:22
Saviqfginther, see all the red https://jenkins.qa.ubuntu.com/job/unity8-autolanding/ https://jenkins.qa.ubuntu.com/job/unity8-ci/ ? ;P13:22
mzanettifginther: hehe13:23
mzanettifginther: short summary:13:23
mzanettithe enabling of the local ppa broke mediumtests jobs13:23
mzanettifginther: what I did:13:23
mzanettiI merged the 2 hooks needed for the local ppa into pbuilderjenkins so we don't have to pull them from your bzr branch13:24
mzanettifginther: then I modified the mediumtests jobs to receive the additional parameters required by those hooks13:24
mzanettifginther: seems to work again now13:25
fginthermzanetti, oh, that sounds good13:25
nic-doffaykgunn, is there a design doc for easy reference?13:25
mzanettifginther: still to do: get rid of your bzr branch and switch existing builders to use hooks from pbuilderjenkins13:25
mzanettifginther: I hope I did not mess up something else :/13:25
mzanettibut it looks good now13:25
fginthermzanetti, I should have thought about that yesterday13:25
fginthermzanetti, very cool13:26
mzanettiSaviq: fginther: Victory! https://code.launchpad.net/~mzanetti/unity8/remove-unused-file/+merge/17356813:27
mzanettimerged ^^13:27
fginthermzanetti, nice. thanks for fixing13:27
mzanettidandrader: you should be able to merge your stuff now13:27
Saviqnic-doffay, http://design.ubuntu.com/apps/building-blocks/option-selector those images show exactly that13:27
nic-doffaySaviq, cool.13:27
kgunnnic-doffay: yep....was going to paste the same13:29
Saviqdednick, paulliu standup13:30
dandradermzanetti, ok, trying out13:31
mzanettifginther: I did some more research regarding jenkins slowness and updated our doc13:33
fginthermzanetti, thanks, I'll take a look when I get a moment to breathe13:33
paulliuSaviq: ok13:33
mzanettifginther: not in a hurry with this... just wanted to let you know13:33
Saviqfginther, btw, are any of the ps-saucy-server- machines unhooked?13:36
* Saviq doesn't get the "are offline" messages :P13:37
fgintherSaviq, yes, i386-3 is. I'll get it updated and back online in a few minutes13:38
Saviqfginther, right, nothing that involves us, we're using amd64 :)13:38
fgintherSaviq, yes, the amd64 ones are all enabled13:38
mzanettiSaviq: FYI: we're using i386 for mediumtests13:39
Saviqmzanetti, yeah, ok, I was looking at qmluitests13:39
nic-doffaySaviq, kgunn the music scope in that link?13:49
nic-doffayAll I see is the option selector!13:49
Saviqnic-doffay, what are you looking for in that spec?13:49
Saviqnic-doffay, just put some dummy data in13:50
kgunnnic-doffay: i think that's people13:50
=== Guest86601 is now known as mfisch
=== mfisch is now known as Guest37969
tsdgeosSaviq: so i fixed the show header thing, but the test is extremely unstable when run against 5.0 (works fine against my self compiled 5.1)13:58
tsdgeoswhat do we do? commit the fix with the test commented?13:59
Saviqtsdgeos, yes please13:59
dednicklarsu: ping13:59
larsudednick: ähi14:00
larsuugh, stupid ä key14:00
dednicklarsu: :) hey. having a bit of troulble with the indicators-client autopilot tests on jenkins.14:01
dednicklarsu: the indicator-battery process doesnt run and when started, it just segfaults.14:02
dednicklarsu: works on my desktop though14:02
larsudednick: lol, we're still using indicator-battery?14:02
larsudednick: got a backtrace?14:02
mzanettilarsu: just out of curiosity. what else would we use?14:03
larsumzanetti: indicator-power, the one we use on the desktop. I hacked up battery before the first phone demo so that we had a battery indicator at all14:04
larsu(indicator-power wasn't compatible with the new stuff back then)14:04
mzanettilarsu: ah ok.. didn't know. is it compatible with our stuff now?14:04
dednicklarsu: http://pastebin.ubuntu.com/5858481/14:04
larsumzanetti: should be, but I don't know if it already supports the phone profile14:05
dednicklarsu: doesnt look very useful14:05
mzanettiok. thanks14:05
larsumzanetti: doesn't look like it. But adding that to indicator-power is probably more worthwhile than fixing battery14:06
dednicklarsu: no, indicator-power doesnt have a phone profile14:06
larsudednick: ugh, no debug symbols14:06
dednicklarsu: we're also using indicator-time instead of indicator-datetime14:06
larsudednick: but a warning: "failed to query backlight devices"14:06
larsudednick: I know, same story there14:06
dednicklarsu: VM14:07
larsuthese things were never meant for long time use14:07
dandradermzanetti,  https://code.launchpad.net/~stolowski/unity8/social-preview-bindings/+merge/173041 <- that was a funny one. CI set it to "Merged" and then back to "Need Review" :D14:07
mzanettidandrader: not entirely sure if this is really funny :D14:07
larsudednick: where does the latest code for indicator-battery live?14:07
dednicklarsu: indicator-power/phablet i think14:08
* larsu has a vague recollection of putting it into some +junk branch14:08
larsudednick: thanks14:08
FernandoMiguelanyone can point me in the direction of debugging unity ?14:10
FernandoMigueldoing $ killall -9 unity-panel-service every few minutes, is no longer an option :(14:10
mzanettifginther: I think we need the local repository also for ci, not only autolanding. Could you enable that for us?14:10
FernandoMiguel  Installed: 7.0.2+13.10.20130705.1-0ubuntu1 on ubuntu 13.1014:11
fginthermzanetti, hmm, that should be also work14:11
fginthermzanetti, hmmm14:11
mzanettifginther: oh... not sure where I was looking14:12
mzanettiseems there, yes. sorry for the noise14:12
larsudednick: that brightness warning seems to be the issue. Whoever wrote that code doesn't check for NULL in the case that the brightness devices can't be found14:14
dednicklarsu: ah. i c14:14
larsudednick: I'll cook up a patch if you'll test it ;)14:15
* larsu doesn't have a device for testing set up right now14:15
dednicklarsu: sure. i'll have to test it on jenkins though14:16
tsdgeosmzanetti: can you verify https://code.launchpad.net/~aacid/unity8/fixShowHeaderHalfShownAtTop/+merge/173727 fixes the bug for you?14:17
mzanettitsdgeos: on it14:18
tsdgeosgreyback_: if you can have a look too14:20
Saviqmzanetti, gentle reminder:14:22
Saviqdh_install: usr/share/unity8/plugins/mocks/Unity/Launcher/qmldir exists in debian/tmp but is not installed to anywhere14:22
Saviqdh_install: usr/share/unity8/plugins/mocks/Unity/Launcher/libMockLauncherPlugin.so exists in debian/tmp but is not installed to anywhere14:22
greyback_tsdgeos: couldn't you use QSKIP() instead of commenting it all out?14:22
mzanettiSaviq: yep... just seen it and running a build14:23
tsdgeosgreyback_: i can, want me to?14:23
greyback_tsdgeos: and what bug it it? Could you attach it to the MR please?14:23
tsdgeosmzanetti: did you open a bug?14:23
mzanettitsdgeos: no14:24
greyback_tsdgeos: I'd prefer it, as you get "Skip" prints to remind you the test was skipped.14:24
tsdgeosgreyback_: ↑↑14:24
Saviqgreyback_, scroll so that thick separator is half-on screen, scroll to the sides14:24
Saviqgreyback_, → full breakage14:24
Saviqit's starts to behave like The Incredible Machines14:24
Saviqdednick, hey, you getting anywhere with autopilot?14:27
nic-doffaySaviq, does the expandable item only exist for pre expanded lists?14:27
Saviqnic-doffay, rephrase please14:28
dednickSaviq: yep. jenkins is using a vm which has no backlight interface, causing the battery indicator to crash and test to fail. larsu fixing up a patch for me to test.14:28
dednickSaviq: can reproduce on local VM14:28
Saviqdednick, awesome14:28
Saviqnic-doffay, if you mean contained vs. full-width; expanding vs. expanded, all 4 combinations should be possible14:29
nic-doffaySaviq, that's what I meant.14:29
nic-doffaySaviq, you would the expanding option be treated with mouse clicks though?14:30
nic-doffayIf an item is expanded then it shouldn't be able to be selected and collapse the list...14:30
Saviqnic-doffay, if "expandable: false", you can't collapse14:30
nic-doffaySaviq, that's what I meant so it will only be available for non-expandable lists?14:31
Saviqnic-doffay, collapsing? yes14:31
Saviqnic-doffay, but expandable: false is just a simple constraint on top of the regular expandable14:32
Saviqnic-doffay, which results in it being always full-height14:32
nic-doffaySaviq, yeah.14:32
nic-doffaybut that's not what I meant.14:33
nic-doffayI'm probably not explaining myself properly.14:33
nic-doffayImagine this scenario:14:33
nic-doffayThere is an entire list which is collapsible. If it's full of expanding items would the only way to collapse the list be to select the header one?14:33
nic-doffayThinking of the OptionSelector right now.14:34
nic-doffayWhich would be full of expanding icons.14:35
nic-doffaySaviq, because at the moment the option selector selects options.14:35
nic-doffayIt's a bit counter intuitive if that feature is disabled in it imo.14:36
Saviqnic-doffay, no, tapping outside of the item would collapse it, too14:36
Saviqnic-doffay, and according to the spec actually scrolling the view would, too14:36
nic-doffaySaviq, cool I'll read it closer.14:37
tsdgeosSaviq: the thing that installs the dependencies .deb when running ./build wants me to remove libunity-scopes-json-def-desktop and install libunity-scopes-json-def-phone14:50
tsdgeosi've said no :D14:51
Saviqtsdgeos, you can install it14:51
Saviqtsdgeos, it's just used build-time14:51
dandradergreyback_,  do you know where /system/bin/ubuntuappmanager comes from?14:51
dandrader(the repo)14:51
tsdgeosSaviq: ah14:51
greyback_dandrader: sorry no. That's inside the android chroot, no idea where that comes from. sergiusens?14:52
dandradergreyback_, yes, it's in the chroot14:53
tsdgeosit's from the qt-ubuntu thing, no?14:53
tsdgeosricmm should know afair14:53
dandradertsdgeos, no, qtubuntu is the qt platform abstraction plugin. ubuntuappmanager is the process that talks to surfaceflinger to manipulate stacking etc AFAIU14:54
greyback_tsdgeos: nope, nothing to do with qtubuntu. That's ubuntu side14:54
tsdgeosstill i'd ask ricmm :D14:54
tsdgeosgreyback_: ok, QSKIPed the test14:56
greyback_tsdgeos: thanks.14:57
tsdgeosgreyback_: it's even a smart QSKIP!14:59
larsudednick: sorry for the delay, was distracted. Patch is here: https://code.launchpad.net/~larsu/indicator-power/phablet-fix-brightness-crash15:00
larsudednick: and a MR: https://code.launchpad.net/~larsu/indicator-power/phablet-fix-brightness-crash/+merge/17374715:02
dednicklarsu: ta. will take a look15:03
FernandoMiguelis there any # for user support?15:17
mzanettiFernandoMiguel: I think you're in the correct channel. But we unity8 people spam the channel so much that unity7 question might drown in the noise :/15:19
FernandoMiguelmzanetti: I assume 8 is the next version? I'm on 13.10 :S15:20
mzanettiFernandoMiguel: yeah. the phone+tablet+dekstop thing15:20
FernandoMiguelanyone I can bug about this?15:20
FernandoMiguelI should affect more users as they join 13.10 testing15:20
Saviqnic-doffay, https://code.launchpad.net/~saviq/unity8/improve-openeffect/+merge/17375015:21
mzanettiFernandoMiguel: usually unity7 people are around too. but I haven't seen one of them today yet15:21
nic-doffaySaviq, will get to it soon.15:21
Saviqnic-doffay, k15:21
FernandoMigueltips on how to debug it so I can at least file a bug?15:21
FernandoMiguelcause saying: it's eating one core, aint enough15:21
* mzanetti has absolutely no clue about unity715:21
=== om26er_ is now known as om26er|dinner
FernandoMiguelhad no idea it changed so much15:22
FernandoMiguelguess you guys need a new $15:22
mzanettiFernandoMiguel: its a rewrite. in a differnt language :)15:22
mzanettitsdgeos: hey, do you think we could somehow suppress the -Wpedantic warnings coming from the private Qt includes used in LVWPH?15:23
mzanettitsdgeos: I'm always worried they hide real ones15:23
tsdgeosmzanetti: drop the -Wpedantic? :d15:23
mzanettitsdgeos: I'm thinking more about #pragma push  etc15:23
mzanettibut I never used that with gcc15:23
tsdgeosmzanetti: same here, no pragma magic on my side15:24
* mzanetti thinks back to the dark days when he had to use msvc :D15:24
tsdgeosbut i can see what mr google says15:24
tsdgeos#pragma GCC diagnostic push15:26
tsdgeos#pragma GCC diagnostic ignored "-pedantic"15:26
tsdgeosseems should work15:26
tsdgeoslet's see15:26
mzanettitsdgeos: yeah, need to #pragma pop it again. just wrap the #include in this. should do15:27
dednicklarsu: 18+    devices = g_udev_client_query_by_subsystem (client, "bcklight");15:27
dednicklarsu: did you mean to change to bcklight from backlight?15:27
greyback_tsdgeos: approved, will wait for CI before top approving15:27
larsudednick: woah, of course not. I used that to reproduce the crash. Thanks for catching it :)15:28
Saviqnic-doffay, I had to resubmit with a prerequisite https://code.launchpad.net/~saviq/unity8/improve-openeffect/+merge/17375215:28
dednicklarsu: :)15:28
tsdgeosis anyone else getting this?15:28
tsdgeosQFontDatabase: Cannot find font directory '/usr/lib/x86_64-linux-gnu/fonts' - is Qt installed correctly?15:28
tsdgeosMirv: ↑↑ ?15:28
Saviqtsdgeos, yeah, all of us are getting that15:28
Saviqtsdgeos, it's qmlplugindump or whatever15:29
tsdgeosthat's new isn't it?15:29
larsudednick: pushed a patch. Embarrassing :P15:30
dednicklarsu: cool. happens to everyone :)15:30
Saviqtsdgeos, yes, dednick added the qmltypes thingie recently15:31
SaviqMirv, but yeah, it might be interesting to you, too, just run `qmlplugindump Foo 1` and it will spew out the fonts not found error15:31
tsdgeosmzanetti: https://code.launchpad.net/~aacid/unity8/disablePedanticQtPrivateHeaders/+merge/17375515:33
nic-doffaySaviq, k15:34
=== dandrader is now known as dandrader|afk
mzanettitsdgeos: perfect15:36
larsudednick: is that branch set up for automerging?15:42
dednicklarsu: i have no idea15:42
dednicklarsu: probably not15:43
larsudednick: ya, that's what I'm thinking as well. I'll merge it manually15:43
dednicklarsu: actually, it has jenkins ci runs on a previous merge15:44
larsudednick: oh okay. I'll wait a bit then :)15:44
dednicklarsu: will it automatically be uploaded to the phablet-team ppas? or is a manual trigger?15:49
dednicki'm guessing manual15:49
larsudednick: no clue, probably manual, yes15:49
=== dandrader|afk is now known as dandrader
kenvandinefginther, when making changes to autolanding in cu2d-config, do you need to update something in jenkins?16:06
kenvandinefginther, can you look at https://code.launchpad.net/~ken-vandine/cupstream2distro-config/autoland_friends-app_sdk/+merge/17371816:06
fgintherkenvandine, we usually need to re-deploy the jobs16:06
kenvandinethat's it though?16:07
kenvandinei can do that :)16:07
kenvandineor is there another redeploy that you need?16:07
fgintherkenvandine, nope, just the one. Are you using the deploy-cupstream2distro-config job?16:09
fgintherupdate-stack does not deploy the jobs to s-jenkins16:09
fgintherkenvandine, you would need to use cu2d-update-ci16:10
fgintheror just use the jenkins job :-)16:10
* kenvandine doesn't know how to do either... maybe i shouldn't know :)16:10
=== om26er|dinner is now known as om26er
=== Guest37969 is now known as mfisch
=== mfisch is now known as Guest41197
nic-doffaySaviq, so I'm assuming the OptionSelector should also have a variable delegate?16:35
nic-doffayOr just an expandable delegate which is set to either true or false?16:35
Saviqnic-doffay, that's a design question16:35
nic-doffaySaviq, ok so nothing has been decided on yet.16:35
Saviqnic-doffay, I'd assume icon + text16:35
Saviqnic-doffay, unless I don't understand your question16:35
Saviqicon optional16:36
nic-doffaySaviq, k16:36
MirvSaviq: yeah it has always spewn out that, I haven't seen a configure option that would affect it16:46
SaviqMirv, k16:46
Mirvtsdgeos dropped from the channel it seems16:46
* Mirv -> night16:47
mzanettiSaviq: do we store the stdout somewhere on the device?16:50
Saviqmzanetti, ricmm should know16:50
Saviqmzanetti, I think we do16:50
greyback_mzanetti: yes in ~.ubuntu-session or ~.ubuntu-touch-session16:51
mzanettigreyback_: cheers16:52
mzanettiom26er: replied to the bug16:54
Saviqfginther, /me no get it http://s-jenkins:8080/job/unity8-autolanding/54/ http://s-jenkins:8080/job/unity8-autolanding/55/16:55
Saviqfginther, 54 landed https://code.launchpad.net/~paulliu/unity8/fix-video-lens/+merge/17367416:55
Saviqfginther, it's in trunk https://code.launchpad.net/~unity-team/unity8/trunk16:55
Saviqfginther, but it's merging once more in 55?16:55
Saviqfginther, and the branch isn't updated that it's Merged, and the changelog entry doesn't contain the "Merged branch lp:xxx" thing...16:56
fgintherSaviq, looking16:56
om26ermzanetti, thanks. I commented some stuff16:59
=== olli_ is now known as olli
fgintherSaviq, possibly it was merged by hand?17:00
Saviqfginther, ah17:01
Saviqfginther, http://s-jenkins:8080/job/unity8-autolanding/54/console17:01
om26erto keep upto date should I still flash the newest saucy-preinstalled-touch-armel+mako.zip or is that now covered with apt-get dist-upgrade ?17:01
Saviqfginther, or is that ok?17:01
fgintherSaviq, I'm looking through the jenkins logs.17:01
mzanettimterry: any idea on this? https://bugs.launchpad.net/unity8/+bug/119944117:02
ubot5Ubuntu bug 1199441 in Unity 8 "[regression] greeter does not appear on some occasions" [Undecided,New]17:02
Saviqfginther, shouldn't have been merged manually17:02
mzanettiom26er: I flash all the time...17:02
fgintherSaviq, by the time 54 finished, lp:unity8 had already been updated17:03
Saviqfginther, interesting...17:03
fgintherSaviq, 55 is running because of the incorrect status on the merge proposal17:03
Saviqfginther, so maybe http://s-jenkins:8080/job/unity8-autolanding/53/parameters/? merged it?17:04
Saviqprobably not17:04
Saviqfginther, ok, nvm, let's see what happens17:04
fgintherSaviq, I'll dig a little more, just in case the jenkins tools have a bug17:05
Saviqfginther, thanks17:05
dednickSaviq: ping17:38
Saviqdednick, pong17:38
dednickSaviq: howdy. larsu has fixed the bug in indicators-battery and it's been merged. we need to get it into the ppas.17:39
dednickSaviq: do we need to bump the version or something?17:40
Saviqdednick, it's daily-released, right?17:43
Saviqdednick, so we probably should just wait for it to release, and that's it17:44
dednickSaviq: I have no idea if it's daily-released. where can i check?17:49
Saviqdednick, in debian/changelog of it17:50
dednickSaviq: actually, it's a bit of a hack branch of indicators-power, in lp:indicators-power/phablet17:50
Saviqdednick, http://bazaar.launchpad.net/~indicator-applet-developers/indicator-power/phablet/view/head:/debian/changelog17:51
Saviqdednick, it needs to be bumped-released, right sergiusens?17:51
Saviqdednick, so you need to MP a debian/changelog bump for saucy17:52
Saviqdednick, so that http://s-jenkins:8080/job/indicator-power-phablet-autolanding/ kicks in17:52
mterrymzanetti, looking18:01
mzanettimterry: I haven't seen this before18:01
mterrymzanetti, weird, nor have i18:01
mzanettimterry: could it be because he only dist-upgrades instead of flashing?18:01
mterrymzanetti, don't know why it would be18:01
mzanettimterry: shouldn't... but who knows18:02
mterrymzanetti, not even sure how that would happen.  either the power button signal or the greeter.show() wouldn't be working for him18:02
mzanettimterry: what if the lightdm plugin doesn't load? I guess that would prevent the shell from starting already18:03
dednickSaviq: how do you generate the changelog entry from bzr history? manual or is there an automated tool?18:04
Saviqdednick, I bet there's some tool, I just never remember what it is :)18:04
dednickSaviq: :) yeah, i'm guessing the daily-release does it somehow. nevermind. will just manual it.18:05
mterrymzanetti, yeah, I would bet18:05
Saviqdednick, there's "debcommit" but it actually works the other way 'round18:05
Saviqmterry, bzr log → debian/changelog, any tool?18:06
Saviqdednick, yeah, I believe that's a custom tool, though18:06
mterrySaviq, I've never used one, always the other way with debcommit18:08
mterrySaviq, but I bet didrocks built something for his daily release stuff18:08
Saviqmterry, that's what I thought, thanks18:08
dednickSaviq: http://pastebin.ubuntu.com/5859125/ ? look right to you?18:09
dednicknot sure about the version18:09
Saviqdednick, I'd probably continue the previous scheme, so 13.10.1phablet118:09
dednickSaviq: I think it's the the .1 is the month after of the release.18:10
dednickSaviq: so it'd have to be a 13.10.-3 ;)18:12
Saviqdednick, TBH if it's higher - it's good!18:13
dednickSaviq: https://code.launchpad.net/~nick-dedekind/indicator-power/phablet.release.bump/+merge/17378518:21
Saviqdednick, for the sake of being picky, can you please revert the whitespace change?18:23
dednickSaviq: sure18:23
dednickSaviq: harder than you think when you have save removing the whitespace in your editor :)18:24
Saviqdednick, I know! ;D18:24
Saviqdednick, uncommit, shelve, commit ;)18:24
dednickSaviq: done18:33
Saviqdednick, thanks18:34
=== Guest41197 is now known as mfisch
=== mfisch is now known as Guest1021
Saviqfginther, ugh... unity8-autolanding seems to take an hour now (1/2h before), is that expected? or maybe just queue?19:04
fgintherSaviq, the armhf queue is really long right now, that's probably the issue.19:05
Saviqfginther, ok19:06
fgintherSaviq, I did find a jenkins tools bug related to the weird commit. Rolling out the fix now.19:56
Saviqfginther, thanks19:56
Saviqmzanetti, can you `bzr commit --unchanged` launcher-backend and push so that jenkins picks it up with the scheduler?21:16
Saviqmzanetti, it feels like it's been rebuilt too many times now and it's not really doing the right thing21:16
mzanettiSaviq: nah. I just manually killed the job and re-triggered it because it was again on a non updated panda21:17
mzanettiSaviq: I'm currently walking through all pandas and upgrading the hooks21:17
mzanettiSaviq: ok. last one done. the hooks issue should be gone for good now21:18
Saviqmzanetti, thanks21:19
mzanettiSaviq: I wonder why the DashBar test started failing in jenkins22:01
Saviqmzanetti, did you see more than one failure?22:11
mzanettiSaviq: 7 in a row22:12
mzanettiSaviq: passes locally22:12
Saviqmzanetti, ugh22:12
Saviqmzanetti, it's a 5s timeout, think it might simply be timing out?22:14
Saviqmzanetti, both tests are failing at22:14
SaviqtryCompare(panel, "animating", false) // wait till the animation ends22:14
Saviqmzanetti, unless something changed in the sdk...22:14
mzanettiSaviq: no... its the compare() above22:14
mzanettiah no22:15
Saviqlast toolkit release was 16h ago, so not that22:16
mzanettiSaviq: uh... after upgrading my machine it fails locally22:18
mzanettisame failure22:18
Saviqmzanetti, what did you upgrade?22:18
Saviqyeah, it's the SDK for sure22:18
Saviqwill pinpoint the rev if can reproduce22:19
=== salem_ is now known as _salem
Saviqmzanetti, it's the last commit in the toolkit22:23
mzanettiSaviq: and its a but in the panel22:23
mzanettithe "animating" property goes once to true and never back to false22:23
mzanettigood we're testing their stuff22:23
mzanettiwhy do I keep on writing "but" instead of "bug"?22:24
mzanettiSaviq: should I report a bug or are you already taking action?22:26
Saviqmzanetti, I'll file, can you find out where we've picked the new SDK from? so that we can disable it for the time being, maybe?22:27
mzanettiSaviq: Get: 4 http://ppa.launchpad.net/ubuntu-sdk-team/ppa/ubuntu/ saucy/main qtdeclarative5-ubuntu-ui-toolkit-plugin amd64 0.1.46+13.10.20130709bzr600saucy0 [257 kB]22:28
mzanettiSaviq: is it actually still a good idea to use that ppa?22:28
mzanettiprobably not22:28
Saviqmzanetti, if it's daily released anyway, probably not22:28
mzanettiSaviq: ppa is removed and the 3 jobs retriggered. I'm off for bed... see you tomorrow.22:36
Saviqmzanetti, thank you, cheers22:36

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!