robruveebers: force is fine. Or if you just want to rebuild one you can specify which one.00:25
veebersrobru: coolio, thanks01:10
robruveebers: you're welcome01:18
robrukgunn: "invalid package" means your silo isn't configured to include that package in the sync.01:18
kgunnrobru: i'm not familiar with syncing like that, do you guys do that upload of the src pkg? or how does it work ?01:19
robrukgunn: what request is it?01:19
kgunnrobru: sorry, silo 54 alberta's sync of mir to vivid+o01:20
robrukgunn: well for starters this request is totally wrong in every possible way01:20
kgunngood to know01:20
robrukgunn: you can't mix MPs and syncs. those are different things01:20
kgunnrobru: what's your recommendation in this instance ?01:21
robrukgunn: the sync syntax changed a bit but "sync:mir" isn't valid the new way or the old way, that's totally out to lunch.01:21
robrukgunn: what are you trying to do?01:21
kgunnso here's what we got01:21
kgunnmir, usc, qtmir are all in wily properly...and i think we just want to promote all those into vivid+o01:21
robrukgunn: so you just want to copy those three packages from wily to vivid+o without any changes at all?01:22
kgunnrobru: correct, rebuilt of course....01:22
kgunnfor gcc considerations01:22
robrukgunn: right01:23
kgunnrobru: and actually...i think usc and qtmir can simply be rebuilt01:23
robrukgunn: ok, those MPs are empty. I'm not sure where he got the idea to do that.01:23
kgunni don't think they differ01:23
kgunnhe was flying unguided i think01:23
robrukgunn: empty MPs are only needed if you're doing a no change release of trunk (eg if you landed changes in trunk without using the train). not for syncs.01:23
robrukgunn: ok so here's what you do. empty out the merges field. put the source names you want in the sources field (WITHOUT the 'sync:' bit)01:24
robrukgunn: the rest of the fields look good01:24
robrukgunn: then click assign to configure the silo, then click build and it'll do a source copy that rebuilds against the appropriate gcc01:25
kgunnrobru: and if i am syncing like this, even tho there's no diff for qtmir/usc....do i still list those as pkgs in the "manual src" block ?01:25
kgunnwhich will in effect rebuild them01:25
robrukgunn: yes, if you don't include them in the sources field then the train won't know what packages to copy in the sync.01:26
kgunnrobru: oh and for qtmir-gles...it's kinda weird01:26
kgunndoes it autoknow ?01:26
robrukgunn: it auto knows that what you put in the sources field are the names of the packages it's going to sync for you.01:27
kgunnrobru: right but as i recall qtmir-gles links to the pkg biuld version from qtmir....which is why we always built it last, unless someone automated that01:27
kgunne.g. i guess i may need a ppa for qtmir-gles01:28
robrukgunn: ok, you might need an MP for the -gles that updates the debian/watch01:28
kgunni'll at least get mir/usc/qtmir built...and i can sort the qtmir-gles in the morning01:28
robrukgunn: it's worth trying. I'm not sure off the top of my head how sync code will handle a -gles package. if the PPA accepts the orig.tar it should be fine, if it uses debian/watch it'll get confused. unless you get lucky and get the same silo, lol ;-)01:29
kgunnthanks for the help tho01:30
robrukgunn: you're welcome01:30
robrukgunn: but you can't have any MPs in the silo config, you have to remove the -gles one too01:30
robrukgunn: syncs and MPs are totally incompatible within the same silo01:31
robrukgunn: if -gles doesn't work as a sync you'll have to do it in a separate silo later.01:31
robrukgunn: and you should file a bug about that, that should be supported by the train. in fact the train could go so far as to automatically set debian/watch for you each time so you don't have to worry about it.01:32
kgunnoh...i would like that very much01:32
robrukgunn: file the bug against lp:cupstream2distro01:32
robrushouldn't even be hard, but I have some other priorities first.01:32
kgunnrobru: fwiw...i just read scrollback...i left the gles mp in there, and the sync seemed to work ?01:33
kgunnor did i just do something undefined01:33
robrukgunn: it might be working since you specified which packages to build in a way that excluded the -gles mp but generally the behavior of MPs and syncs together in a silo is undefined, I'm not sure what'll happen when you try to build the MP01:34
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
robruwho is doing QA at this hour?04:45
robruoh, veebers, it's you04:45
robruyou want i should publish that?04:45
veebersrobru: heh :-) yes please04:46
veebersrobru: speaking of 'at this hour' are you ever offline? ;-)04:46
robruveebers: I usually sleep between 4AM and 4:15AM.04:47
veebersrobru: lol ^_^04:47
robruveebers: just kidding, I usually sleep in actually, but it's only 10pm here!04:47
robruveebers: oooh, look at Mr Fancy Pants fixing his build by ignoring errors!04:48
=== chihchun_afk is now known as chihchun
veebersrobru: hah, that's due to new flake8/pep8 rules coming along which I disagree with, but perhaps I will come around to liking them and remove those ignores04:59
robruveebers: heh, yeah, I think dobey got hit by something similar as well, I'm not sure what the new errors are personally05:00
alf_chihelp: Hi! We have been consistently getting failures in job 'mir-android-vivid-i386-build' for the last day or so: " libboost-all-dev : Depends: libboost-python-dev but it is not going to be installed"07:07
alf_cihelp: Hi! We have been consistently getting failures in job 'mir-android-vivid-i386-build' for the last day or so: " libboost-all-dev :  Depends: libboost-python-dev but it is not going to be installed"07:07
alf_cihelp: Any idea how we can fix this? It's blocking all mir CI jobs07:08
Mirvbfiller: renatu charles: I got permission to publish 024, but you will need to fix bug #1491255 for the next release, to track API07:15
ubot5bug 1491255 in indicator-transfer (Ubuntu) "Add .symbols file for tracking API" [High,New] https://launchpad.net/bugs/149125507:15
Mirvogra_: could you merge + publish both https://code.launchpad.net/~timo-jyrinki/ubuntu-seeds/ubuntu-touch.wily_remove_friends/+merge/267648 and https://code.launchpad.net/~timo-jyrinki/ubuntu-seeds/ubuntu-touch.wily_add_indicator-transfer-download-manager/+merge/269846 (latter needs approval, but it's related to this 024 landing now, adding a plugin package)07:16
MirvI can upload the vivid overlay meta package change07:16
morphisMirv: thanks for publishing silo 907:18
morphisMirv: when I setup a sync silo to get something from wily, does it fetch that from the proposed pocket too?07:23
Mirvmorphis: no problem. yes I'd say it'd fetch from proposed pocket too, but remains to be seen :)07:24
morphislets try it07:24
Mirvdbarth__: hi, repeating from yesterday, did you see robert's comments on 052 silo about it being a dual silo but configured for wily for some reason? it's QA granted, so if you want I can reconfigure it as dual again and publish, but we just want to understand why it's marked as 'wily' at the moment.07:26
Mirvbregma: did you see sil2100's comment in bileto last week about additional changes needed to silo 027 / Libertine before it can be published? a symbol file would be best added (the archive admins will want it), and an arch:all correction.07:28
morphisMirv: seems that this doesn't work: 0.229 does not seem to be a CI Train generated version number, series version change is not supported for non-train uploads.07:32
morphisI remember ... non train upload can't be changed in their release version ...07:33
morphislets do this different then07:33
=== zbenjamin_ is now known as zbenjamin
Mirvmorphis: ah right again, we need to just continue doing manual uploads07:34
Mirvzbenjamin: would you have time today to verify the silo 025 / QtC so that I could publish it?07:34
morphiswould be the best if we change lxc-android-config to be citrain based finally ...07:34
Mirvmorphis: well that would be something, yes07:35
zbenjaminMirv: meh i totally forgot about that... multitasking pretty heavily atm :D07:35
=== chihchun is now known as chihchun_afk
Mirvzbenjamin: I know the feeling, no problem :)07:36
bzoltan_Mirv:  who to quick ask to double the size of this PPA? https://launchpad.net/~ubuntu-sdk-team/+archive/ubuntu/tools-development07:43
morphisMirv: any idea what I have to change for that? never did that before07:44
Mirvbzoltan_: file an internal RT ticket and refer to that on webops channel07:44
bzoltan_Mirv:  OMG .. .really? Still?07:44
bzoltan_Mirv:  I need it in a minute...07:45
Mirvbzoltan_: well that's my guess, they probably get some karma for closing tickets07:45
Mirvmorphis: take .bzr-builddeb from eg http://bazaar.launchpad.net/~phablet-team/camera-app/trunk/files - that's probably about it. for the first release, add a manual new changelog entry in your MP with UNRELEASED as status. with those the train might be able to build it.07:46
Mirvmorphis: oh right, we need project :D07:46
Mirvjust a moment07:47
Mirvbzoltan_: well ask the vanguard on webops directly then07:47
morphisMirv: let me check with ogra_  too07:47
morphisogra_: is there anything which prevents us from switching lxc-android-config to be citrain based?07:47
Mirvyeah, that should be useful07:48
ogra_morphis, except of my lazyness to do the fiddling for it there never was one :)07:48
Mirvogra_: ok, me and morphis can fiddle it then :)07:49
morphisogra_: great!07:51
Mirvmorphis: ok you can now push branches against     lp:lxc-android-config07:51
morphisMirv: so is that fine even if we have a version gap between vivid-overlay and wily atm?07:51
Mirvstarting with eg the addition of .bzr-builddeb and a new changelog entry, and that can then be tried to be built07:51
Mirvmorphis: we could do a vivid branch for overlay07:52
Mirvwell, I can do that now too07:52
morphisI would try to prevent us from doing that07:52
morphisjust takes more time to merge things etc.07:52
morphisthere is one change in wily I need to double check if that causes regressions on vivid07:53
Mirvmorphis: well sure, if you think we can land to trunk to overlay, that's always the best07:53
=== chihchun_afk is now known as chihchun
psivaaalf_: do you have the link for the job?08:08
alf_psivaa: https://jenkins.qa.ubuntu.com/job/mir-android-vivid-i386-build/08:23
psivaaalf_: ack, thanks. I'm in the middle of debugging something else. will take a look at this in a bit08:25
alf_psivaa: Great thanks08:26
morphisMirv: but looks good08:32
robrualf_: generally depending on boost-all-dev is frowned upon, you might be able to make some progress by selecting the individual boost packages you really need.08:33
robrualf_: but other than that i dunno08:34
morphisMirv: there we go: https://code.launchpad.net/~morphis/lxc-android-config/add-citrain-support/+merge/26985908:37
robruMirv: I'm surprised to see 52 in the NEW queue, i thought there was a bug that made publishes skip that08:37
mandelrobru, might be very late, but does the ci bot use wily, is it using gcc 5? I'm getting diff symbols in my wily machine than in the bot :-/08:38
robrumandel: the train uses a pbuilder that matches the series is building for. So you get gcc5 in wily and 4.9 in vivid08:39
mandelrobru, hm.. I wonder what is going on :-/08:39
robrumandel: not sure, I'm not a symbols expert08:39
robrumorphis: did you check your packaging against https://wiki.ubuntu.com/DailyRelease/InlinePackaging ?08:40
morphisrobru: no08:40
mandelrobru, don't worry, at least I'm 100% sure that it is not the bot, that is enough info08:41
robrumorphis: those are the official train packaging guidelines, you should probably look it over. Review looks good for what's there but there might be other things missing08:41
morphisrobru: it looks so08:41
morphisatleast the build failed08:42
robruabeato: you can't release a wily trunk to vivid, either do a dual silo or branch for vivid08:46
robrumorphis: not sure about that error there, can't find the orig.tar... It might be because your trying to change the version from native to nonnative... Try pushing those changes to trunk and then putting a null mp in the silo.08:48
abeatorobru, I was trying to see if I could create 2 MPs from one branch, with one MP targeting vivid and the other wily08:48
abeatorobru, but does not seem possible08:49
Mirvmorphis: ah, you found the wiki page thanks to robert too!08:49
robruabeato: no, that's a horrible idea. What would happen when they merge to trunk? You'd have conflicting changelogs08:49
Mirvrobru: my latest info is that source NEW:s go to the queue (and that's why we don't need to ask preNEW-review for them), while binary new:s are skipped so we need to manually ask for the reviews08:49
robruabeato: if you want to release to both from one trunk you need to do a dual silo08:49
morphisrobru: I just changed the packaging bits, but if I understand you right we have to merge my MP now first before going ahead, right?08:50
abeatorobru, my idea is that CI train might be able to handle that08:50
robruabeato: is there a reason that you can't use a dual silo?08:50
abeatorobru, well after the gcc5 switch we have many projects branched08:50
abeatoso I was experimenting a bit08:50
Mirvmorphis: I think there's a possibility that train doesn't support native packages anymore, meaning that you need "0.230-0ubuntu1" version number instead08:50
morphislet me try that08:51
Mirvnative packages did work at one point but it was more of a bug probably08:51
Mirvand now that lxc-android-config has an upstream project, it's not native anymore to Ubuntu :)08:51
Mirv"kind of"08:51
robruabeato: there is a way to do it from one trunk and keep using dual silo, but it takes a bunch of packaging voodoo. Talk to michi about it he has a proof of concept that works08:53
robruI should get some documentation together for that...08:53
=== Mirv changed the topic of #ubuntu-ci-eng to: Train trouble? ping trainguards | CI problems? ping cihelp | Train: http://bit.ly/1hGZsfS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: OTA-6 final freeze in effect, but the vivid-overlay landing gates opened. sil2100 away, small trainguard gap between EEST & RDT (robru daylight saving time) timezones
abeatorobru, I had actually noticed that there is a silo from him that generates the dependencies dynamically apparently08:54
abeatorobru, it would be great if michi or somebody shares that in the mailing list :)08:54
robruMirv: lol08:54
robruabeato: it's 2 AM for me, I'm afk can't get the details. But michi has a good example if you just read what he did. You can make an override_dh_auto_build that generates the correct debian/control with correct values per distro08:56
abeatorobru, thanks, and take some rest08:57
robruabeato: i mean override_dh_auto_clean, grep his source for that and you can see how it works08:57
robruabeato: can't sleep, clowns'll eat me08:58
morphisMirv, robru: just adding -0ubuntu1 doesn't seem to be enough08:59
morphisstill asking for the tarball08:59
morphisah I see09:01
morphisused - instead of +09:01
robrumorphis: that is really strange, the whole point of split mode is that it makes thr tarball by dropping the debian dir. Double check you have it set right? I don't see anything obviously wrong...09:01
Mirvmorphis: - should be good, I'm testing something on my own09:01
MirvI don't see anything wrong anymore either09:01
morphisif I used + with a local bzr bd run it doesn't ask anymore for the tarball09:02
morphisand the info "This package has a Debian revision number but there does not seem to be..." goes away09:02
morphiscitrain correctly notes: "dch warning: Previous package version was Debian native whilst new version is not"09:03
MirvI wonder what I managed to do when I get build success without it trying to merge the MP in the first place :D https://ci-train.ubuntu.com/job/ubuntu-landing-002-1-build/213/console09:05
MirvI was just experimenting based on morphis' branch https://code.launchpad.net/~timo-jyrinki/lxc-android-config/randomtest/+merge/26986609:06
Mirvmorphis: I don't know what's going on in my test silo, but I was trying to add a fake non-native entry plus use a date code (which includes + in upstream version and the - for being a non-native package)09:07
robruMirv: silo 2 is a manual source...? Force rebuild means it watches the ppa twice as hard i guess.09:07
Mirvmorphis: ah, now I notice. delete the debian/bzr-builddeb.conf09:08
Mirvrobru: it is not, that's the fun of it. https://requests.ci-train.ubuntu.com/#/ticket/29809:08
Mirvmorphis: and don't change the rest of it, that'll probably fix it09:08
robrumorphis: uh "+0ubuntu1" is not a thing that makes sense, if that works it's only by accident. You should try to ape a real version with the date stamp and the train will run with it09:08
Mirvmorphis: just keep the 0.230-0ubuntu1 and bzr rm debian/bzr-builddeb.conf09:09
robruMirv: the console log you linked was silo 2 not 24.09:09
morphisMirv, robru: done, let me try this now09:10
Mirvrobru: wow, quite a coincidence that I actually run a build job for a wrong silo that has the same package :D09:10
mzanettirobru, hey, a small bug report for bileto: when there's a merge conflict in a branch, it prints that with a link to the branch, but the link includes a dot at the end which breaks it09:13
robrumorphis: well that looks good, both uploads accepted09:14
mzanettirobru, here's an example: https://requests.ci-train.ubuntu.com/#/ticket/28509:14
robrumzanetti: oh haha. Can you email me with a link to the log that has that? I'll fix it during my shift tomorrow09:14
mzanettirobru, ack09:15
=== greyback__ is now known as greyback
mandelrobru, so, this mr https://code.launchpad.net/~mandel/ubuntu-download-manager/wily-add-appid-metadata/+merge/269364 is building on vivid, yet is aiming for wily.. at lets so it says here => https://jenkins.qa.ubuntu.com/job/ubuntu-download-manager-vivid-armhf-ci/28/console09:45
mandelrobru, any idea?09:46
robrumandel: yeah that's not the train you looked to there, nothing to do with me09:47
Mirvmandel: it's 2.45am for hi.. okay, he's there09:47
mandelrobru, sorry sorry!09:48
mandelMirv, do you know anything about that? The ci bot using the wrong release.. :-/09:48
robruMirv: for some reason at 11 pm i put on a 4 hour YouTube video thinking that was a good idea09:48
Mirvmandel: try highlighting cihelp for that ^09:48
Mirvrobru: sounds wise09:48
psivaamandel: i'll take a look at this. i'd need some time. digging some other issues at the moment. but i'll make a record of your issue and come back to you09:51
mandelpsivaa, ok, is not a show stopper, I'll add a comment in the MR09:52
mandelpsivaa, moving to gcc 5 has this problems09:52
=== alan_g is now known as alan_g|llunch
=== chihchun is now known as chihchun_afk
cjwatsonMirv,bzoltan_: for future reference, the place to ask for PPA changes is https://answers.launchpad.net/launchpad, preferably not RT or webops10:02
cjwatsonwebops can do it of course but we prefer to distribute load away from them where possible10:03
kgunntrainguards so in silo 54, i was doing a src sync from wily to vivid+o, mir and u-s-c built fine, but qtmir just seems to wait and time out10:11
kgunnis something with a twin not capable of being sync'd at all ?10:11
robrukgunn: I'm not aware of any limitations on the non-gles one being synced. Should be the same...10:12
kgunnrobru: dude...you're up too :)10:13
robrukgunn: i need help10:13
kgunnrobru: so you were helping me last night....and qtmir is the only pkg refusing to build10:14
kgunnas i watched the output file, it just spins saying "qtmir...waiting"10:14
kgunnor no "watching"10:14
bzoltan_cjwatson: OK10:15
robrukgunn: the fact that it's trying to upload "0ubuntu2" is quite suspect, can you confirm you didn't already sync this version? 0ubuntu1 is either already in the PPA or the archive10:17
kgunnrobru: ah...i just found out....someone did something naughty10:18
kgunnvivid+o is ahead of wily10:18
robruOoh good10:18
kgunnso i just need to land qtmir latest in wily and rebuild....i think10:18
kgunnthis is gonna be a mess10:18
kgunncause then i think unity-api is involved somehow10:18
robrukgunn: not sure, sorry.10:19
kgunnrobru: ok, my teams mess thos10:20
kgunnsorry to bother10:20
robrukgunn: no worries10:20
robrukgunn: the train now supports a way to have dual landings even for packages affected by gcc5 abi but it's a little experimental, you should coordinate with michi, he has a working proof of concept.10:22
kgunnrobru: thanks10:22
robrukgunn: it's a little bit voodoo but a nice dual landing would clean up your mess ;-)10:22
Mirvit seems it doesn't hurt I'm having this very late lunch since you two are a fit for each other regarding working hours :)10:23
robruMirv: send help... I need sleep therapy10:24
Mirvyeah you both do10:24
Mirvthis google result seems to be on the topic, but I'm not sure if it's sane https://www.reddit.com/comments/meaq9/the_real_quick_fix_to_get_your_fucked_up_sleep/10:25
abeatoMirv, how can I abandon a silo? I don't see the option anymore10:25
Mirvabeato: if you have a silo assigned, ask a trainguard to free that up, it also makes the request abandoned10:26
robruabeato: Mirv ONLY_FREE_SILO.10:26
robruMirv: landers can free their own10:27
Mirvrobru: oh!10:27
abeatoMirv, ok please free silo 16 then10:27
Mirvabeato: so, click the Merge & Clean and select ONLY_FREE_SILO10:28
Mirvabeato: new instructions, do not ping but free it up yourself :)10:28
abeatoah, ok10:28
robruMirv: i thought you knew, this isn't something that changed recently, landers could always free their own10:28
Mirvthe train is really good nowadays, all the features coming together10:28
Mirvrobru: I simply didn't know..10:29
robruMirv: maybe skim the landingprocess page, i overhauled it last week10:30
* Mirv opens https://wiki.ubuntu.com/citrain/LandingProcess10:30
robruMirv: there was a hilarious part that used to say "ci airline will be ready ~july 2014" so that was in need of updating ;-)10:32
Mirvrobru: any day now!10:33
robruMirv: OK I'm fading... Goodnight!10:37
Mirvrobru: goodnight!10:40
=== chihchun_afk is now known as chihchun
=== alan_g|llunch is now known as alan_g
bzoltan_cjwatson: how those requests are picked up?11:55
cjwatsonbzoltan_: mailed to answer contacts for Launchpad, which includes Launchpad staff11:56
bzoltan_cjwatson: OK, so eventually somebody will pick up. I managed to fill up the PPA :( I somehow feel that PPAs are not designed to host such content...11:58
cjwatsonbzoltan_: how do you mean eventually?  you have no pending requests11:59
cjwatsonoh, six minutes ago11:59
bzoltan_cjwatson: just got two uploads rejected11:59
cjwatsonbzoltan_: would you mind adding me temporarily to ~ubuntu-sdk-team?  then I can make this change (and leave the team immediately afterwards) without having to wait for William to be around12:00
cjwatsonI should get round to getting commercial admin privileges for myself :)12:01
bzoltan_cjwatson: welcome to the team :) i will show you later our backlogs...12:02
cjwatsonbzoltan_: ok, sorted (and left :-) )12:03
bzoltan_cjwatson:  so quickly? I was already looking for a suitable task :D12:03
bzoltan_cjwatson: Thank you12:04
cjwatsonI suspect the LP bug queue is longer than yours, just because it's been around for longer :-)12:04
bzoltan_cjwatson: Yeps... I admit, we have a decent bug queue... but you know, I think like with kids It is already big enough for being left out without our control.12:07
oSoMoNubuntu-qa: all webbrowser-app generic-deb-autopilot-runner-vivid-mako CI jobs are failing, they apparently fail to flash the device, is that a known issue?'13:05
oSoMoN(see e.g. https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/3242/console)13:06
alesageoSoMoN, maybe plars knows ^^13:17
=== chihchun is now known as chihchun_afk
mandeltrainguard so, I have a very very interesting question, anyone wants to hear my idea?13:42
renatuMirv, thanks, for landing silo 24. About the bug #1491255, how do you suggest us to maintain that since the symbols could be different in wily and vivid13:43
ubot5bug 1491255 in indicator-transfer (Ubuntu) "Add .symbols file for tracking API" [High,New] https://launchpad.net/bugs/149125513:43
renatuMirv, until now we are using the same branch (trunk) for both13:43
renatuMirv, creating new branches just because of the symbols files does not look good13:43
renatuMirv, mandel is having this problem right now with download-manager API,13:46
mandelrenatu, Mirv so, as I mentioned, my idea is to have packging branches per distro and just keep the symbol issues there13:47
mandelrenatu, Mirv and keep the same source in trunk. Then when we create a silo we provide the mr for trun and the mr for the pacaking13:48
Mirvrenatu: mandel: right, and it was mentioned michi has a solution for differing symbols too (maybe similar)13:54
mandelMirv, would be nice to have a hang out, brain storm and get this fix asap13:55
mandelMirv, I hate to have to do crazy backport branches just for the symbols, and yes, is no ones fault except for cpp13:55
michimandel, Mirv: have a look at the gen-debian-files script and the rules file in the debian directory in silo 10.13:56
michiThat’s pretty much all you need.13:56
michiIt’s just the usual substition grind with sed and what not.13:56
Mirvmandel: michi: I haven't done this myself yet, but thanks to michi I've added now https://wiki.ubuntu.com/citrain/LandingProcess#Dual-landing_and_handling_symbols14:02
Mirvrenatu: ^14:02
michiMirv: reading...14:02
MirvI think some more documentation on the usage might be handy :)14:03
michiMirv: The trick with the symbols file is to have only one of them.14:03
michiEither wily or vivid, it doesn’t matter.14:03
michiBut don’t try to do a symbols file for both, you’ll go mad that way.14:03
michiBasically, if a symbols file on vivid passes, and the same code is compiled for both vivid and wily, there is no point in having a symbols file for wily too.14:04
michiThat’s because, if a vivid symbols file is good, the only differences in the wily symbols file will be due to compiler differences.14:04
michiDifferent symbol mangling with the new ABI, and differences in inlining.14:04
michiSo, adding a symbols file for wily too won’t show anything that a vivid build won’t show.14:04
michiLonger term, the goal is to get rid of symbols files completely because they are the wrong tool for the job.14:05
michiThe catch some ABI breaks, but only some.14:05
michiEven in C, there are many ways to break ABI that won’t show up in the symbols file.14:06
michiIn C++, there are many more14:06
mandelmichi, well, I was told that QA wants them for OTA714:06
michiabi-compliance-checker is the way to go for C++ and, if we are honest about it, for C as well.14:06
michiQA needs to learn that symbols files don’t do what they are meant to do.14:06
michiIt’s an education campaign.14:06
michiIf a symbols file doesn’t pass, that means the ABI is broken.14:07
mandelmichi,  if we ditch them, we should tell QA, get a company rule and deal with it without symbol files14:07
michiThat, by not means, implies that, if a symbols file passes, that the ABI is not broken.14:07
michiCatches everything a symbols file with catch, plus dozens more cases that a symbols file won’t catch.14:07
michiIt’s a far better way to establish whether the ABI is still intact.14:08
mandelmichi, I;m also a little worried about upstream projects, in ours we can drop them and then add abi-compliece-checker14:08
michiThe problem with acc is that it’s cumbersome to use and difficult to integrate into our CI story14:08
mandelmichi, for me, as long as we do not have to mantain two files etc.. I'm ok14:08
michiI’m working on making it more bearable.14:08
michiFor now, if you have a symbols file for either wily or vivid, that’s cool.14:08
michiJust don’t try and add a second one.14:09
michiIt won’t show you anything new.14:09
michiMaintaining the bloody symbols file has been the bane of our lives for the past two years.14:09
michiThere are an unbelievably stupid number of ways in which that can break, with no ABI in danger whatsoever.14:10
michiThat’s for C++.14:10
michiFor C, it’s not as bad.14:10
michiCheck out silo 10, it’l give you the general gist of things.14:11
michidebian/rules and all the .install files are generated on the fly.14:11
mandelmichi, so, abi-change-checker, I'm never heard about it, is it new, did we build that?14:11
mandelmichi, I'd like to know how it works14:11
michiFor scopes-api, it’s more complex than it will be for most projects because we changed the way we name our soversion recently.14:11
michiIt’s in the archives.14:12
renatumichi, what I understood is that the symbol files will be mandatory now14:12
michiperl script that reads the symbol table and the headers and does a fairly deep inspection of the ABI *at the source code level*.14:12
michirenatu: the symbols files are useless (mostly) to establish that the ABI is still intact.14:12
michiThey don’t do what they are meant to do.14:12
michiSticking to a policy such as this is not productive.14:13
michiSymbols files catch *some* ABI issues, but nowhere near enough of them to matter.14:13
renatubut the QA team are asking us to create it on the projects to accept our MR14:13
michiThey are simply the wrong mechanism to establish ABI compliance.14:13
Mirvmichi: renatu: mandel: I have to stop for today but I updated https://wiki.ubuntu.com/citrain/LandingProcess#Dual-landing_and_handling_symbols by parsing the contents of silo 10 + reading michi's initial explanation above14:13
mandelMirv, good starting point14:14
michithe QA team needs to learn that they are asking for something that causes a lot of maintenance overhead for very little return on investment.14:14
renatumichi, I do not have problems with my .install files the problem will start with symbol files14:14
michiJust skimming.14:14
michihold back on the publishing of this please.14:14
michithe Vivid special-casing is rather unique to our project and won’t apply to most other projects.14:15
michiI’m about to hack this up a little bit more to get rid of the vivid special case altogether.14:15
michiAnyway, the basic idea is simple: have the hook in debian/rules that calls a shell script (or python, whatever).14:16
Mirvmichi: it's a wiki :) added a note at the top that it's a bit WIP14:16
michiThen make that script adjust all the soversion stuff, rename files, substitute the first line into the symbols file, generate a shlibs file, and so on.14:16
michithe idea is that, if you compile the exact same source for both vivid and wily, the only differences that matter are in the debian files.14:17
michiSo, generate the debian files instead of coddling two different series along.14:17
michiIn the end, for identical source, there should be only two things that differ in the final packages: the soname, and the package name.14:18
michiUse a symbols file for vivid, and a shlibs file for wily (until we get abi-compliance-checker working nicely)14:19
michiThat way, we are no worse off than we are now, but we don’t have to maintain two series and two symbols files.14:19
Mirvogra_: can you review https://ci-train.ubuntu.com/job/ubuntu-landing-046-2-publish/lastSuccessfulBuild/artifact/ubuntu-app-launch_content.diff it adds liblibertine-dev dependency and changes to depend on upstart instead of transitional upstart-bin. changelog entry funniness is because of different order wily/vivid landings but seem correct otherwise.14:28
Mirvtedg: you will need to MIR request libertine when you eventually land the new ubuntu-app-launch to wily, even though 046 is vivid overlay only14:29
Mirvdepending on the point of view it might be wanted to be done already before 046 can be published to overlay14:29
tedgbregma: ^14:30
Mirvbregma: tedg: so regarding wily 1. get archive admin to approve it from https://launchpad.net/ubuntu/wily/+queue?queue_state=0&queue_text= 2. file a https://wiki.ubuntu.com/MainInclusionProcess at https://bugs.launchpad.net/ubuntu/+source/libertine 3. when approved, arrange for ubuntu-app-launch release and alert also trainguard so that when publishing it archive admin:s need to be once more contacted14:33
Mirvto do the actual (approved) promotion to main of libertine once ubuntu-app-launch starts needing it14:33
kgunntrainguards long story short, due to proj deps branching (or not) and debian dep numbers....i think i want to do one of 2 things14:33
kgunn1) can i have 1 silo with MPs and 1 silo with src sync rebuild, and guarantee those go together ?14:34
kgunnor 2) have 1 silo with mp's, then create another silo for the src sync...copy over the binaries from that 2nd silo into the first to make it a guarantee ?14:35
ogra_Mirv, looks fine ... if it breaks tedg will get no beer at the next sprint ... so ACK ...14:37
Mirvogra_: serious enough threat, ok!14:39
Mirvkgunn: not understanding the why still, but I'd use 2) ie use a temporary throwaway silo from where you copy when everything is ready to the first silo. it's also easier for QA to test when there's only one final silo.14:40
MirvI need to go now like I probably said a long time ago :) so please wait for robert to wake up for trainguardings14:42
tedgWait, wait. Perhaps bregma should be in charge here, I'm not willing to take those risks!14:46
tedgThanks ogra_ !14:46
nerochiarocihelp: does anyone know why this is failing ? looks like infrastructure problems: https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/3244/console15:26
alf_psivaa: Hi! Did you get a chance to look at the https://jenkins.qa.ubuntu.com/job/mir-android-vivid-i386-build/ failures? Seems like an archive problem?15:38
psivaaalf_: not yet, that's still in the list. sorry being consumed by something else15:39
fginthernerochiaro, it looks like a couple of mako devices have failed. Not sure the cause yet.15:55
fginthernerochiaro, I've triggered a rebuild on that branch15:58
tsdgeoscihelp: all the qmluitests jobs are stuck http://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-vivid/15:59
nerochiarofginther: ok. if it happens on other branches should I alert you ?16:00
fginthernerochiaro, the failed devices have been taken offline, so new runs should be ok for now. But if this caused other runs to fail, we can restart those too16:07
fginthernerochiaro, but at the moment, there is a shortage of working devices16:07
mzanettipsivaa, what tsdgeos tried to say above is that all the slaves are offline and can't be started by jenkins: http://s-jenkins.ubuntu-ci:8080/label/vivid&&amd64/?16:07
fgintherpsivaa, the restart of the libvirt service may have broken the connection to s-jenkins. We may have to restart s-jenkins to correct it16:12
psivaafginther: hmm yea, that could probably be the reason..16:13
fginthers-jenkins.ubuntu-ci needs a restart, sorry for the short notice16:26
=== fginther changed the topic of #ubuntu-ci-eng to: Train trouble? ping trainguards | CI problems? ping cihelp | Train: http://bit.ly/1hGZsfS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: OTA-6 final freeze in effect, but the vivid-overlay landing gates opened. sil2100 away, small trainguard gap between EEST & RDT (robru daylight saving time) timezones, s-jenkins.ubuntu-ci needs a restart
robrukgunn: alright, what's the plan? you want two silos?17:20
* popey creates https://requests.ci-train.ubuntu.com/#/ticket/306 and wonders what happens next.. some bot?18:11
popeyyes, some bot.18:15
kgunnrobru: yep, so question is....do you have magical powers to copy over the pkgs from the silo 24 ppa into the silo 54 ppa ? and do i need to add something else to the silo 54 entry in the train ?18:24
kgunne.g. mir listed in some text block entry to tell the silo 54 configuration that will be there18:25
robrukgunn: I do have magical powers.18:26
robrukgunn: one sec18:26
fginthermzanetti, tsdgeos, the unity-phablet-qmluitests-vivid have been restored18:27
mzanettifginther, thanks :)18:27
robrukgunn: ok I did the copy. you should now add 'mir' to the sources field of silo 54, do the 'assign' and then do a build with WATCH_ONLY18:29
robrupopey: presumably qa will pick that up18:35
kgunnrobru: i must be missing something ^18:40
robrukgunn: yep, you ran a build that deleted all your files but then failed to build anything: https://ci-train.ubuntu.com/job/ubuntu-landing-054-1-build/16/consoleFull so you'll need to rebuild the whole silo now.18:42
kgunnrobru: but i marked it as watch?18:43
robrukgunn: but there's nothing to watch, because everything in your silo was previously deleted. you have nothing. you need to rebuild.18:43
robrukgunn: look at the log I pasted, it literally says "deleting everything" and then when it tries to build it fails and stops.18:44
kgunnrobru: maybe i misunderstood...i thot silo54 would already be empty, except for mir that you just copied in from silo 24, meaning it18:47
kgunnwould "watch" and see mir, but then build the mp's18:47
kgunnand how come when i look in the silo54 ppa i see 6 hrs old pkgs18:48
robrukgunn: i don't... what...18:50
robrukgunn: WATCH_ONLY means that it WATCHes ONLY... I don't understand how you could think that would build anything.18:50
robrukgunn: I also don't understand why you think your silo would be empty? I thought the whole point of this exercise was that you had MPs in there that you'd built?18:50
kgunnrobru: sorry, then i didn't express myself properly... and it's kinda long winded so here goes...18:51
kgunni was trying to get mir that's in wily to vivid+o18:51
kgunnand that mir has an abi break going from old to new in vivid+o18:52
kgunnso i need to rebuild usc & qtmir18:52
dobeytrainguards: eh? 14:35 -queuebot:#ubuntu-ci-eng- dobey, ubuntu/landing-040: Can't publish: Critical data missing from packagelist. Perhaps try a WATCH_ONLY build:18:52
kgunncan't mix mp's & src sync in silo...so, looked at doing one big src sync from wily, but due to other deps (unity-api & qtmir)18:53
robrudobey: right I was going to look at that, one sec18:53
kgunnthat didn't work...so then tried to do all MP's including mir, but then 15.10 v 15.04 complaining ....so18:53
kgunnthis was my last idea...18:53
robrukgunn: your silo is in a broken state and needs to be completely rebuilt. however you got those packages into the PPA, they're orphaned because the necessary branches are not in the silo dir.18:54
kgunndo temp silo src sync for mir, so builds mir happily for vivid+o no complaining18:54
robrukgunn: the thing about versions is easy to fix, you need to branch for vivid rather than trying to build your wily trunk for vivid.18:54
robrukgunn: and in your vivid branch you need to s/15.10/15.04/ on the changelog version of the most recent entry.18:54
kgunnso painful to have 2 branches when it's the same damn src18:55
robrukgunn: well michi is working on a solution to that as I mentioned. but it's a lot of extra work18:56
robrudobey: oh crap, I broke everything, hang on18:56
kgunnrobru: consider for a moment, dual landing allows me not to have to have 2 damn branches....only reason i can't is someone on a 2 degree separation dpendency decided to have 2 branches for what seems to be no good reason18:57
robruslangasek may be better than I am at explaining why two branches are preferrable ^^18:58
dobeythe problem is because your silo has an MP for some other project which doesn't do dual landings?18:58
robrukgunn: anyway, train has support for the magic that lets you dual land, it's up to you to fix your packaging to make it happen, just follow the way michi's doing it if you really want that18:58
slangasekI'm not about to dictate that people *should* use separate branches18:59
slangasekhowever, this kind of drag on dual landings is bound to increase over time19:00
robrudobey: ok I have a fix in trunk, just waiting on IS to roll that out then I can publish it.19:00
slangasekand michi's work isn't ready to land yet in the archive, so I don't recommend adopting that just now if you're trying to unblock something you need landed today19:00
dobeyrobru: ok19:02
robrudobey: ugh, IS guy is on lunch for 1.5 hours already. are you in a hurry to publish that?19:08
dobeyrobru: not critical, but want to get it landed asap19:09
robrudobey: k, if you get sick of waiting for IS to roll out my fix you might want to consider getting a core dev to copy the package to the archive manually, and then merging the silo manually. i have no idea when this guy will get back, already pinged him twice.19:10
kgunnslangasek: what specifically do "no recommend adopting if trying to unblock something landing today"19:22
robrudobey: ok sorry for the delay, published19:22
dobeyrobru: ok, great, thanks19:22
robrukgunn: the packaging changes required to keep dual landing even in the face of gcc5 issues are really staggering, it's not something you can just easily do in a hurry.19:22
kgunngot it19:22
kgunnrobru: but dual landing today just rebuilds no ?19:23
kgunnwhich i like btw19:23
kgunnbin copy seems naughty19:23
kgunn(potentially naughty)19:24
robrukgunn: yes, exactly. dual landing just rebuilds. which is really horribly broken for gcc5 projects because you need to have different binary package names, which means you need to have different debian/control files to achieve that19:24
robrukgunn: so in the train we implemented a hook that allows you to rewrite your debian/control as appropriate for the release you're building for. but it's up to you to write a script that generates the appropriate debian/control19:25
robrukgunn: it's not something you can just drop in and go, it's a really massive architectural shift in your packaging19:25
robrukgunn: if you're curious you can look at what michi's doing in silo 10, it's a big deal19:25
kgunnwhen you say control file, you're implying hard dep on gcc versions19:26
robrukgunn: no, I'm implying that the actual *names* of the binary packages are different for wily than for vivid.19:26
dobeythe ABI is different, therefore package names must be different19:26
robrukgunn: which is not something that debian/control actually supports, so you need to create a meta debian/control that you then pre-process into the correct debian/control as appropriate.19:26
dobeybecause otherwise libmir0 on one will not be the same as libmir0 on the other19:27
robrukgunn: so eg you'd need to rename debian/control to debian/control.in, and then you'd change "Package: libmirN" to "Package: libmir@SONAME@", and then in your debian/rules you make some magic that figures out what the correct @SONAME@ is for vivid vs wily, and then s/@SONAME@/the series-appropriate soname/19:28
robrukgunn: but there's a ton of different places you have to do that, it's all over your install files. like I said, not a quick fix19:29
robrukgunn: oh sorry I've only just noticed. that package copy I did for you failed because the same version was already in the PPA.19:47
bregmahey robru I just rebuilt silo 27 with changes requested by archive admins after a NEW review, is there something that needs to be done to upload to the queue again?20:06
robrubregma: yeah it would need to be published again.20:06
bregmarobru, do I do that or do you guys do that?20:07
* bregma may have been hit on the head and forgotten everything20:07
robrubregma: I'll do it20:07
* bregma was recently on a sprint where his employer is trying to raise an army of sleep-deprived alcoholic zombies20:08
robruslangasek: can I get you to try publishing request 278? apparently checkupload doesn't like NEW stuff ^^20:09
bregmais that a bug in the splendid new bileto code?20:09
slangasekit's by design that you can't publish a new package to the archive without signoff of an Ubuntu dev20:10
slangasekI'm on a call right now, I can give it my full attention in about 20 min20:10
slangasekwhat's with the AttributeError? :)20:11
robruslangasek: attributeerror is coming from lplib, I wrote my code according to spec and caught the ClientError thrown when people aren't authorized to upload.20:15
robruslangasek: ok I have a workaround, just trying to recreate the issue in staging to confirm20:43
slangasekrobru: my attempt to publish: https://ci-train.ubuntu.com/job/ubuntu-landing-027-2-publish/49/20:48
slangasekrobru: despite marking 'ack_packaging'20:48
robruslangasek: thanks, it's clearly a bug in lazr, I'm working on a workaround, just poking around in staging.20:48
slangasekah it's the same backtrace, innit20:49
slangasekok then20:49
robruslangasek: heh, ok workaround has succeeded but it's uncovered a bug in my code now ;-)20:51
robruslangasek: so since this is forcing the issue, what's the story for NEW packages? only core devs can publish NEW?20:52
* slangasek hands robru the norelco yakshaver 200020:52
slangasekrobru: I believe the rule is that core devs and MOTU can upload NEW packages, and other ubuntu-devs cannot20:52
slangasekinfinity, cjwatson, wgrant: ^^ can you sanity check me on that?20:52
robruslangasek: is there an lp team for MOTU? how do I check that?20:52
infinityslangasek: MOTU (which core-dev is a subset of) can upload NEW, yes.  Fuzzier if you're NEWing something detined for main, since you'll never be able to do a second upload if you're not core-dev, but we hand-wave past that. :P20:54
slangasekrobru: can you not rely on the existing checkUpload call to determine this properly?  or is this the part that's failing?20:54
robruslangasek: yes that is exactly what is failing20:54
slangasekrobru: i.e. I would expect the LP API to tell you "is this person allowed to upload this non-existent package? y/n"20:54
infinityrobru: ~motu would be the right team, but obviously hardcoding teams is wrong, since you should be getting the answer from LP.20:55
slangasekrobru: what happens if you call checkUpload() without a source package name?20:55
robruslangasek: checkUpload seems to work fine for packages that exist but it explodes horribly with that AttributeError if you call it on a package that isn't in the archive20:55
slangasektry: checkUpload(sourcepackagename=, ...) except AttributeError: checkUpload(...)20:55
robruslangasek: gimme a sec to poke that20:56
dobeythat AttributeError looks like a python3 vs python2 api issue20:56
dobeyrobru: ^^20:56
robrudobey: yes20:56
robrudobey: I fixed that trivially by monkeypatching.20:56
dobeyah ok20:56
jgdxcihelp: evening, why is webapp tests run for [1]? The params says ubuntu-system-settings test suite only. [1] https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/325220:57
jgdxdoesn't happen for all builds, but this is not the first time I believe20:58
jgdxin #3214 unity8 tests are run20:59
slangasekrobru: hmm bug #82136620:59
ubot5bug 821366 in Launchpad itself "IArchive.checkUpload() requires existing sourcepackagename" [High,Triaged] https://launchpad.net/bugs/82136620:59
robruah, yes20:59
infinityOnly 4 years old.20:59
robruinfinity: they'll fix it any day now21:01
infinityrobru: So, a gross workaround (which may or may not be better than hardcoding a team) would be to trap the expected error from a nonexistant source, and sub in spn=sendmail (which will never, ever, ever leave universe). :P21:03
robruslangasek: Uncaught exception: ValueError: No value for required parameter 'sourcepackagename'21:03
infinitysendmail is in main.  Nevermind.  lolwut.21:04
robruinfinity: sorry I'm just catching up with slangasek's suggestion to omit spn21:04
infinityThere would be others that would fit the bill.21:04
infinityBut gross hack either way.21:04
slangasekrobru: yeah confirmed locally and via api docs that spn is required21:04
robruslangasek: thoughts? hard-code default package name or hard-code team membership21:04
infinityIt's the same end result, really.  We could change which team has upload rights (but probably not), or we could pick a package we're sure will never leave universe, but it might.21:05
robruslangasek: I can say "if no sourcepub is found, check person's teams for MOTU" or whatever easily enough21:05
infinityThe worst fallout from the package hack is that you further restrict uploaders.21:05
slangasekrobru: I think it's easier to hard-code a default package name21:05
robruslangasek: ok, what's a good package to use then? if not sendmail?21:05
infinityNow, let me find something in universe that sucks so bad we'll never promote it, but is so ubiquitous that we'll never remove it.21:06
slangasekinfinity: would something from multiverse work?21:06
infinityslangasek: Yes.  Flash?21:06
slangasekmaybe one of those packages that's blacklisted in seeds21:06
slangasekhmm but libavcodec has an soname so that doesn't help21:07
slangasekoh, except source package - so 'libav'21:08
slangasekrobru, infinity: ^^ my suggestion for a default source package21:08
infinityflashplugin-installer is mine.21:08
robruslangasek: infinity: oh I just tried it with an empty string and it seems to have worked21:09
infinityWell, that works better then.21:09
robruslangasek: infinity: well "worked" in the sense that it told me I'm not authorized21:09
infinityYou still need to trap and retry, but that's fine.21:09
fgintherjgdx, ah. I found that it was configured to use the wrong workspace directory and it was inadvertently picking up results from another test. Should be fixed shortly.21:09
robruslangasek: can I get you to run this publish job with and without ACK? https://ci-train.staging.ubuntu.com/job/ubuntu-landing-001-2-publish/build?delay=0sec21:09
robruinfinity: trap and retry?21:10
fgintherjgdx, 'it' being the mako-20 which ran that test.21:10
infinityrobru: Rerun it with a personid who has rights?21:10
infinityrobru: Well, you need to checkUpload for your spn, then retry if it explodes, no?21:10
robruinfinity: well, that would mean finding a person to run the job, not something I can do in the code21:10
robruinfinity: no, I call getPublishedSources to confirm my spn exists. if it does I set it, if not, empty string21:10
infinityrobru: Oh, or that, sure.  Mine was an optimisation to avoid the check except when needed, but if you getPS for other reasons, yours is cleaner.21:11
robruinfinity: yeah we need to check the component also21:11
robruinfinity: can you run that job I linked ^^ with and without ACK_PACKAGING checked? need to make sure it really works21:12
infinityrobru: Is this going to publish a real thing to a real place?21:13
robruinfinity: no it's staging21:13
robrunot connected to anything. just need to make sure it doesn't say "infinity not authorized to upload"21:13
infinityrobru: Okay.  The big blue "build" button?21:13
robruinfinity: yep21:13
infinityadconrad is missing the Job/Build permission21:14
robruinfinity: did you check the teams on SSO when you logged in?21:14
robruinfinity: core devs should be authorized21:14
infinityNo.  If you need teams, shouldn't you set them mandatory in your SSO config? :P21:14
robruinfinity: that is the case in production but not in staging21:15
robruinfinity: please log out, check the teams, log back in, run the job21:15
infinityOkay, seems to have done a thing when hitting the button without ticking any boxes.21:15
robruinfinity: ah, so that's bad then, if it thinks you're not authorized.21:16
infinityIt did a thing.  Not nothing.21:16
infinityI'm assuming I was authorized.21:16
robruinfinity: the thing it did is that it told you you're not authorized to upload a package to universe. I'd consider that a failure, no?21:16
infinityOh, where do I see that? :P21:17
robruinfinity: click through to the console log21:17
robruinfinity: I'll try this again with libav.21:17
infinityMan, I love jenkins.21:17
robruinfinity: I can't wait to get rid of it ;-)21:18
brendandinfinity, here's your prize for being the first person ever to say that!21:18
infinityrobru: flashplugin-nonfree, please.21:18
infinityrobru: libav will probably get removed some day soon as ffmpeg takes over, and fp-nonfree is multiverse, so a true superset of archive permissions.21:18
infinitybrendand: It was said ironically, I can't possibly be the first.21:19
brendandinfinity, if said ironically, then no, definitely not the first21:19
=== fginther changed the topic of #ubuntu-ci-eng to: Train trouble? ping trainguards | CI problems? ping cihelp | Train: http://bit.ly/1hGZsfS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: OTA-6 final freeze in effect, but the vivid-overlay landing gates opened. sil2100 away, small trainguard gap between EEST & RDT (robru daylight saving time) timezones
* brendand takes back infinity's prize21:19
robruinfinity: what's the source package name? flashplugin-installer?21:19
infinityrobru: flashplugin-nonfree21:20
robruinfinity: ok, run that job one more time with debug checked please21:21
robruinfinity: ok perfect, thanks21:22
robruinfinity: that's confirmed working then, I'll write some tests now21:22
infinityrobru: Hey, it even works if I ACK the packaging changes.21:23
robruinfinity: yep21:23
infinity2015-09-02 21:23:12,462 DEBUG Beginning: phase: ackaging21:24
infinityGood ol' ackaging.21:24
robruinfinity: it's a portmanteau of ACK and PACKAGING. ;-)21:24
infinityrobru: Righto.  For when someone needs to pack the ackage.21:25
slangasekrobru: https://ci-train.staging.ubuntu.com/job/ubuntu-landing-001-2-publish/build?delay=0sec done; Access Denied21:26
robruslangasek: what?21:26
robruslangasek: also infinity beat you to it, but thanks for trying21:26
slangasekrobru: ok :)21:26
infinityrobru: If this code is intended to live more than a few minutes, you might want some obnoxiously loud "XXX: AAAACK, STUPID HACK" comments around the fake spn bit. :P21:29
robruinfinity: I expect this will be in production for 4-8 years. ;-)21:29
robruinfinity: 1-2x the current age of that bug ;-)21:29
* infinity wonders how LP's own queue deals with this.21:31
infinityI would have thought it calls the same method, but, well.  Soyuz, man.21:31
slangasekrobru: why does staging want to know from SSO that I'm a core-dev, as opposed to just knowing that I'm a ci-train-users?21:32
robruslangasek: because core-dev is required for publishing.21:34
slangasekrobru: oh, because that's the existing check, right21:35
slangasekand goes away when this is done ;)21:35
robruslangasek: yeah. jenkins ACLs go by launchpad teams21:35
robruslangasek: you were saying something about code review? https://code.launchpad.net/~robru/cupstream2distro/monkeypatch-lazr/+merge/269988 ;-)21:35
robrudobey: http://bazaar.launchpad.net/~robru/cupstream2distro/monkeypatch-lazr/revision/1070#citrain/recipes/base.py fun times. fun times.21:36
dobeyoh, ugh21:37
robruslangasek: does that mean you'll review or is that sarcasm? ;-)21:37
infinityrobru: When you tested an empty spr, did you test None or an empty string?21:37
slangasekrobru: I am reviewing21:37
robruinfinity: empty string21:37
robruslangasek: thanks21:37
slangasekinfinity: both None and emptystring fail21:38
infinityrobru: Reading some source here, internally, 'None' would work.  Though I'm not sure if that can be represented sanely over an xmlrpc API, which might be the issue.21:38
slangasek(checked with lp-shell)21:38
slangasekNo such source package: ''.21:38
slangasekand with None, it's sourcepackagename: Required input is missing.21:38
infinityslangasek: Yeah, could be a serialisation issue or something, might need to explicitly allow null and the empty string as well to make the wire protcol happier.21:39
infinityslangasek: Cause the internal methods totally don't choke on spn=None.21:39
infinityWhich is why this works for the real queue.21:39
slangasekrobru: approved21:40
robruslangasek: thanks21:40
robruslangasek: ok, seeing as that only affects NEW packages I think I'll roll that out tomorrow batched together with some other stuff. I already did 2 rollouts today, trying not to bother IS too much...21:43
slangasekrobru: well, I believe bregma's publication is blocked until this is fixed21:45
slangasekI think you want to at least /ask/ IS for the rollout21:45
infinityslangasek: No, I did bregma's by hand.21:45
robruyeah ^21:46
robrubregma: although we will need ~ci-train-bot added to ~libertine-team21:46
infinityslangasek: To wily, that is.  We could do the same to the overlay, if that's also blocked.21:46
robruinfinity: oh I did that already. was that premature?21:46
infinityrobru: Oh, if you did vivid-overlay, fine by me, I wasn't sure if it was choking on the same issue.21:46
robruinfinity: yeah the publish job was exploding before it got to the part where it copied to vivid21:47
bregmawhy is it always me?21:48
infinitybregma: Nefarious reasons.21:48
robrubregma: stop making new projects with new teams :-P21:48
slangasekinfinity: ah ok21:49
bregmarobru, should that silo get merged now?21:50
robrubregma: did you add the bot to the team?21:50
bregmaoh, crap, that's what I forgot21:50
robrubregma: after that's done I don't think there's any harm in merging. strictly speaking the train would have waited for the package to migrate fully but the excuses page says "valid candidate" so that should migrate real soon now...21:51
infinityExcept that, entertainingly, it thinks libertine breaks itself on arm64.21:53
robruinfinity: that excuses page is a bit rubbish if "valid candidate" is a false positive.21:54
LaneyValid for the next stage21:55
infinityrobru: It's a two-stage process.21:55
LaneyE: Package 'proot' has no installation candidate21:55
LaneyThat is why21:55
robruinfinity: is that something you can wave through or will this require fixes from bregma?21:55
Laneyhttps://bugs.debian.org/cgi-bin/bugreport.cgi?bug=788113 maybe look at that21:56
ubot5Debian bug 788113 in src:proot "proot: FTBFS on arm64" [Normal,Open]21:56
infinityWhat's pulling in proot?21:57
infinityOh, the python module.21:57
Laneylibertine -> libertine-tools -> python3-libertine -> proot21:57
infinityWhich is why I didn't see it in the arm64 build log. :P21:57
robrubregma: arm64 is for chumps, i wouldn't support it if I were you.21:57
bregmalibertine pulls in proot for the time being because of need in vivid-overlay21:57
LaneyThat bug has a patch you know21:57
LaneySomeone arm64ish could review it21:57
infinityIt does indeed.21:58
infinityAnd I shall.21:58
infinityOkay, that patch alone leads me to believe that proot is gross.21:58
robrubregma: anyway it's sounding like infinity is going to fix proot on arm64 and then you'll be good to go. watch here: https://launchpad.net/ubuntu/+source/libertine and merge when it says "release (universe)"21:59
* Laney goes to bed happy 22:00
infinityIt's going to have the same sadness on ppc*, mind you.22:01
infinityWhere proot isn't ported at all.22:01
fgintherbregma, speaking of libertine... Should ci for lp:libertine/devel be done on wily or vivid+overlay or something else?22:01
bregmafginther, wily please22:02
fgintherbregma, danke22:02
* bregma steps out to buy some groceries for his poor starving family22:03
infinitybregma: OOI, why was fakechroot good enough for later releases, but not trusty?22:03
infinitybregma: Given fakechroot is the same version in trusty->wily, that seems odd.22:05
infinitybregma: (Also, if this is ever meant to be supported, fakechroot has a 2000% better chance at passing an MIR than proot does, from my quick reading of the proot source)22:06
fginthermandel, what's the story on ubuntu-download-manager? That project was never moved to wily. Should it be?22:24
bregmainfinity, I don;t know why proot over fakeroot, I'd have to ask ChrisTownsend and he's EOD23:38
bregmahoping to elide proot by the time we need to MIR23:38
infinitybregma: Well, there's code there for both proot and fakechroot, hence the question.  (And also no dep on fakechroot, so it'll probably fail to work in those cases).23:46
bregmainfinity, we actually did a lot of refactoring at a sprint last week and are aiming to do more in the near future, so either we left some incorrect dependencies in there or we'll find out the hard way we missed some23:49
bregmawe're trying to shoe-horn this project into a use it was not designed for on a system that lacks proper support, so we're doing a delicate dance23:49
bregmaalways good for some laughs and a good story later23:50
infinitybregma: Right.  I need to run to dinner.  To be continued.  But I suspect the proot dep can be dropped today, with a bit of deletion.23:57

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