/srv/irclogs.ubuntu.com/2014/04/10/#ubuntu-release.txt

stgrabertjaalton: so the sssd upstart fix ends up taking 100% of CPU... I found an alternative that seems safe, will upload that now...00:01
jdstrandcan someone also look at oxide-qt? it was approved earlier00:07
jdstrandif you tell me to just do it, I can myself00:07
cjwatsonI don't suppose there's any chance of bug 1297804 being approved?00:12
ubot2Launchpad bug 1297804 in libpipeline (Ubuntu) "FFe: libpipeline 1.3.0" [Wishlist,New] https://launchpad.net/bugs/129780400:12
stgrabercjwatson: approved00:14
cjwatsonoh, that was quick, thanks :)00:15
infinitystgraber: That sssd exec is icky...01:02
stgraberinfinity: does the trick, though I agree that upstream is just buggy there and should be fixed to not fail or eat all your CPU when using a sane stdin...01:05
infinitystgraber: Why not just start it with -D and expect fork?01:05
infinityOr expect daemon.01:05
infinityWhatever.01:05
stgraberinfinity: because sssd is also buggy in that case01:05
infinitystgraber: Err, srsly?01:06
stgraberinfinity: as in, it'll daemonize before it's done parsing your config01:06
stgraberso if your config isn't valid, it daemonized, then exits before the second fork, confusing the hell out of upstart in the process01:06
stgraberand you end up with upstart thinking it's running when it's not, with the only solution to fix the state being a reboot of the system01:06
infinitystgraber: util/server.c seems to imply and attempt to make that not happen. :/01:06
stgraberclearly failing :)01:07
infinityBleh.01:07
infinitySeems like it would be worth fixing correctly instead of working around weirdly.01:07
stgrabersetup sssd to use a krb5 ticket to authenticate, forget to setup said ticket before starting sssd => boom, upstart thinks it's running but sssd exitted01:08
stgrabertotally agree that having sssd be a bit more sane when running in either foreground or background would be nice, though it's not something I have much time to debug and resolve properly right now, so the hackish upstart job at least does the trick and make things work reliably01:09
stgraber(sssd basically spawns 5-6 sub-daemons, all of which read their part of the config, failure from any of those may cause the main daemon to exit. So making sure that the daemonizing case is sane may be a bit challenging...)01:10
infinityWow, it actually scans stdin.  That explains why /dev/zero drove it nuts.01:12
infinityAnd it reads an EOF as an explicit kill.01:12
infinityWHO WRITES SOFTWARE LIKE THIS.01:12
cjwatsonRunning stuff in the foreground is generally the right answer with modern inits anyway ...01:12
infinitycjwatson: It is when it's not written by nutters.01:12
cjwatsonWell yes01:12
infinitystgraber: With /dev/null as stdin, did you get "EOF on stdin - terminating"?01:13
infinityOh, FFS, how do I configure this to run it? :P01:17
infinitystgraber: Erm.  So, I can't make it die with "sssd < /dev/null" ...01:22
infinityOh, maybe -D is the default.01:22
=== FourDollars_ is now known as FourDollars
infinityIndeed it is.01:23
=== superm1_ is now known as superm1
infinitystgraber: http://paste.ubuntu.com/7228797/01:37
infinitystgraber: That makes it stop doing Very Stupid Things with stdin, and </dev/null works again.01:38
jdstrandinfinity: hey, shall I approve oxide-qt-- it got the testing and is in unapproved. I'm happy to have someone else do it too of course01:39
stgraberinfinity: well, the question then is whether there's a case where something spawns sssd and actually wants to feed it stuff on stdin or kill it by closing it01:39
infinityjdstrand: Is it identical to what I reviewed in your PPA before?01:39
jdstrandyep01:39
stgraberinfinity: that'd seem pretty unlikely, but upstream did write those lines and the stdin handler for some reason after all...01:40
jdstrandbit for bit01:40
infinityjdstrand: Then go nuts.01:40
jdstrandthanks! :)01:40
jdstrand(it was just a binary pocket copy)01:40
infinitystgraber: I'd guess it's for debugging.  Not even the testsuite uses it, can't fathom why a user would.01:40
infinitystgraber: Besides, Ctrl-C still works.01:40
infinitystgraber: Take it or leave it.  If you think the mangled upstart job is better, I can confirm that also works, I just think it's awful. :P01:41
stgraberinfinity: so I think I'd rather stay away from distro changes to the upstream code at this point and bring this upstream, then hopefully the next point release will contain their fix and we can use that and drop the upstart job hack01:42
stgraberthough tjaalton is the one who does most of the work on sssd in Debian/Ubuntu so he may have a different opinion01:42
infinitytjaalton: ^01:43
infinitystgraber: Does either of you have a good relationship with upstream?01:43
infinity(Given that upstream seems to be very Fedora-centric, I can't see how this hasn't also been an issue with systemd... Does systemd play differently with FDs by default?)01:43
stgraberyeah, we have a pretty good relationship with upstream and they tend to be pretty quick at fixing issues (like that samba4 crash we reported and got fixed within the hour earlier today)01:44
infinitystgraber: So, I think they should fix their buggy -D implementation, but I imagine that will take Serious Thought(tm).  But making -i not bugger stdin should be trivial.01:45
infinitystgraber: And I can honestly see no reason why it does, except for debugging convenience.01:45
infinityBut.. *shrug*01:45
infinitystgraber: I'll accept your upstart job fix for now, I confirmed that it works.  I'm just going to register my formal grumpiness with it being poop. ;)01:46
stgraberinfinity: I'll file an upstream bug report now so I can forward your grumpiness where it belongs ;)01:48
infinityOkay, based on commit messages, looks like they're using -D under systemd, which would be why this weird -i behaviour isn't biting them.01:52
infinityAnd it could just be luck of job timing that they're not seeing the bug you do with -D.01:53
stgraberbug filed upstream01:53
* infinity washes his hands of it.01:53
slangasekbdmurray: yeah, early release of 1286161 seems ok to me02:15
slangasekhmm, was qtbase-opensource-src auto-accepted, or did someone else review that?02:16
cjwatsonI think I accepted the symbols changes02:18
cjwatsonwell, not the gles thing if that's been uploaded02:18
cjwatsonthe previous change02:19
slangasekstgraber: do you know why the logind upstart job is in a Multi-Arch: same pam package instead of in systemd-services - and why the service doesn't get started on install?02:19
slangasekstgraber: I ask because I happened to have my system in a state affected by bug #1302264 and found myself needing to reboot... NM was unhappy :P02:19
ubot2Launchpad bug 1302264 in systemd (Ubuntu) "systemd-logind assert failure: error.c:319: Assertion failed in nih_error_get: context_stack != NULL" [Critical,Fix released] https://launchpad.net/bugs/130226402:19
slangasekcjwatson: ah, ok - yeah, it's only the gl side that's been uploaded for now02:20
stgraberslangasek: I don't know why the packaging was made this way... pitti may know.02:20
slangasekok02:20
slangasekthe first part seems to have been bug #115607402:21
ubot2Launchpad bug 1156074 in systemd (Ubuntu Raring) "installing systemd-services without libpam-systemd loses device ACLs" [High,Fix released] https://launchpad.net/bugs/115607402:21
slangasekthe second part is a bug introduced in saucy when trying to avoid restarting logind on upgrades02:27
rsalvetislangasek: ^ for qtdeclarative03:47
slangasekrsalveti: so libqt5quick5's exported ABI doesn't change with the gles switch, but two variants need to be built because it will depend on GL-specific symbols when built against it?03:49
slangasek(just confirming that my understanding of the patch matches your intent)(03:49
rsalvetislangasek: 2 things, it'll depend on gl directly and will also depend on the packages built with gl by default, but they export the same abi03:50
slangasekok03:51
slangasekaccepting03:51
rsalvetione thing we need to better investigate later is what to do when a package depends on a common set of gl/gles symbols03:51
rsalvetiI know in linaro we had a small library that could do runtime detection and decide gl/gles in runtime03:51
rsalvetibut not so sure why nobody put a lot of more effort on that03:52
slangasekbecause glue libraries are messy :-)03:52
rsalvetibut in this qt case, we might just follow directly with qt upstream, as they are doing a similar check in trunk (but currently just on windows)03:52
rsalvetiyeah :-)03:52
tjaaltonstgraber, infinity: heh, so it wasn't just the old upstart job that was buggy.. good to know. And I'm fine with the current workaround04:52
* infinity curses component-mismatches trying to yank random universe kernels in again.04:59
=== SpamapS_ is now known as SpamapS
=== maclin_ is now known as maclin
=== knome_ is now known as knome
seb128hey release08:01
seb128could somebody review bamf/compiz/nux in the queue?08:02
dbarthgood morning, so today i'm trying to land a few things:08:09
dbarthaccount-plugins08:09
dbarthwebbrowser-app08:09
dbarthand libunity-webapps and related (they come from silo 8)08:09
seb128Laney, so, nobody reviewed that bamf changes during the night :/08:20
LaneyI imagine it'll get cleared out before final freeze08:21
* Laney will look at other things now08:21
seb128k, fair enough08:23
sil2100seb128: I see compiz, nux and bamf still in the queue - any hope those will move out of UNAPPROVED soon? :)08:42
seb128sil2100, I do as well08:42
seb128I pinged about that earlier08:42
seb128seems like the US team didn't get to those during their shift, let's see if the europeans do08:43
sil2100o/ Thanks08:43
seb128stgraber, infinity, slangasek; did you guys reviewed the queue during your day/kept those on hold for a reason, or it's just that nobody has slots for reviews?08:44
infinityseb128: Did some, but not many.  I'll do a few more before I head to bed.08:48
infinityseb128: Hard freeze tomorrow (well, tomorrow for me) anyway, so it should slow down after that. :P08:49
seb128infinity, thanks ... and yeah, "should", I don't see that really happening though08:50
seb128well, a part of the uploads is going to turn into SRU08:50
seb128but I can see a steady flow of SRUs between release and .108:50
infinityInevitably, yeah.  Seems to be the way with LTSes.08:51
maclinslangasek,hi, I am maclin from Ubuntu Kylin.  I have push new source code of software center to launchpad. All the problems you found have been modified except python2. Could you help to review it?08:55
JackYuslangasek, maclin means the FFe bug #1293299.08:59
ubot2Launchpad bug 1293299 in Ubuntu Kylin "[FFe]upload ubuntu-kylin-software-center into archive" [High,Confirmed] https://launchpad.net/bugs/129329908:59
seb128sil2100, ^ things got accepted, we should have some silos we can m&c in the next publisher runs09:37
seb128(thanks to whoever did the reviews btw!)09:37
cjwatsonyw09:37
cjwatsonwould be nice if somebody could review libpipeline (it has an approved ffe now) so that I can sync man-db09:38
Laneyha, gexiv2 was what I was referring to in #ubuntu-devel09:40
cjwatsonoh, whoops09:40
cjwatsonsorry09:40
cjwatsonI was motivated by seeing several things in p-m that were blocked on it ...09:40
Laneynever mind, fixable09:40
cjwatsonit'll hit NEW anyway, so you have a chance :)09:40
* cjwatson suppresses his "auto-NEW stuff from Debian" twitch09:42
sil2100\o/09:44
sil2100Thanks guys!09:44
sil2100Can't wait for those to migrate out of proposed ;)09:44
cjwatsonlibpipeline> thanks09:56
* infinity tips hat.09:56
* infinity falls asleep in hat.09:57
infinity^-- Can someone do a quick review of that flash-kernel?10:30
infinityI'm told it's tested to DTRT, and I verified the strings are correct.10:31
cjwatsoninfinity: LGTM10:32
Riddellpossible kde-l10n spam ahead11:55
dokojibel, pitti: https://jenkins.qa.ubuntu.com/job/trusty-adt-python3.4-armhf/22/?  looks like the armhf test is run too early. the armhf build is not yet available as you can see from the log12:03
jibeldoko, it's possible, armhf are triggered when x86 tests pass. I'll restart it.12:05
jibel(when the build is available)12:06
dokoneed to fix the aarch64 build anyway12:06
jpdsCan someone please poke ima-evm-utils through NEW?12:28
=== Ursinha is now known as Ursinha-afk
jamespagecould the neutron upload in proposed be rejected please - I missed a bit13:58
infinityjamespage: Done.13:58
jamespageinfinity, thanks13:58
didrocksLaney: sorry to nag you, we have a regression fix from today's landing in webbrowser-app which is really needed for next image14:14
didrocksLaney: it should be easy enough to review :)14:14
didrockshttps://code.launchpad.net/~dbarth/webbrowser-app/notitle/+merge/21516614:14
didrocks(and there is a new unity8 coming which will need a bump in the fauxpackage)14:14
Laneydidrocks: ok, sec14:18
didrocksthanks man :)14:20
=== Ursinha-afk is now known as Ursinha
=== pete-woods1 is now known as pete-woods
didrocksLaney: lost in space and time? :p14:37
Laneyhave patience my child14:39
didrocksthanks Laney14:42
Laneynp!14:43
bdmurrayslangasek: did you want to release ubuntu-release-upgrader to saucy-updates or shall I?14:50
slangasekbdmurray: please go ahead14:50
jamespageinfinity, neutron with missing bits included now in the queue14:54
infinityjamespage: I assume that means you added bits, not that more are missing? :P14:55
jamespageinfinity, I did indeed add the missing bits :-)14:55
zequenceAny core-dev around, that could lend us some upload rights? Bug 129167515:04
ubot2Launchpad bug 1291675 in lmms (Ubuntu) "[FFe] LMMS 1.0.0" [Wishlist,Triaged] https://launchpad.net/bugs/129167515:04
zequenceThis package has some major improvements to the released one. Has been tested and is pretty much ready to go.15:04
infinityzequence: Wasn't ScottK asking you about that last night?15:15
zequenceinfinity: Yep. He approved it as a FFe. There was a repackaging since15:16
LaneyI probably ought not to review gnome-screensaver since it's my own15:24
LaneyWould appreciate it being looked at15:24
cjwatsonaccepted gnome-screensaver15:26
LaneyCheers15:27
Laneyhappy mostly working lockscreen15:27
cjwatsonthat fixes the double-lock?15:27
Laneyunity mainly, but that has a hand15:27
LaneyIt takes over the D-Bus interface from gnome-screensaver, but g-s needed patching to allow itself to be replaced on the bus15:28
Laney(Which is quite a neat facility in D-Bus that I wasn't familiar with)15:28
cjwatsonright15:28
slangasekmaclin, JackYu: is someone taking care of sponsoring ubuntu-kylin-software-center?  I doubt I will have time to review it today, the best thing is to get it uploaded and get it reviewed by whoever's available at the time15:44
slangasekroaksoax: why is bug #1294323 private?  public bugs only for FFes, please15:44
JackYuslangasek, hi, we don't have the rights to upload it.15:45
JackYuslangasek, and neither happyaron...15:46
slangasekJackYu: understood; so you don't have any sponsor lined up for this?15:46
JackYuslangasek, yep, that's the situation:(15:47
slangasekok15:47
slangaseklet me see what I can do about that15:47
JackYuslangasek, great, thanks.15:47
JackYuslangasek, we are waiting for ubuntu-kylin-software-center and ubuntukylin-keyring being uploaded. Then we could upgrade the ubuntukylin-default-settings, so that these two packages would be included into the distribution.15:49
slangasekJackYu: right.  I've uploaded ubuntukylin-keyring, it's in the NEW queue now and should get review from another archive admin15:49
slangasekinfinity: ^^ could you slot that in maybe?15:50
infinityPhrasing.15:50
infinityslangasek: So, this is the keyring with the agreed-upon generated-by-IS key?15:51
slangasekinfinity: that's what I'm claiming15:51
infinityslangasek: Can I independently validate this key in the DC somewhere?15:52
roaksoaxslangasek: yeah I filed a different public bug15:52
slangasekinfinity: you can check it against the RT15:52
slangasekinfinity: RT #6908015:52
infinityslangasek: Ta.15:52
infinityslangasek: Is this just a cargo-cult of ubuntu-keyring?15:57
slangasekinfinity: yes, as noted in the linked FFe bug15:58
infinityOh, who reads bugs?15:58
infinityslangasek: I was just going to whine about the lack of build-* targets, but if it's derived, I don't much care.  debdiffing against ubuntu-keyring to see how close they are.15:59
infinityslangasek: Is it expected that installing this keyring makes apt trust it, or will an installer mangle that?16:08
=== maclin_ is now known as maclin
slangasekinfinity: isn't that what the postinst does? (apt-key update)16:13
infinityslangasek: You'd think.16:14
infinityslangasek: Maybe it doesn't like ascii armored keys and wants an actual keyring?16:14
slangasekJackYu: xnox is reviewing ubuntu-kylin-software-center now for sponsorship16:14
slangasekinfinity: ah; I would think that and did think that, but did not in fact test it16:14
JackYuslangasek, xnox, thanks. maclin and I are here waiting for your questions:).16:15
xnoxJackYu: so far it's ok, one minor dependency is missing which i'll fix up before uploading.16:18
xnoxJackYu: and i'm writting up review comments on the bug report.16:19
xnoxJackYu: no blocking issues found yet, but i'm not done reviewing.16:19
Saviqwho do I bribe to push unity8 through to release? ;)16:19
xnoxSaviq: as per topic "we accept payment in cash, check or beer" and https://launchpad.net/~ubuntu-release/+members#active for the list of people16:19
infinitySaviq: I'll sort it.16:20
Saviqinfinity, awesome, thanks, infinity.beer++16:20
JackYuxnox, got it.16:20
infinityI'll take a raincheck on the beer.16:21
apwthe xen and libvirt combination are best viewed together and are a fix for the migration of machine definitions on upgrade16:22
JackYuxnox, sorry for my bad understanding, you mean you will upload it now and review it later, or you just upload it and we should find another sponsor to review it?16:26
slangasekJackYu: he is reviewing and once he is done reviewing, if there are no blocking problems, he will upload16:27
JackYuslangasek, ah:).16:28
infinityslangasek: Okay, so.  Two things.  It does seem to want to be a gpgp keyring, not an ascii art masterpiece.  But, also, I can't seem to get apt-key update to add it to trusted.gpg.  I *think* it only looks at archive.gpg16:30
infinityslangasek: Shipping it in /etc/apt/trusted.gpg.d/ubuntukylin-archive-keyring.gpg works, though.16:30
infinityslangasek: So, perhaps a symlink in postinst from /etc/apt/trusted.gpg.d/ubuntukylin-archive-keyring.gpg to /usr/share, and skip the other bits?16:31
slangasekinfinity: ok.  Since you've looked at this and understand what's needed, do you want to reject, change, and bounce it back in my direction for counter-review, or do you want me to fix it up and resubmit?16:31
infinityslangasek: Sure, I can do that.16:31
slangasekinfinity: do you intend to generate the binary keyring at build time, OOI?16:39
infinityslangasek: We don't seem to for ubuntu-keyring.16:40
slangasekinfinity: yeah; I would argue that this is a bug in ubuntu-keyring too :)16:42
infinityslangasek: Perhaps, yeah.  But I'm not sure it's one I want to fix this instant.16:42
infinityslangasek: We can do both together some day.16:42
slangasekyep16:42
infinityslangasek: Not like an .asc is really any more readable than a .gpg, it's just more vi-friendly.16:42
slangasekjust makes it a little harder for me to review and make sure you're not MITMing me ;)16:42
cjwatsongrub2 won't be until early tomorrow morning I suspect, by the time I've built this, uploaded to Debian, and synced16:44
cjwatsonThe grub2 patch will be http://paste.ubuntu.com/7231546/ unless something goes horribly wrong for some reason16:47
cjwatsonI guess I could hack the .changes and upload the exact same thing to Ubuntu if people think I should16:48
cjwatsonWould be nicer to have the proper audit trail though16:48
infinitycjwatson: Letting the sync happen $later seems fine.16:49
slangasekI think it's unfortunate that the better we do at keeping things in sync with Debian, the slower it makes our updates in Ubuntu during critical times16:53
slangasekconsidering Debian was discussing reducing dinstall back down to daily, we should probably think about getting better syncing support from incoming16:53
cjwatsonYes, I agree on that16:54
infinityTo sync securely from incoming, all we need is access to the buildd repo.16:56
cjwatsonIncoming is signed these days, isn't it?16:57
infinityWe probably just have to ask nicely, and retool the LP sync stuff.16:57
infinitycjwatson: incoming isn't, but incoming/buildd is.16:57
cjwatsonOh, right16:57
cjwatsonDunno if it would need that much retooling, it could just be another suite?16:57
infinityslangasek: Okay, try that one on for size.16:58
cjwatsonI assume incoming/buildd is only private so that the world doesn't add it to their apt sources16:58
slangasekinfinity: I have a call now, it'll be a couple hours before I can look at it - is that ok?16:59
infinityslangasek: Doesn't bug me.16:59
infinitycjwatson: Yeah, pretty sure that was the reasoning.  And so we could unaccept if really, really necessary.17:00
infinitycjwatson: But I don't think the unaccept case matters for Ubuntu syncing, we'd just sync a newer version in that strange case.17:00
infinity(or revert manually, or whatever)17:00
cjwatsonRight, so auto-sync should still work off unstable, and we could shove it into Debian/incoming and people could use syncpackage -d incoming if they're in a rush17:01
infinitycjwatson: Yeah, that seems ideal to me.17:01
cjwatsongina will do its usual tracking of what's in what suites without much trouble AFAICS17:01
cjwatsonwgrant: ^- any qualms with the above?  otherwise I can go ask ftpmaster17:02
infinitycjwatson: Assuming we poll something not too unfriendly, I can't see that they'd mind.  I hope.17:03
cjwatsonI was thinking hourly17:04
infinitycjwatson: (And, really, buildds hit those Packages/Sources files constantly, one more buildd-like machine only hitting Sources.gz every 15-60m or something would be nothing)17:04
infinityThe queue runs every 15 or 20, doesn't it?17:04
infinityI always forget.17:04
cjwatsonAnd it'd be debmirror, but could be debmirror over HTTP if buildds don't get rsync17:04
cjwatsonNot sure17:04
infinitycjwatson: Well, we could check a cached Sources for freshness, then trigger a mirror, I assume.17:05
infinityThough, I imagine debmirror has such a facility anyway.17:05
cjwatsondebmirror fetches Packages/Sources and only grabs what it doesn't have17:05
* infinity nods.17:05
infinityAnd we only want Sources, so just that.17:05
cjwatsonEr right17:06
cjwatsonSo --arch=none17:06
infinityIf we could get a push trigger, that would be even better, but I won't push my luck.  A shortish poll would still be way ahead of our current situation.17:07
infinityAnd unless they've gone high tech since I ran a Debian buildd, if we want to do polling, it's literally just them generating a u/p pair for us in htaccess and letting us loose.17:08
cjwatsonCould just ask what's reasonable, indeed.17:08
infinityPossibly also IP limited, now that DSA demands buildds not be on sketchy cable modems. :P17:09
xnoxslangasek: JackYu: given the time constraints I have sponsored the package, with a few changes which are proposed for merging back into trunk at https://code.launchpad.net/~xnox/ubuntu-kylin-software-center/14.04-release/+merge/21525817:10
infinityAhh, phew.  "Good" news.  That testsuite regression on POWER8 isn't a regression.  Breaks in the old version too.17:11
JackYuxnox, got it, thanks.17:11
apwLaney, whoopsie-preferences as requested17:16
Laneyfanx17:16
JackYuxnox, since it is in the NEW queue already, should I request another sponsor to review and accept it?17:17
infinityJackYu: We'll get to it.17:18
JackYuinfinity, wow, great!17:19
Laneyapw: Soz, LP's having a little nap and now I've got to go17:29
LaneySomeone else'll probably get to it17:29
apwyep, np17:29
=== HoloIRCUser is now known as zhangchao
Laneyor me if I come back on later17:30
Laneysee you17:30
jamespageplease could the neutron upload for trusty be accepted - I'd quite like to cycle that through our test lab with the rest of the rc packages today17:37
infinityslangasek: In light of your previous verbal agreement on pulling the IBM branch, can you review the above? ^^18:54
infinityslangasek: It was tested in a PPA on all arches and then independently on POWER8 as well.18:54
slangasekinfinity: will queuediff eventually give me a diff for that or do I need to download the whole schleblottle?19:05
infinityslangasek: I see a diff in the queue...19:05
infinityslangasek: Missing -Q unapproved?19:06
slangasekinfinity: queuediff doesn't take a -Q19:08
slangasekinfinity: ok, diff has shown up now19:09
infinitycjwatson: Is https://bugs.launchpad.net/maas/+bug/1306164 relevant to partman as well?19:16
ubot2Launchpad bug 1306164 in MAAS "MAAS fast-path EFI install creates ESP that's too small" [Critical,Fix committed]19:16
rsalvetislangasek: ^ another simple one (qtlocation-opensource-src)19:17
slangasekrsalveti: ICMP redirect infinity; I have a few reviews on my plate already19:17
infinity;)19:17
rsalvetislangasek: sure np, just a heads up as this is related with the gl/gles topic19:19
tjaaltonslangasek: I'll reply to the mesa sru email in a bit, but another matter is that I pushed a change to pam adding pam-configs/mkhomedir, disabled by default. it's in bzr19:24
tjaaltonslangasek: it's not of much use for any automatic setup, since pam-auth-update doesn't know how to enable such configs19:26
tjaaltonuninteractively, there's a bug for that I'll try fixing soon19:26
cyphermoxcould any archive admin check on the archive machine where the cupstream2distro code runs, to see if it has received a file like "packagelist_rsync_landing-018-trusty" recently?19:30
cyphermoxci train publishes don't seem to be working atm19:30
cyphermoxrobru: ^19:30
cyphermoxI'm currently unable to provide a clearer indication of where this data is supposed to end up though :(19:31
robrucyphermox, the two logs showing publication are https://ci-train.ubuntu.com/job/landing-018-2-publish/7/console and https://ci-train.ubuntu.com/job/landing-018-2-publish/8/console19:33
robrubut that package hasn't shown up anywhere19:33
robrucyphermox, also https://ci-train.ubuntu.com/job/landing-019-2-publish/1/console19:33
robrubasically we're looking for phablet-tools and indicator-keyboard, they should have been published but got lost in the ether19:34
cjwatsoninfinity: people's usual complaint about partman's are that they're too large, so I don't think so19:34
cjwatsoninfinity: partman-auto has the minimum set to 512M19:35
cjwatsoninfinity: although there might be a decimal confusion there ...19:35
Laneycyphermox: I guess it's /home/ubuntu-archive/cu2d/incoming on snakefruit, which is empty19:35
infinitycjwatson: Oh, if it's already 512M in partman, then this curtin upload matches that.19:35
cjwatsoninfinity: I think we may be using 512MB rather than 512MiB though19:35
cjwatsoninfinity: but don't have time to investigate tonight19:36
infinitycjwatson: I think the changelog claimed 100, and I didn't go digging.19:36
cjwatsonSomebody should check whether we're at least 512MiB rather than just MB.  If nobody else does then I will tomorrow.19:37
cyphermoxLaney: thanks19:38
cyphermoxLaney: any idea if something appears to be stuck running there, or something?19:39
Laneycyphermox: only vaguely educated guess though ...19:39
cyphermoxLaney: it seems reasonable anyway19:39
Laneysec19:40
cyphermoxLaney: I'm trying to figure out what part is breaking that makes the packages not get really published, so if there's nothing in the incoming directory (seems the right one) on snakefruit, then maybe the jenkins instance is unsuccessful at pushing the rsync files19:41
cjwatsonThere's eight auto-accepts running, which is kinda odd19:41
cjwatsonBut unrelated19:41
cyphermoxaye19:41
Laneysnakefruit pulls the files19:42
cyphermoxLaney: could you check if maybe the crontab got deleted?19:42
cyphermoxassuming it's as I expect, a cron job running to do this19:42
Laneyit did not19:42
cjwatsonI think LP had a hiccup and a job got stuck19:42
robruLP certainly had a hiccup...19:42
cjwatsonJust checking, please excuse slow network19:42
Laneythere's an instance of it running19:42
cjwatson20814 17:20:01      \_ /usr/bin/python /home/ubuntu-archive/cu2d/cupstream2distro//copy2distro --no-filter19:42
cjwatsonelderly19:42
cyphermoxah19:42
cyphermoxcould well be that yeah19:43
cjwatsonI've killed it19:43
cyphermoxthanks19:43
robrucyphermox, so do we have to republish now?19:43
cyphermoxrobru: let's wait, I expect the next run to pick up the files... hopefully19:43
robrucyphermox, ok19:43
cjwatsonplease don't republish until we've traced19:43
cjwatsonunfortunately I'm uploading grub2 over wet string at the same time ...19:43
cyphermoxcjwatson: in the meantime I'm trying to check which packages we're expecting to see published by citrain that haven't made it yet19:44
robrucyphermox, it's phablet-tools and indicator-keyboard19:44
cjwatsonha, there, it hadn't pulled because an old job was stuck19:46
cyphermoxyup19:46
cjwatsonso kill old job -> next one runs and pulls19:46
cjwatsongood19:46
cyphermoxthanks!19:46
robrucjwatson, Laney, cyphermox: thanks19:47
cjwatsonLaney: sorry if I stepped on your toes, I think I took a while to remember that you had access and I was IRC-lagged ...19:48
cjwatson(and was half-expecting to have to check the package copy job logs, which are more restricted)19:48
LaneyNo worries, glad it was easy enough to sort out19:48
infinityslangasek: Unsure if thorough review or distracted vorlon.20:00
slangasekELUNCH20:00
slangasekor maybe that's SIGLUNCH20:00
slangasekback now though20:00
infinityELUNCH is so 1998.  It's iLunch now.20:00
infinityHAHAHA.  Such a shameless copy and paste that I forgot to change the name/.20:02
* infinity rejects and resends the u-d-a one.20:02
LaneyFix "fozen" too20:02
infinity<3 fozen.20:02
infinityI'm half asleep.20:02
infinityThere, slightly less typoey.  My turn for lunch.20:06
elfyinfinity: aah - good, so we're not releasing Saucy Salamander again then :)20:12
infinityelfy: We could try.20:13
wgrantcjwatson: That's probably fine.20:37
infinityslangasek: Oh, there's also the kylin-keyring there that I ping-ponged back to you.20:39
* infinity lunches harder.20:39
slangasekinfinity: yah, am reviewing it now; was just comparing to how other things set up shop under /etc/apt/trusted.gpg.d20:39
* slangasek gets very confused by the output of 'mk-build-deps' telling him this package has no build-deps20:40
infinityslangasek: I'm not sure if there's a precedent here.  There's apt-add-repository which adds keyrings directly to the directory, but that's cause it's only mangling config, not shipping packaged files.20:40
infinityslangasek: Shipping in /usr/share and having a non-dpkg-owned symlink felt right to me, since deleting the symlink should be preservable configuration IMO.20:41
slangasekinfinity: debian-archive-keyring installs files directly under /etc, but I don't /like/ that precedent20:41
slangasekinfinity: your maintainer scripts aren't set -e20:42
slangasekwhat happens if ln -s fails!20:42
slangasek(accepting anyway; you can fix it in postproduction :P)20:43
slangasekinfinity: if you happen to have a chance to throw a bzr mp at lp:~ubuntukylin-members/ubuntukylin/ubuntukylin-keyring, I'm sure JackYu and maclin would appreciate it20:43
infinityWell, now I hope it does fail, just to spite you.20:43
infinityslangasek: If by "MP", you mean "commit directly, because core-dev obviously has rights", sure!20:44
slangasekinfinity: or that20:44
infinityHey neat, did someone just upload firefox in my name? :P20:45
* infinity was holding off on that to batch up some porting fixes, but... Whatever.20:45
robruany chance we could get unity, indicator-session, and webbrowser-app in before the freeze? they're all bugfixes20:46
robru(published in citrain earlier today)20:46
* infinity really wishes the queue told me who signed uploads, it's disconcerting to see uploads from me that arent.20:47
* slangasek blinks at the linux package in the queue20:54
infinityslangasek: You said you wanted half the base system.20:55
infinityslangasek: We live to give.20:55
=== infinity changed the topic of #ubuntu-release to: Released: Trusty Final Beta | Archive: Final Freeze | Trusty Tahr Release Coordination. Please don't upload things during freezes where you shouldn't, or be prepared to apologise to the release team | we accept payment in cash, check or beer | melior malum quod cognoscis
cyphermoxinfinity: I ship you beer to which address re: this 11-second-late indicator-bluetooth upload? :)21:01
infinityLooks like you might owe slangasek a beer.21:02
cyphermoxindeed21:02
stgrabercyphermox: you owe me a beer now, I was the one ignoring the time and clicking accept ;)21:03
slangasekyeah, wasn't me21:04
cyphermoxstgraber: yeah, but I was obvious telling you about it when it showed up here21:04
cyphermoxstgraber: next lunch at McKibbins ;)21:04
stgrabersure :)21:04
bdmurrayslangasek: shouldn't we disable apport sending crash reports to launchpad now?21:19
stgraber20 minutes ago would have been better, but yes, we should21:19
bdmurrayI'm sorry I was slow thinking about someone else's thing21:20
bdmurrayAnyway, shall I upload it?21:21
NoskcajIs https://code.launchpad.net/~noskcaj/ubuntu/trusty/xfburn/bugfixes/+merge/215094 safe to upload? it's a new bugfix release for xfburn21:22
bdmurrayuploaded apport21:29
stgraberNoskcaj: that package is seeded and we are in final freeze, so my guess would be, no21:30
Noskcajok21:31
slangasekbdmurray: hmm, did we never get to the point where we keep whoopsie enabled throughout the cycle but keep launchpad disabled by default throughout the cycle?21:31
slangasekthis is probably one of those questions I ask every 6 months and then forget the answer21:32
slangasekso what's the story with the linux in the queue?  Is that for the first SRU?21:32
bdmurrayslangasek: there is still a fair bit of development that needs to happen on errors for that to be the case21:32
slangasekbdmurray: ok - sounds like something we should talk about immediately after opening unctuous umbrellastand21:33
gilirany chances to accept lxsession + lubuntu-default-settings ? They fix some annoying bugs on the lubuntu ISO21:34
stgrabergilir: so lubuntu-default-settings seems to introduce a new package, yet I don't see anything depending/recommending it, what's the purpose of this?21:36
barryuvular uakari21:36
gilirstgraber, it's a new binary to add only on the ISO, it should be only on the seed21:37
gilirstgraber, it only adds a .desktop to disable light-locker on the ISO21:37
stgraberok, accepted that one and will binNEW the package to universe when it's built21:42
infinityslangasek: No, that's for the archive.21:43
infinityslangasek: Just not bothering to accept until all the binaries roll in.21:43
stgrabergilir: lxsession is realistically impossible to review because for a bugfix release it's pretty massive. I assume you tested it well before upload?21:44
gilirstgraber, thanks21:44
gilirstgraber, yes, sorry about this, I tested it during last days and it's working fine21:45
RiddellKDE incoming!21:46
stgrabergilir: ok, the .vala seems vaguely sane, so I'll accept it21:47
gilirstgraber, if it's easier, you can check the git tree directly, it's the last 7 commits : http://git.lxde.org/gitweb/?p=lxde/lxsession.git;a=shortlog21:47
stgrabernah, the diff is fine once you ignore all the generated .c files21:48
stgraberRiddell: you're aware you're past final freeze right? :)21:48
stgraberRiddell: you'll owe us at least one beer per package21:48
Riddellstgraber: too late, kde4libs was in before freeze and so the rest must follow21:48
gilirstgraber, ok, thanks :-)21:48
slangasekgilir: while we're at it, there's a lubuntu-artwork in the queue; I don't understand the changelog entry, there are no bugs listed, and we're at the deadline - what should be done with this?21:49
slangasekRiddell: er, how does it follow that they must be accepted because kde4libs was?21:49
slangasekinfinity: ok, *why* is there a new kernel in the queue, post-kernel-freeze?21:50
slangasekinfinity: since when are we playing fast and loose with the kernel freeze deadlines?21:50
infinityslangasek: You'd have to ask the kernel team why they felt it was appropriate, but it seemed sane enough (well, sane as can be) when I reviewed it, at least.21:51
gilirslangasek, it's not critical for the release, just moving a directory in the same binary package (from 1 theme to another), it can be SRUed if you prefer21:52
slangasekinfinity: but the kernel freeze doesn't just exist for the benefit of the kernel team, revving the kernel also impacts the rest of the team during a critical window (because we have to futz with d-i etc)21:52
infinityslangasek: we have to upload d-i anyway, to be fair.21:52
slangasekgilir: I don't see how "moving a directory in the package" qualifies under either the SRU or freeze guidelines, maybe you could explain this in terms of user-facing impact?21:53
slangasekinfinity: oh, do we?  For grub, or?21:53
infinityslangasek: eglibc, grub, flash-kernel, kmod...21:53
slangasekinfinity: heh, ok.21:53
infinityslangasek: There've been a fair few udebs changed since the last build.21:53
infinityudev...21:54
stgraberroaksoax: hey, so you know we're past final freeze right?22:01
gilirslangasek, basically, metacity theme is currently in an inherited theme, not the main artwork theme. This commit moves the metacity theme to the main artwork theme, so both themes have the settings available (main and inherited). It's not critical because we don't use metacity by default on lubuntu.22:01
stgraberseems to me kinda late to land new maas features22:02
slangasekstgraber: there was an FFe request before final freeze, with discussion that took until now to converge on an upload22:03
stgraberslangasek: ok22:03
slangasekgilir: ok, so if "it's not critical" I would say it's best to reject this and leave it for 14.10 entirely, unless you want to argue otherwise?22:04
stgraberslangasek: hmm, the FFe was never acked22:04
stgraberslangasek: bug 130583922:04
ubot2Launchpad bug 1305839 in maas (Ubuntu) "FFe: Support for Third Party Driver Installation" [Critical,New] https://launchpad.net/bugs/130583922:04
gilirslangasek, no, I'm fine with rejecting it22:04
slangasekgilir: ok, rejecting - thanks22:05
slangasekstgraber: not sure if the FFe ack happened outside the bug, but infinity was involved in the discussion22:06
slangasek(in fact, he was the one driving the changes required)22:06
stgraberslangasek: ok, I'll let infinity deal with the upload then22:08
jamespageis anyone in the release team looking at the neutron upload I put in the queue ~8 hrs ago?22:26
xnoxlooks like kubuntu will be bug-free rock-solid by thursday =)22:30
slangasekjamespage: I'm looking at it now23:03
slangasekxnox: are you cherry-picking bug #1303891 in?23:03
ubot2Launchpad bug 1303891 in upstart "new initctl should fallback to old reload signal semantics, if pid1 doesn't export reload dbus method" [High,Triaged] https://launchpad.net/bugs/130389123:03
xnoxslangasek: yes, had a volleyball match, now back to complete upstart upload.23:21
slangasekxnox: no spiking the packages over the queue23:22
xnox=))))))))))))))))))))))))23:23
xnoxslangasek: i dig what you did there.23:23
slangasekjamespage: looking at this neutron upload, I'm concerned you may have some missing Replaces between neutron-common and neutron-*-agent; conffiles have moved between the packages, but neutron-common doesn't Replace the previous owners23:24
slangasekjamespage: I've reuploaded neutron with proper breaks/replaces, I think I'd like someone else to review it now before it goes in to make sure I didn't screw anything up23:38
roaksoaxs23:40
roaksoaxslangasek: are we good to accept maas too?23:41
slangasektjaalton: yes, I saw your mkhomedir commit, launchpad told me ;)  and yeah, there's no noninteractive way to enable profiles, that's been a longtime request and something it would be great to see fixed23:41
slangasekroaksoax: I think we were going to let infinity review maas since he's already familiar with the pending changes23:42
slangasekstgraber: if you're around and can review the neutron reupload, I'd appreciate it23:42
xnoxslangasek: i'm treating all uploads from now on as SRUs, thus requiring bug # references and sru template / test-case in good standing. E.g. would help in case an upload doesn't make into "release" but gets re-diverted to be an SRU.23:44
xnoxinfinity: "final release of Saucy Salamander in a week." looks like it's not just timezone challenged =)))))23:45
slangasekxnox: that will certainly make the release team's job easier when reviewing23:46
JackYuhi, release team, good morning:)23:57
xnoxJackYu: i still didn't sleep yet =)23:58
JackYuxnox, I see. You are very busy today. Hope you have a nice weekend:)23:59

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