/srv/irclogs.ubuntu.com/2014/07/30/#ubuntu-ci-eng.txt

slangasekcjwatson: robru is internetless for the night, thanks for kicking it off (I'd promised him I would some time around nowish)00:43
imgbot=== trainguard: IMAGE 158 DONE (finished: 20140730 01:05) ===01:04
imgbot=== changelog: http://people.canonical.com/~ogra/touch-image-stats/158.changes ===01:04
imgbot=== trainguard: IMAGE 159 building (started: 20140730 02:05) ===02:04
imgbot=== trainguard: IMAGE 159 DONE (finished: 20140730 03:35) ===03:34
imgbot=== changelog: http://people.canonical.com/~ogra/touch-image-stats/159.changes ===03:34
bzoltan1hello, is anybody here?03:36
alecuhello bzoltan1, I am. But probably not much use03:37
alecubzoltan1: perhaps you can try pinging the train-guards?03:37
bzoltan1alecu:  I wonder if it is possible to get a silo for line3803:38
bzoltan1rsalveti:  said that he will take care of it during his shift03:38
rsalvetibzoltan1: latest qtcreator is already in the archive03:39
rsalvetiit was a minor patch/bugfix, which was already merged in trunk03:39
rsalvetibzoltan1: https://launchpad.net/ubuntu/+source/qtcreator03:40
bzoltan1rsalveti: ohh man, I am sorry :) You rock!03:41
bzoltan1rsalveti:  yes, it now it was merged :) I was bugging the kubuntu devs about it03:41
veebersrobru, trainguards: I see now that traincon-0 is gone, what's the remaining step for me and silo-10?04:45
tvossgood morning07:23
tvosssil2100, good morning07:24
sil2100tvoss: morning!07:24
sil2100thostr_: hey! I'm now trying to publish silo 17 and in the packaging diff I see that you guys are modifying existing symbols - will that not break the ABI?07:35
sil2100thostr_: are those symbols used somewhere in external apps?07:37
jibelsil2100, good morning, so main issue today on 158/159 bug 135015207:40
ubot5bug 1350152 in apparmor (Ubuntu) "[mako #158] apparmor denies access to /etc/ssl/openssl.cnf" [Critical,Confirmed] https://launchpad.net/bugs/135015207:40
sil2100jibel: hi! I just looked on the smoketesting and saw 2 apps not starting, so this is the reason? It's not a transient error now is it?07:41
jibelsil2100, no it is not transient07:41
sil2100I saw new uploads for most click core apps in 158, might be related I guess?07:41
sil2100jibel: thanks for bringing this up :)07:42
jibelsil2100, not sure, because there was no update for gallery-app07:42
jibeland it's affected by this07:42
sil2100Oh, it is? Since the dashboard is not refreshed for me so I don't see full test results for 15907:43
sil2100And no gallery-app there07:43
jibelsame problem with camera-app and no update07:46
sil2100Chipaca: hey!07:50
Chipacasil2100: hoi!07:50
sil2100Chipaca: is the merge you want to release reviewed by someone? ;) It's not top-approved (not sure if you have anyone to review it ;p)07:50
* Chipaca wonders if sil2100 and queuebot are the same person07:51
Chipacasil2100: top-approving....07:51
Chipacadone07:51
sil2100;p07:51
sil2100Thank you!07:51
sil2100I know selfie top-approvals are not too good, but sometimes there is no other choice..!07:52
Chipacasil2100: in ubuntu-push the branches all go through peeer review before reaching the merge-to-trunk stage07:55
Chipacathe only thing i'm approving here is the changelog edits07:55
sil2100Good to know07:56
Chipacasil2100: peer review and automated testing, hence 'automatic'. automatic to trunk involves paperwork and manual testing.07:56
Chipacaand if issues are found, back to peer review (usually. if it's a dumb packaging bug i'll fix it and then merge back and have that peer reviewed instead)07:57
Chipaca(this happened in this landing, btw; found an issue with one of the dbus endpoints; https://code.launchpad.net/~chipaca/ubuntu-push/fix-clear-persistent/+merge/228712 resulted.)07:58
Chipacasil2100: which reminds me. Are you an ubuntu developer? And if so, could you endorse my PPU application?08:02
Chipacai've been having trouble getting people to go from agreeing to endorse it to actually endorsing it. have had it sitting there for something like two months already.08:02
sil2100Chipaca: I'm a MOTU, not sure if I can but let me take a look at the rules :)08:03
Chipacasil2100: :) ok.08:03
brendanddoes anyone know how to get more verbose logs out of ubuntu-app-launch?08:04
* ogra_ curses ... why do i always forget that 08:04
davmor3ogra_, sil2100: somethings come up I should be back for 11, but I'll miss the meeting let me know if you want something dogfooding when I get back08:05
davmor3catch you later guys08:06
brendandactually here's the syslog from starting calculator: http://paste.ubuntu.com/7902644/08:07
brendandapparmor denied08:07
brendandi guess that's not expected08:07
ogra_seen bug 1350152 ?08:07
ubot5bug 1350152 in apparmor (Ubuntu) "[mako #158] apparmor denies access to /etc/ssl/openssl.cnf" [Critical,Confirmed] https://launchpad.net/bugs/135015208:07
brendandogra_, somehow i overlooked that when reading victor's email. thanks08:09
ogra_:)08:09
brendandogra_, who can fix that one?08:10
brendandogra_, also, why no test results for those apps in CI (even if they would be all red)08:10
brendandhmm, well at least gallery and calculator are not there08:11
brendanddropping letters and camera are08:11
ogra_brendand, see plars' email08:16
ogra_psivaa, can we get the missing tests onto the dashboard somehow already or are there still issues ?08:16
psivaaogra_: the uitk went on  its infinite loop. so rerunning it again. might take a bit for it to appear. the rest should appear soon.08:17
psivaaogra_: but with similar blocks of failures as we saw yesterday08:17
ogra_psivaa, thanks !08:18
thostr_sil2100: @mediascanner: those ABI breaks are not visible outside mediascanner as music app uses qml plugin. But to be 100% sure we're giving it another test round... give us 10 minutes08:19
ogra_bah, the last seed change breaks desktop-next08:22
sil2100thostr_: thanks, just need to make sure about that before I +1 it to the archive, since we don't want ABI breaks without soname bumps08:22
ogra_ah, not the last one but the one before ... hmpf08:24
sil2100ogra_: another breakage in desktop unity8?08:25
ogra_sil2100, well, installing qrtmir-android on desktop unity808:25
vilahi there,08:32
vilaI OTA'ed from #133 to #157 but after reboot I'm still on #13308:32
vilaI then tried adh shell > system-image-cli -v which also said upgrading to 157 but still 133 after reboot08:33
vila/etc/ubuntu-build says 133 too08:33
vilaknown issue ? Any idea ?08:33
ogra_vila, i had no issue08:47
sil2100bzoltan: hi!08:48
bzoltansil2100:  hello08:49
bzoltansil2100:  what did i screw up again?08:49
sil2100bzoltan: so, we seem to be haunted by some infinite loops from time to time in UITK (rare, but still happening) during dashboard testing08:49
vilaogra_: good for you ;-) What can I look/try ?08:49
sil2100bzoltan: oh, nothing! I hope ;p08:49
sil2100bzoltan: but could you guys take a look? psivaa fetched some logs on when the actual loop happens:08:50
sil2100bzoltan: http://pastebin.ubuntu.com/7902557/08:50
bzoltansil2100:  sore08:50
bzoltansure I mean08:50
ogra_vila, hmm, not sure08:50
sil2100bzoltan: this is how the logs looked like, with unity8 being on top from what I remember08:50
sil2100psivaa: ^ is that right?08:50
ogra_vila, there are logs in /cache/recovery/ i think08:51
psivaasil2100: right, when it happened unity8 was on top08:51
sil2100bzoltan: as always, might not be related to any changes your team did, but you're our first contact here ;)08:54
bzoltansil2100:  no worries :)08:54
vilaogra_: https://pastebin/canonical.com/114462/08:54
ogra_bah ... 2fa08:55
vilaogra_: hehe, same here ;)08:55
ogra_vila, hmm, looks all fine08:55
bzoltansil2100: That this function used to cause troubles many times. elopio is the one who could tell the real deal08:56
ogra_vila, and "system-image-cli -i" as well as the UI say you are not on 157 ?08:56
ogra_according to that log you should08:56
vilaogra_: yes, both say 13308:57
cjwatsonsil2100: hm, I'm confused, I thought you were sheriff yesterday but I just checked the calendar and it says that should have been me.  Want me to take today?08:57
ogra_vila, weird08:57
ogra_vila, and do you get a new upgrade offered when you click on upgrades in system-settings ?08:58
vilaogra_: even if system-image-cli says: last update: 2014-07-30 10:55: 29 (I retried an OTA)08:58
vilaogra_: yes, for 15708:58
ogra_vila, and did you try to run it again  ?08:58
sil2100cjwatson: ok, sure thing, would be grateful as I have some paperwork to finish and then doing some real test landing and tweaks for the other-distro bits of citrain08:59
vilaogra_: as of now, 2 OTA, 2 system-image-cli08:59
=== cjwatson changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard (general help): cihelp | CI Train support: cjwatson | CI Train Status: #157 promoted | Known issues: Read http://bit.ly/1k4uBJR before asking about the spreadsheet changes. Both queuebot and http://bit.ly/1mDv1FS know your silo status before the spreadsheet does.
cjwatsonsil2100: ok, anything I should know from the meeting?  I guess we can mostly publish fairly freely ...09:00
vilaogra_: trying 'system-image-cli -b 0 -v' just in case09:01
sil2100cjwatson: yeah, normal landings resumed - we're concentrating on getting the current new blockers resolved (the apparmor issue making application launch problems) but other then that, it's normal09:01
sil2100cjwatson: we're thinking of kicking a new image soon anyway as there has been a change to lxc container handling that we'd need tested09:02
vilaogra_: got channel ubuntu-touch/utopic, upgrade path is 15709:02
ogra_vila, sounds right09:03
ogra_perhaps the file is corrupt or some such ... you coudl try to wipe /cache/recovery/09:03
ogra_that should force a re-download09:03
vilaogra_: when is /etc/ubuntu-build ?09:03
tvosspete-woods, ping09:04
ogra_thtas put in place as last step during the upgrade iirc09:04
vilaogra_: the file is not there after reboot, only the logs (will triple check once that -b 0 finish)09:04
tvosspete-woods, ping09:06
cjwatsonpete-woods,tvoss: any reason I can't publish silo 8?  I see it conflicts with silo 109:11
tvosscjwatson, not from my side, pinged pete-woods but he did not come back to me, yet09:12
vilaogra_: doh, it worked ! Thanks for the debugging vibes or whatever trick you used ;) May be it's just because it rebooted at 11h11...09:12
ogra_haha09:13
thostr_sil2100: can you reconfigure silo 1709:14
cjwatsonthostr_: I'll do it09:14
cjwatsonthostr_: (see topic)09:14
thostr_ack09:15
cjwatsonthostr_: conflicts with silo 509:15
pete-woodstvoss: hi09:15
cjwatsonthostr_: can you agree this with Saviq/mhr3?09:15
tvosspete-woods, are we good to go for silo 8?09:16
thostr_given that silo 5 builds right now let's land that first then09:16
Saviqthostr_, go for it, we'll manage09:16
Saviqthostr_, 5 won't land until tomorrow at least09:16
thostr_Saviq: ok, then I'll take the slo09:16
pete-woodstvoss: yep09:16
thostr_cjwatson: ok, then silo 17 first09:16
pete-woodssooner the better really, it fixes a serious scope bug09:16
tvosscjwatson, ^09:18
tvosscjwatson, good to go for silo 809:18
tvosscjwatson, will rebuild silo 1 after that09:18
LaneySaviq: the /wizard/, maybe - but u-s-s itself ought to work09:18
cjwatsonthostr_,tvoss: ok09:21
LaneySaviq: do you know why this dependency isn't on -wizard, but on u-s-s itself?09:25
Laneymterry: ^-- could you check?09:47
davmor2Morning all09:47
davmor2sil2100: you want me to dogfood something?09:47
sil2100davmor2: hey! Not yet ;)09:48
mterryLaney, read scrollback to see that some dep of USS should be on the wiz, but which one?09:48
sil2100davmor2: the current image has some issues that we need jdstrand for09:48
mterryLaney, wiz used to be in same binary package, so I wouldn't be surprised09:48
davmor2sil2100: oh that doesn't sound good09:48
Laneymterry: the qtmir thingy09:49
Laneysec09:50
cjwatsonsil2100: Are we waiting for the security folks to wake up before addressing this apparmor bug?09:50
mterryLaney, yeah...  that could be on the wizard only I believe09:50
sil2100cjwatson: we know that jdstrand was working on it yesterday, and so far he's our only contact regarding these issues09:51
Laneymterry: It might be sensible to do ... (one second)09:51
cjwatsonmkay09:51
sil2100https://bugs.launchpad.net/dropping-letters/+bug/1350152 <- this is the bug (or at least one manifestation of it)09:52
ubot5Launchpad bug 1350152 in apparmor (Ubuntu) "[mako #158] apparmor denies access to /etc/ssl/openssl.cnf" [Critical,Confirmed]09:52
Laneymterry: https://bazaar.launchpad.net/~laney/ubuntu-system-settings/qtmir-wizard/revision/844 10:03
mterryLaney, it used to theoretically support non-mir mode by dynamically loading unity-mir.  I see that with the change to qtcomp, there is still a check for QT_QPA_PLATFORM and a variable called isMirServer, which makes me think we still support non-mir architectures.  greyback?10:04
mterrynot just theoretically -- I ran it in non-mir10:05
Laneywhy was this added then?10:05
Laneythis happened just yesterday10:05
mterryLaney, where supported, we want to use Mir10:05
mterryLaney, though I suppose that c(sh)ould be a recommend rather than depend10:06
Laneythe first attempt was a build-dep to force it to not build on non-qtmir arches10:06
ogra_cjwatson, does my last touch seed change need more than a publisher run ?10:07
* ogra_ doesnt think so but i want to be sure10:07
mterryLaney, greyback is about to sit down, he can explain whether we need it as a recommend or depend anyway10:11
mterryLaney, I'm not sure what the problem we're causing with the dep is right now?10:11
Laneymterry: It's making my allergy flare up10:12
mterryLaney, greyback confirmed in verbal talks that non-mir fallback still works10:12
mterryLaney, so we can leave Arch as any10:12
cjwatsonogra_: We don't use tasks at present for touch, so it doesn't need a special publisher run but does need a metapackage update10:12
Laneymterry: so Recommends?10:12
Laneyassuming something else pulls it in10:12
Laneyor the arch restricted dep if not10:13
ogra_cjwatson, ah, heh, i thought it was exactly the other way round ... will do a meta rebuild then10:13
cjwatsonogra_: Oh, but desktop-next uses tasks10:13
mterryLaney, sure I'm fine a Recommends, seems cleaner10:13
ogra_ah, good10:13
cjwatsonogra_: You may be right, I guess this has no effect on the metapackages ...10:13
cjwatsonogra_: So yeah, just give it a couple of publisher runs to settle10:13
ogra_good10:13
Laneymterry: yay, thanks, MP in a minute10:14
mterryLaney, sorry to irritate your allergies  :)10:14
ogra_just want to have desktop-next buildable again10:14
Laneymterry: wasn't you - I traced it to https://code.launchpad.net/~unity-team/ubuntu-system-settings/use-qtComp/+merge/22554010:14
mterryLaney, yeah but I think I reviewed the packaging to that10:14
LaneyI bet it made u-s-s uninstallable and so angered proposed-migration to which the rest of the MPs were a response10:14
mterryLaney, and didn't notice the arch problem10:14
Laneyah well10:15
cjwatsonLaney: correct10:15
cjwatsonhilarious, everyone at the time was insisting that it was totally broken without mir10:15
LaneyDefinitely not the main application or else I don't know what I'm developing on. :)10:16
LaneyI would have believed someone saying that about the wizard though, but this is in a different binary package anyway10:17
sil2100Chipaca: ok, so I checked and most probably normally I should be able to endorse your application, but hm, it doesn't seem I sponsored any packages for you!10:20
sil2100Chipaca: most probably because when I was doing that, it wasn't noted as a sponsor by me10:21
Chipacasil2100: isn't every landing a sponsorage?10:21
sil2100Chipaca: now it is, before we weren't mentioned as sponsors of a package... so for now I guess Ubuntu will notice only one sponsoring made by me10:22
sil2100And besides, I am a MOTU since over a month only, so not too long10:22
Chipacasil2100: wfm :)10:22
sil2100Chipaca: in any case, could you send me a link to your application? ;)10:22
Chipacasil2100: maybe. https://wiki.ubuntu.com/Chipaca/PPU10:22
mterryLaney, did you mean to leave the "Architecture: any" removal in your merge?10:34
Laneymterry: nope10:35
Laneylook now10:35
tvosssil2100, any update on the propagation status of silo 8?10:37
cjwatsontvoss: please ask the current sheriff rather than bothering sil :)10:42
tvosscjwatson, hey there :) any update on the propagation status of silo 8?10:43
cjwatsontvoss: unity-scopes-{api,shell} are waiting for autopkgtests to run10:43
sil2100No worries, not bothered ;p I'm just busy with a package right now, so sorry for not being responsive!10:43
tvosscjwatson, ah :)10:43
cjwatsontvoss: there's something of a queue (http://d-jenkins.ubuntu-ci:8080/view/Utopic/view/AutoPkgTest/) so might take a bit, but nothing appears to have gone wrong as yet10:44
tvosscjwatson, cool, thank you10:45
thostr_cjwatson: can I get a silo for line 3110:47
cjwatsonthostr_: as it happens I was just in the process of assigning that10:48
thostr_:)10:48
=== MacSlow is now known as MacSlow|lunch
=== alan_g is now known as alan_g|lunch
=== tvoss is now known as tvoss|lunch
jdstrandhi11:46
jdstrandso, bug #1350152 is a new bug11:46
ubot5bug 1350152 in apparmor (Ubuntu) "[mako #158] apparmor denies access to /etc/ssl/openssl.cnf" [Critical,Confirmed] https://launchpad.net/bugs/135015211:46
jdstrandit was not seen with 157 and is entirely different than the two issues seen yesterday11:46
* jdstrand notes that there were 3 entirely different apparmor denials11:47
ogra_jdstrand, did anyone claim it was ?11:47
jdstrandyes11:47
ogra_oh11:47
ogra_(surely isnt)11:47
jdstrandin another email thread and it was mentioned in this channel that I was working on this11:47
ogra_heh, how would you ... the issues were only found while you slept :)11:48
jdstrandI didn't see this bug yesterday and only just saw it a moment ago11:48
jdstrandexactly :)11:48
jdstrandanyway11:48
jdstrandsomething in the platform changed11:48
ogra_it seems for all apps ssl is involved but they seem to look in different locations11:48
jdstrandwe have the openssl abstraction in the networking policy group11:48
jdstrandapps that don't have the networking policy group are seeing this denial11:49
jdstrandie, they are using openssl now when they didn't used to11:49
ogra_http://people.canonical.com/~ogra/touch-image-stats/158.changes11:49
ogra_gnutls update ?11:49
cjwatsonwhole batch of transitions landed11:49
cjwatsonyou wouldn't expect gnutls to change openssl things though!11:50
ogra_well, yeah11:50
cjwatson(and no matches for openssl.cnf in its diff)11:51
cjwatsonit might be easier to track down directly with strace or similar11:51
jdstrandI don't have an up to date emulator handy11:51
* mterry wants a new image11:57
ogra_sil2100, where do we stand with that ? ^^11:58
ogra_(i assume the apparmor triaging will still take a while anyway)11:58
* jdstrand starts regenerating an emulator image, but it is going to take hours11:58
ogra_yeah11:58
sil2100Ok, if that's the case, let's build an image now11:58
jdstrandwhere do we stand on system-image updates with the emulator?11:58
sil2100We can build one before the evening meeting11:58
sil2100(in case some fixes fly in)11:59
sil2100ogra_: should I kick the image?11:59
ogra_sil2100, go for it :)11:59
ogra_jdstrand, probably an rsalveti question11:59
psivaajdstrand: just to make sure.. i dint claim it was the same case, but asking if it was likely.. apologies if that caused further cofusion11:59
* ogra_ has to much HW to use the emulator :P 11:59
=== psivaa is now known as psivaa-lunch
jdstrandthere was no confusion on my end11:59
jdstrand:)11:59
jdstrandit isn't the same issue11:59
mterrysil2100, I want one to test a lxc-android-config change before I can push in locking support12:00
jdstrandpsivaa-lunch: no worries12:00
psivaa-lunchright not sure if there was confusion about my question though12:00
sil2100mterry: right :) We also wanted to have an image with the new lxc-android-config, but wanted to know if the apparmor issue is, by any chance, not a 'quick 5 minute fix'12:00
mterrysil2100, gotcha12:00
sil2100mterry: an image should be building now12:01
* mterry pumps fists12:01
=== cprov changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard (general help): cprov | CI Train support: cjwatson | CI Train Status: #157 promoted | Known issues: Read http://bit.ly/1k4uBJR before asking about the spreadsheet changes. Both queuebot and http://bit.ly/1mDv1FS know your silo status before the spreadsheet does.
ogra_mterry, i assume with that we can finally land the locking stuff ?12:02
mterryogra_, right, assuming everything still checks out12:02
ogra_it will, it will :)12:02
jdstrandI wonders if it is curl12:02
mterryheh, I'm much more pessimistic than you, after my last few months of trying to land things  :)12:02
* ogra_ has a hammer in the drawer in case it wont :) 12:02
imgbot=== trainguard: IMAGE 160 building (started: 20140730 12:05) ===12:04
ogra_such a nice rround number12:05
davmor2ogra_: bound to go wrong then ;)12:05
ogra_davmor2, well, we knoe some apps wont start :)12:05
sil2100Apps are sooo passe12:06
ogra_heh12:06
ogra_davmor2, this is for testing that the system still works after upgrading ... we moved /etc/passwd|group|shadow arouns a last time ...12:07
ogra_*around12:07
ogra_(teh proper fix for the quick hack i added initially when we couldnt boot anymore)12:07
=== MacSlow|lunch is now known as MacSlow
davmor2ogra_, sil2100: so what happened between 157 and 158 to break the universe, /me prepares to blame that ogra_ bloke if it isn't him it'll be that popey bloke12:09
ogra_davmor2, read mails :P12:09
popeynot me guv12:09
ogra_plars wrote one ;)12:09
davmor2ogra_: haven't got that far through my mail stack yet :D12:09
ogra_sjust start from the top, not the bottom :P12:10
cjwatsontvoss|lunch: silo 8 merged, so you can resync 1 now12:11
plarsdavmor2: don't worry, I'll go back and get results for 158 if needed. since 160 is about to hit us, let me try to get fixed credentials for the dashboard first if that's ok :)12:12
davmor2plars: no rush dude I'm gonna get to your email12:12
ogra_plars, i think we can skip 158 ... wont be much different from 15912:12
ogra_(there was only a seed change between the two)12:13
plarsogra_: when I got it up and running, I think 159 had just hit and I didn't see that I was at a transition there, or I would have started both before going to sleep12:13
ogra_no worries12:14
cjwatsonHm, I see that the RTM branch procedure didn't include ofono-phonesim, but that's needed for some autopilot tests12:37
cjwatsonIs there a seed anywhere that includes that stuff?12:38
cjwatsonAlso uoa-integration-tests12:38
cjwatsonOtherwise I might create one just to help with the branching12:39
=== psivaa-lunch is now known as psivaa
=== Ursinha changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard (general help): Ursinha | CI Train support: cjwatson | CI Train Status: #157 promoted | Known issues: Read http://bit.ly/1k4uBJR before asking about the spreadsheet changes. Both queuebot and http://bit.ly/1mDv1FS know your silo status before the spreadsheet does.
jdstrandwhere can I get the click for the calculator?13:11
=== alan_g|lunch is now known as alan_g
cjwatsonjdstrand:13:12
cjwatsonhttp://people.canonical.com/~ubuntu-archive/click_packages/13:12
jdstrandoh, that is convenient13:13
jdstrandthanks!13:13
cjwatsonogra_: could you check out https://code.launchpad.net/~cjwatson/ubuntu-seeds/touch-tests/+merge/228846, please?13:21
cjwatsonI've run germinate over it and it seems to be producing basically plausible results13:22
cjwatsonhttp://paste.ubuntu.com/7904805/ <- the touch-tests result file13:22
=== tvoss|lunch is now known as tvoss
ogra_cjwatson, approved13:25
cjwatsonstar, thanks13:26
cjwatsonpete-woods: https://code.launchpad.net/~pete-woods/net-cpp/uri-builder/+merge/222614 looks like the changelog needs to be remerged/reordered13:27
cjwatsondoesn't look right for that change to be inserted into the middle13:27
tvosscjwatson, is there a way to install a file via packagename.install only for specific architectures?13:28
cjwatsonseveral13:28
cjwatsonyou can preprocess the file, or sometimes dh-exec works, or you can use foo.install.<architecture> files13:29
cjwatsondo you have an example?13:29
tvosshttps://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-001/+packages13:29
imgbot=== trainguard: IMAGE 160 DONE (finished: 20140730 13:30) ===13:29
imgbot=== changelog: http://people.canonical.com/~ogra/touch-image-stats/160.changes ===13:29
tvossfails on archs that don't have android gps HAL support as the respective provider does not get compiled in, and not even the config file /etc/gps.conf is installed13:29
tvosscjwatson, however, /etc/gps.conf is in debian/libubuntu-location-service2.install13:30
ogra_didnt dh_install have an option to use the arch name for the .install file ?13:30
ogra_seems the manpage doesnt know about it13:30
* mterry installs image 16013:30
ogra_but i seem to remember having used it before13:30
cjwatsonogra_: that was one of my suggestions above.  please leave this to me?13:31
ogra_yeah13:31
ogra_i was just wondering why the manpage is silent about it13:31
cjwatsonit's awkward to have two people giving advice :)13:31
davmor2ogra_, sil2100: so 160, what should it fix and is it worth my time looking at it?13:31
cjwatsonit's in debhelper(7) because it's a general facility, not specific to dh_install13:31
ogra_oh, right !13:31
ogra_thanks13:31
ogra_davmor2, it should only prove that you still have a running system after upgrade (like i said above ... we are testing the passwd|shadow|group change with it)13:32
davmor2ogra_: right okay so I ota and should have a system that boots right?13:33
cjwatsontvoss: ok, give me a minute13:33
ogra_davmor2, right13:33
tvosscjwatson, thank you :)13:33
ogra_davmor2, and the same on a fresh flash indeed13:33
cjwatsontvoss: ok, so there are two sensible alternatives here, and I'll leave it up to your taste which you choose13:36
davmor2ogra_: Good news I am into unity8 on mako13:36
davmor2ota13:36
cjwatsontvoss: you can revert the change to debian/libubuntu-location-service2.install, create debian/libubuntu-location-service2.install.with-gps with the extra entry, and symlink debian/libubuntu-location-service2.install.amd64, debian/libubuntu-location-service2.install.armhf, and debian/libubuntu-location-service2.install.i386 to that13:37
mterrykenvandine, heyo, if I wanted to land my u-s-s locking branch today, any conflicts?13:37
cjwatsontvoss: or you can move debian/libubuntu-location-service2.install to debian/libubuntu-location-service2.install.in and arrange to preprocess it with sed or similar, by adding a make rule to debian/rules to generate it and having override_dh_install depend on that new target; you'd also want to add debian/libubuntu-location-service2.install to debian/clean in that case13:38
pete-woodscjwatson: updated :)13:38
davmor2ohhhhh I like the icon for the app in the start up frame with the spinner :)13:38
cjwatsontvoss: (silo 8 ready for you, if you didn't notice)13:39
=== Ursinha is now known as Ursinha-afk
=== pete-woods is now known as pete-woods|lunch
tvosscjwatson, thank you, I will go for option one. and thanks for the silo, will finish off silo 1, first, though13:40
cjwatsontvoss: ok13:40
ogra_davmor2, awesome13:41
kenvandinemterry, are you ready now?13:41
cjwatsonpete-woods|lunch: surprised to see no symbol file updates in that branch.  are you planning to just get that from the first build failure or something?13:41
kenvandinemterry, i'll have some stuff i want to try to land later today13:41
ogra_kenvandine, locking ?13:41
kenvandinemterry, but not all ready yet, so go ahead13:42
mterrykenvandine, I'm basically ready.  Final testing13:43
kenvandinemterry, sweet13:43
kenvandineogra_, ask mterry, i'm just know it has to land along with some other stuff13:43
pete-woods|lunchcjwatson: at least when I made that branch, net-cpp didn't do symbol tracking13:43
pete-woods|lunchperhaps that's changed, though13:43
* pete-woods|lunch re-running bzr bd13:44
cjwatsonpete-woods|lunch: oh, seriously?  ugh, ok13:44
pete-woods|lunchit's not my project :p13:44
=== pete-woods|lunch is now known as pete-woods
pete-woodsokay, it has changed now, will update the symbols file13:46
brendandsil2100, messaging-app is ok, but there is a definite issue with notes-app. i need to file a bug13:50
pete-woodscjwatson: okay, have added the symbols now13:52
=== pete-woods is now known as pete-woods|lunch
ogra_brendand, you mean beyond the apparmor denial ?13:53
brendandogra_, notes didn't have an issue with that13:53
ogra_ah, k13:54
brendandogra_, at least it doesn't crash on start13:54
mterryrobru, heyo!  I think silo-012 is ready to land13:56
mterrysil2100, ^13:57
robrucjwatson, ^ I haven't started my shift yet ;-)13:57
cjwatsonpete-woods|lunch: thanks.  sorry, yet another thing - wouldn't it be usual to bump the minor number for new interfaces rather than the patch number?  That would make this 1.1.0 rather than 1.0.1 (more importantly, NET_CPP_VERSION_MINOR 1 and NET_CPP_VERSION_PATCH 0)13:57
cjwatsonpete-woods|lunch: (I didn't mention that before since wasn't certain that there was new ABI involved, but there certainly is if you have to change .symbols)13:58
mterryrobru, sorry man  :)13:59
mterryrobru, I'm in London, my timing is all messed up13:59
=== Ursinha-afk is now known as Ursinha
ogra_mterry, he is in europe too ... just pretending to not work :P13:59
mterryrobru, oh there's a CI train support person in the title now13:59
robrumterry, I'm in Strasbourg, my timing is messed up too ;-) But I know my shift doesn't start for 2 more hours ;-)13:59
robrumterry, yeah13:59
robruwell maybe sil and cj have plans for what to land, I don't want to step on their toes until after the daily hangout14:00
=== plars changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard (general help): plars | CI Train support: cjwatson | CI Train Status: #157 promoted | Known issues: Read http://bit.ly/1k4uBJR before asking about the spreadsheet changes. Both queuebot and http://bit.ly/1mDv1FS know your silo status before the spreadsheet does.
sil2100brendand: thanks!14:04
tvosscjwatson, could use some help here: https://ci-train.ubuntu.com/job/landing-001-1-build/152/console14:10
cjwatsontvoss: symlinks in debian/ only work in 3.0 source formats14:10
tvosscjwatson, oh ... adjusting14:10
cjwatsontvoss: can you make this be 3.0 (quilt)?14:11
cjwatsonsorry, forgot to mention that, everything I maintain has been 3.0 for years :)14:11
tvosscjwatson, I only need to adjust source/format, correct?14:11
cjwatsontvoss: I believe so; there's no manual patch system junk to undo in this case, or anything.  I'm less familiar with whether bzr bd will be happy, but hopefully it should be14:13
cjwatson(have mostly stopped using bzr for packaging ...)14:13
tvosscjwatson, ack, just giving it a spin locally14:13
mterrycjwatson, I forget what's involved in landing a silo, but I'd like to initiate the process for silo 01214:23
mterrypresumably it's as simple as pressing the publish button?14:23
cjwatsonmterry: yes, though I'd have done it for you if I hadn't been distracted - I'll just do it now14:26
mterrycjwatson, oh OK thanks14:26
* mterry is excited to see this land finally14:26
ogra_lock 'n loll !!!14:28
mterryogra_, :)14:31
mterrykgunn, ^14:31
barryanybody know what's up with this build failure?  https://launchpad.net/~canonical-ci-engineering/+archive/ubuntu/ci-airline-phase-0/+build/6222466   is this an archive problem and do we need a no-change rebuild of zope.proxy?14:36
cjwatsonbarry: does it fail against plain trusty locally?  I notice that that PPA contains zope.proxy14:38
cjwatsonbarry: so I'd guess it to be a problem local to that PPA14:39
barrycjwatson: yeah, it's probably local to the PPA because z.untrustedpython isn't in trusty14:39
plarscjwatson: do we know when the official stable-proposed will happen? I don't think there have been any updates to the staging one right?14:40
barryi guess i won't worry about it until someone!=me complains :)14:40
cjwatsonplars: see thread on ubuntu-phone@14:40
plarsogra_, sil2100, davmor2: we have live results again14:40
ogra_yay14:41
sil2100plars: \o/14:41
sil2100Thanks :)14:41
cjwatsonplars: I was about to copy a few more packages into the current RTM branch and ask for a rerun of a few tests in touch_stable, but wasn't planning to do an image rebuild14:41
jdstrandok, finally have a emulator up for the openssl.cnf denial14:41
plarscjwatson: ok14:42
ogra_oh, why is touch_stable so much less broken ?14:43
ogra_must be the name14:43
cjwatsonogra_: it was branched from some time ago14:44
ogra_ah14:44
cjwatsonthere are three test suite failures that are down to the touch-tests branch.  The other tests, no idea, they don't look like they're related to the archive branch so I'm not planning to investigate much14:45
bzoltan1I just pushed the 160 and many apps do not start... calculator, camera14:55
ogra_bzoltan1, read your emails :P14:57
ogra_(we have apparmor issues for a handfull of apps)14:57
bzoltan1ogra_: I missed that mail14:59
pmcgowanbzoltan1, jdstrand is looking into it now14:59
bzoltan1pmcgowan: thanks14:59
pmcgowanbzoltan1, apps are accessing openssl.cnf for some reason15:00
alecuping trainguards. Can I ask for a silo for click-scope, row 34?15:00
pmcgowanbzoltan1, anything added to the toolkit to do that?15:00
cjwatsonalecu: I haven't assigned that because it clashes with silo 515:00
bzoltan1pmcgowan:  not as I know15:02
alecucjwatson: thanks for the heads up, I'll see about solving that.15:05
cjwatsonalecu: if the owners of that silo will be landing after you and are happy to rebase, I'll assign, but please check first15:05
alecusure15:06
pete-woods|lunchcjwatson: will ping tvoss for his preference :)15:11
cjwatsonpete-woods|lunch: thanks, sorry for the back-and-forth15:12
=== pete-woods|lunch is now known as pete-woods
mhr3cjwatson, it's fine to give the override to the apps scope, already done that for the mediascanner one15:13
jdstrandok, if I downgrade gnutls and curl, there is no denial15:15
jdstrand3.2.15-1 and 7.36.0-2ubuntu115:15
pete-woodscjwatson: okay, that's updated now, too15:16
pete-woodshopefully I will not make further trivial errors :)15:16
pmcgowanjdstrand, interesting, whats the tie in to an app?15:18
cjwatsonjdstrand: can you see exactly where in that chain?  I don't see anything in either diff15:19
cjwatsonsil2100: ooh15:20
* sil2100 is in the middle of eliminating all the small typos he's been making just now15:22
ogra_cute !15:22
jdstrandpmcgowan: I don't know yet15:23
tvossrsalveti, ping15:23
jdstrandcjwatson: I haven't been able to yet15:23
cjwatsonNeither gnutls28 nor curl mentions openssl.cnf in its source15:23
cjwatsonI guess something is calling out to openssl for some reason?15:23
tvossrsalveti, just checked syslog, no more denies on /etc/gps.conf with silo 1, setting tested to true15:24
ogra_cjwatson, jdstrand i see there are fixes for the nss build in curl ... could it be related to i.e. calling getpwent() by the app which then goes through nss with somehow then (for whatever reason) looks for ssl stuff ?15:29
ogra_(as you might see i'm wildly guessing here)15:29
cjwatsonno idea15:29
jdstrandcjwatson: I did strings on all the libs, the only thing that references it is /lib/i386-linux-gnu/libcrypto.so.1.0.015:29
cjwatsonthis is why I suggested strace15:29
jdstrandI tried strace, but I lack the fu to see what library is doing the access15:30
tvossjdstrand, is that with every app?15:30
ogra_tvoss, only a bunch15:30
cjwatsonjdstrand: ltrace -S might be worth trying too, if it works15:30
ogra_gallery, camera ...15:30
cjwatson(IME ltrace is not the most reliable tool in the box, but useful when it does work)15:30
jdstrandtvoss: as it happens, the networking policy group specifies the openssl apparmor abstraction, which allows that access. in other words, this only affects apps that do not specify the networking policy group15:31
tvossjdstrand, hmmm ... which apps do try the access?15:31
tvossjdstrand, just straced the camera app, did not see the open call15:31
jdstrandthey are listed in the bug. calculator is the one I am looking at15:31
jdstrandI'm told camera-app too15:32
ogra_and gallery15:32
jdstrandtvoss: you need at least image 158 on arm. I'm using 162 on x8615:32
tvossjdstrand, I'm on the latest image here15:33
ogra_tvoss, and gallery starts for you ?15:33
pmcgowanhttps://bugs.launchpad.net/gallery-app/+bug/135015215:33
ubot5Launchpad bug 1350152 in apparmor-easyprof-ubuntu (Ubuntu) "[mako #158] apparmor denies access to /etc/ssl/openssl.cnf" [Critical,In progress]15:33
tvossogra_, nope, just tried15:33
ogra_right15:34
tvossogra_, is the denial the cause of the startup issue?15:34
jdstrandtvoss: latest promoted or latest proposed? latest promoted doesn't have the issue15:34
ogra_tvoss, seemingly15:34
ogra_tvoss, you coudl disable apparmor and test for laughs ...15:34
tvossogra_, interestingly, starting the app from the command line works15:35
ogra_tvoss, well, i guess that doesnt go through UAL15:36
tvossjdstrand, cjwatson I don't get a denied in /var/log/syslog when starting gallery from the command line15:37
ogra_because you dont call aa-exec which UAL does15:37
ogra_(i guess)15:37
tvossogra_, ah yeah ... true15:38
=== Ursinha is now known as Ursinha-afk
ogra_strace and frineds should still show you if/when it tries to open the file15:39
jdstrandtvoss: how are you starting it?15:39
* jdstrand is using: /tmp/trace aa-exec-click -p com.ubuntu.calculator_calculator_1.3.291 -- qmlscene -qt5 ubuntu-calculator-app.qml --desktop_file_hint=~/.local/share/applications/com.ubuntu.calculator_calculator_1.3.291.desktop15:40
jdstrand(from within phablet-shell)15:40
jdstranderr15:40
jdstrandget rid of that first /tmp/trace :)15:40
jdstrandaa-exec-click -p com.ubuntu.calculator_calculator_1.3.291 -- qmlscene -qt5 ubuntu-calculator-app.qml --desktop_file_hint=~/.local/share/applications/com.ubuntu.calculator_calculator_1.3.291.desktop15:41
jdstrand(that is after a cd to /usr/share/click/preinstall/com.ubuntu.calculator/current15:41
tvossjdstrand, phablet@ubuntu-phablet:/usr/share/click/preinstalled/com.ubuntu.gallery/current$ strace ./gallery-app --desktop_file_hint=/usr/share/applications/webbrowser-app.desktop 2> /tmp/strace.txt15:41
tvossjdstrand, interestingly, that gives me a call to open on open("/usr/lib/ssl/openssl.cnf", O_RDONLY|O_LARGEFILE)15:42
jdstrandtvoss: that doesn't run under confinement15:42
tvossjdstrand, right15:42
tvossjdstrand, still interesting that the access is different with no confinement15:43
om26erfginther, Hi! Can you please tell whats going wrong here: https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/1802/console ?15:46
om26erI see its extracting address-book-service-testability deb but when it comes to installing it its not able to find the package15:46
fgintherom26er, does this happen to be a new package that isn't in the archive yet?15:48
=== Ursinha-afk is now known as Ursinha
om26erfginther, yes15:48
fgintherom26er, I was wondering when this was going to bite us. The test tries to install packages from the archive first before looking at the local packages. I might be able to find a fix15:51
jdstrandltrace showed nothing15:52
tvossjdstrand, \o/15:52
jdstrandI'm inclined to just allow the access in the templates since allowing the access allows the app to start, but it would be nice to know why this happened15:53
jdstrand(and the app didn't magically need networking)15:53
rsalvetijdstrand: ogra_: xnox was working on emulator updates (system-image), not sure what is the current status though15:53
* jdstrand nods15:55
sil2100ogra_: hm, I need to jump out somewhere right now, could you lead the meeting?15:58
jdstrandso, if I downgrade just curl, I don't see it, but that doesn't mean it isn't in gnutls since the new curl needs the new gnutls15:58
sil2100ogra_: just give me a sign on the updates15:58
jdstrandCurl_ossl_init: call OPENSSL_config for initing engines15:59
xnoxjdstrand: sounds scary. If gallery app tries to open openssl.cnf, it's probably got transitively linked to openssl for which most of our apps to do not have exception. (all but notes-app)16:00
jdstrandI think it is this: http://curl.haxx.se/mail/lib-2014-06/0003.html16:00
jdstrandxnox: I think this has to do with the compat stuff16:00
* sil2100 jumps out now16:00
xnoxjdstrand: should die with fire, as no need on ubuntu-touch? one can't and shouldn't modify non-default openssl stuff =)16:01
xnoxjdstrand: or like, handle the denial / initialisation gracefully.16:01
jdstrandxnox: no on ewants to modify, it is just a 'r'16:01
=== gatox is now known as gatox_lunch
tvosscjwatson, I'm confused, what am I doing wrong here? https://ci-train.ubuntu.com/job/landing-008-1-build/185/console16:04
cjwatsontvoss: that's due to https://launchpad.net/ubuntu/+source/dbus-cpp/4.0.0+14.10.20140718-0ubuntu2 - you could merge that manually onto lp:dbus-cpp first (just a changelog delta), or since it's just a no-change rebuild we could ignore that.  which would you prefer?16:07
tvosscjwatson, how do I merge manually?16:08
cjwatsontvoss: oh, I can do it16:09
cjwatsonjust a direct push to lp:dbus-cpp, but since I apparently have access I'll sort it out16:09
tvosscjwatson, would be great to get trunk in sync with distro :)16:10
cjwatsontvoss: done, you can retry16:11
tvosscjwatson, looking good16:12
tvosscjwatson, for silo 1, any eta on the migration?16:13
brendandsil2100, ogra_ - notes-app bug https://bugs.launchpad.net/notes-app/+bug/135036116:13
ubot5Launchpad bug 1350361 in notes-app "test_note_focus_on_second_click_outside failed on mako #159" [Undecided,New]16:13
ogra_sil2100, ^^^ for the mail16:13
cjwatsontvoss: the publisher seems to be particularly slow at the moment - I'll look into that16:15
cjwatsontvoss: it's still publishing to -proposed16:15
* tvoss hugs the poor publisher16:15
jdstrandI've updated the bug and am working on an update for apparmor-easyprof-ubuntu now16:16
cjwatsontvoss: I've seen this kind of thing before - I think it just needs an ANALYZE.  have asked webops for that16:17
* ogra_ hugs jdstrand 16:18
ogra_awesome !16:18
davmor2jdstrand: already man you rock \m/16:25
jdstrandthanks!16:25
cjwatsonok, publisher hopefully faster now16:26
jdstrandI'll upload it after I'm done running the test plan16:26
ogra_robru, perhaps wait with an image build for that fix16:26
robruogra_, alright16:26
cjwatsonpointless ~10-minute delays FTL16:26
robruogra_, so I just tried opening some apps. only the most recently used app live updates in the switcher.16:26
davmor2jdstrand: does the testplan now include opening the apps that didn't open?16:26
ogra_robru, right, only the running one16:27
robruogra_, i thought you said they all did? that would be *way* cooler ;-)16:27
davmor2jdstrand: daft question I understand but it might be worth adding for future test runs16:27
jdstranddavmor2: no. but it does include running the smoke image tests, which test for various file access, which I just updated for this16:28
ogra_robru, heh, it would look good but eat your battery :)16:28
jdstranddavmor2: I of course did test that the updated package fixes it for the calculator16:28
jdstrandin other words-- yes, I am testing it and adding a test for the future16:28
davmor2jdstrand: thats cool then as long as it was covered for future releases :)16:28
jdstrandit is16:29
jdstrandoh, we have -dev3 now16:29
robrujdstrand, do you need a silo for that or are you just going to upload it directly?16:31
jdstrandrobru: will upload directly16:36
robrujdstrand, cool, gimme a ping when that's done so I can kick an image. thanks!16:36
jdstrandsure thing16:37
camakofginther, I've now branched mir/0.6. would you mind setting it up for jenkins?16:40
=== robru changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard (general help): plars | CI Train support: robru, trainguards | CI Train Status: #157 promoted | Known issues: Read http://bit.ly/1k4uBJR before asking about the spreadsheet changes. Both queuebot and http://bit.ly/1mDv1FS know your silo status before the spreadsheet does.
=== robru changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard (general help): plars | CI Train support: robru, trainguards | CI Train Status: #157 promoted | Known issues: Both queuebot and http://bit.ly/1mDv1FS know your silo status before the spreadsheet does.
cjwatsonrobru: thanks16:41
robrucjwatson, you're welcome!16:41
cjwatsongah, publisher still slothful, am out of ideas.  if it's still like this next week then I'll see if I can work with William to improve things16:50
cjwatsonit might need the new DB servers plugged in before we can improve it much :-/16:50
=== alan_g is now known as alan_g|EOD
=== gatox_lunch is now known as gatox
jdstrandrobru: apparmor-easyprof-ubuntu 1.2.13 uploaded17:07
robrujdstrand, thanks!17:07
jdstrandnp17:07
cjwatsonplars: Could you please re-run the online_accounts_ui, dialer_app, and messaging_app suites in touch_stable (for all platforms that have each one)?  I copied some more packages into the dry-run archive, so those suites should be happier now.17:10
plarscjwatson:  will do17:14
robrujdstrand, was your upload accepted? I don't see it in proposed.17:29
robrutvoss, alecu, Saviq, kenvandine: anybody around to use a silo if I give you one? I have a couple.17:32
robruboiko, ^17:32
boikorobru: me! :)17:33
robruboiko, ok ;-)17:33
kenvandinerobru, soon :)17:33
Saviq:D17:36
jdstrandrobru: it is listed here: https://launchpad.net/ubuntu/+source/apparmor-easyprof-ubuntu/1.2.1317:38
jdstrandit built17:38
jdstrandhttps://launchpad.net/ubuntu/+source/apparmor-easyprof-ubuntu/+publishinghistory17:38
jdstrandshows it pending for proposed17:39
jdstrandI think I saw somewhere that the publisher is a little slow today. maybe that is it17:39
robrujdstrand, oh yeah, right.17:39
robruSaviq, want a silo then?17:39
Saviqrobru, don't think I have anything that's worth a silo now17:40
Saviqkgunn, unless we want dash-as-app in?17:40
Saviq↑17:40
robruSaviq, the spreadsheet shows a pending request from you and mhr317:40
Saviqrobru, does it now!17:41
robruSaviq, line 3517:41
Saviqrobru, that's in silo17:42
robruoh wait17:42
robruhrm17:42
robruSaviq, ah, got confused with line 25, which says it's superceded by 35. ok, deleting that17:43
alecurobru: me too!17:43
robrualecu, ok!17:43
robrualecu, you want line 33?17:44
alecurobru: yes, please. And I've got permission from mhr3 to override silo 5 that also holds click scope.17:44
robrualecu, boiko: ok please build17:50
boikorobru: thanks17:51
robruboiko, you're welcome!17:51
robrucyphermox, ok, you got 1717:54
robrucyphermox, will publish 3 soon, just waiting for some stuff in proposed so I can kick an image17:57
cyphermoxrobru: sure, thanks!17:57
robrucyphermox, you're welcome!17:58
plarscjwatson: we seem to be having trouble with the screen unlocker now. Going to try switching up the order but I don't suspect it will help17:58
cjwatsonhuh, can't see why that would have changed in stable17:59
plarsdbus.exceptions.DBusException: org.freedesktop.DBus.Error.AccessDenied: Rejected send message, 1 matched rules; type="method_call", sender=":1.24" (uid=32011 pid=2178 comm="python3 -c import dbus, logging;from unity8 import") interface="com.canonical.powerd" member="requestSysState" error name="(unset)" requested_reply="0" destination=":1.1" (uid=0 pid=899 comm="/usr/bin/powerd ")17:59
plarscjwatson: I think we've seen this bug in the unlocker before, let me check to see if it looks the same as the one I filed previously17:59
plarscjwatson: the one I've normally seen before is https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1347183 but maybe it's the same cause?18:00
ubot5Launchpad bug 1347183 in unity8 (Ubuntu) "unity8 unlocker code fails sometimes" [Undecided,Confirmed]18:00
cjwatsonplars: I know nothing18:01
robrubfiller, we're out of silos, you'll have to wait abit18:05
bfillerrobru: np, I have a couple hoping to release shortly too18:05
robrubfiller, sure, just gotta build an image before I can release anything18:06
tvossrobru, yup18:06
tvossrobru, line 41 could use one18:06
robrutvoss, oh yeah, now you respond ;-)18:06
kgunnrobru: can i getta silo reconfig on silo14 ?18:07
kgunnrobru: and not sure i thanked you y'day for last minute tap dance on qtcomp...so thank you18:07
robrukgunn, you're welcome!18:07
robrukgunn, also yes18:08
ToyKeeperkgunn: The fix for the poweroff-dialog-on-screen-on already landed, right?18:27
ToyKeeperI just ran into that twice in a row on image 159.18:28
* sil2100 is back18:31
sil2100Ok, doing teh e-mailz18:31
sil2100ogra_, brendand: thanks18:32
ogra_sil2100, so since 160 had not much changes we wanted to wait for 161 ... apparmor fix is in flight in the archive and robru will roll an image once it is in18:34
robruogra_, sil2100: yep apparmor is in proposed, will kick an image as soon as I see that land18:34
sil2100o/18:36
sil2100Thanks guys, excellent news that a fix is in18:37
sil2100This would mean that we might have something for tomorrow morning18:37
ogra_i also just uploaded an lxc-android config that should fix audio recordinng18:37
ogra_(sits in proposed as well and should make the next publisher run)18:38
kgunnToyKeeper: hmmm...guess its just racy...18:39
kgunncan you reopen ?18:39
ToyKeeperkgunn: Looks like it was never closed...  but I did update it.18:43
* ogra_ grumbles 18:45
ogra_http://people.canonical.com/~ogra/ubuntu-touch/broken-download.png18:45
tvossdavmor2, around?18:46
davmor2tvoss: I am currently yes18:47
ogra_gatox, ^^^i assume i shouldnt see the dbus error there (it stays if i resume the download and the button keeps saying "resume")18:47
gatoxogra_, no18:48
ogra_http://people.canonical.com/~ogra/ubuntu-touch/broken-download2.png18:49
ogra_for the record :)18:49
plarsmterry: doe https://jenkins.qa.ubuntu.com/job/utopic-touch_stable-mako-smoke-daily/6/console look to be similar to that other unlock bug I filed? I can't seem to get past this one by rerunning18:49
mterryplars, oh!?  always reproducable?18:50
plarsmterry: I've run it 4 times or so on that image, and hitting it every time so far18:51
plarsmterry: note that this is on the stable-staging-proposed channel image18:51
mterryplars, I can look at it tomorrow18:52
oSoMoNrobru, hey, can silo 2 be published?18:56
robruoSoMoN, yep, once apparmor finishes landing and I kick an image18:56
oSoMoNrobru, ok, thanks. Is there an ETA for the apparmor landing?18:57
robruoSoMoN, it's in proposed already, autopkgtests are running. not sure. should be an hour or two I guess.18:57
cjwatsonautopkgtests are done18:58
cjwatsonso needs publisher, proposed-migration, publisher18:59
cjwatsonan hour?18:59
cjwatson(p.s. grumble it wasn't this slow not that long ago)19:00
cjwatsonmaybe I shouldn't have given up on my optimise-dominator branch so early19:00
* ogra_ hopes lxc-android-config makes the same publisher run 19:02
ogra_but it looks like it is one cycle behind :/19:02
=== fginther changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard (general help): fginther | CI Train support: robru, trainguards | CI Train Status: #157 promoted | Known issues: Both queuebot and http://bit.ly/1mDv1FS know your silo status before the spreadsheet does.
davmor2popey: can you try and play a video from the video scope please  I'm getting No video selected to play19:22
davmor2ToyKeeper: ^19:22
ToyKeeperdavmor2: Under what conditions?19:27
davmor2ToyKeeper: just got to the video scope select a video that is on the device from the carousel and then on the details page hit play19:29
ToyKeeperdavmor2: Yeah, but what image?  It worked fine last time I tried, on 159.19:29
davmor2ToyKeeper: 16019:29
ToyKeeperSure, after I update.19:30
pmcgowandavmor2, ack I see that here19:35
davmor2pmcgowan: I blame ogra who will no doubt blame mterry :)19:36
ogra_its all mterry's fault !19:36
davmor2see19:36
davmor2ogra_: I'm wondering if it is a side effect of the permissions fix19:37
ogra_what exactly ?19:37
davmor2ogra_: I can see them in the scope but mediaplayer won't play videos saying there are none19:37
robrukenvandine, you got silo 2019:38
kenvandinerobru, you sure are anxious to get rid of silos :)19:38
kenvandinerobru, thx!19:38
robrukenvandine, you're welcome!19:38
pmcgowandavmor2, this worked in 159?19:38
ogra_davmor2, well, that cant really be related to the passwd changes ...19:38
kenvandinei'm hoping to add a couple more branches to that, if i can get ci passing19:38
davmor2pmcgowan: ToyKeeper said yes I went from 157 to 16019:39
robrukenvandine, branches on the same source project? or different ones?19:39
ogra_at least i couldnt imagine how ... since the passwd stuff obviously works if you can start the session19:39
pmcgowan158 had a ton of landings, but not 159 or 16019:39
ogra_davmor2, did you check syslog for denials ?19:39
ogra_pmcgowan, well, 160 was specifically built for the passwd change ...19:40
pmcgowanok19:40
davmor2ogra_: not had time to look anywhere yet busy with bug confirmation for rtm14 tags19:40
mterryogra_, what happened?19:40
ogra_locations service and push client just got lucky to hit the archive at the right time ;)19:40
ogra_mterry, seeems davmor2 cant play videos from the scope anymore19:40
mterryogra_, apparmor again?19:41
davmor2ogra_, mterry: local videos show in the scope and then you hit play and mediaplayer say it can't there are no videos19:41
ogra_mterry, well, not sure19:42
pmcgowanJul 30 15:42:13 ubuntu-phablet kernel: [14261.663293] type=1400 audit(1406749333.369:13404): apparmor="DENIED" operation="open" profile="/usr/bin/mediascanner-service-2.0" name="/var/lib/extrausers/passwd" pid=3153 comm="mediascanner-se" requested_mask="r" denied_mask="r" fsuid=32011 ouid=019:42
jdstrandthat was fixed a while ago19:42
pmcgowanthats 16019:43
pmcgowanunless I ave cruft19:43
pmcgowanhave19:43
ogra_jdstrand, we changed it a bit ... the file is now an actual file again and the dir is a bind mount ... up to 159 the file was a bind mount19:44
ogra_could that have any impact ?19:44
jdstrandpmcgowan: apparmor_parser -p /etc/apparmor.d/usr.bin.mediascanner-service-2.019:45
jdstrandpmcgowan: actually, just this: apparmor_parser -p /etc/apparmor.d/usr.bin.mediascanner-service-2.0 |grep extrauser19:45
jdstrand  # When using libnss-extrausers, the passwd and group files are merged from19:45
jdstrand  /var/lib/extrausers/group  r,19:45
jdstrand  /var/lib/extrausers/passwd r,19:45
pmcgowansame19:46
jdstrandogra_: it shouldn't19:46
ogra_jdstrand, even if an old profile exists where the file was a bind mount ?19:47
ToyKeeperdavmor2: Same here, on image 160 videos give me the same error.19:47
jdstrandogra_: wait, what changed?19:47
jdstrandI thought we were talking about the libnss-extrausers file19:47
jdstrandpmcgowan: can you paste the output of 'stat /etc/apparmor.d/usr.bin.mediascanner-service-2.0 /etc/apparmor.d/cache/usr.bin.mediascanner-service-2.0'19:48
ogra_jdstrand, http://launchpadlibrarian.net/181042722/lxc-android-config_0.178_0.179.diff.gz19:48
jdstrandyeah, that shouldn't matter. the path in the denial is the same19:49
ogra_ok19:49
pmcgowanhttp://pastebin.ubuntu.com/7907782/19:49
jdstrandpmcgowan: I'm at a loss19:51
pmcgowanthats not good !19:52
jdstrandlet me see something19:53
jdstrandpmcgowan: can you do 'tar -zcvf /tmp/aa.tar.gz /etc/apparmor.d' and put aa.tar.gz somewhere?19:54
pmcgowanjdstrand, email ok?19:55
jdstrandpmcgowan: sure19:55
robruogra_, ugh, this is the logest migration EVAR19:57
ogra_yeah19:57
jhodappfginther, can I get a silo for line 46 in the spreadsheet?20:01
jhodappor robru ^20:02
robrujhodapp, nope, none left, sorry20:03
jhodapprobru, ok np, it's not critical that I have one today20:03
jhodapprobru, I'll check back tomorrow20:03
bdmurrayslangasek: given the SystemImageInfo in https://errors.ubuntu.com/oops/e3443c10-176b-11e4-ad0f-fa163e5bb1a2 - what do you think we would want a counter for? "version version" or "device name:version version"?20:20
ogra_ugh20:22
ogra_140 packages of KDE stuff just hit the publisher ... i guess that wont help speeding it up20:22
robruogra_, ah, lp says it's migrated. just waiting for rmadison to catch up with lp20:24
robruI mean apparmor20:24
ogra_ah, sad, i was hoping lxc-android-config would make it too20:24
pmcgowandavmor2, do you see those apparmor denials on mediascanner?20:26
pmcgowanor ToyKeeper ^^20:26
ToyKeeperpmcgowan: I get a bunch of those on first boot anyway, because my post-flash script pushes a bunch of media with the wrong permissions.20:27
ToyKeeperI need to modify it to push into /tmp, chown, then move it into /home.20:27
jdstrandwait20:27
pmcgowanok20:28
jdstrandthe denials we are interested in are specifically ones for /var/lib/extrausers/passwd20:28
ToyKeeper(instead of pushing into /home then chowning it)20:28
jdstrandToyKeeper's denials are different20:28
pmcgowanI should have been more specific20:29
ogra_personal denials for everyone ... yay20:29
pmcgowan I fear the day is coming when I need to wipe20:30
jdstrandtyhicks: hey, let's wait for sbeattie20:30
tyhicksack20:30
pmcgowanthe original question is why dont vids play20:31
pmcgowanjdstrand, seems my issue is separate from just playing videos20:31
ogra_pmcgowan, the media-player log should reveal that ... or the url-dispatcher20:31
ToyKeeperpmcgowan: The videos get thumbnailed just fine, but the media player doesn't play them.20:31
jdstrandtyhicks, sbeattie: hey, so pmcgowan has a bizarre situation where the policy for usr.bin.mediascanner-service-2.0 has '/var/lib/extrausers/passwd r,' in it, but is still getting a denial20:32
ToyKeeperpmcgowan: I see something odd with music too.  The scope preview can play them, but the music app launches as if no file were specified.20:33
jdstrandtyhicks, sbeattie: this happens after verifying the profile with:20:33
jdstrand$ apparmor_parser -p /etc/apparmor.d/usr.bin.mediascanner-service-2.0 |grep extrauser20:33
slangasekbdmurray: we would want counters for version ubuntu, and version version20:33
jdstrand  # When using libnss-extrausers, the passwd and group files are merged from20:33
jdstrand  /var/lib/extrausers/group  r,20:33
jdstrand  /var/lib/extrausers/passwd r,20:33
ToyKeeperpmcgowan: So, I suspect it's not a permission issue at all, but rather a matter of the scope passing the filename to the app.20:33
jdstrandtyhicks, sbeattie: running apparmor_parser -r /etc/apparmor.d/usr.bin.mediascanner-service-2.0 didn't help20:33
slangasekbdmurray: oh, sorry - I mean, 'version ubuntu' and 'device name:version version'20:33
pmcgowanToyKeeper, but that stuff didnt change in the image that we know of20:34
ToyKeeperpmcgowan: The music player can play normally if I select a song from within the app, but the video player doesn't have that functionality.20:34
jdstrandtyhicks, sbeattie: the denial is: Jul 30 15:42:13 ubuntu-phablet kernel: [14261.663293] type=1400 audit(1406749333.369:13404): apparmor="DENIED" operation="open" profile="/usr/bin/mediascanner-service-2.0" name="/var/lib/extrausers/passwd" pid=3153 comm="mediascanner-se" requested_mask="r" denied_mask="r" fsuid=32011 ouid=020:34
slangasekbdmurray: because 'version version' alone tells you the current build number on the channel but doesn't tell you which channel20:34
jdstrandtyhicks, sbeattie: let me forward you the tarball of his /etc/apparmor.d20:34
slangasekbdmurray: so we want 'version ubuntu', for "which rootfs build is this"; and 'device name:version version', for "which device image is this"20:34
jdstrandtyhicks, sbeattie: emailed20:35
* sbeattie looks20:36
ogra_slangasek, probably not much different from what we use for "build number" on the dashboard :) http://ci.ubuntu.com/smokeng/utopic/touch/20:36
slangasekogra_: well, the point is that we want to be able to correlate crashes across devices, and within a single device20:36
slangasekso we need two sets of counters20:37
ogra_yeah20:37
jdstrandtyhicks, sbeattie: I checked the stat of the files: http://pastebin.ubuntu.com/7907782/20:37
tyhicksjdstrand: have you tried to reproduce it?20:37
jdstrandtyhicks, sbeattie: no one else sees it20:37
jdstrandtyhicks, sbeattie: we had to apparmor_parser -R to get it to stop cause the mediascanner kept restarting cause it would crash on the failure20:38
tyhicksjdstrand, pmcgowan: what device and kernel version?20:38
pmcgowantyhicks, Linux ubuntu-phablet 3.4.0-5-mako #32-Ubuntu SMP PREEMPT Wed Jul 16 10:39:52 UTC 2014 armv7l armv7l armv7l GNU/Linux20:38
pmcgowanjdstrand, I haven't wiped in some time, perhaps I have some bit rot20:39
pmcgowanjdstrand, dont want to start a goose chase20:39
jdstrandpmcgowan: well, the thing is, this is very, very odd20:39
tyhicksbut that would be buggy bit rot20:39
pmcgowanyeah20:39
jdstrandas in, this should never happen20:39
jdstrandI asked tyhicks and sbeattie in case I am overlooking something20:39
jdstrandtyhicks, sbeattie: I wonder if it has something to do with profile replacement20:40
bdmurrayslangasek: ack20:40
jdstrandtyhicks, sbeattie: eg, if we loaded the profile now (or had Pat reboot), and see if the errors are still there20:41
jdstrands/errors/denials/20:41
tyhicksjdstrand: rebooting would be the most complete test of that20:41
jdstrandpmcgowan: can you reboot? when done, can you see if there are denials post reboot and also paste 'sudo aa-status'?20:42
pmcgowanjdstrand, yep20:42
tyhicksjdstrand: if that doesn't fix it, I'd like to skip the cached binary profile (or just purge the cache) when doing a reload of the profile20:43
tyhicksI think -T option to apparmor_parser would do that20:43
sbeattieand/or --show-cache so we see what the parser thinks of the cache state20:46
pmcgowanjdstrand, tyhicks no denials, and I don't see mediascanner running which is probably correct20:47
pmcgowanhttp://pastebin.ubuntu.com/7908231/20:47
jdstrandpmcgowan: can you add a video to the ~/Videos folder, with mtp? (or just make sure it is copied in place as the phablet user)20:48
veebersrobru: oops, silo-10 now has all it's MPs approved. What needs to happen for it to publish? :-)20:48
oSoMoN_robru, what’s the status of the apparmor publication and the new image you were going to kick?20:49
tyhickssbeattie: where I was going with the -T option was to see if the cached binary was corrupted at all - maybe a few bits were flipped in that particular pathname or in that particular access bit field20:50
pmcgowanjdstrand, no denials but I don't see the scanner kick in20:50
jdstrandpmcgowan: not sure how to get it to start20:51
jdstrandpmcgowan: maybe got to the Videos scope?20:51
pmcgowanit should with the folder change20:51
pmcgowanI did20:51
pmcgowanknow here that log file is?20:52
jdstrand/home/phablet/.cache/upstart/*mediascanner*log (though, you might want to expand the glob to make sure you snag the right file)20:52
jdstrandpmcgowan: /home/phablet/.cache/upstart/mediascanner-2.0.log it looks like20:53
pmcgowanjdstrand, odd its not starting20:53
pmcgowanwtf20:54
jdstrandjhodapp: fyi, apparmor-easyprof-ubuntu 1.2.14 uploaded for your micshm path change20:54
pmcgowanjdstrand, ogra_ just mentioned he cant see new videos, we have several issues here20:54
jdstrandpmcgowan: what does the log say?20:55
pmcgowanjdstrand, now video is playing20:56
pmcgowanjdstrand, the log was never opened, only old .gzs20:56
jdstrandlooking at my phone, it seems mediascanner service should be a long running process20:56
jdstrandpmcgowan: ah, so mediascanner is running now?20:56
pmcgowanjdstrand, no20:56
pmcgowanbut mediaplayer succeeded to play the video20:57
pmcgowanI cant add new ones, but can play old ones20:57
jdstrandpmcgowan: did you modify its upstart job or have a silo enabled or something?20:57
pmcgowanjdstrand, nope20:58
bdmurrayslangasek: would we want to query on "device image" and "ubuntu release" or would you consider those separate?20:58
jdstrandpmcgowan: start mediascanner-2.020:58
jdstrandpmcgowan: do that as the phablet user20:58
pmcgowanjdstrand, i can do a flash and see if anything gets better20:58
pmcgowanok20:58
jdstrandI would expect that to generate the log file20:59
pmcgowanits crashing again20:59
jdstrandpmcgowan: with apparmor denials?20:59
pmcgowanyep20:59
slangasekbdmurray: hmmm we do reset the channel numbers at the start of the release, I think, in which case the release needs to also be part of it20:59
slangasekstgraber: ^^ can you confirm?20:59
jdstrandtyhicks: do you want to take over here with the alternate options?21:00
pmcgowanjdstrand, terminate called after throwing an instance of 'std::runtime_error'21:00
pmcgowan  what():  Could not look up user name: Permission denied21:00
pmcgowanfrom mediascanner log21:00
jdstrandpmcgowan: how about 'sudo apparmor_parser -rT /etc/apparmor.d/usr.bin.mediascanner-service-2.021:01
jdstrandpmcgowan: that will reload the profile without the cache file21:01
tyhicksah, we're back to crashing21:01
tyhicksgood21:01
jdstrandtyhicks: it didn't start on boot, so I had him start it manually via its upstart job21:02
tyhicksok21:02
jdstrandwhich showed the denials21:02
pmcgowanjdstrand, that fixed things up it seems21:02
pmcgowannot crashing, no denials21:02
jdstrandvery interesting21:02
tyhicksyeah21:02
* sbeattie hrms21:02
jdstrandtyhicks, sbeattie: could this be fallout from the caching patches the we have?21:03
* tyhicks is thinking21:04
jdstrandpmcgowan: ok, last thing. can you do: rm -f /etc/apparmor.d/cache/usr.bin.mediascanner-service-2.0, then reboot?21:05
tyhicksjdstrand: so you *just* changed that profile in your last apparmor upload (a couple days ago) and the cached binary didn't get updated21:05
jdstrandtyhicks: well, the stat I did showed that the cache was regenerated on 4/28, the day of the upload21:05
jdstrandtyhicks: I changed the nameservice abstraction. I didn't add any other patches21:06
jdstrandtyhicks: https://launchpad.net/ubuntu/+source/apparmor/2.8.96~2541-0ubuntu221:07
tyhicksjdstrand: and that change had already landed in the image that pmcgowan was running?21:07
sbeattiejdstrand, tyhicks: regenerating the cache file locally using the parser as built from the 2.8.96~2541-0ubuntu2 generates a slightly different cache file than what was in p mcgowan's tarball21:09
pmcgowanjdstrand, rebooted21:10
jdstrandtyhicks: yes, pmcgowan is running the latest image. I verified that 'apparmor_parser -p /etc/apparmor.d/usr.bin.mediascanner-service-2.0 | grep extrausers' had what I expected first21:10
tyhickssbeattie, jdstrand: I'm wondering if his image had the usr.bin.mediascanner-service-2.0 cached binary from 2.8.96~2541-0ubuntu121:11
pmcgowanseems quiet21:12
jdstrandtyhicks, sbeattie: so in case you weren't aware, jj had all those patches to make it so you could run the parser against an apparmor features dir, and rsalveti starting using that to generate server cache files recently21:12
tyhicksif that was the case, it would be missing the '/var/lib/extrausers/passwd r,' rule21:12
pmcgowanscanner running and no denials21:12
jdstrandtyhicks: why wouldn't it have updated on boot?21:12
jdstrandtyhicks: yeah that was what I wanted to verify in my question to #security21:13
tyhicksjdstrand: I don't know for sure, but possibly from timestamp oddness between the text profile and the binary profile21:13
jdstrandtyhicks: your theory seems to go with what sbeattie mentioned about the cache being slightly different21:14
jdstrand(it would be a 2 rule difference)21:14
tyhicksrsalveti: when you generate the apparmor profile cache for the system images, where do you pull the profiles from?21:14
sbeattiejdstrand: well, oddly, my locally compiled cache file is slightly smaller than the one in the tarball.21:15
rsalvetijdstrand: tyhicks: we're not copying the cached files for the ones available in /etc/apparmor.d/cache21:15
jdstrandsbeattie: heh21:15
rsalvetiwhen updating the image21:15
sbeattiepmcgowan: now that you've rebooted, can you paste the output of 'stat /etc/apparmor.d/cache/usr.bin.mediascanner-service-2.0'21:15
rsalvetibecause the currently logic in the initrd that copies over the files doesn't replace them if they are already there21:15
tyhicksrsalveti: jdstrand recently uploaded apparmor 2.8.96~2541-0ubuntu2, which only made a small policy change, and it looks like the policy from 2.8.96~2541-0ubuntu1 may be what was cached21:16
rsalvetibut, they should still be updated by apparmor during boot21:16
rsalvetiif not, that's a bug21:16
pmcgowansbeattie, http://pastebin.ubuntu.com/7908468/21:16
tyhicksrsalveti: hrm... ok, we'll look into why the parser didn't catch the new profile21:17
sbeattiepmcgowan: thanks. it does show there's a difference in what was in the cache previously (Size: 78713) versus the working version (Size: 78689)21:18
jdstrandtyhicks, sbeattie: so, I don't think two people necessarily need to work on it. can you guys work out who should look at it? (my preference is for sbeattie to keep plugging away at abstract sockets)21:20
jdstrandtyhicks, sbeattie: but I appreciate both of you helping to triage it21:20
pmcgowanseems my work here is done21:20
tyhicksthanks pmcgowan21:21
jdstrandpmcgowan: you device should be ok now. we'll need to do some investigating21:21
pmcgowangood stuff21:21
tyhicksjdstrand: it probably makes sense for me to look at it anyways, since I've been digging around this profile caching code for the last week or so21:21
tyhickssbeattie: is that ok with you? ^21:21
sbeattietyhicks: fine by me.21:21
jdstrandrsalveti: so, to be super clear, when you run the serverside stuff, you will pull in /etc/apparmor.d from the latest apparmor to make sure it has the latest policy?21:22
jdstrandtyhicks, sbeattie: thanks again guys :)21:22
rsalvetijdstrand: the problem is that we're not always copying the files when booting the system21:24
imgbot=== trainguard: IMAGE 161 building (started: 20140730 21:25) ===21:24
rsalvetithe logic in the initrd first check if the dest file is already there, if so, don't overwrite it21:25
rsalvetieven if the content is different21:25
rsalvetithat's not a problem for click packages as they are named with the version id21:25
rsalvetifor the files available in /etc/apparmor.d, that's indeed an issue21:25
rsalvetiso we ended up not updating the cache files if they are already there, with an older version21:25
rsalvetiand we just let apparmor to recreate such cache during boot21:26
tyhicksoh21:26
rsalvetithe problem here is that for some reason apparmor is not updating the cache when booting the system21:26
tyhicksI misunderstood earlier21:26
rsalvetiand still trying to use the older version21:27
jdstrandah21:27
jdstrandrsalveti: so, if the cache file is newer than the system policy file, then it won't update21:27
jdstrandcause we aren't forcing a cache regeneration21:28
jdstrandeven though the policy actually changed in an abstraction21:28
* jdstrand looks at the upstart job21:29
rsalvetijdstrand: but in this case the cache file is older, isn't it?21:29
tyhicksjdstrand: the parser should probably be checking the timestamps of abstractions when deciding if it needs to update the cache file21:29
jdstrandon Pat's system, no, it wasn't21:29
rsalvetithat doesn't make much sense then21:30
jdstrandtyhicks: yeah. you see what I am saying, right?21:30
rsalvetiif it's newer, then it got automatically generated during boot21:30
jdstrandrsalveti: I think it does21:30
* sbeattie wonders what a stat of the abstraction looked like in comparison21:30
jdstrandrsalveti: ie, sometime ago, he got a new mediascanner-2.0 profile21:30
jdstrandrsalveti: it updated the cache21:30
rsalvetiright21:30
jdstrandthen, he did an image update21:30
jdstrandthe abstraction changed21:31
jdstrandso the timestamp on the profile didn't21:31
rsalvetioh, got it21:31
rsalvetikind of a race21:31
tyhickseh, more of a buggy parser21:31
jdstrandthe upstart job has logic to attempt to deal with that21:31
jdstrand(in the absense of the parser doing the right thing)21:31
tyhicksit should be checking all timestamps of #include'ed files21:31
tyhickslets just make the parser do the right thing21:32
jdstrandoh sigh21:32
jdstrandthe upstart job logic only deals with this for clicks21:32
tyhickshmm21:32
jdstrandtyhicks: do you see it?21:32
jdstrand        if [ -n "$run" ]; then21:33
jdstrand            aa-clickhook -f21:33
jdstrand        fi21:33
tyhicksI haven't looked at the upstart job21:33
jdstrandtyhicks: if apparmor changed, we need to force a cache regeneration21:33
jdstrand(and run aa-clickhook -f)21:33
jdstrandtyhicks: do you have it handy?21:34
tyhicksjdstrand: I do - looking at it now21:34
jdstrandtyhicks: it is the  [ -x /usr/bin/aa-clickhook ] hunk21:34
jdstrandtyhicks: it is wrong for how the server side is handling precaching for system policy21:35
jdstrandtyhicks: cause it only cares about click and assume system policy will just work21:35
tyhicksjdstrand: but if the parser was fixed to check the timestamps of all of the #include'ed files, couldn't that entire hunk go away?21:37
jdstrandtyhicks: the 'apparmor' part could21:37
sbeattieI thought the parser did that, but could be mis-remembering21:37
jdstrandtyhicks: not the apparmor-easyprof-ubuntu part21:37
sbeattiealso, in this case, it still would've avoided the cache21:38
tyhickssbeattie: I'm blindly assuming that it doesn't - I'll verify in a moment21:38
sbeattieerr, I mean, used the cace21:38
sbeatties/cace/cache21:38
tyhickssbeattie: why? was the abstraction's timestamp not newer than the cache file?21:38
tyhicks(I thought we didn't know the abstraction's timestamp)21:38
sbeattiethe mtime is about an hour before the cache file21:39
sbeattie$ stat ~/tmp/etc/apparmor.d/cache/usr.bin.mediascanner-service-2.0 | grep ^Modify21:39
sbeattieModify: 2014-07-28 08:28:53.000000000 -070021:39
sbeattie$ stat ~/tmp/etc/apparmor.d/abstractions/nameservice | grep ^Modify21:39
sbeattieModify: 2014-07-28 07:54:05.000000000 -070021:39
jdstrandtyhicks: we have to for apparmor-easyprof-ubuntu because we need to regenerate the policy files from templates21:39
tyhicksjdstrand: oh, right!21:40
stgraberslangasek: haven't looked at the whole backlog, but yes, each series has its own channel and when the channel is created it starts at buildid 121:40
tyhickssbeattie: how did that happen?21:41
stgraberslangasek: then we change the target of the devel-* aliases to the new series which basically means the devel-* channels appear to rollback from buildid 100 or so down to 121:41
stgraberslangasek: the system-image client tracks both the buildid and the channel alias target so that if the target changes it assumes its internal buildid to be 0 and forces a full update to the new build21:41
sbeattietyhicks: presumably the apaprmor cache update occurred locally on pat's phone in between the time the ubuntu2 package was built and him applying the update.21:42
jdstrandtyhicks, sbeattie: could he have booted before running a system-image update (therefore, had a cache from that day), then did a system-image update which updated the abstraction?21:43
jdstrandsbeattie: that would explain why we don't see more widespread issues-- it had to be timed right21:44
jdstrandor rather, wrong :)21:44
tyhicksyeah21:44
tyhicksman, that's a pain21:44
fgintherbrendand, I think I have the job fixed that was causing your address-book-service jobs to fail. I've retriggered a build to the results21:45
jdstrandtyhicks: so, wouldn't updating the upstart job to also force regenerate system policy like we do for click policy fix this?21:46
brendandfginther, i didn't even know you were looking at it - thanks :)21:46
fgintherbrendand, om26er mentioned it to me earlier today21:47
=== fginther changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard (general help): cihelp | CI Train support: robru, trainguards | CI Train Status: #157 promoted | Known issues: Both queuebot and http://bit.ly/1mDv1FS know your silo status before the spreadsheet does.
jdstrandtyhicks: ie, if the apparmor md5sum changed since last time, regenerate?21:47
jdstrandtyhicks: (and by force regenerate, I mean removed /etc/apparmor.d/cache/*21:48
jdstrand)21:48
jdstrandremove*21:48
tyhicksjdstrand: yeah - that would do it21:49
om26erfginther, did you find a solution for the otto issue ?21:49
tyhicksjdstrand: we'd probably need to use clear_cache() from /lib/apparmor/functions, so we'd need to do that after sourcing in that file21:49
fgintherom26er, yes :-)21:49
jdstrandtyhicks: right21:49
om26erfginther, has it been "deployed" ?21:50
jdstrandtyhicks: can you add that to your list of tings to update in the next apparmor upload?21:50
tyhicksjdstrand: just did that :)21:50
fgintherom26er, yes, I was in the middle of finding the build you sent me so that I can re-start it21:50
jdstrandtyhicks: I'm happy to review the upstart job changes if you want21:50
jdstrandtyhicks: thanks!21:50
jdstrandok, nice.21:50
jdstrandmystery solved21:50
tyhicksjdstrand: yeah, I'll def want you to review it21:50
om26erfginther, https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/1824/console21:51
jdstrandman, who knew that md5sums stupid hack had legs21:51
fgintherom26er, ahh sweet. I already restarted that one21:51
jdstrandaka, caching is hard :)21:51
brendandfginther, can i rerun the job then?21:53
fgintherbrendand, already did21:53
fgintherbrendand, om26er, it's s-jenkins.ubuntu-ci:8080/job/address-book-service-ci/274/21:54
om26erfginther, cool.21:54
om26erI see it did pass in otto after your change http://s-jenkins.ubuntu-ci:8080/job/autopilot-testrunner-otto-utopic/1834/console21:54
jdstrandogra_ (and anyone else who was paying attention): pmcgowan's libnss-extrausers denial is triaged. This is not a widespread problem (no bugs) since it relied on a very specific timing of events. that said, we know the issue and will fix it in the next apparmor upload, scheduled for some days out21:58
jdstrandtyhicks: can you file a bug for that? ^21:59
tyhicksjdstrand: ack - will do22:06
jdstrandtyhicks: thanks! :)22:17
imgbot=== trainguard: IMAGE 161 DONE (finished: 20140730 22:55) ===22:54
imgbot=== changelog: http://people.canonical.com/~ogra/touch-image-stats/161.changes ===22:54

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