/srv/irclogs.ubuntu.com/2012/01/12/#ubuntuone.txt

=== yofel_ is now known as yofel
JamesTaitGood morning, all! :D09:32
mandelmorning all!09:39
gatoxgood morning11:09
mandelgatox, morning!11:11
mandelgatox, can you do a super stupid review for me?11:16
gatoxmandel, of course!11:18
gatoxshoot11:18
mandelgatox, https://code.launchpad.net/~mandel/ubuntuone-dev-tools/give-me-batch/+merge/8834211:20
mandelgatox, I was tired of having to write python C:\Pythoon17\Scripts\u1trial on windows, so I did that :)11:20
mandelgatox, if you add C:\Python27\Scripts to the search path in cmd u1trial and u1lint will work :)11:21
gatoxmandel, nicee11:22
mandelgatox, I don't know why i did not do it earlier :)11:26
gatox:P11:26
mandelgatox, most probable cause, I did not want to write 'python "%~dpn0" %*'11:27
mandelhehe11:27
gatoxmandel, only a couple of lint issues11:32
gatoxi added to the MP11:33
gatoxmandel, if you have a minute, could you review this? https://code.launchpad.net/~diegosarmentero/ubuntuone-client/fix-links/+merge/8660511:35
mandelgatox, let me look11:36
mandelgatox, lint issues should be fixed11:51
mandelgatox, 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 trunk11:51
gatoxok11:52
gatoxmandel, mmmmmm there is still a lint issue in setup.py11:53
mandelgatox, wait, they are not.. it seems that pylint is returning me issues with the .bats..11:53
gatoxmandel, there are pep8 issues too.... but i don't know if that is checked too11:54
mandelgatox, let me clean that..11:54
gatoxmandel, ok11:54
gatoxmandel, let me know11:54
gatoxmandel, except for that..... is a +111:54
ralsinagood morning!12:01
gatoxralsina, hi!12:03
gatoxlisettte, ping12:04
nessitahello everyone!12:08
gatoxnessita, hi12:08
nessitahello gatox!12:08
nessitagatox: shall I review something for you today?12:08
mandelralsina, nessita buenas!12:08
mandelgatox, can I have a re-review?12:09
nessitahola mandel!12:09
gatoxnessita, yes.... all my branches should be ready! alecu made a comment in network-detect.... but i'm creating a new bug for that12:09
gatoxmandel, ok12:09
nessitamandel: could you please review https://code.launchpad.net/~nataliabidart/ubuntuone-client/make-it-work/+merge/87767 ?12:09
mandelgatox, would be revno 57712:09
nessitagatox: a new bug for the assetEqualPixmap?12:10
mandelnessita, yes12:10
mandelralsina, how are you feeling about a tiny review in ubuntuone-dev-tools?12:10
ralsinamandel: willing!12:10
gatoxnessita, 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 that12:10
mandelralsina, is a nice convenient script addition for people on windows :)12:10
mandelralsina, there you go: https://code.launchpad.net/~mandel/ubuntuone-dev-tools/give-me-batch/+merge/8834212:10
mandelralsina, 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
gatoxmandel, Tests OK, Test it IRL ok. +112:11
mandelnessita, running the tests, if they pass in P and Windows I'll approve12:12
nessitagatox: right, but did you changes the exception thingy?12:12
gatoxnessita, i'm submitting that code now12:12
nessitaawesome12:13
gatoxnessita, i wanted to check with lisettte if "Cannot get network state" is ok12:13
nessitagatox: yes please, check with her12:13
ralsinamandel: cool!12:16
gatoxnessita, can i assign the assertEqualPixmaps to me?12:16
nessitagatox: not yet, just assign ubunutone-desktop+ please12:16
gatoxnessita, ok12:16
ralsinamandel: +1 and set to approved12:18
mandelralsina, sweet!12:19
mandelralsina, I was really tired of having to be doing python + full_path_u1trial :P12:19
mandelnessita, FYI I got an error on windows with sdtool, if the same error is in trunk I'll approve.. and will file a bug12:26
nessitamandel: what error? so we report or fix...12:26
mandelnessita, waiting for all the tests to finish, I saw it while running them12:27
nessitabu, my computer rebooted itself12:31
mandelnessita, it seems that we all have very unstable systems..12:32
webm0nk3yralsina: I approved it, but no tests for this? https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/register_plus_login/+merge/7412712:42
ralsinawebm0nk3y: that one is not going to get merged, sorry I forgot to mark it WIP12:43
webm0nk3yralsina: ok12:43
ralsinawebm0nk3y: it's a workaround until we have a decent solution12:43
webm0nk3yk12:43
mandelnessita, FYI paste.ubuntu.com/801693 in the make it work branch, looking if it is in trunk12:43
mandelwhich I suppose since you did not touch that at all12:43
nessitamandel: looking!12:44
nessitamandel: ah, it should be in trunk as well... since gatox fixed that a couple of branches ago...12:44
nessitagatox: right? ^ I remember you asking about http://paste.ubuntu.com/801693/12:45
gatoxnessita, let me check12:45
gatoxnessita, yes..... let me check if it is in trunk..... or in my u1-client-fix-links branch12:45
gatoxnessita, mandel  it's fixed in my ubuntuone-client-fix-links branch12:46
cjohnstonrye: did you get my ping from the other day?12:46
nessitagatox: right, that's what i thought. Thanks!12:46
gatoxmandel, that you can re-review :P https://code.launchpad.net/~diegosarmentero/ubuntuone-client/fix-links/+merge/8660512:46
nessitamandel: so you should get that in trunk as well, for now12:46
mandelgatox, sure, one min12:46
gatoxmandel, thanks12:47
mandelnessita, I'll approve since the error are in trunk and not related to your branch12:47
nessitamandel: thanks!12:47
mandelgatox, FYI running tests on windows.. which take a gazillion years!12:58
gatoxmandel, for what?12:58
mandelgatox, fix-links12:58
gatoxmandel, ahhhhh for my branch12:58
gatoxmandel, buy an alienware :P you are not going to be able to have a second monitor.... BUT TESTS RUN FAST!!!!!12:59
gatoxjejeje12:59
mandelgatox, well, I have a number of vms running, so they good slow due to that :)12:59
gatox:P12:59
mandelgatox, and did a partial upgrade.. which has done something funny to my kvm setup, apparently I need to reboot13:00
nessitagatox: did you push the fix to network-detect?13:00
gatoxnessita, no...... i'm waiting for lisettte's pong13:01
gatoxnessita, should i push it with that message?? and we can change it later....13:01
nessitagatox: hum.... let me think... what message you use, so far?13:02
gatoxnessita, "Cannot get network state"13:03
nessitagatox: let's change it to:13:03
nessita"The network state can not be detected."13:03
gatoxnessita, ok13:03
gatoxpushing.....13:04
nessitagatox: all green for fix-links on windows!!!13:04
gatoxnessita, GREAT!13:04
gatoxand it should be all grean for network-detect too :P13:04
gatoxnessita, push done13:04
nessitagatox: why all the removals to run-tests.bat? :-/13:05
gatoxnessita, ohhhhhhhh..... that was a mistake..... i was testing something on windows13:05
gatoxreverting that13:05
nessitagatox: please13:05
gatoxnessita, done13:07
mandelgatox, why are you removing the style checks on windows?13:10
mandelgatox, as well as the removal of the result from coverage13:11
mandelgatox, here: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/fix-links/+merge/8660513:11
gatoxmandel, i already revert that13:11
mandelin the run-tests.bat13:11
alecuhello #ubuntuone peoples and peoplas!13:11
mandelgatox, oh, ok, so I have and old version..13:11
nessitahola alecu13:11
mandelalecu, buenos dias13:12
gatoxmandel, i've just reverted13:12
mandelgatox, 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
mandelgatox, 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 code13:18
mandelok, I need to reboot due to the upgrade13:19
nessitagatox: did you push the restore of run-tests.bat?13:20
gatoxnessita, yes13:20
gatoxmandel, can you add the needs fixing for that?13:20
mandelgatox, already added it :)13:20
gatoxmandel, ok13:20
* mandel reboots13:21
nessitagatox: is fix-links pushed?13:36
nessitaI merged a while ago and wasn't13:37
nessitawill retry13:37
gatoxnessita, it should..... i'm fixing mandel's comments....13:37
gatoxmandel, ping13:37
gatoxnessita, at least in the page the run-tests.bat change was reverted13:38
* mandel lunch13:49
alecugatox, thanks for your review... there was some conflict while doing the merge, so a testing class was repeated!13:50
gatoxalecu, ahhh13:50
gatoxalecu, and hi! :P13:50
gatoxalecu, let me know when you need a re-review13:51
alecusure, thanks!13:51
dobeyhmm13:52
nessitaralsina: are we having the weekly call today?13:58
alecugatox, I've pushed the changes; lp is still rescanning the branch.13:58
ralsinanessita: yes13:58
nessitaralsina: would you know how long will it be? approx13:59
ralsinanessita: 20 minutes13:59
gatoxalecu, ok..... i'll run the tests and lint checks in the meanwhile13:59
nessitaralsina: really? :-) I may have an appt at 12:45... and though that it will overlap with it14:00
ralsinaunless my mumble isincomprehensible as yesterday, in which case I will try to find a good place and do it tomorrow14:00
ralsinanessita: I'll keep it brief14:00
nessitaralsina: I can reschedule this thingy... no problem...14:01
ralsinanessita: it's ok, I don't *want* it to be long :-)14:01
nessitaralsina: :-)14:02
nessitaalecu: would you please un-donkey me?14:23
alecuundonkey?14:24
nessitaalecu: "desburrarme"14:24
nessitaalecu: why would you do Response(bytes(reply.readAll()), headers) <- the bytes() adding confused me14:24
alecunessita, bytes is an alias for "str"14:25
nessitaalecu: ok, and that would encode the unicode using the system default encoding?14:26
nessitaare you sure we're not getting unicodedecodeerrors there?14:27
dobeynessita, ralsina: you got my mail re: CD space, right?14:27
nessitadobey: yes! thanks for the detail on that14:27
ralsinadobey: got it!14:27
nessitadobey: 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
dobeynessita: more yes, but not a lot14:28
dobeywell, not a lot on gwibber. and i don't expect our stuff will be a huge issue either14:28
nessitadobey: and we should do that, or we expect platform to do it?14:29
dobeynessita: i will probably do it, since i've already mostly done it anyway14:29
nessitadobey: and how that fits your roadmap work? I mean, how much load is it? can we fit it in your schedule?14:30
dobeyalthough, will need some testing, to ensure all the account types still work14:30
dobeynessita: it's very little load. i can probably finish it tomorrow14:30
dobeyit shouldn't take long at all even14:30
alecunessita, sorry, was fighting with a telecentro customer representatives14:31
alecunessita, so, with that I'm making sure that I'm passing bytes (or str, in fact), from the QSomethingBytes returned by readAll()14:32
nessitaalecu: ah, ok, but we're sure those are not unicode, right?14:32
alecunessita, let me check the exact type14:32
nessitadobey: ok then, please be sure to mention this in the weekly call so ralsina can update his spreadsheet14:32
ralsinadobey: if it's a day or 2, it doesn't move anything onthe roadmap14:33
dobeygiven the progress i made on it in ~1hr last night, i doubt it will take that long :)14:34
ralsinadobey: awesome14:34
alecunessita, so, readAll returns a QByteArray: http://doc.qt.nokia.com/4.7-snapshot/qiodevice.html#readAll14:35
alecunessita, so we should specifically not worry about unicode issues there.14:35
dobeyi hope my mumble still works; was trying to get audio tools working last night14:36
nessitaalecu: awesome, thanks for looking that yp14:38
nessitaup*14:38
dobeymandel: what problems you having with nautilus?14:42
dobeyhrmm, i guess right now is a bad time to start upgrade to P14:42
nessitagatox: after running your branch i get a lot of new (test) dirs in my userhome!!! perhaps this is also happening in trunk, I'll check14:47
gatoxnessita, please let me know14:48
nessitagatox: but this is "serious", tests can not write the user home... I guess something was left once we removed the setting of the env HOME14:48
nessitagatox: I will report a bug with details14:48
gatoxnessita, can you explain me how to reproduce it to check here?? just run the test on linux and check the home?14:49
nessitagatox: this is on windows14:49
gatoxahhhhh ok14:49
gatoxnessita, are you talking about the fix-links branch?14:49
nessitagatox: you will get stuff in your home such as test_udf_quota_exceeded14:50
gatoxnessita, true14:50
nessitagatox: 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_home14:50
nessitagatox: we need to do this in the devtools TestCase base class14:51
gatoxnessita, ahhhh14:51
gatoxfiuuuuu :P14:51
nessitagatox: fiuuu?14:51
dobeyhrmm14:51
* gatox sigh..... don't want more issues in his branch :P14:51
gatoxnessita, ^14:52
nessitagatox: so, for starters, this grep should give nothing on u1client:14:52
nessitagrep HOME tests/14:52
nessitabut it does, so that's the first problem14:52
=== o0o0o is now known as greenkobold
gatoxnessita, mmmmm i don't get anything with that grep14:52
gatoxin u1-client14:53
gatoxfrom trunk14:53
dobeyis passing a directory to grep a magic thing?14:53
nessitagatox: add -r14:53
dobeyor you mean egrep?14:53
gatox:P14:53
dobeyor -r :)14:53
nessitadobey: I have an alias, grep='grep -rn --color'14:53
mandeldobey, the gtk theme is out for example14:54
nessitagatox: grep -r --color HOME tests/14:54
mandeldobey, it does not stay in the launch bar etc..14:54
gatoxnessita, yes yes.. i already saw it....14:54
dobeymandel: oh, that means gnome-settings-daemon probably crashed14:54
gatoxnessita, do you want me to fix that in another branch?14:54
mandeloh, and I'm back :)14:54
nessitagatox: yes, I will report a bug. But this needs to be high priority14:54
gatoxnessita, ok.... i'll do it right now...... let me know if the branch is ok....... i reallyyyyy want to merge that branches14:55
dobeymandel: i wonder why apport doesn't come up for that though :-/14:55
mandeldobey, no idea and I have not looked into it too much to be honest14:56
nessitagatox: one second, I'm still trying to tst it IRL when I saw all those unwanted dirs in my home14:56
gatoxnessita, ok14:56
dobeyhrmm, and rhythmbox trunk is very crashy for me now on oneiric even14:57
dobeyi wonder if it's crashy on P too14:57
mandelin my case I have broken packages too due to libreoffice retuning a 1 during installation...14:59
dobeywell i'm happy to remove libreoffice, but my packages aren't broken in that manner :)14:59
nessitagatox: bug #91538015:00
ubot4Launchpad bug 915380 in ubuntuone-client "Windows: Tests are modifying the real user home (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/91538015:00
gatoxnessita, ack15:00
nessitagatox: 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 great15:01
gatoxnessita, ok15:01
=== Guest5160 is now known as adorilson
=== yofel_ is now known as yofel
ralsinanatalia: 26 minutes! not bad! ;-)15:28
ralsinanessita: ^15:28
dobeyhmm15:28
nessitaralsina: yeah!15:28
nessitaalecu: sorry I left !15:29
nessitaalecu: I will re login :-D15:29
alecunessita, my mumble is trying to reconnect15:30
nessitaalecu: ready when you are (or when your mumble is :-P)15:31
dobeyok. *not* going to spend $18K for a monitor15:38
gatoxdobey, 18k??????15:39
dobeywell, $17,59915:39
gatoxdobey, for a monitor?? which kind of monitor is that?15:40
dobeyhttp://www.cdw.com/shop/products/Barco-Coronis-Fusion-6MP-DL-LCD-display-6MP-color-TFT-30.4in/2443189.aspx15:42
dobey3280x2048 30"15:42
gatoxdobey, wow...... but..... crazy expensive15:43
dobeyyeah :(15:44
* gatox lunch..... brb15:46
=== gatox is now known as gatox_lunch
ralsinadobey: isn't 18ms kinda slow for a 18K monitor? ;-)15:47
dobeyslow?15:47
dobeyoh, 18ms is slow i guess15:47
dobeyit's not really a gaming monitor, and i don't know if 18ms speed matters as much for endoscopy :)15:48
ralsinahehe15:48
ralsinadobey: endoscopy much?15:48
dobeyi think the old sgi monitor i have might be 18ms15:48
dobeyno, i just want really high resolution screen15:49
dobeyor at least better than what i have now15:49
dobeyi don't want to get a bigger screen, with a lower dpi15:49
dobeyeveryone seems to be killing off the good displays and making everything be 1080p now :(15:50
ralsinadobey: http://www.nextag.com/NEC-MultiSync-MD213MC-21-651397759/prices-html15:50
ralsinathat's 2048x153615:50
ralsinaportrait orientation, cool for coding! ;-)15:51
dobeyralsina: and almost $350015:51
dobeyralsina: my current screen is 2048x1152 :)15:51
dobeyat 23"15:52
ralsinadobey: compare to 18K ;-)15:52
dobeyand i paid $200 for this one :(15:52
dobeyralsina: and that NEC is 24ms, so slower :)15:52
ralsinahttp://erclk.about.com/?zi=5/342s15:53
alecunessita, http://twistedmatrix.com/documents/current/core/howto/process.html15:53
ralsina27" 2560x144015:53
dobeybetter, but still too expensive i think15:53
ralsinadobey: off-mainstream high resolutions arecrazy expensive :-(15:53
alecunessita, twisted.internet.utils.getProcessValue15:54
dobeywell medical displays are crazy expensive, yeah15:54
ralsinadobey: pixel density is pretty much the same asyours, it's just a bit bigger15:54
dobeyyeah15:54
dobeybut i don't really want to spend $1000 to basically have what i currently have15:54
dobeyi just need to find me someone who has a nice kit package to use the 15" or 17" 1920x1200 panels, with a workstation15:55
dobeyhrmm15:57
dobeywhat are the specs for the 17" macbook pro exactly...15:57
ralsinadobey: http://accessories.us.dell.com/sna/productdetail.aspx?c=us&l=en&s=soho&cs=ussoho1&sku=2XU2312&dgc=SS&cid=52102&lid=134249015:57
ralsinadobey: 650 for two monitors?15:57
ralsinadobey: 3800x1080 ;-)15:58
dobeyralsina: yaeh, but they're 1080p15:58
dobeyand 23"15:58
dobeyhmm15:59
dobeythe MBP response time is 16ms15:59
dobeywhich isn't too bad15:59
ralsinadobey: that, with two portraitmoitors, would be awesome15:59
dobeyi could do with 2 MBP displays on my desk16:01
kpokubuntuone is not doing any sync on my computer16:02
kpokI mean uploading16:03
dobeyduanedesign, rye: ^^16:03
kpokany help?16:04
ryekpok, hello. What Ubuntu version are you running?16:04
kpokubuntu 11.1016:05
kpokhey16:05
ryekpok, ok, what status does Ubuntu One Control Panel show? Synchronization in progress or something else?16:05
kpokyes.. File sync in progress16:06
kpokbut it doesnt upload anything16:06
dobeyi suspect it keeps reconnecting16:07
ryekpok, 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
kpokeach file is around 400 mb or so16:07
kpokI can try adding smaller files16:07
kpokbut I want those files to be uploaded16:07
nessitaalecu: http://docs.python.org/library/subprocess.html#popen-objects16:07
kpokand yes, it keeps reconnecting too16:08
ryekpok, could you please try add a small file to see whether it uploads faster (the uploads are concurrent)16:10
ryeverterok, ^ S3 being slow?16:10
kpokrye, no luck with any smaller files either :(16:14
ryekpok, 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 something16:14
kpokokay16:15
verterokrye: no idea, just back from the bank...let me sync logs and check16:16
ryeverterok, i will re-ping once i get the logs16:16
verterokrye: server logs :)16:16
alecunessita, http://developer.gnome.org/glib/2.30/glib-The-Main-Event-Loop.html#g-child-watch-add16:16
ryeverterok, I want to check that the client is ok16:16
verterokrye: ll16:17
verterokkk too16:17
dobeyok i think i need to get me some lunch16:18
dobeybbiab16:18
kpokrye, I sent the log files to the said email16:20
ryekpok, got the message in the system, checking16:20
ryekpok, could you please pastebin the output of u1sdtool --waiting (to https://paste.ubuntu.com ) and give the URL?16:23
kpokhttp://pastebin.ubuntu.com/801897/16:25
kpokrye, ^^16:26
ryekpok, ok, and what is u1sdtool --current-transfers ?16:26
kpokrye, http://pastebin.ubuntu.com/801900/16:28
ryekpok, ok, i will now check the server logs for your session information16:29
kpokokay thanks rye, I ll wait. :)16:30
ryeverterok, how about adding server/slave id to session?16:31
=== alecu is now known as alecu_lunching
=== zyga is now known as zyga-afk
=== gatox_lunch is now known as gatox
mandelha! I cracked the bastard.. I just need to write better tests..16:48
nessitagatox: seems like fix-links will be approved! I tested IRL and looks great16:50
gatoxnessita, awesome!!16:52
dobeyralsina: 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:55
nessitaalecu_lunching: vote and comment added to https://code.launchpad.net/~alecu/ubuntu-sso-client/restful-client/+merge/8823016:56
mandelalecu_lunching, nessita, I have a way to remove the timed tests in the filesystem notifications on windows so that we are more deterministic17:05
nessitamandel: let me know! but after lunch, I'm starving :-D17:06
nessitalunchtime!!!17:06
mandelnessita, sure :)17:07
dobeyheh17:14
gatoxnessita, alecu_lunching this branch should be ready: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/network-detect/+merge/7704917:24
verterokrye: it could be added in the same session id, but we can't add new fields to protobuf messages in old clients :(17:25
verterokrye: well, we can...but no way to make all the installed clients use the new storage-protocol package :)17:26
ryeverterok, is sessionid allocated on the client?17:26
verterokrye: it's server side info17:26
=== zyga-afk is now known as zyga
ryeverterok, is it a string or strictly UUID?17:27
verterokrye: we could add the extra info like this: sessionid:<extra-data>17:27
ryeverterok, eeexactly17:27
verterokrye: yes, that's what I initialy proposed :)17:27
verterokrye: I think the client treats it as string, but need to check17:27
verterokrye: could you file a bug requesting adding this extra info to the session id? (assign it to foundation+)17:30
=== alecu_lunching is now known as alecu
ryeverterok, done17:40
verterokrye: thanks17:40
ryeverterok, for kpok the S3 responds with RequestTimeout17:42
verterokrye: no TCPTImedOut errors?17:44
verterokrye: might be a transient issue...you know S3 it's expected to fail17:45
verterok:)17:45
alecugatox, I'm re-reviewing your branch; can you re-review mine? https://code.launchpad.net/~alecu/ubuntu-sso-client/restful-client/+merge/8823017:47
gatoxalecu, of course17:47
gatoxon it17:47
kpokrye, any possible fix for the issue ?17:52
ryeverterok, it complains that we are not sending bytes17:52
ryeverterok, https://pastebin.canonical.com/58052/17:52
gatoxalecu, +1! nice!17:52
ryekpok, what's your upload speed?17:53
kpok512 kbits ps17:53
ryeverterok, and there are no concurrent uploads, only one file17:53
verterokrye: we have bytes buffered to send...might be a load related issue or a bug :)17:53
verterokrye: yes, doesn't looks like a client side upload issue17:54
verterokrye: check the difference between bytes recv vs sent17:54
ryeverterok, can we add something that will notify us upon S3 error message that tells, whether we have something in buffer to send17:54
verterokrye: we have bytes ready to be sent17:54
ryeaha17:55
ryeverterok, recv is from client, sent is to s3 - right?17:55
verterokrye: yes17:55
ryeinteresting17:55
ryekpok, could you please re-paste the output of u1sdtool --current-transfers?17:56
alecunessita, I've added the refactor you requested. Do you want to run the tests again, before I set the branch to Approved?17:56
nessitaalecu: nopes, I trust you17:56
nessitathanks!17:56
kpokrye : uploads  - 0 and downloads - 017:57
ryekpok, u1sdtool --status ?17:57
ryekpok, is it QUEUE_MANAGER and IDLE ?17:58
verterokrye: yes, this is a server issue, I bet for high load (we are getting a lot more load than usual)...but could be a bug17:58
kpokrye, http://paste.ubuntu.com/801987/17:59
=== beuno is now known as beuno-lunch
ryekpok, working? u1sdtool --waiting - does it list some filenames ?18:02
kpokrye, not working, but its listing file name. http://paste.ubuntu.com/801997/18:05
kpokrye,  a smaller file got uploaded, not the big one, so it was working briefly18:07
dobeyred branches are the best branches18:10
alecugatox, pingt18:16
alecu-t18:17
alecugatox, I don't understand why you removed all of this: NETWORK_DETECTION_ERROR = _("The network state can not be detected.")18:17
dobeyralsina, nessita, alecu, gatox, mandel: the start of the future: https://code.launchpad.net/~dobey/libubuntuone/init-restructure/+merge/8841618:17
ralsinadobey: yay!18:17
ralsinadobey: I like futures! I want to live there!18:17
dobeyit's pretty neat18:18
dobeyi hear there are free kit-kat bars for everyone!18:18
ralsinadobey: yes. Some people call it a communist chocolatey hellhole18:18
gatoxalecu, 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
ralsinadobey: but not me!18:18
dobeyheh18:18
ralsinadobey: gtk3-mono won't be gi-based? Weird18:19
dobeyi think doing gi in mono is hard, and it just hasn't been done yet, though i think they want it18:19
dobeybut gtk3 mono is well, lacking resources18:19
alecugatox, ok, I understand then the bit about removing  self.critical(...)18:19
alecugatox, but I still think keeping the try except is good18:20
gatoxalecu, yep18:20
gatoxnessita, ^18:20
nessitaalecu: design will be proposing a more generic workflow for handling unexpected errors... something like hooking to the except hook and doing something tehre18:20
alecugatox, because: what would happen if networkstate.is_machine_connected throws an error before returning the deferred?18:20
nessitaalecu: why you would keep the try-except? what would you do in that case?18:20
mandeldobey, I'll look at it tom morning, is that ok?18:21
dobeymandel: hopefully you won't have to :)18:21
mandeldobey, although is a nice branch, just deleting :)18:21
alecunessita, gatox, if I get an error trying to detect the network I would let the program proceed18:21
alecunessita, gatox: as if it had connection18:21
dobeyyeah, it's all red, with 2 lines of green in there :)18:21
mandelok, people EOD for me, see you all tom18:22
nessitaalecu: hum...18:22
gatoxmandel, bye18:22
mandelalecu, I'll ping lisettte  tom morning about the ui for proxy18:22
nessitaalecu: can be, I did not think about that18:22
nessitagatox: what do you think?18:22
alecumandel, cool, see you tomorrow18:22
gatox........ that's what i explain to lisettte18:22
gatoxthat's why i put the try-except18:23
alecugatox, I've added a comment on this to the MP18:24
gatoxnessita, 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
gatoxmaybe it's not necessary to show an special error message18:26
gatoxbut probably the page will be the right choice18:26
nessitagatox: can you explain further on the 'right' choice? :-)18:26
ralsinamaybe we should offer a "go ahead even if it looks like there is no network" just in case18:27
gatoxnessita, 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
gatoxunless the exceptio persist18:28
gatox(if we can't detect) i mean18:28
nessitagatox: my fear is that if we got an exception, we'll got it all the time18:29
gatoxnessita, yes.... that can be another scenario too18:29
nessitagatox: so, I would do this:18:30
ralsinagatox, nessita: if network detection fails (by exception) I would assume there is a network18:31
nessita* do nothing in this branch, not even have a try-except18:31
nessita* if we have user reports saying "it crashed", add a try-except and handle gracefully with design inout18:31
gatoxnessita, ok...... alecu agree18:31
gatox?18:31
alecuack18:31
nessitaralsina: yes, that too, but we should add a message del estilo "We could not detect your network conn, we assume is there"18:31
nessitaralsina: and for that we need a new string from design, no?18:32
ralsinanessita: yes. But not lock the user out in an eternal "try again" loop18:32
ralsinanessita: yep18:32
alecunessita, if we fail to detect we should not show anything and go ahead, not even a message or a button or a retry.18:32
nessitaralsina: 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 explode18:33
kpokrye, smaller files are being uploaded but larger files are not.18:33
ralsinadoesn't sound like a good idea. We can't have things explode "to test"18:33
ralsinaBut if we are confident it will not explode... try/except and log, without the string18:34
alecuralsina, I think nessita means that having this explode will mean some other error in our code18:34
alecuralsina, so, catching errors there may mask some other problem in our code.18:34
ralsinaalecu: sure. It always means that ;-)18:34
verterokkpok: is the same large file as before?18:35
kpokyes verterok , its the same file.18:35
kpokand its not uploaded18:35
verterokkpok: ok, could you pastebin the logs?18:36
verterokkpok: ~/.cache/ubuntuone/log/syncdaemon.log18:36
ralsinaI would like this much moreif we are going to have the generic exceptionhook18:36
verterokkpok: 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 logs18:37
nessitaralsina: we are, in some future18:37
ralsinanessita: hmmm you don't sound convinced of it being *our* future. But hey, te doy la derecha.18:38
kpokverterok, http://paste.ubuntu.com/802032/18:38
nessitaralsina: 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 roadmap18:39
ralsinanessita: "some future" sounded tome a bit more vague than that :-)18:39
ralsinanessita: so it's a +1 from me on that. Go ahead18:39
nessitaralsina: some future because roadmpa future is so blurry to me right now... :-)18:40
ralsinanessita: hehe18:40
ralsinanessita: it's not blurry, it's *flexible* ;-)18:40
nessitajuaz18:41
verterokkpok: hmm, looks like the client is stuck after getting a "no more space" (IOError: [Errno 28] No space left on device) error18:41
verterokkpok: do you tried restarting it? u1sdtool -q; u1sdtool -c18:41
nessitaralsina: how can I create a QStringList from a python list? seems silly but I'm stuck18:42
ralsinaQStringList(list)18:42
ralsinaOrmaybe QStringList(*list)18:42
nessitajust that? thanks!18:42
nessita(the doc really confused me)18:42
ralsinanessita: OTOH, most things that take QStringLists also take lists18:42
ralsinanessita: (should be all but I am not betting)18:43
verterokkpok: the client compress the file before uploading it, so it needs enough space on disk to store the compressed copy18:43
gatoxnessita, ralsina alecu http://memegenerator.net/instance/1315518818:43
ralsinagatox: why are you having chipaca say your lines ;-)18:43
gatoxjejejje18:43
kpokverterok, I ll restart ubuntu to free up space and let you know the status18:45
=== zyga is now known as zyga-afk
nessitaalecu: I got it working!18:54
alecunessita, url or it didn't happen!18:54
nessitaalecu: https://pastebin.canonical.com/58058/18:56
nessitaTo run in  linux18:56
nessitapython spawn-test.py --glib18:56
nessitapython spawn-test.py --qt18:56
nessitain windows: the same! :-P18:56
nessitabut look at the path I'm using as the control panel path18:56
nessitayou may need to tweak that18:56
=== beuno-lunch is now known as beuno
alecunessita, cool, I'm looking at the code, and it looks perfect, I'll trust that it works :-)18:58
alecunessita, you'll probably use G_SPAWN_DO_NOT_REAP_CHILD instead of 2 in the final code, right?19:00
nessitaalecu: can't find the constant in the python module19:00
nessitaalecu: did you find it?19:00
alecunessita, I'm certainly not sure how to get at that constant using GI19:00
alecunessita, just wondering about that too19:01
alecunessita, probably dobey knows ^19:01
nessitadobey: 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
alecugatox, lol at the XX guy19:02
dobeyGLib.SpawnFlags.DO_NOT_REAP_CHILD19:02
gatoxejejejeje19:02
dobeyprobably19:02
ryeNOTICE: we are currently investigating the failures for mobile uploads and file downloads from the web.19:03
gatoxi'll be offline for  a few minutes.... i'm doing some network detection tests on linux19:03
kpokverterok, http://paste.ubuntu.com/802059/ <--- new log19:03
nessitadobey: oh, that simple19:03
nessitaalecu: ^19:03
nessitadobey: thanks!19:03
ralsinanessita: if we want to implement to --switch-to handling in linux, it's not all that hard19:04
nessitaralsina: hum?19:04
nessitaralsina: it already works in both OSes19:05
ralsinanessita: well, on windows, if there is a u1cp running, starting a second one makes the first one restore its window19:05
ralsinanessita: and it doesn't switch to the right page19:05
verterokkpok: thanks, what's the output of u1sdtool --current-uploads19:05
nessitaralsina: ah... is ok. This is juts a proof of concepts for sso... so no need for now19:06
ralsinanessita: ack19:06
gatoxyeyyyyy...... i've network detection is working under windows and linux..... in a more "multiplatform" way :P19:06
gatoxalecu, question19:06
gatoxalecu, it's a dbus question...... so, i assume you might know19:07
gatoxalecu, 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
gatoxremoving the signal  avoid to listen to future network state changes19:08
verterokfacundobatista: hi, kpok client seems to be stuck: http://paste.ubuntu.com/802059/ any ideas?19:09
facundobatistaverterok, why stuck?19:11
verterokfacundobatista: sorry, one upload is stuck19:12
verterokfacundobatista: previous run: http://paste.ubuntu.com/802032/19:12
facundobatistaverterok, IOError: [Errno 28] No space left on device19:13
verterokfacundobatista: yes, so I told him to restart the client19:13
verterok:)19:13
verterokfacundobatista: but now isn't even attempting to upload the file :/19:13
verterokfacundobatista: it got a bunch of disconnects, and then nothing19:14
alecugatox, looking19:16
facundobatistaverterok, how do you know? that upload may be one of the two queued ops19:16
verterokfacundobatista: there is nothing in the logs19:17
facundobatista2012-01-13 00:26:09,119 - ubuntuone.SyncDaemon.Main - NOTE - ---- MARK (state: <State: 'QUEUE_MANAGER'  (queues WORKING  connection 'With User With Network')>; queue: 2; hash: 0) ----19:17
facundobatistaverterok, ^19:17
alecugatox, that seems to be the way the upper layer expects to be called19:17
verterokfacundobatista: yes, but no Upload, nor nothing19:17
alecugatox, so yes, we could change it to *not* remove the signal handler19:17
facundobatistaverterok, yes, look at line 3819:18
alecugatox, but we should be extra careful that the upper layers are ok with that layer calling the result_cb more than once.19:18
gatoxalecu, yes, i remove it and made a couple of tests.... and network detection is working fine without the removal19:18
alecu*extra*19:18
gatoxalecu, roger that!19:18
alecugatox, make sure this not also used from syncdaemon, control panel and above.19:18
gatoxalecu, ok19:19
verterokfacundobatista: yes, a warning that it was disconnected19:19
alecugatox, and also the windows.py will need to be changed too.19:19
facundobatistaverterok, I mean, the upload is there!19:19
verterokfacundobatista: yes, it's there...but how do we know it's "running"19:19
gatoxalecu, 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 used19:20
facundobatistaverterok, it's not running, client needs to get to QUEUE_MANAGER state19:20
facundobatistaverterok, do you have more logs?19:20
verterokfacundobatista: nope, that's all. I'm waiting to get u1sdtool --current-transfers output19:21
facundobatistaverterok, we should have more logs, at least MARKs showing the state19:21
verterokfacundobatista: this log is fresh just after the client restart19:22
kpokverterok, http://paste.ubuntu.com/802095/ <--- for u1sdtool --current-transfers19:22
verterokfacundobatista: nm, it's working19:22
verterokkpok: ok, so it's uploading19:23
verterokit will probably take a while19:23
kpokits not.. its stopped right now.. :(19:23
kpokI just keep getting  libnotify popups that its uploading19:23
kpokthis is probably the fourth or fifth time I ve got a popup19:24
verterokkpok: please pastebin the logs again19:24
kpokokay sure19:24
verterokkpok: you migth got disconnected from the server, it will resume uploading as soon it's connected again19:25
kpokisnt there something wrong if it keeps disconnecting every few minutes? Shouldnt there be a steady upload?19:25
dobeykpok: yes, something is wrong19:27
kpoklatest pastebin ---> http://paste.ubuntu.com/802106/19:28
kpoklog19:28
verterokfacundobatista: what does this mean: "No Pong response, disconnecting the client" ?19:29
verterokfacundobatista: the server didn't replied on time?19:29
facundobatistaverterok, exactly19:29
verterokkpok: yes, the server is having some problems, but should upload steady as soon it gets fixed.19:30
kpokokay I can wait .. thanks everyone for the time and effort. :)19:31
kpokI ll come back here if it doesnt get resolved in 24 hours.. is that okay?19:32
verteroksure19:32
kpokokay later19:33
kpokcheers19:33
dobeyralsina, nessita: any good reason for me to not convert rhythmbox-ubuntuone over to autotools rather than distutils?19:49
ralsinadobey: why would you?19:49
ralsinadobey: is it broken?19:49
dobeyralsina: so that we don't have to use hardcoded paths in setup.py, i can translate settings schemas properly, and whatnot19:50
dobeyor i'll just get rid of this setting anyway19:51
ralsinadobey: if it's useless, why not19:51
ralsinadobey: our autotools skill level as a team is too low, is my main concern19:52
dobeywell, it's not totally useless19:52
dobeybut i do hate gsettings19:52
dobeyit's the first-run setting key, so we can select the u1 source by default19:53
dobeyalthough certainly not going to be usable in this first run19:53
dobeyso yeah i'll just delete it for now19:53
ralsinaok19:54
dobeyneed to take a break though. bit of rsi :-/19:58
gatoxeod for me!! see you tomorrow people20:02
ralsinabye gatox!20:02
gatoxralsina, bye20:02
=== gatox is now known as gatox_away
=== JanC_ is now known as JanC
dobeymaybe a good time to do the upgrade to precise20:16
ralsinaEOD for me. Have a nice evening!20:37
dobeycheers ralsina20:39
dobeyabout 2 hours remating for upgrade :-/20:40
nessitaok, I'm off20:56
nessitabye all!20:56
dobeywhoot21:36
dobeyhave a clean ./run-tests in rhythmbox-ubuntuone now21:37
dobeycrikey22:22
dobeywhy is firefox using half a core22:22
dobeyand what is up with my fonts22:22
dobeyman firefox is crashy now :(22:32
=== gatox_away is now known as gatox

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