[00:00] CardinalFang: Hi! Thanks for the review. You've double-approved ubuntuone-files-java-client - was one of the approvals for u1f progressbar branch? [00:00] karni, no, I don't think so. [00:00] CardinalFang: ok [00:01] :) [06:41] U1 down for DB maintenance [07:11] and service should be back now [07:49] hi guys :) [07:49] is there a client for Mac OS or any other way to download the entire contents from Ubuntu One directly to my Mac? [08:19] Agravain: nope we havent got one yet [08:56] fagan: ok, thank you for replying [09:07] Good morning! [11:06] good rmoning everyone! [11:13] nessita, good morning [11:14] hi gatox, how is it going today? [11:14] nessita, nice! :P [11:21] ralsina: ping === zyga is now known as zyga-doctor === yofel_ is now known as yofel [11:38] ralsina, ping [11:50] gatox: if you need anything, you can ask me, I may be able to help you. Also, can I have a trivial\isima review for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/dislexia-no-more ? [11:50] nessita, ok..... reviewing... i was looking for ralsina because i have some question about one of his branches that need review [11:50] gatox: ah, I can't answer that :-D [11:51] :P [12:14] good morning! [12:14] hi ralsina [12:14] hi nessita [12:14] gatox: good morning amd feel free to ask :-) [12:14] ralsina: were you able to review my "inherited" branch? :-) [12:14] and pongs [12:15] nessita: almost done, only need to run tests on linux [12:15] great! [12:19] ralsina, i left a comment in your branch [12:20] gatox: ok! [12:22] nessita: +1 [12:23] nessita, ahhhhhh +1 to your branch..... i forgot to tell you [12:24] :-) [12:27] gatox: answered your comment [12:27] * gatox reading... [12:30] ralsina, just an idea... maybe i'm terribly wrong :P..... but couldn't this be done with a qthread... where the process is executed inside the thread and when the thread finish (emit: finished) execute that function?? so in this way it's not going to be an arbitrary number [12:31] gatox: I am fairly sure we are using python threads elsewhere, and you can't mix those and QThread if I remember right [12:32] ralsina, ok..... was just an idea :P [12:33] ralsina, i'm having some problems with the installer.... the one in trunk [12:34] ralsina, i delete my local copy and rebranch it..... and it stills continue...... when i open the installer...... it shows me the license page....... with a finish button.... i can't go to the next page :S [12:35] ralsina, and i have everything up to date [12:37] gatox: yes, the brach that fixes that is merging right now [12:37] gatox: the problem was: if it couldn't start sso-client, it failed that way [12:38] ralsina, ahhhhhhhh okok [12:41] dobey: ping [12:42] ralsina, i'm looking at Bug: #823414... should i launch the uninstall when the user click Disagree or not? Was this defined? [12:42] Yes, we should [12:43] We should ask the user if he's sure (with our own dialog), then launch uninstall, then show a "it's uninstalled" message. [12:43] ralsina, ok [12:44] bad side is, currently the uninstaller is invisible, not sure why :-) [12:47] ralsina, ok... i'll take a look at that [12:48] gatox: ok, ask lisette for words and stuff ;-) [12:48] ralsina, ok [12:51] words? :) [12:52] lisette: http://www.youtube.com/watch?v=w_Rut4qm33g :D [12:52] lisette: give us words!!! :-P [12:53] aargh, i am blind now! [12:53] lisette: they are one of the greatest Irish bands ever [12:53] they were our backstreet boys [12:54] lisette: you mean deaf? :-D [12:54] nessita: and deaf! [12:54] The lack of musical taste in this team is terrible [12:55] fagan: dude, you actually like avril lavigne! :-) [12:55] so, you may wanna consider the musical taste issue is the other way around :-P [12:55] nessita: hah yeah but I was in the generation that that was acceptable [12:56] fagan: you, calling me, old? [12:56] * nessita prepares the counterstrike [12:56] *cough* *cough* [12:56] I'm old but with honor! and prefect musical taste ;-) [12:56] * nessita runs [12:57] nessita: na your not that old but you were in the good old generation that had power metal and nirvana [12:57] :-) === zyga-doctor is now known as zyga [13:00] me [13:00] me [13:00] ralsina nessita dobey alecu gatox standup [13:00] fagan, ack [13:00] me [13:00] me [13:01] alecu, dobey? [13:01] Ill go they can catch up [13:01] DONE [13:01] * Finished off the redirects [13:01] CURRENT [13:01] * Reading some books waiting for another assignment [13:01] BLOCKED [13:01] * Nope [13:01] nessita: go [13:02] DONE: finished dont-shutil-move branch (waiting for alecu's review), tried and failed miserably to propose a clean solution for bug #827343 [13:02] TODO: propose some solution for bug #827343 [13:02] BLOCKED: nopes [13:02] NOTES: I'm leaving at noon today to go to university [13:02] NEXT: gatox [13:02] Launchpad bug 827343 in ubuntuone-client "Send platform details when pinging for credentials (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/827343 [13:02] DONE: [13:02] Fixed: Bug #819965, Bug #820872, Bug #820874 [13:02] TODO: [13:02] Add Tests. Bug #823203, Bug #823414 [13:02] BLOCKED: [13:02] No [13:02] Launchpad bug 819965 in ubuntuone-windows-installer "Congratulations typos (affects: 1) (heat: 17)" [Undecided,New] https://launchpad.net/bugs/819965 [13:02] ralsina, go [13:02] gatox: Bug 820872 on http://launchpad.net/bugs/820872 is private [13:02] gatox: Bug 820874 on http://launchpad.net/bugs/820874 is private [13:02] DONE: closed bug #825473, merging fix for bug #825491, worked on bug #825513, small fixes on the installer-builder-branch [13:02] gatox: Bug 823203 on http://launchpad.net/bugs/823203 is private [13:02] TODO: close those two bugs, autostart/credentials testing [13:02] BLOCKED: no [13:02] gatox: Bug 823414 on http://launchpad.net/bugs/823414 is private [13:02] Launchpad bug 825473 in ubuntuone-windows-installer "License page may show a "Finish" button (affects: 1) (heat: 36)" [Critical,Fix committed] https://launchpad.net/bugs/825473 [13:02] Launchpad bug 825491 in ubuntuone-windows-installer "The "Local Folders" page tries to do things without credentials (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/825491 [13:02] Launchpad bug 825513 in ubuntuone-windows-installer "The local folders page is not up to spec (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/825513 [13:03] me [13:04] λ DONE: bug #824666, bug #824669, bug #826772 [13:04] λ TODO: more installer fixes, deal with tech board [13:04] λ BLCK: None. [13:04] Launchpad bug 824666 in couchdb-glib (Ubuntu Oneiric) (and 1 other project) "ftbfs on all archs (affects: 1) (heat: 6)" [High,Fix released] https://launchpad.net/bugs/824666 [13:04] Launchpad bug 824669 in evolution-couchdb (Ubuntu Oneiric) (and 1 other project) "ftbfs on all archs (affects: 1) (heat: 6)" [High,Fix released] https://launchpad.net/bugs/824669 [13:04] Launchpad bug 826772 in ubuntuone-installer "Installer currently hardcodes 'natty' as series (affects: 1) (heat: 6)" [Critical,In progress] https://launchpad.net/bugs/826772 [13:04] alecu [13:05] dobey: I need to talk to you about bug #820345 [13:05] Launchpad bug 820345 in ubuntuone-client (Ubuntu) (and 1 other project) "Please remove UbuntuOne from MessagingMenu (affects: 2) (heat: 14)" [Undecided,Confirmed] https://launchpad.net/bugs/820345 [13:06] DONE: worked on bug #824819, started re-review of nessita's branch [13:06] BLOCKED: by a few more broken things I found on windows while fixing tests, like http://pastebin.ubuntu.com/666883/ [13:06] TODO: see if the above only happens to me, work on #824003 [13:06] Launchpad bug 824819 in ubuntuone-client "SD and tests need to wait until a watch is removed (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/824819 [13:06] nessita, ralsina, gatox: can you guys try the above pastebin? [13:06] alecu: sure [13:06] alecu: linux or windows? [13:06] ralsina: ok [13:06] nessita, windows [13:06] alecu, ok === m_conley_away is now known as m_conley [13:07] gatox, make sure to do "from ubuntuone.syncdaemon import tritcask" first. [13:07] alecu: I have 100 all the tiem [13:07] alecu, ok [13:07] time* [13:08] alecu: but it's been ages since I rebooted this VM [13:08] alecu: I get 100 all the time too [13:08] alecu: we had an issue with this, but verterok fixed it [13:09] nessita, ralsina: 100 is perfect [13:09] alecu: you 100% up to date? [13:09] nessita, this happens after I merged trunk. [13:09] nessita, I should try on trunk [13:09] alecu, http://paste.ubuntu.com/667280/ [13:09] alecu, i don't get 100 all the time [13:09] wow [13:09] gatox: you booted this machine today? [13:09] alecu: len(set(len(set([tritcask.DataFile._get_next_file_id() for n in range(100)])) for i in range(1000))) == 1 [13:09] nessita, no..... should i do that? [13:10] gatox: nopes [13:13] nessita, gatox: virtualbox on 32bits or 64? [13:13] alecu: 32 for me [13:13] alecu: ah, sorry, I misunderstood [13:14] alecu: host is 64, guest is 32 [13:14] nessita, the same on my case. [13:14] alecu, 32 [13:14] alecu, same as nessita [13:14] alecu, the vm is 32... host 64 [13:16] anyway: this shows that time.clock is not being reliable (at least under vbox): it's returning the same value for calls near in time [13:16] this broke a few of the tests, and it's been hell to debug [13:17] alecu: you sure you're running latest trunk? we added a fix for this recently (bah, verterok did) [13:17] * verterok looks around [13:17] alecu: hmmm... check if you have virtualbox's "fix client's clock" set [13:17] and/or windows's NTP equivalent (in control panel, date/time) [13:17] verterok: hey there! alecu is having bad-bad issues with tritcask file generation, id's gets duplicated [13:18] nessita: yes, reading the backlog [13:18] ralsina, windows' ntp updates the time *just* once every day [13:18] alecu: ok, so that's not it [13:18] alecu: tritcask is now using time.time() + time.clock() it has more resultion/accuracy than time.time on linux :p [13:18] ralsina, also, I've tried setting the vbox time update flag, and the result is the same. Let me fetch the article that explains it... [13:19] alecu: do you have a WindowsTimer inside ubuntuone/syncdaemon/tritcask.py? [13:19] nessita: I was about to ask that :) [13:19] maybe it's using a installed tritcask? [13:19] alecu: maybe your syncdaemon is loading an "old" pyc for tritcask, from an old install, for example? [13:19] right [13:20] "print tritcask" should clear that [13:20] nessita, verterok: let me check that. [13:21] btw, I tried with: vboxmanage setextradata "VBoxInternal/Devices/VMMDev/0/Config/GetHostTimeDisabled" "1" [13:21] as explained here: http://the-starport.net/freelancer/forum/viewtopic.php?topic_id=4100 [13:21] but it seems to fix a different issue. [13:29] nessita, verterok: I added a "print '*'," inside WindowsTimer.time, in trunk, and I just made sure I'm using the latest version, and I still get the same behaviour. [13:30] it seems that time.clock is not moving forwards :P [13:30] alecu: that's weird :( [13:31] verterok, I even tried adding three zeroes to the multiplier in _get_next_file_id [13:32] alecu: could you try this: python -c "import time; time.clock(); print [time.clock() for i in range(10)]" [13:33] verterok, !!!! [13:34] verterok, on the second run it was all zeroes!!! :P [13:34] verterok, http://pastebin.ubuntu.com/667307/ [13:34] verterok, on the first run they were sensible values [13:34] your clock is broken(?!?) [13:35] this is a big WTF [13:35] verterok, and it happens 50%: http://pastebin.ubuntu.com/667310/ [13:36] alecu: is this a win 7 vm, right? [13:36] verterok, win 7 32bits, on the latest virtualbox on a natty 64bits [13:36] ok, I don't have the same env :( [13:37] verterok, gatox managed to reproduce this, but neither nessita nor ralsina were able to reproduce it. [13:37] maybe virtualbox is broken? [13:37] verterok, yeah, it looks like it. [13:37] alecu: all are using the same virtualbox version? [13:37] we can switch to the timeit module, I suppose :-( [13:37] ralsina: your windows is not a vm but a real install, right? [13:38] nessita: yes, a real one [13:38] ralsina: no we cant :( [13:38] ralsina: timeit uses time.clock in windows :p [13:38] he [13:39] ralsina: can you run the "test" several times with a clean reboot to see if we can reproduce this outside VB? [13:39] ralsina, nessita trivial branch: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/congratulations-typos/+merge/71690 [13:39] gatox: looking! [13:39] no, I can't reboot because I am in the management call :-( [13:40] Hi [13:40] ralsina: maybe later? :-/ [13:40] I have a little problem uploading 1.4 GB zip file on ubuntu one free account... [13:40] in about one hour, yes [13:41] nessita, ralsina: I need to be afk for an hour or so. [13:41] upload was done, but "please wait" was still there, and would never go away... :S [13:41] alecu: were you able to finish my review? [13:41] is it possible to upload such a big file? [13:41] nessita, not yet [13:41] Milos_SD: hi, is this using the web or the desktop client? [13:41] alecu: ok, I'll share some comments with you when you get back [13:42] verterok, web [13:42] nessita, some comments on that branch? [13:42] rye: around? [13:42] verterok, around [13:42] rye: hi! Milos_SD is having some problems uploading via the web...do you know something about that? :) [13:42] alecu: yeap -> confirmed that dont-shutil-move does not fail when committting a partila file that is failing in trunk. maybeBadNews is this: http://pastebin.ubuntu.com/667286/ (watch warnings over files) [13:43] Milos_SD, could you please try going to https://one.ubuntu.com/files/ and see whether the file has been uploaded indeed, w/o closing the first tab [13:43] verterok, I don't even know if I can upload specific file from desktop client [13:43] alecu: but if you need to go, we can talk later (but before 12ART please) [13:43] nessita, I closed that tab a long time ago :) [13:43] rye, I closed that tab a long time ago* [13:44] nessita, ack. [13:44] nessita, i'm changing something in my branch... let you know in a while [13:44] there was no network activity at my side... but "please wait" was still there :S [13:44] gatox: ack [13:46] Milos_SD, ah, and the file is not there, I supposed? [13:46] rye, yes... there is no file there :( [13:52] rye, maybe there is some problem with time spent uploading? I have only 512 Kbps upload, so it would take up to 7 hours to upload 1.4 GB file... [13:53] Milos_SD, is your IP address static? Could you please PM me your current IP address? [13:53] rye, it is not static... but I have cable internet, so lease is very high [13:54] Milos_SD, well, "static enough" - I want to see whether we have reports of your upload failing [13:56] nessita, branch updated [13:57] gatox: ack [14:06] gatox: this does not look right: [14:07] "launch the dashboard to get more information about the sync in progress". I think it should be "sync progress" and not "in progress" [14:07] gatox: another question, you added 2 checkboxes but the logic for those is not added? [14:08] nessita "the sync in progress" is grammatically correct. So is "the sync progress", of course :-) [14:08] ralsina: both are gramatically correct in isolation, but in the context used I don't think "in progress" is correct [14:09] ah, I just read it with a different "accent" and I see the other possibility [14:09] nessita, i've created another bug to add the logic of those checkboxes, because the bug says that the Congratulations page only have some typos... in order to submit that and don't delay the branch for other issue... [14:09] hehe [14:09] "launch the dashboard to get more information about the sync that is in progress" [14:09] exactly [14:09] nessita, should i close the new bug and do it all together? [14:09] gatox: then don't add the checkboxes in that branch... [14:09] gatox: just submit the typo in this branch [14:09] so you can fix that bug [14:10] also, unless the width change is actually necessary, there is no point changing that line [14:10] nessita, ok... and about the text... i copy the text from the wireframe [14:10] that was updated [14:10] gatox: ok then [14:12] nessita, ok..... branch updated..... in the next one (later) i will add the checkboxes with the logic [14:13] great [14:21] gatox: approved and globally approved === gatox_ is now known as gatox === teknico_ is now known as teknico [14:33] nessita, verterok: is it still useful if I reboot and try the tritcask thing? [14:33] ralsina: afaik, yes, we have no new ideas in that front [14:34] ok, this is going to take a while... [14:46] hi! will there ever be an Ubuntu One appliance? I have a school district that's using iFolder and Ubuntu One would make a great replacement if they could store data locally [14:52] nessita, verterok: rebooted twice, windows 7 64 bits, I always get good results [14:52] alecu: just in time! see what ralsina says [14:52] nessita, are you still available to tell me about that issue? [14:52] alecu: yes sir [14:52] ralsina: great news for tritcask, bad news for virtualbox(?) [14:52] alecu: did you get any chance to look at the paste? [14:52] I will now try a VM see what happens there [14:53] nessita, I'm looking at the paste, but I don't understand what's broken in it [14:54] alecu: I'm not sure if something is broken, I saw this: 2011-08-16 10:08:59,510 - ubuntuone.SyncDaemon.filesystem_notifications.GeneralProcessor - WARNING - Ignoring path as we don't have enough permissions to track it: 'C:\\Users\\Test\\Ubuntu One\\fideos: dos?.png' [14:54] alecu: and that "looks" buggy [14:54] verterok, I remember that you told me that tritcask cannot use an incremental numbering because of something..... what was it? [14:54] alecu: the path is not readonly, but is not dir. So I see that log warning and makes me think that maybe we have a bug in the add watcher code? [14:55] alecu: tritcask needs the timestamp in the data file name as it's a way to easily sort the files [14:56] verterok: on a XP vm I get much *smaller* values, but not 0: https://pastebin.canonical.com/51293/ [14:56] verterok, what if we add a dot-numbering to the filenames? [14:56] we can change that, but it means a metadata migration in linux [14:56] verterok, as in timestamp-sequence.cask ? [14:56] verterok, oh, right. [14:57] ralsina_, small is nice. Zero is plain wrong. [14:58] ralsina_: those values looks good [14:58] alecu: do you have the same python version? any weird import from __future__ (like, from __future__ import round :p)? [14:59] version of virtualbox? [14:59] verterok, I'll take a look [14:59] alecu: I understand what you'r trying to solve with the sequence...but I think the problem is something outside sd implementation [14:59] verterok, it surely looks like it. And I'm willing to concede it's a virtualbox bug [15:00] verterok, but this means we will not be able to run any kind of tests in virtualbox [15:00] nessita, I think I found it: "os.access(path, os.R_OK)" [15:00] alecu: also the sequence workaround might not be enough, we need to think on a more broad change [15:01] nessita, it's using path as a byte sequence [15:01] alecu: use kvm? :) [15:01] alecu: ack, I'll file a bug [15:01] nessita, in is_ignored() in ubuntuone/syncdaemon/filesystem_notifications.py [15:01] alecu: I did the change using virtualbox, so it's working there [15:02] verterok, it's working for ralsina and nessita on virtualbox too... but it fails for gatox too [15:02] verterok, but if I was nessita or ralsina_, I would not trust it either... [15:02] hmmm [15:02] alecu, gatox, nessita: what is your VBox version? [15:02] gatox, what's you cpu? amd? [15:03] alecu: so, 3 - 2...we win! [15:03] verterok, hahahaha [15:03] alecu: model name : Intel(R) Core(TM) i5 CPU 750 @ 2.67GHz [15:03] I'm sure nessita and ralsina and verterok are on intel [15:03] I'm on an amd [15:03] gatox? [15:03] yes, i5 here [15:03] alecu: VB is a 4.1.0 [15:04] I am in 4.0.4 so it's probably not the version [15:04] alecu, ralsina_ vbox version: 4.0.10_ubuntu r72426 [15:04] alecu, sorry for the delay...... about window don't let copy the version and it closes everytime it lose the focus :P [15:04] alecu, my host is intel core2duo [15:05] 4.0.12 r72916 [15:05] gatox, :-) [15:05] hmmm... here's an idea. Alecu: on one terminal, start a "dir /s c:\" [15:05] and on another try the clock thing [15:06] ralsina_, good point [15:06] There are some old bug reports about the clock being stuck if the VM is idle [15:06] also, please everybody: let's paste the version of the "virtualbox guest additions"; mine: 4.0.10 r72479 [15:07] I should probably update them to the latest version too. [15:07] How about let's try updating everything to the latest and it may fix itself? [15:08] ralsina_, the thing is that my vbga do not match my vbox [15:08] ralsina_, running dir/s did not help [15:08] alecu: interesting [15:09] alecu: where can I see that version? [15:09] alecu: I installed VB from the website .deb, not from our repos [15:09] nessita, me too [15:09] nessita, in the tray bar, hover over the vbox icon [15:09] nessita, open the traybar with the up arrow if needed. [15:10] nessita, 4.1.0 ? [15:10] 4.1.0r73009 [15:10] hmmm... then my vbox must be from the repos instead. [15:11] alecu: returning to the watch issue, I also noticed in http://pastebin.ubuntu.com/667286/ that the WARNING appears twice, which sounds suspicious (specially since we have threads involved) [15:11] gatox: where did you install VB from? [15:11] CardinalFang, thisfred_: I have a bug for you -> https://bugs.launchpad.net/ubuntu/+source/desktopcouch/+bug/780972 <- do you have any idea what's causing it? [15:12] Launchpad bug 780972 in desktopcouch (Ubuntu) "desktopcouch-service crashed with Unauthorized in request(): ('unauthorized', 'Authentication required.') (affects: 144) (dups: 9) (heat: 698)" [High,Confirmed] [15:12] nessita, from repository [15:12] CardinalFang, thisfred_: because of that, dc doesn't work at all for me on oneiric [15:12] gatox: ah! [15:13] nessita, "deb http://download.virtualbox.org/virtualbox/debian maverick contrib" [15:13] maverick? hum [15:13] alecu: no natty? [15:14] nessita, so mine's from upstream [15:14] alecu: can you try this one? http://download.virtualbox.org/virtualbox/4.1.0/virtualbox-4.1_4.1.0-73009~Ubuntu~natty_amd64.deb [15:14] nessita, I very likely installed it when natty was being developed [15:14] nessita, sure! [15:14] alecu: or choose the best for you: http://download.virtualbox.org/virtualbox/4.1.0/ [15:18] rodrigo, I'll have a quick look [15:18] thisfred_, thanks! [15:21] nessita, the duplicated logs seem to come from the loop in freeze_commit [15:21] looking [15:22] nessita, it seems to me that there are a pair of events coming from the same path, and being ignored there [15:26] alecu: I followed the code and I'm not sure what 2 pair of events are being ignored [15:27] lunch time, bbiab :) [15:27] nessita, do you have more context for this log? [15:27] nessita, more lines above the ones you pasted? [15:28] alecu: the whole run is: http://pastebin.ubuntu.com/667282 [15:33] nessita, ubuntuone/platform/windows/filesystem_notifications.py, process_IN_MOVED_TO() [15:34] alecu: without checking, this code looks duplicated from linux.... is that soe? [15:34] nessita, there are the two calls to is_ignored [15:34] right [15:34] nessita, I've no idea if this is duplicated [15:35] Can I buy music using the phone? [15:36] alecu: how did you end up checking process_IN_MOVED_TO? I don't see the "move" in the log === kklimonda is now known as Guest54756 [15:39] nessita, I looked at what was calling is_ignored() [15:39] nessita, and that place looks likely, because there are two calls in a row [15:40] alecu: if we're having a move event in this case, we're in problems (ie we're getting events that we should not be getting) [15:41] which brings me back to the fact that we need to fix the eq_inotify tests... [15:41] alecu, ralsina_, gatox, Chipaca (if available): can we have the mumble? [15:41] nessita: fine by me [15:43] nessita, ack [15:44] alecu: can you join mumble please? [15:53] dobey: rodrigo_: https://bugs.launchpad.net/ubuntu/+source/desktopcouch/+bug/780972 confirmed, I have no idea why though. Could be new couchdb with missing /broken patches, could be broken js in new couchdb again, could be something changed in the way we store the oauth keys. I really have no time to debug this right now. [15:53] Launchpad bug 780972 in desktopcouch (Ubuntu) "desktopcouch-service crashed with Unauthorized in request(): ('unauthorized', 'Authentication required.') (affects: 144) (dups: 9) (heat: 698)" [High,Confirmed] === Guest54756 is now known as kklimonda [15:57] dobey: rodrigo: no errors in the couchdb logs though (just 401s), so it may not be couchdb itself at all, and we're just not sending the right tokens, or not storing them in couchdb in the first place. === beuno is now known as beuno-lunch [16:12] nessita, http://webchat.freenode.net/ [16:14] * nessita -> teaching duties [16:14] bye all! [16:23] ralsina_, it seems that it's going to be an easy fix change the messagebox for the label in all the pages [16:28] thisfred: pensive panda [16:28] pessimistic pony [16:29] ponderous plato [16:41] not sure why, but for some reason i am wary of products with "super quality" in their name [16:45] ralsina_, ping [17:10] ralsina_, let me know when you are back... i've a branch almost ready for revision.... but there is something i'm not quite convince === beuno-lunch is now known as beuno === alecu is now known as alecu-lunch [18:31] gatox: I'm back sorry, family things [19:07] ralsina_, actually when critical it's called from controller, it is called with 4 params: self.view, self.view.wizard().app_name, CAPTCHA_LOAD_ERROR [19:08] yes... [19:08] ralsina_, is it of if i modified that calls to just pass the string?? the other params now that it's just a simple label are no longer necessary [19:08] *of = ok [19:08] is it ok....? [19:08] I don't really remember that code [19:09] is critical() defined on the wizard or in the page? [19:09] wizard [19:09] or is it gettig a real QMessageBox? [19:09] critical creates a qmessagebox on the wizard and show it [19:10] and in controllers if the param message_box is None, it creates a QMessageBox [19:10] ok, cool then it's ok :-) [19:11] ralsina_, ok.... i'll keep working on that... i'm having problems with some tests, but the code seems to be ready [19:12] what tests? [19:17] ralsina_, they are some tests actually testing "critical", but now i changed the behavior of that function... just that [19:17] oh, ok === alecu-lunch is now known as alecu [20:16] I am stopping for a while. Will work some more late tonight, so please email review requests etc. [20:24] ralsina_, ok, i'm finishing the tests for my branch [20:24] so i'll let you know [20:24] cool [20:25] ralsina_, i was more time writing fixing tests than code :P [20:25] gatox: he, that's standard ;-) [20:41] Ready for review: https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/bug-820872/+merge/71773 [20:41] https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/bug-820872/+merge/71774 [20:54] have a good evening all [20:55] dobey, bye === m_conley is now known as m_conley_away === yofel_ is now known as yofel [22:01] hello everyone! [22:01] alecu: ping [22:05] ralsina_: you around? [22:06] nessita: hi [22:07] ralsina_: hi there. Would you know if alecu finished the review of my inherited branch? LP shows no review from him [22:07] nessita: didn't hear from him after you left [22:07] oh [22:08] nessita, hi [22:08] alecu: hi there! [22:08] nessita, I left the test running... [22:08] and here he is :-) [22:08] let me approve it [22:08] alecu: thanks! :-) [22:09] if noone needs anything right now, I'll go back to playing "Battle beavers" with Tato :-) [22:09] I will do reviews late tonight though [22:10] ralsina_, what's "Battle beavers"? [22:10] alecu: http://www.abcjuegos.net/juego/battle-beavers [22:10] not as interesting as it sounds ;-) [22:10] :-) [22:11] nessita, alecu: if I could get a review for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/too-soon/+merge/71426 it would be great [22:13] ralsina_: in that branch, question: [22:13] shoot! [22:13] ralsina_: when do you stop the timer? I mean, I would expect the timer to be turned on only when "entering" the folders page, and stopping it when leaving it [22:14] it is started when entering, but forgot to stop it, right [22:14] otherwise looks like it wastes resources when is not needed... [22:14] I will add a stop when switching to the next page [22:14] ralsina_: also, please add tests! :-) (faking the timer, I would advice) [22:14] nessita: the function that the timer calls has tests [22:15] ralsina_: I hate having this conversation!!! :-) [22:15] ok, let's not have it then. I will figure out more tests, I suppose. [22:15] you need to test every single line of code you add. This is new functionality, we want tests for it... [22:15] The timer is not new functionality [22:16] it has been there for a month [22:16] but sure, I will add it [22:16] thanks [22:16] ralsina_: also, we should have a test for the self.ui.add_folder_button.backend.account_info failing, to ensure we;'re no having errors like before [22:16] Since I already I test what the timer triggers, I could add a test that the timer is startted, I suppose. But don't worry, I will add 10 or 12 more tests. [22:17] bye, see you all tomorrow. [22:34] nessita, i send you a branch for review [22:34] sent* [22:35] 2 branches :P [22:35] gatox: have the links handy please? [22:36] nessita, https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/bug-820872/+merge/71773 [22:36] nessita, https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/bug-820872/+merge/71774 [22:36] nessita, they are related [22:39] gatox: so, one question... [22:39] gatox: is the sso code calling the critical method that is defined in windows installer? [22:40] that is puzzling my head, shouldn't that be the other way around? [22:40] gatox: or critical is also defined in sso? [22:44] nessita, no, it's calling the critical method in the installer, because the controller receive an instance of the wizard [22:45] nessita, sorry, of the pages [22:45] but with the pages you can access the wizard [22:45] gatox: what if someone uses a wizard that does not have the critical method defined? [22:46] gatox: should the sso side send a siganl, and the other side should connect to the signal and do something in that case? [22:46] nessita, sorry.... not following...... there is only one wizard..... [22:46] gatox: what I mean is: [22:46] gatox: the sso client is a generic project that can be used from any app, not only from the installer, so [22:46] nessita, wait wait.... i know where is the confusino [22:46] you do? [22:46] I read then [22:47] gatox: where is the confusion? [22:48] the sso controller.... receive a message_box... this message_box is really an instance of wizard, but can also be a QMessageBox.... the critical method in the wizard works as a wrapper for create a QMessageBox and call the critical method (before)... now this method show a form error label... but the critical method, belongs to the message_box object that you give to the sso [22:48] from the installer when you create a controller you establish what this message_box object would be..... in this case is self, because the installer wizard implement "critical" [22:49] The message_box object is anything that has a "critical" method. By default it's a QMessageBox. [22:50] nessita, yes, if you don't pass a references for something at the message_box param... it will create a QMessageBox [22:51] gatox: understood now! [22:51] nessita, nice [22:51] ralsina: you're back! I was writting an email to you [22:51] nessita: I thought I had to run, but it turns out I did not [22:51] so I have a little more time [22:52] therefore, pushed the branch with the test for when there is an exception on account_info() [22:52] the only "new" code is moving those lines inside the try/except so that test should cover them. [22:52] There is no way to stop the timer yet because there is no "next page" method, that's on the next branch [23:10] ok, I gotta make some dinner, I'm starting to starve [23:11] see ya all tomorrow [23:11] bye! [23:11] nessita, bye