JamesTaitHappy Friday, everyone! :)08:32
=== zyga is now known as zyga-afk
* mandel reboots due to updates10:33
gatoxgood morning!11:03
mandelgatox, buenas!11:19
gatoxmandel, buenas11:19
=== zyga-afk is now known as zyga
nessitahello everyone12:02
gatoxnessita, hi12:02
nessitagatox: would you be able to do a review? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/validate-in-sd/+merge/10004112:05
gatoxnessita, yes12:06
nessitagatox: depends on latest nigthlies of u1client12:06
gatoxupgrading like a baus12:06
nessitabaus? :-)12:06
gatoxnessita, didn't you see the video?12:07
nessitaapparently not :-P12:07
gatoxnessita, http://www.youtube.com/watch?v=NisCkxU544c12:07
mandelnessita, morning12:07
gatoxis really famous :P12:07
* mandel super quick errands!12:07
nessitahola mandel12:08
mandelnessita, before I go, for the autoupdate we need a UFe :(12:08
mandelnessita, missing strings..12:08
nessitamandel: when you come back, we will talk about it12:08
nessitamandel: we may not mark for translations those12:08
nessitaor we may try to use other strings already there12:09
mandelnessita, ok, I'll take very few mins, I need to pick up a pair of keys downstairs12:09
nessitamandel: ack12:09
gatoxalecu, buenas12:18
alecuralsina, I've got branches to fix the issues on windows. I did a lot of IRL on them and auth proxy, non auth proxy and non proxy cases seem all to be working ok.12:19
alecuI still need to create the bugs and do some cleaning up.12:19
gatoxnessita, +112:19
mandelnessita, I'm back12:21
nessitamandel: would you have a list of the new strings?12:21
mandelnessita, looking for them right now, give me a few seconds12:21
nessitawill grab some milk12:21
mandelnessita, this are the two new ones: http://paste.ubuntu.com/907123/12:22
nessitamandel: are those the only new needed ones?12:28
mandelnessita, yes12:28
mandelnessita, those are the title on the balloon shown for the update and the actual message12:28
nessitamandel: ok, what about making the titte "Ubuntu One"?12:29
mandelnessita, I have not double check with design or ux (I just realized) but there should be just two12:29
mandelnessita, sure, using the app name sounds reasonable :)12:29
nessitamandel: let me grep a little for the second12:29
nessitahey! the ubuntu font issue in Qt apps is now fixed!12:32
nessitaseems like the update made it to ubuntu12:32
alecunessita, gatox, mandel, ralsina: finding this was an awesome moment: http://stackoverflow.com/questions/1736015/debugging-a-pyqt4-app12:32
nessitaalecu: looking12:33
alecuit's basically calling QtCore.pyqtRemoveInputHook before doing pdb.set_trace()12:33
alecuand it allows you to debug qt apps in a sane way.12:33
gatoxalecu, i think there is a package in pypi that does that12:33
nessitaalecu: wow!12:33
gatoxthe other day we had that problem, and martin gaitan upload it i think12:34
alecugatox, awesome. What is it called? and... Why didn't you tell us about it?!!!!12:35
mandelnessita, wait wait wait, why didn't we know about this?!12:35
nessitamandel: about the pdb thing?12:35
mandelalecu, gatox ^ :P12:35
ralsinaalecu: oh, nice :-)12:35
gatoxalecu, :P don't remember.... let me check12:35
mandelnessita, yes, it was a joke :P12:35
mandelnessita, with no context I just realize is a shit joke hehe12:36
ralsinaand good morning!12:36
gatoxalecu, can't find it... i'll ask later to martin which is the name12:38
nessitamandel: regarding the new string, I would say let's ask ralsina and joshuahoover12:39
nessitaralsina: how late is it to ask a UIFe for a new string?12:39
mandelnessita, I agree12:39
ralsinanessita: incredibly?12:39
nessitaralsina: mandel needs one new string for the autoupdate dialog12:39
ralsinanessita: that's windows only12:39
ralsinanessita: so we may not set it up for translations, since we are not translating windows until after P release12:40
nessitaralsina: yes, but the string will in the multiplatform project and will be listed for translation (if we mark it)12:40
nessitaralsina: right, that was my suggestion12:40
ralsinanessita: so let's do that :-)12:40
nessitabut wanted to confirm is too late first12:40
ralsinatoo late and not needed12:40
ralsinaat least not strictly needed12:40
nessitamandel: ack? ^ (do change the title to "Ubuntu One" please)12:41
mandelnessita, ack!12:41
psypher246hi u1 team. any update on the authentication issue with u1, updated to 12.04 beta and still unable to log in, been weeks now12:43
nessitapsypher246: hi there. There is no issue as far as I know, was someone in particular helping you when you first reported this?12:44
nessitaralsina: would you please review https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/validate-in-sd/+merge/100041 ?12:44
ralsinanessita: on it!12:45
psypher246nessita: when it first bombed out i got an erro which lead me to this bug (https://bugs.launchpad.net/ubuntu/+source/ubuntuone-client/+bug/711162) i have asked about it on the channel a few times but got no response12:46
ubot5Ubuntu bug 711162 in ubuntuone-client (Ubuntu) "ubuntuone-login crashed with ValueError in call_async(): Unable to guess signature from an empty dict" [High,Confirmed]12:46
psypher246since updating about 3 weks agho u1 will not log in12:46
nessitapsypher246: that error means one of the U1 dbus services is not working for you. Let's debug12:46
psypher246nessita: ok12:47
nessitapsypher246: can you please pastebin the output of running the following in a terminal "ps aux | grep ubuntu"?12:47
nessitapsypher246: great, can you please close the controlpanel and kill ubuntuone-syncdaemon?12:48
nessitapsypher246: also, could you please close/kill the ubuntuone-indicator as well? (just to be sure what we have running)12:49
ralsinanessita: +112:49
nessitapsypher246: so, after confirming that no ubuntuone nor ubuntu-sso-login process are running, please pastebin the output of:12:49
nessitaU1_DEBUG=True python /usr/lib/ubuntu-sso-client/ubuntu-sso-login12:50
nessitapsypher246: perfect, do not kill that. Can you now paste the output of (run in another terminal):12:51
nessitaU1_DEBUG=True python /usr/lib/ubuntuone-client/ubuntuone-login12:51
psypher2462012-03-30 14:51:43,295 - ubuntuone.credentials - INFO - Starting Ubuntu One login manager for bus u'com.ubuntuone.Credentials'.12:51
nessitapsypher246: great, now in a third terminal, would you please run u1sdtool -c?12:52
nessitapsypher246: a UI for login should pop up for you12:52
nessitapsypher246: let me know how that goes12:53
psypher246don't think it's coming, both other terminlas just spat out a bunch of output and stating it's being shutdown, will pastebin12:53
nessitapsypher246: please, thanks12:53
psypher246and http://paste.ubuntu.com/907164/12:54
nessitapsypher246: ok, so, what does u1sdtool -s says?12:54
nessitashould be in READY state12:55
mandellunch time for me!!! I'm starving!12:55
* mandel lunch12:55
psypher246State: AUTH_FAILED12:55
psypher246    connection: With User With Network12:55
psypher246    description: auth failed12:55
psypher246    is_connected: False12:56
psypher246    is_error: True12:56
psypher246    is_online: False12:56
psypher246    queues: IDLE12:56
psypher246fyi never changed pw and not able to see how you entre a new one12:57
nessitapsypher246: so, seems like you do have credentials in the system... would you please open seahorse and search for any entry with UBuntu One in the name?12:58
nessitapsypher246: also, please let me know if you see your current device (computer) listed in https://one.ubuntu.com/account/machines/12:58
psypher246yes the machine is listed12:58
psypher246don't see u1 saved in keys, which tab am i looking for?12:59
nessitapsypher246: passwords12:59
alecugatox, nessita, ralsina, thisfred, urbanape: can I ask for reviews?13:00
gatoxnessita, can you check if this sounds good for you? https://code.launchpad.net/~diegosarmentero/ubuntuone-client/unicode-info/+merge/10000713:00
ralsinaalecu: indíd!13:00
thisfredalecu: reviewing the first13:00
nessitapsypher246: could you please also confirm that you see your current device (computer) listed in https://login.ubuntu.com/+applications13:00
gatoxralsina, can you check this branch please: https://code.launchpad.net/~diegosarmentero/ubuntuone-client/unicode-info/+merge/100007 ?13:01
psypher246passwords only has one item, login, can't do anything with it13:01
ralsinagatox: sure!13:01
psypher246used to be able to expand i think?13:01
joshuahoovernessita, ralsina: sorry, was wiping tears from my eyes seeing that there was a suggestion that we may have needed a uife ;)13:01
gatoxralsina, nessita i tested that on windows and linux and it works13:01
ralsinajoshuahoover: sorry! ;-)13:01
gatoxreturn unicode in both cases13:02
psypher246nessita: yes i see my device listed twice, once under software center and then again  under u113:02
nessitajoshuahoover: no worries, we will not mark the new string for translation, so no new UIFe needed13:02
ralsinaalecu: we missed 1:1 yesterday, quick mumble?13:02
joshuahoovernessita: cool :)13:02
nessitapsypher246: ok... did you find your u1 entry in seahorse?13:02
dobeypsypher246: right-click on login and choose "unlock" perhaps if it is locked?13:03
psypher246nessita: i only see" paswords: login"13:03
psypher246nope only lock13:03
nessitapsypher246: right click will allow you to "unlock" it?13:03
nessitadobey: no?13:03
psypher246even if i lock it and then unlock again still no subfolders13:03
ralsinadobey: ping13:03
ralsinadobey: we need to have a video lens package ready for upload on short notice if/when we get a different URL for queries13:04
ralsinadobey: sorry about the rush job13:04
dobeypsypher246: i think there is a bug in gtk+ which seahorse is hitting :(13:04
dobeyralsina: uh ok13:05
* thisfred reviews https://code.launchpad.net/~alecu/ubuntuone-client/fix-broken-tunnel/+merge/10013313:05
davidcalleralsina, dobey, just ping me when you have it if you want.13:05
nessitapsypher246: ok, we can workaround this :-). If you open controlpanel, from the u1 icon in the messaging menu, what do you get/13:05
ralsinadavidcalle: oh, awesome :-)13:05
dobeyralsina: i suppose i need to set up nightlies for it too :P13:05
ralsinadobey: probably but not urgent, since we are in hell-freeze already and nothing is going to change except this13:06
psypher246nessita: the u1 client opens13:07
nessitapsypher246: does the info loads? in particular in the last tab (Account)13:07
ralsinagatox: just to be on the safe side, since options.path_info comes from an external library, I would check that it is a string before trying to decode it13:07
psypher246nessita: yes13:08
ralsinagatox: since u'á'.decode('utf8') fails miserably13:08
psypher246nessita: personal detail has my username and email, your services is blankl13:08
gatoxralsina, sound reasonable.... i'll do that13:08
ralsinagatox: just to futureproof it against whatever13:08
nessitapsypher246: would you please share a screenshot of that?13:08
dobeypsypher246: ah! the bug with the expanding in seahorse appears to be an issue in the theme :-/13:09
dobeyat least for seahorse it's the theme. evolution seems to still be broken13:09
psypher246dobey: do you perhaps have the bug nr for that?13:10
dobeypsypher246: i don't know if there is a bug filed for it yet. i just confirmed the problem though :)13:10
nessitapsypher246: one second, need to check something13:10
dobeymaybe it is a gtk+ bug13:12
nessitapsypher246: I'm doing some queries for your account, will get back to you asap13:15
psypher246nessita: ok thx\13:15
nessitagatox: did you notice that the overlay in ussoc is no longer a transparent overlay? :-/13:17
gatoxnessita, mmmmmm nop13:17
nessitagatox: when you have some slot, would you check if that's the case for you?13:17
gatoxnessita, i can do that now, before getting innto the seg....13:17
nessitagatox: please13:19
nessitagatox: aaahhh I think that was a fix that ralsina added...13:21
psypher246dobey: logged the issue: https://bugs.launchpad.net/ubuntu/+source/seahorse/+bug/96920313:21
ubot5Ubuntu bug 969203 in seahorse (Ubuntu) "12.04 - seahorse (passwords and keys) does not allow me to unlock or expand passwords" [Undecided,New]13:21
nessitagatox: perhaps, when he set that every widget has to have white background, the overlay broke13:21
gatoxnessita, ok..... i'll check that now..... is there an issue for this?? or just do it!13:21
nessitagatox: please open a bug if you can reproduce and fix :-)13:21
ralsinanessita: I did not, but maybe I did one widget too many13:21
gatoxnessita, roger that13:21
ralsinagatox: should be a simple fix though, sorry :-/13:22
gatoxralsina, yes13:22
gatoxno problem13:22
dobeywow, i totally just hit a not so fun issue in launchpad13:28
dobeypsypher246: i've marked your bug as a dup of mine, which is bug #96920913:29
ubot5Launchpad bug 969209 in gtk+3.0 (Ubuntu) "Missing expander arrows in some tree views" [Undecided,Confirmed] https://launchpad.net/bugs/96920913:29
psypher246ok u have way more info :)13:29
psypher246dobey: thx13:29
alecuralsina, sorry, didn't see your request to mumble.13:34
alecuralsina, shall we do it now?13:34
ralsinaalecu: I'm about to mumble with beuno, maybe in a bit?13:37
alecuralsina, sure.13:37
ralsinaalecu: also, I posted a needsinfo in oe of your branches13:37
alecuralsina, awesome. Yes, I added that bit to both branches, and I forgot to add the tests to the -sso one.13:38
ralsinaalecu: cool, with the test, +1 from me13:39
ralsinaalecu: pre-approving :-)13:39
nessitapsypher246: still waiting on some queries (just FYI)13:39
dobeypsypher246: appears there's already fix which should get into precise soon13:41
psypher246ok cool13:41
gatoxralsina, question: "why did you add this:     background-color: white; for the qwidgets in the qss?"13:44
gatoxjust trying to understand which was the problem13:44
gatoxralsina, to see how to change that..... and don't break what that was fixing13:46
ryepsypher246: hi13:47
psypher246hi rye13:47
* rye reads the backlog13:47
ralsinagatox: to make it work with dark color schemes13:47
ralsinagatox: see the attached bug for my branch13:47
gatoxralsina, ok..... i'll look that13:48
ralsinagatox: just put an objectname on the overlay and give it a style13:48
ralsinagatox: it will win by higher specificity13:48
ryepsypher246: so, the syncdaemon status is AUTH_FAILED, from u1sdtool --status, right?13:49
nessitarye: and from controlpanel status13:49
psypher246rye: yes13:49
dobeys/work with/ignore/ :)13:50
nessitaralsina, gatox: we already have a section for the overlay in the qss where we modify it by name, I think13:50
ralsinanessita, gatox: so let's just set the bg color13:50
gatoxralsina, i'm playing with that13:50
ryepsypher246: do you have old-style tokens in the seahorse? I suppose based on the bug report you _might_ have them but they are not accessible, right?13:51
psypher246most likely13:51
ryepsypher246: is it precise?13:51
dobeypsypher246: does "login" still show there if you search for "UbuntuOne" in seahorse, or does it disappear?13:52
psypher246dobey: no nothing happens13:54
nessitabriancurtin: hola! any update on the uninstall issue? was wondering if I should seek more reviews or fix anything13:54
briancurtinnessita: nothing yet, still trying to get it working13:54
dobeypsypher246: so "Passwords: login" still appears in the list, with "ubuntuone" in the search box?13:55
dobeyyou have an old style token in your keyring then :)13:55
psypher246the searchbox has a "missing icon" picture in the box itself as well13:55
briancurtinnessita: im just going to start creating new accounts since i cant figure out where non-admin user creds are being stored in order to get the fresh install to run13:55
nessitabriancurtin: sorry :-(13:56
dobeypsypher246: ok, that's probably a theme issue13:56
briancurtinnessita: no worries, it's all a part of the game :)13:56
dobeypsypher246: if you go to https://one.ubuntu.com/account/ and remove the device entry for your computer, then quit the u1 control panel if it's running, and start it back up, what happens?13:58
psypher246dobey: ok cool, no gonna have to re-upload though?14:01
mandelralsina, are you very busy atm?14:01
nessitapsypher246: nopes14:01
ralsinamandel: kinda, how can I help you?14:01
mandelralsina, I was wondering if we could move or 1-1 a little earlier14:01
dobeypsypher246: no, it's just like logging out and back in. if you have files already uploaded, you should be fine14:02
ryedobey: the local entry will not be removed, since sso does not manage it, only reads. we need to remove the token programmatically14:02
psypher246dobey: control panel still hasthis devive in the list14:02
ralsinamandel: let's do it right now14:03
mandelralsina, superb!14:03
psypher246wheni try remove the device on the web it does not remove it, just takes me to the service screen14:03
psypher246worked from the control panel though14:04
=== urbanape` is now known as urbanape
psypher246dobey: ojk re-added this device still getting auth failed14:05
psypher246but my name is there and all the data usage stats14:06
ryedobey: i suppose the filesync servers use somehow different way of detecting the correct token to use14:09
=== yofel_ is now known as yofel
dobeyrye: it seems like syncdaemon is getting the old token, and control panel is maybe getting the new one14:11
dobeybut not sure14:11
ryepsypher246: could you please run this script - http://paste.ubuntu.com/907263/ - this will remove the token from gnomekeyring14:12
rye(if i correctly specified the arguments)14:12
ryepsypher246: wait, no14:13
ryepsypher246: http://paste.ubuntu.com/907265/ - this one14:13
psypher246u1: 5: u1: Syntax error: "(" unexpected14:15
psypher246names script u1 fyi14:17
ryepsypher246: are you running it as "python scripname" ?14:17
psypher246oooh sorry i thought it was bash14:17
ryepsypher246: sorry, should have specified that, if it errors out with NoMatchError, then no, you did not have the token, otherwise if it prints nothing, then it removed the token14:18
nessitapsypher246: if you re added the computer, you need to restart syncdaemon14:18
nessitapsypher246: since it will not pickup automatically the new token14:18
ryepsypher246: well, this means there is nothing to delete... at least nothing i am searching for14:19
psypher246should i just wait till that other seahorse bug is fixed?14:19
psypher246i just restarted like nessita said, connected now14:20
nessitapsypher246: YEY14:20
psypher246i would have tried removing it and adding, i am just always so worried about having to re-upload14:20
psypher246I have lost my storage that i bought?14:21
ryepsypher246: tokens don't have direct connection to your data, you can create new ones and they will link to the same account. The only thing that matters is that you should not authorize your computer with another ubuntu one account, in this case you will get ROOT_MISMATCH14:22
psypher246how can i check when my 40gb expires?14:22
ralsinagotta run for a doctor's appointment, will be back in 80 minutes, and have no lunch. See you all soon!14:22
ryepsypher246: Mar-05-2012 Expired14:22
psypher246don't recall getting a warning\14:23
psypher246should I?14:23
nessitagatox: is this ready https://code.launchpad.net/~diegosarmentero/ubuntuone-client/unicode-info/+merge/100007 ?14:24
gatoxnessita, yes14:25
nessitagatox: who suggested adding the isinstance?14:25
ryewebm0nk3y: ^ psypher24614:25
gatoxnessita, ralsina told to be nice to check that options.path.... is in fact a str14:25
briancurtinnessita: alrighty. i *finally* got my Popen uninstaller to work successfully on a XP non-admin account (still has to be executed from "run as"). it works fine on an admin account as well. i can test it on Win7 now if you'd like14:26
nessitagatox: I would definitely not use isinstance, that means we have no idea what we're getting and we're just guessing :-). I would definitely recommend a try-except UnicodeDecodeError14:26
webm0nk3yrye: ?14:26
briancurtinnessita: however, i still can't explain why the old way, shellexecute, does not work. i'm looking into that as well14:26
gatoxnessita, ooooook14:26
psypher246rye: is there no system which notifies you that your storage is going to expire, looking at email I don't see anything14:26
nessitabriancurtin: could you please check with gatox and mandel about that? they built the code the first tme14:27
nessitabriancurtin: and they may defend it, or help debugging it14:27
mandelbriancurtin, may I have some context? :)14:27
briancurtinmandel: the "uninstall_application" at the bottom of this MP: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/uninstall/+merge/10001214:28
* mandel looks14:28
briancurtinwhen i run IRL, i get a traceback (i'll have to dig it up, don't have it handy) that the ShellExecute call does not work. however, if i just change it to use subprocess.Popen, everything is fine14:28
briancurtini'll dig up that traceback14:28
briancurtinmandel: with ShellExecute being used, I get https://pastebin.canonical.com/63408/ (Access is denied.)14:29
mandelbriancurtin, let me see..14:29
briancurtinso the path being given to it is correct, and just passing it to Popen works...but somehow ShellExecute can't do it. this was taken from somewhere in ubuntuone-windows-installer14:30
briancurtini switched it with """Popen([uninstall_path, "--mode", "win32"])""" and it works fine14:31
mandelbriancurtin, without runas?14:32
mandelbriancurtin, what user do you have? have you tried with a non admin user?14:32
mandelbriancurtin, I would be very happy if we can remove runas because it will allow users with an AD account :)14:32
briancurtinmandel: i've tried the Popen way with both admin and non-admin on XP. i'm about to start trying on Win 714:33
briancurtinmandel: the installer itself needs runas, at least on XP. since this uninstall is called during the installer (when you disagree with the license), the process is already elevated and doesn't need explicit elevation to run uninstall14:35
briancurtini'm going to build an installer that uses ShellExecute without runas and see what happens14:35
mandelbriancurtin, please do, using runas is a dirty horrible trick14:36
mandelbriancurtin, and is possibly related to the installer working in a funny way regarding the privileges14:36
gatoxnessita, i've just push the changes into u1-client-unicode-info14:39
nessitagatox: thanks!14:39
briancurtinFWIW, if we don't need runas, then we also don't need ShellExecute. it's effectively just doing a bare CreateProcess with no special "verb" (the runas)14:39
mandelbriancurtin, and it would be great!14:42
psypher246thanks for all the help guys in getting me back up and running, gotta go, have a great weekend!14:42
mandelbriancurtin, also fixes the AD users bug which I had no idea how to solve and I suspected that we would have because people in corporate networks use it14:42
briancurtinmandel: do you have a reference to that? The installer will still need to be started by right clicking and selecting "Run as..." (at least on XP). in my tests i have Administrator, brian (an admin), and limited (a non-admin) - Admin and brian can just click on the installer and it goes. limited can't -it fails to even begin copying the files14:44
alecumandel, gatox: can I ask for one review on this? https://code.launchpad.net/~alecu/ubuntuone-client/fix-broken-tunnel/+merge/10013314:44
mandelbriancurtin, to the AD bug?14:45
mandelalecu, sure14:45
gatoxalecu, ok!14:45
briancurtinmandel: if there's a LP #14:45
mandelalecu, read the u1-internal we have a very stupid bug related to proxy, I'll fix asap14:45
mandelbriancurtin, sure there is, let me find it :)14:45
briancurtinmandel: we won't be able to install to "C:\Program Files" without admin elevation. this is part of the reason why python installs to C:\PythonXY, so everyone can have it even if their sysadmin doesn't give admin access :)14:45
mandelbriancurtin, yeah, that is why we used the dirty runas, but it certainly makes no sense..14:48
mandelbriancurtin, if we remove runas from everywhere you are fixing bug 89447114:50
ubot5Launchpad bug 894471 in Ubuntu One Control Panel "Autoupdate is broken on machines that have joined a domain with AD" [Undecided,Confirmed] https://launchpad.net/bugs/89447114:50
gatoxalecu, nessita ralsina dobey ?15:04
* gatox listen the sound of the wind......15:05
alecume should write some notes15:05
mandelthe wind sounds like that in spain..15:05
gatoxmandel, lazy wind!15:06
* dobey needs to plan some holiday15:06
gatoxdobey, to listen the wind in spain?15:06
dobeybut to go to spain15:07
mandeldobey, you know you are welcome :)15:07
gatoxi would say gatox go..... the rest can be last.......15:07
* thisfred recommends Portugal: the people are much nicer there :P15:07
urbanapegatox: go15:08
gatoxPropose a branch for the encoding problem in u1sdtool --info. Fixing the overlay issue in sso (almost done).15:08
gatoxStart working on: #940669 ubuntu-sso-login crashed with SIGSEGV in QSocketNotifier::setEnabled()15:08
gatoxurbanape, go15:08
urbanapeDONE: got the new buildout dev environment up, interviewed someone for the iOS position15:08
urbanapeTODO: get ubuntu-sso-client branch back into play and finished, diagnose a crasher in the iOS Music app15:08
urbanapeBLOCK: None15:08
urbanapemandel: por favor15:08
mandelDONE: All hands, if anyone else asks me for a review I will hunt them down and make them suffer as much as I do in that web page. Got autoupdate branch up to date and working.15:08
mandelTODO: IRL for the autoupdate branch. Propose and move to bug 96928015:08
mandelBLOCKED: no15:08
mandelbriancurtin, go!15:08
ubot5Launchpad bug 969280 in Ubuntu One Client trunk "If http_proxy is enabled then an incorrect assumption about https_proxy is made" [High,In progress] https://launchpad.net/bugs/96928015:08
briancurtinDONE: testing windows branches almost every second of the day. heard the XP startup sound more than i ever needed to15:08
briancurtinTODO: figure out this windows uninstall/autoupdate situation, eventually move back into windows read/write for testing/jenkins improvements15:08
briancurtinBLOCKED: no15:08
briancurtinNEXT BATTER: thisfred15:08
thisfredDONE: reviews / u1db mapping indexes progress TODO mapping indexes BLOCKED no NEXT dobey15:08
dobeyλ DONE: team meeting, bug #961342, bug #968555 (protocol)15:08
dobeyλ TODO: more bug #968555, bug #96926215:08
dobeyλ BLCK: none.15:08
ubot5Launchpad bug 961342 in ubuntuone-client (Ubuntu Precise) "GI error while trying to show notifications" [High,Triaged] https://launchpad.net/bugs/96134215:08
ubot5Error: Launchpad bug 968555 could not be found15:08
ubot5Launchpad bug 969262 in libubuntuone trunk "Error while getting credentials: The '/org/freedesktop/secrets/collection/login/24' object does not exist" [High,In progress] https://launchpad.net/bugs/96926215:08
alecuDONE: IRL testing of sd on windows, a few fixes for the tunnel: bugs #969150, #969157, #96915915:08
alecuTODO: catch up with SD bug backlog15:08
alecuBLOCKED: no15:08
alecuCOMING UP: nessita15:08
ubot5Launchpad bug 969150 in Ubuntu One Client "Proxy tunnel is trying to use QtDbus on Windows" [High,Triaged] https://launchpad.net/bugs/96915015:08
ubot5Launchpad bug 969159 in Ubuntu Single Sign On Client "Windows keyring uses twisted deferToThreads even when no reactor is installed" [High,In progress] https://launchpad.net/bugs/96915915:08
ubot5Launchpad bug 969157 in Ubuntu One Client "Tunnel is not using the system proxy on Windows" [High,In progress] https://launchpad.net/bugs/96915715:08
nessitaDONE: a lot (landed like 5 last-minute branches yesterday, for controlpanel mostly)15:09
nessitaTODO: some more, add missing tests to controlpanel15:09
nessitaBLOCKED: no15:09
nessitaTODO.append: reviews15:09
nessitaany comments anyone?15:09
mandelalecu, he, bug 969159 makes sense, the keyring was never updated not to work without a reactor15:09
ubot5Launchpad bug 969159 in Ubuntu Single Sign On Client "Windows keyring uses twisted deferToThreads even when no reactor is installed" [High,In progress] https://launchpad.net/bugs/96915915:09
nessitaeveryone: reminder that next week is national holiday in argentina on Mon, Thru and Fri15:10
alecumandel, already fixed and about to land :-)15:10
nessitaI will be working Tue, Wed and Thru15:10
mandelalecu, sweet!15:10
alecunessita, not monday *thru* friday!15:10
nessitaalecu, gatox: could you please specify which days will you be working next week?15:10
alecunessita, just mon, thu, fri.15:10
nessitaalecu: LOL15:10
gatoxnessita, tue, wed, thu, fri15:11
alecunessita, all: I'll only be working tue and wed.15:11
nessitaeom then?15:11
* alecu needs to check canonicaladmin to see if it's in order.15:11
mandelnessita, I'm updating my holiday, I'm working on monday and tues for what is worth it, the rest of the week I'm out15:11
nessitamandel: thanks for the update15:12
briancurtinthe US needs to get some of these many-day-a-week holidays going15:13
urbanapebriancurtin: we do. it's called March Madness.15:17
briancurtinah this is true. it became second spring break in college15:20
gatoxalecu, +115:21
* gatox lunch+bank15:21
urbanapebriancurtin: So, I got the buildout complete, and while the ubuntuone-dev-tools egg is built, I don't see u1trial anywhere in the devsetup.15:21
nessitabriancurtin, mandel: did you guys reach any conclusion in the uninstall issue?15:22
briancurtinnessita: i'm running on win7 right now a few cases, will report the final findings shortly15:22
=== gatox is now known as gatox_away
mandelnessita, if we can really get rid of the runas and shell execute we will, is cleaner code15:23
briancurtinurbanape: ah, i haven't written the equivalent of what on windows is the "env.bat" script (in devsetup/) - it copies u1trial and u1lint to devsetup/bin, then sets up the PATH and whatnot15:23
mandelnessita,  briancurtin, is doing IRL for that15:23
nessitamandel, briancurtin: nice! we should apply the same solution to both: the autoupdater and the uninstaller, agreed?15:23
mandelbriancurtin, I just realize, for the autoupdate, we need the runas because the execution is done as the user that executes the control panel15:24
mandelnessita, ^15:24
briancurtinmandel: i haven't looked explicitly at autoupdate, just uninstall, but i see it there as needing the same investigation15:24
nessitamandel: the uninstall as well, no?15:24
nessitabriancurtin: ^15:24
briancurtinthe uninstall does not work with runas15:24
briancurtin(on XP, testing on 7 now)15:24
mandelbriancurtin, nessita, if we do a release with that we should make sure that rmcbride and elopio know about it so that we don't brake things15:25
nessitabriancurtin: excuse my ignorance, but not sure what that means15:25
briancurtinnessita: whether you are an admin or a limited user, uninstall does not work when "runas" is an option to ShellExecute15:26
briancurtinit gives that "access is denied" error15:26
briancurtinnessita: your uninstall branch either requires the "runas" option to ShellExecute to be removed, or to use Popen15:27
briancurtinnessita: but this is only what i've seen on XP within a workgroup. apparently it may be different in a domain. it may also be different in Win7, which is what im looking at now15:27
briancurtinbut what i'd expect is the majority of users, people using Windows on their home computer which is just a workgroup, your uninstall branch won't work for them15:28
briancurtinah-ha...looks like there's a difference between XP and 7 here. still running tests, but i think i know how to fix this15:31
nessitabriancurtin: just FYI, officially we don't support XP (we always try to), but when in doubt, prioritize win 715:36
nessitamandel: so you said you'd work Mon and Tue only?15:36
mandelnessita, yes, let me update my calendar and add it to the online services one15:37
nessitamandel: according to the canonicaladmin, you're in holiday Mon and Tue15:37
nessitaand on swap day on Wed15:37
nessitamandel: and you work Thu and Fri15:37
mandelnessita, yes, I talk with ralsina, the house I was going had an accident and had to change mon and tues15:38
nessitamandel: you broke a house without even being there?15:38
dobeylunch, bbiab15:38
mandelnessita, hm.. weird, I though I ask for 5/4 and 6/4 let me check15:38
nessitamandel: your breaking powers amazed me15:38
mandelnessita, frozen pipes.. water everywhere :(15:39
briancurtinnessita, mandel: here's what we have...https://pastebin.canonical.com/63477/ i think we should not use "runas" for the uninstall15:49
nessitabriancurtin: from that, I would say +115:50
nessitabriancurtin: will change that after having lunch15:50
nessitabriancurtin: thanks a lot for testing this!15:50
* mandel looks15:51
mandelbriancurtin, so in all the systems * ShellExecute (without runas) - success, right?15:54
briancurtinmandel: yep, that seems like the best route. i don't currently have a Vista VM, though15:54
mandelbriancurtin, do you have the script you use for testing? I want to try it on Vista, is an evil OS and will like to know what it does15:55
mandelbriancurtin, I have one, usually never boot it but got it just in case :)15:55
briancurtinmandel: i just created three different installers with the code changes, then ran them manually15:55
briancurtinmandel: i can upload them somewhere if you want to run them15:55
mandelbriancurtin, sure, put them in a u1 folder and share it with me :)15:55
mandelnessita, FYI I'll do the same tests for Vista15:59
ralsinaI'm back!16:03
alecuralsina, all the fixes for the tunnel issues on windows have landed on trunk.16:16
alecuralsina, should we make branches to merge them against stable?16:16
nessitabriancurtin: the removal of the runas parameter has been pushed to revno 30916:16
nessitabriancurtin: please let me know if I blindly fixed the windows tests correctly16:17
mandelbriancurtin, FYI I might take a little to test the installer, waiting for vista to do updates..16:17
briancurtinnessita: will look16:18
ralsinaalecu, nessita: good question. If we do that, we can try to make a windows release that is synced to 2.99.9216:20
ralsinanessita: are we still merging everythig on this release, or cherripicking?16:21
nessitaralsina: I will update stable branch with what we have in trunk on tuesday16:21
nessitaralsina: we're merging everything16:21
ralsinanessita: cool then. Alecu, no need.16:21
alecuawesome then.16:22
nessitaralsina: shall I propose a branch that wipes everything outside 'scritps' from the windows-installer project?16:42
nessitamandel: how's the autoupdate branch going?16:42
nessitaralsina: if we ever need something from there, will be in trunk's history16:43
mandelnessita, doing IRL tests16:43
ralsinanessita: sure16:43
nessitaralsina: do you guys use anything outside scritps/ now?16:43
ralsinanessita: no16:44
* briancurtin stepping away for a few minutes16:45
dobeymeh i don't want to look at this bug :-/16:45
nessitadobey: just close your eyes16:47
nessitamandel: I would advice merging https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/uninstall/+merge/100012 in your autoupdate branch16:48
dobeyi'm still working on that as a means to fix bugs16:48
nessitamandel: since it may generate conflicts for you16:48
mandelnessita, ok, will do now and see what goes wrong16:49
nessitamandel: let me know, I can help16:50
nessitawill have lunch now16:50
nessitabut I will be back16:50
* alecu says it's time for lunch, errands and whatnot.16:58
=== alecu is now known as alecu_errands
gatox_awayfinally back!17:09
gatox_awaythe transit is a MESS! and the streets are full of people (that scares me :P jjee)17:09
=== gatox_away is now known as gatox
nessitagatox: and is hot!17:12
gatoxnessita, also!17:12
gatoxralsina, after your unicode post..... do you want to review this? https://code.launchpad.net/~diegosarmentero/ubuntuone-client/unicode-info/+merge/10000717:12
ralsinagatox: indeed :-)17:13
ralsinagatox: like manuel says, this project is an endless source of blog posts ;-)17:13
ralsinagatox: that would actually be UnicodeDecodeError, UnicodeEncodeError :-)17:13
ralsinaif path_info is unicode, it will give a DECODE error. If it's invalid bytes, it will be ENCODEERROR17:14
gatoxralsina, ahhhhhh true17:14
ralsinaor viceversa17:14
gatoxif it was unicode es Encode17:14
mandelnessita, FYI there are some conflicts fixing and lets see if tests pass :)17:15
nessitamandel: ack17:15
gatoxralsina, done17:16
ralsinagatox: checking17:16
nessitaralsina: you can't have an encode error there17:16
nessitaralsina: since we're never encoding, just decoding17:16
ralsinanessita: yes you can17:16
nessitaralsina: how?17:16
ralsinanessita: try u'á'.decode('utf-8')17:16
ralsinanessita: SURPRISE! :-)17:16
nessitahum, why is raising a UnicodeEncodeError?17:17
gatoxunicode magic17:17
ralsinanessita: http://t.co/wChRKHzF17:17
nessitaah, ok, thanks17:18
nessitaI learnt something new17:18
mandelnessita, add_to_autostart is gone, right?17:18
nessitamandel: no, is moved so is alphabetically ordered17:18
mandelnessita, ah, ok I did not see it up there, thx17:19
dobeyralsina: are unicode objects actually guaranteed to be utf-8 though?17:20
ralsinadobey: they are guaranteed to be ENCODABLE as utf-817:20
ralsinaasking if a unicode object "is utf8" is meaningless.17:20
nessitaralsina, briancurtin: when you can, https://code.launchpad.net/~nataliabidart/ubuntuone-windows-installer/wipe-it/+merge/10020417:20
ralsinaexcept as internal representation which is unimportant17:20
nessitamandel: also, if you could review the uninstall branch, that would rock17:21
mandelnessita, certainly, let me fix merge issues and take a look at the the mp17:21
ralsina-9056! nice!17:21
nessitamandel: sure17:21
briancurtinnessita: will look shortly17:21
nessitaralsina: and +0!17:21
nessitabriancurtin: no rush17:21
ralsinanessita: that branch should close a bunch of bugs, too! ;-)17:22
ralsinanessita: +9056 on that branch17:22
mandelnessita, in https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/uninstall/+merge/100012 the perform update does need the runas AFAIK since it is a process that will require the admin right as is spawned by a process that does not have them17:29
mandelbriancurtin, do you think so? ^17:29
nessitamandel: then that would break, no? (as per briancurtin's tests)17:29
briancurtini don't know specifically for the update one, ive only looked into uninstall. they may operate differently17:29
nessitabriancurtin: ah, ok, shall I restore the runas there then?17:30
mandelnessita, I would do so and I'll make sure we use the correct one in the autoupdate branch17:30
nessitaok, will revert that change there17:30
mandelnessita, that way we reduce the chance to get it wrong :)17:30
nessitamandel: true17:31
mandelnessita, I'll do IRL for this specific case17:31
briancurtinnessita: i'm not sure yet. also i think the way you have it written, removing hte "runas" parameter entirely, will not work. what i meant was to make that paramter ""...running the installer in 1' to double check this17:31
nessitabriancurtin: ah, ok17:31
briancurtinnessita: yeah you will need to add back in a "" where the "runas" used to be17:35
nessitabriancurtin: added, committed and pushed to revno 31017:35
nessitamandel: ^17:35
nessitamandel: also restored the runas for updater17:37
mandelnessita, sweet I'll remerge with mine :)17:37
briancurtinnessita: uninstall MP approved17:54
nessitabriancurtin: awesome, thanks!17:54
mandelnessita, in here: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/uninstall/+merge/100012 in FrozenTestCase is there a way to get the cleanup calls closer to what the clean?17:56
mandelnessita, if it is no possible is not an issue17:56
mandelnessita, https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/uninstall/+merge/100012 +!17:58
mandel+1 :P17:58
nessitamandel: we can have them closer, but then we'd need duplicate some calls, because the same cleanup may be needed by any guard17:58
mandelnessita, then is fine, the branch is approve I trust your decision17:59
ralsinadobey: +118:02
gatoxoverlay done..... it was more difficult than i thought18:07
gatoxit wasn't just a style thing18:07
ralsinagatox: really?18:09
gatoxralsina, yap..... i tried setting a specific style for the overlay, with background-color: none/transparent... but didn't work18:10
gatoxi needed to set the stylesheet directly to the widget, to delete any previous style...... and change a bit the paintEvent for the animation18:10
gatoxnot the paint event, but who paint event is being listened18:11
gatoxi'll propose now18:11
nessitaralsina: any chance you review this? https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/unicode-syncing-error/+merge/9999018:19
ralsinanessita: sure!18:19
ralsinanessita: I had that queued yesterday, must have skipped it18:20
nessitaralsina: np18:20
mandelnessita, ralsina FYI I have I did the IRL of the update and works ok but I have tests failing and I ran out of time, I' work on thins on monday and will ping briancurtin to help me fully test it18:21
ralsinadobey: lp:~diegosarmentero/ubuntuone-client/unicode-info just bounced very weirdly on tarmac18:21
ralsinadobey: giving an error about a PDF of all things18:21
briancurtinmandel: ack, let me know whatever you need18:21
nessitamandel, ralsina: we need this landed today if we want to have it in stable, no?18:21
ralsinamandel: awesome18:21
nessitamandel: what if briancurtin follows up on that? ralsina, is that possible18:22
nessitain stable next Tues, I mean18:22
ralsinamandel: oh, I thought you meant you ran out of time in the test fixes you were working yesterday!18:22
gatoxralsina, nessita review please: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/overlay-transparent/+merge/100216 (and with pretty pictures :P)18:22
mandelnessita, I can work a little longer later today, just give me an hour to take the dog out a little18:22
ralsinamandel, nessita, briancurtin: yes, please, brian, take over mandel's branch, let's try to merge that today18:22
mandelralsina, nop.. on the autoupdate18:22
nessitamandel: no need to over work, perhaps briancurtin can follow up18:23
briancurtinmandel: can you send me the branch and some info about what's going on with it?18:23
ralsinamandel: what nessita said, we are not THAT hurried18:23
nessitagatox: I really really really not like putting the qss like that, in the code. You 100% there is no other way?18:24
nessitagatox: have you asked ralsina for more options, using qss only?18:24
mandelralsina, ack18:24
mandelbriancurtin, let me push the branch for you, give me a sec18:24
gatoxnessita, i'll try another option and let you know18:25
ralsinanessita, gatox: nothing comes to mind, really, I don't understand the problem yet18:25
gatoxnessita, ralsina let me try another option........ but, something positive (at least for me)..... is that if you reuse this component (the loading overlay) the other projects won't need to know the qss for this component to look properly18:26
nessitagatox: and that's a con... each project should be able to customize the overlay as the project needs18:28
nessitagatox: another project may need the background orange a the bullets white18:28
gatoxnessita, ok.... the other option didn't work either..... i can keep with this if you want.... trying more options.... but i'm not sure that is going to be doable just with the qss18:31
ralsinagatox: I don't understand why the QSS is not applied. You know?18:31
gatoxralsina, the qss is being apply...... but for some reason, i can't remove the background........ mmmmmmm another idea!18:32
nessitagatox, ralsina: so, looking at gatox's branch, I realized that the indentifier frm_box is just the "small" box with the bullets in it18:32
nessitagatox, ralsina: I think we should set a name for the "parent" of that box, no?18:32
nessitaand set the style to it18:32
ralsinanessita: the parent is the big one?18:32
nessitagatox: did you try that ^?18:33
dobeyralsina: looking18:33
nessitaralsina: "should be"18:33
gatoxnessita, yep18:33
nessitagatox: and that did not work?18:33
dobeygatox: there's a lint error in your branch18:33
dobeyralsina: ^^18:34
gatoxnop..... i did a setObjectName to the LoadingOverlay class..... and didn't work18:34
gatoxsetting the style for that name in the qss18:34
dobeyralsina: the pdf message isn't a fatal error18:34
gatoxdobey, really?? a TODO?18:34
dobeygatox: no a unicode error in bin/u1sdtool18:34
gatoxahhh the other branch18:35
dobeyin unicode-info yeah18:35
dobey    160:  local variable 'UnicodeEncodeError' is assigned to but never used18:35
gatoxahhhhhhhhhh silly me......18:35
dobeygatox: oh your code is wrong18:36
dobeywho approved that!18:36
dobeyyou need parens!18:36
ralsinaargh, my fault18:36
* ralsina goes read the python tutorial as penance18:36
gatoxmy bad, i forgot that18:37
mandelbriancurtin, the branch is here: bzr push lp:~mandel/ubuntuone-control-panel/auto-update-looping-call18:37
mandelbriancurtin, the idea is to add a looping call that will check for updates every x minutes, will pop a message in the app tray icon and will perform the update when the user clicks on it18:38
mandelbriancurtin, there are two tests failing due to the crazy merge I had to fix18:38
nessitagatox: I'm testing a branch where the styled is applied, but something is not letting the 'transparency' work18:39
briancurtinmandel: ack, thanks, i'll start working on it18:39
nessitagatox: so, I added this to the style:18:39
nessita     93 LoadingOverlay {18:39
nessita     94     background: red;18:39
nessita     95     border: 3px solid green;18:39
nessita     96 }18:39
mandelok, all have a great weekend!18:39
mandeland ar, have a great easter :P18:39
ralsinamandel: easter is next sunday?18:39
nessitagatox: and I have a redish background with green border, the problem is that the redish is not full red, but like it had something applied to it that make sit softer18:39
ralsinaanyway have good weekend you too!18:40
nessitaralsina, gatox: can this be explained with your knowledge? http://ubuntuone.com/5trjjPDlwqzRWFRO7hkGha18:40
nessitaralsina, gatox: that's the result of applying the style I pasted above ^18:41
briancurtinnessita: since i'm doing a similar thing with manuel's branch that you did with my volume_manager notify branch...is there some formal way of "taking it over", or do I just branch from his code and push to my own?18:41
gatoxnessita, why is not fully red?18:41
ralsinanessita: two widgets, one transparent the other red, one on top of the other?18:41
gatoxbecause the painter is adding a transparency18:41
nessitabriancurtin: no other way other than hijacking :-)18:41
nessitagatox: you tell me why is not fully red18:41
gatoxnessita, yes18:41
dobeynessita: LoadingOverlay has an opacity less than 118:42
gatoxnessita, which was the questio?18:42
nessitagatox: what is the background red not fully red?18:42
nessitawhy* sorry18:42
gatoxnessita, ah... so the answer was correct18:42
nessitagatox: what's the answer?18:42
dobeynessita: LoadingOverlay has an opacity less than 118:42
nessitadobey: "where"? :-)18:42
dobeynessita: i don't know, i'm not looking at the code. but that is *why* :)18:43
gatoxnessita, the painter that "paint" :P the widget has this to fill the area: QtGui.QColor(255, 255, 255, 135)18:43
gatoxwhere the last number is the opacity18:43
gatox255 will be not transparent18:43
nessitagatox: let me play with that then18:43
gatoxalready did18:43
nessitaI made it work18:44
nessitaI'm getting the transparent background18:44
nessitalet me share the branch18:44
nessitagatox: this is working in my env http://pastebin.ubuntu.com/907682/18:45
nessitaralsina, gatox: that diff gives me http://ubuntuone.com/5trjjPDlwqzRWFRO7hkGha18:46
gatoxyes..... :S18:46
ralsinanessita: looks good to me!18:46
gatoxnessita, are you going to propose that?18:47
nessitagatox: all I've learn for qss I've learnt from you, so you can consider this your win18:47
nessitagatox: no no, you do it18:47
gatoxnessita, ralsina changes pushed....18:51
nessitagatox: thanks!18:51
gatoxdobey, https://code.launchpad.net/~diegosarmentero/ubuntuone-client/unicode-info/+merge/100007 updated18:52
nessitagatox: approved!18:53
gatoxnessita, thanks...... and sorry about that.... i was too focused try to make the thing transparent! that i miss to do that completely :S18:54
nessitaman, I love the ubuntu font not being bold anymore18:54
gatoxand with objetName wasn't working18:54
dobeynessita: the overbolding is fixed now?18:54
nessitagatox: yes, I tried objectName... I don't like saetObjectName, since I think is buggy when more than one widget is created18:55
nessitadobey: YES YES YES18:55
nessitagatox: anyways, the only thing I would like you to take from this is that if you find doing some nasty Qt things, ping ralsina for help... because if we reach a place where Qt make us do nasty things, we need to rethink higher level stuff18:56
nessitagatox: or not apply the fix altogether18:56
dobeystill bold to me in control panel18:56
ralsinanessita, gatox +118:56
nessitadobey: did you update today and rebooted?18:56
ralsinagatox: same thing goes to nessita and I, nasty stuff needs more eyes18:57
gatoxroger that18:57
dobeynessita: why reboot?18:57
ralsinadobey: font cache18:57
nessitagatox: anyways, thanks a lot for working on this18:57
nessitadobey: with unity that's the only thing that works for me to apply changes18:57
nessitadobey: but I haven't tried that hard18:58
nessitait looks SO much better now18:58
dobeyralsina: nonsense :)18:58
nessitaralsina: are bold font gone for you as well?18:58
ralsinadobey: hahaha18:58
dobeynessita: ah, i do see a font update now, that wasn't there earlier18:58
ralsinanessita: never had it because I am not using ubuntu font18:58
nessitadobey: I have non-bold fonts since this morning 9am ART18:58
ralsinanessita: I only had it on my test-fresh-user accounts18:59
dobeymaybe i didn't do an update this morning then, or forgot to do dist-upgrade18:59
dobeyit's all just a blur to me18:59
nessitadobey: hum, perhaps you should quit drinking scotch when you wake up :-P19:01
nessitaralsina: any nice way to access the last row in a treewidgetview?19:01
ralsinanessita: hmmm... is it a firt-level item?19:01
nessitanice would be without grabbing an iterator and going to the end19:01
nessitaralsina: yes19:01
ralsinanessita: give me 1'19:01
ralsinanessita: know the text?19:02
nessitaralsina: the text that is inside the row?19:02
ralsinanessita: topLevelItem(topLevelItemCount())19:02
elopionessita: what would be a maximum acceptable time for the control panel to connect19:03
ralsinawith necessary object names inserted19:03
ralsinanessita: I am not 100% sure that works right if items are sorted on insertion though19:03
nessitaralsina: is ok, we do not sort19:03
ralsinaare NOT sorted19:03
ralsinanessita: yes we do19:03
nessitaelopio: hum... to connect to the web?19:03
nessitaralsina: heh?19:03
dobeynessita: what else would i drink with the gummy bears?19:03
nessitadobey: milk!!!19:03
dobeywith gummy bears? crazy19:04
ralsinanessita: I recall it not being the same order as the volume list but I may be remembering wrong19:04
nessitaralsina: we order the volume list and then fill in the folder treewidget, so strictly speaking we do not re-order the treeview19:04
ralsinanessita: then it's ok19:04
nessitaralsina: thanks19:04
nessitaelopio: sorry, I'm not sure what to answer since I'm not sure what you're specifically asking :-)19:05
elopionessita, I'm making a script that clicks the connect button and waits for the control panel to say it is connected.19:08
elopionessita: but I need to set a maximum time for the script to wait until connected.19:08
nessitaelopio: you mean the 'file sync connect' button?19:09
elopionessita: yes.19:09
nessitaelopio: even if we assume you have netowrk, and the speed of the connection is within average,  the connection can take a very long time, since it depends on our U1 servers answering to syncdaemon19:10
nessitaelopio: if we have issues in the server, you can "never" connect19:10
nessitaelopio: under normal circumstances, I would say 30 seconds19:10
nessita(as a maximun)19:10
nessitaor a minute19:11
elopionessita: here it's taking around 1:30. For now I'm going to set a really big maximum so doesn't fail because of the load in the server.19:11
nessitaelopio: ack19:11
elopiobut one good usability goal in the future would be to make that a maximum of, i don't know, 30 seconds.19:12
nessitaelopio: main issue is that the proces of 'connect' involves, among others, a server rescan19:12
elopioI'm sure we have a lot of other issues to take care :)19:12
nessitaelopio: and network, which is impossible to restrict19:12
elopionessita: but if my network is overloaded, we should say: "it's taking a lot of time, check your connection or patiently wait". Like gmail does.19:13
elopiojust wait for ever doesn't sound nice.19:14
nessitaelopio: right19:14
elopionessita, ralsina: good news are that the qt-testability driver works great for the automation.19:15
ralsinaelopio: awesome!19:16
dobeyyay the bold is gone19:23
dobeyalso, ubuntu is a really compact font19:23
gatoxneed to reboot..... brb19:29
=== alecu_errands is now known as alecu
nessitaralsina: just FYI, the last one is  folders.topLevelItem(folders.topLevelItemCount() - 1)19:33
ralsinanessita: hahaha19:34
ralsinayou know what they say, there are 2 difficult things, naming stuff, cache invalidation, and off-by-one errors19:34
briancurtinnessita: yeah i dont know what is going on with this mandel branch. i fixed one thing to get tests further along, but i'd say there's a 1% chance of this being figured out by the end of the day :/19:54
nessitabriancurtin: may I help somehow? have traces I can look at?19:55
briancurtinnessita: here's the branch https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/mandels-updater and here's the trace https://pastebin.canonical.com/63500/19:56
ralsinabriancurtin: did you merge latest trunk into it?19:57
ralsinathat part should not have changed at all19:57
briancurtinralsina: i started from trunk, merged mandel's branch19:57
nessitabriancurtin: branching and looking19:57
ralsinabriancurtin: he probably reverted a fix I did in main19:58
nessitabriancurtin: anyways, I'm seeing a LoopingCall there that worries me, since in the controlpanel we no longer have a reactor (on linux, we do on windows)19:58
ralsinanessita: the autoupdater stuff should be windows-only19:58
ralsinanessita: but it doesn't look well factored out19:59
nessitaralsina: yes, we could use a qtimer the same, no?19:59
ralsinanessita: indeed19:59
ralsinanessita: harder to test, maybe19:59
nessitaralsina: also, we agreed with mandel that the strings will not be marked for translations, and I see this19:59
nessitaUPDATES_TITLE = _('Updates')19:59
nessitaUPDATES_MESSAGE = _('There is a new update available')19:59
nessitaperhaps mandel did not push everything?19:59
ralsinanessita: in which case we are out of luck20:00
nessitaralsina: and the branch is creating a LoopingCall in multiplatform code!20:00
ralsinanessita: right20:00
briancurtinon linux the LoopingCall is in ubuntuone\controlpanel\gui\qt\task\linux.py, on Windows it uses twisted (if that's what you were talking about earlier)20:01
nessitaralsina: and there's new code in the systray test file? what does have to do autoupdate with systray?20:01
ralsinanessita: when called with some options, u1cp's window is not instantiated20:01
ralsinanessita: also, that's where notifications should come from20:02
ralsinanessita: as in "balloons"20:02
nessitaralsina: but this is not the code and MO we had in the former windows-installer, right?20:02
nessitaralsina: I thought that  we were just moving what we had in installer to controlpanel20:02
ralsinanessita: no, but that was for checking updates n startup only20:02
nessitaralsina: or did I misunderstood?20:03
ralsinanessita: this is about having regular checks for updates. I don't know what you talked with mandel, but that's what he told me20:03
nessitaralsina: on the call yesterday... what we agreed about this?20:03
ralsinanessita: that there was no autoupdate code, and that we would add it20:04
ralsinanessita: the specifics of which one of the two ways to autoupdate were not mentioned20:04
ralsinanessita: and I guess everyone assumed what he wated20:04
nessitaralsina: ...moving what we had in the windows-installer to controlpanel, that's what I recall. Perhaps I just thought that?20:04
ralsinanessita: maybe20:05
ralsinanessita: we could do the "just on startup" for this release20:05
ralsinanessita: and kick the looping check for the next one20:05
nessitaralsina: that was my idea all this tume20:05
nessitaralsina: given the time constraints20:05
ralsinanessita: ok, so miscommunication20:05
ralsinanessita: right20:05
ralsinanessita: when mandel spoke about his branch he was talking about this, he never had a branch for the "on startup" checl20:05
nessitaralsina: I can see 2 options:20:06
mandelralsina, nessita ein?20:06
nessita* ask briancurtin to make a brand new branch with the code that checks only at startup (moving from installer to controlpanel)20:07
nessitamandel: one sec20:07
mandelralsina, if the looping call is run as on launch, it does the check on startup which is when the app is started, later it will be done once every 24 hours20:07
nessita* wait for this branch to land next week, and to fully test it, and make another stable-3-0 update after the update for releases, making that an update for
ralsinano, let's go with the first option20:08
nessitaralsina: the last option means we don't ship this code in the tags and updates we're doing on Tuesday, but I can make another stable-3-0 update after Tuesday20:08
ralsinasince the previous release had only on-startup checks, it's not a regression if this doesn't go in20:09
nessitamandel: apparently there was some miscommunication about what each one of us meant with 'auto update' feature20:09
mandelnessita, ah..20:09
nessitamandel: I thought you were moving *just* the (working) code from installer to controlpanel20:09
nessitamandel: and I guess you understood you should finish the autoupdate "cool" branch20:09
mandelnessita, yes, that is what I though..20:09
nessitamandel: right, sorry for not being more explicit20:10
mandelnessita, well, sorry for not understading.. my brain 'subcsociente' surely wanted to do the 'cool' one20:10
nessitamandel: anyways, now brian can't make the test suite pass, and I see the diff and is  very very large to land in a rush (it may break Ubuntu "very easily")20:10
mandelnessita, ok, makes sense, I can do more testing on monday with IRL etc.. or I can get a branch with a single one, as you wish20:11
nessitamandel: so, the safest path I guess is just adding the windows isolated code (from installer). But we can ask briancurtin to do that :-)20:11
nessitamandel: so, enjoy the weekend, you can keep fixing this branch next weel20:12
briancurtinnessita: so what should i be adding?20:12
mandelnessita, indeed, is saver to just move the working code20:12
nessitamandel: but please do not land it until we agree "is safe" to do so20:12
mandelbriancurtin, nessita , sorry for 'el marron'20:12
nessitamandel: is ok20:12
mandelnessita, I'll add you as a reviewer :)20:12
nessitabriancurtin: in the windows-installer project, you will find some code that runs the autoupdater at controlpanel startup, we should move that to the controlpanel20:13
mandelok, I'm off to watch tv, laters!20:13
nessitabriancurtin: let me point you to it20:13
nessitabriancurtin: ubuntuone_installer/gui/qt/utils/windows.py, methods are_updates_present, check_updates (you will need to do some magic to not need the 'gui' parameter), and perform_update20:15
nessitaralsina: that code needs to interact with the UI ^ :-/20:15
nessitaralsina: to confirm with the user he wants to upgrade20:16
nessitaralsina: I'm not sure that's doable today... what do you think?20:16
nessitabriancurtin: _get_update_path is already in controlpanel code, is the get_exe_path function20:16
briancurtinnessita: ack20:17
nessitabriancurtin: anyways, I'm not that sure that's doable in a couple of hours20:17
briancurtinnessita: i'm also the wrong person to be rushing GUI-specific stuff :/20:18
nessitabriancurtin: indeed20:18
nessita(at least for now ;-))20:18
briancurtini'll get better :)20:18
nessitaralsina: what were the plans about the windows release for next week? regarding you being on vacations, was any plan for anyone else to move that forward?20:19
nessitabriancurtin: perhaps we killed ralsina with this talk?20:21
nessitaralsina: you alive? :-D20:21
* alecu wishes we have at least *one* windows release for internal testing.20:21
alecubefore we start spreading it around.20:21
alecuwe don't need signing for that, right?20:22
nessitaalecu: I'm pretty sure we can have one without autoupdate code20:22
nessitabriancurtin: am I lying?20:22
alecunessita, that would be awesome. But if we get to test autoupdate, it would be even better.20:22
briancurtinnessita: we have one that i sent out last week or so20:22
nessitaalecu: have any suggestion to solve this autoupdater issue?20:23
briancurtinnessita: oh actually we said don't use that one20:23
briancurtini can create an installer from trunk in a few minutes if we want to send it out20:23
nessitaalecu: ^20:23
alecubriancurtin, that would be awesome.20:24
alecunessita, no ideas on the autoupdater, no.20:24
ralsinanessita: sorry, was afk20:24
alecunessita, I've not seen that code in depth. Nor in surface.20:24
ralsinalet's do a test build on monday, brian knows how20:25
nessitaalecu: I have in surface, rough estimate to move it and having working, I'd say one day20:25
nessitaralsina: and what about the other things I asked? :-)20:25
ralsinathen I can coordinate the RT moving forward on tuesday if the tet build is ok20:25
nessitaralsina: about autoupdater20:25
nessitaralsina: we have no autoupdater, and apparently we will no have one for MOn20:26
ralsinaabout autoupdater? I said let's do the on-startup now and move the other until next release20:26
nessitaralsina: right, and I said:20:26
ralsinanessita: the autoupdater on startup is not that hard. I can do that and propose tonight20:26
nessita(05:15:57 PM) nessita: ralsina: that code needs to interact with the UI ^ :-/20:26
nessita(05:16:11 PM) nessita: ralsina: to confirm with the user he wants to upgrade20:26
nessita(05:16:26 PM) nessita: ralsina: I'm not sure that's doable today... what do you think?20:26
nessitaralsina: is complex as in the code is outside the qt/ dir, and the autoupdater requires an UI20:27
nessitaralsina: how would you solve that?20:27
ralsinanessita: what code is outside qt/ dir? sorry if I am dense, but I don't remember20:27
nessitaralsina: the whole utils/ dir20:27
nessitawhere we have all the windows specific20:27
ralsinalet me think 1'20:28
nessitaralsina: so the UI needs to check for updates, if there are some, present the user a confirmation dialog, if the user agreeds, the UI should call the thing that actually do the autoupdate20:28
ralsinawe could just launch the autoupdater on interactive mode20:28
ralsinabut that needs admin access. Sigh20:28
nessitaralsina: and the other options does not need it?20:29
ralsinanessita: no, just checking not-interactively doesn't20:29
nessitaralsina: but doing the actual install it does, right?20:29
ralsinayou are only asked for creds after you agree to update20:29
ralsinawhich is not annoying20:29
ralsinawhy not take that from utils and put it in qt/ with a noop if platform is linux?20:30
ralsinawe want no platform-specific code in qt at all?20:30
nessitaralsina: only that or the whole utils dir?20:30
ralsinanessita: only the autoupdater20:30
ralsinaand then, when we want to do the looping call it20:31
ralsina's just a qtimer triggering that same thing20:31
nessitaralsina: bah, not sure why I asked, I'm -1 to either... we're risking breaking things20:31
nessitaa new python module will break freezes20:31
ralsinanessita: grmbl, but that means we never get to do a windows release again20:31
ralsinayou understand that?20:31
nessitaeven if in linux is a no-op (but the module has to be installed, etc)20:31
nessitaralsina: what about the logic I proposed?20:32
nessita"the UI needs to check for updates, if there are some, present the user a confirmation dialog, if the user agrees, the UI should call the thing that actually do the autoupdate"20:32
ralsinanessita: the only way to check for updates is calling autoupdate.exe (or however it's called) and checking exit code20:32
ralsinanessita: that *is* how it works20:32
nessitaralsina: and what's the problem with that?20:33
nessitasorry if it's obvious, I just don t see it20:33
ralsinanessita: I don't follow you. mumble?20:33
nessitabriancurtin: wanna join us?20:33
briancurtinnessita: need a few minutes, waiting 2' for the microwave20:34
nessitasure, let me know20:34
briancurtinnessita: logging in now20:38
gatoxeod here..... have a nice weekend everyone!20:44
alecuok, guys, gal: have a great weekend.20:58
alecusee you some on tuesday.20:59
alecuralsina, have a great vacations.20:59
alecuthis is an early EOW for me, since I stayed up late yesterday fixing a few bugs... :-)20:59
ralsinaalecu: same for you!21:01
dobeyhave a good weekend/holiday/whatever all21:06
nessitaok, I'm of21:34
nessitabye all!21:34
ralsiname too. BYe!21:34
briancurtin2installer ready - http://ubuntuone.com/77HUgqLg7CbrISFAgclG1F - email sent21:39
briancurtin2and im out of here as well. i need a beer.21:40

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