/srv/irclogs.ubuntu.com/2012/03/09/#ubuntuone.txt

dfNullanyone up to speed on the sso API for U1. I have it mostly working buy have some questions about the tokens and how revocation might work for the user03:08
=== karni is now known as karni-holiday
karni-holidaydfNull: You still here?03:37
dfNullyeah, Hi03:38
karni-holidaydfNull: If you are, I could probably help03:38
karni-holidaydfNull: What language are you using?03:38
dfNullwe can do this later...youre supposed to be away03:38
karni-holidaydfNull: I'm about to get some sleep ;) Come on :)03:38
dfNulljava....you sent me your REST api stuff earlier in the week03:38
karni-holidayah cool03:39
karni-holidaydfNull: If you use a revoked token, The U1FileAPI will throw Unauthorized Failure (failure with getStatusCode() == HttpStatus.SC_UNAUTHORIZED)03:39
dfNullI've got auth tokens working but am puzzled why I have to use "Ubuntu One" and the host name for the app token?03:40
dfNullwhen I use that it works03:40
dfNullbut using anything else fails03:40
karni-holidaydfNull: It's because when you do sso.pingUbuntuOne(), Ubuntu One pulls tokens from sso, and filters them by "Ubuntu One @ " string pattern03:40
karni-holidaydfNull: If they match, the token is saved to U1 token cache.03:40
dfNullbut then when I look in U1 web UI under my account I see my android device listed again, since I'm also running the U1 client from youguys03:40
karni-holidayOr at least, that's how you should think about it.03:41
karni-holidaydfNull: If you're asking about the token, you need to use it in the format of "Ubuntu One @ <your stuff here>"03:41
karni-holidayNormaly it should be the host name/device name.03:42
karni-holidayBut you could put your application name in there.03:42
karni-holidaydfNull: Did that answer your question?03:42
dfNullhmmm...well I tried that with something other than androids Build.Device (hostname equiv) and your REST api wrapper fails03:42
dfNullfails auth at least03:43
dfNullyeah, that at least lets me know I should be able to vary the host name part03:43
karni-holidaydfNull: HAve you looked at lp:~karni/ubuntuone-files-java-library/cli-ubuntuone ?03:43
dfNulland I can work with that03:43
karni-holidayhttps://code.launchpad.net/~karni/ubuntuone-files-java-library/cli-ubuntuone03:43
karni-holidaydfNull: There's a line saying: final AuthenticateResponse response = sso.authenticate("Ubuntu One @ CliUbuntuOne");03:44
karni-holidayIt could also be "Ubuntu One @ Cli U1 whatever_you-like" what not :)03:44
dfNullOk, I will look at that. was looking at03:45
dfNullhttps://code.launchpad.net/~ubuntuone-android-hackers/ubuntuone-files-java-library/trunk2.x03:45
dfNullmost of which is broken, but got me going03:45
karni-holidaydfNull: Broken?03:45
karni-holidaydfNull: https://code.launchpad.net/~ubuntuone-android-hackers/ubuntuone-files-java-library/trunk0.x03:46
dfNullmuch of the command line stuff is commented out03:46
karni-holidaydfNull: cli-ubuntuone (the link I provided a few lines above) is a fully functional command line client03:46
dfNullthe uri for listing volumes has an extra / at the end, etc.03:46
dfNullok, I will look at that one03:47
karni-holidaydfNull: It has been extracted to a separate project from trunk2.x (which is now trunk0.x)03:47
dfNullI'm in a good place now know that I should be able to vary the hostname part03:47
dfNullwhich then lets someone revoke access via the web UI03:47
dfNullyeah, I noticed that the original one you sent was 40403:47
dfNullthanks....hope youre away for work and not away on holiday doing IRC03:48
karni-holidaydfNull: "the uro for listing volumes nas an extra / at the end" - I don't know what you're talking about :<03:49
karni-holidaydfNull: API uri is https://one.ubuntu.com/api/file_storage/v1/volumes03:49
karni-holidaydfNull: And this is what getVolumes is calling: http://bazaar.launchpad.net/~ubuntuone-android-hackers/ubuntuone-files-java-library/trunk0.x/view/head:/main/src/com/ubuntuone/api/files/U1FileAPI.java03:49
karni-holidayline 40903:49
karni-holidaydfNull: Well, I started holiday yesterday, but I'm very happy to answer questions related to SSO and U1FileAPI :)03:50
dfNullthe one at trunk2.0x had extra trailing slash..IIRC03:50
karni-holidaydfNull: Do not hesitate to leave me priv messages or e-mail me at michal.karnicki at canonical dot com, so I can follow up :)03:50
dfNullhad to grab source and fork to make getVolumes to work03:51
karni-holidaydfNull: I've rearranged things yesterday. the trunk2.x is now trunk0.x, so it's same code.03:51
karni-holidayhm o_O03:51
karni-holidaythis is the last part of the uri: private static final String VOLUMES = "volumes";03:51
karni-holidayI see no trailing slash (and there was none, if I am not mistaken. I can be mistaken :) ).03:52
karni-holidaydfNull: Anyhow, this should be working :) You can bzr branch the project trunk at lp:ubuntuone-files-java-library - it's most up to date03:52
karni-holidayand points to trunk0.x (tech preview of the library)03:52
karni-holidayIt'll become 1.0.0 as soon as we believe is stable and has no further critical feedback from developers.03:53
dfNullhmmm...all I can say is the source I had wouldnt give me volumes until I changed final URI uri = new URI(HTTPS, META_HOST, API_PATH + "/volumes", null);03:53
karni-holidaydfNull: Have fun and if you find any problems, please do file bugs on the project page :)03:53
dfNullyeah, mercurial and git guy but time to try bazaar I guess03:54
karni-holidayhmm03:54
karni-holidayah :)03:54
dfNullthanks and go away to enjoy holiday!03:54
karni-holidaydfNull: Haha, my pleasure! Keep sending your feedback :) Cheers!!03:54
karni-holidaydfNull: holly shoot, wait a second. haven't you branched the old library version by any chance :/03:56
karni-holidayThere's no such code as you just wrote :O03:56
karni-holidaydfNull: The old code is used by Ubuntu One Files on Android (so it's production code), but it wasn't near enough good to become public (as in, official)03:57
karni-holidaydfNull: Please make sure you are using U1FileAPI class to talk to U1, if not - ouch ouch, please branch lp:ubuntuone-files-java-library and sorry for all the confusion. We were rearranging things for the library release, and I think that bit you!03:58
dfNullhuh? I got stuff from: https://code.launchpad.net/~ubuntuone-android-hackers/ubuntuone-files-java-library/trunk2.x03:58
karni-holidayHmm. In that case, you got the right code. Do you remember where did you change the URI uri = new URI(HTTPS, META_HOST, API_ATH + .... ?03:58
karni-holidayWhich file was it?03:58
dfNullwhich is now gone.03:58
karni-holidayResourceClient.request(..) line final URI uri = new URI(getHost().getSchemeName(), getHost().getHostName(), path, query, null); ?03:59
dfNullRestApi.java03:59
karni-holidaydfNull: omg. you haven't branched trunk2.x :<03:59
dfNullno, I copied local03:59
karni-holidayyou branched lp:ubuntuone-java-files-client few days ago, instead lp:ubuntuone-java-files-client/2.x03:59
dfNullreally?04:00
karni-holidaydfNull: Ok, any how - I am terribly sorry. You are looking at the old, un-official code :<04:00
dfNullhmmm04:00
karni-holidaydfNull: The new one is SO MUCH BETTER :)04:00
dfNullwell..I got it working mostly, nonetheless04:00
karni-holidaydfNull: I am so sorry. The old library is written quite ad-hoc, and is not that pleasant to work with (static calls with multitue of parameters)04:01
karni-holidaydfNull: I'll say this one last time - I am sorry, this was a misunderstanding. You've branched the old library - which is, well, production code, so it's not bad, but the new one is waaay better.04:01
dfNullyeah, well I've seen worse. Hadnt used signpost before and that was handy04:01
karni-holidaydfNull: :)04:01
karni-holidaydfNull: If you have time and nerve to change your code, you could upgrade to the tech preview of the new version04:02
karni-holidaydfNull: Notably, U1FileAPI class instead of RestApi class04:02
karni-holiday(Just mentioning a key point that differs the libraries. Everything has been written from scratch.)04:02
dfNullyah the jar I have is ubuntuone-files-java-client-rev304:02
karni-holidaydfNull: Right, that's the old one. Have a look at https://launchpad.net/ubuntuone-files-java-library04:03
dfNullfrom which I had to get source for RestApi.java and make my own version to fix the volumes uri, etc04:03
karni-holidaydfNull: The jar is now libUbuntuOneFiles-0.1.0-rc.1.jar04:03
karni-holidaydfNull: Well, you don't need to get source from the jar, there's source on launchpad, right :)04:03
karni-holidayShoot, 3.5 hours of sleep and then hours on the bus. I should be going now..04:04
karni-holidaydfNull: You've got nerve to have used RestApi, I'm actually pleased someone other than me (and Ubuntu One Files for Android) has used that code :)04:04
dfNullyeah, that what I meant. started with jar and went to get source from bzr for RestApi.java04:04
karni-holidayah cool04:05
karni-holidayThe jar didn't naturally contain the sources, only compiled classes.04:05
dfNullor javadoc in this case :/04:05
karni-holidayno java doc? uhh that's a bug04:05
dfNullgo away and thank you!04:06
dfNullHOLIDAY!04:06
karni-holidaydfNull: I hope you will consider upgrading to the new version, it's written way better and the source contains java doc. it also has the cli-ubuntuone example ^_^04:06
karni-holidayhaha, take care dfNull !04:06
karni-holidayCatch you in 1.5 week04:06
* karni-holiday disappears04:06
dfNullI will use new and hope to have something to keep our keepass files safe when u return04:07
karni-holidayRock on, dfNull !04:07
karni-holiday:)04:07
=== m_conley is now known as m_conley_away
JamesTaitHappy Friday - wait, what, again? Already? ;)09:22
dfNullI think time dilates as you age!09:25
dfNullYay! got a simple POC going on Android using the new ubuntuone-files-java-library-rc.109:28
mandelmorning all09:44
dfNullmorning!09:46
dfNullnice work on ubuntuone-files-java-library-rc.1! thanks! I've got enough of an android client working now that I can sleep tonight. have fun!09:50
mandeldfNull, laters!09:51
* mandel rebooting09:53
=== webmaster is now known as davidcalle
ralsinagood morning!10:39
ralsinamandel, can I get a simple review? https://code.launchpad.net/~ralsina/ubuntuone-control-panel/go-native10:39
ralsinamandel: and maybe a not-so-simple one: https://code.launchpad.net/~ralsina/ubuntuone-control-panel/tab-tab-tab/+merge/9668710:40
mandelralsina, sure, let me finish first with the ones from alecu10:42
ralsinamandel: cool, those are more urget10:51
mandelralsina, you mean the proxy ones, right?10:51
ralsinamandel: yes10:51
gatoxmorning!10:52
mandelgatox, buenas, feeling any better?11:01
gatoxmandel, yep.... taking some pills and a special diet :P11:01
ralsinagood morning gatox11:01
gatoxralsina, hi11:01
mandelgatox, special diet? do you eat babies now?11:02
gatoxralsina, i'm reading nessita's email.... could you fix the button thing?11:02
gatoxyep..... i'm seeing the screensshots right now11:03
gatoxralsina, who have the idea of changing the top and bottom padding on focus?11:17
mandelyay! all of alecus reviews done \o/11:35
mandelralsina, moving to yours11:35
mandelralsina, for https://code.launchpad.net/~ralsina/ubuntuone-control-panel/go-native/+merge/96123 should we add tests for that?11:37
nessitabuenas!12:19
gatoxnessita, hola12:19
gatoxnessita, i answer to your email12:19
nessitahi gatox!12:19
nessitagatox: how are you?12:19
gatoxnessita, better12:19
nessitagatox: you *sure*? :-)12:20
gatoxnessita, yes.... the doctor gave me some pills, special diet, and drink a lot of water because i was dehydrated because of the fever.... :P so, i was really sick jeje12:20
nessitagatox: and how many rest days did he give you? ;-)12:21
gatoxnessita, i feel silly not thinking about reducing the highlighted area during focus.... grrrrrr12:21
nessitagatox: is ok, I did not come up with that, ralsina did12:21
gatoxnessita, well..... they always gave two.....  but that is for people who HAS to go to work.... i'm feeling better today... and i have all the weekend to rest :D12:22
gatoxreally!12:22
mandelnessita, morning! one question, landing in trunk, how does that work? I mean, if a branch is approved, when can we set it to be merged etc..12:22
nessitamandel: hola! yes, releases were made on Wed, so go ahead12:23
mandelnessita, also, we need some reviews for alecu, all the ones related with https://bugs.launchpad.net/ubuntuone-client/+bug/92920712:23
ubot5`Ubuntu bug 929207 in Ubuntu One Client "[FFE] Proxy "tunnel" for syncdaemon" [High,In progress]12:23
mandelnessita, super! gracias :)12:23
nessitamandel: yes, will continue reviews today12:23
mandelnessita, great, I have done all of them with some comments and a couple of very simple need fixing, if we can land this today it would be great since QA will have time to test it before beta212:24
nessitayes12:24
nessitamandel: do you know if qa has env set with proxy?12:25
mandelnessita, I'll be sending an email to them on how to at least set a very simple squid3 on P12:25
mandelnessita, with a sample configuration for auth and non-auth ports12:25
nessitamandel: ack, would you please cc me?12:26
mandelnessita, sure!12:26
mandelnessita, actually, I'll write it now so I don't forget12:26
ralsinagood morning again! (for real this time)12:28
ralsinamandel: there are tests, it checks that it gets the constant as argument12:29
ralsinagatox: that's why I am the manager, I know all the dirty hacks12:29
gatoxralsina, jeje groso12:30
ralsinanessita: no, they don't have a proxy setup but a squid+auth is very easy to configure on linux12:30
ralsinain fact, I have a hack that would let us have a proxy where you auth with your canonical credentials in a public IP! Hey, I will do that now :-)12:31
mandelralsina, ok, I'll do an irl and will add my +112:31
ralsinagatox: not for today, but this is going to totally own your weekend: from PyQt4 import QtDesigner12:34
gatoxralsina, do you  mean for ninja?12:35
ralsinagatox: yep12:35
ralsinagatox: gives you access to all of Designer to integrate in apps12:35
ralsinagatox: apparently, I have been told, have not tried it ;-)12:35
mandelgatox, you should do the same with ninja12:36
gatoxralsina, we have been looking at that (i don't know if they updated that) but that was reallyyyyyy limited.... because the actual useful things of the qt designer part were private in qt, so riverbank didn't do the wrap12:36
mandelgatox, and then import ninja in ninja!12:36
ralsinagatox: ok, then, look for pynoto, they have a hack using private headers ;-)12:36
gatoxralsina, cool! thanks! :D12:36
gatoxralsina, i totally going to take a look at that!12:37
mandelralsina, gatox, nessita do you sue evolution for mail? I get errors with the canonical smtp server :(12:46
gatoxmandel, thunderbird12:47
nessitamandel: thunderbird12:47
ralsinamandel: thunderbird12:47
ralsinahahaha12:47
ralsinamandel: conclusion: evolution sux0rz12:48
mandelis it meant to appear now next to me, like beatlejuice?12:48
mandelralsina, a little, just a little :P12:48
nessitaralsina, gatox: could you please review? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/focus/+merge/9665412:50
gatoxnessita, yep, on  it12:50
ralsinamandel, nessita, gatox, elopio, rmcbride: I have setup a public authenticated proxy. When you want to try using proxy support, if you don't want to setup your own squids, ping me ;-)12:54
gatoxralsina, ack12:54
ralsinanessita: sure12:54
nessitaralsina: nice!12:54
ralsinanessita: the explore button was already disabled. It just didn't *look* very disabled12:55
nessitaralsina: exactly, it was cause by the style being defined for "every state" instead for :enabled12:55
ralsinanessita: ack but then maybe the changelog should be changed a bit?12:56
ralsinanessita: or not, I don't really mind12:56
nessitawill edit12:56
ralsinait's casual friday I won't have strong opinions until monday12:57
nessitaadded (style-wise) clarification12:57
ralsinanessita: good idea to start using declarations for colors12:57
nessitaralsina: can we do that?12:58
nessitaralsina: I just added a comment12:58
ralsinanessita: I think so, let me check12:58
nessitabut not sure if "variables" can be defined12:58
ralsinawe could pre-process with cpp at least12:58
nessitaralsina: you sure is worth it?12:59
ralsinanessita: well, right now we have ... 78 color declarations12:59
ralsinanessita: if I had a code file with 78 color literals, what would you say? ;-)13:00
nessitaralsina: 78 colors but the uniq of that is much less, no?13:00
ralsinaok, no, it's only 54 colors13:00
ralsinanessita: probably the uniq is much fewer, which is more reason to unify in constants somehow13:01
gatoxralsina, nessita maybe i'm wrong.... but the qss can use things from the resources...... so "maybe" (big maybe).... we can define that colors constants in the qrc13:01
ralsinagatox: that's an idea. It doesn't have syntax for variables or constants, AFAIK, let me dig resources13:01
nessitaralsina: I'm not sure how much is worth the effort, but will not block any work for that... as long as the solution is not more complex that adding a color in a file and using it :-)13:02
nessitaralsina: can we talk a little about the go-native branch?13:02
ralsinanessita: of course, no blocking anything13:02
ralsinanessita: I am just filing it in my "think in the background" folder13:02
ralsinanessita: sure, let's13:02
ralsinanessita: mumble or here13:02
nessitahere13:02
nessitaralsina: so, I was cool with having qt in utils, but then I realized that it will break the ubuntu packaging of the project, let me explain further13:03
nessitaralsina: deja-dup depends on the backend of the cp which has utils, so deja-dup pulls the backend of the control panel in the CD13:03
nessitaralsina: so our backend can not pull qt :-/13:03
ralsinanessita: ack13:04
ralsinanessita: ok, I can just add the constants in folders.py13:04
nessitaralsina: sorry I did not realized this sooner13:04
ralsinanessita: or create a platform-speciic module in gui/qt13:04
ralsinanessita: no problem, it smelled bad to me too13:04
nessitaralsina: could we please have an utils inside gui/qt/ that can be multiplatform13:04
ralsinanessita: one small argument for putting the constant in folders: this is temporary because it's a workaround for a Qt bug13:05
nessitaralsina: do we know if the bugs was already fixed?13:05
ralsinanessita: AFAIK, it's meant to be fixed in Qt 4.913:05
nessitaok13:05
nessitaralsina: not sure what's your suggestion about the fix then :-)13:06
ralsinanessita: I would rather put it in folders.py with a big note about removing it when qt 4.9 arrives13:06
nessitaralsina: works for me, as long as we don't keep adding the same in other modules (ie as long as we don't add more if platform == win in other modules)13:07
ralsinanessita: no, it's just a one-time thing13:07
ralsinanessita: I can put a "don't do this" in the comment, too13:07
nessitaralsina: please13:08
ralsinanessita: cool,fixing then13:08
nessitadobey: when you come in, I wanted to mention I hit this yesterday on an updated precise bug #85306013:14
ubot5`Launchpad bug 853060 in ubuntuone-installer (Ubuntu) "ubuntuone-installer crashed with GError in function(): Failed to execute child process "ubuntuone-control-panel-gtk" (No such file or directory)" [High,Incomplete] https://launchpad.net/bugs/85306013:14
nessitadobey: of course the failure was regarding the -qt control panel13:15
nessitadobey: installer version was 2.99.9013:15
gatoxnessita, +1 to your branch13:22
nessitagatox: thanks!13:22
ralsinanessita: updated go-native as we discussed13:26
nessitaralsina: ack13:26
nessitaapproved alerted13:26
ralsina nessita: thanks!13:27
alecuhello!13:27
gatoxalecu, hi13:27
alecuhi gatox! feeling better?13:28
gatoxalecu, yep, thanks :D13:28
nessitahola alecu13:28
mandelalecu, morning!13:33
alecuhola mandel! thanks for all the reviews13:33
mandelalecu, I did all the reviews, there are some comments that you might or might no follow and a small lint error in one of them13:34
mandelalecu, all should be very easy to fix :)13:34
alecumandel, I'm fixing those so you can re-review in a few minutes.13:34
alecugreat!13:34
mandelalecu, superb! I'll microwave some pasta and will do them asap13:34
* mandel lunch13:34
nessitaralsina: you wanna remove the code in ubuntuone/controlpanel/utils/__init__.py now, in go-native?13:41
ralsinanessita: ugh, sorry13:42
ralsinanessita: I am getting a bunch of fails on tests now, will ping you when that branch is good again13:44
nessitaralsina: thanks13:49
ralsinasigh, I have been doing much harder stuff, why is this stupid "change one constant" branch driving me nuts13:55
dobeynessita: what was the actual full traceback?13:57
nessitadobey: the exact same traceback as the bug report changing ubuntuone-control-panel-gtk by buntuone-control-panel-qt13:58
nessitadobey: and also, after the crash, the qt control panel was not installed, so seems like the spawning was made too early13:58
nessitadobey: I had to install it by hand, looked like no package was installed at all13:59
ralsinaand as usual, the answer is, because sometimes I am dumb14:00
dobeynessita: well i guess it's not easy to debug now since you installed the package by hand14:01
nessitadobey: well, I needed it :-/14:01
nessitaalecu: approved with a fix request https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-client/+merge/9507714:09
alecunessita, great! thanks.14:10
nessitaralsina: any update in my focus branch review?14:10
ralsinanessita: yes, running tests now14:10
nessitaralsina: oh neat!14:10
ralsinanessita: works nice IRL, and code review14:10
ralsinanessita: +114:11
nessitaralsina: thanks! wanna merge that in your branch, so I then review it? (referring to tab-tab-tab)14:12
ralsinanessita: sure, will do that now. Here is the much smaller go-native14:12
nessitawhere? I can't see it! :-P14:12
ralsinanessita: https://code.launchpad.net/~ralsina/ubuntuone-control-panel/go-native/+merge/9612314:12
nessita(thta's small)14:12
ralsinanessita: I feel stupid about that branch, I hope now it's simple enough that it can be correct ;-)14:13
ralsinanessita: I'll wait for focus to merge and then will merge trunk14:13
nessitasure14:13
nessitaralsina: looks good, running tests and about to test IRL14:14
gatoxralsina, LOL.... love the comments here: https://code.launchpad.net/~ralsina/ubuntuone-control-panel/go-native/+merge/9612314:15
nessitame too :-P14:15
ralsinagatox: if someone did that and suffered the wrath of the nessita because I failed to warn him, I would feel bad for 10 minutes at least.14:15
gatoxjejeej hey! you have a typo :P14:15
gatoxtemporary14:16
ralsinaNOOOOOOOOOOO THAT BRANCH IS CURSED!14:16
gatoxjejeje14:16
nessitaralsina: approved on this end14:16
gatoxi only realize because it was so funny that i read it twice14:16
gatoxjeje14:16
ralsinagatox: pushed "temporary"14:16
gatox:P14:16
nessitaralsina: focus is merged now14:17
nessitadobey: trying to reproduce in my clean french precise vm14:18
nessitagatox: speaking of which, in that ^ vm, the "forgot password" is unclickable14:19
nessitagatox: let me file a bug for you :-)14:19
gatoxnessita, ok14:19
nessitadobey: ugh, tarmac is requesting daily build for all the ubuntus for u1cp14:20
gatoxnessita, when you have a moment please re-review this one: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/too-verbose/+merge/96364 and let me know if those loggings are correct14:21
dobeynessita: ok, will fix14:21
nessitagatox: ack!14:21
ralsinanessita: tab-tab-tab ready for you14:26
nessitaralsina: thanks!14:27
* gatox needs to resolve conflicts in a .ui file.... :S14:28
ralsinagatox: ouch14:29
gatoxyap14:29
nessitagatox: oh... reset password?14:29
gatoxnessita, aja14:29
nessitaperdón14:29
nessitaneed help?14:29
gatoxnessita, no... i was reviewing my branch to see if it was ok, and i found that....14:29
gatoxi'll let you know when it's ready14:30
* mandel back14:34
mandelI think I ate too much..14:34
mandelalecu, please let me know when ever you need a second review14:37
nessitamandel: can you squeeze a fix if any ussoc branch of yours?14:38
nessitamandel: in ssl_dialog, there is a docstring with 4 "14:38
nessita     65 class SSLDialog(QDialog):14:38
nessita     66     """"Dialog used to show SSL exceptions."""14:38
nessitamandel: and that's busting the syntax highlight :-)14:39
mandelnessita, the other branches have been merged, but I certainly can add it in the next one14:39
nessitamandel: thanks14:39
mandelnessita, sorry for that, vim does not go crazy if that happens..14:40
nessitamandel: it should :-P14:40
mandelnessita, it would be helpful to avoid this things, that is true14:40
dobeyi guess i need to figure out how to set up/use some vms :(14:44
dobeynessita: hrmm, why can't i remove the other devices i have connected, in the qt panel?14:45
nessitadobey: design decision14:45
dobeyGRRRRRRRRRR14:45
nessitadobey: I don't like it either14:45
dobeythat is so broken14:46
nessitadobey: but they say is kinda a "security" measure, if someone different from you access the controlpanel in a single devicew14:46
dobeynessita: right, so they click on the big orange button, and it loads up the web page, and they click remove on there instead14:46
nessitaright (?)14:46
nessitadobey: adding the remove button in the remote devices is trivial, wanna talk with design? :-)14:47
dobeyno. my doctor says i need to keep my blood pressure down14:47
ralsinaI don't eve like having the "remove *this* device" button since it leads 50% of the time to rot_mismathc14:48
nessitadobey: ah, ok. Mine says I need to keep my gastric juice down :-D14:48
nessitagatox: too-verbose comment added, branch is looking great, we really need this done (I know is boring though)14:48
dobeynessita: also, when i move the mouse over the tabs, it *feels* like it should switch the cursor to the pointy-finger url cursor instead14:49
gatoxnessita, ok, i'll review it as soon as i finish with the ui thing14:49
nessitadobey: because they get underlined?14:49
dobeynessita: and the tabs light up. if feels like it's a web page.14:49
dobeynessita: there is a bug about the underlining already, right?14:50
nessitadobey: yes, but as far as I know we are not changing that... sorry14:51
nessitadobey: I honestly don't have a strong opinion about that, I don't like underline at all there, but we need to have the same UI across OSes... so14:51
nessitadobey: if you feel strong about this, I welcome you to talk with design, I will build the branch14:51
ralsinagotta go, school run then lunch, so back in about 80 minutes, I suppose14:51
dobeywell we can remove it on windows too14:51
nessitadobey: yes14:52
dobeynessita: do you know the bug #?14:52
nessitadobey: sure14:52
nessitabug #93983914:52
ubot5`Launchpad bug 939839 in Ubuntu One Control Panel "Qt control panel shouldn't underline tab title on hover" [Undecided,Triaged] https://launchpad.net/bugs/93983914:52
dobeyyay bug bot14:52
nessitadobey: let me know if you get new info on this14:52
dobeyi just commented on the bug14:56
alecumandel, "use sys.exit in main specially in the case were there is no proxy" <- why?14:59
alecumandel, the main function is already returning in that case14:59
mandelalecu, to state that the process did not have a normal termination or something like that14:59
nessitame14:59
mandelalecu, it seems logical to say, hey we finished and did nothing because there is no proxy14:59
mandelme14:59
alecumandel, but it's perfectly ok to exit if the settings say "no proxy"15:00
alecumandel, so that case is a normal termination15:00
mandelalecu,  if is normal termination ok, I can change my vote :)15:00
alecumandel, the bits that start that process (in the following branch) already handle all those cases15:00
alecumandel, ok, great.15:00
nessitagatox, dobey, ralsina, alecu, urbanape, me?15:01
gatoxnessita, ok, the ugly reset one is ready15:01
gatoxme15:01
dobeynessita: what about you?15:01
mandelalecu, ok, I did not review that taking into account the following ones15:01
nessitadobey: I said me already :-)15:01
nessitadobey:  (11:59:50 AM) nessita: me15:01
dobeynessita: and just filed bug #950873 about device removal15:01
ubot5`Launchpad bug 950873 in ubuntuone-control-panel (Ubuntu) "Cannot remove other devices from control panel" [Undecided,New] https://launchpad.net/bugs/95087315:01
alecumandel, oh, and regarding "return settings" vs "if settings: return True" <- I like it more to return a boolean instead of the dictionary.15:01
dobeymeh15:01
alecumandel, I think it's cleaner...15:01
nessitaalecu: in that case do:15:01
nessitaalecu: return len(settings) > 015:02
alecunessita, sounds fine.15:02
urbanapeme15:02
nessitaalecu: did you say me?15:02
alecume15:02
* nessita got lost with so many messages15:02
nessitaok, let's!15:02
nessitaDONE: did *not* start with bug #933697, instead tackled more reviews and bug #942020 and bug #94903515:02
nessitaTODO: more reviews and start with bug #933697 this time15:02
nessitaBLOCKED: nopes15:02
nessitaNEXT: mandel15:02
ubot5`Launchpad bug 933697 in Ubuntu One Control Panel trunk "[FFE] Integrate missing pages to Qt Control Panel" [High,Triaged] https://launchpad.net/bugs/93369715:02
ubot5`Launchpad bug 942020 in Ubuntu One Control Panel "The selected tab has a weird orange highlight" [Medium,Fix committed] https://launchpad.net/bugs/94202015:02
mandelDONE: Finished Qt webclient code. Reviews, reviews, reviews.15:02
mandelTODO: IRL tests for contorl panel, libsoup ssl.15:02
mandelBLOCKED: no15:02
mandelgatox, please15:02
ubot5`Launchpad bug 949035 in ubuntuone-control-panel (Ubuntu) "Explore buttons for folders not synced to the machine should be inactive" [Undecided,New] https://launchpad.net/bugs/94903515:02
gatoxDONE:15:03
gatoxTook a sick day, investigate the focus button problem, fixed the logging branch, resolve conflicts in the ugly-reset branch.15:03
gatoxTODO:15:03
gatoxKeep fixing bugs from my queue.15:03
gatoxBLOCKED:15:03
gatoxNo15:03
gatoxdobey, go15:03
dobeyλ DONE: finished packaging new rb, rb-u1 upload, twisted fix upload proposed15:03
dobeyλ TODO: installer15:03
dobeyλ BLCK: none.15:03
dobeyurbanape15:03
urbanapeDONE: pushed an initial branch (still needs cleanup)15:03
urbanapeTODO: more refinement, out for the afternoon, getting lasik exam15:03
urbanapeBLOCK: None15:03
urbanapealecu: go15:03
alecuDONE: worked on webclient tunnel branch #929212, reviews for mandel, fixes on the reviews15:03
alecuTODO: more fixes, finish #92921215:03
alecuBLOCKED: no, because mandel and nessita are helping with the reviews.15:03
* nessita will keep helping15:04
nessitaany comments anyone?>15:04
alecunop15:04
nessitaNOTE: starting next week, I will be leaving every Tuesday at noon ART, university classes start next weeks15:04
nessitanext week*15:04
dobeyno comments, only complaints ;)15:04
alecudobey, complain away!15:05
dobeyoh i will. i will.15:05
nessitaand next Tuesday in particular, I will be taking half od day off, so I will not be at all :-)15:05
nessitaok, I guess is eom then15:05
* nessita needs to buy some eggs15:05
nessitaeom!15:05
* nessita -> groceries hunt15:06
nessitabrb15:06
dobeypor medialunas?15:06
nessitadobey: nopes, to make an omelet :-)15:06
dobeymedialunas de eggs15:07
gatoxjeje15:07
* gatox lunch15:07
=== gatox is now known as gatox_lunch
mandelralsina, we have or 1-1 now, right15:08
mandel?15:08
mandelalecu, so, I have been getting stupid errors with some of the tests because libsoup is reading my proxy settings, we should do something about it, maybe something like the dbus test but a gsettings tests or something15:21
dobeymandel: ugh, it's reading the real settings?15:23
mandeldobey, yes, terrible..15:23
dobey:(15:23
dobeywe need to fix that15:23
mandeldobey, I was wtf for a while..15:23
* nessita is back15:28
dobeymandel: hrmm. i wonder if it's getting those values from the environment, rather than gsettings :)15:30
mandeldobey, I'm in the process of debugging the bloody thing..15:32
dobeymandel: export GSETTIGS_BACKEND=memory in the run-tests script15:34
mandeldobey, what does that do?15:34
dobeymandel: makes gsettings use an in-memory db, which will be initially empty. so basically, not break the tests15:35
mandeldobey, awesome!15:35
dobeymandel: if it doesn't work, or breaks in some other way, let me know :)15:35
ralsinamandel: 1:1 in about 45' is ok?15:41
ralsinamandel: I am about to have lunch15:41
mandelralsina, sure15:41
alecumandel, pushed fixes to *-process branch15:48
mandelalecu,  looking right now!15:48
=== JanC_ is now known as JanC
mandelalecu, can it be that the mock server cancels a request? I getting a canceled request with libsoup + ssl cert15:50
alecumandel, what? do you have a branch I can take a look at?15:51
mandelalecu, give me a sec and I'll pus it15:51
mandelalecu, lp:~mandel/ubuntu-sso-client/libsoup-ssl-dialog15:53
mandelalecu, try running: u1trial -r gi -t test_ssl_fail_dialog_user_accepts ubuntu_sso/utils/webclient/tests/test_webclient.py15:53
alecumandel, http://pastebin.ubuntu.com/876189/16:00
alecumandel, it looks like libsoup is saying that.16:00
mandelalecu, yes, but I wonder why, ssl-strict is set to false so it should be doing something besides a cancellation16:01
=== gatox_lunch is now known as gatox
alecumandel, where are you setting "ssl-strict" ?16:07
alecunevermind, found it16:07
mandelalecu, there :)16:07
mandelalecu, I also added a full no secure debugging with all the info16:08
mandelalecu, it seems that it does not event create the connection16:08
alecumandel, found it!16:11
alecumandel, _assert_ssl_fail_user_accepts returns a deferred16:11
alecumandel, and trial is not getting at that deferred.16:11
nessitaralsina: tabtabtab reviewed16:11
alecumandel, so: two options:16:12
alecumandel, have all 4 tests return the deferred returned by _assert_ssl.......16:12
alecumandel, or make each test a @defer.inlineCallback, and yield on the value returned by _assert_ssl....16:13
mandelalecu, let me try it out :)16:13
alecumandel, does it make sense? ^^^^16:13
mandelalecu, thx!16:13
alecuno prob... it was a hard one to see :-)16:14
gatoxneed to restart16:14
gatoxno need to restart anymore.... suddendly everything get fix :P16:15
mandelalecu, it fixes it!!! sweet I'll get the ssl dialog spawn in no time :)16:16
alecumandel, lovely!16:16
mandelalecu, FYI running the tests in the branch you updated16:16
alecumandel, btw: I've found this: http://valadoc.org/Soup-2.4.gir/Soup.Message.get_https_status.html16:16
alecumandel, while looking for the issue.16:16
alecumandel, perhaps you already think of using that method?16:17
mandelalecu, no I did not, specially because I don't seem yo find it in the c docs..16:18
* mandel looks16:18
alecumandel, I found it's present in the python object (I was looking at the dir() of that object, when it caught my eye)16:19
mandelalecu, in the soup session?16:20
alecuin the soup message16:20
alecumandel, here you go: http://developer.gnome.org/libsoup/stable/SoupMessage.html#soup-message-get-https-status16:20
mandelalecu, oh, in the message! yep that one I new about :)16:20
alecumandel, ok, cool.16:21
mandelalecu, +1 on https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-process/+merge/9621216:22
dobeyneed lunch, bbiab16:22
mandelalecu, did you fix the pylint issue for https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-useit/+merge/96213 ?16:22
alecumandel, I'm looking at it right now.16:23
alecumandel, I'll push in 5'16:23
ralsinamandel: 1:1 on mumble?16:23
mandelalecu, cool, let me know16:23
alecumake that 1016:23
mandelralsina, sure, let's do it!16:23
mandelralsina, can you hear me?16:25
ralsinamandel: you can't hear me?16:25
ralsinamandel: I can hear you16:25
mandelralsina, nop, let me restart, the same happened yesterday16:25
ralsinamandel: ok16:25
=== zyga is now known as zyga-afk
pmatulisnessita: re bug #944110, i'm trying to restart the client but it doesn't seem to work remotely16:34
ubot5`Launchpad bug 944110 in Ubuntu One Client "Permissions changed on unmodified files" [Wishlist,Confirmed] https://launchpad.net/bugs/94411016:34
pmatulisdbus.exceptions.DBusException: org.freedesktop.DBus.Error.NotSupported: Unable to autolaunch a dbus-daemon without a $DISPLAY for X1116:34
nessitapmatulis: hi there16:35
nessitapmatulis: yes, U1 is not supported on headless install or headless access, you need a dbus session running and without an X you don't get one16:35
pmatulisnessita: maybe i'll reboot my home 'puter then16:36
nessitapmatulis: is this to gather DEBUG logs?16:36
pmatulisnessita: yes, as per your instructions16:36
pmatulisnessita: i noticed on Home 'puter the syncdaemon.conf file was different from my Work 'puter16:36
nessitapmatulis: right, sorry :-/16:36
nessita(about the need of reboot)16:37
nessitapmatulis: note that syncdaemon will no be started unitl your login to your x session with your username16:37
pmatulisgah!16:37
pmatulisnessita: i have this on Home.  sounds like a default:16:38
pmatulis[__main__]16:38
pmatulisfiles_sync_enabled = True16:38
nessitapmatulis: yes, that's correct16:38
pmatulisk16:38
alecumandel, fixed lint issue, now pushing.16:39
alecumandel, it's pushed now.16:40
mandelalecu, ok, taking a look16:41
alecunessita, ralsina, gatox, dobey, thisfred, urbanape: plis, reviews?16:43
alecuhttps://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-process/+merge/9621216:43
alecuand https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-useit/+merge/9621316:43
thisfredsure16:43
nessitaalecu: I was waiting for you to fix the needs fixing from mandel16:43
gatoxalecu, i'll review the first one right now16:43
nessitathisfred: I'll take use-it16:43
alecunessita, great, thanks.16:43
nessitagatox: I think between thisfred and I got those 216:43
gatoxnessita, ok16:44
nessitagatox: but thanks!16:44
thisfrednessita: k, I'll do the first16:44
ralsinaalecu: got it16:44
ralsinastupid question, on what file is the list of UDFs?16:44
pmatulisnessita: k, i'm good.  i forgot that i have auto-login on Home16:44
alecuralsina: eric and natalia are already on it.16:44
nessitapmatulis: that's handy :-)16:44
alecuralsina, probably in some metadata. Only verterok may know :-)16:45
pmatulisnessita: yeah16:45
ralsinaalecu: even better :)16:45
ralsinaalecu: ok, so I will just delete the metadata, because I think I just turned Ubuntu One into a UDF ;-)16:45
alecuralsina, cleanest way is "u1sdtool --list-folders" but it's likely you already knew that.16:45
pmatulisnessita: i presume you only need logs from the 'puter that lost the permissions?16:45
alecuralsina, "Ubuntu One" is usually a UDF, only with an empty volume uuid16:46
pmatulis(when it happens again)16:46
nessitapmatulis: well, ideally from both, since I'm betting there was a modification triggered by the other computer... but is just a guess that allows me to explain the perm lost :-)16:46
pmatulisnessita: ok, will send both if it happens again16:46
nessitapmatulis: unless we have a very dark and hidden corner case, syncdaemon never modifies a files unless the server notifies of a modification happening in another device16:47
pmatulisnessita: sorry to hear that 'cause i did not change 35 shell scripts16:48
nessitapmatulis: sometimes, some programs, will modify your files without you noticing. I know the following  example does not apply to your use case, but for example many music players edit tags in mp3 files16:48
nessitaand the user himself never "changed" the file. So, with your logs, I can detect if there was an actual modification, and when16:48
pmatulisnessita: k, thanks for your help16:49
nessitapmatulis: thank you!16:49
nessitagatox: https://bugs.launchpad.net/ubuntu-sso-client/+bug/95095316:51
ubot5`Ubuntu bug 950953 in Ubuntu Single Sign On Client trunk "I can not click the "Password forgotten" link" [High,Triaged]16:51
gatoxnessita, weird..... but ok :P16:51
nessitagatox: this happened in a clean vm install, no nightlies, french lang16:52
nessitagatox: 100% reproducable16:52
thisfredalecu: +1, two small comments, but nothing that needs changing16:54
alecu\o/16:54
alecuthisfred, good point on str.format()16:58
alecuthisfred, I'll change that, since I don't see it used everywhere else in u1-client.16:59
ralsinanessita: replied on tab-tab-tab, the loop in test_focus_order is tricky to get right, I would rather have it fail if someone changes FAKE_VOLUMES_INFO (and I'd rather we don't change it anyway ;-)16:59
thisfredalecu: yeah, it's a shame, because I like the new syntax much better, but it seems to be a failed experiment at this point17:00
alecuthisfred, anyway, I'll do it in the last branch of this series, since it's used there too.17:00
thisfredalso having it be the same as in C has its advantages17:00
thisfredalecu, sure17:00
mandelalecu, sorted!17:05
mandelok, EOD for me, need to go and visit the brother, havea great weekend!17:05
alecumandel, thanks for all the reviews.17:06
alecumandel, hope your brother gets well soon!17:06
alecumandel, and have a great weekend too.17:06
alecuok, time for lunch and some errands. See you guys laters!17:09
=== alecu is now known as alecu-away
=== zyga-afk is now known as zyga
ralsinanessita: I am looking at lisette's focus doc, and specifically at the settings tab (devices looks ok to me!). Currently, we open focused on "Get more storage" but "Apply these settings" is  a orange default button. So, users may be  surprised, think they are applying settings and get sent to buy stuff instead17:23
ralsinanessita: how do you feel about making "get more storage" not orange? I know we have to ask design,  but it really makes everything harder17:23
ralsinanessita: or at least not start focused there17:24
verterokralsina: there is no file with the UDFs, it's all in tritcask17:31
ralsinaverterok: thanks17:32
nessitaralsina: trying to follow up in that...17:36
nessitaralsina: so which one is the default button that shouldn't be from your POV?17:37
ralsinanessita: having "get more storage" be a default button is weird17:37
ralsinanessita: or at least we should focus on "apply changes" when we switch to the settings tab17:38
nessitaralsina: so, I'm not sure I follow. The app as a whole will have  the focus on whatever widget the user last tabbed in, no?17:38
nessitaralsina: when the user swicthes to folder we don't change the focus, same for devices or account, I don't find consistent changing the focus for settings17:39
ralsinanessita: well, the default button is supposed to be bound to "Enter"17:39
ralsinanessita: and space is used to activate the focused button instead17:40
nessitayes, when you're "in" a set of widgets with a default button. So you say that clicking on a setting does make hitting enter trigger the apply of the new settings?17:40
ralsinanessita: I see I was extra-annoyed because I was using u1cp-qt --switch-to settings to test the tab  ordering17:40
gatoxbrb.... reboot17:40
ralsinanessita: right17:40
nessitalet me try that17:41
gatoxeverytime i said reboot: the problem get fix17:41
ralsinanessita: or clicking on "settings", then enter, should apply (even if it's useless)17:41
dobeynessita: ugh. also, the window title of the sso qt dialog is "ubuntu-sso-login-qt". that's not good17:41
nessitaralsina: so, before try it (I'm logging in to U1), it smells like a focus issue, not a "default button" thingy, no? cuz even if we make the buy more storage non -default, the same problem is there, no?17:42
nessitadobey: yesm already reported and assigned to gatox17:42
ralsinanessita: I think it's both. We have two "extra-special-orange" buttons, and enter does the wrong thing17:43
nessitaralsina: but the buttons are located in different "widget sets"17:43
dobeynessita: and no icon :-/17:43
ralsinanessita: doesn't matter, it's a dialog17:43
nessitaralsina: so they are default for different scopes17:43
nessitaralsina: is not a dialog :-) is a mainwindow17:43
ralsinanessita: that's nerdtalk there :-)17:44
ralsinanessita: basically, that sounds like an artificial distinction, when you consider the UI as a whole17:44
nessitadobey: yes, that's a very old bug for ussoc, but we never got the proper name to use for the "ubuntu" icon17:44
gatoxnessita, which one is that bug? i can't find it in my queue17:44
nessitagatox: https://bugs.launchpad.net/ubuntu-sso-client/+bug/94974417:45
ubot5`Ubuntu bug 949744 in ubuntu-sso-client (Ubuntu) "The ‘Sign In’ window does not have a title" [High,Triaged]17:45
dobeynessita: i'm not sure we should use the "ubuntu" logo icon exactly.17:45
nessitadobey: for the sso service, we definitely will not use the u1 icon17:45
nessitadobey: if we would have a sso icon, we should use that17:45
dobeynessita: the u1 icon was not my suggestion :)17:45
nessitadobey: I know, I was being pro active in answers :-P17:45
nessitaralsina: not sure what you mean with artificial distinction...17:46
dobeyhmm17:47
ralsinanessita: that we know it's a separate widget hierarchy, but to theuser it's a window with two orange buttons in it (or 3 in accounts)17:47
dobeyand login.ubuntu.com of course just uses the same thing as the rest of ubuntu17:47
ralsinanessita: nevermind, I will try to write a coherent email about this and send to you and design :-)17:47
nessitaralsina: ok, sounds good17:47
nessitathe UI thingy is getting annoying, too many people expressing their opinions (which is fine) but not much we can do without a clear path from the design team17:48
ralsinag able to move from twitter to facebook using arrows)17:48
nessitaralsina: I would ignore that and let the user move with tab17:48
ralsinanessita: agreed, I was being part of the problem there17:48
nessitais easier and more intuitive?17:48
ralsinanessita: pressing enter to activate the highlighted button is a reflex to many users17:49
nessitaralsina: any idea why only space works?17:49
nessita(for some buttons)17:49
ralsinanessita: but as I said, I will try to write a reasonable email17:49
nessitaok17:49
ralsinanessita: like which ones?17:49
nessitaralsina: twitter and facebook, get help17:49
dobeynessita: they aren't buttons?17:50
ralsinanessita: you are supposed to use space to activate buttons. Having enter work is the strange one :-) But let me do some checks17:50
nessitadobey: they are17:50
ralsinadobey: yes they are17:50
nessitaralsina: if it's expected, that's fine by me17:50
ralsinanessita: I am double-checking17:51
nessitaI expect a focused button to be activated by enter, but I'm a power user17:51
ralsinanessita: toy app with buttons in it, they activate by space, not by enter17:51
nessitaack17:52
dobeyugh17:52
dobeythis is nasty17:52
ralsinadobey: I assume ugh means "on gtk they activate with enter and not space"?17:52
dobeyno17:52
dobeyugh means "ugh, they *are* buttons"17:53
ralsinadobey: we could make them labels, if you want. You would not be able to see the difference, and you would not be able to activate them by kbd though.17:53
dobeyugh and now i have seen the border around the fb/twit icons17:53
dobeyralsina: you can see the difference. though i don't know where any labels are in there which you can tab to17:54
ralsinadobey: the border when focused was  added so you know you tabbed there17:54
ralsinadobey: it was invisible before17:55
dobeyralsina: the one that covers up the text?17:55
dobeyralsina: and i see a border on the fb/twit icons when they aren't focused17:56
nessitadobey: can you share a screenshot please?17:56
nessitaalecu ping17:57
nessitano alecu? bo17:57
gatoxnessita, ralsina here is the branch for the title thing: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/window-title/+merge/96802 is reallyyyy really trivial17:58
ralsinanessita: I see a visible border when unfocused but it doesn't cover any text17:59
ralsinaoops, that was for dobey ^17:59
nessitathere's a border? but they both have background: transparent18:00
dobeyralsina: no, the get help and the giant white button above the tabs, have the bottom part of the text covered up though, when they are focused18:01
ralsinadobey: not here for the get help, and what "giant white button"?18:02
dobeyralsina: "go to the web for blah blah blah blah"18:03
nessitadobey: can you show what you refer to with a screenshot please?18:03
dobeynessita: http://ubuntuone.com/67YpqFQR8wPvK7cS20QUqT shows the borders around the icons18:04
ralsinadobey: nope, no overlapping there, either. Are you using current trunk?18:04
dobeynessita: you might need xmag18:04
nessitadobey: I have it18:04
dobeyralsina: i'm using nightlies18:04
nessitadobey: debugging now18:04
nessitadobey: ok, I see them as well, would you please file a bug for me?18:05
dobeyralsina: oh, get help is a bit weirder. get more storage covers up text on the boottom though18:07
dobeyas does go to web blah blah18:07
ralsinadobey: I would love some screenshots of that, please18:07
dobeyralsina: yes, am working on it it18:08
dobeyit even18:08
ralsinadobey: because get more storage now should have a black halo outside when in focus18:08
dobeyi can't focus them all at the same time and just take one screenshot, you know :)18:08
nessitaralsina, gatox: about https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/window-title/+merge/96802, I'm now considering that perhaps we should not put the app_name as title, but the ussoc name (Ubuntu Single Sign On), since the window is for that service, and there is a trade off between customizing sso and maintaining the branch of sso18:08
ralsinadobey: and "go to web for..." should have a gray halo18:08
ralsinanessita: +1 on generic Ubuntu Single Sign On18:09
gatoxnessita, as you prefer18:09
nessitabranch -> brand18:09
nessitagatox: the thing is that is not my call :-/18:09
gatoxahhhhh18:09
gatoxnessita, who? design?18:09
dobeyralsina: and they do18:09
nessitagatox: not really, ISD18:09
ralsinadobey: ok, I will just wait for the shots then :-)18:10
nessitagatox: is like we re using, as a dumb example, paypal, and we were not saying that we're using paypal18:10
gatoxnessita, i understand18:10
nessitagatox: so we're using a service that should have a brand of its own, and we can't just remove all references to sso18:10
nessitagatox: let's change to "Ubuntu Single Sign On" for the title, non translatable string18:11
nessitaralsina: agreed? ^18:11
ralsinanessita: yes18:11
gatoxok, fixing18:11
nessitadobey: YEY u1sdtool finishes now!18:13
nessitadobey: may I see the patch you proposed yesterday? (link to the mp please)18:13
dobeynessita: for twisted?18:14
nessitayes18:15
dobeyhttps://code.launchpad.net/~dobey/ubuntu/precise/twisted/fix-935756/+merge/9661718:15
gatoxnessita, done18:15
nessitagatox: thanks18:17
* dobey ponders just writing a gtk+ control panel in vala18:18
dobeyoh18:19
dobeyspeaking of gtk+18:19
nessitaok, I need to *start coding* today, so will finish title review and then go underground18:19
dobeyblah, my scrollback is cut off18:19
nessitaspent the whole day so far answering style complaints ;-) and reviews18:19
nessitagatox: will finish too-verbose on monday, ok?18:20
nessitano rush with that one, you may seek another review in the mean time18:20
dobeyJanC: btw, if you want a separate application for ubuntuone music store, it would take all of about 15 minutes to write a python script that does exactly that.18:21
dobeyhttp://ubuntuone.com/5IvaCQhMzY2jQpYliO57TM18:21
dobeyhttp://ubuntuone.com/4HwY018NyJ5IYPbVgVSqzI18:21
dobeyhttp://ubuntuone.com/6tXJfPd5XXlWO2I2lJw0ob18:21
dobeyralsina, nessita: ^^ ugliness in buttons18:22
ralsinanessita: haha, go and code with my blessing! ;-)18:22
ralsinadobey: the blue square is something else, and we are aware of that18:22
ralsinadobey: although for me it's pink-orangish18:22
dobeyralsina: it may be blue for me because of gtk+ theme18:23
ralsinadobey: the "get more storage" button just doesn't do that here18:23
dobeyam not using ambiance at the moment18:23
ralsinadobey: right18:23
dobeyusing the upstream gnome 3 one, so i can read the text on my gnome-panel18:23
ralsinadobey: and I see no overlapping in "Go to the web"18:24
dobeyralsina: i don't know. i'll file a bug i guess18:24
ralsinadobey: not about the 1st or 3rd ones18:25
ralsinadobey: the 1st we know, the 3rd I don't see the wrong18:25
dobey?18:25
dobeyralsina: you don't see the bottoms of the "p" and "g" characters cut off?18:26
dobeyralsina: it's the same bug as the get more button. it's just a different color18:26
ralsinadobey: ha, chrome was showing me like 30% of the image18:26
ralsinadobey: I only saw "Go to the web"18:26
nessitaralsina: the  "get more storage" button must be our padding hack18:26
nessitaralsina: perhaps we need to make the padding bigger18:27
dobeynessita: i think all these problsm are "our <insert something here> hack"18:27
ralsinanessita: 1000px should be enough ifit was going to work18:27
nessitadobey: maybe18:27
nessitaralsina: yeah, but that triggers another issue...18:27
nessitaralsina: if the "there was a problem window" is raise, the "show deatils" button has that padding applied18:28
ralsinanessita: ok, so let's use 100% padding then18:28
nessitaralsina: you can try it by putting an "assert False" inside the folders load() function18:28
ralsinaor 50%18:28
nessitaralsina: wanna try it in a branch? will review *not today* but on monday for sure :-P18:28
ralsinanessita: sure18:28
nessitagatox: approved with a comment https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/window-title/+merge/9680218:30
ralsinadobey: do all the problems go away if you use -style windows? other than it looking like a windows app, mind you ;-)18:30
dobeyralsina: it *already* looks like a windows app :(18:31
dobeymaybe you missed the e-mail where i said that at greater length :)18:31
ralsinadobey: yadda yadda18:31
gatoxnessita, ok18:32
dobeyralsina: some of the buttons are weird in a slightly different way, but the text isn't really getting cut off. the underline on "Get help online" gets covered up it looks like, and the focus dotted line intereferes with the p character, but i don't think it's actually "hidden" beyond the obvious confusing white pixels coalescing in the same area18:33
ralsinanessita: the border around facebook and twitter is about those buttons not being set to flat18:33
nessitaralsina: oh my bad then18:34
ralsinanessita: np, trivial fix18:34
gatoxnessita, should i look for another aprroval? i already push the changes with the constant18:36
nessitagatox: yes, please ask ralsina (mainly for the nature of the change, not the diff itself)18:37
gatoxnessita, ok.... ralsina please when you have a minute, really trivial branch: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/window-title/+merge/9680218:37
ralsinagatox: got it18:37
urbanapeoff to get my lasik consultation. see you guys on Monday18:38
gatoxralsina, thanks18:38
ralsinagatox: +118:38
dobeycheers urbanape; don't go blind18:38
gatoxralsina, great18:39
urbanapehere's hoping18:39
urbanapeyay, science18:39
dobeyfml. this weather is hell on sinuses18:42
ralsinadobey: you have 2' to see if lp:~ralsina/ubuntuone-control-panel/tweaks fixes help / twitter / facebook buttons for you?18:49
dobeysure18:50
ralsinadobey: I added some padding around the get help because the halo felt too tight, but that's not final18:50
dobeysince now is the time when millbank falls off the earth18:50
ralsinadobey: yeah, we are all alone us americans18:50
dobeyralsina: that branch fixes those issues, yes. the arrow circle thing moves when i focus/unfocus the buttons with them though18:52
ralsinadobey: yes, that's a bug in the gtk theme for qt18:52
ralsinadobey: it draws the icon as if it were not a flat button18:53
ralsinadobey: I can do a fix, though18:53
dobeyeh18:53
dobeyi'm not as bothered about that18:53
ralsinadobey: me neither18:53
=== jalcine_ is now known as jalcine
dobeyand i still think the buttons are ugly, but at least the text doesn't get covered up now18:53
ralsinadobey: and there is no blue highlight anymore, right?18:53
dobeyralsina: right18:53
ralsinadobey: how about on the "go to web" one?18:54
dobeyit didn't have a blue hilight before18:54
dobeyand still doesn't18:54
dobeybut the text is visible now18:54
ralsinadobey: ok, good18:55
dobeycome back alecu19:01
gatoxnessita, about the forgotten passoword link not clickeable: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/forgotten-link/+merge/9681419:08
nessitaack19:08
nessitagatox: nice catch19:08
gatox:D19:09
nessita;-)19:09
ralsinagatox: care to take a look at bzr+ssh://bazaar.launchpad.net/~ralsina/ubuntuone-control-panel/tweaks/ and tell me if you like the changes?19:10
gatoxralsina, ok, looking19:11
ralsinagatox: very minor tweaks to make the gotoweb buttons a bit less weird (text doesn't wiggle on focus)19:11
gatoxralsina, maybe you can change also the padding of the QTabBar::tab:focus {19:18
gatoxfrom:19:18
gatox    padding-left: 1000px;19:18
gatox    padding-right: 1000px;19:18
gatoxto use with percentaje19:18
gatoxpercentage19:18
gatoxralsina, everything looks fine!19:19
ralsinagatox: sure19:19
ralsinagatox: I am trying to hack gotowebbutton to set the icon via background-image to remove it from the code19:20
ralsinagatox: also so it stops fidgeting when I tab ;-)19:20
gatoxralsina, and fixing the icon-size from the qss instead?19:23
gatoxfixing i mean..... set the icon-size of the button19:24
gatoxor it would be the same situationn as now?19:24
ralsinagatox: the problem is not size, it's position19:24
gatoxahhhhh19:24
ralsinagatox: when the button gets focus, the clipping area gets pushed 2 pixels down, and there goes the icon19:24
gatoxahhhh i see19:25
ralsinagatox: I blame elves19:25
gatoxjejej19:25
gatoxdoes anyone know where i can report a bug about the hud?19:29
dobeyhttps://code.launchpad.net/~dobey/ubuntuone-client/transient-notify/+merge/9681519:33
dobeywhee19:33
gatoxralsina, i think you forgot to add your +1 here: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/window-title/+merge/9680219:52
ralsinagatox: oops, there19:52
gatoxralsina, thanks19:52
gatoxok, EOD here!! see you on monday people! have a great weekend!20:05
alecunessita, ping20:16
alecunessita, "the whole ubuntuone.proxy namespace will be distributed in another binary package"20:16
alecunessita, in fact, the tunnel_client.py *needs* to be in the ubuntuone-client package where syncdaemon is.20:16
alecunessita, tunnel_client.py and bin/ubuntuone-proxy-tunnel can be in the different package.20:18
alecusorry20:18
alecunessita, tunnel_server.py and bin/ubuntuone-proxy-tunnel can be in the different package.20:18
alecusince tunnel_server is the only one that depends on Qt.20:18
alecuand tunnel_client can work just fine if bin/ubuntuone-proxy-tunnel is not found (minus proxy support, that is)20:19
nessitaalecu: ah, ok...20:27
nessitathat's a little more complex than I thought, packaging and concept wise20:27
nessitaalecu: why does u1client need tunnel_client if no proxy nor proxy process available?20:28
alecunessita, tunnel_client is pure twisted. There's a class in it named TunnelRunner, (that starts the qt tunnel process).20:30
alecunessita, a TunnelRunner has a get_client method that returns either a tunnel client or the default reactor.20:31
=== zyga is now known as zyga-weekend
alecuin action queue, in the bits where reactor.connectTCP or connectSSL is called, a "client" is used instead, gotten from the tunnel runner.20:31
dobeyalecu!20:32
nessitaalecu: why not just having the tunnel runner in the default install, and not the tunnel_client20:32
nessita?20:32
dobeyalecu: is bug #703100 still relevant?20:32
ubot5`Launchpad bug 703100 in Ubuntu One Client "pynotify is a non-asynchronous library" [Medium,Triaged] https://launchpad.net/bugs/70310020:32
alecunessita, that can be done, sure.20:32
alecuhey thre dobey.20:32
nessitaalecu: would you think is cleaner? if we put the tunnelrunner outside proxy we can have the whole proxy namespace in another bianry package20:33
alecudobey, I think it's still relevant, yes, since the whole twisted loop in SD is waiting every time pynotify is called.20:34
alecunessita, let me take a better look.20:34
nessitaalecu: thanks20:34
alecunessita, but a priory, yes.20:34
dobeyalecu: but it's like 5ms per call, right, and we don't call it very often…20:34
alecudobey, how did you measure "5ms"?20:35
alecudobey, I agree we don't call it very often, yes.20:36
dobeyalecu: i didn't, hence the "like 5ms" meaning "it's obviously very small"20:36
alecudobey, perhaps we can lower the priority?20:36
alecudobey, the bug priority I mean.20:36
dobeyalecu: well, given the first number for the bug is 7, and it's not fixed yet, i'm pretty sure it has a low priority :)20:37
alecudobey, if we measure it to be really fast, then I agree with removing the bug.20:37
dobeyalecu: do you know how we can measure it?20:38
alecudobey, yes. But not by hard :-)20:38
dobeyalecu: write a test that logs the times?20:39
alecudobey, or time.time() - start_time, and logging that.20:39
alecudobey, are you interested in solving that bug *today*?20:39
dobeyalecu: no. i was looking for another bug which i did fix already, and that was in the results list, so i thought i'd ask since you filed it, and it's assigned to you :)20:40
thisfredin general, performance testing in python is best done in pystones, since actual speed is *very* machine dependent. Also usually it's best to call the function you're testing at least a few thousand times, and throw away the outliers, since the first call can be disproportionally expensive20:49
thisfredbut if you're looking for a ballpark figure, likely none of this matters ;)20:50
JanCthisfred: timing a function many times is general good benchmarking practice, not only for Python  ;)20:51
JanCcalling it many times to time it20:51
thisfredsure :)20:51
thisfredhttp://code.activestate.com/recipes/440700-performance-testing-with-a-pystone-measurement-dec/ for a pystone example20:52
alecunessita, so, where do you suggest that we should put the runner in?20:54
ralsinaI think I have made all the widgets stop moving when focused, and all orangeish squares disappear except for the checkboxes20:54
ralsinaAs well as getting rid of the icon-in-code in GotoWebButton20:54
alecunessita, also this should also work if *all* of the code was in the syncdaemon package proper.20:55
ralsinaOTOH, I want to poke my eyes with my mate because it was a pain in the arse to get "right" and noone is going to like it on monday ;-)20:55
alecunessita, since the runner is resilient to the tunnel process crashing because Qt is not installed.20:55
alecuralsina, we may be able to not like it earlier, if you point us at a branch :-)20:56
alecuralsina, no need to wait till monday!20:56
nessitaalecu: I did not follow you, sorry20:56
ralsinaalecu: feel free to hate it as soon as you like ;-) https://code.launchpad.net/~ralsina/ubuntuone-control-panel/tweaks20:57
dobeythisfred: mostly i care about "is this ever going to actually be a problem in any case we care about"20:57
alecunessita, let's say for a moment that we *do not* create a different binary package for the tunnel process.20:57
nessitaalecu: ok20:57
alecunessita, and stuff all of it into the u1-client package.20:57
dobeythisfred: and i'm pretty sure the answer is "no" and we're calling all kinds of other sync APIs without any similar worry20:57
nessitaalecu: aja20:57
alecunessita, what would happen?20:57
nessitaalecu: ImportErrror on Qt is my guess20:57
dobeyalecu: i would say no20:57
alecunessita, the runner will try to start the tunnel process20:57
nessitaalecu: before that20:58
alecunessita, and the tunnel process will fail20:58
dobeyunless you are going to handle qt not being there20:58
nessitaalecu: ImportError, no?20:58
alecunessita, but the runner knows how to deal with the tunnel process not starting20:58
thisfreddobey: right, just responding to one of my python "triggers" ;)20:58
alecunessita, it just ignores it, and uses the default reactor, just like if proxies are not configured.20:58
nessitaalecu: we can't release to ubuntu a code that import qt4 and then not list that on Depends20:58
alecunessita, what if it just "recomends" it?20:59
nessitaalecu: the same, recommends pulls packages in20:59
alecunessita, also, what if we release this on nightlies?20:59
dobeywe can't depends/recommends on pyqt in ubuntuone-client20:59
dobeynightlies should be as close to proper ubuntu packages as possible20:59
nessitaalecu: I think is much much cleaner to have a separated binary package for the tunnel, where we can set the dependencies properly21:00
alecunessita, dobey: ok21:00
dobeyit should be a separate binary package21:00
nessitaalecu: that's also what we commit to with the ubuntu-release team...21:00
=== jalcine is now known as JackyAlcine
nessitaalecu: so, I would put *just* the runner, which is more like a chooser, in ubuntuone/tunnel_safe_not_crashing_chooser_runner.py21:01
=== JackyAlcine is now known as jalcine
nessitaand import that from action queue. And make that module, check if it has available whatever it needs, and start the super-duper-tunnerl server if available21:01
nessitaalecu: and also check if there is a concrete need of using it, no?21:02
dobeyhow is it starting it?21:02
dobeyhard coding the path?21:02
nessitaalecu: we should not start it if there is no proxy support need21:02
nessitadobey: no, it uses LIBEXEC21:02
alecunessita, the only way to know if proxy support is needed is trying to start it.21:02
alecunessita, because we can't have qt stuff on sd21:02
dobeynessita: from clientdefs?21:03
nessitadobey: yes21:03
dobeyok, good21:03
nessitaalecu: but what if I have qt, and no proxy needs? will I get the tunnel proxy running?21:03
nessitadobey: from https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-useit/+merge/9621321:04
nessita319+        from ubuntuone.clientdefs import LIBEXECDIR21:04
nessita320+        return path.join(LIBEXECDIR, TUNNEL_EXECUTABLE)21:04
ralsinaEOD for me, I'll send some emails about the style changes I am trying late tonight21:04
nessitaralsina: ack21:04
dobeynessita: ok. i didn't need the paste :)21:04
alecunessita, if you have qt installed, and have no proxies configured, the tunnel process will be started, will check if proxies are needed and then die. And the runner will find out about this and not use the tunnel.21:04
nessitadobey: well, I prefer you noticing that code now and not when we're releasing :-)21:04
nessitaalecu: ah, ok then21:05
dobeyugh, where are these ubuntuone icons at21:05
dobeygrr21:05
nessitaalecu: so, I think what I mentioned before may work? what do you think?21:05
alecunessita, I think I can make it work.21:07
nessitaalecu: is it too much overhead for you? can I help somehow?21:07
alecunessita, I would have liked this to get to nightlies, so we can start to test it.21:08
alecunessita, and work on this change on a different branch.21:08
nessitaalecu: the proxy namespace is not being distributed ATM in mightlies21:09
alecunessita, ok.21:09
nessitaalecu: so this branch, when landing, will just broke nightlies21:09
nessitaso, we should also fix the packaging in nightlies as close as ubuntu21:10
nessitaas close as how ubuntu will have it, I mean21:10
alecuok, perfect.21:10
nessitaotherwise we're missing one important piece of the testing (not having/having the thing available)21:10
nessitaalecu: sorry for the delay... I know you wanted this ready today21:10
alecunessita, no problem: I'm just one review from landing this, so it's much closer than I've been all week :-)21:11
nessita:-)21:11
alecunessita, I'll finish this fix tonight, and you can review it first thing in the morning, and then we'll try to get nightlies fixed in the week.21:12
alecuI mean, "first thing *monday* morning"21:12
nessitaalecu: +121:12
nessitaof course ;-)21:12
alecunessita, thanks!21:12
nessitaalecu: thank you :-)21:12
dobeyhmm21:27
dobeyhrmm21:53
dobeyi can't find these icons21:54
nessitabye all!22:00
dobeyindeed22:04
thisfredlater!22:07
dobeylater22:16
=== yofel_ is now known as yofel

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