/srv/irclogs.ubuntu.com/2015/10/08/#ubuntu-ci-eng.txt

=== chihchun_afk is now known as chihchun
robruMirv: I don't know if it makes sense for you, but dual silos can now have manual sources, so if you're doing the same qt packages in vivid and wily, the train can handle that in a single silo now05:36
Mirvrobru: yeah I was thinking about it, it sounds really good. so far I've needed two silos for all OTA Qt fixes!05:37
Mirvrobru: it's not the same packages, but same patch being applied. wily has 5.4.2, vivid 5.4.1. but I guess since I upload the manually train is fine with that?05:37
robruMirv: yeah train won't enforce the versions matching or anything. it'll just watch the builds & do the publishing05:38
Mirvrobru: the train rocks!05:38
robruMirv: thanks. long way to go still ;-)05:38
robruMirv: I'll be around for a bit (just playing video games). ping me if anything explodes05:39
Mirvrobru: ok, have fun05:39
robruThanks05:39
=== chihchun is now known as chihchun_afk
=== marcusto_ is now known as marcustomlinson
=== chihchun_afk is now known as chihchun
SaviqUrsinha, we don't need anything special for our qml tests these days. we run them all in xvfb so we only basically need CPU07:58
seb128hey there08:05
seb128seems like there is an issue with the uitk/rc-proposed that makes some store apps not start08:05
seb128bug #150390108:05
ubot5`bug 1503901 in ubuntu-ui-toolkit (Ubuntu) "Authenticator app won't open with latest SDK" [Undecided,New] https://launchpad.net/bugs/150390108:05
seb128should probably be an ota7 blocker08:05
seb128Mirv, ^08:07
Mirvseb128: targeting so08:07
seb128Mirv, thanks08:07
jibelseb128, thanks for the bug, continue testing OTA7. ;)08:12
seb128jibel, yw!08:12
sil2100Blockers make me sad08:13
jibelseb128, did you reproduce it?08:13
jibelit works fine here08:13
seb128no, just tried and it works (but I don't have an active config in that app so maybe the clear state is not enough to hit the issue), but mzanetti did apparently, he is discussing in on #sdk08:15
mzanettiyes, found the issue08:16
mzanettihttp://paste.ubuntu.com/12712813/08:16
mzanettihere's a standalone example to repro it08:16
jibelmzanetti, it is a problem in the sdk or the app?08:23
mzanettiwell, the app doesn't have the prettiest code ever in there, but nevertheless it should not crash the SDK08:24
mzanettias this was working until now I guess it should be fixed in the sdk08:24
jibelokay08:24
seb128the code does nothing buggy/illegal08:25
jibelbzoltan_, can you look at bug 150390108:25
ubot5`bug 1503901 in Canonical System Image "Authenticator app won't open with latest SDK" [Critical,New] https://launchpad.net/bugs/150390108:25
jibelbzoltan_, it'll block OTA7 as it breaks existing apps08:25
jibelmzanetti, can you update the bug report with your findings please.08:27
mzanettidone08:27
jibelthanks08:27
bzoltan_Mirv:  how happy the CI train is with git repos?08:56
anpokcihelp: the krillins used by our mir-mediumtests-runner-touch are those flashed with rc-proposed images?08:57
psivaaanpok: yes,08:59
psivaarc-proposed/ubuntu/krillin/version-299 was the latest08:59
sil2100bzoltan_: not very happy I suppose08:59
sil2100bzoltan_: it's still hard coded to bzr from what I know08:59
=== chihchun is now known as chihchun_afk
* psivaa -> errands09:00
anpokpsivaa: thx..09:01
=== chihchun_afk is now known as chihchun
zbenjaminsil2100: sigh :(09:03
zbenjaminsil2100: any chance this will change? Like now? :D09:04
=== chihchun is now known as chihchun_afk
sil2100zbenjamin: it's in the plans, yes09:11
=== chihchun_afk is now known as chihchun
Mirvbzoltan_: same answer as sil210009:14
bzoltan_Mirv:  thanks09:15
=== sil2100 changed the topic of #ubuntu-ci-eng to: Train trouble? ping trainguards | CI problems? ping cihelp | Train: http://bit.ly/1hGZsfS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: OTA-7 #1 candidate testing in progress, landing gates open
zbenjaminsil2100: that would be awesome09:30
seb128jibel, oSoMoN_, shouldn't the settings from silo 49 land for ota7 if the confined webbrowser landed? (it has a fix needed to make the non click .desktop be used)09:34
oSoMoN_seb128, it would be nice to have indeed, but not essential I would say09:35
seb128oSoMoN_, well, the silo is safe and waiting for qa since monday and fix an annoying bug09:35
seb128unsure why it didn't get verified/published09:35
oSoMoN_seb128, not sure either, but it’s tagged for next milestone, that would explain why QA didn’t validate it09:36
seb128where are things "tagged"?09:36
oSoMoN_seb128, in the trello board that tracks QA’s work, and I believe the tags are assigned based on the LP milestone of the corresponding bug reports09:37
seb128where is that trello again?09:37
oSoMoN_jibel, can you confirm the above?09:37
mzanettijibel, do you know if this is going to be included for OTA-7 https://bugs.launchpad.net/ubuntu/+source/network-manager/+bug/148087709:37
ubot5`Ubuntu bug 1480877 in location-service (Ubuntu) "Access points' "PropertiesChanged" dbus signals freeze UI on mobile devices" [Undecided,Incomplete]09:37
oSoMoN_trello.com/b/AE3swczu/silo-testing09:37
seb128thanks09:37
jibelmzanetti, the fix for network-manager is in OTA7, not sure if there is also a fix need for location-service and indiator-network09:58
jibelneeded*09:59
jibelseb128, it didn't land because none of the fixes were targeted for this OTA10:01
mzanettijibel, ah ok, great... just asking because it rendered my phone completely useless on the conference I've been the last two days10:02
mzanettibut I wasn't up to date with the very latest image either10:02
=== oSoMoN_ is now known as oSoMoN
UrsinhaSaviq: thank you10:23
SaviqUrsinha, basically, the only need we have there is that Jenkins runs our autopkgtests, which now run all our QML tests10:24
UrsinhaSaviq: right, we were trying to understand if hardware was required for the slaves, if not it's easier10:26
=== chihchun is now known as chihchun_afk
bzoltan_sil2100: Mirv: the bugfix branch for #1503901 is already in silo. I have fresh archive tests, so i expect the silo to be QA ready tomorrow monring.11:25
bzoltan_mup: are you lazy or what?11:26
bzoltan_https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/150390111:26
ubot5`Ubuntu bug 1503901 in ubuntu-ui-toolkit (Ubuntu) "Authenticator app won't open with latest SDK" [Critical,In progress]11:26
jibelbzoltan_, not before tomorrow morning?11:27
jibelbzoltan_, we already started regression testing and an update of uitk invalidate almost everything11:27
bzoltan_jibel: The build is ready in half an hour11:27
jibelk11:28
bzoltan_jibel:  it is a bug what was reported 2am my time. It was tagged up just 2 hours ago. I have the fix in silo already.11:29
=== alan_g is now known as alan_g|lunch
bzoltan_jibel: popey: I could use a good list of click packages what come with even the simplest AP test to extend the UITK test plan.11:31
bzoltan_popey: I would wget the click packages, install on the device and run the AP tests if they have such11:32
Mirvbzoltan_: thanks!11:32
sil2100bzoltan_: thanks!11:43
=== _salem is now known as salem_
seb128jibel, ok, should we land it still? or declare it a minor bug with the new feature and delay to next ota update?11:58
=== alan_g|lunch is now known as alan_g
rvrpopey: Are you in charge of reminders-app (Notes)?12:00
popeyrvr: wassup?12:03
rvrpopey: There are some missing strings in the app12:03
rvrpopey: A language update would be cool12:04
rvrpopey: For example, in Spanish, "close" button is presented in English. Also, the Evernote prompt at first start also has a phrase still in English.12:05
rvrpopey: I think both are now translated in Launchpad12:05
popeyrvr: Ok, I'll put a call out for language update, thanks rvr12:06
jibelseb128, bug 1501428? I don't think it's minor as it exposes wrong or missing information in security settings. We should try to land it and include it iun next respin but I'll defer to Pat for the final decision12:10
ubot5`bug 1501428 in ubuntu-system-settings (Ubuntu) "TrustStoreModel fails to expose application name for non-click app" [High,In progress] https://launchpad.net/bugs/150142812:10
seb128jibel, right, good ... just to know, what should be done next time to flag the silo as "should be consider for the coming ota"?12:12
seb128because I failed at that12:12
seb128IRC naging doesn't seem to be an answer either12:12
seb128(well it works, but we shouldn't rely on that)12:12
jibelseb128, add a canonical-devices-system-image task. It'll be reviewed by the project team and targeted for an ota12:13
seb128k12:13
seb128thanks12:13
jibelseb128, if it's not reviewed on time and you think it's important pinging people helps12:14
jibelseb128, the MR is not reviewed and top approved12:14
jibelhttps://code.launchpad.net/~ken-vandine/ubuntu-system-settings/lp1501428/+merge/27306612:14
seb128jibel, that's not in the landing/silo?12:16
seb128jibel, https://requests.ci-train.ubuntu.com/#/ticket/45212:16
seb128jibel, we wanted to land the safe/simpler version of the fix which is in the silo for that ota and review/land Ken's change for next one12:17
jibelseb128, right, sorry, I was checking the ones attached to the bug12:18
=== alan_g is now known as alan_g|lunch
jibelpmcgowan, what do you think about landing silo 49 in OTA7? it contains a fix for bug 1501428 ie the item is blank for the webbrowser in security settings.13:33
ubot5`bug 1501428 in ubuntu-system-settings (Ubuntu) "TrustStoreModel fails to expose application name for non-click app" [High,In progress] https://launchpad.net/bugs/150142813:33
jibelpmcgowan, other fixes in the silo are minor and low risk13:33
pmcgowanlooking13:34
jibelwe blocked ota6 for a similar issue with the camera13:35
pmcgowanjibel, yes looks good to me13:38
pmcgowanfall out of previous landing13:38
pmcgowanI will tag the other bugs too13:38
mzanettipmcgowan, I think we want to fix this too: https://bugs.launchpad.net/ubuntu/+source/unity-system-compositor/+bug/150214513:41
ubot5`Ubuntu bug 1502145 in unity-system-compositor (Ubuntu) "rc-proposed r140, krillin: screen does not blank after timeout expires" [Undecided,Confirmed]13:41
pmcgowanmzanetti, but there is no fix there :(13:45
pmcgowanis that a recent regression I assume?13:45
=== alan_g|lunch is now known as alan_g
pmcgowanmzanetti, ?14:07
mzanettipmcgowan, no, no fix yet, but the issue is quite critical IMHO14:09
pmcgowanmzanetti, and it can be reproduced?14:10
mzanettipmcgowan, my krilling here seems to turn itself on all the time and never turn off again14:11
pmcgowanjust krillin or do we know14:11
pmcgowanI am not seeing on mx414:11
mzanettiI'll try repro with something else14:12
ChrisTownsendtrainguards: Hi, we no longer need https://requests.ci-train.ubuntu.com/#/ticket/401 since the change in that silo was merged in a different request yesterday.  Do I just click "Abandon" to get rid of it?14:18
sil2100ChrisTownsend: yes, should be enough :)14:19
ChrisTownsendsil2100: Ok, thanks14:19
sil2100jibel, davmor2, rvr: is ubuntu/landing-023 in the queue to land?14:19
sil2100Since it's UITK so I suppose it's good to land this ASAP, right?14:19
jibelsil2100, not marked ready for QA but we will take it14:20
sil2100Ah, right, it's QA Required still14:20
sil2100bzoltan_: piing14:20
bzoltan_sil2100:  tell me14:22
sil2100bzoltan_: how's silo 23 going? Ready for QA sign-off soon?14:22
bzoltan_sil2100: Hmm... do you guys want me to run a real UITK Test Plan? That takes ~24h. The archive test to compare the silo tests with I have started 10 hours ago and in few hours it is ready... teh base14:23
bzoltan_sil2100:  but the build is there, I ca do sanity check on it... but that is not a proper UITK test plan14:24
sil2100jibel: ^ what does QA think? How much testing would you like on that one?14:24
bzoltan_sil2100: jibel: the change compate to the released version is subzero14:25
bzoltan_sil2100: jibel: it is a cherry pick landiing with few lines of code change14:25
jibelbzoltan_, what is the risk of potential regression introduced by this change?14:27
bzoltan_jibel: less then any landing before... check the diff, it is super minimal.14:31
jibelbzoltan_, I saw the diff but what is the impact on the rest of the toolkit? to target testing around this area instead of re-running things that are not impacted at all14:32
bzoltan_jibel: I do the test plan anyway tonight .. so I will tell in the morning if I find something. But I would suggest to proceed with this version14:32
greybacklol at citrain page with JS disabled15:20
anpokhehe15:27
mzanettipmcgowan, so far no luck reproducing on mako.. still happening on my krillin all the time15:32
pmcgowanmzanetti, how do you trigger it?15:35
mzanettipmcgowan, I don't do anything special... I just upgraded to the latest rc-proposed last night and it happens15:36
pmcgowanjibel, ^15:38
mzanettifaenil reported this on monday already... I just didn't upgrade earlier because I was travelling15:39
Saviqsil2100, hey, do you know what's the status of dbgsym packages in PPAs? I read somewhere LP supports publishing those now, but has to be enabled? could we enable in silos and overlay?15:47
sil2100Saviq: the overlay has those enabled IIRC15:48
sil2100Not sure about silos tho15:48
Saviqsil2100, hmm I wonder, wouldn't it mean train has to copy them actively from silos?15:50
Saviqsil2100, like, overlay can only have them if it builds itself, when copied, they have to be there in silos in the first place?15:50
Saviqsil2100, oh, silos do have them, but maybe not in the archive? apt can't see them fwiw15:52
sil2100Saviq: http://ppa.launchpad.net/ci-train-ppa-service/stable-phone-overlay/ubuntu/dists/vivid/main/debug/ <- the overlay seems to have them too15:53
Saviqsil2100, ok, but they're not added to Packages?15:54
Saviqi.e. I can't simply use apt to install them?15:54
Saviqor should I add main/debug?15:54
Saviqthink that's what I was missing15:54
Saviqsry15:54
Saviqdidn't know how to work with that yet15:54
pmcgowanmzanetti, sorry was otp, do you just turn the phone on and it doesn't timeout?16:01
mzanettipmcgowan, yep16:01
mzanettipmcgowan, also, if I receive a notification that turns it on, it won't time out either16:02
pmcgowanmzanetti, jibel could not reproduce it which may be a clue in itself16:02
pmcgowanwhat does upower say its state is16:03
mzanettipmcgowan, seems u-s-c is holding it awake. alf is looking into it too16:05
pmcgowanok16:06
seb128does anyone see why https://ci-train.ubuntu.com/job/ubuntu-landing-009-1-build/369/console is failing?16:06
seb128hum16:07
seb128./debian/rules: Permission denied16:07
seb128wth?16:07
seb128cihelp, ^ (or is that a trainguards issue?)16:13
fgintherseb128, that would be trainguard16:14
seb128fginther, thanks16:14
robruseb128: sorry was in meeting16:34
seb128no worry16:34
seb128seems like CI build doesn't like the fact that indicator-session debian/rules is not +x16:35
seb128but that was not an issue before16:35
seb128and dpkg-source and other tools usually fix that16:35
seb128unsure what changed that makes it a problem16:35
robruseb128: what changed if we started calling "rules clean" before builds16:36
seb128seems buggy16:36
seb128and why?16:36
robruseb128: i can make it chmod first but it's easier if you just set the permission for now16:36
seb128I did16:36
seb128still a bug in the system16:37
robruseb128: we did this so people could hook into that and munge their control files prior to building, eg so they can have different abi from same source package16:37
seb128I see16:37
seb128well in which case adding the chmod +x seems like a good thing to do16:38
seb128or ignoring the failing clean calls16:38
robruseb128: yeah i can fix it but will take some time to get the fix in production16:39
seb128that's ok, I did a chmod +x mp for indicator-session meanwhile16:39
robruOk16:39
seb128robru, thanks16:39
robruYou're welcome16:40
=== alan_g is now known as alan_g|EOD
bzoltan_robru: jibel: rvr: The silo23 with the tiny fix is good for QA validation17:36
rvrjgdx: ping17:42
jibelbzoltan_, good, alesage is on it17:44
alesagejibel, bzoltan_ ack17:45
rvrjgdx: "Open System Settings and call a SIM Service number containing a #, e.g. +47005#1"17:45
bzoltan_alesage: jibel:  super, thank you. Feel free to ping me, I try respond even during the night17:45
rvrjgdx: I think my SIM service numbers don't have any hash characters17:46
jibeltrainguards you can publish silo 23 it is for ota719:27
robrumterry: around for a packaging ack? https://requests.ci-train.ubuntu.com/#/silo/2319:40
mterryrobru, hiyo sure19:40
robruthanks19:40
robruoh hah, it's just the trivial debian/watch19:41
robruI should fix the train to not consider that as requiring an ack...19:41
robruslangasek: can you remind me again why the -gles packages need to be separate source packages and can't just be build from the same sources?19:42
mterryrobru, yeah19:42
robrumterry: need you to publish for now please19:43
mterryrobru, was about to ask if I was looking at right diff  :)19:43
mterryrobru, I did19:43
mterryoh whoops19:43
mterrynow I did19:43
robrumterry: yeah the first time you click build it just logs you in and redirects to the form19:44
robrumterry: I'll be so happy when we finally get rid of jenkins19:44
jgdxrvr, right, it's difficult to test19:51
rvrjgdx: So, what can I do?19:58
slangasekrobru: you can't build -gles and non-gles as part of the same source package because of conflicting build-deps20:01
robruslangasek: why do the build deps conflict? What if we made the build deps coinstallable so that we could unify the source packages?20:02
robruslangasek: what if we made a debian/rules that installed one build dep for over build then switched to the other for the other build? ;-)20:03
slangasekrobru: they conflict because GL and GLES have conflicting APIs and there's no reasonable way to make them coinstallable without doing a lot of build system work20:03
slangasekrobru: then that debian/rules would be shot out of a cannon20:03
robruHehe20:03
jgdxrvr, I'm thinking20:04
robruslangasek: what if the rules file did the builds in two separate chroots with the different build deps?20:04
slangasekrobru: you don't get to do anything that requires root in a debian/rules.20:05
robruslangasek: what if we invented a user mode chroot? ;-)20:05
robruslangasek: what if, instead of the train enforcing users to manage gles twins, the train just made gles copies similarly to the way it makes vivid copies. Would reduce a lot of hassle for people i think20:07
jgdxrvr, are you able to make the device rw to test this?20:09
rvrjgdx: Yes20:09
jgdxrvr, okay, then I guess there's a way. You have to +rw and install python3-dbusmock20:10
rvrjgdx: Checking20:13
jgdxrvr, next up, stop ofono. $ sudo stop ofono ?20:19
jgdxlet me know how it goes20:19
rvrjgdx: ofono stop/waiting20:21
jgdxrvr, okay, then $ sudo python3 -m dbusmock --template ofono20:24
jgdxand in a new terminal, do phablet-shell and then $ gdbus call -y -d org.ofono -o /ril_0 -m org.freedesktop.DBus.Mock.AddProperty "org.ofono.SimManager" ServiceNumbers "<{'Foobar': '+47#000'}>"20:24
jgdxshould output ()20:24
jgdxopen system settings and test20:24
rvrhmm20:25
rvrDone20:28
rvr()20:28
jgdxrvr, okay, System Settings -> Phone -> Service numbers -> Foobar -> Call20:29
rvraha!20:30
rvrDone20:30
jgdxthe dialer should have +47#00020:30
rvrYes20:30
jgdxnot just +4720:30
rvrRight!20:31
kgunnrobru: you'd be the bomb if you automated twins20:33
robrukgunn: I really hate the way gles are handled currently and I'd love to clean that up20:36
robrukgunn: like, it really bothers me on a deep and personal level ;-)20:36
kgunn:)20:36
robrukgunn: only because I know that a better way exists.20:36
robrukgunn: can't guarantee a timeline but I'm looking into solutions20:37
kgunnme likie20:38
jgdxrvr, what'd you get20:38
rvrjgdx: Yes, I got +47#00020:38
jgdxrvr, wheee! Thanks for you patience20:39
rvrjgdx: I'm going to do some additional checks before approving the silo20:39
jgdxokay, I'm going to bed. Night!20:40
rvrjgdx: Good night!20:41
robrukgunn: did you ever notice that the train can't generate diffs for the non-gles version of packages that have -gles twins? I just noticed today and it doesn't seem like it's a result of anything I've changed recently, but I have a hard time believing that the train always failed to diff ubuntu-ui-toolkit20:44
kgunnhuh...i never noticed20:51
robrukgunn: it's due to the watch file... when it builds the gles variant, it downloads the orig.tar over the non-gles one, then renames it to the -gles one, leaving the non-gles one without an orig.tar to diff. pretty basic stuff, it's been using debianwatch files all along...20:53
robrukgunn: but yeah I never noticed until today. there must have been diffs in the past...20:53
robruwat21:24
=== salem_ is now known as _salem

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