| lool | Not clear to me why upstart-app-launch is listed here | 00:22 |
|---|---|---|
| lool | would someone be able to review? | 00:22 |
| lool | I mean, I'm surprized it's help up | 00:24 |
| lool | ah via liburl-dispatcher1 | 00:24 |
| lool | via a Suggests?! | 00:24 |
| lool | gah | 00:24 |
| cjwatson | click is in Ubuntu desktop supported too | 00:24 |
| cjwatson | Suggests are ignored | 00:24 |
| lool | ah right click | 00:25 |
| === plars_ is now known as plars | ||
| pitti | ^ for the upower sync, in the queue it's quite hard to review | 04:32 |
| pitti | we currently have a git snapshot, and the effective delta is just two commits plus some test suite updates | 04:32 |
| pitti | http://cgit.freedesktop.org/upower/commit/?id=db89e5a3 | 04:32 |
| pitti | http://cgit.freedesktop.org/upower/commit/?id=ecc4e37 | 04:33 |
| pitti | and we really really need those two fixes, they break a lot in GNOME | 04:33 |
| pitti | (that is, the bug that they fix :) ) | 04:33 |
| pitti | oh, and http://cgit.freedesktop.org/upower/commit/?id=03c4ab | 04:33 |
| pitti | cheers :) | 05:26 |
| === rsalveti_ is now known as rsalveti | ||
| infinity | slangasek: ^-- If you're still around, care to give a once-over to those? New upstream version, but only a handful of bugfixes. The git log is only a few commits. | 07:02 |
| dholbach | hiya, can somebody please take a look at bug 1235737? seems like another bug (already approved) in the sponsoring queue depends on it | 08:37 |
| ubot2` | Launchpad bug 1235737 in postbooks (Ubuntu) "FFe: Sync postbooks 4.1.0-2 (universe) from Debian sid (main)" [Undecided,New] https://launchpad.net/bugs/1235737 | 08:37 |
| infinity | cjwatson: Since you're awake now, want to poke at those gdb/gdb-doc uploads? bugfix-only, and rather minimal. | 08:49 |
| cjwatson | infinity: Was just starting on them, indeed | 08:50 |
| infinity | cjwatson: Sorry about the lazy copy/paste changelog from upstream, but, well. Lazy. I did test it, though. :P | 08:51 |
| cjwatson | Yeah, no problem | 08:53 |
| Laney | dholbach: sync away | 08:55 |
| Laney | I'll subscribe u-sponsors to some other ones | 08:56 |
| Laney | there's a little stack of postbooks things | 08:56 |
| dholbach | thanks Laney | 08:57 |
| ogra_ | infinity, hmm, these touch kernels went through completely untested | 09:04 |
| infinity | ogra_: Who was meant to test them and how? | 09:05 |
| apw | ogra_, they weren't untested, all of yesterday was spent testing them | 09:05 |
| ogra_ | (they were not supposed to be uploaded yet) | 09:05 |
| ogra_ | apw, well, touch images have no landings without the landing team nodding it off | 09:05 |
| ogra_ | infinity, i was supposed to test them with the autopilot tests before they get uploaded | 09:06 |
| apw | the rest of the bits of the landing landed already yesterday | 09:06 |
| ogra_ | apw, which AP tests did you run ? do you have a test matrix or something, did all tests pass ? | 09:06 |
| apw | ogra_, not me | 09:07 |
| apw | ogra_, i waas more pointing out the process is opaque and confusing to those of us using it | 09:07 |
| ogra_ | the ychanges touch click package behavior, apps might misbehave | 09:07 |
| apw | they are presumably stuck in -proposed anyhow by your britney block | 09:08 |
| ogra_ | they just went into the archive | 09:08 |
| ogra_ | there is no britney lock for universe packages | 09:08 |
| cjwatson | apw: It would if the touch landing team bothered to use the facilities I'd set up for them in any breadth | 09:08 |
| infinity | ogra_: Touch people have the power to block their packages. | 09:08 |
| ogra_ | infinity, which doesnt help us, since android (which consumes the kernel) builds from proposed | 09:09 |
| infinity | ogra_: Anyhow, if you're deeply concerned, test away. But these changes went into the master kernel yesterday, and the userspace side is all landed. | 09:09 |
| ogra_ | infinity, point is that per policy they would have gone through the spreadsheet stuff, which they didnt, without approval on that sheet nothing has to be uploaded | 09:10 |
| cjwatson | Point is that you lose. | 09:10 |
| cjwatson | I mean, it's inevitable with this process. | 09:10 |
| cjwatson | Or lack thereof. | 09:10 |
| ogra_ | well we'll change the process to something more sane after release | 09:11 |
| cjwatson | Pre-upload gating while choosing not to use the technical means available is infeasible. | 09:11 |
| infinity | ogra_: I've uploaded a lot of stuff that's on touch images without the spreadsheet. Why didn't you yell at me for glibc? :P | 09:11 |
| ogra_ | cjwatson, the technical means available will be used after release, i want that as much as you ... but currently all uploads are held for the Mir switch on touch ... if we need to fix a bug on the android side for this we cant rebuild it without pulling in the kernels now | 09:12 |
| ogra_ | infinity, i'm not yelling | 09:13 |
| cjwatson | Well I don't know what you propose to do about it now. | 09:13 |
| cjwatson | (And I will say: the arm64 porting work is not going to stop for this.) | 09:13 |
| ogra_ | nothing just pointing it out so it doesnt happen again | 09:13 |
| cjwatson | (I know that's not about the kernel, but it is about other uploads.) | 09:13 |
| infinity | ogra_: This was a change that was wanted. The userspace side landed. The kernel needed to. The idea of gating things one upload per image is insane. Full stop. | 09:14 |
| infinity | ogra_: But dropping process arguments, the kernels are in. Oh well. | 09:14 |
| ogra_ | infinity, discuss that with asac and lool ... i'm just pointing out something so it doesnt happen again | 09:15 |
| xnox | ogra_: security uploads trumps anything. | 09:15 |
| cjwatson | It's likely to happen again. | 09:15 |
| infinity | And again, and again. | 09:15 |
| xnox | ogra_: and it was security upload. | 09:15 |
| cjwatson | doko_: Curious. No dh_autotools-dev_{update,restore}config in libsigc++-2.0? | 09:15 |
| ogra_ | xnox, ?? | 09:15 |
| cjwatson | doko_: And no clean handling | 09:16 |
| ogra_ | xnox, i'm talking about the fout touch kernels that were uploaded to saucy | 09:16 |
| ogra_ | *four | 09:16 |
| xnox | ogra_: so do I. | 09:16 |
| doko_ | cjwatson, clean handling is there | 09:16 |
| ogra_ | and they were supposed to be held back | 09:16 |
| doko_ | using the existing one | 09:16 |
| ogra_ | until after Mir landed | 09:16 |
| cjwatson | doko_: Oh, right, generic code | 09:17 |
| doko_ | cjwatson, didn't want to interfer with the manual autoreconf in this package | 09:17 |
| lool | cjwatson: Problem here is that the kernels get copied into android; so we couldn't upload android anymore as soon as the kernels were uploaded to proposed (hints dont help this case) | 09:17 |
| xnox | ogra_: have you seen the bug they are fixing together with the matching apparmor uploads? (ps it's 6 kernels, goldfish and normal as well) see bug #1208988 | 09:17 |
| ubot2` | Launchpad bug 1208988 in AppArmor "AppArmor no longer mediates access to path-based AF_UNIX socket files" [High,Triaged] https://launchpad.net/bugs/1208988 | 09:17 |
| cjwatson | Also: if these were supposed to be held back, why did nobody tell the release team? | 09:17 |
| lool | so completely not related to hints | 09:17 |
| infinity | lool: Or, you could upload android and get the new kernels. This is bad, why? | 09:17 |
| ogra_ | xnox, no, and thats not what this is about ... | 09:17 |
| cjwatson | You can't not communicate about specific things that were supposed to be held back and then expect that to happen. | 09:17 |
| lool | infinity: I might have to upload android and not want the new kernels | 09:17 |
| cjwatson | Since pre-upload gating is infeasible. | 09:17 |
| lool | infinity: but I can't technically stop that | 09:17 |
| lool | and we had agreed to coordinate the uploads later today | 09:18 |
| lool | but well | 09:18 |
| lool | I guess there was a hiccup in this part | 09:18 |
| ogra_ | cjwatson, i just did ... for the next time :) | 09:18 |
| infinity | lool: I understand that you might want that, I don't understand WHY you want that. This apparmor change is, AFAIK, necessary. | 09:18 |
| lool | cjwatson: I didn't expect anything to be held back | 09:18 |
| infinity | lool: It's bizarre to slow yourself down by only testing one tiny thing at a time. | 09:18 |
| xnox | ogra_: lool: apparmor changes went in (security bug) and if kernels didn't go in, your userspace and containment would be borked. Taking one without the other, means borked images. | 09:18 |
| lool | infinity: it is, but there are other changes (ureadahead for instance) in the kernels, the toolchain changed, we need to test the kernels and we might have to land android to fix tests before we take new kernels | 09:19 |
| lool | xnox: No, we actually confirmed they could go in in this order | 09:19 |
| lool | xnox: and separately | 09:19 |
| xnox | ogra_: lool: and android package makes it's own sources where it downloads the kernels from, so you can easily change it to pull kernels from -release pocket if you wish to do so. And then hints would gate keep it. | 09:19 |
| lool | xnox: yeah, I can workaround this specific case | 09:19 |
| infinity | lool: Erm, the ureadahead commit already existed in 2/4 kernels. You intentionally want them to behave differently? | 09:19 |
| infinity | lool: And that's literally the only change other than the apparmor stuff. | 09:20 |
| lool | infinity: and being rebuilt | 09:20 |
| xnox | ogra_: lool: but multiple times, i've been requested to pull from -proposed to speed up image respins with updates (e.g. hybris, initramfs-tool-ubuntu-touch, etc) | 09:20 |
| lool | it's not like toolchains broke kernels in the past :-) | 09:20 |
| lool | xnox, infinity, cjwatson: I am personally fine with what just happened | 09:20 |
| lool | I would like us to use hints more next cycle | 09:20 |
| infinity | Then why are we talking about it? | 09:20 |
| lool | they wouldn't help in this case | 09:20 |
| xnox | lool: touch kernels are build with toolchain that doesn't change any more (4.6 and 4.7) | 09:20 |
| lool | infinity: exactly :-) | 09:20 |
| apw | right the toolchain changes do not affect these kernels at all, because the kernle doens't work if you compile it with those; which of course is an epic recommendation for the compiler you are building the rest of the stuff with | 09:22 |
| xnox | apw: =))))) | 09:23 |
| lool | also, I think we actually copied the kernel binaries | 09:23 |
| lool | which got tested | 09:23 |
| lool | apw: do you know if cking tested on Mir too? | 09:24 |
| infinity | I did copy with binaries, yes. | 09:24 |
| cking | lool, I tested with Mir enabled on maguro + manta | 09:24 |
| lool | cking: awesome | 09:25 |
| lool | infinity: great | 09:25 |
| * seb128 scratches head | 09:27 | |
| seb128 | why did I receive an "[ubuntu/saucy] gtk+3.0 3.8.4-0ubuntu3 (Accepted)" email this night when that version as already in saucy for a while | 09:27 |
| seb128 | e.g https://launchpad.net/ubuntu/+source/gtk+3.0/+publishinghistory | 09:27 |
| cjwatson | seb128: Probably arm64 port -> new copy | 09:28 |
| cjwatson | Known to produce somewhat confusing mails when proposed-migration copies the new binary into the release pocket | 09:29 |
| === doko_ is now known as doko | ||
| xnox | cjwatson: maybe it's useful to send an email to ubuntu-devel about the emails? cause pretty much every uploader of everything that gets build on the new port will receive these confusing emails? | 09:30 |
| cjwatson | Actually no that's not true | 09:31 |
| cjwatson | The bulk of builds are happening in the release pocket (because reasons) | 09:31 |
| cjwatson | The only cases where you get confusing mails are uploads to -proposed after the port was initialised | 09:31 |
| * xnox has been getting a lot of emails already and FTBFS emails as well, wrt arm64. | 09:31 | |
| cjwatson | Yeah, I realise that but that's tied up with details of hardware and I'm not sure what I can say publicly on that | 09:32 |
| lool | did you just say arm64? | 09:32 |
| cjwatson | I don't want to accidentally slag off a partner on a public mailing list. :-P | 09:33 |
| xnox | ok. | 09:33 |
| seb128 | cjwatson, ok, thanks | 09:33 |
| cjwatson | (And hopefully the major cause of odd build failures will be fixed soonish ...) | 09:33 |
| cjwatson | lool: Yeah. :-) | 09:34 |
| === exekias_ is now known as exekias | ||
| lool | hud in unapproved has notably a workaround for when upstart mis-sets the dbus env vars in user sessions (this is apparently an upstart bug with set-env -g that we're still looking into); hud crashing is causing lots of test failure noise on touch, so greatly appreciated if we can land this :-) | 11:27 |
| lool | there's a complex hud change coming up in the next hours/days to address deeper problems with it, but this one should help quite a bit already | 11:27 |
| Laney | Is a binary-only promotion still OK to do? | 11:34 |
| cjwatson | Should be | 11:34 |
| Laney | I want to make sessioninstaller dep on python-commandnotfound for bug #1102434 | 11:34 |
| ubot2` | Launchpad bug 1102434 in gnome-control-center (Ubuntu) ""System settings -> Colour", fails due to missing "gnome-color-manager" package." [Low,Triaged] https://launchpad.net/bugs/1102434 | 11:34 |
| Laney | ta | 11:34 |
| seb128 | Laney, did you get the apturl fix tested/uploaded? | 11:34 |
| cjwatson | No Python 3? :-( | 11:34 |
| Laney | seb128: yeah | 11:36 |
| seb128 | Laney, great ;-) | 11:36 |
| Laney | cjwatson: apparently not | 11:36 |
| apw | the linux and linux-signed uploads are to fix the arm64 linux-libc-dev (and other cross environments) builds | 12:10 |
| Laney | lool: where is the workaround? | 12:13 |
| xnox | Laney: in user-session's init hud.conf.in | 12:20 |
| Laney | xnox: Maybe I have the wrong diff | 12:20 |
| lool | Laney: the hud thing? | 12:21 |
| lool | Laney: in unapproved | 12:21 |
| Laney | I don't believe that change is in the upload | 12:21 |
| xnox | lool: Laney: well, we want http://bazaar.launchpad.net/~indicator-applet-developers/hud/trunk.13.10/revision/326 which doesn't seem to be included in the unapproved upload. | 12:22 |
| xnox | (sync) | 12:22 |
| lool | Laney: checking | 12:22 |
| Laney | queue -Q unapproved -s saucy-proposed fetch hud | 12:22 |
| lool | Laney: you're right, it's not there | 12:22 |
| Laney | :-) | 12:22 |
| Laney | want to re-upload? | 12:22 |
| lool | Laney: yes | 12:23 |
| * Laney rejects | 12:23 | |
| lool | need to figure out by cu2d didn't pick up though | 12:23 |
| lool | Laney: thanks | 12:23 |
| jdstrand | xnox: fyi, regarding borked images because userspace was updated but the kernels weren't-- that actually wasn't the case this time. the userspace could go in before or at the same time. it was just the kernels couldn't go in before | 12:35 |
| jdstrand | ah, I see someone mentioned that already | 12:36 |
| xnox | jdstrand: right the argument was that whilst apparmor can have block and not migrate to release pocket, android package is built with pulling kernels from -proposed (avoiding all blocks) | 12:37 |
| xnox | jdstrand: thus when everything is in -proposed, one had to take apparmor or otherwise be stuck with updated kernel & out of date apparmor. | 12:38 |
| jdstrand | sure. I am not going to participate in that conversation except to say that while I understand the desire and utility to gate the touch images, the process needs improvement and I'm glad that it is on the agenda for the next sprint | 12:39 |
| ogra_ | xnox, that hud fix will not have your filtering i guess ... can we have that too ? | 12:39 |
| xnox | jdstrand: ack. | 12:39 |
| xnox | ogra_: upstart upload? | 12:39 |
| ogra_ | xnox, for the dbus bridge filtering | 12:40 |
| ogra_ | wasnt that upstart ? | 12:40 |
| xnox | ogra_: there was no dbus bridge filtering, i had a patch for upstart-udev-bridge to drop that one event and not generate any upstart events for it. is that what you mean? | 12:40 |
| * ogra_ checks the bug again | 12:41 | |
| ogra_ | xnox, oh, righ t | 12:42 |
| ogra_ | thats what i mean | 12:42 |
| xnox | ogra_: ok. jodh doesn't want it upstream, but it can be uploaded into the ubuntu package. | 12:44 |
| ogra_ | i think that would make sense | 12:45 |
| ogra_ | (so would having the udev side fixed too i guess) | 12:45 |
| xnox | ogra_: it is unknown if mir needs the udev event or not. | 12:46 |
| xnox | ogra_: as far as I understood yesterday's discussions. | 12:46 |
| xnox | ogra_: but i guess that can be tested. | 12:46 |
| ogra_ | yeah :) | 12:46 |
| xnox | ogra_: since kernel was changed to emit them.... | 12:47 |
| ogra_ | well, it is only emitted on maguro | 12:47 |
| ogra_ | mako runs without it | 12:47 |
| ogra_ | so i dont think we need it outside of the kernel | 12:47 |
| xnox | ogra_: yes, but propbably maguro's binary blobs want it. | 12:47 |
| ogra_ | right | 12:47 |
| lool | Laney: ^ | 13:04 |
| === vila is now known as vila-laaaate-lun | ||
| xnox | ogra_: if you want about upstart upload, please make it go through all the right processes. | 13:05 |
| xnox | ogra_: as you wish =) | 13:05 |
| zul | can i get a release team meber to ack https://bugs.launchpad.net/ubuntu/+source/python-cinderclient/+bug/1236901 | 13:10 |
| ubot2` | Launchpad bug 1236901 in python-cinderclient (Ubuntu) "FFE for python-cinderclient 1.0.6" [High,New] | 13:10 |
| === vila-laaaate-lun is now known as vila-late-lunch | ||
| lool | Laney: Still around for the hud thing? :-) | 13:33 |
| === vila-late-lunch is now known as vila | ||
| Laney | lool: back now, was lunching | 13:35 |
| === arges_ is now known as arges | ||
| Laney | that workaround :( | 13:46 |
| ogra_ | Laney, well, look at the original start process of thge HUD | 13:49 |
| ogra_ | Laney, it fits well :P | 13:49 |
| Laney | I like my eyes unbled :P | 13:49 |
| seb128 | Laney, did you accept it? | 13:50 |
| Laney | ya | 13:50 |
| seb128 | that seems buggy | 13:50 |
| seb128 | http://launchpadlibrarian.net/153097115/hud_13.10.1%2B13.10.20131002.1-0ubuntu1_13.10.1%2B13.10.20131009-0ubuntu1.diff.gz | 13:50 |
| ogra_ | (a dbus service, that calls a shellscript (with -hack- in its name actually) to trigger upstart to start an upsart job | 13:50 |
| ogra_ | ) | 13:50 |
| seb128 | what is that apport hack | 13:50 |
| seb128 | it's not even documented in the changelog | 13:50 |
| seb128 | the printf line | 13:50 |
| seb128 | (dropping the author as well seems weird) | 13:51 |
| Laney | I divined that they want to report when this situation happens | 13:51 |
| Laney | "and reporting a bug" | 13:52 |
| ogra_ | looks like debug logging | 13:52 |
| seb128 | that should be documented in the changelog... | 13:52 |
| seb128 | shrug | 13:52 |
| ogra_ | well, pete-woods is in -touch | 13:52 |
| seb128 | it's coming from ted | 13:53 |
| seb128 | http://launchpadlibrarian.net/153097115/hud_13.10.1%2B13.10.20131002.1-0ubuntu1_13.10.1%2B13.10.20131009-0ubuntu1.diff.gz | 13:53 |
| ogra_ | i think they cross review their MPS | 13:53 |
| seb128 | oh well, hacks on hacks | 13:53 |
| ogra_ | *MPs | 13:53 |
| * seb128 uninstall huds | 13:53 | |
| ogra_ | yeah | 13:53 |
| Laney | I thought about arguing that they should be using normal dbus activation instead of this weird stuff | 13:54 |
| Laney | but then I decided to stay away from the battlefield | 13:55 |
| ogra_ | http://paste.ubuntu.com/6213176/ | 13:56 |
| ogra_ | thats the current hud startup process | 13:56 |
| ogra_ | so the patch fits in well :P | 13:56 |
| ogra_ | its all about keeping the style right :) | 13:57 |
| seb128 | hehe | 14:01 |
| xnox | ogra_: well at least i patched it to use "sh" instead of "bash" =/ | 14:07 |
| ogra_ | heh yeah | 14:07 |
| ogra_ | that will buy us precious bootspeed | 14:07 |
| ogra_ | every nanosecond counts | 14:07 |
| * ogra_ would like ot get back under 30sec | 14:08 | |
| === barry` is now known as barry | ||
| jodh | ogra_, jibel: could you try https://bugs.launchpad.net/ubuntu/+source/unity/+bug/1235649/comments/44 ? | 14:22 |
| ubot2` | Launchpad bug 1235649 in upstart (Ubuntu Saucy) "uevent spam causes session upstart to consume massive amounts of memory on Ubuntu Touch" [Critical,Confirmed] | 14:22 |
| ogra_ | jodh, yes (see #ubuntu-touch) will do soon | 14:22 |
| jodh | ogra_: thanks | 14:22 |
| ogra_ | jodh, i fear it wont help on maguro with the uevent spam though | 14:23 |
| jodh | ogra_: right. There are a few problems going on here :) | 14:23 |
| ogra_ | yeah | 14:23 |
| ogra_ | well xnox fix works pretty well | 14:23 |
| ogra_ | apart from the fact that udevd consumes a bit more CPU afterwards | 14:24 |
| jibel | jodh, what do I need to do? patch Panel.qml and Shell.qml and verify if I can reproduce? | 14:24 |
| ogra_ | jibel, yeah, see if the session init ram consumption raises still ... on mako | 14:25 |
| jibel | ogra_, okay, doing that now | 14:28 |
| Laney | Probably → #-touch :-) | 14:29 |
| Laney | stgraber: queuebot went away | 14:30 |
| ogra_ | having lunch too :) | 14:30 |
| doko | please could somebody review python3.3 and openjdk-7 ? just needed for the aarch64 bootstrap | 14:34 |
| cjwatson | python3.3 accepted, waiting for diff for openjdk-7 | 14:37 |
| cjwatson | oh, it's there | 14:38 |
| cjwatson | really must apiify diffs | 14:38 |
| * ogra_ hugs xnox | 14:57 | |
| ogra_ | YAY !!!! | 14:57 |
| xnox | ogra_: can I have MIR on grouper please? =) | 14:59 |
| ogra_ | xnox, so be it ... | 14:59 |
| ogra_ | xnox, upgrade to 89 | 14:59 |
| ogra_ | ;) | 14:59 |
| xnox | ogra_: i do system image, is 89 published in devel-proposed channel? or which channel do I want? | 14:59 |
| ogra_ | saucy-proposed, yeah | 15:00 |
| ogra_ | (or devel-proposed) | 15:00 |
| rsalveti | ricmm_: ^ once it gets approved | 15:14 |
| ricmm_ | rsalveti: great | 15:16 |
| lool | (would be nice to get the libhybris upload reviewed; just adds a header) | 16:05 |
| stgraber | queued cleaned up | 16:11 |
| seb128 | stgraber, thanks! | 16:13 |
| Laney | excellent | 16:13 |
| Laney | I loaded the ubuntukylin-default-settings diff and had to go off to mop up my tears | 16:14 |
| stgraber | ;) | 16:14 |
| Laney | not the change itself, but the context in the diff | 16:15 |
| Laney | cp /some/logo /the/ubuntu/logo/in/g-c-c | 16:15 |
| * seb128 pokes Laney in the eyes | 16:15 | |
| seb128 | or maybe not | 16:15 |
| Laney | then I get to have it read to me by a screen reader | 16:15 |
| Laney | that seems worse :( | 16:16 |
| seb128 | Laney, yeah, sorry about that :/ | 16:16 |
| stgraber | right, I accepted the change because it wasn't making things any worse, actually it made things slightly better, but the context looked pretty scary indeed | 16:16 |
| Laney | indeed | 16:16 |
| seb128 | but yeah, the kylin setting has quite some hacks | 16:16 |
| ypwong | Hi all, at what time will saucy be frozen? | 16:16 |
| happyaron | and this wallpaper package needs approval, for ubiquity to recognize the new wallpaper | 16:25 |
| happyaron | I think cjwatson did not modified the settings in ubiquity so a link must be created for it here. | 16:25 |
| ypwong | well, I meant about what time the final freeze will be in effect | 16:26 |
| happyaron | Laney: well, yes, I tried to revert as many changes as possible in previous upload, if you looked at that version, you'll be scared even more... | 16:26 |
| Laney | thanks for your efforts :-) | 16:27 |
| Laney | ypwong: usually 2100 UTC | 16:27 |
| ypwong | Laney, thx | 16:27 |
| happyaron | oh, gosh, I need to upload another version of -default-settings, it ftbfs... | 16:27 |
| ypwong | happyaron, the patch in the bug doesn't work? | 16:28 |
| happyaron | it works, but removing the unity.mo make it fail | 16:29 |
| happyaron | sorry everyone I'm in a big rush cuz I'll have no network in 2min... | 16:29 |
| dobey | is there any way to tell who accepted a particular upload to saucy? | 17:21 |
| slangasek | dobey: no | 17:41 |
| slangasek | this causes great consternation for the team | 17:41 |
| dobey | :( | 17:42 |
| dobey | slangasek: can i convince you to accept the 3 uploads of mine that you rejected last night? | 17:43 |
| slangasek | dobey: well, you can try, but I don't see why I would be persuaded that we should do no-change rebuilds of packages just to get 13.10 as the version number | 17:44 |
| dobey | slangasek: because we want to have versioning consistency across the ubuntuone projects/packages. we have been doing this for several cycles now. | 17:45 |
| slangasek | with freeze exceptions? | 17:45 |
| dobey | freeze exceptions? | 17:46 |
| slangasek | we're in post-beta freeze | 17:46 |
| slangasek | and the RC is tomorrow | 17:46 |
| slangasek | and package version number consistency is very low on my list of priorities | 17:46 |
| dobey | well, the other 6 packages i uploaded were accepted without problem, with only a version number bump. | 17:48 |
| stgraber | haha, looks like we've got a conflict here :) | 18:01 |
| stgraber | I'll reject them both and re-upload one with both changes | 18:01 |
| slangasek | dobey: right, I guess that's why you want to know who accepted a particular package? But I presume those other packages were uploaded earlier | 18:05 |
| slangasek | I guess if you can figure out who did the other accepts and talk them into un-rejected, I won't stand in your way | 18:05 |
| stgraber | those were accepted by me as they weren't really causing any harm and there weren't too many of them, I then changed my mind after mentioning the problem to dobey so I'm not going to accept any more of those | 18:07 |
| stgraber | (I wasn't the one who rejected the rest though) | 18:08 |
| dobey | no slangasek rejected them | 18:08 |
| * slangasek nods | 18:09 | |
| dobey | i don't see why it is a "problem" | 18:10 |
| stgraber | you're wasting review time, buildd resources and libraring space for nothing | 18:11 |
| dobey | i didn't even know they had to be reviewed until after i uploaded stuff and was getting "waiting for review" messages. final freeze isn't until tomorrow | 18:14 |
| stgraber | we've been freezing the archive between the final beta and release for the past 2-3 cycles now | 18:14 |
| dobey | and i wouldn't really call a very short build time on an emtpy build queue a "waste of resources" | 18:14 |
| stgraber | I uploaded that lxc so would be nice if someone else could review it in the queue | 18:24 |
| jdstrand | can someone look at apparmor-easyprof-ubuntu real quick? it is required for (at least) grouper with mir | 18:41 |
| shadeslayer | could someone also approve user-manager ( has a bug fix that allows setting the user avatar ) and soprano ( fixes a bug when the db gets corrupted and causes problems in nepomuk ) | 18:47 |
| shadeslayer | ScottK doesn't seem to be around | 18:48 |
| stgraber | shadeslayer: looks like you're adding/changing strings in there... | 18:50 |
| shadeslayer | stgraber: which package? | 18:50 |
| doko | whoever did accept exempi, thanks and please accept exiv2 too | 18:50 |
| stgraber | shadeslayer: user-manager | 18:51 |
| shadeslayer | stgraber: okay | 18:53 |
| shadeslayer | stgraber: can we get it in either way? because apparently Riddell asked for some UI modifications which resulted in string changes | 18:56 |
| shadeslayer | stgraber: we just didn't upload the updated git snapshot | 18:56 |
| stgraber | shadeslayer: if you get a UIFe, then sure (you usually need whoever works on your documentation and translations to ack the change for that) | 18:57 |
| shadeslayer | okay | 18:57 |
| slangasek | stgraber: lxc, there are no linked bugs for either of the changes; verbose++ please? | 19:02 |
| slangasek | stgraber: 0011-ubuntu-cloud-prep-hook-fix-debug-helper-to-not-inapp.patch at least is self-explanatory from the shell, but the apparmor change I feel I need context for | 19:02 |
| stgraber | slangasek: bug 1237543 is for that cherry-picked patch (the bug was filed after hallyn fixed it upstream and in the package, that's why we don't have a LP:#xxxxx in there) | 19:04 |
| ubot2 | Launchpad bug 1237543 in lxc (Ubuntu) "lxc-create (or clone) of cloud template with '--cloud' exits failure" [Low,In progress] https://launchpad.net/bugs/1237543 | 19:04 |
| stgraber | slangasek: the other one was reported by vila on IRC. Basically the new apparmor blocks dbus by default. | 19:04 |
| stgraber | slangasek: so a standard saucy container would be unable to use dbus with our current profile. | 19:04 |
| zul | can someone look at python-cinderclient FFE please (#1236901) | 19:05 |
| stgraber | slangasek: fixing that needs adding "dbus," to our default profile, but we can't just do it unconditionaly since the same source package is backported to all releases down to precise | 19:05 |
| stgraber | slangasek: and any release < saucy will fail to parse an apparmor profile containing "dbus" | 19:05 |
| slangasek | stgraber: ok. So I'm generally pretty opposed to applying per-version hacks in debian/rules instead of just expressing those differences in different per-release VCS branches in the natural way. Is there a reason that couldn't be done here, even though it's a backport? | 19:07 |
| stgraber | slangasek: I'd rather avoid having to update 8 packaging branches and having to change all our automatic build scripts to know which branch to pick up when pushing new builds... | 19:09 |
| slangasek | sure; whereas I'd rather avoid having debian/rules doing magic like applying dpkg --compare-versions to lsb_release ;) | 19:10 |
| stgraber | (we support precise, quantal, raring, saucy in both upstream-daily and ubuntu form, so I alrady have to sync between two branches every time we change something, having to maintain per-series would be a bit of a pain) | 19:10 |
| slangasek | in practice this delta would only require two extra branches (saucy vs. pre-saucy) if you wanted to do it that way | 19:11 |
| slangasek | and effectively it would be safe to auto-merge | 19:11 |
| slangasek | (except when it wasn't safe, which would probably be a clue to look at it :) | 19:11 |
| stgraber | yeah but that'd still prevent us from using requestbackport since the automated backporting tools just take from a release and automatically spit out an updated source package ready for upload | 19:12 |
| slangasek | there were provisions for sourceful backports, I thought. requestbackport doesn't handle that workflow? | 19:12 |
| stgraber | requestbackport takes a source series, a destination series and a source package | 19:13 |
| stgraber | *source package name | 19:13 |
| slangasek | yah | 19:13 |
| slangasek | so I'll allow this, because my objections don't really have anything to do with the release freeze | 19:14 |
| slangasek | but at minimum, I would recommend that you invert the debian/rules handling, to make the backports the special case so that 'dbus' is part of the actual source | 19:14 |
| slangasek | (not for saucy, but for the future) | 19:14 |
| stgraber | slangasek: well, my plan for the future is to move the apparmor profiles upstream and have it be a .in file | 19:15 |
| slangasek | ok | 19:16 |
| stgraber | since we already detect most distros in our configure.ac and I believe also export the distro version as a variable | 19:16 |
| slangasek | anyway, the point I want to express is that I don't think it's a good idea for part of the "source" of the package to be hidden in debian/rules | 19:17 |
| slangasek | and now I'll get off my soapbox :) | 19:17 |
| stgraber | slangasek: oh I agree and I was pretty annoyed by it when I got the report today as none of the apparmor abstractions would give me something that worked on all the releases I care about... so I basically ended up with 3 choices 1) hackish debian/rules 2) separate source package for backports 3) only support session+system bus and block accessibilty bus | 19:19 |
| stgraber | 2) meant quite a bit of time to invest in the coming years, 3) meant breaking the desktop QA test environment at least and 1) was just ugly, so ugly it ended up being :) | 19:20 |
| jjohansen | stgraber: you supposed to pitch a fit at us and tell us how much this sucks so we are motivated to fix it | 19:21 |
| jjohansen | stgraber: truth is this is bad, and something we need to fix, but I don't have a good solution atm | 19:21 |
| stgraber | jjohansen: oh, I certainly wanted to blame you ;) but short of having the parser accept any invalid keyword, I'm not sure what you could have done differently. | 19:22 |
| stgraber | block by default is always how apparmor worked, so that makes sense and the dbus keyword was only introduced this cycle, so any older parser will fail... I guess if we had a all-dbus abstraction around since precise that would have been mapped to "dbus," in saucy, then I could have used that, but you'd have had to be pretty good at predicting the future a couple years ago to do that ;) | 19:24 |
| * mdeslaur hands jjohansen a crystal ball | 19:24 | |
| stgraber | well, actually there was a 4th alternative in my list above, backport apparmor to all releases where we backport lxc, but I figured this would probably cause even more problems ;) | 19:25 |
| jjohansen | stgraber: well 2 things | 19:25 |
| jjohansen | 1. the parser should be setting some conditional variables, then at the very least you could have wrapped the rule in if $COND_X in the policy (this should have been done a long time ago) | 19:25 |
| jjohansen | 2. there are plans to extent the parser so you can tell it how to parse new rules as part of policy, and a little rule to ignore dbus could have been done that way. | 19:25 |
| jjohansen | Of course either of those mean backporting the changes | 19:25 |
| jjohansen | mdeslaur: well its a problem we have known about for years | 19:25 |
| jjohansen | stgraber: yeah, backports are a pita | 19:26 |
| mdeslaur | jjohansen: yeah | 19:26 |
| stgraber | jjohansen: ah, those two sound pretty nice, can we have that for 14.04? :) | 19:26 |
| jdstrand | I'll answer that with 'unlikely' | 19:26 |
| jjohansen | hehe | 19:26 |
| stgraber | ok :) | 19:26 |
| jdstrand | stgraber: we have a pretty big pile of stuff for 14.04 which includes lxc | 19:27 |
| jjohansen | stgraber: well if you can find us some resources to work on it ... :) | 19:27 |
| stgraber | anyway, on my personal apparmor priority list, stacking is way more important than that stuff for 14.04 | 19:27 |
| jdstrand | stgraber: *perhaps* 14.10 would get something like that, we'll see | 19:27 |
| jjohansen | stgraber: you would say that :) | 19:27 |
| jdstrand | stgraber: yeah-- stacking is coming and planned. I know we've been saying that for a while, but a lot of work was done for that already and we hope to deliver it in 14.04 | 19:28 |
| jdstrand | at least something quite a bit better than what we have now | 19:28 |
| slangasek | stgraber: done differently> they could also have backported support for the 'dbus' keyword to older releases | 19:37 |
| slangasek | even if it was a no-op | 19:37 |
| stgraber | oh, apparently I'm TIL on irssi, I didn't know that (just got a build failure for arm64) | 19:48 |
| infinity | Yeah, I've been discovering all sorts of things I forgot I was TIL on. :) | 19:52 |
| infinity | doko: Erk, why are up updating config.{sub,guess} in patches? | 19:52 |
| infinity | doko: That's just a recipe for failure next time. | 19:53 |
| infinity | doko: Something like http://launchpadlibrarian.net/152821294/bind9_1%3A9.9.3.dfsg.P2-4_1%3A9.9.3.dfsg.P2-4ubuntu1.diff.gz is much saner, and doesn't lead to a painful-to-rebase patch. | 19:54 |
| stgraber | I saw a few done with dh-autoreconf though I didn't check if there was a pattern there (!doko => dh-autoreconf, doko => massive ugly patch) | 19:54 |
| doko | infinity, well, for two or three packages, I tried to introduce autoreconf, and it did fail | 19:54 |
| slangasek | infinity, stgraber: you're both slackers, you should be more like me and make sure everything you touch gets in sync with Debian so that you can stop being TIL ;) | 19:54 |
| doko | so I assume at this stage the patch is the safest way not interfer with a build system | 19:54 |
| infinity | doko: Hence my pointing at the above that uses autotools-dev, not autoreconf. | 19:54 |
| infinity | doko: (--with autotools-dev, if it's dh(1), or as above, if it's old-style) | 19:55 |
| stgraber | slangasek: I'm TIL because I fixed a utf-8 related bug a while back, that got merged upstream so I've been doing standard Debian merges since as we patch irssi to suggest irc.ubuntu.com apparently... | 19:55 |
| slangasek | stgraber: right :) | 19:55 |
| doko | infinity, and this does work when files are in subdirs? | 19:56 |
| infinity | doko: Yup, it finds all copies. | 19:56 |
| doko | and figures out the right -I m4 flags? | 19:56 |
| doko | I give it a try for the next package | 19:56 |
| infinity | Err, what now? It literally just replaces config.sub and config.guess. | 19:56 |
| slangasek | stgraber: also, is 91_ssl_proxy.patch still in our delta? :) | 19:56 |
| slangasek | ah, no apparently that's upstreamed in Debian \o/ | 19:57 |
| roaksoax | All. So we are looking to upload a new maas release that contains various fixes and improvements to the previous FFe upload we made. We have discovered a couple of regressions that we are currently working on fixing. So we are looking into doing the upload tonight if things go as expected | 19:57 |
| infinity | doko: It's not a reconf (autoreconf is for that), it just updates config.{sub,guess}, but reliably, and at build-time. | 19:57 |
| doko | ok, then let me redo irssi | 19:57 |
| roaksoax | for which we might require delaying the release candidate remastering | 19:57 |
| stgraber | slangasek: not according to my merge changelog at least | 19:58 |
| slangasek | stgraber: yeah, so evidently my patch is not part of the delta, at least :-) | 19:58 |
| doko | infinity, irssi & jigit | 20:24 |
| TheDrums | tyhicks: Thanks for taking a look at the dbus bug, and for the quick fix. | 20:24 |
| cjwatson | stgraber: Generally I use dh_autoreconf when the existing build system looks fairly recent and I've test-built to make sure it's safe and does the job, and dh_autotools-dev_* otherwise | 20:36 |
| cjwatson | stgraber: And I brutalise all my own packages until dh_autoreconf is safe and does the job :-) | 20:37 |
| infinity | cjwatson: "... and does the job" being a key point, as autoreconf seems to miss on updating config.* in some cases. | 20:38 |
| cjwatson | infinity: Yes. Fortunately I understand when and know how to deal with it. | 20:38 |
| cjwatson | infinity: It's not a problem for packages that use automake as well as autoconf. | 20:38 |
| infinity | Would be nice if it just used the same logic as dh_autotools-dev with the find-and-replace-all-copies thing. | 20:38 |
| cjwatson | Yeah, it wouldn't hurt; though I sort of feel autoreconf itself should do that | 20:39 |
| infinity | (If autogen/etc then goes and replaces again, it's not a big deal, as you just restore backups in clean, and life is good) | 20:39 |
| cjwatson | Feels wrong for a high-level wrapper to be doing it | 20:39 |
| infinity | Sure, auto* should get it right, but it doesn't always seem to. The higher level wrapper papering over that wouldn't be bad, IMO. | 20:39 |
| cjwatson | The reason it currently gets it wrong is that it basically just calls a sequence of lower-level tools, and the ones that update config.{guess,sub} are automake and libtoolize. | 20:41 |
| slangasek | if someone would do me the honor of reviewing glm in the queue, that's a fix needed to unbreak cross-installability of unity8 build-deps | 20:41 |
| cjwatson | autoconf on its own doesn't. | 20:41 |
| infinity | cjwatson: Right. Well, contrary to (documented) protests to the contrary, mixing dh_autoreconf and dh_autotools-dev works, if you get the sequence order right, but it's icky. And would be nice if it was smart enough to just integrate one into the other to always get it right implicitly. | 20:44 |
| cjwatson | I still prefer my recipe for it, since I know exactly how it interacts :-) | 20:45 |
| cjwatson | (Would be nice if it worked OOTB, sure) | 20:45 |
| stgraber | cjwatson: any known problem with autopkgtest? | 20:49 |
| cjwatson | stgraber: first I've heard of it, but I've been out for a few hours | 20:49 |
| stgraber | cjwatson: lxc was accepted over an hour ago, britney says it's waiting on autopkgtest but I don't see any test scheduled on Jenkins | 20:49 |
| stgraber | http://10.98.0.1:8080/view/Saucy/view/AutoPkgTest/job/saucy-adt-lxc/ shows ubuntu9 and no ubuntu10 | 20:50 |
| cjwatson | The running one at the top is 10, I think | 20:51 |
| stgraber | the one that showed up 10s ago, yep, looks like it ;) | 20:51 |
| stgraber | guess I wasn't patient enough | 20:51 |
| cjwatson | Maybe the first request got lost for some reason, not sure | 20:51 |
| cjwatson | I think generally the next p-m cycle will re-request if that happens | 20:52 |
| slangasek | cjwatson: so bug #1208800 was an MIR for click, which was accepted by jdstrand; it seems that this is the only reason click (and various dependencies that are still being iterated) are in the 'supported' seed, which has caused a couple of those to be caught in the queue that shouldn't have been. Can we just drop this from supported and put in it the touch seed where it belongs instead (for now)? | 21:14 |
| ubot2 | Launchpad bug 1208800 in click (Ubuntu) "[MIR] click" [Undecided,Fix released] https://launchpad.net/bugs/1208800 | 21:14 |
| slangasek | click isn't any more or less "supported" than the rest of the phone image, AFAICS | 21:14 |
| cjwatson | slangasek: It's in desktop because I expect people to install it there for development | 21:14 |
| cjwatson | or in supported, whatever | 21:15 |
| slangasek | but then wouldn't we want that pulled via the sdk package? | 21:15 |
| slangasek | which is also not seeded | 21:15 |
| cjwatson | well, I expect click to be used other than via the sdk | 21:15 |
| cjwatson | and indeed there's the problem that that's not seeded | 21:15 |
| cjwatson | can we just leave it as it is for now? it's not all that big a problem | 21:16 |
| slangasek | my point is that the current seeding is adding friction where I think it shouldn't | 21:16 |
| cjwatson | we could make an exception for it in auto-accept rather than having to move it around in the archive | 21:16 |
| slangasek | because the apparmor stack is (was?) still being worked on | 21:16 |
| cjwatson | AIUI that's quite practical | 21:16 |
| slangasek | for click-and-all-its-transitive-deps | 21:16 |
| slangasek | ? | 21:16 |
| cjwatson | it doesn't have *that* many | 21:16 |
| cjwatson | half a dozen or so I think | 21:17 |
| slangasek | ok | 21:17 |
| slangasek | if you and stgraber can sort that out, then that's the thing I really care about | 21:17 |
| slangasek | it just seemed inconsistent to me to have it directly seeded | 21:18 |
| slangasek | so I thought that might've been the easier solution :) | 21:18 |
| cjwatson | it just seems like a retrograde step to move it back | 21:18 |
| slangasek | stgraber: could you please whitelist click and its dependencies (as indicated by cjwatson) for queue auto-accept? | 21:20 |
| cjwatson | aha, new is now occasionally involving five binaries :) | 21:20 |
| slangasek | whee | 21:20 |
| stgraber | slangasek: I added a whitelist and whitelisted click, click-apparmor and ubuntu-system-settings | 21:24 |
| jdstrand | stgraber: can you also whitelist apparmor-easyprof-ubuntu? | 21:25 |
| jdstrand | stgraber: python3-apparmor-click (from click-apparmor) pulls it in | 21:25 |
| stgraber | slangasek: I didn't whitelist any of their dependencies because it's not impossible that those may get seeded or put in the packageset for other reasons (if we noticed one that gets frequent update, I'm fine with individual whitelisting) | 21:25 |
| stgraber | jdstrand: ok | 21:25 |
| stgraber | done | 21:26 |
| jdstrand | thanks | 21:26 |
| cjwatson | stgraber: upstart-app-launch was the other one in my list | 21:26 |
| jdstrand | stgraber: you just whitelist sources or binaries, or both? | 21:26 |
| stgraber | jdstrand: source | 21:27 |
| cjwatson | stgraber: ubuntu-system-settings was only in touch to begin with, at least according to seeded-in-ubuntu | 21:27 |
| jdstrand | ok | 21:27 |
| slangasek | cjwatson: yeah, I think ubuntu-system-settings was the one you fixed the lubuntu supported seed for earlier | 21:27 |
| stgraber | cjwatson: ah, good, will unwhitelist it then. It used to get stuck in the queue because of lubuntu supported | 21:27 |
| cjwatson | slangasek: right | 21:27 |
| jdstrand | url-dispatcher may be another candidate | 21:27 |
| cjwatson | stgraber: yeah, that was an obscure seed bug | 21:27 |
| slangasek | stgraber: and yeah, upstart-app-launch was the other one I knew about - thanks | 21:27 |
| cjwatson | jdstrand: no, that's in Ubuntu daily-live images | 21:27 |
| cjwatson | I haven't checked why, but it's more deeply embedded than other things listed | 21:28 |
| slangasek | cjwatson: it's in universe? | 21:28 |
| stgraber | ok, updated. Any other I'll just add as I waive them through the queue during review | 21:28 |
| slangasek | I mean, I see the same thing from seeded-in-ubuntu, but I'm confused by what it shows | 21:28 |
| cjwatson | liburl-dispatcher1 | 0.1+13.10.20131003-0ubuntu1 | saucy | amd64, armhf, i386, powerpc | 21:28 |
| cjwatson | liburl-dispatcher1-dev | 0.1+13.10.20131003-0ubuntu1 | saucy | amd64, armhf, i386, powerpc | 21:28 |
| stgraber | so far the click apparmor stuff has been the one I had to let through the most I think | 21:28 |
| cjwatson | url-dispatcher | 0.1+13.10.20131003-0ubuntu1 | saucy | source | 21:28 |
| cjwatson | url-dispatcher and url-dispatcher-tools binaries are in universe | 21:28 |
| slangasek | oh right, source v. binary | 21:28 |
| cjwatson | http://people.canonical.com/~ubuntu-archive/germinate-output/ubuntu.saucy/rdepends/ALL/liburl-dispatcher1 | 21:29 |
| slangasek | so no love for url-dispatcher | 21:29 |
| cjwatson | reverse depends: indicator-bluetooth reverse recommends: unity | 21:29 |
| slangasek | interesting | 21:29 |
| cjwatson | and indeed several other indicators | 21:29 |
| cjwatson | datetime, power, sound | 21:30 |
| slangasek | yes, we want to make sure there's a common framework for dispatching those urls from your bluetooth headset, so that attackers don't have to search too hard ;) | 21:30 |
| smoser | hey. i jus tuploaded a simplestreams simplestreams_0.1.0~bzr315-0ubuntu1.dsc . that replacews the bzr314 that i did earlier. | 21:31 |
| smoser | if someone could nak the 314 that'd be thank ful. | 21:31 |
| slangasek | done | 21:31 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!