/srv/irclogs.ubuntu.com/2015/07/16/#ubuntu-ci-eng.txt

=== chihchun_afk is now known as chihchun
=== chihchun_afk is now known as chihchun
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
Mirvmorning05:19
anpok_hi05:58
anpok_we finished testing silo-004 yesterday ..05:58
anpok_ping trainguards: there is one thing remaining05:59
anpok_we started with dual landing and switched to wily only landing becaue we have additional source packages..05:59
anpok_but the landing ppa for silo 004 still has the vivid+overlay packages in it..05:59
anpok_I hope this does not cause troubles..06:00
anpok_.. oh and additionally it is still marked as requres QA signoff - not sure if that is needed for a wily landing..06:00
Mirvanpok_: no it doesn't cause trouble, we can remove the packages not supposed to be there anymore06:09
Mirvanpok_: and eh, yes, it does not need QA signoff :)06:09
Mirvanpok_: do you want to make a copy of the vivid packages already built there, or are you ok reuploading/rebuilding them?06:13
anpok_hm yes .. what do I need to download?06:21
anpok_Mirv: I started a new row for syncing to vivid+overlay .. can we copy the additional source packages over to that silo then..06:27
anpok_all but glmark2.. we decided to bump glmark2 in overla06:27
anpok_so we would use the wily version there..06:28
Mirvanpok_: ok, just a moment06:31
Mirvanpok_: ok, all vivid packages from 004 except glmark2 copied to https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-053/+packages - does that look correct?06:34
Mirvanpok_: I will now remove the vivid packages from 004 and prepare it for the wily publishing06:36
Mirvanpok_: so I'm running WATCH_ONLY build of 004, don't worry06:37
anpok_ok06:38
Mirvanpok_: should I copy the 004's glmark2 wily to 053 for vivid?06:39
anpok_Mirv: yes06:42
anpok_the wily one yes06:42
Mirvanpok_: ok, glmark2 2014.03+git20150611.fa71af2d-0ubuntu2 building for vivid now in 05306:44
Mirvanpok_: regarding 004, all MP:s would need approval: https://ci-train.ubuntu.com/job/ubuntu-landing-004-2-publish/67/console06:45
Mirvif you've the permissions to top-approve them, just go through them and do it, but I can't do it06:45
anpok_Mirv: oh! not all of them .. I will trigger the necessary people06:52
Mirvanpok_: thanks!06:53
mardycihelp, does anyone understand why this build is failing? https://ci-train.ubuntu.com/job/ubuntu-landing-052-1-build/3/console07:29
mardylooks like it cannot create the source tarball07:29
Mirvmardy: ping trainguards, not cihelp, if thre's a CI Train problem07:49
Mirvmardy: my guess would be that you need to add .bzr-builddeb directory to your (first ever) landing (or push such a thing directly to the trunk first)07:52
Mirvmardy: and the .bzr-builddeb/ would have default.conf like http://bazaar.launchpad.net/~system-settings-touch/ubuntu-system-settings/trunk/view/head:/.bzr-builddeb/default.conf07:52
Mirvmardy: then try the building again07:52
mardyMirv: ah, I'll try that, thanks07:52
sil2100jibel, davmor2: do we know if there is anyone working on fixing the customization hooks?08:03
jibelssweeny and cwayne were discussing a fix yesterday evening. I don't know what the conclusion has been08:05
jibelif an08:05
jibely08:05
=== marcusto_ is now known as marcustomlinson
sil2100mzanetti: hey! Where is qml-module-qbluetooth built from?08:18
davmor2mzanetti: say outer space it will fry his head ;)08:22
sil2100;)08:23
sil2100Today my head is frying anyway, I feel worse than yesterday08:23
mzanettisil2100, you mean where to code for it is located?08:25
sil2100Yes, what source package it's coming from08:28
mzanettiouter space, yes08:29
mzanettisil2100, http://code.qt.io/cgit/qt/qtconnectivity.git/08:29
mzanettiqtconnectivity508:29
mzanettiit also contains QtNfc08:31
mzanettinot sure how far we are with the kernel enablement there08:32
mardytrainguards, can you help me solve this build issue? https://ci-train.ubuntu.com/job/ubuntu-landing-052-1-build/lastBuild/console08:38
sil2100mardy: looks like a geniune failure, looking at the build logs08:39
sil2100But it seems it  failed building for all arches08:39
sil2100I see a cmake error there08:39
sil2100mardy: are you using google mock in your project? Since it seems if you do, you're missing a dependency to google-mock08:40
sil2100mardy: and cmake is failing to find one of its directories, as it doesn't seem to be installed08:41
mardysil2100: I think you are right, but I cannot understand how you got to this conclusion; I don't see anything about google mock in the logs08:43
sil2100mardy: https://launchpadlibrarian.net/211794576/buildlog_ubuntu-wily-i386.online-accounts-api_0.1%2B15.10.20150716-0ubuntu1_BUILDING.txt.gz <- here for instance08:43
sil2100mardy: the train only gives you links to build logs of the packages that failed building, so I browsed it through for errors08:44
mardysil2100: oh, indeed; I guess I didn't explore the jenkins' interface well enough :-)08:44
sil2100And saw this:08:44
sil2100CMake Error at cmake/FindGMock.cmake:6 (add_subdirectory):08:44
sil2100  add_subdirectory given source "/usr/src/gmock" which is not an existing08:44
sil2100  directory.08:44
sil2100:)08:45
mardysil2100: thanks a lot, I'll get it fixed08:45
sil2100Excellent, you're welcome08:45
mzanettisil2100, hey, I'm about to train dednick to be a lander, can you grant him permission please09:25
sil2100mzanetti: sure, on it09:28
sil2100mzanetti: done09:29
mzanettithanks :)09:29
sil2100ogra_: hey! Can I bother you with https://code.launchpad.net/~sil2100/ubuntu-seeds/ubuntu-touch.wily-qtbluetooth/+merge/264977 for a moment? Once this is approved I'll upload the vivid-packaged version to the overlay10:11
alextu #launchpad10:25
psivaaLaney: I've commented on the wily lvm desktop bug, please take a look. I need to be afk for a bit now, and be happy to answer any questions whey i come back10:57
=== psivaa is now known as psivaa-afk
Laneypsivaa-afk: I think it's likely to be cyphermox rather than me (trying hard not to own this bug), but can you point to where you can see the failure in the logs or give a link/instructions for how to run it outselves?11:00
cyphermoxwhat's the bug no?11:00
Laneyhttps://bugs.launchpad.net/ubuntu-test-cases/+bug/147488711:01
ubot5Ubuntu bug 1474887 in Ubuntu Test Cases "wily-desktop-amd64-smoke-lvm is failing" [Undecided,New]11:01
* Laney eyes cyphermox up suspiciously early11:02
cyphermoxdoubtful any preseed changes are going to fix a system that already boots11:02
cyphermoxLaney: yeah, couldn't sleep any more... which is weird because I went to bed pretty late after a few hours ingressing11:03
cyphermoxpsivaa-afk: my guess is that the test itself is wrong for LVM. It looks for the very first line in the output of mount to start with /dev/mapper, which probably won't be the case11:07
cyphermox(because the first entry in mount is usually not the mountpoint for /)11:07
* sil2100 off to lunch11:13
cjwatsonFYI, all newly-created amd64/i386 LP builds will now be dispatched to scalingstack (lcy01-*/lgw01-*), even if they're in non-virtualised archives such as silos11:16
cjwatsonRetries will still go to the bare-metal build farm for now, until we're confident enough in things to do the more invasive database surgery11:17
cjwatson(Retries of builds created prior to this change, that is)11:17
=== _salem is now known as salem_
Mirvsil2100: hmm, I can't free a silo due to an error.. https://ci-train.ubuntu.com/job/ubuntu-landing-035-3-merge-clean/6/console12:26
Mirvsil2100: actually, it apparently got far enough in https://ci-train.ubuntu.com/job/ubuntu-landing-035-3-merge-clean/5/console already so that it disappeared now from dashboard, even though that too ended in an error12:26
Mirvweird12:26
sil2100Mirv: hm, this error usually means the silo was already freed12:27
sil2100Mirv: you sure it wasn't auto-merged or you didn't run the job twice?12:27
Mirvsil2100: no, as you can see there it removed packages first in that job nr 512:29
sil2100Then maybe the silo was in overall in a busted state.12:30
Mirvsil2100: probably when the silo is never formally built (build job not run), that happens. I was only manually uploading packages for testing.12:32
sil2100Ah, no watch_only?12:34
sil2100ogra_, slangasek: could any of you review/approve this branch? https://code.launchpad.net/~sil2100/ubuntu-seeds/ubuntu-touch.wily-qtbluetooth/+merge/26497712:35
Mirvanpok_: just two empty commits non-approved anymore, I guess those don't need special permission to top-approve?12:45
Mirvtrying.12:46
Mirvanpok_: now this is more problems for you though, there are new, non built commits in two of the branches so you need to investigate that: https://ci-train.ubuntu.com/job/ubuntu-landing-004-2-publish/68/console12:47
kyrofatrainguards: last I heard, I can't actually land anything in the archives via the spreadsheet replacement beta. Is that still the case?12:47
sil2100kyrofa: hey! Not sure if you can, I think it's still not possible currently, as I think those are using the staging PPAs12:48
Mirvanpok_: it seems those are only a changelog entry change and comment change by you, so you'll need to decide whether you want to rebuild the packages for those or not12:48
kyrofasil2100, alright, thanks for the info! I wanted to make sure before I tried to get a new project silo'd12:50
Mirvsil2100: actually, I think we cannot even force publishing something if there's a branch that has a newer commit that what was built? ie rebuild needed always?12:53
sil2100Mirv: I think currently there's no way to force-publish it besides doing a copy-package12:54
sil2100You need to rebuild or revert the commits from the branch12:54
sil2100Revert and do a push --overwrite12:54
kyrofatrainguards: I'm preparing a new project to go through the CI train. Does the debian/changelog version need to be anything special for the train magic to happen?12:59
sil2100kyrofa: no, the train will try to be smart and change it to something following its format - be sure to just add a changelog entry that's UNRELEASED with the correct upstream version number13:00
sil2100So, for instance, something like 1.2.3-0ubuntu1 should just work13:00
sil2100The train should mangle it into something it understands during build13:01
kyrofasil2100, alright, very good. Can you please point me to the spreadsheet?13:04
sil2100https://wiki.ubuntu.com/citrain13:06
sil2100(it's a redirect)13:06
kyrofasil2100, perfect!13:06
jdstrandjohn-mcaleely: hey, is bug #1360403 for OTA-5? I saw in bug #1439101 you mentioned OTA-5 will fix a lot of things, but 1360403 actually makes mms work flawlessly for me (t-mobile).13:08
ubot5bug 1360403 in nuntium (Ubuntu) "MMS does not work with T-Mobile US" [High,In progress] https://launchpad.net/bugs/136040313:08
ubot5bug 1439101 in telephony-service (Ubuntu) "impossible receive MMS any operator on bq aquaris E4.5 " [High,Confirmed] https://launchpad.net/bugs/143910113:08
jdstrandjohn-mcaleely: I imagine it would fix things for others as well (obviously you would want to talk to abeato-- I'm just mentioning this as a user)13:09
jdstrandjohn-mcaleely: (this is already in silo 2 btw)13:09
jdstrandjohn-mcaleely: I should be more clear-- rc-proposed (ie all of OTA-5) *plus* silo 2 makes mms work great for me13:10
davmor2jdstrand: no then ota5 is done, ota 6 will have the fix13:10
jdstrandthat is what I figured. that's too bad (not for me, I will just install silo 2 and be done with it)13:11
davmor2jdstrand: There are a bunch of underlying fixes that abeato and awe did for ota5 that I think make silo 002 work, but fixed issues with mms for many others.  So I think there will be a few more improvements along the way.13:12
kyrofacihelp: Can you verify that autolanding has been disabled for unity-scope-snappy?13:12
jdstrandsure, I understand that (I've been following this very closely). just saying, ota-5 plus silo 2 is the first time mms has ever worked for me13:13
fgintherkyrofa, yes, it's disabled now13:13
ogra_pfft ... MMS ... send a postcard ... thats cheaper13:13
jdstrandand I thought it worth mentioning that it is as close as it is, especially since ota-5 is delayed. it isn't my call, just providing info13:14
kyrofafginther, so if I approve MPs nothing will happen without the CI train, correct?13:14
jdstrandogra_: heh13:14
ogra_sil2100, sorry, currently wrangling with a snappy prob ... i'll get to your MP later13:14
abeatojdstrand, which is your operator?13:14
jdstrandtmobile13:14
fgintherkyrofa, yes, approved MPs will be ignored by the ci s-jenkins now13:14
ogra_jdstrand, no joking ... MMS in europe is like 1.20€13:15
kyrofafginther, awesome, thank you!13:15
abeatojdstrand, ok t-mobile us wait for ota-613:15
fgintherkyrofa, you're welcome13:15
john-mcaleelyjdstrand, sadly, I understand it missed the ota-5 cut13:16
john-mcaleelyso will land in ota-613:16
davmor2ogra_: yeah but after the greek affair that's what 3 cents ;)13:16
seb128ogra_, non-sense, in France most subscriptions include them for free13:16
seb128like for 5€/month you can get 2 hours call and unlimited sms/mms13:16
jdstrandhey, like I said. not my call-- I'm extremely pleased to install silo 2 on my device13:16
jdstrandjust thought I'd pass along the info13:17
davmor2seb128: no pay as you go in the uk include them, I think they are in contract sims though13:17
abeatojdstrand, note however that many fixes for mms already landed, there was just a small thing needed for t-mobile us, but I think most of the other operators should work now13:17
davmor2seb128: roughly 60p in the uk up to £113:17
seb128davmor2, yeah, most people in france has sim contracts, not pay as you go13:17
seb128have*13:17
abeatojdstrand, for instance Free in france13:18
ogra_seb128, pfft, france ... communists !13:18
ogra_:D13:18
seb128:-)13:18
john-mcaleelythey're not in my UK contract13:18
john-mcaleelywhich makes testing tedious13:18
ogra_yeah, same here13:19
davmor2john-mcaleely: no expensive not tedious ;)13:19
ogra_i got free SMS13:19
ogra_but MMS cost a fortune13:19
john-mcaleelytedious to read the bill :-)13:19
abeatofortunately in my case the price dropped from 1 eur + VAT to 0.5 a couple of months ago13:19
john-mcaleelyabeato, I guess you're a bulk user :-)13:19
abeatobut anyway I had to spend like 30/40 eur in MMS last month, john-mcaleely knows that :D13:20
john-mcaleelyindeed13:20
abeatojohn-mcaleely, probably the only MMS bulk user in Spain these days :)13:20
john-mcaleelyha13:20
davmor2abeato: Europe13:23
abeatodavmor2, maybe EMEA ;)13:23
davmor2abeato: infact, the world bar the USA13:24
abeatolol13:24
rvrjdstrand: ping13:59
=== charles is now known as jordan
jdstrandrvr: hey14:18
rvrjdstrand: Hi, I'm testing silo 11 and I found some issues with apparmor14:18
rvrjdstrand: http://paste.ubuntu.com/11887897/14:18
rvrjdstrand: The popup is stuck loading the login page14:19
rvrjdstrand: During installation, I downgraded to apparmor-easyprof-ubuntu 1.3.12, the version in the silo PPA.14:19
rvrThe one in the overlay PPA is 1.3.1314:20
jdstrandrvr: let me circle back to you. got pulled into a meeting right after I said 'hey'14:21
tsdgeoscoolio14:22
* tsdgeos tries14:22
rvrawe_: ping14:30
awe_rvr, pong14:32
rvrawe_: How to test silo 6?14:32
awe_rvr, I would run your standard network test cases;  if you want to test the specific changes for the APN editor, then you have a choice of either doing so manually using ofono scripts, or using the corresponding system-settings silo for the apn editor14:35
awe_but you'd need to check with jgdx as to it's current status14:35
awe_rvr, I think you testing should be primarily focused on ensuring there are no regression14:36
awe_s14:36
rvrawe_: Ok14:36
kyrofafginther, while autolanding is disabled for unity-scope-snappy, is it still running tests etc. for MPs?14:49
anpok_Mirv: I rebuilt (004) .. I just realized that the vivid mir package in the former dual landing silo was outdated.. can you copy the wily ubuntu-system-compositor and mir package instead15:07
kyrofacihelp: while autolanding is disabled for unity-scope-snappy, is it still running tests etc. for MPs?15:14
josephtkyrofa: yes, it should be.15:14
kyrofajosepht, hmm, alright, maybe I'm just being paranoid. It's been over an hour is all15:17
josephtkyrofa: I'll take a look in a few minutes. Do you have a link to the MP handy?15:18
kyrofajosepht: https://code.launchpad.net/~kyrofa/unity-scope-snappy/improve_debs/+merge/26464015:18
kyrofajosepht, I appreciate your eyes :)15:19
jdstrandrvr: ok, back15:22
jdstrandrvr: so, 1.3.12 and 1.3.13 are equivalent for what you are testing15:22
rvrjdstrand: Ok15:22
rvrjdstrand: So, do you think those denials are harmless?15:23
jdstrandI do, based on a conversation with mardy before15:23
jdstrandlet me find the bug15:23
jdstrandrvr: this is https://bugs.launchpad.net/ubuntu-system-settings-online-accounts/+bug/146879215:25
ubot5Ubuntu bug 1468792 in Online Accounts setup for Ubuntu Touch "various apparmor denials when using ubuntu-account-plugin template" [Critical,In progress]15:25
jdstrandlet me check something15:25
jdstrandah right15:26
jdstrandrvr: so, what is happening is that we allow this: owner @{HOME}/.cache/online-accounts-ui/id-*-@{APP_PKGNAME}_@{APP_APPNAME}/15:26
jdstrandrvr: but we intentionally don't allow this: @{HOME}/.cache/online-accounts-ui/15:27
jdstrandrvr: something needs to create @{HOME}/.cache/online-accounts-ui/ on the apps behalf.15:27
jdstrandrvr: it is believed that this is another bug:15:27
jdstrandactually, no15:28
jdstrandI thought it was related to another one, but I was wrong15:28
jdstrandrvr: last I heard, with apparmor-easyprof-ubuntu 1.3.12 and higher, all relevant denials are fix and that if there is a problem it is somewhere else15:29
jdstrandrvr: I'm going to comment in the bug regarding @{HOME}/.cache/online-accounts-ui/15:29
rvrjdstrand: It is point 1 of that bug15:29
rvrjdstrand: "1. There is a denial to create this directory if it does not exist already" /home/phablet/.cache/QML/Apps/online-accounts-ui15:30
rvrHmm15:30
rvrname="/home/phablet/.cache/QML/Apps/online-accounts-ui/"15:31
rvrYup15:31
kyrofajosepht, that MP ran! Did you poke it, or did it just take time?15:32
josephtkyrofa: I poked it15:32
kyrofajosepht, heh, thanks :)15:32
slangaseksil2100: qml-module-qtbluetooth, you've analyzed the impact on the size of the phone image and everyone's happy with the result?15:33
josephtkyrofa: np, it thought it had already looked at rev 32 so it didn't try to run again.15:33
jdstrandrvr: I'm quite confident the  /home/phablet/.local/share/applications/ is harmless. I'm less so about /home/phablet/.cache/QML/Apps/online-accounts-ui/15:34
jdstrandrvr: can you do: 'mkdir -p /home/phablet/.cache/QML/Apps/online-accounts-ui/' then ttry again?15:35
rvrjdstrand: Sure15:35
sil2100slangasek: I checked that and was fine, but I didn't consult with anyone ;)15:37
rvrjdstrand: Jul 16 15:37:30 ubuntu-phablet kernel: [   52.552819]type=1400 audit(1437061050.590:131): apparmor="DENIED" operation="mknod" profile="com.ubuntu.developer.rmescandon.asana_account-plugin_1.0.0" name="/home/phablet/.cache/QML/Apps/online-accounts-ui/ef91bab385a7f63fa8bbf22bbf9d1bdf" pid=3546 comm="QQmlThread" requested_mask="c" denied_mask="c" fsuid=32011 ouid=3201115:38
sil2100I think it was around ~260KB of additional space required15:38
AlbertAfginther: is it possible to enable listing the mir-clang-ts-wily-amd64-build jobs through a public URL (similar to:  https://jenkins.qa.ubuntu.com/job/mir-clang-wily-amd64-build/)?15:38
jdstrandrvr: ok-- that indicates two things-- one, the denial is not harmless and two, there is a bug in the silo because /home/phablet/.cache/QML/Apps/online-accounts-ui/ef91bab385a7f63fa8bbf22bbf9d1bdf is not app-specific, and it should be15:40
rvrjdstrand: Ack15:40
rvrmardy: ^15:40
rvrdbarth: ^15:42
dbarthreading15:42
dbarthah, so not harmless after all :/15:44
jdstranddbarth: you might see comments 12 and 13 from bug 146879215:44
ubot5bug 1468792 in Online Accounts setup for Ubuntu Touch "various apparmor denials when using ubuntu-account-plugin template" [Critical,In progress] https://launchpad.net/bugs/146879215:44
slangaseksil2100: ok.  I'm not sure what the policy should be on adding new interfaces to the phone, this seems like something that should be checked with product management or so?15:45
dbarthhmm, yet i was able to create the account, whereas rvr could not in his testing15:46
dbarthrvr: could you double check the package versions that effectively got installed on your test phone?15:46
rvrdbarth: Hmm15:47
rvrdbarth: Grrrr... citrain tool didn't install the packages15:49
rvrdbarth: Rechecking15:49
sil2100slangasek: I think this was a request from management as well as it's a feature request from higher-up, but I don't know, I'm managing seeds since recently15:49
dbarthrvr: ah!15:49
rvrbecause of the downgrade problem15:49
sil2100slangasek: I could check with pmcgowan now15:49
sil2100pmcgowan: hey! mzanetti requested adding qml-module-qtbluetooth to the seeds for bluetooth file transfer support15:50
sil2100pmcgowan: just want to confirm it's approved by product management15:50
mzanettiright. I didn't talk to anyone...15:51
mzanettiI just installed it to my phone, had a little play, it seemed working and so I thought I'd start the discussion15:51
sil2100Oh, so I misunderstood15:51
mzanettibut yeah... I still think we should have it15:51
sil2100Well, still new to this process, in the past I was only getting requests from management15:52
sil2100pmcgowan: you think this will be fine? It's an additional ~260KB of space (with new dependencies)15:52
pmcgowansil2100, is that a standard Qt 5 module15:52
pmcgowanthat seems fine15:52
sil2100pmcgowan: yes, it's from Qt515:52
mzanettiit has been released upstream by now, yes15:52
pmcgowanok15:53
mzanettiwhen we first put together the seed it was still marked "experimental"15:53
sil2100slangasek: ^ we have approval, sorry for that, will make sure to confirm every change with the PT15:53
slangaseksil2100: ok, merged15:54
ogra_was that the seed change ?15:54
slangaseksil2100: I guess this needs an ubuntu-touch-meta upload?15:54
slangasekogra_: yes15:54
ogra_ah, thanks15:54
sil2100slangasek: I have the vivid-overlay version prepared15:54
* ogra_ would have had time now :)15:54
sil2100slangasek: I can prepare a wily version too, but I never remember if I should prepare it by hand or is there some magic way to build it out of the branch15:55
slangaseksil2100: ah, this is being changed for vivid too? hmm I'm not sure how that will work15:55
* sil2100 uploads the overlay version in teh meantime15:55
sil2100slangasek: we always prepare a separate version of the ubuntu-touch-meta and upload15:56
ogra_slangasek, sil2100, err, wait15:56
slangaseksil2100: er, you're talking about a vivid-overlay /package/, not a merge?15:56
sil2100slangasek: yes, package15:56
ogra_slangasek, sil2100, qml modules need to go into the sdk-libs15:56
ogra_(and slangasek is right, that would need a framework bump i guess)15:56
slangaseksil2100: the metapackages are supposed to be autogenerated from the seeds using the ./update command15:56
sil2100ogra_: oh?15:57
ogra_slangasek, not for releases releases :)15:57
ogra_slangasek, there we always need to hack the meta directly15:57
ogra_because the seeds get locked on release day15:57
slangasekhmm15:57
ogra_*released releases15:57
sil2100If the place for it is in sdk-libs then indeed it's not so easy15:58
ogra_well, all qml modules live there15:58
rvrdbarth: Doesn't load either15:58
ogra_and essentially make up our framework15:59
ogra_not sure what our policy for vivid is here15:59
ogra_as an architect :)15:59
dbarthrvr: ?!15:59
dbarthrvr: with the same error messages?15:59
ogra_but i imagine we cant just enhance the framework without any version bumps16:00
sil2100Ok then, let's hold off getting this change merged/approved16:00
sil2100It's not critical enough and we need to put a bit more thought into it16:00
ogra_sil2100, i guess you want tvoss inpout for that one first16:00
ogra_bumping in wily shouldnt be a prob though16:00
sil2100Right16:01
davmor2sil2100: meeting16:02
slangaseksil2100, ogra_: the merge was committed to the seed branch, but I'll hold off on uploading anything until we have a final decision16:02
pmcgowansil2100, ogra_ is right, that becomes part of the sdk16:03
rvrdbarth: jdstrand: Jul 16 15:51:19 ubuntu-phablet kernel: [   74.711347]type=1400 audit(1437061879.696:123): apparmor="DENIED" operation="exec" profile="com.ubuntu.developer.rmescandon.asana_account-plugin_1.0.0" name="/usr/lib/arm-linux-gnueabihf/oxide-qt/chrome-sandbox" pid=3750 comm="online-accounts" requested_mask="x" denied_mask="x" fsuid=32011 ouid=016:03
rvr 16:03
ogra_slangasek, well, wily is -dev anyway, any chanes there should be ok ... this is just about vivid ... where seeds arent used anyway16:04
pmcgowansil2100, ogra_ we did just add a 15.10-dev framework a bit belatedly16:04
pmcgowanthe fw definition is a bit unwieldy16:04
slangasekogra_: if it's supposed to go in sdk-libs, that affects the seed for wily also.16:06
jdstrandrvr: you need the webview policy group16:06
ogra_slangasek, yes, i'm only talking about wily :)16:07
dbarthrvr: oh, did you apply comment #3 from the bug report ? that's the key16:07
ogra_slangasek, vivid doesnt have seeds anymore16:08
dbarthie, the package doesn't have the right policy embedded, and it needs an upgrade via:16:08
ogra_(not functional seeds at least)16:08
dbarth  cd /var/lib/apparmor/profiles16:08
dbarth    sudo apparmor_parser -r click_com.ubuntu.developer.rmescandon.asana_account-plugin_1.0.016:08
rvrdbarth: Ah16:09
dbarthrvr: sorry, that was not totally obvious from the spreadsheet comment16:09
jdstranddbarth, rvr: comment #3's point '1' is already resolved in apparmor-easyprof-ubuntu 1.3.12 and higher16:11
rvrdbarth: jdstrand: I executed that, and the popup is still stuck loading the login page16:14
jdstrandrvr: what dbarth said isn't quite enough16:14
jdstrandrvr: you need to look at /var/lib/apparmor/clicks/*asana*, then add 'networking' and 'webview' to the policy_groups16:15
jdstrandrvr: then you need to: sudo rm -f /var/lib/apparmor/profiles/click_*asana* ; sudo aa-clickhook16:15
dbarthrvr: well, that's the whole of comment #3, ie download the improved profile from mardy, which should go into the next version of the package16:15
jdstranddbarth: no16:16
jdstranddbarth: stable-phone-overlay has all the fixes that can be applied16:16
jdstranddbarth: to apparmor-easyprof-ubuntu16:16
rvrjdstrand: dbarth: Is the click package wrong?16:16
jdstrandrvr: if it doesn't contain 'webview', then yes, it is wrong16:17
jdstranddbarth: ie, I incorporated mardy's fixes16:18
jdstranddbarth: but the silo is not operating correctly in the manner I described in comments #12 and #1316:19
dbarthjdstrand: which click package version should we use? i refer to the one received by email, not the store upload which is unaccessible for us16:24
rvrjdstrand: dbarth: Now it loads16:25
dbarthrvr: what changed?16:25
rvrdbarth: I added the profile changes that jdstrand described16:26
jdstrandwell, hold on-- you had to do the mkdir still, right?16:27
dbarthok16:27
rvrjdstrand: Yes, let me remove it16:28
jdstranddbarth: the asane attached to my email with 'Subject: Re: account-plugins apparmor confinement' on 2015/06/26 has the corrected policy groups16:30
jdstrandasana*16:30
rvrjdstrand: It loads, even after removing the directory :-/16:32
jdstrand:\16:33
jdstrandrvr: sending you the updated click16:33
rvrjdstrand: Ack16:34
jdstrandrvr: I think the test environment is compromised. I advise restarting the tests with the click I provided16:34
rvrReflashing16:34
jdstrandI don't really know what online accounts is doing behind the scenes16:35
dbarthyup, agreed16:35
dbarthi can do the same here16:35
jdstrandI suspect that parts of it completed despite the failure16:35
dbarthrvr: you on wily, right?16:35
jdstrandand therefire simply removing the dir didn't reset it16:35
rvrdbarth: Nope, overlay PPA16:36
rvrdbarth: ubuntu-touch/rc-proposed/bq-aquaris.en16:36
dbarthah ok, switching there then16:36
rvrjdstrand: dbarth: It loads17:09
rvrAfter reinstalling everything17:09
rvrAnd using jdstrand's click package17:09
=== alexabreu is now known as alex-abreu
dbarthrvr: right, i have the same here; worked like a charm (again ;) using my corporate google account to login even17:19
dbarththough i do get a few denials due to the webview wanting to go over the board, but that's cosmetic, and can be managed in a separate bug17:20
rvrdbarth: I can also create another account in Ubuntu17:20
rvrOne17:20
rvrdbarth: jdstrand: Approving the silo17:27
dbarthrvr: thank you17:29
=== psivaa-afk is now known as psivaa
cwaynetrainguards: hiya, can i get a silo for line 72? it's part of the dconf custom fix17:33
cwaynesil2100: ^17:33
robrucwayne: on it17:33
cwaynerobru: <317:34
robrucwayne: you didn't specify what series to release to17:34
sil2100o/17:35
robrusil2100: goodnight17:35
sil2100Not yet! Just waving hands to cwayne o/17:35
robruheh17:35
sil2100robru, cwayne: can that be dual landed?17:35
cwaynedon't see any reason why not17:36
sil2100Ok, if that's an MR then it makes sense17:36
cwaynesil2100: so funny story, seems we had a fix for this in place over a year ago (we didn't realize there was a bug to fix, we were just going to move to a precompiled db)17:36
robrucwayne: ok silo 3517:37
cwaynethis'll require some fixes to the custom build scripts as well, so we'd need to coordinate its landing sil210017:37
cwaynerobru: thanks17:37
robrucwayne: you're welcome17:37
sil2100cwayne: hm, is this well tested?17:38
sil2100cwayne: ok, so we'll need a coordinated custom release, right? For OTA-5 we'll only release this fix for arale17:38
cwaynesil2100: ssweeny just tested it, yeah17:38
sil2100For vegetahd and krillin we won't do an OTA-5 respin17:39
cwaynesil2100: ack, ill work on getting a custom tar ready so we can test both together17:39
sil2100cwayne: excellent, thanks :)17:39
sil2100davmor2, jibel: ^17:39
robrukgunn: nah you gotta wait until u-s-s-o-a finishes migrating before rebuilding17:39
kgunnrobru: ah...was just reacting to dashboard17:40
robrukgunn: yeah it's kind of a race condition, dashboard is set as dirty when the other silo publishes, but rebuilding doesn't work until the other silo merges17:40
robrukgunn: no, it's exactly a race condition, no kind of about it ;-)17:41
robrukgunn: fortunately it'll get marked dirty again when the merge happens17:41
kgunnsure17:41
kgunnnp17:41
psivaacyphermox: i've commented on your MP for lvm test fix17:46
cyphermoxok17:46
jordantrainguards, line 73 is ready for a silo18:16
dobeytrainguards: if a landing is for vivid without the overlay ppa specified, it will get pushed out to vivid-updates as an SRU, right?18:16
fgintherrenatu, The buteo jobs have been running for about a day now, can you confirm they are setup correctly? https://jenkins.qa.ubuntu.com/job/buteo-syncfw-qml-ci/ and https://jenkins.qa.ubuntu.com/job/buteo-sync-plugins-contacts-ci/18:18
renatufginther, thanks. Everything looks ok. The "/buteo-sync-plugins-contacts-ci" will fail because some missing deps.18:21
robrudobey: yep, but it's up to you to make sure you get all the SRU paperwork filed and ping sru people to prod it along, otherwise it just sits in UNAPPROVED for *literally months*18:25
robrudobey: average lead time on a train SRU is typically 2 months18:25
dobeyrobru: ok. i was just making sure that the train handled that sort of thing already. not that i need to do it right now :)18:26
robrudobey: yep that's been supported since the beginning.18:26
dobeyok. i'm going to move a couple more projects over to CI train, so just clarifying before i do all the work ;)18:27
robrujordan: silo 55, note conflicts in silos 3, 4, and 4618:28
kyrofatrainguards: I don't seem to have permission to modify the CI train spreadsheet. How do I get that?18:29
robrukyrofa: one sec18:29
* alecu vouches for kyrofa18:30
kyrofaGot it robru, thank you!18:30
robrukyrofa: you're welcome18:30
robrukyrofa: let me know if you have any questions about the spreadsheet. also, don't spend too much time learning about it because it's all changing next week18:31
jordanrobru, ack. These are little four-liner patches in each repo, but I'll confirm that there's not overlap with 3, 4, and 4618:31
robrujordan: it's more like, after you release this silo, make sure the owners of 3, 4, and 46 know they need to rebuild ;-)18:31
jordanrobru, ack18:32
robrujordan: or the other way around depending on who publishes first. communication is key18:32
robrukgunn: safe to rebuild now ^18:32
kgunnthanks!18:33
robrukyrofa: ok, silo 1119:39
kyrofaThanks robru!19:40
robrukyrofa: you're welcome19:40
robrukgunn: do you have a core dev lined up for silo 4? I see Laney is responsible for gtk there but not sure about the rest19:40
kgunnrobru: uh.... :P19:41
kgunnrobru: we have RAOF on our team who is one19:41
robrukgunn: lots of packaging changes! can you get him to review? https://ci-train.ubuntu.com/job/ubuntu-landing-004-2-publish/69/artifact/19:41
kgunnhe'll be on in about an hour or so19:41
kgunnrobru: ok, after that, is he good to own all the action...or does he need to ping back here ?19:42
robrukgunn: he can ping me and I can publish the silo.19:42
kgunnok..19:42
kgunnah crap.... i think he might still be on vacation19:42
kgunnmterry much thanks!19:50
kgunnrobru: mterry taking a look...19:51
robrukgunn: mterry great19:51
* mterry waves at robru19:51
robrumterry: hey buddy, long time19:51
mterry:)19:51
mterryLaney, I'm looking at https://ci-train.ubuntu.com/job/ubuntu-landing-004-2-publish/69/artifact/gtk+3.0_packaging_changes.diff and I see that gtk+3.0's debian/control.in didn't get entirely updated -- looks like it got updated in one place, but debian/control got updated in two places.  Right?19:56
mterryI don't think it's a critical omission19:57
mterryLaney, but maybe fix it in bzr19:57
mterryrobru, an empty changelog is no good right?  https://ci-train.ubuntu.com/job/ubuntu-landing-004-2-publish/69/artifact/unity-system-compositor_packaging_changes.diff20:01
cyphermoxpsivaa: so, should I file a new merge against the other branch or are you including the changes in your branch?20:01
robrumterry: bah20:02
psivaacyphermox: i'd ask you to do an MP against the other branch please20:02
psivaacyphermox: that MP has to be approved and merged by ubuntu-server-developers anyway20:03
robrukgunn: https://ci-train.ubuntu.com/job/ubuntu-landing-004-2-publish/69/artifact/unity-system-compositor_packaging_changes.diff do you have any idea what's going on here? aside from the empty bullet point there's a giant chunk of seemingly duplicated changelog, I have no idea why.20:04
cyphermoxpsivaa: https://code.launchpad.net/~mathieu-tl/ubuntu-test-cases/fix_lvm/+merge/26503920:04
cyphermoxI can do the merge myself..20:04
kgunnrobru: no idea20:05
* kgunn looks at mps20:05
robrukgunn: yeah I guess the MP is busted.20:05
robrukgunn: I don't know who's idea it was to copy & paste 0.0.5+15.10.20150506.1-0ubuntu1 into a new entry for wily, don't do that20:06
kgunnrobru: i know who the culprits most likely are....will relay the message20:06
psivaacyphermox: I have approved it20:06
robrukgunn: please make the changelog entry like this: http://paste.ubuntu.com/11889568/ then rebuild20:07
kgunnack20:08
robrukgunn: thanks20:08
robrukgunn: oh and just rebuild that one, not the whole silo ;-)20:09
kgunnrobru: yes of course20:09
robrugreat20:09
kgunnrobru: am i that dumb of a manager ? :-P20:09
kgunndon't answer that20:09
robrukgunn: I can't remember who but I've seen it happen, just making sure ;-)20:10
kgunnrobru: totally possible with me...thanks for the reminder :)20:10
robrukgunn: you're welcome!20:10
robrukyrofa: please have a skim over our packaging guidelines for the train, it may help resolve the build failure you're seeing: https://wiki.ubuntu.com/DailyRelease/InlinePackaging20:11
cwaynefginther: hi, I can apt-get install stuff in my jenkins job if i need it right?20:13
cwaynelike it's all run in a chroot or something so i wouldn't be polluting anything i mean20:14
robrukyrofa: particularly the part about split building20:14
kyrofarobru, oh, is that what it's whining about?20:15
kyrofarobru, what about the package version?20:15
robrukyrofa: yeah there's a few packaging niggles I noticed in your branch. split building is a big one, also add '-0ubuntu1' to your changelog version and it should be able to figure the rest out from there.20:16
kyrofaThat's what I had initially and it gave me the native/non-native complaint, so I removed the "-0ubuntu1" to see if that fixed things. It didn't :P20:16
kyrofarobru, ^^20:16
robrukyrofa: you should definitely be using -0ubuntu1 in the train.20:17
kyrofarobru, okay, will do. And a native format?20:17
robrukyrofa: if you mean dropping quilt, yes.20:17
kyrofarobru, yeah, 3.0 (native)20:18
robrukyrofa: yeah read the page, you should drop debian/source entirely20:19
kyrofarobru, ah yes, right at the top :)20:19
kgunnrobru: just makin' sure i got this right...i added the unrelease entry20:28
kgunnhttp://bazaar.launchpad.net/~unity-system-compositor-team/unity-system-compositor/trunk/revision/22720:28
kgunnbut i did not delete20:28
kgunnhttp://bazaar.launchpad.net/~unity-system-compositor-team/unity-system-compositor/trunk/revision/22220:28
kgunncorrect ?20:28
kgunnor should i have deleted that one20:28
robrukgunn: nope, need you to delete what was added in 222, it's totally redundant and wrong20:28
kgunnright20:29
kgunni get it now20:29
robrukgunn: if you look a bit further down there's an identical stanza with identical version number.20:29
kgunnright20:29
kgunnredundant i see20:29
robrukgunn: thanks for taking care of that ;-)20:29
kgunnrobru: oh...meant to say, i didn't delete, cause it said entry was from ci train  bot20:30
kgunnrobru: oh...i see what happened20:30
kgunnanpok did that...but just cnp'd it20:30
robrukgunn: the original one was from the train bot, but somebody got copy&paste happy there ;-)20:30
kgunnah ha20:30
kgunnyeah20:30
kyrofarobru, seems to be working now, thank you for the link :)20:30
kgunnthrew me for a minute...but i see the commit on lp20:30
robrukyrofa: you're welcome!20:31
anpok_hm?20:32
anpok_I had issues with building u-s-c jenkinks complained about missing change log entries..20:33
anpok_and I remeber merging stuff20:33
robruanpok_: yeah you did it wrong :-P20:33
robrukgunn: ok that's weird. try making the most recent entry 0.0.5+15.10.20150506.2-0ubuntu1 or something higher. it doesn't scan the full changelog it just does a lexical comparison of the most recent changelog entry with the most recent distro version when it complains there20:35
fginthercwayne, it depends on exactly how the job is setup. Most use chroots, but there may be a couple that operate outside of that environment. Can you PM me the job(s) in question?21:32
kgunnrobru: ok, tried a couple of times...it's still upset at the changelog...not really sure what's happening here21:36
kgunnwondering if this is why anpok copied it that wily release in there?21:37
robrukgunn: I guess so.21:38
robrukgunn: did you try my suggestion?21:39
robru <robru> kgunn: ok that's weird. try making the most recent entry 0.0.5+15.10.20150506.2-0ubuntu1 or something higher. it doesn't scan the full changelog it just does a lexical comparison of the most recent changelog entry with the most recent distro version when it complains there21:39
kgunnrobru: yes21:39
robrukgunn: then why does the changelog still say '0.0.5-0ubuntu1' when I look at the merge? :-P21:40
kgunnrobru: ohhhh, i thot you meant the one under the unreleased...my bad21:40
* kgunn goes to clean mess21:40
robrukgunn: yeah, no, when the train says "is missing from the changelog, which has up to" that means that it's compared the most recent changelog entry against the most recent distro version and doesn't like what it saw. changing anything other than the most recent version number in the changelog won't make a difference.21:42
robrukgunn: sorry that wasn't clear21:42
robruboiko:  https://ci-train.ubuntu.com/job/ubuntu-landing-008-2-publish/lastSuccessfulBuild/artifact/telepathy-qt5_packaging_changes.diff/*view*/ is this for real? that's a lot of patches you're dropping.21:50
boikorobru: yep, we upstreamed almost everything21:51
robruboiko: wow congrats21:51
boiko:D21:51
boikorobru: thanks, we actually upstreamed them awhile ago, but we were waiting for an upstream release for quite some time21:51
robruboiko: ah ok21:51
boikorobru: in the end both me and salem_ got write access to telepathy-qt and in theory we can do releases now too21:52
robruboiko: nice!21:52
robrukgunn: great looks good22:10
kgunnrobru: yep, altho mterry gone for day i think22:11
robrukgunn: fine by me, we fixed the issue he raised ;-)22:11
kgunnrobru: alright! sounds good...people starting to nag :)22:12
kgunnheading to the house....bbiab22:12
=== salem_ is now known as _salem
=== _salem is now known as salem_

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