/srv/irclogs.ubuntu.com/2013/09/30/#ubuntu-ci-eng.txt

fgintherthomi, pong?00:27
fgintherthomi, I'll should be back in 60-90 minutes00:30
fgintherthomi, are you there?01:26
thomifginther: am now - you?01:28
fgintherthomi, yo01:32
fgintherthomi, sorry about me email short on details. I'm trying to determine if there is a general pattern for what the ubuntu-ui-toolkit is doing.01:33
fgintherthomi, I know that the retry code I used is not the way to go.01:34
thomifginther: hey01:34
fgintherthomi, good evening01:34
thomifginther: I looked at your branch, but it wasn't clear to me exactly what the problem was you were trying to solve01:34
thomifginther: so maybe we ought to have a hangout about this tomorrow first thing?01:34
fgintherthomi, sorry about that, was a little rushed trying to send you something before I had to leave the house01:35
fgintherthomi, hangout tomorrow would be good01:35
thomifginther: yeah no worries01:36
thomifginther: it'll have to be first thing tomorrow though, since I have an appt in town afterwards01:36
thomifginther: I'll create a calendar event, so we both know when we're free01:36
fgintherthomi, that works for me, please schedule what works for you01:36
thomifginther: scheduled.01:37
thomifginther: our clocks have changed, which means NZ + US now have more overlap01:37
fgintherthomi, \o/01:37
thomipresumably the US clocks will change soon as well, which should make it even easier :)01:38
fgintherthanks for offering to help, hopefully I'll understand the problem a little better tomorrow as well01:39
fgintherhave a good night01:39
thomino worries01:39
loolheya06:48
looldidrocks: hangout?06:50
didrockslool: still answering emails, ok in ~20 minutes?06:50
loolsure06:52
looldidrocks: I'm on07:06
didrockslool: coffee and I'll be there07:07
loolhttps://code.launchpad.net/~bfiller/mediaplayer-app/remove-no-display/+merge/18809207:24
looldidrocks: LP #123258807:52
ubot5Launchpad bug 1232588 in upstart-app-launch (Ubuntu) "--desktop_file_hint still seems required for Mir" [High,New] https://launchpad.net/bugs/123258807:52
didrocksvila: https://plus.google.com/hangouts/_/b31c28f31fcf9e093dc094c78119b1080db93ff708:33
viladidrocks: sry, can't right now,08:35
didrocksno worry ;)08:35
viladidrocks, lool: some notes about the issue you mentioned during the week end: http://paste.ubuntu.com/6174698/08:36
viladidrocks, lool: i.e. I looked, it could be fixed, do we want to or is it overkill ?08:36
viladidrocks: no news about dns migration, will check later08:37
ogra_damn, my daily DLS reconnect seems to now fall exactly into the meeting :(08:41
ogra_*DSL08:41
=== vila is now known as vila-afk-biab
loolvila-afk-biab: There was some news about DNS migation somewhere08:58
loolI heard we were switching DNS/DHCP in some lab since last friday, but can't remember where08:58
loolvila-afk-biab: fixing the changelog: up to you, I just thought I'd pass it on since you had recently fixed the "-" handling08:58
loolvila-afk-biab: There are certainly more urgent things to do though  ;-)08:59
loologra_: lol08:59
loologra_: perhaps you can reboot it in the morning?08:59
ogra_well, or late at night08:59
loolsome folks here have a cron to reboot their cable connection in the middle of the night to have a stable IP during the day08:59
ogra_yeah, my reconnect used to be around 4 or 5am ... not sure why it moved09:00
loolit's tired of waking up so early, like me?09:00
ogra_i guess i'll set up a cronjob09:00
ogra_heh, likely09:00
looldidrocks: Ah there was landing 51 for some click updater stuff already and I was pinned on it, probably when I got pinged by ralsina last week09:02
didrockslool: yeah, I wondered if it's the same09:02
didrockslool: please feel free to remove my duplicate if it's the same ;)09:02
looldidrocks: I think yours covers slightly more, so will keep both09:03
didrocksok09:03
didrockssil2100: while I'm busy doing other things, do you mind landing friends?09:07
didrockssil2100: no need for testing, it's just a Standards-Version bump09:07
didrocks(juts to clean the page)09:08
sil2100didrocks: any testing needed? ACK09:08
sil2100didrocks: done!09:09
sil2100AFK for a quick momen09:09
sil2100t09:09
didrocksMirv: I added request #58, can you look and test that with psivaa?09:10
didrocksMirv: divide the work as you wish, I think someone should take desktop, the other phone09:10
didrocks(the changelog sounds good)09:10
Mirvdidrocks: ok!09:13
psivaadidrocks: Mirv: let me know either way09:14
Mirvdidrocks: there's a blocker bug though since libusermetrics doesn't build, I filed a bug today09:14
didrocksMirv: ok, please ping upstream as well so that they know of it09:15
Mirvbut if excluding libusermetrics, then ok09:15
didrocksMirv: and we can publish without that one09:15
didrocksyep ;)09:15
Mirvdidrocks: I'm planning to, ted is not line yet09:15
didrocksok ;)09:16
Mirvpsivaa: can you the the phone, upgrade the indicators stack and run tests there? the desktop - Unity7 tests - is a bit tricky but so I'm thinking I could run it because I know the pitfalls there and how to interpret the results because we've been releasing unity7 for a long time...09:18
Mirvmy sentences suck today09:18
psivaaMirv: ack will test the phone :)09:19
Mirvpsivaa: thanks!09:19
psivaayw :)09:19
didrockssil2100: Mirv: lool: ok, found out why the customization project wasn't listed. (configuration branch was not refreshed). Should be fine now09:37
didrockssil2100: please work on landing the customization one as well as discussed09:37
didrocksMirv: ok, so I just branch qtsystems and review?09:37
didrocksMirv: ah, we'll need a DEP5 format (so that we know if it's reported upstream) please09:40
didrocksping me once done09:40
=== vrruiz_ is now known as rvr
=== vila-afk-biab is now known as vila
looldidrocks: why don't we just bzr pull the config regularly from jk?10:06
didrockslool: jibel convinced me to not do it, but TBH, I don't see anything block it (on that side at least)10:07
didrockslool: the whitelist still need to be manual (on snakefruit), but on magners, I think we could10:08
jibellool, didrocks IIRC the problem is that if you reconfigure jobs while they are running, jenkins aborts them (if that's what your referring to)10:10
jibelyou're10:11
didrocksjibel: this is just about bzr pull on magners I guess10:11
didrocksnot deploying the config I guess10:11
jibelif it is just a pull that's fine10:11
didrockslet's cronify that then10:11
jibelbut at some point we talked about reconfiguring automatically10:11
didrocks(done)10:12
didrocksyeah10:12
didrocksfor that we need to pull out of jenkins a bunch of stuff10:13
didrocksbut I think this plan is now dead…10:13
didrockssil2100: back?10:15
loolwhy don't we do that as part of starting the tasks or something?10:15
loolor just change the jobs to read from the config10:15
loole.g. instead of a "complex" job definition, you just make a thin wrapper around config/$job-name10:16
didrockslool: at first, we didn't want to rely on the config files, but yeah, it happens it's what needed now10:16
didrockslool: but TBH, this is not something we can tackle on anymore before V1, we are pulled in a lot of other directions ;)10:16
viladidrocks: what's V1 here ?10:21
vilacu2d V1 ?10:21
didrocksvila: phone version 1?10:21
vilaoh, bah, silly me10:21
didrocks:)10:21
viladidrocks, lool: I know fginther takes care of having no jobs in flight while reconfiguring for upstream-merger. That's certainly something the CI engine should do transparently in any case, I for one will push for it, as well as leaving as little control to jenkins as possible for scheduling (been there, suffered from that, don't want to try again ;)10:24
didrockssil2100: how was friends published? Did you check? I see everything in red on that stack10:30
Mirvdidrocks: DEP5 added to qtsystems.10:30
* vila catches up backlog10:30
* cjwatson adds landing ask 123 for click 0.4.910:31
vilalool: yeah retoaded said he will attempt to further test the dns migration during the week end but I got no feedback so I'll re-check later and will relay the feedback here10:31
didrockscjwatson: I guess this is backward compatible and tested against all the other click packages, even if some were not yet landed? (http://people.canonical.com/~platform/cu2d/results)10:34
didrocksMirv: sponsored, thanks!10:34
Mirvthanks!10:34
cjwatsondidrocks: Should be entirely backward-compatible, though I'm reflashing now to run tests10:34
didrockscjwatson: ok, feel free to upload once the tests are done (filing landing ID 60)10:35
cjwatsondidrocks: Are the .click files for the unlanded packages available anywhere?  (Though TBH I think it would be fine to test against the ones in http://people.canonical.com/~ubuntu-archive/click_packages/ - AFAIK they're constructed the same way)10:35
didrockscjwatson: ah, I only know about the .debs that are waiting (see the *-click-* packages in http://people.canonical.com/~platform/cu2d/results), not sure about the .click one TBH10:36
didrocksMirv: yw ;)10:36
cjwatsondidrocks: Oh, it won't affect that at all, but I'll run what tests I can find10:37
didrocksthanks ;)10:38
didrockssil2100: also, have you disabled autopilot the other day for real? I see still some builds in the ppa10:38
didrocksseb128: mind merging back your changes to indicator-keyboard trunk please? ;)10:39
seb128didrocks, waiting for somebody to approve https://code.launchpad.net/~seb128/indicator-keyboard/saucy-manual-upload/+merge/188065 since friday...10:43
seb128didrocks, not sure why the CI didn't kick in again after I pushed the new revision though10:44
didrocksseb128: ah, you proposed that back, thanks! seems upstream isn't responsive then.10:44
seb128didrocks, well, I think they were waiting for CI10:44
didrocksseb128: what about kick in? it doesn't find the latest revision in the archive10:44
didrocksah10:44
didrocksupstream merger10:44
seb128right10:44
seb128let's just approve it10:44
didrocksyeah10:44
seb128can you do that?10:44
seb128thanks ;-)10:44
didrocksdone ;)10:45
didrocksthanks to you!10:45
didrockssil2100: I'll go soon exercising, I hope you will be back once I'm back so that you can process friends +  ubuntu-touch-customization-hooks updates (also see my other questions)11:01
didrockssil2100: also, it will be nice if you can evaluate the settings stack at the same time11:01
sil2100ACK11:26
sil2100didrocks: I published the friends stack, as when I was publishing the stack it was still waitonstacks11:27
didrockssil2100: maybe try rerunning it?11:28
didrocksor restoring the .bak11:28
sil2100hmmm11:28
sil2100I wonder what happened11:29
sil2100cp: cannot stat `/var/lib/jenkins/cu2d/work/saucy/friends/*_friends-app.xml': No such file or directory11:30
didrockssil2100: interesting, maybe you just got unlucky and published just after the stack was unlock?11:31
didrocksunlocked*11:31
sil2100didrocks: should I re-run the whole stack normally or just do a force publish again?11:34
didrockssil2100: I think restore the previous .bak11:34
didrocksand then try publishing11:34
didrockssil2100: better?11:39
sil2100didrocks: tried restoring the .bak, but it seems I don't have some perms like: cp: cannot open `friends.bak/libfriends_0.1.2+13.10.20130929.1-0ubuntu1_source.changes' for reading: Permission denied11:40
sil2100didrocks: is it ok without those?11:40
didrockssil2100: should be11:40
didrocksas long as you have the .project11:40
sil2100hmmm11:44
sil2100http://10.97.0.1:8080/view/cu2d/view/Saucy/view/Friends/job/cu2d-friends-saucy-3.0publish/42/console11:45
sil2100I wonder whatsapp11:45
didrockssil2100: bzr: ERROR: Cannot lock LockDir(file:///var/lib/jenkins/cu2d/work/saucy/friends/libfriends/.bzr/branch/lock): Permission denied: "/var/lib/jenkins/cu2d/work/saucy/friends/libfriends/.bzr/branch/lock/yyzntm7kvu.tmp": [Errno 13] Permission denied: '/var/lib/jenkins/cu2d/work/saucy/friends/libfriends/.bzr/branch/lock/yyzntm7kvu.tmp'11:45
sil2100Permissions look ok,11:45
didrockssil2100: are you connected to mangers? I can't apparently11:47
sil2100didrocks: it takes around 1 minute to connect ;/11:47
sil2100So you need patience11:47
sil2100drwxrwxr-x+ 2 desktop-team jenkins 4096 Sep 29 21:23 lock11:47
didrocksoh11:48
didrocksyou did cp11:48
didrockswhy not mv?11:48
didrocks(or even cp -p :p)11:48
sil2100I did a mv of the old one, and then cp -a ;)11:48
sil2100Ok, so I'll do a mv then!11:48
didrocksweird, if you cp -a, desktop-team shouldn't be the owner11:49
didrocksso yeah, rm -r friends11:49
didrocksmv friends.bak friends11:49
sil2100cp -a friends.bak/ friends11:49
sil2100k ;)11:49
didrockssil2100: so, we're good now?11:58
sil2100didrocks: with friends, yes! Waiting for misc to finish12:01
didrockssil2100: ok, did you read about AP which seems to still be in the ppa and building?12:01
didrockssil2100: did you forgot to deploy the backout on Friday?12:01
didrocks(I think we should remove it again and remove it)12:01
didrockssil2100: while you are waiting on misc stack, maybe you can start the settings one? (running AP tests for ubuntu-system-settings-online-accounts + some dogfooding for system-settings?)12:02
sil2100Aye aye! I was almost sure I was deploying *something*, but I guess I didn't!12:04
sil2100heh12:04
sil2100Maybe I redeployed the wrong branch12:04
=== alan_g is now known as alan_g|lunch
didrocksmaybe…12:07
didrocksMirv: psivaa: how are the indicators tests looking?12:10
Mirvdidrocks: argh! I just figured out you said 'changelog' looked fine, not the 'changes' (packaging changes) regarding indicators!12:10
Mirvdidrocks: so published, a moment ago, but..12:11
didrocksoh?12:11
didrocksok, let me look at the FS12:11
Mirvdidrocks: so this's because the cu2d was read, and it didn't show the packaging changes under the publish stack12:11
Mirvs/read/red/12:11
didrocksMirv: yeah, there is a trick for that12:11
didrocks(foo + AUTO_PUBLICATION)12:11
didrocksMirv: it was red because of the tests?12:11
Mirvit's http://10.97.0.1:8080/view/cu2d/view/Saucy/view/Indicators/job/cu2d-indicators-saucy-3.0publish/lastSuccessfulBuild/artifact/packaging_changes_indicator-bluetooth_0.0.6+13.10.20130927-0ubuntu1.diff + http://10.97.0.1:8080/view/cu2d/view/Saucy/view/Indicators/job/cu2d-indicators-saucy-3.0publish/lastSuccessfulBuild/artifact/packaging_changes_indicator-datetime_13.10.0+13.10.20130927-0ubuntu1.diff + http://10.97.0.1:8080/view/cu2d/view/Saucy/vi12:11
didrocksok, let me look :)12:12
Mirvdidrocks: red because of libusermetrics build failure. I did autopilot locally and so did psivaa12:12
didrocksah ok ;)12:12
didrocksMirv: ok, it seems to be the same thing everytime, so yeah, looking good12:13
Mirvyeah it looked good to me too, but sad still to me. I'm writing a note about special case handling with foo + AUTO_PUBLICATION.12:15
loolralsina: around?12:17
didrocksMirv: yeah, this one is really tricky ;)12:17
Mirvdidrocks: "do no use manually" says the hint for that :)12:17
didrocksMirv: when you run it, ensure that the stack is not running at all12:17
didrocksthere is no safety net12:17
didrockshence this "do not use manually" :p12:17
ralsinalool: here, good morning!12:18
loolralsina: good morning!12:18
loolralsina: we would like to land click-package stack, that is everything listed in the first section of http://people.canonical.com/~platform/cu2d/results12:18
Mirvright. I tend to not to ever do anything if stack is running anyhow (or stop it before).12:18
loolralsina: a) are you aware of any issue  b) do you miss anything there before it may land  c) any suggested tests we should focus on?12:19
loolmy plan was to update to these, test installing + running + removing a click, and checking for click updates, albeit I always get "No updates found"12:19
ralsinalool: the main thing bugging me there is that testing click-updater gave us conflicting results. But since it was totally broken, even the worst case is an improvement12:19
ralsinalool: well, to test for updates, we are installing some old clicks via pkcon then checking12:20
loolcan I download old clicks from somewhere?12:20
ralsinalool: we have a few, let me get you the URLs12:20
loolthanks12:21
loolone is enough I guess12:21
loolsmall  :-)12:21
ralsinalool: there is a copy of old hello world somewhere, gatox is loking for it12:21
gatoxlool, hi, ralsina asked me to give you this old clicks: http://ubuntuone.com/0mHE7D9wQLkd2DXBJROxTi - http://ubuntuone.com/3KnGeEQjvt9nXul0JLLDew12:22
loolgatox: ty12:23
mandellool, ralsina can I propose landing a critical fix first in the ubuntu-download-manager (that was there for a long time but had no reviews?)12:24
mandellool, ralsina ideally, files should not cleaned after they have been downloaded, yet in case they are not I added => https://code.launchpad.net/~mandel/ubuntu-download-manager/throw-error-exists/+merge/18752712:25
sil2100didrocks: eh, it seems cordova-doc build is hanged ;/12:28
sil2100didrocks: "Started 6 hours ago"12:28
sil2100didrocks: and it doesn't move at all12:28
sil2100I'll abort it maybe?12:28
sil2100Ok, no, wait, it moves12:28
sil2100But why 6 hours?12:29
LaneyCan someone make the CI re-run on https://code.launchpad.net/~laney/ubuntu-system-settings/measure-size-critical-fix/+merge/188020 ?12:31
loolmandel: looking12:32
mandellool, take into account that in the case where the file is not removed by the client we are appending to it, and that is a terrible idea. It is also the possible cause to see those 200% progress results12:32
loolLaney: done12:33
Laneymerci12:33
ralsinalool: let's land that as well, but it has no correlation with the click stack, it can go in any order (if that helps)12:34
cjwatsonsil2100: (if this didn't get resolved while my connection was bouncing)  Have you checked that the logtail isn't progressing?12:36
loolmandel: hmm looking at the diff, I'm not too convinced with the file handling stuff12:37
cjwatsonsil2100: In fact, it looks like the logtail *is* progressing, so it's not huge, just slow due to png compression12:37
cjwatson*not hung12:37
mandellool, what worries you?12:37
loolmandel: that the download service itself is running unconfined and writing to computed pathnames12:37
loollike appending an id at the end of a filename to create a new one12:38
sil2100cjwatson: as I mentioned: "Ok, no, wait, it moves", but I wonder why it takes 6 hours already12:38
loolI'm not sure how secure and private the ids are, but it seems like this could be abused with a symlink created by a malicious app to write outside of its app dir12:38
sil2100cjwatson: since damn, it's i386, 6 hours for one package is a LOT12:38
cjwatsonsil2100: png compression can be enormously slow on systems with lots of pngs12:39
cjwatsonunfortunately12:39
cjwatsons/systems/packages/12:39
mandellool, the downloader is, when running as a system app, writing anywhere it wants, hence the error, the appending is just happening when we are running for a request of a confined app, in that case we are writing to XDG_DATA/APP_ID/filepath(uuid if needed)12:39
sil2100cjwatson: thanks12:39
cjwatsonsil2100: you can make sure all pngs are already optimised in the source (or that it doesn't make enough of a difference in size to be worth it), and then export NO_PNG_PKG_MANGLE=1 while running dh_builddeb12:40
loolmandel: exactly, so if I'm app==APP)ID, what happens if I create a filepath-uuid -> ~/.upstart/my-job.conf symlink and make download manager download an upstart job?12:40
cjwatsone.g. override_dh_builddeb:\n\tNO_PNG_PKG_MANGLE=1 dh_builddeb12:40
=== thostr_ is now known as thostr_brb
loolmandel: anyway, to box this I propose we land whatever you think are good fixes that pass the testsuite and aren't regressing click downloads or system-image testsuite; we can worry about the confined app use case later and get a security review12:42
loolwe need to ask security team about apparmor profiles anyway12:42
mandellool, so, you mean that you point me to download an upstartjob, then if you are confined I'll write it to XDG_DATA/APPID/my-job.conf12:43
mandellool, what is wrong with that?12:43
mandellool, then, I think we should land that branch12:43
loolmandel: the symlink part was that it would write the download to ~/.upstart/my-job.conf for an unconfined app12:50
mandellool, we can always return an error if the file is present12:50
mandellool, and let app developers deal with it12:51
loolmandel: right; you need to do the write in a secure way, race free12:51
mandellool, can you add a comment and I'll update the branch so that if present, we raise an exception12:51
loolmandel: typically if(!exists(file)) {write(file)} is racy12:51
mandellool, I fear that the click scope is not cleaning the files and that will break it :-/12:52
=== alan_g|lunch is now known as alan_g
loolmandel: anyway, I dont think this relates, reading the diff just made me realize this isn't ready for prime time for the confined apps12:52
mandellool, so, I'll add that fix and we won't land it 'til I have checked that the click-scope is doing things correctly12:52
fginthermorning12:52
loolwill log a bug12:52
mandellool, thx12:52
loolmandel: let's land your fix though12:52
mandellool, ok, I'll pay attention to that, we need to ask security, I'd like their input for this12:53
loolmandel: it's non-trivial to get it right, e.g. that applies when creating the download directory too12:54
cjwatsonmandel: I believe you're right; I've noticed in the past that if I use the click scope to install a package, then manually make it look like a lower version on the filesystem, then use click-updatemanager to try to upgrade it, that the download manager creates a download file consisting of two copies of the package concatenated, and the click-updatemanager reports 200% progress and then fails to install the invalid file12:54
mandelcjwatson, yeah, that is an issue, both in the downloader and the scope, that branch fixes it just changing the downloader side12:55
loolmandel: https://bugs.launchpad.net/ubuntu-download-manager/+bug/123314912:55
ubot5Ubuntu bug 1233149 in ubuntu-download-manager "Must write downloads of confined apps securely" [Undecided,New]12:55
mandellool, thx12:55
* mandel goes to have some food12:55
looldidrocks: so the above important bugfix from https://code.launchpad.net/~mandel/ubuntu-download-manager/throw-error-exists/+merge/187527 is pending merging, but is part of click-package stack; I'd like to get it in too, that way we update all packages of the click-package stack today  ;-)12:58
loolmy only problem is that I'm falling asleep12:58
loolralsina: gah, would you please a) top approve https://code.launchpad.net/~mandel/ubuntu-download-manager/throw-error-exists/+merge/187527  b) tell your team to top approve stuff and not just approve it or upstream merger doesn't merge  c) give me / others permissions to top approve your branches?  Thanks!  :-)13:07
ralsinalool: approved, mandel forgot to do it after it was agreed to land it13:07
cwayneasac: ping13:09
cjwatsoncwayne: He's on holiday13:09
cwaynecjwatson: ah, thanks.. a  /nick asac-holiday could've been helpful :)13:10
cwaynedidrocks: ping13:10
sil2100ogra_: hi! Do you have a moment ;)?13:13
ogra_sil2100, shoot13:14
sil2100ogra_: I need core-dev ACKs for: http://10.97.0.1:8080/view/cu2d/view/Saucy/view/Settings/job/cu2d-settings-saucy-3.0publish/lastSuccessfulBuild/artifact/packaging_changes_ubuntu-system-settings_0.1+13.10.20130930-0ubuntu1.diff and http://10.97.0.1:8080/view/cu2d/view/Saucy/view/Settings/job/cu2d-settings-saucy-3.0publish/lastSuccessfulBuild/artifact/packaging_changes_ubuntu-system-settings-online-accounts_0.2~+13.10.20130930-0ubu13:14
cjwatsonsil2100: ack on the first, nack on the second13:16
cjwatsonsil2100: Shouldn't be "Section: debug"13:16
cjwatsonHah, other packages are like that, but it's still wrong :)13:16
ogra_thanks for checking cjwatson ... i seem to not be able to reach that machine13:16
sil2100cjwatson: will you give a conditional ACK if I promise to prepare a merge for changing that? ;)13:17
cjwatsonsil2100: I'd suggest just deleting the Section: line13:18
cjwatsonsil2100: I'm just checking some context for the rest of it13:18
cjwatsonsil2100: Surely you need to handle dh_python2/dh_python3 integration too?13:19
cjwatsonsil2100: It's not generally OK to just dump files into /usr/lib/python*/dist-packages/ without the use of any Python helper13:19
sil2100cjwatson: well, sorry, but that's what 90% of our touch -autopilot packages do right now, so even if it's not really ok, that's the standard that is currently used13:21
sil2100cjwatson: ;)13:21
cjwatsonI'm afraid that doesn't convince me to ack it13:21
sil2100cjwatson: I'm not the one doing the packaging here13:22
cjwatsonsil2100: I think you basically need (a) Build-Depends: python (>= 2.7); (b) dh $@ --with=python2; (c) add ${python:Depends} to Depends13:22
cjwatsonThat still doesn't convince me to ack it :)13:22
cjwatsonYou aren't going to convine me to ack something incorrect13:22
cjwatson*convince13:22
sil2100cjwatson: ok then, I'll wait for didrocks to decide on what to do with this release13:22
sil2100cjwatson: then why didn't the release team reject all the other packages if they are incorrect?13:22
cjwatsonThe above sequence should be basically all you need, simple enough13:22
cjwatsonsil2100: I didn't see them :-P13:22
sil2100cjwatson: right13:22
ogra_the wonderful world of auto-approval13:23
sil2100didrocks: ping, the settings stack is blocked for now, see ^ for context13:23
cjwatsonMight also need "X-Python-Version: 2.7" since autopilot.pro hardcodes 2.713:24
cjwatsonIn which case Build-Depends: python (>= 2.6.6-3~) is actually more correct13:24
cjwatsonGiven that this is a five-minute build, this should take about ten minutes to fix13:25
cjwatsonI'll go and fix overrides for the other autopilot packages in the archive; that "Section: debug" has a good chance of causing problems once ddeb support is fully implemented in Launchpad13:26
cjwatson(As in, the packages will probably vanish from the primary archive)13:26
cjwatsonOh good grief, all *-dbg are in debug.  I wonder if I misunderstood that section ...13:28
cjwatsonLooks like I did.  Hmm.13:28
cjwatsonsil2100: I withdraw the "Section: debug" part of my comment; I think it might perhaps be better for *-autopilot to be in some other section, but we can override it in the archive if we so choose and it isn't a blocker of any kind at the package's end13:30
didrockslool: ack13:35
didrockssil2100: cordova-doc: please abort I guess13:35
didrockssil2100: oh, no, it doesn't. Hum, the build seems very long13:36
didrockssil2100: so, let's try to release it so that we don't have it again13:36
sil2100didrocks: so we should wait for it to finish? (7 hours running already)13:37
didrockssil2100: yeah, and then, we ensure it's in13:38
sil2100ACK13:38
didrockssil2100: keeping a look on that?13:40
sil2100didrocks: I'll keep a tab opened, hope it will finish building till the end of the day!13:41
didrockssil2100: cjwatson: we have examples in other packages how we handle building with python13:43
didrockssil2100: look at unity8 (debian/rules)13:43
didrockssil2100: so I doubt 90% of the packages are doing that, for those I looked at least…13:43
cjwatsonI'm preparing a patch for friends-app now by way of example13:44
sil2100didrocks: well, notes-app, gallery-app, friends-app13:44
sil2100didrocks: messaging-app as well13:44
cjwatsonAnd yeah, unity8 is better13:44
sil2100didrocks: I would say that all *-app packages that we daily release have that13:44
cjwatsonSo let's fix them and not get it wrong in new code13:44
sil2100didrocks: so it's no wonder someone added the autopilot packaging in the same way13:45
cjwatsonLike I say, it's really not hard, I'm preparing a patch that people can crib from13:45
didrockssil2100: yeah, same issues than autotools…13:45
didrockscjwatson: +113:46
sil2100didrocks: if you guys say it's invalid, well, I won't argue as yes, it's a bit strange to not use setup.py - I only wonder how we were able to push so many packages with this into the archive, I thought that it's acceptable since that happened13:48
didrockssil2100: for unity8, I've implemented a setup.py upstream for this13:49
cjwatsonNot using setup.py is also weird, but it wasn't what I commented on.13:49
cjwatsonsil2100: https://code.launchpad.net/~cjwatson/friends-app/dh-python2/+merge/188338 - shouldn't be harder than that for the others13:51
cjwatson(Of course some other core dev should probably eyeball that)13:51
sil2100cjwatson: is python:any valid? Just asking since somehow somewhere I remember we had a problem with using that (I might me mixing up problems though)13:53
seb128sil2100, https://launchpad.net/ubuntu/+source/qtbase-opensource-src/5.0.2+dfsg1-7ubuntu5 is what you remember13:55
sil2100seb128: thanks! heh, something similar was echoing in my ears and I couldn't differentiate it from the noise13:56
LaneyIs there a hidden point to the libfriends upload that just bumps standards-version?14:01
fgintherretoaded, dns resolution is failing on the qa machines (i.e. ps-saucy-server-amd64-1, ps-android-sandybridge). Is this a known issue?14:04
cjwatsonsil2100: Yes, it is valid.  Some details of it were only fixed up in the last few months14:04
sil2100cjwatson: I get a lintian error on your branch - is that normal?14:05
sil2100cjwatson: E: friends-app source: missing-build-dependency-for-dh-addon python2 => python | python-all | python-dev | python-all-dev14:05
retoadedfginther, let me poke but it may be related to me changing the DHS/DHCP services for those networks.14:05
cjwatsonEr, by which I mean "two weeks ago"14:05
cjwatsonsil2100: Yes, that's Lintian being out of date and can be ignored14:05
cjwatsonsil2100: FWIW, while the change seb128 pointed to would have been blocked by the infrastructure bug that I fixed on 16 September, it would have been blocked anyway because the previous state was just wrong; you can only use :any dependencies if the target of the dependency is Multi-Arch: allowed14:07
cjwatsonsil2100: But python is Multi-Arch: allowed, and is indeed the canonical example of this kind of dependency14:07
cjwatsonWe have quite a few of these in the archive now14:07
sil2100ACK, awesome14:15
sil2100cjwatson: https://code.launchpad.net/~sil2100/ubuntu-system-settings-online-accounts/dh_python2/+merge/188340 <- is the same change ok here?14:22
cjwatsonsil2100: Yep, looks fine14:24
cjwatsonsil2100: That has my ack with that merge14:24
sil2100cjwatson: thanks14:24
cwaynemfisch: next day or two according to pete-woods14:25
* cjwatson sees that security has taken over the build farm for a while14:28
lool== Building click-package stack ==14:28
mandelralsina, lool I just top approve when I have two reviews, that is why that branch was not merged, let me know if you want me to change that :)14:28
cjwatsonUploaded click 0.4.914:28
loolto pick up the ubuntu-download-manager rev that got merged a couple of minutes after the tick14:28
didrocksgreat ;)14:28
ralsinamandel: I top-approved it anyway14:28
loolmandel: If you have a stricter policy than one review that's fine; usually folks reviewing just approve + top approve things when they are happy with it, and the upstream merger jenkins does another approve if it builds and passes the configured tests14:29
mandellool, we usually do 2 human reviews + jenkins bot, if all those 3 are ok then the person that proposes the branch does the top approve (in case the mp had comments for non blocking changes for example)14:30
loolmandel: ok; nervermind then, this is special to your team, and I cant blame you guys for doing one more review than what others do14:33
didrockssil2100: apart from that, testing the settings went ok? so you can publish them both once rebuilt?14:35
didrockssil2100: also, I feel cordova-docs is *almost* merged ;)14:35
didrocksbuilt*14:35
didrockssil2100: also, the customization hooks have progressed?14:36
didrocks(not sure if you followed, but I got why it didn't show up this morning)14:36
sil2100didrocks: testing went fine, ran the AP tests on the device as well, but some of the tests are actually desktop-only14:37
sil2100didrocks: did some dogfooding, all green14:37
cjwatsonYeah, cordova-docs is built now14:37
didrockssil2100: please publish whatever you can, will be helpful before the meeting to know what's still needed for image 7114:38
cjwatsonI think I'll rebalance the i386/amd64 builders back to 4/4, since there's a bit of a queue14:38
cwaynedidrocks: the customization hooks were updated last week but haven't landed yet14:39
cjwatson10 pending amd64 builds14:39
didrockscwayne: yeah, we are working on publishing it14:39
cwaynedidrocks: cool, thanks. just saw customization-hooks and figured i'd give some context :)14:39
sil2100didrocks: will rebuild settings once my merge gets in14:40
didrockscwayne: yeah, no worry, thanks for following! :)14:40
didrockssil2100: ok, so on customization first? or this just need publishing? not sure if your hands are free or not? (there is a new package pending)14:41
sil2100didrocks: I can tackle ubuntu-touch-customization-hooks now, any additional tests that need to be ran for that?14:41
bfillerfginther: we are quite blocked by otto autopilot failures on multiple MR's14:42
didrockssil2100: I don't think there is any TBH14:42
didrockssil2100: did you fix AP trunk btw?14:42
didrockssil2100: I mean, removed from the ppa and deployed the removal?14:42
bfillerfginther: sounds like you and omer are discussing a plan but we need this fixed asap. have a ton of stuff to land14:43
sil2100didrocks: by fix you mean, removed? Yes14:43
sil2100didrocks: it's removed from the PPA and disabled in the stack14:43
fgintherbfiller, hey, we discusses a potential plan this morning with omer and iftikhar to disable the otto tests until the underlying problem is fixd14:43
didrockssil2100: thanks a lot man :)14:43
bfillerfginther: sounds good, can we do that now?14:43
didrocksfginther: what's the issue?14:43
sil2100didrocks: I'll publish misc - checking what components are in ;)14:43
didrocksgreat!14:44
fgintherbfiller, yes, trying to, unfortunately there is a network issue blocking everything else at the moment14:44
fgintherbfiller, but will update the jobs as soon as possible14:44
fgintherdidrocks, it looks like tab switching is unreliable - https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/123285714:44
ubot5Ubuntu bug 1232857 in Ubuntu UI Toolkit "Autopilot tests failing when objects disappear" [Undecided,New]14:45
sil2100didrocks: juust... cordova-docs is still pending publication ;/14:45
bfillerfginther: thanks14:45
didrockssil2100: yeah, do the customization stuff and you can switch on that one just after I guess14:45
didrocksthen 2 down!14:45
didrocksfginther: which autopilot version are you using?14:46
fgintherdidrocks, 1.3.1+13.10.20130930-0ubuntu114:47
didrocksfginther: where are you taking it?14:49
didrocksfginther: it's been multiple days we have issues with AP in daily-build ppa FYI14:49
didrocksso it's now removed14:49
didrocksplease retry before disabling (now that this version is removed)14:49
fgintherdidrocks, yes, that's from the ppa14:49
fgintherdidrocks, will give it a fresh try14:49
didrocksfginther: please keep me posted14:50
looldidrocks: I intend to push mediaplayer-app from media stack alone (by doing the rename .project dance); any objection?  this is just the NoDisplay=true removal to fix activation from scopes14:50
didrocksbfiller: "fix for flaky autopilot tests", really really really? \o/14:50
didrockslool: ok, you did run the mediaplayer-app AP tests?14:51
=== greyback is now known as greyback|food
didrockslool: not sure if it's using unity8 to test scope activation14:51
sil2100didrocks: is cordova-docs preNEWed?14:52
didrockssil2100: I think I did, let me look again at it14:52
didrocks(looking at the last commits)14:52
sil2100didrocks: if yes, then I'll publish it along with ubuntu-touch-customization-hooks now ;)14:52
didrockssil2100: my requested change is in, so +114:53
didrockssil2100: refreshed the whitelist as well14:53
bfillerdidrocks: indeed!14:59
didrocksbfiller: you will get in front of the queue for image 72 I guess :p14:59
didrocksbfiller: will release the ubuntu-keyboard back (we just fixed trunk with mismatch in the archive)14:59
looldidrocks: I did not run AP tests; they don't pass for sure (bug filed) since the removal of SceneSelector; however the change is trivial in the .desktop and I ran with this change locally15:00
loolbfiller: (did you see my bug about mediaplayer AP tests?  ;-)15:00
bfillerdidrocks: thanks15:00
didrockslool: I just wanted to ensure we don't regress any other tests we can have in unity8 in the way we launch apps15:00
bfillerlool: we supposedly skipped the tests for scenne selector, let me check with renato15:01
looldidrocks: this doesn't change the way we launch apps though, it changes just the mediaplayer-app .desktop file so that upstart-app-launch stops refusing to launch it15:01
didrockslool: ok, ack then15:02
loolbfiller: Yes, some indeed seemed disabled, but at least another one seems related and fails https://bugs.launchpad.net/ubuntu/+source/mediaplayer-app/+bug/1233020 and there's another failure where I can't tell whether it relates15:02
ubot5Ubuntu bug 1233020 in mediaplayer-app (Ubuntu) "Autopilot tests not passing without SceneSelector" [Undecided,New]15:02
bfillerlool: what environment did the tests fail on?15:03
loolbfiller: upstream merger15:03
bfillerlool: is that the VM based environment or running directly on device? just curious as we are seeing VM issues that fginther is reporting and wondering if this related15:04
loolbfiller: I don't know, I think there's a link in the bug15:04
didrocksbfiller: TBH, I think you are getting AP issues, not VM issues (but let's see if Francis confirms)15:04
loolright, there's a link thre15:05
loolso I wanted to play with click stack, but download-manager didn't build on amd64 yet15:05
cjwatson15:38 <cjwatson> I think I'll rebalance the i386/amd64 builders back to 4/4, since there's a bit of a queue15:06
cjwatson15:39 <cjwatson> 10 pending amd64 builds15:06
cjwatsonlool: ^-15:06
bfillerloo, didrocks : I think this is the same problem that is preventing many of our MR's from landing on different apps15:06
bfillerlool: ^^15:06
bfillerlool: can you run the test on a device? I bet it will work fine15:06
loolcjwatson: thanks15:07
loolbfiller: not right now15:08
loolbfiller: will try later, if you beat me just say so in the bug report and close it15:08
loolbfiller: and poke fginther about it I guess :-)15:08
=== greyback|food is now known as greyback
loolbfiller: http://reports.qa.ubuntu.com/smokeng/saucy/touch_ro/4470/mediaplayer-app-autopilot/ seems to back your analysis though15:10
* didrocks will bet on AP trunk15:10
sil2100didrocks: *sigh* my settings merge got rejected by jenkins15:11
sil2100A unit test failure ;/15:11
looldidrocks: so I just set FORCE_PUBLICATION, nothing else, and rebuild with renamed .project files?15:11
cjwatsonsil2100: As was my friends-app merge for some autopilot-related reason I can't sort out.  Don't know if it's the same as lool's failure above15:12
fgintherlool, bfiller, those failed tests were on the hardware env15:12
cjwatsonhttps://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-saucy/480/testReport/junit/friends_app.tests.test_timeline_view/TestTimelineView/test_toolbar_with_mouse_/15:12
didrockslool: FORCE_PUBLICATION, then once in, rename the .project file, and rebuild with "foo" to recreate the state15:12
didrockssil2100: argh, can you get some help upstream?15:12
looldidrocks: Gah, getting this http://10.97.0.1:8080/job/autopilot-saucy-daily_release/2236/15:13
looldidrocks: seems like the same issue as upstream merger15:13
loolbfiller: ^15:13
loolso real failures15:14
didrockscjwatson: lool: AP is the one in the archive, so different15:14
loolMismatchError: After 10.0 seconds test on SceneSelector.opacity failed: 1 != dbus.Double(0.0, variant_level=1)15:15
loolthis is definitely one of the bogus tests I intend to ignore15:15
loolok, forcing this one through15:15
didrocks(I mean, it's not AP trunk's fault like what we saw somewhere else)15:16
loolpushed mediaplayer-app, rebuilding media stack with "foo"15:17
fgintherretoaded, any ideas what the DNS issue is? I'm getting lots of backed up jobs. If I need to I'll try to convert them to using IPs15:18
retoadedfginther, all of the VMs/systems that have rebooted should be working correctly. Let me proceed through all of you running VMs to a) make sure they are getting their IPs from the new DHCP server and b) are resolving correctly.15:20
fgintherretoaded, do we need to reboot ps-android-sandybridge also?15:21
rfowlerfginther: i just rebooted it 1/2 hour ago15:22
didrocksbfiller: do you mind expanding the "use qml bindings for infographics"?15:23
didrocksbfiller: like, is there any bug report?15:24
retoadedfginther, that was already done and it has the correct information. I believe it's problem is that psoglav is using the older DNS server so using the IP for now would be a better option which has been done.15:24
bfillerdidrocks: done15:24
fgintherretoaded, ack15:24
bfillerno bug report, just apps are supposed to use new api15:24
bfillerwhich is qml15:24
didrocksbfiller: ok, sounds good, thanks ;)15:24
bfillernp15:25
doanacasac: is there anything i can do to help get autopilot unblocked from the landing-pipeline?15:25
retoadedfginteher, later this afternoon I will switching the last network (10.97.2.0/24) over to the new DNS/DHCP servers. Once that is done I will need to reach into every system and restart networking so they all get the correct information and then update all of the servers.15:26
didrocksdoanac: asac is on vacations15:26
retoadedIt will be time consuming but will get done.15:26
didrocksdoanac: right now, from our tests, AP trunk regressed all apps + unity8 tests15:26
didrocksdoanac: if you can get a fresh image, update to AP trunk and run all AP apps + unity8 to confirm it, that will be helpful15:27
didrocks(and then fixing ;))15:27
doanacdidrocks: interesting. i'm not sure AP had any functional code changes.15:27
ogra_sergiusens, ^^^15:27
didrocksdoanac: it seems that it regressed one way or another (multiple confirmation by sil2100 and Saviq)15:28
didrocksdoanac: we don't really know more though, had to go back on prod side :/15:28
sergiusensogra_, doanac from what I know, only thing we changed was a hook file which isn't even used with autopilot directly but from the click hook system15:30
sergiusensdidrocks, ^^15:30
doanacsergiusens: there are more changes that have queued up now15:30
sergiusensdoanac, that sucks, can we daily release with only our change?15:30
didrockssergiusens: doanac: quite a lot: https://code.launchpad.net/~autopilot/autopilot/1.315:31
=== Ursinha is now known as Ursinha-afk
sergiusensdidrocks, doanac can we distro patch with just http://bazaar.launchpad.net/~autopilot/autopilot/1.3/revision/336 and http://bazaar.launchpad.net/~autopilot/autopilot/1.3/revision/335 ?15:33
didrockssergiusens: hum, this is getting complicated, can't we work with thomi to get the issues fixed?15:34
didrockssergiusens: because ensuring AP doesn't regress is taking 2 hours, I will rather fix the real version15:34
didrockssergiusens: when/why do you need that right now?15:34
didrocks(just trying to assess)15:34
sergiusensdidrocks, it's blocking click package testing for two weeks now15:35
didrockssergiusens: well, I think we should try to get thomi on board to fix AP issue, maybe that's not a biggie and we can get that in tomorrow15:35
sergiusensdoanac, we can always side fix this and put the hook in phablet-config and make the hooks rerun with a tmpdir15:38
doanacsergiusens: yeah. even with the fix, I'm still seeing something bad. I'm sending you an email now to explain how to re-create it15:39
=== Ursinha-afk is now known as Ursinha
sergiusensdoanac, great, I'll look into it15:41
=== gatox is now known as gatox_lunch
Laneycan someone please retry https://code.launchpad.net/~mterry/unity8/statsWelcomeScreen/+merge/184153 too? The failures all look transient to me15:48
LaneyAlternatively tell me if me clicking on the rebuild link actually works15:50
Laney:-)15:50
sergiusensLaney, rebuild works if logged in to jenkins behind the firewalls15:53
Laneyoh, probably not then15:53
LaneyI have no account there afaik15:53
sergiusensLaney, look at what's on the MR (latest comment)15:54
Laneysergiusens: how timely15:54
sergiusensyeah15:54
Laneythe CI was still busted :P15:54
sergiusensdoanac, saw your email, will look at it after a quick bite15:55
* lool waits for click 0.4.9 to be on ports16:00
didrocksrobru: around?16:02
didrockshttps://plus.google.com/hangouts/_/76bca9c02d41106488aa07d8aae1b40d8b6d09a416:02
robrudidrocks, yep, just up16:02
fgintherLaney, looking16:04
Laneyfginther: mterry pushed another commit, probably will kick itself off now16:05
fgintherLaney, ack16:05
ogra_rsalveti, will we get the last bits of the media stack over night ?16:09
rsalvetiogra_: we hope so, in a few hours actually16:10
ogra_ok16:10
rsalvetiwe can dream, right?16:10
ogra_haha16:10
ogra_yeah16:10
ogra_didrocks, ^^^16:10
didrocks\o/16:15
fginthersil2100, https://code.launchpad.net/~fginther/cupstream2distro-config/add-thumbnailer/+merge/18838316:24
fginthersil2100, adds the thumbnailer which was just discussed16:24
robrulool, ok, so it sounds like you are going to sping image 71 right now, and then wait some hours for me to land a few things, and then spin 72 after that?16:28
loolrobru: we're going to spin 71 and 72 immediately thereafter, then you can land stuff if you like16:28
loolbut need some testing before we can spin 7116:28
looland my phone is just screwed, gah, will take some time to fix16:28
robrulool, hmm, didrocks has assigned me to land a few things for 72 ;-)16:28
didrocksrobru: yeah, try to land as much as you can for 72 (there are quite small things)16:29
looldidrocks: for 73 I guess16:29
didrocksrobru: we can retarget for 73 if you don't finish them for 7216:29
looldidrocks: where should we state the system-image .debs to test?16:29
robrudidrocks, ok, no worries then. I will begin landing now and if they land for 72 or not is fine16:29
looldidrocks: I was thinking a block hint on it, + upload to -proposed would be in order, what do you think?16:29
loolor we can do PPA16:30
didrocksrobru: looks good, maybe some stacks will need for rebuilding (like the apps one I guess)16:30
didrockslool: PPA maybe? It's not that long to build16:30
robrudidrocks, ugggggh, 79 test failures in app stack... fun16:31
looldidrocks: with a ~ppa?16:31
looldidrocks: slangasek can't participate in much system-image testing right now due to a power outage taking his wireless down16:31
didrocksrobru: yeah, please relaunch, the autopilot in the ppa was busted16:32
didrockslool: yeah, ~ppa16:32
didrockslool: argh, of course16:32
loologra_, didrocks: Sorry will take me a while to test because I have to reflash; should be 10-15mn from now16:33
ogra_lool, no hurry16:34
didrockslool: this gives time for robru ;)16:34
robrudidrocks, ok, i believe everything that has been asked of me is building... now we wait ;-) (err, read emails while waiting... :-)16:41
didrocksrobru: heh, thanks!16:41
=== gatox_lunch is now known as gatox
=== fginther is now known as fginther|lunch
didrocksrobru: do I miss anything? (I answered you by email)16:50
robrudidrocks, no, just miscommunication. it's fine i guess16:51
didrocksrobru: ah nice ;)16:51
robrudidrocks, oh, um, is manual publishing disabled? I ran indicator stack and it seems to have published on it's own (I just did a regular build run, it should not have published)16:53
robrudidrocks, http://10.97.0.1:8080/view/cu2d/view/Saucy/view/All/job/cu2d-indicators-saucy/ build 63 is green, i was expecting yellow16:54
loolcjwatson: I'm getting this https://bugs.launchpad.net/ubuntu/+source/click/+bug/1233280 but perhaps there are image building fixes needed along 0.4.9?16:54
ubot5Ubuntu bug 1233280 in click (Ubuntu) "Permission error while removing preinstalled click before installing any other click" [Undecided,New]16:54
loolcjwatson: I mean, there is no /opt/click.u.c/.click in the image16:54
didrocksrobru: I guess indicator juts has nothing to build :)16:55
didrocksand then publish16:55
didrockshum, no16:55
robrudidrocks, hummm, the latest commit from lp:indicator-keyboard did not show up in daily release ppa16:55
didrocksthere is this libubermetric16:55
cjwatsonlool: Huh, no, shouldn't be16:56
didrocksrobru: I think that Mirv messed with the file system maybe there is something wrong16:56
didrockslet me look16:56
cjwatsonlool: Different traceback, I assume?16:56
loolralsina: Hmm I'm getting 404 when long-pressing on dropping-letters; I'm not sure whether it's a bug or just because we renamed it or something, would you mind passing this to the right folks (appstore or click scope I guess)?16:56
loolcjwatson: yes, permission denied16:57
didrocksrobru: in fact the stack is running16:57
loolcjwatson: can't remove it even after installing other clicks16:57
cjwatsonlool: Oh, sorry, that's a new bug16:57
didrocksrobru: however, 64 is a typo of mine16:58
robrudidrocks, yes, i just re-ran it now myself16:58
didrocksrobru: it's not indicator-keyboard, but ubuntu-keyboard16:58
didrocks(fixed the spreadsheet)16:58
robrudidrocks, oh16:58
didrockssorry about that16:58
didrocksjust autotyping ;)16:58
didrocksrobru: it seems you are already rerunning the services stack16:59
robrudidrocks, i just did now that you told me to :-P16:59
didrocksrobru: ahah, you're fast :p16:59
cjwatsonlool: bah, right, <- idiot16:59
cjwatsonlool: can I upload a new click version that fixes just this?16:59
loolcjwatson: yes please  :-)16:59
didrocksogra_: you will need to track that one before image 71 ^17:00
looldidrocks ogra_: ^ I think this regresses removals, so waiting on this one for the image17:00
didrockslool: no worry!17:00
loologra_, didrocks: Going for dinner17:00
didrockslool: enjoy17:00
cjwatsonlool: it'll be http://paste.ubuntu.com/6176317/17:00
cjwatsonafter testing17:00
loolcjwatson: ok17:00
cjwatson(that's hard to read but it's mostly indentation changes - http://paste.ubuntu.com/6176321/ with diff -b)17:00
loolit looked like it was either something like that, or another syscall ld_preload thing for removals, glad it's the former  :-)17:01
* ogra_ is happy to wait ... not on a race :)17:01
loolwow getting an update in update manager17:02
cjwatsonYep, that works17:02
cjwatsonlool: It's not a regression, I might note - I just didn't fix bug 1232066 hard enough in the last upload17:02
ubot5bug 1232066 in click (Ubuntu) "click unregister on preinstalled app causes exception" [High,Fix released] https://launchpad.net/bugs/123206617:02
cjwatsonlool: Oh, actually, you're right, it is a regression17:02
cjwatsonMaybe17:03
cjwatsonlool: Do you get the same traceback after installing other click packages?17:03
loolcjwatson: can't easily tell17:04
loolgetting a pkcon error17:04
cjwatsonlool: You have a traceback in the bug though17:04
cjwatsonSo try   pkcon remove 'com.ubuntu.sudoku;0.4.3;all;local:click'17:04
cjwatsonAnd then paste the traceback from that17:05
=== jdstrand_ is now known as jdstrand
=== psivaa is now known as psivaa-afk
cjwatsonlool: ... any luck?  I have a fix, just would like to make sure I fully understand what's going wrong for you17:10
cjwatsonOh, he said he was going for dinner.  I'll just upload then since it WFM17:11
cjwatsonlool: click 0.4.10 uploaded17:13
loolcjwatson: just back from dinner17:13
cjwatsonYou eat quickly17:14
looljust a plate of spaghettis with bolognaisa17:14
loolsorry, bolognase I guess17:14
loolcjwatson: pkcon remove 'com.ubuntu.sudoku;0.4.3;all;local:click' worked17:14
cjwatsonOK, I don't know what the problem with "can't remove it even after installing other clicks" was then but hopefully this addresses it too17:16
bfillerfginther|lunch: these are the MR's we have that are blocked by current autopilot/CI issues: https://docs.google.com/a/canonical.com/document/d/1u5rQCAshO3ZVfl61MKtw2RF_Y_BvE9AIcjMTNbGWrxA/edit#17:17
bfillerfginther|lunch: please give me an update when these can be unblocked.17:18
loolcjwatson: is it enough to rm -rf /opt/click.ubuntu.com/{*,.*} to get to the same state as a fresh image?17:18
loologra_: will promote click-package stack17:18
loologra_: Actually we can probably do better than what I had proposed for system-image17:19
cjwatsonlool: Might leave cruft around from hooks17:19
cjwatsonlool: Assuming correctly-written hooks, it should be OK if you reboot after doing that17:19
loologra_: a) build now, b) upload system-image, c) build, d) build17:19
looltest from c) to d)17:19
cjwatson(Which has the effect of rerunning hooks)17:19
loolok17:19
cjwatsonlool: But17:19
loolthere is always a butt17:19
cjwatsonlool: Don't run the command you quoted!17:19
cjwatsonlool: .* expands to include ..17:19
loolerf, ok, it was just for IRC17:20
cjwatsonrm -rf /opt/click.ubuntu.com/{*,.click}17:20
loolinteresting, zsh is being slightly more helpful than bash here in expanding .* to things starting with .17:21
loolcjwatson: bah, unapproved17:22
looldesktop seed17:23
loolvia pk plugin I guess17:23
=== bfiller is now known as bfiller_lunch
lool(click 0.4.10 approved by Stéphane)17:35
=== fginther|lunch is now known as fginther
robrulool, do we care about libusermetrics for today's images? i can't get it to build, don't understand the failure.17:44
ogra_robru, make the committer fix it17:45
ogra_and set it back to TODO17:45
robruogra_, yeah, there's already a bug17:45
ogra_if it is broken it is broken ...17:45
loolrobru: isn't that already in the image?17:51
loolrobru: what's the issue?17:51
robrulool, https://bugs.launchpad.net/libusermetrics/+bug/123300317:51
ubot5Ubuntu bug 1233003 in libusermetrics "Tests failing since Sep 27th" [Critical,Confirmed]17:51
robrulool, there is an old one in the image. i'm  just asking if it's important to get it in17:51
loolThe following tests FAILED: 4 - usermetricsoutput-unit-tests (Failed)17:52
loolrobru: it's not particularly important17:52
robrulool, ok, then i'll focus on other things ;-)17:52
loolrobru: it looks like the issue are that some tests trigger valgrind errors17:53
robrulool, i don't know the first thing about valgrind. maybe you can work with pete-woods on resolving that? ;-)17:53
loolrobru: just point him at the failure log on i38617:53
robrulool, well, he marked the bug as fixed even though it's not fixed, so...17:54
loolcjwatson: new click is all fine, thanks17:54
robrulool, ogra_: what is the best way to do a factory reset of an n7? Can phablet-flash be used to wipe the device? i want to clear all user settings and make it as "stock" and default as possible17:57
ogra_phablet-flash ubuntu-system --channel saucy --no-backup17:58
robruogra_, thanks17:58
ogra_(--no-backup is the "wipe" command)17:58
robruogra_, ah, it didn't show up in 'phablet-flash -h'17:58
ogra_yeah, would be in phablet-flash ubuntu-system -h17:59
loolyou need the actual command to see all the --help options17:59
robruahhhh17:59
sil2100fginther: ping!18:06
=== bfiller_lunch is now known as bfiller
fginthersil2100, hello18:07
sil2100fginther: https://code.launchpad.net/~sil2100/cupstream2distro-config/onlinemusic_scope_add/+merge/188417 <- hello, could you take a look, approve and prepare the mergers for it? :)18:07
sil2100fginther: thanks!18:07
fginthersil2100, approved18:12
sil2100fginther: awesome! *hugs*18:12
sil2100See you tomorrow guys18:13
fginthersil2100, later18:13
loologra_: Ok, gtg; could you check for click 0.4.10 to be in saucy in rmadison then build an image ?18:20
ogra_indeed18:20
loologra_: ty!18:20
robrulool, ogra_: hummm, ok, so one of my tasks is to release libunity and unity-scope-home. but unity stack is blocked by 43(!) failing autopilot tests (but the tests are on unity, not libunity or the scope). do you think it's safe to manually copy those two packages from the daily PPA to distro? or should we just punt on that? I reported a bug to unity already18:32
robrusimilar situation with mediaplayer_app AP tests holding back camera-app release.18:39
ogra_build 71 running18:41
ogra_robru, sorry, i have not much clue about the CI infrastructure, cant give you a hint here18:44
robruogra_, ok, well then I have no choice but to not release this then. bug has been filed, will ping appropriate upstreams shortly.18:45
ogra_probably lool can help later18:46
ogra_he said he'd be back18:46
robruogra_, let me know when image 72 is done, i'll help test the image upgrades too18:47
fgintherbfiller, the otto tests are working again after reverting to an older autopilot per didrocks advice. notes, gallery, camera and webbrowser have all passed.18:47
fgintherbfiller, will retrigger recent failing jobs18:47
ogra_robru, yeah, thats still hours away18:48
robruogra_, ok, great. I EOD in 6 hours ;-)18:48
robrufginther, do you think that'll help with mediaplayer-app, as well? https://bugs.launchpad.net/mediaplayer-app/+bug/123141818:51
ubot5Ubuntu bug 1231418 in mediaplayer-app "TestPlayerWithVideo.test_time_display_behavior seems to fail consistently" [Critical,Confirmed]18:51
fgintherrobru, nope, that test was already executed with the autopilot 1.3.1+13.10.20130918-0ubuntu118:56
robrubuh. ok thx18:56
bfillerfginther: please retrigger all the MR's found in that sheet19:23
fgintherbfiller, ack19:23
cjwatsonlool: new click> great19:46
kgunnfginther: just one more time...thanks for moving the mir CI amd tests to the dedicated host machines, that has been a huge plus for us!!!19:51
kgunnits been damn smooth ever since19:51
fgintherkgunn, you're welcome. I was I could have similar results for eveything else19:51
kgunn;)19:52
cjwatsonhttps://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-amd64/70/console is transient.  Can somebody please retry?19:53
cjwatson(from https://code.launchpad.net/~cjwatson/friends-app/dh-python2/+merge/188338)19:53
fginthercjwatson, on it19:57
cjwatsonthanks19:58
robrufginther, do you have any idea why jenkins would fail with a hash sum mismatch here? https://code.launchpad.net/~cjwatson/friends-app/dh-python2/+merge/18833820:00
cjwatsonrobru: that happens occasionally even on the primary archive build daemons due to bad timing20:01
cjwatsonit's a "try again" thing20:01
robrucjwatson, ok, will do20:01
cjwatsonwe cycle the archive so frequently that the odds of happening to hit it in the middle of an update aren't as small as you might hope20:01
robruheh, fginther beat me to it20:01
fginthercjwatson, curious, would it be safe if the job itself immediately did a retry of the apt-get operation?20:02
cjwatsonand even worse for PPAs20:02
robrucjwatson, what do you mean by 'cycle'? are you just referring to the churn of updated packages?20:02
cjwatsonfginther: probably, yes, but you would want to do that only on certain types of failures and have a limit to avoid infloop20:02
cjwatsonrobru: I mean each publisher run20:02
fginthercjwatson, I would likely cap it at 3-5 retries20:02
robruah20:02
cjwatsonthe indexes for a given archive are unfortunately spread over multiple files so it's possible to get an inconsistent view20:03
cjwatsonwe've talked about fixing it, but time ...20:03
fginthercjwatson, I see the problem several times a day now. followed by having to retrigger a job that took 2 hours to make it through CI and it becomes a bit of a headache...20:04
cjwatsonfginther: the PPA publisher runs every 10 minutes or so in practice if there are changes20:04
cjwatsonso the odds of hitting it are probably non-negligible, indeed20:05
bfillerrobru: ping21:01
bfillerfginther: is jenkins down?21:15
fgintherbfiller, not the one I'm looking at21:15
renatofginther, could you take a look on this MR: https://code.launchpad.net/~renatofilho/mediaplayer-app/fix-1231418/+merge/18843321:16
fgintherbfiller, upstream merger and public jenkins are responding for me21:16
bfillerfginther: waiting on this MR too: https://code.launchpad.net/~renatofilho/ubuntu-ui-toolkit/fix-1213046/+merge/18622321:17
fgintherrenato, your MP is at the front of the build queue21:18
renatofginther, thanks21:19
bfillerfginther: many of the MR's in the sheet are still not working21:19
bfillerfginther: in fact don't see any that passed yet21:19
bfillerfginther: this one approved but never merged: https://code.launchpad.net/~ted/gallery-app/single-instance/+merge/18661121:21
bfillerall the browser ones still failing tests21:21
loolheya21:21
fgintherbfiller, one moment21:21
robrubfiller, pong.21:22
bfillerrobru: please see my email re: ubuntu-keyboard21:22
bfillerrobru: it needs a new release as the one you guys did today has issues21:23
robrubfiller, ah, ok. sure thing.21:23
robrubfiller, hummm, some miscommunication it seems. this very morning didier told me that i needed to get a release of ubuntu-keyboard out the door ;-)21:24
bfillerrobru: I know, not sure why exactly. I never requested one. I think it's because he reverted one last week21:25
robrubfiller, yeah. the asks page just says that he's reverting the revert.21:26
bfillerrobru: I think that's right, but in the meantime we've added new things and unfortunately something was broken.. but my MR should fix it21:27
robrubfiller, ok, no worries. as soon as jenkins lands it in trunk, I will kick off a new release build.21:27
bfillerrobru: thank you21:27
robrubfiller, you're welcome. I just subscribed to the MP so I should get notified as soon as it's ready21:28
fgintherbfiller, do you have time for a hangout?21:28
bfillerfginther: yes21:28
plarscamera, address-book, and clock seem to be crashing: http://reports.qa.ubuntu.com/smokeng/saucy/touch_ro/4494/21:31
plarson mako, just camera crashed21:31
plarsah, it's url-dispatcher and camera actually21:31
plarslool, bfiller: ^ any idea on camera? I don't see an update to camera-app in 71, nor to url-dispatcher (but the url-dispatcher one doesn't seem to always happen)21:36
loolplars: can you reproduce the camera crash?21:38
loolplars: I'd try downgrading qtvideo-node21:38
plarslool: likely yes - it happened on both mako and maguro, along with lots of failures21:38
looladdress-book is url-dispatcher21:40
* lool upgrades to 7121:41
plarslool: qtvideo-node didn't seem to upgrade in 71 either21:41
loolplars: that was 70 though21:41
bfillerplars: no idea, we haven't changed anything that would cause that21:41
bfillercould be qtvideo-node21:41
plarsbfiller: something either broke it in 71, or it's just random enough that we got (un)lucky this time but not in 7021:42
mfischinfographics broke in 71 for sure21:51
loolplars: so the error is:21:53
lool#2  0x401179ba in g_logv (log_domain=0x0, log_level=G_LOG_LEVEL_ERROR,  format=format@entry=0xbf18 "Unable to connect to D-Bus: %s", args=...,  args@entry=...) at /build/buildd/glib2.0-2.38.0/./glib/gmessages.c:98921:53
loolfrom the url-dispatcher crash in addressbok21:53
loolplars: this looks fishy; either dbus is going away because everything is going away (session), or there is some bad upstart deps somewhere21:54
loolurl-dispatcher starts on started dbus21:55
loolso I'd rather vote on the former21:55
plarslool: that seems consistent with the log here: https://jenkins.qa.ubuntu.com/job/saucy-touch_ro-maguro-smoke-ubuntu-clock-app-autopilot/92/artifact/clientlogs/url-dispatcher.log/*view*/21:56
plarslool: I'll see if I can reproduce it locally while I'm watching21:56
loolplars: just to finish exploiting the crash file: the message is actually "Could not connect: Connection refused", and the reason it's crashing is because it's using g_error which will cause a crash dump22:09
loolI mean, it basically causes an abort to get a backtrace22:09
* lool => bed22:10
plarsthe camera one is easily reproducible for me, the url-dispatcher one is harder to hit it seems22:17
rsalvetilool: ogra_: will update the seeds and spin a new image23:01
robrufginther, https://code.launchpad.net/~bfiller/ubuntu-keyboard/disable-predictive-text/+merge/188448 humm, no jenkins after 2hrs, can you take a look? thx23:04

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