/srv/irclogs.ubuntu.com/2011/07/05/#ubuntuone.txt

=== zyga is now known as zyga-afk
mandelmorning all!07:46
* mandel is surprise how good 3g works in mallorca...07:46
faganmorning07:48
wimpunkmorning07:53
ryemandel, 3g... mmmm.... do want...07:59
mandelrye: I'm surprise it works this well…. I'll upload a pict of my new office :)08:00
mandelrye: working looking at the mediterranean is great :P08:00
ryemandel, I'll send you a postcard full of envy08:01
mandelhahaha08:02
faganIreland is sunny too08:02
fagan:)08:02
mandelrye: at least you are not working on windows and dealing with its stupid encoding errors… god I hate the windows file system...08:02
faganmandel: do you not develop on mac and pull into the vm?08:03
ryemandel, well, you know... there were 3 standard encoding for cyrillic, koi8-r, windows-1251 and, finally, utf-8. I had all of the filenames encoding on my machines resulting in so many issues... ext fs do not specify the encoding of the filenames at all08:04
ryemandel, but not working on windows is good enough08:04
mandelfagan: I do, but I'm dealing with a bug I can only reproduce irl… I need to understand how windows deals with this kind of paths. Funny thing, I just discovered it because I have some nordic music and the bastards use some funny char in the songs name :P08:05
mandelrye: yeah.. at least in spanish we just have accents and ñ :)08:05
faganmandel: ahhh ok now I get it08:06
ryemandel, here, have a cyrillic letter Zh, Ж :)08:06
mandelrye: that is fucked up hahaha08:07
faganwow its raining now but its still really sunny08:07
* mandel wainsting time while windows updates...08:19
duanedesignmorning all08:29
faganmorning duanedesign08:33
* fagan needs to head to the shop real quick 08:33
wimpunkI see there regularly are people asking about couchdb sync but never get an answer. Is there any good place to ask about it?08:53
duanedesignrye: what is the latest news about the couch sync?08:54
ryeduanedesign, new couchdb is being tested on staging. The beautiberry shard (user ids 0 - 475000) has replication disabled, all other shards are working08:55
duanedesignwimpunk: ^08:57
wimpunkduanedesign: thx!08:57
duanedesignyour very welcome08:57
duanedesign.508:58
mandelrye: are we allowed to answer with such techy things to users...08:59
* mandel is going to love to answer windows bugs..08:59
ryemandel, well, the user id is public info, the shard names are not secret (and they are not visible from outside) and it is better to give a definite answer for technical question (and replication is somehow a technical term)09:00
ryeduanedesign, let me update ubuntuone-account-info script so that it works in natty, it will print the account id and other info out09:00
=== zyga-afk is now known as zyga
mandelrye:  :P09:01
ryemandel, speaking of windows bugs...09:01
* mandel hides09:02
mandelrye: tell me :)09:04
ryemandel, I don't have a hands on experience with windaemon yet and i need some assistance for...09:05
wimpunkrye: btw, tnx for those beautiful scripts. Pretty useful for people who wants to know what's going on.09:06
ryewimpunk, thanks for the kind words. I just hate when i don't know what's happening09:07
wimpunkrye: +1 :-)09:07
* rye feels writing scripts for windows is quite a bit harder :(09:23
* fagan tea break09:38
lalejandHi, no news about that ? "2011-06-16 Syncing of contacts in Evolution is not working for some users due to a server issue. We are looking into this and trying to resolve it ASAP. " My couchDB addressbook crash makes the personal one unusable also after a while.09:42
jo-erlendlalejand, I'm no expert, but I don't see how the couchdb server outage could cause your addressbook to crash.09:44
lalejandjo-erlend : maybe it's not related then :)09:45
lalejandI will see that when the couchDB one will be fixed.09:45
lalejandreading the comments in launchpad about a related bug, I understood that the problem was fixed for users one by one. I just wanted to say "not yet fixed for me" :)09:47
jo-erlendlalejand, perhaps it would be wise to say which bug you're referring to?09:51
lalejandon https://wiki.ubuntu.com/UbuntuOne/Status#Contacts it says "'An issue preventing Evolution contacts and Firefox bookmarks from syncing has been discovered and we're working on fixing it. Bug 788206"09:52
ubot4Launchpad bug 788206 in ubuntuone-servers "Oauth authentication against couchdb.one.ubuntu.com fails (affects: 16) (heat: 86)" [High,Confirmed] https://launchpad.net/bugs/78820609:52
jo-erlendlalejand, that may not be the current issue. If you look in ~/.cache/desktop-couch/log/desktop-couch-replication.log, you can see if you get errors about a server returning an error 503. I don't think that's caused by OAuth issues.09:55
lalejandjo-erlend : this file ends with "HTTPError: HTTP Error 400: BAD REQUEST 2011-07-05 11:50:16,039 DEBUG    finished replicating"09:56
lalejandbut file sync looks okay09:57
jo-erlendah. Ok.09:57
lalejandho, but ine U1 window it says that my internet connection is bad :/09:57
jo-erlendlalejand, have you tried to delete the key from your gnome keyring  and log back on?09:57
lalejand"L'information n'a pas pu être récupérée. Vérifiez votre connexion à Internet."09:58
lalejandjo-erlend : nope, I try right now09:58
lalejandjo-erlend : is it that : "UbuntuOne token for https://ubuntuone.com" I just delete it ?09:59
jo-erlendyes.09:59
jo-erlendthen, when you open the Ubuntu One client, it should ask you to log on.10:00
lalejandjo-erlend : it says "an exception representing a server-side error"10:02
jo-erlendlalejand, what says that and when?10:02
lalejandjo-erlend : I deleted the key, disconnected from U1 and reconnected in the U1 param window, it asked me to create an account or click on some button if I have already one, I clicked the button, I entered my email and password (login/pwd that I use to connect to the web interface) and I have a red message in the window saying that10:04
jo-erlendwhat does your logfile say?10:05
lalejandjo-erlend, Ho I just tryed again, it worked now10:05
lalejandlet me check if things are ok now10:05
jo-erlendbecause that is correct. It should now say something about an error 503. That means the OAuth issue was corrected and that you now only have the same problem everyone else has. :)10:05
jo-erlendno, I don't think your couchdb will sync. That's what the server-side error is about.10:06
lalejandjo-erlend : in the main tab of U1 window it says something like "information couldn't be retrieved. Please check your internet connexion"10:06
jo-erlendit is much more helpful if you paste the past entries from your logfile.10:07
lalejandjo-erlend : ok so the server-side bug isn't fixed since one month now ?10:07
lalejandok10:07
lalejandI paste10:07
jo-erlendI don't think it's been down so long. I encountered it before this weekend.10:08
lalejandjo-erlend : http://pastebin.com/qwLsHWpR10:08
jo-erlendoh, I haven't seen that before. I don't understand what it means either.10:10
jo-erlendwhat version of Ubuntu are you using, btw?10:10
lalejandargh, if I go to the "connected devices" tab, I see 3 times my laptop :(10:11
Chipacahow often does tarmac run? mandel, do you know?10:11
lalejandjo-erlend : and all 3 added today pfff10:12
jo-erlendlalejand, what operating system do you use?10:12
mandelthat is weird, a 400 error… I would have expected something completely diff10:12
lalejandnatty10:12
lalejandin the web-interface I see also 3 times my laptop10:13
jo-erlendmandel, yes, me too. But he said something about an addressbook crash. Perhaps that's caused some issues?10:13
jo-erlendlalejand, I've seen that too, and I don't know why it happens, but I don't think that's caused any problems for me.10:13
lalejandjo-erlend : ok, but it's weird :)10:14
lalejandwhat if I delete all the associated devices, and connect again ? It will replicate what is on the server to my laptop and then I will loose last version on my files here ?10:15
jo-erlendno, I don't think so. What you can do, is to create a new user, log in as that user and open Ubuntu One there.10:16
jo-erlendbut as I said before; couchdb sync doesn't work for anyone at the moment, so your databases won't work. But the error 400 might go away.10:17
lalejandjo-erlend : in order to verify what ?10:17
jo-erlendthe connected devices shouldn't cause that problem.10:17
lalejandjo-erlend : ok10:17
jo-erlendbut if you want to see if it's a problem on your local data, you can create a new user and log onto ubuntu one with that. Since that'll be a blank account, there isn't anything on it to cause any problems so you should be able to log in.10:18
jo-erlendI mean a new account on your PC, not Ubuntu One.10:18
lalejandin https://wiki.ubuntu.com/UbuntuOne/Status#Contacts they say the bug is present since the 16th of june !10:19
lalejandjo-erlend : ok10:19
jo-erlendlalejand, "the bug"?10:19
jo-erlendthere might be several different ones.10:19
lalejandjo-erlend : "2011-06-16 Syncing of contacts in Evolution is not working for some users due to a server issue. We are looking into this and trying to resolve it ASAP."10:19
karniCan someone translate this for me? Google Translate can't handle it well: "Вовремя появилась. Аналогичные удалил."10:20
jo-erlendright. But now, no couchdb sync works at all, and I think that's a new issue. But that will also affect Evolution. So I think they're two different bugs.10:20
jo-erlendtwo different issues, I should say.10:20
lalejandjo-erlend : ok10:21
jo-erlendlalejand, can you try what I suggested, just to confirm?10:21
lalejandjo-erlend : ok10:21
lalejandjo-erlend : should I close my session ?10:23
jo-erlendlalejand, no, that's not necessary.10:23
lalejandok10:23
jo-erlendyou can just switch to that other user and log onto Ubuntu One.10:23
jo-erlendI would be more than a little surprised if that didn't just work.10:24
lalejandjo-erlend : ok I have the same error message on the main tab in U1 window ("information could not be retrieved, ..."). U1 is downloading stuff to my HD, looks like things that are in my U1 folder.10:27
jo-erlendlalejand, please paste the logfile from that account.10:28
lalejandreplicating log ?10:28
jo-erlendyes.10:29
lalejandjo-erlend : http://pastebin.com/gSQ2d8kD10:34
jo-erlendno errors there. That's the complete file?10:36
lalejandyes10:36
jo-erlendwell, as I said, I'm no expert. But at least now you have something to show the experts. :)10:37
lalejandups, now I got errors10:37
lalejandhttp://pastebin.com/Cta6SJ2Z10:38
lalejandthis is the complete file from 1 min ago10:38
jo-erlendyes, that looks more like the ones we're all getting... Except, I get 503.10:39
lalejandarf10:40
jo-erlendbut that probably means there are some issues on your clientside. But I have no idea how to troubleshoot them, so that's for someone else.10:41
lalejandjo-erlend : on my alternative session I have error 500, and on mine I have error 400 :/10:41
jo-erlendyes, but 500 is not a problem. That's on the serverside, so that'll be fixed. :)10:44
jo-erlendI have to go. Good luck :)10:44
mandel1@ping10:44
ubot4pong10:44
lalejandjo-erlend : thx for yout help10:46
lalejand"your"10:46
mandelwhere are all the argentinians?10:49
lalejandis there a specific help/hotline for paying U1 users ?11:06
ryelalejand, /me is reading the scrollback11:08
ryehmmm11:09
ryeok, ATM the sso/u1 login is working, reading further11:11
ralsinagood morning!11:13
ryekarni, <karni> Can someone translate this for me? Google Translate can't handle it well: "Вовремя появилась. Аналогичные удалил." - "Appeared in time. Removed the similar ones" but i need more context11:13
ryekarni, for cyrillic you can poke me directly11:13
mandelralsina: morning!11:15
mandelI was wondering where everyone was...11:15
ryelalejand, so, regarding 400 BAD REQUEST (which should be 401 UNAUTHORIZED soon, actually) - this means that the account tokens that are stored locally are not usable, i.e. they have been removed from the server etc. Since it appears like you have reauthorized your machine the desktopcouch might not have picked up the new credentials (which looks like a bug)11:15
lalejandrye : just filed a bug report : https://bugs.launchpad.net/evolution-couchdb/+bug/80589711:16
ubot4Launchpad bug 805897 in evolution-couchdb "couchDB adressbook doesn't work (affects: 1) (heat: 6)" [Undecided,New]11:16
ryelalejand, you can restart desktopcouch - /usr/lib/desktopcouch/desktopcouch-stop; killall desktopcouch-service; and then run /usr/lib/desktopcouch/desktopcouch-get-port - that will restart replication with new token for sure11:16
ryelalejand, what I am very interested is the internal server error message11:16
lalejandrye : I try that on my alternative session ? or mine ?11:16
ryelalejand, let me check your account11:17
lalejandrye : one precision : I have nightly PPA packages for U111:19
ryelalejand, that should not matter for server-side errors, so that's ok11:20
ralsinamandel: sorry, I started working at 8AM today! Lazy me! ;-)11:23
mandelralsina: you lazy!!!11:24
mandelhehe11:24
faganralsina: you can always say there are lazier and get up at 7AM11:24
mandelralsina: I fixed correctly the encoding issue in that last branch and I'm waiting for the bzr push to finish… I hope it does not brake due to my connection :P11:25
* fagan needs to do a big find and replace on this rst thingy 11:26
faganmuffinresearch: little question I have the names of all of the images but I dont know what to put in before it so I can have the right link to it. What will the url be for the images?11:27
faganI can just do a s/::/::url and it would be good11:28
fagan(at least for all the images)11:28
mandelralsina: can you please take a look at https://code.launchpad.net/~mandel/ubuntuone-client/fix-805372/+merge/6688711:29
karnirye: sweet, thanks. it means the person installed Ubuntu One Files, and removed similar software ;) \o/11:29
mandelralsina: and remind me to ping alecu or nessita about it :P11:29
* karni goes back to sleep11:29
fagannight karni11:30
ryekarni, night :)11:30
karni\o =)11:30
ryelalejand, ok, it is a server-side bug, currently filed as critical and i am starting to poke developers11:30
lalejandrye : ok I will have news via my bug report ?11:31
ralsinamandel: looking11:32
lalejandrye : is there something I can do ?11:32
mandelcheers11:32
ralsinamandel: much better11:34
ryelalejand, could you please confirm that it was working before thursday/friday last week?11:35
mandelralsina: yes, that does the trick in a nicer way, and will only put funny chars in the logs since __str__ is just used for logging…11:35
ryelalejand, OTOH the couchdb/evolution nightlies may be an issue that prevents it from working in natty. rodrigo_, ping - do you have any way to test latest ubuntuone nightlies with evolution couchdb?11:36
ryerodrigo_, on natty11:36
rodrigo_rye, the nightlies don't build afaik, but once they do, they should be available for natty, shouldn't they?11:37
lalejandrye : nope, it was not working before thursday/friday las week11:37
ralsinamandel: +1 but wait for someone that knows more syncdaemon's approval11:38
lalejandrye : mi couchDB addressbook is broken from a longer time (maybe related to https://wiki.ubuntu.com/UbuntuOne/Status#Contacts "2011-06-16 Syncing of contacts in Evolution is not working for some users due to a server issue. We are looking into this and trying to resolve it ASAP")11:39
ryerodrigo_, hm, i mean that latest nightlies for natty do not work with evolution at all, leading to weird timeout problem dialogs11:40
* fagan break 11:40
ryelalejand, let me finally update the script to find out the account info... which will not work due to the bug11:40
ryegrrr11:40
rodrigo_rye, ah, haven't tested in natty for long, so will have a look11:40
lalejandrye : should I leave PPA packages and use regular ones ?11:41
ryelalejand, to get ubuntuone addressbook locally working, i am afraid, yes11:41
lalejandrye : ok, How can I downgrade ?11:42
ryelalejand, there is a ppa-purge application in ubuntu repositories11:42
mandelralsina: ok, although is more of a 'someone that know more sd on windows...'11:43
mandeland we are not that many ;)11:43
ralsinamandel: exclusive club that one :-)11:43
czajkowskimandel: oi i hope you have a lot of bugs there today mister!! cheeky11:43
mandelprecisely that one :P11:43
mandelczajkowski: hahaa it was an easy comment to make ;)11:44
* mandel thinks opera turbo is a placebo...11:44
czajkowskimandel: may the karma gods be evil to you today and fill your inbox full of bugs! :p11:44
mandelczajkowski: or a very slow connection hehehe11:44
nessitahello everyone!11:45
ralsinahola nessita!11:47
mandel@ping11:49
ubot4pong11:49
lalejandrye : I just noticed that I have also rye/ubuntunone-extras :)11:49
lalejandrye : this is for the icon in the tray, right ? I don't remember11:50
ryelalejand, that will work... most likely11:50
lalejandry : I can leave it then11:50
lalejandrye : I purge the ppa:ubuntuone/nightly, should I do something special now ?11:50
ryeyeah, /api/quota is not affected, accessing /api/account crashes11:50
ryelalejand, u1sdtool --quit; and stop desktopcouch - /usr/lib/desktopcouch/desktopcouch-stop; killall desktopcouch-service11:51
lalejandrye : ok11:51
lalejandrye : stop: Unknown job: desktopcouch11:51
lalejandah sorry11:52
ryelalejand, um...11:52
=== teknico is now known as teknico_away
ryeok11:52
lalejandwhat do I have to write ,11:52
lalejandstop desktopcouch - /usr/lib/desktopcouch/desktopcouch-stop ?11:52
lalejandor /usr/lib/desktopcouch/desktopcouch-stop ?11:52
* ralsina <=======[coffee]11:53
ryelalejand, just /usr/lib/desktopcouch/desktopcouch-stop11:53
ryeralsina << coffee;11:53
* rye hates bitwise overloading of C++ for streams11:53
ralsinarye: I am C++ incompatible ;-)11:53
lalejandrye : ok everything done11:53
lalejandI start it again ?11:54
ryelalejand, start ubuntuone-control-panel, it will start syncdaemon as well, but will show errors regarding account information, desktopcouch replication may be working and evolution.. you need to restart evolution too -- evolution --force-shutdown11:55
ryeand start evolution again11:55
nessitamandel: hi there! Could you please send your report from yesterday?11:56
lalejandrye : my laptop appears 4 times now in "connected devices"11:56
lalejandrye : \o/ couchDB addressbook looks ok11:57
ryelalejand, that's ok, you can delete other laptop entries from control panel/devices11:57
lalejandrye : ok11:57
mandelnessita: sure, sorry I left early and forgot11:58
lalejandrye : about mobile contact sync --> so we should not use it ?11:58
lalejand"on https://wiki.ubuntu.com/UbuntuOne/Status#MobileContacts it says that Mobile contacts are inactive. On http://voices.canonical.com/ubuntuone/?p=943 it repeats that Mobile Contact is OFF. But on https://one.ubuntu.com/mobile/ Contact sync is still present. So it is working or not ? We should use it or not ?"12:00
nessitafacundobatista: hola! and ping12:03
ryelalejand, hm, i am editing the wiki now and the service has been shut down, the /mobile/ page will need a copy update which I am sure is being done at the moment12:03
lalejandrye : so no mobile contact sync for now. Do we have an idea of when the new service will be available ?12:04
nessitafacundobatista: regarding your review in https://code.launchpad.net/~mandel/ubuntuone-client/fix-803984/+merge/66758, the solution that mandel proposed is what we discussed in a call with Chipaca and ralsina. We decided we will force the 'b' everywhere, except when the caller explicatively states that he does not want the binary mode on.12:04
ryelalejand, i don't have a timeframe at the moment12:05
nessitafacundobatista: the sentence "those who know how to use open" has proven to be False, since the syncdaemon code did not have the open calls with the proper mode12:05
lalejandrye : ok12:05
nessitamandel: you reverted the changes in the branch?12:07
mandelnessita: yes I did12:09
nessitamandel: why?12:09
mandelnessita: I mentioned yesterday, the changes are not big and in python 3 you will have to pass the b on linux, so why not doing it?12:10
nessitamandel: not following. What we will do with all the new bugs because we forgot a 'b' until we migrate to python 3? :-)12:11
mandelnessita: fix them?12:11
nessitamandel: we can *avoid* them, remember, "it just works". If we forget even one 'b', we will have thousand of users that will not be able to use the service12:12
mandelnessita: I mean, I dont know, I had a needs fixing with a very decent reasoning behind it from a devel from chicharra…12:12
nessitamandel: but in the meeting we agreed something else, and we even discussed what facundobatista mentioned12:13
mandelnessita: and then I mentioned it on the meeting on monday with ralsina and Chipaca, and we dicussed the comment from facu12:13
nessitamandel: and what did you agree on?12:14
mandelnessita: to follow facus comment...12:14
ralsinanessita: reverting, as suggested by one of the chicharra developers.12:14
Chipacanessita: can you take a look at the vm plz12:16
ryelalejand, the server release today will fix the account issue12:16
nessitaChipaca: which one? :1 or :3?12:16
Chipacanessita: :112:16
Chipacanessita: :3 is gone12:16
nessitaChipaca: that looks like the contents of :312:18
Chipacanope, :3 was win7_base, this is win7_00112:19
nessitaChipaca: ah, true, my confussion12:20
nessitaChipaca: well, that looks really good :-)12:21
Chipacanessita: still a lot of issues with it, but yes12:21
nessitaChipaca: you started only the control panel there?12:22
Chipacanessita: yes12:22
nessitaok, I'll try with the other services12:22
Chipacanessita: syncdaemon gives a bunch of key errors12:22
nessitaChipaca: do we have that in trunk?12:22
nessita(the styling)12:22
Chipacanessita: not yet, pep8 issues12:22
Chipacaspeaking of the devil12:22
* Chipaca grins at DiegoSarmentero 12:22
=== zyga is now known as zyga-food
ralsinaCan I get a review for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_804600/+merge/66731 ? It has screenshots!12:26
nessitaalecu: hi there! could you please send  the report from yesterday? I'm hungry to know what happened with the u1cp :-)12:32
nessitaralsina: sure12:32
ralsinanessita: thx!12:33
alecunessita, hi there. My branch to make signals removeable needs one more review: https://code.launchpad.net/~alecu/ubuntuone-client/removable-signals/+merge/6680712:34
alecunessita, it's for the error where deferreds where being called twice.12:34
nessitaalecu: I'll review after ralsina's12:35
nessitaDiegoSarmentero: ping12:35
alecus/where/were/12:35
DiegoSarmenteronessita, pong12:36
nessitaDiegoSarmentero: hi there! how is it going?12:36
DiegoSarmenteronessita, fine, i've to polish some ui, but it is on the path :P12:37
nessitaralsina: these two should be swapped, right?12:37
nessita181+    congratulations_ui,12:37
nessita182+    choose_sign_in_ui,12:37
ralsinanessita: alphabetical! Forgot :-(12:37
ralsinathey are so close ;-)12:37
ralsinanessita: pushing in 1'12:37
ralsinanessita: pushed12:40
nessitaralsina: ack12:40
Chipacaralsina: you could say 'ch' comes after 'c'12:41
Chipacaralsina: and then we'd all laugh at how *old* you are12:41
ralsinaHmmmm.... ch came after c until 15 years ago. I amover 15, indeed :-)12:43
ralsinamandel alecu fagan nessita thisfred dobey DiegoSarmentero standup in 9'12:51
thisfredack12:51
DiegoSarmenteroack12:51
faganI wonder why my nick doesnt highlight comments if my name isnt the first line12:52
faganweird12:52
dobeymeh12:52
=== teknico_away is now known as teknico
dobeybecause your irc client is lame12:52
duanedesignfagan: i had that issue too...trying to remember how i fixed it12:52
fagandobey: its irssi12:52
nessitahi dobey! question: was this fixed by your branch in ussoc? bug #805244 (report says ussoc 1.3.1 was used, so I guess no?)12:53
ubot4Launchpad bug 805244 in ubuntuone-control-panel (Ubuntu) "Method "Create Collection" with signature "a{sv}" on interface "org.freedesktop.Secret.Service" doesn't exist (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/80524412:53
dobeyralsina: btw, your branch yesterday with the voting issue; it was because you requested two reviews, not because two are required.12:54
ralsinadobey: I know12:54
dobeynessita: that is a different issue than the one i fixed, so maybe an update in gnome-keyring broke some other API :(12:54
ralsinadobey: I couldn't find how to remove one after I asked, though, so I voted it myself :-)12:54
facundobatistanessita, ping12:54
nessitadobey: can I assign that bug to you?12:54
ralsinafagan: crappy irc client?12:54
dobeynessita: sure12:54
nessitafacundobatista: pong (about to have standup)12:54
nessitadobey: thanks!12:54
facundobatistanessita, hola!12:54
faganralsina: is irssi crappy?12:55
* fagan always thought it was nice 12:55
facundobatistanessita, che, should I tell visual c++ to install mssql server?12:55
ralsinafagan: no idea, never used it. But if it doesn't highlight your name, by definition, it is12:55
nessitafacundobatista: hola! and nopes12:55
facundobatistanessita, or it is not needed?12:55
facundobatistanessita, thanks!12:55
nessitafacundobatista: anytime12:55
faganralsina: I think it might have something to do with my config12:56
mandelralsina: why do you always choose an odd number to let us know about the stand up?12:56
dobeyfagan: irc clients are like cars. stock is lame.12:57
ralsinamandel: the alarm starts at 10, then I notice, the I finish something, then I ping. Usually takes 1-3 minutes :-)12:57
fagandobey: yeah irssi would be pretty lame if you didnt config it at all12:57
fagan:D12:57
mandelralsina: that is what she said :P12:57
dobeylol12:57
fagandobey: I was looking into plugins but it seemed like a lot of effort12:57
mandelhehe12:58
dobeyfagan: you don't need an extension for that12:58
dobeyfagan: /help hilight12:58
faganahhh ok then12:58
faganthanks dobey12:58
duanedesignfagan: this is what I had in my ~/.irssi/config. Looks like I had to specify my nick. Hope that helps http://pastebin.com/WgVwGvkq12:59
=== m_conley_away is now known as m_conley
nessitamandel: aft5er an issue that DiegoSarmentero had during the weekend with our xdg, I reported:  #80592712:59
nessitabug #80592712:59
ubot4Launchpad bug 805927 in ubuntuone-client "UnicodeEncodeError when importing windows' version of xdg (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/80592713:00
ralsiname13:00
nessitame13:00
DiegoSarmenterome13:00
faganme13:00
faganduanedesign: nice thanks13:00
thisfredme13:01
mandelme13:01
alecume13:01
fagango ralsina13:01
nessitadobey: me?13:01
ralsinaDONE: windows call, closed #805461 #804562 #804569 #804271, proposed branches for #805405 #805290 #80392913:01
ralsinaTODO: close the ones I have proposed, #804610 #800376, calls13:01
ralsinaYOUR TODOS: objectives13:01
ralsinaBLOCKED: no13:01
ralsinaNEXT: nessita13:01
ralsinafagan: bit premature there13:01
faganralsina: ah sorry forgot about dobey13:01
dobeyme13:01
nessitaDONE: day off. The day before that: support to DiegoSarmentero so he had his setup ready to go for development, tested and fixed instructions at https://wiki.ubuntu.com/UbuntuOne/Contribute/WindowsTesting (they are certified now!)13:02
nessitaTODO: make control panel tests pass, assitye the rest, have 'account api' call with web team?13:02
nessitaBLOCKED: nopes13:02
nessitaNEXT: DiegoSarmentero13:02
DiegoSarmenteroDONE13:02
dobeyralsina: that's what she said ;)13:02
DiegoSarmenteroSome minor improves in the UI, ProgressBar in the right upper corner replaced for an overlay animation while Ubuntu One is connecting to the server.13:02
DiegoSarmenteroTODO13:02
DiegoSarmenteroWorking on Table style. Polish ConnectingWidget and add Ubuntu Font.13:02
DiegoSarmenteroBLOCKED13:02
DiegoSarmenterono13:02
faganDONE13:02
fagan* Fixed some of the photo bits in the rst13:02
faganTODO13:02
fagan* Finish off the branch13:02
faganBlocked13:02
fagan* nope13:02
faganthisfred: go13:02
thisfredDONE: Celebrated the birth of grilling hamburgers or something like that TODO: Check mail | Bug #762722 BLOCKED: no NEXT: mandel13:02
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/76272213:02
mandelDONE: finished bug 803984 and bug 805372, looked at the other encoding bus, we should be suing str for paths, pass the to sd, and encode them to mbcs later so ensure things do not brake13:03
mandelTODO: Ensure filesystem notifications do not use utf813:03
mandelBLOCKED: no13:03
mandelalecu, go13:03
ubot4Launchpad bug 803984 in ubuntuone-client "Windows: Inifite loop between upload started -> upload error (TRY_AGAIN) (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/80398413:03
ubot4Launchpad bug 805372 in ubuntuone-client "Windows pyinotify implementation cannot deal with utf8 (affects: 1) (heat: 360)" [High,In progress] https://launchpad.net/bugs/80537213:03
alecuDONE: a branch to fix signals not being removed (bug #805471); a few reviews for ralsina; more debugging on u1cp->sso13:03
alecuTODO: more debugging. expenses! objectives?13:03
alecuBLOCKED: no13:03
ubot4Launchpad bug 805471 in ubuntuone-client "Signals not removed means deferreds called twice (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/80547113:03
alecuNEXT: dobey13:03
dobeyλ DONE: hack day, us holiday13:03
dobeyλ TODO: bug #805244, start the shiv^Hm?, more fixes/triage/etc13:03
dobeyλ BLCK: None.13:03
ubot4Launchpad bug 805244 in ubuntuone-control-panel (Ubuntu) "Method "Create Collection" with signature "a{sv}" on interface "org.freedesktop.Secret.Service" doesn't exist (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/80524413:03
ralsinaok, comments!13:04
ralsinaPlease everyone, objectives?13:04
ralsinathisfred and dobey, got yours, need to approve them13:04
* fagan has a standing objective to be awesome if that counts 13:04
thisfredambitious!13:05
nessitaralsina: yes, objectives. Can we mumble about them after our daily mumble?13:05
ralsinanessita: of course13:05
mandelralsina, nessita, alecu: when do we have mumble?13:05
ralsinafagan: you are excepted from the objectives TODO13:05
mandelI need to have lunch...13:05
nessitaChipaca, ralsina, alecu, mandel: mumble at 10:30 ART?13:05
ralsinanessita: ok13:05
faganralsina: yeah I know :)13:06
mandelagg objectives..13:06
nessitamandel: you have 25 minutes, if that's enough13:06
mandelnessita: would not mind 10/15 more, just because I'm at my parents place and they are old people...13:07
mandelthey could choke ;)13:07
nessitamandel: so, not sure what you prefer then13:08
mandelnessita: at 45 is ok for you?13:08
DiegoSarmenteronessita, did you submit how to fix the bug reported about xdg?13:08
nessitait is to me, let me ask Chipaca13:08
nessitaChipaca: can you make it to the mumble in 33 minutes?13:09
nessitaDiegoSarmentero: yes, is bug #805927, you can subscribe :-)13:09
ubot4Launchpad bug 805927 in ubuntuone-client "UnicodeEncodeError when importing windows' version of xdg (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/80592713:09
mandelDiegoSarmentero: I fixed it, you have to pull from lp:~mandel/+junk/xdg13:09
DiegoSarmenteromandel, ok!13:10
mandelDiegoSarmentero: wait, I'm talking about a diff bug, is there a unicode error in xdg?13:10
mandelmierda..13:10
mandelputo fs de windows y la madre que lo pario13:10
mandelDiegoSarmentero: I'll take a look at it asap13:10
DiegoSarmenteromandel, no problem, i fix it on my machine... so it's working here13:10
nessitamandel: go have lunch, we will wait for you13:11
mandelDiegoSarmentero: can you put that changes in the bug, that way I dont have to think much ;)13:11
nessitaralsina: ping13:12
ralsinanessita: pong13:12
nessitaralsina: run-tests will not finish in my machine13:12
ralsinanessita: weird13:12
* ralsina checks13:12
DiegoSarmenteromandel, nessita already put the changes... it was just converting the string to unicode: https://bugs.launchpad.net/ubuntuone-client/+bug/80592713:12
ubot4Launchpad bug 805927 in ubuntuone-client "UnicodeEncodeError when importing windows' version of xdg (affects: 1) (heat: 6)" [Medium,Triaged]13:12
nessitaralsina: any clues?13:12
ralsinanessita: what's the last thing it printed?13:13
nessitamandel: so, how long you estimate for us to have the meeting?13:13
nessitaralsina: all test run OK, then,. nothing else was printed, and the command never returned13:13
ralsinanessita: if you ctrl+c/13:14
ralsinaI meant what happens if you ctrl+c?13:14
nessitaralsina: "Terminate batch job? Y/N"13:14
ralsinanessita: it happened to me once, I re-ran and it finished13:14
nessitaralsina: so, the script is running13:15
nessitalet me check13:15
nessitaalecu: about to approve your branch13:15
ralsinaand now it stuck again. Seems to happen "sometimes", although pretty often13:15
ralsinanessita: for some reason u1trial never ends13:16
nessitaralsina: is it u1trial or the style chek?13:17
nessitaralsina: is never ending here13:17
ralsinanessita: u1trial13:17
ralsinanessita: run            "C:\Python27\\python.exe" "C:\Python27\\Scripts\u1trial" ubuntuone_installer --gui --reactor=qt413:18
nessitayeah, doing that right now13:18
nessitaralsina: yes, same behvior here13:19
ralsinaSince all the tests end, I have no idea why u1trial does that13:19
nessitaralsina: did you ask dobey?13:19
ralsinanessita: no, but let's :-)13:19
ralsinadobey, I hav a problem! ^13:19
dobeyhrmm?13:20
ralsinanessita: never noticed because I always ctrl+c the run-tests to avoid the style checks when testing :-(13:20
ralsinadobey: for some reason u1trial is not stopping after all the tests are finished13:21
nessitaralsina: you should not avoid style check while developing!!!!!!13:21
dobeyralsina: on windows, or on linux?13:21
ralsinanessita: style checks on windows show hundreds of useless lines (about Ui files) and push the tests out of the buffer13:21
ralsinadobey: windows13:21
dobeyweird13:22
ralsinanessita: I forgot chipaca and I have the mgmt call in 6 minutes.13:24
alecudobey, ralsina: I get the same when running "u1trial --reactor=qt" in windows.13:26
=== zyga-food is now known as zyga
dobeyyay qt :(13:26
nessitaralsina: how long is that meeting? can you please talk about plans and account API?13:27
nessitaralsina: ping? ^13:32
nessitaalecu: your branch approved13:33
nessitaralsina: how can I IRL test yours?13:33
nessitaralsina: also, I don't think we should offer "Back" in that screen13:34
alecunessita, thanks. I saw that a few minutes ago :-)13:34
ralsinanessita: it's a one-line change nt to offer it, but why?13:34
nessitaalecu: ah! re: the unicode stuff, we should talk about that during the mumble call, which will take place soon in the future (we need to align mandel's and bosses' agendas)13:35
nessitaralsina: because at that point, I think, it makes no sense?13:35
nessitare looking the screenshot13:35
ralsinanessita: it just sends you back to the license page. Unless the user did something that can't be undone, there's not much reason not to offer the expected "Back"13:35
nessitaralsina: the screen says "UBUntu one installed!", what can teh user do if he goes back?13:36
ralsinanessita: read the license again :-)13:36
nessitaralsina: but the message say "ubuntu one has installed" <- is done and can't be undone by going back13:37
ralsinanessita: ok, convinced. Will push without a back button in 1'13:38
nessitaralsina: can you please add IRL testing isntructions? I will run it here13:38
ralsinanessita: sure13:38
nessitaralsina: also, any answer to: "how long is that meeting? can you please talk about plans and account API?" (I want to make some mate if I have time)13:39
ralsinanessita: 1 hour usually. I will ask about accounts13:39
nessitathanks, please let us know when you guys are free13:39
gordhey hey all, its now got to the point where i have so much stuff in u1 i can't sync it all on to my /home partition for lack of space. any plans of following symlinks or something this cycle so i can use my slow HDD's to sync to rather than the fast SSD i use on my home partition?13:40
ralsinanessita: pushed without "Back" and added IRL instructions13:41
ralsinavolunteers for bug #805899 ?13:42
ubot4Launchpad bug 805899 in ubuntuone-client (Ubuntu Oneiric) (and 1 other project) "Can't sign in to U1 from Live session (affects: 1) (heat: 8)" [High,Confirmed] https://launchpad.net/bugs/80589913:42
ralsinathisfred, dobey? ^13:42
alecugord, I think there's no plan for symlinks during this cycle.13:43
* thisfred looks13:43
dobeyralsina: dup13:44
thisfredralsina: hmm, looks like we're missing dependencies on the CD13:44
mandel1nessita: I'm bacl13:44
mandel1back13:44
dobeythisfred: eh?13:44
ralsinadobey: can you mark it dup?13:44
dobeyralsina: already done13:45
ralsinadobey: cool13:45
ralsinadobey: thanks!13:45
gordalecu, oh thats a shame, i guess there is no solution to my problem other than stop syncing things or get a larger SSD right?13:45
mandelnessita: ups, the irc client let me down...13:45
thisfreddobey: well if the dbus service we need isn't running, I'm guessing something on the live CD is missing that we need (and that may not be explicitly listed in our dependencies)13:45
dobeygord: is it all in ~/Ubuntu One, or in other folders that you've synchronized as well?13:46
thisfreddobey: oh, already known, nm13:46
dobeygord: because you can choose to not sync some of those other folders to that disk13:46
thisfreddobey: I think gord wants to sync more, not less,13:46
alecugord, perhaps mounting your hdd as /home (or /home/gord) and having symlinks to the sdd for the folders you want to be fast. But it sounds like a lot of trouble.13:46
nessitamandel: you had lunch already?13:47
mandelalecu: so, regarding the encoding, the problem is that pyinotify gets the paths as a sequence of bytes (str in my email, ups!) and sd has the logic to deal with them and use utf-8 against the server, on the windows side we get the paths as a encoded string in mbcs, so we have to make the pyinotify implementation to use bytes so that we need to change nothing on the other side13:47
mandelnessita: yes I have13:47
nessitamandel: ok. Bosses remembered they have the weekly management call, so we're waiting for them13:47
alecunessita, do you know if we can have udfs for folders symlinked in /home from another filesystem?13:47
dobeyalecu: you can't13:48
dobeyalecu: symlinks get nothing13:48
gordyeah i need to sync more but not on my /home partition, what are the requirements u1 needs to allow syncing? does it have to be in /home? certain filesystems?13:48
thisfredalecu: afaik no links, sym or hard will ever be synced13:48
nessitaalecu: nopes, and the reason is we don't allow UDFs to be symlinks13:48
dobeygord: has to be under /home13:48
nessitaalecu: part of UDF validation is checking that the path is not a link13:48
thisfredwhich I still disagree with, but there you go. I think dropbox shows it can be done in a non-confusing way13:49
alecunessita, so there's no solution to gord's problem above? having a small fast ssd as /home, and no way to sync a bigger hdd?13:49
dobeygord: because there is no way for us to guarantee we can sync to the same place outside of /home on other systems.13:49
dobeyprecognition is not a feature of u1 :)13:49
thisfreddobey: which is why we should sync them as real files13:49
nessitaalecu: not at the moment, unless mounting the FS via some fancy mount method13:50
dobeythisfred: sync what?13:50
alecuthisfred, gord, I agree that we should sync symlinks too.13:50
alecunessita, who may know what that fancy method is?13:50
dobeysyncing symlinks is useless13:50
thisfreddobey: sync symlinks as actual files they represent, just like dropbox does (I think). Unless they exist as symlinks on the target system13:51
alecudobey, s/syncing symlinks/syncing stuff from other filesystems/13:51
dobeythisfred: i see no actual value in it13:51
thisfreddobey: I've used it and I do13:51
gordyeah i don't actually want to sync symlinks, i just want to "mv ~/Music/ /storage/Music; ln -s /storage/Music ." and have u1 play nice with it13:52
thisfredright13:52
dobeywhich is basically impossible to do13:52
thisfredso u1 should follow symlinks13:52
thisfredwhy?13:52
thisfredwell if the disk is not mounted you want to not delete it all I guess13:52
ryein case /storage is a mountpoint and it gets unmounted, syncdaemon will be happily removing the files from online storage13:53
ryebeen there, done that13:53
nessitaalecu: let me look the bug report with some instructions13:53
thisfredbut you could maybe autounsubscribe folders on filesystems that are not mounted13:53
dobeybecause we don't necessarily have write access to same place on ohter systems13:53
dobeyand what happens if it's an external disk, and it gets plugged into another system, that's also synced with u113:53
thisfreddobey: If we treat it as a folder in the place it is mounted, it will just create a new folder on the other system13:54
dobeyuntil we are providing a proper back-up service, handling anything outside home is not feasible13:54
gordwell you don't sync the symlink surely? in my case the music folder on system B would just get created and populated as a directory Music in home13:54
thisfredunless you already have that folder there, and then it will sync into it13:54
thisfredwhat gord says13:55
joshuahooverfagan: ping13:55
dobeythe problem is that pretty much anything we decide to do with symlinks/folders outside home, will never actually be what the user wants, as long as we are a sync service, and not a back-up service.13:55
thisfredgord: This discussion has been had 3 years ago, though, so I don't hold out much hope for a policy change13:55
alecuthisfred, we should push for one, because gord is not the first user to ask for it.13:56
alecuat least a policy review.13:56
thisfreddobey: it will be what the user wants: if the user knows what symlinks are, they are sophisticated enough to understand the solution we provide, which will have some corner cases, I admit13:56
alecuI have the exact same issue on my new desktop.13:57
dobeythe only people really asking for it are hackers at canonical, and ubuntu/debian developers. :)13:57
thisfredyes, but if we can help them without breaking it for the other users, why not?13:57
dobeythisfred: it will never be what the user wants, unless the user already knows what we're going to do and they consciously decide that is what they want, after the fact13:57
nessitaalecu: can't find the bug report, but I think jdo was a commenter there13:57
thisfreddobey: again, like it is with dropbox13:57
dobeythisfred: but going into it, it's almost never what they want13:57
thisfreddobey: I agree you have to find out how it works, and then adapt to that13:58
dobeyi don't know what dropbox does with symlinks, but i guarantee it's the wrong thing13:58
thisfreddobey: but it makes things possible that are not possible now13:58
dobeyif it doesn't Just Work (TM), then it's broken.13:58
thisfreddobey: no it's not, it works just fune13:58
thisfredfine13:58
dobeyno it doesn't13:58
thisfredfine13:58
dobeyhow do you sync a symlink to windows?13:58
thisfredIt does what I want it to do13:59
dobeyyes, and you are not anywhere near a typical use case :)13:59
thisfreddobey: As I keep saying, it's only a symlink on the original system13:59
thisfreddobey: which is the problem people keep asking us to solve, nothing more13:59
dobeythat is only the first time you've said that13:59
thisfredno13:59
gordi don't actually care about symlinks, its just there doesn't seem to be a solution to having a lot of data in ubuntu one and trying to use ubuntu one with small drives13:59
dobeyyes :)13:59
dobeyand it's not the problem people keep asking us to solve, unfortunately14:00
dobeybut anyway, whatever14:00
alecugord, so, as a temp solution, you may "mount /dev/hdd ~/mount"; then make udfs out of ~/mount/Music, etc, then symlinks in ~/Music to ~/mount/Music14:00
dobeythere are much worse problems to deal with14:00
thisfreddobey: read what gord is asking, that is exactly the problem he is having, he wants to sync stuff not in his home. He doesn't care about symlinks14:00
alecugord, but be very careful with the thing that rye commented above, about not unmounting that while syncdaemon is running14:01
dobeyi read what gord is asking.14:01
thisfredalecu: still have to be very careful with that: unmount your external HD, and it might delete all your music14:01
alecugord, because if it detects that those files were deleted, it will start deleting them on the cloud.14:01
thisfredright14:01
alecugord, it's not external, right?14:01
gordalecu, huh and that should work? will have to have a play, and yeah its internal14:02
thisfredalecu: I think if SD knew about symlinks, or was just plain able to sync stuff outside home, we could detect unmounted filesystems and unsubscribe them or mask them temporarily pretty easily14:03
dobeygord: if you're going to do the mount thing, make sure it gets mounted before you log in from gdm/lightdm14:03
thisfredThis is what my music player does for disks that aren't mounted14:03
alecuthisfred, absolutely. Sounds great.14:03
dobeythe problem is that sd uses pyinotify directly14:04
dobeythe problem is that "sync stuff outside of home" is not feasible.14:04
thisfredWell I don't know much about pyinotify, but I very much doubt it sends 'deleted' events for files on disks that are unmounted14:05
dobeythere is "back-up stuff outside home" which is a different thing14:05
dobeyand we are not a back-up service yet14:05
dobeythisfred: what i mean, is that if it used gio, a lot more useful info would be available14:05
thisfredIt's totally feasible, as long as you let go of the idea that all UDFs have to live in the same place on each device14:05
* nessita -> brbs14:05
dobey(plus it would be cross-platform already)14:05
thisfredwhich assumption the mobile clients have already broken14:06
dobeythisfred: as long as you ignore the fact that we are a sync service, yes14:06
thisfreddobey: well let's switch to that14:06
dobeythe mobile client doesn't sync14:06
thisfreddobey: we're syncing files and folders, not filesystems14:06
thisfreddobey: yes it does, on a file by file basis14:06
dobeyno, it browses and lets you download a file. it's the same as the web client14:07
thisfredanyway, discussion interesting, but not going anywhere :)14:07
dobeyso hell, just use the web client then; it's a solved problem14:07
thisfredwell in that case it should be made to14:07
dobey∎ :)14:07
dobeyi'm sure making 10000 folders and uploading 5000 files individually would make an interesting 6 years of one's live14:08
dobeylife14:08
thisfredI don't understand what you're saying there14:09
thisfredIf I explicitly download a file from U1 to my mobile client, I don't see why it shouldn't be kept in sync14:09
dobeyi'm saying the argument that android breaks the sync assumption in some way is moot, because it's the same as the web client14:09
thisfredwell, that's great for a first iteration, but I'm sure we're more ambitious than that in the long run14:10
dobeyno i think it does keep it in sync on android14:10
dobeybut it's nothing at all like solving the problem of syncing external drives on a PC14:11
dobeythey are very different problem spaces14:11
thisfredwell it's all sync, it has nothing to do with backup14:11
dobeyit isn't all sync14:11
dobeyanyway, this is going nowhere14:11
thisfreddobey: well then you use confusing language since that's what you just called it14:12
mandelkiss kiss kiss14:12
mandel:P14:12
thisfred:)14:12
thisfredpistols at dawn? ;)14:13
dobeytoo early14:13
thisfredtrue14:13
thisfreddrinking contest at midnight?14:13
dobeythat works14:14
mandelhehehe14:14
nessitamandel: where are logs now, in windows, using latest xdg?14:21
mandelAppData\Local\xdg\cache\14:21
mandelnessita: we are using the xdg_cache path for logs and partial files14:22
nessitamandel: ack, thanks. I m about to report 2 not-minor bugs in syncdaemon. One makes local rescan crash so syncdaemon does nothing else14:22
mandelnessita: ok, I'm close to be done with the UploadCMd tests so that facundo can give the green light and will move to those before I deal with the utf-8 issue14:23
nessitaack14:23
mandelnessita: that way we can talk about the udf8 issue at the meeting, but it is a no brainer :P14:24
nessitamandel: no brainer?14:24
nessitanot sure what that means in this context14:24
mandelnessita: facil14:24
nessitamandel: hum, I doubt it :-) unicode and file path names are never easy14:25
mandelnessita: well, I know where the code has to go, and I should be able to understand it… and I have no brain :P14:26
mandelnessita: have you had any news regading the mumble?14:27
nessitamandel: as soon as the manager finish their call. ralsina, Chipaca: any ETA for our daily mumble?14:28
ralsinasorry guys, it started a bit late, and we are all a bit late14:28
nessitamandel: I'm about to report paste.ubuntu.com/63844614:29
nessitaralsina: you added IRL testing instructions to the commit message in https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_804600/+merge/66731 :-)14:29
dobeygrr, power blip14:29
mandelnessita: can you do me a favour, can you get into mumble and let me know if you can hear me?14:30
ralsinanessita: yikes, fixing14:30
nessitamandel: yes, let me report this bug first14:30
mandelnessita: sure14:30
ralsinanessita: there14:31
alecumandel, back to the unicode issue. Above you said "on the windows side we get the paths as a encoded string in mbcs"14:31
mandelalecu: yes14:31
alecumandel, in python, it means either a secuence of bytes in "UTF-16/UCS-2" encoding, or either a unicode object.14:32
alecumandel, what do you mean by mbcs?14:32
alecumandel, so, how does pyinotify gets the paths on windows?14:32
mandelalecu: http://docs.python.org/howto/unicode.html#unicode-filenames14:32
aleculooking14:32
alecuooook.14:33
mandelalecu: the paths are returned by a call to COM14:33
mandelalecu: does it make sense?14:33
alecumandel, ok. But that COM call returns a type(unicode) object, right?14:35
mandelalecu: yes14:35
mandelalecu: the code works as following, we get the events from COM, get the to a Q so we do not loose any events, in a diff thread, read the Q and create pyinotify events as the ones you would get on Linux14:36
alecumandel, ok, and internally syncdaemon handles everything as type(unicode). The conversion from "linux filesystem encoding" to type(unicode) is done on the linux platform specific stuff.14:36
alecumandel, ahhhh ok.14:37
nessitamandel: bug #80598114:37
alecumandel, then that means that the "synthetic pyinotify events" you are creating should have "bytes"14:37
ubot4Launchpad bug 805981 in ubuntuone-client "Windows: KeyError during local rescan, which makes syncdaemon do nothing else (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/80598114:37
mandelalecu: yes, exactly14:37
alecumandel, should have "bytes in some encoding" as the filenames14:37
alecumandel, that's why you'll convert the type(unicode) filenames as returned from COM into the "bytes" required by the pyinotify events using some encoding. Say "utf-8".14:38
alecudid I understand correctly?14:39
CardinalFangEww, found why Android music app doesn't load some  album art.  "GraphicsJNI  E  VM won't let us allocate 5120000 bytes"14:39
CardinalFang"bitmap size exceeds VM budget"14:39
* CardinalFang boggles.14:39
ralsinaCardinalFang: WTF is that image? A 1024x768 BMP/ :-)14:39
mandelalecu: yes, that is what we have to do14:39
beunoCardinalFang, nice!14:39
alecuCardinalFang, here's ten bucks. Get a gig'o'ram.14:39
beunoCardinalFang, that's because we don't resize the images on the server14:39
ralsinaalecu: in a phone? Good luck with that :-)14:40
beunowe send whatever is embeded14:40
CardinalFangalecu, in my phone?  Yes please.14:40
mandelalecu: but ofcourse we also have to make sure that the os_helper layer knows about this too14:40
beunoCardinalFang, in 3 months or so we should be able to resize images, but, you know, 3 months14:40
CardinalFangbeuno, I can do it at download time.14:40
alecuralsina, as long as the image has to be uncompressed before displaying, packing it with jpeg2000 would not be useful either :-)14:41
beunoCardinalFang, that'd be awesemoner14:41
alecumandel, why does the os_helper layer need to knows about this too?14:42
* alecu does not know about the os_helper layer.14:43
Chipacanessita: now?14:43
dobeygrr, evil broken APIs14:43
alecumandel, btw: please let me review those branches.14:43
mandelalecu: sure, I'll add you as a requested review14:44
mandelalecu: the os_layer wants to be writing to the fs in the correct encoding, that is mbcs which ever that is in your installation14:44
mandelnot plain utf814:44
mandelI mean, not the utf8 we use in the server side14:44
alecumandel, let's not care about server side.14:45
alecumandel, server side we use type(unicode), we use utf8 for the network, and we use type(unicode) internally in syncdaemon as well.14:45
nessitaChipaca: yes!14:45
mandelalecu: god…14:46
alecumandel, but for writing on windows from python we should be using the same type(unicode) objects we use inside syncdaemon.14:46
ralsinasorry, was fetching coffee14:46
mandelalecu: are you sure about that?14:46
alecumandel, we should not be converting that to the encoding specified by mcbs, because python should do that in the best way.14:46
mandelalecu: ha.. should do that, we need to confirm that assumption :P14:47
alecumandel, as a general python knowledge, I am sure. But perhaps syncdaemon does something different I am missing.14:47
nessitaChipaca: coming to mumble?14:47
Chipacanessita: am there14:49
dobeyugh14:52
dobeynessita: why is sso tests printing log messages to console in middle of tests?14:52
nessitadobey: got a trace? (we changed nothing related AFAIK)14:53
dobey    test_delete ... ERROR:dbus.connection:Unable to set arguments ({'org.freedesktop.Secret.Collection.Label': dbus.String(u'sample_keyring', variant_level=1)}, '') according to signature u'a{sv}': <type 'exceptions.TypeError'>: Fewer items found in D-Bus signature than in Python arguments14:53
dobey                                                       [OK]14:53
dobeyi am getting that in trying to fix bug #80524414:54
ubot4Launchpad bug 805244 in ubuntu-sso-client (Ubuntu) (and 1 other project) "Method "Create Collection" with signature "a{sv}" on interface "org.freedesktop.Secret.Service" doesn't exist (affects: 2) (dups: 1) (heat: 16)" [Medium,Triaged] https://launchpad.net/bugs/80524414:54
dobeyand trying to maintain compat with older version14:54
dobeyand i am trapping TypeError and not re-raising it, so not even sure why it would end up in a log message, since i'm not logging it14:54
joshuahooverfagan: ping14:56
ralsinamanuel: lost you on mumble15:21
nessitadobey: we saw this behavior a lot in the past, when something "breaks" in the background somehow the debug mode gets enabled15:21
dobeymakes no sense :(15:26
dobeysoi'm surprised that people are actually hitting this bug15:27
dobeyit implies there's no default keyring set up yet, which would be very odd15:27
dobeyi guess maybe on a fresh install is possible, but still quite odd15:28
dobeyespecially if on wifi15:28
dobeynessita: https://code.launchpad.net/~dobey/ubuntu-sso-client/fix-805244/+merge/6691815:32
dobeynessita: i am pretty sure that solves the bug, but the test thing is bugging me still :-/15:32
thisfreddobey: well the latest report was from someone running the live CD, there it's less surprising I gyess15:36
dobeyeh, i just wish people would stop breaking API with no sane way to keep compat15:37
mandelalecu: ping15:44
alecumandel, pong15:44
mandelalecu: got an encoding related branch for review, got time for it?15:45
mandelthis is the one: https://code.launchpad.net/~mandel/ubuntuone-client/fix-805372/+merge/6688715:45
dobeyChipaca: the lint issues need to be fixed in the branch from DiegoSarmentero you are trying to land. why is it not proposed/owned by DiegoSarmentero directly?15:45
alecumandel, looking15:45
mandelthx15:46
nessitadobey: ack, sorry for the delay was on a call15:49
* mandel walks dog15:49
Chipacadobey: because his 'push' was failing. But he's got it working, and fixed the lint issues, and will be pushing it later today.15:49
dobeyok15:50
jo-erlendwhy does computers show up several times in the u1 client?15:56
dobeynessita: no worries; i'm going to head off to lunch right now. thanks :)15:57
dobeybbiab15:57
Chipacajo-erlend: http://askubuntu.com/q/19792/71115:59
* ralsina goes eat16:00
=== ralsina is now known as ralsina_eating
jo-erlendChipaca, that seems to indicate that the computer will be listed once per application16:00
jo-erlend? That can't be right?16:01
Chipacajo-erlend: not sure what you mean16:01
=== DiegoSarmentero is now known as DiegoSarmentero_
=== DiegoSarmentero_ is now known as DiegoSarmentero
Chipacajo-erlend: what part indicates that? once per what application? what do you mean by 'application'?16:01
jo-erlendChipaca, not your answer, but the other one. Your answer doesn't really answer the question, I think. Why is a computer authorized several times?16:02
Chipacajo-erlend: because you deleted the key from your keyring, for example16:02
Chipacajo-erlend: several ways, but that's probably the most common16:03
jo-erlendChipaca, «If you have multiple device entries for Ubuntu One you can use the following steps to determine which devices to keep and which to delete. Be aware that it is not uncommon to have two entries. One for Ubuntu One and one for Tomboy.»16:03
jo-erlendthat's what I meant. That didn't sound right.16:04
Chipacathat is absolutely correct16:04
Chipacajo-erlend: *in seahorse*, there'll be one key per application16:05
jo-erlendah! I misunderstood then.16:06
Chipacajo-erlend: if you log in to software centre (to purchase things, for example), you'll have a key for it also16:06
Chipacathe ubuntuone-control-panel (or in older ubuntus, ubuntuone-preferences) will list only those that are for Ubuntu One16:06
Chipacajo-erlend: server-side, on the sso site you'll also have one key per app, whereas on the u1 site you'll only have the u1 keys16:07
jo-erlendright. Think those keys in gnome-keyring should be better labeled.16:09
nessitamandel: poing16:14
nessitamandel: ping16:14
nessitamandel: ping16:30
nessitaok, lunchtime16:33
mandelnessita: pong16:45
* mandel back16:45
alecumandel, ping16:46
mandelalecu: pong16:46
alecumandel, I'm trying to understand the branch that fixes pyinotify.py16:46
mandelok16:47
alecumandel, it seems that the only place used for that output_format is in the __repr__ of the event, right?16:47
alecumandel, that means that the pyinotify in the windows dir is handling file names as type(unicode)16:47
mandelalecu: yes, that is correct, which was giving an exception when we wanted to write the logs16:49
alecumandel, ok. And btw: why do we include pyinotify inside our own code?16:49
mandelalecu: because that is a subset of pyinotify, is just the events, i has no code regarding how those events are gotten from the OS16:50
mandelalecu: so instead of implementing something similar, I grabbed the minimun I needed and used it16:50
mandelthe license etc is ok, right?16:51
alecumandel, I think not. If the owner signed a contributor agreement... blah blah blah.16:51
mandelalecu: really? mierda contributor agreement...16:52
mandelalecu: so wants the point of open source if we cannot use it, and ofcourse I cannot do easy_install pyinotify on windows...16:52
alecumandel, also... I understood from what we talked before that we were going to handle all the filenames in the "windows synthetic pyinotify events" as byte. But this clearly shows that we are handling them as unicode, so I'm puzzled.16:52
alecumandel, good point on the open source bit.16:53
mandelalecu: well, right now, the formatter brakes if the data is unicode, which only happens with path, but nevertheless it should ot brake16:54
mandels/ot/not16:54
mandelalecu: so, we fix that in case we have unicode comming from somewhere, next we deal with the file paths so that they use byte and not unicode, which the formatter per encapsulation, should not care about16:55
* mandel hopes he makes sense16:55
alecumandel, oh, ok. that makes sense.16:55
mandel:D16:55
alecumandel, I'm approving the branch, and we'll ask everybody on the contrib agreement after they return from lunch. Or tomorrow.16:56
mandelalecu: ok… but this contributor agreement is a PITA...16:57
alecumandel, I know.16:57
=== beuno is now known as beuno-lunch
nessitamandel: any updates regarding the bug you filed to pywin32 upstream?17:01
nessitamandel: I'm running a non patched version and I'm having no issues at all17:01
mandelnessita: I was going to send the version of pywin32 I was using, but I need to use a clean vm.. will do later this evening17:02
mandelI don't want to loose time on the vm right now17:02
nessitamandel: ok, for now I removed those steps (the patching of the win32) from the instructions17:03
nessitadobey: approved17:04
mandelok17:04
nessitaralsina_eating: you need more reviews?17:04
nessitamandel, alecu: you need reviews? (IRL @ win7 in particular)17:04
nessitaif not, I'll start making the u1cp tests run in win717:04
alecuI'm fine, thanks.17:05
mandelnessita: so far I don't17:05
mandelnessita: I'm moving to your rescan issue17:07
nessitamandel: yey!17:07
nessitamandel: I had to remove the file in order to have a working syncdaemon17:07
nessitamandel: so you may need to create your own ugly?.png file to play with :-)17:07
mandelnessita: got plenty of those :)17:08
nessitamandel: then how come you did not hit this bug?17:08
mandelnessita: because I never did I rescan… I always got them and remove the files because I was testing other things...17:09
mandelnessita: you got further than I did in the IRL tests17:09
mandelI think I hit something else before that...17:09
nessitamandel: the rescan is just the LOCAL_RESCAN stage, which takes place before any other action (INIT -> LOCAL_RESCAN -> [LOCAL_RESCAN_DONE] -> everything else17:10
nessitaso I'm puzzled17:10
mandelpuzzled? why>17:10
mandel???17:10
nessitamandel: why you did not hit this17:11
mandelI got to the point where the files where written on the fs, then removed everything Ubuntu One/ xdg etc...17:11
mandeland never got to do a rescan with data in the hd17:11
nessitamandel: ah, I see417:11
* alecu lunches17:13
=== alecu is now known as alecu-lunch
=== alecu-lunch is now known as alecu
=== ralsina_eating is now known as ralsina
=== beuno-lunch is now known as beuno
mandelnessita: I found the issue with the local_rescan, listdir has to be fixed to ensure that the paths are not returned with utf8 chars, got the test done will push as soon as I have fixed the method18:15
nessitamandel: on mumble, one sec18:18
facundobatistamandel, what do you mean "with utf8 chars"?18:20
mandelfacundobatista: fear not ;) I mean that we swap ? for an interrobang so that the user can see his illegal path on windows, the pyinotify implementation we have takes care of that case and swaps it, but listdir was not doing so18:21
mandelfacundobatista: so metadata ignores the fact that we have funny utf8 char, and we do not loose files so that the user can have them synced.. also the chars are very strange ones I'd would be very very strange if the create a file with that path on windows18:22
ryemandel, we have files causing oopses having \n in them18:23
ryemandel, i mean real newline18:23
mandelrye: really… people should take a test to be able to use a computer… why would they have a \n?18:24
mandelthat most be an app doing something stupid...18:24
mandelwell, I'm going to call it a day, catch you all tom18:30
* mandel EOD18:30
nessitamandel: report pliz?18:30
ralsinaI'm back but will have to babysit for a little while. Will make up for it late tonight. Ping me because I am around!18:37
dobeyhrmm18:38
dobeyis there a nifty way to get the byte difference between two strings in python?18:38
dobeyhrmm, or maybe that's not what i want18:39
ralsinabyte difference? As in "what bytes are different?"18:44
dobeyyes, or if there are any different bytes18:45
dobeyam having problem with launchpad api :(18:45
ralsinaalecu: different bytes? s1 != s2 :-)18:46
dobeyand str() of an entry object in the launchpadlib api, gives the URL for the resource, but to my eyes they look the same18:46
alecuralsina, what?18:46
dobeythough python is saying 1 != 218:46
ralsinadobey: using filter?18:47
nessitaralsina: since humanize is defined in ubuntuone/controlpanel/gui/__init__.py, the test should be in ubuntuone/controlpanel/gui/tests/ (and you put it inside qt tree). I'll move it18:47
dobeyralsina: no. i'm just doing if foo == bar:18:48
ralsinadobey: apply a filter function over zip(s1,s2) and you should get the exact difference18:48
dobeyand it's failing, when it shouldn't18:48
dobeyit's quite weird18:49
ralsinanessita: I put it there because the gtk tree doesn't use this humanize, but that's ok18:49
dobeybecause it's not failing for everyone :(18:49
ralsinadobey: but you have a case that fails?18:49
dobeyralsina: yes, multiple it seems, and multiple that work. is very weird18:50
ralsinadobey: if s1 and s2 are the allegedly alike strings: filter(lambda t: t if t[0] == t[1] else None, zip(s1,s2)) [untested like crazy]18:51
ralsinadobey: sorry I meant you have a specific example copied somewhere that fails :-)18:51
dobeyWTF: https://api.launchpad.net/1.0/~adorilson != https://api.launchpad.net/1.0/~adorilson18:52
nessitaralsina: did you talk to Claire re: sso spec?18:54
ralsinanessita: pinged her, got no response18:55
nessitaralsina: could you please write her an email?18:56
ralsinanessita: sure, with you an copy18:56
nessitathanks!18:56
ralsinadobey, are those two copied and pasted?18:56
dobeyralsina: that line is a copy and paste of a direct print statement. i added quotes around them after that, and no visible character differences within quotes :(18:58
ralsinadobey: could there have been a trailing space?18:58
nessitadobey: can you print repr of the strings?18:58
dobeyWTF: "<person at https://api.launchpad.net/1.0/~adorilson>" != "<person at https://api.launchpad.net/1.0/~adorilson>"18:59
dobeyis repr18:59
nessitadobey: you're using == for comparing objects that do not define __eq__?19:00
nessitathat will compare id's of the objects19:00
dobeyi don't know if they define __eq__ or not19:00
nessitaseems like no?19:00
dobeythen why would it work sometimes?19:01
ralsinadobey print thingie.__eq__ and you will know19:02
dobey<bound method Entry.__eq__ of <person at https://api.launchpad.net/1.0/~adorilson>>19:02
dobeyso it is defined19:02
dobeyso wtf :)19:02
ralsinadobey, therefore, it sometimes fails because __eq__ is buggy ;-)19:02
dobeyugh19:18
dobeythe etag is different for some reason19:18
ralsinadobey: could be different apaches answering?19:21
dobeyi hope not19:22
dobeyralsina: and i'm not sure it matters as i think this is a value stored in the db19:23
ralsinaetag? In the db? weird.19:23
dobeywell not sure19:25
dobeybut the etag seems to have two hashes in it19:25
dobeyand the first one is different, but the second is the same19:25
dobeyit's quite odd19:25
dobeyand this is a very annoying bug for us :(19:26
ralsinaetags are for cache invalidation, comparing objects based on their etags is kinky, at least.19:28
dobeysigh19:31
dobeyand #launchpad-dev is totally dead19:51
beunodobey, it's post-thunderdome sprint19:52
beunoI'd guess people are jetlagged, on holiday and tires19:52
nessitadobey: did you and ralsina find any solution to the issue where u1trial gets "stuck" when using qt4reactor in windows?19:59
ralsinanessita: I have been looking at it20:00
dobeynessita: no, i'm not sure what to say about it. and i fixed the sso bug, and ran into this lp bug20:00
ralsinanessita: without much success. It seems there is a thread leftover or something like that, based on a message I get when stopping it20:00
dobeyit would be nice to know what is keeping it running exactly, and if the same thing happens on linux with qt420:01
nessitaralsina: QOject ::killTimers: bla?20:01
ralsinanessita: right20:01
nessitadobey: it does not happen on linux, confirmed20:01
dobeyok20:01
ralsinabeuno: people should not do sprints in the thunderdome. Not ergnonomic at all.20:02
ralsinanessita: and debugging stuff on windows is... semi-impossible?20:02
dobeyralsina: well, it makes room for new hires20:03
nessitaralsina: we can always move the style check to the top, before the tests are run20:03
ralsinadobey: yes, but HHRR complains about the turnover, and maintenance about the blood.... not worth it.20:03
ralsinanessita: agreed20:03
ralsinaThat way It's "fixed" with a Ctrl+C20:03
ralsinanessita: want me to add it to the branch you review?20:04
nessitaralsina: does your branch pass stule checks?20:04
ralsinanessita: yes!20:04
nessitaralsina: then please!20:04
dobeyralsina: is something in the windows tests creating another qt main loop or thread or something?20:05
dobeyand not properly disposing of it20:05
nessitadobey: in the control panel case, it's the exact same suite on both ends20:05
ralsinadobey: could be20:06
nessitadobey: so not sure how that can be happening20:06
ralsinanessita: pushed with the tests at the bottom20:07
nessitaralsina: that branch was already approved by me, you knew that, didn't you? :-)20:10
ralsinanessita: for some reason some launchpad mails take foreeeeeeever to arrive20:10
ralsinaSo no20:10
nessitaralsina: ah :-)20:11
ralsinaBut hey, that means I can merge it ;-)20:11
ralsinanessita: are you ok with 1-review for the installer?20:11
nessitaralsina: yes (given the shortage of reviewrs)20:12
ralsinanessita: ok, +1ing myself and will stop asking for 2nd reviews20:12
nessitaralsina: heh, I mean I'm ok with this branch, but i think in general ew should aks 2 reviews20:13
ralsinanessita: ok ;-)20:13
ralsinalet's take it case by case20:13
ralsinanessita, alecu: review please (not urgent): https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_803929/+merge/6682720:22
=== Stanley is now known as Guest71626
pandinusHello what's exactly the problem "Our Notes, Contacts and Bookmarks database is not responding at the moment."20:30
alecunessita, ralsina: I need your opinion on some matter...20:30
ralsinaalecu: sire20:30
nessitaalecu: shoot20:31
alecunessita, ralsina: in syncdaemon tool, I have a long number of methods. Long.20:31
alecunessita, ralsina: I need to apply a new decorator to *most* of them20:31
nessitabut not all?20:31
alecu*all* of syncdaemon tool public methods, except for __init__ and a few others.20:31
ralsinaalecu, nessita: and a tiny tiy branch for SSO ready for review: https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_805405/+merge/6678420:31
ralsinaalecu: want to do it via instrospection?20:31
alecu(__init__ and _get_dict and wait_connected are the ones that do not need it)20:32
alecuralsina, I was thinking metaclasses. Or something like that.20:32
nessitaalecu: override __get_attribute__?20:32
alecuperhaps I can even rename wait_connected to some other thing...20:32
ralsinaalecu: well, it does bring python black magic into syncdaemon which is scary. I prefer nessita's idea20:32
nessitaralsina: needs tests, also, question: you know that error['__all__'] will not show all the errors?20:33
ralsinanessita: how unintuitive :-)20:33
nessitaralsina: heh20:34
nessitaralsina: __all__ are those errores at form level20:34
ralsinanessita: it's used on a specific case (bad answer to recaptcha) and there, the only field is __all__20:34
alecu(just counted, it's 51 methods :P)20:34
nessitaralsina: not sure what you meant. The __all__ is sent by the SSO server and can have a variety of errors, not only bad captchas20:35
alecunessita, so, __get_attribute__ sounds reasonable, but I wonder if it is a bit of black magic also.20:35
alecuperhaps adding the 51 decorators makes more sense. Or perhaps not, because if we add yet another method we'll have to remember to add the decorator.20:35
ralsinanessita: then what should I display? There is nothing else when the problem is the recaptcha20:35
nessitaalecu: yes to __get_attribute__ being a bit of black magic, but since the def will be in the same module, I prefer this solution to 51 decorators20:36
ralsinaI prefer __get_attriute__ to metaclasses20:37
nessitaralsina: you need to handle all the fields from that error dict. You can have specific errors for specific bits, such as 'email' and 'password'20:37
alecuok, you guys have convinced me. Thanks!20:37
nessitaralsina: when you get a __all__ error, you *also* need to show an error at form level20:37
alecuguys and gals :-)20:37
nessita:-)20:37
nessitaralsina: not sure if what I say is clear...20:38
ralsinanessita: I am getting *only* a __all__ saying "Bad recaptcha solution"20:38
nessitaralsina: well, that is your particular case. But the spec is different20:38
ralsinaOr I may be wrong, will test different errors20:38
nessitaralsina: the API says:20:38
nessita"we'll send a dict with errors back to you"20:38
nessitajust like django validation does20:39
nessita(and, implementarion detail, right now is the same dict, AFAIK)20:39
ralsinanessita: will try it and get back to you on the report then :-)20:39
nessitaralsina: so, despite your IRL tests can't display any other field, we need to handle the error dict according to the spec20:39
nessitaralsina: GTK version already does this, we may need to move that to a common location20:40
ralsinanessita: I will look at that code, too20:40
nessitaralsina: see gtk/gui.py _build_general_error_message and20:41
nessitaon_user_registration_error for example20:41
ralsinanessita: but if it's like a django validation, the recaptcha error shouldnot be form level, so there is some other bug there.20:41
nessitaralsina: we handle error['email'], error['password'], error['message'] and error['__all__']20:42
nessitaralsina: There may be a bug there, yes. In that case we need to handle in client and report to upstream20:42
ralsinanessita: yep, will double-check and report20:42
nessitathanks20:42
dobeyalright all. have a good evening20:51
ralsinanessita: here's the current behaviour: If the captcha is wrong, only the "wrong captcha" message is given in __all__, even if other fields are wrong too. Then again, I suspect that is intentional.21:02
nessitaralsina: not sure what your conclusion is then21:03
nessita(I still think we need to properly handle the error dict, since this can change server side and our client will blow up)21:04
ralsinaThat I had to reimplement the method, and it will be done in 2' ;-)21:04
ralsinaOTOH, the GTK code doesn't seem to handle __all__ in on_user_registration_error and the docstring for that method is just wrong.21:04
ralsinaOk, it does handle __all__ sorry. The docstring is still wrong.21:05
nessitaralsina: why?21:17
nessita(may be outdated)21:17
ralsinaBecause it says """""Captcha image generation failed."""21:17
nessita"""Concatenate __all__ and message from the errordict.""" <- seems correct... am I too sick? (this running nose is killing me)21:18
ralsinaAnd that's the docstring for another method21:18
nessitaah, that's another line, let me find it21:18
nessitawhat method?21:18
nessita on_user_registration_error ?21:18
ralsinaon_user_registration_error, right21:18
nessitaralsina: definitely bad copy and paste21:19
ralsinaI can add that change to this branch21:19
nessitaplease21:19
ralsinanessita: care to take another look? I'm afraid I crossed your "needs tests" threshold on https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_805405/+merge/6678421:23
nessitaralsina: from my POV, that change needs tests21:24
ralsinajajajaja21:24
* ralsina goes write a test21:24
ralsinaI hate writing tests in sso-client with the replay mocker21:25
nessitaralsina: I hate that too. I hate that record-and-play mocker lib21:25
nessitaI never use it21:25
ralsinaMe neither. Last night I wasted 1 hour trying to find the right incantation to make the number of calls match21:26
nessitacan I have a couple of reviews for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/run-tests-windows/+merge/66972 ?21:29
ralsinanessita: sure21:29
nessitaand, test for u1cp now run in windows!21:29
ralsinayay!21:29
=== m_conley is now known as m_conley_away
ralsinanessita: I have no idea how to test that branch. Should I fake critical() and write tests that send different kind of error dicts?21:30
ralsinaI meant my branch21:30
nessitalooking21:30
nessitaralsina: yes, and you should call  on_user_registration_error with different error dicts21:31
alecunessita, all tests pass on windows????21:31
alecunessita, congrats!21:31
ralsinanessita: ok, will do that21:31
ralsinaNOOOOO I HATE YOU BZR ON WINDOWS!21:31
ralsinabzr: ERROR: Failed to rename C:/Users/ROBERTO/canonical/run-tests-windows/.bzr/c21:31
ralsinaheckout/limbo/new-7 to C:/Users/ROBERTO/canonical/run-tests-windows/data: [Error21:31
ralsina 5] Access is denied21:31
ralsinaThat means I have to whack my whole ~/canonical where all my branches are21:32
thisfreddog + walk = later21:34
nessitaalecu: yes! the tweaks needed were very little :-)21:35
ralsinanessita: indeed they pass! cool!21:35
ralsinaAnd changes are minimal, so +1 from me21:36
nessitathanks21:38
nessitaI'm gonna to bed now, I'm, not feeling particularly well21:38
nessita(the running nose is morphing into a not minor cold)21:39
nessitasee ya later!21:39
ralsinaalecu: still here?21:59
alecustill21:59
ralsinaI don't  understand the mocker we use in ubuntu-sso-client :-(22:00
facundobatistaalecu, I save a file, SD uploads that file, the bubble notifies about a file uploaded three days ago22:00
alecufacundobatista, juas! Please open a bug, let thisfred know about it.22:01
ralsinaalecu: if you do understand that mocker, could you tell me how to write a single test for on_user_registration_error from this branch? https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_805405/+merge/6678422:01
alecuralsina, my recommendation is "don't use that mocker for new tests"22:01
ralsinaalecu: it's an idea. But I don't have patch, and the test case makes me use replay and so on.22:02
ralsinaAnd self.view.message_box is already mocked22:03
ralsinaAnd I want to put that file on fire :-)22:03
alecuralsina, then create a different TestCase class, make it inherit from twisted.trial.unittest.TestCase and you'll have patch...22:03
ralsinaok, I will go that way22:03
alecuralsina, but if you really want to use mocker....22:03
alecuralsina, let's see.22:03
ralsinaHonestly, I don't but crating a whole testcase different from all the others just to avoid the mocker... fun, but it smells22:04
alecuralsina, that's the issue with mocker. Every time we have to go and fix a test, or create a new one, it's a long time getting again in the mocker mindset. And the weird error messages and...22:05
thisfredfacundobatista: that is very strange, please assign the bug to me and tell me the number22:05
alecuralsina, so, let's say you make it with mocker. What do you want to assert in the test?22:06
alecuralsina, do you have a specific question or just need a pointer into how to use mocker?22:06
ralsinaalecu: a simple test where I call on_user_registration_error with error={} would be a good start22:07
alecufacundobatista, does the "file uploaded three days ago" shows on the latest logs?22:07
ralsinaalecu: I also suspect we are not testing much beyond "things are called in the right order" in lots of tests22:08
alecuralsina, you said "self.view.message_box is already mocked". Where is that?22:11
ralsinaalecu: SetUpAccountControllerTestCase.setUp22:11
=== m_conley_away is now known as m_conley
alecuralsina, well, that's just an instance of the mocker class, it's not mocked the way we mock it when we are not using mocker.22:15
ralsinaalecu: ooooooooooook ;-)22:16
alecuralsina, so, that object that's already mocked is no big deal.22:16
alecu(I know that phrase above sounded very confusing :-) )22:16
ralsinano kidding ;-)22:16
alecuralsina, so my recommendation stands: no more mocker :-)22:16
alecuralsina, do you still want me to build an example with it?22:17
ralsinaalecu: ok, I will do a non-mocker testcase. Hope nessita doesn't hate it.22:17
alecuralsina, nessita hates mocker. I started using it, and she made me see how awful it was.22:18
ralsinaalecu: ha22:18
ralsinaalecu: ok, should be no problem then22:18
ralsinabut now, I EOD22:18
alecubye!22:19
facundobatistathisfred, alecu, ok!22:20

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