=== yofel_ is now known as yofel [09:32] Good morning, all! :D [09:39] morning all! [11:09] good morning [11:11] gatox, morning! [11:16] gatox, can you do a super stupid review for me? [11:18] mandel, of course! [11:18] shoot [11:20] gatox, https://code.launchpad.net/~mandel/ubuntuone-dev-tools/give-me-batch/+merge/88342 [11:20] gatox, I was tired of having to write python C:\Pythoon17\Scripts\u1trial on windows, so I did that :) [11:21] gatox, if you add C:\Python27\Scripts to the search path in cmd u1trial and u1lint will work :) [11:22] mandel, nicee [11:26] gatox, I don't know why i did not do it earlier :) [11:26] :P [11:27] gatox, most probable cause, I did not want to write 'python "%~dpn0" %*' [11:27] hehe [11:32] mandel, only a couple of lint issues [11:33] i added to the MP [11:35] mandel, if you have a minute, could you review this? https://code.launchpad.net/~diegosarmentero/ubuntuone-client/fix-links/+merge/86605 [11:36] gatox, let me look [11:51] gatox, lint issues should be fixed [11:51] gatox, the XXX comments are tests to ensure that lint does not fail when they are present, so please ignore them since they are present in trunk [11:52] ok [11:53] mandel, mmmmmm there is still a lint issue in setup.py [11:53] gatox, wait, they are not.. it seems that pylint is returning me issues with the .bats.. [11:54] mandel, there are pep8 issues too.... but i don't know if that is checked too [11:54] gatox, let me clean that.. [11:54] mandel, ok [11:54] mandel, let me know [11:54] mandel, except for that..... is a +1 [12:01] good morning! [12:03] ralsina, hi! [12:04] lisettte, ping [12:08] hello everyone! [12:08] nessita, hi [12:08] hello gatox! [12:08] gatox: shall I review something for you today? [12:08] ralsina, nessita buenas! [12:09] gatox, can I have a re-review? [12:09] hola mandel! [12:09] nessita, yes.... all my branches should be ready! alecu made a comment in network-detect.... but i'm creating a new bug for that [12:09] mandel, ok [12:09] mandel: could you please review https://code.launchpad.net/~nataliabidart/ubuntuone-client/make-it-work/+merge/87767 ? [12:09] gatox, would be revno 577 [12:10] gatox: a new bug for the assetEqualPixmap? [12:10] nessita, yes [12:10] ralsina, how are you feeling about a tiny review in ubuntuone-dev-tools? [12:10] mandel: willing! [12:10] nessita, yes.... to move that into a common place in sso, and make u1-cp and u1-installer use it from there..... but i prefer not to delay this branch even more for that [12:10] ralsina, is a nice convenient script addition for people on windows :) [12:10] ralsina, there you go: https://code.launchpad.net/~mandel/ubuntuone-dev-tools/give-me-batch/+merge/88342 [12:11] ralsina, if you do a python setup.py install and add C:\Python27\Scripts to the search path in cmd you can use u1trial and u1lint :) [12:11] mandel, Tests OK, Test it IRL ok. +1 [12:12] nessita, running the tests, if they pass in P and Windows I'll approve [12:12] gatox: right, but did you changes the exception thingy? [12:12] nessita, i'm submitting that code now [12:13] awesome [12:13] nessita, i wanted to check with lisettte if "Cannot get network state" is ok [12:13] gatox: yes please, check with her [12:16] mandel: cool! [12:16] nessita, can i assign the assertEqualPixmaps to me? [12:16] gatox: not yet, just assign ubunutone-desktop+ please [12:16] nessita, ok [12:18] mandel: +1 and set to approved [12:19] ralsina, sweet! [12:19] ralsina, I was really tired of having to be doing python + full_path_u1trial :P [12:26] nessita, FYI I got an error on windows with sdtool, if the same error is in trunk I'll approve.. and will file a bug [12:26] mandel: what error? so we report or fix... [12:27] nessita, waiting for all the tests to finish, I saw it while running them [12:31] bu, my computer rebooted itself [12:32] nessita, it seems that we all have very unstable systems.. [12:42] ralsina: I approved it, but no tests for this? https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/register_plus_login/+merge/74127 [12:43] webm0nk3y: that one is not going to get merged, sorry I forgot to mark it WIP [12:43] ralsina: ok [12:43] webm0nk3y: it's a workaround until we have a decent solution [12:43] k [12:43] nessita, FYI paste.ubuntu.com/801693 in the make it work branch, looking if it is in trunk [12:43] which I suppose since you did not touch that at all [12:44] mandel: looking! [12:44] mandel: ah, it should be in trunk as well... since gatox fixed that a couple of branches ago... [12:45] gatox: right? ^ I remember you asking about http://paste.ubuntu.com/801693/ [12:45] nessita, let me check [12:45] nessita, yes..... let me check if it is in trunk..... or in my u1-client-fix-links branch [12:46] nessita, mandel it's fixed in my ubuntuone-client-fix-links branch [12:46] rye: did you get my ping from the other day? [12:46] gatox: right, that's what i thought. Thanks! [12:46] mandel, that you can re-review :P https://code.launchpad.net/~diegosarmentero/ubuntuone-client/fix-links/+merge/86605 [12:46] mandel: so you should get that in trunk as well, for now [12:46] gatox, sure, one min [12:47] mandel, thanks [12:47] nessita, I'll approve since the error are in trunk and not related to your branch [12:47] mandel: thanks! [12:58] gatox, FYI running tests on windows.. which take a gazillion years! [12:58] mandel, for what? [12:58] gatox, fix-links [12:58] mandel, ahhhhh for my branch [12:59] mandel, buy an alienware :P you are not going to be able to have a second monitor.... BUT TESTS RUN FAST!!!!! [12:59] jejeje [12:59] gatox, well, I have a number of vms running, so they good slow due to that :) [12:59] :P [13:00] gatox, and did a partial upgrade.. which has done something funny to my kvm setup, apparently I need to reboot [13:00] gatox: did you push the fix to network-detect? [13:01] nessita, no...... i'm waiting for lisettte's pong [13:01] nessita, should i push it with that message?? and we can change it later.... [13:02] gatox: hum.... let me think... what message you use, so far? [13:03] nessita, "Cannot get network state" [13:03] gatox: let's change it to: [13:03] "The network state can not be detected." [13:03] nessita, ok [13:04] pushing..... [13:04] gatox: all green for fix-links on windows!!! [13:04] nessita, GREAT! [13:04] and it should be all grean for network-detect too :P [13:04] nessita, push done [13:05] gatox: why all the removals to run-tests.bat? :-/ [13:05] nessita, ohhhhhhhh..... that was a mistake..... i was testing something on windows [13:05] reverting that [13:05] gatox: please [13:07] nessita, done [13:10] gatox, why are you removing the style checks on windows? [13:11] gatox, as well as the removal of the result from coverage [13:11] gatox, here: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/fix-links/+merge/86605 [13:11] mandel, i already revert that [13:11] in the run-tests.bat [13:11] hello #ubuntuone peoples and peoplas! [13:11] gatox, oh, ok, so I have and old version.. [13:11] hola alecu [13:12] alecu, buenos dias [13:12] mandel, i've just reverted [13:18] gatox, in _assert_read_link you could have done destination = os.path.join(self.basedir, target) and then pass the target as 'target' or 'target.lnk' [13:18] gatox, also test_create_shares_link_existing_destiny_with_lnk_extension and test_create_shares_link_existing_source_with_lnk_extension can share a fair amount of code [13:19] ok, I need to reboot due to the upgrade [13:20] gatox: did you push the restore of run-tests.bat? [13:20] nessita, yes [13:20] mandel, can you add the needs fixing for that? [13:20] gatox, already added it :) [13:20] mandel, ok [13:21] * mandel reboots [13:36] gatox: is fix-links pushed? [13:37] I merged a while ago and wasn't [13:37] will retry [13:37] nessita, it should..... i'm fixing mandel's comments.... [13:37] mandel, ping [13:38] nessita, at least in the page the run-tests.bat change was reverted [13:49] * mandel lunch [13:50] gatox, thanks for your review... there was some conflict while doing the merge, so a testing class was repeated! [13:50] alecu, ahhh [13:50] alecu, and hi! :P [13:51] alecu, let me know when you need a re-review [13:51] sure, thanks! [13:52] hmm [13:58] ralsina: are we having the weekly call today? [13:58] gatox, I've pushed the changes; lp is still rescanning the branch. [13:58] nessita: yes [13:59] ralsina: would you know how long will it be? approx [13:59] nessita: 20 minutes [13:59] alecu, ok..... i'll run the tests and lint checks in the meanwhile [14:00] ralsina: really? :-) I may have an appt at 12:45... and though that it will overlap with it [14:00] unless my mumble isincomprehensible as yesterday, in which case I will try to find a good place and do it tomorrow [14:00] nessita: I'll keep it brief [14:01] ralsina: I can reschedule this thingy... no problem... [14:01] nessita: it's ok, I don't *want* it to be long :-) [14:02] ralsina: :-) [14:23] alecu: would you please un-donkey me? [14:24] undonkey? [14:24] alecu: "desburrarme" [14:24] alecu: why would you do Response(bytes(reply.readAll()), headers) <- the bytes() adding confused me [14:25] nessita, bytes is an alias for "str" [14:26] alecu: ok, and that would encode the unicode using the system default encoding? [14:27] are you sure we're not getting unicodedecodeerrors there? [14:27] nessita, ralsina: you got my mail re: CD space, right? [14:27] dobey: yes! thanks for the detail on that [14:27] dobey: got it! [14:28] dobey: what I'm not complete sure is how doable is having gwibber dropping webkit. I know you built a testing branch, but more work needs to be done, no? [14:28] nessita: more yes, but not a lot [14:28] well, not a lot on gwibber. and i don't expect our stuff will be a huge issue either [14:29] dobey: and we should do that, or we expect platform to do it? [14:29] nessita: i will probably do it, since i've already mostly done it anyway [14:30] dobey: and how that fits your roadmap work? I mean, how much load is it? can we fit it in your schedule? [14:30] although, will need some testing, to ensure all the account types still work [14:30] nessita: it's very little load. i can probably finish it tomorrow [14:30] it shouldn't take long at all even [14:31] nessita, sorry, was fighting with a telecentro customer representatives [14:32] nessita, so, with that I'm making sure that I'm passing bytes (or str, in fact), from the QSomethingBytes returned by readAll() [14:32] alecu: ah, ok, but we're sure those are not unicode, right? [14:32] nessita, let me check the exact type [14:32] dobey: ok then, please be sure to mention this in the weekly call so ralsina can update his spreadsheet [14:33] dobey: if it's a day or 2, it doesn't move anything onthe roadmap [14:34] given the progress i made on it in ~1hr last night, i doubt it will take that long :) [14:34] dobey: awesome [14:35] nessita, so, readAll returns a QByteArray: http://doc.qt.nokia.com/4.7-snapshot/qiodevice.html#readAll [14:35] nessita, so we should specifically not worry about unicode issues there. [14:36] i hope my mumble still works; was trying to get audio tools working last night [14:38] alecu: awesome, thanks for looking that yp [14:38] up* [14:42] mandel: what problems you having with nautilus? [14:42] hrmm, i guess right now is a bad time to start upgrade to P [14:47] gatox: after running your branch i get a lot of new (test) dirs in my userhome!!! perhaps this is also happening in trunk, I'll check [14:48] nessita, please let me know [14:48] gatox: but this is "serious", tests can not write the user home... I guess something was left once we removed the setting of the env HOME [14:48] gatox: I will report a bug with details [14:49] nessita, can you explain me how to reproduce it to check here?? just run the test on linux and check the home? [14:49] gatox: this is on windows [14:49] ahhhhh ok [14:49] nessita, are you talking about the fix-links branch? [14:50] gatox: you will get stuff in your home such as test_udf_quota_exceeded [14:50] nessita, true [14:50] gatox: yes, but also trunk. And I think this is because of the branch that added xdg_home... not all the calls to setting the HOME env have not be replaced with patching xdg_home [14:51] gatox: we need to do this in the devtools TestCase base class [14:51] nessita, ahhhh [14:51] fiuuuuu :P [14:51] gatox: fiuuu? [14:51] hrmm [14:51] * gatox sigh..... don't want more issues in his branch :P [14:52] nessita, ^ [14:52] gatox: so, for starters, this grep should give nothing on u1client: [14:52] grep HOME tests/ [14:52] but it does, so that's the first problem === o0o0o is now known as greenkobold [14:52] nessita, mmmmm i don't get anything with that grep [14:53] in u1-client [14:53] from trunk [14:53] is passing a directory to grep a magic thing? [14:53] gatox: add -r [14:53] or you mean egrep? [14:53] :P [14:53] or -r :) [14:53] dobey: I have an alias, grep='grep -rn --color' [14:54] dobey, the gtk theme is out for example [14:54] gatox: grep -r --color HOME tests/ [14:54] dobey, it does not stay in the launch bar etc.. [14:54] nessita, yes yes.. i already saw it.... [14:54] mandel: oh, that means gnome-settings-daemon probably crashed [14:54] nessita, do you want me to fix that in another branch? [14:54] oh, and I'm back :) [14:54] gatox: yes, I will report a bug. But this needs to be high priority [14:55] nessita, ok.... i'll do it right now...... let me know if the branch is ok....... i reallyyyyy want to merge that branches [14:55] mandel: i wonder why apport doesn't come up for that though :-/ [14:56] dobey, no idea and I have not looked into it too much to be honest [14:56] gatox: one second, I'm still trying to tst it IRL when I saw all those unwanted dirs in my home [14:56] nessita, ok [14:57] hrmm, and rhythmbox trunk is very crashy for me now on oneiric even [14:57] i wonder if it's crashy on P too [14:59] in my case I have broken packages too due to libreoffice retuning a 1 during installation... [14:59] well i'm happy to remove libreoffice, but my packages aren't broken in that manner :) [15:00] gatox: bug #915380 [15:00] Launchpad bug 915380 in ubuntuone-client "Windows: Tests are modifying the real user home (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/915380 [15:00] nessita, ack [15:01] gatox: please read the last comment. Let's try not to only fix this, but also make some cleanup, so if we could make the patch in a single location and make the tests use a user_home_dir variable (or similar), that would be great [15:01] nessita, ok === Guest5160 is now known as adorilson === yofel_ is now known as yofel [15:28] natalia: 26 minutes! not bad! ;-) [15:28] nessita: ^ [15:28] hmm [15:28] ralsina: yeah! [15:29] alecu: sorry I left ! [15:29] alecu: I will re login :-D [15:30] nessita, my mumble is trying to reconnect [15:31] alecu: ready when you are (or when your mumble is :-P) [15:38] ok. *not* going to spend $18K for a monitor [15:39] dobey, 18k?????? [15:39] well, $17,599 [15:40] dobey, for a monitor?? which kind of monitor is that? [15:42] http://www.cdw.com/shop/products/Barco-Coronis-Fusion-6MP-DL-LCD-display-6MP-color-TFT-30.4in/2443189.aspx [15:42] 3280x2048 30" [15:43] dobey, wow...... but..... crazy expensive [15:44] yeah :( [15:46] * gatox lunch..... brb === gatox is now known as gatox_lunch [15:47] dobey: isn't 18ms kinda slow for a 18K monitor? ;-) [15:47] slow? [15:47] oh, 18ms is slow i guess [15:48] it's not really a gaming monitor, and i don't know if 18ms speed matters as much for endoscopy :) [15:48] hehe [15:48] dobey: endoscopy much? [15:48] i think the old sgi monitor i have might be 18ms [15:49] no, i just want really high resolution screen [15:49] or at least better than what i have now [15:49] i don't want to get a bigger screen, with a lower dpi [15:50] everyone seems to be killing off the good displays and making everything be 1080p now :( [15:50] dobey: http://www.nextag.com/NEC-MultiSync-MD213MC-21-651397759/prices-html [15:50] that's 2048x1536 [15:51] portrait orientation, cool for coding! ;-) [15:51] ralsina: and almost $3500 [15:51] ralsina: my current screen is 2048x1152 :) [15:52] at 23" [15:52] dobey: compare to 18K ;-) [15:52] and i paid $200 for this one :( [15:52] ralsina: and that NEC is 24ms, so slower :) [15:53] http://erclk.about.com/?zi=5/342s [15:53] nessita, http://twistedmatrix.com/documents/current/core/howto/process.html [15:53] 27" 2560x1440 [15:53] better, but still too expensive i think [15:53] dobey: off-mainstream high resolutions arecrazy expensive :-( [15:54] nessita, twisted.internet.utils.getProcessValue [15:54] well medical displays are crazy expensive, yeah [15:54] dobey: pixel density is pretty much the same asyours, it's just a bit bigger [15:54] yeah [15:54] but i don't really want to spend $1000 to basically have what i currently have [15:55] i just need to find me someone who has a nice kit package to use the 15" or 17" 1920x1200 panels, with a workstation [15:57] hrmm [15:57] what are the specs for the 17" macbook pro exactly... [15:57] dobey: http://accessories.us.dell.com/sna/productdetail.aspx?c=us&l=en&s=soho&cs=ussoho1&sku=2XU2312&dgc=SS&cid=52102&lid=1342490 [15:57] dobey: 650 for two monitors? [15:58] dobey: 3800x1080 ;-) [15:58] ralsina: yaeh, but they're 1080p [15:58] and 23" [15:59] hmm [15:59] the MBP response time is 16ms [15:59] which isn't too bad [15:59] dobey: that, with two portraitmoitors, would be awesome [16:01] i could do with 2 MBP displays on my desk [16:02] ubuntuone is not doing any sync on my computer [16:03] I mean uploading [16:03] duanedesign, rye: ^^ [16:04] any help? [16:04] kpok, hello. What Ubuntu version are you running? [16:05] ubuntu 11.10 [16:05] hey [16:05] kpok, ok, what status does Ubuntu One Control Panel show? Synchronization in progress or something else? [16:06] yes.. File sync in progress [16:06] but it doesnt upload anything [16:07] i suspect it keeps reconnecting [16:07] kpok, you may want to try reconnecting and see whether another server will perform better. What are the sizes of the files on average? [16:07] each file is around 400 mb or so [16:07] I can try adding smaller files [16:07] but I want those files to be uploaded [16:07] alecu: http://docs.python.org/library/subprocess.html#popen-objects [16:08] and yes, it keeps reconnecting too [16:10] kpok, could you please try add a small file to see whether it uploads faster (the uploads are concurrent) [16:10] verterok, ^ S3 being slow? [16:14] rye, no luck with any smaller files either :( [16:14] kpok, could you please compress the log files in ~/.cache/ubuntuone/log/ and send them to ubuntuone-support@canonical.com - i want to check whether syncdaemon is actually doing something [16:15] okay [16:16] rye: no idea, just back from the bank...let me sync logs and check [16:16] verterok, i will re-ping once i get the logs [16:16] rye: server logs :) [16:16] nessita, http://developer.gnome.org/glib/2.30/glib-The-Main-Event-Loop.html#g-child-watch-add [16:16] verterok, I want to check that the client is ok [16:17] rye: ll [16:17] kk too [16:18] ok i think i need to get me some lunch [16:18] bbiab [16:20] rye, I sent the log files to the said email [16:20] kpok, got the message in the system, checking [16:23] kpok, could you please pastebin the output of u1sdtool --waiting (to https://paste.ubuntu.com ) and give the URL? [16:25] http://pastebin.ubuntu.com/801897/ [16:26] rye, ^^ [16:26] kpok, ok, and what is u1sdtool --current-transfers ? [16:28] rye, http://pastebin.ubuntu.com/801900/ [16:29] kpok, ok, i will now check the server logs for your session information [16:30] okay thanks rye, I ll wait. :) [16:31] verterok, how about adding server/slave id to session? === alecu is now known as alecu_lunching === zyga is now known as zyga-afk === gatox_lunch is now known as gatox [16:48] ha! I cracked the bastard.. I just need to write better tests.. [16:50] gatox: seems like fix-links will be approved! I tested IRL and looks great [16:52] nessita, awesome!! [16:55] ralsina: also, if i wanted to pay that much for a display, i could just buy a nice used IBM T221 for about $1500, and it's 3840x2400 :) [16:56] alecu_lunching: vote and comment added to https://code.launchpad.net/~alecu/ubuntu-sso-client/restful-client/+merge/88230 [17:05] alecu_lunching, nessita, I have a way to remove the timed tests in the filesystem notifications on windows so that we are more deterministic [17:06] mandel: let me know! but after lunch, I'm starving :-D [17:06] lunchtime!!! [17:07] nessita, sure :) [17:14] heh [17:24] nessita, alecu_lunching this branch should be ready: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/network-detect/+merge/77049 [17:25] rye: it could be added in the same session id, but we can't add new fields to protobuf messages in old clients :( [17:26] rye: well, we can...but no way to make all the installed clients use the new storage-protocol package :) [17:26] verterok, is sessionid allocated on the client? [17:26] rye: it's server side info === zyga-afk is now known as zyga [17:27] verterok, is it a string or strictly UUID? [17:27] rye: we could add the extra info like this: sessionid: [17:27] verterok, eeexactly [17:27] rye: yes, that's what I initialy proposed :) [17:27] rye: I think the client treats it as string, but need to check [17:30] rye: could you file a bug requesting adding this extra info to the session id? (assign it to foundation+) === alecu_lunching is now known as alecu [17:40] verterok, done [17:40] rye: thanks [17:42] verterok, for kpok the S3 responds with RequestTimeout [17:44] rye: no TCPTImedOut errors? [17:45] rye: might be a transient issue...you know S3 it's expected to fail [17:45] :) [17:47] gatox, I'm re-reviewing your branch; can you re-review mine? https://code.launchpad.net/~alecu/ubuntu-sso-client/restful-client/+merge/88230 [17:47] alecu, of course [17:47] on it [17:52] rye, any possible fix for the issue ? [17:52] verterok, it complains that we are not sending bytes [17:52] verterok, https://pastebin.canonical.com/58052/ [17:52] alecu, +1! nice! [17:53] kpok, what's your upload speed? [17:53] 512 kbits ps [17:53] verterok, and there are no concurrent uploads, only one file [17:53] rye: we have bytes buffered to send...might be a load related issue or a bug :) [17:54] rye: yes, doesn't looks like a client side upload issue [17:54] rye: check the difference between bytes recv vs sent [17:54] verterok, can we add something that will notify us upon S3 error message that tells, whether we have something in buffer to send [17:54] rye: we have bytes ready to be sent [17:55] aha [17:55] verterok, recv is from client, sent is to s3 - right? [17:55] rye: yes [17:55] interesting [17:56] kpok, could you please re-paste the output of u1sdtool --current-transfers? [17:56] nessita, I've added the refactor you requested. Do you want to run the tests again, before I set the branch to Approved? [17:56] alecu: nopes, I trust you [17:56] thanks! [17:57] rye : uploads - 0 and downloads - 0 [17:57] kpok, u1sdtool --status ? [17:58] kpok, is it QUEUE_MANAGER and IDLE ? [17:58] rye: yes, this is a server issue, I bet for high load (we are getting a lot more load than usual)...but could be a bug [17:59] rye, http://paste.ubuntu.com/801987/ === beuno is now known as beuno-lunch [18:02] kpok, working? u1sdtool --waiting - does it list some filenames ? [18:05] rye, not working, but its listing file name. http://paste.ubuntu.com/801997/ [18:07] rye, a smaller file got uploaded, not the big one, so it was working briefly [18:10] red branches are the best branches [18:16] gatox, pingt [18:17] -t [18:17] gatox, I don't understand why you removed all of this: NETWORK_DETECTION_ERROR = _("The network state can not be detected.") [18:17] ralsina, nessita, alecu, gatox, mandel: the start of the future: https://code.launchpad.net/~dobey/libubuntuone/init-restructure/+merge/88416 [18:17] dobey: yay! [18:17] dobey: I like futures! I want to live there! [18:18] it's pretty neat [18:18] i hear there are free kit-kat bars for everyone! [18:18] dobey: yes. Some people call it a communist chocolatey hellhole [18:18] alecu, it was based in a long talk between nessita and lisettte ..... and it was decided that the try-except is needed to be removed, because that was there "just in case" [18:18] dobey: but not me! [18:18] heh [18:19] dobey: gtk3-mono won't be gi-based? Weird [18:19] i think doing gi in mono is hard, and it just hasn't been done yet, though i think they want it [18:19] but gtk3 mono is well, lacking resources [18:19] gatox, ok, I understand then the bit about removing self.critical(...) [18:20] gatox, but I still think keeping the try except is good [18:20] alecu, yep [18:20] nessita, ^ [18:20] alecu: design will be proposing a more generic workflow for handling unexpected errors... something like hooking to the except hook and doing something tehre [18:20] gatox, because: what would happen if networkstate.is_machine_connected throws an error before returning the deferred? [18:20] alecu: why you would keep the try-except? what would you do in that case? [18:21] dobey, I'll look at it tom morning, is that ok? [18:21] mandel: hopefully you won't have to :) [18:21] dobey, although is a nice branch, just deleting :) [18:21] nessita, gatox, if I get an error trying to detect the network I would let the program proceed [18:21] nessita, gatox: as if it had connection [18:21] yeah, it's all red, with 2 lines of green in there :) [18:22] ok, people EOD for me, see you all tom [18:22] alecu: hum... [18:22] mandel, bye [18:22] alecu, I'll ping lisettte tom morning about the ui for proxy [18:22] alecu: can be, I did not think about that [18:22] gatox: what do you think? [18:22] mandel, cool, see you tomorrow [18:22] ........ that's what i explain to lisettte [18:23] that's why i put the try-except [18:24] gatox, I've added a comment on this to the MP [18:26] nessita, alecu yes.... i think at least we can just show the "network_detection_page" which has a "try again" button, if the network detectionn fails..... [18:26] maybe it's not necessary to show an special error message [18:26] but probably the page will be the right choice [18:26] gatox: can you explain further on the 'right' choice? :-) [18:27] maybe we should offer a "go ahead even if it looks like there is no network" just in case [18:28] nessita, yes, instead of doing nothing..... if we can detect if there is any network connection (because of an exception)...... show the network detection page...... if it was because of an exception, the user will have the chance to press "try again" and continue...... [18:28] unless the exceptio persist [18:28] (if we can't detect) i mean [18:29] gatox: my fear is that if we got an exception, we'll got it all the time [18:29] nessita, yes.... that can be another scenario too [18:30] gatox: so, I would do this: [18:31] gatox, nessita: if network detection fails (by exception) I would assume there is a network [18:31] * do nothing in this branch, not even have a try-except [18:31] * if we have user reports saying "it crashed", add a try-except and handle gracefully with design inout [18:31] nessita, ok...... alecu agree [18:31] ? [18:31] ack [18:31] ralsina: yes, that too, but we should add a message del estilo "We could not detect your network conn, we assume is there" [18:32] ralsina: and for that we need a new string from design, no? [18:32] nessita: yes. But not lock the user out in an eternal "try again" loop [18:32] nessita: yep [18:32] nessita, if we fail to detect we should not show anything and go ahead, not even a message or a button or a retry. [18:33] ralsina: right, so my proposal is kinda rough: remove the try-except and let this explode in the user face. The thing is we don't know how can this explode, so far we think we did all right and will not explode [18:33] rye, smaller files are being uploaded but larger files are not. [18:33] doesn't sound like a good idea. We can't have things explode "to test" [18:34] But if we are confident it will not explode... try/except and log, without the string [18:34] ralsina, I think nessita means that having this explode will mean some other error in our code [18:34] ralsina, so, catching errors there may mask some other problem in our code. [18:34] alecu: sure. It always means that ;-) [18:35] kpok: is the same large file as before? [18:35] yes verterok , its the same file. [18:35] and its not uploaded [18:36] kpok: ok, could you pastebin the logs? [18:36] kpok: ~/.cache/ubuntuone/log/syncdaemon.log [18:36] I would like this much moreif we are going to have the generic exceptionhook [18:37] kpok: looking at your last paste of --waiting output, I think you'r hitting a bug in the server (the fix is already in the pipeline to get applied) but need to confirm with the logs [18:37] ralsina: we are, in some future [18:38] nessita: hmmm you don't sound convinced of it being *our* future. But hey, te doy la derecha. [18:38] verterok, http://paste.ubuntu.com/802032/ [18:39] ralsina: not sure what you mean... I wanted to say that we are working on an generic except handler but we first need input from design and then room in the roadmap [18:39] nessita: "some future" sounded tome a bit more vague than that :-) [18:39] nessita: so it's a +1 from me on that. Go ahead [18:40] ralsina: some future because roadmpa future is so blurry to me right now... :-) [18:40] nessita: hehe [18:40] nessita: it's not blurry, it's *flexible* ;-) [18:41] juaz [18:41] kpok: hmm, looks like the client is stuck after getting a "no more space" (IOError: [Errno 28] No space left on device) error [18:41] kpok: do you tried restarting it? u1sdtool -q; u1sdtool -c [18:42] ralsina: how can I create a QStringList from a python list? seems silly but I'm stuck [18:42] QStringList(list) [18:42] Ormaybe QStringList(*list) [18:42] just that? thanks! [18:42] (the doc really confused me) [18:42] nessita: OTOH, most things that take QStringLists also take lists [18:43] nessita: (should be all but I am not betting) [18:43] kpok: the client compress the file before uploading it, so it needs enough space on disk to store the compressed copy [18:43] nessita, ralsina alecu http://memegenerator.net/instance/13155188 [18:43] gatox: why are you having chipaca say your lines ;-) [18:43] jejejje [18:45] verterok, I ll restart ubuntu to free up space and let you know the status === zyga is now known as zyga-afk [18:54] alecu: I got it working! [18:54] nessita, url or it didn't happen! [18:56] alecu: https://pastebin.canonical.com/58058/ [18:56] To run in linux [18:56] python spawn-test.py --glib [18:56] python spawn-test.py --qt [18:56] in windows: the same! :-P [18:56] but look at the path I'm using as the control panel path [18:56] you may need to tweak that === beuno-lunch is now known as beuno [18:58] nessita, cool, I'm looking at the code, and it looks perfect, I'll trust that it works :-) [19:00] nessita, you'll probably use G_SPAWN_DO_NOT_REAP_CHILD instead of 2 in the final code, right? [19:00] alecu: can't find the constant in the python module [19:00] alecu: did you find it? [19:00] nessita, I'm certainly not sure how to get at that constant using GI [19:01] nessita, just wondering about that too [19:01] nessita, probably dobey knows ^ [19:02] dobey: would know how can I access these flags from gi.repository.GLib http://developer.gnome.org/glib/2.30/glib-Spawning-Processes.html#GSpawnFlags ? [19:02] gatox, lol at the XX guy [19:02] GLib.SpawnFlags.DO_NOT_REAP_CHILD [19:02] ejejejeje [19:02] probably [19:03] NOTICE: we are currently investigating the failures for mobile uploads and file downloads from the web. [19:03] i'll be offline for a few minutes.... i'm doing some network detection tests on linux [19:03] verterok, http://paste.ubuntu.com/802059/ <--- new log [19:03] dobey: oh, that simple [19:03] alecu: ^ [19:03] dobey: thanks! [19:04] nessita: if we want to implement to --switch-to handling in linux, it's not all that hard [19:04] ralsina: hum? [19:05] ralsina: it already works in both OSes [19:05] nessita: well, on windows, if there is a u1cp running, starting a second one makes the first one restore its window [19:05] nessita: and it doesn't switch to the right page [19:05] kpok: thanks, what's the output of u1sdtool --current-uploads [19:06] ralsina: ah... is ok. This is juts a proof of concepts for sso... so no need for now [19:06] nessita: ack [19:06] yeyyyyy...... i've network detection is working under windows and linux..... in a more "multiplatform" way :P [19:06] alecu, question [19:07] alecu, it's a dbus question...... so, i assume you might know [19:08] alecu, in u1-sso-client/ubuntu_sso/networkstate/linux.py, why the signal is being removed in the call_result_cb method (self.state_signal.remove())? [19:08] removing the signal avoid to listen to future network state changes [19:09] facundobatista: hi, kpok client seems to be stuck: http://paste.ubuntu.com/802059/ any ideas? [19:11] verterok, why stuck? [19:12] facundobatista: sorry, one upload is stuck [19:12] facundobatista: previous run: http://paste.ubuntu.com/802032/ [19:13] verterok, IOError: [Errno 28] No space left on device [19:13] facundobatista: yes, so I told him to restart the client [19:13] :) [19:13] facundobatista: but now isn't even attempting to upload the file :/ [19:14] facundobatista: it got a bunch of disconnects, and then nothing [19:16] gatox, looking [19:16] verterok, how do you know? that upload may be one of the two queued ops [19:17] facundobatista: there is nothing in the logs [19:17] 2012-01-13 00:26:09,119 - ubuntuone.SyncDaemon.Main - NOTE - ---- MARK (state: ; queue: 2; hash: 0) ---- [19:17] verterok, ^ [19:17] gatox, that seems to be the way the upper layer expects to be called [19:17] facundobatista: yes, but no Upload, nor nothing [19:17] gatox, so yes, we could change it to *not* remove the signal handler [19:18] verterok, yes, look at line 38 [19:18] gatox, but we should be extra careful that the upper layers are ok with that layer calling the result_cb more than once. [19:18] alecu, yes, i remove it and made a couple of tests.... and network detection is working fine without the removal [19:18] *extra* [19:18] alecu, roger that! [19:18] gatox, make sure this not also used from syncdaemon, control panel and above. [19:19] alecu, ok [19:19] facundobatista: yes, a warning that it was disconnected [19:19] gatox, and also the windows.py will need to be changed too. [19:19] verterok, I mean, the upload is there! [19:19] facundobatista: yes, it's there...but how do we know it's "running" [19:20] alecu, yes, i made a couple of changes there and in the dbus and ipc interface in the way the consume the network_detection module..... because the multiplatform was not being used [19:20] verterok, it's not running, client needs to get to QUEUE_MANAGER state [19:20] verterok, do you have more logs? [19:21] facundobatista: nope, that's all. I'm waiting to get u1sdtool --current-transfers output [19:21] verterok, we should have more logs, at least MARKs showing the state [19:22] facundobatista: this log is fresh just after the client restart [19:22] verterok, http://paste.ubuntu.com/802095/ <--- for u1sdtool --current-transfers [19:22] facundobatista: nm, it's working [19:23] kpok: ok, so it's uploading [19:23] it will probably take a while [19:23] its not.. its stopped right now.. :( [19:23] I just keep getting libnotify popups that its uploading [19:24] this is probably the fourth or fifth time I ve got a popup [19:24] kpok: please pastebin the logs again [19:24] okay sure [19:25] kpok: you migth got disconnected from the server, it will resume uploading as soon it's connected again [19:25] isnt there something wrong if it keeps disconnecting every few minutes? Shouldnt there be a steady upload? [19:27] kpok: yes, something is wrong [19:28] latest pastebin ---> http://paste.ubuntu.com/802106/ [19:28] log [19:29] facundobatista: what does this mean: "No Pong response, disconnecting the client" ? [19:29] facundobatista: the server didn't replied on time? [19:29] verterok, exactly [19:30] kpok: yes, the server is having some problems, but should upload steady as soon it gets fixed. [19:31] okay I can wait .. thanks everyone for the time and effort. :) [19:32] I ll come back here if it doesnt get resolved in 24 hours.. is that okay? [19:32] sure [19:33] okay later [19:33] cheers [19:49] ralsina, nessita: any good reason for me to not convert rhythmbox-ubuntuone over to autotools rather than distutils? [19:49] dobey: why would you? [19:49] dobey: is it broken? [19:50] ralsina: so that we don't have to use hardcoded paths in setup.py, i can translate settings schemas properly, and whatnot [19:51] or i'll just get rid of this setting anyway [19:51] dobey: if it's useless, why not [19:52] dobey: our autotools skill level as a team is too low, is my main concern [19:52] well, it's not totally useless [19:52] but i do hate gsettings [19:53] it's the first-run setting key, so we can select the u1 source by default [19:53] although certainly not going to be usable in this first run [19:53] so yeah i'll just delete it for now [19:54] ok [19:58] need to take a break though. bit of rsi :-/ [20:02] eod for me!! see you tomorrow people [20:02] bye gatox! [20:02] ralsina, bye === gatox is now known as gatox_away === JanC_ is now known as JanC [20:16] maybe a good time to do the upgrade to precise [20:37] EOD for me. Have a nice evening! [20:39] cheers ralsina [20:40] about 2 hours remating for upgrade :-/ [20:56] ok, I'm off [20:56] bye all! [21:36] whoot [21:37] have a clean ./run-tests in rhythmbox-ubuntuone now [22:22] crikey [22:22] why is firefox using half a core [22:22] and what is up with my fonts [22:32] man firefox is crashy now :( === gatox_away is now known as gatox