/srv/irclogs.ubuntu.com/2011/09/08/#ubuntuone.txt

JamesTaitMornin' all!08:17
feastyIve got a file that is stuck uploading. u1sdtool --current-transfers shows it as uploading and that it has completely uploaded all the data but the file doesnt exist any more. I've rebooted the machine and stopped/started the service multiple times and it won't go away. I've even tried overwriting the file but it still won't go. Can anyone tell me how to clear the queue please?10:16
clem-lcparrino: hi, I just got your email10:16
clem-lcparrino: I'll be here all day, don't hesitate to send me a query when you're available.10:17
facundobatistaHolaaa10:48
nessitahello everyone!11:21
gatoxnessita, hi11:21
nessitahi gatox11:22
gatoxnessita, are you free for 2 really small reviews? :P JUST TWO TODAY! jeje11:25
nessitagatox: shoot! I'll do them right after ralsina's cloud to cloud11:26
gatoxnessita, https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/prettify/+merge/7421911:26
gatoxnessita, https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/setup-button-fix/+merge/7411811:26
facundobatistaHola nessita11:29
nessitahola facundobatista!11:32
gatoxralsina, ping11:53
gatoxblajk1, ping11:54
gatoxsomeone ping? jeje11:55
=== nessita is now known as someone
someonegatox: pong11:56
=== someone is now known as nessita
gatoxJAAAAAAAAAAA11:56
gatoxnessita, legen..... wait for it....... dary! jeje11:57
nessitajuaz11:57
nessita:-)11:57
gatoxnessita, for a moment i thought that "someone" was a real person ejje11:58
nessitagatox: you have so much to learn :-P11:58
gatoxjeje11:58
nessitadobey: when you start your work day, would you please help me debugging why the client-dailies seems not to be building after the branches land? cp and sso are buidling just fine (well, are trying to be built at least, we know that controlpanel is not building in M and O :-/)12:04
nessitagatox: do you have the mock from andrews for the signin screen? I don't like the lineedits centered12:22
nessitagatox: the rest looks good12:22
gatoxnessita, yes i have the mock.... and has everything centered12:23
nessitagatox: I have a mock from lisette with everything to the left :-P12:23
nessitagatox: is in the share12:23
gatoxnessita, should i revert that change?12:23
nessitaunder client/disconnect_flow12:24
gatoxnessita, it looks kind of ugly for me12:24
* gatox looking..12:24
nessita004.png12:24
gatoxnessita, true..... only that the labels and textfields seems to be bigger....12:24
gatoxnessita, do you want to see andrews'?12:25
nessitagatox: yes please12:25
nessitagatox: bigger than what?12:25
ralsinagatox: pong12:25
gatoxnessita, that what it was now.... i mean.... the labels in that screen in this moment looks really small12:26
gatoxralsina, hi12:26
ralsinagood morning12:26
nessitagatox: we could make that a little bigger and add more left padding. Let's do this: let's revert the signin.ui, and I'll talk to design team about this. Please share with me the mock andrews sent.12:27
nessitahi ralsina12:27
ralsinahello nessita12:27
nessitaralsina: I reviewed you cloud-to-cloud branch, honestly it did not work very smoothly for me... I added comments, and I have a screenshoot if you need. Let me know!12:28
gatoxralsina, i was looking at some pages in the wizard while fixing some bugs.... and i found some inconsistency between some pages..... for example..... lisette designs tend to align everything to the left, and i found some pages where the text fields has some spacer at the left margin (to align that text field at the center or something), and some labels above text fields that include ":" (for example: "Email address:") and lisette always u12:28
gatoxse labels without ":".... i start fixing that details so the ui has always the same style... but i wanted to check with you if you think that is ok12:28
gatoxi was thinking to include that changes if you want in the branch i'm working on12:29
ralsinagatox: of course it's ok12:34
gatoxralsina, awesome! :D12:35
ralsinagatox: but which one will it be? with/without ? left-aligned or centered?12:35
gatoxralsina, everything left-aligned..... and everything without ":"12:35
ralsinagatox: ok, go ahead12:36
ralsinanessita: I will re-check that branch, maybe something broke from all the merging last night12:36
nessitaralsina: let me know12:37
gatoxralsina, do you know where is the page with the "reset password" button?? i can't find that!!12:40
nessitagatox: under login -> I forgot my password12:40
gatoxralsina, not the reset password page...... i mean the one with the e-mail text field12:41
ralsinagatox: forgotten_something.ui12:41
nessitaah12:41
gatoxralsina, ok!12:41
gatoxthanks12:41
ralsinanot literally that name :-)12:41
gatoxralsina, yes yes..... i found it!.... it started with forgotten jeje12:46
alecuholas!12:56
gatoxany back to the future fan boy here? http://alt1040.com/2011/09/nike-lanza-las-zapatillas-de-back-to-the-future-22-anos-despues12:56
nessitahola alecu!12:57
ralsinanessita: could you run the tests of cloud-to-cloud and tell me if you still have the error? Not restart the review, just the tests12:58
ralsinahola alecu12:58
nessitaralsina: sure!12:58
dobeynessita: hmm, ok12:59
nessitadobey: thanks (I requested a build manually this morning, but wasn't building since sep, 02)12:59
nessitame13:00
alecume13:01
gatoxme13:01
nessitaralsina, dobey?13:01
ralsiname13:02
nessitaDONE: reviews, bug #80702113:02
nessitaTODO: reviews, bug #83990713:02
nessitaBLOCKED: nopes13:02
nessitaNEXT: alecu13:02
ubot4Launchpad bug 807021 in ubuntuone-control-panel "QT UI: we need to handle errors from the backend (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/80702113:02
ubot4Launchpad bug 839907 in ubuntuone-control-panel "Windows: Creating UDFs doesn't make them appear in u1cp (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/83990713:02
nessitaubot4: YOU ARE BACK!13:02
ubot4Factoid 'YOU ARE BACK!' not found13:02
dobeyme13:02
alecuDONE: finished errdict branch, a bit of IRL for it remaining13:02
alecuTODO: work on restart fixes13:02
alecuBLOCKED: yes. Kid sick, no kinder; lets find the best time for mumble13:02
alecuNOTE: I'm taking a conference day tomorrow, to go to CISL 201113:02
alecuNEXT: gatox13:02
gatoxDONE:13:02
gatoxBug #829358, and several fixing in minor details to increase styling consistency.13:02
gatoxTODO:13:02
gatoxRemaining UI Bugs.13:02
gatoxBLOCKED:13:02
gatoxNo13:02
ubot4gatox: Bug 829358 on http://launchpad.net/bugs/829358 is private13:02
gatoxralsina, go13:02
ralsina7/9/1113:02
ralsina------13:02
ralsinaDONE:13:02
ralsina* Finished with-systray branch (merge pending)13:02
ralsina* Proposed branch to improve the whole cloud-to-computer/viceversa pages13:02
ralsina* Proposed branch to add --installer flag to the wizard13:02
ralsina* Removed warning in tests (merge pending)13:02
ralsina* Worked on put-in-autostart13:02
ralsinaTODO:13:02
ralsina* build release! that ism, if I finish three things first13:02
ralsina* installer details, cloud-to-cloud13:03
ralsinaBLOCKED: no13:03
dobeyλ DONE: bug #835093, poked at more deps issues13:03
dobeyλ TODO: bug #840072, bug #838778, get stuff off CD13:03
dobeyλ BLCK: None.13:03
ubot4Launchpad bug 835093 in ubuntuone-control-panel (Ubuntu) (and 1 other project) "Grammar Error in Control file (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/83509313:03
ubot4Launchpad bug 840072 in ubuntuone-installer (Ubuntu) (and 1 other project) "Installer needs to install more packages (affects: 2) (dups: 1) (heat: 14)" [Medium,Confirmed] https://launchpad.net/bugs/84007213:03
ubot4Launchpad bug 838778 in ubuntuone-installer (Ubuntu Oneiric) (and 2 other projects) "After installing Ubuntu One, there are 2 Ubuntu One launchers in System Settings (affects: 34) (dups: 10) (heat: 148)" [High,Triaged] https://launchpad.net/bugs/83877813:03
ralsinaok, comments?13:04
nessitaany comments?13:04
ralsinaEOM13:04
ralsinaabout the mumble, is something like yesterday good?13:05
ralsinaabout 3PM13:05
nessitaralsina: ok with me13:06
gatoxralsina, yep13:06
alecuralsina, I have some issues: my wife is working at that time13:06
ralsinaalecu: then you propose a time :-)13:06
nessitaralsina: all test OK with no warnings13:06
alecuralsina, what about 12.30?13:06
ralsinaalecu: ok13:06
ralsinaeveryone else, is 12:30 ok?13:07
nessitaalecu: sounds good13:07
alecucool, thanks13:08
ralsinanessita: confirmed the "white page" problem with cloud-to-cloud, it's because of the merges, I will fix it ASAP so it's there for your second review shift13:35
nessitaralsina: ack. Can you please confirm you get the computer-to-clound page as well? I confirm I'm not getting it13:35
ralsinanessita: no, it's crashing in the middle of setting things up: http://pastebin.ubuntu.com/685270/ [at least it's an easy fix]13:35
ralsinagatox broke it! [points accusing finger]13:35
ralsinaDoing a test rye suggested I got a oops: http://pastebin.ubuntu.com/685261/13:35
ralsina OOPS-ID-2077appserverZBGFIGFbHACAaEEeBbIeBEJGDJfABfDCa23466613:35
ubot4https://lp-oops.canonical.com/oops.py/?oopsid=ID13:35
ralsinaargh, wrong channel13:35
mterryHeyo!  I have been experiencing not-infrequent "SSLError: The read operation timed out" errors when Deja Dup uploads files to the cloud.  Does anyone know why that might be?13:41
mterrySee bug https://bugs.launchpad.net/ubuntu/+source/deja-dup/+bug/844369 for an example13:41
ubot4Launchpad bug 844369 in deja-dup (Ubuntu) "Deja-Dup backup to Ubuntu One failed (affects: 1) (heat: 6)" [Undecided,New]13:41
CardinalFangOoo.  Nice.  I saw duplicity backs up to Ubuntu One.  I didn't know about D'ej`a Dup.13:43
CardinalFangDamn Compose key stopped working.13:43
alecumterry, there's been ssl errors in our servers as of lately. I'm not sure of the status, but maybe rye or duanedesign have more info.13:44
Beretnote syncing broken?13:45
Beretstatus says it's up13:45
mterryalecu, it has not been merely a very-recent phenomenon13:45
alecu(sorry, my daughter just woke up.)13:54
alecumterry, yup, it's been like that for some days now.13:54
nessitaralsina: I have good news and bad news13:55
alecumterry, don't know when it started, but just today I heard people here having issues creating new users.13:55
ralsinanessita: bad first?13:55
nessitaralsina: the udf list from syncdaemon, after udf creation, is not updated13:55
nessitaralsina: the good news is that the UI is, indeed, reloading that list13:55
ralsinanessita: so, it's a syncdaemon bug13:55
nessitaralsina: so I'll go debug syncdaemon side13:56
ralsinanessita: good luck, here's a snorkel :-)13:56
nessitaralsina: is *extremely* weird, I can't think of how this is happening (knowing the code). We may be catching this somewhwere13:56
nessitaralsina: and is very likely a windows issue13:56
nessitaralsina: let you know when I know more13:56
nessitaralsina: were you able to finish my review?13:57
ralsinanessita: not yet13:57
ralsinanessita: but soonish13:57
nessitais ok13:57
* nessita dives in syncdaemon code14:00
ralsinabeuno: pindonga is telling me that the timeout problem with my account's ping *may* be u1-related after all14:02
dobeynessita: is there a u1client branch ready to land?14:04
nessitadobey: not that I know of...14:04
dobeyyeah, seems my merges folder for it had a lot of rejected/superseded proposals still :-/14:11
alecunessita, ralsina, gatox_away: the branch to fix the errdict issues: https://code.launchpad.net/~alecu/ubuntu-sso-client/everybody-hates-errdicts/+merge/7460614:17
ralsinaalecu: queuing14:17
=== m_conley_away is now known as m_conley
alecuralsina, gatox_away: I have not used the installer in a while, and I just followed all steps while testing my branch14:24
alecuralsina, gatox_away: it's looking great!!!!14:24
ralsinaalecu: and...14:24
ralsina\o/14:25
alecuralsina, what about the "IMAGE GOES HERE?" in the final page?14:25
ralsinaalecu: it's in a branch that's not merged yet14:25
ralsinanessita: +1 on your branch14:25
nessitayey!14:25
alecucool14:25
ralsinanessita: cloud-to-cloud is working well again, ready for your review in your next batch14:30
nessitaack!14:30
nessitaalecu: approved!!! and thanks for working on this :-)14:47
nessitaalecu: error handler makes much more sense now!14:48
alecunessita, :-)14:48
alecunessita, the nice thing about this branch is that the -error tests now work from inside the "blocking" thread in the server, all the way out to the callback in the client14:50
alecunessita, and the tests themselves are much smaller14:50
nessitaalecu: yes, I really like the refactoring. A couple of iterations more and we get rid of mocker14:51
alecunessita, that's the idea :-)14:51
alecunessita, anyway, this last branch only applies to SSOLogin, not to SSOCredentials14:51
nigelbaquarius_: Hey, do you or someone from your team want to talk at Ubuntu User days? :)14:51
alecunessita, I'm opening a new bug so we can fix the tests in SSOCredentials as well14:52
nessitaalecu: great, thanks. The bug is for "only" fixing the tests, or we also have this failure in production code?14:52
alecunessita, there was no use of errdict in SSOCredentials itself, so we should not have the failure14:53
nessitaalecu: ok, good news then (?) :-)14:53
alecunessita, I started doing the fixes to that in that same branch, but the bit that runs in a thread was giving me a headache14:54
alecunessita, so I decided to fix only what was needed14:54
nessitaright14:54
=== gatox_away is now known as gatox
gatoxI'M BACK!!!!!!!1 TRAFFIC WAS AN INCREDIBLE MESS!!!!!!!!15:06
nessitagatox: STOP YELLING! :-)15:08
gatoxjeje15:08
ralsinaIRL testing of adding things to autostart on windows is a huuuuuuuuuge pain15:12
=== karni_ is now known as karni
dobeybbiab, need to get lunch15:31
=== kklimonda is now known as Guest27664
aquarius_nigelb, heya! we may do indeed; that sounds like a good question to ask joshuahoover, perhaps15:32
joshuahoovernigelb: yes, i think we'd like to present, i need to confirm15:35
ralsinaalecu, nessita, gatox: mumble?15:42
nessitaright!15:42
gatoxralsina, ok15:42
alecudone15:42
nessitagatox: ping16:04
gatoxnessita, pong16:05
nessitagatox: I see your bug list and there are 5 High marked as Triaged, are those actually In progress?16:05
gatoxnessita, yes, i'll change the statusnow16:05
nessitagatox: thanks!16:05
nessitaalecu, ralsina: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/delay-reload-folders/+merge/7463316:11
aleculookin16:12
nessitaalecu, ralsina: I will work on bug #843835 and bug #80070516:12
ubot4Launchpad bug 843835 in ubuntuone-control-panel "Qt UI: "Explore" should be disabled for folders that are not syncing to the local device (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/84383516:12
ubot4Launchpad bug 800705 in ubuntuone-control-panel "After subscribing/unsubscribing, folder icon is not properly enabled/disabled (affects: 1) (heat: 3)" [Medium,Triaged] https://launchpad.net/bugs/80070516:12
ralsinanessita: that works?16:12
nessitaralsina: at least in my VM, wann confirm?16:12
nessitawanna*16:12
ralsinanessita: sure!16:12
alecunessita, there are no tests!!!16:13
nessitaalecu: yes there are! I changed implementation, no functionality visible from the outside16:13
=== Guest27664 is now known as kklimonda
nessitaalecu: you can look at test_reload_volumes_info_on_success in delay-reload-folders/ubuntuone/controlpanel/gui/qt/tests/test_folders.py16:14
nessitaalecu: what I'm not testing is the call to callLater, which should not be there at all, since is a cheap hack16:15
ralsinaa test there would be like ... a confession :-)16:15
alecunessita, ralsina: will this bit of code be used also when creating udfs from the installer?16:17
ralsinaalecu: no problem since you can only create UDFs *after* you see the folders list :-)16:17
nessitaralsina: but that is True for the control panel as well, no?16:18
ralsinanessita: in u1cp you create udfs from the folders tab16:18
nessitaralsina: yes16:18
nessitawhere the folder list is already loaded16:18
ralsinanessita: in installer, you leave that view, then you can create udfs, which are added to the list manually16:18
nessitaah16:19
nessitawhy manually?16:19
ralsinanessita: well, I create them and add them to the list, instead of asking for a new list16:19
ralsinanessita: delay-reload-folders doesn't seem to work IRL for me :-(16:21
nessitaralsina: can you re try by increasing the callLater to 2?16:21
ralsinanessita: sure16:21
nessitathis is an ASCO16:22
nessitawe can't be putting delays in there16:22
ralsinaagreed16:22
nessitawe'll never get a delay that fits everyones computers16:22
nessitaralsina: you sure you're using my branch, with proper pythonpath and all?16:23
ralsinanessita: yep16:23
ralsinaugh, udf creation is failing. I need to debug this more16:27
nessitaralsina: do you get the error popup?16:29
ralsinano16:29
ralsinabut the UDFs are not created at the website, so something is failing in syncdaemon. I am staring clean now16:30
nessitaralsina: if folder creation is failing, and you're not getting an error popup, smells like you're not using my branch... because in it we have the handle-errors branch, which catches that and shows a popup16:31
ralsinassl handshake failure16:32
ralsinait's failing in syncdaemon16:32
nessitaralsina: right, is the error in the controlpanel debug trace?16:32
ralsinano, only on syncdaemo16:33
nessitaralsina: weird!16:33
ralsinayes, I am starting again, with no metadata, clean account, no previous UDFs, etc16:33
ralsinanessita: the "show a dialog for everything" may not be a great idea. I am getting one about webclient every time I close u1cp16:35
karniguys, where are syncdaemon logs on Ubuntu? I thought they were in .cache16:38
karniah, .cache/ubuntuone/log  :)16:38
karnifacundobatista: Can I send you my SD logs? I noticed that every time I upload a larger file (this time 22MB), the upload progresses up to 100%, but then sits at that 100% before completeion for long. Would you have a look?16:40
facundobatistakarni, put verterok in copy16:40
karnifacundobatista: ok tnx16:41
facundobatistakarni, but we'd need for them to be TRACE logs16:41
karnioh shoot16:41
karniright16:41
karnifacundobatista: Could you remind me one last time how to launch SD in TRACE mode?16:41
facundobatistakarni, edit your ~/.config/ubuntuone/syncdaemon.conf16:42
karnifacundobatista: DEBUG is not sufficient, is it?16:42
facundobatistakarni, nop, as for this kind of issues we need to study all the dialog client-server16:43
ralsinanessita: ok, things seem to fail here because of the SSL handshake errors. I ust can't test it :-(16:43
karnifacundobatista: ok :) thank you!16:43
karnifacundobatista: I'll send the logs next time I have this issue in that case. Editting my syncdaemon.conf now.16:44
facundobatistakarni, no problem16:44
nessitaralsina: we should fix the webclienterror instead ;-)16:45
ralsinanessita: of course16:45
nessitaI'm getting traces in syncdaemon...16:45
nessitaassertion errors from paths16:45
gatoxnessita, ralsina when you have a minute: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/text-style/+merge/7464016:47
ralsinanessita: could we only show that if DEBUG is set? (unless that's already the case, of course)16:47
nessitaralsina: we could, but that's no the idea. If the backend fails, the user will not know and nothing will happen, and he will not know why...16:48
nessitaralsina: we should try fixing the webclienterror asap16:48
ralsinanessita: showing every exception to the user as it is, is not something that can be shipped16:48
ralsinafor testing? yes16:48
nessitaralsina: you approved the branch! :-)16:48
ralsinanessita: I make mistakes16:48
nessitaralsina: we all do, but my point is: how were you expecting that we handle errors then? I'm not sure what the alternative is16:49
ralsinanessita: with DEBUG, it's ok to show those. Without DEBUG, we should log them.16:49
nessitaralsina: but the user experience will be aweful16:49
nessitaralsina: imagine this scenario:16:50
nessitaralsina: you try to login, and if the user/password do not match, nothing happens, and you have no idea why16:50
dobeyheh16:50
nessitaralsina: if folder creation fails, nothing will happen, and you (as user) will have no idea why16:50
ralsinanessita: that is not what happens :-)16:50
ralsinanessita: if user/password don't match you get an error message, not a dialog with an exception in it16:50
nessitaralsina: nopes, you get a dialog with a proper error in i16:51
nessitat16:51
nessitaralsina: try it! :-)16:51
nessitasame for folder creation error, device list retrieval, file sync settings change, etc16:51
ralsinawhen exactly are you saying I get "a dialog with a proper error in it" exactly?16:51
nessitaralsina: in the controlpanel, when you try to sign in with wrong user/pass16:52
ralsinanessita: users are not signing via control panel16:52
nessitaralsina: not 100% of the time, agreed, but 5% maybe? my point is we can't hide all the errors from the user, because we're not recovering from them (just not failing)16:53
nessitaralsina: think of folder creation. What if it fails?16:53
ralsinano, the only way to signin via control panel is to remove the credentials manually and then start it from cmd. 0.001% ;-)16:53
nessitaralsina: no, is by removing a device16:53
nessitaremoving the local device, sorry16:54
ralsinafolder creation as in udfs? I am not getting a message anyway16:54
nessita(which is the only that can be removed from the QT interface)16:54
nessitaralsina: if syncdaemon is sending the error, you will get it. Perhaps syncdaemon is crashing itself16:54
ralsinanessita: not crashing, apparently, also not passing it. No idea what happens there. have you tried UDF creation lately? Maybe it's just me16:55
nessitaralsina: yes, I tried it 5 times today debugging the folder list reaload issue16:57
ralsinanessita: then it may be something again with my account, except it's a new one :-(16:58
ralsinanessita: on windows, right?16:58
nessitaralsina: so, about the error popup, the user should practically never got that. The key issue here is that we're having errors we should not be handling, and we should fix that16:58
nessitaralsina: yes, windows every single time16:58
nessitawe should be handling, I mean :-)16:58
nessitaralsina: can you please file a bug about the webclienterror?16:59
ralsinanessita: I am not getting it ATM, if I get it again I will16:59
nessitaralsina: yes, is a timing issue, related to closing the controlpanel while a REST request is in place16:59
nessitaalecu: I LOVE I can shutdown syncdaemon with ctrl-c now :-)17:00
nessitaok, lunchtime!!!17:00
alecu:-)17:00
ralsinaalecu: http://pastebin.ubuntu.com/685432/17:03
gatoxelopio, ping17:07
elopiogatox, hello.17:07
gatoxelopio, hi!, can you please tell me where did you see this: https://bugs.launchpad.net/ubuntuone-control-panel/+bug/838628 because i can't find it17:08
ubot4Launchpad bug 838628 in ubuntuone-control-panel "wrong wording on the preference to sync shared folders (affects: 1) (heat: 6)" [High,In progress]17:08
elopiogatox, that's the settings tab of the windows dashboard. There was a list of three checkboxes. I'm not sure if that screen is still the same though...17:11
gatoxelopio, ok..... let me check.... if i have any doubt i'll ping you again :P17:12
elopiogatox, yeah. I'll also grab the most recent installer and give it a try again.17:12
alecuralsina, how did that pastebin happened?17:17
ralsinaalecu: adding a udf17:17
nigelbaquarius_ / joshuahoover: Great! Thanks :) (sorry was away for dinner)17:17
ralsinaalecu: it seems nessita did a commit "enforcing suggested paths t be unicode'17:17
ralsinaalecu: revno 1100 of ubuntuone-client17:18
ralsinabut that was a month ago, so I don't know how this could happen17:18
nigelbjoshuahoover: When just ping me when you guys are ready with a timeslot. You can pick one from https://wiki.ubuntu.com/UserDays :)17:18
joshuahoovernigelb: will do17:18
ralsinaBut at least that explains why udfs are broken for me. I don't know why just for me though :-(17:19
* nessita is back17:28
nessitaralsina: we did the path enforcing check when we fixed all the unicode/bytes path handling17:29
nessitaralsina: I'm getting traces like that as well, since today17:29
nessitaralsina: so something landed in trunk that is breaking path assumption, I'll debug17:29
nessitaralsina: can you please file a bug?17:29
ralsinanessita: thanks17:29
ralsinanessita: sure17:29
ralsinaagainst u1-client or u1cp?17:29
nessitaralsina: is not specifically related to udf creation, though17:29
ralsinano, it just triggers it17:30
nessitaralsina: u1client, I'm getting those for Uploads17:30
ralsinaack17:30
Monotokohey guys...any admins around? I think U1 gave me too much space... XD17:31
MonotokoYour storage: 1% in use (229.4 MB of 25.0 GB) Your current storage is composed of: 5GB with Ubuntu One Free17:32
dobeyMonotoko: do you have the music streaming package?17:32
Monotokodobey, no...I did have then I canceled it because I have Spotify, I'm on the basic package now17:33
Monotokothis was a few months ago though17:33
nessitabeuno: ^17:33
beunoMonotoko, did you have the yearly music plan?17:34
Monotokobeuno, no I had the monthly one17:34
gatoxelopio, sorry to bother you again..... can you please send me an screenshot of the page where are you seeing the bug i mention earlier.... so i can find it quickly and know what are you looking at?17:34
ralsinanessita: bug #84500917:34
ubot4Launchpad bug 845009 in ubuntuone-client "unicode path asserts are failing (affects: 1) (heat: 6)" [Critical,Triaged] https://launchpad.net/bugs/84500917:34
Monotokobeuno,  it even tells me on the homepage I am not subscribed to the music plan17:34
beunoMonotoko, what is your email address?17:35
Monotokobeuno, can I PM you with that?17:35
beunoMonotoko, please17:35
nessitaralsina: thanks! I'll start on that ASAP, since I just proposed a branch for review:  https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/able-enable-disable/+merge/7465217:37
ralsinanessita: I'll review that one in the meantime17:37
nessitaralsina: shall I do cloud-to-cloud before starting that debug-branch?17:37
beunoMonotoko, interesting, that does look like a bug. Thanks for lettings us know!17:37
ralsinanessita: no17:37
nessitaack17:37
Monotokobeuno, no problem :) can I just ask am I going to be downgraded or can I use the space? XD17:38
ralsinanessita: start debugging, alecu and/or gatox can take a look at cloud-to-cloud, but the path stuff is way more important17:38
beunoMonotoko, we're going to fix the problem, I'll see if I can get you an extra year of storage for reporting it  :)17:39
Monotokobeuno, :D17:39
elopiogatox, don't worry. I should have attached the screenshot :)17:39
=== beuno is now known as beuno-lunch
elopiogatox, just give me some minutes to download the new installer.17:39
gatoxelopio, Thanks!17:40
dobeysomeone make a u1client branch already :)17:40
nessitadobey: I'm about to! but I depend on debugging :-/17:41
nessitaI mean, I depend on finding the bug first, which can be tricky17:41
nessitafacundobatista: ping17:42
facundobatistanessita, pong17:42
nessitafacundobatista: hi! I need your knowledge on the Upload action queue command. I never got this right: who sets the tempfile name?17:42
nessitafacundobatista: is it the ZipQueue in _compress?17:43
facundobatistanessita, AFAIR, the one who compresses the file17:43
facundobatistanessita, I think so17:43
nessitafacundobatista: ack, thanks, I'll keep debugging from there17:44
nessitaralsina: can I have the complete log from your run?17:45
ralsinanessita: will ahve to do a new one17:47
ralsinanessita: will have it in 2'17:47
ralsinanessita: it's not happening now!17:51
* ralsina is about to go crazy17:51
nessitaralsina: right, it depends on an upload being compressed generating a temp file17:51
ralsinaooooook17:51
ralsinano wonder it was so randim17:51
ralsinaSo it has to be a folder with a large file?17:52
nessitaralsina: right, it smells like is an 'old' bug, but we never hit it so far (or we were not aware we hit it(17:52
nessita))17:52
ralsinabetter today than next week17:52
nessitaralsina: perhaps, I'm not sure (I m not very familiar with the ZipQueue)17:52
ralsinadoesn't seem to make a difference17:53
nessitaralsina: is ok, I'll use what I have17:53
ralsinastill need the log? Because I couldn't  reproduce17:53
nessitais ok17:53
ralsinaok17:53
=== joshuahoover1 is now known as joshuahoover
elopiogatox, I added the screenshot18:03
ralsinaI am going to take 30' and have some tea. I need to cool down a bit.18:05
gatoxelopio, thanks!18:06
elopiogatox, np.18:07
elopioI'm going to have lunch.18:07
=== elopio is now known as elopio-lunch
gatoxmandel, ping18:26
Monotokohmmm...is there any iPhone app other than the music one?18:32
nessitagatox: mandel?18:33
Monotokocontacts backup would come in useful...18:33
nessitagatox: mandel is on holidays! :-)18:33
gatoxnessita, i was looking for mandel about the bug i mention to you.... no need anymore18:33
nessitaah!18:33
dobeyMonotoko: i think a new contacts app is in the works18:33
Monotokodobey, excellent18:34
=== beuno-lunch is now known as beuno
alecugatox, ping19:09
gatoxalecu, pong19:09
alecugatox, can I pass you this bug? https://bugs.launchpad.net/ubuntu-sso-client/+bug/84508319:09
ubot4Launchpad bug 845083 in ubuntu-sso-client "Registration error message is not parsed (affects: 1) (heat: 6)" [Medium,New]19:09
alecugatox, it's the one we discussed on the meeting19:09
gatoxalecu, yep!19:09
gatoxalecu, is it ui stuff or something else?19:09
alecugatox, I've traced it all the way thru the ipc methods, and it keeps being a dict, so it seems to be a bug in the function detailed in the bug19:10
alecugatox, I think we should ask nessita for input on how the parsing of these error messages is done on the gtk side19:10
gatoxalecu, ok, i'll take a look19:10
ralsinathere is a _build_generic_error_message we can make public and use19:10
alecuralsina, that's the function that "gives up" while parsing the dict.19:11
ralsinareally?19:11
ralsinawhoa19:11
alecuralsina, take a look at the function, and at the screenshot in the bug: https://bugs.launchpad.net/ubuntu-sso-client/+bug/84508319:12
ubot4Launchpad bug 845083 in ubuntu-sso-client "Registration error message is not parsed (affects: 1) (heat: 6)" [Medium,New]19:12
alecuralsina, anyway, this looks like a "medium" or even "low" issue19:12
ralsinaso it's a sort of error message it's not expecting19:12
ralsinawelllllllllll19:12
ralsinait's not a crasher but it's not something we can ship19:13
ralsinaas in "every time you fill a field wrong you get a dictionary thrown to your face" :-)19:13
alecuralsina, well, the users would end up learning a whole lotta new words.19:13
ralsinaso, alecu, no more changes coming for the errdicts branch?19:14
ralsinaif so, I am approving it19:14
alecuralsina, yes: no more changes coming.19:14
alecuralsina, yay!19:14
ralsinacool +1 and merging19:14
ralsinawe are down to one critical error and two ugly patches19:15
ralsinaalecu: wanna start something else, or you need to look after amelia?19:15
nessitaralsina, alecu: the assertion error in paths thing is "heavy" :-(19:16
ralsinaif you are up to another thing: closing u1cp from trunk shows a dialog with a webclient error, find and kill ;-)19:16
ralsinanessita: expected it19:16
alecunessita, can I help with that?19:16
ralsinanessita: so, we are not shipping tonight19:16
nessitaralsina: I'm mumbling with facundo trying to designed a good solution19:16
ralsinanessita: ack19:16
nessitawe narrowed the problem19:16
nessitaalecu: I would prefer if you can tackle the webclienterror19:17
nessitaralsina: have the bug report handy?19:17
ralsinanever crated that one19:17
alecuralsina, I'm currently babysitting, yes, so I'm not exactly "focused", but I will take a look when Luli returns19:17
ralsinaalecu: cool thanks19:17
ralsinaAnd.... I have to go pick mine form kinder19:17
alecunessita, you mean the webclient error when there are pending web requests?19:17
alecunessita, do you have a bug for that?19:18
=== elopio-lunch is now known as elopio
nessitaalecu: yeap, ralsina is creating one :-)19:18
alecunessita, or a way to reproduce?19:18
ralsinano, I am going to kinder, but I will create one in an bit. Alecu: start u1cp, then close it :-)19:18
nessitaalecu: open controlpanel in either linux or windows, and close before the webclient request are done19:18
alecunessita, cool.19:19
alecuralsina, nessita: do you guys mean the "dialog box" that's shown even after the control panel window was closed?19:25
nessitaalecu: yes, that is just evidencing a traceback that we had since we added the twisted webclient19:25
alecuralsina, nessita: it's says "WebClientError ... -1... etc"19:25
nessitaalecu: that webclient "query" also prevents the controlpanel to close19:25
nessitaalecu: yes19:26
alecunessita, I can't close the control panel anyway19:26
nessitaalecu: I meant: that webclient "query" also prevents the controlpanel process to finish19:26
alecunessita, even if I wait19:26
nessitaalecu: why not?19:26
alecunessita, I open the control panel, wait for all the queries to finish, then I try to close it with the X19:27
alecunessita, but the process does not stop.19:27
alecuwell, right now it did close :P19:27
nessitaalecu: right, "something" is happening there, never had the time to debug. Can't confirm is the same webclient thingy, but I would like to think it is :-D19:27
alecuok, I'm looking into it.19:29
nessitaalecu: thanks, I will file the bug since ralsina is afk19:31
alecucool, thanks.19:31
dobeywhoo, i finally found a trivial bug to fix!19:33
nessitaalecu: the webclienerror: bug #845105 (can you please attach the stacktrace to it?)19:34
ubot4Launchpad bug 845105 in ubuntuone-control-panel "QT UI: WebClientError when closing the windows (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/84510519:34
alecuack19:35
nessitaalecu: the "sometimes it takes a while to close" bug #84510619:35
ubot4Launchpad bug 845106 in ubuntuone-control-panel "QT UI: sometimes the process does not finish when closing (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/84510619:35
alecunessita, my guess is that the second one is related to qtreactor19:36
alecunessita, but I'll take a good look too19:36
nessitathanks!19:37
nessitafacundobatista: just FYI, the open_file().name bug is: bug #845108. Do you need a bug report for the issue you're kindly helping us with?19:37
ubot4Launchpad bug 845108 in ubuntuone-client "Windows: open_file returns a file object whose 'name' is not a utf8 byte sequence (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/84510819:37
dobeyhooray!19:41
dobeywho wants to do a trivial review? :)19:41
facundobatistanessita, nah19:41
dobeyhttps://code.launchpad.net/~dobey/ubuntuone-client/print-version/+merge/7467219:41
nessitadobey: I will19:41
nessitadobey: approved19:48
dobeyyay19:50
dobeynow we can get some more debug info about nightlies build requests19:50
gatoxnessita, ralsina when you have a minute: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/some-ui-fixes/+merge/7467319:51
nessitagatox: ack19:51
gatoxlaunchpad not updated yet19:51
ralsinagatox: on it19:53
FuryWhen I look on the status page I see the symbol, that contacts are not working. When I click on it I only get informations about a service outage two weeks ago.20:01
FuryDid someone just forget to update the details or did someone forget to remove the symbol and I theres a failure on mc PC?20:01
gatoxnessita, ralsina when you ran out of quota in control panel, is there any message right now?? or any error message?? to catch that and implement it nicer..... or this should be done from scratch20:03
gatox?20:03
nessitagatox: from scratch20:03
nessitagatox: anyways, is not like you have to do big implementation, let me show you20:04
nessitagatox: please open this file:20:04
nessitaubuntuone/controlpanel/gui/gtk/gui.py20:04
gatoxnessita, ok, nothing from ui.... but is there any message from sso or something to determine if the user ran out of quota?? that is what i want to know20:04
nessitagatox: and go to line20:04
nessitagatox: follow my pointer20:04
gatoxnessita, yep? line?20:05
nessitagatox: go to line 1512, and then 148020:05
nessita_update_quota20:05
nessitagatox: see how we paint the quota in red if it's below a threshold20:05
gatoxnessita, yep20:06
nessitagatox: that's all you have to do. Translated to the QT UI, you have to:20:06
nessita1- got to controlpanel.ui20:06
nessitago*20:06
nessitasorry, controlpanel.py20:06
nessitaubuntuone/controlpanel/gui/qt/controlpanel.py20:06
gatoxnessita, yep, got it20:07
nessitaprocess_info20:07
nessita(line 89)20:07
gatoxyep20:07
nessitain that method, you should apply a similar logic with similar tests to the GTK ones20:07
gatoxnessita, ok! no problem!!20:07
gatoxthanks!20:07
nessitagatox: only that in QT we can just apply a property to the label, and define a QSS style, yes?20:07
gatoxnessita, right20:08
nessitagatox: so we don't hardcode the "red" for the quota usage20:08
gatoxnessita, of course20:08
nessitagatox: also note that we paint the quota in 'red' when the free space is less than QUOTA_THRESHOLD, which is defined in a common location for both QT and gtk20:08
gatoxnessita, ok20:09
nessitagatox: question regarding the sso branch:20:12
nessitais this correct?20:12
nessita   <property name="rightMargin">20:12
nessita    <number>1</number>20:12
nessita   </property>20:12
nessitasmells like a bug spirit20:12
nessitaI think you meant 0?20:12
gatoxnessita, why?20:12
gatoxahhhh let me see20:13
nessitamargin of 1 is weird20:13
joshuahooverFury: contacts sync is not working for most users, we're working on fixing it, which has us doing work on both the server and client side20:13
joshuahooverFury: that work is in-progress20:13
duanedesignFury: hello20:14
gatoxnessita, right...... fixing.....20:14
nessitagatox: next question, why did you removed all the placeholdertext?20:14
dobeywell crap20:14
duanedesignd'oh joshuahoover beat me to it20:14
gatoxnessita, DESIGN DECISION!! lisette registered a bug for that.... she says that we don't want that, only the labels above the text fields20:15
nessitagatox: AAAAHHHHH20:15
Furythanks20:15
nessitagatox: these design crowd does not get aligned20:15
gatoxnessita, what?20:16
nessitagatox: I agree having that twice sucks20:16
gatoxnessita, ahhhhhhh yes!20:16
nessitagatox: but the captcha solution entry does not have a label, right?20:16
gatoxnessita, i think that is better as lisette says20:16
gatoxnessita, nop..... is pretty clear that....... and if we put a label there we should increse the height of the wizard.... she says that we can let captcha without label20:17
ralsinausers know captchas20:17
nessitagatox: right, so for captcha we leave the placeholder, yes?20:17
nessitait makes thing clearer20:17
gatoxnessita, i remove every placeholder but i can restore that one if you want.....20:18
nessitagatox: I prefer not to remove the placeholder for the entry that has no label20:19
gatoxnessita, ok20:19
nessitagatox: I think in this case is only the captcha solution, right?20:19
ralsina+1 on that20:19
gatoxnessita, yes20:20
nessitagatox: also, why did you removed the newline in SET_NEW_PASSWORD_LABEL?20:21
nessitagatox: please note that legend is also used from the GTK side20:21
ralsinanessita: saw the movement on the asserts bug... since EOD is near can you catch me up with it, for ETAs and such?20:22
gatoxnessita, ah ok.... the "\n" was not being taken into accout, so the space looks better, that's why i remove that20:22
gatoxnessita, i'll restore that20:22
nessitaralsina: sure! but, whose EOD's?20:22
ralsinanessita: I assume yours20:22
dobeynow i need to find another trivial bug to fix20:22
nessitagatox: why was not taken into account?20:22
nessitaralsina: I'll be here one more hour, but let's mumble and I'll give you the updates20:23
ralsinanessita: cool20:23
ralsinanessita: will be in mumble in1'20:23
nessitasure20:23
gatoxnessita, the label was ignoring the \n20:24
nessitagatox: is that a QT thing?20:24
gatoxnessita, nop.... i think something in between the translated string or something was causing to remove that \n20:25
nessitahum20:28
nessitaweird20:28
gatoxyep20:28
ralsinagatox, nessita, alecu: taking 2|3 hours off, will be back late tonight to see if we can ship and/or do the build20:33
ralsinaif anyone needs me urgently, phone me :-)20:34
alecuralsina, ack!20:34
gatoxralsina, ack20:34
nessitaack20:34
nessitaralsina: did you finish my reviews, by any chance?20:35
j0nrello20:35
j0nrI make use of ubuntuone and ubuntuone music for Android.20:35
j0nrI currently have my music collection synced to the U1 server20:35
j0nrI would like to move my local music collection from one machine to another. Is this going to be possible without syncing the whole lot again?20:36
beunoj0nr, you're going to copy over the music yourself from one machine to the other?20:37
j0nrbeuno: yes20:37
beunoj0nr, when did you sync this music for the first time?20:37
j0nrumm a few months ago20:39
nessitagatox: I added my previuos comments to the sso MP, moving to controlpanel now20:39
beunoj0nr, if you're on Natty (11.04), U1 will detect that you've already uploaded it and not do it again20:40
gatoxnessita, ok..... i fix sso already anyway20:40
nessitagatox: ah, ok20:40
gatoxrest for a minutes........ brb20:41
j0nrbeuno: well.........this kinda leads onto my next question. I am setting up a home server using 11.04 server edition. I know I cannot use U1 on server edition (right?) so I thought I would put my music on the server, then mount it back on my laptop as a samba share. If that mount point for the share is the same name as the U1 music sync (currently ~/Music) would that work? I imagine it could play havoc if for some reason the mount doesn't mount 20:43
beunoj0nr, right, I don't know what will happen with mounts, to be honest20:44
beunoI would expect U1 to assume you deleted everything if the mounts doesn;t load before U1 does20:44
j0nrbeuno: would it them proceed to sync, i.e. delete everything on the U1 server20:44
beunoj0nr, yeap, i think it would20:45
beunoit would then maybe re-add them when they appear20:45
j0nrnot ideal20:45
beunobut it sounds like a pretty risky thing to do  :)20:45
j0nrand I can't run U1 from the server?20:45
beunonot officially, but rye is around early in the day, he may have pointers on how you can hack it to work20:46
dobeyj0nr: i would highly recommend not using a mount like that. it won't end well :)20:49
j0nryeah I agree20:50
j0nrjust want my music on a file server,so other devices at home can access it and also sync it to U120:51
dobeyjoshuahoover: why not just share it from within banshee/rhythmbox/whateveryouuse20:51
dobeyerr20:51
dobeyalso, j0nr not joshuahoover20:51
j0nrdobey: what do you mean?20:52
dobeyj0nr: well they have DAAP sharing support. i don't know what devices you want to share it to though.20:52
j0nri am trying to get away from having my music just on a laptop... i want it on a central file server20:52
j0nrthat file server runs ubuntu server....20:53
dobeyj0nr: another option is to just install rygel on your workstation, and point it at your music library there. it will share it over UPNP/DLNA. it's what i use20:53
j0nrbut how do I also sync to U1?20:53
dobeyj0nr: i am talking about keeping the files where they are now20:53
j0nryeah, they are just on my netbook. I don't like the netbook being too important... i.e. if it breaks or needs reformatting it wont matter20:54
dobeyyou don't have a workstation pc?20:55
j0nrnope :)20:55
j0nrwish I did20:55
j0nrI currently do everything from my netbook20:55
dobeyclearly you don't have much music then :)20:56
j0nrthe server is a low power thing not suitable for gui ubunt20:56
j0nru20:56
j0nrdobey: about 14Gb at the mo of music20:56
j0nris that a lot or a little?20:56
dobeya little20:57
j0nrheh21:02
j0nrits a lot to upload to U1 when you only have like 50k upload speed21:02
j0nrtook like 48 hrs21:03
j0nrthats why I don't wanna have to redo it.21:03
nessitagatox: when you come back, I had to add a needs fixing for the controlpanel branch, I just realized the wording for the checkbox had a incorrect (semantic) meaning21:03
dobeyeek21:03
* dobey hugs his bandwidth21:03
j0nrdown is ok, but up sucks21:04
beunoj0nr, U1 should for the most part never re-upload something once you've uploaded it21:05
j0nrbeuno: what if I renamed a U1 synced folder locally? i.e. the top level one?21:05
j0nrI have ~/Music checked to sync21:06
j0nrwhat ifI renamed it?21:06
beunoit should be fine21:06
j0nror what if I moved its contents to another U1 synced folder?21:06
beunoshould still not re-upload it21:06
beunogive it a try, see for yourself  :)21:06
j0nr:)21:06
=== m_conley is now known as m_conley_away
j0nrso, "IN THEORY", if I mounted an empty share, checked it to sync, then moved my music into it... it would just move on the server?21:08
beunoyes, although, again, once there's a mount point, all bets are off21:08
dobeywe do not support mount points at all, not even a little 'unofficially' bit :)21:12
ralsinanessita i did I think21:20
ralsinadid I miss one?21:21
nessitaralsina: both? did the folder reload list finally worked for you?21:21
nessitaralsina: this one https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/delay-reload-folders/+merge/7463321:21
ralsinanessita: never was able to try it because of the problem with the paths21:21
ralsinalet me give it another try21:21
nessitaralsina: please, I think you then said you could not reproduce the assertion failure, no?21:22
ralsinayes but did not try this one again21:22
ralsinanessita: trying now21:22
dobeyhave a good evening everyone!21:23
nessitagatox: I still dead people21:23
nessitaI mean, I still see the CAPTCHA_SOLUTION_ENTRY removed from the branch21:24
nessitagatox: weird thing is that there is a mocker test that should fail, but is not failing :-(21:24
nessitagatox: AH! is failing on windows but not in linux21:25
nessitaCRAP21:25
nessitagatox: ok, all failures and comments added to the MP21:27
ralsinanessita: it doesn't fix it for me with 1, yes with 221:27
nessitaralsina: CRAP**N21:27
nessitaralsina: ok, I'll change it, but this sucks :-)21:27
ralsinanessita: indeed21:28
ralsinanessita: OTOH, if this hides the symptoms for some users and there is really no functional breakage, it's a valid workaround until we can fix it properly next week or so21:28
nessitaralsina: yadda yadda ;-)21:29
ralsinahehe21:29
nessitaralsina: chaging...21:29
ralsinanessita: we have fixed so many bugs the last two months.... we can fix 4 or 5 more next week, no sweat ;-)21:29
nessitaralsina: Pushed up to revision 220 (2 seconds instead of 1)21:32
ralsinalisto, la mando a trunk21:33
ralsinaoops, E_WRONLANG21:33
nessitajuaz21:33
nessitaralsina: I still don't get the computer-to-cloud page in the cloud-to-cloud branch :-/21:49
nessitaralsina: comment added....21:51
alecunessita, ralsina: I think I found why that dialog is shown21:52
nessitaalecu: I know why the dialog is shown :-)21:53
nessitaalecu: I added that code21:53
alecunessita, why?21:53
nessitaalecu: the issue is that we should not be getting a webclienterror21:53
alecunessita, why not? it's being shut down!21:53
nessitaalecu: I added a generic handler to catch any error21:53
alecunessita, that's insane!21:53
nessitaalecu: ...21:53
nessitaalecu: wanna mumble it?21:54
alecunessita, we should not be blindly handling *every* error21:54
alecushure21:54
alecusure too21:54
nessitaalecu: loading mumble21:54
gatoxback....... :P22:00
alecuhello gatox22:32
gatoxalecu, hi22:32
alecuafaik the release should be tonight, but it will likely won't be22:32
gatoxalecu, ahhhhhh ok.... i just wanted to know to be sure if i could finish with some stuff before :D22:33
nessitagatox: we haven't solved some important issues (low layer), so, it may be tomorrow22:33
nessitagatox: did you saw the 2 reviews I made?22:33
gatoxnessita, yep..... fixed already22:33
nessitagatox: I will finish those tomorrow first time, I'm too burnt out now22:36
gatoxnessita, of course!22:37
gatoxnessita, i took a revitalizing nap! jeej22:37
=== Guest87485 is now known as gord
alecugatox, can I ask you a Qt question regarding "Signals and Slots Across Threads"?22:40
gatoxalecu, yes! sure22:41
alecuI'm looking at this: http://doc.qt.nokia.com/stable/threads-qobject.html#signals-and-slots-across-threads22:41
gatoxalecu, yep.....22:41
alecuand I'm trying to understand in which thread will the signal handlers be called in qtreactor22:42
alecugatox, the qtreactor is creating a QSocketNotifier22:42
alecugatox, and it's connecting it like this:22:43
alecu        QObject.connect(self.notifier, SIGNAL("activated(int)"), self.fn)22:43
alecugatox, so, I wonder what type of signal-slot connection type it will use (according to the docs above)22:43
gatoxalecu, that should be direct connection22:44
alecumy guess is that in order for it to be safe, it should be using either "Queued" or "Blocking Queued"22:44
gatoxalecu, , which is the most common case22:44
alecuoooooh!22:44
gatoxalecu, maybe i'm wrong, but you usually use direct connection22:44
alecugatox, my gut feeling is that qtreactor is probably unsafe :-(22:45
gatoxalecu, you mean thread safe?22:45
alecugatox, yup.22:45
gatoxalecu, because direct connection works properly with threads.... i don't know the specific case.... but any direct connection can communicate any thread with the main thread when it's needed without causing any conflict22:46
alecugatox, the thing is that twisted functions should only be called from the main thread22:47
alecugatox, and qt starts a new thread for each QSocketNotifier22:47
alecugatox, but that's ok22:47
gatoxalecu, do you have any log of the bug or something?22:48
alecugatox, what's wrong is that if the qsocketnotifier slot is called on a different thread it may break twisted.22:48
alecugatox, no logs. I found this trying to understand the weird behaviour we have some times while shutting down.22:49
alecugatox, anyway, I would have to prove first that the slots are being called in the wrong thread22:49
gatoxalecu, ah..... probably we should ask ralsina to be sure..... but if you really need that, you could connect qsocketnotifier to the main thread, and let the main thread to call twisted..... as a kind of wrapper.... just a thought22:50
alecugatox, right: modifying qtreactor so it connects the slots using the "Queued" type22:50
alecugatox, ok, thanks for the info.22:51
alecugatox, I just tested, and it seems that all the calls are being done from the main thread.23:00
alecugatox, so, it's not a thread safety issu23:00
gatoxalecu, so it's not related to a qt thing23:00
alecugatox, yup: that part of the qtreactor is safe :P23:01
nessitaralsina: we need a review in this branch from facu https://code.launchpad.net/~facundo/ubuntuone-client/temporary-files-cleanup/+merge/7469923:42
ralsinanessita: got it!23:42
nessitaralsina: did you check the cloud to cloud thingy?23:42
ralsinanessita: not yet, I am supposed to be cooking :-)23:43
nessitaah23:43
nessitaI m supposed to be taking a bath23:43
nessitapero son rachas :-P23:43
ralsinanessita: is that an account with UDFs or a "clean" one?23:44
nessitaralsina: an account with UDFs23:44
ralsinaweird23:44
nessitaralsina: the cloud-to-computer looks great23:44
nessitabut computer-to-cloud is not there23:44
ralsinaok, I'll take another look after dinner23:44
ralsinacould you try one with no UDFs?23:44
nessitaralsina: honestly? not today :-(23:45
ralsinanessita: understood23:45
ralsinago bath :-)23:45
nessitaralsina: I'm trying to fiinsh last 2 branches from diego23:45
nessitaralsina: so they land and you dont have to do anything special23:45
nessitagatox: sso one approved!23:48
gatoxnessita, awesome23:48
nessitaonto the cp now23:48
nessitagatox: cp +1'ed!23:52
nessitaok, I'm off crowd23:56
nessitasee ya all tomorrow!23:56
* nessita will get some rest23:56
nessita(for the final countdown!)23:56

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