=== eu is now known as Guest92369 === Guest92369 is now known as adorilson === eu is now known as Guest21731 === rodrigo__ is now known as rodrigo_ === rodrigo__ is now known as rodrigo_ [09:51] morning all! [09:51] dobey, on it! [09:52] how to move ubuntu one default folder to another partition in 11.10 [09:58] Goooooood moooooooornin' all! [10:00] how to move ubuntu one default folder to another partition in 11.10 [10:11] amandeep: what is it you're wanting to do? [10:12] i want to move it to a drive with lots of space [10:17] ??? [10:18] amandeep, currently we only support folders under your home directory [10:21] Thnx alot for the info.... [10:21] JamesTait, can you do me a favour and try something in your machine for me? [10:22] mandel: As long as it doesn't involve rm -rf anything, yes. ;) [10:22] JamesTait, I'm having some issues with squid and I wonder if it has to do with my machine setup or something :) [10:22] JamesTait, can you please install squid in your system? [10:23] JamesTait, Let me pass you a config file I have so that you can try it [10:23] mandel: I actually have squid installed on my other machine - that's a Lucid box though, is that any good? [10:23] JamesTait, sure, I just want to find out why the bloody things does not start when you pass -f [10:23] JamesTait, let me paste the config file I'm using [10:23] one sec [10:26] JamesTait, can you please use the following config http://paste.ubuntu.com/745758/ [10:31] mandel: I might have to create a couple of directories, bear with me. [10:33] JamesTait, yes, the temp/spool/squid and the temp/squi [10:33] temp/squid sorry [10:33] JamesTait, once you have those two you can do squid -z -f config_file [10:33] Exactly. :) [10:35] mandel: I get just a single line of output: 2011/11/22 10:34:24| Creating Swap Directories [10:35] mandel: Let me see if there's a useful log somewhere. [10:36] JamesTait, that is correct, it creates the cache dirs, after than try [10:36] squid -N -f config_file [10:36] JamesTait, actually add -X to that [10:36] so squid -N -f config_file -X [10:36] JamesTait, does it work? cause I have a stupid rename error :( [10:36] Aborted,(core dumped) [10:37] Wit lots of "2011/11/22 10:36:39| storeUfsDirGetNextFile: opendir: temp/spool/squid/0F/FF: (2) No such file or directory" [10:38] mandel: Just checking scrollback to see if there's anything more useful than that. [10:39] JamesTait, that is not a surprise since you have no cache (although the stupid file is present) [10:39] JamesTait, I get FATAL: storeUfsDirCloseTmpSwapLog: rename failed [10:39] which I do not understand, and if I run squid in the default location it works.. [10:40] Permissions? [10:41] "FATAL: storeUfsDirCloseTmpSwapLog: rename failed" you say? [10:41] I get that right after all those other entries. [10:43] JamesTait, so you ahve the same issue.. WTF [10:43] I hate squid [10:43] * mandel is not a sysadmin for a reason [10:46] mandel: http://paste.ubuntu.com/745780/ [10:48] JamesTait, yeah, I have the same.. let me look more into this.. [10:48] puto squid [10:49] Maybe get strace on the case. [10:51] Well that didn't help. rename("temp/spool/squid/swap.state.new", "temp/spool/squid/swap.state") = -1 ENOENT (No such file or directory) [10:51] Maybe it need absolute paths. [10:52] mandel: Absolute paths seems to have worked. [10:56] facundobatista: is it me, or did amandeep go off in a huff? [10:56] Chipaca, we are clearly not solving his/her problem [11:01] hi [11:04] Hola gatox [11:06] facundobatista, how are you this morning? [11:06] gatox, fine, you? [11:06] JamesTait, really? WTF [11:07] JamesTait, la madre que los pario.. well, I own you a big one! [11:07] facundobatista, fine..... trying to wake up :P [11:07] gatox, morning! [11:07] mandel, hello! [11:08] gatox, did one of your reviews, I have to look at the next one [11:08] mandel, great! thanks!....... trying to fix the bug yesterday i felt like coding an onion...... so many layersssss :P [11:10] gatox, we really really need to simplify that [11:10] mandel: You're most welcome. :) [11:10] mandel, i agree [11:12] JamesTait, well, now I know how to start the bloody proxy service then :) [11:15] mandel: My main gripe with squid-deb-proxy has been that squid isn't IPv6-capable - at least not the version in 10.04. [11:16] JamesTait, squid is just one of the versions of proxies we have to tests.. II guess I'll write a test case for an ISA :( [11:16] it hurst just to think about it :P [11:16] it could be worse [11:16] it could be an ISA *BUS* [11:17] Hey! I still have a firewall box with an ISA bus! [11:17] JamesTait: right! you're in charge of writing the proxy tests [11:17] Anyway... tea required. [11:18] mandel: j/k. it's still you, all the time. [11:18] Actually, if you're going to be like that, maybe something stronger. [11:18] :-P [11:18] haha [11:18] Chipaca, don't play with my feelings [11:19] * JamesTait needs to hit up some Windows developers. [11:19] Anyway. TEA! [11:19] I mean it this time! [11:19] mandel: ok :) [11:19] mandel: how're the ribs? [11:20] Chipaca, fine, I'm back to 100% :) [11:20] mandel: w00t :) [11:48] Chipaca, I'm trying to look at getting a empty port to be used by the squid proxy.. I'm thinking of using socket.bind(('', 0)).getsockname()[1] is there a better way? [11:48] Chipaca, I know there can be a race condition between getting the port and starting squid.. also if you pass 0 to squid it hates you :( [11:49] mandel: trunk-2a/utilities/utils.py/get_arbitrary_port [11:49] Chipaca, which project? [11:50] mandel: IOW, yes, except use 'localhost' instead of '' [11:50] because '' has issues with IPv6 [11:51] ok [12:04] morning all [12:05] duanedesign, hi! [12:07] how are you today gatox [12:08] duanedesign, really fine, finishing with a bug, you? [12:08] gatox: oh cool. Which bug if i may ask? [12:10] duanedesign, one about confirmation code when the user cancel the registration process and then try to login using the same email.... the hardest part is already done :D [12:10] ahhh [12:11] great! [12:11] :D [12:17] gatox: how do you fix that bug? [12:18] very carefully ;) [12:18] duanedesign, jejeje [12:19] karni, i'm talking about the client..... when the user execute the installer and choose to sign in with the email already registered, and the server answer not validated email, i change the actual page to verification code page [12:19] * duanedesign runs from karni [12:19] karni, and change the flow of the wizard to continues from there [12:20] gatox: ok. so we have the same bug in the wizard I just wrote for Android ;P thanks. [12:20] karni, jjejeje [12:20] it seems [12:26] gatox: so when user attempts to register with same username+password, and the registration returns "email already used", you use that username+password to make a Basic Auth authenitcated call to login.ubuntu.com/.../?ws.op=me just as if the user was already registered? [12:28] karni, no..... when the server response that the email is already used but not validated, i show the verification page in the wizard, and that page already has the implementation that collect the user email and password and the code that the user should enter in that page and then try to send a request to validate that email as usual (if the user had completed the registration process previously) [12:30] gatox: collect username and password from where? you guys store users password? [12:30] gatox: also, does the ws.op=register response contain information that email is used *but not validated* ? I have only seen SSO return "E-mail already used." [12:30] karni, no, in the current user sign in page, the user type her email and password in order to login..... and the verification code page, can ask the wizard the info that the password or email fields has [12:31] its the same application wizard..... different pages [12:31] gatox: ah, that is perfect [12:31] gatox: so you do use the same login and password, okey [12:31] yes [12:32] karni, the real problem was to intercept the not validated message and send a message to the installer to show the proper page..... the rest it's just reusing the current logic [12:34] gatox: which URL does give you a response for "not validated"? is it login.ubuntu.com/api/1.0/something?ws.op=me ? [12:34] karni, let me show you the code..... [12:34] * gatox looking... [12:35] gatox: Thanks [12:36] karni, look here: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntu-sso-client/trunk/view/head:/ubuntu_sso/main/__init__.py#L88 [12:36] * karni loks [12:36] * karni looks ;d [12:36] in the login you have a callback to success that check if the email is validated [12:36] Chipaca: http://www.h-online.com/open/news/item/Canonical-dropping-CouchDB-from-Ubuntu-One-1382809.html [12:38] karni, is that what you were looking for [12:38] ? [12:38] gatox: I'm looking for the code of processor.is_validated(credentials) [12:39] karni, do you know which object is that?? give me a sec, i'll tell you [12:39] gatox: I've no idea, it's line 103 form your link. thanks! [12:41] karni, ubuntu_sso.account.Account [12:41] gatox: thank you! [12:42] karni, let me know if you need something else..... i'm still with that issue [12:42] gatox: yes, it uses sso_service.accounts.me() :) thank you :) [12:42] :D === rodrigo__ is now known as rodrigo_ [13:00] good morning! [13:02] ralsina, hi! [13:02] mandel: ping [13:02] ralsina_, pong [13:03] mandel: I see nothing happened with the windows release? Is there a problem? [13:04] ralsina_, let me check, I have not look at the rt today [13:04] mandel: I have :-) [13:05] ralsina_, and we do not have the .exe yet? [13:05] * mandel looks [13:05] mandel: I uploaded the unsigned exe on friday [13:05] mandel: https://rt.admin.canonical.com/Ticket/Display.html?id=49209#txn-783031 [13:06] mandel: ok, seems like the 49210 RT is not done because the installer has not been signed and published [13:06] mandel: I'll re-take it [13:07] ralsina_, yes, that I know I was asking question yesterday about that [13:07] mandel: got any answers? [13:08] ralsina_, no, that he did not know and that there are the super-losas de ones that do the signing [13:09] ralsina_, I don't know why on Friday it was market as incomplete.. [13:09] and they changed from GSA to LOSA.. no idea why [13:10] it has to change to l-sa when it's time to put it on the web, looks like a miscommunication [13:10] it moved too quick and fell on the floor [13:11] mandel: fixed, it's top of their queue now [13:14] ralsina_, ok cool, ping me if they let you know and I'll open the other rt with the correc txml [13:14] mandel: ok [13:24] czajkowski: yep, saw that [13:24] czajkowski: thanks [13:25] Chipaca: wasn't me was my boss, but tis interesting read [13:26] * mandel hates squid sooooo much [13:27] mandel: fry it! [13:27] mandel: I know all about squid, so feel free to ask me [13:29] ralsina_, awesome, so why does this subprocess.Popen(['squid', '-f %s ' % os.path.abspath('./squid.conf')] fails telling me it cannot find squid.conf when it is there [13:29] mandel: probably it's not being executed where you think it is [13:30] ralsina, but is an abspath [13:30] mandel: yes, but it's abspath of "./squid.conf" which is the cwd [13:30] So if it's executing in the "wrong" place, the abspath is wrong too [13:32] ralsina, I'm doing that from the same dir as the .conf from a python RELP [13:32] print os.stat(os.path.abspath('./squid.conf')) [13:33] mandel: trust the computer :-) [13:33] ralsina_, I get the stat result no problems what so ever [13:34] Ok, the other problem is that you are using Popen wrong, but I need to check the docs ;-) [13:34] mandel: subprocess.Popen(['squid', '-f', os.path.abspath('./squid.conf')]) [13:35] ralsina, ah, thx! it was that.. what a day.. [13:35] ralsina, we do have an issue then in the dbus runner too [13:36] mandel: if it had all been a single string it would have worked too [13:36] czajkowski: i had no idea you were related :) [13:36] czajkowski: s/related/involved/ [13:36] ralsina_, momento estupido :P [13:37] mandel: what's the problem with the dbus runner? [13:39] czajkowski: it's been picked up all over the twitterverse [13:39] Chipaca: it's interesting to see future plans are underway [13:39] ralsina, code style things, no to worry atm [13:39] mandel: ack [13:40] Chipaca: interesting to see no comment from couchbase [13:41] czajkowski: is it? [13:41] czajkowski: (not being cute; seriously don't know) [13:42] * mandel lunch [13:47] Chipaca: see pm [13:47] * czajkowski steals mandel lunch and runs away [13:47] czajkowski: haven't got a pm [13:47] czajkowski: are you identified? [13:48] aye [14:14] mandel, ping [14:15] Hey alecu how are you dude :) [14:15] hey fagan! long time no see! [14:15] Yeah 3 months been moderately busy [14:16] fagan, what have you been working on? [14:16] alecu: Well was working to find a job first now working on an ipad game with a few friends [14:17] alecu: was at a small company for a while but it didnt really suit [14:17] fagan, ipad game! cool [14:17] alecu: yeah we have some people from cambridge doing the art for it. [14:19] alecu, ping [14:19] gatox, pong [14:19] gatox, I owe you a review, right? [14:19] alecu, how are you this lovely morning? (gatox prepares alecu for a review) [14:19] jejejejejee [14:19] gatox, no te conviene.... [14:20] gatox, I've got 9 branches for review :-) [14:20] alecu, wowww [14:21] alecu: thats just showing off [14:21] fagan, hahahha [14:21] alecu, is it my branch for the other day included in that 9? [14:21] * fagan thinks alecu is splitting things into 1 line branches :D [14:21] gatox, no! I've got 9 branches of mine that need review! [14:22] alecu, ahhhhhhhhhh [14:22] gatox, they are all the backports of the same bug that touches lots of projects: [14:22] gatox, take a look here: https://bugs.launchpad.net/ubuntuone-client/stable-2-0/+bug/692597 [14:22] Launchpad bug 692597 in ubuntuone-storage-protocol (Ubuntu) (and 22 other projects) "If the user has the date/time/timezone wrong, connection fails (Expired timestamp error) (affects: 18) (dups: 6) (heat: 142)" [Undecided,New] [14:22] gatox, and look at "Related branches" [14:22] * gatox taking a look :P [14:23] alecu, nnaaaaaaaaaaaaaa [14:23] alecu, ........ so........ not a lovely morniing for you........ [14:23] jejejej [14:24] gatox, I've been working on this on and off for the last two weeks... and on friday I was finishing the u1-client branches [14:24] gatox, today I've got only the control panel branches left [14:24] gatox, but now I need to start chasing reviewers... :-) [14:25] gatox, luckily alll the branches are very similar to branches that are already merged into trunk [14:25] alecu, well.... let mme know if i can help you with a review [14:25] gatox, let's trade! [14:26] gatox, so, I still owe you a branch... do you have any other more branches to review? [14:26] (besides that one) [14:27] alecu, the most important one is the one about unicode...... the other one is really trivial and already has mandel approval.... i prefer that you review this one (that you already have been reviewing): https://code.launchpad.net/~diegosarmentero/ubuntuone-client/unicode-issues [14:28] gatox, I'm starting with that right now [14:28] alecu, gatox: I can do a coupld oe reviews today [14:28] couple of reviews [14:29] ralsina_, choose the one you like from this tiny list: https://bugs.launchpad.net/ubuntuone-client/trunk/+bug/692597 [14:29] Launchpad bug 692597 in ubuntuone-storage-protocol (Ubuntu) (and 22 other projects) "If the user has the date/time/timezone wrong, connection fails (Expired timestamp error) (affects: 18) (dups: 6) (heat: 142)" [Undecided,New] [14:29] ralsina, great.... i have a trivial one.... and i'm finishing another one that nessita left me about: trying to log in when an email not validated [14:29] ralsina, let me give you the trivial one that it's ready [14:29] I have the mgmt call in 45 seconds, but I can do reviews in a couple of hours [14:29] ralsina, no problem [14:29] ralsina, https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/congratulations-page/+merge/82596 [14:30] ralsina, the other one is going to be ready in half an hour.... so i'll give you the link later [14:35] gatox, let me know when you finish with that and start reviewing, so I'll guide you on reviewing one of my branches. [14:36] alecu, ok..... i'll let you know as soon as i finish with this..... only a couple of tests more, and done [14:36] great! [14:43] * mandel back [14:43] alecu, pong [14:44] hi mandel! [14:44] alecu, hola! [14:44] mandel, two things: an important one, and a not so important one :-) [14:44] alecu, how was the holiday, short, right? [14:44] alecu, shoot [14:45] alecu, but before you say it, I did not know she was your sister :P [14:45] hehehe [14:45] mandel, fine! I just asked for the day because my wife had a small holiday too. [14:45] mandel, hahaha [14:45] mandel, the important one: I neeeeeeed you to get me one of the galactica expansions! [14:45] mandel, I'll pay it, with blood if necessary. [14:45] alecu, ok, email me the one amd I'll find it for you :) [14:45] mandel, cool! [14:46] mandel, the other issue: how's the "squid integration" coming? had any issues? [14:47] alecu, I had a small issue starting squid, mainly me being stupid, so I should have a tests case for today that starts squid and uses it, next would be to extend that testcase so that we can add more ocnfig things [14:47] alecu, at the moment starts squid at a random port and that is all, I need to ensure auth config etc [14:47] mandel, cool [14:48] mandel, perhaps the squid conf could make some urls require an auth and some other urls not require auth [14:48] mandel, that way the tests can call a method from the testcase and get the first url or the other [14:49] alecu, hm.. good idea, is a matter of finding the correct config for that and how to doit nicely more than anything else [14:49] alecu, also ading the tests for the test case :P [14:49] mandel: I can do the config for that no problem [14:49] mandel, right. In any case, remember that we have people with *lots* of squid experience in canonical [14:49] mandel, like ralsina_ :-) [14:49] hehe [14:50] ralsina_, as soon as I have the test case we can take a look at it, since I'm squid agnostic :) [14:50] mandel, or life-less, that was (or is) a squid maintainer. [14:54] alecu, I'll keep you updated, once is there we can start writing some tests :) [15:00] me [15:01] ralsina, alecu, gatox , dobey ? [15:01] me [15:01] mandel: on call, DONE: was sick. TODO: not be sick. BLOCKED: no [15:01] heh [15:01] jeje [15:01] me [15:01] me [15:01] lets go then [15:01] DONE: Progrees on the proxy test case. Got squid running at a random for for the test case. I need to find a nice api for the auth settings. [15:01] TODO: More squid test case, specially make it work on windows. gatox review about unicode. [15:01] BLOCKED:no [15:01] gatox, please [15:02] DONE: [15:02] Fixed login with not validated account [15:02] TODO: [15:02] Test for this branch :P, fix more bugs from my bug queue [15:02] BLOCKED: [15:02] No [15:02] dobey, go [15:02] λ DONE: fixed u1client SRU, bug #882732 (SRU proposal), reviewed mandel's devtools branch [15:02] Launchpad bug 882732 in banshee (Ubuntu Precise) (and 3 other projects) "Race Condition setting active source at startup (affects: 5) (dups: 1) (heat: 40)" [High,In progress] https://launchpad.net/bugs/882732 [15:02] λ TODO: move some stuff around, review some code, make some awesome [15:02] alecu: vamos [15:02] λ BLCK: No. [15:02] DONE: backporting u1-client branches for bug #692597; day off [15:02] TODO: reviews, 1-1, backport u1cp branches [15:02] BLOCKED: no [15:02] Launchpad bug 692597 in ubuntuone-storage-protocol (Ubuntu) (and 22 other projects) "If the user has the date/time/timezone wrong, connection fails (Expired timestamp error) (affects: 18) (dups: 6) (heat: 142)" [Undecided,New] https://launchpad.net/bugs/692597 [15:07] ok, I need to go for 30 min, dog needs to walk [15:29] alecu: can you post a status update in bug #692597 please? [15:29] Launchpad bug 692597 in ubuntuone-storage-protocol (Ubuntu) (and 22 other projects) "If the user has the date/time/timezone wrong, connection fails (Expired timestamp error) (affects: 18) (dups: 6) (heat: 142)" [Undecided,New] https://launchpad.net/bugs/692597 [15:30] alecu: post it in the bug, I mean :-) [15:31] ralsina_, sure [15:31] alecu: thanks [15:34] te [15:34] sorry [16:03] * mandel back [16:09] * gatox lunch! brb === gatox is now known as gatox_lunch [16:29] lunch bbiab === ralsina_ is now known as ralsina [17:01] alecu, ping === beuno is now known as beuno-lunch [17:03] mandel, pong [17:04] gatox_lunch, in expand_user(), in this line: "result = path.replace('~', xdg_home, 1)" [17:04] alecu, on question, how are we getting the proxy settings from the system? [17:04] gatox_lunch, what happens if path contains the "~" character twice? [17:05] alecu, specially taking into account the two approaches [17:05] mandel, we are not getting them *yet*, but the plan is: [17:05] * on windows or mac, qt gets the settings [17:05] * on ubuntu, we get them from gsettings. [17:05] mandel, oh, you mean for the integration tests? [17:06] alecu, yes, the SquidTestCase needs to patch the methods used for that === gatox_lunch is now known as gatox [17:06] mandel, for the integration tests, the tests themselves will choose if they need to use: a) no proxy, b) anon proxy, c) authed proxy. [17:06] alecu, only replace the first one [17:07] gatox, oh, right! I missed the "1" as the last parameter! [17:07] gatox, dumb me! [17:07] gatox, sorry :-) [17:07] alecu, no problem! [17:08] alecu, hm.. then in the setUp of the tests you can say, get_aut_proxy_settings for example, otherwise we don't match very well with the current services implementation in ubuntuone-dev-tools [17:09] alecu, the DBustTestCase patches the methods that get the Session which makes sense [17:09] right [17:09] mandel, but for proxies it looks different to me [17:10] mandel, my guess is that we should choose to use proxies or not from each individual test [17:10] alecu, we would like to be able to get a test case to be ran in the three diff scenarios, right? [17:10] alecu, or just a subset? [17:11] mandel, that can be an option too: have a test be repeated in each of the three scenarios [17:11] alecu, that is the one I was thinking to be the most common one.. [17:11] alecu, why don't you write a test with what you would imaging using, I'll do the same and lets see where we get [17:11] mandel, but anyway we want to separate the "discovery of the configuration" from the "using the proxy" into different sets of tests. [17:12] mandel, so, these tests you are working on are about "using the proxy" [17:12] mandel, and I would not want to mix the "discovery of config" into that. [17:13] mandel, yup, it makes sense. I'll try writing some tests in half an hour or so. [17:13] mandel, if it's past your EOD, I'll mail it [17:13] alecu, it probably is, I'll do the same know.. [17:13] s/know/now [17:16] ralsina, gatox, I'd like your input too ^ [17:17] * ralsina reads [17:17] ralsina, gatox, it would be nice to have a decent api for that [17:17] mandel, i'm lost.... about what? [17:17] mandel, from alecu ping? [17:17] +1 that config discovery is a whole different thing [17:17] :P [17:18] mandel, I think I got disconnected when you pasted *something* [17:18] mandel, because I don't understand what you'd like the input on [17:21] mandel, alecu +1 to separate those things.... i would be easier to control which one may be causing problems in some point and understand the code [17:22] alecu, on how you would like the tests to be written [17:22] alecu, ralsina, gatox for example: http://paste.ubuntu.com/746122/ [17:23] not that any of those are good ideas [17:24] I'm more interested in how you want to write the tests than the implementation... I can think about that later [17:30] mandel, http://paste.ubuntu.com/746137/ [17:31] mandel, I think that way is simpler [17:31] mandel, the tests would only be on the first class; the second and third class would only have a smallish setUp method [17:34] alecu, looks nicer, now, how do they get the proxy seetings, like port number etc.. just set them in a self.proxy_settings? [17:34] mandel, and setupAuthProxy and setupAnonProxy could be used directly from other tests methods in other testcases [17:34] alecu, hm, if you allow that you will have to be careful with the tearDown and cleanUps [17:35] mandel, our webclient library will probably use some function to get the settings. Those two methods (setupAuthProxy and setupAnonProxy) should just "self.patch" that function. [17:35] alecu, aha, that is what I was think, but ofcourse we don't know that yet [17:37] alecu, I need to think how well those that map with the current design of ubuntuone-dev-tools since services are ALWAYS started by the test runner [17:38] dobey, ping [17:38] mandel, that's great. We want one instance of squid running all the time. [17:38] mandel: hi [17:39] mandel, it's the responsibility of the twin functions above to let the webclient know whether to use the squid or not. [17:39] dobey, I have a question, at the moment the DBusRunner is always ran by the TestRunner, is there a reason for that, rather than doing it in the setUp of the test case? [17:40] dobey, I've been talking with alecu about the API that the SquiTestCase should have and he came with a nice idea: http://paste.ubuntu.com/746137/ [17:40] mandel: it wasn't working when i did it in the setUp/tearDown; and spawning killing a service for every test is also quite expensive [17:41] mm [17:41] mandel: yes; you really do not want to spawn a new squid for every single test [17:42] alecu, dobey is right, is going to be very expensive to do that [17:42] mandel: i'm not sure what the best solution is for that, but this also goes back to the discussion we had yesterday [17:42] dobey, I had uglier ideas like: http://paste.ubuntu.com/746122/ [17:42] i don't think we want to be using squid in unit tests [17:42] mandel, I know. And I'm not proposing to start/stop the squid on each test. [17:43] alecu, ok, so we start the service and those methods only patch the x method that gets the proxy settings, that makes sense [17:43] mandel, I'm proposing the webclient library using it or not on each test. [17:43] mandel, exactly. [17:44] alecu, I misunderstood you [17:45] mandel, I agree with dobey that we probably don't want this to be run during unit tests [17:45] mandel, so probably these tests should be run separately, with a different script [17:46] alecu, that is not a big problem, we can later make u1trial ignore them or not in a nice way [17:47] maybe devtools is the wrong place to even have this [17:48] dobey, ideally it would be nice to have eveything in a central place for all the projects to be tested with proxy.. so where do you think it should be moved? [17:50] mandel: a central place for it to exist makes sense; but i am not sure it should be done via the u1trial services magic. mostly becuase i wouldn't want to suggest other people to use it, and having it there does the exact opposite of that :) [17:51] wouldn't want to suggest that it's sane for people to have that in their unit tests, that is [17:52] dobey, yeah.. but it is so easy to do it there, that I think we are already doing it ;) [17:52] dobey, the team chat is tom, right? [17:52] mandel: so maybe in devtools is an ok place to have such a thing, but probably via some other means. perhaps a script or set of tools, for doing integration testing with larger instance system services like squid/apache/mysql/etc [17:52] ralsina, when you have a moment..... review please: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/not-validated-account/+merge/83054 - https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/not-validated-account/+merge/83055 [17:52] mandel: thursday; but i won't be on it :) [17:53] dobey, dammed.. [17:54] dobey, so.. I'll read a little of how to do proper integration tests with python, then with all the info I'll see if I have a better idea of how to integrate with devtools [17:54] mandel: actually; probably it belongs in ubuntuone-testing, which is where the tests stuff lives that gets run in jenkins [17:54]