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

nessitadobey: approved, but with a comment to fix a wrong docstring00:45
tntcThe ubuntu one mobile player supports m4a, but doesn't grab album info and stuff?03:47
tntc(on Android)03:47
igormorgadohello! how can I mount/sync my ubuntu one storage in a computer without X/Gnome? any howtos? which package? i cannot find u1sync in 11.04. any tips?04:44
mandelmorning08:25
faganbe back in an hour need to get milk08:56
fagan(and other bits and bobs but wont be that long)08:56
mandelChipaca: ping09:50
Chipacamandel: pong09:50
mandelChipaca: do you have any clue of a mode similar to r+w to be used with os.fdopen? r+w is an illegal mode on windows..09:51
Chipacamandel: 1 sec09:51
Chipacamandel: r+w is an unspecified mode anywhere09:53
Chipacamandel: you want either r+, w+, or a+09:53
Chipacaw+ truncates the file, a+ is read-and-append. You probably want r+.09:53
mandelChipaca: ok, I wonder how has r+w has worked 'til now...09:54
Chipacamandel: massive good luck?09:54
Chipacamandel: also, it works on ubuntu :)09:54
Chipacabut it's not specified09:54
Chipacaso up to the platform coders09:54
Chipacaunix people seem to have taken the DWIM approach09:55
Chipacabah, i dunno09:55
Chipacar+w has two different possible interpretations, you see09:55
Chipacaso erroring out is quite reasonable :)09:55
mandelChipaca: yes, I suppose we meant r+ I'll change it and will run the tests to see09:55
duanedesignmorning!09:58
duanedesign.509:58
Chipaca.5?09:58
duanedesignChipaca: was supossed to be /5 :P09:59
Chipacaheh09:59
duanedesignnot awake 10 minutes and i have my first typo09:59
Chipacaduanedesign: that sounds about right10:00
Chipacamandel: r+b, right?10:06
mandelChipaca: yes, stupid EOL on windows..10:06
Chipacamandel: here: http://www.youtube.com/watch?v=ZJfZTr8zMsc10:08
mandelChipaca: que grande!10:10
karniWe broke 1000 downloads of the files app in less than 16 hours10:57
karniCurrently 1068 total installs :)10:57
=== Iroslaw_ is now known as Iroslaw
mandelkarni: well done!11:05
karnimandel: Thanks Mandel!11:05
mandelkarni: but why do you own that many phones?11:05
mandel:P11:05
karnimandel: :D11:05
duanedesignkarni: awesome \0/11:36
karniduanedesign: \o/ :)11:37
duanedesignkarni: i was just about to tweet about that11:37
karni=)11:37
* mandel at the bank will be back soon11:39
ralsinamorning!11:46
duanedesigno/11:53
ralsinahi duanedesign11:54
duanedesignhowdy11:55
duanedesigngoing to be a hot one here today11:56
duanedesign103F11:56
* mandel walks dog....12:02
nessitagood morning everyone!12:43
nessitamandel: I'm reviewing your branch12:44
mandelnessita: great12:50
nessitamandel: trivial needs fixing:12:50
nessitaDocstring should be written as a command, so these two needs fixing:12:50
nessita"""Performs the file...12:50
nessita"""Returns if an...12:50
mandelnessita: which branch?12:51
nessitahttps://code.launchpad.net/~mandel/ubuntuone-client/fix-802726/+merge/6637312:51
mandelack12:51
ralsinamrning nessita12:54
mandelralsina, nessita: can I also get reviews for https://code.launchpad.net/~mandel/ubuntuone-client/fix-803507/+merge/6641812:54
ralsinamandel: sure!12:54
nessitaralsina: hello12:54
mandelnessita: changes made12:56
ralsinanessita: I am a bit blocked because design has told me there are updates for the wireframes, because they were working directly on the visual designs, and there is no "master document" we can refer to right now. I am waiting to see what the changes are, and should have them soon.12:56
nessitaralsina: hum12:57
nessitaralsina: ideally, we should not be getting updates from design right now, we can't be changing the UI at this point... do you know what kind of changes are they?12:57
ralsinanessita: not yet. I hope it's nothing major. It was mentioned in the context of the installer, though. And since the installer doesn't yet match the wireframes...12:58
mandelyes, is kinda late for that...12:58
nessitaralsina: your work so far on the installer, was based on signed off wireframes?12:59
ralsinanessita: I had on one hand signed off texts, and on the other, the 2.5 wireframes, which I asked design whether those were the ones I should follow, ad was told yes.13:00
nessitaralsina: was that conversation via email?13:00
ralsinanessita: let me check13:00
ralsinanessita: apparently not mail. I think it was in mumble in budapest, in our room?13:02
ralsina"our" I mean my, with you present13:02
nessitaralsina: if it was, I wasn't there, I left for the airport as soon as we finish the control panel wireframes13:03
nessitaralsina: ok, we need to confirm we're working on signed off specs, otherwise is our bad to have things moving without that :-/13:03
ralsinaagreed.13:03
nessitaralsina: let's talk about that in the mumble. In the mean time, what are you working on while blocked?13:03
ralsinaI will take care of this. However, this is only about the installer, so it's not TERRIBLY hard to fix right now.13:04
ralsinanessita: can't mumble, kid asleep next room :-(13:04
nessitaralsina: in "the mumble", the one we have after the stand up :-)13:04
ralsinanessita: oh, ok13:05
ralsinaI may have something more solid by that time13:05
ralsinaWhat I can work with: there are some pretty basic things in the SSO UI that need work, that should fill a couple of hours13:05
nessitaralsina: such as?13:05
ralsinalike the error dialog shows the repr of the error dict13:06
nessitaoh, right13:06
nessitaralsina: do you have any spec about how to handle that?13:06
nessitamandel: also: Text conflict in ubuntuone/platform/windows/filesystem_notifications.py13:07
mandelralsina: and some review for me :)13:07
ralsinanessita: I thinkI have it on lisette's docs. I am looking now13:07
ralsinamandel: yes, and reviews :-)13:07
mandelnessita: hmm ok I'll take are of that/13:07
mandelha, is an empty line.. lame13:08
mandelralsina, nessita: I really do not see how I can add extra tests for this: https://code.launchpad.net/~mandel/ubuntuone-client/fix-803622/+merge/66425 it is covered by the action_q tests...13:09
nessitamandel: if you had to fix something then it was obviously not covered :-P (haven't seen the diff yet)13:09
ralsinanessita: r+w is undefined, but it works on Linux13:10
ralsinaand by "it works" I mean "it works like r+b"13:10
nessitamandel: I think you should add a test and confirm that os.fdopen was called with r+b instead r+w13:11
ralsinaSo on linux the change is a no-op13:11
nessitaralsina: right, but we need to avoid the possible regression13:12
ralsinaexactly, so patch open and see the args13:12
mandelaggg13:12
nessitamandel: os.fdopen can be patched with no further issues13:12
mandelnessita: I know.. but I'm lazy13:13
mandelanyway, conflict resolved13:13
nessitamandel: DON'T be :-)13:13
nessitamandel: trust me, everything can be tested. Chipaca taught me that. But you the right to complain about that for a while (not for toooo long)13:13
nessitayou "have"...13:14
mandelI'll do it… still a PITA13:14
ralsinamandel: for every test we write, a little kitten gets its wings or something like that.13:15
nessitamandel: can you please remove the extra line you added in platform/windows/filesystem_notifications.py, right before13:16
nessita     @longpath(paths_indexes=[1])13:16
nessita    def _path_is_dir(self, path):13:16
ralsinaOTOH, since this change is platform-dependent, we could just instantiate that class in a windows-specific test. If it passes, it's not regressed ;-)13:16
mandelnessita: ok13:17
nessitaralsina: explicit is better than implicit :-) so, having a dedicated test stating what we are about, is easy for the next one reading and understanding the code13:17
mandelnessita, ralsina: we also have the start method implemented looking at the reg to get the installation path: https://code.launchpad.net/~mandel/ubuntuone-client/sdtool_start_method/+merge/6644813:18
ralsinanessita: yes, but basically, if the tests were ran on windows, this *would* have been caught by all of them.13:18
nessitaralsina: True13:18
ralsinaSo the underlying problem was "tests didn't work on windows", not lack of coverage.13:19
mandelexactly :)13:19
nessitaralsina: yes, I agree. But when committing a fix to the repo, is even better if we have a test ensuring that the change is correct (and remains correct in the future)13:20
nessitaso, I think we all agree :-)13:20
ralsinanessita: ok, convinced :-)13:20
mandelnessita, ralsina: one of the main issue for not running the tests is bug 76051813:20
ubot4Launchpad bug 760518 in ubuntuone-client "All tests are broken on Windows when trying to remove the tests files (affects: 1) (heat: 9)" [Medium,Triaged] https://launchpad.net/bugs/76051813:20
mandelI could move to fix that one so that we can see most of the tets running and passing13:20
nessitamandel: do you have something else in your plate before that?13:21
nessitalike, any other pending bug13:21
mandelnessita: I have 4 branches for review and I think bug 802498 should be addressed first13:21
ubot4Launchpad bug 802498 in ubuntuone-client "Error when doing a partial commit on windows (affects: 1) (heat: 6)" [Critical,Triaged] https://launchpad.net/bugs/80249813:21
mandelnessita: also, I'd like to see if after those merges we start uploading files, maybe they were making the q to screw up for some reason13:22
nessitamandel: hum, so I'm not understanding what you will be working next13:23
mandelnessita: add tests for r+b and move to bug 80249813:23
ubot4Launchpad bug 802498 in ubuntuone-client "Error when doing a partial commit on windows (affects: 1) (heat: 6)" [Critical,Triaged] https://launchpad.net/bugs/80249813:23
mandelnessita: if I have time, make tests run on windows13:23
mandelor some other bug if needed13:23
nessitamandel: makes sense. I'm testing branches on windows as well (IRL), so I'm a bit slower today13:24
mandelnessita: ok, no problem when the bug fixes I have added land I'd like to do a IRL to see if we do update files or is a diff bug13:25
mandelI'm going to grab some early food, will be back in 20 min or so13:25
* mandel lunch13:25
nessitaack13:26
ralsinamandel: +1 on https://code.launchpad.net/~mandel/ubuntuone-client/fix-803507/+merge/6641813:38
=== m_conley_away is now known as m_conley
=== teknico is now known as teknico_away
nessita6 minutes to stand up!13:54
spacebug-Hello again. I still have the problems with big files not being uploaded. (from the program that is. Web upload works great). Last time I was here, my files said to have been uploaded to the server but had 0 bytes content. Now I installed the ubuntu one file app for my android yeterday and when loggin in on that I also the my big files uploaded with 0 bytes content. The u1sdtool and my ubuntyuone indicator both tells me file13:59
spacebug-is being uploaded and I see the % go up to 100%. 'u1sdtool --current-transfer' gives bytes written = deflated size. Still the folder never gets synced and transfer is never really done. Sometimes (at random it seems), moving the file out of the dir 'Ubuntu One' and restarting with 'u1sdtool -q; u1sdtool -c' seems to work, and sometimes not. Small files always work, they get uploaded and synced instantly (even during a big file13:59
spacebug-'hang')13:59
faganmoi14:00
nessitame14:00
nessitaalecu, ralsina, mandel, dobey, thisfred?14:00
ralsiname14:00
dobeyme14:00
thisfredme14:00
mandelme14:00
spacebug-oh, a meeting going on?14:00
faganspacebug-: for a few mins14:00
nessitaspacebug-: yeah, no worries14:00
ralsinaspacebug: it's our daily standup, will take 10 minutes or so14:00
spacebug-ok14:01
spacebug-I waint ;)14:01
fagango?14:01
nessitago!14:01
faganWORK IN PROGRESS14:01
fagan* rst port of the tutorial pages14:01
faganBlocked14:01
fagan* nope14:01
fagannessita:  go14:01
nessitaDONE: lots of syncdaemon @ windows testing! finally. Also: bug #803598, bug #80356014:01
nessitaTODO: bug #803228, more IRL testing and debugging14:01
nessitaBLOCKED: nopes14:01
nessitaNEXT: ralsina14:01
ubot4Launchpad bug 803598 in ubuntuone-client "Windows: log is too crowded with "Processing events with threashold" messages (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/80359814:01
ubot4Launchpad bug 803560 in ubuntuone-control-panel "sd_client code should be multiplatform (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/80356014:01
ubot4Launchpad bug 803228 in ubuntuone-control-panel "Make test run and pass on windows (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/80322814:01
ralsinaDONE: reviews, worked on bug #800376 (blocked by sd_client), finished bug #800381, started bug #803661 and bug #80367714:01
ralsina(blocked by design updates), expense report, cleaned Canonicaladmin queue.14:01
ralsinaTODO: check updates from design, finish pending bugs, continue SSO cleanup or wizard workflow, whatever unblocks first.14:01
ralsinaBLOCKED: mostly14:01
ralsinaNEXT: dobey14:01
dobeyλ DONE: bug #803638, reviews, bug #80347514:01
dobeyλ TODO: objectives, more bug fixing/triage14:01
dobeyλ BLCK: None.14:01
ubot4Launchpad bug 800376 in ubuntuone-windows-installer "Add "syncing your computer to the cloud" page to the installer (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/80037614:01
ubot4Launchpad bug 800381 in ubuntuone-windows-installer "Add "sync now or later" page to the installer (affects: 1) (heat: 19)" [Undecided,Fix committed] https://launchpad.net/bugs/80038114:01
ubot4Launchpad bug 803661 in ubuntu-sso-client "The Qt existing user login widget is incomplete. (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/80366114:01
ubot4Launchpad bug 803677 in ubuntuone-windows-installer "Make the "current user sign in" page match the design wireframe (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/80367714:01
ubot4Launchpad bug 803638 in ubuntu-sso-client "[nightlies] Tests failing on Lucid (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/80363814:01
dobeythisfred: your roll14:01
ubot4Launchpad bug 803475 in lazr.restfulclient (Ubuntu) "Missing dependency on python-simplejson (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/80347514:01
thisfredDONE: Bug #762722 TODO: Test and wrap #762722 fix BLOCKED: no NEXT: mandel14:01
ubot4Launchpad bug 762722 in ubuntuone-control-panel (and 1 other project) "Opening the Control Panel during sync removes progress indicator (affects: 1) (heat: 9)" [High,In progress] https://launchpad.net/bugs/76272214:01
nessitamandel?14:02
nessitaalecu?14:02
nessitaiuuuuhuuuu14:03
mandelone sec14:03
mandelDONE: fix for bug 801277 bug 803828 bug 803507 bug 803622 and bug 80272614:03
mandelTODO: add tests for bug fix 803622 and bug 80249814:03
mandelBLOCKED: no14:03
ubot4Launchpad bug 801277 in ubuntuone-client "The start method is not implemented in the windows sdtool (affects: 1) (heat: 62)" [Medium,In progress] https://launchpad.net/bugs/80127714:03
ubot4Launchpad bug 803828 in ubuntuone-client "Tests in test\windows\tests_tools do not restore the mocked objects (affects: 1) (heat: 6)" [Low,In progress] https://launchpad.net/bugs/80382814:03
ubot4Launchpad bug 803507 in ubuntuone-client "OSError when deleting a folder (windows version) (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/80350714:03
ubot4Launchpad bug 803622 in ubuntuone-client "ValueError: InvalidMode ('r+w') (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/80362214:03
ubot4Launchpad bug 802726 in ubuntuone-client "KeyError (Unhandled Event in INotify) in ubuntuone\syncdaemon\filesystem_notifications.py (affects: 1) (heat: 8)" [Medium,In progress] https://launchpad.net/bugs/80272614:03
ubot4Launchpad bug 802498 in ubuntuone-client "Error when doing a partial commit on windows (affects: 1) (heat: 6)" [Critical,Triaged] https://launchpad.net/bugs/80249814:03
nessitaalecu: you around?14:04
nessitaany closing comments anyone?14:04
nessitamandel, ralsina, alecu, Chipaca: meeting in 25 minutes?14:04
mandelsure thing14:04
joshuahooverfagan: ping14:04
faganjoshuahoover: ping14:04
joshuahooverfagan: how are faqs and tutorials to rst going?14:04
nessitasmells like EOM and snal time14:04
nessitasnack14:04
ralsinanessita: ack14:05
nessitaspacebug-: hey there, I've pinged verterok for help14:05
verterokspacebug-: hi14:05
spacebug-nessita: ah ok tnx14:05
faganjoshuahoover: still working on it im going pretty slow today so will have something for friday to show14:05
nessitaspacebug-: he may be able to help debug your issue14:05
spacebug-ok tnx14:05
spacebug-hi verterok14:05
faganjoshuahoover: its a good bit different to the faq since there are pics too14:05
* fagan needs a nap 14:05
verterokspacebug-: is the upload at 100% for a long time?14:05
alecuhello!14:05
aleculet me write my notes, and then I'll go "me!"14:06
spacebug-verterok: yes, this one has been for 8 hours ort so14:06
verterokspacebug-: ok, you'r hitting bug in the server14:06
joshuahooverfagan: right, with pics, i got those and put in a branch for muffinresearch...14:06
verterokspacebug-: it's a known issue that sometines big uploads get stuck at 100%, we are working on a fix14:06
joshuahoovermuffinresearch: any thoughts on how fagan should reference the pics in the tutorials as he converts to rst?14:06
ChipacaNeoti: yes14:06
Chipacaum14:06
ralsinaI need two reviews for https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_803661/+merge/66456 it's pretty simple14:06
Chipacanessita: yes14:06
spacebug-verterok: ah ok, well good its known and its been worked on14:06
verterokspacebug-: sadly, there is no woraround :/14:07
faganjoshuahoover: ah ok I thought id stick them in the branch wit mine14:07
verterok*workaround14:07
faganjoshuahoover: its cool though14:07
ralsinajoshuahoover: if the images have a known full URL, use that, it's easiest14:07
joshuahooverralsina: right, make sense14:07
spacebug-verterok: so I just wait till someone fix the bug?14:07
ralsinajoshuahoover: you can use relative URLs but it gets very annoying very quick14:07
verterokspacebug-: I'm fixing it14:07
muffinresearchjoshuahoover: is there a merge proposal for your images branch?14:08
joshuahooverfagan: here's the branch i gave to muffinresearch: lp:~joshuahoover/ubuntuone-servers/tutorial_images14:08
spacebug-verterok: ok ;)14:08
faganralsina: ah so use full urls thats a good suggestion14:08
verterokspacebug-: we arelady have a proposed fix, we are testing it and will be rolledout to productin "soon"14:08
joshuahoovermuffinresearch: no, i wasn't sure if you wanted me to propose the merge or if you were managing merges on your branch and then doing a bigger merge to trunk14:08
verterokspacebug-: "soon" depends on the tests :)14:08
faganjoshuahoover: cool will use those names for reference14:08
alecume14:09
alecuDONE: a branch to move ubuntu-sso-client from txnamedpipes to qtreactor (bug #803433). Started a branch to move ubuntuone-client from txnamedpipes to plain windows reactor (bug #803640)14:09
alecuTODO: finish moving ubuntuone-client, move ubuntuone-control-panel to qtreactor (bug #803641)14:09
alecuBLOCKED: no14:09
ubot4Launchpad bug 803433 in ubuntu-sso-client "Use qt-reactor (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/80343314:09
ubot4Launchpad bug 803640 in ubuntuone-client "Replace txnamedpipes with the standard twisted reactor (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/80364014:09
fagan(the names of the images I mean)14:09
ubot4Launchpad bug 803641 in ubuntuone-control-panel "Replace txnamedpipes with the qtreactor (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/80364114:09
muffinresearchjoshuahoover: it'll be easier to get this into trunk so we can roll it out - that way we'll get the rst pointing at the images on the media server directly14:09
alecusorry I missed the standup.14:09
spacebug-verterok: ok good. Just wanted to know that someone knew the problem. I really like the service ;)14:09
spacebug-well tnx for now, have a great day everyone!14:09
faganso cool then ill finish it by tomorrow but going for a break now since I need to sleep for a bit14:11
ralsinadobey: while I don't usually ask for the second u1-hackers review, I don't approve the branches without it :-)14:11
ralsinadobey: just to avoid getting two identical mails14:11
ralsinafagan: sleep? it's 2PM!14:12
dobeyralsina: if it needs two reviws, you should request it14:12
ralsinadobey: ok, will do14:12
faganralsina: tired, got actually no sleep last night14:12
=== teknico_away is now known as teknico
ralsinafagan: working?14:12
joshuahoovermuffinresearch: care to review? https://code.edge.launchpad.net/~joshuahoover/ubuntuone-servers/tutorial_images/+merge/6646114:12
faganslept after work yesterday threw my cycle off a bit14:13
faganralsina: slow and steady today :)14:13
faganill get back after a quick rest and get more of this rst port done14:13
nessitaalecu: FYI, control panel already uses qtreactor14:14
ralsinafagan: grmbl. You know, it's part of our job to be *at* our job on the hours we are supposed to. Sleeping regular hours is important.14:14
faganralsina: Well I will be just taking a break for an hour or 214:15
alecunessita, did you change it? or did we never ported it to txnamedpipes?14:15
ralsinafagan: just something to keep in mind.14:15
nessitaalecu: we never port it14:15
faganralsina: sorry about it but id say it would make me a little more productive14:15
nessitaalecu: hum now that you mention, windows. main may be txn14:15
alecunessita, nice.14:15
nessitasorry14:15
* nessita checks14:15
ralsinanessita, alecu; efficiency through procrastination14:15
alecuralsina, I think I better take a nap too14:16
nessitaalecu: you are right :-) windows.main uses txn14:16
faganbut yeah regular sleep is a good thing (I think thats what ralsina was trying to get across)14:17
faganfor waking up and being productive and all that14:17
ralsinafagan: basically, that sleeping at night and being awake at 2PM is part of being a responsible adult, which is part of the job description :-)14:18
ralsinaOf course we all have bad nights/days/whatever, so... it happens.14:18
faganralsina: yeah bad night that sums it up14:19
thisfredralsina: what should we set for target dates? april 12 or oct 12?14:44
dobeythisfred: December 21, 2012. You won't be able to do anything after that anyway14:48
thisfredcan I just put 'rapture'14:49
dobeyQuetzlcoatl will return14:49
thisfred?14:49
thisfredPopocatepetl14:49
dobey"Survive the impending alien invasion."14:50
thisfredJuly 4 is X-day, I'm not making any plans beyond that14:50
thisfredJuly 5 I mean14:51
dobeyheh14:51
thisfredJuly 4 is something else. I forget14:51
dobeyjuly 4 is us holiday14:51
thisfredYeah, I lied, I know that14:51
dobeysomething to do with Jeff Goldblum saving us all14:52
thisfredwith an Apple laptop14:55
thisfredand a copy of TDL-414:55
dobeyyep14:56
ralsinathisfred: it's objectives for the next 12 months.14:56
thisfredralsina: awesome14:56
dobeyso i got a mail today from someone wanting to buy one of my domains… for $1514:57
dobeyi think i'll pass14:58
mandelnessita: can you re-review this guy: https://code.launchpad.net/~mandel/ubuntuone-client/fix-802726/+merge/6637315:01
nessitasure!15:02
mandelnessita: regarding this https://code.launchpad.net/~mandel/ubuntuone-client/sdtool_start_method/+merge/66448 the test were ran on windows, right?15:03
nessitamandel: of course :-)15:04
mandeloh, that is strange...15:04
nessitamandel: hum, maybe PYTHONPATH issues?15:05
nessitalet me re try15:05
nessitamandel: it worked!15:05
nessitamandel: seems like I had the PYTHONPATH unset15:06
nessitamandel: what I don't like about the test is that notepad is actually, really, opened15:06
mandelnessita: I actually wanted to do that to test that with the real subprocess would work15:07
mandelwith the deferred and all15:07
nessitamandel: but we don't want to test subprocess, "subprocess works" is True for us15:07
nessitamandel: can we please mock subprocess.Popen? otherwise is not longer a unittest15:07
mandelnessita: I can mock it if you want to, is not too hard15:08
nessitaplease... if we want to run this in a bot we may not be able to open IRL stuff15:08
ralsinawe need a windows port of /bin/true to test that :-)15:09
nessitaralsina: mocking Popen will do for now :-) (luckily)15:09
mandelok, I'll do that then :)15:09
mandelnessita: the other review, the one about w+b, does it look ok?15:10
nessitamandel: yes, about to try it on windows15:10
mandelgreat :)15:11
mandelnessita: those tests are broken, make sure you run the command with the -t and the regexp of the tests15:11
mandeldo you know the commands?15:11
nessitamandel: no, can you please add that to the MP?15:11
mandelsure15:11
nessitamandel: for now I'm testing the IRL syncdaemon15:12
mandelnessita: command added15:13
nessitasyncdaemon running15:13
nessitaand I see tons of AQ_UPLOAD_STARTED!15:13
nessitamandel: guh, uploads are not succeding15:16
nessitamandel: but they are going further than before15:16
mandelnessita: ok, can you add a bug for that, and I'll do that one right now after the fixes in the merges15:16
nessitamandel: sure15:18
nessitamandel: ok, I need to run the magic command now :-)15:20
mandelnessita: great, landing that will get us a step closer, I have also just pushed the sdtool branch with subprocess being mocked.15:22
nessitamandel: yeah. About the upload error I will ask help to facundo, so you keep working in the same path we agreed on15:22
mandelralsina: what about a review for the spaniard: https://code.launchpad.net/~mandel/ubuntuone-client/sdtool_start_method/+merge/6644815:22
mandelnessita: sure, I'll try to reproduce the partial  file commit error then15:23
ralsinamandel: trade! https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_803661/+merge/6645615:23
nessitamandel: great15:23
mandelralsina: ok15:23
mandelnessita: which is the bug you filled, jus tto take a quick look and see if there is any thing that rings a bell15:24
nessitamandel: can't fill it yet, VM was out of contorl15:24
nessitacontrol15:24
nessitamandel: I'll let you know, but basically every single UPLOAD was failing with TRY_AGAIN15:25
* facundobatista is glad to help15:25
mandelweird...15:25
nessitamandel: does the magic command line need a reactor?15:29
mandelnessita: no, did you get an error?15:29
nessitayeap, pasting now15:29
nessitamandel: paste.ubuntu.com/63581315:30
mandelnessita: looks like I do get the tests to run without passing the reactor15:33
mandelnessita:  add —reactor=txnp to the command, and please provide the regexp I have and not the name of the testcase, otherwise it won't run any tests15:33
mandelu1trial is not that smart, it looks at test names and ignores testcases names15:33
nessitawhy not? -t TestCaseName works on linux15:33
nessitamandel: that's a lie! :-)15:33
nessitaI use test case names all the time15:34
nessitaor is this a windows thing?15:34
mandelnessita: maybe I did a typo, but in mine it did not work...15:34
* mandel double checks15:34
nessitait worked here (passing --reactor=txnp)15:35
mandelnessita: ha, I'm stupid I was in the wrong branch…15:35
nessita:-)15:35
ralsinamandel: about https://code.launchpad.net/~mandel/ubuntuone-client/sdtool_start_method/+merge/66448 all tests pass but I *don't* see notepad15:36
mandelralsina: yes, nessita wanted to have that mocked :)15:36
mandelralsina: so you are not meant to see it15:36
ralsinamandel: cool then ;-)15:37
* nessita is always guilty15:37
ralsinadidn't know it was already mocked, had the old diff15:37
ralsinamandel: +115:38
ralsinanessita: this is the current wireframe for the installer, which will be signed off and finalized tomorrow morning https://docs.google.com/a/canonical.com/drawings/d/10Io1nRr0mGr_EW5mHTKErS6s2oxVnF7uCDdIzuqYB50/edit?hl=en_US15:43
ralsinaAnd renamed "Final spec we promise not to change."15:43
nessitaralsina: so, that is not the one you've been using :-/15:44
ralsinanessita: it's a new copy of the one I have been using + design work + 3 changes15:44
ralsina1) per-folder checkboxes on the local folders page15:44
ralsina2) A wizard-progress widget on the left15:44
ralsina3) there is design, so I can use the assets (or not and leave it for later)15:45
nessitaralsina: I will have to trust you on this one (re the amount of changes). Let's remember this for future developments, so we all (I know I didn't confirm this was signed off) work on signed off specs15:46
nessita:-)15:46
ralsinaLive and learn.15:46
ralsinaBetter than all the alternatives.15:46
nessitaor, like fagan would say, live and sleep :-P15:47
ralsina;-)15:47
nessitamandel: ./tests/syncdaemon/test_action_queue.py:15:49
nessita    5653:  local variable 'tmp' is assigned to but never used15:49
mandelO_o on it15:49
nessitamandel: from https://code.launchpad.net/~mandel/ubuntuone-client/fix-803622/+merge/6642515:49
nessitamandel: also, did you catch this trivial needs fixing? https://code.launchpad.net/~mandel/ubuntuone-client/fix-803507/+merge/6641815:53
mandelyes, on it15:53
mandelnessita: do you know which comment to add when you want to disable the var not use message with pyflakes?15:54
nessitamandel: I don't think there is a way15:55
nessitamandel: can't you assert something on tmp?15:56
nessitamandel: or remove "tmp = " altogether15:56
dobeymandel: you can't disable specific warnings in pyflakes15:56
dobeymandel: if the variable isn't being used, remove the variable :)15:57
mandelok15:59
mandelsuper lame15:59
dobeymandel: don't write useless code :)15:59
dobeywhy would you assign to a variable you're not using, anyway?15:59
mandelwanted to be clear, but not a big deal16:01
nessitamandel: hit me baby one more time!16:02
nessita(with reviews)16:02
mandelthat + twitter status == mandel scared16:03
mandelnessita: unused var fixed16:03
nessitamandel: approving (the rest will be take care by tarmac)16:03
dobeyat least it's not google+ status16:03
nessitagoogle+: sometimes I'm scared of you!16:05
mandeldobey: I wonder how that good that things is… have you seen it?16:06
dobeygoogle+? yeah, i am on it. but i don't understand it16:06
mandeldobey: what do you mean?16:09
ralsinaalecu, nessita: I could use a second review of https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_803661/+merge/6645616:09
mandelnessita: https://code.launchpad.net/~mandel/ubuntuone-client/fix-803507/+merge/66418 updated16:09
nessitaralsina, mandel: on those16:09
ralsinanessita: thx16:09
dobeymandel: i mean google+ is very weird. it has this thing called "circles"16:10
dobeymandel: "friend circle" "family circle" etc…16:10
dobeymandel: and the UI is annoying16:11
mandeldobey: circles as in circles of friends?16:11
dobeyright16:12
ralsinadobey: like facebook lists16:12
dobeyinstead of groups, they called them circles i guess, and made the UI suck16:12
dobeyralsina: yeah, but you can't use lists in fb to restrict privacy; you can use circles for that in google+16:13
ralsinadobey: yes you can, in facebook. http://www.facebook.com/help/?page=76816:14
ralsinadobey: specifically, http://www.facebook.com/help/?faq=1207416:14
dobeysecond one is not relevant since it's assumed if you know what friend lists are, you already have them :)16:15
thisfrednessita: re: control panel auth:16:16
dobeybut ok, so the UI sucks for doing that on FB, so nobody uses it, because it's insanely hard to use16:16
thisfrednessita: I cannot connect: I threw away the token, then I thought I had the wrong password maybe to authenticate the device, asked for and got a new one, but whenever I enter it, I see the string 'An exception representing an authentication failure'  in red, which is a terrible error message. (Looks like we're showing an exception doc string??)16:16
nessitamandel: directory removed!!!!16:16
mandelnessita: is that good or bad?16:16
nessitamandel: good, OsError went waya16:16
mandeltoma16:17
mandel!!16:17
dobeythisfred: that isn't a docstring16:17
thisfredIt sure looks like one16:17
nessitaaway16:17
nessitathisfred: you have the time wrong?16:17
nessitaI agree terrible error to show, is not a docstring, is coming from the SSO server16:17
nessitathisfred: most likely, your time is off16:17
dobeythisfred: well is't a horrible string, yes16:17
thisfrednessita: maybe, I have no indicators :)16:17
nessitathisfred: bug #70350716:18
ubot4Launchpad bug 703507 in ubuntu (and 1 other project) "I can't log in - "an exception representing an authentication failure" (affects: 4) (heat: 42)" [Undecided,Confirmed] https://launchpad.net/bugs/70350716:18
nessitathisfred: "date" on a terminal16:18
thisfrednessita: yep it's an hour off16:18
nessitathisfred: that, plus all the errors you got re auth failures, indicates time is off16:18
dobeyproblem solved16:18
nessitathisfred: oauth does not work without the problem time16:18
nessitathisfred: the time thing is way out of our reach. Yes, we should be showing better error messages.16:19
ralsinathisfred: meta-whatever, xterm -e "watch date"16:19
mandelthisfred: do you still need a review for https://code.launchpad.net/~thisfred/ubuntuone-client/add-counter16:20
thisfredmandel: sure, not urgent, but if you're bored :)16:21
thisfredyay u1 works again16:22
dobeyralsina: "watch ddate" is more fun16:29
nessitaralsina: running you branch on windows now. Can I have some intructions about running tests and IRL test in the MP?16:29
ralsinanessita: sure, in 1'16:29
thisfredman ddate | grep X-Day16:30
dobeyok, lunch. bbiab16:35
ralsinanessita: added IRL instructions. However, it doesn't show you the page subtitle, because the default is an empty string.16:37
ralsinaI should add a test that the subtitle is set as the right property of the QWizardPage, right?16:38
ralsinadobey, thisfred, one of you who has oneiric, can check bug #803930 please?16:40
ubot4Launchpad bug 803930 in ubuntuone-control-panel (Ubuntu) "Services tab fails with: Value could not be retrieved. (Unauthorized: ('unauthorized', 'Authentication required.')) (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/80393016:40
thisfredralsina: was just looking at that16:40
ralsinathisfred: cool16:40
nessitaralsina: ok, thanks. I tried to run the installer with the new sso, to see if I got the text in the window (I still don't), and I notice that sso complains about the installer trying to register twice. I filed bug #80395216:42
ubot4Launchpad bug 803952 in ubuntuone-windows-installer " Client tried to register twice (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/80395216:42
nessitaralsina: maybe that's Invalid, I'm not sure, I'm letting you know to debug and triage16:43
ralsinanessita: checking that bug right now16:43
nessitanot urgent, I think16:43
ralsinanessita: if you use the installer instead of show_gui you should have a subtitle16:43
nessitaralsina: I got no text at all16:43
nessitaralsina: and this time they did connect, but SSO complained with that bug16:43
ralsinanessita: still not connecting :-(16:43
ralsinaoh, ok16:43
nessitaralsina: you are running latest uSSOc?16:44
ralsinanessita: as of early this morning16:44
ralsinaI'll pull16:44
nessitaralsina: I need to ask this: you sure you don't have an older version installed in site-packages?16:44
ralsinanessita: I actually install sso every day16:44
nessitaralsina: ok, I'll let you debug that bug (is not top priority, can be later). I'll use show_gui for now16:45
ralsinanessita: ok, cool16:45
nessitaralsina: oh, tests do not pass on windows16:48
ralsinasso tests?16:48
ralsinanessita: confirmed that trunk sso and trunk installer don't work together: I get this on the installer console: Failure: twisted.internet.error.ConnectionRefusedError: Connection was refused b16:49
ralsinay other side: 10061: WSAECONNREFUSED.16:49
ralsinaThat's because we are not in the same transport/reactor right now. So, need to fix that first16:50
nessitaralsina: ok16:50
ralsinaalecu: do you have moving the installer to qt4 reactor in your TODO? I am doing it myself now16:51
nessitaralsina: yes, sso tests16:51
ralsinanessita: ok, put the error in a needsfixing, I'll get to it soon16:51
=== beuno is now known as beuno-lunch
nessitaack16:53
nessitadobey: did you sso branch landed yesterday?16:53
nessitadobey: I see it did. That broke tests on windows :-/16:54
mandelnessita: the thing with the logs being stored in Temporary Internet Files is driving me crazy, I'm gonna fix that before I move on, the thing is that the bug is from +junk branch I have, should I add that code in the u1client code as a window module to be installed when needed?16:55
nessitamandel: we can't add that to the u1client source tree16:55
mandelnessita: not even in contrib?16:56
nessitamandel: we need to so something about it, but not adding it to the source tree. No contrib, anyone branching in linux will be confused16:56
nessitasince xdg is already provided. That needs to be a dependency16:56
nessitamandel: please remind me to talk about this tomorrow in the mumble, yes?16:57
mandelnessita: I can create a small project called winxdg that is installed as xdg, is not too much overhead, what do you think?16:57
mandelnessita: yes, I'll remind you16:57
nessitamandel: let's talk tomorrow. Thanks!16:57
mandelwell, I need to take the dog out, bbl16:57
nessitaack16:58
nessitadobey: I've filled bug #803958, if you could please help us having the test working on windows as well, it would be great16:59
ubot4Launchpad bug 803958 in ubuntu-sso-client "urllib2.URLError when running the tests in windows (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/80395816:59
nessitadobey: (I assigned it to you)16:59
nessitaralsina: test failure added (the one not related to ^). Also, I requested tests (as usual) :-).17:10
ralsinanessita: cool, thx17:11
Cerrdorhow do I install ubunut one thing for banshee?17:13
nessitamandel, facundobatista: seems like syncdaemon can't rotate logs on windows :-(17:14
nessitaI'll file a bug17:14
facundobatistanessita, why is that?17:15
dobeynessita: weird17:15
facundobatistanessita, maybe because the old is still open and windows locks it?17:15
alecuralsina, pong. Here's the bug for qtreactor in installer: bug #80367517:15
ubot4Launchpad bug 803675 in ubuntuone-windows-installer "Replace txnamedpipes with qtreactor (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/80367517:15
nessitafacundobatista: why? no idea... but I ran with TRACE, which generated a huge amount of info17:15
ralsinaalecu, ok thanks!17:15
bert_voegeleCerrdor : sudo aptitude install banshee-extension-ubuntuonemusicstore17:16
Cerrdorthar ya go17:16
Cerrdorshanks17:16
alecuralsina, feel free to asign it to yourself, etc.17:16
nessitafacundobatista: and I only have a syncdaemon.log file, which is evidently not the "first one"17:16
Cerrdorand ubuntu one in my application menu?17:16
nessitadobey: the "weird" refers to the log not rotating or the bug report re test in window?17:17
bert_voegeleCerrdor : should be under Preferences17:17
nessitafacundobatista: sorry, I was wrong!17:17
nessitamandel: ^17:18
Cerrdoryou tha man17:18
nessitafacundobatista, mandel: seems like listing all the *.log files will not include others such as bla.log.bar17:18
dobeynessita: windows tests17:19
nessitadobey: ah. I pasted the trace, so if you could help us in that end it will be highly appreciated17:19
facundobatistanessita, listing where?17:20
nessitafacundobatista: on a cmd.exe17:20
facundobatistanessita, ok17:21
CerrdorI want a banshee plugin that will send what im listening to into an IRC room anyone know where to find that?17:21
dobeynessita: i think the test was already broken on windows, based on that error. it doesn't seem to have the full path17:21
dobeyCerrdor: i doubt there is one. but there might be a plug-in for your IRC client that can get the current playing song from banshee. you should ask in the banshee channel through probably17:22
CerrdorI tried banshee and it is abandoned17:23
Cerrdorlol17:23
bert_voegeleCerrdor : solution depends on your IRC client17:23
Cerrdorxchat17:23
dobeyCerrdor: banshee's irc channel is on gimpnet, not freenode17:23
Cerrdorohhh17:23
Cerrdorwtf for lol17:23
nessitadobey: but the test_account is mutiplatform, we don't have special cases for windows17:25
nessitafacundobatista: logs sent, I'll report the bug while you see the logs17:25
dobeynessita: well clearly that's wrong17:26
nessitadobey: I have too many things in my head, can you please be more specific?17:26
nessitadefine "that"17:26
dobeynessita: it looks like os.path.abspath, and os.path.curdir, are not working on windows?17:27
dobeyor something.17:27
dobeyhwy is the path '' in the traceback?17:28
ralsinaalecu, nessita, mandel: reviews for very small branch? https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803675/+merge/6649117:28
dobeynessita: can you print the url being passed to urlopen() right before it's called to see what it's passing in?17:28
dobeynessita: i'm guessing maybe it's not escaped properly or something17:29
bert_voegeleI'm getting "Service Temporarly Unavailable" (503)  for https://couchdb.one.ubuntu.com since a couple of weeks and desktop logs grow huge. someone know a solution?17:29
nessitadobey: I can, after lunch, I'll ping you bak17:34
nessitaback*17:34
dobeyok17:34
=== beuno-lunch is now known as beuno
nessitalunchtime@!17:52
ralsinanessita: https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_803661/+merge/66456 ready for re-review17:53
=== teknico is now known as teknico_away
mandelralsina: have you tested that branch?18:38
ralsinamandel: the one for nessita? Yes, on windows.18:38
mandelralsina: I think it wont work, I have not tried it yet, but I have a very string feeling about that18:38
mandelralsina: I meanhttps://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803675/+merge/6649118:39
* nessita is back18:39
ralsinamandel: looking...18:39
ralsinamandel: yes, tried it IRL and it works18:39
ralsinamandel: even get credentials and everything18:40
mandelralsina: If I remember correctly you have to create the instance of the QApplication before you install the reactor https://github.com/thisfred/qtreactor18:40
mandelralsina: look at Using the Qt4Reactor18:41
mandeland in the diff is after18:41
mandelralsina: am I missing something?18:41
ralsinamandel: hmmmm makes sense. Wonder why it seems to work, though18:41
thisfredthisfred?18:42
mandelthisfred: first hit in google when I looked for qtreactor ;)18:42
thisfredgit clone https://github.com/ghtdak/qtreactor18:42
thisfredlol18:42
thisfredall I did was fork it18:43
ralsinamandel: I; ll flip it18:43
mandelralsina: so there is something wrong… I have not tried it but looks like it should be using a QCoreApplication and not a QApplication meaning that the UI could not be isntantiated18:43
mandelralsina: that should fix the possible issue, but it would be nice to now why it works on your machine...18:43
ralsinamandel "would be nice", sure18:44
mandelralsina: also, your removed the interleaver, but did not remove the app.exec call which is wrong, in the diff I should be able to see a reactor.run()18:45
mandellike here: https://github.com/thisfred/qtreactor/blob/master/ghtTests/trivialscript.py18:46
thisfredSTOP PINGING ME :)18:46
mandelthisfred: your fault, stop forking projects!18:46
thisfredfork you!18:46
mandelthat is what she said18:47
mandelralsina: did I make sense?18:47
ralsinabut without the exec how do I start the event loop?18:48
dobeyhmmmm18:48
ralsinaoh, run(), sure18:48
ralsinamandel: did the changes, seems to work exactly the same to me. But hey, you know better.18:49
mandelralsina: and really_quit is wrong too, since you do not have to call app.quit but reactor.stop18:49
ralsinamandel: yes, but it made sense because I had app.exec() :-)18:50
ralsinathose go in pairs18:50
mandelralsina: yes, I was mentioning it in case you forgot it in the run change :)18:50
ralsinaThere, pushed :-)18:50
ralsinaalecu, mandel: reviews for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803675/+merge/66491 would make me super happy for about 10 minutes18:59
* ralsina is starting to have too many branches piling19:00
mandelralsina: this approved: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803675/+merge/6649119:01
ralsinamandel: yay!19:01
mandelralsina: but check that in your python path you were not using the old code because it did not work in my machine19:01
ralsinachecking...19:01
ralsinamandel: verified via a huge print in the main()19:02
alecuralsina, on it19:02
mandelweird....19:04
nessitaralsina: I'm back to your review now19:11
ralsinanessita: cool19:11
nessitadobey: can you please add a comment in the bug report asking what thing you want printed where? otherwise I'm a bit lost19:12
alecuralsina, probably not related to your latests changes, but I get this: https://pastebin.canonical.com/49234/19:12
ralsinaalecu: yes, separate problem19:13
ralsinaalecu: probably one branch or the other merged incomplete :-(19:14
ralsinaalecu: is that on the fix_803675 branch?19:14
alecuralsina, yes19:15
dobeynessita: https://bugs.launchpad.net/ubuntu-sso-client/+bug/803958/comments/219:15
ralsinaok, will push a fix in 1'19:15
ubot4Launchpad bug 803958 in ubuntu-sso-client "urllib2.URLError when running the tests in windows (affects: 1) (heat: 6)" [High,Triaged]19:15
nessitaralsina: are the test suite passing in your fix_803661 branch? (except those from generate)captcha)19:15
nessitaralsina: I get 2 failures19:15
ralsinanessita: they pass for me19:15
ralsinanessita: paste the errors please?19:15
nessitasure19:15
nessitaralsina: paste.ubuntu.com/635920/19:17
ralsinanessita: ack19:18
alecuralsina, again, probably not related, but when pressing enter on the password, the "Sign in" button should be activated.19:18
ralsinaalecu: accelerators are not implemented yet. Had a question about them in last night's report19:18
alecuralsina, also, when entering bad login info, a dialog pops up with the webservice error.19:18
alecuralsina, I'm sure you know about that :-)19:19
ralsinaalecu: popups not done yet either :-)19:19
ralsinanessita: I was running the tests wrong :-( Should have a fix in 15' or so19:19
alecuralsina, finally: clicking on "Sign In" should disable said button while the webservice call is active, otherwise I can keep clicking on it.19:20
nessitaralsina: ack19:20
alecuralsina, other than that, branch approved ! :-)19:20
ralsinaalecu: blocking the UI while busy is not implemented yet ;-)19:20
alecuhahahaah19:20
nessitaralsina: I know you asked about accelerators, but those are not related with default buttons, no?19:20
alecuralsina, do you have bugs for this items to be done? or do you keep some list?19:20
nessitaralsina: clicking <enter> on a text entry should activate the default button19:20
ralsinanessita: kinda. In a wizard, the default button can't be one inside the page19:21
nessitaralsina: ok... not sure how that answers my question re: accelerators are not directly related to default buttons19:21
ralsinaBasically, that I need to do a whole review of how to interact with the keyboard. And that one way to fix it is override the default [enter] shortcut19:22
alecuralsina, what about making the "Sign In" button be part of the wizard buttons?19:22
ralsinaalecu: have to ask design, and change SSO19:22
alecuralsina, ok19:22
* alecu gets back to code19:23
nessitadobey: url printed and added19:28
dobeyah19:29
dobeythe C: needs to be changed to C|19:29
nessitamandel: you gone?19:29
dobeynot sure the best way to do that, though19:29
dobeyother than to make that test partially platform-dependent19:30
ralsinaalecu: can you tell me what is the result of  type(1337) in your python?19:32
alecuralsina, __elite__19:32
ralsinaalecu: seriously :-)19:33
ralsinaif it's long, that's why the test fails for you (it's int here)19:33
ralsinaAnd in ay case, I have to fix a line of code19:33
alecuPython 2.7.1 (r271:86832, Nov 27 2010, 18:30:46) [MSC v.1500 32 bit (Intel)] on19:34
alecuwin3219:34
alecuType "help", "copyright", "credits" or "license" for more information.19:34
alecu>>> type(1337)19:34
alecu<type 'int'>19:34
alecuralsina, does it help?19:34
nessitaralsina: besides the test failure, there are lint issues on linux (added to the MP)19:35
ralsinaalecu: ok, no idea of *why* it fails, but it may be fixed now.19:37
ralsinanessita: on SSO? Yes, those were there already but I can fix them.19:37
dobeyralsina: hrmm, int vs long should only matter if it's a very large number, or the byte ordering is different. and i don't think windows deals with the latter well at all :)19:37
nessitaralsina: on *linux* (they were not there)19:38
ralsinadobey: basically if it's a number, I have to call humanize() on it19:38
ralsinanessita: I didn't touch anything on the linux files19:38
* ralsina pinky-swears he didn't19:38
nessitaralsina: I know. When running the test suite in linux, there are lint issues on the windows code that needs to be fixed19:38
nessitaralsina: if you look the comment, those are pretty fixeable issues19:39
ralsinanessita: ok, that makes more sense. Will check.19:39
ralsinanessita: fixed https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_803661/+merge/66456 -- because of PYTHONPATH I was running the tests on trunk :-(19:57
nessitaralsina: ack19:59
nessitafacundobatista: nested branch approved20:02
facundobatistanessita, nice!20:03
nessitaralsina: ubuntu_sso/qt/tests/test_windows.py:20:04
nessita    132:  [C0301] Line too long (91/79)20:04
facundobatistanessita, well, the not-nested one, right? ;)20:04
nessita:-)20:04
ralsinanessita: grrrrrr on me20:04
nessitaralsina: indeed! :-P20:04
facundobatistaralsina, 91????20:04
facundobatistaralsina, not 80, or 81... 91??? jajaj20:04
nessitabad news is that syncdaemon is stuck in windows :-/20:04
nessitahas 2 items left in the queue, and is doing nothing else20:04
alecunessita, remember that the txnamedpipes reactor for windows is broken even when connecting thru tcp. That means that random errors you see on windows may be atributable to that.20:06
alecunessita, just a few more hours :-)20:06
nessitaalecu: right... good point. I will see thru the logs, so far all errors were, well, real errors in the code :-)20:07
alecunessita, well, that makes sense as well :-)20:07
ralsinanessita: I had fixed that, I forgot to push it :-(20:08
nessitaralsina: oh!20:08
alecunessita, ralsina: please, can you review my first u1client branch? https://code.launchpad.net/~alecu/ubuntuone-client/pipe-refactor/+merge/6651420:08
ralsinanessita: so there, ready for re-re-review. I am starting to feel incredibly stupid today.20:08
ralsinaalecu: yessir!20:09
nessitaalecu: sure, after I finish with ralsina's20:09
alecunessita, ralsina: this branch is only refactor, so I can move the code from txnamedpipes to qtreactor in a smaller branch20:09
dobeyralsina: it's called realizing you're just a puny human. :)20:09
ralsinadobey: I am hardly puny compared to most humans (eric excepted). I do feel stupid, but there is nothing wrong with my size :-)20:10
dobeyralsina: stomach size != brain size :)20:12
thisfredBRAAAAINS20:13
nessitaralsina: APPROVED, finally20:13
ralsinanessita: yay, I guess20:13
nessitaalecu: your turn sweety20:13
* nessita is in the mood "thanks" to babasonicos20:14
nessitaalecu: can you please merge trunk in?20:15
nessitaText conflict in tests/platform/windows/test_tools.py20:15
nessita1 conflicts encountered.20:15
alecuuhhh20:16
nessitaalecu: you let me know when is done?20:36
alecunessita, sure. I found another thing that broke after merging reactor and running tests, so I'm fixing it.20:37
nessitaack20:37
alecunessita, fix and pushed (hopefully!)21:00
nessitalooking21:01
ralsinaalecu, nessita: I have a much smaller branch I hope can pass without 5 re-reviews. You can approve it by looking at pictures if you want ;-) https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803677/+merge/6646621:02
alecuralsina, will look at pictures and approve.21:02
alecuralsina, I trade for one review of mine21:02
alecuhttps://code.launchpad.net/~alecu/ubuntuone-client/pipe-refactor/+merge/6651421:02
* ralsina adds pictures of pretty blondes21:02
ralsinaalecu: was looking at it already :-)21:02
alecuralsina, approved!21:02
nessitaralsina: queueing that up21:03
ralsinanessita: ok, no rush for this one21:03
alecuralsina, I'm getting this: http://pastebin.ubuntu.com/635990/21:12
alecuralsina, and the fields are empty :-(21:12
nessitaalecu: branch looks good, running tests21:13
alecuralsina, my path is set right, and the sso is running21:13
alecuralsina, perhaps it's because the sso on trunk is not qtreactor based?21:13
ralsinaalecu: old SSO21:13
ralsinaalecu: and you need to run setup.py build on SSO before testing this branch, as well21:13
alecuaha21:14
nessitaralsina: is this https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803677/+merge/66466 as WIP for some reason?21:14
nessitaralsina: so, aren't there tests that test the calls you're making to CurrentUserController and related?21:15
ralsinanessita: I don't know how to remove it :-)21:15
alecuralsina, building sso worked.21:15
ralsinanessita: there are tests for how CurrentUserController works, on SSO21:16
alecuralsina, now I get the labels, I get no stacktrace, but I get this warning on stdout: QWizardPage::addField: Duplicate field 'email_address'21:16
ralsinaalecu: harmless21:16
nessitaralsina: right, but within the installer we should have tests about how the installer calls the CurrentUserController21:16
alecuralsina, cool21:17
nessitaralsina: we should have every piece of non-generated code tested\21:17
ralsinanessita: hmmmm ok, easy to add. But I can only test that I am setting the right title/subtitle. And I would have to import those from gui.py... I will add it ayway.21:17
nessitaralsina: yes, that's the perfect test21:17
nessitaralsina: seems silly, but pays in the long term21:18
ralsinaSo, import two constants, call the function with them, and see I get them back? That's exactly the SSO test for title/subtitle :-)21:18
alecuralsina, one tiny nitpick: "...with your existing Ubuntu Oneusername": there's no space between "Ubuntu One" and "username"21:18
alecuralsina, other than that, approved.21:18
ralsinanessita: ok, I see the possible regression now (changing the call and forgetting to set the subtitle)21:18
ralsinaalecu: fixing the typo, nessita, adding the test21:19
nessitaralsina: the assert is about the call being properly made, not the label being set with the proper string21:19
nessitaralsina: the test about the label having the proper string is in SSO, and I agree there is no point in duplicating21:19
nessitais also discouraged, since we may change the subtitle to be added as, crazy example, a tooltip21:20
ralsinanessita: but the only arguments I am passing are those two strings.21:20
nessitaand in that case we should change 2 places <- crazy21:20
nessitaralsina: I can go and change those string to 'perro' and 'gato' and I expect a test to fail21:20
nessitanot the constants, but the args in the call21:20
ralsinanessita: hmmmm ok, I think I get it now. let me write the test.21:21
nessitaralsina: can you see my point?21:21
ralsinaSo, I have to check that MainWindow.current_user_controller._title is the right constant and the same for subtitle.21:22
nessitaralsina: well, not _title since is not a public attr. YOu can patch the class and assert about the args passed21:22
ralsinaright21:23
ralsinait's faked already so I can do that21:23
nessitagreat!21:23
nessitaalecu: approving21:23
alecuwoohoo!21:23
* alecu is rushing to finish u1client21:23
ralsinanessita: added test21:30
nessitaack21:32
ralsinaalecu: +121:32
alecuralsina, thanks!21:33
ralsinaI'm about to EOD now. report being sent in a bit, but all things considered, not really productive today :-(21:46
alecubug #80367221:47
ubot4Launchpad bug 803672 in ubuntu-sso-client "Do not start two instances of this process (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/80367221:47
dobeyhrmm, i just remembered something regarding perf reviews21:48
dobeybut i can bug tomorrow21:48
nessitaralsina: I'll leave the review result in the MP21:52
ralsinadobey shoot21:52
nessitahopefully is an approve and nothing is left in that area21:52
ralsinanessita: if it's approve mark it for approval so I start with only one open branch tomorrow :-)21:52
nessitaof course21:52
nessitaralsina: approved. Did you see alecu's comment? Please just remove the space between the app_name and the text "username"          Reply21:58
ralsinanessita: checking...22:01
ralsinanessita, alecu: is that in the subtitle?22:02
nessitayes22:02
nessita"Sign in with your existing Ubuntu One username and password."22:03
alecucan't find any .srt file!22:03
nessitasounds better to say22:03
nessita"Sign in with your existing username and password."22:03
nessitaalecu: .srt?22:03
nessitalol22:03
nessitaI'm SO slow at this point22:03
alecubrb22:04
ralsinanessita: the texts came from design. Anyway, I asked for final signoff on these, because the SSO screens were not in the final texts document22:04
nessitaok22:05
ralsinareport sent22:07
ralsinaand EOD achieved :-)22:07
nessitaralsina: amen22:07
* nessita eoding as well22:07
* alecu is about to push...22:12
nessitaboy or girl?22:13
nessita:-P22:13
alecuwe'll find out soon...22:14
=== m_conley is now known as m_conley_away
dobeyalright, i'm off. good evening all!22:15
ralsinaalecu, I must be too tired but I don't understand the comment about the space in https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803677/+merge/66466 :-(22:16
alecuralsina, let me take a look.22:17
nessitaoh, I approved that already22:18
alecuralsina, it seems you have already fixed it here: http://bazaar.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803677/revision/1622:18
nessitabecause of what you've said about signing off the texts22:18
ralsinaoh, it's ADD the space :-)22:19
ralsinathe comment says REMOVE the space22:19
ralsinanessita: yes, this is idle curiosity22:19
alecudoh22:19
alecuralsina, sorry! too sleepy today.22:19
nessitaok, I'm off22:19
nessitabye!22:19
alecunessita, ralsina: https://code.launchpad.net/~alecu/ubuntuone-client/use-standard-reactor/+merge/6652122:20
ralsinabye nessita!22:20
alecufor review, tomorrow :-)22:20
nessitaalecu: will review that later22:20
nessitabye!22:20
ralsinaalecu: put it in the report, I will do mine early and awake22:20
alecubye!!!!22:20
beunothat's a lot of candidates!22:25
beunothat's great22:25
beuno(ignore that)22:25
* ralsina whistles, looks up and pushes those two lines under the rug22:25
thisfreddog o'clock, later all22:32
* alecu eods23:36
alecubye all23:36

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