[00:11] <cjwatson> bluesabre: Usually somewhat later on Thursday
[00:11] <bluesabre> cjwatson: thanks, I've been in panic packaging mode :)
[14:17] <Riddell> stgraber: you're the iso tracker dude?
[14:38] <Riddell> stgraber: I added Kubuntu Plasma 5 to isotracker, how do I get it to show up under Utopic Daily ?
[14:43] <Laney> Riddell: I think you have to have it match etc/qa-products in lp:ubuntu-cdimage and then it should show up after a build
[14:43] <Laney> semi-educated guess
[14:43] <Laney> (In that file the product name doesn't contain "Desktop")
[14:51] <Riddell> doen't seem to help to remove the "Desktop" from isotracker product
[14:52] <Laney> has a build finished?
[14:53] <Riddell> not since I added it
[14:53] <Riddell> maybe I should just be patient
[14:54] <Laney> 17 22 * * *for-project kubuntu-plasma5 cron.daily-live --live
[14:54] <Laney> I'd see if it happens after then
[15:04] <cjwatson> [5~/wg 24
[15:04] <cjwatson> sigh
[15:04] <Riddell> queuebot: ooh?
[15:30] <Riddell> how does this file get edited? http://releases.ubuntu.com/include/kubuntu.css
[15:30] <cjwatson> by hand, I'm afraid
[15:30] <cjwatson> happy to plonk in a new version
[15:35] <Riddell> cjwatson: could you   cp kubuntu.css kubuntu-plasma5.css; sed s,Ubuntu,Oxygen, -i kubuntu-plasma5.css
[15:37] <cjwatson> Riddell: done
[15:39] <Riddell> cjwatson: then eye over and pull the change I just commited to ubuntu-cdimage
[15:41] <cjwatson> Riddell: single-element tuples don't work like that
[15:41] <cjwatson> ("kubuntu") is just grouping not a tuple, so foo in ("kubuntu") is true if foo is "k" or "u" or "b" or "n" or "t"
[15:42] <cjwatson> Riddell: also you have a bunch of test failures
[15:42] <cjwatson> Riddell: I think mostly because you're missing file=header on various print calls, but make sure to use ./run-tests
[15:43] <cjwatson> for the former problem I would just write 'if self.project == "kubuntu":' etc.
[15:43] <cjwatson> and save in-tuple for the case where you have more than one
[15:44] <Riddell> mm ok, that seems unintuitive
[15:45] <Laney> you need the trailing comma for one-element tuples
[15:45] <cjwatson> Yeah, that works too, but style in cdimage is usually just to use equality in such cases
[15:45] <Laney> Sure, just explaining the python syntax
[15:49] <Riddell> that seems like the sort of unintuitive change in behaviour you'd expect from JavaScript not Python :)
[15:49] <Riddell> cjwatson: ubuntu-cdimage updated
[15:52] <cjwatson> Riddell: thanks, but still produces junk on stdout during the tests which indicates missing file=header on print, and there's a test failure if you have pep8 installed
[15:54]  * Riddell investigates
[15:56] <Riddell> cjwatson: third time lucky, ubuntu-cdimage updated
[16:00] <cjwatson> Riddell: great, thanks - deployed
[16:00] <Riddell> yay
[16:23] <bdmurray> slangasek: I'd like to release this ubuntu-release-upgrader SRU to trusty although one bug failed verification (the fix for that one is still in the works).
[16:25] <stgraber> Riddell: sorry, at a conference with pretty bad wifi. If the product exists on the tracker, it should automatically show up with the next build, so long as the cdimage branch contains the proper mapping between internal cdimage product name and qatracker product name.
[16:26] <stgraber> Riddell: if the mapping doesn't exist or is wrong, you'll see an error message in the iso build log
[16:26] <Riddell> stgraber: ok I'll check back tomorrow thanks
[16:26] <stgraber> Riddell: KeyError: "Product 'Kubuntu Plasma 5 amd64' not found"
[16:27] <stgraber> Riddell: http://people.canonical.com/~ubuntu-archive/cd-build-logs/kubuntu-plasma5/utopic/daily-live-20140820.log
[16:27] <Riddell> right, and now it has "Kubuntu Plasma 5 amd64" so hopefully that's all good
[16:28] <stgraber> ok, so the next daily build should just show up then
[16:33] <slangasek> bdmurray: should we reupload dropping the bug reference from the changelog, to avoid confusion?
[16:34] <slangasek> gaughen: docker.io accepted into trusty-proposed now
[16:35] <gaughen> slangasek, thank you!
[16:37] <bdmurray> slangasek: ah, yeah that'd make sense. can that be done using the same version number or is a new one required?
[17:05] <slangasek> bdmurray: new one required
[17:09] <bdmurray> slangasek: ack, I'll get that uploaded then
[17:09] <slangasek> bdmurray: cheers
[17:22] <wxl> hey not to be jumpy but are we quite ready to start beta1 testing?
[17:24] <elfy> beginning of next week wxl - I've not seen the "who's doing beta" mail yet either
[17:24] <wxl> elfy: ok just don't want to miss it this time XD
[17:24] <elfy> I'll try and remember to ping you if you like :)
[17:25] <wxl> elfy: as long as someone does ;)
[17:26] <elfy> heh
[18:00] <bdmurray> slangasek: uploaded
[18:02] <slangasek> bdmurray: looks good, accepted
[18:20] <bdmurray> slangasek: there is no need to reverify the fixed bugs correct?
[18:25] <slangasek> bdmurray: based on that diff, I'm happy to skip reverification
[22:00] <bdmurray> In case anybody is interested I've update the metarelease fiels for the release-upgrader in trusty-updates
[23:08] <bdmurray> cjwatson: is it expected for packages to exist in -updates and -proposed after a run of sru-release?  see systemd and ubuntu-release-upgrader
[23:08] <cjwatson> Yes
[23:08] <bdmurray> for how long?
[23:09] <cjwatson> We don't have a copyPackage with move semantics yet (I've been working on that, haven't quite finished), so they have to be removed manually
[23:09] <cjwatson> Until somebody processes the "-proposed cleanup" block in pending-sru.html
[23:09] <cjwatson> I'll do that now
[23:10] <bdmurray> cjwatson: got it, thanks!
[23:10] <cjwatson> You probably haven't noticed since I tend to reflexively process it when I'm bored :)
[23:11] <bdmurray> well and I've been so buried with the error tracker
[23:11] <cjwatson> that batch is removed now