/srv/irclogs.ubuntu.com/2011/06/20/#ubuntuone.txt

=== m_conley is now known as m_conley_away
duanedesignmorning alll11:38
karniyo my man, duanedesign !11:39
* mandel walking dog12:03
nessitahello everyone!12:22
duanedesignhello nessita12:23
nessitahi duanedesign!12:23
* fagan break12:27
karnihi nessita !12:27
faganhey nessita12:28
* nessita waves12:28
nessitamandel: ping12:39
nessitamandel: can you please fix what alecu asked in https://code.launchpad.net/~mandel/ubuntu-sso-client/use_txnamedpipes/+merge/61935 ?12:39
nessitafagan: have you seen mandel around?12:42
mandelnessita: sure12:47
mandelnessita: I guess ou have not backlog, said I went to walk the dog :P12:47
nessitamandel: no, I have not, sorry :-)12:48
nessitamandel: how's the dog?12:48
mandelnessita: noisy, smelly and dirty, so excellent well heheh12:49
nessitahehh12:49
jussiHrm, I dont suppose anyone know's if joshua hoover wil be on IRC at some stage?12:52
faganwas it fathers day in other countries yesterday?12:52
jussiNot here.12:53
nessitafagan: yes12:53
nessitajussi: yes, he will be here, but in about 2 hours12:53
jussinessita: excellent. :)12:53
faganjussi: anything we can help with12:53
mandelnessita: crazy question, do you know what is scintilla?12:53
nessitajussi: or maybe less, 1 hour. His nickname is joshuahoover12:53
nessitamandel: in which context?12:53
mandelnessita: I had to patch pywin32 for our project and I have a missing lib… and I've never heard of it12:54
nessitahttp://www.scintilla.org/ScintillaRelated.html ?12:54
nessitamandel: can I see the error?12:54
mandelnessita: it just somplains about a missing dll during a move done in the setup.py of pywin3212:57
nessitamandel: can I see the error?12:57
nessita:-D12:57
mandelnessita: sure, let me pastebin it12:58
nessitamandel: let me know when your branch if fixed, so I also review. Also, you could please review alecu's branch? https://code.launchpad.net/~alecu/txnamedpipes/add_qt_integration/+merge/6507213:00
mandelnessita: ofcourse13:01
nessitamandel: and I would like to have a quick talk with you in mumble, if possible13:03
mandelnessita: can we do it as soon as I finished with what I'm doing right now, I wanna avoid the context switch13:04
nessitamandel: sure13:04
mandelnessita: what is our policy if we have to patch pywin32, shall I add the compiled bin in the tree or add the diff of the patch13:04
nessitamandel: may I ask what is pywin32? :-)13:04
mandelnessita: I hope the patch will be accepted (is adding a W at the end of a method) but it wont be done in time13:04
mandelnessita: is the python extensions that allow to access the win api13:05
mandellike win32security etc..13:05
nessitaright13:05
nessitaso, did you fill a bug and attach the patch?13:05
mandelnessita: I just fnished running the tests and they pass, so yeah :)13:06
mandelnessita: but wont be ready on time for us, that I'm sure...13:07
nessitamandel: I know, but we need to track this upstream. Do you have the link to the bug? in the mean time, I would advice you send the pywin32 version to us by email, and then we may need to distribute that in our installer13:07
mandelnessita: ok, doing that first then we can have the mumble13:08
nessitasure13:08
faganOk I think ill look into u1 in O and see what I can fix. I know there is something with SSO and bindwood wasnt working for me which might be a couch server side thing so ill have a look at both and see what I can find out.13:16
faganI looked at the bug list and nothing big jumped out at me13:17
faganI dont think anyone else is looking at either issue so ill fiddle about and see13:18
mandelnessita: there is just a single module to be changed in the pywin32 exetensions, so I'll just send that one rather than the full compiled bin13:18
mandelnessita: also, do you know if there is a way to link a bug from tracker in sourceforge to lp? or to import an hg branch to an lp one so that we have something in a +junk branch just in case?13:19
mandelnessita: is this enough for upstream: https://sourceforge.net/tracker/index.php?func=detail&aid=3323058&group_id=78018&atid=551954#13:20
ubot4mandel: Error: <Bugtracker.plugin.Sourceforge instance at 0x2b46ea8> bug 3323058 not found13:20
nessitamandel: I'm not sure about bug linking13:20
nessitamandel: I think there is a way, but you should ask in our internal channel13:21
mandelnessita: ok on it13:21
nessitamandel: do not forget to attach your patch to the bug report13:27
mandelnessita: upstream one? yes, that has been done :)13:27
nessitagreat!13:27
nessitamandel: I'm ready in mumble when you are13:30
mandelnessita: ok, one sec, I need to push a branch with the fix before I forget13:30
nessitamandel: sure!13:30
mandelnessita: on my way13:33
nessitaok, I'll make some mate before the stand up13:52
* nessita brbs13:52
mandelnessita: udf.accentors, are those suggested paths (that is server realted ones) or local paths???13:53
nessitamandel: if I recall correctly, local paths13:57
nessitayes, local paths13:57
mandelnessita: I'm asking facundo to be 100% sure13:58
nessitamandel: verterok is the guy to ask, but he may be off due to our National Holiday13:58
nessitaalecu, dobey, thisfred, fagan, mandel: stand up in 1'13:59
faganye13:59
alecuhellooooooo!13:59
fagannessita: ralsina not around today?13:59
faganor is he still on hols13:59
nessitafagan: nopes, Flag Day in Argentina13:59
faganahh13:59
nessitafagan: alecu and me are swapping13:59
fagannessita: that explains it14:00
fagan:)14:00
faganme14:00
alecuwe love to work.14:00
nessitaeverything is so quite around here... there is 0 car in the street14:00
faganalecu: dont we all14:00
nessitame14:00
nessitado we?14:00
nessita:-P14:00
nessitahi alecu!14:00
nessitamandel, alecu, dobey: please say me14:00
alecuI'll do it as soon as I'm finished writting notes14:01
fagannessita: well I dont think id like it any more other than if my job was drinking and chasing women14:01
=== m_conley_away is now known as m_conley
nessitafagan: seriously? I think you've proven yourself that the drinking is not your thing :-D14:01
alecume14:01
fagannessita: not enough practice14:01
nessitamandel, dobey?14:02
fagannessita: if I was doing it professionally id be a lot better14:02
dobeyme14:02
nessitafagan: go!14:02
faganDONE14:02
fagan* Looked at the bug list and didnt find anything that wasnt already fixed or was something that I could figure out easy enough to assign to myself.14:02
faganTODO14:02
fagan* Im on 11.10 and I have a few issues so ill go look into them and see if either I can fix them or I can point out whats going wrong and where so I can ask for help or what ever.14:02
faganBlocked14:02
fagan* nope14:02
fagannessita: go14:02
nessitaDONE: bug #798429, bug #798687, reviews, meetings14:02
nessitaTODO: bug #798964, bug #798413, reviews, windows port pushing14:02
nessitaBLOCKED: nopes14:02
nessitaNOTES: I need to go to university tomorrow instead of Wednesday, I'll be leaving after stand up and daily windows catch up meeting14:02
nessitaNEXT: alecu14:02
ubot4Launchpad bug 798429 in ubuntuone-control-panel "Implement functionality for adding a new cloud folder (affects: 1) (heat: 24)" [High,Fix committed] https://launchpad.net/bugs/79842914:02
ubot4Launchpad bug 798687 in ubuntuone-control-panel "Migrate credentials querying to CredentialsManagementTool (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/79868714:03
ubot4Launchpad bug 798964 in ubuntuone-control-panel "Add new folders tweaks (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/79896414:03
alecuDONE: got u1cp-qt running on windows14:03
alecuTODO: push all remaining branches so they can land14:03
alecuBLOCKED: just got up, eyes still sticky14:03
alecuNEXT: dobey14:03
ubot4Launchpad bug 798413 in ubuntuone-control-panel "Provide a method in backend to list only devices info without querying local settings from syncdaemon (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/79841314:03
dobeyλ DONE: swap day14:03
dobeyλ TODO: fix more stuff14:03
dobeyλ BLCK: None.14:03
nessitaalecu: was your face! :-)14:03
mandelme14:03
nessitawash*14:03
nessitamandel: go!14:03
mandelDONE: worked on bug #799707 which broke my sync. pywin32 has a bug in which it uses GetFileSecurity instead of  GetFilesSecurityW that means that the code won't handle utf8 chars. Got caugh is some problems with the landlady and sorted it out.14:03
mandelTODO: loads... push missing promissed proposals. Fix us_txnamedpipes in sso client. land fix fo 799707.14:03
mandelBLOCKED: unless some other stupid life event happens, no :)14:03
ubot4Launchpad bug 799707 in ubuntuone-client "The windows os_helper does no have support for illegal windows paths (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/79970714:03
alecunessita, will do!14:03
nessitacomments anyone?14:04
thisfredme14:04
thisfredDONE + TODO: more refinement of solution to bug #779851 / reviews BLOCKED: no14:04
ubot4Launchpad bug 779851 in ubuntuone-client "Ubuntu One's Unity progress bar is uninformative when transferring a single file (affects: 2) (dups: 1) (heat: 18)" [Wishlist,In progress] https://launchpad.net/bugs/77985114:04
nessitathisfred: GO!14:04
thisfredsry, was making coffee, had no idea of the time14:04
nessitathisfred: we forgive you14:04
nessita(I think(14:04
faganthisfred: you werent that late :)14:04
thisfredwhew!14:04
nessita))14:04
nessitaok, seems like eom to me14:04
faganok on to debugging14:05
nessitamandel: how long till you get lunch?14:05
mandelnessita: if you wanna mumble I can wait14:05
nessitaalecu, Chipaca, mandel: mumble in 10 min? (ie 10:15ART)14:05
thisfredLIKED: outloud.fm over turntable.fm in the end, it's just way less hassle to use14:06
* alecu is wearing a new set of clean eyes14:06
alecunessita, ack14:06
* nessita will grab some abrigo, she's freezing14:06
thisfredalecu: better not click any links mandel sends you in that case ;)14:07
alecujuas!14:07
mandelnessita: I'll heat the pasta and will be back for mumble14:10
* nessita is wearing 2 pants now, and tons of coats14:12
mterryChipaca, hello!  I remember you saying you were working on drumming up a designer to review deja-dup.  Any progress?  Ideally I'd like to see the review happen this week14:13
karnirye: Do you have your U1 gallery script url handy?14:13
ryekarni, umm, should be on the blog14:14
karnirye: got it!14:14
karnithanks14:14
ryekarni, lp:~rye/+junk/ubuntuone-galleria14:14
karnihttp://blog.rtg.in.ua/2010/12/ubuntu-one-gallery.html14:14
karnitnx14:15
ryei should definitely post something14:15
duanedesignrye: do you know of a bug report for the issue that prevents resuming downloads of files greater then 25mb?14:45
ryeduanedesign, ummm argh, i am not sure i have filed one and most likely i haven't, eventually the file will be saved, but for large files (e.g. 50+ Mbs) the process can stall completely14:47
duanedesigni have a bug report from a user that has that isue14:48
duanedesign bug 79687714:48
ubot4Launchpad bug 796877 in ubuntuone-client "ubuntu one not syncing some files (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/79687714:48
duanedesignwas just curious if their was a report i could dupe it against14:49
mandelnessita, alecu: if you can take a look at this it would be appreciated: https://code.launchpad.net/~mandel/ubuntuone-client/illegal_windows_chars/+merge/6519514:51
* mandel moves to fix the comment from ubuntu-sso-client/use_txnamedpipes14:51
alecumandel, ack14:52
nessitamandel: ack14:55
mandelalecu, nessita: https://code.launchpad.net/~mandel/ubuntu-sso-client/use_txnamedpipes/+merge/61935 has been updated and the unused var is gone14:55
nessitaack14:56
* mandel moves to fix file system notifications14:56
alecumandel, after that, see if you can land https://code.launchpad.net/~alecu/txnamedpipes/add_qt_integration/+merge/6507214:56
mandelalecu: maybe before :)14:56
mandelalecu: just pushed the merged version to lp:txnamedpipes14:58
alecumandel, thanks.14:59
nessitamandel: there is no tarmac running for txnamedpipes?15:01
nessitamandel: easy to fix needs fixing in https://code.launchpad.net/~mandel/ubuntu-sso-client/use_txnamedpipes/+merge/6193515:01
mandelnessita: on it15:02
nessitajussi: you still waiting for joshuahoover?15:03
nessitajussi: I just saw an email from him letting us know that he won't be around today, but he will tomorrow. Can I help you somehow?15:03
mandelnessita: just pushed an update to use_txnamedpipes without the unused imports15:04
nessitamandel: thanks! questions regarding the other branch:15:04
nessitamandel: was it intented that the class OSWrapperTests(BaseTwistedTestCase): disappeared?15:05
mandelnessita: yes, since with the changes all the test in os_helper are enough since the test the same code in a more balck box manner15:06
nessitaok15:06
mandelnessita: know we run the tests on windows twice, with legal paths and with illegal ones15:06
nessitamandel: how do we do that?15:06
mandelnessita: the special tests in test_os_helper for the platform. the have the same logic but use illegal paths, I wanted to use the same, but you cannot be doing ope(test_file) where test_file is illegal, you have to ensure that you use the legal path to create it15:07
mandelnessita: that ^ would in the case were we are testing a method that acts on an existing file (ex: rename)15:07
nessitaI see15:07
nessitamandel: another question:15:08
nessitahow come test_path_file_exist_yes pass if you're not creating the file?15:08
nessitamandel: nevermind15:08
nessitaI just saw the open() in setUp :-)15:08
mandel:)15:08
dobeyla la la15:17
nessitamandel: some needs fixing in the u1client branch15:22
thisfreddobey: I changed some things about my progress bar branch, because I realized we could make it more accurate even. Since it changed I would love it if you could rereview. We apologize for the inconvenience. https://code.launchpad.net/~thisfred/ubuntuone-client/better-progress-bar/+merge/6488715:28
mandelnessita: looking15:28
dobeythisfred: i shall grant you this one favor. and there will be a time i shall come to collect the debt. </godfather>15:29
thisfredIf I ever reproduce, you can have my firstborn15:30
faganthisfred: the firstborn is always the good one you dont want to give that one up15:33
fagango for the second one15:33
thisfredwell, I intend for it to remain academic15:34
faganthisfred: hah15:34
nessitafacundobatista: ping15:35
facundobatistanessita, pong15:36
nessitafacundobatista: any clues regarding bug #798979? it has several dups15:36
ubot4Launchpad bug 798979 in ubuntuone-client (Ubuntu Oneiric) (and 2 other projects) "ubuntuone-syncdaemon crashed with KeyError in add_watch(): '/home/caravena/Ubuntu One' (affects: 6) (dups: 6) (heat: 56)" [High,Triaged] https://launchpad.net/bugs/79897915:36
alecumandel, more needs fixing15:37
mandelalecu: ok, looking15:38
facundobatistanessita, we'd need debug logs15:38
* fagan break 15:39
facundobatistanessita, but it seems that inotify is not being able to add a watch to the "Ubuntu One" dir15:39
facundobatistanessita, why would that happen? don't know15:39
nessitafacundobatista: can you (or any otehr chicharrero) request debug logs and do some follow up, please?15:42
nessitaother*15:42
nessitamandel: more needs fixing on ubuntu-sso-client/use_txnamedpipes/+merge/6193515:44
mandelnessita: ok15:45
facundobatistanessita, is it only happening in oneiric?15:46
dobeyoh15:47
dobeynessita, facundobatista: there is a new pyinotify in oneiric, btw15:47
facundobatistadobey, with "new" you mean just a new version, or a "new and shiny and we broke all apis"?15:48
dobeyfacundobatista: version 0.9.1 which had some changes, so that API /may/ be broken15:49
facundobatistadobey, nessita, is the ubuntuone-client running on *any* Oneiric?15:49
nessitafacundobatista: no idea where is happening, that's why I ask you (u1foundations) to do some follow up15:50
nessitafacundobatista: nopes on desktop end (yet)15:50
dobeyfacundobatista: i don't know for sure. the NM stuff should be causing breakage anyway15:51
facundobatistadobey, what NM stuff?15:51
nessitadobey: but mterry branch landed...15:52
dobeyfacundobatista: NM 0.9 broke API, so everywhere we use NM dbus API to check for connectivity, it thinks we're offline15:52
nessitadobey: so no breakage there, right?15:52
dobeynessita: in trunk, not in oneiric proper15:52
facundobatistadobey, ok, thanks15:52
dobeynessita: and this bug report lists the oneiric proper version15:53
dobeywhich reminds me, that i need to make a release of sso15:53
nessitaright. Anyway, the trace is very pyinotify specific15:53
dobeyhrmm, actually we should fix the other txsecrets issue first too :-/15:53
dobeyoh15:54
dobeyfacundobatista: but syncdaemon will still run, just never connect to the server. so it will still create watches15:54
nessitamandel: any branch ready for re-review?15:54
dobeyfacundobatista: and with new pyinotify, i guess this failure comes up :)15:54
mandelnessita: no, I'm stuck with an error when doing one of the changes you request for u1lient15:55
nessitamandel: ah, ok. Let me know, I'll start a branch of my own15:55
nessita(wanted to see if I should wait to do my context swicth)15:55
dobeythisfred: ok, so "pydoc sum" seems to be entirely useless for me understanding what sum expects as arguments, and what it returns exactly16:02
thisfreddobey: expects a list, returns sum of values in list16:03
thisfredwell, any collection would work, I expect16:03
dobeythisfred: well i understand the metaphorical description of what it does. i was looking for something a bit more technical though16:03
thisfredsum=0; for i in argument: sum += i; return sum16:04
dobeydoes it return a float always, an int always, or sometimes one or the other depending on content of the sequence?16:05
thisfredthough I'm not sure that's how it's implemented (this would work on generators, I'm not sure that sum does)16:05
thisfreddobey:  the latter16:05
thisfreddobey that's just the behavior of + though16:05
thisfrednothing to do with sum16:05
thisfred>>> 0.1 + 116:06
thisfred1.116:06
thisfred>>> 1 + 116:06
thisfred216:06
thisfred>>>16:06
dobeywell, could do return float(result) or something16:06
thisfredwhy though?16:06
thisfredthat would be really unexpected16:07
dobeydude. it's python. everything it does is unexpected afaict.16:07
dobeyanyway16:08
dobeyoh16:08
dobeyyour code is just hard to read because python is flippantly insane16:08
dobey:-/16:09
alecumandel, nessita: in about 20' I'll be afk for about one hour. (returning home from the in-laws)16:09
mandelok16:09
nessitaalecu: ack16:09
dobeystupid python16:11
thisfreddobey: what's the problem? I'll gladly make my code easier to read16:13
mandeldobey: that is what she said16:13
facundobatistadobey, what would you expect sum to do? always return a float, so lose the unbound behaviour of ints in large sums, or always return an int, truncating your floats?16:14
dobeythisfred: the progress = float(\n bit. i had to look extra hard to see that it was really only the left-of-divisor being converted to float16:16
facundobatistadobey, also, "floats" and "ints" are only part of the numbers that Python handles... what if you sum complex numbers, or fractions?16:16
thisfreddobey: I can break the line at a different point I guess16:16
dobeyfacundobatista: i would expect the documentation to be more explicit about what it actually does16:16
facundobatistadobey, well, it's quite explicit: "it returns the sum of a sequence numbers" <-- it does *exactly* that16:17
=== clarita_ is now known as clarita
dobeyfacundobatista: i will agree to disagree.16:19
facundobatistadobey, ok16:19
nessitamandel: how is it going? need help?16:46
mandelnessita: no, nearly done, I found a stupid bug thaks to your review :)16:46
mandelwas fising it16:46
nessita:-D16:46
nessitagreat!!!16:46
dobeylunch time, bbiab16:50
nessitathisfred: when you have a moment, could you please update https://bugs.launchpad.net/ubuntuone-control-panel/+bug/781875? (ping chad if needed, please)17:29
ubot4Launchpad bug 781875 in ubuntuone-control-panel "ERROR - ReplicationSettingsChangeError: args (<ubuntuone.controlpanel.dbus_service.ControlPanelBackend at /preferences (affects: 1) (heat: 5)" [Undecided,Confirmed]17:29
nessitamandel: ping17:30
nessitamandel: as far as I know, this bug #797258 is already fixed? if so, could you please link the branch and update status?17:30
ubot4Launchpad bug 797258 in ubuntuone-control-panel "The correct reactor is not used depending on the platform (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/79725817:30
thisfrednessita: will do17:31
nessitathisfred: thanks!17:31
mandelnessita: going17:31
nessitamandel: also, remember to use the --fixes=lp:<bug number> when submitting branches, so when the branch is scanned is attached to the bug report, and when the branch lands the bug report is marked as Fix Committed17:32
nessitamandel: another question, do you remember what was the issue with pylint when having this issue with pyqt? http://pastebin.ubuntu.com/629907/17:34
mandelnessita: agg anoying, I remember the exception, there was an issue in the order of the imports, I think it was related with QtCore and QtNetwork. It was fixed by importing them in a diff order17:35
mandelnessita: those it ring a bell?17:35
nessitamandel: ack, thanks!17:35
nessitayes, thanks17:35
nessitamandel: shall I get lunch or your branches are ready for re-review?17:37
mandelnessita: get lunch, will be ready in a few mins17:37
nessitaack17:37
* nessita -> lunch!17:38
* fagan EOD17:39
mandelnessita: in which project is bug irc://irc.freenode.net:6667/#797258 17:41
mandelor hal where is #irc://irc.freenode.net:6667/#797258 17:41
mandeloh, stupid me I have to say bug #irc://irc.freenode.net:6667/#797258 lame...17:42
thisfredwhat irc client are *you* using :)17:42
mandelthisfred: why?17:42
thisfredmandel because it prefixes everything you start with a '#' with  'irc://irc.freenode.net:6667/'17:42
mandelreally? wtf adium!!17:43
thisfredwhich is why ubottu is not picking it up ;)17:43
mandelthisfred: can you tell him to do it… I cannot believe I'm ignored by a bot17:43
thisfredbug #797258 17:44
ubot4Launchpad bug 797258 in ubuntuone-control-panel "The correct reactor is not used depending on the platform (affects: 1) (heat: 6)" [High,New] https://launchpad.net/bugs/79725817:44
mandelthisfred: txh!17:44
thisfredmaybe every bug *should* have its own irc channel, much like it has its own email address :)17:44
alecuback17:46
dobeythisfred: care to follow up on https://code.launchpad.net/~thisfred/ubuntuone-client/i18n-test-strings/+merge/63982 ?17:49
mandelalecu, nessita: feel free to take a second look at https://code.launchpad.net/~mandel/ubuntuone-client/illegal_windows_chars/+merge/6519517:51
nessitamandel: sure17:53
mandelalecu: your comment about using a method for the exception checking was great, the code looks a lot nicer :)17:54
alecumandel, :-) re-re now.17:54
=== beuno-honeymoon is now known as beuno
karninessita: Hi hi :) \o17:55
karninessita: Can you tell an example for 2 errors considering one field ('email') as noted by you here https://bugs.launchpad.net/canonical-identity-provider/+bug/62495517:55
ubot4Launchpad bug 624955 in canonical-isd-qa-old (and 1 other project) ""Email already registered" error is not being sent inside a list (affects: 1)" [Undecided,New]17:55
thisfreddobey: ah yes, sry, that was still waiting somewhere in my inbox>017:55
nessitakarni: I'm not sure what you're asking :-)17:56
mandelalecu, nessita: I'm away for 30 min, need to walk the dog17:56
nessitamandel: ack17:56
karninessita: Sorry. You wrote in the bug report, that we expect such a reponse: {'status': 'error', 'errors': {'email': ['error1', 'error2']}}17:57
karninessita: Can you give me an example of 'error1' and 'error2' ?17:57
karninessita: I'm wondering why is it a list, and not a simple string value.17:57
karninessita: Also, I was a little surprized when I saw a python-like json key '__all__' (see first comment in that bug report)17:58
nessitakarni: the error report aims to unify the error reporting for any call, and maybe not for email but for other fields you may have more than one error17:58
nessitakarni: so, the report was born when I noticed that some fields will send a list of errors, and some others will send a single string17:59
karninessita: I see, that makes sense.17:59
alecumandel, there are still some instances of the old == 123 check17:59
karninessita: Thank you!17:59
nessitakarni: I don't recall any specific error for email other than already registered :-)17:59
nessitayou're welcome!17:59
karni:)17:59
alecumandel, lines 459 and 638 in the diff.18:00
nessitamandel: also, you didn't apply the suggestion of using os.path.splitdrive instead of all the custom logic?18:01
nessitamandel: needs fixings added to the branch18:05
nessitanow for real, lunchtime!18:10
karninessita: So, the '__all__' JSON key is a catch phrase for any other errors?18:21
nessitakarni: is the key to signal errors at form level18:24
nessitakarni: not specific to any field18:25
* nessita is back18:25
karninessita: Cool :)18:26
* mandel back18:27
mandelalecu: are those isntances exactly the same, I was very careful not to ver generalize for a reason18:28
mandelnessita: I must have overread the split drive comment, will sort it out18:28
nessitamandel: thanks, also please fix what I mentioned about logger.exception18:29
alecumandel, I don't care if you use the same function; what I meant is that there are two places with "if e.winerror and e.winerror == 123" instead of """if getattr(e, "winerror", None) == 123"""18:29
mandelalecu: oh, that! I understood it was not using the same function…18:30
alecunp18:30
* alecu is having a late lunch18:30
mandelnessita: use_txnamedpipes should not longer have pep8 issues just pushed it (r725)18:39
* mandel hates all this multiplatform things at times… is so hard to remember to check in all platfroms18:40
nessitamandel: ack!18:40
mandelnessita: if you give a +1 can you do me th favour of approving the  branch too, I might forget later18:46
nessitamandel: which branch18:46
nessita?18:46
nessitasso +txn?18:47
mandelnessita: yes, sso + txn18:47
nessitaon it right now18:47
nessitamandel: done, also I set the proposal to Approved18:49
mandelnessita: you rule!18:49
nessitalol18:49
nessitaChipaca: ping18:50
nessitaChipaca: unping18:50
nessitamandel: which is the fastest way to implement a throbber (spinner) in QT?18:51
nessitamandel: this is what I'm referring to http://developer.gnome.org/gtk/2.24/GtkSpinner.html18:52
* mandel looks18:52
nessitamandel: a solution without depending on an external animated gif will be the ideal18:52
mandelnessita: haha I was going to propose that one :P18:53
nessitasince for that we will need the animated spinner to be provided by the design crowd18:53
nessitamandel: and also, the spinner will be to be customized to each platform...18:53
nessitato look native, I mean18:53
nessitamandel: what about building a progressbar that does not show progress but only a piece that goes from left to right until the info is loaded?18:54
nessita(not sure if I made myself clear)18:54
mandelnessita: yes.. well usually you have to use a QMovie and a gif18:55
nessitamandel: can't the progressbar idea be implemented without that?18:55
nessitain GTK is what is called "activity" mode18:55
mandelnessita: you can use a progressbar and set it to be loading, that is easy18:56
nessitabecause a progress bar has  different look depending on the OS as well18:56
nessitamandel: what's the method name? I'm looking to http://doc.qt.nokia.com/latest/qprogressbar.html18:56
mandelnessita: ok, let me find that for you18:57
nessitaso, how can I set a progressbar to be in loading/activity mode without setting explict % progress?18:57
mandelone min18:57
nessitasure!18:57
nessitamandel: ah! http://old.nabble.com/ProgressBar---Activity-Mode-td29152774.html18:58
nessita"Just set the minimum and maximum to 0 and you'll get exactly what you18:58
nessitawant. "18:58
nessitagreat! thanks!18:58
nessita:-P18:58
mandelnessita: ha, and why dont they put that in the docs?!18:58
mandelbastards18:59
nessitaright!18:59
mandelnessita, alecu: please https://code.launchpad.net/~mandel/ubuntuone-client/illegal_windows_chars/+merge/6519519:18
nessitamandel: in 5 minutes19:18
mandelif I forgot something is clearly telling me that I should be done for the day :P19:18
nessitamandel: is this correct? os.chmod(fixed_path, 0o664)19:24
nessitaseems like there is an extra o there?19:24
nessitasame for os.chmod(fixed_path, 0o444)19:24
nessitamandel: let me know when you're here, seems like we're not understanding eachother re os.path.splitdrive19:25
nessita:-(19:25
mandelnessita: I did use os.path.splitdrive19:26
nessita    if os.path.isabs(path):19:26
nessita        # we need to remove the volume letter, thee reason is that we19:26
nessita        # will have issues with it and the separator19:26
nessita        partial_paths = os.path.splitdrive(path)[1].split('\\')19:26
nessita        partial_paths = partial_paths[1:]19:26
nessita        path = os.path.join(*partial_paths)19:26
nessitamandel: did you in _is_illegal_path(path)?19:26
mandelnessita: ha, I used it wrong, I should fix that and call it a day19:28
mandelnessita: regarding the extra o, that are the exact same values found in the os_helper from linux19:29
nessitamandel: is there any chance you propose the SyncDaemonTool branches today? Meaning, are they proposable without any extra work on them?19:29
mandelnessita: if I propose them we will have merge issues with the fixes regarding the notifications, from exp I prefer to propose that one first and then sd tool19:30
mandelnessita: I can always go to the gym swim a little and get back to finish those19:31
thisfrednessita: mandel: the 0o is because of forward compatibility, I think: http://www.python.org/dev/peps/pep-3127/19:31
nessitamandel: I'm worried that we're 3 days behind with the SDT, and I can guess there will be some fixing needed to those branches, so I was hoping that alecu and me could review those while you were gone, so you get feedback as soon as possible19:31
nessitamandel: can't you propose the branches setting the proper prerequisites?19:32
mandelnessita: yes, I can create a pipeline so that if there are fixes to commit they will be done, but before that I'll like to at least finish the current fix I'm in, otherwhise I'll screw it up later19:33
nessitamandel: ok19:33
nessitamandel: you let me know when the splidrive fix is ready?19:33
mandelnessita: running the tests right now19:33
nessitathanks!19:34
karninessita: Quick question. Failure on validate_email returns {'errors': {'email_token': ["Bad email token!"]}} -- should it not go with {'status':'error', 'errors' : .... } pattern? I'm asking you, because achuni is already gone from IRC :(19:38
karninessita: It feels inconsistent.19:39
nessitakarni: very likely to be an inconsistency19:39
karninessita: Smilarily, instead of returning {'status':'ok', 'email':'foo@bar.com'} it returns on success {'email':'foo@bar.com} :(19:40
nessitayeap19:40
karninessita: Ok, thanks. I'll take that to ids19:40
nessitakarni: that is why we need a 2.0 API :-)19:40
karni*isd19:40
nessitakarni: thanks!19:40
karninessita: Agreed!! :)19:40
mandelnessita: os.path.splitdrive pushed19:45
nessitamandel: ack, looking19:48
pmatuliswould someone be so kind as to demonstrate how to file sync with u1sdtool?  i do '--create-foler' and it just hangs forever19:48
nessitapmatulis: are you running u1sdtool within an X session?19:48
pmatulisnessita: in a terminal, so i guess yeah19:49
nessitapmatulis: a graphical terminal, right?19:49
nessitapmatulis: if so, what does u1sdtool -s reports?19:50
pmatulisnessita: sure, terminator19:50
pmatulisState: READY19:50
pmatulisqueues: WORKING19:50
pmatulisnessita: i have since stopped the create-folder comand19:51
nessitapmatulis: so, your syncdaemon is not connected. You should first do: u1sdtool -c19:51
nessitapmatulis: --create-folder blocks until the command is finished, and since your syncdaemon is not connected to the cloud, the command will not finish19:51
pmatulisnessita: i did that before but i closed the resulting window.  i thought that was for adding a computer19:52
pmatulisnessita: or an account19:52
pmatulisnessita: i have already added my computer via the web ui19:52
nessitapmatulis: nopes, -c is "connect". It will prompt for SSO account details if there aren't any in your keyring19:52
pmatulisnessita: hmmm19:52
nessitapmatulis: if you have a u1client new enough, you can set the autoconnect setting to True in the conf file19:53
nessitapmatulis: u1sdtool -c should no longer read "READY"19:53
pmatulisproblem with SSO is that you tend to forget the original p/w19:54
nessitamandel: ping19:55
nessitapmatulis: the same SSO screen offers password retrieval19:55
nessitamandel: I'm about to approve, I'm running the tests now. One thing: you removed the docstring for one of the methods, can you please restore that?19:59
nessitamandel: also,19:59
nessita./ubuntuone/platform/windows/ipc.py:19:59
nessita    1246:  undefined name 'get_creds_proxy'19:59
nessitapmatulis: so, is it working now?19:59
pmatulisnessita: i think you nailed it19:59
nessitapmatulis: syncdaemon ir actually transferring from-to the cloud when its states is QUEUE_MANAGER + WORKING20:00
pmatulisnessita: my previously sync'ed folder (prior to my recent re-install) is showing up in --list-folders as is my test folder20:00
pmatulisnessita: very very nice!  i have QUEUE_MANAGER and IDLE20:01
nessitapmatulis: you're all synced up then!20:01
pmatulisnessita: so to have this kick in upon boot what do i do?  note that i'm running lubuntu20:01
dobeyit should be doing that already, assuming lubuntu follows the autostart spec20:03
nessitadobey: I think he means the autoconnect?20:04
nessitapmatulis: when your session starts, is it that syncdaemon is not running or is not connected?20:04
dobeynessita: yes, and that is on by default20:04
nessitadobey: right, let's see the version he has (autoconnect is rather new)20:05
nessitamandel: yes?20:05
pmatulisnessita: i haven't tested it yet as i just go it working20:05
pmatulisnessita: but i guess autoconnect sounds good  :D20:06
nessitapmatulis: let us know. In theory, syncdaemon should autostart as a process and should autoconnect, unless you explicitly disable that setting in the conf file20:06
pmatulisnessita: which resides where?20:07
dobeypmatulis: what version of ubuntuone-client do you have installed?20:07
pmatulis1.7+r1004-36~natty120:08
nessitapmatulis: ~/.config/ubuntuone/syncdaemon.conf (may not exist if you're running default settings)20:08
nessitapmatulis: you should get the autoconnect working then20:09
pmatulisnessita: it exists.  'on = False'20:10
nessitapmatulis: AHA20:10
nessitapmatulis: but wai20:10
nessitawait20:10
nessitapmatulis: that is throttling, by any chance?20:10
pmatulisnessita: it does mention it, yes20:11
pmatulisnessita: just 4 lines in there20:11
nessitapmatulis: can you please pastebin that file?20:11
pmatulisnessita: yes20:11
pmatulishttp://paste.ubuntu.com/629985/20:12
alecumandel, approved (minus some typos on the merge proposal)20:12
mandelnessita: I'm looking at 1246:  undefined name 'get_creds_proxy'20:14
mandelalecu: ok, I'll fix them right away20:14
nessitapmatulis: right, so your syncdaemon is indeed set to autoconnect20:14
nessitapmatulis: if it does not work, let us know20:14
pmatulisnessita: nice.  so i'll see tomorrow morning.  thanks a lot!20:14
nessita:-)20:14
mandelnessita: I'm confused, what is worng with '1246:  undefined name 'get_creds_proxy''20:15
nessitamandel: lint is complaining :-)20:15
mandelnessita: is that on linux?20:16
nessitayeap20:16
* mandel looks20:17
* mandel hates multiplatform20:17
nessitamandel: once those are fixed, you can approve globally the merge proposal20:18
mandelnessita: cool20:19
mandelnessita: which method did I remove the docstring from?20:20
nessitamandel: none, it was my mistake while reading the diff20:21
nessitaI just rechecked20:21
mandelnessita: ok, so I'm not THAT stupid hehhe20:22
nessitamandel: you're not stupid at all, you're just tired20:22
mandelnessita,alecu: will fix typos and approvethen20:22
nessitamandel: I can see why20:22
* mandel dinner, will be back shortly20:33
czajkowskimandel: walking the beast?20:39
alecunessita, mandel: https://code.launchpad.net/~alecu/ubuntuone-client/fix-credentials-defer/+merge/6526721:24
nessitaalecu: on it!!!21:24
nessitaalecu: is the CMT working with that fix?21:25
alecunessita, what's CMT? Country Music Television?21:25
nessitaheh21:25
nessitaCredentialsManagementTool21:26
dobeyalecu: exactly21:26
alecunessita, it seems to be working, but now I'm hitting a PB/jsonrpc issue I need to debug further.21:26
nessitaalecu: did you applied mandel's branch that swicths ussoc to txn?21:27
alecunessita, yes, I'm using "use_txnamedpipes" on the sso21:27
alecunessita, but my u1cp branch still seems to want to use pb21:28
nessitahum21:29
nessitathat smells21:29
nessitamandel: you eod'd already?21:29
alecuhere's the full error log: http://pastebin.ubuntu.com/630022/21:36
alecuhmm... perhaps I need to update the local txnamedpipes21:37
dobeyhave a good evening alls21:51
=== m_conley is now known as m_conley_away
nessitaalecu: running tests now21:59
nessitaalecu: keeps working perfect on linux22:08
nessitaapproved22:08
alecunessita, great. I tried it on linux, and it seems to work fine.22:08
alecunessita, btw: do you have any idea on this? http://pastebin.ubuntu.com/630033/22:09
nessitayeah, I tried it myself. No luck on windows?22:09
nessitalooking22:09
nessitaalecu: hum, I saw a branch/bug passing by, let me look that for you22:09
nessitaalecu: in any case, you need to change the import to be:22:09
nessitafrom ubuntuone.platform.credentials import CredentialsManagementTool22:10
nessitathat is the proper way of using that22:10
nessitaalecu: bug #79972222:11
ubot4Launchpad bug 799722 in ubuntuone-client "IPC on windows uses the old CredentialsManagementTool and has an import error (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/79972222:11
nessitahum, mandel linked the wrong branch to that bug report22:11
nessitaah no, he actually fixed that22:11
nessitaalecu: ^22:11
nessita318-from ubuntuone.credentials import CredentialsManagementTool22:12
alecugreat.22:12
nessita319-from ubuntuone.platform import get_creds_proxy22:12
nessita320+from ubuntuone.platform.windows import CredentialsManagementTool22:12
nessitamandel: ping22:12
nessitaI guess he eoded22:12
nessita(which makes sense)22:12
nessitaok, I'll take a quick break to get something to eat22:13
alecunessita, thanks.22:13
alecunessita, I'm hitting an error here: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc.py#L124522:28
alecunessita, this code tries to use get_creds_proxy, but it's not imported, and furthermore, it tries to register callbacks without using CMT22:29
thisfredgotta walk the dogs, bbiab if anyone needs me22:29
nessitaalecu: looking22:29
nessitaalecu: that code looks awfully broken22:32
nessitaalecu: can you fix?22:32
alecunessita, sure. Any pointers?22:32
nessitayes, let me grab/find that22:33
nessitaalecu: hum, there is no current code using CMT, but pointers are:22:35
alecunessita,  I think I should get at the "get_creds_proxy" and then call "register_to_credentials_found" there22:36
nessitaalecu: I think the get_creds_proxy should never be used22:37
nessitasee line (from that url) 126522:37
nessitathe CredentialsManagementTool is created, and the deferred returned by either management.find_credentials() or management.register({'window_id': '0'})  should be used22:37
nessitaso, no signal connection should be needed22:37
alecunessita, oh, right.22:38
nessitaalecu: from my POV, self._register_to_signals() should be removed22:38
alecunessita, both find_credentials and other cmt methods already return deferreds22:38
alecunessita, so: you are right, we should be removing signals.22:38
nessitayeap. So, the caller to self._request_token should handle the deferred success (the creds are returned)22:38
nessitaor deferred failures (something bad happened)22:38
alecunessita, great, got it now.22:38
alecuthanks!22:39
nessitaalecu: anytime! I think you got the idea, but see line 1231, the only missing bit would be to handle errors there22:39
nessitaalecu: hum, also, the body of on_credentials_found_cb should be next to token retrieval22:40
nessitayes?22:40
nessitathat should be all that's needed (along with the removal of _register_to_signals)22:40
alecugot it, cool, thanks.22:41
nessitalet me know, I'll be around since my branch still needs work :-/22:41
=== AlanChicken is now known as AlanBell
=== threeve_ is now known as threeve
pmatulisnessita: i added some stuff under the directory given by --list-folders and it did not show up on the web ui.  i'm on a separate system right now so i cannot troubleshoot23:04
nessitapmatulis: first thing to check would be if the folder is subscribed. Also, please enable debug in your logs by adding this piece to your .conf file:23:05
nessita[logging]23:05
nessitalevel = DEBUG23:05
nessitapmatulis: and then restart syncdaemon23:05
pmatulisnessita: alright, i'll do this tomorrow morning23:05
nessitapmatulis: let me know, I'll be around :-)23:06
pmatulisnessita: not sure about 'subscribed', will look23:06
nessitapmatulis: when issuing --list-folders, check the subscribed flag. Example:23:06
nessitaid=<something> subscribed=True path=/home/nessita/Documents23:06
nessitaalecu:  any luck? is the CMT integration progressing? can I help?23:37
alecunessita, I've been working on the tests, but right now I'm writing the report and EODing23:38
nessitaalecu: makes sense, I should so myself...23:39
nessitabut this freaking stuff will not work23:39
karninessita: Can we ping the sso-done-so-get-tokens before we verify the email with email token?23:45
nessitakarni: nopes, that fails23:46
karninessita: So we need that after we validate the mail, correct?23:46
nessitafails on server side, I mean, since that code depends on the preferred_email23:46
nessitakarni: yessir :-)23:46
karninessita: Roger, thank you :)23:46
nessitayou're welcome!23:46
karninessita: You're very helpful (and cheerful!). I like that :)23:47
nessitathanks! :-)23:47
nessitayou're very thankful, and I appreciate that23:47
karni^_^23:47
karninessita: Last question for today. Can we OAuth sign the ping or it needs to be Basic Authentication?23:49
karniOh it can't.. that's the point, to get the tokens from SSO xD23:49
karniRight?23:49
nessitakarni: the ping has to ping OAuth signed with NO plain text, but the HMAC encoding23:51
nessitakarni: so not only it can, but it should :-)23:51
karninessita: Oh, it's OAuth signed! Doh, I thought Basic Auth =) Yay \o/23:51
nessita:-)23:52
karninessita: I always used Plaintext singer with Ubuntu One Files. Do you happen to know if the Files API will accept the HMAC encoding as well?23:55
nessitakarni: yes, they do. If they don't, is a huge issue.23:55
karninessita: Cool! Thank you =)23:55
nessita:-)23:55

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