/srv/irclogs.ubuntu.com/2015/01/07/#ubuntu-quality.txt

=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== g__ is now known as Guest48587
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== vrruiz_ is now known as rvr
elopiogood morning team.14:33
elopioalesage: how is it going with system settings?14:33
rhuddieelopio, hey good morning14:37
elopiohello rhuddie14:37
rhuddieelopio, I was thinking about querying device for maps support14:37
rhuddieelopio, something like this seems to work: http://pastebin.ubuntu.com/9687891/14:38
rhuddiewhich checks if you have the click app installed14:38
elopiorhuddie: ok. Sounds fine. I  guess we could have different agps providers in the future, but then we will just update that method.14:41
elopiorhuddie: I would call the capability agps, instead of maps.14:41
rhuddieelopio, yes, you could make it as specific as you needed it to be. even have different methods for different providers if you need to know which one(s) are supported14:42
elopiorhuddie: yesterday we were wondering if the apt-get update thing should be reported as aa bug to autopkgtest. Maybe it should be done by the setup scripts, maybe with a flag or something.14:43
elopiocould you please file a bug for that, and add a comment with the bug # in your branch?14:44
rhuddieelopio, sure, I'll raise the bug. But I merged that branch earlier this morning, do you want me to revert it?14:45
elopiorhuddie: no, that's fine. You can add the comment directly to trunk, no need to MP that.14:45
rhuddieelopio, ok14:46
elopiorhuddie: second thing, you said that you filed a bug for the wizard in rtm, right? Could you please make a branch to skip that test if we are on rtm, also with a comment linking the bug?14:46
rhuddieelopio, sure. that will require the config helper so we know if we are on rtm. should I just add those as helpers for now?14:48
elopio rhuddie: yes, I think you should. Didn't you have a method to get the channel, in your last branch?14:49
rhuddieelopio, yes I do14:49
rhuddieelopio, the file I look at is only available on touch, not desktop14:50
rhuddiedon't think that should be a problem for now, but if we move these helpers to autopilot we would obviously need to make it work with desktop too14:50
elopiorhuddie: that's good for now. I think that on desktop you could use something like /etc/lsb-release. But if you want, just raise an exception leaving that part for the future.14:52
elopiorhuddie: and final comment for today... :) Last time I called a click command they told me to better use the click db.14:54
elopiohttp://bazaar.launchpad.net/~click-hackers/click/trunk/view/head:/click/commands/list.py#L2914:54
elopioyou can do something like db = Click.DB(); db.read(); registry = Click.User.for_user(db, name=os.get_env('USER'))14:55
rhuddieelopio, thanks I'll take a look.14:56
elopiothat is also good because you won't have to parse a string for the answer. You can query the registry.14:56
rhuddieelopio, fyi, this is the bug for rtm wizard: https://bugs.launchpad.net/ubuntu-rtm/+source/ubuntu-system-settings/+bug/139957615:00
ubot5Launchpad bug 1399576 in ubuntu-system-settings (Ubuntu RTM) "Cannot introspect settings wizard" [High,In progress]15:00
elopiorhuddie: great, thanks.15:01
=== chihchun is now known as chihchun_afk
elopioalesage: where are you?15:55
alesageelopio, here :) , sorry other pings, waiting on a build of system-settings to get your answer15:56
elopioalesage: ok. I'm getting my env ready to give it a try.15:56
=== G4MBY is now known as PaulW2U
alesageelopio I need to be updated on how to run an autopilot test on the phone, is the best way to shell into the phone and run from the source tree (for a unity8 autopilot tests which I don't think is dep8-able)?16:07
elopioalesage: it all depends what you are running.16:07
elopiobest way is to do what jenkins does, https://wiki.ubuntu.com/Touch/Testing16:07
elopioif you are debugging, you might want to make the phone writable, install bzr and emacs, and get the branch into the phone.16:08
alesageelopio, right, I'm usually doing that--just want to be sure that some nifty utility hasn't come into existence as I'm advising paulliu on how to run indicators tests on phone16:09
alesageelopio, balloons, what's your fav unity8 test, page-object-model-wise?16:16
elopioalesage: the new one: https://code.launchpad.net/~canonical-platform-qa/unity8/edges_demo_test/+merge/24488916:18
=== roadmr is now known as roadmr_afk
alesageelopio, thx16:22
balloonsi'll go with leo alesage :)16:23
alesageballoons, very wise choice16:24
rhuddiealesage, as we now need 2 approvals, could you take a look at: https://code.launchpad.net/~canonical-platform-qa/ubuntu-sanity-tests/config-helpers/+merge/24575416:26
alesagerhuddie, ok shall do in a little bit16:26
rhuddiealesage, thank you16:26
alesagerhuddie, nothing interesting to add to that MP, approved :)16:47
rhuddiealesage thanks :)16:47
=== roadmr_afk is now known as roadmr
alesageelopio, have you worked with paulliu before?  he's helping with the indicators helpers, may have questions about how best to implement the page object model17:06
elopioalesage: for a short time when he was working on pay.17:06
alesageelopio, aha ok17:06
elopioalesage: tell him to ask here or on ubuntu-autopilot.17:07
alesageelopio, done :)17:07
elopiopaulliu: oh, right, you are here :)17:07
elopiopaulliu: if you need a hand, just ping ubuntu-qa.17:07
=== chihchun_afk is now known as chihchun
alesageelopio I'm running the sanity suite on a fresh flash and I'm failing due to apt-get 404's, have you witnessed?  feel a bit blocked by this18:10
elopioalesage: do you have trunk?18:10
alesageelopio, I'm reviewing https://code.launchpad.net/~canonical-platform-qa/ubuntu-sanity-tests/serial/+merge/24559918:11
elopioalesage: I'll merge it with trunk.18:11
elopioalesage: give it a try to see if Richard's apt-get update solves it.18:12
alesageelopio, k18:12
alesageelopio, surprised if so b/c a manual apt-get update still 404s :( , running now18:13
alesageelopio, appears to have worked18:16
elopioalesage: awesome. I also tried the apt-get update myself, but I think what we are doing now is a little different. It temporarily mounts the partition as writable.18:17
alesageelopio, o I see18:17
alesagedark science18:18
elopioyeah, I didn't understood it all. But Richard should have filed a bug by now.18:18
alesageelopio, were you able to discover anything re: my settings blockage?18:52
alesageelopio, here's the latest result on Jenkins FWIW http://s-jenkins.ubuntu-ci:8080/job/generic-deb-autopilot-runner-vivid-mako/603/18:53
elopioalesage: no, I was waiting for you to tell me if you needed a hand.19:04
elopioI will get to it after the standup.19:05
alesageelopio, ok yes I might need a hand at least to brainstorm19:05
elopioonly one error, that's pretty good. And no longer pagecomponent errors. We can work with that.19:05
alesagebut this sound failure is a mystery (and I've skipped all the other failures :) )19:07
elopiohey veebeers, is the --test-timeout flag released in rtm?19:28
dobeyok, i'm confused.19:33
dobeythe autopilot tests need to be included within the click package, to be runnable with the x-test stuff in the manifest, with adt-run, right?19:34
dobeyotherwise adt-run foo.click couldn't possibly work, right?19:36
dobeyalso, isn't filemanager supposed to be a fat package?19:45
=== roadmr is now known as roadmr_afk
balloonsdobey, terminal and fm both got published as fat i believe19:59
dobeyballoons: i just installed the update for fm on my mako, and looking at the file system, i only see the arm binaries20:01
balloonsdobey, adt run works by pulling tests from the source tree @ revision20:03
balloonsas soon as someone pushes an update its likely to lose fat package support, as people have to manually build the fat package20:03
balloonsso tests don't go in the click20:03
dobeyhuh20:05
dobeyi added the same stuff that fm has in it's CMakeLists.txt to install the ap tests, in pay-ui, and when i build the click the console log shows the files being installed to the place where the .click is built from20:06
dobeyso i don't need to add that?20:07
Letozaf_balloons, hi20:09
balloonsLetozaf_, hi20:09
Letozaf_balloons, I found out that editing a topic on shorts app does not work, shall I file a bug ?20:10
Letozaf_balloons, I found it out working on the edit topic tes20:10
Letozaf_test20:10
balloonsdobey, ? not sure what you are trying to do, but no. the tests are pulled from source (or passed in). the click should be the same as the end user20:11
balloonsLetozaf_, that's with the changes I take it?20:11
balloonsand yes, a bug is alwasys useful :)20:12
dobeyballoons: i'm trying to add autopkgtests to the pay-ui click. i was looking at filemanager for hints on how to do it.20:12
Letozaf_balloons, I was wondering If I was wrong as if you read in my mp: https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/fixed-test-edit-topic/+merge/245411 Joey and Roman don't seem to agree with me20:12
Letozaf_balloons, by the way: howzit ?20:13
balloonsdobey, your manifest does need to be formatted properly, but it probably is already. can i see the source?20:13
balloonsLetozaf_, not bad at all thanks20:14
Letozaf_balloons, looks like you are typing quite quikly :)20:14
balloonshah. hunt and peck super speed mode!20:15
Letozaf_balloons, :-)20:15
dobeyballoons: i haven't added the autopkgtests bits to it yet. i thought the tests had to be installed, so i split the work into two tasks20:16
balloonsdobey, ahh. yep, adt can just be pointed at an installed click and run the tests for it by pulling the click manifest and working it out20:18
balloonsor pass it tests; or tests and a click20:19
=== roadmr_afk is now known as roadmr
alesageelopio, given what we talked about in stand-up maybe I'll just switch to temporarily skipping those tests, open a bug for each, then fix the non-Jenkins-related ones (i.e. the 'about' test we discussed yesterday)20:55
elopioalesage: yes. Make a branch with the skips. Run it 3 times to make sure it's green. Then make a branch that depends on that one, where we will remove the skips and solve the issues.20:58
elopioit's only required to file bugs if the first branch lands before the second one is ready.20:58
alesageelopio, ok now I know the process thx20:59
elopiook, I'm off to pick my bike from the mechanic, and have lunch.21:04
elopiobbl21:04
balloonsLetozaf_, sorry i missed your other ? earlier21:22
balloonsLetozaf_, did you need help with that mp?21:23
Letozaf_balloons, I am reporting a bug as changing a fedd's topic does not work21:23
Letozaf_balloons, then I will put a note in the mp21:24
balloonskk21:24
Letozaf_balloons, :)21:24
dobeyuhm22:44
dobeyballoons: still here?22:44
balloonsdobey, for a moment yes22:46
dobeyballoons: do you know if doing adt-run on a click will always create a user session with xvfb or what?22:47
balloonsdobey, it depends on the runner you choose22:52
dobeyi presume the runner is the thing where i put "autopilot" in the manifest?22:52
balloonsi should use the proper terms.. the virt server i guess it's called22:53
balloonsadt-run [options] <source package> --- <virt-server> [<virt-server options>]22:53
balloonshttps://people.debian.org/~mpitt/autopkgtest/README.running-tests.html22:54
dobeythat readme doesn't really answer my question :-/22:56
dobeyoh22:56
dobeyvirt-server isn't what you meant22:56
dobey--setup-commands is?22:56
* dobey builds a click and tries23:01

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