[00:02] robru, plars, doanac` we are good for the test target change (no need to install camera-app-autopilot and gallery-app-autopilot) [00:03] sergiusens, according to the plan, plars should handle that. [00:03] robru, yeah, was just a general broadcast :-) [00:03] sergiusens, ok great [00:04] sergiusens, ok so i'm a little bit confused: we got the debs *out* of the seed... but where is the change that makes those click apps be preinstalled? === Ursinha is now known as Ursinha-afk [00:08] robru, lp:click-sync [00:08] robru, it's not a package though [00:08] robru, it's something run on snakefruit [00:09] sergiusens, what's snakefruit? the image builder? [00:10] robru, archive [00:10] robru, meh, it's not the archive per se [00:10] sergiusens, ah ok. well it looks like you have it under control. very well, carry on ;-) [00:11] robru, but it does have access to http://people.canonical.com/~ubuntu-archive/click_packages/click_list [00:15] * sergiusens will brb [00:53] If r224 doesn't land soon, I might not be testing it today. [00:56] ToyKeeper, r224 of what? [00:56] image 224? [01:02] robru, do you know of anyone else who can make the testing changes on ci? [01:10] Yes, image 224. Looks like it may have landed though. My phone lied to me, probably because I was messing with PPAs. [01:10] sergiusens, nope, sorry. well, maybe fginther [01:11] ToyKeeper, nope, just 223 so far. I don't think we're ready to start 224 just yet [01:15] Alright, I'm heading out for dinner, but i should be back within an hour. [01:17] Hmm, where did balloons manage to get r224? [01:17] Oh well, will check later. [01:37] sergiusens: robru: doanac`: psivaa: ok, the mp I proposed earlier for removing the pkg installs for gallery and camera, as well as om26er's unlocker change have both been merged. [01:54] plars, thanks! === Ursinha-afk is now known as Ursinha [02:15] plars, awesome [02:15] * sergiusens goes to bed [02:15] ty [02:24] kgunn, what's the scoop? are you giving up on the mir landing for now? === kalikiana_ is now known as kalikiana [02:54] robru: if you're still on...yeah...i can give up my silo....i'm gonna leave that line & MP's in place...i'll try again after we debug a little... [02:57] kgunn, "can"? no need to give it up right now if you're still working. [02:59] if you can leave a note with someone...tvoss will need me out to unlock platform-api...which i'm happy to allow him....until then, some of my guys might use the ppa [02:59] robru: ^ [03:03] kgunn: please also send an email to the ml [03:03] rsalveti: which ml ? [03:06] kgunn, was more wondering if I should expect a publish request from you anytime soon, or if I can EOD ;-) [03:06] kgunn, sure thing [03:06] robru: no way...go to bed :) [03:06] or watch tv :) [03:06] kgunn, only 7PM here ;-) [03:06] not gonna happen tonight [03:07] kgunn: ubuntu-touch/phone [03:07] kgunn: didrocks had a plan to land mir today [03:07] so please just give a heads up that it's not going to happen [03:07] so he knows what happened once he's online again [03:22] robru: the build happens at 3am utc [03:22] so yeah, cdimage should be done by now already [03:22] actually, in ~20min [05:11] morning [05:30] Mirv: morning [05:30] Mirv: it seems florian got the needed fixes for webkit [05:44] rsalveti: url or it didn't happen! [05:44] Mirv: he updated the bug [05:45] bug 1207270 [05:45] bug 1207270 in Ubuntu UI Toolkit "Font size too small in browser with Qt 5.1 / DPR rework" [Critical,In progress] https://launchpad.net/bugs/1207270 [05:45] there's an additional unity-mir change [05:46] rsalveti: oh, so qtwebkit does not need to revive the DPR patch, but unity-mir instead? [05:46] yeah [05:47] ok, great, I'll build again today and get pope_y testing too [06:11] === Building image 225 === [07:02] === Image 225 is done === [07:13] didrocks: image 225 should be available in a few minutes in system image [07:13] didrocks: probably asac explained already, but all tests failed because they were looking for a python module that's available in unity8-autopilot [07:13] rsalveti: hey, oh, what change does it contain? It was just for the infra revert, right? [07:13] yeah [07:14] that was not installed by default before either (only as a side effect of gallery-app/camera-app debs) [07:14] didrocks: just infra, but we don't know how to trigger the tests again [07:14] rsalveti: right, we still have an issue with the infra in general where no tests are run but we have 83% of tests passing :) [07:14] so we just decided to do a new image [07:14] yeah, sounds the best way :) [07:14] didrocks: right, well :-) [07:14] thanks for looking at that! [07:15] (and for the messages on the phone ML, was nice to get all updates in the morning) [07:15] the problem is that we don't know how many tests we should be running before actually running then [07:15] *them [07:15] yeah, too bad mir didn't land, more fun for today :-) [07:15] right, and also, if no test run, everything is counted as a test: [07:15] setup [07:15] systemsettle [07:15] phablet-test-run failing [07:15] systemsettle [07:16] teardown :p [07:16] phablet-test-run is clear not 20% of the global results :) [07:16] yeah [07:16] rsalveti: I'm not upset to not have Mir today TBH ;) But maybe that will be after Qt 5.2? [07:16] probably [07:17] as the last big fix for qt5.2 should be in today I guess [07:17] Mirv should know more, but florian got some additional patches for that issue [07:19] ok, anyway, let's try to ensure we get Mir soonish [07:19] it prooved the dogfooding was done seriously, so it's not a failure per say :) [07:29] rsalveti: hey. I still have input problem with DPR [07:30] yeah :-) [07:30] Mirv: hm =\ [07:30] I just updated https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1207270 - graphically it's now good, aside from the one glitch I'm mentioning there [07:30] Ubuntu bug 1207270 in Ubuntu UI Toolkit "Font size too small in browser with Qt 5.1 / DPR rework" [Critical,In progress] [07:30] Mirv: even after unity-mir? [07:30] alright [07:30] rsalveti: yes, well, it's more minor, ie unlocking is harder than before [07:30] not sure if it's worth landing to Qt 5.2 PPA or not, ie is it better now than without fixes [07:31] there are two images and a video there [07:32] right, awesome [07:33] let's try to get Kaleo to fix the remaining issue asap then === popey_ is now known as popey [08:02] didrocks: the landing-003 was freed from Mir, but prepare-silo still claims unity-mir is attached to there. is there a need to do reconfiguration of landing-003 with null, or something else? [08:02] Mirv: hum, shouldn't, is landing-003 really freed? [08:02] let me look [08:03] Mirv: who freed it? [08:03] didrocks: o/ [08:04] how did you do that? :p [08:04] didrocks: with the menu option [08:04] argh [08:04] remember, it's written "admin only", it's not what it's supposed to :) [08:04] so, this is really when the spreadsheet is out of sync with the backend [08:04] remember that the spreadsheet can't run jobs in the backend [08:04] the way to free something is: [08:04] http://162.213.34.102/job/landing-003-3-merge-clean/33/ [08:04] -> build with "only free silo" [08:04] right: "merge and clean with "ONLY_FREE_SILO" selected." [08:05] yep :) [08:05] this would have removed the line on the spreadsheet as well :) [08:05] (so, normally you don't need that menu option) [08:05] argh indeed. I'm updating my notes [08:05] and doing merge-and-clean [08:05] thanks! [08:05] sorry, and thanks to you [08:05] Mirv: do you have any suggestion on how to make that menu option really scary? :p [08:05] Mirv: no harm done ;) [08:06] it's just like, let's say, the spreadsheet is going crazy and can't sync from the backend… [08:06] didrocks: "didrocks only"? :) [08:06] Mirv: ahah, that can be :p [08:06] ok, just need to wait on the ppa to be freed [08:07] and you can take the lock! [08:07] yes, preparing some manual uploads meanwhile [08:07] (waow, they did build quite a lot of packages) [08:07] http://162.213.34.102/job/landing-003-3-merge-clean/34/console [08:07] you can see how many times people retried :p [08:07] yeah, I saw there was some middle-of-the-night efforts already the day before [08:07] yeah [08:07] but I liked the fact that they prefer to back out [08:07] and communicate [08:08] very positive experience in the end [08:08] yes, the process was working well [08:08] also, this "do something while waiting for Qt 5.2" worked well with robru [08:08] he e-mailed me on the status and also reminded to rebuild the new trunk against Qt 5.2 [08:09] for qtorganizer5-eds [08:09] excellent [08:09] Mirv: do you have a lock on the toolkit? [08:10] Mirv: as for clock app, we may have a fix to be able to promote an image [08:10] (but need more infos first) [08:10] didrocks: yes, a lock is there on the ui-toolkit, can be removed temporarily if/when needed. [08:11] Mirv: ok, let's keep an eye on that [08:31] Almost though Freenode was down [08:36] didrocks: balloons posted a new weather app at 5am (bzr 217) and asked me to test it.. I get 8 failures of 23. [08:36] http://paste.ubuntu.com/7048809/ [08:36] http://paste.ubuntu.com/7048811/ [08:37] popey: so, more than the previous version? [08:37] popey: are the changes related to test only? [08:38] not analysed, just ran and pasted so far [08:52] didrocks: aha! I think I may know why.. [08:52] * popey tests again [08:52] popey: all your fault again, I assume! :) [08:53] cjwatson: was it that there is zero way of rebuilding or removing a single arch of "successfully" built package in the PPA? I've gone through the landing-006 PPA and found 12 packages that will be a problem for proposed migration because they built for powerpc or pp64el with the old Qt [08:53] of course, my fault the phone dropped off wifi, and I have no 3g data allowance left ☻ [08:54] cjwatson: so the problem comes from that they depend on libqt5core5 instead of the new libqt5core5a [08:55] there was a time where powerpc build was not available of qtbase, and then when ppc64el was added there were some uploads made before qtbase was rebuilt for that (in those cases powerpc binary is fine but ppc64el isn't) [08:55] popey: ah, and as the tests are not isolated… :p [08:55] so either those should be handled at the proposed migration time or I'll reupload 7 packages manually and have CI Train rebuild 5 packages [08:57] didrocks: would you please point me the bug report about the Clock app issue? [08:58] sure [08:58] bzoltan: I think it was bug #1285958 [08:58] bug 1285958 in Ubuntu Clock App "Reccuring Alarm can't be set for sunday" [High,Confirmed] https://launchpad.net/bugs/1285958 [08:58] didrocks: thanks ... that is what I have too :) [08:59] bzoltan: I heard qtorganizer5-eds was half the fix [08:59] bzoltan: not sure about the other half :p [08:59] didrocks: I am looking after that one ... btw, why it is not "Critical"? [09:00] bzoltan: because the sdk team set it to high? [09:00] Zsombor Egri (zsombi) [09:00] Undecided → High [09:00] and bill set to high as well before [09:01] didrocks: if you consider a bug to be critical, please ping me in the future. I can take care of the prioritization. [09:01] bzoltan: sure, will do that. You were supposed to be in touch through Nicholas though [09:01] not sure why he didn't happen [09:01] bzoltan: as we thought first it was in the application itself [09:02] I just learnt yesterday it was in the sdk [09:02] didrocks: indeed.. FAILED (failures=2) [09:02] getting better [09:02] bzoltan: when I did pinged you [09:02] popey: better than the previous version for you, with the same setup, right? [09:02] better than the above 8 fails [09:03] will roll back and run again to compare once I have 3 runs under my belt [09:03] popey: ok, can you also try to dogfood that one like crazy, as: can we promote an image with that version? [09:03] "that version" - the one in the store? [09:03] It's not in the image yet [09:04] store has 211, I'm testing 217 [09:04] popey: the new one [09:04] ok [09:05] didrocks: as far as I know it was/is a problem what required fix from both sides. Anyhow, we are working on it now. I will keep you posted. [09:05] bzoltan: thanks a lot, please do, we'll get that landed ASAP and kick an image with it (potentially the image we'll promote) :) [09:05] the so-awaited-pre-qt5.2 image :) [09:08] didrocks: One communication mistake we made ... I was not t old and so I was not aware that the so-awaited-pre-qt5.2 image has higher priority than the also so-awaited-qt5.2 image [09:10] sil2100: didrocks: initial indication is that gallery app has some issues during the conversion to click.. camera is good [09:12] bzoltan: ah, probably, I tried to make it clear in my email, but the idea is always to decouple risk [09:12] psivaa: ok, please look at the gallery-app one :) [09:13] didrocks: yea, the tests are looking at /home/phablet/.local/share/com.ubuntu.gallery/gallery-app/database/gallery.sqlite whilst that's available in /home/phablet/autopilot/gallery_app/data/default/.database/gallery.sqlite during the click setup [09:13] didrocks: of course... I am not against it :) we just need to double handshake these priorities [09:13] psivaa: thanks for the heads up, too bad... [09:14] bzoltan: yep :) [09:14] psivaa: oh, so the tests need to be changed as well, do you think you can try ugly-patching those once the run is done to see the results? [09:14] psivaa: ah, I thought they ran the gallery-app tests before publishing and everything was ok [09:15] sil2100: I think gallery-app (debs) was still installed [09:15] I'm ready to bet all my money on that [09:15] and popey's one as well :) [09:15] can I bet popey's money as well? [09:15] didrocks: i could do that.. but i'd let the other tests to finish first [09:15] ahah ;) [09:16] didrocks: maybe, although I saw Sergio talking to Robert about wiping the system clean and installing the click ones for testing ;p [09:16] seb128: we all can, I heard [09:16] seb128: s/money/bitcoins/ [09:16] But I don't want to bet your and popey's money for that! [09:16] popey: what, you are a billionar then? :) [09:16] * sil2100 wants some dogecoins [09:16] hah. 0.00000000000000001 bitcoin millionaire.. right? [09:16] ;) [09:16] Wow wow, such currency, much worth [09:16] sil2100: yeah, but wiping won't remove the debs [09:23] cjwatson: FYI, deployed the ppcel64 list (need to improve the logic in a long term and get a couple of hours to think about it). I'll need to do the same change for arm64 [09:26] didrocks: from the logs I see they were removing the debs first and then installing the clicks, but maybe they missed up on something indeed! [09:27] sil2100: interesting… [09:30] popey: time to dress and join us! [09:56] sil2100, silo for row 40 pls? [09:56] mhr3: lookin'! [09:59] mhr3: assigned! [09:59] sil2100, ty [09:59] yw [10:01] ralsina_: hi! I'm sorry but I'm going to free up the unity-scope-click silo so that Qt 5.2 landing can continue. unity-scope-click is the only package remaining from Qt 5.2 landing [10:02] ralsina_: the silo wasn't marked as being tested, so that's why it cannot be released either. anyhow, it will be quick to reassign the silo and build after Qt 5.2 has landed. [10:04] cjwatson: FYI I've started the powerpc/ppc64el related rebuilds in landing-006. my belief is that after those are done there are no other immediate ppc issues. [10:05] because ppc64el was added late, some builds will start only when the copy will be made [10:06] didrocks: I'm upgrading my device to the latest image, I have a hunch that I know what might be wrong with gallery-app, but need latest to be sure === WebbyIT is now known as rpadovani [10:11] Mirv: I can remove them from trusty-proposed after copying, but I might be able to remove those binaries [10:11] Mirv: is it still needed or are you just rebuilding everything? [10:11] Mirv: I don't think the web UI exposes it, but you should be able to remove single builds from a PPA using the API [10:12] didrocks: thanks [10:12] cjwatson: well I'm doing the rebuilds now, shouldn't be too much of a problem. in case I missed something, some ppc mangling may be needed still but I went through the whole list and opened DEBIAN/control of the binary package to see [10:13] cjwatson: it's good to know that there's such a thing in the API. in that case I could do / find out it too. [10:13] I'm nowadays somewhat used to LP API via python :) [10:13] didrocks, I am looking at the results and feeling bad :/ [10:14] Mirv: the method is BinaryPackagePublishingHistory.requestDeletion [10:14] sergiusens: no worries ;) [10:14] sil2100, I can fix it easily though [10:15] Mirv: actually, it looks like I included PPA handling options in remove-package in lp:ubuntu-archive-tools [10:15] sergiusens: it simply seems that gallery-app has problems with the right database directory [10:15] Mirv: so you don't even need to use the API directly, you should just be able to use that script [10:15] sergiusens: oh, I was looking into that just now, but I guess you have more power over the process [10:15] sil2100, it's easier than that ;-) [10:15] Mirv: just be careful to use the -b option :-) [10:15] sergiusens: so, if only you have a moment, please ;) [10:16] sil2100, it just needs to be ignored [10:16] Mirv: sorry I was up a bit late this morning, could've saved you some builds ... [10:16] As I just now upgraded to the latest image [10:16] sergiusens: you mean, the lack of the database in the AP tests? [10:16] cjwatson: oh, nice. again, good to know. no problem, it's not the first time that PPA is seeing rebuilds after rebuilds.. [10:17] sil2100, yeah; it's a save my state so I keep my database after the tests are run [10:17] sergiusens: well, I guess you know more on how things look, so I leave the fix to you ;) Thanks! [10:19] sil2100: keep us posted :) [10:19] sergiusens: ah, what was the issue? You didn't remove the .deb packages? [10:19] sergiusens: the fix will be in the click app itself or anything else? === vrruiz_ is now known as rvr [10:21] cjwatson: Mirv would be able as well to use remove-package on the ppa, right? [10:21] ah, right [10:21] didrocks, in the test setup [10:22] didn't backlog after that line, ignore me :) [10:22] sergiusens: ok, leaving it to you then :) [10:22] sergiusens: there was some additional fun infrastructure issue related to that as well btw (unity8-autopilot not pulled anymore) [10:22] sergiusens: so, it relaunched a discussion with the QA team :) [10:22] didrocks, yeah, saw the unlock one [10:23] didrocks, I knew that wouldn't be me, but was scared anyways :-P [10:23] heh :) [10:23] sergiusens: see, any safe change always have unseen side-effects :p [10:23] (that's why we should really build an image as per airline from the ppa) [10:24] didrocks, well, I never said migrating to click was easy; not with all the infra stuff that needs to happen in parallel at least [10:24] and the test rewriting [10:24] too many non click environment assumptions exist in them [10:25] right [10:25] anyway, let's try to get everything under the same process [10:25] and dream about the future :) [10:26] didrocks, well I saw a comment from ev saying he wants to not use PPAs for anything, I guess the same thing would apply for not using branches as well [10:26] that would somehow fit in the airline [10:27] yeah [10:27] to be clear, what I said was that I want our infrastructure running the LTS as much as possible. If a non-testbed system needs additional packages, I consider that a leaky harness. If we absolutely must install packages and they're not in the LTS, we should backport them and put them in a PPA. [10:27] ev: no ppa? [10:28] I do not want us upgrading to trusty all over the place. [10:28] hum, not sure you are talking about the same thing [10:28] ev, oh, I misunderstood, so the PPA part was fine then? [10:28] building packages and testing those packages :) [10:28] that's the only place I mentioned PPAs, as far as I recall [10:28] didrocks, ignore me then ;-) [10:29] :) [10:29] back in a tick [10:30] didrocks, I was trying to get infrastructure changes under control as well ;-) [10:31] sergiusens: I'm *happily* ignoring you! (kidding :p) [10:34] sil2100, didrocks, wrt row 40, since there's a qml plugin now in mediascanner2, ppc et al builds won't finish, so even though it's saying that it's still building, it already built everything that can be built [10:35] qml plugin? uuh [10:35] mhr3: ah, so it's built on ppc in the distro? [10:35] didrocks: yes, I remember that being on ppc [10:35] didrocks, yea, it didn't have qml plugin until now [10:36] mhr3: ok, let me check the rdepends [10:36] didrocks: yes, the PPA owner can remove publications [10:37] ok, seems we don't have any rdepends from libmediascanner-2.0-0 that are on those archs [10:37] mhr3: so, I'll be able to remove from the distro [10:37] didrocks, cool, thx [10:37] cjwatson: meanwhile, you can stop the build (if the rest is published) and start testing [10:37] argh [10:37] mhr3: ^ [10:37] sorry cjwatson :) [10:38] didrocks, already tested [10:38] didrocks, do i just abort the jenkins job? [10:38] ok, let me remove the packages [10:38] mhr3: yep [10:38] k [10:38] mhr3: so, removing powerpc, ppc64el and arm64 [10:39] +1 [10:39] fortunately, this dance will be soon not needed [10:39] Mirv: ack, np. It was a matter of which thing got done first :-) [10:40] ralsina_: we normally try to discuss with you first, but timezone one and seeing the rush in Qt 5.2, wasn't easy, sorry man :) [10:41] ralsina_, i warned you yesterday :) [10:43] didrocks: much happier with 225 + new weather app. I have all the apps open now and still no crash, I'll go back to opening and closing them all a couple of times which is a likely user trait (open an app, can't remember why, close it, remember why and open it again) and see if that triggers anything [10:43] mhr3: http://paste.ubuntu.com/7049288/ -> done [10:43] mhr3: you will be able to rerun build with "watch only" if you want to get it clean (after next publisher cycle) :) [10:44] davmor2: excellent! [10:45] didrocks, will do, thx [10:50] didrocks: indicators are back to their old tricks of closing as soon as you select and item, so you can't select geoip and gps for example [10:51] davmor2: oh, this is fairly new, right? [10:52] davmor2: wasn't that one disappearing with a reboot? [10:52] didrocks: it's an old issue that was fixed and now seems to be back [10:53] I will reboot to check [10:53] need to go to mother-in-laws back in an hour [10:53] ok, see you! [11:03] sil2100, can you be my reviewer? https://code.launchpad.net/~sergiusens/gallery-app/test_fixes/+merge/209891 [11:07] sergiusens: sure ;) [11:07] Let me check [11:09] sergiusens: code looks ok, testing in a moment :) [11:11] sil2100, I was compelled to refactor; but I don't want to introduce any errors [11:19] * didrocks likes the safety approach from sergiusens :) [11:20] sergiusens: so far it seems to be working fine on my device, but I have been wondering if maybe we shouldn't remove the database/thumbnails after the tests are finished when there was nothing previously? [11:20] sergiusens: doesn't seem to be serious enough for us to care though [11:24] sil2100, shouldn't be a problem as we remove it prior to running [11:24] sil2100: ok, going out for a run, so please feel free to work with sergiusens to get those released :) [11:24] sil2100, could be considered though [11:24] bzoltan: refresh us when you get any news on the sdk fix for clock-apps [11:24] didrocks: of course [11:25] didrocks: sure ;) [11:27] sil2100, I preloaded l42 in the train to get the silo [11:27] sergiusens: ok, let's consider that some other time [11:27] Anyway, all tests pass now, yay [11:28] sergiusens: approved [11:29] Let me assign a silo [11:31] Mirv: eeek! [11:31] sergiusens: hmmm [11:31] sergiusens: come to think of it, do we need a silo for gallery-app? It's click now, right? What's the way it gets released now? [11:32] sergiusens: since I can't assign a silo because of Mirv's Qt 5.2 landing, and I don't know now if we should ask Mirv to release the lock for it for a moment or maybe we can simply release it to the click store some other way [11:34] sil2100, can I merge to trunk myself? [11:35] if I can do that; that solves the issue [11:36] davmor2: #225, first run of music app unity seemed to barf.. [11:37] sergiusens: I guess we'll have to deprovision gallery-app from CITrain anyway, let me poke Mirv just to be sure [11:37] Mirv: ping [11:37] sil2100, I think Mirv just needs to rebuild due to the qreal issue [11:38] davmor2: how do I know if this is the known unity issue? [11:38] If it's an empty merge rebuild then I guess we have green light on pushing [11:40] sil2100: pong [11:41] sil2100: so why it needs to be deprovisioned if it's click now? not that I'd otherwise have anything against it [11:45] davmor2: got white screen starting system settings too [11:47] elopio: any more luck with AP test runs / adb issues? [11:49] Mirv, sil2100 with that logic, you don't need to lock it yourself either ;-) [11:50] Mirv, I need to merge to trunk; that's where the tests come from [11:50] sergiusens: I need to lock it so that when Qt 5.2 is landed there is no package in the archives that depends on libqt5core5 which would prevent proposed migration [11:51] sergiusens: just merge it ;) [12:01] popey: working fine here [12:01] popey: is there a crash file in /var/crash? [12:01] popey: either for unity8 or qmlscene [12:02] davmor2: yes, one for unity8 [12:02] popey: okay so I'll pm you with what to do as soon as I open the file [12:05] sergiusens: so, how's it going? [12:06] sil2100, hey, missed that comment; merged and no building the click [12:07] Awesome! [12:08] popey: didrocks: I'm going to do a bootstrap install and just ensure that everything is working as smoothly from a full install rather than just an update [12:08] sil2100, this might be a problem in the future as gallery is click + deb (and so are all the other ones in bfiller's umbrella) [12:08] the core community apps don't have this problem as they don't land in distro [12:08] didrocks, popey: but for me so far big thumbs up I don't know what happened but everything has been opening fine [12:10] davmor2: mine was a clean fresh install of the previous image, updated to 225 [12:10] sergiusens: we're planning to land core apps in universe soon... [12:11] popey, hmm; that's going to be interesting [12:11] popey, why not just click? [12:11] * sergiusens wonders about contradicting goals [12:12] sergiusens: click packages don't work on desktop do they? [12:12] popey: so my updated version was working really well but it is updates with data from a couple of days ago and I don't want false hope pinned on possibly altered files :) [12:12] unless that's changed since I last spoke to cjwatson about it [12:13] not yet, I still hope to get it working for 14.04 but it'll be tight [12:13] and there's still the issue of presenting unconfined click apps being a dangerous thing to do [12:13] so actually I'll probably only get the infrastructure in place for 14.04 if that, it won't be something we'll be able to advertise [12:14] we can't really do click apps properly on X because we can't confine them properly [12:15] sergiusens: ^ [12:15] popey, they do [12:15] err they don't [12:15] well they don't work confined; true [12:15] ok, you can install them by hand, kind of [12:15] but there's no aptdaemon integration [12:16] cjwatson, I just upstart-app-launched com.ubuntu.calculator_calculator_${version} on unity7 [12:16] and getting the right click install invocation is tricky. we shouldn't be advertising them to users [12:16] right [12:16] sure, it can be done by hand [12:16] it's only sensible for developers though [12:16] reasonable way to test/develop your app, not a reasonable way to deliver it [12:17] we agreed at the client sprint that we oughtn't taint the general message of click by advertising it when it's basically a malware vector [12:17] I would just focus on them working with unity8/desktop which should be the focus next cycle anyways (unless priorities changed) [12:18] well next LTS cycle anyway, yes [12:18] right [12:20] davmor2: are you flashing clean right now? [12:21] popey: just finished now I'm transferring things onto it [12:22] popey: and without the fix weather app is crashing [12:24] davmor2: can you try this... swipe to home after putting music on it - I see no thumbnails yet - tap a song and click play, quickly [12:24] before it has a chance to get thumbnails etc [12:25] thats when I had the unity crash [12:25] will do let me get the music on first :) [12:27] yeah, you kinda need to be quick before mediascanner does its stuff [12:27] Mirv: just to double check - there is no lock required for dbus-cpp, location-service or platform-api regarding Qt5.2, right? Didn't see those on the pad ;) [12:28] sil2100: yep, those don't depend on Qt [12:28] popey: yeah I think I missed it all mine have album art (well all the ones that it thinks has album art) :) [12:30] \o/ [12:31] popey: davmor2 too: beta1 is obsolete -> if you reflash, only add landing-006 (without reflashing or manually upgrading, you keep older versions of unity8 at least) [12:31] s/upgrading/downgrading/ [12:33] Mirv: nice, does 006 have the dpi fix in play now? [12:34] popey: I'm bound to reflash some point I'll try it again as the music is transferring see if I can't trip it up that way :) [12:37] sil2100, didrocks can I get a silo for the process-cpp landing, and one for dbus-cpp, too? [12:41] popey, didrocks: weather app updated now everything works :) [12:41] davmor2: DPR fix by apt-add-repository ppa:canonical-qt5-edgers/qt5-beta-proper - I'm uncertain if to put it yet in landing, testing and after that opinions very much welcome! it turned out the graphical glitch is no more, the unlocking/input problem only one remaining and Kaleo is now awake === tvoss is now known as tovss|test === tovss|test is now known as tvoss [12:48] tvoss: I already assigned for dbus-cpp, it's building [12:48] tvoss: let me see the process-cpp one [12:48] sil2100, thanks [12:48] dbus-cpp is dead, long live dbus-go [12:48] :P [12:49] tvoss: so, line 7 is ready for silo assignment, yes? Since we didn't touch it as it's set to Ready: No ;p [12:49] sil2100, can we still get landing for components in the old "autolanding" schema (e.g not in CI train)? [12:49] didrocks: with a hunder packages the package building status check is a bit slow, maybe it could be parallized a bit some day http://162.213.34.102/job/landing-006-1-build/65/console [12:50] seb128: no, it's unwired now [12:50] didrocks, then manual uploads? ;-) [12:50] didrocks, it still autocommit to trunk so I was unsure [12:50] seb128: I don't think so, we need to sort out and find a lander [12:50] seb128: yeah, that's going to be removed next week [12:50] sil2100, referering to 35 here [12:50] didrocks, so if we have no lander we get stucked? [12:51] seb128: we need to find someone owning the component [12:51] didrocks, I guess I'm just going to wait for you to need to land the new icons for touch, that's going to mean landing ubuntu-themes one way or another :p [12:51] seb128: we can have an existing lander taking it as well [12:51] I don't want to take it [12:51] and I didn't find anyone to do it [12:51] tvoss: oh, ok [12:52] seb128: who touch this package the most? [12:52] so I guess I'm just going to wait for somebody to need theme updates enough to sort it out [12:52] tvoss: that one is set to Ready: no as well, but let me see if we can assign [12:52] didrocks, design? [12:52] tiheum [12:52] didrocks, you? ;-) [12:52] Mirv: it's only the checking, and packages are not checking once published on all archives, not really sure we need to parallelize that [12:52] didrocks, I saw your mrs for the packaging cleanups, adding a new binary, etc [12:52] seb128: I'm happy to take it if needed [12:53] thanks [12:53] didrocks: yeah, might be [12:54] tvoss: it might be a problem with assigning process-cpp, as Mirv has a lock on unity-mir right now for Qt 5.2 [12:55] sil2100, ack [12:56] brb, jumping out for lunch [13:08] Mirv: the device issues where supposed to be fixed, but now it says error: device not found :/ [13:08] I'll retry. === josepht changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: josepht | CITrain support - US: robru, cyphermox, rsalveti - EU: sil2100, Mirv, didrocks | CITrain support no answer: use mup bot after 30 minutes, but choose right timezone | Landing instructions: http://goo.gl/8H1Du3. Landing in degraded mode (see http://goo.gl/J1EqPW) [13:11] we need to stop the haptic feedback on camera snapshot button, If you zoom in and try to take a photo is blurs it all because of the vibration :) [13:12] ahah ;) [13:16] :( [13:17] elopio: just update: landing-006 PPA _only_ now [13:17] didrocks: https://bugs.launchpad.net/camera-app/+bug/1289353 only bug to date [13:17] Ubuntu bug 1289353 in camera-app "Haptic feedback on the shutter button makes for fuzzy zoomed in photos" [Undecided,New] [13:18] davmor2: ok, sounds gooddddd! :) [13:19] Mirv: it seemed to be flashing. I stopped it, updated the PPA parameters, and ran it again. [13:20] elopio: ok, let's see.. [13:20] psivaa: do you mind looking at the test results and telling if all issues are known ones/due to the crash? [13:21] didrocks: sure, just a sec. was trying to rerun gallery_app with the modified path in the test.. having some issues [13:21] psivaa: don't worry too much on that one. sil2100/serio were on it I guess :) [13:24] didrocks: the difference in 225 is that there is webbrowser and messaging app crashes and have the associated failures [13:24] psivaa: ok, d8f8? [13:24] didrocks: the other failures were flakiness/qmlscene crash and known clock app failure [13:25] didrocks: let me check that [13:25] ok ;) === pete-woods is now known as pete-woods-lunch [13:26] didrocks, popey: so after playing on this for an hour this is the only thing in /var/crash http://paste.ubuntu.com/7049915 [13:26] davmor2: not bad, so no application crash on regular application startup? [13:27] what's the desktop hook crash? [13:27] didrocks: not so far only tested the core apps post fresh flash so I'm going to install a few of the common click apps and see what happens then [13:28] not seeing such a thing on errors.u.c or LP [13:28] didrocks: but it is looking promising so far :) [13:28] davmor2: ok :) [13:28] cjwatson: I get it on every fresh install [13:28] cjwatson: yeah, I don't know how e.u.c is supposed to work and publish the data, I've several reports which never showed up [13:28] (from the phone) [13:29] oh, there is something on errors.u.c, it's just making my browser incredibly slow ... [13:29] Yeah, I have it here too [13:29] didrocks: messaging app is d8f8 [13:31] cjwatson: I wonder if it is something to do with the touch start guide as it is on all fresh installs without actually unlocking the phone [13:31] didrocks: and the same with webbrowser app crash [13:32] davmor2: no, it's a click hook [13:32] those run on session startup [13:32] davmor2: does yours contain "AppID: com.ubuntu.notes_notes_1.4.242"? [13:34] davmor2: this is the thing that updates ~/.local/share/applications/. it seems to have an appropriate .desktop file for notes here though, and the .crash file is labelled "RecoverableProblem" [13:34] cjwatson: duplicate sig for icon-path.....notes_notes_1.4.242 [13:35] cjwatson: and AppID: is indeed com.ubuntu.notes_notes_1.4.242 [13:36] it's not notes' fault, though [13:37] psivaa: I don't really mean it but "great" :) [13:37] psivaa: thanks a lot for looking! [13:37] oh, maybe it is [13:37] cjwatson: I'll leave that one with you then :) [13:37] didrocks: haha, yw :) [13:38] lunch [13:38] davmor2: right, the problem is simply that notes-app.desktop says Icon=notepad but there's no notepad icon in the package [13:39] davmor2: bug 1267882 [13:39] bug 1267882 in notes-app "notes-app, errors from click-hook, missing icon path" [High,Confirmed] https://launchpad.net/bugs/1267882 === plars is now known as plars-away [13:56] popey: care to confirm https://bugs.launchpad.net/camera-app/+bug/1289353 [13:56] Ubuntu bug 1289353 in camera-app "Haptic feedback on the shutter button makes for fuzzy zoomed in photos" [Undecided,New] [13:57] didrocks: sil2100: quick FYI http://162.213.34.102/job/landing-006-1-build/65/console seems to have gotten confused that various ppc64el builds are ongoing while those builds do not exist [13:58] at least some of the packages were uploaded before ppc64el was added to the PPA. but that didn't happen before I now fixed a couple of other ppc builds today. [14:00] Mirv: hum… let me look then, seems I was too quick on how fixing it (relying on what I've done on other archs) [14:01] popey: if you install a click app does the click store part refresh the whole scope? [14:01] Mirv: it's really really interesting, it's in the ARCHS_TO_EVENTUALLY_IGNORE list though [14:02] with: [14:02] # an arch we will ignore for publication if latest published version in dest doesn't build it [14:02] I guess they don't exist in dest [14:02] (and it was working for other components like powerpc until then) [14:03] Mirv: and seems it works for the rest, like http://162.213.34.102/job/landing-003-1-build/62/console === jasoncwarner___ is now known as jasoncwarner [14:03] I wonder if copying the source with binaries over itself would create the missin gbuild record [14:03] *missing build [14:03] I think it probably would [14:04] (sorry hangout after hangout) [14:04] cjwatson: ahah, that might explain… [14:04] I don't immediately have anything new to build though, before DPR input branch is final [14:04] cjwatson: don't you think the "no signer" as well will be an issue? [14:05] what "no signer"? [14:05] cjwatson: https://launchpad.net/~ci-train-ppa-service/+archive/landing-006/+packages [14:05] oh, on +packages? that just means it's a copy [14:05] ok [14:05] from a Debian import I presume [14:05] try: copy-package -p ci-train-ppa-service --ppa-name landing-006 -b qtsvg-opensource-src [14:06] I think [14:06] hang on a sec [14:06] didrocks, Mirv: things like ubuntu-system-settings are locked until qt5.2 lands right? (nothing urgent, we have some bugfixes/tweaks and I was pondering putting a landing ask for those) [14:06] cjwatson: sure [14:06] seb128: I think you're part of his set, let me check [14:07] seb128: yeah [14:07] popey: alarms were working in last promoted image, right? [14:08] didrocks: yep, can you try the above? [14:09] cjwatson: sure [14:09] didrocks: on a hangout atm... [14:09] it's actually supposed to be forbidden by copy-package but that check isn't working due to a bug somewhere; but that's all to the good :) [14:09] cjwatson: done [14:10] there you go, https://launchpad.net/~ci-train-ppa-service/+archive/landing-006/+build/5668902 [14:10] magic [14:10] thostr_: hi! I'll try landing your landing-001 in a moment [14:10] so you can do that for all affected packages [14:10] ah nice :) [14:10] Mirv: ^ [14:10] thanks cjwatson [14:11] morning [14:11] hey rsalveti [14:11] that's magic, thanks cjwatson :) [14:11] didrocks: hey, did our revert work in the end? [14:12] rsalveti: the Mir one? we flushed out yet, so that Mirv finish the Qt 5.2 landing [14:12] syet [14:12] -yet [14:12] ECANTTYPE [14:12] didrocks: oh, cool, but was asking about the dashboard revert [14:12] hopefully can we can fix remaining qt5.2 issues [14:13] rsalveti: oh, asac's fix was good and working for the time being :) [14:13] we had an issue due to gallery-app click conversion [14:13] (tests looking at the wrong directory) [14:13] but sergiusens fixed it [14:13] and weather-app fix is in [14:13] we are just pending on the clock-app fix (in the sdk?) from zsombi and bzoltan for now to decide if we can get an image promoted [14:14] davmor2 is collecting data on the number of crash in "normal" usage [14:14] so that we can assess [14:14] didrocks: zsombi is on the Qt5.2 standup, he will tell you the details after that [14:14] thanks bzoltan [14:15] ogra_: would have been good if you used the train for mtp btw (as there is an upstream branch) [14:16] didrocks,Mirv: so I guess you just need to run: copy-package -p ci-train-ppa-service --ppa-name landing-006 -b qttools-opensource-src libqtdbusmock unity-voice libqtdbustest libdbusmenu-qt qtserialport-opensource-src qdjango qtscript-opensource-src [14:17] we have ten ppc64el builders, so ;-) [14:19] * didrocks did to win some time for Mirv [14:19] * sergiusens reads [14:20] asac: I'm wondering what you would like to see on the qt5.2 jenkins results to be happy about releasing it. [14:20] taking the notes app as an example, their user acceptance tests are failing but not because of anything that affects a user story. [14:20] rsalveti, didrocks, the gallery fix should land in the next image build [14:20] asac: it's a combination of poorly written tests, with not using the right tool, and a bug we are hitting by a combination of those two things. [14:20] didrocks: thanks for helping [14:21] asac: if you want the dashboard to be green, it will take some time to improve the tests. [14:21] sergiusens: published in the store, right? [14:21] ogra_: daft question can't you wait adb on mtp loading? [14:21] sergiusens: didrocks: great [14:21] hey didrocks, it looks like we need to bump our Nux ABI, which should really have a package version change -- do I just go ahead and manually dch the debian/changelog or is there a better way under ci-train [14:21] davmor2, no, they are separate and can run independently [14:21] ogra_: ah okay now it makes more sense :) [14:22] bregma: just do it manually in debian/changelog. The commit message associated to it though won't be collected (as you change debian/changelog manually) [14:22] bregma: but please, don't land that for now, as it may impact the Touch image [14:22] davmor2: I can't get the alarm working even on the promoted image [14:23] nux impacting touch still? [14:23] didrocks, we're hours away from landing, I'm just trying to get our ducks in a row [14:23] bregma: ok, ensure all you dependencies are sorted out :) [14:23] sergiusens: we still install libunity-core, so nux as well [14:23] sergiusens, havent you heard ? we port from Qt to nux now .. the 5.2 landing is to painful :P [14:24] didrocks, how do I cleanly delete line 42? do you have a script? [14:24] sergiusens: it didn't land? [14:24] ogra_, what? [14:24] :) [14:24] sergiusens: I meant, no silo/package side? [14:25] didrocks: first crash by the look of it, I used OSMtouch opened a link and then closed the browser once the link had opened, crunch. Looks like the phone is just locked [14:25] didrocks, I followed sil2100's rec [14:25] davmor2: please write and report :) [14:25] aren't we having a session at the next UDS to discuss moving Unity8 from Qt to Nux? [14:25] sergiusens: we won't have any desktop .deb package landing ever? [14:25] didrocks: it's a click package right now, so we're not really using CITrain for those anymore, right? [14:26] didrocks, this doesn't affect desktop; and the deb is still needed for desktop [14:26] sergiusens: but you want your branch to be merged to trunk,right? [14:26] * sergiusens would prefer getting rid of debs sooner; but seems it won't be happening soon [14:26] bregma, what are you landing next? [14:26] didrocks, it's already merge (think of it as a translation update) [14:26] ah you did it manually [14:26] ok [14:26] ;p [14:27] so yeah, just delete the line [14:27] or put "Landed" [14:27] in column 3 [14:27] manually [14:27] it will turned green [14:27] It's a translation merge, it's translating from 'broken tests' to 'working tests' [14:27] sil2100, for the record; citrain is still necessary; debs are still required [14:27] * sergiusens wants to run aways from the process :-) [14:27] didrocks: think it might of been the frame buffer the phone seems fine just the gfx stuck and no crash [14:28] davmor2: maybe you want someone from kgunn's team to remotely debug? [14:28] sergiusens: I think I understand, but here anyway we would have problems with Mirv's Qt merge [14:28] didrocks: had to reboot to get it back up and see if it was reproducible [14:28] seb128, we're trying to prepare the lockscreen, we'll want to coordinate some landings with you for that [14:29] sil2100, yeah, it does't matter here particularly [14:29] sil2100, but I would prefer an either/or approach instead of an and [14:30] it will be restack on top of Mirv's landing [14:30] for the deb part [14:30] bregma, right, that's why I'm asking, I'm still pondering the details (Trevinho started an email discussion early today, should probably have CCed you, about changing gnome-screensaver to trigger the unity locking under Unity) [14:30] elopio: at best same results as on main dashboard [14:30] elopio: if not, we should look at the cases [14:30] and understand then a bit [14:30] asac: elopio: the only failure you should see now are the clocks and weather app ones [14:30] didrocks: and crashes randomly [14:30] no? [14:31] asac: hum, we still have crashes with 5.2? [14:31] asac: all crashes we have are the d8f8 ones [14:32] * didrocks erases user-data [14:33] seb128, as long as we're making progress: I'd like to land everything in a single silo (I've started line 43 for this) [14:33] bregma, +1 [14:33] bregma, thanks, it's about time to land that ... that's the one remaining feature right? then bug fixing? ;-) [14:33] seb128: bregma: pleae do test your landing on touch [14:34] seb128, right [14:34] it's really not the time to break it :) [14:34] bregma, btw you might want to do a landing to fix windows shrinking first, before people hunt your team down because that's driving them crazy [14:35] didrocks, yeah, don't worry, I don't think that's likely to land before qt5.2, locking is going to need integration with GNOME components and indicators and we didn't sort out those details yet [14:35] didrocks, asac: yes, I understand that all the errors not in the main dashboard are a regression caused by the qt5.2 upgrade. But if it doesn't affect the current user experience at all, I'm wondering why do we see it as critical and a blocker, instead of high priority to be fixed before april. [14:35] seb128, ack, the lockscreen may stretch into next week at the current rate [14:35] elopio: because then it's hard to know if we have another regression or if the failure is impacting even more [14:35] elopio: and we missed real regressions due to that in the past [14:36] popey, ping [14:40] tvoss: pong [14:41] popey, hey there, can you help with testing of silo 3? [14:41] popey: and my question first! :p [14:41] heh [14:41] * didrocks reflashed/bootstrapped in between [14:42] didrocks: no, because the clock app can't be updated in latest promoted [14:42] well, thats partly the problem. [14:42] popey: so, we don't have user regression on that one, just tests regression? [14:42] on clock apps [14:43] well, alarms dont work on #194, so no. [14:43] it did work in the past, right? [14:43] or am I crazy? [14:43] only in non-promoted images [14:43] ok [14:43] #194 is last promoted right? [14:43] yep [14:43] It's what I have on my main phone [14:43] and no alarm clock in the indicator when I set alarms [14:43] popey: so, there actually only test regression on clock [14:43] ok [14:43] no user-visible regression? [14:44] right [14:44] ok [14:44] didrocks: well, yes. The thing is that we are finding some of these bugs through autopilot just by coincidence. It's not the right tool for it in most of the cases. [14:44] what I'm trying to define is a process to go back to green. [14:44] If what we want is to get all the bugs fixed, then I don't have to do anything. Report the bug and wait for the devs to fix it. [14:44] If what we want is just to be green again, I can workaround the issue and make all the tests pass. [14:44] What I would really want is to refactor the tests so we leave autopilot only checking user stories, and we use lower level tools and tests to check everything else. === pete-woods-lunch is now known as pete-woods [14:45] elopio: it's not waiting for the devs, it's getting status, pinging the devs, trying to get ETAs and so on [14:45] that will leave us with autopilot tests still failing for some time. Then they will go green, but the bug will not yet be fixed. What will be failing now are the unit tests. [14:46] elopio: well, putting stuff that degrade test results don't allow us to know if there is a regression then [14:46] and we never recover [14:46] look at Mir last cycle [14:46] and maguro [14:46] we never went back to 100% [14:46] and we never knew if it was a race or a regression, loosing hundreds of hours [14:46] but yeah, agreed that some part should be unit [14:46] so [14:46] 15:44:17 elopio | If what we want is just to be green again, I can workaround the issue and make all the tests pass. [14:46] -> seems to me the best path? [14:47] ogra_: anything again building an image now? [14:47] nothing at all [14:49] ==== image #126 building ==== [14:49] didrocks: well, no. Unless we block new releases for a project based on the autopilot results on the dashboard, AND on the critical bugs reported against the project. [14:49] sil2100: popey: davmor2: this will be the promotion candidate ^ [14:50] !!! [14:50] didrocks: 3 === and surely it is 226 right? [14:50] zoiks [14:50] elopio: why critical bugs and not on autopilot results on the dashboard first? [14:50] I like the sound of that [14:50] 126 is a bit retro ☻ [14:50] this would be so much easier for me to discuss if you speak spanish, or we were at a bar drinking :) [14:50] heh :) [14:50] didrocks: because I would like autopilot to test only user stories, right? [14:51] * sil2100 didn't even notice the typo ;p [14:51] elopio: I think it's too late on the cycle for doing that reshaping [14:51] we learnt what gave "landing things with test failing" multiple times already [14:51] didrocks: I don't want my phone back in the stoneage [14:51] our current suite is testing a lot more than that, and that's why I'm having so many troubles checking the results. [14:51] davmor2: ahah :) [14:51] elopio: agreed, but I don't think that the Qt 5.2 transition will be a way to block on that [14:51] so I would propose, landing 5.2 finishing trusty [14:52] and then, we can have that as a goal [14:52] repurposing AP to AP tests [14:52] and not unit test [14:52] but not something that needs to be decided on the fly [14:53] davmor2: hmm, I can't seem to reply to a message by using the indicator - is that a known problem? [14:54] didrocks: it lasted too long weather app finally didn't open :D That's not too annoying though 2hr 30 minutes of hammering apps one mir lock up one app not opening :) [14:54] sil2100: worked here [14:55] davmor2: yeah, I won't call that "user blocker" [14:55] I will call that "popey blocker" :p [14:55] "If I do x, y, z, and then back o and p, I'm getting d" [14:55] sil2100: just retested and it worked again [14:56] Strange [14:56] For me the 'Send' button was grayed out all the time [14:56] didrocks: ToyKeeper is better, If I drag in all the 4 sides I get 6 images on the screen [14:56] dum dum [14:57] davmor2: ahah :) [14:57] Mirv: Dum dum [14:57] didrocks: I agree it's late, and its a goal for next cycle. [14:57] so what I'm asking is, now what should I do with the notes? [14:57] As you said, get a dev to dig into the issue and get it fixed, that's ok. But in the mean time, [14:57] if I workaround the bug or improve the test, we will get back to green, but as we are not blocking on critical bugs that will mean we will release with the regression. Am I right? [14:57] dum dum dum dum dum didledum and a shark [14:58] I'm trying to EOD but the builds linger still a small bit [14:58] elopio: I'm not sure to understand the case. If you improve the test, the test will fail if the behavior doesn't work anymore, and so, we'll pick regressions, right? [14:58] didrocks: no, this is the case: https://bugs.launchpad.net/notes-app/+bug/1288876 [14:58] Ubuntu bug 1288876 in notes-app "When clicking the header, the notes are not collapsed" [Undecided,New] [14:59] elopio: ok, so this is a bug which is a regression [14:59] and it's making some AP tests failing, right? [14:59] we find that not because it's part of a user story. We are getting the error just because somebody thought it was a nice idea to start all the tests clicking the header and checking that everything is collapsed. [15:00] if I were to improve the tests, I would just remove that step and turn it into a qml runner test that will not be displayed on the dashboard but block on MPs [15:00] ok, but the fact that notes are not collapsed anymore is not wanted? [15:01] didrocks: well, as a user, I don't care if the notes are collapsed or not when I click the header. [15:01] as a user, I want to add notes and read notes, and that doesn't affect the user story at all. [15:02] elopio: sure, but I'm not talking about that [15:02] what the bug probably means is that the header is not clickable when it has only one item. [15:02] like what's the expected behavior? [15:02] is it the one with Qt 5.2 [15:02] or 5.0 [15:02] and that's a problem of the header, to be tested on the toolkit, not on the notes. [15:02] right [15:02] but again, you are mixing 2 valid things :) [15:03] with makes one bigger valid things? :) [15:03] - ok, the test shouldn't test that, and should really really not due that in all test :) [15:03] s/things/thing [15:03] - but here, we are talking about 5.2 transition ONLY and we expect to have the same behavior than 5.0 [15:03] am I right? [15:03] yes, you are. [15:04] so I guess you have nothing to change in the test itself in that case (for now) and the bug/behavior change itself should be fixed [15:04] for 5.2 transition [15:04] then, yeah, decoupling the concerns will be a nice thing to do :) [15:04] ok, I can wait after 5.2 is released to improve the test. [15:04] yeah, I would really suggest that :) [15:05] treat the tests just as a non regression checker behavior-way [15:05] even if it's testing too much things [15:05] or not at the right place [15:05] now, zsombi seems to be really busy with the other blockers. [15:05] didrocks: who would you suggest to take care of this bug? [15:05] then, this work of refinement will be needed anyway [15:05] elopio: is it a sdk one? [15:06] seems so, either notes-app or sdk [15:06] didrocks: maybe. I don't know how to know. It could be the sdk, or it could be the notes app using the sdk in a way it's not supposed to. [15:06] yeah [15:06] elopio: I think the sdk guys will be the best to analyze this one [15:06] as they already started [15:06] drilling down to a small case [15:07] so bzoltan's team ^ [15:07] didrocks: do you have permissions to make the bug critical? I don't. [15:07] ogra_: if you broke my mtp, I'll hunt you down to death! ;) [15:07] elopio: oh sure [15:07] lol [15:08] didrocks: yes sir [15:08] didrocks, what do you do with mtp ? [15:08] elopio: maybe affect as well sdk? [15:08] ogra_: I don't do anything, I just want to promote an image ;) [15:08] :) [15:08] didrocks: maybe. I suppose I can add it, and they will remove it if it's not the case. [15:08] ogra_: ok, let's say "if you broke my adb…" :p [15:08] yeah :) [15:08] adb didnt change [15:08] elopio: please, and then, I'm bumping the priority [15:08] ogra_: yeah yeah… I heared that already :) [15:09] didrocks: what can I do for you? [15:09] bzoltan: so, this bug it's really not a big one affecting the user in a big way, but it could be an important regression. [15:09] https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1288876 [15:09] Ubuntu bug 1288876 in Ubuntu UI Toolkit "When clicking the header, the notes are not collapsed" [Undecided,New] [15:09] so it's blocking the 5.2 release. We need a dev to take a look at it. [15:09] elopio: why not to ask the app's dev? [15:10] bzoltan: I don't know. I'm following didrocks' suggestion. Who's the notes dev? [15:10] elopio: there could be like a million different reason for that. [15:11] elopio, talk to bfiller, ugo works on notes, but remember also it is being replaced by reminders [15:11] elopio: done [15:11] elopio: nerochiaro [15:11] yeah, between the 2 teams, you should get to something :) [15:11] bfiller: any input about that? [15:12] elopio: lets disable it [15:12] it's going away [15:13] not worth spending any time on [15:13] elopio: you don't have the same behavior on any other apps, right? [15:14] bfiller: so, yes, but the failing test is not the important part here. The important part is to check if qt5.2 is introducing a regression here that might affect something else. [15:14] sil2100: didrocks: final final update, Qt 5.2 landing is now complete. except for the Kaleo's qtubuntu + unity-mir + ubuntu-ui-toolkit branches from bug #1207270 since he's still working on the swipe input regression although otherwise they now work as of today. [15:14] bug 1207270 in Ubuntu UI Toolkit "Font size too small in browser with Qt 5.1 / DPR rework" [Critical,In progress] https://launchpad.net/bugs/1207270 [15:14] didrocks: there is no other application that I no of that has only one tab. [15:14] Mirv: so, they will ping us? [15:14] elopio: even in the click store? [15:14] this for sure doesn't affect tab switching. [15:15] Mirv: excellent, should we poke the SDK team about that one leftover issue? [15:15] that's what the tab is for. [15:15] didrocks: sorry, no other application /that I know/ [15:15] popey: do you know about any app in the store having that particular case? ^ [15:15] didrocks: I asked Kaleo to keep the bug + branches up-to-date, I believe pmcgowan & co may check the EOD situation, ie. shall we land the branches to the PPA [15:15] there's system settings, but it has no behavior when you click the tab. [15:16] Mirv: so no reconfigure will be needed, only rebuilds if anything, yes? [15:16] didrocks: what use case? [15:16] sil2100: Kaleo is the one, yes [15:16] sil2100: reconfigure needed, since those three branches are not in the config at all since before today it was pretty broken [15:16] Ah, ok [15:17] popey: something like that: https://bugs.launchpad.net/notes-app/+bug/1288876 [15:17] Ubuntu bug 1288876 in Ubuntu UI Toolkit "When clicking the header, the notes are not collapsed" [Critical,New] [15:17] Mirv: so we'll be keeping an eye on Florian then, just hope they will also ping us directly when the fixes are ready [15:17] popey: so clicking on the tab "dismiss" or "ack" an edit [15:17] sil2100: ppa:canonical-qt5-edgers/qt5-beta-proper, if used in addition to landing-006, gives Florian's branches readily compiled as they are currently [15:17] sil2100: yep, I hope so too [15:17] didrocks: hmm, not seen that, sorry. [15:18] popey: no "good" :) [15:18] hah [15:18] pmcgowan: I think we are fine if we don't know of anyone using that "feature" to break that behavior between 5.0 and 5.2? [15:18] pmcgowan: well, apart from clock apps, which is rewritten [15:18] notes* [15:19] didrocks, reading, so notes tab behavior somewhat busted with 5.2? [15:20] ah one tab [15:20] yeah I think we can whitelist this [15:20] pmcgowan: yeah, basically it seems that if you edit a note and tap on the tab to dismiss/ack the change with one tab, it doesn't end up the current transaction [15:20] pmcgowan: and popey-who-install-always-all-click-apps-from-store doesn't know of any application using that behavior [15:21] so I guess we can call that behavior change a non issue, just checking with you :) [15:21] didrocks, yeah, not likely to fix that, will deprecate note apps first, but I see the point about checking it vs sdk [15:21] yeah [15:21] ok, so elopio, seems the way to go for now is just to remove that part of the test [15:21] making sense? [15:21] thanks pmcgowan for the confirmation :) [15:22] np [15:22] didrocks: yeah, it does. [15:22] elopio: let me remove the sdk from the bug then [15:22] I mean, the whole problem shouldn't be exposed here anyway. [15:22] yeah [15:23] actually, there's something I can try to see if there's actually a bug in the sdk. [15:23] * elopio goes to code. [15:23] elopio: bug updated and assigned to you then :) [15:23] sure :) [15:25] jaja, that's what I get for talking to much, one more bug to my bag ;) [15:26] ;) [15:35] sil2100: what about silo1? doesn't seem to be published yet? [15:35] sil2100: also, can you get me a silo for line 44 [15:36] thostr_: almost done with silo 1, looking at line 44 [15:39] ping josepht [15:39] balloons: pong [15:40] josepht, can you check and see if a package is installed or missing on the core apps jenkins merge bots; it's passing tests which should be failing and we believe it's because it's missing an eds package [15:40] thostr_: silo assigned, publishing 1 now [15:40] sil2100: thanks [15:41] josepht, the package is qtorganizer5-eds [15:48] didrocks: hows 226 coming along? [15:48] davmor2: building nicely, did you finish and prepared the report on crashers? [15:49] like is it good enough, not yet, horrible? :p [15:50] didrocks: I'm yet to have one, still only got the upstart-app-launch_desktop-hook in /var/crash [15:50] davmor2: oh, so no crash even hammering the click apps? [15:51] didrocks: that's right [15:51] great ;) [15:51] ah speaking of which weather just died on me === gatox is now known as gatox_lunch [15:52] didrocks: first qmlscene crash let me confirm it is d8f8 [15:53] davmor2: yeah, then remove the crash file [15:53] to get more :) [15:56] didrocks: so I think this is the same issue popey saw earlier #0 0xb5e7b150 in ?? () [15:56] didrocks: ah no different again [15:56] davmor2: ah? [15:57] didrocks: on a plus side it was definitely the weather app that triggered it [15:57] ok ;) [15:57] keep us posted on social and other heavy-js apps [15:59] didrocks: http://paste.ubuntu.com/7050648/ === bschaefer_ is now known as bschaefer [16:01] davmor2: ok, we saw some corrupted stacks which were the same [16:01] so identical frames, let's put under the same "V8 issue" [16:07] fginther, sergiusens : is there a place where I can download the deb (amd64) for this build? http://jenkins.qa.ubuntu.com/job/thumbnailer-trusty-amd64-ci/20 [16:07] * bfiller trying to avoid buiilding it [16:07] didrocks: hey didier, the webbrowser / oa reverts, is that automatic? cause this branch still has the commit from that silo [16:07] https://code.launchpad.net/~phablet-team/webbrowser-app/trunk [16:07] rev 454 has not been reverted [16:07] oh [16:07] ==== Image #226 built ==== [16:07] davmor2: ^ [16:07] \o/ [16:07] dbarth: it will have it, once qt 5.2 lands [16:08] bfiller, it's not saved, but the jobs can be quickly changed to do so [16:08] davmor2: jump on it! [16:08] * davmor2 jumps on it and 226 breaks into a million tiny shards [16:09] :p [16:09] didrocks: installing === pmcgowan_ is now known as pmcgowan [16:09] rebootin' [16:09] fginther: don't worry about it [16:12] balloons: do you have a link to an example job? [16:12] didrocks: I'm going to do a bootstrap to give it every chance being as it is so important :) [16:12] davmor2: yeah, it completely deserves a bootstrap :) [16:12] davmor2: continue on the click apps look I guess [16:14] didrocks: so facebook, googlemaps, g+, osmtouch and twitter were all fine, as I say only crash was weather and that took nearly 3 hours of constant hammering opening reopening interacting with alll the different apps to get that to happen :) [16:14] bfiller, no artifacts are stored for that [16:14] davmor2: ok, sounds solid! [16:14] go davmor2 go davmor2 [16:14] didrocks: now everyone cross your fingers that 226 is the same [16:15] * didrocks wants to see fingers crossed here as well [16:15] (but continue working :p) [16:15] josepht, the clock-app jobs specifically [16:16] josepht, http://91.189.93.70:8080/job/generic-mediumtests-trusty/ [16:16] josepht, as part of http://91.189.93.70:8080/job/ubuntu-clock-app-ci/ [16:20] fginther: is mediumtests-slave offline for a reason? I need to get into it. [16:21] balloons: from dpkg-l.post.log it looks like it's not installed [16:21] josepht, ok, could we get it installed? I can open a bug to do so [16:22] josepht, yes, it's an old saucy image, it shouldn't be used by anything [16:22] josepht, reading the backlog [16:23] balloons: yes please open a bug [16:23] fginther: sorry I was looking at the wrong node [16:23] josepht, no worries [16:24] balloons, so the clock app tests need a dependency? [16:24] didrocks: well the guide works \o/ [16:24] davmor2: SHIP IT! [16:25] ;p [16:25] josepht, I'll have to check on this, there aren't any ssh keys setup for anyone else === gatox_lunch is now known as gatox [16:25] popey: managed to hit play on a blank piece of music, music-app took it's time to open but played in the end [16:25] fginther: ack, thanks [16:26] didrocks: could you run "copy-package -p ci-train-ppa-service --ppa-name landing-006 -b qtdeclarative-opensource-src"? Should let qttools-opensource-src build [16:27] cjwatson: sure, done. (thanks for looking, too many things at the same time) [16:27] cjwatson: I can give you access to the ppas if you need, but you will get a lot of more spam :) [16:27] no thanks :-) [16:27] balloons, do you know if that's going to wreck other tests? that's the problem we had earlier... And if so, is there a reason the clock-app tests can't specify this as a dependency? [16:28] balloons, and I can confirm it's not currently installed [16:28] didrocks: I'll try to check through for other things. I can use my buildd-admin privileges to retry stuff [16:28] cjwatson: ok, as you wish :) [16:28] didrocks: qtpim-opensource-src too [16:29] done [16:31] didrocks: qtwebkit-examples-opensource-src qtwebkit-opensource-src qtx11extras-opensource-src qtxmlpatterns-opensource-src [16:31] sorry, will hopefully be done soon, just trying to chase that jenkins failure ... [16:32] cjwatson: no worry, rather thanks you for looking into that :) [16:33] fginther, yea, I'll have to dig into the clock dependencies.. if it's specified it should be getting installed eh [16:44] fginther, so I suppose I would argue since it's in the images, it should be in the test environment. That said, it'll be added for clock as a dependency [16:49] balloons, I guess that is a weird thing, but I think it's the right answer as long as we're using apt. I assume this goes away when we start installing and testing these as click packages (although I admit to not knowing how the mechanics of that works) [16:49] fginther, yea, that would be even closer to the real thing :) === kalikiana_ is now known as kalikiana [17:01] grrr google [17:01] ogra_: davmor2: sil2100: joining? [17:01] Ah! [17:01] ogra_: it's because the mpt change! :) [17:02] mtp* [17:02] so let me upload a revert then if google will start working [17:02] even [17:02] ahah [17:02] :P [17:02] damn [17:02] plars-away: really away? :p [17:02] ev: anyone else from your team to join the "kick the image" side? [17:03] didrocks, I'm considering myself off the hook with http://ci.ubuntu.com/smokeng/trusty/touch/flo/226:20140307.2:20140304/7039/gallery_app/ [17:04] didrocks: hm? I'm a bit heads down on Mt Hood. Do you need people to help with something? === josepht changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cihelp | CITrain support - US: robru, cyphermox, rsalveti - EU: sil2100, Mirv, didrocks | CITrain support no answer: use mup bot after 30 minutes, but choose right timezone | Landing instructions: http://goo.gl/8H1Du3. Landing in degraded mode (see http://goo.gl/J1EqPW) [17:05] ev: just ensure that someone is looking/rerunning image #226 for CI test results [17:05] if needed [17:05] popey: have a look as the camera app icon on the home page and the apps page do they look the same? [17:05] sil2100, ping [17:06] davmor2: no [17:07] psivaa: ^ can you have a look? [17:07] tvoss: pong, did you see that dbus-cpp and others successfully built on the PPA? [17:07] tvoss: on the first run! [17:07] ev: ack, will do [17:08] psivaa: thanks! [17:08] sil2100, yup, pete-woods and Wellark are helping with the testing [17:08] sil2100, so do you need any further information on the merge proposal or are we good? [17:09] that is, mps for location service [17:09] ev: yw [17:11] tvoss: I think we're cool - could you just double-check if the list of merges in the landing is correct and has all the necessary ones? === retoaded changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: retoaded | CITrain support - US: robru, cyphermox, rsalveti - EU: sil2100, Mirv, didrocks | CITrain support no answer: use mup bot after 30 minutes, but choose right timezone | Landing instructions: http://goo.gl/8H1Du3. Landing in degraded mode (see http://goo.gl/J1EqPW) [17:15] davmor2, can you send along the pastebin again? did you file a bug? and what version of weather? [17:17] balloons: http://paste.ubuntu.com/7050648/ [17:21] sil2100, ack [17:21] davmor2, what version did you run? ohh I suppose it was old as r226 contains new weather [17:22] davmor2, I just assume you hit the bug which was fixed [17:22] balloons: the updated version on 225 [17:22] davmor2, how did you install it ? [17:22] balloons: however I got the same error on 226 which it the new one [17:23] balloons: from click [17:23] davmor2, blah, fine [17:23] davmor2, bug? [17:23] balloons: not yet busy busy testing everything [17:25] sil2100, list of mps look good [17:25] pete-woods, Wellark are you guys done with the testing? [17:26] tvoss: I'm happy it works [17:26] pete-woods, ack [17:26] Wellark, ? [17:26] tvoss: I think antti is running the Unity AP tests right now [17:26] popey: did you file a bug for the camera getting the line traveling across it [17:26] pete-woods, ack [17:26] he's AFK, though [17:28] sil2100, you happy with pete's feedback? [17:28] mhr3_, there's a dependency problem with your silo 2 landing [17:28] mhr3_, http://162.213.34.102/job/landing-002-2-publish/lastSuccessfulBuild/artifact/packaging_changes_mediascanner2_0.100+14.04.20140307-0ubuntu1.diff [17:29] mhr3_, the new package depends on "mediascanner (= ${binary:Version})" but that's wrong, should probably be "mediascanner2.0 (= ${binary:Version})" [17:29] popey: it looks like if you zoom in any and then click to focus on something then take a shot it starts to do the scrolling line [17:29] robru, indeed [17:30] mhr3_, can you add a commit and rebuild? then I can publish it [17:30] robru, can't, not my branch [17:31] robru, can only create new mp [17:31] mhr3_, bah. ok, please do, then I can reconfigure for you [17:31] sil2100, ? === tvoss is now known as tvoss|test === tvoss|test is now known as tvoss [17:33] urgh, qtdeclarative/ppc64el is going to need the usual patch we applied to a bunch of places [17:34] robru, so just this http://paste.ubuntu.com/7051141/ right? [17:34] mhr3_, close, it needs to be '2.0' to match the binary package name [17:35] robru, ah, right, good that i checked [17:35] ;-) [17:35] brb [17:38] robru, added mp, pls reconf and feel free to build too, i will eow in a bit [17:38] mhr3_, ok no worries. [17:39] robru: reusing the new reconf one? :) [17:39] didrocks, yes, it's working! ;-) [17:39] \o/ [17:39] phew ;) [17:39] didrocks, thanks for that, it's pretty slick. [17:39] robru: great! glad that you like it. Will be officially announced next week and opened to everyone :) [17:40] didrocks, ahhh I was wondering when. excellent! [17:40] robru: just be aware that if you add a new component that isn't in the list, it will yell and force you to use prepare-silo [17:40] as it's the goal of it :) [17:40] didrocks, perfect [17:40] * mhr3_ waves [17:40] see you mhr3_! [17:40] mhr3_, bye [17:41] enjoy weekend didrocks, robru! [17:41] didrocks, sil2100 do we see some u8 crashes on the current image? === bfiller is now known as bfiller_afk [17:42] tvoss: we do [17:42] u8 + random apps [17:42] didrocks, hmmm ... wellark is reporting 5 crashes when running the u8 ap test suite [17:43] yeah [17:43] we can have less or more [17:43] didrocks, is that in the baseline image, too? [17:43] robru, crap, forgot to set prereq branch for the mp, although i guess it should work without that as well [17:43] well, you can check as well :) [17:43] and see all my emails on the phone ML [17:43] but yeah, 5 can happen [17:43] you can check them [17:43] didrocks, ack [17:43] the address is finishing by d8f8 [17:43] (from frame 0) [17:43] tvoss: so easy to check ^ [17:44] didrocks, that's the v8 issue, correct? [17:44] tvoss, pete-woods: ping [17:44] tvoss: yep [17:44] Wellark, see backlog [17:44] didrocks, I'm seeing 2 crashes locally [17:44] mhr3_, should be fine since they're in the right order. it's already building [17:44] tvoss: ack [17:45] so unity8 ap tests are flaky [17:45] Wellark, could you check for the address in the crashes? [17:45] how do I retrace them manually+ [17:45] ? [17:45] Wellark, hmmm, might even be enough to look at the crash file. didrocks^? [17:45] yeah [17:45] just unpack the crash file [17:45] gdb the coredump [17:46] and look at frame0 [17:46] no need for symbols [17:46] as we just need the address [17:46] didrocks, thanks [17:46] yw :) [17:48] didrocks: how do I "unpack" the .crash file? [17:48] Wellark: apport-unpack /tmp/foo [17:48] and then, you will have the CoreDump in /tmp/foo [17:49] didrocks: ack [17:50] davmor2, I can't recreate your crash in weather. when you have a chance, more info please [17:52] hum [17:52] doanac`: psivaa: anything change in the way you install deps? http://ci.ubuntu.com/smokeng/trusty/touch/manta/226:20140307.2:20140304/7038/messaging_app/ [17:53] it's like if ofono-phonesimd isn't installed [17:53] asac: FYI ^ [17:53] (don't have the console log for now) [17:53] yeha [17:53] i noticed that too [17:53] doanac`: psivaa: at worse, once the tests are finished, can you ssh, install the dep (ofono-phonesimd) if not there and try? [17:54] didrocks: this is on manta though.. i'll try once the tests complete [17:54] psivaa: yeah, let's wait on mako, it's running as we speak it seems :) [17:54] manta also has no working BT yet [17:54] so the Bt indicator test will always fail [17:55] psivaa: ogra_: you're right, they were as well failing on manta [17:55] weird that it has this side-effect (it's failing on the setup) [17:55] ok, ignore for now, let's really focus on mako [17:55] current results look promising :) [17:56] well, the craches get populated at the end :) [17:56] Now this is interesting, after meddling with my device I now have a landscape keyboard when in portrait and the other-way around ;) [17:56] Useful feature [17:57] thats dual user mode [17:57] how long is this qt5.2 "no change rebuild" going to take? [17:57] ogra_: :D [17:57] bringing people together and such :) [17:59] dobey: it's not a non change rebuild exactly, people are working on latest bugs, early next week we hope [18:00] didrocks: we have branches piling up for the click scope, and the silo we had for a few of them got wiped away because of the rebuild thing [18:01] dobey: well, your lander was contacted first [18:01] dobey: but if you can help getting the 5.2 Qt things out, your help is welcome :) [18:02] what qt 5.2 things? [18:02] Wellark, any insight? [18:02] dobey: bugs/regressions [18:02] tvoss: still running the ap tests [18:03] Wellark, ack [18:03] tvoss: tvoss seems I got one crash at least [18:03] ok, checking now [18:03] didrocks: so i should just ignore the "no change rebuild" MPs from Mirv and ask to get the other branches landed? [18:04] Wellark, okay [18:04] dobey: no, they have a lock on the component, don't throw their testing away [18:04] dobey: check with your landers, he knows how that work [18:04] even on things that are not on the "ci train"? [18:05] didrocks: messaging on mako came back all passing [18:05] dobey: you can land anything that are not locked by the Qt 5.2 (through the ci train) [18:05] psivaa: thanks \o/ [18:06] :) [18:08] didrocks, tvoss: the adress of frame 0 is 0x3cb0d8f8 [18:08] Wellark: so yeah, the V8 crash [18:08] didrocks: forgot asking this during the meeting - did you get any news from Kaleo regarding that last-standing 5.2 bug? [18:08] ok. ignoring them then [18:08] The one Mirv mentioned [18:08] Wellark, perfect, thank you [18:08] sil2100: just that he's working on it [18:09] no more news [18:09] sil2100, ^, so pete-woods, Wellark and me tested silo 3 [18:09] sil2100, as far as we can tell, things work as good as before [18:09] tvoss: no suprises, no CPU eatage and location-service working normally? [18:09] sil2100, yup. Wellark, pete-woods could you guys please check top, too? [18:10] Wellark, pete-woods, tvoss: just to make sure the 100% CPU is really not an issue ever again ;) [18:11] sil2100, yup, but that was caused by gcc intermixes back then :) [18:11] anyway, better safe than sorry [18:11] didrocks: you think we could also publish this one today? Me or Robert could also double check if it's not causing any regressions, but as long as it's not eating up 100% CPU or something similar, it shouldn't even interfere in existing integration tests [18:11] sil2100: yeah, can you double check to ensure before landing? then, it will be fine for me IMHO [18:12] anyway, WE are going to promote #226 isn't it? [18:12] isn't it? :) [18:12] didrocks: the components to be released are dbus-cpp, location-service and platform-api (p-api has only a build-dep bump, so a rebuild with latest dbus-cpp) [18:12] Ah, right ;) [18:13] sil2100, would appreciate you giving the silo a spin, too [18:13] tvoss, sil2100, pete-woods: np. we will run the tests again and this time look at top aelo [18:13] seriously speaking. didn't notice any 100% cpu usage so we should be all fine [18:14] tvoss: sure thing, upgrading in a moment and spinning [18:14] sil2100: tvoss: Wellark: we are as well usually trying to get lock up from the browser from a location webpage, might be worth looking at it [18:14] (and it's not winter anymore, so not cold to have the window opened for 10 minutes!) :) [18:15] tvoss: in the meantime, as you're the official lander, could you set the landing 003 to 'Tested: Yes' on the spreadsheet? ;) [18:18] didrocks: is there a bug filed in LP? [18:18] balloons: I opened it, added a city, change the F to C [18:19] I've done all that and more.. you can reproduce consistently? [18:19] and of course the tests do the same [18:19] balloons: then I changed the backend once that was working and crash, then I opened it back up changed back to the other backend and crash again [18:19] didrocks, pete-woods tried to get a fix in the browser and managed to do so after the usual 10 minutes :) [18:19] tvoss: ah, excellent and top wasn't crazy after that? :) [18:19] balloons: pass but have you checked the /var/crash dir? [18:20] balloons: for the second crash everything worked but the crash still appeared [18:20] didrocks, not for me, but it might be a 100% [18:20] davmor2, I can.. but I don't visually see any issues [18:20] didrocks, that's well known :) [18:21] sil2100, didrocks setting it to tested [18:21] great ;) [18:21] sil2100: missing a submitter checklist on this one, please add it [18:21] davmor2, ahh.. it's the re-opening that might be what you mean [18:21] sil2100: https://code.launchpad.net/~phablet-team/location-service/bump_deps/+merge/208169 [18:21] balloons: while I am testing I run adb shell and then watch -d ls /var/crash [18:21] tvoss: mising a submitter checklist on this one, please add it [18:21] tvoss: https://code.launchpad.net/~thomas-voss/location-service/switch-to-transact-method/+merge/209451 [18:22] balloons: that highlights everytime a file ends up in there [18:22] davmor2, I think I finally reproduced [18:22] tvoss, sil2100: https://wiki.ubuntu.com/Process/Merges/Checklists/location-service [18:22] balloons: \o/ [18:22] davmor2, thanks for the explainations [18:22] balloons: no worries [18:22] balloons: it is a lot more stable than it was :) [18:23] Wellark: will add, thanks ;) [18:24] sil2100: we are ready to land the MR, so only thing blocking is the checklist [18:24] and me and pete are way over EOD :) [18:25] Wellark: that MR will only land when we click the publish button anyways ;) [18:25] sil2100, do you add the checklist? [18:25] tvoss: to his own MR, you still need to do yours :) [18:25] it's a small one [18:26] just copy and paste [18:27] davmor2: balloons: don't break the images guys, and keep us posted! [18:27] * didrocks really EOW now :) [18:28] didrocks: nooooooooooo [18:28] davmor2: what's up? last minute crisis? :p [18:28] tvoss: added [18:29] didrocks: No I just don't think you should go :P [18:29] tvoss: ok, by some dogfooding I see it working pretty ok [18:29] didrocks: Keep your eyes out for the email, It will be long [18:29] Didn't get a GPS signal yet though [18:29] Ok, need to drive to pick up my girlfriend, be back later to do the final publish [18:29] sil2100: takes about 15 minutes [18:29] davmor2: sure ;) (my wife disagrees with you and thnk I should go and help painting the kitcken :p) [18:29] tvoss, Wellark, pete-woods: give me a final +1 if all is ok and I'll publish once I'm back ;) [18:30] didrocks: go man go, life won't be worth living else ;) [18:30] heh, see you guys! [18:31] enjoy [18:31] Could somebody with ci-train-ppa-service access please run "copy-package -p ci-train-ppa-service --ppa-name landing-006 -b qtsystems-opensource-src" to create the ppc64el build? [18:31] (from lp:ubuntu-archive-tools) [18:31] sil2100, Wellark pete-woods adjusted https://code.launchpad.net/~thomas-voss/location-service/switch-to-transact-method/+merge/209451 [18:36] davmor2, I lied.. the crash was mediascanner [18:36] weather is fine :-) [18:36] balloons: haha [18:37] Wellark, anything to add? [18:37] tvoss: nope, looks good. [18:37] Wellark, ack [18:37] pete-woods, ? [18:37] tvoss: now, only waiting pete-woods to approve the last two [18:38] tvoss: done [18:38] then sil2100 or robru need to switch testing to green on the landing sheet [18:38] Wellark, what's tested? [18:38] silo 3 [18:38] robru: silo 003 [18:39] we would like to see that landed please :) [18:39] tvoss: ^ [18:40] Wellark, ok, I can publish it soon, I just have to poke at it a bit [18:40] robru: please continue double cehcking etc. [18:40] until we have shot qt5.2 [18:40] asac, yes [18:41] we want to preserve ability to do a firedrill fix on monday after we promoted the image monday morning [18:41] thanks [18:41] robru: can you help cjwatson ? [18:41] or was he already helped? [18:42] asac, oh i dunno. what did he need? [18:42] 18:31 Could somebody with ci-train-ppa-service access please run "copy-package -p ci-train-ppa-service --ppa-name landing-006 -b qtsystems-opensource-src" to create the ppc64el build? [18:42] cjwatson: i think we should give you access to such things [18:42] 18:31 (from lp:ubuntu-archive-tools) [18:42] asac: I didn't want the mailspam :) [18:42] asac: it's a transient problem, new uploads won't be affected, it's just for things that happened before we switched on ppc64el, so just trying to clean things up a bit [18:42] sil2100, so pete-woods, Wellark and me are +1 [18:42] cjwatson: what team is it? [18:42] ci-train-ppa-service [18:43] I'd actually rather not be able to affect that directly [18:43] cjwatson: yeah, but in general i feel saver if we have our strong core-devs able to do something in case there isa big problem [18:43] hmm [18:43] cjwatson: your call [18:43] i don't think I have that access. [18:43] it's not urgent, anyway, I just noticed its build-deps aren't satisfiable in any case, so drop it [18:43] cjwatson: let me see. maybe i can give you access and remove after [18:43] cjwatson: yes, i own that team [18:43] let me know [18:43] https://launchpad.net/~ci-train-ppa-service [18:43] it'll need https://code.launchpad.net/~cjwatson/kubuntu-packaging/qtdeclarative-porting/+merge/209981 first, don't worry about it [18:43] ok [18:44] cjwatson: seems quite a few already are in that team [18:44] think all CI-trained core devs are addded there [18:44] yeah, I haven't had training [18:44] probably should at some point but it's not the most urgent thing [18:45] stgraber ought to be there if that's the condition, though? [18:45] robru: anything you need from us anymore? or can it be landed now without any additional input from me, pete-woods or tvoss ? [18:45] but probably best ask him :) [18:46] Wellark, ideally tvoss should be around in 2-3 hours to merge & clean after the publish, but if he wants to EOW, I can take care of it. [18:46] Wellark, but only tvoss, everybody else can go if they're done [18:46] robru: so if tvoss needs to EOW, you can still take care of everything? OK. good. [18:46] Wellark, yep [18:46] :D [18:46] great! :) [18:47] thanks. [18:47] Wellark, pete-woods have a good weekend! [18:47] robru: you, too [18:47] robru: you too! [18:47] ogra_: since you might get tricked into believing that we promtoe the image when jfunk gives a +1 [18:47] ogra_: please dont do that until monday morning [18:48] try to get up early though [18:48] we dont want to end up having people work weekend in unlikely case there is something that we need to react on [18:48] robru, I really have to eow [18:48] tvoss: man, go off [18:48] robru, would appreciate your help [18:48] asac, i wouldnt do any promotion without either your or didrocks approval anyway [18:48] thanks [18:49] asac, thanks :) [18:49] yeah. just saying because jfunk will send an official mail [18:49] * tvoss waves goodbye [18:49] right [18:49] tvoss: good luck and good weekend! [18:49] * asac hugs tvoss [18:49] tvoss, i'm on it, have a good weekend! [18:49] * Wellark gives tvoss a man-hug [19:00] tvoss: and good luck as well :-) [19:08] sergiusens, https://code.launchpad.net/~sergiusens/phablet-tools/emu_prov/+merge/207440 works fine. I asked for the option to not do the setup if I wish [19:12] balloons, #2 is an option I use myself :-) [19:13] sergiusens, well then, it must exist! [19:15] balloons, all these options make me think it's time to move aways from a script though :-) [19:16] well, yes I know it makes things feel more and more cobbled together [19:18] still making the change [19:19] balloons: https://bugs.launchpad.net/ubuntu-weather-app/+bug/1289536 [19:19] Ubuntu bug 1289536 in Ubuntu Weather App "Crash opening weather app and switching backends" [Undecided,New] [19:19] davmor2, you know I still don't believe you :-) [19:21] davmor2, I did find this, but.. https://bugs.launchpad.net/ubuntu-weather-app/+bug/1287982 [19:21] Ubuntu bug 1287982 in Ubuntu Weather App "Refreshing when no weather data is found crashes the app" [Undecided,Incomplete] [19:21] no one believed me :-) [19:21] rsalveti, can you ack this? https://code.launchpad.net/~nskaggs/phablet-tools/add-cmake-dependency/+merge/204269 [19:24] robru: can I get a silo for line 45? [19:24] let me check [19:26] sergiusens: why recommends in this case ^? [19:26] from the description it needs cmake to function properly [19:26] shouldn't it be depends then? [19:27] fginther, hey is the new unlocker not being used in production anymore ? [19:33] rsalveti, probably right; I'm splitting that package into tiny bits today as well [19:34] om26er, it's in use, what's not working? [19:35] fginther, just wanted to make sure, i seem to have noted something in didier's email, so was under the impression the script was reverted [19:35] sergiusens, rsalveti so put it back into depends then? It's a trivial thing, but tools fail for new users if we don't install things we need :-) [19:37] sergiusens: how to proceed then? [19:37] rsalveti, sure [19:37] rsalveti, balloons, yeah, or do like the devscripts where mk-build-deps asks for equivs to be installed [19:37] but doesn't make the whole package depend on it [19:38] rsalveti, ok, you got silo 1. please build [19:38] robru: thanks [19:38] rsalveti, you're welcome [19:44] sergiusens, not sure how to setup mk-build-deps to ask for equivs [19:47] * robru --> lunch [19:51] balloons, no worries, assign the problem to me and I'll get it proposed [19:56] sergiusens, done. https://bugs.launchpad.net/ubuntu/+source/phablet-tools/+bug/1289558 [19:56] Ubuntu bug 1289558 in phablet-tools (Ubuntu) "Click-buddy requires cmake, but it's not specified in dependencies" [Undecided,New] [20:03] davmor2: http://goo.gl/l5i9SV is not public [20:03] pmcgowan, ogra_, asac, didrocks(sciving off painting kitchens), popey please may I draw your attention to the phone mailing list there is a nicer breakdown of the email I sent and a big thumbs up from qa enjoy :) [20:03] ogra_: should we then promote 226? :-) [20:03] rsalveti: sorry jfunk set it up [20:05] rsalveti, I believe the plan is first thing monday to avoid weekend danger [20:05] pmcgowan: yeah :-) [20:05] not a good thing to do on a friday evening [20:05] davmor2, was that terminal font thing a known issue? [20:06] pmcgowan: yeap it's happened on and off for a while, it's my EOD and EOW so Monday first things I'll add the bugs and file any new ones [20:06] davmor2, ok asac and I were just wondering [20:06] pmcgowan: all you have to do to fix it is set a font size [20:07] davmor2, slacker not working the weekend? [20:08] davmor2: oh sorry. sent another mai. ill check that new mail [20:08] pmcgowan: I know right, some people have wives that insist on them not going near computers all wekend cause they'll wind up working :) [20:09] pmcgowan: and She is lovely, but scares me more than you guys ;) [20:09] davmor2, understood [20:10] asac: it has minor niggles and the weather app is flakey but mostly only the weather app so we are happy big thumbs up [20:10] rsalveti: the doc should be public now too [20:11] asac: and popey is happy too so :) [20:11] davmor2: thanks [20:13] asac: is that good enough for you? [20:16] right guys I'm off enjoy your weekends [20:17] davmor2: i dont see that mail :/ [20:18] davmor2: is that hidden within a thread? [20:19] pmcgowan: do you see the mail by davmor2 to phone ML? [20:19] asac, no I see it [20:19] image r226 Testing Results [20:19] pmcgowan: on the public ubuntu-phone list? [20:20] * asac goes to archive [20:20] it is in archive, not in my inbox [20:20] guess google is again trying to slow us down :P [20:21] asac, stop using tbird ;) [20:21] davmor2: what is missing is what got fixed that was bad in last image [20:21] davmor2: like "click apps didnt get updates" [20:21] and making a tablet to put them against each other. [20:21] anyway, i wont bother [20:21] goes into micro managing :P [20:21] * asac still feels jfunk should have sent a big message [20:22] click scope being busted [20:22] is scary [20:22] and we need to know if it existed in previous build [20:22] otherwise we dont know [20:22] really [20:22] asac, its not busted its just wonky, has been for quite a while [20:22] so yeah the comparison wasnt done [20:22] not sure about 194 [20:22] right, but i wanted a comparison as we make calls based on last image vs. this [20:22] the fact that they say we dont know is scary [20:22] i asked for that explicitely for days [20:23] well I dont know, dav probably does [20:23] well, he marked it as "New breakage, needs to be checked if it existed in previous build" [20:23] pmcgowan: see https://docs.google.com/document/d/1UB-bk31kLZMlsDFT81LWoFg1tZt_lLeKcRVLIExrAi8/edit [20:23] the RED with white font [20:23] so thats a regression [20:24] he said needs investigation [20:26] asac, I had asked about that very issue, and I think it was ralsina_ that said its expected the way the scope works, but at some point needs to be improved for sure [20:26] pmcgowan: well, that i dont really care about. i want to know was it in previou build or not. if it was not in previous build i want a mail from jfunk [20:26] or maybe it was saviq [20:26] putting the new regressions in comparison to the old bugs [20:26] and making a good argument [20:26] we can also decide that here [20:27] pmcgowan: what issue? [20:27] and probably will, but i feel that QA should be able to make tha tcall [20:27] if its not a clear black and white decision anymore [20:27] sure nt disagreeing [20:27] but having a good assessment and describing symptoms, arguing why and so on :) [20:27] we should learn how to do that [20:27] yeah. i will continue to ask him fo rthat [20:27] personally would really like to understande what this really is :) [20:28] ralsina_: click scope is buggy in latest image [20:28] ralsina_: see davmors mail to touch mailing list... [20:28] ralsina_: err here: https://docs.google.com/document/d/1UB-bk31kLZMlsDFT81LWoFg1tZt_lLeKcRVLIExrAi8/edit [20:28] ralsina_: Glitch after each app install it removes the section from the scope, flashes and then returns the updated list. [20:28] in section Click packages [20:28] asac: right [20:29] asac, it needs to refresh the entire sections to remove the app from available and put it in installed [20:29] asac: all the scope does is invalidate the results because one result has moved from one section to the other [20:29] there is no api for moving it [20:29] exactly [20:29] sorry didnt see you there [20:29] ralsina_, so did this change recently? [20:29] maybe the refreshing in the dash can be made smarter or happen before the preview is closed, or something [20:29] ralsina_, seems more intrusive in the latest [20:29] "it has to refresh"? ... what does the user experience exatly? [20:30] pmcgowan: no, we had the call to invalidate there, and then the dash started actually invalidating. No change in the scope there like for a month [20:30] i dont want to know what the fix is, but what the experience is right now :) [20:30] it clears the page and repaints it [20:30] asac: otherwise the installed app will be listed in the store section, not the installed section [20:30] ralsina_, I think part of the issue is the user sees the scope, then it blanks and refeshes [20:31] so just some flickering? [20:31] ralsina_, would be good if it was already blanked and simply restores for example [20:31] asac, yes [20:31] but no user impact? [20:31] pmcgowan: right, so things could be made smoother by changing order of things, to avoid flickering [20:31] sure? [20:31] yes [20:31] I do it all the time [20:31] I logged that bug ;) [20:31] waht ralsina says above sound more like the app continues to stay in the wrong section (e.g. store, not installed) [20:31] no it moves [20:31] just flashy [20:31] pmcgowan: you do what? if its just flickering then you dont need to do anything :) [20:32] asac: it would stay in the wrong section if we did not invalidate the results (whcih then causes the flashing) [20:32] when you install an app [20:32] ralsina_: ok but we ALWAYS invalidte after install? [20:32] asac: yes, we have to [20:32] coming back from the app install view, it shows the scope, then clears then paints it with the app in the right place [20:32] pmcgowan: how long is that flickering action in seconds? [20:32] 0.1? 1, 10? :) [20:33] < 1 [20:33] yeah <1 [20:33] depends on network tho [20:33] we could refresh the old results and make it take no time, but since the scope has to be stateless we can't. [20:35] just did one and around .1 secs, very quick [20:35] doesnt sound critical [20:35] we should track it though [20:35] ralsina_, that transition could be improved [20:35] agreed [20:35] its not nice [20:35] pmcgowan: +1 yes [20:36] we can make the rick test... will he complain :)? [20:36] pmcgowan: the dash could be smarter, get the new results and then do something pretty [20:36] ralsina_, has it always been this way? [20:36] is it worse now for some reason? [20:36] pmcgowan: since the dash started doing invalidation it has always been the same [20:37] ah so before you would not see the new app without searching [20:37] the other way to decide if it would block is to check if we would have catched this as part of our normal image validation procedure. [20:37] asac, now I remember [20:37] asac, so it changed, but to fix the bug that the new apps were not shown [20:37] at some point [20:38] if neither rick-will-start-buggin-devs and that test are negative its good for promotion :P [20:38] pmcgowan: can we at least upgrade click apps now? [20:38] :) [20:38] we can witht he upgrade app still [20:38] not yet in settings [20:39] so even that wasnt fixed yet? wow [20:39] probably backed up on CI [20:39] so no improvement over 194 [20:39] its fine, run the app upgrader [20:40] pmcgowan, it was worse before; installing a new app kept the old ref and the app was unlaunchable until a search was triggered [20:40] sergiusens, exactly [20:40] yep [20:41] pmcgowan: the doc says in "settings" section: updates work [20:41] just auto download option is ignored [20:41] pmcgowan: are you disagreeing with that? [20:41] asac: maybe that means system image updates [20:41] this was the biggest issue we had identified in 194 [20:41] the click scope is weird, it's the only scope where using the search result changes the search results :-P [20:41] it was tracked by CI, then was claimed fixed [20:41] etc. [20:42] asac, I do not think app updating is in system settings [20:43] 21:38 < asac> pmcgowan: can we at least upgrade click apps now? [20:43] 21:38 < asac> :) [20:43] 21:38 < pmcgowan> we can witht he upgrade app still [20:43] 21:38 < pmcgowan> not yet in settings [20:43] so "not yet in settings" is a fewature? [20:43] guess then it just means that the upgrade app didnt work in 194 [20:43] yes, its staus quo [20:43] so all good [20:43] dont have any other explanation [20:43] there is a branch waiting to make it so [20:44] updating apps is not in settings, that was reverted, use the updater until https://code.launchpad.net/~diegosarmentero/ubuntu-system-settings/click-updates/+merge/208567 merges [20:44] yeah. if it was never in, then thats surely not what i mean [20:46] #213: [20:46] Full list of changes in Friday's email. However, just want to mention (if you didn't read other exchanges on this mailing list), that this version introduced a download manager issue on click apps. [20:46] maybe thats the one? [20:46] #216: [20:46] * this one contains the bug fix for the download-manager with click app issue. Dogfooding confirmed it's now fixed. [20:47] but that wasnt in the promoted image [20:48] anyway [20:48] * asac notes it down to post-mortem what was going on here :) [20:49] ogra_: do you know what was broken with click apps not updating? [20:49] what exactly that was :P [20:50] asac: that was that the fix for the system-image issue with download manager (atomic downloads) broke click apps downloads because noone followed the full test plan because of assumptions and lack of devices [20:50] click app updates was never in settings [20:50] ralsina_: right. but that wasnt in 192 [20:50] so its not the thing [20:51] what ways to update click apps do we expect to have? [20:51] 1. upgrade app [20:51] 2. scope? [20:51] 3. automatic? [20:51] "expect" as in "now" or "expect" as in "someday"? [20:51] ralsina_: 192 [20:51] in 192, just the upgrade app [20:52] ralsina_: 170? [20:52] there was one image where the upgrade app was bogusly removed, don't recall the number [20:52] ralsina_: ok i think thats 192 [20:52] unless we had some other way to upgrade apps [20:52] ok, so that was it [20:52] nope, no other way [20:52] wqe never had another way to upgrade apps? [20:52] no [20:52] good [20:53] that thing is back now. so thats fixed :) [20:53] * asac doesnt know if its right, but at least can get some rest with this believe now [20:53] hehe [20:54] asac: good enough to go to bed on a friday indeed [20:54] yep [20:55] asac: just as a heads up, starting monday I am phasing off as a manager into a dev position in lucio's team, so click scope will be in thostr's capable hands [20:55] pmcgowan: ralsina_: sorry for annoying on this one. we have a way to improve our problem tracking planned, then i could just look it up [20:55] asac: of course I'll be around to help as much as needed [20:55] i am happy now. we achieved a lot this week [20:55] and are good for a qt shot :) [20:55] ralsina_: thanks! [20:56] we certainly did [20:56] yep. qt cracking [20:56] image fixing [20:56] ;) [20:56] * asac phases out from active discussions for the day [20:56] cu around monday [20:57] See you on Monday! [20:57] err wednesday... :) [20:57] i am most likely off [20:57] but we will see [21:04] hmm, i presume all the "no change rebuild" merges are blocked on qt5.2 landing, or something? is there a silo for that, or what? [21:10] dobey, yep, silo 6 [21:10] dobey, we're looking at landing those monday [21:12] robru: and what about the rebuilds for things that aren't on ci train yet? [21:12] asac you still about? [21:13] dobey, what isn't in ci train yet? everything should be in there by now. if not, I dunno, you'd have to ask Mirv how he'll handle those [21:13] robru: ubuntu-purchase-service isn't [21:14] dobey, I guess Mirv will have to migrate it to citrain on monday then. not sure [21:16] asac: it was the camera and weather click apps that were updated I needed to make sure that both of those were included in the new build but had updated them through the click store on 225 [21:17] at least I think it was the the camera app anyway :) [21:18] oh, meh, ok [21:20] asac: the title is [Ubuntu-phone] Image r226 Testing Results I kept it separate so it stood out from the list. [21:46] davmor2: yes, was received, just came through late. have a great weekend [21:47] davmor2: sure that 226 just had those two changes? [21:47] http://people.canonical.com/~ogra/touch-image-stats/20140307.2.changes [21:47] davmor2: did we do smoke tests on 226? [21:47] popey: ? [21:48] davmor2: popey: there have been other changes, so if we havent even booted etc. we run kind of risky :) [21:53] dont worry [21:54] jfunk confirmed that you guys tested 226 [21:54] thanks and enjoyu! [21:55] asac: no we tested on 226. All I was saying was I upgraded from 225 to 226 but had already updated the click apps on 225 so did a fresh install to make sure that 226's image had actually pulled them in. [21:55] asac: as they were kinda key to making weather app more stable [21:55] asac: not perfect but more stable :) [21:55] asac: if there is anything else I'll stick around for a bit [21:56] davmor2: nice. so extra care. thanks [21:59] plars-away: i think i'm supposed to chat with you ? [Bug 1236525] Re: unity8 killed/crash then restart can result in mir unable "could not unblank display" [21:59] bug 1236525 in unity-mir "unity8 killed/crash then restart can result in mir unable "could not unblank display"" [Medium,Triaged] https://launchpad.net/bugs/1236525 [22:02] one quick question for whomever....wrt "Special landing mode process while we are recovering on ubuntu Touch" [22:02] if Qt5.2 has a project associated with it during this time before "final qt5.2 landing" [22:03] does that lock out others ?? (e.g. i'd love to reattempt mir - we fixed our 1 regression we found, but qt has unity-mir as a rebuild) [22:05] I gotta head out, anybody need anything before I EOW slightly early? [22:06] kgunn, yeah, projects can only exist in one silo at a time. qt5.2 is locking an enormous number of things right now === retoaded changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cihelp | CITrain support - US: robru, cyphermox, rsalveti - EU: sil2100, Mirv, didrocks | CITrain support no answer: use mup bot after 30 minutes, but choose right timezone | Landing instructions: http://goo.gl/8H1Du3. Landing in degraded mode (see http://goo.gl/J1EqPW) [22:18] kgunn: is this related to the update we got to the unlocker that didn't work? [22:19] plars-away: not completely sure...i got a mail from alexander but didn't quite know what it meant...ended with "talk to plars" :) [22:20] kgunn: If it's what I think it is, om26er is probably the better one to talk to. He's the one that patch came from. But is there a specific question? I'm not really familiar with that bug [22:20] ok, i'm ending soonish...but i'm sure anyone will be glad to help euro mornin monday [23:04] kgunn: plars-away: i just dropped a comment that wahtever patch landed that was referring to the bug was backed out [23:04] as it broke all tests [23:05] that was an infrastructure patch i referred to... guess it was omer etc. [23:05] i dont know mnow ,mor :) [23:05] have a great weekend === bschaefer_ is now known as bschaefer