=== Trevinho_ is now known as Trevinho [00:34] Saviq, I don't know why it's failing on CI [00:35] I mean, those two tests.. === Guest72010 is now known as balloons === TheMuso` is now known as TheMuso === duflu_ is now known as duflu [08:53] MacSlow: any help with the notification thing? [08:54] s/help/luck [09:00] tsdgeos, not yet... still stuck with all those ap-tests failing... [09:01] tsdgeos, I've no idea why they all fail with http://pastebin.ubuntu.com/6827212 [09:01] MacSlow: are you running them with make autopilot? [09:01] tsdgeos, yes [09:03] you're compiling in /tmp/bla/unity8/ ? [09:03] tsdgeos, yes [09:06] MacSlow, that error says autopilot can't find the running unity8 process to introspect - maybe not running with testability? [09:06] MacSlow, got a branch so we can test? [09:07] Saviq, for this I'm just using plain unity8 trunk [09:08] MacSlow, and all tests fail for you like that? [09:10] Saviq, yeah [09:10] MacSlow, feels like a dbus issue, when did you last reboot? [09:10] MacSlow, like your dbus session got confused between your terminal and upstart, maybe [09:11] tsdgeos, Saviq: about an hour ago... so I compiled unity8, cd builddir, LC_ALL=C make install, LC_ALL=C PYTHONPATH=../tests/autopilot autopilot run ... [09:11] I can do a reboot and see if that solves anything... [09:16] Saviq, hi [09:16] Cimi, yo [09:16] Saviq, can you give me feedback on https://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/203387 ? [09:16] Saviq, I would like to know what you wanted with the preview widgets [09:17] Saviq, apart from adding tests and cleaning code, is something like this correct? [09:18] Cimi, don't actually integrate it with the old previews, they're going away [09:19] Saviq, so I dimly separate them, but leave them unused in a folder? [09:19] *simply [09:19] Cimi, yes, and test them [09:19] ok [09:19] Cimi, they need to be self-contained, too [09:20] Cimi, so the actual Audio { } component needs to be inside, too [09:20] Saviq, but this will imply having one audio component per track [09:20] Cimi, that's my next sentence [09:20] Cimi, in this particular case, the widget needs to support a model, not just one item [09:21] Cimi, so you need a Column+Repeater here [09:21] ok [09:22] jamesh: hi! Sorry for the mediascanner2 LP problems - I actually thought that the team configuration I prepared was enough [09:22] tsdgeos, Saviq: same problem even after a reboot... but starting unity8 via the run script works as usual [09:22] jamesh: I made you an administrator of the mediascanner team now so you have ultimate power [09:22] sil2100: yeah. It looked like mediascanner and mediascanner2 had the same ownership config, so I wonder what is different? [09:22] MacSlow, if you have unity8 installed system-wide, remove ~/.config/upstart/unity8.conf [09:23] I can certainly update things on the old project [09:23] MacSlow, otherwise cp data/unity8.conf ~/.config/upstart/ [09:24] jamesh: that's indeed a valid question [09:26] Saviq, copying over data/unity8.conf to ~/.config/upstart did the trick... although I'm not seeing any of the assets (images) in the notifications [09:28] Cimi, it won't be a real model, but an array of hashes instead, so access them like so: modelData['title'], modelData['subtitle'], modelData['source'] [09:29] Saviq, so I'll just create a property var for this, and write a mock? [09:29] Cimi, yup [09:29] tsdgeos, hm... with the fixed unity8.conf none of the notification ap-tests fail for me on the desktop (using trunk of both... unity-notifications and unity8) [09:31] MacSlow: you sure you're using trunk unity-notifications? try usign http://10.98.3.7/archive//head.unity8/trusty/qtdeclarative5-unity-notifications-plugin_0.1.1+14.04.20131030.1bzr196pkg0trusty0-0ubuntu1_amd64.deb [09:31] * Saviq reboots [09:31] tsdgeos, just pulled lp:unity-notifications about 30 minutes ago... [09:32] tsdgeos, I retry with that .deb [09:38] MacSlow, I can confirm, with the .deb it fails [09:41] tsdgeos, Saviq: yes... that .deb also fails for me here. [09:42] MacSlow: i have the same when using the deb that results from compiling unity-notifications trunk [09:42] MacSlow: you sure you did install the deb after compiling it? [09:42] tsdgeos, Saviq: I'll work on the fix (for the test) now [09:43] tsdgeos, I saw that I used fixed trunk I did locally yesterday and copied over the wrong directory. [09:43] tsdgeos, sorry for the mixup [09:47] mzanetti: ping [09:48] tsdgeos: pong [09:49] mzanetti: i was having a look at https://bugs.launchpad.net/unity8/+bug/1271676 and found out it's because of the preview [09:49] Ubuntu bug 1271676 in Unity 8 "Launching an app from the app scope causes app scope to jump to the top" [High,Triaged] [09:50] hmm.. ok [09:50] mzanetti: can it really happen in rendererLoader.item.onClicked that scopeView.scope.activate will trigger a preview? [09:50] yes.. that's how it normally works [09:50] for everything except installed apps [09:51] wait... need to read the code [09:51] well, everything except installed apps goes through the else, no? [09:52] tsdgeos: true... should not happen in there [09:53] so basically could move all the previewlist stuff inside the else? [09:54] I'd say yes... [09:55] when you guys are creating all the new Scopes API's, are you keeping the previews in mind? so we eventually will have a PreviewModel which is just filled with data instead of throwing all those pointer forth and back [09:55] mhr3, see https://docs.google.com/a/canonical.com/document/d/1NmiM4UCnJgf6IEawmfyTOHRNAA5ZGrqpyrPqPOibwc8/edit#heading=h.6fng39a8onr for a reference on how data for widgets will be provided [09:55] erm [09:55] Cimi, ↑↑ [09:55] mhr3, ignore [09:56] Saviq, dziekuje [09:56] Cimi, nie ma za co [09:57] mzanetti, I'm not entirely sure what the lower-level plan is there, mhr3, pstolowski? [09:57] Saviq, meaning? [09:57] "lower-level plan"? [09:58] mhr3, how will the plugin expose the preview as received from the scope? how will we match previews to result items? [09:58] Saviq, the plugin exposes previews as models of the widgets [09:59] you request a preview for specific result [09:59] mhr3, current issue is the preview just comes in at a random time, and there's no way to associate it with a result item, and we're jumping all kinds of hoops because of that [09:59] mhr3, yeah, but it comes back async, how do we match it back to the result? [09:59] Saviq, i changed the semantics the way i mentioned yesterday [10:00] Saviq, ie a preview request gives you back an empty model right away, once all data is there a signal is emitted [10:00] up to you to ignore it :) [10:00] ok.. the model right away is good [10:00] yup, we can then associate it with the result [10:01] ideally it would be a model just the same size as the actual item model, just containing the prviewdata :) [10:01] the day we get rid of [10:01] terminate called after throwing an instance of 'boost::exception_detail::clone_impl >' [10:01] what(): could not activate surface with eglMakeCurrent [10:01] will be an awesome day [10:01] haha... indeed [10:01] but this one is not as bad as "address already in use" [10:03] mzanetti, hm, theoretically the result model could have previewdata role, a that thing would be filled in only if you asked for it [10:03] but that doesn't play overly nice with being able to do that lazily [10:04] mhr3: you can just fill the stuff on demand... [10:04] with an additionaly property, "loaded: bool" so we know if we should show the spinner instead [10:05] but we wouldn't need to catch a signal with a pointer later and try to find what it belongs to... [10:06] mzanetti, i can easily emit such a signal on the preview model itself if you want that [10:06] mhr3: for defining something I'm a bit far off the topic right now... would need to read through it again. [10:07] but I'm sure Saviq can handle this ;) [10:07] mzanetti, mhr3, if a role in the model, it'd have to be a stack of previews (think payment confirmation page) [10:07] ah yeah... that's another issue [10:07] which might not be an overly bad idea anyway [10:09] mhr3, data() is only called on a role when that particular role is requested, so having that there might be a good enough idea [10:09] Saviq, but it's const :P [10:09] so isn't exactly lazy-init friendly [10:10] mhr3, not like that can't be worked around ;) [10:10] but those workarounds are awful [10:11] mhr3, we should write down all the usecases and see what's problematic where and why [10:11] mzanetti, btw, remember how you spent countless hours to get the "highlight item under preview" work properly? [10:12] yeah [10:12] mzanetti, you'll be pleased (?) to know it's going away completely - previews are going to cover full screen now [10:12] its totally broken all over already :/ [10:12] \o/ [10:13] and there's going to be a back button, simply [10:14] Saviq, for me it doesn't feel right, previews can be dynamic and stuffing it into the model makes it feel like it's immutable [10:14] mhr3, only the mapping between result and "preview stack" is immutable, what's in that preview stack is a different issue [10:17] still, you'll never need multiple previewstacks active [10:17] mhr3, that is true [10:18] mhr3, well, when switching between previews we will [10:18] mhr3, can't destroy the old one before requesting / creating the new one [10:18] that's looking at it from a different angle [10:19] mhr3, sure, the old one isn't "active" any more, and arguably we could just use a snapshot of it [10:19] mhr3, OTOH it depends on how persistent their state is supposed to be [10:19] mhr3, but I doubt it is meant to be persistent [10:19] and if you were deep in the stack and come back i would expect it to start over [10:20] mhr3, you would probably be prevented from going away from an unconfirmed or cancelled page (again, preview confirmation) [10:20] mhr3, and yeah, once you switch away you start over [10:21] Saviq, i'd say there isn't a strong need to put it in the result model [10:22] but we do have the option to change it if we do want it that way [10:22] mhr3, sure, you're probably right, we can request a stack for a particular result on getting into a new preview, and the old one would get deleted when its preview page gets off screen [10:24] mhr3, as long as you make sure that a preview for a particular result will always end up in the stack it's been requested for, it should be fine [10:24] mhr3: you going to be in office today? [10:25] dednick, yea, for 2nd half :) [10:26] Saviq, i think that navigating in the stack will be done by invoking methods of the preview itself [10:26] mhr3, yeah, so they can be "confined" to that particular preview stack, sounds good to me [10:27] mzanetti, ↑ ? [10:27] mhr3: hehe, now that you have to do your 8:30am start you're slacking off again?! [10:28] dednick, shush, you came in at 9:30, i can see it here, imagine you'd have to be there an hour earlier :P [10:29] um, yeah :) trip in was a bit of a mightmare [10:29] Saviq: not entirely following... but you know what the issues are with how it works right now... if you're happy with it I'm sure were fine [10:30] Saviq, anyway, we don't have api that would tell us about nested previews yet... cc pstolowski :) [10:30] mzanetti, workflow would be like so: activating an item will get you an empty preview stack, that you can start displaying straight away, the preview data would come into that model and any subsequent preview interaction will be "confined" to that preview stack for that result [10:31] mzanetti, when you switch to a different preview (left/right), you'd request another preview stack for that result, and drop the other one when it goes off screen [10:32] but that's mostly how it is right now, no? [10:32] mzanetti, problem right now is that you don't get the model straight away [10:32] that's one, yes [10:32] mzanetti, you request a preview and you never know when it'll come (hence the cancellation we did) [10:32] the other is that I need to build a model in qml which reflects the scope category but holds previews [10:33] have to run to a right-edge meeting now... [10:33] mzanetti, yeah, but that model is only ever max 3 items, and the append/prepend operations are pretty simple I'd think [10:33] mzanetti, sure [10:39] Saviq, what this means though is that we can't update the result models when preview is showing [10:39] (when we have async notifications about changes) [10:40] mhr3, I think the fact that we're not showing the results any more precludes result updates anyway [10:40] mhr3, as suddenly switching between items would get you somewhere you didn't expect [10:41] well that would be weird even if we were showing the results [10:41] plus weirdness like current result disappearing [10:42] Saviq: I've probably spammed you a bit with merge proposals before I left yesterday"today" [10:43] mhr3, indeed [10:43] karni, it's fine, didn't respond to the emails, though, wanted to wait for you to show up [10:43] Saviq: :) [10:44] Saviq: fire at will when you have a moment, I'm all ears [10:44] karni, re: carousel, yes I know the problem, didn't dig into it, though, Carousel has a huge number of properties I've no idea what they do ;P [10:45] Saviq: so, it's something we're aware of? (i.e. there is a problem, and we already know)? [10:45] karni, yes, we are aware, no one looked at a solution yet, though [10:45] yeah, fixing that one will be challenging. [10:45] Saviq: ack [10:47] karni, as for the filtergrid, I suspect there's a binding loop somewhere there, and you're falling victim of that [10:49] Saviq: It seemed like sometimes FilterGrid wouldn't "unwind", the opposite to the way it nicely collapses. The fix works fine, the problem only surfaces when you start *changing* collapsedRowCount in the mean time. [10:49] karni, for https://code.launchpad.net/~saviq/unity8/fix-card-header-height/+merge/203411 - if you feel like it, approve away, otherwise we can ask mzanetti to put another set of eyes on it [10:50] Saviq: can you explain me just one line from your MP? line 91, header.y + header.height [10:52] karni, that's the lower edge of the header [10:52] karni, so its top edge + its height [10:52] Saviq: Right. I understand that, what I don't understand why would you test that against *height* [10:53] karni, hmm let me look again [10:53] karni, because that's the height I expect the card to be [10:54] karni, we need the card's bottom edge to be the same with the header's bottom edge [10:54] ha [10:54] gotcha [10:54] and we're only testing on 1 card, so bottom edge == height [10:54] Saviq: I feel confident top approving this [10:55] karni, go for it, then [10:59] Saviq: how about https://code.launchpad.net/~unity-team/unity8/unity8-fix-carousel5/+merge/203444 ? [11:00] karni, well, doesn't x <= 6 eq x < 5 for integers? ;) [11:01] no it doesn't [11:01] no :D [11:01] hehehe [11:01] ^ ^ [11:01] karni, but anyway, I'd rather not touch that value in lp:unity8, as it's working fine here [11:01] karni, make the test work with <= 6 for now [11:01] Saviq: shall I rebase to new-scopes? [11:02] karni, and we'll tweak it in new-scopes for the new rule, yes [11:02] Saviq: ack! [11:09] mzanetti, ok, shall we go through the release process then? === _salem is now known as salem_ [11:11] mzanetti, I'm thinking let's push trunk to lp:unity8/devel, uncommit trunk up to the latest release, propose a merge from devel to trunk, and go through the process with that? [11:12] wonder how we can get the changelog message for this, though [11:13] Saviq: okay, this effectively just adds a tests, that passes in unity8 trunk [1]. I'll get the tweak ready for new-scopes now, aye? [1] https://code.launchpad.net/~unity-team/unity8/unity8-fix-carousel5/+merge/203444 [11:13] karni, yup [11:15] karni, one thing: var renderer...var renderer - the second "var" is not needed, and will be frowned upon by qtcreator, for example [11:15] Saviq: oh, that wasn't intentional. shall I fix? [11:15] karni, yes please [11:15] ok [11:16] didrocks, do you have the daily release dch generator somewhere so that we could use it locally? [11:16] Saviq: hum, it's a in library, but what are you trying to aciheve? [11:16] achieve* [11:16] Saviq: fixed [11:16] didrocks, we're going the staging/devel branch route for lp:unity8 [11:17] Saviq: hum, talk with asac, it's not what we want people to go with [11:17] didrocks, as the overhead for Train CI is just to big for us for every MP [11:17] *too [11:17] didrocks: maybe you should not create guidelines that make it impossible to devel stuff if you don't want that [11:17] Saviq: well, it seems you didn't even try it and already decided about the guidelines [11:18] Saviq: would have been great that you try before talking about the overhead [11:18] especially as you are bundling multiple MPs together [11:18] so not sure what you mean by overhead [11:18] didrocks, well, I wasn't even schooled yet, but we just know how long it takes to build/run the test suites locally [11:18] didrocks, manual testing time [11:18] Saviq: and you are not applying the same level of quality in your "devel trunk"? [11:19] didrocks: we are not doing manual testing for a one liner, no [11:19] didrocks, we do, but we let CI do the testing for us [11:19] depending on the one liner of course :D [11:19] tsdgeos: well, you can bundle multiple MPs all together [11:19] and accept/reject all of them [11:19] didrocks: that means that everyone has to wait for the very obvious oneliner fix because it has to be bundled [11:19] Saviq: *shrug* ... i am with didrocks that you should really first try using the system and see how fast you can go before investing in workarounds [11:20] bundling MPs is just a workaround for the thing being a pain [11:20] Saviq: you realize that you can land 100 MPs in one shot if the quality your engineers submit is good [11:20] didrocks, meaning that we can put several MPs into Approved, and land them all together? [11:20] Saviq: right [11:20] asac, no, I was not [11:20] Saviq: thats the point :) [11:20] as seb128 is doing with system-settings [11:21] ok well that changes things [11:21] Saviq: use the system for a bit... it ultimately is the same approach. you take a bunch of MPs, create a temp branch for everything you feel is safe to land [11:21] and land it all in one shot ... [11:21] asac, so if I need the temp branch, what's the difference between a staging branch that we merge periodically into trunk then? [11:21] Saviq: look at https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=drive_web#gid=1 [11:22] Saviq: see that there are 2 MPs listed here [11:22] the merge is done for you [11:22] so I don't need the temp branch [11:22] Saviq: you disconnect the process [11:22] larsu: ping [11:22] Saviq: the temp branch is created by the system for you (just used this to explain it) [11:23] Saviq: how about you join and get trained? [11:23] asac, I was planned to, but was off last week [11:23] right. so lets move you in, then you will see [11:24] asac, when's the next slot for training? [11:24] Saviq: guess we can make a special session for you this week? [11:24] didrocks: ? [11:24] like in 1h :)? [11:24] lol [11:25] dednick: hey, what's up? [11:26] larsu: howdy. q about the message indicator. It doesnt use telepathy does it? [11:26] tsdgeos, +Component.onCompleted: console.log("SDFDFSD", scope.id) [11:27] need that or won't work! [11:27] j/k [11:27] dednick: no, but there's talk that it should [11:27] larsu: ok. [11:27] asac, not necessary, we've got mzanetti and tsdgeos trained (although apparently there's been some miscommunication, then) [11:27] Hrm... I pulled trunk and I have failing tests for testGenericScopeView, even though they were passing last evening. [11:28] Saviq: i wasn't trained, just mzanetti [11:28] dednick: https://bugs.launchpad.net/indicator-messages/+bug/1267063/ [11:28] tsdgeos, ah ok [11:28] Ubuntu bug 1267063 in indicator-messages (Ubuntu) "Retain user-selected presence state, optimize daemons usage" [Undecided,New] [11:28] Saviq: re (sorry. had to go to buy some food) [11:28] mzanetti, don't be [11:28] asac, didrocks, we'll get the first process through and I'll join the next planned slot [11:29] larsu: cool. thanks [11:29] Saviq: we are going to do a landing with thostr_, I won't reash all the concepts, but I'm happy to prepare that with you in advance [11:30] didrocks, I'm yours [11:30] Saviq: do you have time like now for a quick hangout? [11:30] so that I explain the basic workflow [11:30] then, you will follow their first landing :) [11:31] didrocks, sure [11:32] * didrocks is going to a free room [11:36] Saviq: https://plus.google.com/hangouts/_/76cpj79bppufnts1b6cjfcq8u8 [11:43] mhr3: Cimi: in new-scopes, testGenericScopeView fails on test_hiddenPreviewOpen, test_previewCycle, test_previewOpenClose, test_showPreviewCarousel, test_show_spinner - are you guys aware of this? Is this because of undergoing preview work? [11:44] karni, the tests weren't much updated so far [11:44] mhr3: okay, was just double checking you guys aware we have failing tests. === MacSlow is now known as MacSlow|lunch [12:14] tsdgeos: is https://bugs.launchpad.net/unity8/+bug/1271676 on your radar to be fixed soon? It's quite jarring and will be obvious when giving demos at MWC [12:14] Ubuntu bug 1271676 in Unity 8 "Launching an app from the app scope causes app scope to jump to the top" [High,Triaged] [12:14] popey: it is fixed in a MR [12:14] popey: needs a test, but then i found out an existing test was broken, so i'm fixing that test first [12:14] ♥ [12:15] thanks [12:15] popey: so if you need it *now* ping saviq and we can fast track it without test [12:15] not *now* but /soon/ [12:15] otherwise wait till the end of the day and i hope i've fixed the mess [12:15] great great! [12:15] glad I asked === dandrader is now known as dandrader|afk [12:41] mzanetti: https://code.launchpad.net/~aacid/unity8/testCyclePreviewBroken/+merge/203529 [12:44] tsdgeos: hmm. fails to build === dandrader|afk is now known as dandrader [12:44] mzanetti: really? [12:45] crap... yeah. but my fault... it just printed mock preview warnings exactly before crapping out [12:45] mzanetti: i changed 2 qml files, how can it fail to build? [12:45] ... what are the odds... [12:46] mzanetti: it's not your fault really, it's my fault when i redid the previewlist to be global instead of per genericscope [12:49] tsdgeos: approved, thanks [12:50] * mzanetti goes back to the right edge animation [12:51] tsdgeos: hey, maybe you don't mind taking a look :)? https://code.launchpad.net/~unity-team/unity8/unity8-fix-carousel5/+merge/203444 [12:51] tsdgeos: and tweak for new-scopes (that's what we decided with Saviq) https://code.launchpad.net/~unity-team/unity8/new-scopes-fix-carousel5/+merge/203517 [12:52] tsdgeos: If you prefer to leave it to Michał, that's fine [12:52] he approved the first one already [12:53] looks ok to me, i guess he'll aprrove the second one later [12:53] tsdgeos: yeah, thanks :) [13:01] tsdgeos, Cimi, dandrader, elopio, kgunn, greyback, mzanetti, karni, MacSlow|lunch, dednick: please have a look at https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8 and see if it sounds sane, bear in mind it's ultimately us who decide whether answers to those questions are good enough, so if e.g. there was no design review, because it's still too difficult to get it, it's fine [13:01] * dandrader reads on [13:01] * karni looks [13:02] Saviq, how about setting an highlighted word to type that we all sync in our irc clients? so when we want to gather attention to the whole team we simply type that keyword [13:02] * mzanetti uses "all:" [13:02] Cimi, mzanetti, you decide ;) [13:02] like unityuiteam: hello team [13:03] I think it's nice [13:03] "expound" that not a word I see too often :) [13:03] causes a few false positives when someone pastes "dh_install:" error messages [13:03] or cimirocks [13:03] lol [13:03] +1 ^^ [13:03] cimirocks: standup [13:03] :D [13:03] cimirocks is actually my playstation network id :) [13:04] i like uiteam [13:04] Cimi: we used @all for hilight [13:04] greyback, I had to look it up ;D [13:04] :) [13:04] i actually use "expound" [13:04] that worked for 2 of the teams I've been on [13:04] karni, but a lot of people can say all [13:04] kgunn, yeah, you wrote it, too [13:04] Cimi: "@all" not "all" [13:04] in normal chat [13:04] karni, ok, other teams then :) [13:04] Cimi: whatever works for you. this works for u1-client and phone delivery :) [13:05] kgunn, I adapted the checklists a bit, hope the changes are fine with you: https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8?action=diff&rev2=18&rev1=16 [13:05] Saviq, "Did you perform an exploratory manual test run of your code change? " would like to have "test run" properly defined. e.g.: run unity8 on the device and play around with it, performing a couple of fundamental usecases and something that your MP affects, if any" [13:05] haha I like cimirocks [13:05] Saviq: yep, looked good...i'm for paring them down to bare minimum [13:05] karni, +1 ;) [13:05] dandrader, we have the definition in https://wiki.ubuntu.com/Process/Merges/TestPlans/Unity8 [13:06] dandrader, but it's more extensive than what we want per-MP [13:06] dandrader, per-MP we just want to apply reason === alan_g is now known as alan_g|lunch [13:06] Saviq: "First please copy & paste into the MP the checklist " <- into the description field? [13:06] Saviq: when you say "core-dev" you mean what exactly? (for the packaging change bullet point) [13:06] greyback, right, I want it to be a team [13:06] mzanetti: maybe into a comment ? [13:06] greyback, but there isn't one yet [13:06] ok [13:07] greyback, so ubuntu-unity, /me fixes [13:07] if description is weird [13:07] Saviq: description can be altered, comments not [13:07] kgunn, mzanetti I'm for description, yes [13:07] so you could change something from "no" to "yes" or add a better expound later :D [13:07] right...i did think comment let's you reply to expound [13:08] but i'm originally for description [13:09] bbiab, a.m. house duties [13:11] Saviq, not sure I got it. so "exploratory manual test" is more lax than https://wiki.ubuntu.com/Process/Merges/TestPlans/Unity8 ? [13:11] dandrader, yes [13:11] dandrader, it's more about testing your code change and any related functionality, rather than checking for regressions [13:12] Saviq: what's the plan on getting an image to test? [13:12] mzanetti, add a ppa, dist-upgrade from it [13:12] Saviq, ok. better have it written down there somewhere [13:12] mzanetti, for now, will get improved later [13:12] Saviq: is that already working or tbd? [13:12] mzanetti, works [13:12] which ppa? [13:13] mzanetti, it's a per-landing silo [13:13] mzanetti, https://launchpad.net/~ci-train-ppa-service/+archive/landing-004/ for example [13:14] mzanetti, when you create a landing, you get assigned one of them [13:14] yep [13:14] didn't know it creates a ppa we can use [13:14] mzanetti, the ppa is there, it just cleans it up and uploads new packages [13:15] mzanetti, there's 10 "silos" at the moment [13:15] yeah, I know [13:15] mzanetti, and each has a PPA [13:15] Saviq, so the reviewer also changes the MP description? [13:15] dandrader, yes [13:16] dandrader, or well [13:16] dandrader, actually maybe not [13:16] dandrader, because there can be more reviewers [13:16] Saviq, why not in a regular comment? [13:17] dandrader, just changed [13:17] dandrader, reviewers put their answers in review comments [13:18] Saviq, so merging to trunk is going to be manually done by someone and that guy will perform the TestPlan? [13:18] dandrader, no, not manually [13:18] dandrader, the "Lander" lists the MPs to be landed in https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=drive_web#gid=0 [13:19] dandrader, all the MPs are built into a corresponding PPA, the test plan is to be executed [13:19] dandrader, packages are put into archive and code merged into trunks [13:22] Saviq: You seem to be quite busy.. this one is almost the same as the one you approved, just for new-scopes: [13:23] https://code.launchpad.net/~unity-team/unity8/new-scopes-fix-carousel5/+merge/203517 [13:23] karni, yeah, will wait for the one to trunk to merge [13:23] karni, and merge trunk into new-scopes [13:23] Saviq: ach, gotcha! [13:23] :) [13:23] karni, can you please fill the answers in https://code.launchpad.net/~unity-team/unity8/unity8-fix-carousel5/+merge/203444 [13:23] sure [13:23] dandrader, fill in answers in https://code.launchpad.net/~dandrader/unity8/draghandle_lp1269022/+merge/202484 please [13:24]