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

AmacidiaSo I guess I'm already to start contributing, I have test drive installed and synced the latest version of raring.04:31
AmacidiaDo I just work on Week 3 tests ?04:32
AmacidiaOrca perhaps?04:32
balloonsAmacidia, hello04:53
AmacidiaHi balloons04:53
AmacidiaHow are you?04:53
balloonsnot too bad.. About to shut down for the evening, but let me answer your questions quickly if I can04:54
balloonsone of the was you can contribute is by writing/running tests..04:54
AmacidiaThanks balloons. I'm thinking of starting off by work on week 3 tasks as indicated by the schedule. Am I correct ?04:55
balloonsAmacidia, yes, indeed go for it04:55
AmacidiaI do some programming at work, however I don't work directly with python.04:55
balloonsthis is the close of the cadence week for week 304:55
balloonsAmacidia, ahh.. well, if your willing to give it a try, we can help.. the python isnt too bad04:56
AmacidiaI can definitely look into this week for sure. I'm not sure if the classes being planned for this month cover writing tests do they?04:57
balloonsanyways, hopefully the iso tracker makes sense to you.. I'm generally around 1400-2200 UTC so feel free to ping, or email04:57
balloonsAmacidia, no, but we have hackfests04:57
balloonsthere are some planned for this month04:58
balloonsit does cover this stuff :-)04:58
AmacidiaOh well that would be awesome04:58
balloonsI don't have the dates handy, I think 3 are scheduled right now04:58
balloonsit was sent to the list in an email04:58
balloonsif you don't find the dates, just ask.. ;-_)04:58
AmacidiaI didn't join the lists too long ago...04:58
balloonsI'll be on my proper pc again tomorrow, so we can chat more then04:59
balloonsits night for me here at least.. so good night!04:59
AmacidiaTake care, nice chatting with you.04:59
balloonsnice meeting you! happy testing!04:59
pittiGood morning05:01
AmacidiaHey pitti05:06
AmacidiaI'm new to testing, so this might be a dumb question, but I'm looking at the daily iso tracker for raring and it doesn't seem like a lot of people have submitted test results, is that normal?05:08
pittiAmacidia: yes, it is; we have some intense testing around milestones and regularly over the cycle, but not every day05:11
AmacidiaThanks pitti, so would you say I'm wasting my time working on daily iso testing?05:12
pittino, feedback on those is always welcome05:12
pittiwe have some automatic tests to ensure that they install at all, but there are a lot of problems only humans can see05:12
Amacidiasounds good05:13
jibelGood morning08:08
=== Noskcaj is now known as Noskcaj-afk
=== Noskcaj-afk is now known as Noskcaj-sleeping
=== _salem is now known as salem_
pittijibel: FYI, restarting ubiquity autopkgtest; that's supposed to work again, right?12:10
jibelpitti, thanks, it's back to normal12:38
dkesselhello :)13:35
dkesselhas anybody got time to look at my file-roller autopilot branch ? it is not listed and executed by autopilot, and I can't find my mistake13:36
dkesselmy branch is here: lp:~d-kessel/ubuntu-autopilot-tests/file-roller13:37
=== fader_` is now known as fader_
=== plars_ is now known as plars
zygaroadmr: how are lp:ubuntu/$series/checkbox repositories related to the ubuntu source package for checkbox?15:16
roadmrzyga: awesome question15:17
roadmrzyga: I don't think there's anything that auto-packages the bzr branch and generates the package; from what I've read, uploaders are still required to build and dput the package in addition to uploading the branch15:18
zygaroadmr: so they are related in theory but in practice that's just smoke and mirrors?15:18
zygaroadmr: can we upload to lp:ubuntu/$whatever/checkbox ourselves?15:19
roadmrzyga: yes to the smoke and mirrors (if I understand the material correctly)15:19
roadmrzyga: and we should be able to upload there ourselves, yes15:19
zygaroadmr: ok15:21
zygaroadmr: that's pretty good actually15:21
roadmrzyga: yes, it'll be quite useful15:22
roadmrzyga: of course if we want to upload to non-development releases (i.e. SRU) we need to get approvals first anyway15:22
zygaroadmr: wait, how does that work?15:23
zygaroadmr: if there was a can_upload(package, repo) function, how would you specialize it for package=='checkbox'15:24
roadmrzyga: I *think* packages are held in some sort of queue and someone has to approve them moving to the actual archive15:24
zygahmm15:25
zygaat all times?15:25
zygaor only during freezes?15:25
roadmrzyga: I'm not 100% sure really, I need to read up on it a bit15:25
zygaroadmr: ok, it would be good to get that understood15:26
roadmrzyga: for the development release, there's no queue/approval needed if there's no freeze15:26
zygaroadmr: and I'd like to get plainbox into ubuntu as well15:26
roadmrzyga: after ff, of course, all uploads need some sort of approval15:26
zygaroadmr: having plainbox in the archive would simplify some stuff for me15:26
roadmrzyga: yay, we can go the debian route but I'm sure there's a way to upload to ubuntu first/only, man, so much stuff to read and so little time..15:31
zygaroadmr: I'd rather just get it to ubuntu now15:32
zygaroadmr: then if needed push to debian15:32
roadmrzyga: makes sense15:32
zygaroadmr: I got access to the QA lab btw15:36
roadmrzyga: yay! I also got the vpn keys, haven't configured them though15:36
zygaroadmr: we should sync this week to understand how to use that stuff15:37
zygaspineau: did you have the time to look at https://code.launchpad.net/~zkrynicki/checkbox/zyga/+merge/14186515:43
spineauzyga: I've started. Need a closer look to the tests now15:44
zygaspineau: thanks15:44
zygaspineau: I'd like to get it landed to unblock the next branch which does a few big changes15:45
spineauzyga: I understand15:46
zygaspineau: also if there is anything that I can do to make such changes easier to review, tell me please15:46
balloonsdkessel, I'll have a look15:52
dkesselballoons, thanks16:04
dkesselballoons, i will be back later16:06
balloonskk16:06
zygahttps://code.launchpad.net/~zkrynicki/checkbox/zyga/+merge/14216417:12
* zyga takes a look at all the merge requests17:16
balloonsdkessel, hmm.. not sure what's going on with your tests17:17
dkesselthat makes two then :)17:17
balloonsohh17:17
balloonslol , I see it17:17
dkesselbtw it would seem logical to name tests by the executables or packages. which is not the case for terminal, for example17:18
balloonshmm.. yes, something to think about17:19
balloonswhat did we do for gnome-terminal?17:19
balloonsahh.. just terminal17:19
balloonsok, so kill the '-' in the name, then it works.. also you have some non-ascii quotes in there17:20
balloons'17:20
balloonsdkessel, the reason is you can't have a dash in the python module name17:21
dkesseloh :) lol17:21
balloonsyes, hence my lol when I saw it17:22
balloonsI looked at your filename and it clicked17:22
dkesselmeh :) but the folder naming would be ok?17:22
balloonsit should align17:23
balloonsso you should drop the dash there also..17:23
balloonsyou could leave it and it would work, but it would be confusing17:24
balloonsand there goes your naming convention dkessel :-(17:24
dkesselyup....17:24
balloonsunderscores are allowed though17:24
balloonspotentially you could use that instead17:24
* zyga is unappy about https://code.launchpad.net/~sylvain-pineau/checkbox/bug1091633/+merge/14105417:24
dkesseli've never seen an underscore in a package name, so it might work to replace dashes with underscores.17:25
balloonsI believe it should.. I'm not sure I'm a fan, heh17:25
balloonsbut yea, it should work17:25
h01gerunderscore in package name will not work17:26
balloonsh01ger, why do you say that? I did a quick change and it seems to recognize the module still17:27
h01gerbecause underscores in package names are forbidden by debian policy as they seperate the package name from the version17:28
balloonsh01ger, ohh yes yes.. I was speaking about python module names17:30
balloonsI think we crossed some terminology dkessel17:30
h01gerah, ok17:30
balloons:-)17:30
dkesselso using underscores would enable automation of stuff - a script could replace underscores by dashes and then know which package is tested17:33
* zyga looks at roadmr's patches17:34
roadmr:)17:36
roadmrthanx17:37
balloonsdkessel, hmm.. we could look to have a 1 for 1 replacement17:38
balloonsundersocre probably not a good idea as a character, as it's used validly for other things17:38
dkesselhmm ok - i'll wait before i commit my renames :)17:39
dkesselhmm python identifiers seem quite limited17:48
dkesseland *_* is a valid unrestricted identifier17:48
dkesselhttp://docs.python.org/2/reference/lexical_analysis.html#identifiers17:49
* zyga is back from dinner18:12
dkesselis there an autopilot helper method to select a specific value in a combobox? if not, how would i do that given the item i want to select may not be at the same position in different program versions?18:15
balloonsdkessel, I've been working with alesage and thomi on getting introspection and gtk apps going.18:18
balloonsthat I believe would be your solution18:18
balloonsfor the moment, you have to assume the positionj18:18
zygaroadmr: https://code.launchpad.net/~roadmr/checkbox/1084986-report-missing-dependencies/+merge/137702 commented but not marked as needs fixing as nothing here is really a deal-breaker18:21
roadmrzyga: let's see18:21
roadmrzyga: thanks! I'll fix all that stuff :)18:22
zygaroadmr: don't feel required to test everything I comment on18:29
zygaroadmr: I want to keep fluid coding possible18:29
zygaroadmr: only when you also agree that something is better otherwise18:29
roadmrzyga: the defaultdict suggestion looks good to me, what I'm doing is indeed kludgy18:30
zygaroadmr: looking at it just now, typically I add a space after # but PEP8 won't complain if you don't do that18:30
roadmrzyga: I didn't know += was that much slower, definitely good to change to join18:30
zygaroadmr: it's great to see the comments BTW, they make understanding the 'why' part much easier18:30
roadmrzyga: as for the formatting thing, I think it was for it to fit in 80-columns (otherwise pep8 tears me a new one) but I'll try to prettify it, and thanks for catching the lack of i18n18:31
zygaroadmr: yeah, += in a for loop is quadractic while the same "" join is not, basically python does not overallocate string buffers since strings are immutable18:31
zygaroadmr: yeah, I assumed as much, maybe you can put it ine a separate variable so that it's got more space18:32
roadmrzyga: oh that sounds good, that way I'm not constrained by the surrounding indentation18:32
=== Ursinha_ is now known as Ursinha
letozaf_Hi guys!19:54
Amacidiahey leozaf20:04
AmacidiaI'm new around here, submitted my first test result last night actually20:05
dkesselgood evening20:06
letozaf_hello all20:09
letozaf_Amacidia, welcome, well I'm not a "guru" in testing but I can try to help you :)20:10
letozaf_dkessel, did you try autopilot?20:11
dkesselyup i did. i started with the file-roller test conversion. i have converted the first steps, but then i got stuck.20:11
dkesselnow i know i used the wrong syntax :) don't use a '-' in a test file name :)20:12
letozaf_:)20:12
letozaf_so what do you think about it ?20:13
dkesseli think it is nice. but it need some polishing and extensions to be more useful.20:14
dkesselbut they are working on it i heard20:16
letozaf_that's great news20:21
=== jackson_ is now known as Noskcaj
Noskcajmorning everyone20:25
letozaf_good morning (well in Italy it's night  :D  )20:26
balloonshello20:26
letozaf_hello balloons20:27
balloonshello, so let's see if thomi is about and see if he made any progress on the autopilot stuff we asked about :-)20:37
thomiHi20:38
thomiI'm about20:38
dkesselhehe :)20:38
balloonsHappy Tuesday!20:38
thomiThanks :)20:38
letozaf_:D20:38
thomiwe just decided a few minutes ago to add a new feature to ap, which should help you guys a lot. We'll add a new command, probabvly something like this:20:38
thomi'autopilot launch gedit' -> will launch gedit with introspection enabled, so you can use 'autopilot vis' to look at it's internals.20:39
dkessel=)20:39
Noskcajballoons knows its Tuesday, yay20:39
thomiregarding the gedit introspection, there was an issue with the autopilot-gtk package in raring, which I'm told has now been fixed20:39
balloonsexcellent.. so have you been able to get a successful example?20:39
thomiso I should be able to push that branch now...20:40
balloons:-)20:40
thomialesage: do you have your gedit example handy?20:40
balloonsNoskcaj, lol.. yes, Tuesday and summer20:40
balloonsI can be taught20:40
Noskcaj:)20:40
alesagehi thomi, yes you can find it at lp:~allanlesage/+junk/UDS_AP_session20:40
=== salem_ is now known as _salem
zygaroadmr: as for UEFI, I'll make a small efi image for netboot testing tomorrow, do you think we could try spending a few hours on pair programming?20:42
roadmrzyga: sure!20:43
roadmrzyga: I didn't get anything uefi-related done today :/ so tomorrow we can certainly work for that20:43
zygaroadmr: have a look at that email I've sent on tuesday last week please20:44
zygaroadmr: the one with a few links to the wiki on how to get this up20:44
zygaroadmr: I'd like to try that on a real machine20:44
roadmruefi + netboot notes20:44
zygaroadmr: ok, till tomorrow then :)20:44
zygaroadmr: if you can look at the rather big MP, feel free: https://code.launchpad.net/~zkrynicki/checkbox/zyga/+merge/14216420:45
zygaroadmr: no need to review it entirely, just post your notes20:45
zygaroadmr: most of the diff is code motion20:45
thomiballoons: this is what you want: http://bazaar.launchpad.net/~allanlesage/+junk/UDS_AP_session/view/head:/tests/test_gedit.py20:45
roadmrzyga: excellent, I'll have a look20:45
thomiballoons: the two key things are: 1 - derive from GtkIntrospectionTestMixin and 2 - use self.launch_test_application(...)20:46
roadmrzyga: for pxe netboot, we usually extract the iso and then patch the initrd on the fly. As I understand it, for UEFI we'd have to then reassemble the ISO with the initrd hacks, right?20:51
balloonshmm20:51
balloonsdoesn't seem to blow up20:51
roadmrzyga: just some preliminary asking around, I'd have to review the initrd patches we apply, they may be unnecessary if a full iso (rather than nfs) is used20:52
zygaroadmr: I honestly don't know20:53
zygaroadmr: the way we boot today, we netboot the old-school installer, right20:53
roadmrzyga: yep20:54
zygaroadmr: but I really don't even know how that works, I understand how we pxe boot the kernel but what is after that is a mystery to me really20:54
roadmrzyga: we also need a way to load our custom preseed20:54
zygaroadmr: I could try to mimick that locally but raring still has no driver for my new laptop's eth20:54
roadmrzyga: actually feel free to ignore me, I may be talking out of my ass since I haven't read the uefi page fully20:55
zygaroadmr: preseeds are a part of the installer but how do we get there is something I don't yet understand20:55
zygaroadmr: I think you are right in describing what works now20:55
zygaroadmr: how efi affects that is unsure20:55
zygaroadmr: from what I read efi netbooting is very primitive with free tools so far20:55
roadmrzyga: I imagine something like building a custom iso with our preseed, but an alternative would be passing a kernel parameter, which may be easier than doing the meccano thing with isos20:55
balloonscool stuff20:55
zygaroadmr: and grub 2 has none of it20:55
zygaroadmr: so grub somehow gets mixed with a mini iso20:56
balloonsok, so things are working ;-) now just have to talk about using autopilot vis and looking at your dbus session20:56
balloonsdkessel, you still about?20:56
zygaroadmr: so that a few stacks of tools get to a point where you basically do what you did in bios (after booting)20:56
dkesselyes balloons20:56
roadmrzyga: well last time I tried any form of uefi netbooting, I didn't get past the mini iso thing (felt too kludgy back then)20:56
zygaroadmr: native-aware boot would be better but is not implemented by grub, efilinux (IIRC) has some code for that but it's still far cry from working20:57
roadmrzyga: since it now seems to be the recommended way, I guess we will have to do it that way :)20:57
zygaroadmr: we should ping cjwatson to know if there's some movement there20:57
balloonsso, want to try doing some introspection on fileroller?20:57
balloonsyou needed it for something yes>20:57
zygaroadmr: yeah, unless there's a more "native" solution that's what was recommended last time I've checked20:57
roadmrzyga: yep. It's not even that late so I may have time to do a bit of experimentation today20:57
zygaroadmr: ok, I'll take a break now20:57
dkesseli wanted to manipulate a combobox, yes20:57
zygaroadmr: I'll talk to you tomorrow :)20:58
roadmrzyga: enjoy, we can continue this tomorrow20:58
zygaroadmr: have a good evening :)20:58
dkesselis a new autopilot version up?20:58
roadmrzyga: enjoy your rosca :)20:58
balloonsalesage, the fix is also in the ppa right?21:00
balloonsIf I remember your not on raring dkessel21:00
alesageballoons, correct--please ping if you find different :)21:00
dkesselballoons, i have setup a quantal vm21:00
dkesselalesage, autopilot-gtk_0.1-0ubuntu0 is the old version, right?21:03
balloonsapt-cache show libautopilot-gtk021:04
alesagedkessel, yes--21:04
balloonsI have Version: 0.4-0ubuntu1+bzr14pkg0raring121:04
alesagedkessel, install libautopilot-gtk-dev21:04
dkesselgot that21:04
alesageand actually dkessel you may want to remove autopilot-gtk_0.1bla21:04
alesageas it's from before a renaming21:05
dkesseloh ok21:05
dkesselbug: autopilot is missing --version ;)21:07
dkesselok got the same version as balloons now21:07
balloons:-)21:09
dkesselnow how do i run file-roller with introspection?21:11
balloonsok, so autopilot has this tool called autopilot vis21:12
dkesselyeah, i tried that21:12
thomidkessel: very soon you'll be able to do 'autopilot launch file-roller'21:12
balloonsyou can try running it now.. by default, you'll see unity as a connection21:12
thomidkessel: right now you need this:21:13
thomialesage: I forgot what the env variable is... remind us please?21:13
balloonsthomi types faster than me ;-)21:14
alesagethomi GTK_MODULES=autopilot-gtk:$GTK_MODULES file-roller, I believe21:14
alesagedkessel ^^21:14
* dkessel tries21:14
dkesselalesage, aaah :) and there is a new "Root" connection21:15
alesagedkessel, and a pile of GTK+ widgets underneath :)21:16
thomidkessel: yeah, that's the one21:16
balloonsall sorts of stuff21:16
* dkessel spends the next day poking around in gtk stuff :)21:16
thomidkessel: please let us know if you find problems21:16
* thomi writes 'autopilot launch' now21:17
thomishouldn't take too long21:17
dkesselhmm - hard to identify a program. both name and title of the gtkwindows are empty ?21:18
thomidkessel: Gtk seems to create a lot of garbage in it's widget tree21:19
thomidkessel: the bits you're looking for will be in there somewhere21:19
alesagedkessel, it takes some digging21:19
dkesselok21:19
thomibut it's a lot less clean that a Qt app for sure21:19
thomithe standard practise is to write a class called an 'emulator' that retrieves the useful bits from the introspection tree for your tests21:20
balloonsyou have some unity emulators in this way21:21
balloonsbut it would be good to get some gtk examples.. so we can work on that know21:21
balloonsthomi, actually does it make sense to have emuators for us?21:22
balloonswould we be able to avoid writing one for each app?21:22
thomiballoons: it makes sense to have functions that get commonly used components.... so if the component chsanges it's position in the introspection tree you don't need to update all your tests21:23
thomi'emulator' is a terrible term for non-unity tests I realise21:23
thomiso yeah, I think it makes sense21:24
dkesselok, it is a start. i have the dialog and i have identified some of the components in it. however, for the test, i guess the "id" attribute values are generated at runtime? so i would have to guess and write some code like "get the first combobox in this dialog and select entry xyz in there"21:33
thomidkessel: alesage: the way to do this is to find something unique to that widget. If the app is well written, you can say "select the ComboBox with the name 'someUniqueName'"21:37
balloonsdkessel, Im also having a look around21:38
thomiif that's not possible, you may need to get a bit more creative - select the dialog box, then iterate over it's children or something similar21:38
thomipro tip: autopilot's select_single, select_many and get_children_by_type are all very powerful - you can specify several filters...21:38
dkesselthomi, balloons : i guess the real thing would be submitting patches upstream to get some real names in there...21:38
thomifor example:21:38
thomicombo_box_widget = app.select_single('ComboBox', name='foobar', someOtherProperty=123)21:39
alesagedkessel is this bug relevant? https://bugs.launchpad.net/autopilot-gtk/+bug/108239121:39
thomithe first parameter is the type name. The other parameters are filters to apply to the selection21:39
ubot5Launchpad bug 1082391 in autopilot-gtk "Needs proper widget identifiers" [Undecided,New]21:39
thomiI guess having the widget ids visible would certainly help21:40
thomialesage: did ted come back to you with anything useful WRT that?21:41
alesagethomi yes I got a brief intro, we'd have to decide how to expose 'em21:41
dkesselyes, alesage - i guess that "affects me" ;)21:41
thomialesage: easy to do?21:42
alesagethomi unknown, I'll have to review :/21:42
alesagethomi will study for AWG and discuss then?21:43
balloonsa search would be helpful :-)21:44
thomiballoons: in the vis tool?21:45
balloonsyea, that would work around the annoying widget spam21:46
balloonsin the interim21:46
dkesselhm autopilot -h says: "Opens visualisation in xdot when not passed an output path to write file to. is it still possible to dump the entire tree to a file?21:47
dkesseloops... missing " after "to."21:47
thomidkessel: oops, that's a bug21:48
thomidkessel: we haven't used xdot for a while, I'll clean up the help string21:48
dkesselok :)21:49
dkesseli don't like dot files anyway21:49
thomigraphvis can't handle trees of that size anyway21:49
thomi...which is why we stopped using it21:49
dkesselyup - it crashed when i handled it a graph of our producation software once too....21:50
dkesselthanks for your work so far guys!21:51
dkessel...and i'm off22:16
balloonsbah.. I cannot find the popup in the tree ;-(22:58
balloonsany thoughts on finding a pop-up window.. in this case, the gedit prefs window?22:59
balloonsI'm guessing it's a child somewhere.. hmm23:01
balloonssadface.. I simply can't find it..23:13
alesageballoons, yeah this can be quite frustrating :(23:16
balloonsI thought I had it.. maybe spawning under the gtkmenu tree.. but nope23:16
alesageballoons beware having to 'refresh' the autopilot vis tree--thomi sez it's possible to do so by clicking on a root node, then the child nodes are refreshed23:19
alesageballoons, if you're looking at a modal dialog then it's probably all the way up the tree23:20
balloonsalesage, ok.. not sure what you mean23:20
thomithat's another feature we need. balloons if you file a bug I'll try and get around to doing it...23:20
balloonsI have to click the top level root node perhaps23:20
balloonsthomi, a search you mean? It would make this much easier23:20
balloonsand I could ignore the nonsense spawning23:20
thomiballoons: both a search function, and a way to explicitly refresh the tree23:21
balloonsalesage, lol.. I'm not going to look through it again, but I'm noting it23:21
balloonsthomi, ok, let me file request23:21
balloonsI'll go through it again tomorrow23:22
balloonsI want to see if what I've got works without that23:22
balloonsAttributeError: 'BamfWindow' object has no attribute 'select_single'23:26
balloonsalright.. good night all.. good stuff, glad this is working now..23:27
balloonshave to parse through it a bit, and the enhancements your making will clean this up nicely23:27
balloonsthanks again23:27

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