/srv/irclogs.ubuntu.com/2015/07/31/#ubuntu-devel.txt

dokohttps://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-039/+sourcepub/5267908/+listing-archive-extra00:09
dokoLaney, slangasek: apparently introduced by new gstreamer00:09
synthorheya01:04
synthoranyone knows about this one: https://twitter.com/temasek71/status/626688248841023488 ?01:04
synthori'm sitting here since hours asking myself why cyanogenmod build fails with broken pipes01:04
synthortested my memory modules01:04
synthorand then i find a twitter post, reboot the old kernel and the build runs through01:05
synthor3.13.0-59 doesn't compile,  3.13.0-58 does01:05
sarnoldsynthor: can you reproduce on a current kernel?01:08
synthornot atm, i rebooted -58 and now the cyanogenmod build runs through for an hour or so01:12
synthorlater i can, of course01:12
pittiGood morning04:11
pittislangasek: gosh, what a shameful britney bug; reproduced in a test case and fixed now05:49
jamespagebdmurray, wily has a sufficently new version that it already contains the fix for that bug06:15
pittiinfinity, slangasek: ah! http://anonscm.debian.org/cgit/debian-release/britney2.git/commit/?id=b9f6417351 introduces :any dependencies06:26
pittihm, but we have that already06:28
stgraberkees: I'm in Europe this week, might be easier to chat next week when I'm back in north america :)06:33
infinitypitti: I don't think britney itself has issues with :any for migration/installability checks.06:33
pittiinfinity: the reverse dep calculation does06:34
pittiI fixed it now06:34
pittiinfinity: http://bazaar.launchpad.net/~pitti/britney/britney2-ubuntu-swiftresults/revision/464 ?06:34
infinitypitti: Hrm.  Curious.  What would that have affected other than autopkgtest triggers, I wonder?06:36
infinitypitti: If it was horribly broken, you'd think we'd have noticed.06:36
pittiinfinity: well, it just didn't consider a Depends: python:any in rdepends, so any installability check etc. would ignore this06:37
infinitypitti: installability checks use forward deps, not revdeps.06:38
infinitypitti: Perhaps it might have tripped up the autohinter occasionally.06:38
pittiRDEPENDS is also being used in doop_source(), is that not used for installability checks?06:38
pittiin _check_packages() too06:39
pittiso I figure what *could* happen now is that this change now causes britney to hold back packages that it previously didn't06:39
infinityDunno.  I'd have to read a lot more, I'm just pondering how it wasn't obviously broken (and it wasn't), given the numeber of things with foo:any deps.06:39
pittihopefully this is actually justified in most cases, but if it's wrong, then it would err on the safe side, wouldn't it?06:39
infinityAnyhow, regardless, your patch seems safe and obvious enough.06:40
pittiis there any other suffix than :any?06:40
infinity:native06:41
pittia split(':')[0] might be more appropriate06:41
infinityAlthough, wait.  Tell me the output from the thing you patched doesn't get fed to the bits Colin fixed?06:41
infinityCause then they'd suddenly be wrong.06:41
infinitySince his stuff explicitly only allows an :any dep on things that are M-A:allowed (as the spec requires).06:41
infinityIf you strip :any before you get to that check, then you're regressing that.06:42
pittiregister_reverse() actually happens after that (cjwatson changed read_sources(),  which calls register_reverse near the end)06:42
pittiinfinity: ipython doesn't have M-A: allowed06:42
pittipython does06:43
infinitypitti: No, python does.06:43
infinityYou can only dep on foo:any if foo is M-A:allowed06:43
pittiah no, not even read_sources, but get_dependency_solvers()06:44
pittibut a package without M-A: can have a Depends: foo:any, no?06:44
infinitypitti: Sure.  The point is that for installability checks, you need the full dep, a stripped version is a lie.06:45
pittianyway, the above commit does not actually filter them out for the "packages" maps calculation, only in the subsequent installability test06:45
pittiyes, http://bazaar.launchpad.net/~pitti/britney/britney2-ubuntu-swiftresults/revision/464 does not modify the DEPENDS field06:45
pittibut when reading the DEPENDS field (which includes the :any) it currenlty tests whether that value is a known package06:46
pittibut "python:any" is not a package name06:46
pittiso in the dict lookup it must strip the :any suffix06:46
pittiof course packages[DEPENDS] should continue to have the :any suffix (and I'm not changing that)06:47
infinitypitti: Mmkay.06:48
pittiI'll send a Debian bug about that too06:50
pittiinfinity: you don't happen to know the Debian britney2-tests suite? I'm getting http://paste.ubuntu.com/11971566/06:50
infinitypitti: Not sure how to make the testsuite go.06:51
pittiit doesn't cover ":any" stuff right now06:51
infinitypitti: Do we trigger on rbuilddeps too or just rdeps, I can never remember?06:52
pittiinfinity: so far (in old and new infra) only on rdeps06:52
infinityI guess just rdeps, or we'd have a lot more tests running.06:52
infinitypitti: Anyhow, if rbuilddeps were ever in the cards, you'd have to worry about both :any and :native06:52
pittiENOTENOUGHHORSEPOWER06:52
pittiwe don't even trigger autodep8 tests ATM, for the same reason06:52
infinity:native is unique to build-deps (it doesn't make sense for deps, since "native" is the default assumption anyway).06:53
pittiinfinity: if there's only these two, I think  split(':')[0] is better -- ':' can't ever occur in a package name, so stripping off all qualifiers for the package dict lookup seems safer06:54
infinitypitti: Yeah, that's what I did in the old-skool sbuild fork, since I didn't care about supporting cross-building. :P06:54
infinitypitti: Made life much simpler to just ignore M-A and pretend it wasn't a thing.06:54
pittiwell, britney should take it into account06:55
infinitybritney's meant to be growing actual MA support soon anyway.06:55
infinityWhich means cross-arch deps without hacks.06:56
infinityWhich I still think is a mistake, but we'll see if people abuse it. :P06:56
infinityRight now, we have exactly one cross-arch dep, I think (wine64->wine32), and I think that's about the right number.06:56
infinityOh, and build-essential-$arch->libc:$arch.  So, two.06:57
infinitys/libc/libc-dev/06:57
dholbachgood morning07:00
pittiinfinity: ah, got the tests to work, still all pass07:08
pittiforwarded to Debian bug 79419407:10
ubottuDebian bug 794194 in release.debian.org "britney: Strip off Multi-Arch qualifiers in reverse dependency calculation" [Normal,Open] http://bugs.debian.org/79419407:10
Laneydoko: fixing it, thx for noticing07:53
Laneydoko: shall I just upload to 39 or wily or what?08:03
=== jamesh__ is now known as jamesh
dokoLaney, fixing in gstreamer, or thumbnailer?08:22
Laneythumbnailer?08:22
Laneyyou mean tp-qt5, and it's there08:22
dokoLaney, yes, 39 would be fine.08:27
=== vrruiz_ is now known as rvr
pittisbeattie: FYI, new apparmor seems to break content-hub: http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#apparmor09:39
jjohansenpitti: do you know what kernel that was run on?09:40
sbeattiepitti: yeah, I haven't dug into it yet, need to dig out the bits to reproduce it.09:41
dokoLaney, https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-039/+build/7741720  new gnome components?09:41
pittisbeattie: ack, just wanted to make sure you're aware, as email notification isn't very effective for these09:41
Laneydoko: sec, on the phone09:41
pittijjohansen: https://jenkins.qa.ubuntu.com/job/wily-adt-content-hub/lastBuild/ARCH=amd64,label=adt/artifact/results/testbed-packages/*view*/ says 4.1.0-3.309:41
jjohansenpitti: I asked because the kernel team is rolling out the 4.2 rebase it is possible that could cause some regressions as well09:42
jjohansenpitti: thanks09:42
pittiI should start referring to the new system on http://autopkgtest.ubuntu.com/packages/c/content-hub/wily/amd64/ :)09:43
pittijjohansen: so, there ^ you can look at the debci log09:43
pittiyou see in the "base system diff" that the kernel did not change, but apparmor changed from 2.9 to 2.1009:44
pitti(also, python3.4, but that's most probably not the cause)09:44
jjohansengot it09:44
pittithis is a much more useful view than the old jenkins stuff09:44
pittisbeattie: please let me know if you have trouble reproducing it (using adt-run etc.)09:45
* pitti runs it locally for fun09:46
pittiand yay for tests which just say "FAIL" without saying what/why09:46
pittiadt-run --apt-pocket=proposed -U content-hub --- qemu adt-wily-amd64-cloud.img09:48
pittisbeattie: ^ reproduces locally; I suggest running with -s to get a shell after the failure09:49
Laneydoko: why gnome?10:04
Laneydoko: this is a touch component10:04
dokoLaney, why not? it was a question10:04
Laneywell if you have a reason then I want to know so that I can see it :)10:04
dokoLaney, please could you merge the new glibmm2.4 into silo 39?10:04
dokoseb128 isn't yet here10:05
Laneyi.e. if it's something I might be able to fix easily10:05
Laneyumm10:05
Laneyyou mean 2.45.41.is.2.44.0-0ubuntu1?10:05
dokoprobably, the ppa page says: glibmm2.4 - 2.45.41-0ubuntu3~gcc5.2 (Newer version available)10:06
Laneyis it reapplying this diff https://launchpadlibrarian.net/213023058/glibmm2.4_2.45.41-0ubuntu1_2.45.41-0ubuntu3%7Egcc5.2.diff.gz ?10:06
dokoyes, looks ok10:07
Laneyok, will do10:07
dokoand then could you stop uploading stuff for 4-6 hours, which is in the ppa?10:07
LaneyI know you want to push it today, won't upload anything :)10:08
Laneymaybe mail -devel to tell everyone else10:08
cjwatsondoko: do you want auto-sync disabled now then?10:13
dokoLaney, sent10:15
dokocjwatson, yes, would be good10:15
cjwatsondoko: done10:16
caribouI'm looking for a sponsor for the merge of rsyslog (Bug: #1464201)10:44
ubottubug 1464201 in rsyslog (Ubuntu) "Please merge rsyslog 8.9.0-3 (main) from Debian unstable (main)" [Wishlist,Confirmed] https://launchpad.net/bugs/146420110:44
caribouI'm returning from vacation near feature freeze so I'd like to see this one in Wily so it gets shaken out before our next LTS10:44
=== andyrock_ is now known as andyrock
Laneydoko: glibmm2.4 merged10:52
dokoLaney, ta, and I had to do a glib-networking rebuild in the ppa11:01
dokorenamed libproxy11:01
Laneydoko: will you NMU these renames?11:03
dokoLaney, nmu where?11:04
Laneyunstable11:04
dokosurely not all, need to write an email first ... want to call for "adopt a library transition"11:04
=== MacSlow is now known as MacSlow|lunch
=== _salem is now known as salem_
=== MacSlow|lunch is now known as MacSlow
pittiLaney: hmm, still stunned about https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-wily/wily/amd64/u/ubuntu-system-settings-online-accounts/20150731_085942@/log.gz13:32
pittiLaney: it looks like cloud-init isn't done yet at this point and the host name is still "ubuntu", i. e. it didn't get around to updating it yet13:32
Laneypitti: it's happening through multiple retries?13:33
pittibut we wait until /var/lib/cloud/instance/boot-finished exists, so it should be done13:33
Laneywhy should that be package specific?13:33
pittiLaney: http://autopkgtest.ubuntu.com/packages/u/ubuntu-system-settings-online-accounts/wily/amd64/ (and i386) failed four times in a row yesterday13:33
pittiLaney: but if I run it manually on teh worker it succeeds13:33
pittiLaney: yeah, no idea13:33
pitti(package specific)13:33
pittiI suppose it isn't really, as it happened to other runs too13:33
Laneycould just be unlucky13:34
Laneyyou mean if you run the same adt-run command it works?13:34
pittie. g. https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-wily/wily/i386/r/ruby-rails-assets-jquery-textchange/20150730_165419@/log.gz13:34
pittiright13:34
pittiit fails fast, as adjusting the apt sources (from --apt-pocket=proposed) is pretty much the first thing it does13:34
pittiI'm running a loop with /home/ubuntu/autopkgtest/tests/testpkg-simple/ (which is very quick), to see whether I can hit it13:35
pittiLaney: just wondering if you happen to have an idea abou that13:35
pitti    if ! timeout 30m $SSH 'while [ ! -e /var/lib/cloud/instance/boot-finished ]; do sleep 1; done'; then13:35
pittiI don't see what's wrong with that13:35
Laneywhy do you think that's the problem?13:40
Laneyisn't it that this hostname error causes sudo to fail?13:40
pittithe sudo error points out that /etc/hostname is "ubuntu", not the correct host name yet13:40
pittior cloud-init fails, but then it shuoldn't create /var/lib/cloud/instance/boot-finished13:41
Laneypitti: maybe some clue on the console?13:47
pittiLaney: I guess my first step will be to make it fail with 16 instead of 20, and then think how the ssh setup script can do nova console-log only in such failure cases13:57
pittimeh, restarted it again, failed again; so at least it's moderately reproducible13:58
=== adrianalves is now known as adrian
synthoris a kernel-dev here?14:20
pittithat's a trick question -- better just ask your actual question14:20
pittisynthor: ^14:20
synthorthe latest 14.04 kernel 3.13.0-59 causes segfaults14:20
synthor3.13.0-58 seems to be fine14:20
synthorJul 31 16:17:52 prosyn kernel: [11720.503201] ld[28708]: segfault at bae6b2d0 ip 00000000bae6b2d0 sp 00000000ea4e7d80 error 14 in ld-2.19.so[2b55bae6a000+23000]14:20
synthorwhile compiling cyanogenmod14:21
synthorif no segfault occurs, it'll create broken pipes14:21
synthorrebooting the old kernel, everything runs fin14:21
synthore14:21
pittisynthor: can you please report this as a bug, with the complete dmesg and reproducer?14:22
pittiif possible, attach just a single ld command with pre-compiled object files that triggers this, as "build cyanogenmod" is not a very tractable (and well-defined) reproducer14:22
=== dholbach_ is now known as dholbach
synthori'll check the makefiles for the command which triggers the bug14:24
pittisynthor: if building doesn't show the full command line, but uses "quiet" output, running "make V=1" often helps14:24
pittithen, if you can reproduce the crash with just running the single ld that it runs -> win14:25
synthorclang: error: unable to execute command: Segmentation fault (core dumped)14:25
bdmurraypitti: is there any reason the security SRU of apport, version 2.14.1-0ubuntu3.11, would be producing less informative crashes about /sbin/ip from iproute2?14:33
synthorhard to track it down....those android makefile-system is just a monster ^^14:39
synthori greped the calls, but now i have to look which programs they're callin14:39
pittibdmurray: /bin/ip is not a suid/sgid program, behaviour for "normal" 755 executables shouldn't have changed14:40
pittibdmurray: i. e. I don't see a reason14:40
bdmurraypitti: Could it be called another program?14:44
pittibdmurray: what does "less informative" mean here? the security update would cause suid/sgid programs to not get a report at all, or a root owned one, i. e. you wouldn't get a bug report unless a user is an admin14:45
pittiis it that?14:45
pittithe update doesn't have any impact on the quality of stack traces if you mean that14:45
bdmurraypitti: The reports for iproute2 are incomplete - particularly missing StacktraceAddressSignature14:46
bdmurraypitti: example here https://errors.ubuntu.com/oops/1b8c3c20-3607-11e5-89c7-fa163e525ba714:47
pittibdmurray: do we have a core dump for those? maybe it's truncated or even 0 bytes?14:49
dokoslangasek, ready to pull the trigger? and start the copy?14:50
pittigo, gcc, go!14:50
bdmurraypitti: no, we don't ask for a coredump if there is no SAS and its not on armhf14:52
dokostarted the copy now ...15:00
* Laney hears the screams of britney from here15:10
ogra_teach her how to sing then !15:10
pittiLaney: hah!15:15
=== FJKong is now known as FJKong_afk
* Laney hopes that this is a eureka moment hah15:15
pittiLaney: http://paste.ubuntu.com/11973890/15:15
Laneyhahaha15:15
pittiI thought POSIX would allow up to 255 chars15:16
pitticloud-init seems to think differently15:16
Laneyseems to be passing through an error from hostname?15:16
pittiactually not c-i, it's hostname itself15:16
dokocopy is finished15:16
pittiLaney: it seems the limit is 64 chars15:17
Laneywhy didn't this happen when you ran it yourself?15:17
bdmurraypitti: so no ideas about iproute2 crashes other than truncated crashes?15:17
pittiLaney: I used --name adt-test, to avoid interfering with the running tests; silly me :)15:17
Laneyhehe15:17
pittibdmurray: no, I'm afraid; I tried "ip route & killall -SEGV ip" in a loop, but that's too slow to crash it; I figure one had to re-compile ip to force a crash and see what happens15:18
bdmurraypitti: I used "ip monitor" and got a good crash15:19
pittiLaney: that also explains ruby-rails-assets-markdown-it-diaspora-mention and the other funny long one :)15:20
pitti"ones"15:20
Laneyis it always setting the machine name to the instance name?15:22
Laneycan't think that the hostname matters so much, unless I'm missing something15:22
pittiyeah, would probably be even better to just hardcode it to "ubuntu"15:22
pittithis seems to be the default, maybe c-i has some "hostname:" option15:23
Laneylooks like it does15:24
pittismoser: ^ it seems the host name defaults to the cloud instance name given to nova boot; is there a c-i option to change it? (nothing in http://cloudinit.readthedocs.org/en/latest/topics/examples.html)15:25
pittiChangeLog: - added cloud-config option 'hostname' for setting hostname15:25
pittiah!15:25
smoserpitti,  you can also do 'manage_etc_hosts: localhost'15:26
smoserwhich will make sudo not cry at you "unknown host" business.15:26
pittiah, instead of "true"?15:26
pittiI have "manage_etc_hosts: true"15:26
pittifor exactly that reason15:26
smoserah. maybe thats right. let me read doc that is there.15:27
pittismoser: but if I give my instance a very long name, it'll overflow hostname's 64 char limit15:28
pittismoser: I don't need that, just "adt" or so is enough as hostname; I just want to be able to identify my cloud instances from "outside"15:28
smoserhttp://bazaar.launchpad.net/~cloud-init-dev/cloud-init/trunk/view/head:/doc/examples/cloud-config.txt#L44715:28
pitticool, "hostname: adt" works fine15:29
pittithanks smoser15:29
smoserfor identifying them, if you launched them on openstack, you can provide that hostname at launch time15:29
smoserin fact you're required to provide one.15:29
pittiI didn't explicitly give one, it just seemed to default to the name of "nova boot"15:30
smoserah. ok. i thought that nova boot required one.15:30
smosernova boot --key-name=brickies --flavor=m1.small --image=b9a4cbd6-82ae-4d53-8231-dc956aca100d smoser-hostname15:30
pittismoser: right, it does15:30
pittismoser: but my name is e. g. adt-wily-amd64-ubuntu-system-settings-online-accounts-20150731-15045615:31
pittiand that causes http://paste.ubuntu.com/11973890/15:31
smoserah.15:31
pittiLaney: ok, fixed and retrying tests15:32
pittiand with that, good weekend everyone!15:32
Laneypitti: prima! schönes Wochenende!15:34
=== soee_ is now known as soee
=== Malsasa_ is now known as Malsasa
=== superm1_ is now known as superm1
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== _salem is now known as salem_
=== salem_ is now known as _salem

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