/srv/irclogs.ubuntu.com/2014/06/27/#ubuntu-ci-eng.txt

=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
=== fginther is now known as fginther|away
=== fginther|away is now known as fginther
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
rsalvetipopey: are you able to reproduce bug 1334940?03:26
ubot5bug 1334940 in mediascanner2 (Ubuntu) "Can't scan videos on ubuntu touch (flo, image 100)" [Undecided,New] https://launchpad.net/bugs/133494003:26
=== wgrant_ is now known as wgrant
Saviqsil2100, can we kick M&C on silo 5 do you think?07:16
tvossgood morning07:23
tvosssil2100, can I get a silo for line 28?07:24
sil2100o/07:31
sil2100tvoss: let's see if there will be no conflicts07:35
tvosssil2100, there most likely will be, but it's really only build system setups07:35
tvosssil2100, also note that the list of mps is not complete, yet07:35
sil2100tvoss: anyway, silo 008 is for you07:56
tvosssil2100, thank you07:56
tvosssil2100, I will add a bunch of MPs now, and ping you once done to ask for a reconfigure07:57
sil2100tvoss: ok :)08:02
=== sil2100 changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cihelp | CI Train Status: #87 promoted | CI Train Sheriff: sil2100 | Known issues: The spreadsheet is extremely slow, http://people.canonical.com/~rbpark/citrain/ knows the statuses before the spreadsheet does. Long mako wait times on s-jenkins.
sil2100davmor2: hey, meeting?08:31
seb128sil2100, thanks for handling that settings landing! seems like you can m&c it now as well08:51
t1mpany ideas why jenkins is so slow these days? It takes >10h for a happroved MR to land in a branch08:52
mhr3psivaa, ping?09:04
mhr3psivaa, would you happen to know anything about foo:native not working on jenkins?09:04
ogra_t1mp, there was a mail09:05
psivaamhr3: do you have a link for the job09:06
mhr3psivaa, here's one https://jenkins.qa.ubuntu.com/job/unity-team-unity-scope-click-devel-utopic-amd64-ci/129/console09:06
mhr3psivaa, another https://jenkins.qa.ubuntu.com/job/mediascanner2-utopic-amd64-ci/38/console09:07
mhr3psivaa, and there will be few more09:07
t1mpogra_: ah yes, I see now.09:10
t1mpthanks.. I hope a solution can be found09:10
oSoMoNsil2100, hey, would you mind publishing silo 13?09:11
tvosssil2100, could you reconfigure silo 27 please?09:25
sil2100o/09:27
sil2100One moment please and I'm handling your requests09:27
LaneyYour call is important to us09:28
tvosssil2100, can I push build and see the world explode?09:30
tvosssil2100, pushed the button :)09:30
psivaamhr3: sorry, could not find much info as to why it's failing except:09:36
psivaash: 1: x86_64-linux-gnu-gcc-4.9: not found09:36
psivaadpkg-architecture: warning: couldn't determine gcc system type, falling back to default (native compilation)09:36
mhr3psivaa, hmm, wouldn't you anyway get that error *after* the dep checks?09:37
mhr3psivaa, plus it's not really an error... just a warning :)09:38
popeysil2100: ogra_ i can't land camera it in the store right now because the frameworks has been updated..09:39
psivaamhr3: ok, but i dont see that in passing jobs.09:40
ogra_to something non existing ??09:40
popey-dev2 was pushed to click and the camera app uses it, but the docs, the store and the reviewers tools aren't updated09:40
popeyexists on the device, yeah09:40
popeybut nowhere else09:40
popeylooks like a conversation in #phablet last night09:40
popeywish people would use the public channels09:40
psivaamhr3: the installed version of gcc in the host is: 4.8.2-19ubuntu1, if that points to anything.09:41
mhr3psivaa, well what changed is the addition of gcc-4.9:native to the control file, and apparently pbuilder doesn't like that, i've heard that you were stripping the native suffix because of that, but i guess that stripping isn't global?09:42
ogra_sigh09:42
psivaamhr3: i have no idea about that :), sorry. i'll ask fginther when he's around.09:42
* sil2100 sighs09:42
psivaatoo many sighs :)09:43
mhr3psivaa, ok, thanks for looking into it anyway09:43
* mhr3 sighs too09:43
sil2100tvoss: the reconfigure has failed it seems, let me reconfigure again09:43
tvosssil2100, hmmm, the silo is building09:43
psivaamhr3: np.09:43
sil2100tvoss: ok, it might not have built all the components then09:44
sil2100tvoss: let me re-run it in a moment09:44
tvosssil2100, yup, the build failed09:44
ogra_popey, ah, to prevent it from being offered to images with older frameworks09:45
popeyyes09:45
ogra_k09:45
ogra_do we have any ETA for the update of the tools ?09:45
ogra_(and docs)09:45
sil2100tvoss: ok, the build failed also from a different reason, let me try and get to the source of it09:47
popeyogra_: jdstrand owns the docs and tools, asked james for ETA on store09:47
ogra_thanks09:47
sil2100hmmm09:49
sil2100tvoss: so, it seems that the unity-scope-click merge proposal branch targets lp:unity-scope-click/devel which is not up to sync with the archive09:49
sil2100tvoss: not sure if lp:unity-scope-click/devel should be used? What is the main branch for this project? I see lp:unity-scope-click being in sync with the archive09:51
sil2100I think you'll have to ask the author to retarget the merge09:55
sil2100ogra_, popey: you think we should wait with 102 for that to happen?09:56
ogra_sil2100, no, lets build one09:56
sil2100Ok then, let's kick a new image09:57
popey09:57
popeyogra_: before you do..09:58
ogra_yeah ?09:58
popeyany chance you can add my libkeychain to the image?09:58
ogra_that will cost another publisher run, but yeah, i can09:59
popeyhttps://code.launchpad.net/~popey/ubuntu-seeds/add-keychain/+merge/22466709:59
popeythanks09:59
tvossthostr_, ^ please see sil's comment10:00
=== tvoss is now known as tvoss|food
cjwatsonmhr3: As I noted in whatever unity-scope-click MP it was, I really think the advice to use g++-4.9:native is wrong - even though there does seem to be something suspicious here going on in jenkins.10:10
mhr3cjwatson, well pbuilder not supporting :native is what's wrong10:14
mhr3cjwatson, i'll let you fight with xnox about the need for :native, imo explicitely picking gcc version on individual packages like this is already what's wrong :P10:15
xnoxcjwatson: well the intention is to protect from c++11 abi breaks.10:17
xnoxcjwatson: yet, without breaking cross compilation....10:17
xnoxcjwatson: not add that dependency at all?10:18
* xnox ponders where pbuilderjenkins is maintained.10:18
fgintherpsivaa, mhr3, the ':native' stripping is not global. I'm applying it to unity-scopes-api now10:33
mhr3fginther, well, it would need to be applied to about ~15 projects10:34
mhr3at least10:34
psivaafginther: ack, thanks. i'll take a look at it later for future ref :)10:36
cjwatsonmhr3: No, there are excellent reasons to control the timing of C++11 ABI changes10:37
cjwatsonxnox: The thing we need to do to make explicit compiler build-dependencies work for cross-building isn't to use :native, but to have sbuild know when to turn g++-4.9 into g++-4.9-$triplet10:39
cjwatsonmhr3: Thomas is working on that set of packages, after discussion with me and Steve10:39
cjwatsonxnox: g++-4.9:native doesn't help the cross case - sure, it'll give you a compiler that executes on the build architecture rather than on the host architecture, but it'll also target the build architecture so it's no good10:41
cjwatsonxnox: I've been over this extensively with Wookey :-)10:41
mhr3cjwatson, i know, i just don't like that they're even needed... anyway that doesn't mean that half of unity stack should stop being cross buildable10:41
cjwatsonxnox: I agree that this set of changes regresses cross-compilation, but realistically that's less important than controlling ABI changes in this case10:42
cjwatson(Well, may regress; I don't know what the current state of cross-building for this particular set of packages is)10:42
cjwatsonmhr3: Like I say, we can and should fix this centrally10:42
mhr3cjwatson, and meanwhile any development can suffer?10:42
cjwatsonmhr3: This is just making a well-known problem cover a few more packages10:42
cjwatsonmhr3: Um, development suffers either way, this change isn't being made for no reason!10:43
cjwatsonmhr3: We had to revert a toolchain defaults change until this was sorted out10:43
cjwatsonmhr3: There is no currently-available method that both preserves cross-building and permits control of C++11 ABI changes across g++ version changes, to the best of my knowledge.  You could always not use a C++ version with experimental toolchain support :P10:45
cjwatson(But that ship has sailed, I know)10:45
xnoxcjwatson: yes, i am aware that there is no way to declare cross-compilers as a build-dependency. Without adding any dependency the package will fail to build from source, given the export in the debian/rules. The extra dependency helps to keep native build working even after g++4.9 -> g++4.10 default change, but will not help x-building.10:45
fginthercjwatson, For CI, we have to strip out :native as pbuilder doesn't parse it correctly even though we are already doing native builds10:46
cjwatsonfginther: yes, I know, but that apparently isn't working quite right in all cases, given this conversation10:46
fginthercjwatson, this is applied as a hook and currently only to the projects when needed10:46
cjwatsonfginther: https://jenkins.qa.ubuntu.com/job/unity-team-unity-scope-click-devel-utopic-amd64-ci/129/console10:46
xnoximho, just adding the export of CC/CXX in debian/rules is suffcient to guarantee package builds against expected compiler abi.10:46
cjwatsonfginther: aha, why not globally?10:47
cjwatsonxnox: No, because the point is to decouple these changes from our toolchain default switch10:47
fginthercjwatson, it was new and I didn't want to introduce it global until it was needed globally10:47
cjwatsonxnox: That would make it impossible to continue building against g++-4.8 after we switch to g++-4.9, for instance, or vice versa10:47
cjwatsonxnox: Which is the point (not to do that for an extended period of time, but to give those projects control over when they switch because they have to coordinate SONAME changes and such)10:48
fginthermhr3, cjwatson, it's been working fine so far, so I see no reason to not apply it to all projects. It's just that it's time didn't come until today :-)10:48
xnoxcjwatson: right. There was proposal emailed to allow specifying cross-compilers, such that they are appropriately remapped using a build profile something a keen off "gcc-4.9 <!cross-building>, ARCH-gcc-4.9 <cross-building>"10:48
cjwatsontvoss|food: ^- you might want to be aware of the above conversation10:49
xnoxbut exact semantics have not been implemented yet.10:49
xnoxI think we will be discussing this at the Debconf.10:49
cjwatsonxnox: right, exactly, that's what I discussed with Wookey years ago10:49
xnoxack. so same thing.10:49
cjwatsonxnox: AFAIK the only thing that needs to be sorted is the way to tell which packages this should be applied to; I think I suggested a control field in the various relevant compiler packages, although I forget the name we settled on10:50
cjwatson(and of course implementation in dpkg-checkbuilddeps and sbuild)10:50
brendandogra_, you won't believe the explanation for the autopilot bug :)10:50
tvoss|foodcjwatson, ack and thanks10:50
cjwatsonbrendand: this one weird autopilot bug, discovered by a mom10:51
brendandcjwatson, earn $$$ without doing anything!!!10:52
brendandogra_, here's the guilty landing http://paste.ubuntu.com/7710576/10:52
brendandogra_, start with #9510:52
brendandogra_, then install qtubuntu-sensors10:53
brendandogra_, run the test suite and autopilot gets dbus *errors*10:53
=== oSoMoN__ is now known as oSoMoN
brendandogra_, then install ubuntu-application-api2-touch and they turn from errors into delays10:53
brendandogra_, so i guess they tried to fix the dbus error incorrectly10:54
brendandogra_, i do wonder though why autopilot has to scan the whole bus10:54
brendandogra_, i need to ask them that10:54
brendandogra_, so what can we do? revert that? i guess we need to get ricmm and/or rsalveti involved10:59
=== MacSlow is now known as MacSlow|lunch
ogra_brendand, show it to ricmm11:24
sil2100brendand: \o/11:25
brendandogra_, ok11:26
sil2100brendand: yeah, as per ogra_'s proposition, let's wait for ricmm to comment on this11:26
sil2100ricmm: ^11:26
h0x15Suggestion: some html page with an graphic representation of number of failures/crashes vs build number. Something like http://ci.debian.net on ci.ubuntu ?11:40
ogra_h0x15, do you mean http://ci.ubuntu.com/smokeng/utopic/touch/ ?11:41
ogra_or more detailed http://ci.ubuntu.com/smokeng/utopic/touch/mako/99:20140626.1:20140625/8744/11:42
h0x15yes, but visual grafig11:42
h0x15chart11:43
thostr_could somebody reconfigure silo *11:44
thostr_silo 8 that is11:44
ricmmsil2100: so whats the issue?11:45
ricmmogra_:11:45
ogra_ricmm, seems our testing got extremely slow sinse papi211:45
ogra_*since11:45
ogra_or rather with the new qtubuntu-sensors11:46
ricmm95 is the orient11:46
ricmmnot papi211:46
ogra_see brendand above11:46
ogra_right, sorry11:46
* sil2100 off to lunch11:47
ricmmbrendand: extremely slow in what way? do you have a bug number with the information?11:47
ogra_ricmm, the UITK autopilot tests turned into a several hour job ... there is a dbus timeout that adds ~30sec per loop11:48
brendandricmm, https://bugs.launchpad.net/autopilot/+bug/133467611:48
ubot5Ubuntu bug 1334676 in ubuntu-touch-meta (Ubuntu) "Applications have a dead open dbus connection (since image #96)" [Critical,New]11:49
ogra_all other AP tests got slow too11:49
thostr_trainguards: can anybody reconfigure silo 811:50
ricmmbrendand: and reverting the changes from the usensord landing fixes it?12:01
ricmmit might be dbus-cpp which we use to reach the usensord service12:01
ricmmwhich became apparent now that usernsord is a dbus service12:02
ricmmI'll examine after lunch if you dont mind12:02
ricmmas its getting cold12:02
brendandricmm, yes12:02
=== alan_g is now known as alan_g|lunch
popeyogra_: sil2100 camera updated in store12:20
thostr_sil2100: can you reconfigure silo 812:24
tvossdavmor2, around?12:30
tvossmandel, oSoMoN you fine with me setting silo 11 to tested?12:30
davmor2tvoss: I am12:35
tvossdavmor2, hey there :)12:35
sil2100thostr_: sure12:35
sil2100popey: \o/12:35
sil2100thostr_: I'm on lunch now so it's taking a bit longer for me to notice pings :)12:36
tvossdavmor2, I filed the issue you encountered in bug https://bugs.launchpad.net/location-service/+bug/133510212:36
ubot5Ubuntu bug 1335102 in platform-api "Stopping positioning updates/the gps provider should release wake lock" [High,Confirmed]12:36
tvossdavmor2, mandel, oSoMoN are you guys then fine with me setting silo 11 to tested?12:36
davmor2tvoss: yeap everything works in 011 and it doesn't really break anything.  Except for killing the battery :)12:36
tvossdavmor2, ack, thank you12:36
mhr3is something wrong with s-jenkins?12:47
mhr3can't reach it12:47
=== renato_ is now known as Guest94157
=== alan_g|lunch is now known as alan_g
mhr3if only i knew its ip12:58
popeymhr3: $ host s-jenkins.ubuntu-ci12:59
popeys-jenkins.ubuntu-ci is an alias for mayura.ubuntu-ci.12:59
popeymayura.ubuntu-ci has address 10.98.3.1312:59
popeymhr3: working fine here13:00
oSoMoNtvoss, yup, as far as I’m concerned silo 11 is good to go13:00
mhr3popey, thx, works via ip here too, just no dns it seems13:00
mhr3i'll blame the upgrade that i just did13:01
ricmmtvoss: ping13:05
=== josepht changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: josepht | CI Train Status: #87 promoted | CI Train Sheriff: sil2100 | Known issues: The spreadsheet is extremely slow, http://people.canonical.com/~rbpark/citrain/ knows the statuses before the spreadsheet does. Long mako wait times on s-jenkins.
jdstrandogra_ (and popey): tools are fine (we don't have to change anything atm-- we will in the future so keeping them in mind with new frameworks is a good idea). I used to be able to edit the docs, but part moved out to a spreadsheet that I can't edit. I've asked lool to update it13:07
popeysweet, thanks13:08
imgbot=== trainguard: IMAGE 102 building (started: 20140627 13:15) ===13:15
alan_gjosepht: there seems to be something odd with https://jenkins.qa.ubuntu.com/job/mir-team-mir-development-branch-autolanding/13:16
alan_gI'd expect it to correspond with the "ready to land" in https://code.launchpad.net/mir/+activereviews13:16
alan_gBut 804 is still in the latter and the other six in the latter are not in the former.13:16
alan_gAny idea what's going on?13:16
kgunnUrsinha: ^ sorry if i'm a redundant ping13:16
josephtalan_g: let me take a look13:17
mhr3fginther, so is the :native stripping going to be global?13:18
=== stgraber changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: josepht | CI Train Status: #87 promoted | CI Train Sheriff: sil2100, stgraer | Known issues: The spreadsheet is extremely slow, http://people.canonical.com/~rbpark/citrain/ knows the statuses before the spreadsheet does. Long mako wait times on s-jenkins.
jdstrandfyi, just sent to the list. I did an emergency no-code-change upload for bug #1334940, but that has an unrelated testsuite failure13:26
ubot5bug 1334940 in mediascanner2 (Ubuntu) "Can't scan videos on ubuntu touch (flo, image 100)" [Critical,Fix committed] https://launchpad.net/bugs/133494013:26
josephtalan_g: it looks like the generic-land job failed to aquire a lock on bzr+ssh://ps-jenkins@bazaar.launchpad.net/~mir-team/mir/development-branch/13:27
jdstrandI have alerted Satoris, who is looking into it13:27
josephtalan_g: I'll try re-running that build13:27
sil2100davmor2: once 102 is built, could you take a look at it promotion-wise?13:28
davmor2sil2100: if the issue with not displaying music or videos is still there then no13:28
alan_gjosepht: that may take some time.13:29
alan_galf__: ^^13:31
sil2100davmor2: you mean the mediascanner bug?13:34
josephtalan_g: sorry, I meant the generic-land build, it's pretty quick and it failed again trying to acquire the lock.13:35
davmor2sil2100: yes13:35
davmor2sil2100: people having no access to their music or videos kinda sucks13:35
sil2100davmor2: I see a fix in the archive... I wonder if it made it to this image13:36
sil2100Of course it didn't make it ;/13:36
sil2100As it's still in -proposed13:36
davmor2sil2100: it failed to build see mailing list team are on it now I think13:37
alan_gjosepht: weird. I just tried running break-lock --verbose ... it reported nothing13:37
sil2100Ah, right, email from 15 minutes ago13:38
sil2100Ok, so I'm starting to feel pressure here...13:38
josephtalan_g: Unable to obtain lock  held by alan-griffiths@bazaar.launchpad.net on taotie (process #28697), acquired 20 hours, 57 minutes ago.13:39
alan_gjosepht: That's me. But...! I'll try merging that MP locally and push it - see what happens then13:40
josephtalan_g: hold on13:40
mandeltvoss, +1 from me for silo 1113:42
josephtalan_g: it seems to have merged that time13:42
sil2100ricmm: how's the dbus-related regression analysis proceeding?13:42
alan_g\o/13:42
ricmmsil2100: we are looking into it13:45
alan_gjosepht: thanks. Any idea why jenkin's hasn't picked up the other jobs?13:45
ricmmsil2100: but no definitives. it should be related to using dbus-cpp to reach usensord13:45
ricmmtrying to figure out which bit is actually timing out13:45
josephtalan_g: it also looks like jenkins has kicked off builds for the others as well13:45
ricmmbrendand_: do you know exactly what it is that autopilot is trying to do?13:45
brendand_ricmm, the bug should say. but if it's not clear i can describe it in more detail13:46
alan_gjosepht: where do you see that?13:46
josephtalan_g: s-jenkins13:46
alan_gjosepht: Ah. I was looking at https://jenkins.qa.ubuntu.com - and that doesn't show the pending ones.13:48
ricmmbrendand_: describe it in more detail, what exactly is autopilot trying to do by probing into connection peers?13:48
alan_gjosepht: thanks for the help13:48
ricmmbrendand_: to find it is a testability interface?13:48
ricmmif it is*13:48
josephtalan_g: my pleasure13:48
ricmmbrendand_: also, got a test that fails every time?13:49
ricmmor is it all of them13:49
brendand_ricmm, yes it seems to be trying to find if that connection has the introspection interface13:49
brendand_ricmm, i run ubuntuuitoolkit. using --verbose13:50
ricmmbrendand_: ok13:50
brendand_ricmm, you should see it hang for ~30 seconds, and then the error described13:50
brendand_ricmm, i think this log from pitti might be helpful: http://paste.ubuntu.com/7709407/13:51
=== MacSlow|lunch is now known as MacSlow
ricmmbrendand_: right, and the bug there is what, the app exit behaviour?14:02
ricmmI'm not seeing any timeouts in that log14:02
brendand_ricmm, the messages at the bottom14:03
brendand_ricmm, it seems to suggest a problem closing the connection14:03
ricmmthose only show when the application exits14:03
brendand_ricmm, ok i won't read too much into it then14:04
brendand_ricmm, i'll leave it to you. if you have any more questions for me or need me to try anything, let me know14:04
ricmmbrendand_: so if its the usage of the dbus usensord interface, reverting qtubuntu-sensors to anything before 24.3 should fix it14:04
ricmmis that correct?14:05
brendand_ricmm, well doing qtubuntu-sensors:armhf from 0.6+14.10.20140623.3-0ubuntu1 to 0.6+14.10.20140624.3-0ubuntu1 broke it14:06
brendand_ricmm, i didn't try then reverting it again14:06
brendand_ricmm, i can do that now14:07
ricmm23.3 fixes it just checked14:07
brendand_ricmm, read my last comment on the bug though. if you *just* upgrade qtubuntu-sensors then it is broken in a different, less subtle way14:07
brendand_ricmm, ok cool14:07
ricmmif you just upgrade qtubuntu-sensors the API wont be available from platform-api14:08
brendand_ricmm, yeah14:10
brendand_ricmm, so is it that the new api is broken then?14:10
ricmmthe API is using dbus-cpp to open a connection to the vibration service14:10
ricmmsomething there is setting itself up in a way that it baffles autopilot14:11
ricmmthomas will help in a minute as he knows that code best14:11
ricmmit'd be great if I could know the exact autopilot lines that are doing this probing14:11
ricmmwhich them timeouts14:11
brendand_ricmm, you can reproduce it without AP like this: http://pastebin.ubuntu.com/7709477/14:13
brendand_ricmm, using qdbus14:13
brendand_ricmm, i could give you the autopilot lines as well if you want14:14
ricmmbrendand_: ah thats amaizing :)14:14
brendand_ricmm, but probably easier with those steps14:15
ricmmsure, but this looks good enough14:15
ricmmbrendand_: the one thing I dont get is why autopilot needs to probe into the application's connections14:17
ricmmis that to see if the application connected back to autopilot? if it finds itself in the tree?14:17
fginthermhr3, yes, it will be made global. It will take a day or two to roll it out, so if another project needs it right now, we'll need to update it individually.14:19
brendand_ricmm,  i need to find that out. but i don't think it's worth getting hung up on it. it can't be good that some dead dbus connection is being left around like that, right?14:23
ricmmbrendand_: its not a dead dbus connection, its a timing out introspect call14:27
ricmmbrendand_: the bug lies either with usensord or dbus-cpp14:27
ricmmbrendand_: I just want to know why the need to introspect the tested app's connections to find itself14:27
=== alan_g is now known as alan_g|tea
jdstrandhey, we need a/win 3214:29
jdstrandmeh14:29
plarscjwatson, fginther, ev: I think we can skip the hangout if you want. I just sent an email. cjwatson: your response was very helpful and I think I have a better idea of what is needed.14:31
boikosil2100: hi, could you please reconfigure landing-006? renato just told me there are some extra MRs needed for that one14:32
evcool14:32
cjwatsonplars: Ah, good timing, I was just about to engage in battle with audio14:33
fgintherplars, ack14:33
sil2100boiko: sure14:33
cjwatsonplars: Cool14:33
jdstrandhey, can I have a silo for mediascanner2 (line 31)14:35
sil2100boiko: reconfigured14:35
jdstrandit is a no-code-change update to bring the packaging in sync with my emergency upload from today and then a ftbfs fix on top of that. the fix is to disable a test that is broken due to a change in sqlite behavior14:36
=== alan_g|tea is now known as alan_g
jdstrandthat test failure doesn't indicate that mediascanner2 is broken (I'm told) and simply needs to be updated14:36
sil2100tvoss: ok, so we might need to rebuild location-service from silo 11 - the changelog is too poor... there is a lot of changes but they're not mentioned in the changelog :|14:37
sil2100The core devs would kill me if this would go to the archive14:37
sil2100Especially oSoMoN14:37
sil2100I mean14:37
sil2100ogra_:14:37
* sil2100 has some typing problems it seems14:37
sil2100oSoMoN: ignore my ping ;)14:37
* ogra_ hopes oSoMoN also check for proper changelog though ;)14:38
* oSoMoN can also kill14:38
oSoMoNogra_, not just improper changelogs, also poor commit messages14:39
ogra_yeah14:39
jdstrandnote, this mediascanner2 landing is crucial for the mediascanner2 to be able to scan on the device, because the apparmor fix to fix that can't land without the fix for the ftbfs14:42
stgraberjdstrand: you got silo 714:42
jdstrandthanks!14:42
stgraberAlbertA: you've got silo 1314:43
sergiusenssil2100: hey, I don't understand this error; care to explain?14:43
sil2100sergiusens: hey! Which one?14:44
sergiusenssil2100: lolz; Friday woes :-) https://ci-train.ubuntu.com/job/landing-002-1-build/120/console14:44
sergiusensforgot to paste the link :)14:44
imgbot=== trainguard: IMAGE 102 DONE (finished: 20140627 14:45) ===14:45
sil2100ohoho, oh my14:45
imgbot=== changelog: http://people.canonical.com/~ogra/touch-image-stats/102.changes ===14:45
sil2100This is one absurd error14:45
sil2100sergiusens: I noticed you have a gift of making citrain confused!14:45
sil2100Let me dive into this one14:46
stgrabertvoss: libjsoncpp-dev and libnet-cpp-dev are universe packages, location-service is in main, so you'll need to file an MIR for each of them before I can let your package in14:46
sil2100Ah14:46
sil2100sergiusens: ok, I see the problem I think14:46
sil2100sergiusens: I mean... not sure if that's it, but it might lead to the confusion of citrain14:47
sil2100sergiusens: so, the problem seems to be that there is a mismatch between the lp-project name (lp:udm) and the source package name (golang-udm)14:48
tvossstgraber, ack14:48
sil2100sergiusens: the principle of all citrain (and also cu2d) project is that the lp project name needs to be the same as the source package name14:48
tvossstgraber, or alternatively patch out those dependencies, correct?14:48
stgrabertvoss: right, dropping those two dependencies works too :)14:49
sil2100sergiusens: this migth be the problem, and it's not really fixable in citrain as it's as I mentioned - it's a principle which CI Train enabled projects need to follow :)14:49
tvossstgraber, can I patch them out from the existing MR?14:50
stgrabertvoss: yeah, that should be fine, update the current MR, rebuild it in the silo and that should do the trick14:51
jdstrandsil2100: hrmm, failure to merge the source :\14:51
sil2100jdstrand: where?14:51
looljdstrand: I've updated the spreadsheet and gave you write acess14:51
tvossstgraber, I'm confused, isn't the unity-scopes-api in main?14:51
jdstrandsil2100: it merges fine here. I have a feeling it is because of the changelog. the MR updates the changelog to sync to what is in the archive, but then expects something more14:51
sil2100Ah14:52
sil2100Let me take a look14:52
jdstrandsil2100: should we update the changelog too?14:52
stgrabertvoss:  unity-scopes-api | 0.5.1+14.10.20140626-0ubuntu1 | utopic/universe | source14:52
stgrabertvoss: so, no14:52
tvossstgraber, ack and thx14:52
tvossstgraber, my bad then14:52
jdstrandsil2100: eg, to have ubuntu3?14:52
sil2100jdstrand: wait, from what I see the build failure in silo 007 is caused by something else?14:52
sil2100(we're talking about 007, right?)14:53
jdstrandoh, right, not approved14:53
jdstrandyes, let me address that14:53
sergiusenssil2100: we had this discussion once; and didrocks told us it was supported though14:53
stgrabersergiusens: it sure should be, otherwise we wouldn't be able to land any of the system-image stuff14:54
ricmmsergiusens: so we know the issue, go-dbus has no default error if the interfaces are no introspectable14:55
ricmmsil2100: ^14:55
ricmmsergiusens: that wasnt for you, was for sil14:55
sergiusensI assumed :-)14:55
ricmmso sergio will try his best, but ultimately this exposes an architectural issue with autopilot14:55
ricmmbrendand_: ^14:55
sergiusenssil2100: I'll fix that and you fix my issue ;) fair trade :-)14:55
ricmmI still dont understand why it needs to probe into the application's connections to check if the application has connected to autopilot14:55
sil2100sergiusens: ok, let me dig in deeper then ;) Didn't know Didier removed this restriction14:56
sil2100Ah14:56
sil2100hm14:56
sergiusensyeah, stgraber brings me some confidence that it should be working14:56
sergiusenssil2100: also, this is the first train landing for this14:56
sil2100Ok, I think I know what's up then ;) But let me double check first before I spout out another theory!14:56
sil2100Yeah, this might be the reason, I think citrain might be confused by the lack of tags14:57
sil2100sergiusens: could you try tagging in bzr the revision that's released into the archive?14:57
sergiusenssil2100: in trunk I asume; will do14:58
sil2100i.e. 0.1-0ubuntu1 in trunk, probably revision no 214:58
sil2100Yeah :)14:58
* sergiusens tries build15:01
sergiusenssil2100: still breaks15:03
sergiusenssil2100: I can force build I guess15:03
=== josepht changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cihelp | CI Train Status: #87 promoted | CI Train Sheriff: sil2100, stgraer | Known issues: The spreadsheet is extremely slow, http://people.canonical.com/~rbpark/citrain/ knows the statuses before the spreadsheet does. Long mako wait times on s-jenkins.
sil2100sergiusens: let me take a look again then15:04
brendand_ricmm, it might not be an 'issue' though - it might be essential for autopilot to do it, so please - keep looking on your side for a solution15:05
balloonsfginther, can you have a look at https://code.launchpad.net/~nskaggs/reminders-app/fix-evernote-sdk-python3/+merge/223493 again? I see the pep8 errors again that you tried to get rid of earlier in the week. I fixed the errors in the code we want to be checking15:05
brendand_ricmm, i really don't understand autopilot well enough to know if there is another approach that can be used. we need to wait for someone who does to come online15:06
sil2100Ah!15:06
sil2100sergiusens: I see the problem... it's a bit more trivial it seems15:06
brendand_ricmm, did you find any root cause yet?15:07
brendand_ricmm, is it the go-dbus thing you mentioned?15:07
sil2100sergiusens: so, only now I noticed that trunk still has 0.1-0ubuntu1 in the changelog as UNRELEASED15:07
sil2100sergiusens: so... please make a quick direct push to trunk where you change 0.1-0ubuntu1 to utopic and maybe re-tag this one to 0.1-0ubuntu115:08
sil2100sergiusens: it should work like a charm then15:08
sergiusenssil2100: ah, that can actually be it; is there a way to remove the tag easily though?15:08
sergiusensor only with --overwrite?15:09
sil2100sergiusens: I'm not sure... I know I did it a few times, but I don't remember if I didn't have to --overwrite in the end15:09
sergiusensack, I'll figure it out15:09
sil2100As it's a new project I guess even --overwrite wouldn't hurt I guess... still, you might try doing a bzr tag --delete and then re-tagging it and pushing without it15:10
sil2100Who knows? Maybe it'll allow it15:10
ogra_if you are the only committer --overwrite is ok15:10
fgintherballoons, did the packaging change recently? It looks like it's now looking in the temporary install path, should just add that to the exclude list15:10
balloonsfginther, yes, I needed to add those modules to the packaging.. failed to do that originally :-)15:11
fgintherballoons, I'll have to update the hook then, hang on15:12
ricmmbrendand_: we have the solution on our side, just saying that we need to figure out why autopilot is doing this, because it is certainly a hack to find if the client connected back to autopilot15:17
ricmmso this fix, and the subsequent autopilot investigation, are separate things15:17
ricmmbrendand_: the problem is that go-dbus doesnt provide introspection by default15:17
ricmmbrendand_: so the way to fix it right now is to manually add introspection to usensord, which is the offending service15:17
ricmmwe'll fix it np15:18
brendand_ricmm, sure - if autopilot doesn't *have* to use such a technique then it certainly shouldn't. like i said, i'll pursue it with them15:20
brendand_ricmm, you could comment on the bug with such analysis as well15:21
=== retoaded changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: retoaded | CI Train Status: #87 promoted | CI Train Sheriff: sil2100, stgraer | Known issues: The spreadsheet is extremely slow, http://people.canonical.com/~rbpark/citrain/ knows the statuses before the spreadsheet does. Long mako wait times on s-jenkins.
sergiusenssil2100: it's building now15:33
sil2100sergiusens: excellent15:33
sergiusensthanks15:33
tvosssil2100, seb128 to make sure: silo 8 shouldn't be blocking any other landings right now15:34
seb128tvoss, there is nothing in 8?15:35
seb128oh15:35
seb128it didn't even build15:35
seb128tvoss, no, it should not15:35
sil2100davmor2: hm, and what about the ringer thing in the end?15:35
tvossseb128, even if it did build, it is just for staging the toolchain specific changes right now15:35
tvossseb128, sil2100 can you guys hand over that information to the next "shift"? :)15:36
tvossmandel, ^15:36
seb128tvoss, right, that's a test silo15:36
tvossmandel, feel free to go ahead with your landing15:36
seb128tvoss, just write that in the comment column15:36
davmor2sil2100: I'm still running tests15:36
sil2100tvoss: sure ;)15:37
tvosssil2100, seb128 updated the comments column :) thanks15:37
sil2100stgraber: ^ just so you know, silo 008 is a test silo and shouldn't block assignment of other silos15:37
davmor2sil2100: ringer seems to be working at least :)15:37
seb128tvoss, yw15:37
sil2100davmor2: but is the blocker bug still happening? ;)15:38
sil2100davmor2: I know we won't promote anyway now, but I want to at least know the situation15:38
sil2100I'm starting to feel the pressure of no promotions for a week15:38
mandelsil2100, seb128 it would be ideal to land what we have in line 16 that uses dbus-cpp before tvoss work15:38
sil2100Monday is TRAINCON-0 already15:38
davmor2sil2100: it's a race condition I'm sure of it.  So it isn't happening this time that's not to say it might not kick in again.  I would say down grade it.  Keep watching for it and when it is reproducible leave the device in that state and then get the devs involved till it is actually fixed15:39
sil2100davmor2: right, but is it something you would block on right now promotion-wise?15:43
stgrabersil2100: yeah, I noticed, I ran into this with jdstrand's landing15:44
davmor2sil2100: no that's what I'm saying downgrade it from promotion blocker to other bugs of interest or whatever the section is called and just monitor it there till some has a device back in that state and then work with the devs till it is fixed15:45
davmor2sil2100: it's to random to block on15:45
sil2100Ah15:45
sil2100davmor2: ACK, I guess that sounds sane15:45
boikosil2100: thanks (/me was out for lunch)15:52
sil2100I'll be there in a moment16:01
popeystupid browser crash16:08
sergiusensbrendand: ricmm: mandel at least according to the dbus spec; implementing introspection is optional (it says MAY and to use the RFC definition of it)16:08
mandelsergiusens, I know it is optional for objects to implement the interface... but it is very useful and everyone assumes it is there16:26
mandelsergiusens, but you know what they say when you assume ;)16:26
sergiusensmandel: then they need to request a spec update ;-)16:26
mandelsergiusens, probably, or we just have to provide an easy way to provide the instrospect for the libs, I'm thinking about how to do it with dbus-cpp 'cause it has the same issue16:27
sergiusensmandel: then usensord is not the only one causing slow downs ;-)16:27
sergiusensmandel: should just fix autopilot then ;-)16:27
mandelsergiusens, no, the dbus-cpp ones have the exact same issue, but I can just push the go-dbus problem to you hehe16:28
sergiusensmandel: if I fix it, you are next in line :-P16:28
mandelsergiusens, I do agree, the way that autopilot works is wrong, makes waaaaay to many assumptions with no scientific base16:28
mandelsergiusens, if I'm next.. can you take some time ;)16:28
sergiusenslet's just call it an autopilot bug then16:28
sergiusenslolz16:31
sergiusenssil2100: ^^16:31
sergiusensjust in case ;-)16:31
sergiusensnot just usensord16:31
sergiusensmandel: btw, media-hub would solve it for you ;-)16:33
=== gatox is now known as gatox_lunch
mandelsergiusens, it is defiently not just sensord, all daemons that do not have an Introspect interface implemented will have it, yet, there is a bug  in ygo-dbus16:33
ogra_scientific base ? whats that ?16:34
mandelsergiusens, it should raise a NoSuchInterface error instead of doing nothing :)16:34
mandelogra_, something I just pulled out of a hat ;)16:34
sergiusensmandel: it returns MethodUnkown16:34
ogra_mandel, you magician you  !!!16:34
brendand_<sergiusens> let's just call it an autopilot bug then16:35
brendand_^ NO16:35
sergiusensmandel: http://paste.ubuntu.com/7711951/16:36
mandelsergiusens, yet autopilot has a timeout?16:36
sergiusensbrendand_: reasoning?16:37
mandelsergiusens, but, you do have an object in / correct?16:37
sergiusensmandel: nope; that's why I called both; just in case16:37
sergiusensreturns an error in both cases16:37
mandelsergiusens, well, an error should be returned and autopilot should deal with it, dbus does make a diff when the object is not there => http://dbus.freedesktop.org/doc/api/html/dbus-protocol_8h_source.html16:40
brendand_sergiusens, so you're saying usensord doesn't do anything wrong?16:40
mandelsergiusens, I would use #define DBUS_ERROR_UNKNOWN_INTERFACE "org.freedesktop.DBus.Error.UnknownInterface"16:40
ricmmwell the case here is different, usensord (go-dbus) just timesout16:40
mandelseb128, and #define DBUS_ERROR_UNKNOWN_OBJECT "org.freedesktop.DBus.Error.UnknownObject"16:40
ricmmmandel: does dbus-cpp return an error for unavailable interfaces?16:40
mandelseb128, sorry,16:40
mandelsergiusens, ^^16:40
ricmmright, thats what the other services on the bus that arent introspectable return16:41
sergiusensricmm: my dbus-send calls return errors16:41
mandelricmm, I need to check I don't know it by heart16:41
sergiusensricmm: http://paste.ubuntu.com/7711951/16:41
mandelsergiusens, I think is better to use org.freedesktop.DBus.Error.UnknownInterface for the interface, that would be the only bug I can see16:41
sergiusensmandel: I could return UnkownInterface; that's easy16:42
mandelbrendand_, the fact that a dbus object is not introspectable is not a bug at all since there is no guarantee that the object implements it, that said, an error should be returned and not just a timeout16:42
sergiusensbrendand_: I'm saying autopilot should not rely on things that aren't mandatory on the dbus spec16:42
mandelsergiusens, that would be ideal, since with that error we know is not a missing method but an interface not implemented16:43
mandeland is easier to spot16:43
sergiusensmandel: I'll fix that then16:43
mandelsergiusens, love it :)16:43
ricmmsergiusens: this return of yours is after a patch? or is that what you get by default on the bus16:44
brendand_mandel, does the dbus spec have an opinion on how to handle that? does it say an error should be returned?16:44
ricmmmethod call sender=:1.373 -> dest=:1.284 serial=2 path=/; interface=org.freedesktop.DBus.Introspectable; member=Introspect16:44
ricmmthis is the blocking call16:44
mandelbrendand_, and error should be returned and it it is a known one: org.freedesktop.DBus.Error.UnknownInterface16:44
brendand_mandel, i think autopilot handles the situation where the correct error is returned just fine16:45
mandelbrendand_, so if you get that error in the reply, you know that you are missing the interface and you can skip the test with a nice message16:45
sergiusensricmm: what is :1.284 ?16:45
brendand_mandel, so if that's what the spec says and usensord is not doing that, then the bug is in usensord16:45
mandelbrendand_, if that is the case the only issue is go-dbus not returning the error16:45
brendand_mandel, but to repeat what i said before, i am going to suggest to them that autopilot might want to be cleverer about this eventuality16:46
mandelbrendand_, we have to blame it lower in the stack, you will have the same issue with all daemons that us go-dbus16:46
mandels/us/use16:46
ricmm:1.28516:46
brendand_mandel, what is timing out exactly? it's not autopilot because qdbus also does the same thing16:46
jdstrandsil2100: hey, so I got the MR approved and updated the changelog manually but am seeing this: https://ci-train.ubuntu.com/job/landing-007-1-build/83/console16:47
mandelbrendand_, you get a timeout error from go-dbus16:47
jdstrandsil2100: do I need to reconfigure the ppa?16:47
sergiusensmandel: can you show me the timeout error?16:48
sergiusensmandel: I am not seeing it16:48
mandelsergiusens, this is from ricmm => http://pastebin.ubuntu.com/7709477/16:48
sergiusensmandel: brendand_ I don't see a timeout here http://paste.ubuntu.com/7712004/16:49
mandelsergiusens,  1.15816:49
brendand_mandel, sergiusens - please update https://bugs.launchpad.net/autopilot/+bug/1334676 with your findings so the autopilot team can decide if they need to do anything in response16:49
sergiusensmandel: usensord is a named service16:49
ubot5Ubuntu bug 1334676 in qtubuntu-sensors "Applications have a dead open dbus connection (since image #96)" [Critical,New]16:49
brendand_gotta go now16:49
mandelsergiusens, hm..16:50
mandelsergiusens, we need to get the exact tests that ricmm did16:51
brendand_mandel, http://pastebin.ubuntu.com/7709477/16:52
brendand_mandel, i asked him to follow those - he should confirm though16:52
ricmmyea, just follow that16:52
ricmmyou'll see it failing to introspect that connection16:52
ricmmhowever I'm not entirely sure what that means, im no dbus expert16:52
ricmmcertainly if you introspect usensord's name you'll get an error, no timeout16:53
ricmmbut if you target that connection, it dies16:53
mandelricmm, if it is a named service you will need to use as a named one and that might be the issue16:55
mandelI need to run to do some errands before everything close16:56
mandelsergiusens, I think that if you are returning the error the issue is the what you mentioned, it is a named service16:57
AlbertAsil2100: landing-013 ready to go16:57
sergiusensbrendand: ricmm mandel well I'm going to investigate some more; but it isn't as simple as I originally thought16:58
fgintherSaviq, does lp:unity-system-compositor/devel-mir-next replace lp:~mir-team/unity-system-compositor/development-branch ?16:59
=== alan_g is now known as alan_g|EOW
sergiusensricmm: mandel http://paste.ubuntu.com/7712099/17:06
ricmmsergiusens: works the second time?17:08
ricmmthats not the usensord connection tho17:08
sergiusensricmm: yeah...17:08
sergiusensnope17:08
ricmmthe usensord connection fails all the times17:08
sergiusensricmm: so is it possible that the unnamed connection is the client connection? I'm not sure how that is supposed to work though...17:12
jdstrandI need help with silo 717:12
jdstrandit is telling me to adjust the changelog, which I've done and it still fails17:13
jdstrandhttps://ci-train.ubuntu.com/job/landing-007-1-build/84/console17:13
jdstrandI don't know what to do at this point17:13
ricmmsergiusens: im not sure either, one sec17:14
ricmmtrying some stuff17:14
sergiusensricmm: asking pitti on #ubuntu-devel17:14
jdstrandretoaded: ^ (re silo 7)17:15
ogra_jdstrand, "a change in debian/changelog with no UNRELEASED or no content in the changelog."17:16
ogra_jdstrand, note the "no UNRELEASED"17:16
jdstrandyeah, I know17:16
ogra_change "utopic" to UNRELEASED17:16
ricmmsergiusens: well it certainly is the pid of the client17:16
ricmmso the connection might indeed be the client end17:16
* ogra_ thinks that sentence needs serious re-phrasing17:16
ricmmnot sure how introspection works in that case17:16
* jdstrand reads the message again17:16
jdstrandI guess I got hung up on "If you modify debian/changelog in your commit, the commit message for the merge proposal isn't used"17:17
ogra_you should not have no UNRELEASED, no17:17
retoadedjdstrand, that would need one of the CI train pocs17:17
jdstrandogra_: I'm confused, should I have UNRELEASED or utopic?17:17
ogra_negation FTW :)17:18
ogra_jdstrand, right, thats what i mean ... you should have UNRELEASED in there17:18
jdstrandok, let me try that17:18
ricmmtvoss: are you still around?17:18
sergiusensricmm: yeah, I don't either17:20
ricmmsergiusens: maybe dbus-cpp needs to run the bus executor in a thread?17:20
=== sil2100 changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: retoaded | CI Train Status: #87 promoted | CI Train Sheriff: stgraber | Known issues: The spreadsheet is extremely slow, http://people.canonical.com/~rbpark/citrain/ knows the statuses before the spreadsheet does. Long mako wait times on s-jenkins.
ricmmfor pure-clients dbus-cpp doesnt run the bus on its own17:20
ricmmso incoming messages dont get processed17:20
ricmmbecause there are no registered method handlers17:20
ricmmsergiusens: going to give that a shot and see what happens17:22
ricmmI have a bad feeling that it will work17:22
ricmmlol17:22
sergiusensa BAD feeling?17:22
sergiusenslol17:22
ricmmyea17:22
sergiusenstoo much star wars :-)17:23
jdstrandok, that seems to be working17:37
jdstrandogra_: thanks17:37
=== gatox_lunch is now known as gatox
ricmmsergiusens: seems to work, lol17:45
ricmmgod I love these wild guesses17:45
elopiofginther: the job failed with:18:03
elopiocould not import package ubuntu_experience_tests: No module named ubuntu_experience_tests18:03
elopiocould it be that it's not installing the deb, or that phablet-test-run is trying to run with py2?18:03
sergiusensricmm: hawesome18:14
rsalvetiricmm: stop breaking stuff :P18:17
ricmmrsalveti: go to china or something18:20
ricmmrsalveti: tired of you18:20
rsalvetiricmm: doing that tomorrow, no worries18:20
=== renato_ is now known as Guest23257
sergiusensfginther: I see you used your personal account to take ownership of com.canonical; this basically means you have to use your account for everything (the store doesn't support multiple accounts)18:31
sergiusensfginther: followup, a question; did you build this with click chroot?18:31
ricmmsergiusens: alive?18:55
ricmmsergiusens: https://code.launchpad.net/~ricmm/platform-api/thread-for-client/+merge/22488818:55
sergiusensyeah18:55
sergiusensjust quiet18:56
sergiusensah, cuando veo c++ me tengo que concentrar :-P18:56
=== retoaded changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cprov | CI Train Status: #87 promoted | CI Train Sheriff: stgraber | Known issues: The spreadsheet is extremely slow, http://people.canonical.com/~rbpark/citrain/ knows the statuses before the spreadsheet does. Long mako wait times on s-jenkins.
=== stgraber changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cprov | CI Train Status: #87 promoted | CI Train Sheriff: - | Known issues: The spreadsheet is extremely slow, http://people.canonical.com/~rbpark/citrain/ knows the statuses before the spreadsheet does. Long mako wait times on s-jenkins.
balloonsfginther, did you manage to update the hook for reminders? Also, is the clock-autolanding job ok? This MP doesn't seem to want to be picked up for landing: https://code.launchpad.net/~nskaggs/ubuntu-clock-app/new-pep8-fixes/+merge/224853. It's unclear what jobs are being used for clock atm, looking at jenkins19:34
=== rsalveti_ is now known as rsalveti
tvossstgraber, still around?20:07
tvossstgraber, in case you come back: patched out all the universe build deps and have a packaging in the silo20:09
tvossdavmor2, mandel ^ would be great if you guys could give it a spin20:09

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