[07:44] does ubuntuone for iphone also run on the ipad? === zyga is now known as zyga-afk [09:28] dobey, ralsina will update the proposals [09:30] morning all! [09:40] Good morning all! :D [10:08] mandel: no I won't [10:08] and good morning! [10:13] ralsina, I forgot the I :P [10:13] ralsina, ups hehe [10:13] mandel: hehe [10:14] mandel: nice work on the creds-qt script! [10:14] dobey, I meant 'dobey, ralsina I will update..' [10:14] ralsina, I need to fix that one, there is some refactoring needed so that we have a windows and linux and provide the correct qss on windows [10:14] ralsina, I've ran it a looks ugly.. very ugly, also, we need an icon from lisettte [10:15] yes [10:15] however, it's not terribly crucial that the proxy credentials dialog is pretty [10:15] ralsina, also I founf bug 961027 with the release you sent [10:15] Launchpad bug 961027 in Ubuntu One Control Panel "on_download_finished signal is not correctly used in the Windows IPC" [Undecided,New] https://launchpad.net/bugs/961027 [10:15] and it's a two-line fix, copied from the usoc gui main [10:16] mandel: yes [10:16] ralsina, yeah, is simple, but I like things to be nice :) [10:16] I also got a more misteryous one about using "_ " as a formatting character [10:16] which I have not reported yet [10:17] ralsina, he, it removes the _ because it takes them like an action? [10:17] In any case, I need to hack on the qss loading so it becomes parametric, so we can do things like $separator_color in it instead of having 60 #abcdef in it [10:17] mandel: I suspect we are sending %_ somehow to the logger [10:18] mandel: or something similar [10:18] ralsina, also, I've set up a auth proxy on windows and I'm not getting the creds auth asked.. so I think that QNetworkProxyFactory.setUseSystemConfiguration is not really working.. [10:19] weird, I did get that before [10:19] ralsina, I guess I have to set the proxy in the NetworkAccessManager to use QNetworkProxy.Default [10:19] I will do some IRL and reporting today I guess [10:20] ralsina, can you test that because afaik in my system it is no using the proxy, and it is a 1 liner :) [10:20] mandel: not right now, but yes [10:21] ralsina, can you give me some background about this bug 958938 [10:21] Launchpad bug 958938 in Ubuntu Single Sign On Client "On windows, the proxy credentials cannot be retrieved" [High,Confirmed] https://launchpad.net/bugs/958938 [10:22] mandel: I think that's why the creds are not requested. See https://pastebin.canonical.com/62693/ [10:22] mandel: I just got that with an authed proxy [10:24] ralsina, hm.. let me take a look [10:53] ralsina, keyring is buggy.. I'm really considering doing my own lib for that called secrets with a sync and async api [10:54] mandel: sure. But is there a way around that for today? ;-) [10:54] ralsina, I think so :) [10:54] mandel: yay [10:54] ok, time to be a dad for bit, then walk, then breakfast. Will be back in about 90 minutes === zyga-afk is now known as zyga [11:29] good morning! [11:29] gatox, morning! [11:29] gatox, did you fix the bug? [11:30] mandel, yes! :D i couldn't be more happy :P [11:30] gatox, what was the issue>? [11:30] Hey, all. Will probably be out this morning. Lex still has pink-eye, and I need to run him to the doctor. He'll likely stay home, and hopefully Amber can take the afternoon off. [11:31] urbanape, ack [11:32] mandel, it was some problem between some part of the code not clean and the package not being generated correctly.... and mmy brain completely dry to look at the same thing for a long time :P [11:33] gatox, weird.. [11:33] mandel, yes....... the good news, is that it was an important bug and now it's fixed [11:33] mandel, i'm reading your blog..... good post [11:34] about pyqt twisted reactor [11:34] gatox, evil qtreactor.. [11:34] agree [11:46] hello everyone! [11:48] nessita, morning! [11:49] nessita, hi [11:50] hola mandel, gatox [11:50] ralsina: hello! you around? [11:51] ralsina, when you are around, look at this: http://ubuntuone.com/4ae2jDxfp5LKhpXYKeYTIT :D \o/ [11:52] gatox: what did you install to get performance indicators? [11:53] nessita, let me remember [11:54] nessita, i look for "indicator" in the software center..... and install the system load indicator [11:55] crap, I still need the reviews to move forward with the releases [11:56] ralsina: I need your reviews ASAP, please :-) [12:10] nessita, mandel do you know which is the dbus api to publish a file and get the link? it wasn't really clear for me yesterday [12:10] gatox, from dbus or the rest api? [12:11] mandel, i was using dbus for everything [12:11] gatox, no idea.. wask in #chicharra they should know [12:11] i tihnk that might be change_public_access..... but i have to check that [12:12] mmm.... yes.... it seems to be that....... and then you have to listen the signal PublicAccessChanged if i'm correct [12:12] * gatox answering himself [12:15] gatox, if you hear voices, get worried [12:15] hello #ubuntuone! [12:15] mandel, crap...... :P [12:15] alecu, hi! o/ [12:15] alecu, morning! [12:17] gatox: have you check on d-feet? that's the easiest way to digg that [12:17] nessita, yes..... i was checking that.... but i wasn't sure..... but i think that it is the one i mention [12:18] gatox: yes, that's it (I didn't read that :-D) [12:18] nessita, :P thanks [12:19] alecu: would you please help me with one review? is the update of the stable-3-0 branch for u1client, and I guess you can check if it makes sense [12:19] nessita, gatox: I heard you guys were needing a replacement for deferToThread for windows [12:19] nessita, sure, I can make reviews [12:19] alecu: https://code.launchpad.net/~nataliabidart/ubuntuone-client/stable-3-0-update-2.99.91/+merge/98535 [12:19] alecu: re deferToThread, I think there is no need... right mandel? [12:19] nessita, gatox: did you end up putting the qt4reactor back just in order to use deferToThread? [12:20] alecu, yess [12:20] nessita, no, I fixed it :) [12:20] alecu: on windows we agreed we never could let the qt4reactor go since we depend on that for IPC [12:20] alecu: so we did not "put it back", apparently we just weren't using it right [12:21] nessita, ok. [12:21] mandel, so, how did you fix it? do you need a review on that branch? [12:21] alecu, and the deferToThread issue was due to this: http://www.themacaque.com/?p=1067 [12:21] alecu, already landed [12:23] mandel: you know that we hve the same issue in linux, without any reactor involved? (now that you mention that) [12:24] nessita, what? deferToThread no working? [12:25] mandel: no, doing an exec on a dialog and eveything freezing [12:26] nessita, dammed, that is annoying, did you try to set WA_ShowModal to false? [12:26] mandel, the solution in your blogpost sounds right [12:27] mandel: but I want the dialog to be modal... and no did not try that [12:27] mandel, but I see a small issue that can lurk in there. [12:27] nessita, then we are stuck with show, I wonder why did the qt people add it [12:27] alecu, shoot! [12:28] ralsina, how did you set the proxy in you system? which version of windows is it? [12:28] mandel, since ._exec freezes every event going to the parent window, we *have* to make sure that nothing can be clicked on the parent window of the dialog when we use your solution. [12:28] mandel, otherwise breakage can occur [12:29] mandel, since the code that was doing the ._exec may rely on nothing changing on the parent. [12:29] does it make sense? [12:30] alecu, yes it does, in the exampe case is for the proxy_creds dialog so we have no problem, but for control panel etc.. it is indeed something to consider [12:30] (I've been thru similar situations in other unspeakable frameworks, and changing from modal to modeless dialogs usually brings this issue.) [12:30] alecu, one question, have to tested proxy support on windows? how did you do it? [12:31] mandel, if it is for the proxy credentials then we *might* have that same issue. [12:31] mandel, think of this: [12:31] you pop up the credentials dialog. [12:31] mandel, but the user clicks again on the "ok" button in the parent. [12:31] alecu, the credentials dialog is a diff process from any other [12:31] mandel, or in the "cancel" [12:32] mandel, oh, ok. [12:32] mandel, anyway: are the parent buttons being disabled while the child is showing? [12:32] mandel, (nevermind if they are in different processes or different mainloops) [12:33] I'll test it right now. [12:33] alecu, exactly, the problem you are stating I think is already fixed by gatox and nessita :) [12:33] nessita, gatox: awesome! [12:33] alecu, I'm setting the proxy in the windows control panel things and I'm not getting a 407 [12:33] alecu, eventhough I do not have the creds and is a auth proxy [12:34] mandel, but does it connect? [12:34] alecu, yes, control panel gets all the data [12:34] mandel, alecu: the u1cp gets disabled when the u1cp knows is popping up a window, which is *not* the case for credentials [12:34] mandel, anyway: you should ask brian since I remember reading in the IRC logs that he got proxies working on windows. [12:35] alecu, I think ralsina got them working too [12:35] nessita, so we need to let the u1cp know when proxy credentials are being requested [12:36] alecu: well, is very very tricky to do that, since... how would you do that? [12:37] * alecu thinks [12:38] ralsina: I need reviews from you :'( [12:39] nessita, I would do it like this: [12:40] nessita, when the control panel creates the webclient it passes an optional parameter. This parameter is a function, that takes a boolean, and we can name the argument something like this: "requesting_proxy_credentials" [12:41] nessita, whenever the webclient needs to show the credentials dialog, it calls that function with True [12:42] alecu: any chance the webclient emits a signal when "is processing"? [12:42] nessita, the control panel already knows this: [12:43] nessita, it's from the time webclient.request is called until the deferred returned by it is fired (with either callback or errback) [12:44] nessita, perhaps the control panel could inherit the webclient, add that specialization to request and use that specialized class to do requests. [12:44] alecu: the thing is that all this happens in the backend... not in the client [12:44] alecu: client -> frontend I mean [12:45] right. [12:45] alecu: and right now the frontend will disable the part that is being loaded [12:45] but not the whole UI [12:45] nessita, then I think it's safe enough. [12:45] so if the account info is being requested to fill in the account tab, the "file sync status" widget and the get storage button are not disabled [12:45] neither are the 'get help button', nor the twitter or facebook button [12:46] nessita, the worst thing that can happen in all these cases is the proxy credentials dialog being thrown to the background [12:46] ralsina, pin [12:46] g [12:46] nessita, I think we should pass the window id so the proxy creds dialog is shown on top too [12:47] alecu: well, following what you mentioned before to mandel, the user could click on the "parent"? [12:47] nessita, yes, the user could click on the parent. And the parent will get focus, but the areas being requested would be grayed out. [12:48] nessita, and if the user clicks on a different area, another creds dialog will pop up. [12:48] alecu, nessita yet, he could close the window, right? [12:48] and the proxy creds dialog will be left there, or am I wrong? [12:48] mandel: yes [12:48] I guess so, yes [12:48] mandel, yes, the proxy creds dialog would be below other windows. [12:48] ideally we need to make the Qt dialog be 'transient for' other window id [12:49] mandel, *you* need to make sure that the process is terminated when the parent dialog is closed. [12:49] alecu: but how can mandel know that? [12:49] nessita, that will help with it not being "below" the parent. But not with it closing when the parent closes. [12:50] alecu: if a window is transient for another, the parent can not be closed (at least in gtk) [12:50] nessita, does this happen when they are in different processes? [12:51] alecu: yes, the former sso worked like that. Sso was a process, and the window id was the control panel one (another process) [12:51] (sso process was the same as the sso UI process back then, but I think it applies the same) [12:51] nessita, great then. It's probably a WM thing, so it should work on Qt too. [12:52] alecu: anyways, I asked ralsina some time ago help for doing this... and unless we go deep in the wm stack, there is no way to do it [12:53] mandel, so: in _launch_proxy_creds_dialog you should keep the reference to the process spawned by spawn_program. And in WebClient.stop you should make sure that the process is killed. [12:53] nessita, I don't understand what is that you want to do. [12:53] nessita, (I mean, with ralsina) [12:54] nessita, closing the child when the user clicks to close the parent? [12:54] alecu: no, setting a window to be transient of another window given its window id [12:54] alecu, nessita that means that we need to change the runner implementation, which uses glib or qt, to return the process use for the creds dialog [12:54] alecu: that can not be done from qt [12:54] Hello again, sorry I am late, had to do something. [12:55] nessita: I did a few of the reviews, did I miss any? [12:55] nessita, I'll take a look at transient windows from qt later today. [12:56] mandel: pong? [12:56] ralsina, ho are you testing auth proxies on windows? [12:56] ralsina: unless I'm very wrong, I emailed you asking for 3 reviews and got only one done :-/ [12:56] mandel, probably we need to change that, yes. I think that the right way would be to return a tuple with the current deferred, and some way to close the process. [12:56] nessita: ack, I did dobey's too yesterday. Will do them now. [12:57] mandel: by using my public proxy which I have told everyone about a few times already. [12:57] ralsina, I know that, but how did you tell the system that you are using a proxy? [12:58] alecu, ok, I'll file a bug for that and will take care of it as soon as I find out why I cannot get proxy working on windows [12:59] mandel: I configured it on chrome [12:59] mandel: which gave me the system's configuration dialog [12:59] mandel, it sounds like an interesting bug, but let's better look at the bug list and see if there's something more urgent than that. [12:59] alecu, sure [12:59] mandel, I think today is a fine day to make plans rather than dive into code. [12:59] alecu: doing a widows release soon is not low priority [13:00] btw: I love it when a plan comes together. [13:00] alecu: no, I need to get windows bugs fixed. Sorry. [13:00] ralsina, I'll try the chrome path, but it seems that doing it from the control panel does not work [13:00] alecu, you are missing the cigar, I'm sure there is unicode for that :) [13:01] mandel, this is it: 烟 [13:02] mandel: it's the same dialog [13:02] ralsina, hi there!! did you see my screenshot? [13:04] gatox: not yet [13:04] ralsina, here is the link: http://ubuntuone.com/4ae2jDxfp5LKhpXYKeYTIT (when you can) [13:05] ralsina, that is why I'm puzzled [13:06] mandel: in fact, every app in the system picked it up. [13:06] ralsina, in my case, all apps but u1 are using it [13:07] ralsina, cause the release you made has proxy support, right? [13:07] mandel: yes [13:07] mandel: if it didn't have it, why would I get proxy errors on stderr [13:09] ralsina, I know, I'm just discarding possible reasons [13:09] nessita, I'm reviewing this branch of yours: https://code.launchpad.net/~nataliabidart/ubuntuone-client/stable-3-0-update-2.99.91/+merge/98535 [13:10] nessita, if I do a diff with trunk, the only thing that changes is the version in AC_INIT in configure.ac [13:10] ralsina, I think is my system since I just for an error telling me it could not lauch a process within my projects dir.. [13:10] alecu: yes [13:10] nessita, what kind of review you need? [13:10] alecu: you should diff against stable-3-0 [13:11] alecu: is a review that should check that features there have an approved FFe, and that no unwanted changes are there, mainly [13:11] Chipaca, aquarius, beuno_, joshuahoover - online via dongle + windows machine [13:11] nessita, right. I branch stable-3-0. I merge in your changes. I get trunk minus that small diff. === beuno_ is now known as beuno [13:11] nessita, oh, ok. [13:12] cparrino, wrong channel? :) [13:12] nessita, in that case, I see this, that has no FFe: Fix tunnel spawning code so that it works on windows. [13:12] alecu: yes, but the idea is not compare against trunk, but compare the diff against stable-3-0 [13:12] beuno - only channel available to me atm [13:12] ay [13:13] beuno, should we trust this guy? It looks like a fakeparrino. [13:14] cparrino, what's your favorite christmas pudding? [13:14] beuno: is easier than that, just start to argue with him, and of he plays along, he's he :-D [13:14] lol [13:15] (you can recognize me in the same way ;-)) [13:15] cparrino, if manuel was to shit on something.. what would it be? [13:15] :P [13:16] haha [13:16] ralsina, I', stupid, I left sso running in a terminal from a branch I was working on.. [13:17] mandel, there's no such thing as "my favorite english pudding" [13:17] mandel: ayayayay [13:17] alecu - no jodas! [13:18] :-) [13:18] alecu, checks out. [13:18] ralsina, I though it closed like the linux one.. [13:18] mandel: no, it doesn't [13:18] :-) [13:18] ralsina, yep, I've realized that, why is that? [13:19] mandel: because it starts too slow and because there was a bug where if you start it twice quick it starts twice [13:19] ralsina, oh, cute [13:19] mandel: and if you start it twice none of the instances reply on IPC [13:20] nessita, I don't see a FFe for bug #956407, yet it's in the MP [13:20] Launchpad bug 956407 in ubuntuone-client (Ubuntu Precise) "Apport recipe attaches old/useless log files" [Medium,Triaged] https://launchpad.net/bugs/956407 [13:21] nessita: reviews finished [13:21] alecu: right, but that bug is not a new feature... just a bug fix [13:21] ralsina: thanks! [13:21] ok [13:22] alecu: we still can land bug fixes until final freeze [13:22] alecu: which is april 12th [13:25] nessita, I have just seen this add_info for apport function. I did not know it even existed. I think we should have it include the proxy.log file. [13:25] nessita, I'll open a new bug for that. [13:25] alecu: awesome [13:26] alecu: does the proxy.log contain any 'private' info? [13:27] dobey, I think not, but I should double check. I just realized that syncdaemon.log is not included, so it might make sense not to include it either. [13:27] I'll create the bug anyway, and we can check if we should include it or invalidate the bug. [13:27] right [13:28] it would be nice if we could automagically redact things in the logs [13:30] dobey, perhaps by including some special sequence while printing it in the logs? [13:30] dobey, like: SECRET("passWd"), and it gets translated into SECRET("XXXXX") [13:32] alecu: depends on how we're currently doing it. but something like that [13:33] it would be nice if we could register regex matches in apport somehow, like r'password="(.*)"' and apport would just replace the (.*) match with "[REDACTED]" or something [13:33] but alas [13:35] alecu: any other thing to note from the branch? [13:36] nessita, I'm halfway thru reviewing the code. [13:36] alecu: nice! [13:38] nessita: are there any left that i need to look at? [13:38] dobey: reviews? nopes :-) [13:38] aye, ok [13:39] i see ralsina approved the u1client one [13:39] dobey: yes, but alecu is also looking at it [13:39] right [13:39] dobey: since he added most of the proxy code, I wanted to make sure the update had his approve [13:39] ping me when it lands, and i'll make the tarball [13:39] dobey: yes [13:42] I'm off to have lunch [13:42] * mandel lunch [13:48] nessita, +1 [13:48] alecu: thanks! [13:48] nessita, should I change the status too? [13:49] alecu: already done, thanks :-) [13:49] nessita, i have a doubt regarding this issue: https://bugs.launchpad.net/ubuntuone-control-panel/+bug/944256 [13:49] Ubuntu bug 944256 in Ubuntu One Control Panel "Using ubuntuone-control-panel-qt defaults to limit bandwidth" [High,Triaged] [13:50] nessita, which should be the fix: no limit? a different limit? [13:53] i'm not nessita, but i would go with no limit. the great majority of users are likely to be on high-speed connections with bandwidth to spare. if this was 15 years ago i'd probably think differently (but i'd also be 12 years old) [13:56] gatox: is not like you need to change the default :-) [13:56] gatox: there is bug that is missreading the defaults from syncdaemon [13:56] gatox: if you look at /etc/xdg/ubuntuone/syncdaemon.conf [13:56] you will see that the default is: [13:56] nessita, ahhhhhhhh that's the bug!!... i didn't understand that from the bug report [13:57] [bandwidth_throttling] [13:57] on.default = False [13:57] read_limit.default = 2097152 [13:57] write_limit.default = 2097152 [13:57] gatox: so, in general, controlpanel should not change the defaults from the syncdaemon's default [13:57] nessita, the bug report just say: I installed ubuntuone-control-panel-qt on a clean system and it defaulted to limit bandwidth to 2048 kb/second. [13:58] gatox: rigth, so since controlpanel uses syncdaemon defaults, it means we're showing something wrong [13:58] ok [14:00] nessita, ping [14:01] nessita, I'm trying to install ubuntuone-client-proxy in a VM that has an older version of ubuntu-sso-client. [14:01] nessita, it says that ubuntuone-client will be updated (perfect) but it does not mention ubuntu-sso-client being updated. [14:02] alecu: what version of ussoc do you have? [14:02] nessita, I think that might be an error, since ubuntuone-client now uses the webclient from ubuntu-sso-client [14:02] nessita, 3.1+r922-42~precise1 [14:03] wow that's old... well, in nightlies is very messy to put dependencies using revno version [14:03] so we're just requiring ussoc >= 3.1 [14:03] nessita, oh, ok. === yofel_ is now known as yofel [14:03] alecu: you should update "by hand" as well, but the production package will be requiring ussoc 2.99.91 [14:03] so no worries there [14:04] nessita, so, this won't happen in the main repo, right? if it only happens on nightlies I don't care :-) [14:04] great [14:04] thanks! [14:04] :-) [14:05] dobey: did you make the tarball for protocol? [14:06] nessita: yes [14:06] dobey: buuuu you did not let me know :-) [14:06] I was staring at the version=2.99.91 and was confused [14:07] you were off playing with the kids at school :P [14:08] dobey: heh [14:10] i discovered that bzr merge-upstream does uscan for you, yesterday [14:10] dobey: yes it does, that's why we need to fix the watch file every time [14:11] dobey: so you uploaded protocol 2.99.91 to ubuntu as well? [14:12] i haven't uploaded it yet, no [14:12] dobey: will you or shall I? [14:12] i will [14:13] nessita, in the latest nightlies this file is missing: /usr/lib/ubuntu-sso-client/ubuntu-sso-proxy-creds-qt [14:13] alecu: oh... hum... let's see [14:14] nessita, and obviously sso fails when it tries to get the credentials. [14:14] nessita, also, control panel fails in an ugly way [14:14] alecu: how does it fail? when doing what? [14:15] nessita, (it pops up a dialog with that error, I'm trying to reproduce to write the steps) [14:15] alecu: yes, dialog popping up is the expected way [14:15] alecu: the dialog sometimes freezes, and I think that's caused by the same exec_ issue that mandel had [14:18] nessita, btw: the new control panel with the integrated wizard is awesome. I still don't like the cloud picture, and the text that's too bold, but it's awesome nonetheless :-) [14:18] nessita, gatox, ralsina: congrats! [14:18] alecu: too bold? [14:19] alecu: change your default font to non-ubuntu-font and it's fixed ;-) [14:19] ralsina, but my font is the default! :-) [14:19] alecu: thanks!!! [14:20] alecu: ok ok ok and thanks! [14:20] alecu: ok, great catch about the -qt binaries, will tweak packaging dailies now [14:21] alecu: the whole code for those binaries is under the same namespace as the qt UI, right? [14:21] oh no! [14:22] there is a ubuntu_sso/webclient/qtnetwork.py outside ubuntu_sso/qt [14:22] gah this will be complicated [14:22] nessita, don't know about namespaces. mandel? [14:22] alecu: how does the webclient chooses the implementation use? [14:23] nessita, ubuntu_sso/utils/webclient/__init__.py [14:23] alecu: hum, this name is no longer valid, no? "is_qt4reactor_installed" [14:24] nessita: let's do the 1:1 close to EOD so we can review how the releases went, ok? [14:24] ralsina: perfect [14:24] alecu: also, I think I tested this, and asking for QCoreApplication.instance() will give you not None even if there is a QApplication running [14:24] nessita, right: we should have used a different name there when we replaced the check with PyQt instead of qt4reactor [14:25] nessita, I know that mandel made a branch that took it out, then another to add it. [14:25] *to add it back. [14:25] alecu: trunk has it [14:25] nessita, right: he took it out as you or ralsina requested. Then he put it back [14:25] alecu: also... will you please help me with something? webclient/common.py:247 [14:25] alecu: why he put it back? [14:26] nessita: panic [14:26] nessita, 247, yes. [14:27] alecu: so... what if qt is not available, ergo de SSL_DIALOG program is not availble? [14:27] nessita: he said he got different objects on both tests, so it kinda scared him into putting it back in. Worst case it's redundant, not important [14:27] alecu: I know we don't need to support a non-qt dialog, but what will happen in that case? boom? nothing? [14:28] nessita, probably we would get a similar WebClientError [14:28] alecu: is there any chance to test that in the next 10 minutes? or is it too much to ask? [14:28] besides general usage testing and some of the non-auth proxy stuff i did the other day, does anyone have any particular areas they want me to look at on windows? [14:29] nessita, oh, wait. [14:29] ralsina: why did he panic? is the same instance https://pastebin.canonical.com/62722/ [14:29] alecu: /me waits [14:30] nessita, we disabled ssl overrides. Now any ssl error is a webclienterror. [14:30] alecu: not sure what that means, sorry :-$ [14:30] nessita, ubuntu_sso/utils/webclient/qtnetwork.py, line 85 [14:31] nessita: I would have to find logs but he was getting two different objects at some point. [14:31] alecu: seeing that line, but really not sure what that means IRL, when a user is actually using this [14:31] ralsina: ok, no need to do that now [14:32] nessita: right [14:33] alecu: requesting a new nightlies build that will ship the qt UIs for proxy support only when the ussoc-qt package is installed [14:33] will build in 9 minutes approx [14:33] nessita, sorry, Lucila was telling me about the mirror that we need to replace in our car. :P [14:33] nessita, she got her sideview mirror "crashed" too. [14:33] alecu: $855 is costing me to replace my mirror [14:33] :-( [14:34] nessita, our costs ~ARS 250 [14:34] alecu: since my car is under warranty, I can only repair it at the official shop [14:34] we are going to Av. Warnes in a few hours. [14:34] the chevrolet original mirror is $610, and the "mano de obra" $250 [14:34] a crazyness [14:36] alecu, yes, I did add it back 'cause I had an issue with it, but can be remove, I can double check [14:36] alecu, what about namespaces? [14:37] mandel, I don't care about removing those bits right now. nessita had some concerns because she requested only checking for the coreapp instance. [14:37] mandel, but in any case, it's harmless. [14:37] yes [14:37] mandel, regarding namespaces... [14:38] mandel, nessita asked: "the whole code for those binaries is under the same namespace as the qt UI, right?" [14:38] mandel, but now she realized the answer is "no" [14:38] because there are some bits that are in the webclient namespace. [14:38] alecu, which are 'those' binaries? [14:39] alecu, and if it is about qt, yes, we have code under webclient [14:40] mandel: so, we "can't" (is complicated) code that requires qt under webclient. Not code that do not support not having qt around... [14:40] alecu, mandel: wanna mumble about this? I need to solve it asap to release to ubuntu, and freeze is today [14:41] dobey, http://pastebin.ubuntu.com/893693/ [14:41] nessita, sure, give me 5 mins (updates are taking place) [14:41] nessita, I'll ping you as soon as mumble works [14:41] dobey, should I open a bug for it? [14:41] mandel: I m there [14:41] mandel, nessita: I'm fetching the laptop to mumble. [14:42] alecu, nessita, mandel: do you need me on that call? [14:42] alecu: yeah. and sigh. [14:43] although i guess that's a bug in pygobject [14:44] dobey, does this mean that no notifications will be shown at all? [14:45] alecu: you get that on precise? [14:46] dobey, up to date precise with nightlies [14:46] dobey, in fact, let me update the rest of precise, because I only updated the nightlies. [14:46] dobey, I'll ping you back. [14:48] nessita, ralsina, gatox: you may want to check this screenshot of the error dialog: http://ubuntuone.com/0gcDwz2oNBMDOrUNUj89R6 [14:48] alecu: yes, I saw that, I need to fix it [14:48] alecu, wow! [14:48] alecu: yes, saw that [14:48] ralsina, gatox: should I open a bug for it? [14:48] alecu: please and assign to me [14:48] ralsina, ack [14:48] alecu: yes please [14:49] alecu, it's a feature...... we really encourage users to hide the details [14:49] alecu: is the dialog also frozen sometimes? [14:49] :P [14:51] ooh! [14:51] nessita, alecu FYI waiting for updates to finish to be able to mumble.. [14:51] a progress bar! [14:51] * mandel hates libre office [14:52] mandel, click it off, dude! [14:53] (Stan: Dude, what the fuck is wrong with German people? ) [14:56] lol [14:56] alecu, nessita done, opening mumble, yet should we talk after the stand up? [14:56] alecu: weird. so i didn't get the "Foo is uploading" notification, but no TypeError in logs either [14:56] it's almost done [14:56] mandel: sure [14:57] (or maybe i just missed the notification because it pops up in a bad place) [15:00] me [15:00] me [15:00] me [15:00] me [15:01] alecu, dobey ralsina ? [15:01] dobey: with this new ussoc packaging issue, I will not be able to do the u1client packaging and upload to ubuntu today. I'm happy to do it next time, though :-) [15:01] me [15:01] me [15:01] DONE [15:01] oops [15:01] DONE: finished landing branches to complete the u1cp wizard. Proposed update-3-0 branches for ussoc, u1client, u1cp and u1windowsinstaller. [15:01] TODO: releases [15:01] BLOCKED: ussoc packaging may be broken [15:01] NEXT: gatox [15:02] DONE: [15:02] Fixed the reactor issue, couple of reviews, update the state of some issues that were already closed. Started working on Bug #944256. [15:02] TODO: [15:02] Finish with limit bandwidth bug, keep working on some issues in my queue. [15:02] BLOCKED: [15:02] urbanape: standup? [15:02] No [15:02] Launchpad bug 944256 in Ubuntu One Control Panel "Using ubuntuone-control-panel-qt defaults to limit bandwidth" [High,Triaged] https://launchpad.net/bugs/944256 [15:02] briancurtin, go [15:02] DONE: debugging, call, 1-1, add_to_autostart branch, some testing last night and this morning [15:02] TODO: insert add_to_autostart call in case where no wizard runs on startup, whatever else needs to be done for release, probably get back to Mac stuff [15:02] BLOCKED: none [15:02] NEXT: mandel [15:02] DONE: Fixed creds dialog not working on windows due to using exec_ in the dialog. Looked at proxy on windows and the diff issues we have with the proxy creds. Also found an IPC bug comming from sd in ralsinas last release. [15:02] TODO: Fix proxy support and proxy creds on windows. Move to next bug in the list. [15:02] BLOCKED: no [15:02] ralsina, please [15:02] no notes yet,sorry [15:02] DONE: broken wifi while traveling meant I had to take a forced freaky tuesday. Been doing IRL of the nightlies today [15:02] TODO: more IRL to get the nightlies in shape [15:02] BLOCKED: not today [15:02] NEXT: ralsina [15:02] meh [15:03] NEXT: dobeeh [15:03] DONE: started perf reviews, proposed --installer option, reviews, build windows for QA , manager-like-substances handled TODO: fix styling bugs on windows, herd cats, random other stuff BLOCKED: no [15:03] λ DONE: releases, uploads [15:03] λ TODO: u1client release, finish uploads, bug triage/fixing, some closed milestone reorg [15:03] λ BLCK: none. [15:04] any comments anyone? [15:05] ralsina: is urbanape with us today? [15:05] nessita: he should be, but I am locked out of canonicaladmin until I can get back to linux [15:05] so maybe I misplaced him [15:05] nessita: you forgot thisfred? [15:05] ralsina, nessita, he said earlier that he was with some problems.... let me check the logs [15:05] oh right [15:05] he's gone [15:05] dobey: he's off today [15:05] dobey: having a happy birthday I hope [15:05] thisfred is off the rest of the week [15:05] gatox: ack [15:05] yeah [15:06] just realized, when tab complete wasn't working :) [15:06] heh [15:06] dobey, that would have never happened with gatox, he would have pinged someone else :P [15:07] ralsina, nessita urbanape says: Hey, all. Will probably be out this morning. Lex still has pink-eye, and I need to run him to the doctor. He'll likely stay home, and hopefully Amber can take the afternoon off. [15:07] hehe [15:07] mandel, jejje [15:07] alecu, mandel: mumble then? [15:08] alecu: ping? [15:09] I'm there [15:14] uh oh [15:14] ze progress bar hasn't gone away [15:14] and i haven't gotten any notification about the upload [15:15] brb, quick run to the coffee shop (forgot to buy coffee at grocery store :/ ) [15:16] mandel: ping [15:16] ah, now there is a TypeError [15:16] weird [15:16] or anyone who can help me with SSO login, it broke when I moved from 11.10 to 12.04 -.- [15:16] broke? [15:17] stopped working [15:17] can you be more specific about what isn't working exactly? [15:18] ok [15:18] I was accessing the dbus interface in com.ubuntu.sso in /credentials [15:18] called com.ubuntu.sso.ApplicationCredentials [15:19] now I don't seem to have this service anymore [15:20] nessita: ^^ is this just the "old/deprecated api" getting removed? [15:20] apol, pong! [15:20] apol, how exactly? [15:20] dobey: one sec, mumble [15:21] mandel: hmm... I don't know what other info to give :/ [15:22] apol, sso has service activation, nessita is the master of this piece of code :) [15:22] mandel: it sounds like the old api being removed; and one needs to use /com/ubuntu/sso/blah/blah as the path now [15:23] so... what/where is the new api? [15:24] apol: please be patient, and nessita will help in a few minutes. :) [15:24] ah ah [15:24] sorry [15:25] apol: https://wiki.ubuntu.com/SingleSignOn/UbuntuSsoClient (will give more details after I finish this call) [15:26] * apol waits... :) [15:27] nessita: yes, that's what I was using before... maybe I need to install more packages? [15:28] ok, lunch time for me. bbiab [15:29] apol: did you read the paragraph that reads: [15:29] "NOTE: formerly, the ApplicationCredentials interface was implemented under the /credentials object path. That interface is deprecated and should not be used. However, current applications using this interface will be able to do so until the Ubuntu 11.04 release inclusive, since it won't be removed it until 11.10. " [15:29] * briancurtin chugs coffee [15:29] apol: so, the /credentials path is not the same as /com/ubuntu/sso/credentials [15:30] apol: /credentials path is deprecated, you should use /com/ubuntu/sso/credentials instead [15:30] ah [15:30] * apol tries... [15:30] apol: the API changes a bit, so the same calls as before will not work [15:31] apol: please check the API in that wiki page :-) [15:35] wonderful [15:36] dobey: would you please let me know if I need to do something specific to add the new dep python-openssl to ussoc? [15:36] (re: packaging) [15:38] ralsina: should we change SHOW_CMD (conf.py) back to False now or just right before the official release? [15:38] this new API has a method called register which is a C reserved keyword, so it's not possible to generate the C interface of your dbus [15:38] nessita: where you aware of that? [15:38] It's quite a common way to deal with dbus in Qt [15:38] briancurtin: I keep it True for QA builds and turn it off for release [15:39] apol: not sure what you mean... would you please explain a bit? [15:39] sure [15:39] ralsina: good plan [15:39] in Qt, to use a dbus interface, what you do is to generate the xml file of the interface using a line such as: [15:39] qdbus com.ubuntu.sso /com/ubuntu/sso/credentials org.freedesktop.DBus.Introspectable.Introspect > ubuntu_sso_dbus_interface.xml [15:39] then it generates a .h file [15:40] like this http://paste.ubuntu.com/893773 [15:41] apol: ack. So, the problem is...? [15:41] nessita: the problem is that there's a method called "register" and you can't have anything called "register" [15:42] I probably can find ways to workaround this, but well, it's still workarounding [15:42] apol: can't you generate the .h so the method names use the namespace they are in? in dbus you have hierarchy (service name, object path, interface) so you can group your method logically [15:43] well, the method is ComUbuntuSsoCredentialsManagementInterface::register [15:43] it's a method inside ComUbuntuSsoCredentialsManagementInterface class [15:43] apol: and, given that the method is under a specific path/interface, I don't see a conceptual reason to rename it... we can't rename methods considering all the languages reserved words... [15:43] ok then [15:43] never mind [15:44] apol: can you propose another name for it, that would make more sense in that context? [15:44] apol, what is the complies telling you? [15:44] such as sso_register (first name I came up with) [15:45] nessita, the problem that apol has is that if he wants to auto generate the api via the qdbus xml he finds a register method, since register is a key word in c++ the compiler is probably going bananas [15:45] dobey: for u1client, be sure to require ussoc >= 2.99.91, since the new use of webclient requires it [15:46] mandel: I understand that [15:46] mandel: http://paste.ubuntu.com/893779 [15:46] mandel: but what I say is that we can't rename our methods considering every language reserved keyword... [15:46] nessita, yeah.. I know, is a PITA coming from the c++ compiler [15:47] mandel: perhaps the generation can be tweaked and a given prefix can be added to every method name? [15:47] such as sso_ [15:47] register_credentials? [15:47] nessita, just what I was going to say! [15:47] nessita, stop reading my mind! [15:47] mandel: lol [15:47] apol, yes, or even just register_ to be closer to the dbus definition [15:47] apol: just FYI, the method goal is not registering credentials but registring a new sso account [15:48] oops, so many typos, sorry [15:48] apol: can you add a fixed prefix to every method? [15:48] i don't really know [15:48] I'll investigate [15:48] apol, qdbus does not have many params, but what about adding an extra step in the middle, grab the xml and append _ to any cpp keyword? [15:49] -sys.maxsize on prefixing methods something. i would need to have lazer eye surgery [15:49] s/something/with something/ [15:50] mandel: well, I'm seeing if qdbusxml2cpp has something like that [15:50] apol, are you cheking the generated files in a source control? or are you doing this as a make/cmake step? [15:50] cmake yes [15:52] apol, afaik you should not be running that as a make step, but just do it the first time and then adapt the generated code [15:52] apol, since there is no guarantee that the machine compiling the code will have the sso dbus interface [15:52] i know [15:52] I have the xml versioned [15:53] what I do compile-time is to run qdbusxml2cpp [15:55] nessita, ralsina, mandel: good news: nightlies u1cp, sso and sd are crossing an authenticated proxy just fine. [15:55] alecu: AWESOME NEWS [15:55] alecu: really, great great news [15:55] alecu: awesome! [15:55] alecu, I'm updating the clean vm and will try the software center behind a proxy [15:55] apol: just out of curiosity, what are you doing with it? :-) [15:56] ralsina: I'm working on kubuntu's software center, I'm using it to login for the review and ratings service [15:56] apol: great! [15:57] apol: my next sunday project is better integration with KDE (maybe in a month of two) [15:57] apol: but I need to refresh my C++ *a lot* [15:58] ralsina: if I/we can help you, don't hesitate to get in touch :) [15:58] apol: will do! [15:59] apol: you may want to try this for a taste of things to come: https://launchpad.net/u1-toys [15:59] gatox: something happened with the reset password page [15:59] gatox: the error is no longer shown :-/ [15:59] :) [16:00] nessita, mmmmmm is there any bug for that? [16:00] gatox: opening [16:00] nessita, thanks [16:01] gatox: https://bugs.launchpad.net/ubuntu-sso-client/+bug/961315 [16:01] Ubuntu bug 961315 in Ubuntu Single Sign On Client "Qt UI: when resetting my password using an invalid reset code, the operation fails and no error is shown" [High,New] [16:01] nessita, thanks [16:05] dobey, I've tried dist-upgrading the whole Precise VM, and I still get those GI errors when notifications are to be shown. [16:05] dobey, the only issue is notifications not being shown; SD keeps working fine otherwise. [16:07] dobey, I'm creating a new bug for this... shall I assign it to you? [16:08] dobey, or should I assign it to the notifications GI project? [16:09] I have to switch to linux to test some styling bugs, should be back in a few minutes. Keep talking to me, I have logs ;-) [16:10] alecu, dobey: I packaged ussoc and tested it on a clean VM (without proxy involved). Will upload as soon as dobey gives me some ack about the python-openssl package [16:13] nessita: another question, why do you use "a{ss}" maps instead of "a{sv}"? It would make sense to me if it always were strings [16:13] but considering that the winid has to go there, a{sv} would make more sense [16:13] apol: they are always strings so we can use the dbus bindings from python [16:13] apol: using variants from python is more complicated than what we need [16:13] ugh xD ok [16:14] apol: so, the windid is expected as string and cast to int [16:14] casted* [16:16] u1cp/gui people: i added a call to UbuntuOneWizard.done to add SD to the Windows autostart. i need the same functionality to occur for users who don't run through the wizard (they already have creds) - i'm not sure where this should go [16:16] briancurtin: you added what where? :-) [16:16] briancurtin: not sure I understood [16:16] nessita: https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/add_to_autostart [16:17] i took add_to_autostart from u1-win-installer and added it to the current u1cp Wizard, so SD will startup automatically [16:17] but it only does so in the case where it's a first time install. if you are upgrading and already had creds, we don't currently add to the autostart, but need to find a place to [16:18] briancurtin: why not add that code to the installer itself? [16:18] nessita, I've not installed the ubuntu one control panel, that is, I'm in a clean P vm, and ubuntu-sso-client-qt is installed, is that correct? [16:19] mandel: unless something is pulling it in, which I doubt, you should not have -qt [16:19] mandel: you sure? [16:19] nessita: im not sure if we can do that - looking [16:20] nessita, 100% sure [16:20] nessita, I just added nightlies via add-apt-repository ppa:ubuntuone/nightlies [16:20] briancurtin: is extremely difficult to know, when there are crendentials in the system, if the user just installed [16:20] nessita, and did a sudo update-manager -d to make sure I have the latests of the latests [16:21] mandel: something in our nightlies is pulling that in... but definitely is not the case in ubuntu [16:21] 110% sure [16:21] nessita, I'll remove it then [16:22] otherwise the platform team would be screamiung at our faces [16:24] dobey, bug #961342 [16:24] Launchpad bug 961342 in Ubuntu One Client "GI error while trying to show notifications" [High,New] https://launchpad.net/bugs/961342 [16:25] alecu: you can assign to me and i'll look into it [16:25] done. [16:25] ralsina: do you know what I need to add to the debian pacakging to deal with the python-openssl license isue? [16:25] nessita: just add the python-openssl dep [16:26] oh, a dobey is here! :) [16:26] nessita, nevertheless, requesting sso creds over auth proxy in software center fails with an error stating 'Authentication failed' [16:26] dobey: didn't you mention we needed to "grant" the license or something like that? [16:26] nessita, which is the 407 error from the proxy [16:26] mandel: aaaaarghhhh so confusing :-/ [16:26] but I guess we'll have to settle with that [16:27] nessita, it does not brake in a crazy manner, but I guess that we can add a bug to return an exception with a more decent error [16:27] ralsina, bug #961348 [16:27] Launchpad bug 961348 in Ubuntu One Control Panel "Huge button on the dialog shown on errors" [Low,Confirmed] https://launchpad.net/bugs/961348 [16:27] nessita: we need to tweak the license text a bit to add an exception yes; but we'll do it after this release. anything that needs it now, just add it. [16:27] mandel: yes, like "proxy in the way, can't do anything else" [16:27] dobey: ack then [16:27] uploading before leaving for lunch [16:28] nessita, I'll add the bug and tag it with the u1-tag, should be a simple one to fix [16:28] nessita: we can add to the autostart via installer, but we should really only do it after the user has logged in. is there a good place in u1cp that would only execute right after login? [16:28] nessita, one questions, if this messages are shown in the ui, shouldn't they be translated? [16:28] nessita, not now, not another UIFe [16:29] hey all: I need to run some car related errands, will be back later. [16:30] briancurtin: yes, ubuntuone/controlpanel/gui/qt/controlpanel.py -> on_wizard_finished [16:30] briancurtin: but I guess is the same you had before? [16:31] briancurtin: there is no place to detect when there was credentials in the system *already*, but the controlpanel just installed [16:31] I added it to wizard.py. looking at controlpanel.py now [16:32] briancurtin: is basically the same, though I like it a bit better on the controlpanel. Also, please make sure to add a test confirming the call to the add_to_autostart is made [16:33] nessita: thanks, i'm looking into adding it here [16:33] (here = on_wizard_finished) [16:33] nessita, FYI bug 961357 [16:33] Launchpad bug 961357 in Ubuntu Single Sign On Client "Proxy authentication errors should be clearer" [Medium,Triaged] https://launchpad.net/bugs/961357 [16:34] ralsina, FYI moving back to windows === elopio_ is now known as elopio [16:44] * gatox lunch [16:48] briancurtin: I just realized there is a much better option [16:48] i'm interested [16:48] briancurtin: I think you need to call to autostart inside the controlpanel's main when the --installer option was passed... [16:48] briancurtin: though now that I'm writing that, we still have the "new credentials" issue, no? [16:49] nessita, mandel: I have it working now, thanks for the help [16:49] apol: nice!!! [16:50] it would be cool if you looked into the register renaming though, the code looks really ugly :( [16:50] apol, superb! [16:50] nessita: in main via --installer does seem better, i'll look into it. i'll see what it would have to do for new creds - maybe it just stays inside the wizard's done function as well (it's harmless to call it multiple times) [16:50] apol, problem is, we not only have cpp clients.. and since we mostly we use python we did not see that comming [16:50] apol, good things is, you wont get one called if/while/try/except etc :P [16:51] mandel: well, take it as positive criticism [16:51] if you want people to use it, at least [16:52] apol, I know, I know, era broma :) [16:52] mandel: thanks! [16:52] apol, problem is, we are very late in this cycle, but.. why don't you add a bug? [16:53] where would I report such a bug? [16:54] apol, here https://bugs.launchpad.net/ubuntu-sso-client/ [16:55] apol, go to report a bug and explain all the details, we will later deal with it (not at the moment due to the deadline) [16:56] i'll see what I can do [16:58] ralsina, turns out that automatic detection on windows is 'heavier' on windows than manual, which meant that altough I had manually set up the proxy it was not used.. [16:58] ralsina, puto windows! [16:59] apol, sorry for the burden, but trust me we will do our best to get it working [16:59] mandel: no sure what that means. When you set to "automatic" the manual is disabled [16:59] ralsina, yes [16:59] ralsina, and is a tiny little check box.. [17:00] mandel: he, ok, so now you know :-) [17:09] dobey: ussoc 2.99.91 uploaded, if you want to move forward with u1client [17:09] (which must depend on that) [17:10] nessita: ok, thanks [17:15] ralsina, I found the bug on windows, applicationProxy on windows returns a hostName that is '' therefore the credentials manager complains because you cannot have '' as the key of a pwassword [17:15] password* === gatox is now known as gatox_lunch [17:22] mandel: WTF? [17:22] oh, got it [17:22] ralsina, and to make it more annoying the proxy type is 2, which means that qt is not able to detect the proxy.. [17:22] I read that as the hostname beig "therefore the credentials manager..." [17:23] ralsina, that would be an awesome proxy name [17:23] lunchtime on this end [17:23] ralsina, but, yeah, proxy detection, the way we have it, is broken, but there is a fix [17:27] alecu, it seems that the proxy setting on windows return without a proxy, which is a proxy with hostname '', port 0 and type 2 [17:27] alecu, it means that we might have to query the system to set the proxy in the setup [17:28] gotta run for a few minutes, brb === vds` is now known as vds [17:55] ralsina, FYI windows bug fixed, setting tests and will propose [17:55] mandel: awesome! [17:56] ralsina, I hate that qtreactor, makes debugging really hard with pdb [17:56] mandel: you mean the constant "event loop already running"? That's Qt actually [17:56] mandel: I just 2> /dev/null [17:57] ralsina, yeah, is what I do too.. on linux, but windows.. [17:58] mandel: yes, on windows, you just can't [17:58] mandel: maybe with bash for windows you can [17:59] ralsina, well, is sorted so lets get this merged hehe [17:59] mandel: ack === gatox_lunch is now known as gatox [18:02] oof, hello, all. [18:03] briancurtin: ping [18:03] urbanape: pong [18:03] heya, you up for any pairing today? [18:04] urbanape: trying to add the last half of a branch for release, then yes [18:04] awesome, thanks. [18:04] I'll be here. Amber's watching Lex for the rest of the afternoon and tomorrow morning. [18:08] sweet, got it working. commence testing [18:09] gatox: so, I'm doing the review for https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/backend-problem/+merge/98513 [18:10] nessita, aja [18:10] ralsina, do you mind if I EOD and propose tom? [18:10] gatox: any chance you do not remove and add the main.py, but bzr move it? the way is now, the diff (and the bzr history) will show that file as removed and a new file added [18:10] gatox: so, the ideal way to handle that is (in a clean branch): [18:10] bzr mkdir ubuntu_sso/qt/main [18:11] bzr move ubuntu_sso/qt/main.py ubuntu_sso/qt/main/__init__.py [18:11] mandel: no, it's ok [18:11] and then apply your changes [18:11] gatox: same for the test file [18:11] bzr mkdir ubuntu_sso/qt/main/tests [18:11] nessita, in a clean branch or this one? [18:12] bzr move ubuntu_sso/qt/tests/test_main.py ubuntu_sso/qt/main/tests/test__main.py [18:12] gatox: I don't think you can do this in this one [18:12] since you already remove it and re add it [18:13] ralsina, ok, thx! tom you will have the working version, I'll ask you and alecu for reviews :) [18:13] all., EOD here! [18:13] see you tom! [18:14] nessita, but maybe i could revert that..... ok.... i'll try in a new branch [18:14] gatox: if you can revert, you're welcome to [18:14] I have no idea how to do revert that and preserve the changes [18:22] nessita, so, here it is: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/main-moved/+merge/98703 [18:22] nessita, i think you should accept that before, so i can merge my backend branch with trunk [18:22] nessita, or should i mark this one as prerequisite? [18:23] what do you prefer? [18:23] gatox: a single branch, of course :-) [18:23] gatox: merge your changes in that branch ;-) [18:23] nessita, and then drop the backend branch? [18:23] gatox: exactly [18:24] gatox: the branch originally should not have removed and re-added files [18:24] nessita, ok, i'll do that [18:24] gatox, nessita, alecu: two very simple branches for review, just qss changes, with pictures: https://code.launchpad.net/~ralsina/ubuntuone-control-panel/fix-zion-inverted/+merge/98701 https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix-zion-inverted/+merge/98696 [18:24] gatox: we should bzr move files when we're moving then [18:24] ralsina, on it [18:24] them [18:30] ah great. not even going to try to explain it, but this add_to_autostart will be a fun merge. [18:31] briancurtin: enjoy! and welcome [18:31] dobey: would you please let me know when u1client is uploaded? I'm trying to avoid having it in source dep wait [18:32] nessita: what needs the new version? [18:33] dobey: nothing specific, but you know how much I love to depends on the latest version for consistency sake (controlpanel does depend on 2.99.91 ussoc) [18:34] nessita: stuff should only depends on the newest version necessary. [18:36] dobey: yes, I usually agree. But since we're gonna say "version 2.99.91 of U1 supports proxy" I would like the controlpanel 2.99.91 to have the same proxy support as u1client, but if u1client is not 2.99.90, that will not be true... [18:37] (since controlpanel web request will have proxy support, but syncdaemon will not work) [18:37] Also, https://code.launchpad.net/~ralsina/ubuntuone-control-panel/u1cp-windows-styling/+merge/98704 should cause no visible changes in linux, fix bug #961229 on windows [18:37] Launchpad bug 961229 in Ubuntu One Control Panel "On Windows, control panel has focus styling artifacts" [Undecided,New] https://launchpad.net/bugs/961229 [18:37] ralsina: any reason why the underline was removed from the tabs when focused? [18:37] ralsina: visually, not in code I mean [18:37] nessita: on what branch? [18:38] ralsina: I noticed that since a couple of days ago [18:38] nessita: hmmm I have the underline here [18:38] ralsina: no particular branch, just IRL [18:38] ralsina: when tabbing into a tab? [18:38] ralsina: not when hovering it [18:38] gatox: you let me know when the branch is ready? [18:39] nessita, yes.... i'm testing it IRL.... because something went wrong and it wasn't working the new branch now [18:39] ack, thanks [18:39] nessita: yes, by tabbing. But then again, I have not been in ubuntu much lately, so maybe there is something broken there. I do have them in KDE right now [18:40] ralsina: hum... [18:40] I don't have them in ubuntu... [18:40] Let me switch to unity since I have to test some stuff there [18:40] (just double checked) [18:40] ralsina: nice [18:41] nessita, done: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/main-moved/+merge/98703 [18:41] ack [18:42] gatox: also, can you please explain in the commit message how you fix the backend getting stuck? :-) when that goes to the changelog, does not provide that much info to someone reading the release notes [18:42] nessita, ok [18:47] * alecu is back [18:48] nessita: I have underlines usig 3.1+r292-49~precise1 [18:48] nessita: I have not, however, updated the system in a few days, I only updated u1cp and dependencies today [18:49] ralsina: is your system up to date? [18:49] ah, je [18:49] that ;-) [18:49] ralsina, is this necessary? https://code.launchpad.net/~ralsina/ubuntuone-control-panel/fix-zion-inverted/+merge/98701 [18:49] gatox: oh, yes [18:49] ralsina, in the other branch you are setting background-color: white to QWidget [18:49] gatox: look at the screenshot [18:49] spinbox should inherited [18:49] gatox: that is on sso [18:49] nessita: the new binary package for u1client will hold it up a bit btw [18:49] ralsina, ahhhhhhhh my bad [18:50] ralsina, only look at the code.... thought that was the saame project [18:50] gatox: if I set qwidget to white here, I break all the app [18:50] dobey: ah, right... [18:50] ralsina, roger that.... i thought it was the same qss..... [18:50] but it's uploaded now [18:53] dobey: thanks [18:54] gatox: could you please commit with --fixes and the bug number for the branch https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/main-moved/+merge/98703 ? also, in the commit message it should specify that the fix applies to windows only [18:54] ok [18:55] (otherwise people from linux, suffering from this - like perrito - will think we fixed it for them as well) [18:55] which so far we did not :-/ [18:58] nessita, i realize there wasn't a bug for this.... the bug was only for linux..... creating now [18:58] gatox: thanks [19:01] nessita: I am not sure I understand what you mean by "I wonder why you added a new method start_from_license instead of redefining setStartId in the QWizard" [19:02] ralsina: this question may be a non-sense, I might be getting something wrong from the qwizard documentation... but why adding a custom method when the api provides one? [19:02] nessita: if you change the startID outside __init__ you need to restart the wizard [19:02] which is exactly what the new method does [19:03] ralsina: right.. but in my head, by doing that (providing a method in our custom wizard), you're leaking info to the mainwindow about the controlpanel widget structure [19:03] ralsina: so it would make more sense (to me, please correct me) to have the start_from_lincese in the controlpanel widget [19:03] nessita, done [19:03] nessita: I could add that method to contrlpanel instead [19:03] right [19:03] and let le controlpanel widget do whatever it considers necessary, like doing startID on the wizard and restarting it [19:04] ralsina: so no need to add the method in the wizard itself, given that its API already provides what is needed [19:04] no? [19:04] gatox: ack [19:04] makes it awkward to test, and means I have to add a installer parameter to control_panel too [19:04] or no, forget that [19:04] ralsina: why? [19:04] ok, forgotten [19:04] :-) [19:04] I'm just saying to have: self.ui.control_panel.start_from_license() [19:04] right [19:04] instead of accessing the "internal" (not necessarily hidden) wizard [19:05] bah, am hungry again [19:06] nessita: usually, I don't consider UI components private (or I would call it _ui instead of ui) but sure, I will do that and keep it in mind for the future. [19:13] great, crashing test. [19:13] eh, maybe its not me. the baseclass crashes as well [19:17] is u1cp ubuntuone.controlpanel.gui.qt.tests.test_gui.MainWindowTestCase currenty passing on Linux? [19:18] nessita, alecu: https://code.launchpad.net/~mandel/ubuntu-sso-client/libsoup-ssl-dialog/+merge/97057 should be 'rejected' right? [19:19] dobey: not sure... it also has several branches depending on it [19:19] briancurtin, yes, they are working on liunx [19:19] briancurtin: yes [19:20] nessita: but we aren't doing ssl proxies, because nothing supports ssl to the proxy, right? [19:20] i know this works IRL but the tests crash python :/ [19:20] dobey: right [19:21] dobey, right, but that branch checks ssl against the server, not against the proxy. [19:21] oh [19:21] hmm [19:22] dobey, and for a library that might be useful at some point if we want to allow the user to accept invalid certificates for some servers. [19:22] dobey, but certainly we do not want users to accept invalid certificates for *our servers*. [19:23] alecu: unless of course, it's connecting to a different server that isn't *our* server [19:23] so the branch is not needed right now, but may make sense at some point if we want our desktop apps to download say, pictures or videos or something from user defined servers. Or something like that. [19:24] well, anyway; it's probably good to note, that practically nobody gets ssl right, anywhere. [19:25] but it's gotten more right than proxies usually. [19:25] also; fml, dandelions grow way too fast :( [19:25] only because proxies are always wrong [19:29] if anyone has a minute for lp/bzr question, can you let me know how to un-screw this up: https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/add_to_autostart -- rev293 is ralsina's branch that i had applied and didn't know it would get pushed (294 used to be called something different, but i remerged onto a clean checkout) [19:29] i might just export it as a diff and just start over... [19:30] briancurtin: that's what I would do [19:34] ralsina: since it depends on your --install branch, once i get a clean u1cp branch, how do i apply your branch but not have it be a part of any work i checkin/push to my own branch (so yours is dependent) [19:35] briancurtin: you merge my branch and in the proposal declare it as prerequisite [19:36] ralsina: ah, so it's lp that figures out that it's not my work? [19:36] briancurtin: right, the proposal does it [19:36] ralsina: cool, thanks [19:39] briancurtin, usually when I merge from a side branch I put the branch name in the commit, so it's easier to track. Like "merged from lp:~ralsina/project/branch" [19:48] nessita: I get some strange test failures in installer-option and can't find why. Have a minute to give me a hand? [19:49] ralsina: sure [19:50] nessita: I am getting failures in UbuntuOneWizardCloudToComputerTestCase which I have not touched [19:50] ralsina: have a trace? [19:51] nessita: https://pastebin.canonical.com/62768/ [19:52] ralsina: wanna push the branch that is giving you that? [19:52] they look like legit "test" errors... but if you have not changed anything... [19:52] nessita: pushed, it's lp:~ralsina/ubuntuone-control-panel/installer-option [19:52] branching [19:52] nessita: the tests there are a bit weird. For example, the "button behaviour" tests depend on absolute page IDs [19:53] nessita: which is why I had to shift them all when I added a page at the beginning [19:53] ralsina: yes, they do. We can improve them by putting a name instead of the id and in the test doing the getattr in the self.ui [19:53] I can do that after I get rid of these :-) [19:54] ralsina: at the time, I consider that a good intermediate solution between time-results, considering that I thought the page order will not change (I don't like adding the license page if we're never showing it...) [19:55] nessita: I *could* add the page in start_from_license... [19:55] nessita: but that's even hackier [19:55] ralsina: right [19:55] so, no [20:00] ralsina: if you test your branch IRL, the test failure are reflecting IRL bugs. For example, in the cloud-to-computer page, the back button is gone [20:00] nessita: hmmmm ok, I'll dig then [20:00] ralsina: you forgot to remove something: [20:00] self.setButtonText(self.NextButton, self.next_button_text) [20:00] AttributeError: 'UbuntuOneWizard' object has no attribute 'next_button_text' [20:01] nessita: where's that? [20:01] ralsina: grep for it, i got that in the test run [20:01] nessita: ok [20:01] from the trace: [20:01] File "/home/nessita/canonical/controlpanel/review_installer-option/ubuntuone/controlpanel/gui/qt/wizard.py", line 252, in initializePage [20:01] self.setButtonText(self.NextButton, self.next_button_text) [20:01] AttributeError: 'UbuntuOneWizard' object has no attribute 'next_button_text' [20:02] nessita: got that one, and that explains it. Thanks! [20:02] ralsina: still 2 tests failures, they may be legit [20:02] ralsina: my guess is that a CommitButton does not make currentIdChanged be emitted [20:03] but some other signal [20:03] nessita: well, that's totally undocumented anyway [20:03] it's an implementation detail of QWizard, really [20:07] ralsina, nessita eod for me..... and i really need to go to the university to an import errand..... i'll try to apply nessita's comments to the backend branch tonight [20:07] gatox: ok, thanks! === gatox is now known as gatox_away [20:10] ralsina: why changing the next button to be a commit button? [20:12] nessita: so the user can't go back [20:12] you can't "disagree" on a license [20:13] ralsina: but the way it was before, you couldn't go back either. Though I understand is more correct using CommitButton [20:13] nessita: before there was no way to see the page, so I am not sure what you mean :-) [20:15] nessita: you mean having a next and removing the back in the next page? [20:16] I am just trying to use the features provided instead of doing things manually as much as possible [20:16] ralsina: https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/windows-autostart/+merge/98720 -- i'm confident in the branch via IRL testing, but due to some GUI tests crashing on Windows (higher up the chain that my test) i'm not exactly sure what to do [20:17] briancurtin: you can run your specific tests [20:17] ralsina: even running my specific test fails due to a class higher up the inheritance chain [20:17] briancurtin: using u1trial, although I am fuzzy in the details beyond running all the tests in a single file by u1trial --reactor=qt --gui foo/bar/test_stuff.py [20:17] briancurtin: oh, that sucks [20:18] briancurtin: you get the same failure in trunk? [20:19] ralsina: yes, since we already are customizing every button layout on every page [20:19] ralsina: you could see the license page by manually changing the show_license to True and running it from source [20:19] was working great :-) [20:20] nessita: hahaha, ok, I just tried to make it nicer, and use the features we have [20:20] nessita: but I can revert if you really feel it's the right thing to do [20:21] ralsina: not really, I just did not know about commitpage [20:21] commitbutton [20:21] commit* [20:21] yeah, ok, I have the tests fixed now, need to add one for start_from_license and I'm done with it [20:22] ralsina: yep, with trunk it fails [20:23] briancurtin: then if with good IRL tests it passes, the tests should not block you, I guess [20:23] nessita: are you ok with landing briancurtin's page with preexistant failing tests in windows? [20:23] nessita: I am starting on test-fixing on friday [20:23] ralsina: yep, i've tested on 7 and XP, locally and via a bundled installer [20:24] ralsina: can I see the trace? [20:24] briancurtin: ^ [20:24] nessita: there's no trace per se...it crashes the Python interpreter [20:24] briancurtin: ah, that's very likely a qt seg fault [20:24] briancurtin: are you patching "everything" that you should? [20:24] :-) [20:24] briancurtin: I can look at the branch to confirm that [20:25] what branch is it? [20:25] nessita: https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/windows-autostart/+merge/98720 [20:26] branching [20:26] briancurtin: which one is the last test being reported in the terminl? [20:27] nessita: when running a clean u1cp trunk it fails - i'm not aware of any current branches that need to be applied for that, but i could have missed it [20:27] briancurtin: no branch pending is supposed to fix it [20:28] nessita: it gets to test_animation_is_active (AccountPanelTestCase), but the issue is further down in some base class. the test that i added fails, then i ran the tests of the base class and they fail, and on up the chain [20:28] but that's just very early on in the tests to be run [20:28] briancurtin: how are you running the suit? [20:28] suite [20:29] nessita: i started by just doing run-tests to run everything. even when i run just the individual test i added (by doing u1trial and passing -t with the exact class to run) it crashes [20:29] briancurtin: are you passing -r qt4 --gui? [20:29] if you do not use --gui, UI tests will explode [20:30] booting my windows VM now [20:30] nessita: i'm 99% sure i was doing that but it's past my command history (or i closed the window). i will try again and be 100% sure to use that [20:30] nessita: finished fixing installer-option. Calling dibs on your morning review slot tomorrow [20:31] ralsina: dibs granted [20:31] ralsina, briancurtin: overall, I think we can land a branch if all is green in linux, but I think we need to fix this (sooner or later) [20:32] so will try to debug a little [20:34] nessita: yep, i'm running with --gui [20:34] nessita: python ..\..\bin\u1trial --reactor=qt4 --gui -p"ubuntuone\controlpanel\dbustests" -i "test_linux.py, test_libsoup.py" ubuntuone [20:36] also crashes when i give it a -t to filter down to specific tests (like the one i added) [20:36] nessita: yes, I filed a bug for these test failures yesterday, I think [20:37] briancurtin: reproduced, will give it 5 minutes of debugging [20:39] ja, found out what;'s the proble,m [20:39] but have no solution to propose [20:40] briancurtin, ralsina: this avoid the crashes http://pastebin.ubuntu.com/894239/, but is not sematically correct to remove that line... [20:40] ralsina: any idea why showing a widget makes qt seg fault? [20:40] nessita: not the slightest [20:40] nessita: unless we are not having a QApplication for some reason [20:40] will add a print before the .show() call [20:41] nessita: god news are, a core dump on the crash should give useful info [20:41] ralsina: this is windows we're talking about, no? [20:41] :_P [20:41] nessita: oh, right, windows [20:41] nessita: me engolosiné despues de 4 horas de ubuntu [20:42] ralsina: when adding these prints: [20:42] 271 print '\n\n\n===========================', repr(QtCore.QCoreApplication.instance()) [20:42] 272 print '\n\n\n===========================', repr(QtGui.QApplication.instance()) [20:42] I got: [20:42] =========================== [20:42] =========================== [20:43] nessita: ok, so. We could bisect the failure, I guess [20:43] ralsina: so the seg fault is a pain in the butt [20:43] ralsina: bisect how? [20:43] ralsina: we could get a nice dump file if i ever have time to finish https://bitbucket.org/briancurtin/minidumper [20:43] ralsina: every single ui tests fail with that [20:43] nessita: try revision 100, see if it fails, and so on [20:43] nessita: we used to have working windows tests in the past, we could find the revno that broke them [20:43] ralsina: I added the self.ui.show() in the past 4 weeks [20:44] so clearly that broken it [20:44] but why self.ui.show() will break it? [20:44] they weren't broken too long ago. i can run the manual bisect if you want [20:44] nessita: no idea whatsoever, but why did you have to add it? [20:44] briancurtin: you'we welcomed to [20:45] briancurtin: you can run bzr blame to see the change for that line, so we can confirm that broke it [20:45] ralsina: 2 reasons: several tests were doing the self.ui.show() in them and the adding a cleanup for hiding... and also, I consider that semantically is more correct to test how a widget *looks*, which requires the widget is shown, no? [20:45] ralsina, briancurtin: 100USD bzr blame says nessita ;-) [20:45] actually, nataliabidart [20:46] nessita: the problem may be that we are faking something and then we are showing() something that's not a widget somewhere [20:46] * nessita knows she added it [20:46] ralsina: not in this tests [20:46] these* [20:46] nessita: the only things that need showing are checking for geometry and isVisible() [20:46] we're faking backends [20:47] ralsina: leaving the discussion of to show or not show appart for a while, qt should not seg fault when showing a widget, no? [20:47] nessita: it usually doesn't :-) [20:48] ralsina: this is the trace http://pastebin.ubuntu.com/894254/ [20:48] (what I could get from windows, of course) [20:48] hehe [20:49] Oh, right, old 0a9e! (not really) [20:49] nessita: BTW, you probably should update your Qt/PyQt [20:50] ralsina: to which version? [20:50] nessita: Qt 4.8.0 PyQt 4.9.2 IIRC [20:50] 4.9.1 [20:51] nessita: rev 264 http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-control-panel/trunk/revision/264.2.11 [20:51] not that it will help here, of course [20:51] briancurtin: so yes, adding the show breaks it [20:51] I won! I won! [20:52] ralsina: found the problem [20:52] nessita: what's self.ui.destroy? [20:52] briancurtin: instead of removing the show, change this: [20:52] ralsina: YES :-D [20:52] use deleteLater! [20:52] 300 self.addCleanup(self.ui.hide) [20:52] ralsina: what's deleteLater? [20:53] ralsina: can't we delete it ASAP? [20:53] nessita: deleteLater deletes it after it gets to the event loop [20:53] ralsina: is there some method to deletes it sonner? (when calling it) [20:54] the problem is, if you remove the object before reaching the event loop, then maybe a signal is triggered with a stale pointer [20:54] nessita: it will delete it soon enough :-) [20:54] ralsina: not really... is not deleted until the whole suite finishes [20:54] I just tested it IRL [20:54] nessita: i think our line numbers aren't matching up. do you want that as the last line of the setUp right there? [20:54] add a processEvents as cleanup [20:54] briancurtin: sorry, was doing tons more of tests, will give a proper diff [20:55] a cleanup that does thing.deleteLater() ; QtCore.QCoreApplication.instance().processEvents() should delete immediately [20:55] ralsina: ack [20:55] nessita: if even with that it's not deleted, then we may be keeping a reference somewhere [20:56] that worked [20:56] briancurtin: diff is: [20:56] nessita: http://stackoverflow.com/questions/4888189/how-delete-and-deletelater-works-wrt-to-signals-and-slots-in-qt [20:56] http://pastebin.ubuntu.com/894265/ [20:59] briancurtin: with that diff ^, I got: FAILED (errors=16, successes=988) [20:59] which is much better than a crash :-D [21:00] nessita: ah, there we go. working [21:00] briancurtin: and I will leave that fix to you, since symlinks are involved ;-) [21:00] nessita: yes, my favorite part of python on windows! [21:00] nessita: are cleanups stacked? So it's hide->deleteLater->processEvents? [21:01] ralsina: yes, guaranteed [21:01] nessita: cool then [21:01] briancurtin: merry christmas!!! [21:01] * ralsina goes play Another One Bites the Dust [21:02] ralsina: let's do 1-1- tomorrow? [21:02] before the weekly call [21:02] nessita: yeah, I am drained [21:02] me too, and I need to clean my habitat a bit [21:02] dust and pelusas are eating me [21:02] nessita: and still have to fix a bunch of crap on windows, which I really don't want to [21:03] but it's taking shape :-) [21:04] yes [21:04] ok, I'm off crowd [21:04] see ya all tomorrow [21:04] bye nessita! [21:04] bey all! [21:07] later all [21:09] bye dobey [21:10] briancurtin: if you get to propose that branch today, I can add it to tomorrow's installer for QA [21:10] i'm proposing the symlink branch now, will be proposing the branch with nessita's fix in a minute [21:10] specially since the test failures don't seem to be IRL fails [21:10] beuno: cool [21:10] oops, that was for briancurtin [21:19] ralsina: https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/nessitas-crash-fix/+merge/98735 and https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/no-symlinks-on-windows/+merge/98736 are ready [21:20] now that i have this nessita fix i'm going to try out the autostart tests that i was ignoring before when it was broken [21:20] globally approved the crashfix [21:22] briancurtin: on the symlink test, it's better if you move that part to a separate test and make it skipIfOs("win32", "No symlinks on windows" ) [21:23] ralsina: noted - i'll fix that after the other branches [21:23] Or maybe doing that is a pain in the ass [21:23] wait, it is a pain in the ass [21:23] since that's not a test, and is rather part of the setup [21:24] approving what you have [21:36] and EOD for me [21:37] byr briancurtin and whoever else is still around :-) [21:37] ralsina: i'll mail you with status and branches once im done [21:38] and the test i added now works with all of the other branches, so i'll repropose the autostart branch