[00:26] cihelp, jenkins isn't building music-app [00:26] https://code.launchpad.net/~vthompson/music-app/fixes-1259962/+merge/198601 [05:05] cwayne: sorry, didn't get a good chance to reconfigure the jobs before, but I merged the change already. I just pushed and merged it all now, and re-started the custom jobs here: http://q-jenkins:8080/view/Touch/view/Ubuntu%20Touch%20Master%20Jobs/job/trusty-touch_custom-mako-smoke-master/ [05:05] cwayne: we should check on them tomorrow morning, but they will all run now anytime there's a new custom image [07:57] Morning [09:13] Mirv: hello! Did you notice any infra failures today maybe? [09:14] sil2100: intel failing right now there running apps stack ap:s [09:14] nothing else [09:15] I got one good result before that === ev changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cihelp (EU CI members sprinting this week) | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - [09:33] ev: joining hangout? [09:43] psivaa: hello! [09:43] psivaa: heya [09:44] psivaa: could you retry terminal and unity8 on maguro after recording hte URLs of failing tests? [09:44] lool: will do [09:44] psivaa: also would need webbrowser on mako [09:44] lool: will do [09:45] psivaa: cool, could you ping us when done? [09:45] psivaa: would like to kick an image build once it's done [09:45] lool: sure [09:45] psivaa: seems we duplicate here :) [09:45] err in -touch [09:46] :) [09:46] so yeah. let us know when done so we can swiftly kick [09:50] lool: sorry, couldn't make it. Running a sprint on juju and ci in bluefin today [09:52] ok [09:53] sil2100: building it now first. I think we had the rule for this week that cu2d desktop tests need to pass as well before releasing anything, so let's see them first and I'll continue on my Qt stuff [09:56] Mirv: thanks! [10:03] Mirv: regarding testing the bug, from the merge request I see that it's indeed just switching between 12h and 24h locale, looking at some PM time and checking if the indicator changes [10:19] lool: asac: the three tests have been run and passed. sending you the earlier failed links in a mail [10:20] psivaa: \o/ [10:21] awesome [10:21] sil2100: good to start a build? [10:22] lool: fire it up! [10:22] 99.5% [10:22] that's pretty good [10:23] building [10:23] sil2100: Do we want to promote image 70? [10:24] sil2100: maybe check with popey and others [10:24] sil2100: just let me know here if you want it promoted [10:24] I can check on mako, but you need someone (omer) with maguro to test too [10:25] hmm [10:25] lool: how can we get those final tests fixed? [10:25] om26er: ^^ [10:25] om26er: we have like 3 AP tests... can you just crush them? :) [10:26] popey: if you have a free moment then check, if you can - I would personally prefer promoting an image with the datetime fix, but any promotion doesn't hurt ;) [10:26] om26er: http://ci.ubuntu.com/smokeng/trusty/touch/mako/70:20131217:20131211.2/5503/ubuntu-weather-app-autopilot/ [10:26] om26er: if you can fix those i will give you whatever you want :) [10:26] 2!!! [10:34] asac, sure. get me a pay raise ;) [10:35] om26er: how much? :) [10:35] what is needed for a new ubuntu-ui-toolkit release? It has been in the landing asks page for a while now [10:36] asac, i'll settle for 10% but before april. [10:36] working on the failures now [10:37] om26er: seems its just a DB not being created [10:37] not sure whats going on with the .ini files [10:37] and who should create those [10:38] in create_blank_db [10:38] something is missing [10:38] :) [10:38] feels all it wants is to figure what location to use for the sqlite file [10:38] not sure why the algo is so weird and goes through .ini files in Databases directory === xnox_ is now known as xnox [10:50] sil2100: 70 looks good to me on mako [10:51] popey: thanks! [10:52] om26er: could you give image 70 a spin on maguro as well, just like popey ? [10:52] sil2100, Ok, updating. maguro is too slow to be fun :) [10:52] ;D [10:56] sil2100, how to run autopilot tests for click packages ? [10:56] om26er: http://paste.ubuntu.com/6588007/ i use that script [10:57] ./testing.sh lp:ubuntu-rssreader-app ubuntu_rssreader_app [10:57] like that [10:58] popey, thanks [10:58] om26er: https://wiki.ubuntu.com/Touch/Testing <- this guide works for me as well [10:58] So just phablet-config autopilot --dbus-probe enable [10:58] And then: phablet-click-test-setup --click com.ubuntu.calculator; phablet-test-run ubuntu_calculator_app [10:58] etc. [10:59] Damn, my neighbour again started drilling [10:59] Doesn't help with concentrating [11:07] bug 1192711 is a little irritating [11:07] bug 1192711 in mediaplayer-app "video player won't play a second video without closing completely" [High,Confirmed] https://launchpad.net/bugs/1192711 [11:07] not a regression, seems to have been there a while, but I'd not noticed [11:21] sil2100: https://code.launchpad.net/~timo-jyrinki/cupstream2distro-config/indicators_missing_deps/+merge/199265 [11:22] Mirv: do you know if both libupstart-app-launch1 and libupstart-app-launch2 are required to be installed right now? [11:24] om26er: any clue yet? [11:24] om26er: sergiusens knows how to run AP test for click [11:24] oh already answerd :) [11:27] Can someone help me figure out why most of the autopilot tests for ubuntu-system-settings are borked (the ones which try to select anything, it seems), please? [11:27] asac, so the issue is the test suite expects a certain directory to be there but its not able to find it because that directory is only created once the weather app is started [11:28] om26er: i got so far... but it also expects .ini files in that directory :) [11:29] so i dont know where that .ini is supposed to come from [11:29] i think this is super bogus code to be honest :) [11:29] asac, works fine for me after I started the weather app manually [11:29] om26er: where is that directory created? [11:29] borked on the device only, that is --- they work on the desktop... [11:29] om26er: i didnt see anythign in code creating that directory ... nor the .ini file [11:29] sil2100: in the previous cases it seemed to be, at least earlier, that if the older one was removed it did not work [11:29] asac, ~/.local/share/com.ubuntu.weather/ [11:30] om26er: where in code i mean :) [11:30] oh wait [11:30] yeah .../Databases [11:30] i still dont see anywhjere in code that this dir is created [11:30] also the code checks for .ini files [11:30] asac, exactly that directory is not created by the test suite, rather it is automatically created when the app is started [11:30] anyway. seems you already have it running, so debugging shoudl be simple [11:31] om26er: which code is doing that? [11:31] om26er: is that something our sdk does? does it also create the .ini files? [11:31] asac, the app is using QtLocalStorage for that [11:31] asac, the dir is created when using LocalStorage [11:31] ok seems the test doesnt use it [11:31] kk [11:32] so the tests should be using that :) [11:32] sil2100: anyhow, the cu2d tests are not green so if I should test on device or not [11:32] instead of raw mkdirs etc. [11:32] err raw os. stuff [11:32] interesting that i get no direct hit when googling for QtLocalStorage [11:32] is that no public API? [11:33] sergiusens, I am thinking to check if that directory exists and if it does not just start the app and close it in the suite. sounds hacky ? [11:33] sil2100: I understand that we won't release unity stack until it's green, but what about indicators? do you think I can go ahead and test on device with those results currently in cu2d and publish if device ok? [11:33] sounds hacky :) [11:33] but makdirs that directory would make sense [11:33] :) [11:33] om26er, that was there before and actually removed from the test suite to make it more stable [11:33] hehe... just dont know about the .ini and if that info in ther ematters [11:33] om26er, I don' t get it; I tested by deleting the dir completely when first accepting [11:34] so why cant the test use qt local storage as well? [11:34] or call an app func to do the setup [11:34] hmm. guess its not nicely available in python :) [11:34] asac, QtQuick.LocalStorage [11:35] right. so if we would have qmltests rather than AP things would be easier? [11:35] anyway. i feel fro the test all that matters is that we give it SOME db filepath [11:35] that is valid [11:35] but qmltests are not integration tests [11:36] so whatever we give too it its good and if its even in the right .local/ directory owned by the app its even better [11:36] asac, the app is supposed to create everything it needs on it's own, and it does [11:37] sergiusens: right, so its a race? [11:37] sergiusens: where does the test start the app? [11:37] om26er, asac, the test code just needs to contemplate the inexistence of a prior database, unless it mangles with it before launch [11:37] sergiusens: so we might want to change the find_db into a poll that waits for 60 seconds? [11:37] and only then gives up? [11:37] :) [11:37] asac, that may be the case; not sure if it waits for everything [11:38] asac, yeah, that's what's been happening with the tests [11:38] Mirv: let's test it on the device, since I guess not-green tests on indicators right now basically might be related to some desktop issues [11:38] Mirv: let's make this one exception [11:42] btw ci, need a release for autopilot to get some emulator love http://www.youtube.com/watch?v=n2pVpjBFiaI [11:42] cihelp that is [11:43] nice one [11:43] sergiusens: lets get one 100% image [11:43] then we land autopilot... hwoever, i wouldprefer if we could pick just what you need [11:43] and leave the rest out [11:43] There's a lot of autopilot changes pending still [11:43] check much stuff people have accumulated http://people.canonical.com/~platform/cu2d/results [11:43] i would prefer if we only land exactly what we understand out of those [11:43] I guess we could release this, but I would like to get all the other pending bits out separately first [11:44] at best all, but only after looking at each change [11:44] :) [11:44] sil2100: lets get 100% green before pulling in AP [11:44] AP folks should be testing the hell out of their AP [11:44] so once we take it its clean [11:45] asac, sil2100 would a package patch do? It's a one line add to a map, that's about it for now [11:45] robru: ping [11:45] I can prepare a debdiff [11:45] sergiusens: not before we have 100% :) [11:45] get us the fix for this test :) [11:46] then we cna land whatever you want [11:46] hehe [11:46] asac, ack, I can wait [11:46] sergiusens: help fixing the test [11:46] its just this DB thing [11:46] at best next image would have the fix [11:46] sergiusens: that one-liner, would it help in getting us green, or is it related to something else? [11:46] sergiusens: check with om26er if he needs help etc. [11:46] sil2100, not related [11:46] om26er, do you need help? [11:47] om26er, I can review whaever you've done so far [11:47] sergiusens: then let's wait and release it with all the AP changes then - we need to release those ASAP anyway ;) [11:47] sil2100: please send a mail to thomi,sergiusenss etc. that they should start running APs all day long against their AP package from daily-build [11:47] and give us a list of APs that are green [11:47] after that ... at best with logs :) [11:49] sil2100: alright [11:52] sergiusens, I only figured what was the problem. now working on workarounds/solutions [11:52] sergiusens, will get help from you if i run out of ideas [11:54] ack [11:54] Mirv, sil2100 do you know what is blocking the UITK release? the flaky test from https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0Au6idq7TkpUUdGNWb0tTVmJLVzFZd0doV3dVOGpWemc&usp=drive_web#gid=0 was fixed a while ago [11:55] it was this one https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1259476 [11:55] Ubuntu bug 1259476 in ubuntu-ui-toolkit (Ubuntu) "Flaky gallery.test_textinput.TextInputTests.test_textfield_numbers, fix or disable" [Undecided,New] [11:55] timp: hi! [11:55] sil2100: hello :) [11:55] timp: from what I see we required an ACK from you guys and a re-addition to the Landing Asks once the flacky test is gone :) [11:56] timp: so, if you can, please add it and we'll do everything we can to release it ASAP [11:56] sil2100: I don't have edit rights for that doc [11:56] sil2100: can I get rights or should I ask zoltan to add it tot he list? [11:56] *to the list [11:57] om26er: did I see sil2100 bribe you to look at the weather test failures? Are you looking at those? [11:58] popey, yes, I am working on it [11:58] timp: it's usually the manager that adds it [11:58] om26er: great. let me know if you need any testing of it done [11:58] Mirv: okay. [11:58] timp: if you can, please ask Zoltan :) Thanks! [12:00] okay, asking. thanks [12:00] robru: unping :) [12:03] sil2100: it is there nwo :) [12:03] *now [12:03] I seem to have a typing problem today [12:15] sergiusens, summary is: the test code adds certain fixed locations to the database (hamburg and London in this case) before starting the app so that the tests have some data in the app to interact with. for injecting these pre-defined db entries it needs to have an already created database. [12:15] sergiusens, the notes-app does exactly the same thing. it check if the db directory is there and if its not it starts and quits the app [12:18] om26er, ok, there's old code in there that calls a launch_and_close_app === gatox is now known as gatox_dr [12:40] sil2100: can you please keep me informed about the uitk release? If there are any issues please let me know so we can fix them :) thanks [12:40] timp: sure, thanks! :) We'll add it to our todays release list [12:40] great [12:41] Mirv: how's the progress on indicator-datetime ? [12:41] popey: hmm, you said you have some bluetooth audio device for Ubuntu Touch testing? [12:41] sil2100: it's starting to be done, I'm wondering though who to bug with packaging changes [12:42] lool: did we ever build an image :)? [12:43] ah seems we have 71 [12:43] nice [12:43] psivaa: something was going wrong on mako it seems [12:43] guess you are on it? [12:44] Mirv: that's a valid question! Maybe try poking Laney or lool I guess? [12:44] Later we can always pester Ken or Mathieu [12:45] ack. pick any friendly core-dev :) [12:46] There's loads in #ubuntu-devel, but sure [12:46] Laney: http://pastebin.ubuntu.com/6588664/ (to fix http://pad.lv/1256061) [12:46] Launchpad bug 1256061 in indicator-datetime (Ubuntu) "clock forcefully switches from 24h to 12h AM/PM format once the panel clock is loaded" [Undecided,New] [12:46] maybe you could look up my question from earlier that went unanswered to thank me back :-) [12:47] oh it's this [12:47] the merge request does not explain the details (small fixes) https://code.launchpad.net/~charlesk/indicator-datetime/lp-1256061/+merge/198968 but seem sok [12:47] I told him it's better to use locale-gen directly [12:47] but fine [12:47] sil2100: no [12:48] sil2100: oh, actually, my daughter has a speaker, let me see if that works [12:48] popey: since someone mentioned on the ML that bluetooth devices still don't work with the latest image [12:48] Laney: thanks. ooh, autopilot test failures.. did you see the logs already? [12:48] which logs? [12:49] Laney: settings failures, there are some tests failing in the CI [12:49] show me? [12:49] it's probably the same thing [12:49] http://q-jenkins.ubuntu-ci:8080/job/autopilot-trusty-daily_release/957/label=autopilot-nvidia/ + http://q-jenkins.ubuntu-ci:8080/job/autopilot-trusty-daily_release/957/label=qa-intel-4000/ [12:49] for some reason on intel test_about.StorageTestCase.test_installed_apps and on nvidia test_about.LicenseTestCase.test_licenses_page [12:50] /var/local/autopilot/autopilot.log: AssertionError: The following apps were started during the test and not closed: [] [12:50] is it that? [12:50] sil2100: it doesn't detect my bluetooth speaker [12:50] but I don't know if it ever did, because I have never tested this [12:50] CraB [12:50] Laney: looks like that. a) it seems like apport, b) it's different from your "most tests failing" [12:51] it is [12:51] yep, http://q-jenkins.ubuntu-ci:8080/job/autopilot-trusty-daily_release/957/label=autopilot-nvidia/artifact/results/autopilot/videos/ubuntu_system_settings.tests.test_about.LicenseTestCase.test_licenses_page.ogv [12:51] system-image-dbus crash [12:52] psivaa: hello! Do you have a Ubuntu Touch compatible bluetooth device by any chance? [12:52] seems like it's not our fault [12:52] also that runs on desktop ... [12:52] the problem I had was only on the phone [12:53] Laney: bypassing those, are the selection problems using selection via UI Toolkit somehow? just wondering if SDK people could help debugging the errors [12:53] it uses select_single from autopilot, don't know what that does [12:54] that's at least used also by ubuntu-ui-toolkit which has all tests passing, so maybe it'd be worth asking AP people like thomi directly [12:56] ok === alan_g is now known as alan_g|lunch [12:59] popey, here: https://code.launchpad.net/~om26er/ubuntu-weather-app/fix_db_tests/+merge/199278 [12:59] popey, will the CI run on that branch ? === cjohnston changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cjohnston | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - [13:06] om26er: lemme test it first [13:07] it wont if nobody approves it [13:09] testing now [13:17] sil2100, Mirv: Feel free to bug me for packaging reviews if I'm around [13:18] lool: thanks :) [13:20] om26er: sergiusens do we have a breakthrough on those tests yet :)? [13:20] asac, I gave something to popey to test [13:20] WFM [13:20] wfm too [13:20] cool [13:20] :) [13:20] nice [13:20] * popey commented on the mr [13:20] who need to approve so this can land? [13:21] did something else land as well on trunk after we updated in image last time? [13:21] lets get that removed :) [13:21] mhall119: can probably approve https://code.launchpad.net/~om26er/ubuntu-weather-app/fix_db_tests/+merge/199278 for us [13:21] when he awakens shortly. [13:23] popey, om26er: with this fix in, the weather app AP tests are fixed? [13:23] thats the plan [13:23] sil2100, atleast the one's that I was given link to [13:23] om26er: +beer for you! [13:25] sil2100, ;) [13:26] asac, nothing; the apps with the most failures in testing have been apps that only hd test updates for the past two months [13:32] sil2100: i dont have any devices with me at the moment :/ [13:33] om26er: sorry to bother you again, but maybe you by some lucky chance have some UT compatible bluetooth devices? ;) [13:33] sil2100, I don't have. the one in my car is not supported by Ubuntu touch [13:35] sergiusens: hi! I had some questions related to LP: #1234361 [13:35] Launchpad bug 1234361 in lxc-android-config (Ubuntu) "Bluetooth panel not getting scan results" [High,Fix released] https://launchpad.net/bugs/1234361 [13:35] sergiusens: someone reported on the ML that it's not fixed - and I can't find anyone that would have the equipment for re-testing this [13:37] sergiusens: do you have the tools to check if this is fixed with the latest image? [13:38] sergiusens: ok... please help the fix get into the store and then lets build an image [13:38] psivaa: mako still has no resutls for 71 [13:39] http://ci.ubuntu.com/smokeng/trusty/touch/maguro/71:20131217.1:20131211.2/5521/ [13:39] (sorry if you answered) [13:39] plars: ^^ [13:39] asac: looking at that, sorry dint notice that [13:40] sil2100: did we promote 70 or do we have that in the bank and already well tested, but hope for 71 to be at least as good and push that? [13:42] sil2100, I don't have bluetooth headsets, but I can check from the cli and see if I find anything [13:46] asac: from what I see, 71 doesn't have anything more than 70 (no packages updated) [13:47] asac: so let's promote 70 anyway, since 72 will only be ready for tomorrow morning [13:47] asac: if there is of course no cost of promoting an image that is [13:47] It doesn't matter if we promote 70 or 71 basically [13:48] But just to have a clear concious we can promote 70, since that's what popey and om26er used for testing [13:48] lool: ^ could you promote 70 ? [13:49] sil2100: yup [13:49] Thanks! [13:50] asac: just got to my desk, looking now, unless psivaa has already [13:50] plars: the devices is shown as offline soon after flashing [13:51] probably need to assign another device. mako-06 looks ok to do that [13:52] psivaa: wow, it even shows in adb but shows offline [13:53] psivaa: yeah, you want me to switch it, or do you want to? [13:53] plars: i am in the sprint,so if you could. that would help [13:53] psivaa: ack [13:53] thanks plars [13:56] rfowler: when you are around, could you take a look at mako-02? It's marked offline for some reason, so nothing can run on it. I've moved all the jobs from it to 06 for now [13:57] asac: jobs are restarting now for mako on image 71 [13:59] om26er, popey where's the MR [13:59] ? [13:59] https://code.launchpad.net/~om26er/ubuntu-weather-app/fix_db_tests/+merge/199278 [14:00] morning [14:01] FYI, image promoted [14:01] (70) [14:01] lool: my phone can't see it [14:01] pfft ... gimme 72 ... with 24h clock [14:02] popey, om26er ironically, the tests fail on jenkins [14:02] sergiusens, that's not related to my code [14:02] falky tests are flaky :/ [14:04] ogra_: patience! [14:04] :D [14:05] popey: something is wrong with the push mirrors [14:07] indeed [14:08] hmm maybe it's a cron and I had never realized [14:09] it's usually immediately after ogra_ says it's published [14:11] lool, nusakan seems dead here [14:11] or at least inaccessible [14:11] * ogra_ cant ssh [14:11] popey: it's being fixed [14:11] ogra_: you need to use sshebang [14:11] ogra_: or update your ssh config to use FQDN [14:12] ogra_: works fine for me [14:12] since when ? [14:12] I don't use sshebang) [14:12] ogra_: since some hours ago [14:12] it worked on monday [14:12] it worked wihtout .canonical.com up til this morning :-) [14:12] ah [14:12] my config has used nusakan.canonical.com for ages [14:12] I raised it with #is, but it's intended [14:12] * ogra_ fixes /etc/hosts [14:13] hmm, nothing to fix there it seems [14:13] 91.189.89.127 nusakan nusakan.canonical.com [14:13] but indeed, using the FQDN works [14:13] cjwatson: ssh nusakan used to work without .canonical.com from the jump host, allowing to have a single .ssh/config entry for *.canonical.com + nusakan; now I have two [14:14] ogra_: this is in your .ssh/config [14:14] I doubt that /etc/hosts matters. Fix ~/.ssh/config [14:14] ogra_: your proxycommand needs a FQDN [14:14] ogra_: nc -q0 nusakan.canonical.com or nc -q0 %h.canonical.com [14:14] The host name is looked up on the proxy jumphost, not on your machine [14:14] And I'm pretty sure you can't write to chinstrap's /etc/hosts :) [14:14] oh, ok [14:15] thanks, that fixes it === alan_g|lunch is now known as alan_g [14:21] #70 on the way \o/ [14:22] \o\ [14:22] It's a bird, it's a plane! No, it's #70! [14:23] IS should have fixed it entirely now [14:23] I see #70 on trusty [14:26] om26er, check my comment on the MR [14:27] plars: I'm going to 1ss in a bit... my car was completely frozen and had to thaw it before the next storm hits [14:29] rfowler: ugh, be safe [14:30] rfowler: it's no rush on our end, its up and running for now === gatox_dr is now known as gatox [14:46] sergiusens, pushed. looks better now ? [14:47] sergiusens: did you have a moment to do that cli bluetooth sweep? [14:47] om26er, let me check; let's also get balloons on now that he's online [14:47] sil2100, no [14:48] om26er, looks better, let's wait for ci [14:48] om26er, are you looking @ weather or calendar? [14:48] balloons, weather [14:48] balloons, weather, and its cold. [14:48] balloons, https://code.launchpad.net/~om26er/ubuntu-weather-app/fix_db_tests/+merge/199278 [14:48] sergiusens: om26er: are we close :)? [14:48] om26er, it's 35 C [14:48] the next image will be cut in 2h approx [14:48] would be nice to have it in :) [14:49] asac, I can't approve MRs for community apps, but if it works, I'm fine with the solution [14:49] balloons: can you plesae check that out? [14:50] balloons: we have the fix pending for 100% dashboard gren on mako [14:50] and i want to get this in opportunistically for next image [14:50] om26er, why are you readding launch and quit? [14:51] balloons, these tests assume ~/.local/share/com.ubuntu.weather/Databases/ to be there. It is only created once the app is started [14:52] balloons, since we manipulate an already existent database, and if the suite is not able to find one it fails without telling why it failed [14:53] om26er, we have the ability to create a db if one doesn't exist in the test [14:53] I don't believe this changes anything [14:53] launch and quit would be a regression imho [14:54] balloons, remove ~/.local/share/com.ubuntu.weather/Databases/ on your phone and run those tests, they fail with the traceback that's in the dashboard [14:54] balloons, then start the app with hands and run those tests again and they will pass [14:55] balloons, from what I saw from the test code is that it tries to add a few database entries to an already existent database, it does not create a whole new database [14:55] balloons, line 165 in __init__.py [14:56] om26er, I don't doubt your assessment. However, the fix imho is not to go back to the launch and quit thing. Let's just create a db from scratch if we need to [14:59] balloons, not really sure if that's going to be a simple task ? these are Qt LocalStorage databases, and there aren't nice APIs for python [15:00] om26er, well I understand the desire to push this in, but it's going to be tough to review beyond I would really like to not see launch and quite come back into this test [15:02] balloons, another nasty thought is: clear the db, launch the app, copy the db and ship that with weather app tests. In the test suite make sure to move any old db, copy the shipped one over to the default location, run the tests. Profit! [15:04] om26er, I would be happier with that kind of approach [15:04] if we can't build it on the fly, just ship it with the tests and copy it it to set it up that way [15:05] balloons, ok, I can work on that. btw how do these tests get in in the first place ? [15:07] ah, i would assume CI test runner passed those tests because that db was already there due to not having a clean environment. [15:12] balloons, popey om26er we need a happrover [15:13] sergiusens, om26er is going to make the tweak I asked for [15:13] balloons, ack, feels lke a recap from an earlier conversation ;-) [15:13] sergiusens, :-) I'm sure he works quick and we'll have it [15:34] cwayne: looks like the attempt I made last night with custom on the new mako had a strange problem, going to retry it and see what happens, but that's not something I've seen happen before [15:36] plars ah, ok [15:36] i was going to ping you since i saw all 0's, but I thought it might be too early :) [15:46] Hey, having some CI issue I can't seem to figure, anyone mind taking a look? https://code.launchpad.net/~nicolas-doffay/ubuntu-ui-toolkit/fix-1242647/+merge/197176 [15:54] sergiusens: balloons: om26er: stuff in? [15:54] people are waiting with aggressive uploads [15:55] that will brick us for weeks [15:55] i can hold the line for a bit longer :) [15:55] but... :) [15:56] asac, just pushing the fix [15:59] asac, which means it's another hour until it's fully in [16:00] sergiusens: we can wait with kicking the build a bit after the meeting, so hour more is ok === Ursinha-afk is now known as Ursinha [16:02] sil2100, really depends on the infra after the review... [16:03] sergiusens: yes, thats fine [16:03] right in time for our standup === doanac changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: doanac | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - [16:03] to agree on kicking [16:05] om26er, please ping when you need approval [16:14] popey: do you still need somebody to top-approve https://code.launchpad.net/~om26er/ubuntu-weather-app/fix_db_tests/+merge/199278 ? [16:17] mhall119: i think balloons is on it. [16:23] plars: mako-02 is up [16:23] rfowler: cool, thanks - what was going on with it? [16:24] plars: it visually seemed just fine [16:24] rfowler: odd, adb had it marked offline [16:24] plars: ya... not sure [16:44] om26er, how's it going? [16:50] balloons, https://code.launchpad.net/~om26er/ubuntu-weather-app/db_stab2/+merge/199321 [16:51] om26er, :-) [16:54] balloons, om26er I added 2 comments [16:59] sergiusens, om26er, balloons: so I guess we won't make it with the fix before the meeting ;) ? [16:59] it'll be during the meeting :-) [17:00] robru, kenvandine, cyphermox, lool: meeting? [17:00] balloons, om26er also need to use the module path to determine where to copy the database from, look at the music app as an example [17:01] sergiusens, so I do os.path.join inside os.path.expanduser wouldn't that hurt readability more ? [17:01] asac: coming? [17:01] sil2100: yep [17:02] om26er, do it in two steps, but doing a + is not pretty and could concatenate the wrong path [17:03] om26er, alternatively, just have db_file and drop db_path and use path.dirname [17:06] tests do run fine on my device [17:06] I think Sergio has nailed the syntax bits [17:06] sergiusens: all in? [17:06] :) [17:06] e.g. can we kick ? === gatox is now known as gatox_lunch [17:07] do I need to start a build? [17:07] asac, read like 5 lines above; MR got in 10 minutes ago [17:07] asac, still under review [17:09] sergiusens, pushed [17:11] sergiusens: i dont understand the steps it takes [17:11] before we can build an imge [17:11] hence i am only asking about the final YES its all ready [17:11] or no, i have to do X,Y still :) [17:11] asac, ok, fyi, we are at step 1 [17:12] asac, I'll get back to you when it's in [17:12] sergiusens: so what steps are left? [17:13] and whats the ETA? [17:13] i need to tell folks that they have to wait with uploads etc. [17:13] sil2100, https://code.launchpad.net/~ken-vandine/cupstream2distro-config/no_ap_settings/+merge/199325 [17:14] asac, needs to get review approved, tested and then merged by the autobuilders, then build the click; test the click, then push to store; then have it synced [17:14] sil2100, that should be enough to keep it from block on the check step [17:14] asac, as I said, an hour after the MR is in [17:14] ahh sergiusens sorry.. I thought it was a bit quicker after the MR was in [17:15] I said we'd laid in the MR very shortly, and I didn't think it would take much beyond that [17:15] balloons, well it's mostly waiting on infra most of the time [17:15] balloons, I can also blindly push without testing myself [17:16] sergiusens: ping when you want the click package testing [17:16] popey, ack [17:17] sergiusens, I'm running the MR with and without db prexisting [17:18] balloons, the problem isn't going to be click, but when running on a clean env as a deb [17:18] as a click we cd into the autopilot dir [17:19] but as a deb we on desktop, that's not the case [17:26] kenvandine: could I ask for commenting that out instead of removing? :) Otherwise I'm sure I'll forget that the test should be there ;p [17:26] ok [17:28] sil2100, pushed [17:30] Thank you! Approving [17:32] om26er, sergiusens made some further suggestions. I'm happy with it as he is suggesting. It runs great for me. [17:32] om26er, with those tweaks I think we're good [17:34] balloons, done [17:35] aah there are more suggestions [17:36] om26er, balloons also look at how I'm getting the module path http://paste.ubuntu.com/6589958/ [17:36] didnt' test that part yet though [17:42] om26er, thanks for keeping on this.. It's worth getting it right before it goes in.., [17:46] balloons, om26er with the changes I proposed, still get all tests passing as click [17:46] tested on desktop and device with and without db [17:46] we're releasing u-s-s? [17:46] neat [17:47] Laney: yes \o/ [17:48] would have reviewed the outstanding branches if I knew [17:48] Laney: kenvandine handles that, we'll deal with the AP tests later - no need to keep the package waiting, since we're not yet using those extensively [17:48] balloons: isnt the current code good enough? you can still improve after its out :) ... all we need is code that fixes the tests for now [17:48] Laney, we can land it again [17:48] it's okay, I'm looking at something there [17:48] it had just been ages [17:48] balloons, sergiusens pushed you suggestions. [17:49] *yours [17:49] * asac sorry for being inheritently impatient :) [17:49] those tests select a QQuickView [17:49] trying to copy what gallery-app does instead [17:49] * asac goes off and waits for another hour [17:50] asac, it is prone to fail on desktop [17:50] om26er, did you add the module path detection? [17:50] ;) [17:50] sergiusens, no, I am not adding that ;) [17:51] om26er, why? [17:51] sergiusens, I don't understand that well and I am not really productive at this O'clock [17:51] do we care about desktop? not sure what it means if we say it fails there [17:52] asac, you don't today, but you will tomorrow and when you do you won't be able to pinpoint the problem as easily [17:53] asac, I'll abstain and leave it to balloons and popey [17:53] ok. [17:53] balloons: popey: whats next? did we give up? [17:53] hmmm [17:53] * popey is on a hangout, will catch up in a moment [17:53] asac, they need to approve the MR [17:53] sergiusens, balloons, om26er: so no weather-app update today, right? [17:53] or you can summarise in one tweet size line [17:53] asac, and then I continue [17:54] sil2100: i hope the answer is that we get it [17:54] soonish [17:54] sil2100, I was under the impression we could go as soon as the MP landed, but sergiusens informed me it takes a bit. [17:54] sil2100, asac it will be in [17:54] \o/ [17:55] sergiusens, balloons: could you guys poke us once the fix is in and ready to be fetched by a new image? [17:55] balloons: when will the MP land though [17:55] i think thats where sergiusens is stuck [17:55] sil2100, that was the plan all along [17:56] sergiusens, om26er commented with "Not creating the dir in the test code. shutil.copytree calls makedirs so no need" [17:56] asac, I was reviewing the MR and detecting possible failures for the future; that's all; it's been like this forever with these apps; just want to do it once and not have to go over this again [17:56] om26er, balloons let me take over the MR [17:57] ok. dont make rocket science out of this [17:57] :) [17:57] but keep engineering standards high [17:58] yes, at this point we're not after quick fixes.. we'd had those for awhile. Being agonizingly detailed on these MP's is what has gotten us to this point [17:58] as in, so close to 100% with core apps not falling over randomly :-) [17:58] well, the goal shoujld be to have a fix soon [17:58] whatever best solution is a vailable [17:58] if code is not good enough you can iterate [17:58] balloons, I fixed that in my proposal [17:59] balloons: well, so we have big destructuve landings happening later today [17:59] hence i want this in to get to complete cgreen etc. [17:59] let me know what is left [17:59] and then lets do it [17:59] sergiusens, you make a new mp or ? [18:01] balloons, I will resubmit [18:02] balloons, I don't have privs, but can you resubmit that MR with lp:~sergiusens/ubuntu-weather-app/db_create === alan_g is now known as alan_g|EOD [18:04] sergiusens, https://code.launchpad.net/~sergiusens/ubuntu-weather-app/db_create/+merge/199333 [18:04] sergiusens, add a logging statement under if not os.path.exists(db_path): if you do create a db [18:07] balloons, done [18:11] first results on desktop and phone are good.. running again [18:12] jenkins builder is running === gatox_lunch is now known as gatox [18:19] plars, looks like touch_custom still has some issues [18:24] balloons: soooo... will it be merged soon :) ? [18:24] \o/ [18:29] cwayne: indeed, that's very strange considering how stable the other builds have been [18:29] cwayne: anything odd about that mako? [18:31] not that i know of plars [18:33] plars, hm, looks like it is having an issue with the theming perhaps? [18:34] cwayne: is it confusing the ap tests somehow? [18:34] plars, i'm not 100% sure, is it installed with --no-backup or -b? [18:34] i wonder if it's looking for an old theme or something [18:34] is there any way to get ssh access to it? [18:35] cwayne: -b, yes [18:35] maybe our customizations really is breaking stuff, in which case, good thing we did this! [18:36] cwayne: if you need ssh access, talk to rfowler or retoaded about getting ssh to a system called "ashes", it hangs off of there [18:36] rfowler, ^ [18:43] who's doing the latest image build? can somebody ping me when it's done? [18:51] we are waiting on MP landing still [18:52] and store upload i guess [18:55] cwayne: try logging into ashes [18:56] rfowler, what's the ip? [18:56] cwayne: it's in dns [18:56] or is it just ashes.ubuntu-ci [18:56] cwayne: yes [18:56] rfowler, k im in [18:57] cwayne: enjoy [18:57] rfowler, which device is it? [18:58] cwayne: 01ce848e48dfa6a2 [19:00] hm, the theming stuff is all correct there [19:00] maybe autopilot doesn't like it for some reason? [19:01] sil2100, yes "soon". have you heard that before? [19:13] :) [19:13] sil2100, mp is top approved so it's up to jenkins now ;) [19:14] \o/ [19:14] Do you know when it will be available for the image to pull it in? [19:15] sergiusens: ^ ? [19:15] sil2100, 30' [19:15] speeding [19:16] plars, so i see how the tests are failing, i still don't get *why* though, especially since it seems to work when i run it manually [19:19] sil2100, code hasn't merged yet... it's in the pipes, as soon as that's done, the equivalent of a daily release for a click package happens, goes to the store, then there's a sync to the ubuntu-archives [19:19] cwayne: I was going to try rerunning some of them, but the number of failures on each one of the failing tests looks to be the same as the last time it ran too [19:19] cwayne: pretty much the same failures on both 67 and 68 [19:20] plars, so it's apparmor denying requests for the custom theme we have [19:20] cwayne: ah [19:20] plars, but what doesnt make sense to me is that when you actually are using it, the custom theme shows up just fine [19:20] cwayne: doesn't seem to affect all apps, but it does seem to affect certain apps consistently [19:21] doanac, my friend. vanguard i trust :-) Can you have a look at this: https://code.launchpad.net/~vthompson/music-app/fixes-1259962/+merge/198601. Jenkins isn't landing the merge for some reason. The builders are failing [19:21] plars, right, but im looking at an emulator instance now with the devel-proposed-customized, and the gallery app is 100% properly themed [19:22] plars, and there's no errors, so something's weird [19:22] somehow running it from autopilot is making it denied [19:24] sergiusens: ok, let's wait till then [19:24] plars, is autopilot confined in some way? [19:26] cwayne: yes, it all runs identically to how we run on the other images [19:26] balloons: sorry. was grabbing lunch. looking now [19:26] plars, so what's happening is it's getting denied looking for /custom/xdg/data/themes when running from autopilot [19:26] but when actually using the device, it's allowed [19:27] sergiusens, re: https://bugs.launchpad.net/ubuntu-ci-services-itself/+bug/1259568 - I don't have anything scripted yes [19:27] Ubuntu bug 1259568 in Ubuntu CI Services "Create android builder to build AOSP branch from phablet.ubuntu.com repo" [High,Triaged] [19:28] 'yet' [19:28] fginther, ack; so what we need to script is rather easy and was sort of stripped out of the build setup when going public, what we can do something similar [19:29] fginther, if you are working on something; I can review and add your missing parts [19:29] fginther, are you working on something that sets up the environment and another that works on the build? [19:30] sergiusens, right, a script to just set up the env would be adequate [19:30] plars, is there a reason syslog isn't an artifact of the test runs? [19:31] sergiusens, I can put something together quick with the steps I performed as a start [19:31] fginther, sounds good [19:35] popey, https://myapps.developer.ubuntu.com/dev/click-apps/147/ [19:36] cwayne: iirc, it should come in the console output stream [19:36] doanac: can you confirm? is that what we get in utah with the 'with device.tail_syslog()'? [19:37] sergiusens: ack [19:38] plars: that is correct [19:40] plars: ie - the syslog should be in the jenkin job's console output [19:41] balloons: making progress on your issue. looks like the clock drifted on the build slave. working to get it fixed now [19:42] i'm not seeing it there, i should be seeing the apparmor denials [19:46] sergiusens: approved [19:46] balloons: you can top-approve and it should work now [19:46] doanac, k.. ty! [19:51] popey, yay [19:52] cjwatson, do you mind manually triggering the click sync please? [19:56] fginther: Hey, to follow on that Unity armhf debugging from yesterday, I know have a test mesa in a PPA we can try. [19:57] *now [19:57] ChrisTownsend, cool, what's the PPA? I'll kick off a build [19:58] fginther: ppa:rocket-scientists/staging [19:58] For armhf using that branch you used yesterday. [19:58] fginther: Thanks! [20:12] sergiusens: how's the sync? [20:13] sil2100, it's in, automated sync hits at 11' or 13' after the hour [20:13] we are good [20:13] sergiusens: once an hour? [20:13] \o/ [20:14] So I guess we're ok [20:14] cyphermox: hi! Do you have a moment to press the build button? [20:14] Q: how can we confirm that its really in (like: the image build will pick it up)? (like in archive we use rmadison)? === doanac changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cihelp | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - [20:18] asac, it's in click_list and livecd-rootfs has a hook that picks it from there [20:18] sure [20:18] asac, so if it doesn't, we have a mitm :-) [20:18] kk [20:18] just checking :) [20:18] lets go then [20:19] if you confirmed it like that [20:24] fginther: Could you send me the link to the build whenever you have it kicked off? [20:26] plars, alright, so we've got an anwser for why all the clicks were failing on touch_custom [20:26] still unsure of gallery-app though [20:26] sil2100: done [20:27] ChrisTownsend, http://s-jenkins.ubuntu-ci:8080/job/unity-trusty-armhf-ci/61/console [20:28] fginther: Thank you [20:29] cihelp: I just noticed today that the job that updates the autopilot documentation has been broken for a while - look s like a networking issue: http://s-jenkins.ubuntu-ci:8080/job/autopilot-docs-upload/ [20:29] I wonder if someone could take a look for me please? [20:29] thomi, looking [20:30] My guess is that since the machine moved lab, the firewall rules that IS look after will have to be changed [20:30] but that's just a guess :) [20:30] \o/ [20:30] cyphermox: thanks! [20:30] See you tomorrow everyone [20:30] thomi, timing of when the job started to fail would support that guess [20:31] fginther: yeah [20:31] thomi, I'll get a ticket filed to update the firewall [20:31] thanks [20:31] fginther: I assume once the FW is fixed you'll re-run the job? [20:32] thomi, I can if I remember to do so :-) [20:32] it usually takes them a while to respond to the request [20:32] fginther: thanks - I guess you'll run it to test the FW rules anyway [20:33] yeah, I usually hassle them on IRC :) [20:40] thomi, is maitri.canonical.com the correct destination? [20:43] fginther: yes [20:59] fginther, btw can we get automerge on lp:usensord? and some upcoming daily release prep? [21:03] sergiusens, I can help with setting up automerge, sil2100 and company can help with the daily-release prep [21:04] fginther, sure, just not sure how to make the request these days :-) seems the team is pending a merge :-P [21:30] sergiusens, this will take more time than usual, lp:usensord requires dh-golang on the host which isn't available for raring (which the build machines are currently running) [21:34] fginther, we just want trusty [21:34] oh [21:34] still raring? [21:36] sergiusens, it's a build host thing: http://s-jenkins.ubuntu-ci:8080/job/usensord-trusty-i386-ci/2/console [21:37] sergiusens, my plan was to move them to saucy eventually, this appears to be the trigger to start that process [21:41] fginther: daily-release-executor? [21:41] fginther, it's not super hard to backport dh-golang btw; just a bunch of perl scripts [21:42] cyphermox, I was waiting for the job to finish before change the connection method, looks like that just happened [21:42] cyphermox, should be back in a few minutes now [21:48] cyphermox, it's back up now [21:49] thanks [22:00] sergiusens, can I get back to you tomorrow on lp:usensord? [22:01] ChrisTownsend, the unity armhf build passed with your PPA [22:17] fginther, sure [22:17] fginther, I will have around 5 more jobs using dh-golang aprox fwiw [22:17] soon more to come [22:26] fginther: got a second? [22:26] thomi, yep [22:27] thomi, I do have a hard stop in 30 minutes [22:27] fginther: before I can get the new autopilot released to distro, I need to prove that it doesn't break anything... [22:27] in the past you made us a jenkins job to do this, I believe? [22:27] I'm trying to figure out how I can do this again, but there are so many 'autopilot' jenkins jobs out there I'm not sure what I should be looking at [22:27] thomi, yes, we did that for the 1.4 effort [22:28] fginther: right [22:28] do you remember which job it was? [22:31] thomi, found it: http://s-jenkins.ubuntu-ci:8080/job/autopilot-trusty-master/ [22:31] sweet, let's see.... [22:32] thomi, it worked by pulling the latest python-autopilot from the experimental ppa [22:32] thomi, is this for python3-autopilot? [22:32] fginther: sure [22:32] fginther: actually, I'm not sure [22:32] fginther: this is for whatever you use normally [22:33] python-autopilot in that case [22:33] yeah [22:34] so target_branch is the project whose tests I want to run... what do I put in landing_candidate? [22:35] landing_candidate is actually the value that holds the branch to test, target_branch is the 'trunk' branch [22:35] sorry, that's confusing [22:36] landing_candidate is the branch to test, target_branch is the 'trunk' branch. They can be identical [22:37] fginther: ok.. and this won't actually land anything though, right? [22:37] thomi, nope, it just buids and tests [22:38] awesome, thanks [22:38] thomi, hope that helps [22:38] fginther: testing it out now [22:40] fginther: are there still only 4 test runners in the pool? Just wondering how far I can push this [22:40] don't want to starve the rest of Canonical [22:40] thomi, yes still have just the 4 [22:40] ok, thanks [22:46] thomi: if you have some time, can you cast a look at the autopilot tests in lp:ubuntu-system-settings on the device please? All but the most basic ones are broken & none of us has the autopilot smarts to figure it out [22:47] Laney: got a link to a failure page? [22:48] thomi: umm, a pastebin, don't know if there was an official run [22:48] thomi: http://paste.ubuntu.com/6586198/ was one robru did yesterday [22:49] * thomi looks [22:49] I guess it's some weirdness around how vrruiz set it up originally [22:50] ok, it looks like maybe the object names have changed or something. I can add it to my list of things to look at, but it's unlikely I'll get to it before tomorrow morning... [22:50] it passes on desktop [22:50] that's probably an important piece of information that I forgot to say :-) [22:58] interesting [23:20] Laney, thomi : yep those failures are on mako [23:22] my brain is too small to understand it