/srv/irclogs.ubuntu.com/2014/10/01/#ubuntu-ci-eng.txt

bfillerrobru: I need rtm silos for line 81,82, and 83 when you have a chance00:44
bfillerrobru: scratch that, 80, 81, and 82 I mean00:44
robrubfiller: oh goodies00:44
robrubfiller: well, let's do everything in 3's today! You got silos 6, 9, and 12.00:46
bfillerI love threes :)00:46
robrubfiller: i landed some changes to the build job today, let me know if anything explodes (should be good, I tested it all day before landing)00:49
bfillerrobru: will do00:49
robrubfiller: thanks00:49
robruoooh01:17
bzoltantrainguards: I have flipped the "tested" switch for the silo17 with the QtC plugin. It has changes in the debian/ folder as it provides the new -autopilot package. This change was one of the main target of the 14.10 SDK tools.03:53
Mirvrobru: mine has many builds (=manual uploads), but it's a preparation silo that will stay there for a good while so the status is not very useful to have updated04:03
robruMirv: OK no worries, just wanted to test the newly-refactored build job, but bill built a bunch, looks good.04:23
Mirvbzoltan: I'm worrying slightly more about the autopilot tests being a new feature and there being no FFe04:26
Mirv(than the packaging changes as such)04:27
bzoltanMirv:  the package change is that it introduces a new package04:54
bzoltanMirv:  Well, the autopilot package is not pulled by any dependency and does nothing special than tests the QtC plugin.04:55
Mirvbzoltan: yeah, it's nothing special, but because it's a new package it will stop at archive admins' hands anyway so I'm just wondering before-hand what they'd think about it04:59
Mirvadding tests of course sounds of course always welcome, and the click support additions are more like bug fixes themselves, so maybe it's alright05:00
Mirvbzoltan: couldn't you do the debian/rules file installations from the qmake files instead of hand-copying so many files?05:01
Mirvnot that it would be completely wrong, but ideally there wouldn't be any need for dh_override_auto_install05:02
bzoltanMirv:  I  lost the net, I am not sure if you have seen my last two lines.05:57
Mirvalright, no I didn't, but I see them now in msg05:59
bzoltan2Mirv:  I made two new lines with the UITK landings... both RTM and Utopic... whichever can land first. Obviously RTM is the priority. It has 4 RTM bugs fixed.07:15
brendandogra_, i'm testing silo 7 now - i guess it will require me to go through all the recovery mode dance etc, etc?07:21
tsdgeosis jenkins qa down?07:45
tsdgeosjenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/1444 doesn't load here07:45
tsdgeosand obviously it came back after 5 minutes the moment i mentioned it here :D07:46
Mirvbzoltan2: ok. rtm first I'd say07:51
bzoltan2Mirv: Yes, I agree07:52
Mirvbzoltan2: sorry, took a while, assigned and kicked a build08:04
bzoltan2Mirv:  no hurry :) thanks08:05
Mirv(and rekicked)08:06
Mirvthe build job has been reimplemented by robert, that first error sounds a bit like something might be a amiss. forcing (ignore step) + ignore dual works though.08:06
davmor2Morning all08:18
brendanddavmor2, do you know where to get the custom tarball?08:41
davmor2brendand: yes from the channel08:41
brendanddavmor2, ok08:42
=== vrruiz_ is now known as rvr
Mirvpstolowski: thostr_: ^ both your unity-scope-mediascanner2 and mediascanner2 landings how now rtm silo and are building, so should be ready for testing soon.09:18
pstolowskiMirv, thanks09:18
Mirvpstolowski: in your scopes-shell landing, the MP URL is not MP URL but branch09:19
Mirvthe fix-delete-later one09:20
pstolowskiMirv, ah, sorry, fixed09:20
Mirvthanks!09:20
mzanettitrainguards: silo utopic/006 tested. ready to go.09:22
mzanettihmm... seems its not mirrored yet to a rtm silo.09:23
mzanettiwe might want to do that first I guess09:23
Mirvkgunn: mzanetti there's unbuilt and untested unity 8 rtm-011 landing "Unity8 indicator polish fixes ("09:25
Mirvmzanetti: it's enough the sync silo is synced before the utopic one is cleaned09:25
mzanettiMirv: oh... I thought kgunn tested the indicators polish for rtm already09:25
mzanettiMirv: I can test that now09:25
Mirvmzanetti: ok. build it first, though :)09:26
mzanettiack, building09:26
mzanettiMirv: hmm... what does that mean? (silo rtm/11)09:27
Mirvmzanetti: err.. I don't know09:29
Mirvas usual, I'll workaround instead of trying to understand the error since it's beyond me09:29
mzanetti:D09:29
mzanettithat's you get the job done09:29
ogra_mzanetti, yo ...09:31
mzanettiogra_: :)09:31
ogra_mzanetti, we seem to see unity8 crashes in app tests with the last utopic image09:31
Mirvmzanetti: ok the packages are binary copied now manually, just needs maybe 10-20mins so that they are Published instead of Pending09:31
Mirvas seen at https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu-rtm/landing-011/+packages09:32
mzanettiMirv: so I can already start testing that ppa now, right?09:32
mzanettiogra_: have some more details?09:32
ogra_mzanetti, if you scroll to the bottom of http://ci.ubuntu.com/smokeng/utopic/touch/mako/264:20141001:20140929.1/10758/ubuntu_calculator_app/09:32
Mirvrobru: possibly interesting build failure https://ci-train.ubuntu.com/job/ubuntu-rtm-landing-011-1-build/40/console (workarounded now by manually copying packages)09:32
ogra_there is a .crash file09:32
ogra_unity8 itself also seems to have a bunch of errors doing its own tests in utopic http://ci.ubuntu.com/smokeng/utopic/touch/mako/264:20141001:20140929.1/10758/unity8/09:33
ogra_mzanetti, it seems rtm is fine ... so i think it would be good to find the issues before something migrates over09:35
mzanettiogra_: that's a race condition... we're investigating into that09:36
ogra_cool, thanks !!09:36
mzanettiogra_: but I think its in rtm too... just not happening always09:36
ogra_ah09:36
mzanettiogra_: not sure what's up with the calculator tests yet... will check out09:36
mzanettiogra_: do you know how I get the calculator tests on my device?09:36
ogra_https://wiki.ubuntu.com/Touch/Testing09:37
ogra_under "Running click tests"09:37
mzanettiogra_: hmm... seems to fail with an exception09:39
mzanettilist index out of range09:39
ogra_do you have the patest phablet-tools ?09:39
ogra_*latest09:39
dbarthhi trainguards; silo rtm 16 ready to land now09:42
mzanettiogra_: I didn't have that. now I upgraded. still happening though09:42
ogra_weird09:43
mzanettiwell... let me upgrade everything... might take a bit09:43
Mirvmzanetti: yes, now ready09:48
mzanettiMirv: thanks09:49
Mirvdbarth: you probably mean rtm 018? it's goind for QA signoff now.10:05
ogra_mzanetti, hmm i cant reproduce the unity8 crash when running the calculator tests here ...10:25
* ogra_ re-runs again 10:25
mzanettiogra_: phablet-test-setup still not working for me10:25
ogra_mzanetti, did you run "phablet-config autopilot --dbus-probe enable" first ?10:26
ogra_thats what i do ...10:26
ogra_and then:10:26
ogra_phablet-click-test-setup --click com.ubuntu.calculator;  phablet-test-run ubuntu_calculator_app10:26
cjwatsonbzoltan2: That new version of click is in utopic now, by the way.  Not RTM yet - need to test that today and put through QA.10:27
ogra_runs fine here10:27
mzanettiogra_: hmm.. no... didn't run that... from that wiki page I understood that's to be ran after installing the click packages... will try10:27
cjwatsoncamako,robru: Which specific builds are you talking about here?  All the virtualised builders in the Launchpad build farm are on trusty nowadays, but it's true that the devirtualised farm is still on precise.10:28
cjwatson(Mostly.  Being devirtualised it's not totally homogeneous, especially across architectures.)10:28
ogra_mzanetti, "phablet-click-test-setup" standalone would just install all click tests ..10:29
ogra_(no need for that)10:29
mzanettiogra_: yeah... I tried with adding --click ...10:30
cjwatsoncamako,robru: The problem with upgrading the devirtualised build farm has historically been concerns about what effect it's going to have on building security updates for older but still-supported distributions (say, lucid-security or precise-security).  Possibly we can get over that ...10:30
lool(about to publish this one, any reason not to?)10:41
loolthen will need a rtm silo  :-)10:41
dbarthMirv: indeed rtm 1810:42
bzoltan2cjwatson: All right, the SDK counterpart has landed too few hours ago. Good job :) thank you.10:42
Mirvlool: oh, you're our tvoss now? how about testing rtm-001 with location-service?10:46
Mirv(the previous version, already in utopic)10:46
loolMirv: hmm pretty sure that wont work10:46
loolor rather, that it needs a coordinated landing10:46
Mirvlool: anyhow, that ^ can't make it into rtm before the previous landing, since ^ already includes the previous landing (that landed together with dbus-cpp) also.10:49
Mirveither the 001 should be combined into the upcoming location-service rtm silo, or 001 should be landed first.10:49
loolMirv: right; what I believe is that 001 or anything subsequent needs a coordinated update of the custom tarball10:50
loolit might be less effort for QA etc. to do a single location-service landing with everything I guess10:50
Mirvok. added some notes to both landings in the spreadsheet for now.10:51
Mirvcombining might be as easy as updating rtm-001 description and resyncing the location-service from utopic. but it's up to you to decide. certainly it'd mean less QA work.10:51
Mirvinterestingly, the dbus-cpp change already landed together with media-hub10:58
bzoltan2Mirv:  do you know who is the LP guru/jedi I could ask to change the UITK project setup?11:07
cjwatsonWhat do you need changing?  (I probably don't have relevant permissions, but I expect I could figure out who does)11:10
Mirvbzoltan2: ^ same question11:11
Mirvmzanetti: stop breaking the good train! :)11:13
mzanettiMirv: :D11:13
mzanettiI don't even know what I'm doing to break it11:13
Mirvmzanetti: the -gles branch's watch file is wrong, should point to 00711:14
mzanettiah right.... that's the other one...11:14
mzanettiwill update11:14
mzanettithanks11:14
bzoltan2cjwatson: I would need to add new milestones11:28
bzoltan2cjwatson: and it seems that only Kaleo could do that, but he is nor really working on the project anymore11:29
cjwatsonbzoltan2: What have you tried?  The project driver should be able to do that.11:30
cjwatson(= ~ubuntu-sdk-team)11:30
Mirvbzoltan2: you seem to be admin of both https://launchpad.net/~pspmteam/+members and https://launchpad.net/~ubuntu-sdk-team/+members , so I cannot immediately see why you couldn't have "Create milestone" button on https://launchpad.net/ubuntu-ui-toolkit/trunk11:30
cjwatsonbzoltan2: If you don't, then I can (due to being in ~registry), but I believe you should be able to do it yourself.11:32
bzoltan2cjwatson: \o/ cool... Me idiot I was looking for that button on the milestones page11:33
Mirvpsivaa: first .crash retraced https://bugs.launchpad.net/ubuntu/+source/mediascanner2/+bug/137621911:33
ubot5Ubuntu bug 1376219 in mediascanner2 (Ubuntu) "qmlscene crashed with SIGABRT in raise() in music_app" [Undecided,New]11:33
* Mirv afk, back later11:53
ogra_brendand, what happened to the fix of phablet-click-test-setup ? would it be ready ?11:54
ogra_(i just ran into it with the patest unity8 landing ... cant run the tests on latest utopic anymore)11:54
ogra_*latest11:54
asacsil off today?11:55
=== alan_g is now known as alan_g|lunch
ogra_asac, ge cracked his wrist11:59
ogra_*he11:59
brendandogra_, it hasn't been merged yet: https://code.launchpad.net/~nskaggs/phablet-tools/fix-1371241-try2/+merge/23638512:01
asacogra_: ouch12:01
ogra_brendand, yes, i was planning to do a clean up of the phablet-tools backlogs today (there are phablet-shell and demos-setup fixes too that could land together)12:01
bfillerMirv: ubuntu silo 1 and silo 9 are ready to be released, but they are not in sync with rows in spreadsheet. In fact there is no row in spreadsheet for silo 1 at all12:02
ogra_brendand, heh, i think thats the wrong one :P12:02
brendandogra_, i wasn't aware of any other12:03
cjwatsonbrendand: do you have an opinion on whether https://launchpad.net/ubuntu/+source/glibc/2.19-10ubuntu2 would need QA testing for ubuntu-rtm as well as dev testing?  The emulator doesn't seem to be broken on RTM right now, but it's a potential time-bomb that could be tripped by fairly innocent changes to other packages12:03
cjwatsonand grr, looks like I still need to poke at autopkgtests there12:03
ogra_cjwatson, how hard would a rollback be ? we could just land it, have a full smoke tests and if needed roll it back12:04
cjwatsonogra_: probably relatively straightforward at least if I'm around to do the right removes/copies12:04
ogra_well, i'd do it that way then ... just let the automation catch potential issues and be prepared to roll back12:05
ogra_but thats indeed time consuming12:05
cjwatsonI didn't think "just land and roll back if needed" was the approved philosophy for RTM though :P12:05
brendandogra_, what other change to phablet-click-test-setup?12:05
=== cprov changed the topic of #ubuntu-ci-eng to: Need a silo? Ping train support: trainguards | Need other help? Ping vanguard: cprov | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Please inform robru of all build failures today.
ogra_brendand, hmm, might actually be the right one ... i'm a bit irritated that everything is yellow on that page12:07
ogra_brendand, yeah, applying that here helps ... i'll try to land it then12:09
brendandcjwatson, yeah i think so12:12
bzoltan2Mirv:  We have a cool UX fix in the line 88. It will take an hour or two to release it.12:15
om26ercprov, Hi! the dashboard have results for calendar app, while its removed from the image. Is there a reason not to remove that from the dasboard ?12:18
cprovom26er: perhaps there is a stray line in the spreadsheet, let me check12:19
cprovom26er: do you mean the silo 9, right ?12:21
om26ercprov, I didn't see the silo. I am talking about this dashboard http://ci.ubuntu.com/smokeng/utopic/touch/mako/264:20141001:20140929.1/10758/12:22
om26erweather app is no longer in the image but its results are still there.12:23
cprovom26er: oh, sorry, I was thinking about the train dashboard (biased, sorry).12:24
om26er;)12:24
cprovom26er: let me check why it is still listed, I will come back to you.12:24
om26ercprov, alright, thanks12:24
cprovom26er: ubuntu_weather_app is a click package and is tested from the click-store based on a jenkins configuration file we control.12:36
cprovom26er: that explains why it is being tested, even if it is not included in the image.12:36
popeysince when was it removed from the image?12:37
om26erpopey, last week I believe.12:37
popeywat, no. calendar was removed.12:38
om26ercprov, ok, I will talk to plars on that and see what we can do.12:38
cprovom26er: np12:40
popeyom26er: http://people.canonical.com/~ogra/touch-image-stats/258.changes 258 had an update, i see no removal.12:40
om26erpopey, I don't see it on my phone12:41
popeyom26er: what device / channel / version ?12:41
om26erI noticed it when the 'Upcoming event' menu stopped opening the calendar, first I thought that was a bug12:42
om26erpopey, image 264 - utopic-proposed12:42
popeyom26er: wait, you said _weather_12:42
popeynow you mention calendar.12:42
popeyI know calendar was removed. I'm asking about weather.12:42
om26erI said calendar above.12:43
popey13:23:05 < om26er> weather app is no longer in the image but its results are still there.12:44
popeyno, you said weather12:44
om26erpopey, <om26er> cprov, Hi! the dashboard have results for calendar app, while its removed from the image. Is there a reason not to remove that from the dasboard ?12:44
om26erseems the second time, even I forgot to call it calendar :D12:45
popeyheh12:45
popeylolz ☻12:45
cprovom26er: oh, okay then12:45
=== alan_g|lunch is now known as alan_g
ogra_sergiusens, i'd like a top approval for https://code.launchpad.net/~ogra/phablet-tools/fix-phablet-shell-without-local-key/+merge/23669612:47
pstolowskiMirv, hey, if it's not too late, can I squeeze one more MP for same project  into line #86, rather than create a new request?12:52
ychengtrainguards help needed for landing12:53
dbarthtrainguards can i get a silo reconfig on utopic 14?12:55
dbarthextra polish, but comes in 2 new branches12:56
ychengtrainguards I have no permission for the spreadsheet, and jhodapp has help me to use silos 00312:58
ychengtrainguards aha, I don't have the revision that I test it on.13:00
ychengtrainguards maybe I should get the version and comes back13:00
=== ycheng is now known as ycheng-afk
pstolowskitrainguards, Mirv may I ask for reconfig of silo-004 (#86) - one more MP added?13:01
* Mirv back13:02
=== Ursinha changed the topic of #ubuntu-ci-eng to: Need a silo? Ping train support: trainguards | Need other help? Ping vanguard: Ursinha | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Please inform robru of all build failures today.
jhodappMirv, what is ycheng-afk missing for landing utopic silo 3?13:07
ogra_brendand, it is in line 90 ... just waiting for a signoff from sergiusens for one branch there13:14
Mirvbfiller: 001 looks line line 39 with spreadsheet having lost the id.13:14
Mirvbfiller: I've restored the id now13:15
Mirvbfiller: so you an set it as tested as usual13:15
bfillerMirv: ok, line 63 marked as tested but not showing up as ready to publish13:16
Mirv009 funnily doesn't reflect the correct status in the spreadsheet13:16
Mirvbfiller: yeah, we've had that before, no idea what's it's related to. thanks for pinging about that.13:16
Mirvrobru: another example: 1. assign silo 2. build 3. get https://ci-train.ubuntu.com/job/ubuntu-landing-006-1-build/52/console13:21
Mirvbzoltan2: building at https://ci-train.ubuntu.com/job/ubuntu-landing-006-1-build/53/console13:21
Mirvpstolowski: sure you can13:21
Mirvpstolowski: you just need to ask for a reconfig after you've added it13:22
pstolowskiMirv, great, thanks. yeah, i updated and asked a few irc lines later ;)13:22
Mirvpstolowski: I'm just getting there :)13:22
Mirvpstolowski: dbarth: reconfig requests done.13:24
Mirvycheng-afk: jhodapp: so qtubuntu-camera on utopic?13:24
jhodappMirv, yeah13:24
Mirvycheng-afk: jhodapp: I'm not sure how, but there's the silo, and no line for it whatsoever13:25
jhodappMirv, yeah me neither...it landed in RTM already13:25
jhodappMirv, we just need it to sync and land in utopic now13:26
pstolowskiMirv, grr, i think i hit ctrl+z one time too many and removed that change a few minutes ago; just added it back, sorry for that :(13:26
Mirvycheng-afk: jhodapp: the silo 003 has version 20140918 of qtubuntu-camera, while both utopic and rtm already have 20140924. so at the very least, the current contents of silo 003 seem obsolete and the silo should be freed, plus a new line added for any new landing13:26
dbarththank you sir!13:26
dbarth:)13:26
sergiusensogra_: what is it waiting for me again?13:26
pstolowskiMirv, can you re-config one more time please?13:26
=== ycheng-afk is now known as ycheng
sergiusensogra_: ssh key and shell13:27
sergiusens?13:27
ychengMirv: jhodapp I have no permission for the spreadsheet, what I should do now ?13:27
jhodappMirv, sounds reasonable13:27
Mirvpstolowski: done!13:28
jhodappMirv, do we need a new empty MR then or something?13:28
ogra_sergiusens, yeah13:28
sergiusensogra_: if I run that twice, won't it fail to check properly?13:28
pstolowskiMirv, thanks!13:29
Mirvycheng: jhodapp: so just to be precise, the current silo comments are what landed in rtm at https://lists.canonical.com/archives/rtm-14.09-changes/2014-September/000438.html and to utopic as part of the _next_ upload (from trunk) at https://lists.canonical.com/archives/utopic-changes/2014-September/010654.html13:29
sergiusensogra_: as you have an mkdir in there13:29
ogra_sergiusens, not if you did what it asked for13:29
Mirvycheng: jhodapp: if you are only worried about the silo contents getting to utopic + rtm, there's nothing to be done13:29
ogra_sergiusens, yes, because known_hosts needs to exist13:29
sergiusensogra_: well, consider that you run lots of phablet-shell instances and miss the first notice13:29
jhodappMirv, oh so both already have the MR then is what you're saying?13:29
Mirvycheng: jhodapp: so what happened is that your utopic landing didn't happen, but the next landing did and it included both the "fix bug on use the lowest resolution instead of highest one" and the "Ensure androidControl is valid before using it to change exposure settings" since it was built from https://code.launchpad.net/~phablet-team/qtubuntu-camera/trunk13:30
Mirvjhodapp: yes.13:30
ogra_sergiusens, i wouldnt call that the typical use case (i would also not expect a developer to not own a ssh key indeed)13:30
Mirvso, I'll just free up that silo13:30
ogra_sergiusens, if you run it a second time it wuill fail with a key error13:30
jhodappMirv, oh excellent, so this can just be freed then and ycheng's landing is done13:30
ogra_sergiusens, but at least it tells you the first time what to do13:30
jhodappycheng, seems there's nothing for you to do now :)13:31
Mirvycheng: all done already, thank you! ;)13:31
jhodappthanks Mirv13:31
sergiusensogra_: exit 1 then I guess would be better, wouldn't it?13:31
ychengMirv, jhodapp: I'll de-assign me on that issue, thanks !13:31
ychengMirv, jhodapp: hmm, I can wait for it's landing and then de-assigned myself.13:32
ogra_sergiusens, well, if anyone checks the exist status i assume yes :)13:32
ogra_let me fix that13:32
jhodappycheng, it has already landed, you don't have to do anything13:32
Mirvycheng: the silo ^ was freed, it has landed and nothing more needs to be done13:32
ychengjhodapp: aha, ack, thanks13:32
jhodappycheng, Mirv will remove the line from the spreadsheet13:32
Mirvjhodapp: thanks13:32
ychengjhodapp: Mirv got it.13:32
ogra_sergiusens, exit 1 pushed13:33
=== ycheng is now known as ycheng-afk
thostr_Mirv: what is now with line 69?13:36
thostr_Mirv: I don't see the conflict as the "conflict" is said to have landed?13:37
Mirvbregma: ^ ignore. triggered wrong silo, aborted, running watch_only build to get the existing status back.13:37
Mirvthostr_: it was a conflict, not anymore, now assigning a silo13:37
thostr_Mirv: thanks13:37
* Mirv reaches EODish soon, sil2100 is away today and robert will be here in 2h. I've some errands to run.13:47
ogra_sergiusens, i changed the MP a bit more now ... please take another look if you have time14:01
thostr_Mirv: was the package in rtm silo 22 actually built? or just the utopic one copied because build was hit too quickly?14:12
cwaynejdstrand: it just occurred to me, the cache is going to appear broken until we have an rtm rootfs with the new apparmor-easyprof14:39
jdstrandcwayne: I don't understand14:51
jdstrandcwayne: do you mean apparmor-easyprof-ubuntu? (apparmor-easyprof is a different package)14:51
jdstrandcwayne: assuming you meant apparmor-easyprof-ubuntu, yes, you would not want to push your custom tarball until the krillin image has 1.2.3014:52
jdstrands/krillin image/krillin rootfs/14:52
jdstrand(which is what I was getting at earlier on deferring to you on the timing of server side stuff)14:53
cwayneright, sorry, I know they're different packages, I'm just lazy I guess14:53
cwaynejdstrand: right, well it's that + we needed it in the archive to build the new custom tarball anyway14:53
jdstrandright, and you need it in the archive to build the rootfs14:54
mzanettitrainguards: please reconfig silo 7 once more14:54
mzanettihad to add a qtubuntu-gles-sync14:54
cwayneogra_: any idea when we might expect a new rtm rootfs?14:56
ogra_cwayne, 3am UTC14:57
cwayne:/14:57
cwaynedavmor2: so i guess custom is off again til tomorrow :)14:58
ogra_cwayne, well, if needed we can indeed just build one15:02
cwayneogra_: is there anything else that would prevent us from doing a build?  it's not super-duper-critical, but it'd definitely be good to have this in today15:05
rsalveticjwatson: is there anything we can do to unblock glibc from proposed?15:05
rsalvetiit seems the failed test cases were always failing15:05
ogra_cwayne, well, we just need to make sure everything is in the archive before we start the build15:05
cjwatsonrsalveti: yes I'm working on it, no they weren't always failing15:05
ogra_theer are two new ones15:06
rsalvetiat least from the jenkins jobs most of them had failures before that, but might be my impression15:06
rsalveticjwatson: but cool, thanks15:06
ogra_most of them are overridden though15:06
cjwatsonthe two remaining started failing quite recently15:06
cjwatsonI need to make sure they have nothing to do with glibc before forcing anything15:06
rsalvetiyeah15:06
cwayneogra_: ack, the only thing needed from my end would be apparmor-easyprof-ubuntu version 1.2.3015:12
ogra_cwayne, ok, rmadison says it is in both archives15:13
ogra_i'm in a meeting atm but will trigger an image then15:13
cwayneogra_: thanks15:13
cwaynedavmor2: would you be able to test custom after that ^ or would it be too late15:14
davmor2cwayne: yeap sure15:14
=== plars changed the topic of #ubuntu-ci-eng to: Need a silo? Ping train support: trainguards | Need other help? Ping vanguard: plars | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Please inform robru of all build failures today.
balloonsogra_, can you approve and push in with the next release of phablet-tools? https://code.launchpad.net/~nskaggs/phablet-tools/fix-1371241-try2/+merge/23638515:23
ogra_balloons, see line 90 on the spreadsheet :)15:23
balloonsogra_, :-)15:23
ogra_balloons, i'm landing three MPs, but one is waiting for signoff ( sergiusens took a look but didnt sign off yet)15:24
ogra_i'm confident i can land this within the next hours though ....15:24
ogra_(i really needed your fix today here(15:24
balloonsogra_, same, I ran into the issue again and it reminded me15:26
ogra_sergiusens, ^^^ can i have that phablet-shell MP approved ?15:36
sergiusensogra_: well I didn't test yet15:36
* sergiusens doesn't approve without testing15:36
ogra_ok15:36
mzanettianyone here with powers to reconfig silo7?15:45
ogra_mzanetti, robru should be up soon15:45
mzanettiah, great15:45
cwayneogra_: has a build been kicked?15:52
cjwatsonmzanetti: doing15:54
ogra_cwayne, yes, bot should announce it in a moment15:54
cwayneogra_: awesome, thank you :)15:55
mzanetticjwatson: thanks15:57
imgbot=== trainguards: RTM IMAGE 76 building (started: 20141001 16:00) ===15:59
=== lan3y is now known as Laney
davmor2ogra_: just waiting to see if it updates the issues tracker but it is here https://bugs.launchpad.net/ubuntu-keyboard/+bug/1376337 there are also a couple of sound-indicator issues that should probably be tagged but not blockers16:22
ubot5Ubuntu bug 1376337 in ubuntu-keyboard "Spellcheck holds a single word reply in messaging indicator" [Undecided,New]16:22
Mirvbfiller: not approved https://code.launchpad.net/~renatofilho/address-book-app/release-30-09-2014/+merge/23663016:26
kgunnfginther: hey, so, let's say for a silo we'd like to run our ui tests on unity, would there be a way to either a) automagically have these run? or b) is there a way to check out the source from a built silo to manually run them ?16:27
kgunnfginther: details would be to  ./build.sh, cd builddir, make check (which includes make test and make qmltests)16:29
kgunnwe just keep chasing our tail on landing 1 or 2 broken ui tests16:29
bfillerMirv: fixed16:30
fgintherkgunn, can we discuss this later this afternoon (I've got a few meetings first)?16:30
fgintherkgunn, it's sounds possible, but I want to make sure I understand the whole picture first16:31
kgunnfginther: sure...16:31
kgunnyep16:31
ogra_davmor2, http://people.canonical.com/~lzemczak/issues/ looks great, thanks16:34
davmor2ogra_: there is more yet16:35
ogra_add away :)16:35
popeyogra_: i am seeing a bit of memory leaking in unity8-dash with a lot of scopes enabled...http://paste.ubuntu.com/8473625/16:40
popeyis there a bug for that already?16:40
kgunnpopey: we did fix one recently16:40
popeyok. i have 11 open16:40
kgunnpopey: but it might be different...16:41
popeystarted monitoring memory and haven't touched the device since. it's sat unlocked16:41
popey(odd it's not locking itself)16:41
kgunnpopey: not sure if you can easily tell if this https://code.launchpad.net/~albaguirre/platform-api/fix-120614616:43
kgunnis in the image you're on16:43
kgunn...things taking longer to hit rtm branch it's hard to say16:44
imgbot=== trainguards: RTM IMAGE 76 DONE (finished: 20141001 17:05) ===17:04
imgbot=== changelog: http://people.canonical.com/~ogra/touch-image-stats/rtm/76.changes ===17:04
ogra_cwayne, ^^^ enjoy17:05
sil2100davmor2: hey! I heard you found a regression in our last promoted image - do you have a bug handsy?17:05
davmor2sil2100: they should be in the issues tracker17:06
davmor2sil2100: I'm still looking through the image currently though17:06
ogra_is LP timing out for anyone else ?17:10
dobeyogra_: it is a bit yes17:12
dobeywell, bug page loaded fine, but my MP timed out17:12
ogra_right17:12
ogra_same here17:12
sil2100davmor2: btw. are those blockers for next promotion?17:15
ogra_sil2100, they are in the last promoted image ...17:15
bzoltan1brendand: I have started the validation of the UITK release candidate from the RTM silo14. I see thatthe Unity tests still hang. Is there a special trick i could see the same results as on the CI Dash?17:21
brendandplars, should the unity8 tests still hang?17:22
ogra_no17:22
ogra_they never hung in rtm17:22
cwaynedavmor2: latest custom is available for testing (ubuntu-touch/ubuntu-rtm/14.09-proposed-customized)17:23
cwaynesil2100: ^17:23
sil2100Excellent17:23
ogra_brendand, bzoltan1 http://dashboard.ubuntu-ci:8080/smokeng/utopic/touch_stable/krillin/75:20141001:20140929-d974fdc/503/17:24
sil2100davmor2: how long does the custom tarball testing take for you?17:24
ogra_there were unity8 failures today ... but it surely ran17:24
ogra_(and finished)17:24
plarsbregma: where did you see it hang?17:24
ogra_plars, i think they are referring to the media-hub/gstreamer/whatever hang we had on utopic for a while17:25
bzoltan1ogra_: All right, fingers crossed that I will see the same results with my re-run17:25
ogra_bah17:28
ogra_ 1907 root      20   0   81652   4748   4748 S  99,3  0,5  13:02.57 ubuntu-location17:28
ogra_lool, didnt you land a fix for that ?17:28
ogra_wow17:29
* ogra_ has a black screen with panel after unlocking on image 7617:29
ogra_(and location service eating 100%)17:29
ogra_apps dont start anymore17:30
ogra_showing an eternal splash17:30
* ogra_ reboots, this is unusable :(17:32
cwaynedavmor2: sil2100: just for claiification, the custom tarball needing testing is 1412208333 (can be found by running cat /custom/build_id)17:37
cwaynei've got to run a quick errand, will brb to see if everything's going ok17:37
ogra_cwayne, could we sync that versioning with something ubuntu-ish at some point ?17:39
cwayneogra_: i thought of that, but unsure how system-image server works (i.e. does it care about the version number or just the timestamps)17:43
ogra_cwayne, that i dont know ... but having something like YYYYMMDD.N-$STAMP would be more in line with all other versioning we use17:45
cwayneogra_: i'll poke around, i would definitely like to change it to something more meaningful17:46
cwaynewe need the build_id still because it tells whether or not we do a dconf update, but i dont think it should be used as the version number17:46
ogra_cwayne, talk to stgraber what s-i supports17:46
bzoltan1ogra_:  do you have a trick for silencing that device from adb shell? Not as I would not enjoy the music app's tests at 3am :) But am not sure if my family shares my passion to nightly tests.17:58
ogra_audio you mean ?17:58
ogra_hmm, no17:58
ogra_bzoltan1, you could try something like: amixer -D pulse set Master 1+ mute17:59
ogra_no idea if that would work though18:00
ogra_probably rsalveti knows another way18:00
bzoltan1ogra_: thanks, I will test it18:00
davmor2sil2100: sorry just got back from tea18:02
davmor2sil2100: So I think they will become blockers if they are not fix in the next 7 days but currently shouldn't blocker as it technically isn't a regression but are so user facing that they need fixing if that makes sense18:03
davmor2sil2100: custom tarball changes depending on what is landing but an hour or 218:04
loologra_: no, we don't have a repro for the 100% location-service CPU usage18:06
ogra_ok18:06
loologra_: brendand might have filed it, haven't seen it yet18:06
loolwill look after dinner18:06
ogra_lool, well, it is gone again after reboot18:06
ogra_and i havent noticed it ever before18:07
sil2100davmor2: btw. will you be able to take a look at utopic mako tomorrow? Since maybe we'll be able to promote a new image for that channel at least18:15
sil2100Sicne I guess the mediascanner scope issue seems to be fixed18:15
davmor2sil2100: I can once I get to the box that has my mako flo and manta in ;)18:19
dbarthhi trainguards, can i get a sync for silo utopic 14?18:29
dbarthi need it to populate line 54 for rtm18:29
robrudbarth: ok you got rtm418:31
dbarththank you18:31
robruyou're welcome18:31
dbarthhey, btw, pitti did what's needed to get the langpacks up to date on rtm18:31
robrudbarth: oh cool, so you should be able to just rebuild oxide and have it work? or maybe rebuild isn't even needed18:32
dbarthso that last bug will get fixed on friday18:32
dbartha rebuild is not needed18:32
robrueven better18:32
dbarthright18:32
infinitycamako: Can you give more details on the code that "requires a newer kernel"?18:35
infinitycamako: If it's just a test that can't pass without a newer kernel present, it's fine to conditionally skip that on older kernels.  If it's that your package literally doesn't work with older kernels, that might be a bug on your end, IMO.18:36
infinitycamako: It's generally sane to try to keep things working in a state where they can work in a chroot on top of our oldest supported LTS kernel.18:36
dbarthrobru: and utopic14 is good for publishing18:36
camakoinfinity, thanks for the comments ...18:37
camakoinfinity, this is for Mir desktop18:37
camakoand we are ok not supporting kernels older than 3.1118:37
camakowhere this kernel feature we depend on was introduced18:38
camakoSince mir on the desktop won't be ready for a while18:38
infinitycamako: Right, that's perhaps a fair decision to make.18:38
robrudbarth: https://ci-train.ubuntu.com/job/ubuntu-landing-014-2-publish/38/console please approve your merges18:38
infinitycamako: It won't prompt us to run around and upgrad the buildds in a hurry, though, so you might want to conditionally skip the test, and work out an alternate testing scenario (ie: build with tests enabled in a virt PPA, which are all 3.13 now).18:39
camakoinfinity, yea we discussed this during the review process... So how do you bypass a failing test on an old kernel?18:39
infinitycamako: The test itself should check the running kernel version before attempting to run.18:39
camakoinfinity, oh ok...18:40
cjwatsonHave a helper function that tests uname(2) or something18:40
camakosure, I'll look into that18:40
cjwatsonOr, better, check for availability of the actual kernel feature you're using18:40
cjwatsonTesting kernel version numbers is generally a last resort and not usually a good idea18:40
infinityYeah, what he said.18:41
infinityTesting the feature availability (is it a syscall?) is the sanest route.18:41
camakocjwatson, yes I believe that would be quite straightforward18:41
cjwatsonCool18:41
camakoit's just a flag passed to open18:41
camakoO_TMPFILE18:41
cjwatsoninfinity: Do you know if we have a policy issue with upgrading buildds to trusty?18:41
cjwatsonI know there's a general thing about "don't upgrade metal to trusty, switch to the cloud", but we know that full conversion of buildds to the cloud is (a) in progress and (b) blocked for a while ...18:42
infinitycjwatson: So, yes, there's the "don't upgrade metal" issue, but I think elmo and I have already discussed that buildds are a unique snowflake there.18:42
cjwatsonk18:42
infinitycjwatson: I also am personally annoyed that "the cloud" must mean openstack, as the buildds are ALREADY a cloud and, in fact, the oldest cloud in Canonical. :P18:43
cjwatsonYeah, not maybe as horizontally scalable as we might like but indeed18:43
cjwatsonWe should probably try to figure out who's working on scalingstack/{power,arm}18:43
cjwatsonIf anyone18:43
infinitycjwatson: So, yeah, the only blocker for upgrading to trusty on x86 is time and a ticket, but I've been in no rush because powerpc has a kernel bug to hunt before we can upgrade, and armhf is just going to have the hardware replaced instead of upgraded.18:43
cjwatsonSo maybe midway armhf can just be installed fresh with trusty (it might have to be anyway), and that'll take care of most of mir's needs18:44
infinitycjwatson: But I also don't see any factors other than dogfooding that should force the issue either.18:44
infinitycjwatson: And the dogfooding argument should usually happen before an LTS releases, not after. ;)18:44
infinitycjwatson: Yeah, the midways will be tusty from day 1.18:44
infinityBut Mir builds places other than armhf, so the kernel feature testing before running the test thing will be required regardless.18:45
infinityAnd is also the right way to do it.18:45
cjwatsonYep18:45
infinityAssuming that build environment == runtime environment is pretty much universally wrong, IMO.18:46
infinitycamako: Oh, there are reasons other than kernel version to test O_TMPFILE anyway (and, maybe a good argument to have a forked path in your code, not your testsuite).18:47
infinitycamako: It's not supported on all filesystems.18:48
infinitycamako: So, if this code was running on, say, btrfs, it would explode, even if on a current kernel.18:48
camakoinfinity, good point18:48
cjwatsonYeah, that gets more delicate, you have to make sure your test runs against a file on the relevant fs18:48
infinitycamako: Not sure how one determines this without an expensive try;fail;fall-back, though. :/18:48
cjwatsonYou can probably cache it for each fs18:49
davmor2sil2100, cwayne: I'm not getting a location so I checked the /var/crash folder crashes for trust-store and location ouch18:49
davmor2not sure if that it the custom image fault or the app-armor update18:49
cjwatsonusing statvfs().f_fsid or something18:49
infinitycjwatson: Well, it depends on how often the flag is passed to open(2) during runtime.  If it's frequent, it's worth a startup check to determine if the target FS will support it, then cache the result.18:49
cwaynedavmor2: there was 0 change in custom pertaining to either of those18:50
infinityIf it's a one-time thing, that's a huge performance hit with little gain, when you could just avoid the flag.18:50
cwayneor anything related to HERE18:50
camakowe are opening /dev/shm... and it's a one-time thing18:50
infinitycamako: Ahh, if it's /dev/shm, you're safe.18:50
infinitycamako: That's guaranteed to support it (in newer kernels).18:50
cjwatsonok, so just try O_TMPFILE and fall back if you get EINVAL18:50
infinitycamako: So, going back to "the testsuite can test this" rather than testing it in the code seems reasonable.18:51
cjwatson(fall back => skip test, if you only want to cover this in the test suite)18:51
camakoindeed18:51
camakothanks for your feedback guys.. appreciate it..18:52
davmor2ev: I thought that previous errors was going to get a fix so that the error tracker showed the previous errors :(18:52
davmor2sil2100: you might want to add tags to https://bugs.launchpad.net/unity8/+bug/1373312 and https://bugs.launchpad.net/unity8/+bug/1373313 too18:55
ubot5Ubuntu bug 1373312 in Unity 8 "Phablet album art not coming through from media-hub" [High,Triaged]18:55
ubot5Ubuntu bug 1373313 in Media Hub "Previous & New buttons not hooked up in indicator" [High,Triaged]18:55
sil2100Oh?18:55
sil2100New stuff!18:55
sil2100davmor2: blockers?18:55
cjwatsonsil2100: your landing team mail said that emulator images were broken on the RTM channels - are you sure?  the breakage I'm aware of is only in devel-proposed18:55
cjwatson(possibly devel)18:56
sil2100cjwatson: someone mentioned to us that the emulator image we promoted for ubuntu-rtm was broken18:56
cjwatsonand I was able to use the emulator on ubuntu-rtm as of this morning18:56
cjwatsonwell, ubuntu-rtm/14.09-proposed was fine today18:56
davmor2sil2100: no user facing again but block in 7days if not fixed they got released in image 3 so not a regression18:56
sil2100cjwatson: maybe it's fixed in -proposed, but ogra_ mentioned it being broken in ubuntu-rtm/14.0918:56
sil2100davmor2: ACK18:56
cjwatsonmkay18:56
=== fginther changed the topic of #ubuntu-ci-eng to: Need a silo? Ping train support: trainguards | Need other help? Ping vanguard: fginther | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Please inform robru of all build failures today.
cwaynedavmor2: so how's it look otherwise?19:03
cwaynei need to disappear for an hour or so pretty soon, but ill be able to pop in to publish if needed19:03
davmor2cwayne: so it is broken on 76 ToyKeeper just confirmed so it looks like this isn't an issue with custom, but it means any of the scope that need location don't actually get it19:04
ToyKeeperI see a crash on boot for both.19:04
davmor2jdstrand: I'm going to have a dig but I think there were only a couple of things land in image 76 and your app-armor was one :(19:05
cwaynedavmor2: right, but with 76 the scopes that need location won't get them anyway19:06
ToyKeeperYesterday one of the new default scopes did ask me to allow location permission...  couldn't tell if it worked though.19:06
cwayneso no reason to block custom :)  main thing that changed is we set favorites (if the user has never set them) and we have a new cache to match jdstrand's upload19:06
cwayneso if we block custom, it'll be back to like 5 minute boots on krillin :/19:06
davmor2cwayne: which will mean that there should be a revert of the thing that broke it if that is app-armor we can't land custom either right19:07
cwaynedavmor2: are there any apparmor denials?19:07
davmor2cwayne: only just started digging19:08
jdstranddavmor2: apparmor-easyprof-ubuntu? what broke?19:08
jdstrandI find it pretty unlikely that would've broken anything19:08
jdstranddavmor2: are you talking about indicator-sound? that is confined so not affected by that upload19:09
jdstranderr19:09
jdstrandthat is *not* confined19:09
davmor2jdstrand: location and trust store started crashing on 76 only adb and app-armor changed19:09
jdstrandlocation is also not confined by apparmor19:10
cwaynei can't reproduce btw on 8819:10
jdstranddavmor2: please do 'grep DEN /var/log/syslog' and paste the denials19:11
cwaynewell 14.09-proposed-customized 8819:11
davmor2jdstrand: http://paste.ubuntu.com/8474451/19:12
cwayneall those leaf-net ones are just noise19:13
rsalvetibzoltan1: you can use the cmdline tools from pulse to mute the device19:14
rsalvetidon't use alsa19:14
jdstrandcom.canonical.scopes.timeout_timeout_0.4.10 is not using the connection api but trying to access network-manager directly19:14
jdstrandthat is a bug in that app19:15
bzoltan1rsalveti:  you would not use `amixer -D pulse set Master 1+ mute`?19:15
bzoltan1rsalveti: any better suggestion?19:15
rsalvetipactl probably, let me check19:15
cwaynejdstrand: would you mind logging a bug against hanloon?19:17
jdstrandyelp is a bug in the scopes api-- it is trying to create a directory not specified in https://wiki.ubuntu.com/SecurityTeam/Specifications/ScopesConfinement19:17
cwaynethat one has an upstream bug, i can try and find it19:18
jdstrandit looked different to me19:18
jdstrandcwayne: is hanloon for timeout?19:19
rsalvetibzoltan1: pacmd set-sink-mute sink.primary 119:19
davmor2jdstrand: I can file the bugs if you can point me to what the denial is for19:19
cwaynejdstrand: hanloon is for all the scopes in /custom, so yep19:19
jdstrandoh it seems like a bunch of theme19:19
jdstrandall network manager denials: bug #137640819:21
ubot5Error: Launchpad bug 1376408 could not be found19:21
cwaynejdstrand: thank you19:23
cwaynebut thats also unrelated to davmor2's issue (and in fact none of thses scopes except news have even changed since the last custom tarball release)19:23
jdstrandso, yelp should be using @{HOME}/.local/share/unity-scopes/leaf-net/@{APP_PKGNAME}, but it is using @{HOME}/.local/share/unity-scopes/@{APP_PKGNAME}_@{APP_APPNAME}19:27
jdstrandthat is a bug in yelp of the scopes api19:27
jdstranddavmor2, cwayne: ^19:28
cwaynejdstrand: yelp is only using what the scopes api is giving it19:28
cwayneit could be specific to the go bindings maybe.. but all the scope itself does is call ScopeBase.ScopeDirectory()19:28
cwaynejdstrand: but regardless I'll take a look and see what I can find19:28
jdstrandthen that is returning the wrong thing19:29
jdstrandI can't seem to find the bug19:29
jdstrandI thought there was one19:29
=== boiko_ is now known as boiko
jdstrandI'll file it19:30
davmor2ToyKeeper: can you do grep DEN /var/log/syslog and see if it is similar to http://paste.ubuntu.com/8474451/ please I want to double check that they are not masses apart19:31
cwaynedavmor2: jdstrand: I've got to run for an hourish, please shoot me an email if theres any ?'s, but it really seems like whatever is causing location-service to crash is unrelated (and I can't for the life of me reproduce)19:32
ToyKeeperdavmor2: http://paste.ubuntu.com/8474560/19:32
robruted: do you have any idea what that error meant? wtf?19:33
tedrobru, No, I built the package locally just to be sure. But no.19:34
davmor2jdstrand, cwayne: so ToyKeeper 's result is looking pretty similar to the one I pasted but nothing obvious, I'll keep digging in errors to see if I can see the faults that were filed there19:34
tedrobru, Restarting the build I think it's going find now…19:34
robruted: messed up19:34
=== nik90 is now known as nik90|Dinner
jdstrandcwayne, davmor2: bug #1376416 for yelp19:37
ubot5bug 1376416 in unity-scopes-api (Ubuntu) "apparmor denial for yelp" [High,New] https://launchpad.net/bugs/137641619:37
jdstrandthat leaves two denials:19:38
jdstrandOct  1 18:14:46 ubuntu-phablet kernel: [  232.213753] (1)[9350:scoperunner]type=1400 audit(1412187286.501:143): apparmor="DENIED" operation="open" profile="com.canonical.scopes.timeout_timeout_0.4.10" name="/android/system/lib/libGLESv2.so" pid=9350 comm="scoperunner" requested_mask="r" denied_mask="r" fsuid=32011 ouid=019:38
jdstrandOct  1 18:14:46 ubuntu-phablet kernel: [  232.395523] (0)[9388:com.ubuntu.yelp]type=1400 audit(1412187286.681:145): apparmor="DENIED" operation="mkdir" profile="com.ubuntu.yelp_yelp_1.0.26" name="/home/phablet/.local/share/unity-scopes/com.ubuntu.yelp_yelp/" pid=9388 comm="com.ubuntu.yelp" requested_mask="c" denied_mask="c" fsuid=32011 ouid=3201119:38
jdstrandOct  1 18:45:47 ubuntu-phablet kernel: [ 2093.373992] (0)[25723:webapp-containe]type=1400 audit(1412189147.661:151): apparmor="DENIED" operation="open" profile="com.nokia.heremaps_here_1.0.1" name="/custom/etc/dconf_profile" pid=25723 comm="webapp-containe" requested_mask="r" denied_mask="r" fsuid=32011 ouid=019:38
jdstrandwhoops19:38
jdstrandOct  1 18:14:46 ubuntu-phablet kernel: [  232.213753] (1)[9350:scoperunner]type=1400 audit(1412187286.501:143): apparmor="DENIED" operation="open" profile="com.canonical.scopes.timeout_timeout_0.4.10" name="/android/system/lib/libGLESv2.so" pid=9350 comm="scoperunner" requested_mask="r" denied_mask="r" fsuid=32011 ouid=019:38
jdstrandboth of these are due to changes outside of apparmor-easyprof-ubuntu19:39
jdstranddavmor2: these are also not related to location service19:39
davmor2jdstrand: https://errors.ubuntu.com/problem/361a1c5dde22e0ce444d9aaa23180449a7a52446 could be this fault which looks like it started on utopic yesterday19:40
jdstrandit is getting a little frustrating seeing new denials that no one is reporting19:41
jdstrandcwayne: so, apps aren't supposed to be using dconf19:43
jdstrandcwayne: do you know what /custom/etc/dconf_profile is about with the here app?19:43
davmor2jdstrand: I can run grep DEN /var/log/syslog once a day for you and email to you if you want?19:46
jdstranddavmor2: that could be helpful. I'd really like to see this automated though in the ci infrastructure...19:48
jdstrandie, so that things are blocked if they introduce new denials19:49
davmor2jdstrand: that would be the ultimate in awesome :)19:50
jdstrand:)19:50
davmor2hey ogra_ plars would it be possible to make that happen and maybe add the result to the image results? ^  ie run grep DEN /var/log/syslog and initially just store the output on the image results?19:56
davmor2eventually have it go green if there are none and red if there is :)19:57
plarsdavmor2: jdstrand: is this not covered by the /home/phablet/bin/check-clickhook-rules check that we added before?19:58
davmor2plars: no idea20:00
plarsdavmor2: also, is there any possibility that a test (like security tests) could throw a false positive here?20:01
cjwatsonrsalveti: looks like the last unforced autopkgtest (aria2) is passing locally for me - just double-checking that and then I'll force-skiptest glibc20:02
rsalveticjwatson: great, thanks20:02
cjwatsonrsalveti: thanks for poking zodb earlier20:02
cjwatsonhate these flaky tests20:02
rsalvetinp20:02
rsalvetiyeah20:02
davmor2plars: possibly20:02
davmor2night all, cwayne I'll take another look in the morning but I just ran out of day trying to figure out why location is dying20:04
jdstrandplars: what project is that in again?20:05
jdstrandplars: security tests would totally throw false positives (thousands of them)20:06
brendandmvo_, are you around?20:06
brendandmvo_, there's an issue with the rtm silo for click20:06
jdstrandplars: I think we would just want to block on apps suddenly geteting new denials20:06
brendandcjwatson, ^20:06
brendandcjwatson, actually it's your name next to the test results20:07
plarsjdstrand: we don't really have a way to inspect history while running the test20:07
jdstrandplars: at least as a start. apps are what are confined, and they use the services via their autopilot tests, etc20:07
cjwatsonbrendand: what is it?  I'm only around for a few more minutes and I suspect mvo has finished for the day20:07
jdstrandit could be after20:07
cjwatson(and I'm on vac tomorrow)20:07
brendandcjwatson, it seems the 'stop apps when uninstalling' fix doesn't work20:07
cjwatsonbrendand: yes I already reopened the bug20:07
jdstrandyou'd just need to filter out denials prior to test start20:08
cjwatsonbrendand: it's fine as long as it hasn't actually made things any worse (which AFAIK it hasn't)20:08
cjwatsonit fails because it removes the registration link before trying to stop the app, so can't get its manifest20:08
cjwatsonbrendand: see end of https://bugs.launchpad.net/ubuntu/+source/click/+bug/123213020:08
ubot5Ubuntu bug 1232130 in click (Ubuntu) "Uninstalling an app doesn't stop it" [Medium,Triaged]20:08
cjwatsonbrendand: I don't think this should block landing though - it didn't stop it before and it doesn't stop it now, but it also doesn't break app removal20:09
brendandcjwatson, alright i'll take that into account20:12
cjwatsonwe should absolutely fix it up properly in the next landing though20:12
cjwatsonbrendand: I'm glad to see QA noticed this too though! :-)  Sorry, I should've mentioned it in the spreadsheet to save on worry20:14
mvo_brendand, cjwatson: thanks, I will work on a fix tomorrow20:14
mvo_(and will also find out why the test did not fail for this :/)20:14
cjwatsonmvo_: because you happened to pick a package name to remove that exists in two database layers, so .remove just removes it from the top one, maybe?  though that should've caused the version to be wrong20:16
cjwatsonhopefully it's shallow-ish anyway20:16
mvo_cjwatson: right, I check it out20:16
cjwatsonthanks20:16
cjwatsonrsalveti: glibc forced20:17
cwaynedavmor2: so what's the verdict?20:32
=== nik90|Dinner is now known as nik90
cwayneah nm sorry davmor2 I'd missed your message earlier, tomorrow's good20:39
robrubfiller: you got rtm920:39
=== fginther changed the topic of #ubuntu-ci-eng to: Need a silo? Ping train support: trainguards | Need other help? Ping vanguard: cihelp | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Please inform robru of all build failures today.
rsalvetislangasek: may I ask some help to approve pulseaudio and indicator-sound? so we can completely land silo 1821:59
slangasekrsalveti: ok. is there a bug open / patch to review?21:59
slangasekhmm I suppose clicking through the silo will take me to the diffs22:00
rsalvetislangasek: the main bug is https://bugs.launchpad.net/ubuntu/+source/indicator-sound/+bug/136882722:00
ubot5Ubuntu bug 1368827 in indicator-sound (Ubuntu) "[touch] indicator needs to be able to control volume per audio role" [Critical,In progress]22:00
slangasekah, but maybe not for pulseaudio22:01
slangasekok22:01
rsalvetipulse changes are just bugfixes22:01
rsalvetiand also only relevant to touch22:01
rsalvetithe main one that adds a bit more logic is indicator-sound, but also for touch22:01
rsalvetislangasek: I though we had a FFe for it, but saw that it was not included at bug 137163522:01
ubot5bug 1371635 in Ubuntu "[FFe] standing freeze exception in utopic for Ubuntu Touch-specific packages" [Undecided,Triaged] https://launchpad.net/bugs/137163522:01
rsalvetiso my mistake for trying to push it before checking that properly22:01
rsalvetiI added one comment to add indicator-sound to the list of packages22:02
slangasekright, I responded there - basically, it's not included because the package isn't touch-specific22:02
rsalvetioh, you already replied22:02
slangasekdoesn't mean it shouldn't go in, but it means we should be extra careful that it doesn't regress non-touch stuff22:02
rsalvetioh, right, similar to the other indicators in there I guess22:02
rsalvetilet me check22:02
rsalvetiyou're right, the other indicators in there are common to the desktop22:03
rsalvetisorry, not common22:03
rsalvetislangasek: want me to open a separated bug for this package?22:04
slangasekno, we can use the existing bug report22:04
rsalvetislangasek: updated bug22:12
rsalvetiargh22:12
rsalvetibug 136882722:12
ubot5bug 1368827 in indicator-sound (Ubuntu) "[FFe] [touch] indicator needs to be able to control volume per audio role" [Critical,New] https://launchpad.net/bugs/136882722:12
rsalvetiwhat is wrong with my copy & paste22:12
robrubfiller: rtm1222:21
bfillerrobru: fast service today :) thanks!22:21
robrubfiller: you're welcome!22:21
robrubfiller: just a heads up, if you change your MPs and need a reconfigure, the button for that moved off the dashboard and into the spreadsheet. I'll make a public announcement about this soon but I'm still ironing out some bugs here22:22
bfillerrobru: thanks for the heads up22:23
slangasekrsalveti: so the idea here is that the sink-input-by-media-role:$foo are only present if we're on touch?22:54
rsalvetislangasek: they can be part of the desktop, but the dbus module that export that interface to the indicator is only available on touch22:54
rsalvetithe stream can have basically any role, as the app is responsible for setting that up, we just have a small set of supported ones on touch22:55
rsalvetiand this additional work was to enable this extra dbus interface so it can be used to change volume per roles22:55
rsalvetiso if the dbus interface and the needed roles are not available in the stream-restore internal database, it'll act like today (only controlling the main sink)22:56
rsalvetiif the dbus interface is available, and the required set of roles are available in stream-restore, then the volume will reflect on the current active role (or alert by default)22:57
slangasekok. what package provides this new dbus interface?22:57
rsalvetislangasek: pulse itself, but that's disabled by default, and the config enabling that is part of ubuntu-touch-session22:57
rsalvetiwe have a separated pulse audio config for touch22:58
rsalvetithe pulse changes are just basically to fix that use case (like preloading volume per roles, which was broken)22:58
rsalvetiI'm also sending both pulse patches upstream as we speak22:58
slangasekrsalveti: ok, FFe approved, and indicated this in the bug log22:59
rsalvetislangasek: great, thanks so much22:59
rsalvetigreat, glibc also migrated, next image should make the x86 emulator to work again23:01
rsalvetislangasek: ok, both pulseaudio patches were sent upstream, will update the package later on once they get accepted23:43

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