[00:28] balloons: still around? === TheLordOfTime is now known as teward [04:13] hello? [08:05] good morning jibel and elfy [08:06] hi DanChapman [08:06] Hey DanChapman [08:06] morning elfy [08:06] hi jibel :) [09:22] jibel what would be the best way to apend additional repo's to a sources.list file say to enable universe. I have tried echo "universe" | sudo tee -a /etc/apt/sources.list but that appends to a new line which i don't want. How do i append to the end of the first line? [09:28] jibel never mind, sed -e 's/$/ universe/' does the trick [10:00] DanChapman, that or sudo add-apt-repository universe [10:59] Morning all [11:38] jibel, what would the best way to get the ubiquity runner to work from a local directory. Would it be zipping it up and pack it in when preparing initrd then just unpack & move when run-autopilot kicks off on the other side? [11:40] * DanChapman is getting bored of having to push EVERYTHING! [11:40] DanChapman, this is a solution, another is to use the vvfat driver of qemu, but that might interfere with the installer because it'll see an additional drive [11:40] DanChapman, let me try something [11:41] jibel, awesome thanks :-) [13:30] DanChapman, I tried with vvfat but it makes the test fail because it is presented as sda and the partitioner takes the 1rst volume available. I'll try something else. [13:31] DanChapman, however it is interesting to note that ubiquity doesn't complain if you install ubuntu on the 512MB volume :) [13:33] jibel, did it get past the 'preparing to install' page it usually should stop there with the next button becoming inactive [13:34] jibel, well it did when i trialed it a while back. Should really get a testcase for that [13:34] DanChapman, it does because there is a second disk with the required amount of space [13:35] jibel, thats interesting :-) [13:35] DanChapman, I'll try 9p instead of fat, just to see how the partitioner presents it [13:43] DanChapman, it works with virtfs and it is invisible to partman [13:59] jibel, brilliant :-) thanks for taking the time to figure it out. Will be so much nicer working with it [14:30] davmor2: the tests we are missing on the jenkins jobs are: online accounts ui, ubuntu system settings, ubuntu terminal, ubuntu weather. [14:31] elopio: thanks I'll have a look if I can ever re-trace these crashes [14:33] balloons, jibel, quick question are packages from the partner repository allowed for test dependencies on jenkins? disc0tech is working on some rhythmbox tests and wants to mock a radio station but currently depends on gstreamer0.10-fluendo-plugins-mp3-partner [14:36] elopio: terminal will possibly be down to the fact that the backend needs to be rebuilt against 5.2.1 it only shows the shell and nothing else, Weather app only shows the loading page and goes no further too so I will look if those are fixed now but I doubt they are [14:42] DanChapman, you can install any package from archive.ubuntu.com [14:42] jibel, great thanks :-) [14:47] DanChapman, http://paste.ubuntu.com/7033346/ should save you some bzr pushes :) [14:47] and comments too [14:48] jibel, excellent! thank you so much :-D [14:48] DanChapman, I'm doing few more tests to make sure I didn't break too many things and will push this [15:10] DanChapman, added option -D to r6127, ie run: ./ubiquity-autopilot-runner/run-ubiquity-test -D --sdl ~/iso/ubuntu/trusty-desktop-amd64.iso [15:15] DanChapman, I assume we have a bit more to do then on https://code.launchpad.net/~dpniel/ubuntu-autopilot-tests/test-runner/+merge/206581? [15:38] jibel thats brilliant thanks :-) [15:40] balloons, howdy :-) yes it will also come in handy for that aswell. I'll look into it once i have got rid of this pesky bug in the custom install tests. [16:35] balloons,DanChapman : helloo...is this code enough for reporting the bug http://paste.ubuntu.com/7033834/ [16:36] senan, I'm not sure exactly what the problem was you were encountering. But in general, the smallest amount of code that will show the issue is the way to go :-) [16:38] balloons, its some object identification issue Scan Folder... is not detecting in autopilot3 [16:39] balloons, I'm also not sure what description I should give. :( [16:40] DanChapman, need your help ;) [16:48] senan you can simplify that testcase to just simply selecting the menuitem there isn't any need to go clicking round the UI :-) [16:53] DanChapman, like this http://paste.ubuntu.com/7033915/ [16:54] DanChapman, I'm not sure what description I should give [16:54] pls help me on that too [17:03] senan it doesn't need the test_open_remote_folder the issue is with the '...' / '\u2026' not being recognised/selectable in autopilot3. Also it could do with an assert just to check it's selected [17:04] senan, just describe what the problem is and if you give me the bug number after i'll add any details that are missing [17:27] elopio, is there a good time to have a look at weather today? [17:27] when the sun is up? [17:27] balloons, weathers pretty grey here ;-p [17:27] oh dear [17:27] English humour strikes twice :p [17:28] * balloons shakes head slowly [17:28] :-D [17:28] believe it or not, it's grey here too! [17:28] I bet our grey is greyer :p [17:29] actually I had sun - but shhh [17:29] ohh it surely is. But it's an odd day for us [17:41] balloons, i'm reviewing the rhythmbox test atm. Could i get your thoughts on the utility function naming style? [17:42] DanChapman, quick link to mp? I was just looking at https://code.launchpad.net/~adam-disc0tech/ubuntu-autopilot-tests/fileroller/+merge/208183 [17:43] https://code.launchpad.net/~adam-disc0tech/ubuntu-autopilot-tests/rhythmbox [17:43] balloons, ^^ [17:43] the majority of them are in base.py [17:44] DanChapman, I prefer _ as leading char and I like camelCase [17:45] DanChapman, I believe pep8 suggests function naming to be that way. However, I think most tests stick to _ anyway in the autopilot world [17:46] in regards to this mp, it feels a bit weird, since it feels like he's mixing styles [17:47] balloons, ok cool and what about the given/when/then flow of the tests? [17:47] ohh hehe [17:47] well, I think we spoke a bit about how we'd like to see things; so that would apply here as well [17:48] namely, the object class model idea [17:48] testsuite run ok? [17:49] DanChapman, I think I can appreciate his desire to make tests more readable, but I would rather see it done a different way. I would support renaming and dropping the given, when, and then from the function names [17:50] comments and logger debug statements can fill in instead. [17:50] balloons, yes i agree. Ok well I will I have made a start on the eog test to see how it goes today which i can then show him as a reference. It ran ok but the combobox selection is a bit flaky though (failed a couple of times)so that needs toughening up [17:50] is disc0tech here atm? [17:52] DanChapman, are we comfortable merging with the changes we mentioned above? Again, I think I would pursue the refactor as a separate mp and land this [17:56] balloons, yes I agree I will make the recommendation I have for the combobox but apart from that it works good locally and in the runner [17:56] DanChapman, I'll add a quick note as well. It looks like a nice testcase for a big app.. Good stuff ;-) [17:57] balloons: today is not good for me. I'm doing slides for tomorrow. [17:57] I had it planned for tomorrow, but if you can start earlier, even better. [17:58] balloons, yes it's comae along nicely. He's actually working on improving the radio test case by mocking a radio station so it has no external sources which is cool :-) [18:00] elopio, ahh.. yes do prepare for your talk.. no worries [18:07] DanChapman, balloons : Bug #1287834 [18:07] bug 1287834 in autopilot-gtk "GTK elements with property contains '...' / '\u2026' not being recognised/selectable in autopilot3" [Undecided,New] https://launchpad.net/bugs/1287834 [18:13] DanChapman, balloons : I've added bug number to the test and resubmitted the MP..Dan can you please take a look at the bug number and please add if I missed something === bfiller is now known as bfiller_afk === WebbyIT is now known as rpadovani === bfiller_afk is now known as bfiller === veebers_ is now known as veebers === bfiller is now known as bfiller_afk