/srv/irclogs.ubuntu.com/2015/09/29/#ubuntu-ci-eng.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
robert_ancellI'm trying to land https://requests.ci-train.ubuntu.com/#/ticket/405 into the overlay PPA. Do I click publish myself or do I need someone to do that?02:36
bregmarobru^02:38
robrurobert_ancell: you can probably do it03:00
robert_ancellrobru, ok, trying. Thanks.03:04
robert_ancellrobru, do I need to do anything to close that silo now?03:08
robrurobert_ancell: nope, it's done03:10
robruunless the merge fails for some reason03:10
robruoh there's no merge03:11
robruso it's fine then03:11
robrurobert_ancell: wait why did you put xorg through a silo? I don't understand a) why that's needed in overlay and b) why you published without QA.03:12
robert_ancellrobru, it's XMir for the libertine work. They needed it in the overlay PPA - the only process is through silos right? As it's not used anywhere else I just asked the libertine guys to confirm if this version was better than what they had.03:14
robrurobert_ancell: oh ok, as long as it's not seeded in the phone (yet) I guess that's fine03:22
robert_ancellrobru, cool. Thought I'd done something wrong there!03:23
robrurobert_ancell: technically you could just dput directly to the overlay PPA, I don't think using a silo actually did anything meaningful for you here03:23
robert_ancellrobru, we wanted the silo so people could test before we committed to putting it in there03:23
robert_ancellI guess we were just using the CI train as a PPA provider03:23
robert_ancellBut good practise when this becomes a supported feature.03:24
robrurobert_ancell: ok no worries then. I just find it strange when people use silos without MPs, in my mind the whole value of the silo is the auto-merging of MPs.03:24
robert_ancellYeah, understood.03:24
robruoh god04:44
robruI thought I fixed it so it wouldn't spam the channel at startup04:44
robruMirv: watch out for diffs today, i redid the diffing logic from scratch. Should fix the bug where oxide-qt couldn't be diffed but everything else should behave identically, please ping me if anything is different or unusual.05:00
Mirvrobru: ok05:03
Mirvrobru: do you think someone still actually uses https://launchpad.net/~phablet-team/+archive/ubuntu/tools ? the phablet-tools is at https://launchpad.net/~ubuntu-sdk-team/+archive/ppa for vivid + trusty which all devs probably use05:04
Mirvrobru: but no cost either, so I copied from the SDK PPA05:06
robruMirv: tools ppa is the one i use and the only one i ever updated ;-)05:06
robruMirv: am i the only one using it? Yikes05:06
robruMirv: people must be using tools ppa because there were bugs in citrain tool that i fixed and only copied there, nowhere else.05:08
Mirvrobru: hehe, so many PPA:s :) well at least anyone doing apps would have SDK PPA in use.05:08
Mirvrobru: I know, many do have the phablet tools PPA in use from the old times too, and not all of them use our SDK so...05:09
robruMirv: yeah citrain tool used more by Canonical people, it's not really relevant for community app devs05:09
robruMirv: oh apparently there is an oxide in silo 21 ^ so I'm just regenerating diffs to be sure05:14
Mirvrobru: ok!05:19
robruGood god it takes forever to diff05:20
robruDamn! Still didn't work05:20
robruAt this point i wonder if debdiff itself is the problem05:21
=== chihchun_afk is now known as chihchun
robruMirv: actually I lied to you, there is a difference in the new diffs. You'll no longer see Makefiles/CMakelists in the "packaging diffs". I ok'd this with steve, as that info is contained in the full diff anyway. so now "packaging diffs" are really just debian/*05:55
Mirvrobru: thanks, makes sense.06:00
robruMirv: yeah a few people have complained about that, so diffs should be less noisy now.06:01
robrujust trying to reproduce this oxide issue outside the train, bah06:01
Mirv"ok now finally I'll just rebuild... oh crap, oxide-qt has started using private symbols again"06:14
Mirv-> repacking 4GB + uploading 400MB + waiting 12h06:14
Mirvlife of a Qt maintainer06:15
robrufun times06:34
alf_sil2100: Hi! For ci train request 423, we have know top-approved the MP. Please publish it again (or should I just hit the publish button myself?).08:43
sil2100alf_: publishing :)08:56
alf_sil2100: thanks!08:56
sil2100alf_: published o/08:57
=== chihchun is now known as chihchun_afk
Saviqrvr, hey, I'm confirming on a mako now, but I really wouldn't block on the "failure" on silo 38, OOM kills is not an exact science, and you were testing on a device least likely to do it09:28
Saviqqtmir in that silo is just a packaging change anyway09:29
rvrSaviq: Yeah, I was surprised that the silo had something to do, but I tested without it in the same device, and webbrowser-app was killed. Installed the silo again, and webbrowser (as well as the other apps) weren't killed.09:30
Saviqrvr, if *no* apps are killed means we're doing better on memory usage09:31
Saviqjust launch more of them, open more tabs in the browser, monitor the free memory, it must happen at some point09:31
Saviqany case, I'm just trying on mako that's more likely to hit memory limits09:31
rvrSaviq: http://paste.ubuntu.com/12610164/09:32
rvrOh, shit, it was cut09:32
Saviqrvr, so yeah, unity8 is at 3.2%, dash at 4.1%, apps barely cross the 1% barrier, arale just has plenty of memory is all09:33
Saviq`free -m` would say a bit more about how much memory's left09:33
rvr             total       used       free     shared    buffers     cached09:35
rvrMem:          1892       1850         42         24         42        10609:35
Saviqrvr, yeah, you've still 42 megs available, launch more apps! :)09:35
Saviqand actually, IIRC caches will get freed before OOM kicks in09:36
Saviqrvr, there was quite a bit of static code fixes in one of our unity8 MPs, that could actually reduce memory usage enough to cause what you're seeing09:38
rvrSaviq: Finally, webbrowser-app was killed09:38
=== marcusto_ is now known as marcustomlinson
Saviq\o/09:38
Saviq:D09:38
rvrlol09:38
rvrSaviq: Ok, approving the silo :)09:42
Saviqrvr, great, thanks09:43
Saviqtrainguards, Icanhaspublish here https://requests.ci-train.ubuntu.com/#/ticket/410 please?09:47
=== marcusto_ is now known as marcustomlinson
MirvSaviq: ok, let's see09:50
MirvSaviq: I wonder why it says on trello both passed and still has "blocked" tag too09:51
Mirvrvr: should the tag be removed?09:51
rvrMirv: Yup09:51
Mirvyup09:51
Mirvthanks09:51
rvrjgdx: ping09:52
Mirv(just gles)09:55
=== chihchun_afk is now known as chihchun
SaviqMirv, thanks10:15
oSoMoNtrainguards: can the amd64 and i386 builds of webbrowser-app be retried in silo 16 (for both series), please?10:25
MirvoSoMoN: okay, at it again10:26
oSoMoNMirv, thanks!10:30
=== _salem is now known as salem_
jgdxrvr, pong10:58
rvrjgdx: Hey10:58
rvrjgdx: Is there any way to manually check fix in silo 53?10:59
jgdxrvr, yeah, it's in the test plan11:00
jgdxrvr, step 311:01
rvrjgdx: Oh, cool11:01
MirvoSoMoN: ^11:06
oSoMoNMirv, thanks! out of curiosity, how many attempts did it take to get it to build successfully on all arches?11:07
MirvoSoMoN: well two were fine with the next rebuild, the third after the second and the last one took one more11:09
MirvoSoMoN: so in general it feels roughly 50% change of success11:10
Mirvchance11:10
oSoMoNMirv, thanks! I haven’t given up on actually fixing the unit test yet, but there are more urgent things for now, unfortunately, so until it’s properly fixed I’ll abuse your patience to retry failed builds…11:14
MirvoSoMoN: yeah, I can see from the amount of landings you're doing! no problem. the new patch is the way of things in 5.4.3/5.5.1/5.6 so it's useful to look at it eventually, but it seems non-critical enough.11:16
ChrisTownsendtrainguards: Could you please assign https://requests.ci-train.ubuntu.com/#/ticket/436 ?11:38
sil2100ChrisTownsend: on it, bu11:39
sil2100s/bu//11:39
ChrisTownsendsil2100: Thank you!11:39
sil2100ChrisTownsend: done :)11:40
ChrisTownsendsil2100: ta!11:41
=== alan_g is now known as alan_g|lunch
=== alan_g|lunch is now known as alan_g
=== pat_ is now known as Guest41182
mterryrobru, got a few questions for ya.  1) one of silo 51's uploads is in the UNAPPROVED queue.  How can I find out why?14:26
robrumterry: if there not a freeze on? Check with #ubuntu-release14:28
mterryrobru, I thought beta freeze is over once beta is done  :)14:29
mterryfinal freeze doesn't start until oct14:29
mterrywill ask14:29
mterryrobru, 2) I would like to squeeze silo 41 in today if possible, to make ota7.  I have specific qa testing steps for the patch that's landing.  I can't edit the silo.  But can you point me at who might be doing QA and I can tell them?  (or let me edit silo somehow)14:30
mterry(to add the QA steps)14:30
robrumterry: why can't you edit the silo?14:31
robrumterry: I'm in a meeting with jibel, in half an hour he should be free to discuss that14:32
robrumterry: there's no acl for editing silos just sign in and edit14:33
mterryhmm14:33
mterryrobru, haha, yup.  Wasn't logged in.  :(  I want a big blinking <marquee> at the top for that  :P14:33
robrumterry: the "create new request" form is hidden when you're logged out14:34
mterryrobru, yeah, it's obvious enough if you've had coffee14:34
rvrjgdx: Approving silo 5315:07
sil2100~/.~.15:26
sil2100uh oh15:27
ogra_snakes left and right of walls ?15:27
sil2100~15:31
sil2100Darn cat15:32
ogra_oh, no morew walls !15:32
* ogra_ runs from the snake 15:32
* kenvandine stress tests citrain :)15:32
=== chihchun is now known as chihchun_afk
=== alan_g is now known as alan_g|EOD
alesagerobru, renato adding cmake-extras to his indicator-transfer-buteo project, cmake-extras is in universe, he's wanting to know if that's kosher for a package entering main17:28
alesagerobru or please redirect my q :)17:29
renaturobru, network-indicator already uses that17:29
alesagerenatu, robru nm, appearing in main IIUC http://pastebin.ubuntu.com/12614769/17:32
robrualesage: Projects in main cannot depend on projects in universe, will need a MIR filed17:32
robrualesage: note that is universe. PPAs don't count17:33
alesagerobru ack17:34
renatualecu, should we revert the change in transfer-indicator?17:34
renatualesage, ^^17:34
renatualecu, wrong nick, sorry17:34
alesagerenatu, up to you, I'll pursue the MIR17:35
barryrobru: this is quicker than email :)19:16
barryrobru: your fix-filterdiff-tests branch fails with a ton of pep8 failures19:16
robrubarry: hey ;-)19:16
robruway19:16
robruwat19:16
robrubarry: did pep8 get way stricter in wily?19:17
barryrobru: it's possible some of the defaults got changed.  you should just suppress E402 (but verify the tons of E402 failures).  most E402 is bogus19:17
robrubarry: wait but that means that the tests passed, it only runs pep8 if tests pass ;-)19:17
barryrobru: Ran 408 tests in 7.762s19:18
barryso, yes!19:18
robrubarry: sweet, I'll approve the branch then. I'll have to pull out my wily schroot to check all this pep8 stuff.19:19
barryrobru: have fun with that :)19:19
robrubarry: thanks for all your help!19:22
barryrobru: sure thing!19:23
slangasekrobru: I'm trying to recall where the latest changes to the train left us as far as being able to see who signed off on a given change; as, for instance, https://launchpad.net/ubuntu/+source/mediascanner2/0.107+15.10.20150922.1-0ubuntu1 which introduces an soname change for a library in wily but is not accompanied by a rebuild of its reverse-dependencies20:00
slangasekrobru: I understood we were passing the sponsor's name as an argument to copy-package now, so it should be recorded on this page?20:01
slangasekrobru: but it only lists jamesh, who is not an Ubuntu dev; so either the train didn't properly generate a packaging diff, or it didn't enforce the uploader check correctly20:02
slangasekor it didn't put the right name in the field20:02
robruslangasek: train has always passed the sponsor name to copyPackage, what changed was that we now enforce checkUpload to allow somebody to be a sponsor at all20:04
robruslangasek: if you tell me what silo it published from I'll find the logs and tell you who published20:04
slangasekrobru: please compare https://launchpad.net/ubuntu/+source/mediascanner2/0.106+15.10.20150917-0ubuntu1 and https://launchpad.net/ubuntu/+source/mediascanner2/0.107+15.10.20150922.1-0ubuntu1 - the first (before the recent changes) doesn't list that this is a sponsored upload at all.  The second does list it as a sponsored upload, but is *not* listing the name in the "sponsored" field that I'm expec20:05
slangasekting to see20:05
slangaseklaunchpad should show me who approved the upload that ci-train-bot sponsored, not who the lander was20:06
slangasek(unless the lander is an ubuntu-dev)20:06
slangasek(and is also the one who approved the upload)20:06
slangasekrobru: this appears to be silo 01920:08
robruslangasek: published and ACKed by kenvandine https://ci-train.ubuntu.com/view/2.%20Publish/job/ubuntu-landing-019-2-publish/101/console20:09
slangasekrobru: ok, thanks20:09
robruslangasek: also packaging diff looks sensible: https://ci-train.ubuntu.com/view/2.%20Publish/job/ubuntu-landing-019-2-publish/lastSuccessfulBuild/artifact/mediascanner2_packaging_changes.diff/*view*/20:09
slangasekrobru: that means kenvandine is the one whose launchpad id I want passed to copy-package :)20:09
robruslangasek: it should have been. was it not?20:09
=== bfiller_ is now known as bfiller
slangasekrobru: what I see on that launchpad page I linked you above was 'jamesh', not 'kenvandine'20:10
robruslangasek: dunno what to tell you. copyPackage sets sponsored: http://bazaar.launchpad.net/~cupstream2distro-maintainers/cupstream2distro/trunk/view/head:/copy2distro#L116 and sponsored comes from the rsyncfile here:20:12
robruhttps://ci-train.ubuntu.com/view/2.%20Publish/job/ubuntu-landing-019-2-publish/lastSuccessfulBuild/artifact/packagelist_rsync_ubuntu-landing-019/*view*/ where it clearly says ken-vandine. must be a bug in launchpad not displaying the right thing.20:12
slangasekrobru: ok; I'll open a bug on this against cupstream2distro, and we can figure out if it needs to be reassigned to launchpad20:14
slangasekrobru: btw you actually call syncpackage rather than copy-package, right?20:14
slangaseksince that's the one that has the "--sponsor=" option20:15
robruslangasek: we use lplib not ubuntu-archive-tools20:15
slangasekok20:15
robrualthough I'm starting to wonder if it wouldn't make more sense to convert copy2distro from python to shell, considering that snakefruit doesn't have python3 and this is a frequent source of problems in the train.20:16
slangasekrobru: bug #150103620:16
ubot5`bug 1501036 in CI Train [cu2d] "cupstream2distro passes the wrong launchpad id as the sponsoree when uploading to the distro" [Undecided,New] https://launchpad.net/bugs/150103620:16
slangasekrobru: mm? snakefruit does have python320:17
robruslangasek: well not a new enough version. it's running precise.20:18
robruslangasek: if snakefruit could be upgraded to trusty I'd be so happy20:18
slangasekdon't count on that20:18
slangasekkenvandine: ping, btw :)20:20
kenvandineslangasek, pong20:20
robruslangasek: no I'm not counting on it but it often causes headaches.20:22
robruslangasek: oh, it's not that snakefruit doesn't have python3, it's that snakefruit doesn't have python3-launchpadlib, and my trusty backport is only for trusty, not for precise.20:22
slangasekkenvandine: so silo 019 is stalled in -proposed because it introduces an soname change in wily and the reverse-dependencies were not all included in the silo20:22
slangasekkenvandine: since you signed off on the packaging changes, I think that means you're responsible for following through ;)20:23
kenvandinedamn20:23
kenvandineslangasek, iirc, it didn't break or conflict with the previous20:24
kenvandineso should still be installable, right?20:24
slangasekkenvandine: you don't get to have both of them in wily at the same time20:25
kenvandinejamesh, ^^20:26
slangasekkenvandine: proposed-migration requires all the revdeps to be rebuilt first, and then transition them together - and considering one of the reverse-dependencies is qtubuntu, I'd say there certainly hasn't been a valid integration test of the new lib...20:26
slangasekkenvandine: see also related bug #150085920:26
ubot5`bug 1500859 in qtubuntu-media (Ubuntu) "hard coded dependency on non-existing version of runtime library" [Critical,Confirmed] https://launchpad.net/bugs/150085920:26
kenvandineugh!20:26
kenvandineok20:26
slangasekthe criticality may be somewhat overstated, but it does require a source fix and upload ;)20:27
kenvandineyeah20:27
slangasekrobru: and there's no chance that copy2distro mistakenly picked up an rsync file from jamesh's previous failed publication attempt because that file is only generated if the acl check succeeds, right?20:29
robruslangasek: yes, the acl check happens well before the class that creates that file is even instantiated.20:30
* slangasek nods20:30
robruslangasek: http://bazaar.launchpad.net/~cupstream2distro-maintainers/cupstream2distro/trunk/view/head:/citrain/publisher.py#L213 line 213 vs 222 in the main() function20:31
kenvandinei'm confused...20:31
kenvandinethis was published to the overlay ppa20:32
kenvandinebut... there is no libmediascanner-2.0-4 in the overlay20:32
slangasekkenvandine: this was landing the merged packaging that generates *different* package names for vivid vs. wily for the g++5 ABI transition20:32
slangasekkenvandine: however, it's possible this whole landing was pointless and it should have been left alone - because we already had libmediascanner-2.0-3 in both vivid and wily, and we wouldn't have let that through without a package name change if the ABI had changed, I think20:33
kenvandineoh... i see it20:34
kenvandine2015-09-24 18:55:39,811 INFO Copying mediascanner2 0.107+15.04.20150922.1-0ubuntu1 in vivid to https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay.20:34
kenvandine-Package: libmediascanner-2.0-420:34
kenvandine+Package: libmediascanner-2.0-320:34
kenvandinein the overlay ppa20:34
kenvandinejhodapp, what do you know about this issue?20:35
jhodappkenvandine, which issue?20:35
kenvandinesilo 1920:36
jhodappnot a thing20:36
kenvandinemediascanner soname bump20:36
jhodappkenvandine, I'd check with jamesh20:36
kenvandineyou commented about the bug it caused :)20:36
kenvandines/about/on/ :)20:36
jhodappgot a link?20:36
jhodappthis must have been pre-vacation...everything has since left my brain ;)20:36
kenvandinebug 150085920:36
ubot5`bug 1500859 in qtubuntu-media (Ubuntu) "hard coded dependency on non-existing version of runtime library" [Critical,Confirmed] https://launchpad.net/bugs/150085920:36
kenvandinethat was caused by this landing20:36
kenvandinebut.. silo 19 includes magic to provide a different soname version for vivid vs wily20:37
jhodappkenvandine, oh, don't know much yet other than I wanted to know about the MIR landing process20:37
kenvandinejhodapp, is qtubuntu-media dual landed? or separate wily and vivid branches?20:38
jhodappkenvandine, separate branches atm20:38
slangasekhaha of course20:38
kenvandineok, so you could have separate depends for the 220:38
slangasekso you can't include it in a dual landing silo dur20:38
kenvandineyeah20:38
kenvandinegrumble20:38
slangasekwhy is the dep hard-coded at all?20:38
kenvandineyeah, i'd think you'd get it with shlibs20:39
jhodappslangasek, you mean libmediascanner being listed for qtubuntu-media?20:39
slangasekjhodapp: yes20:39
jhodappI'm pretty sure it can be removed now actually...that is probably a mistake that it's still listed there20:40
jhodappI'm working with qtubuntu-media atm, so I'll check this out20:40
jhodappalthough I don't understand why having it as a dep is an issue...can someone explain that to me with the knowledge that I have little packaging experience20:41
kenvandinejhodapp, silo 19 landed a soname bump of libmediascanner20:42
kenvandineso libmediascanner-2.0-4 is the new binary package name20:42
kenvandineso depending on libmediascanner-2.0-3 forces it to install the older version20:42
jhodappoh I see, ok20:42
kenvandinebut i'm thinking slangasek might be right, this whole landing to have different soname versions for the 2 isn't useful20:43
jhodappwhy is the so-name part of the package name?20:43
jhodappthat seems odd20:43
kenvandineit's the debian way :)20:43
robrukenvandine: oh, this was the landing where the train had that bug that it was diffing the vivid version against the wily version20:43
jhodappreally?20:43
kenvandineso you can have them installed in parallel20:44
kenvandine2 versions of the same lib installable together20:44
jhodappah, so instead of using metadata in the package they use the package name20:44
kenvandineyup20:44
jhodappseems antiquated now20:44
kenvandineslangasek, sorry for letting it in, i thought we only blocked on rebuilding everything if there was a breaks or conflicts with the old soname version20:45
* kenvandine is glad we have -proposed :)20:46
slangasekthe use of the soname in the package name is deliberate and not antiquated :)20:46
kenvandinenot everyone agrees :-D20:47
* slangasek shrugs20:47
kenvandinei'd like parallel installable versions of the same package name :)20:47
jhodappI think it creates a weird situation, like this20:47
kenvandineconary did that, it rocked20:47
jhodappexactly20:47
slangasekkenvandine: go run Red Hat, I'm sure you'll love it?20:47
kenvandinenope :)20:47
kenvandinewith conary we could do libfoo=2.0-0 and libfoo=2.0-120:48
kenvandineboth installed at the same time20:48
kenvandinesame package name20:48
kenvandineit was cool :)20:48
robrukenvandine: oh yeah I forgot you're a conary snob ;-)20:48
kenvandineanyway...20:48
kenvandinenot anymore20:48
kenvandinewe had cool stuff back in the day :)20:49
jhodappmakes sense to me, seems to be a style difference/preference20:49
kenvandinejhodapp, when does jamesh usually show up?20:49
robrukenvandine: yeah conary was a victim of "too technically good to be successful". debian, of course, being *just* good enough, wins out ;-)20:49
jhodappkenvandine, he's UK I believe20:50
kenvandineoh?20:50
kenvandineaustralia.. last i heard20:50
jhodappoh nevermind then20:50
kenvandine:)20:50
jhodappyou know better than me :)20:50
kenvandinei figured you worked more closely than me20:50
jhodappnot really, I just know he works on mediascanner20:50
kenvandinei guess a couple hours20:51
ahayzenjhodapp, jamesh is in Australia :-)20:58
=== salem_ is now known as _salem
robruslangasek: hangout issues, just restarting21:31

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