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

=== jderose1 is now known as jderose
=== rodrigo__ is now known as rodrigo_
JamesTaitGood morning all!08:34
j0nrfolks, morning08:41
j0nrI asked in here a week or so ago about how to backup and restore my synced U1 folders...........I forgot08:42
j0nrI have a backup of my ~/Music folder..... I have a fresh install of U1.... should I put the ~/Music folder onto my fresh install before or after I sign into U1?08:43
mandelj0nr, you have to do it before, the idea is that you have to amke sure that the metadata is not there before your files, but let me confirm this beofre you do anything08:43
mandelkarni, is that ^ right?08:44
karniGood morning everyone!08:44
* karni reads08:44
mandelkarni, I remember we once said the wrong thing.. so I want to make sure :P08:44
karnimandel: j0nr: I believe the user should not copy the metadata over. Just the files. Syncdaemon will first do local rescan, and only then start down/uploading stuff.08:45
karnimandel: Yeah, I think I told the user to copy metadata as well, which was a mistake08:45
j0nrSo restroe my files as they were i.e. copy my ~/Music onto new system. Then sign into U108:46
j0nrjust trying to avoid the need to download all my Music08:46
j0nrsidenote... are you guys in charge of the Android Music App?08:47
karnij0nr: I believe that is correct. Yes, understood. It' be best to ask our support guys, duanedesign or rye. The former is probably still sleeping.08:47
karnij0nr: I'm the developer on it.08:47
karnij0nr: I believe you have something to say :)08:47
ryekarni, you are right, don't copy ~/.local/share/syncdaemon, copy Music only then launch syncdaemon08:48
j0nrJust wondering if it is still alive and kicking... I put it on my phone ages ago (paid) and haven't seen any updates since...08:48
karnij0nr: We just made a few improvements and fixed a playlist related server bug. Should be released soon to the Market.08:49
j0nrkarni: Cool! i was going to say, that one really 'bugs' me...08:49
j0nrthe playlists that get saved, are they specific to U1 Android music client or can they be accessed anywhere esle, or by any other programs? e.g. other music apps on my device? Can I access them on my desktop pc?08:50
karnij0nr: We had some unrelated issues, and this bug hid behind them. This is the only reason it was there for so long, sorry for the inconvenience :<08:50
karnij0nr: To be honest, I'm not sure.. We save them in couch, so theoretically they should be available. rye, do you know anything about that ↑ ?08:51
karnij0nr: (I'm gradually taking over the U1 Music app, sorry that I can't answer that particular question yet)08:52
j0nrAre all playlists the same, e.g. M3U or is it specific to all different applications?08:53
j0nrif you are storing as M3U then any app should be able to make use of them right? or is there problems with each app knowing where to find the list of music?08:53
j0nrSorry I don't know much, just thinking aloud08:53
karnij0nr: Playlists are stored in CouchDB. It's a database on your computer, that syncs with U1. It's not M3U files.08:55
j0nrOk, so I guess that rules out using those playlists in other music apps on my phone08:59
j0nrfair enough.08:59
karnij0nr: unless they talk to our Couch db on U1 servers (which is possible for 3rd party developers)09:00
j0nrok09:00
* mandel coffee09:54
matti+`09:57
mattiOps09:58
matti+109:58
mattiCoffee!09:58
* mandel back10:12
mandelmatti,  lol10:12
matti:)10:15
gatoxhi10:21
mattiHi10:22
mandelgatox, buenos dias!10:41
mandelgatox, truns out it was not deferToThread, but the bloody ubuntuone.platform.tools10:41
gatoxmandel, jejje lot of debugging?10:42
mandelgatox, more like, lots of swearing :)10:46
gatoxjeje10:46
* mandel walk dog11:33
gatoxmandel, did you already propose the branch that fix the problem in the installer?? because i propose another branch yesterday that  requires the branch you submit yesterday... to know, because if the fix change i have to change my branch11:36
=== eu is now known as Guest97336
gatoxbrb12:17
ralsinagood morning!12:53
ralsinaalecu, nessita, dobey, gatox, mandel: standup in 7'12:53
gatoxralsina, hi!12:53
ralsinahola nessita12:54
nessitahello everyone!12:54
gatoxnessita, hi12:55
ralsiname13:00
mandelme13:00
dobeymeh13:00
ralsinanessita, alecu, gatox: say me!13:01
gatoxme13:01
nessitame13:02
mandel@ping13:02
ubot4pong13:02
nessitamandel: go13:02
mandelwe do have a stand up, right?13:02
ralsinamandel: yes13:02
nessitayeah, as far as I know13:02
gatoxmandel, yes13:02
nessitamandel: go!13:02
gatoxralsina, was first13:03
ralsinaDONE: team call, 1-1 with alecu, started resourcing, moved canonicaladmin queue, doctor's appointment. TODO: finish resourcing, work on hiring, nessita's 1-1, 1-1 with the boss, and so on and so forth. BLOCKED: I am feeling rather sick to my stomach, but hey, I feel just as bad in bed13:03
ralsinamandel, now13:03
=== m_conley_away is now known as m_conley
nessitaralsina: sorry, I did not notice you said me first13:03
ralsinanessita: no problem!13:04
ralsinaok, mandel, if you pasted it, it did not work :-)13:04
ralsinadobey, go13:05
dobeyλ DONE: planning call, discussed release/qa procedure with elopio, collected some notes for process e-mail13:05
dobeyλ TODO: send out process e-mail, banshee/libu1 SRUs, bug #467397?13:05
ubot4Launchpad bug 467397 in python2.6 (Ubuntu) (and 6 other projects) "KeyError: 'ROUND_CEiLiNG' when using turkish locale (affects: 16) (dups: 9) (heat: 108)" [Undecided,Confirmed] https://launchpad.net/bugs/46739713:05
dobeyλ BLCK: No.13:05
dobeygatox: go13:05
gatoxDONE:13:05
gatoxFixed some branches, Fixed Congratulations page, working on expanduser.13:05
gatoxTODO:13:05
gatoxFinish expanduser, search & destroy ui bugs.13:05
gatoxBLOCKED:13:05
gatoxNo13:05
gatoxnessita, go13:05
nessitaDONE: roadmap call, reviews, discovered a couple of IPC bugs in u1client, more on the epic branch for u1sdtool multiplatfofm13:06
nessitaTODO: 1-1 with ralsina, reviews, stable branch for installer, more epic branch13:06
nessitaBLOCKED: no13:06
nessitaNEXT: alecu13:06
nessitaNOTE: I will be attending the design sprint next week, and UDS the nweek after that, so email will be the best way of contact me if I'm not in IRC13:06
ralsinaMaybe there was a confusion about standups in yesterday's call. We'll keep on having standups except the day of the team call. I am estimating you will not waste more than 2 hours a week on administrivia. If you think that's too much, I can give you *my* list ;-)13:06
mobile_me13:07
ralsinaAlso, in the email I sent I mentioned weekly email reports. Those will not be needed, I will do them based on the 1-1s13:07
ralsinamobile_: ?13:07
dobeyes mandel13:07
=== mobile_ is now known as mandel1
mandel1yes, stupid irc client13:08
ralsinaok13:08
dobeyrima-tde.net == spain :)13:08
ralsinaaleeeeeeccccccuuuuu alecu!13:08
mandel1DONE: worked on fixing bug https://bugs.launchpad.net/ubuntuone-windows-installer/+bug/878259 added extra tests for the moved code. TODO: propose merges for autoupdate after running tests on N. debug issue on vista. BLOCKED: no13:08
ubot4Launchpad bug 878259 in ubuntuone-windows-installer "Fails to start the installer (affects: 1) (heat: 6)" [High,Confirmed]13:08
mandel1I don't know why I have issues with the pad irc"..13:09
ralsinamandel1: because it's evil and proprietary13:09
gatoxmandel, because is mac13:10
gatox:P13:10
gatoxjeje13:10
gatoxnot mac...... but..... you know jeje13:10
mandel1probably13:10
ralsinaok, alecu seems to be missing in action, so EOM13:11
mandel1like our servers, hehehe13:11
ralsinaand if I may say so, the worst standup ever. I think most of you were not even standing.13:11
gatoxjeje13:11
mandel1ralsina, I was no :P13:12
ralsinaour servers have different freedom capabilities, you insensitive clod!13:12
nessitaralsina: wasn't alecu taking the morning off?13:14
mandel1I go back to lunch, I'll be back I'm 10/15 min13:14
ralsinanessita: right!13:15
* ralsina wishes canonicaladmin would send daily staff rolls13:15
ralsinaof course that would be like, useful, so it's not there13:15
gatoxlol13:15
gatoxnessita, i have a couple of pending reviews13:16
ralsinagatox: I can do one, probably, if you need it13:16
nessitagatox: yeah, I'm already with network-detect13:16
nessitagatox: then I have the 2 you gave me yesterday13:16
gatoxnessita, i fix network-detect..... but i was getting a error with a test for oauth..... not related to my branch :S13:16
gatoxralsina, yep!!1 i need reviewers..... let me give you one :P13:17
nessitagatox: need some help?13:17
gatoxralsina, here is a really small one: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/connect-files/+merge/7961113:18
gatoxnessita, oops..... i commit the test_after_timeout_cache_expires commented.... let me revert that...... i don't know why that test was failing.... i was looking at that13:19
nessitagatox: ok, let me know13:19
gatoxnessita, if you are free..... and you are not going to review network-detect right now..... can you please review the one for congratulations page?? is really small (sso and installer)13:20
ralsinagatox, nessita: shouldn't we yield on connect_files so we actually know if it connects correctly?13:20
nessitaralsina: I think there is no need, there's much we can do at that stage... the controlpanel will try to connect again if not connected13:22
nessitaralsina: if we'd like to handle not connecting at that point we would need a design for that... may be to much13:22
ralsinanessita: ok13:23
nessitaralsina: nevertheless, I can talk this with liseeeete next week13:23
nessitagatox: connect-files is giving conflict13:23
nessitaText conflict in ubuntuone_installer/gui/qt/gui.py13:23
nessita1 conflicts encountered.13:23
gatoxfixing.....13:24
ralsinanessita: I think it may be a good idea, because it will prevent weird errors like "None has no get_rootdir attribute" (wouldn't it?)13:24
nessitaralsina: as far as I see it, it will not prevent those, they will just appear in the installer13:25
ralsinaor is conenct_files to make syncdaemon connect, instead of connecting to syncdaemon>13:25
nessitaralsina: right, the former13:25
ralsinaoh13:25
ralsinathen no point13:25
nessitaconnect == connect a running syncademon to the server13:25
nessitaralsina: I just read carefully the email "The plan". Seems like it was chopped off at the end, and Chipaca played along :-)13:27
ralsinaYes, it seems that way :-)13:28
ralsinaI will post a fixed version with added bits13:28
gatoxnessita, ralsina conflict resolved in connect-files13:28
nessitagatox: ack13:28
ralsinaack13:29
nessitagatox: the successfull page sso branch will need fixing.... sso is a project independent from Ubuntu One, so we can't hardcode the name13:31
nessitagatox: please note that sso is used from software center also, and ideally any other app that wants sso credentials13:31
nessitagatox: so, the SUCCESS message, should not change, or at least change to:13:32
gatoxnessita, ahhhhh ok13:32
gatoxi'll add a new one13:32
nessitaSUCCESS = _('You are now logged in into %{app_name}s. Congratulations!')13:32
nessitagatox: you can't hardcode Ubuntu One, not sure if you got that message :-)13:33
gatoxnessita, ahhhhh ok..... sorry..... i took the text as it was in the bug description13:33
nessitagatox: right, but we need to adapt it a bit according to the project's specifics :-)13:33
nessitagatox: so, always remember: sso is generic, knows nothing about ubuntu one itself13:34
gatoxnessita, roger that! :P13:34
nessitaralsina: shall we 1-1?13:37
ralsinanessita: in 1'13:37
ralsinanessita: ok, whenever you want, I am in mumble now13:39
nessitaralsina: I'm trying to connect but is not letting me13:39
nessitaralsina: the server is closing the connection on me13:40
ralsinanessita: it's ok, whenever it works. Or I can just... call you13:40
nessitaralsina: shall we skype?13:40
ralsinalet's skype13:40
ralsinanessita: you seem offline in skype13:41
nessitaralsina: I'm nidart.work13:41
nessitanbidart.work13:41
nessitagatox: ubuntuone_installer/gui/qt/tests/__init__.py:13:56
nessita    62:  [C0111, FakeBackend] Missing docstring13:56
gatox:(13:56
=== zyga is now known as zyga-afk
nessitagatox: I will be doing another review round after lunch, I will switch to monkey coder role now :-)14:08
gatoxnessita, :P ok14:09
gatoxnessita, the test for oauth is working fine now.... maybe was something blocked in my machine and it needed to be restarted14:41
nessitagatox: what error were you having?14:41
gatoxnessita, also all the branches should be ready now..... i hope without lint issues :S (this weekend i'm going to try to have u1lint working)14:42
gatoxnessita, the test i mentioned earlier was throwing an exception and blocking the process14:42
dobeymeh, i suck at writing e-mail14:45
nessitadobey: if it's of any consolation, we all do14:46
nessitagatox: we will get together and get your env properly working14:46
nessitagatox: this week has been crazy for me with the incoming travel and all14:46
gatoxnessita, yes, i know, no problem..... let me check if i can fix u1lint this weekend, and if i can't i'll let you know next week to see if you can give a  hand14:47
nessitagatox: well, next week I will be a little far away from your home... but sure ;-)14:48
gatoxnessita, uds?14:49
nessitagatox: design sprint and UDS, so I will be out for 2 weeks14:49
nessitagatox: did you read my standup? :-D14:49
gatoxnessita, :P ok...... emmmmm when you get back jeje14:49
mandelnessita, this guy is ready for a re-review whenever you have the time: https://code.launchpad.net/~mandel/ubuntuone-windows-installer/fix-reactor-import/+merge/7984214:50
nessitamandel: ack!14:50
gatoxmandel, you never answer my questionnnnnnnn14:50
gatoxmandel, and here you are :P14:50
gatoxmandel, this is the same branch as yesterday?? i ask because i have another branch that depends on this one.... to keep it up to date14:51
mandelgatox, oh, wich question? I propbably missed it14:53
mandelgatox, and yes, is the same branch with some code that has been moved and more tests14:53
gatoxmandel, ahhhhhh ok14:54
dobeyi think i will have to get this e-mail finished up after lunch. my brain is still all over the place after yesterday :-/14:55
dobeyok, lunch time; bbiab15:13
ReaperIs there an upload limit in ubuntu 1?15:48
gatoxlunch..... brb15:57
mandelnessita, ping16:00
nessitamandel: pong16:01
mandelnessita, I have a very quick question, in control panel, we are using gi AND pygtk, right?16:02
nessitamandel: yes, for now16:03
nessitamandel: I will be migrating that, and very likely removing the GTK UI16:04
mandelnessita, ok, understood16:04
nessitamandel: can I help somehow?16:05
mandelnessita, I had a present for you: http://paste.ubuntu.com/714312/ but then I realized that it would not solve anything because we use pygtk in the widgets :(16:05
nessitamandel: you made that?16:06
mandelnessita, I was going to try and propose that for twisted when some changes so that there is not code repetition, but well, other day16:06
mandelnessita, yes, but based on the gtk2reactor, I'm not THAT smart hehe16:06
mandeland itamar is not in #twisted so that I could merge it nicely16:07
nessitamandel: dobey is already working on this, so I would advice talking to him, if you're really interested. On the other hand, I would also advice not to put time ont his, for now...16:08
nessitamandel: or are you blocked?16:08
mandelnessita, no, I took some time while watching tv, that is all :)16:08
nessitamandel: I think you can safely run the tests for the non-ui stuff in isolation?16:08
mandelnessita, but wanted to try before I EOD, that is all16:08
alecuhello #ubuntuone!16:14
mandelalecu, hello!!!16:14
alecuhi there, mandel!16:15
nessitahi alecu!16:15
* alecu has sweated more today than in the whole previous year.16:17
dobeymandel: that pastebin also doesn't even import successfully :)16:17
nessitaalecu: that's goo news! (?)16:17
dobeymandel: you basically did the exact same thing as someone else did, who proposed it on twisted's trac :)16:17
=== zyga-afk is now known as zyga
alecuon the plus side, I'm almost moved, and starting tomorrow, my office will officially be the back room in my own house.16:18
=== beuno is now known as beuno-lunch
alecugatox, mandel, ralsina: if any of you is having a happy day, doing this review will keep your spirits up: https://code.launchpad.net/~alecu/ubuntu-sso-client/fix-get-token-name/+merge/7959416:21
alecuif it's not a happy day, it will have no side-effects.16:21
mandeldobey, it does import correctly, I have tried it :P16:21
mandeldobey, anyways, it was a 10 min thing while watching tv hehehe16:22
dobeymandel: well, you got lucky then, because it's point at things that don't exist :)16:23
nessitaalecu: and what about amelia's bedroom? you share? :-)16:23
mandeldobey, I don't want good generals, I want lucky ones jeje16:23
dobeyheh16:24
alecunessita, no, it's Luli's deskroom I'm taking over.16:24
nessitaah, heh16:24
alecunessita, amelia's bedroom is usually a battlefield of plush toys, so it's not easy to work there.16:25
dobeyalecu: more than 3 toys? it must be really boring in there :P16:26
alecuhahahaha16:26
gatoxalecu, jeje ok on it16:28
dobeyok, back to e-mail16:34
gatoxalecu, +116:40
dobeyugh, this weather sucks16:41
alecugatox,   |o|16:43
alecu(tie fighter)16:43
nessitaalecu: so, when you have some spare time... I'd need your wisdom about twisted.pb. Turns out I found another bug, where the exceptions from the pb server are not being 'forwarded' to the clients16:46
alecunessita, now would be a great time16:46
nessitaalecu: I read the doc, and tried some approaches, but I had no luck. Spent quite some time yesterday fighting with this... so I skipped the test and moved on, until you came back16:46
nessitaalecu: nice!16:46
nessitaalecu: can you please branch:16:47
alecusure16:47
nessitalp:~nataliabidart/ubuntuone-client/u1sdtool-multiplatform16:47
nessitaalecu: you can check all is green for you by running, in linux:16:47
nessitau1trial tests/platform/test_tools.py16:47
nessitaalecu: once that's green on linux, confirm is all green on windows, but with a SKIPPED test16:48
nessita(JIC, latest revno is 1176)16:48
alecunessita, all green in linux. testing windows now16:51
nessitaalecu: so, the SKIPPEd test is  test_rescan_from_scratch_missing_volume. If you un-skip it, you'll get a trace where the exception received is not the expected, but a TypeError. That TypeError is raised from within twisted internals, complaining that the thing raised from the server is a string, which is "a lie"16:52
nessitaalecu: since the exception is raised from: ubuntuone/syncdaemon/interaction_interfaces.py:128216:53
nessitaalecu: oh, I'll be back in a few minutes, I'm at my mother in law's and she's screaming the lunch is ready :-D16:56
nessitabrb!16:56
alecunessita, on windows I'm getting a lot of these: http://pastebin.ubuntu.com/714368/17:02
alecuon the "interaction_ifaces" module17:02
ReaperI am trying to send a iso into ubuntuone and its not working.. any ideas.. its it too big?17:07
Reaperthe iso is under 5 gigs.17:07
dobeyphew, finally "done" with that mail17:08
dobeyReaper: i have uploaded ISOs just fine.17:08
Reaperhmm i think its user error then17:09
Reaperlol17:09
ReaperI tested a txt file and got that just fine17:09
dobeyReaper: if it's not uploading, it could be network, server, or client issues. but i can't really say without knowing the logs. i can tell you that our servers are *very* busy right now though, with oneiric having just been released :)17:10
Reaperyeah, thats what ive heard17:10
dobeyand there is a bug or two in the client, for which fixes which are being worked on at the moment, that we hope to get out to oneiric in the next couple weeks (SRUs take at least 7 days to get out)17:10
Reaperahh17:11
dobeydo you have a ~/.cache/ubuntuone/log/syncdaemon-exceptions.log ?17:11
dobeyhrmm, i need to get my laptop fixed17:13
=== beuno-lunch is now known as beuno
Reapernot sure, ill have to take a look at it17:17
dobeywtf; now network magically works again on my laptop17:17
ReaperI love when that happens17:17
nessitaalecu: back17:18
dobeyyeah, i upgraded to oneiric on it, and it stopped working17:18
nessitaalecu: can you just run the test_tools tests?17:18
dobeyspent 3 days trying to figure out what it was and how i could get it updated as there were pending updates still17:18
dobeyshut it off for 2 weeks, and it suddenly works again17:18
Reaperlol and now it works. that awesome lol17:18
beunoReaper, we are having some issues with our servers17:18
beunoso that's probably the cause17:19
Reaperthats what I was thinking. I tested a text file earlier today and that went through so ill just hang tight till the servers calm down.17:19
dobeyReaper: just to check, run this in a terminal please: "grep ValueError ~/.cache/ubuntuone/log/syncdaemon*"17:19
Reaperk 1 sec17:19
dobeyReaper: if you see "ValueError: I/O error..." type message, you're also hitting one of the known bugs we're working on a fix for :)17:20
Reaperahh got cha17:20
Reaperyep, getting alot of those errors17:22
dobeynow if i could just get the UI back to how i had it, on my laptop (which i guess isn't going to happen with gnome3) :-/17:22
dobeyReaper: restarting syncdaemon might help with that: u1sdtool -q; u1sdtool -c17:22
dobeyReaper: but if you put a big iso in there again, it will likely crop up again17:23
Reapersays its being uploaded now17:24
Reaperbut its said that before.. its about 991mb17:24
dobeyyeah, the notification will pop up again if you get reconnected, too17:26
Reapergot cha, its nothing big. was just stoked about trying that svc and was freakin because I couldnt get it to work. Just wanted to make sure I wasnt completely retarded lol17:27
dobeyunfortunately it does seem to be our fault. but hopefully we can have a fix soon17:28
nessitaalecu: let me know when you're back17:29
Reapereh, it happens lol17:31
nessitaalecu: ping?18:43
alecunessita, pong18:43
nessitaalecu: hey there, sorry for leaving in the middle of the debugging, not sure if you receive my latest messages18:43
alecunessita, no problem, I disappeared for a bit too.18:44
alecunessita, I'm still trying to follow the code18:44
nessitaalecu: i was saying, could you just run the tests/platform/test_tools.py in windows? that should be all green18:44
alecunessita, and I'm not sure at what point the error is being transformed into string.18:44
alecunessita, I've run that, and now I'18:44
alecuI've commented the skip18:45
alecunessita, and it shows an error where the exception comes as a string18:45
alecunessita, so I'm trying to understand at what point the exception is translated into a string18:45
nessitaalecu: me neither. I read the doc and it said that the exceptions being raised from the pb server should inherit from pb.Error. I did that, I created a new error inheriting from that class, and added a decorator (not in the branch) so every remote method catches everything and raises the specific error, but no luck18:46
alecunessita, I'm reading the PB tutorial, and it says exactly what you say:18:51
alecuhttp://twistedmatrix.com/documents/current/core/howto/pb-usage.html#auto418:51
nessitaalecu: yeah, I may have done something wrong... but I couldn't find what18:51
nessitaalecu: anyways, the doc gives an example where even if you raise a non pb.Error exception, the exception is indeed forwarded to the client, but ugly messages appear server side18:52
nessitaalecu: so that puzzles me a lot18:52
alecuyup, some message should be shown in the log in that case18:52
nessitaalecu: maybe one of the decorators that mandel added are stringyfing that?18:53
nessitaalecu: I looked, and my eyes found nothing, but yours may?18:54
alecunessita, I can't see anything wrong neither. I'm looking at both RemoteMeta and signal, the two places with meta-magic, and it's not being done there.18:57
nessitaalecu: I'm thinking skipping this test and filing a bug, for now, if you don't see anything popping up. This has been "broken" the same until now, can remain like this a bit longer18:58
alecunessita, I'm fine with that, since it's not something that your branch breaks19:00
nessitaright19:00
nessitaalecu: thanks for your time/eyes!19:00
alecunessita, I'll keep looking a bit more19:01
alecunessita, no problem. Sorry if I was a little more sleepy than usual :-)19:01
nessitaalecu: -)19:01
nessita:-)19:01
gatoxok, EOD for me... see you tomorrow... or later :P19:04
alecunessita, I'm pdbing the code, and it seems that it *gets* into """log.msg("Peer will receive following PB traceback:"...."""19:10
nessitaalecu: wtf?19:10
nessitaalecu: how's that possible?19:11
alecunessita, perhaps the twisted logs are not enabled on this tests. Or something like that.19:14
alecunessita, I've added "from twisted.python import log;import sys;log.startLogging(sys.stderr)" before that test starts. Here's the result: http://pastebin.ubuntu.com/714507/19:29
nessitaalecu: I don't see a str cast there, do you?19:33
alecui do not see it either.19:35
alecubut it looks like what the pb docs say19:35
alecunessita, the thing that I don't understand...19:38
alecunessita, why do we have some errors as signals (ie, RootMismatch, QuotaExceeded), and some other errors as exceptions? (VolumeDoesNotExist, etc)19:38
nessitaalecu: the signals you mention, in particular, are not 'errors' but 'states' (por llamarlo de alguna forma)19:40
nessitaalecu: despite that, we can't have signal for every error19:40
alecunessita, InvalidName, BrokenNode are signals too19:40
alecu(on a different object, though)19:40
nessitaalecu: yeah, but my point is what if we have an IOError, an outofmem error, whatever unxpected signal19:41
nessitaalecu: I would like to know that would be propagated to the caller, if unhandled19:41
nessitaalecu: so, we can certainly handle this specific case in the server side19:41
nessitabut worries me that the exceptions are not 'forwarded'19:41
alecunessita, right.19:42
dobeyugh, i can't remove multiple packages at once in software-center19:42
alecunessita, but the thing with signals is that they can be "async"19:42
alecunessita, if I'm returning the error as an exception, it needs to be fast19:42
alecu(kinda)19:43
dobeynessita: isn't there a "GenericError" signal on the main object or something, that just gives you all errors on a dbus signal?19:43
alecuso, that's really the main difference.19:43
nessitaalecu: not sure what that difference is19:43
nessitadobey: this is windows IPC side...19:43
dobeynessita: oh, does it not have the same interface as the dbus service?19:44
alecunessita, the difference is related to the timeouts we discussed the other day.19:44
alecudobey, it should be pretty similar19:45
nessitadobey: I'm refactoring it so interfaces matches. But ATM, exceptions are not being sent from server to client19:45
nessitaalecu: right19:45
dobeynessita: ah. well on linux there is a genericerror signal or something, right?19:47
nessitadobey: yes and no... this specific case was throwing a DBusException19:48
alecunessita, so, we should either: a) subclass all exceptions as pb requires. Or b) catch every exception and match it against a list of known exceptions in each function's decorator, and wrap those exceptions in a pb exception.19:48
alecunessita, b) sounds way over the top, so we should find a cleanish way to do a)19:49
dobeynessita: the server was getting a dbusexception from something else? or what?19:49
nessitaalecu: I tried the (b) approach, and even when raising a pb.Error, the TypeError still remains, that's the puzzle19:50
alecunessita, I'd love to see *that* branch19:50
alecunessita, but first I need to pick up amelia.19:50
nessitaalecu: you can try it yourself: inside the ubuntuone/platform/windows/ipc.py, in rescan_from_scratch, raise a pb.Error('foo')19:50
nessitaalecu: ah, ok, later then19:50
alecunessita, good point, I'll try that. thanks!19:51
nessitadobey: not sure what you ask. When I say server I was referring to the dbus server side... and yes, any unhandled exception in our dbus (and lower) layers will send a dbusexception19:52
dobeyok, some cake for me, and then poking at tcole's branch19:52
=== yofel_ is now known as yofel
dobeyhave a good evening all!21:13
* nessita ->eod21:50
nessitasee ya tomorrow crowd!21:50
=== m_conley is now known as m_conley_away

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