[10:07] <karni> Hello ubuntuone!
[10:07] <fagan> Heya karni
[10:54] <facundobatista> Hola karni, fagan
[10:54] <facundobatista> Buen día!
[10:55] <fagan> hey facundobatista
[10:55] <karni> Hello facundobatista , fagan
[10:55] <karni> facundobatista: Tutu bien (I probably failed miserably)
[10:55] <facundobatista> karni, "all" is "todo"
[10:55] <facundobatista> karni, "todo bien" :)
[10:55] <karni> facundobatista: todo bien!! \o/
[10:55] <facundobatista> :D
[10:55] <fagan> todo :)
[10:58] <duanedesign> morning all
[11:11] <facundobatista> Hola duanedesign
[11:17] <ralsina> morning!
[11:17] <fagan> ralsina: hey
[11:27]  * fagan break
[11:28]  * mandel present
[11:42] <ralsina> hi mandel!
[11:46] <mandel> ralsina: buenas!
[11:46] <mandel> ralsina: so, in 5 min we have the encoding fixed :)
[11:46] <mandel> ralsina: I just need to run the tests on linux
[11:47] <ralsina> Cool!
[11:47] <mandel> ralsina: nothing like a day of not thinking about a problem to get it fixed
[11:47] <mandel> ralsina: did you get my email about mumble + standup and report?
[11:48] <ralsina> mandel: got it
[11:49] <mandel> ralsina: does it make sense what I say?
[11:49] <ralsina> yes
[11:49] <ralsina> in your case, the separation between the mumble and EOD is too small
[11:49] <mandel> ralsina: exactly, and standup is more or less the exact same thing...
[11:50] <ralsina> so the three are closer than for us. About mumble, I am trying to make them much shorter (about 15 minutes)
[11:51] <mandel> ralsina: yeah, that is another thing… an hour which is more or less what we have done does not make sense… we end up talking about everything and about nothing
[11:52] <ralsina> yes 1 hour is way too long
[12:02] <duanedesign> anyone speak Portuguese?
[12:03] <adorilson> duanedesign: I'm
[12:03] <adorilson> I means I'm brazilian
[12:07] <duanedesign> adorilson: thank you, I think i got it :)
[12:08] <duanedesign> adorilson: I had an email in Portuguese and was trying to determine if it was spam or not. It was spam :\
[12:11] <adorilson> duanedesign: :-/
[12:25] <mandel> ralsina: killer branch https://code.launchpad.net/~mandel/ubuntuone-client/pyinotify-non-unicode/+merge/67955
[12:25] <duanedesign> rye: do you have a moment to help me with a syncdaemon.log?
[12:25] <rye> duanedesign, sure, i am going through my tickets so need to tune in
[12:26] <mandel> ralsina: it solves lots of bugs because they all had the same root :P
[12:26] <rye> duanedesign, shoot (as nessita says :) )
[12:26] <ralsina> mandel: \o/
[12:27] <duanedesign> rye: https://pastebin.canonical.com/49735/
[12:28] <duanedesign> rye: I had never seen that many failure: Connection to the other side was lost in a non-clean fashion before
[12:28] <mandel> ralsina: we will need to go through the bugs and close them accordingly, I think there are some more closed but I'm not 100% sure
[12:28] <ralsina> mandel: you have a print in there
[12:29] <ralsina> line 19 of the diff (I do that all the time too :-)
[12:30] <rye> duanedesign, i have never seen any such kind of warnings... facundobatista, ping - https://pastebin.canonical.com/49735/ - it shows lots of Unlink warnings with "Connection to the other side was lost in a non-clean fashion before" - how does this happen (i thought if connection fails we just halt the queues and reconnect)
[12:30] <ralsina> joder, me ha quedado el reporte en drafts?
[12:30] <rye> facundobatista, this is 11.04
[12:31] <facundobatista> rye, they're normal if the command fails, tells you why it failed (in this case, a disconnection)
[12:32] <facundobatista> rye, OTOH, Unlinks are normally fast, so you really need to throw a lot of them and disconnect in the exact moment to see a lot of warnings there
[12:32] <mandel> facundobatista: please tell a look at this: https://code.launchpad.net/~mandel/ubuntuone-client/pyinotify-non-unicode/+merge/67955
[12:32] <ralsina> mandel, after you finish linking bugs, maybe I can get a review of https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix-810053/+merge/67894
[12:33] <mandel> facundobatista: fixes the encoding stuff
[12:33] <mandel> ralsina: sure, although I dont know if I'll have time, I'm been here since 8:45 :P
[12:33] <ralsina> mandel: nevermind
[12:33] <rye> facundobatista, aha, so this is the part where the user has asked to disconnect
[12:33] <ralsina> mandel: I can get a review later, it's ok
[12:34] <mandel> ralsina: I'm done with the bugs
[12:34] <ralsina> damn, launchpad logged me out again
[12:34] <mandel> or at least all I know about, so Ill take a look now
[12:35] <adorilson> duanedesign: how U1 cliente uses the apt ?
[12:35] <ralsina> mandel: I am adding test instructions IRL, and since it involves having ussoc import a module from the installer they are necessary,sorry
[12:35] <mandel> ralsina: first question, why does the bin/ubuntuone-installer-qt use the defaul reactor from twisted?
[12:36] <ralsina> mandel: why not? It only needs to send a few messages to ussoc, which does the rest.
[12:36] <ralsina> mandel: it's almost the same as show_gui, but with a custom ui
[12:37] <mandel> ralsina: so, you send the message and how do you show the UI from the isntaller?
[12:37] <mandel> that bin does not need a QApplication?
[12:37] <ralsina> mandel: no, it doesn't
[12:37] <facundobatista> mandel, ok
[12:37] <ralsina> mandel: ussoc imports a custom UI and displays it
[12:38] <ralsina> mandel: and that's the installer. In a way, it's a *huge* code injection :-)
[12:38] <mandel> ralsina: ah, ok, s you provide ussoc the ui class and it takes care of it, smart..
[12:39] <ralsina> mandel: and makes things a lot simpler on the installer's side, even if it feels very weird
[12:39] <fagan> ralsina: do you have any tasks that are intern sized at the moment. I had a look at the bug list this morning for some inspiration but not much jumped out.
[12:39] <ralsina> fagan: yes!
[12:39] <rye> duanedesign, so, the part of the logs we have is only the last one, that is just before disconnection. We don't know why he has disconnected
[12:39] <fagan> ralsina: cool
[12:39] <rye> duanedesign, this is US#3057, right?
[12:39] <ralsina> fagan: I need you to test this on XP: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_809873/+merge/67911
[12:39] <ralsina> fagan: much improved version of the bundle script
[12:39] <mandel> ralsina: makes sense, I think I'm done with the code review, have you written the instructions?
[12:40] <ralsina> fagan: then setupa clean XP (just install it) and try the generated installer
[12:40] <ralsina> mandel: almost there
[12:40] <fagan> ralsina: cool will do
[12:41] <ralsina> mandel: there they are. This one is actually easier to test using a bundle, the imports are a bit dense ;-)
[12:42] <duanedesign> rye: yes
[12:43] <duanedesign> rye: i looked through all of them and does not seem to have been captured (why he has disconnected)
[12:44]  * ralsina is going to be sorry to say this, but that branch has enough tests that nessita can't complain about it
[12:44] <rye> duanedesign, ok, all my logs are being kept for an indefinite period...
[12:44] <mandel> ralsina: ok
[12:44] <nessita> hello everyone!
[12:45] <ralsina> hello nessita!
[12:45] <rye> ralsina, assertEqual(this.__doc__, "This is a function description") ?
[12:45] <duanedesign> adorilson: I think to install Bindwood, the bookmark sync extension
[12:45] <rye> Equals
[12:46] <ralsina> nessita, I had the report written last night andI have proof that it got stuck in drafts: http://www.screencast.com/t/CXZyzZmXa
[12:46] <ralsina> rye: damn, missed that one!
[12:46] <nessita> ralsina: :-)
[12:46] <rye> duanedesign, so, [logging] ... file_size = 0 - that will keep all the files, however what I think is happening is that due to a massive amount of files to sync SD does not keep up with dbus requests, which makes it look like it is not doing anything
[12:47] <ralsina> And nessita: a branch that I am almst sure you will not have issues with: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix-810053/+merge/67894
[12:47] <nessita> ralsina: so, we need to talk a bit more, last night before sleep I spent some time thinking in what we talked about yesterday, and I realized a couple of issues
[12:48] <nessita> mandel: are you staying for the stand up and the daily call?
[12:48] <ralsina> nessita: ok, but I can't do voice calls yet (kid asleep next door)
[12:49] <adorilson> duanedesign: then it runs something seems "apt-get install bindwood" ?
[12:49] <mandel> nessita: yes
[12:50] <ralsina> mandel, alecu,nessita, dobey, fagan: standup in 10'
[12:50] <ralsina> And DiegoSarmentero has finished his contract, but he's invited ;-)
[12:50] <fagan> :)
[12:53] <DiegoSarmentero> ralsina, thanks
[12:55] <alecu> hello!
[12:55] <fagan> hola alecu
[12:56]  * fagan does know some spanish :)
[12:56] <alecu> Dia is Muire dhuit, fagan
[12:57] <fagan> alecu: very good :)
[12:57] <duanedesign> adorilson: i think that is correct. "gksudo apt-get...."
[12:57]  * alecu knows no irish at all :-) http://wikitravel.org/en/Irish_phrasebook
[12:57] <fagan> I learned about the word todo in spanish this morning
[12:58] <fagan> alecu: yeah it actually  means god and mary abe with you
[12:58] <fagan> but thats how we say hello
[12:58] <fagan> or respond to saying hello in that case
[12:59] <fagan> Irish is such an interesting language when you start translating it
[12:59] <alecu> fagan, right :-). And I'm impressed about "Conas atá tú?"
[12:59] <ralsina> fagan: how catholic!
[12:59] <fagan> ralsina: yeah I know
[12:59] <alecu> fagan, it sounds just like in spanish: "Cómo estás tu"
[12:59] <ralsina> here, old coutry people used to say hi saying "good and holy!" so...
[13:00] <fagan> alecu: yeah the language is kinda half english half latin and half some other weird nordic languages
[13:00] <nessita> me
[13:00] <fagan> me
[13:00] <ralsina> me
[13:00] <DiegoSarmentero> me
[13:00] <mandel> me
[13:00] <nessita> alecu?
[13:00] <nessita> dobey?
[13:00] <dobey> me
[13:01] <nessita>  DONE: bug #809363, started with bug #810100 and bug #810440
[13:01] <nessita> TODO: finish those bugs, careful review to mandel's encoding branch, IRL test od that branch
[13:01] <nessita> BLOCKED: I'm having serious connectivity issues, looking for alternatives ATM
[13:01] <nessita> NEXT: fagan
[13:01] <ubot4> Launchpad bug 809363 in ubuntuone-control-panel "FileSyncStatus needs to be updated to the current design (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/809363
[13:01] <alecu> me
[13:01] <fagan> DONE
[13:01] <ubot4> Launchpad bug 810100 in ubuntuone-control-panel "Windows: Folder display name is ugly (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/810100
[13:01] <fagan> * bundled the dist folder with the exes and gave it to ralsina.
[13:01] <fagan> * went looking for some bugs (didnt find any within my range)
[13:01] <fagan> TODO
[13:01] <fagan> * test ralsina's installer branch
[13:01] <ubot4> Launchpad bug 810440 in ubuntuone-control-panel "QT UI: 'explore' button is too wide and looks ugly (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/810440
[13:01] <fagan> BLOCKED
[13:01] <fagan> * no
[13:01] <fagan> ralsina:  go
[13:01] <fagan> whoops that was a little bit early :)
[13:01] <ralsina> DONE: proposed branches for bug #809873, bug #810053, closed bug #810096
[13:01] <ralsina> TODO: signed urls, embed webkit, bug #800376
[13:01] <ralsina> BLOCKED: no
[13:01] <ubot4> Launchpad bug 809873 in ubuntuone-windows-installer "Add a script to create a bundle and installer for all the related software (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/809873
[13:01] <ralsina> next DiegoSarmentero
[13:01] <ubot4> Launchpad bug 810053 in ubuntuone-windows-installer "Credentials obtained by the installer don't work. (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/810053
[13:01] <ubot4> Launchpad bug 810096 in ubuntu-sso-client "The windows port doesn't expose the ui_module parameter of login_or_register_to_get_credentials (affects: 1) (heat: 6)" [Undecided,Fix committed] https://launchpad.net/bugs/810096
[13:01] <ubot4> Launchpad bug 800376 in ubuntuone-windows-installer "Add "syncing your computer to the cloud" page to the installer (affects: 1) (heat: 9)" [Undecided,In progress] https://launchpad.net/bugs/800376
[13:01] <DiegoSarmentero> DONE:
[13:01] <DiegoSarmentero> U1 Windows Installer UI complete (Waiting for review). Tests for U1 installer ui improved.
[13:01] <DiegoSarmentero> TODO:
[13:01] <DiegoSarmentero> Fix issues in UI if necessary.
[13:01] <DiegoSarmentero> BLOCKED:
[13:01] <DiegoSarmentero> No.
[13:01] <DiegoSarmentero> mandel, go
[13:02] <mandel> DONE: Rested, though about encoding and fixed the bloody bug!!! Canonical HR (I forgot to do my objectives, ups!)
[13:02] <mandel> TODO: Look at local_rescan, there are tests failing but are easy to fix, a diff event is raised from the fs on windows.
[13:02] <mandel> BLOCKED: not that much anymore
[13:02] <mandel> dobey, go
[13:02] <dobey> λ DONE: took stock of duplicate assets, discussed bzr locking, more issues in #809561 to fix
[13:02] <dobey> λ TODO: finish bug #809561, start the shim
[13:02] <ubot4> Launchpad bug 809561 in ubuntuone-dev-tools "Support ignoring paths in u1lint (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/809561
[13:02] <dobey> λ BLCK: None.
[13:02] <dobey> alecu
[13:02] <alecu> DONE: landed fix for bug #806539; started working on bug #806655
[13:02] <alecu> TODO: finish branch for second bug
[13:02] <alecu> BLOCKED: no
[13:02] <ubot4> Launchpad bug 806539 in ubuntuone-client (and 1 other project) "Cleanup API to connect to services (affects: 1) (dups: 1) (heat: 12)" [Medium,Fix committed] https://launchpad.net/bugs/806539
[13:02] <ubot4> Launchpad bug 806655 in ubuntuone-control-panel (and 1 other project) "Windows: SyncDaemonTool should provide a way of registering a callback for status changed notifications (affects: 1) (heat: 56)" [High,Triaged] https://launchpad.net/bugs/806655
[13:02] <alecu> EOM?
[13:03] <ralsina> no comments?
[13:03] <fagan> nope
[13:03]  * ralsina waits the regulation 30 seconds
[13:03] <ralsina> EOM
[13:03] <dobey> i've always got comments
[13:03] <dobey> but i also have restraint
[13:03] <ralsina> dobey: no you don't :-)
[13:03] <mandel> alecu, ralsina, nessita I'd like as many revies as possible for https://code.launchpad.net/~mandel/ubuntuone-client/pyinotify-non-unicode/+merge/67955
[13:03]  * mandel lunch
[13:03] <ralsina> mandel: working on it
[13:04] <dobey> ralsina: no comments? or restraint?
[13:04] <ralsina> dobey: restraint?
[13:04] <mandel> thx
[13:04] <nessita> ralsina, alecu, mandel, Chipaca`: I left my home early this morning hunting for internet and I forgot my headset, so, 2 things: 1- I'd need (please) to have the meting thru skype, and 2- I'd need (please) to have it earlier than usual so I can use the laptop's mic and speakers and do not overlap with the meting of my "office mate", if possible
[13:04] <ralsina> dobey: OTOH, if you do, I am more impressed
[13:05] <ralsina> nessita: mandel just left for lunch
[13:05] <nessita> ralsina: oh, I see
[13:05] <nessita> I typed too slow :-)
[13:05] <dobey> i do. :)
[13:05] <nessita> ralsina: ok, I can grab the laptop and walk around when we have the meeting
[13:05] <Chipaca`> nessita: skype, and we call you on your cell?
[13:05] <nessita> so I guess that...
[13:06] <ralsina> Chipaca` nessita: +1 for that
[13:06] <nessita> Chipaca`: sounds expensive! :-) I can walk away from Matias and use the laptop's mic and speakers
[13:06] <Chipaca`> nah, it's not
[13:06] <ralsina> BTW, Chipaca` you are not Chipaca's derivative, your apostrophe is the wrong one!
[13:07] <ralsina> nessita: if there is a fixed phone it works better and is very cheap
[13:07] <nessita> ralsina, Chipaca`: *thanks*. But let's try the cheap option first, if it does not work, let's go with the cell
[13:07] <nessita> there is!
[13:07] <ralsina> Then it's 2 cents a minute :-)
[13:07] <nessita> ok, then... meeting a las y 45 :-)
[13:08] <ralsina> which is why calling land lines using skype from my cellphone is cheaper than calling with my phone.
[13:08] <dobey> i thought skype call-out pricing was based exponentially on time used
[13:08] <nessita> ralsina: let me know when you can talk, and I share what I was thinking last night re sso + wizard
[13:08] <ralsina> nessita: I will close the door andcall you
[13:09] <nessita> ralsina: no rush on my end
[13:09] <ralsina> nessita: and I was so happy I had it working (with tests too!)  ;-)
[13:09] <Chipaca`> ralsina: note skype also has a per-call cost as well as the per-minute cost
[13:09] <nessita> ralsina: is not your code, is the sso design that may misbehave
[13:10] <dobey> price = 0.0001 * minutes^2 + 42
[13:14] <adorilson> duanedesign: ok. We have a yum version from file http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-control-panel/trunk/view/head:/ubuntuone/controlpanel/gui/gtk/package_manager.py
[13:43] <mandel> nessita, ralsina, alecu: mumble?
[13:44] <nessita> mandel: skype this time
[13:44] <nessita> mandel: can you please join skype?
[13:44] <mandel> nessita: ok, on it
[13:46] <joshuahoover> fagan: ping
[13:46] <alecu> nessita, mandel: skype is on.
[13:47] <mandel> mine too
[13:47] <nessita> Chipaca: can you start the conference?
[13:47] <nessita> (please)
[13:47] <Chipaca> nessita: yus
[13:47] <nessita> danke
[13:47] <Chipaca> nessita: PM me your cell please
[13:48] <nessita> Chipaca: try my skype first,
[13:48] <nessita> I'm already in the yard
[13:48] <nessita> (?)
[13:49] <nessita> is sunny today!
[13:49] <dobey> watch out for coronal mass ejections
[13:55] <fagan> joshuahoover: pong
[14:35] <ralsina> Chipaca: just checking, tom means tom haddon?
[14:35]  * ralsina sucks at names
[14:41] <Chipaca> ralsina: yes
[14:41] <Chipaca> ralsina: and yes
[14:45] <dobey> nessita: https://code.launchpad.net/~dobey/ubuntuone-dev-tools/lint-ignores/+merge/67758 does work now, if you can re-review please :)
[14:46] <nessita> dobey: sure!
[14:46] <nessita> ralsina: will you please remind me the 2 branches that need your review?
[14:46] <ralsina> that need my review?
[14:47] <nessita> ralsina: sorry, I meant, that need my review :-D
[14:47] <ralsina> hehe, https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix-810053/+merge/67894 https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_809873/+merge/67911
[14:47] <ralsina> The second one is probably not for you :-)
[14:48] <dobey> las chicharras es fuerte
[14:48] <ralsina> dobey: "son fuertes" if you mean cicadas are buff
[14:49] <ralsina> "están fuertes" if you mean they are loud
[14:49]  * ralsina hopes for the second, but is willing to accept our new cicadan overlords
[14:49] <dobey> heh
[14:51] <nessita> ralsina: "están fuertes" means they are hot! :-)
[14:52] <ralsina> nessita: that too :-)
[14:52] <dobey> ralsina: http://half-life.wikia.com/wiki/Animal_King
[14:53] <ralsina> nessita: I am pretty sure this is a typo, right? http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc.py#L296
[14:53] <nessita> looking
[14:54] <ralsina> Also on line 301
[14:54] <nessita> ralsina: yes sir, typo + lack of tests it seems
[14:55] <ralsina> Both or just the one in 296?
[14:56] <nessita> ralsina: I don't know, mandel can answer, or you can look at the spec of the class, what signals it promises to throw
[14:56] <ralsina> ok, will check
[14:59] <ralsina> I'll assign to Manuel. AFAICS those signal names or the "corrected" ones are never used anywhere in our code (says grep)
[15:16] <dobey> alright. time to get some lunch. bbiab
[15:19] <alecu> nessita, ralsina: there is some styling issue when clicking on buttons in the control panel, because there's no visible change in the button while it's "pressed".
[15:20] <nessita> alecu: yes, I saw that too
[15:20] <nessita> alecu: can you please file a bug?
[15:20] <alecu> nessita, surely!
[15:22] <alecu> nessita, bug #810543
[15:22] <ubot4> Launchpad bug 810543 in ubuntuone-control-panel "Style does not change while buttons are pressed (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/810543
[15:23] <DiegoSarmentero> nessita, hi! did you have the chance to review my branch for u1 windows installer?
[15:23] <alecu> nessita, should I assign it to DiegoSarmentero?
[15:23] <alecu> DiegoSarmentero, you may want to take a look at this bug ^
[15:23] <nessita> DiegoSarmentero: not yet
[15:24] <nessita> alecu: yes please, we'll see if he can fix it if not we'll seek alternatives
[15:25] <nessita> ralsina: question re your fix-810053 branch
[15:25] <ralsina> nessita: sure
[15:26] <DiegoSarmentero> nessita, alecu yes, i'm taking a look at that bug right now
[15:26] <alecu> DiegoSarmentero, thanks!
[15:26] <nessita> ralsina: I see you removed some constant imports but not all, APP_NAME and TC_URL are still there. What are we using those in the wizard?
[15:27] <DiegoSarmentero> alecu, question: is there any image how the button should looks like when it is pressed?
[15:28] <DiegoSarmentero> alecu, just to know if there is any special specification about that
[15:29] <alecu> DiegoSarmentero, I don't know. nessita do you have any idea? ^
[15:29] <DiegoSarmentero> nessita, alecu, or maybe I can use the same as hover... what do you think?
[15:30] <ralsina> nessita: SSO expects the SSOGui.view to have them
[15:30] <alecu> DiegoSarmentero, probably we should darken a bit the buttons, like the default windows 7 theme does... But I don't know for sure, we may ask lisette as well.
[15:30] <alecu> lisette, ping
[15:30] <ralsina> nessita: in other words, if I remove those two, I get an exception about them being missing
[15:31] <DiegoSarmentero> alecu, ok, i'll do it darken a bit the buttons, let me know if this change
[15:31] <alecu> lisette, is there a design for the "pressed" state of buttons in the control panel? This is so the user gets visual feedback while a button is being pressed.
[15:32] <lisette> alecu: no, not one that I did. Are you using Ubuntu buttons?
[15:33] <alecu> lisette, we are using the orange Qt buttons in the windows control panel.
[15:34] <alecu> lisette, and they get a bit lighter orange color when they are hovered.
[15:35] <alecu> lisette, but it would be useful to have them a "different color" or "a different shading" or even "move the text a few pixels down and right" while they are clicked
[15:35] <lisette> alecu: so there is no click status in those buttons?
[15:35] <nessita> alecu: I have no design for that, so asking lisette is the best option :-)
[15:36] <nessita> ralsina: oh! k
[15:36] <nessita> ok
[15:36] <alecu> lisette, sorry, don't understand about "click status". Do you mean "click feedback" ?
[15:36] <alecu> lisette, I was just wondering if there's any design for that, or if we should just "darken" like the default windows 7 theme.
[15:37] <alecu> lisette, let me rephrase...
[15:37] <lisette> alecu: no there is no design for that, because we were going to use buttons exactly as they appear in Ubuntu
[15:37] <alecu> lisette, oh, ok.
[15:37] <lisette> alecu: do you know what the behaviour of buttons in Ubuntu is?
[15:38] <alecu> lisette, I think the current buttons in ubuntu are a very different shade of orange, but let me check.
[15:40] <lisette> alecu: hmm. I assumed we´d use Ubuntu buttons, and this is why I haven´t specced or cropped any button backgrounds. but I can have a look at it now. If so, do you know hex values / have image backgrounds for the hover state?
[15:42] <nessita> lisette: I'm trying to follow the conversation but I'm a bit lost :-) What does it mean that you use Ubuntu buttons if this design is for windows?
[15:42] <DiegoSarmentero> lisette, i don't know if the hex values for the hover state are the correct ones right now, when you hover over a button you can see it brighter
[15:42] <alecu> lisette, I'm looking at ubuntu buttons in natty, and they are usually gray, only "default" buttons appear orange. (default button is the one that's activated when you press <Enter>)
[15:43] <alecu> lisette, also, buttons in ubuntu use dark font, but the current control panel buttons use white font.
[15:43] <alecu> lisette, also, the buttons in ubuntu "move the text to the right and down" when clicked; standard windows 7 buttons "darken the button" when clicked.
[15:44] <lisette> alecu: i will have to discuss this with the team and get back to you
[15:45] <alecu> lisette, sure! for now, I'll ask DiegoSarmentero to just make the button darker when they are clicked, so the user gets some feedback when they are clicked.
[15:45] <alecu> DiegoSarmentero, is that ok?
[15:46] <DiegoSarmentero> alecu, yep!
[15:46] <lisette> alecu, DiegoSarmentero: cool
[15:48]  * DiegoSarmentero is away: Not here... send me an e-mail! :P
[15:48] <ralsina> nessita: correct way to get the credentials in controlpanel is ubuntuone.controlpanel.login_client.get_credentials()?
[15:49] <nessita> ralsina: yessir!
[15:49] <nessita> ralsina: that will not popup anything
[15:49] <ralsina> and that's a deferred
[15:49] <ralsina> nessita: it's ok, don't want it to :-)
[15:49] <nessita> will return {} or the credentials when the deferred is fired
[15:52] <ralsina> nessita: ok, thanks
[15:54] <ralsina> I feel like an ignrant, but why is this returning a deferred instead of a value?http://pastebin.ubuntu.com/644202/
[15:54] <ralsina> alecu, nessita ^
[15:55] <nessita> ralsina: looking
[15:55] <nessita> credentials = yield get_credentials()
[15:55] <nessita> that can block
[15:56] <nessita> get_credentials() returns a deferred, yielding on it make sthe main loop "advance" until the deferred is fired
[15:56] <dobey> ralsina: because the yield + inlineCallbacks makes it a generator that returns a deferred
[15:56] <nessita> ralsina: is that makes you see the light, yey, if not, I can explain further
[15:56] <nessita> is/if
[15:56] <ralsina> nessita: I meant the whole sign_url returns one
[15:57] <dobey> ralsina: yes, because of what i said :)
[15:57] <ralsina> so I suppose I need some remedial deferred teaching again :-(
[15:57] <nessita> ralsina: becasue in its implementation you're yielding over a method that returns a deferred
[15:57] <lisette> alecu, DiegoSarmentero: making the button darker (instead of moving it) is cool with all of us here; I will have a look at hex values for over and hit.
[15:57] <ralsina> nessita: ok, makes sense
[15:57] <dobey> ralsina: using yield inside a function makes that function a generator
[15:57] <nessita> ralsina: and you're decorating the method with inlineCallbacks, that, like dobey says, makes the method return a deferred
[15:58] <dobey> inlineCallbacks loops through the list, and returns the deferreds
[15:58] <alecu> lisette, great, thanks!
[15:58] <ralsina> nessita, dobey: ok, I think I got it now, thanks for your patience
[16:00] <dobey> ralsina: no problem. it was just as confusing as when i had to have tcole explain it to me in regards to the test cases :)
[16:01] <ralsina> I don't think I get how to actually *do* what I want but I'll probably figure it out ;-)
[16:01] <alecu> ralsina, just call the method, it will return a deferred.
[16:01] <ralsina> alecu: yes, and then?
[16:01] <alecu> ralsina, then .addCallback() and .addErrback() with two functions that you want to be called.
[16:01] <ralsina> I need to check the return value, so how do I wait on it?
[16:01] <dobey> ralsina: i think you need to add a callback
[16:02] <ralsina> ok, yikes, complicated tests
[16:02] <dobey> yes
[16:02] <ralsina> callbacks was what I was avoiding :-)
[16:02] <ralsina> but it's ok, let's do that
[16:03] <alecu> ralsina, you won't be able to avoid callbacks as long as get_credentials is async...
[16:03] <alecu> ralsina, there are no other bits in that function that are async
[16:03] <nessita> che ralsina, can you please remove "print "creds found", args" from the 'found' callback?
[16:03] <ralsina> nessita: sure
[16:04] <ralsina> nessita: damn, I had hopes for that branch to go without needsfixingseseses
[16:04] <ralsina> ;)
[16:04] <alecu> ralsina, so you may get away with "passing" the credentials to that function, and avoiding the deferreds in the signing function.
[16:04] <nessita> ralsina: regarding tests and deferreds, they are amazing once you understand them. You can try proposing some code and we can give you the "enhanced" version of it, and that way I'm sure you will get an idea
[16:05] <ralsina> nessita: ok, will have that in a few minutes
[16:05] <nessita> ralsina: or what alecu says, too, but I'm not sure you have the credentials before calling that function
[16:05] <ralsina> no, I don't
[16:05] <ralsina> So callbacks
[16:06] <alecu> ralsina, well, separating the "getting the credentials" from the "signing stuff" makes for simpler tests anyway,.
[16:06] <ralsina> alecu: yes, but then everywhere I call this I have to get the credentials first. It's easier to test but harder to use
[16:06] <alecu> ralsina, and +1 to nessita's "amazing" comment. Make sure to look at some other tests that are testing similar code.
[16:07] <ralsina> alecu: any suggestions?
[16:08] <alecu> ralsina, any function in sso that has at least a yield on it :-)
[16:08] <ralsina> alecu: ok
[16:09] <nessita> ralsina: or, look at pretty much any test in test_backend.py in the controlpanel
[16:09] <mandel> ralsina: ping
[16:09] <ralsina> mandel: pong
[16:09] <ralsina> nessita, alecu: cool, thx
[16:09] <nessita> mandel: hey there!
[16:09] <mandel> ralsina: so when is the sprint?
[16:09] <mandel> nessita: hola :)
[16:10] <mandel> nessita: I'm staring the split fo rthe branch right now, I found a internetcafe :)
[16:10] <nessita> mandel: while you work on splitting the big encoding branch, please check out some comments that facundo added since he already made a eye-code-review over that branch
[16:10] <ralsina> mandel: not next week
[16:10] <nessita> mandel: so you can apply the fixes while splitting, and save some roundtrip time :-)
[16:10] <mandel> nessita: cool :)
[16:11] <alecu> nessita, do we plan to make the "username" field editable in the control panel?
[16:11] <alecu> nessita, do we even have a webservice call for that?
[16:11] <mandel> ralsina: can you tell me when asap, flight tickets are a hard thing to get
[16:11] <nessita> alecu: nopes, we can't edit user info
[16:11] <ralsina> mandel: should know later today, early tomorrow
[16:12] <nessita> alecu: we should make those fields not to be text entries, actually, I'm waiting for some feedback from blajk
[16:12] <nessita> alecu: and we may even remove that tab altogether
[16:12] <alecu> nessita, cool.
[16:13] <nessita> ralsina: what did you agree with clarita re: web page for services tab?
[16:13] <nessita> (if any agreement happened)
[16:13] <dobey> hrmm
[16:13] <dobey> is there some way we can avoid having binaries stored in bzr?
[16:13] <mandel> ralsina: ok, if you send me a tweet I'l start moving
[16:13] <ralsina> nessita: the agreement was that it will happen, and she's not thrilled by it, but it's what we can do
[16:13] <alecu> nessita, your latest mail seems to suggest otherwise: "Yes in the sense that we have no plans to change that."
[16:13] <ralsina> mandel: moving to argentina? ;-)
[16:13] <alecu> nessita, I read that we'll keep the text entries
[16:14] <mandel> ralsina: mocing to get a flight ticket, the other thing will take longer :P
[16:14] <nessita> alecu: hem... I meant we have no plans to change it as in we have no bug nor task for that
[16:14] <ralsina> mandel: you are a mommy's boy ;-)
[16:14] <nessita> alecu: I can see how that can be read differently
[16:14] <mandel> ralsina: she is a spaniard, remember hehe
[16:14] <alecu> chu!
[16:15] <nessita> alecu: I will clarify, thanks for pointing that out
[16:15] <alecu> nessita, no problem
[16:17] <blajk> nessita, alecu, I write you an email right now!
[16:18] <alecu> blajk, cool, thanks!
[16:18]  * alecu goes to have lunch, and then to the bank.
[16:18] <nessita> blajk: oh, I just send a carification :-)
[16:18] <nessita> clarification*
[16:22] <dobey> ralsina: is there no good way to do PNG generation from a big SVG, on windows, for icons/etc?
[16:23] <ralsina> dobey: well, install inkscape for windows and write a .bat I suppose
[16:23] <dobey> i am not porting icontool to .bat :)
[16:24] <dobey> how are we doing icons on windows anyway? just loading specific ones off disk from a specific location?
[16:25] <nessita> ralsina: another request: can you please replace self.assertEqual(isinstance(self.ui.view, FakeMainWindow), True) by
[16:26] <nessita> self.assertIIinstance(self.ui.view, FakeMainWindow)
[16:26] <nessita> oops
[16:26] <nessita> self.assertIsIinstance(self.ui.view, FakeMainWindow)
[16:26] <ralsina> nessita: sure, didn't know that one
[16:27] <nessita> ralsina: :-)
[16:27] <facundobatista> ralsina, is new for me too
[16:28] <nessita> facundobatista: because of my bad typing? :-)
[16:28] <dobey> ralsina: also, assertTrue is better than assertEqual(foo, True) :)
[16:28] <nessita> ralsina: jic, correct method name is self.assertIsInstance(self.ui.view, FakeMainWindow)
[16:28] <nessita> ralsina: and what dobey says as well
[16:28] <facundobatista> nessita, no, I mean, the real name
[16:28] <ralsina> dobey: I prefer small APIs instead of a new methd for every kind of comparison, but that's just personal taste
[16:29] <dobey> ralsina: "assert foo" :)
[16:29] <nessita> ralsina: that's not-pythonic. If you need to choose between 2 APIs:
[16:29] <nessita> set_foo_enabled(enabled)
[16:29] <nessita> or
[16:29] <nessita> enable_foo()
[16:29] <nessita> disable_foo()
[16:30] <nessita> you should alays go with the second
[16:30] <dobey> neither of those are pythonic
[16:30] <nessita> even if for the implementation you use an internal _set_foo
[16:30] <dobey> @property ftw
[16:30] <ralsina> nessita: in that case there are only 2 methods, there are an infinity of possible assert* methods :-)
[16:30] <nessita> dobey: it was an example
[16:30] <nessita> dobey: and yes, @property FTW
[16:30] <dobey> just saying :)
[16:30] <dobey> @property makes it a single API point, that abstracts those two away :)
[16:31] <dobey> ralsina: well, it's better than writing a lambda every time, to get the result down to true or false to do an assert on :)
[16:32] <dobey> duplicate code FTL
[16:32] <ralsina> nessita: there are some huge test failures in that branch so hold it a bit (I swear I tested this last night :-( )
[16:32] <nessita> ralsina: np
[16:33] <ralsina> nessita: ok, and lunchtime, too
[16:33] <nessita> ralsina: and lunchtime here too
[16:34] <ralsina> So I'll ping you when it's ready
[16:34] <nessita> ralsina: please
[16:34] <nessita> though I have tons to do, so no rush :-P
[16:34] <ralsina> Oh, cool, deleting oneline fies 24 tests ;-)
[16:56] <dobey> nessita: did you get to look at my branch again? :)
[16:59] <dobey> hmm, i should have taken today as holiday too. the weather is too nice outside
[17:03]  * DiegoSarmentero is back (gone 01:14:43)
[17:11] <nessita> dobey: not yet, but is in the queue! :-) popping it up right now
[17:13] <dobey> :)
[17:17] <nessita> dobey: is not working that well, I ran ./run-tests and the u1lint was printed instead of the lint report, see
[17:17] <nessita> http://pastebin.ubuntu.com/644253/
[17:18] <nessita> dobey: and, u1lint -i bin/ gives me lint issues in bin/
[17:19] <dobey> eh
[17:19] <dobey> weird
[17:20] <dobey> nessita: did you run u1lint -i bin or bin/u1lint -i bin?
[17:20] <nessita> dobey: DOH :-)
[17:24] <dobey> oh i see what the issue is with the pylint output
[17:24] <dobey> i am an idiot again :)
[17:26] <dobey> nessita: fixed and pushed
[17:27] <dobey> nessita: wb :)
[17:28] <dobey> nessita: fixed and pushed the pylint output issue, in case you didn't see
[17:29] <nessita> dobey: I didn't, the desktop froze. I tried unity reset in the text terminal with no luck
[17:30] <dobey> nessita: no worries. :)
[17:30] <nessita> any idea why the test output is shown twice?
[17:31] <nessita> (not related with u1lint I think)
[17:33] <nessita> dobey: ^
[17:35] <nessita> dobey: the rest looks good, and works fine now
[17:35] <nessita> so I'll approve
[17:36] <dobey> nessita: run-tests is running u1trial twice, once with glib reactor and once with twisted
[17:37] <nessita> ah!!!
[17:37] <nessita> perfect
[17:37] <dobey> :)
[17:45] <nessita> ralsina, DiegoSarmentero (only if you have spare minutes, no worries otherwise since you're of the hook now): can you please review https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/fix-810440/+merge/67998 ?
[17:45] <nessita> off*
[17:46] <DiegoSarmentero> nessita, yes, no problem!
[17:46] <nessita> DiegoSarmentero: there is a screenshot attached in case you can't branch that
[17:47] <DiegoSarmentero> yes, i'm watching it
[17:48] <nessita> DiegoSarmentero: any hints about how can I remove the frame borders there? (honestly I made no googling yet, so feel free to send me to do that ;-))
[17:50] <DiegoSarmentero> nessita, mmmm it seems to be taking the border from the header... did you try to group those frames in the qss and set the border: none or border-width: 0px; ?
[17:51] <nessita> DiegoSarmentero: that screenshot is using the same qss as the windows one, are you doing that there? :-)
[17:51] <nessita> if not, I will try that
[17:51] <DiegoSarmentero> nessita, nop... it was just an idea.... if you want i can give it a try here
[17:52] <nessita> DiegoSarmentero: nono, I can try it
[17:52] <nessita> DiegoSarmentero: I prefer to squeeze your talent when absolutely needed
[17:52] <DiegoSarmentero> nessita, jeje ok ok
[17:59] <nessita> ralsina, alecu: I just realized we haven't implemented the 'delete this device' functionality, yet!!! I will add that to my queue
[18:00] <DiegoSarmentero> nessita, about your review
[18:00] <DiegoSarmentero> nessita, about the review of your branch..... i mean
[18:00] <nessita> DiegoSarmentero: tell me
[18:00] <nessita> yeah
[18:00] <alecu> nessita, ack with the "delete this device", but priority lower than the rest :-)
[18:01] <nessita> alecu: "rest"?
[18:01] <alecu> nessita, the rest of stuff we are working on! :-)
[18:01] <DiegoSarmentero> nessita, it looks fine by me.... there is just one line that you can delete if you want, because it is unnecessary, but it doesn't affect anyway.... line 9 in the diff: <string notr="true"/>
[18:01] <DiegoSarmentero> nessita, +1
[18:02] <nessita> alecu: ah!!! I thought we were on vacations already! :-P
[18:02] <alecu> juas!
[18:02]  * alecu +1s vacations
[18:03] <nessita> DiegoSarmentero: removing that now...
[18:03] <DiegoSarmentero> nessita, in fact you can delete the property for "styleSheet" really.... line 7 to 10 in diff
[18:03] <nessita> deleting...
[18:03] <DiegoSarmentero> :D
[18:03]  * nessita obeys
[18:03] <DiegoSarmentero> nessita, jejee
[18:03] <alecu> nessita, cparrino owes us vacations on the caribean beaches after this.
[18:03] <nessita> (from time to time, do not get used to that :-P)
[18:03] <nessita> alecu: *at least*
[18:03] <DiegoSarmentero> nessita, no no.... i wasn't expecting that
[18:03] <DiegoSarmentero> jeje
[18:04] <ralsina> I'm back sorry about the long lunch!
[18:05] <nessita> ralsina: relatives finally arrived? :-)
[18:06] <ralsina> nessita: yeah
[18:07] <alecu> ralsina, http://bit.ly/longest-lunch
[18:07] <nessita> ralsina: for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_809873/+merge/67911, is there any chance that you grab the logging.conf and the syncdaemon.conf from launchpad or a repo? I think is not good to have that duplicated in that project, since those files can be easily changed in u1client trunk
[18:08] <ralsina> nessita: I can do that for syncdaemon.conf but logging.conf requires variables
[18:08] <nessita> ralsina: hum... the loggin level, right?
[18:08] <nessita> logging*
[18:08] <ralsina> yes
[18:09] <nessita> ralsina: can't we replace the vriable for the logging level we want for the installer? :-)
[18:09] <ralsina> I could generate it with some obvious code, of course :-)
[18:09] <nessita> so, when releasing betas, we set DEBUG, for finals, INFO
[18:09] <nessita> yeah, just a silly 'something'.replace()
[18:09] <ralsina> nessita: pushed test fixes on  the bug #810053 branch
[18:09] <ubot4> Launchpad bug 810053 in ubuntuone-windows-installer "Credentials obtained by the installer don't work. (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/810053
[18:09] <ralsina> nessita: ok, will do it now
[18:11] <ralsina> nessita, alecu: did any of you try the encodings branch to see if it fixes UDFs / shares?
[18:11] <alecu> ralsina, not me
[18:12] <nessita> ralsina: no, is in my queue
[18:12] <nessita> ralsina: I'm waiting for people using this net conn to release it
[18:12] <nessita> ralsina: should happen soonish (in the next hour)
[18:13] <ralsina> nessita: ok, will wait for it then
[18:14] <nessita> DiegoSarmentero: FYI, all the empty style tags were removed
[18:15] <DiegoSarmentero> nessita, nice!! i probably insert some too... when i was trying the styles in the designer... now i double check even the xml :P
[18:15] <nessita> DiegoSarmentero: ;-)
[18:16] <nessita> ralsina: did you ever confirmed with design that it was ok to replace frames 18 and 19 with current "preferences" tab from control panel?
[18:16] <ralsina> nessita: let me check
[18:17] <ralsina> nessita: argh, no I haven't, se me traspapeló
[18:17] <ralsina> nessita: will do it by email
[18:17] <dobey> ralsina: can you tell me about how our bundle things work on windows?
[18:17] <ralsina> dobey: sure
[18:18] <nessita> ralsina: ok, I'm just bringing that to your plate again. Mail sounds good.
[18:18] <dobey> ralsina: do we build a separate bundle for each project? (sso, cp, client, etc)
[18:18] <ralsina> dobey: no, that's extremely inefficient
[18:19] <ralsina> dobey: basically if we do that, each bundle will contain lots of duplication (for example, control-panel will have most of syncdaemon and sso in it)
[18:19] <dobey> ralsina: oh, we can't make a bundle of just cp without its deps? the bundle has to have everything in it?
[18:20] <ralsina> dobey: a bundle with everything is about 30% of the size of bundles for each
[18:20] <ralsina> also, because of ussoc's design, it needs to see parts of the installer (to make the "replacement ui module" work, so that's not even doable with separate bundles
[18:20] <dobey> that didn't quite answer those two questions :)
[18:20] <dobey> hrmm
[18:20] <dobey> ok
[18:21] <dobey> ralsina: do we have to pull everything from bzr and build it from branch to bundle it?
[18:21] <ralsina> dobey: it's how I did it, to make it reproduceable
[18:21]  * ralsina knows that's probably not a word
[18:22] <ralsina> dobey: but if you have trunk branched (or any branch) you just put it in sources/ and it doesn't download anything
[18:22] <nessita> DiegoSarmentero: did you +1 my branch? (I just reloaded and LP seems not to know that if you did)
[18:22] <DiegoSarmentero> nessita, ahhhhh i'll do it in the page now :P
[18:23] <nessita> :-)
[18:23] <dobey> ralsina: i'm curious if we can make it pull a tarball/zip file as a means of acquiring source for a couple things, vs pulling from branches
[18:23] <ralsina> dobey: sure, it
[18:23] <nessita> ralsina: did you got my review request while you were having lunch?
[18:24] <ralsina> dobey: sure, it's a python script, it can do anything we want ;-)
[18:24] <dobey> ok
[18:24] <ralsina> nessita: nope, will get to it in ... 10 minutes
[18:24] <DiegoSarmentero> nessita, ready!
[18:24] <DiegoSarmentero> nessita, please later if you have a minute review my branch :P
[18:25] <nessita> ralsina: thanks! next to that I have this trivial one: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/add-asset/+merge/68002
[18:25] <nessita> DiegoSarmentero: yes sir, is in the queue
[18:25]  * ralsina goes for the trivial
[18:25] <dobey> ralsina: that makes it a bit easier to deal with the icon generation issue, i think
[18:25]  * nessita realizes that suddenly "The queue" is starting to sound mysterious
[18:25] <ralsina> nessita: +1 on the trivial one
[18:25] <DiegoSarmentero> nessita, thanks! :D
[18:26] <ralsina> dobey: ok, we'll talk about details after it's landed, good for you?
[18:27] <dobey> ralsina: sure i guess. i'm just asking about this right now, as i'm drawing up a bit of a minor re-org plan for some of our code, to make things easier for the cross-platform case, as well as the shim case on ubuntu
[18:28] <ralsina> dobey: sounds useful
[18:28] <ralsina> dobey: basically, there is a "FetchCode.run" function that has to produce a tree of code. How we do that is irrelevant.
[18:29] <dobey> ralsina: yeah, i'm not worried so much about that implmeentation at the moment. just the possibility, and what we're currently doing :)
[18:29] <dobey> since i have no idea how the windows port bits work really :)
[18:29] <ralsina> dobey: I think the bundling thing is pretty flexible so it's probably not the part that's going to cause problems :-)
[18:31] <dobey> ralsina: sure, but it is relevant to my current thinking :)
[18:39] <ralsina> nessita: pushed suggested changes in the branch for bug #809873
[18:39] <ubot4> Launchpad bug 809873 in ubuntuone-windows-installer "Add a script to create a bundle and installer for all the related software (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/809873
[18:39] <alecu> nessita, ping
[18:40] <alecu> nessita, I remember you were discussing some things with mandel regarding the "start" method that was the last to be done, or something like that.
[18:40] <mandel> alecu: in sdtool?
[18:41] <alecu> mandel, around!
[18:41] <alecu> mandel, yes! is that this? http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/linux/dbus_interface.py#L1461
[18:42] <mandel> alw
[18:42] <nessita> alecu: yes, it was done as far as I know
[18:42] <mandel> alecu: yes, start is there…
[18:43] <nessita> alecu: the "start" on windows was the one needed
[18:43] <nessita> alecu: I mean, the one that mandel coded
[18:43] <alecu> mandel, the thing is that I'm hitting a bug when "disconnect" is being called from control panel
[18:43] <alecu> mandel, and I started tracing all the way down to sd, and that bit looks weird, because that push in disconnect I expected it to be in "interaction interfaces"
[18:44] <alecu> mandel, is that right? or am I missing something?
[18:45] <alecu> mandel, hmmm..... not sure. the thing is, that there's a disconnect both in class SyncDaemon (in windows/ipc.py) and both in class IPCInterface
[18:46] <alecu> and the one that's getting called (and fails) is the one in class SyncDaemon.
[18:47] <mandel> alecu: there are 2 disconnect, the actual method and the interface...
[18:47] <mandel> alecu: can you paste the error?
[18:48] <alecu> mandel, http://pastebin.ubuntu.com/644295/
[18:49] <ralsina> alecu: the TIMEOUT_INTERVAL in SSO is used by both windows and linux?
[18:49] <mandel> alecu: one sec, I think I know the issue
[18:49] <alecu> mandel, cool
[18:49] <alecu> ralsina, don't know. checking
[18:49] <mandel> alecu: I'm looking for the code right now
[18:50] <alecu> ralsina, it looks like it's used on both.
[18:50] <mandel> alecu: you want to call this: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc_client.py#L264
[18:51] <mandel> alecu: which is called from this remote object http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc_client.py#L701
[18:51] <ralsina> So ... ugh. We do it for both, or I put that back to 10000 and override on the windows side, if at all possible, or put a platform if there
[18:51] <mandel> alecu: so you are sking the root object http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc.py#L944 to execute a method that he does not have
[18:52] <nessita> ralsina: is not the same, I think...
[18:52] <mandel> alecu: root provides you access to all the different exposed interfaces, being the syncdaemon one of them, so you either request root the correct remote or you use the wrapper I wrote, one of those
[18:52] <nessita> ralsina: oh, it's
[18:52] <alecu> mandel, I think on the control panel side the right method is being called. let me check.
[18:52] <mandel> alecu: let me know if I make sense before I go out
[18:52] <ralsina> nessita: yep
[18:52] <nessita> ralsina: well, you can duplicate that in the main/windows.py file
[18:53] <nessita> ralsina: instead of importing and re-using
[18:53] <ralsina> nessita: right
[18:53] <ralsina> and I put this back to 10000 msec
[18:53] <alecu> mandel, I think the right method on syncdaemon is being called.
[18:54] <mandel> alecu: then the issue is here: SyncdaemonService
[18:54] <nessita> mandel: how's branch splitting going? (I'm asking since you mention you may be going out)
[18:54] <nessita> mandel: how's branch splitting going? (I'm asking since you mention you may be going out)
[18:54] <mandel> alecu: this: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/syncdaemon/interaction_interfaces.py#L702
[18:55] <ralsina> nessita: not even, it's only used in the __init__.py not on the platform-specific ones.
[18:55] <mandel> alecu: looks like he is not getting the correct service object, and instead of getting the actual remote on is getting an IPCRoot, you should check in wither main of platform IPC to see how that class is instantiated
[18:55] <nessita> ralsina: ooooohhhhhuuuuuccchhh
[18:55] <ralsina> nessita: I will set it on the platform modules andimport it in __init__
[18:55] <mandel> nessita: most of it done, I had to think a little in the way I wanted to split it, should be ready for tom morning
[18:56] <nessita> mandel: AR morning or your morning?
[18:56] <mandel> nessita: my morning, as in 10/11 am, so ready for review in the AR morning
[18:57] <nessita> mandel: great!
[18:57] <alecu> mandel, it's created here, right? http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc.py#L348
[18:58] <alecu> mandel, and called from here: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc.py#L979
[18:59] <mandel> alecu: indeed, so the instantiation of the class is wrong and is getting IPCRoot in the constructor instead of the correct class to relay the calls too
[18:59] <mandel> alecu: looks like a trivial bug to fix
[18:59] <mandel> the relay of the methods should work when the correct instance is passed
[19:00] <mandel> alecu: does it make sense?
[19:00] <alecu> mandel, not entirely. Probably in your head does, but I can't keep all the layers in my head yet.
[19:00] <alecu> :-)
[19:00] <ralsina> nessita: +1 on https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/fix-810440/+merge/67998
[19:01] <mandel> alecu: indeed that part is a little ugly hehe to many layers in some points...
[19:01] <nessita> ralsina: ack, I'm not bringing up the VM from cheepaca, I got some better net conn now :-)
[19:01] <ralsina> not or now?
[19:01] <alecu> mandel, you say "the instantation of the class is wrong" <-  which one of the classes is instantiated wrong?
[19:01] <nessita> ralsina: now!~:-)
[19:01] <alecu> how should we instantiate it?
[19:01] <ralsina> nessita: is that your head-is-smoking emoticon? Nice!
[19:02] <nessita> ralsina: see? I'm unstoppable today (?)
[19:02] <alecu> mandel, if you can post of a diff of the fix, and I'll fix and do the tests.
[19:02] <ralsina> nessita, alecu: trivial sso timeout branch, https://code.launchpad.net/~ralsina/ubuntu-sso-client/live-forever/+merge/68010
[19:03] <dobey> hmm, i should get a snack
[19:04] <mandel> alecu: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/syncdaemon/interaction_interfaces.py#L702 should create a SyncDaemon remote object that simply provides the communication layer for SyncdaemonService which is the guy who knows how to disconnect etc...
[19:05] <mandel> alecu: on the linux side we have a similar object that SyncdaemonService (I think is the same name, I cannot remember) that does the same thing with dbus
[19:05] <mandel> alecu: so those objects are the diff platform communication layers whose only concert is to expose the method from the SyncdaemonService which is the class that knows how to do the job
[19:06] <mandel> alecu: in this case it seems that the class calls another remove object (kinda ugly if you ask me) to disconnect
[19:06] <alecu> mandel, "remove object" or "remote object"?
[19:06] <nessita> mandel: by SyncdaemonService you mean SyncDaemonTool? (I'm trying to understand)
[19:06] <mandel> alecu: typo, is a remoTe object
[19:07] <mandel> nessita: no, SdTool is a convinient class that puts together some logic plus dbus/spread calls
[19:07] <nessita> ah
[19:07]  * nessita browses code then
[19:07] <mandel> nessita: the ones we are talking about are the interaction interfaces and their wrappers
[19:08] <nessita> ah
[19:08] <mandel> alecu, nessita: super quick mumble and I explin it fast so that you can work
[19:08] <mandel> I have 15/20 min
[19:08] <alecu> mandel, please, lets.
[19:08] <mandel> nessita: or skype :)
[19:08] <nessita> mandel: skype! ;-)
[19:08] <mandel> ok, I'll cll you both
[19:11] <mandel> alecu, nessita: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/syncdaemon/interaction_interfaces.py
[19:17] <mandel> alecu, nessita: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/linux/dbus_interface.py
[19:18] <mandel> alecu, nessita: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/linux/dbus_interface.py#L434
[19:30] <mandel> alecu, nessita: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/syncdaemon/main.py#L129
[19:37] <alecu> mandel, nessita: http://pastebin.ubuntu.com/644295/
[19:37] <mandel> nessita: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc.py#L944
[19:43] <ralsina> nessita, when you have a minute: https://code.launchpad.net/~ralsina/ubuntuone-control-panel/fix-806654/+merge/68014  [URL signing, I want to explain why I am not mocking oauth on the tests, though]
[19:46] <dobey> ralsina: why would one mock oauth?
[19:46] <ralsina> dobey: it's URL signing, so it's time dependent
[19:46] <ralsina> dobey: so, if you want a repeatable output of oauth'ssigning, you have to mock it
[19:47] <ralsina> Instead, I only test the invariant parts of the URL, that way I also test we are calling oauth correctly
[19:47] <dobey> ralsina: even without the time, nonce is supposed to be unique every time :)
[19:47] <ralsina> Plus we will know if oauth changes something on tests instead of on deployment ;-)
[19:47] <dobey> i don't think we should mock oauth, since it doesn't do any network connection itself
[19:48] <ralsina> dobey: since you know oauth,could you review that branch?
[19:48] <ralsina> it's pretty trivial, but I don't know all that much about it
[19:49] <dobey> ralsina: i don't know why you didn't use throwaway generic readable strings for the token in the test though
[19:49] <dobey> like 'key' and 'secret' instead of 'XfdaXfdsafdkL' stuff :)
[19:49] <ralsina> dobey: because we already use that set of credentials in a few other tests
[19:49] <ralsina> So I am just copying myself ;-)
[19:50] <dobey> ugh, copy/pasted junk is even worse. especially if they're real tokens, which they look like they are :)
[19:50]  * ralsina took them from SSO ;-)
[19:50] <ralsina> but it's ok, I can change that
[19:51] <dobey> it's just weird and i think it makes it a bit harder to read :)
[19:51] <ralsina> dobey: tokens have to be a fixed length or are just arbitrary?
[19:52] <dobey> ralsina: arbitrary, i usually use 'consumer_key' and 'consumer_secret' and such for tests
[19:52] <dobey> as long as they're ASCII, it's fine
[19:53] <dobey> ralsina: did you run pep8 on this?
[19:53] <ralsina> dobey: yep
[19:53] <ralsina> dobey: if I put a readablestring on token or token_secret:
[19:53] <ralsina> exceptions.TypeError: character mapping must return integer, None or unicode
[19:54] <dobey> ralsina: and it didn't complain about lack of space between string and \?
[19:54] <ralsina> dobey: no, but may be because I have a newer pep8
[19:54] <ralsina> besides, it's a continuation line, you can't *ask* for spaces before it
[19:54] <dobey> ralsina: nah, it probably doesn't complain. i don't know if it's pep8 or not. it's just ugly
[19:55] <dobey> continuation characters suck :)
[19:55] <dobey> use parens instead :P
[19:57] <ralsina> dobey: personal taste dude.
[19:57]  * dobey writes a PEP for it
[19:57] <ralsina> PEP-dobey: "it should look like C" :-)
[19:58] <dobey> no
[19:58] <alecu> nessita, mandel: before finding this, my head was hurting. Now it's pul-sa-ting!
[19:58] <ralsina> dobey: there, pushed with spaces and readable keys where those don't break it
[19:58]  * ralsina is suddenly scared of what nessita alecu and mandel have been talking about on skype
[19:58] <nessita> ralsina: you need to put an space before \! ;-)
[19:59] <dobey> haha
[19:59] <nessita> so, "a string" \
[19:59] <nessita> "long"
[19:59] <ralsina> nessita: pep8 didn't complain, but I already pushed that anyway just because I am nice ;-)
[19:59] <dobey> ralsina: see, nessita and i agree on it!
[19:59] <ralsina> oh, get a room you too. Nazis.
[19:59] <ralsina> ;)
[20:00] <nessita> ralsina: we were debugging an issue that alecu was having, and we saw some code is not trivial to see :-)
[20:00] <dobey> o/~ deutscheland deutscheland uber alles! o/~
[20:00] <nessita> that said, I can die in piece
[20:00] <nessita> peace also
[20:00] <ralsina> Yes, we all can die in pieces. :-)
[20:00] <nessita> my brain is shortcutting :-D
[20:00] <alecu> ralsina, it's not called a room. It's called a "bunker"
[20:00] <dobey> haha
[20:01] <dobey> alecu: nice
[20:01] <dobey> ralsina: heh, you didn't change the token/token_secret too, only the consumer :P
[20:01] <nessita> ralsina: I have 4 tabs opened with reviews for you. Which one I should do first? :-D
[20:01] <ralsina> dobey: see above. If I change the tokens,oauth complains
[20:01] <ralsina> nessita: random is ok
[20:02] <dobey> it does?
[20:02] <nessita> ralsina: which one is easier?
[20:02] <ralsina> the sso timeout one
[20:02] <dobey> ralsina: what version of python-oauth is complaining, btw?
[20:02] <dobey> (just wonder if that's a new thing)
[20:03] <ralsina> dobey:  1.0.1
[20:03] <dobey> must be a new thing then :(
[20:03] <ralsina> it makes sense, since the tokens are not just random strings
[20:04] <dobey> the tokens are just random strings
[20:04] <dobey> but it's also python-oauth :(
[20:05] <dobey> ralsina: anyway, looks ok to me
[20:05] <ralsina> dobey: cool, thx
[20:10] <nessita> ralsina: I think that the signing url branch should not be attached to bug #806654, right?
[20:10] <ubot4> Launchpad bug 806654 in ubuntuone-control-panel "QT UI: replace current Services tab with an embedded browser (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/806654
[20:10] <ralsina> nessita: it was sort of necessary for that bug, since we needed to display the signed URL
[20:10] <ralsina> it doesn't *fix* it though
[20:10] <nessita> ralsina: also, that same bug should be closed as won't fix after what we talked with design by email
[20:11] <nessita> ralsina: so, I think we should have a bug to only solve the url signing thingy
[20:11] <nessita> and close the embed a browser
[20:11] <ralsina> I can just change the description to "create a function to yadda yadda
[20:11] <nessita> ralsina: hum, I would like to keep track of the history of the bug, we may re-open in the future
[20:12] <ralsina> ok, will add another one and reattach
[20:12] <nessita> we usually edit the title for typos or better descriptions, not for completely change the goal of the bug :-P
[20:14] <nessita> ralsina: approved timour thingy, going for oauth signing
[20:16] <nessita> ralsina: question. Why do you sign the uri = 'https://one.ubuntu.com/api/1.0/from_oauth' instead of the url itself, and pass a 'next': url param to the oauth request?
[20:17] <ralsina> nessita:IIRC the from_oath page sets the cookie so the other pages see you are logged in
[20:17] <ralsina> and then you are redirected
[20:18] <nessita> ralsina: but... I thought we were signing each page that needed to be sign with credentials...
[20:18] <nessita> I don't understand the cookie thing (in the sense why we need to set a cookie)
[20:18] <ralsina> nessita: if we do that, when you click on a link, you are going to be not-siged-in
[20:18] <ralsina> because the links on the page point to the not-signed pages
[20:19] <nessita> ah... I see
[20:20] <nessita> ralsina: fail cut & paste! +# Authors: Roberto Alsina <alecu@canonical.com>
[20:20] <nessita> (been there, done that)
[20:20] <ralsina> nessita: hahaha, I'm modest
[20:21] <nessita> also, this is other fail! +"""Tests for the humanize function."""
[20:21] <nessita> :-D
[20:22] <ralsina> there, pushed both. And you uncovered my cunning plan to have alecu answer questions about my code!
[20:22] <alecu> hahahah
[20:25] <nessita> ralsina: hum that method will no work if the url has a query string
[20:25] <ralsina> nessita: it should!
[20:25] <nessita> ralsina: if the given url has a query string, that does not work :-)
[20:25] <ralsina> remember it's put on the "next" argument, url-encoded
[20:26]  * ralsina has not checked, of course ;-)
[20:29] <ralsina> nessita: added test for that, works just fine
[20:30] <nessita> ah, let me think this again, then
[20:30] <nessita> ralsina: I'm askin because I struggled with this in the past
[20:30] <ralsina> nessita: pushed that test in r192
[20:30] <nessita> but I may have slightly different issue /me thinks
[20:31] <ralsina> In fact, we always sign urls with parameters (next)
[20:38] <nessita> ralsina: right, my confussion comes when signing url without the next
[20:38] <nessita> that is, signing the url for rest api call, for example
[20:39] <nessita> ralsina: branching the last version to run tests, and approving after that
[20:39] <ralsina> nessita:  cool, thx
[20:43] <nessita> ralsina: hum, I got several lint issues
[20:43] <ralsina> nessita: checking lint is pretty hard on my machine because of the amount of noise I get from the test classes, sorry
[20:44] <ralsina> nessita: if you could pastebin, I'll fix them immediately
[20:46] <ralsina> nessita: I did introduce a pep8 error in the last test (for urls with query) which I fixed now
[20:48] <nessita> http://pastebin.ubuntu.com/644361/
[20:48] <nessita> ralsina: ^
[20:49] <ralsina> The last one is fixed, the others I have to disable
[20:49] <ralsina> Since it's pylint who's wrong :-(
[20:50] <ralsina> and on my notebook.... doesn't give me those. Gives me 200 others, though.
[20:50] <nessita> ralsina: right
[20:51] <ralsina> nessita: pushed the disable
[20:52] <nessita> ralsina: yey
[20:52] <ralsina> Wow, pushing the disabled sounds nasty :-)
[20:52] <dobey> this is why i want to switch to pyflakes :)
[20:53] <nessita> ralsina: GUESS WHAT :-D
[20:54] <nessita> ubuntuone/controlpanel/gui/tests/test_url_sign.py:40:1: E302 expected 2 blank lines, found 1
[20:54] <nessita> ralsina: I'll approve, you can globally approve once ou fix that so tarmac will not reject
[20:54] <ralsina> nessita: http://pastebin.ubuntu.com/644366/
[20:55] <ralsina> but thanks!
[20:55] <nessita> ralsina: I believe you....
[20:55] <ralsina> I swear I am testing things first :-(
[20:58] <dobey> pep8 --repeat
[20:58] <dobey> but pep8 also doesn't exit non-zero on warnings
[21:01] <ralsina> dobey:  --repeat doesn't make a difference if it didn't find any problems in the file
[21:01] <ralsina> ooooooook it does
[21:02] <ralsina> freaking crap
[21:02] <dobey> heh
[21:02] <dobey> pep8 is weird
[21:02] <dobey> it will find other problems after you fix some, sometimes
[21:02] <dobey> even if you do --repeat :(
[21:02] <nessita> ralsina: --repeat does not repeat the same error globally, not per file
[21:04] <ralsina> nessita: Iam having to stop for a while and be a parent. Are you getting to test shares/udfs today?
[21:05] <ralsina> If not, I may do them late tonight
[21:05] <nessita> ralsina: yes if I delay your windows installer branch review which I was starting
[21:05] <ralsina> yes, delay it
[21:05] <dobey> well, i am off. i'll see you all on tuesday. have a good weekend and friday/monday :)
[21:05] <nessita> I can swap and re-start the review tomorrow
[21:05] <nessita> ok, on it!
[21:05] <nessita> ralsina: I'll send the result of the test in the report
[21:05] <ralsina> nessita: I know it works IRL,so I can send today's installer with it anyway
[21:06] <nessita> ralsina: ack
[21:13] <nessita> ralsina: if you're still around, using mandel's branch syncdaemon won't start at all
[21:13] <nessita> :-(
[21:13] <ralsina> :-( indeed
[21:13] <nessita> ralsina: I'll file a bug and send an email to the windows-group
[21:13] <nessita> I just confirmed in trunk syncdaemon is starting
[21:25] <nessita> hum, laptop battery is almost dead, I need to  plug a cable
[21:25] <nessita> brb
[22:13] <DiegoSarmentero> alecu, are you there?
[22:13] <alecu> DiegoSarmentero, here I am
[22:14] <DiegoSarmentero> alecu, i'm at home right now, i was going to fix the issue about the check state of the buttons........  question
[22:14] <alecu> sure
[22:14] <DiegoSarmentero> alecu, all the buttons needs to has a checked state??
[22:14] <alecu> DiegoSarmentero, I think no button needs a checked state
[22:14] <alecu> DiegoSarmentero, this is not about "checked state"
[22:14] <DiegoSarmentero> alecu, i'm confuse
[22:15] <alecu> DiegoSarmentero, this is the visual indicator that a button is "being clicked"
[22:15] <DiegoSarmentero> alecu, ahhhhhhhhhh ok ok
[22:15] <DiegoSarmentero> alecu, i though it was an indicator that the button is pressed
[22:15] <alecu> DiegoSarmentero, try pressing the mouse over a button and holding the button for a few seconds...
[22:15] <DiegoSarmentero> alecu, yes, i know what you mean
[22:15] <alecu> DiegoSarmentero, that's the small visual indication that you have clicked.
[22:15] <alecu> DiegoSarmentero, cool.
[22:16] <DiegoSarmentero> alecu, ok, ready then! working on that!
[22:16] <alecu> great!
[22:19] <ralsina> DiegoSarmentero: sorry I didn't get to call you today at noon. Are you free a little bit now?
[22:20] <DiegoSarmentero> ralsina, yes
[22:20] <DiegoSarmentero> ralsina, at home in peace!!
[22:20] <DiegoSarmentero> :D
[22:20] <ralsina> ok, calling...
[22:21] <ralsina> Diego, PM me the number please?
[22:38] <nessita> DiegoSarmentero: I did not forget about your review, but I ran out of time today, a complicated branch took much time from me
[22:38] <nessita> DiegoSarmentero: I'll do it tomorrow
[22:38] <nessita> ralsina: long email about UDF tetsing was sent by email
[22:39] <karni> duanedesign: omg... are you here still?
[22:39] <nessita> I'm gone now, I'm a little burnt out
[22:39] <karni> duanedesign: I accidentally generated an autoresponse to a support request. I just wanted to give it to joshuahoover :<
[22:39] <karni> joshuahoover: please have a look at RT 3379
[22:39] <joshuahoover> karni: k
[22:39] <karni> joshuahoover: thank you
[22:40] <nessita> bye all!
[22:48] <csgeek> when I run u1sdtool I get a GtkWarning: could not open display  how can I run it the cli tool w/o a gui setup?
[22:58] <alecu> csgeek, at the moment you can't, because currently u1 gets the credentials from the gnome keyring
[22:58] <alecu> csgeek, it might be possible to run it inside an xvfb
[22:59] <csgeek> okay... then I probably should install gnome at the very least
[23:04] <alecu> csgeek, not sure if the whole of gnome is needed; just installing ubuntuone-client should get the necessary dependencies.
[23:08] <alecu> ralsina, ping?
[23:13] <csgeek> okay.  is there a guide on how to connect / associate with an UbuntuOne account?
[23:13] <csgeek> using the u1sdtool
[23:13] <ralsina> alecu: pong
[23:14] <alecu> hi ralsina: I'm getting an error with trunk of sso+sd+control panel; it says sd can't connect because of an auth error.
[23:14] <alecu> ralsina, nessita describes the same in an email.
[23:14] <alecu> ralsina, are you getting that?
[23:14] <alecu> (control panel says sd can't connect, that is)
[23:14] <ralsina> alecu: haven't tried syncdaemon in a while, I was working on u1cp and installer
[23:15] <ralsina> and in u1cp onthings without UI :-(
[23:15] <alecu> ralsina, then don't worry; we'll see about that later.
[23:15] <ralsina> nothing landed on sycdaemon today, right?
[23:44] <DiegoSarmentero> alecu, are you there? i've submited a request merge for the pressed effect on buttons
[23:44] <alecu> DiegoSarmentero, great!
[23:44] <alecu> DiegoSarmentero, want to pass me the link to the merge proposal?
[23:45] <DiegoSarmentero> alecu, https://code.launchpad.net/~diego-sarmentero/ubuntuone-control-panel/button-pressed/+merge/68039
[23:46] <DiegoSarmentero> alecu, let me know if the colors are not the right one..... i use another in the same scale but darker
[23:46] <alecu> DiegoSarmentero, the code looks fine; please set the "commit message" in the merge proposal.
[23:47] <DiegoSarmentero> alecu, ok
[23:47] <alecu> DiegoSarmentero, (it can be the same as "description of the change")
[23:47] <alecu> DiegoSarmentero, I'll test it right away.
[23:49] <DiegoSarmentero> alecu, ready
[23:54] <alecu> DiegoSarmentero, it works fine, but looks a bit weird. We should ask the designer for the exact colors.
[23:54] <DiegoSarmentero> alecu, yep! i had the same feeling
[23:54] <DiegoSarmentero> alecu, i don't have the best taste in colors :P
[23:55] <alecu> DiegoSarmentero, the designer supposedly does :-)
[23:55] <DiegoSarmentero> alecu, ok, let me know when the colors are defined :D
[23:55] <alecu> DiegoSarmentero, let's try the same color gradient that the "non highlighted" button, but with the darker color on top and the lighter below it.
[23:56] <DiegoSarmentero> alecu, ok....... give a sec