/srv/irclogs.ubuntu.com/2014/04/03/#ubuntu-devel.txt

cjwatsonxnox: https://rt.admin.canonical.com/Ticket/Display.html?id=6846800:06
cjwatsonxnox: For now you'll just have to get IS to remove things for you00:06
xnoxcjwatson: whilst modern style porter boxes would be nice, getting rapt to support "remove" command would be nice as well.00:08
xnox(limited to things outside of e.g. minimal and build-essentials or some such)00:09
cjwatsonMaybe, but that just increases the chance of people clashing with each other.  I'd rather we just had modern infra00:09
infinityxnox: Not supporting remove was an intentional design decision when we wrote rapt.00:13
xnoxok.00:13
infinityxnox: Because you removing a build-conflict that happens to be a build-dep that doko's 8h build-in-progress depdends on would suck.00:14
infinity(But yes, we should just move to the Debian way of doing things)00:14
doko?00:15
xnoxdoko: don't worry, we are not _actually_ breaking one of your in-progress 8h builds. it's just hypothetical =)00:15
infinitydoko: You were just being used as a fictional example, not being summoned.00:15
dokooh, I would like YOU to be summoned, and not just as a fictional example00:17
infinitydoko: But, given our history, whenever I think of someone breaking a computer and affecting a long-running build, the hypothetical user is you. ;)00:17
* infinity still remembers the week-long GCC builds on his Amiga...00:17
dokoyeah, glibc sucks with test suites00:17
mwhudsoninfinity: so that patch from andrew fixes the eglibc build for me00:19
mwhudsonoh i guess i should also glom up the test case00:23
infinitymwhudson: Okay, can you give me an email with all the patches you know and love, and I'll get them queued up for the next upload?00:38
infinitymwhudson: So, that should be Will's setcontext patch, Will's testcase for same, and that 2-liner for the alignment issue.00:39
infinitymwhudson: With patch headers pointing to origin, since none of them are in git yet. :/00:39
mwhudsoninfinity: i just commented on  https://bugs.launchpad.net/ubuntu/+source/eglibc/+bug/1279620#900:39
ubottuLaunchpad bug 1279620 in eglibc (Ubuntu Trusty) "stack corruption running "go install launchpad.net/juju-core/..."" [High,Confirmed]00:39
mwhudsoninfinity: i don't know the formatting for patch headers, i can go find it out i guess...00:40
UBUNTUuserhi guys I want to know if ubuntu 14.04 will work will daul boot00:40
blkperlUBUNTUuser: yep00:40
infinitymwhudson: Oh, I don't care if it's formatted in any specific way.  Garbage above the diff is fine.00:40
infinitymwhudson: I can fix it up, I just want the refs.00:40
mwhudsoninfinity: is that bug comment useful? i can email them as well...00:41
UBUNTUuseryou sure I am already having problems with ubuntu 13.1000:42
UBUNTUuserI have a uefi which secure boot00:42
UBUNTUuseris on00:42
blkperlUBUNTUuser: your question is probably better suited for #ubuntu, this is the dev channel00:44
UBUNTUuserok00:45
infinitymwhudson: Nope, that's perfect.00:45
infinityrobert_ancell: Did you want your changelog for that lightdm to actually match reality? :P00:49
robert_ancellinfinity, um... what did I write?00:49
infinityrobert_ancell: s/greeter/cursor/00:49
robert_ancellyeah, that would make more sense00:49
infinityrobert_ancell: Not enough of a nit for me to reject, but feel free to retroactively fix it in your VCS so it's not a lie on the next upload. :P00:50
cyphermoxstgraber: xnox: I'm around now00:50
robert_ancellinfinity, there are many wrong changelog entries :)00:50
infinityrobert_ancell: I don't doubt it. :)00:50
blkperlcjwatson: so since you fixed bug 1300072 my trusty install works but now im running into bug 127432000:51
ubottubug 1300072 in parted (Ubuntu Trusty) "LVM installation fails - regression with parted 2.3-17" [Critical,Fix released] https://launchpad.net/bugs/130007200:51
xnoxcyphermox: well, you could help me understand what's going on in silo11 and why it failed to build =)00:51
ubottubug 1274320 in grub2 (Ubuntu) "Error: diskfilter writes are not supported" [Medium,Confirmed] https://launchpad.net/bugs/127432000:51
cyphermoxsure00:51
infinityrobert_ancell: Also, wtf?  -lstdc++?  Is that not building with g++?00:51
robert_ancellinfinity, I have no idea, but the library build explicitly does that00:51
robert_ancellso making the introspection match fixes the problem00:51
xnoxcyphermox: it said it build everything, and then decided not to upload them.... or something?!00:52
infinityrobert_ancell: Ahh, yeah, it's using cc00:52
infinityrobert_ancell: Building C++ code with cc is usually a sign that one's doing something wrong.00:52
xnoxcyphermox: i'd love to fix that up, if i new what needs changing.00:52
cyphermoxoh, nice.00:53
robert_ancellinfinity, feel free to berate them on https://bugzilla.gnome.org/show_bug.cgi?id=727521 :)00:53
ubottuGnome bug 727521 in build "Fails to build on ppc64el" [Normal,Unconfirmed]00:53
infinityrobert_ancell: Although... This could have more to do with the libtool macros being broken in that build.00:54
infinityrobert_ancell: Did you test with just dh_autoreconf before adding the link hack?00:54
infinitychecking whether the g++ linker (/usr/bin/ld -m elf64ppc) supports shared libraries... yes00:54
infinityErr, wrong paste.00:54
infinitychecking whether the gcc linker (/usr/bin/ld -m elf64ppc) supports shared libraries... no00:54
infinity^-- Indicative of libtool macros needing love.00:54
robert_ancellinfinity, no, I only added it because I couldn't patch the Makefile directly without automake getting confused00:55
infinityrobert_ancell: So, the above needs to be addressed.  And if dh_autoreconf fixes the above, the lstdc++ thing probably isn't needed.00:55
robert_ancellinfinity, I can check, but I think it will need the -l stdc++00:56
infinityrobert_ancell: It really shouldn't if other arches don't.00:56
robert_ancellinfinity, g-ir-scanner is great at getting linking wrong00:57
cjwatsonblkperl: expected, grub can't write to raid, not new00:57
robert_ancellinfinity, huh, you're right00:57
cjwatson(or lvm)00:58
infinityrobert_ancell: Figured.  I'll reject your current upload, then. :)00:58
robert_ancellinfinity, cheers :)00:58
cjwatsonblkperl: and save_env involves writing to /boot/grub/grubenv00:58
cjwatsonblkperl: the best we could do is maybe silence the error - but like I say it isn't a new thing00:59
infinityrobert_ancell: You might want to followup to that upstream bug noting that all they need is a newer/saner libtool.00:59
infinityrobert_ancell: So people don't go hunting in circles some other weird cause. :)00:59
robert_ancellinfinity, should I re-use -0ubuntu2 or do I make a -0ubuntu3?01:00
infinityrobert_ancell: 2 is fine, the rejected one doesn't get used up.01:00
cyphermoxxnox: tbh, I think there might be no way for me to tell you currently01:01
cyphermoxxnox: let's try to run it again :/01:02
xnoxcyphermox: maybe add the two source packages to "Sources:" line?01:05
chilukHey cjwatson , I'm casually looking into https://savannah.gnu.org/bugs/?42026 , and Vladimir thinks that our grub-efi version is differnt from our grub-pc version.  Any clue where to start looking?  The grub-pc version works fine, but the grub-efi fails to discover the amt sol serial port..01:05
xnoxcyphermox: no idea if that will help the train driver or not =)01:05
cyphermoxtwo new source packages?01:07
cjwatsonchiluk: well that's not possible, they're built from the same source.01:07
chilukthat's what I was thinking.01:07
xnoxcyphermox: they are existing, in the archive, and i think they were previously released via train already.01:07
cjwatsonand that patch was long ago.01:07
chilukyeah..01:08
cyphermoxxnox: well if the issue was with versions you should get an error earlier01:08
xnoxcyphermox: right.... hm =) i'm clueless, all I know is how to operate dput =)01:08
cyphermoxunless it's trying to make the exact same version but with a different source tarball, though that shouldn't happen given that we datestamp the version anyway01:08
cyphermoxxnox: let me re-run this step by step01:09
chilukcjwatson  was afraid we were missing a define for the efi build maybe01:09
cyphermoxif it really doesn't work I'll frankenstein the ci... dah01:09
cyphermoxno longer have access to the jenkins box01:09
cjwatsonchiluk: no01:09
cyphermoxxnox: let's just try to rerun, see how it goes, and if it still fails we'll need ev or olli or jfunk to help for now01:10
cjwatsonchiluk: grub_serial_ns8250_add_port could fail for reasons other than the parsing, which would produce the same error01:10
chilukfyi, this isn't official business01:10
cjwatsonchiluk: I think Vladimir has misdiagnosed this01:10
chilukk thanks01:10
cjwatsonchiluk: also grub_serial_ns8250_add_port doesn't seem likely to work on class 3 (or whatever the terminology is) EFI devices01:10
cjwatsonchiluk: you'd need something involving the efiserial driver01:11
chilukany suggestions on how to go about debugging grub?01:11
cjwatsonchiluk: which seems to generate efiXXX style port names01:11
chilukI'm not familiar with that style01:12
chilukare you suggesting trying serial --port=efif0e0?01:12
chilukcjwatson or something similar01:12
cjwatsonI think they're sequential rather than a big hex number01:12
cjwatsonefi0, efi1, etc.01:13
chilukcool give me a sec to check.01:13
cjwatsondon't offhand see how to get a list of registered ports01:13
chilukI'm out of my element here...01:14
cjwatsonoh, the output of "terminal_input" without args should include them with "serial_" prefixed01:14
infinityrobert_ancell: That did the trick, cheers.01:15
robert_ancellinfinity, thank you. I learned some more obscure build system knowledge today :)01:15
chilukcjwatson terminal_input yields "serial_* serial at_keyboard"01:17
cjwatsonchiluk: it is also possible, I suppose, that you have a zombie GRUB image lying around from ages back, so double-check the version on the GRUB menu and not just the binary package01:18
cjwatsonchiluk: maybe your EFI just doesn't probe the serial ports01:18
chilukwhich is possible01:18
chilukI have a bug open with intel as well.01:18
cyphermoxxnox: got it now: https://launchpad.net/~ci-train-ppa-service/+archive/landing-011/+packages01:18
cjwatsonand if it's a bleeding-edge-enough EFI then it may not allow access to the traditional inb/outb style of port-banging01:19
chilukcjwatson, it's showing up as 2.02~beta2-801:19
cjwatsonright, good01:19
xnoxcyphermox: \o/01:19
cjwatsonchiluk: I don't see any debug messages currently being logged by the relevant code, so you'll probably have to add grub_dprintf ("serial", ...) to various places and then "set debug=serial"01:20
infinityIt's a NUC, so about as bleeding edge as it can be without being an engineering sample.01:20
xnoxcyphermox: will test from silo as soon as they are ready, and will ping you about publishing them.01:20
cjwatsonchiluk: grub-core/term/serial.c grub-core/term/efi/serial.c would be the places to start01:20
infinity(I'm not convinced that NUCs aren't pretty much engineering samples, from all the bugs I've heard people complain about)01:20
cjwatsonprobably the latter01:20
chilukah... efi/serial.c missed that one01:20
chilukinfinity, I have mine working pretty darn well./01:20
cjwatsonyou'll probably want to add traces to the loop in grub_efiserial_init01:20
chilukexcept for this efi issue everything else seems pretty decent.01:21
chilukcjwatson, stupid question... what's best way to print out info without printf?01:21
infinitychiluk: Everyone seems to have their pet complaint.  This is yours. :)01:21
chilukinfinity, I even have kvm working reliably-ish.01:21
cjwatsonchiluk: err, I don't quite get the question, grub_dprintf is generally the best way to debug grub01:21
chilukthat's the answer I was looking for.01:22
cjwatsonthere's a grub_printf to print things unconditionally01:22
cjwatsonbut you indeed don't have the standard C library01:22
chilukright..01:22
chilukI haven't screwed with things this early in boot before.01:22
cjwatsonsee e.g. grub-core/disk/efi/efidisk.c for an example of using grub_dprintf01:23
cjwatsonthough it's scattered pretty liberally over the codebase01:23
cjwatsonthe first arg to grub_dprintf is something that you might put in "set debug="01:24
chilukcjwatson, cool thanks again... I'll push this a bit.. considering how many of us are investing in nucs, it'd be nice to have these things work reliably... that and it might be nice to test serial on an efi machine just in case this is a larger problem.01:26
cjwatsonsure, it's possible.  I think it's been confirmed to work at least sometimes :)01:27
chilukfor such old-school tech it should work all the time.01:27
cjwatsonchiluk: serial may be old-school tech, but EFI serial sure isn't.01:30
cjwatsonThe driver only went into 2.02, IIRC01:31
=== salem_ is now known as _salem
cjwatsonOh, no, git says it was in 2.00, just a couple of patches since then01:31
cjwatsonBut even so01:32
blkperlcjwatson: alright thanks, good to know its nothing serious :)01:34
darkxstsomething is pulling in the entire Unity world on Ubuntu GNOME :(01:51
xnoxcyphermox: i have finished testing all packages from silo 11, please upload them into -proposed.02:06
xnoxcyphermox: all is good.02:06
cyphermoxxnox: done02:10
chilukhey infinity can I get some upload lovin on https://bugs.launchpad.net/ubuntu/+source/amtterm/+bug/1301692   I notice there are no patch pilots online right now.02:30
ubottuLaunchpad bug 1301692 in amtterm (Ubuntu) "amtterm errors out with "amtterm: ERROR: redir_data: unknown r->buf 0x29"" [Undecided,In progress]02:30
chilukxnox are you still around and will to upload https://bugs.launchpad.net/ubuntu/+source/amtterm/+bug/130169202:39
ubottuLaunchpad bug 1301692 in amtterm (Ubuntu) "amtterm errors out with "amtterm: ERROR: redir_data: unknown r->buf 0x29"" [Undecided,In progress]02:39
=== thegatta_ is now known as thegattaca
cyphermoxFourDollars: hey, re: your bluetooth patch03:13
=== maclin_ is now known as maclin
FourDollarscyphermox: yes03:14
cyphermoxI'm not convinced it would really fix the crash...03:14
FourDollarscyphermox: Err... XD03:14
cyphermoxI agree with the extra check, but it's freeing the caps list in the device that crashes bluez03:14
FourDollarscyphermox: Because it is freed before.03:15
cyphermoxso that extra check would only guard you from trying the free the device if the device pointer is invalid, not from invalidly freeing the capabilities list if the device pointer is valid but the caps list may not be (as seemed to be the case in your crash)03:15
cyphermoxso while I'm happy with the patch as it is, I can't confirm it's valid as I have no way to really reproduce this state03:16
cyphermoxif you tell me you could reproduce the crasher reliably on your hardware and it fixed it, then all good :)03:17
FourDollarscyphermox: Before it runs to the crash place, it will run unix_device_removed().03:17
cyphermoxoh wait a second03:18
cyphermoxFourDollars: can you easily reproduce this crash? I can't here. I'd really like to see the backtrace for all threads03:32
FourDollarscyphermox: Yes, I can easily reproduce this issue.03:33
FourDollarscyphermox: I need some time to setup the environment because I am working on many different machines for this issue.03:34
cyphermoxoh ok03:34
cyphermoxah, I see what's happening now03:36
FourDollarsDo you stiil need the backtrace for all threads?03:37
FourDollarscyphermox: ?03:38
cyphermoxno, I don't03:39
FourDollarsOK03:40
cyphermoxI see what you're doing now, fully agree with the patch, I'll upload now for trusty, and then for precise03:40
FourDollarsThanks a lot. :D03:40
dannfstgraber: fyi, now that udev links w/ libcgmanager0, d-i ftbfs due to a lack of a libcgmanager-udeb03:55
dannfhallyn: ^03:55
dannfand libnih0 apparently03:57
infinitystgraber: Also, upstart needs udebs now because of that (nih and nih-dbus)03:57
infinityWhich might be a slipper slope...03:57
stgraberwell, our systemd source now build-depends against libcgmanager-dev because we need logind to talk to cgmanager in some cases. I can't think of a good reason for udev itself to do so though, maybe I can get some sanity into upstream's crazy Makefile to avoid that particular one03:57
infinityDisabling cgmanager support in the udev udeb pass might be smarter.03:57
stgraberwell, udev never does anything with cgroups, so there's no good reason for it to be linked to cgmanager to begin with03:58
infinitystgraber: With --as-needed, I wouldn't expect it to be.03:58
infinity./lib/systemd/systemd-udevd04:00
infinitylibcgmanager.so.0 => not found04:00
infinityIt's definitely udevd itself that's linked to it.04:00
infinitystgraber: I'll just add --disable-whatever to the udeb pass and test that.04:01
stgraberinfinity: right, --disable-cgmanager should do the trick04:01
stgraberI'm still pretty confused as to why udev would even use any of that code to begin with, I was under the (apparently wrong) impression that udev was supposed to be kept mostly separate after its merge into systemd...04:02
infinitystgraber: It probably statically links libsystemd, because Lennart and Kai.04:02
infinitystgraber: And your cgmanager stuff is built into libsystemd, so...04:03
infinityTestbuilding now.04:03
chilukhey infinity want to do an upload for https://bugs.launchpad.net/ubuntu/+source/amtterm/+bug/1301692 as well?04:10
ubottuLaunchpad bug 1301692 in amtterm (Ubuntu) "amtterm errors out with "amtterm: ERROR: redir_data: unknown r->buf 0x29"" [Undecided,In progress]04:10
infinitystgraber: http://paste.ubuntu.com/7197140/ Looks like success.04:13
infinitystgraber: And in the queue, if you'd like to do the honours.04:14
stgraberinfinity: looks good, will review in the queue. Thanks for fixing that one!04:14
chilukhey stgraber I see you listed as a patch pilot would you help me with an upload for bug 1301692 ?04:14
ubottubug 1301692 in amtterm (Ubuntu) "amtterm errors out with "amtterm: ERROR: redir_data: unknown r->buf 0x29"" [Undecided,In progress] https://launchpad.net/bugs/130169204:14
stgraberchiluk: it's past midnight here and I'm headed to bed after I'm done reviewing that systemd upload, I'll be happy to take a look tomorrow if you remind me though :)04:16
chiluk sure thing.. I'll harass tomorrow's patch pilot04:17
=== jackson is now known as Guest50945
=== maclin__ is now known as maclin
dholbachgood morning06:47
darkxsthey dholbach06:48
dholbachhi darkxst06:49
darkxstre bug 1294891, was quite clear on exactly what is need with license texts, I put full text for CC-BY-SA 2 and 3 in both COPYING and debian/copyright06:50
ubottubug 1294891 in Ubuntu "Ubuntu GNOME community wallpapers" [High,Incomplete] https://launchpad.net/bugs/129489106:50
darkxsts/was/wasnt/06:50
darkxstis that what you meant?06:50
caribouAre SRU for packages in Universe handled differently than for those in main ?09:04
LaneyNope09:04
caribouLaney: ok,thanks09:05
mptmvo_!!!09:22
mvo_hey mpt09:22
* janimo hugs mvo09:25
* mvo_ hugs janimo09:29
seb128bdmurray, mvo_: would have of you have some spare cycles to look at https://bugs.launchpad.net/ubuntu/+source/ubuntu-release-upgrader/+bug/1294124 ?09:47
ubottuLaunchpad bug 1294124 in ubuntu-release-upgrader (Ubuntu) "do-release-upgrade crashed with AssertionError in confirmChanges()" [Medium,Confirmed]09:47
seb128it's ranked 3rd on the daily e.u.c trusty report with 30 reports in a day09:47
=== vrruiz_ is now known as rvr
jibelseb128, the crash signature is not really accurate for this bug because from the duplicates the root cause of the failure is "MarkUpgrade() called on a non-upgrable pkg: 'brasero'"09:53
seb128jibel, what bug?09:53
jibelseb128, 129412409:53
seb128jibel, you mean it doesn't match https://errors.ubuntu.com/problem/3b1ff63e2f241b3ef46a2ed80e2ae253370b8721 ?09:54
seb128jibel, I'm not sure what you are getting at? it's not a bug? it's mistriaged?09:54
jibelseb128, I mean this signature matches any upgrade that fails because a package is non-upgradable09:55
jibelin that case 'brasero'09:55
seb128where do you see that?09:55
seb128jibel, I'm looking at https://launchpadlibrarian.net/169922204/Traceback.txt ... are we looking at the same thing?09:55
jibelseb128, from apt.log09:55
seb128I mean the signature09:56
seb128where do you see it09:56
jibelhttps://launchpadlibrarian.net/169922201/DuplicateSignature.txt09:57
seb128I don't see anything on https://errors.ubuntu.com/problem/3b1ff63e2f241b3ef46a2ed80e2ae253370b872109:57
seb128jibel, that has the python bt?09:57
seb128jibel, anyway, I'm unsure what you are getting at, even if some packages are not-upgradable that should hit an assert and is a bug still no?09:57
seb128shouldn't*10:00
jibelseb128, right, this is a bug. My point is that the problem on errors.u.c aggregates all the failures caused by non-upgradable packages.10:03
seb128jibel, let's fix the assert and make it report the actual errors then :p10:03
seb128jibel, do you know why those packages (e.g brasero) are non-upgradable?10:04
jibelseb128, not yet, I'll try to reproduce it.10:09
seb128jibel, thanks10:09
* seb128 shakes fist at ted10:10
seb128my desktop greets me at each login with an apport prompt about click :/10:11
pittiseb128: is that still from those few days where a new indicator version pulled in all of click?10:11
seb128pitti, no, I'm having click installed because we use it in the touch settings and I'm testing those on my desktop10:12
seb128things like click list work there10:12
seb128even if the clicks can't be run from unity710:12
=== zyga_ is now known as zyga
Laneyhttps://bugs.launchpad.net/ubuntu/+source/url-dispatcher/+bug/129099710:16
ubottuLaunchpad bug 1290997 in url-dispatcher (Ubuntu) "click crashed with gi._glib.GError in run(): Child process exited with code 139" [High,Triaged]10:16
seb128right10:17
seb128which is why I shake my first at ted :p10:17
seb128(it also tops e.u.c since yesterday)10:18
pittislangasek: as we have logind now taking care of the power button even if nobody is logged in, I think it's time to rip this functionality out of acpid; it leads to bugs like bug 120118010:18
ubottubug 1201180 in kde-workspace (Ubuntu) "Pressing power button turns off the PC ignoring the presence of another session manager" [Medium,Triaged] https://launchpad.net/bugs/120118010:18
pittislangasek: do you agree, or am I missing something here?10:18
=== MacSlow is now known as MacSlow|lunch
xnoxpitti: yes, please!10:38
xnoxpitti: is logind also present on the servers and taking care of login buttons there?10:39
xnoxpitti: atleast acpid shouldn't do anything if there is logind present.10:39
pittixnox: yes, that seems like the easiest solution then (pidof systemd-logind)10:41
pittixnox: I'm not sure whether it's running on all servers, but it might not be10:41
xnoxpitti: it can and does create sessions for serial/ssh users etc. but i'm not sure how far down libpam-systemd & logind are seeded.10:42
pittiI made a comment on the bug, I'll look into that10:47
* Laney finds some systemd-cgmanagery bugs10:48
xnoxsergiusens: hey! Could you please remove ubuntuone click webapp from http://people.canonical.com/~ubuntu-archive/click_packages/click_list ? we no longer want it pre-installed, as u1 file services will be shutdown.10:59
sergiusensxnox: sure; it's a mostly useless web app anyways :-P11:00
sergiusensxnox: is it already unpublished by dbarth?11:01
ogra_yeah, it never had any mobile CSS either11:01
xnoxsergiusens: yes, he unpublished it in the store.11:01
dbarthsergiusens: yup unpublished; see if you need to let your script now11:02
dbarthknow11:02
sergiusensxnox: dbarth done11:02
dbarthcool thanks11:03
sergiusensnext sync should do the right thing at 11 after the clock11:03
pittixnox: FTR, logind isn't on our server images (meh.. we keep dragging along acpid/acpi-support)11:09
pittixnox: so instead of completely ripping it out, let's go for "check for logind" for trusty, yes11:09
pittiacpi-support will survive us all..11:09
xnoxpitti: it will do for another 5 years =)11:10
ypwongseb128, hi!11:10
ypwongseb128, could you or someone in your team take a look at bug 1299855? We need the feature to set the default search engine in ubuntu kylin11:12
ubottubug 1299855 in ubuntu-defaults-builder (Ubuntu) "setting browser.search.defaultenginename in distribution.ini does not have effect" [High,Confirmed] https://launchpad.net/bugs/129985511:12
=== ara is now known as Guest11257
seb128ypwong, hey, sure, adding to our list11:20
seb128happyaron, ^ would you have a free slots to look at that one?11:21
=== mino_ is now known as mino
=== slacker_1l is now known as slacker_nl
infinitypitti: If I'm using run-adt-test, what's the best way to upgrade my base image so it's not updated over and over on every run?12:41
pittiinfinity: just re-run "prepare-testbed amd64" is the easiest12:41
infinitypitti: Won't that download a fresh cloud image all over?12:41
pittiinfinity: with the prepare-testbed VMs it's not trivial to dist-upgrade them; you can do it, though12:42
pittiinfinity: yes, it will; but they aren't particularly big12:42
infinityBigger than a dist-upgrade.:)12:42
infinityBut will keep that in mind the next time it annoys me.12:42
pittiinfinity: so run the image with -hda pristine-trusty-amd64-20140324_084956.img and -hdb pristine-trusty-amd64-20140324_084956.img.seed12:43
pittiwith whichever images you have12:43
pittiand then log in as ubuntu/ubuntu and dist-upgrade12:43
infinitypitti: Also, when can get right of the twisty maze of jenkins-is-the-worst-video-game-ever and have results I can actually navigate, like Debian now does? ;)12:44
infinitypitti: Aaaand a feature request, being able to stuff mirror and apt_proxy info into the image via arguments to prepare-testbed would make me happy.12:48
pittiinfinity: it's in the pipeline :) https://wiki.debian.org/MartinPitt/DistributedDebCI12:51
pittiinfinity: that already exists in autopkgtest12:51
infinitypitti: Which already exists?  Mirror/proxy selection?12:51
pittiinfinity: I don't want to waste too much effort on lp:auto-package-testing and jenkins any more, given that I really want both of them to go away next cycle12:51
pittiadt-buildvm-ubuntu-cloud --help12:52
infinitypitti: Oh, should I be using something other than lp:auto-package-testing?  I was just following a random wiki.12:52
pitti  -m URL, --mirror URL  Use this mirror for apt (default:12:52
pitti                        http://archive.ubuntu.com/ubuntu)12:52
pitti  -p URL, --proxy URL   Use a proxy for apt12:52
pittiand adt-build-lxc just automagically uses whichever apt proxy you have on the host12:52
pittithat doesn't have a --mirror yet, please file a bug if you need that12:52
pittiinfinity: you can use the new stuff; lp:a-p-t is still what we are running in production12:53
pittiinfinity: but locally I don't use it any more, I just run adt-run directly with schroot, LXC, or QEMU (depending on what's appropriate)12:53
pittiinfinity: I spent some time to write /usr/share/doc/autopkgtest/README.running-tests.gz, I hope that's a good enough intro how to run schroot/lxc/qemu and how to build testbeds12:54
infinitypitti: Are there docs for idiots on how to do it that way?  I've just been following the two simple steps on http://packaging.ubuntu.com/html/auto-pkg-test.html#executing-the-test for prepare/run-adt12:55
pittiinfinity: if you have a test which doesn't open network ports or fiddles with the kernel, then using an existing schroot is certainly by far the easiest method12:55
infinitypitti: Ahh, cool.  Alright, I'll look at that at some point.12:55
infinitypitti: And yeah, schroots would be fine for most things (and how Debian seems to be running their tests?), but I'd prefer to match production when I can.  Ish.12:55
pittiinfinity: read that doc and please tell me if anything is unclear; I'm really eager that this is a good intro document, as ohterwise I'll keep explaining this on IRC forever :)12:56
=== _salem is now known as salem_
infinitypitti: Pointing out the doc on the above page might be helpful. ;)12:56
pittiinfinity: don't worry; if it works in a minimal schroot it's very likely also going to work in the fat cloud images12:57
pittiinfinity: yes, I'm going to do that once we actually switch over production12:57
pittiinfinity: I think for now the official answer is still "prepare-testbed every day" and "run-adt-test"12:57
pittiit's just about as expensive as it can get, but it's easy12:57
=== StevenK_ is now known as StevenK
=== wgrant_ is now known as wgrant
=== ara_ is now known as ara
bdmurraypitti: have you had a change to look at my apport-retrace package versions branch?14:16
=== brendand is now known as brendand-afk
stgraberhallyn: any idea what could be causing bug 1301882? from the backtrace it looks like we're stuck in ping...14:17
ubottubug 1301882 in systemd (Ubuntu) "systemd-logind can get hung in cg_enumerate_tasks" [Undecided,New] https://launchpad.net/bugs/130188214:18
bdmurraymvo_: Hi, would you mind having a look at bug 1300465? Its not too important but curious.14:18
ubottubug 1300465 in python-apt (Ubuntu) "description dialog is empty" [High,Triaged] https://launchpad.net/bugs/130046514:18
mvo_sure14:19
mvo_bdmurray: I "stole" #1286161 from you, hope you don't mind14:19
bdmurraymvo_: heh, not at all!14:19
infinityjcastro: He's mine, back off.14:20
infinityErm, ECHAN.14:20
stgraberhallyn: one idea would be to switch to using a single connection to cgmanager and keep it open to avoid all those connect/ping/disconnect but I'm not sure this would have fixed that particular hang. (The reason for all the connect/disconnect is that logind spends most of its time idle and I wanted to be as robust as possible in case something goes wrong with cgmanager and the reconnect code doesn't work)14:21
mvo_what the process to request a FFe for apt 1.0 for trusty? http://people.canonical.com/~mvo/apt_0.9.15.4ubuntu2_to_1.0ubuntu1.debdiff <- its a bit long, mostly cppchecker fixes14:21
stgraberhallyn: an alternative would be to have connect fail after a short timeout if ping doesn't respond, not sure how easy that'd be.14:21
bdmurraymvo_: what might go wrong with the fix for bug 1286161?14:21
ubottubug 1286161 in ubuntu-release-upgrader (Ubuntu Trusty) "13.10 -> 14.04 upgrade failed: initramfs failed to ugprade, udev is not configured yet" [High,In progress] https://launchpad.net/bugs/128616114:21
pittibdmurray: sorry, not this week; the sprint takes pretty much all brain and time, I'm afraid14:24
infinitymvo_: File a but against the apt package titled "[FFe] update apt to shiny version with more cows" , describe the how and why in the bug, and subscribe ubuntu-release.14:25
infinitymvo_: And then convince me directly that it's low impact, and why. ;)14:25
hallynstgraber: I'll take a look14:26
bdmurraypitti: ah, didn't know you were sprinting14:28
mvo_bdmurray: on the phone right now, will reply in a bit14:30
hallynstgraber: do you know if cgmanager had crashed/restarted?14:32
stgraberhallyn: don't know14:33
stgraberLaney: can you post your /var/log/upstart/cgmanager.log in the bug?14:33
hallynok lemme set up a victim vm to torment14:33
Laneystgraber: oh yes, sorry, which one?14:33
Laneyor either I guess14:33
Laneybtw I got it back into the borked state accidentally if you want some data out of it :-)14:34
hallynstgraber: i can't imagine why it would've happened otherwise.  especialy on ping14:34
hallynLaney: you had no containers running right?14:34
Laneyhallyn: I almost certainly did, I usually do14:35
=== pedronis__ is now known as pedronis
Laneyand certainly do now14:35
hallynLaney: oh but that shoudl be fine, i should've asked, the ping hang didn't happen in a container?14:36
LaneyEverything I was doing with loginctl was outside14:36
Laneyhallyn: log is attached14:37
hallynthanks.14:38
mvo_bdmurray: so what might go wrong - well, the ording might break in a different way. there is no indicataion that this is the case and it definitely fixes the bug in the bugreport14:47
mvo_thanks infinity, I will do this14:48
bdmurraymvo_: okay, jibel you could do some automated testing with that new version of apt right?14:48
hallynstgraber: logind doesn't do any threading right?14:49
mvo_bdmurray: yeah, that would be great. I can do some testing on my workstation too. I think its pretty safe (the diff) but we all know that shortly before a release stuff may break :)14:51
jibelbdmurray, sure, if you provide a deb, I can run the tests with this version.14:53
bdmurraymvo_: why don't you upload it to the saucy-proposed queue and I'll approve it later today14:53
mvo_bdmurray: sounds good14:54
bdmurraypitti: the powerd debug symbols seem to be out of date15:01
slangasekpitti: ah yes, if logind is now handling the power button outside of login sessions (hurray), we should indeed drop it from acpid (double-hurray!)15:01
pittislangasek: right, thanks; as xnox pointed out, it's not currently on server images, so for now I just added the "pidof systemd-logind" check15:02
pittislangasek: this acpid{d,support} stuff is annoyingly sticky :(15:02
slangasekpitti: ah, doh15:02
pittibdmurray: checking where it was built and trying to rescue15:03
bdmurraypitti: thanks15:04
pittibdmurray: hm, the ddebs are nowhere on kishi06/toyol/lamiak, so there's nothing to rescue I'm afraid15:06
psusicjwatson: I noticed that it looks like you fixed bug #1065281 some time ago, but it still has one open task against ubiquity ( not release specific ).  Has it really not been fixed in trusty, or does it not affect anything above precise and so the task should be removed?15:06
ubottubug 1065281 in partman-efi (Ubuntu Quantal) "Installer crashed when trying to partition 4k/4k sector hard disks" [High,Triaged] https://launchpad.net/bugs/106528115:06
pittibdmurray: "powerd has no unstripped objects, ignoring"15:07
pittibdmurray: so I think powerd is built without -g15:07
cjwatsonpsusi: ah, see, this is why I hate having ubiquity tasks on bugs that just amount to incorporating new included sources15:07
psusiyep15:08
cjwatsonpsusi: closing, thanks15:08
pittibdmurray: so the build log doesn't have any -On option or -g15:08
pittibdmurray: that's a "cmake does not respect $CFLAGS" kind of bug, I'm afraid15:09
psusibtw, I've been closing a lot of bugs lately that are installer crashes due to booting the installer in efi mode, and trying to install on a disk already set up for bios mode.. I think we really, really need to fix partman-xxx to stop and tell the user they need to reboot in bios mode instead of getting to grub-install failing15:09
psusisound like a good idea, and if so, what is the partman-xxx to file the bug against?  partman-efi, or partman-auto?15:09
stgraberhallyn: it's not extremely clear due to the way all of systemd's code is mixed together... I don't believe logind itself uses threads but I may be wrong...15:10
psusior better yet, switch to using grub-pc instead of grub-efi15:10
cjwatsonpsusi: maybe partman-efi.  not sure we *can* switch to grub-pc at that point, it may not necessarily be able to install15:11
cjwatsonpsusi: certainly not partman-auto15:12
=== brendand-afk is now known as brendand
ogra_stgraber, we are seeing weird issues with the boot on phones (not sure you read the ubuntu-phone ML lately), the system locks up hard after run-init since nothing changed in mountall and cgmanager is the first additional thing to start there, do you know of any issues cgmanager could cause ?15:12
psusicjwatson: why wouldn't it be able to install?15:13
ogra_stgraber, (it only happens ever 20th to 60th boot, which kills the devices in the lab)15:13
ogra_(i noticed plenty of mesaages where gcmanager repawns a few times during a normal boot)15:13
stgraberogra_: and I don't suppose adb is up by the time it hangs?15:14
psusiisn't partman-auto where you pick then guided - install along side option?  I would think that's where you would want to look at the current disk layout and decide if you should force grub-pc instead of grub-efi15:14
stgraberogra_: when cgmanager respawns, do you get a crash entry in /var/crash by any chance?15:14
ogra_stgraber, nope, no adb across run-init ... we have an emergency shell that kicks off wqhen the container cant start and one on panic in the initrd15:14
hallynstgraber: that's bad if it does.  then we need to fix that nih/dbus threading issue15:14
Laneyhallyn: It doesn't AFAICS15:15
hallynah good, my victim is done installing.  now to torture it15:15
hallynok15:15
ogra_stgraber, i'll check, i'm not near the phone atm15:15
dholbachxnox, Paolo just pinged me about https://bugs.launchpad.net/ubuntu/+source/live-build/+bug/1300400 again - he said they want to get out an Italian CD for 14.04 but would need this fixed - if you don't have time, do you know anyone else who could take a look at this?15:15
ubottuLaunchpad bug 1300400 in live-build (Ubuntu) "Please add saucy and trusty strings to releases.sh" [Undecided,Confirmed]15:15
stgraberogra_: it still wouldn't explain why the phone doesn't boot though. Even if cgmanager fails to start or crashes, the worst case scenario should be that the container won't start.15:16
stgraberogra_: but the rest of the system doesn't depend on cgmanager and so everything else should start, including adb15:16
xnoxdholbach: i'll sponsor that patch now.15:17
xnoxdholbach: we can work on a generic solution later.15:18
dholbachxnox, awesome15:18
cjwatsonpsusi: I'd have to go pick over the UEFI spec with a fine tooth-comb to see if there's anything that e.g. a class 3 device might do that could break grub-pc installation.  Dunno15:19
cjwatsonpsusi: platform-specific stuff like this, other than default partition layouts, doesn't belong in partman-auto15:19
cjwatsonpsusi: (also, you're asking a question and then arguing with the answer ...)15:19
ogra_stgraber, unless the android kernel is not safe for something cgmanager tries to do to it15:20
psusiwell, I mean, if the existing disk is MBR partitioned, then obviously the machine normally boots in bios mode, right?  so screw efi15:20
stgraberogra_: doubtful, all cgmanager does is write to plain text files15:21
stgraberogra_: and is basically a copy/paste of what LXC used to do directly itself before15:21
cjwatsonpsusi: well, probably, yeah15:21
ogra_stgraber, ok15:21
xnoxcjwatson: psusi: it can do either, e.g. on Macs they keep MBR and UEFI in sync thus one can boot using either method. That's more or less how bootcamp works.15:21
cjwatsonpsusi: but you definitely want to do it later than partman-auto because it's completely legitimate for somebody to manually zap the existing partition table and replace it with GPT in order to convert to full-fat UEFI15:21
psusixnox: right.. but the disk is gpt ( hybrid )15:22
cjwatsonxnox: that's an irrelevant side issue though, as parted will still see that as GPT15:22
xnoxok.15:22
psusicjwatson: yes, but isn't partman-auto the place where they decide they are not going to do that, but do a side-by-side install?15:22
psusion the existing pure MBR disk?15:22
hallynstgraber: ok, so i have just installed ubuntu-desktop on trusty vm, but cgmanager and logind are not installed.  ?15:23
hallynshouldn't logind be automaticlaly there?15:23
cjwatsonpsusi: they can perfectly well switch to manual partitioning15:23
cjwatsonpsusi: in fact, to overwrite the existing partition table, I think they may *have* to switch to manual partitioning15:23
cjwatsonpsusi: like I say, partman-auto is just simply the wrong place for this15:23
psusi*after* they already chose side by side in partman-auto?15:23
cjwatsonthat has nothing to do with it15:24
cjwatsonlook15:24
stgraberhallyn: it should, is that vm up to date?15:24
cjwatsonconsider an existing mbr disk.  user selects manual partitioning, may or may not replace the entire thing with GPT15:24
psusimaybe I'm misunderstanding the relationship between the menu and the partman modules... I thought that partman-auto implements the options for either guided or manual partitioning?15:24
cjwatsonif it's GPT at the commit stage, then we don't complain, otherwise we do15:25
cjwatsonsimple15:25
cjwatsonpartman-auto does not step in at the commit stage15:25
cjwatsonpartman-efi (and others) do, via check.d scripts15:25
hallynstgraber: yup.  what should bring in logind?  (logind is the pkg name right?)15:25
cjwatsonno, partman-auto does not implement manual partitioning beyond just the single option letting you get into it15:25
stgraberhallyn: systemd-services is the package name15:25
psusiright, that's what I meant, that's where you make the choice15:25
cjwatsonbut that choice is not sufficient information to decide whether to issue this warning15:26
cjwatsonthe proper place to do it is in a check.d script which happens at commit time15:26
cjwatsonand those do not belong in partman-auto15:26
psusiso if you choose the side-by-side option, then at that point, if the disk is mbr, I would think that would be where we want to set a flag to not bother loading partman-efi, and force grub-pc15:26
cjwatsonI'm not going to argue this further15:26
hallynstgraber: oh, hm, that's there15:26
psusiwhat warning?  I'm trying to avoid any kind of warning15:27
cjwatsonnot bother loading partman-efi> you misunderstand the architecture15:27
cjwatsonplease stop it15:27
stgraberhallyn: it should only bring libcgmanager0 though, not the daemon itself15:27
cjwatsonyou asked a question, I gave you the answer ...15:27
stgraberhallyn: so a default desktop install won't have Laney's problem, the hangs are currently restricted to those who have lxc installed15:27
hallynstgraber: oh ok.  and it uses fs if cgmanager is not installed.  got it15:27
hallynok just making sure what i had was sane, thanks :)15:27
cjwatsongrub-installer should be the thing that decides which grub platform to install15:28
cjwatsonif it's necessary to tell the user they can't proceed with their current layout, then that belongs in a check.d script in partman15:28
cjwatsonif it's possible to avoid such a thing, then partman doesn't need to change at all15:29
cjwatsonin neither case should partman-auto be touched15:29
psusiok, but we want to change its decision based on the fact that we are doing a side-by-side install on an MBR disk... is that information availible to grub-installer?15:29
cjwatsonno, there's no need to take that into consideration15:29
cjwatsonall grub-installer needs to know is the partition label in place when it runs15:30
cjwatsonwhich is well after partitioning15:30
cjwatsonit's not relevant how it got that way15:30
psusiahh... ok... got ya15:30
cjwatsonand, you can just as well do a side-by-side install in manual partitioning as in partman-auto ...15:30
cjwatsonit's obviously more effort, but it's entirely doable15:30
cjwatsonpartman-auto is basically just recipes and code for applying them, not anything specifically about the implementation of the various filesystems and labels and such15:31
psusiok, I was thinking we only wanted to do this for the guided install option, but yea, I suppose it also makes sense in manual15:32
cjwatsonI try very hard to keep this sort of thing out of partman-auto because the inevitable result is a bug ten seconds later complaining that the same thing doesn't work in manual partitioning :-)15:33
cjwatsonIt's a lot easier to be hard-nosed about the architecture from the start15:33
psusiwhat triggers partman-efi to be loaded?15:36
psusiso partman-efi's check should throw an error if you are using gpt but don't have an efi system partition ( or maybe bios_grub? ), but if you are using mbr, let it go and have grub-installer switch to grub-pc if using mbr?15:37
psusior perhaps issue a warning that the resulting system may not boot.. hrm...15:41
cjwatsonpartman-efi is Priority: standard, so anna loads it by default if it's available.15:42
psusiprobably want to suppress that warning though if they picked guided install beside15:43
cjwatsonYeah, partman-efi should probably only care about systems with GPT.15:43
cjwatsonI don't think guided install or not should be relevant.  Yes, it's a bug if guided install sets up something wrong, but we still want to know about it early rather than failing in a confused heap that's hell to debug later.15:43
cjwatsonEFI System Partition / bios_grub> I feel that partman-efi should probably allow either, and grub-installer should use that to decide which platform to install15:44
psusiI think the warning makes perfectly good sense if they did a manual setup, but I'd rather not give the user a scary warning if they are doing a side by side install with an existing setup that obviously boots in bios mode just fine15:44
cjwatsonThough partman-efi's job is really specifically about the EFI System Partition, so maybe this actually belongs somewhere like partman-partitioning15:44
cjwatson(which already handles stuff about bios_grub)15:45
cjwatsonWe need to make sure that the logic is such that the warning doesn't appear in such a side-by-side case, and that we install a boot loader which will cope15:45
psusiright. that's what I'm saying15:45
mvo_bdmurray: so for #1300465 - you see pkg.candidate.raw_description, correct?15:46
cjwatsonBut detecting guided partitioning and suppressing the warning in that case is the wrong answer15:46
cjwatsonThis is a safety check - it's better not to proceed than to generate something unbootable due to a logic bug in partitioning15:46
mvo_bdmurray: on what system did you reproduced this? anything I could try to reproduce this?15:46
psusihrm... I suppose I was using the guided partitioning as a surrogate for detecting existing operating systems later15:46
cjwatsonAnd it's furthermore better to at least give users the chance to fix it, rather than have the whole system just be uninstallable15:47
cjwatson(which a check.d warning will naturally do - they can fall back to manual partitioning)15:47
psusiright, but I'd like to avoid even a warning if ther's another OS on the disk that obviously is booting just fine in bios mdoe15:48
cjwatsonWell, we seem to be largely in agreement that we only need to generate a warning if you have a GPT label without any boot-type partition15:49
cjwatsonSo that shouldn't be an issue15:49
psusiright15:49
bdmurraymvo_: yes, I see raw_description but description is empty15:49
cjwatsonWe do need to check the final state though - with BIOS+GPT you could break that existing OS by the changes that you make in the partitioner (not guided, presumably, but you could wipe the bios_grub partition)15:49
cjwatsonAll I'm saying is that the way we avoid issuing warnings with guided partitioning is by not doing wrong things in guided partitioning :-)15:50
cjwatson(Which I think is already the case, as far as this matter goes)15:50
cjwatsonSide-by-side won't touch the boot partition (of either type), and using the whole disk will set up a new one15:51
cjwatsonpartman-auto only has to be aware of the platform to the extent of setting up the right boot-type partition for it, and knowing how to reuse one that already exists15:52
hallynall right reproduced it15:52
cjwatsonAll of that should be in place15:52
bdmurraymvo_: weird, all of those are empty for me15:52
cjwatsonrecipes/atomic is used on BIOS, and has "1 1 1 free $iflabel{ gpt } $reusemethod{ } method{ biosgrub } ." (i.e. "if GPT, reuse an existing bios_grub partition if present, otherwise make a new one")15:53
psusiso should the check for bios_grub || esp be in partman-efi, partman-partitioning, or one in each?15:53
mvo_bdmurray: could you add some debug code into /usr/lib/python2.7/dist-packages/apt/package.py in @property descripton(self)? (around line 330) ?15:53
cjwatsonrecipes-amd64-efi/atomic is used on UEFI, and has "$iflabel{ gpt } $reusemethod{ } method{ efi } format{ } ." (i.e. "if GPT, reuse an existing ESP if present, otherwise make a new one")15:54
mvo_bdmurray: to see if that fails maybe? anything special about this system? or is it a "normal" ubuntu install? (special as in different languages or something)15:54
cjwatsonpsusi: Hm, partman-partitioning/check.d/biosgrub *already* has code that does exactly this15:54
cjwatson# Is there at least one EFI System Partition or BIOS Boot Partition, as15:54
cjwatson# appropriate?15:54
cjwatsonWorst case it has some slight bug and needs to be fixed, but we don't need to write that code again15:55
mvo_bdmurray: do you have any output from "ls /var/lib/apt/lists/*Transla*"15:55
psusiohh?  hrm... it may not be working then15:55
bdmurraymvo_: it happens on a virtual machine too15:55
mvo_bdmurray: I suspect for some reasons the "translations" are not downloaded for you (which means that you don't get the long descriptions)15:55
mvo_bdmurray: the Package file only contains the short summary these days15:55
cjwatsonpsusi: should be visible in the partman log15:55
psusiI'll play around with it15:55
cjwatsonunder check.d/08biosgrub or something like that15:56
mvo_bdmurray: but this does not make that much sense if you have the full description in the pkg.raw_description :/15:56
bdmurraymvo_: yeah, that seems to be it. ls: cannot access /var/lib/apt/lists/*Transla*: No such file or directory15:56
mvo_bdmurray: aha! so the question is why you don't get translations :)15:56
mvo_bdmurray: I assume you never actively disabled them? what do you see with: $ apt-config dump|grep Languages15:57
cjwatsonpsusi: I'd be happy to look through logs of an installation that should have generated such a warning and didn't, if you have one to hand15:57
cjwatson(i.e. manual GPT partitioning and no ESP or bios_grub partition as appropriate)15:57
bdmurraymvo_: its my mirror, sorry15:58
psusicjwatson: I'll try to reproduce it... unless this was a relatively recent fix?  I just know I have seen a number of bugs where people didn't have the bios_grub partition so grub-install failed15:59
mvo_bdmurray: no problem15:59
mvo_bdmurray: good to know whats causing it :)15:59
cjwatsonpsusi: partman-partitioning 83ubuntu2 in precise16:00
cjwatson(before release)16:00
hallynyeah the EMFILE doesn'tlook like a kernel issue - bc it does have a buttlaod of files listed in select16:00
hallynand /proc/pid/fd shows 1024 open files16:01
mvo_infinity: https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1302033 fwiw, but looking over the changelog maybe its better if I just cherry pick stuff16:01
ubottuLaunchpad bug 1302033 in apt (Ubuntu) "[FFe] apt 1.0" [Undecided,New]16:01
infinitymvo_: I like the idea of shipping the new shiny in trusty, but yeah, it might be too late to properly test all those changes.  I'll go through the diff after I've slept and see just how crazy it really is.16:02
infinitymvo_: The build-profiles change is probably trivial and auditable, and I'd like it in, for instance.16:02
mvo_infinity: mostly long, but has some mildly scary bits16:03
mvo_infinity: I'm also a bit torn between shinny and conservative here :)16:03
infinitymvo_: I'll try to get a definitive answer to you soon enough that you can start cherry-picking things you really want if it's a "no".16:03
* mvo_ goes and gets some dinner first16:03
* infinity wonders if it's too late to ship Packages.xz for trusty, or if we can get that through and tested before release day.16:05
infinityhallyn: Did we give up on qemu 2.0, or is that still in testing?16:09
bdmurraymvo_: is bug 1286161 fixed in trusty apt already?16:10
ubottubug 1286161 in ubuntu-release-upgrader (Ubuntu Trusty) "13.10 -> 14.04 upgrade failed: initramfs failed to ugprade, udev is not configured yet" [High,In progress] https://launchpad.net/bugs/128616116:10
stgraberhallyn: does it look like a cgmanager bug or did I mess up something in logind?16:13
hallyninfinity: absolutely did not give up on it.  upstream hasn't released 2.0 yet though,16:14
hallyninfinity: so my plan on monday, if it is not released, is to push based on v2.0.0-rc0 + diffpatch16:14
hallynstgraber: well it's possible that logind is keeping open it's files and so cgmanager never closes them...  not sure yet16:14
infinityhallyn: Well, we're cutting it close.  If you plan to push 2.0ish, and it's well-tested, we should do that sooner, rather than later, and update to a later rc/git/final as we can.16:15
hallynstgraber: tbh since the testsuite doesn't do this it must be somethign logind is doing, but cgmanager should defend itself against it16:15
=== _salem is now known as salem_
stgraberhallyn: ok, so what does it look like, missing disconnect in some case?16:16
hallyninfinity: the 2.0 has been tested quite a bit out of ppa:ubuntu-virt/candidate fwiw, so it wont' be going into the archive green16:16
hallynstgraber: yeah16:16
infinityhallyn: Right, it should still be ASAP is all I'm saying. :)16:16
hallynstgraber: my guess would be the fds we pass in16:16
hallynno wait, you're not doing scm anywhere right?16:16
hallyninfinity: ok, well i can push whenever.  i just pushed a new pkg to ppa.16:17
hallyninfinity: now what i have now is based on a git snapshot tarball.  is that ok, or is it better ot use the qemu v2.0.0-rc0 snapshot + a huge diff?16:17
hallyn(I prefer the former but i have a feeling it's not as 'correct')16:17
stgraberhallyn: nope, not doing a single scm call16:18
hallyninfinity: also since the debian qemu maintainer prefers to have qemu-ssytem-aarch64 be rolled into qemu-system-arm, i figure we should do that now before release16:18
hallyninfinity: any objections to that?  (you've seen the pkg)16:19
stgraberhallyn: I also just re-checked the code and I always call close...16:19
infinityhallyn: Err, which Debian maintainer prefers that?  mjt or vagrant?16:19
hallynmt16:19
hallynmjt16:19
hallynstgraber: well running the testsuite 10 times doesn't leave a single extra open fd...16:19
infinityhallyn: aarch64 and arm really aren't the same platform at all, I feel like perhaps he's missing that.16:19
hallyninfinity: his feeling is if you can run armhf binaries on aarch64, then it's the same situation as x86/x86_6416:20
infinityhallyn: You can't always.16:20
hallynyup we've been over that16:20
hallynanyway i can go either way,16:21
infinityhallyn: aarch64 doesn't require an ARM execution unit, it's an optional bolt-on.16:21
hallynbut if debian will merge them and we don't do it before 14.04 release, then we have to keep a metapackage until 16.1016:21
infinityhallyn: But meh, I don't much care either.  i386/amd64 are in x86, powerpc/ppc64 are in ppc, I don't mind if arm/aarch64 follow suit, even if it's less technically correct.16:21
infinityhallyn: So, if mjt won't move on the topic, bundle 'em up.16:21
infinityhallyn: As to the tarball, I don't care.  There's nothing inherently more or less correct about one method versus the other, so long as you have a trust path of some sort to the original code.16:22
hallyninfinity: if you wanna go make the case in oftc#debian-qemu that it's less technically correct, maybe you can suade him :)  vorlon and i couldnt16:22
hallyncool16:22
infinityhallyn: So, signed tags in the git case, or signed tarballs in the orig case.16:22
hallynok i'll push that today, with or without the merged arm16:22
hallynhm.  signed tags?16:23
infinityhallyn: I'm not going to bother arguing the point, if both you and vorlon tried.  Just merge. :P16:23
hallynmeaning there needs to be a tag by upstream?16:23
hallynor just that the last commit id should show up in the version?16:23
infinityhallyn: Well, or signed commits.  This isn't a "need", per se, just that, well, how else do you know you're not serving me some MITMed git repo of doom? :)16:23
xnoxls16:23
hallyninfinity: still missing the point <duncecap> - how to sign the commit?  right now i have qemu_2.0~git-20140403.de03c31.orig.tar.gz16:24
hallynwhere de03c31 is current git HEAD16:25
infinityhallyn: As in was the commit at the head of your branch signed by anyone?16:25
hallynsigned-off-by, yes16:25
infinityhallyn: And does git log --show-signature show that it was PGP signed?16:29
infinityhallyn: (signed-off != signed)16:29
hallynnope16:30
infinityhallyn: Anyhow, like I said, this isn't some hard requirement, it's just the sanest way to an auditable trust path to someone who says the repo is sane.16:30
infinityThere are tons of snapshots in the archive that aren't signed. :P16:30
hallynthe v2.0.0-rc0 tag isn't signed either actually, so i'll go with the snapshot :)16:30
hallynthanks16:31
hallynoh this'll be fun.   kvm: malloc(): memory corruption: 0x00007f49cc8c3120   (qemu 1.0 on trusty kernel...  probably my own fault)16:32
stgraberLaney: so I believe bug 1301846 shows an actual bug in logind which is usually avoided because logind would just make an invalid fs path and return -1, instead the new code sends that to normalize_controller which contains the assert. I have a fix for this now, doing a test build and will upload after lunch.16:35
ubottubug 1301846 in systemd (Ubuntu) "loginctl crashed with SIGABRT" [Medium,New] https://launchpad.net/bugs/130184616:35
=== roadmr is now known as roadmr_afk
=== roadmr_afk is now known as roadmr
hallynstgraber: oh, how ar eyou opening your dbus connections?  bc if you don't open them as private then they won't really be closed16:37
Laneystgraber: Yeah, I saw it comes all the way from the top but didn't check what logind does itself to avoid passing that in16:37
hallynstgraber: yeah i think this might be the problem,16:38
hallynstgraber: in lxc, we do          connection = dbus_connection_open_private(CGMANAGER_DBUS_SOCK, &dbus_error);16:39
hallyni guess logind is running as a part of a long-running lightdm (?) process.  since it never exits it never closes any of the connections16:39
hallynlemme try my hand at a patch...16:40
hallynstgraber: so i'd try something like http://paste.ubuntu.com/7199494/16:43
hallynlessee if it builds over yonder,16:43
hallynof course if that works, the question remains how cgmanager should defend itself against that16:46
hallynhrmph.  that didn't suffice17:13
hallynd'oh, i see why17:16
hallynlet's try http://paste.ubuntu.com/7199648/17:17
bdmurrayslangasek: looking at bug 1090196 which continues to receive duplicates (see my last comment on it), do you have any ideas what we could do to improve the situation?17:29
ubottubug 1090196 in ndiswrapper (Ubuntu) "ndiswrapper-dkms 1.57-1ubuntu1: ndiswrapper kernel module failed to build [Cannot find kernel build files in ... Please give the path to kernel build directory with the KBUILD=<path> argument to make]" [High,Fix released] https://launchpad.net/bugs/109019617:29
slangasekbdmurray: unseed ndiswrapper as obsolete?17:30
bdmurrayslangasek: on precise?17:30
slangasekoh, probably not so much there :)17:30
slangasekbdmurray: if this was fixed in dkms later, should we maybe backport that fix onto precise?  Or do we know why users are winding up with the image installed but not the headers?17:31
hallynstgraber: : http://paste.ubuntu.com/7199648/ seems to work17:33
hallynsmoser: jcastro: btw that also should fix the problem you had yesterday afternoon i recon' (at least for a bit)17:34
bdmurrayI was guessing that people just read some web page that reads 'apt-get install linux-image-generic-lts-saucy'.17:34
jcastrohallyn, I am free now if you want to take a look17:34
slangasekbdmurray: why would any page say that?  we don't recommend that users install enablement kernels on top of already-installed, already-working systems17:34
hallynjcastro: can you just ps -ef | grep cgmanager to get its pid, then look at /proc/$pid/fd ?17:35
hallyni assume you'll see 1024 fds there17:35
slangasekbdmurray: oh, because there's an askubuntu answer for this question - so it's all jcastro's fault!17:35
hallynstgraber: ^ should I push with that fix?17:35
hallynhm, /me back in 3 mins17:36
smoserhallyn, possible.17:37
smoser$ sudo ls /proc/267/fd | wc -l17:37
smoser20817:37
smoserthat is what was there now.17:37
smoserbut it isn't acting up now.17:37
bdmurrayslangasek: most of what I'm finding seems to recommend linux-generic-lts-* though17:37
jcastroyeah I just did a string search and the lts-* is the one people are recommending17:39
hallynsmoser: log out and in a few times17:39
bdmurrayjcastro: with linux-generic or linux-image-generic though?17:39
jcastroI as searching for linux-image-generic17:40
bdmurrayjcastro: and that wouldn't install the linux-headers which is the problem.17:40
jcastro"sudo apt-get install linux-headers-generic linux-image-generic" seems to be common17:41
jcastrobut I can do a doublecheck17:41
bdmurrayjcastro: okay, if its both that's good17:41
slangasekjcastro: edit proposed on http://askubuntu.com/questions/257617/how-can-i-upgrade-the-ubuntu-12-04-2-kernel-to-3-5-0-23/257623#25762317:41
smoserhallyn, http://paste.ubuntu.com/7199760/17:42
smoserthat is seriously leaky17:42
hallynstgraber: pushed that change to ubuntu:systemd;  i think you're working on another update so i'll leave it there for you.17:42
hallynsmoser: yeah, at least we know the fix for that.  the next q is hwo to have cgmanager protect itself in teh future.  looking at dbus/dbus-timeout.c17:43
jcastrobdmurray, protip: any user can do direct queries of the AU database: http://data.stackexchange.com/askubuntu/queries Handy if you want to search for a specific issue, compare it to say, pageviews of the results to investigate if there is a popular question giving out specific incorrect advice17:44
stgraberhallyn: back from lunch, looking17:46
bdmurrayjcastro: oh neat, thanks17:48
stgraberhallyn: stacked my change on top of that one, test build running.17:54
hallyncool17:54
hallynstgraber: so i'm thinking two things:17:57
hallyn1. have cgmanager keep a count of open connections per uid, limit those to say 5  (maybe 20 for root)17:57
hallyn2. keep a nih_timer for each open connection, reset it on every method call, and close the connection after the default (25s)17:58
stgraberhallyn: 1) would cause some of my machines to fail quite miserably when I start a few hundreds of containers at the same time17:58
stgraberhallyn: 2) sounds fine17:59
hallynmaybe 1 isn't necessary17:59
stgraberhallyn, Laney: just updated to my patched logind here and things appear to be solid, not seeing any fd leaks on login/logout and logind hasn't crashed, uploading18:29
hallyncool18:31
mvo_bdmurray: hello, #1286161 is not yet fixed, I can cherry pick that as well or its fixed with the apt 1.0 update18:46
=== jackson is now known as Guest37026
=== Guest37026 is now known as Noskcaj_
Noskcaj_seb128, Thanks for fixing the gnome-icon-themes-extra bug19:11
seb128Noskcaj_, yw!19:12
=== roadmr is now known as roadmr_afk
barrydoko: what's the current state of cross building python packages?  e.g. https://wiki.ubuntu.com/CrossBuilding  under "Known large-scale problems" it says Python multiarch support, but the debian bug that links to is fixed.   however, cross building e.g. autopilot according to the given instructions fails trying to install dependencies (python2.7).  is the bug fixed in debian but not yet in ubuntu?19:33
dokobarry: which build dependencies?19:34
barrydoko: http://paste.ubuntu.com/7200212/19:35
dokobarry: try with --arch-only19:35
barrydoko: you don't mean `sbuild --arch-only` since sbuild doesn't have that option.  (but --no-arch-all still doesn't help)19:39
dokobarry, for dpkg-buildpackage, it's -B (and -b if you don't want to touch the diff)19:40
psusicjwatson: well darn... I thought that was goign to be nice and easy: I copied a chunk of code from check.d/biosgrub that detects an efi system partition and if not found, fall back to grub-pc, but it seems that you can't make use of parted in grub-installer?  or should you and I just messed something up somehow?19:40
barrydoko: hmm... --no-arch-all is supposed to pass -B to dpkg-buildpackage while --arch-all is supposed to pass -b19:42
barrydoko: should that note about "almost fixed" in the CrossBuilding wiki page be removed?19:47
dokodoes it build for you? if yes, then yes, please remove it19:47
barryit doesn't, but i don't know whether that's caused by the bug or not.19:48
dokowell, try without sbuild19:48
barrydoko: i'm going to say in the context of that wiki page, it is not fixed.  you can cross build e.g. apt just fine with sbuild19:52
dokocjwatson, ^^^  I don't agree with barry here. is this an issue with sbuild?19:53
barrysimilar problem with pure python3 applications19:56
dokobarry, no. try to cross-build zope.interface. this works19:58
* barry tries19:58
dokosorry, can't help you with sbuild19:58
barryyeah no worries, if it's an sbuild problem, i'll file the appropriate bug19:59
barryi'm not going to remove that known problem from the wiki page until and unless i can get it working in sbuild, which is what that page is all about20:00
barryheh, cross building zope.interface *does* work in an sbuild.  i'll look more closely at autopilot's control file20:02
=== roadmr_afk is now known as roadmr
cjwatsonpsusi: there's existing code that uses libparted, just not the parted binary20:28
cjwatsonpsusi: parted_server isn't running so you can't use partman stuff20:28
cjwatsonpsusi: (please mail me if you're still stuck, we don't have very compatible IRC times)20:29
psusicjwatson: yea, that's what my guess was20:34
bdmurraycaribou: can you verify bug 1281066?21:07
ubottubug 1281066 in deja-dup (Ubuntu Saucy) "Needs to backport trusty's fix to handle duplicity's lockfile" [Medium,Fix committed] https://launchpad.net/bugs/128106621:07
Laneystgraber: nice one21:26
bdmurrayLaney: would you mind having a look at bug 1261096? I haven't quite been able to sort out why its restarting whoopsie all the time.21:28
ubottubug 1261096 in whoopsie-preferences (Ubuntu) "whoopsie-prefernces respawns whoopsie in an infinite loop" [High,Confirmed] https://launchpad.net/bugs/126109621:28
bdmurrayrobru: is there a bug refernce for the unity-chromium-extension fix for "Pass string argument to MediaPlayer.init on to backend."?21:29
stgraberLaney: let me know if you get into any more trouble, we want to make sure that stuff is solid before we start shipping cgmanager on all systems whenever we finally land the support in upstart21:32
kirklandcjwatson: I encountered https://bugs.launchpad.net/ubuntu/+source/debian-installer/+bug/1302206 today, installing Ubuntu server21:49
ubottuLaunchpad bug 1302206 in debian-installer (Ubuntu) "wifi configured and working in debian-installer, but not after installation at first boot" [Undecided,New]21:49
kirklandcjwatson: I'm curious if this is designed behavior, if this should have worked?21:50
=== salem_ is now known as _salem
stgraberkirkland: I don't believe netcfg knows how to setup a NM wifi connection at least I can't remember seeing code for that last I patched that part of the code21:56
stgraberoh, though you said server21:56
robrubdmurray, not sure what you're talking about, sorry21:57
kirklandstgraber: I was surprised/impressed that when I chose wlan0 in d-i as my primary interface, it scanned my network, found my essid, prompted me for passphrase, connected, drew an ip21:57
stgraberI'd vaguely expect it to spit out a valid ifupdown config for wifi but maybe the code is less clever than I remembered21:57
kirklandstgraber: I was just disappointed that that working wifi network did not get committed to disk and wasn't operational on reboot21:57
Laneybdmurray: ooooooookay, don't know this code though21:58
stgraberkirkland: yeah and the code has logic to spit out ifupdown or NM config files for most of that, the NM stuff is pretty recent so I wouldn't be surprised that wifi is missing, I'm vaguely surprised it doesn't generate a working ifupdown config though21:58
stgraberkirkland: what's you /etc/network/interfaces like post-install?21:58
LaneyMaybe you could see what whoopsie-preferences is doing21:58
kirklandstgraber: lo only21:58
robrubdmurray, oh, if you're referring to http://bazaar.launchpad.net/~webapps/unity-chromium-extension/13.10/revision/234, you'd have to ask justinmcp, he asked me to SRU it.22:02
bdmurrayrobru: yeah, I'm asking about change that was in 2.4.8+13.10.20131108.1-0ubuntu1 with no bug reference22:03
robrubdmurray, yeah, I don't know it, sorry22:05
bdmurrayrobru: but you uploaded it to saucy -proposed?22:06
robrubdmurray, no? I uploaded it to ppa:ubuntu-unity/sru-staging22:10
robrubdmurray, I gotta step out for a bit. I sent an email to justin asking about the bug and cc'd you.22:13
bdmurrayrobru: okay, thanks22:13
robrubdmurray, you're welcome22:13
bdmurraySweetshark: is there a bug about an issue with OOXML and libreoffice? I'm looking at the libreoffice upload in the precise-proposed queue.22:17
Laneybdmurray: might have fixed it22:35
Laneyyou on amd64?22:35
bdmurrayLaney: yeah22:35
Laneyone sec22:38
Laneybdmurray: okay, give deb http://people.canonical.com/~laney/package-junkyard ./ a go22:40
LaneyLet me know if it fixes it for you and I'll upload in the morning22:40
Laneygoing to bed now, see you22:41
bdmurrayLaney: thanks, that seems like not enough though...23:06

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