ralsina | alecu: 184.82.108.14 port 8888 | 00:30 |
---|---|---|
alecu | lovely, thanks. | 01:53 |
=== JackyAlcine_ is now known as jalcine | ||
=== jalcine_ is now known as JackyAlcine | ||
=== JackyAlcine is now known as jalcine | ||
=== jalcine is now known as JackyAlcine | ||
=== JackyAlcine is now known as jalcine | ||
=== jalcine is now known as JackyAlcine | ||
=== JackyAlcine is now known as jalcine | ||
=== jalcine is now known as webjadmin_ | ||
=== webjadmin_ is now known as jalcine | ||
JamesTait | Good morning all! Happy Friday! :D | 09:36 |
mandel | morning all! | 09:37 |
czajkowski | Aloha | 10:21 |
gatox | good morning! | 11:20 |
popey | dobey: when you're about, if you get a chance could you take a look at http://paste.ubuntu.com/886159/ which shows (line 1060) that rb is scanning a folder I dont want it to. This is resulting in double copies of my music in rb ☹ | 11:21 |
mandel | gatox, morning! | 11:23 |
mandel | so, is anyone here willing to test u1 with proxies? | 11:23 |
mandel | gatox, popey ? pleaaaaaase :P | 11:23 |
gatox | mandel, ok..... but i didn't save the ralsina ip.... so you will need to wait until i ask him that again :P | 11:27 |
mandel | gatox, which irc client do you use? | 11:27 |
mandel | gatox, 'cause if it is xchat you can find the longs under ~/.xchat/xchatlogs/ and then the channel :) | 11:28 |
gatox | mandel, xchat..... but i can't find it in the logs | 11:28 |
mandel | gatox, really? bummer :( | 11:28 |
gatox | mandel, let me check in the file..... but at least is not in the history that the client load | 11:28 |
mandel | gatox, oh, but in the files you have A LOT more :) | 11:29 |
gatox | mandel, it's .xchat2/scrollback for the logs | 11:29 |
mandel | mandel, no, is ~/.xchat2/xchatlogs/ | 11:29 |
popey | mandel: oooh, yeah | 11:30 |
gatox | mandel, i always had the logs in ~/.xchat2/scrollback/... | 11:31 |
gatox | mandel, and npo.... is not there either..... maybe i have the logs configured to don't save much | 11:31 |
mandel | popey, sweet! do you have a proxy server of any type? maybe a squid or something running on P | 11:31 |
popey | i do | 11:31 |
mandel | gatox, oh :( I have a stupid about of logs | 11:32 |
popey | mandel: the worry I have though is that the machine can alsso get to the web without going via the proxy | 11:32 |
popey | so i need to do some iptables fu to block port 80 to anything but the proxy IMO | 11:33 |
mandel | popey, great, lets get our hands dirty then, you need to get the following branches: lp:~alecu/ubuntuone-client/proxy-tunnel-cookies lp:~mandel/ubuntu-sso-client/ lp:~mandel/ubuntu-sso-client/use-qt4-webclient-backend and lp:ubuntuone-control-panel | 11:33 |
mandel | popey, don't worry, we can attempt first without the iptables foo, in theory it does not work, but in my machines does.. so I need a 'cleaner' env to test :) | 11:34 |
mandel | popey, does it use auth? the proxy I mean | 11:34 |
popey | no | 11:34 |
popey | just squid on an ubuntu server | 11:35 |
mandel | popey, ok, lets first start with nonauth, we can quickly later set the auth in a diff port with not too much work :) | 11:36 |
* popey needs to fix his proxy first | 11:36 | |
mandel | popey, let me know when you are ready, should be easy to get you with a non-auth proxy | 11:36 |
mandel | gatox, I'm noticed that running control panel from trunk the loading text is HUGE does this happen to you? | 11:44 |
gatox | mandel, haven't tested that yet..... maybe the branch that was modifying the font sizes landed.... i'm going to check that now | 11:46 |
mandel | gatox, please so, in my is crazy.. I know I'm old but I can still read, you guys did not need to do that ;) | 11:46 |
gatox | mandel, oh yes..... is big.... that for the font thing that dobey was working i assume | 11:48 |
mandel | gatox, he made it bigger? | 11:48 |
gatox | mandel, let me check the qss | 11:49 |
popey | bzr: ERROR: Invalid url supplied to transport: "lp:~mandel/ubuntu-sso-client/": ~mandel/ubuntu-sso-client is too short to be a branch name. Try '~<owner>/+junk/<branch>', '~<owner>/<product>/<branch> or '~<owner>/<distribution>/<series>/<sourcepackage>/<branch>'. | 11:49 |
mandel | popey, ups, sorry i miss type the full branch name, it is lp:~mandel/ubuntu-sso-client/use-qt4-webclient-backend | 11:50 |
mandel | popey, I guess I had a paste fail, sorry | 11:50 |
gatox | mandel, i'll have to check with nessita if that was intended or what | 11:51 |
mandel | gatox, I don't think it is, right? it does not even fit in the control panel window.. | 11:53 |
gatox | mandel, maybe you are seeing something different than i.... i see the font big.... but fit in that area | 11:53 |
gatox | mandel, i'm seeing that the part that specifies the font size of the loading label is missing in the u1-cp qss | 11:57 |
mandel | gatox, hm.. weird | 11:58 |
nessita | hello everyone | 12:11 |
gatox | nessita, hi | 12:12 |
nessita | hola gatox | 12:12 |
gatox | nessita, mandel was telling me that the text from the loading overlay is too big, and i saw that the font size for that in cp was removed fromm the qss.... was that intended? | 12:12 |
mandel | nessita, morning! | 12:13 |
nessita | hola mandel! | 12:13 |
nessita | gatox: yes, we need to use the font from the system | 12:13 |
nessita | gatox: you can propose a branch changing xx-large for x-large if you want | 12:13 |
nessita | I'm oj with it | 12:13 |
nessita | ok* | 12:13 |
gatox | nessita, ok | 12:14 |
nessita | gatox: since now we should never ever use hard coded fonts, just relative fonts (small, large, etc) | 12:14 |
nessita | gatox: what bug are you working on atm? | 12:18 |
gatox | nessita, i was looking at this ones: https://bugs.launchpad.net/ubuntuone-control-panel/+bug/887284 - https://bugs.launchpad.net/ubuntuone-client/+bug/907479 | 12:19 |
ubot5 | Ubuntu bug 887284 in Ubuntu One Control Panel "UI: improve handling of "'s" when appending to names ending in 's' in the Cloud Folders panel" [Medium,Triaged] | 12:19 |
ubot5 | Ubuntu bug 907479 in Ubuntu One Client "u1sdtool -q should not start syncdaemon when syncdaemon is not running" [Medium,Triaged] | 12:19 |
gatox | nessita, but wanted to do the font thing first | 12:19 |
gatox | nessita, do you have anything better in mind? | 12:19 |
nessita | gatox: if it's ok with you, I will ask you to look a couple of bugs regarding translations | 12:20 |
nessita | gatox: https://bugs.launchpad.net/ubuntu/+source/ubuntu-sso-client/+bug/951376 | 12:20 |
ubot5 | Ubuntu bug 951376 in Ubuntu Translations "Several strings untranslatable in ubuntone-client registration" [High,Triaged] | 12:20 |
nessita | gatox: https://bugs.launchpad.net/ubuntu/+source/ubuntu-sso-client/+bug/951371 | 12:20 |
ubot5 | Ubuntu bug 951371 in Ubuntu Translations "Translated strings shown wrong, character set not UTF-8" [High,Triaged] | 12:20 |
nessita | gatox: I will mark them as High and assign to you if that's ok | 12:20 |
gatox | nessita, yep....... shoot | 12:20 |
gatox | i'll take those | 12:21 |
nessita | gatox: thanks. Assigned 2 in ussoc, but I might assing a couple more for the controlpanel (regarding translations as well) | 12:22 |
gatox | nessita, assign assign..... i don't mind | 12:22 |
nessita | :-D | 12:22 |
mandel | nessita, I would literally love you if you could look at line 89 of http://paste.ubuntu.com/886222/ | 12:23 |
ralsina | gatox: 184.82.108.14 | 12:23 |
nessita | mandel: looking! | 12:23 |
ralsina | and good morning1 | 12:23 |
gatox | ralsina, i will write that down | 12:23 |
ralsina | gatox: I suggest un tatuaje tumbero in your belly | 12:23 |
gatox | ralsina, jeejeje | 12:23 |
mandel | nessita, is the one with the executiong of login only | 12:24 |
mandel | ralsina, morning! | 12:24 |
mandel | gatox, on the back of your head, like a hit man! | 12:24 |
ralsina | gatox: hope you remember the high security user/password! | 12:24 |
gatox | ralsina, yes | 12:24 |
gatox | :P | 12:24 |
nessita | mandel: line 89 seen. DO you also have the -gui logs? | 12:24 |
mandel | nessita, let me ask, where can those be found? | 12:25 |
mandel | popey, ^ | 12:25 |
gatox | mandel, in the back of my head i have: 優柔不断な人 ñandú | 12:25 |
gatox | jejeje | 12:25 |
nessita | mandel: same dir as sso logs, file is called sso-login-gui.log | 12:25 |
mandel | nessita, ok | 12:25 |
gatox | mandel, hey! i actually have a chinese symbol in my back :P.... my body supports unicode :D | 12:26 |
popey | mandel: ? | 12:26 |
mandel | popey, can we get the logs from the proxy test you did? | 12:26 |
popey | Failure: ubuntuone.platform.credentials.CredentialsError: dbus.Dictionary({dbus.String(u'errtype'): dbus.String(u'CredentialsError')}, signature=dbus.Signature('ss')) | 12:26 |
popey | thats it | 12:26 |
alecu | hello, all! | 12:26 |
gatox | alecu, hi | 12:26 |
nessita | gatox: do you unicodedecodeerror sometimes? | 12:27 |
popey | mandel: not sure what other logs you want/need? | 12:27 |
mandel | popey, the logs are here: ~/.cache/sso/sso-login-gui.log | 12:27 |
nessita | hola alecu (we're not derailing, no....) | 12:27 |
popey | oh | 12:27 |
mandel | popey, sorry, I'm assuming too much from you :) | 12:27 |
mandel | alecu, morning! | 12:27 |
popey | yeah, assume I'm an idiot and you wont go far wrong ☺ | 12:27 |
mandel | alecu, apparently you have to kick me in the balls, yet I don't know why.. | 12:28 |
gatox | nessita, that would be painful :P | 12:28 |
popey | sso-client-gui.log sso-client.log | 12:28 |
popey | not sso-login-gui.log ☺ | 12:28 |
alecu | mandel, I've spent the whole night practicing kicking myself between the eyes, too, if that helps. | 12:28 |
alecu | mandel, but it seems I need shorter legs... | 12:28 |
mandel | popey, can we get those please, pastebin in canonical just so that is not too public | 12:29 |
ralsina | alecu: or more bendy ankles | 12:29 |
popey | http://paste.ubuntu.com/886237/ | 12:29 |
popey | oops | 12:29 |
ralsina | or more ankles | 12:29 |
alecu | ralsina, mandel: do you guys have 5 mins for a quick mumble? | 12:30 |
mandel | alecu, certainly! | 12:30 |
mandel | alecu, although, give me 5 mins that other person at the office is screaming.. | 12:30 |
mandel | popey, that is the sso-client.log, right? and the sso-client-gui.log? | 12:31 |
popey | -rw-rw-r-- 1 alan alan 0 Mar 6 13:15 /home/alan/.cache/sso/sso-client-gui.log | 12:31 |
popey | empty | 12:31 |
mandel | nessita, ^ | 12:32 |
popey | mandel: is there a -d debug option on ubuntuone-control-panel-qt ? | 12:33 |
ralsina | alecu: sure | 12:33 |
ralsina | alecu: let me start it | 12:33 |
ralsina | popey: no, but you can set U1_DEBUG=True | 12:33 |
mandel | popey, that is the U1_DEBUG=True we did | 12:33 |
nessita | popey: can you please pastebin the output of: apt-cache policy ubuntu-sso-client* | 12:35 |
nessita | ralsina: I'm reviewing your 'tweaks' branch, and I have a question | 12:36 |
popey | mandel: nessita i didnt set U1_DEBUG for the control panel line | 12:37 |
popey | $ PYTHONPATH=~/u1/use-qt4-webclient-backend/:~/u1/proxy-tunnel-cookies/:. ./bin/ubuntuone-control-panel-qt | 12:37 |
nessita | popey: you're running a custom branch? :-/ | 12:37 |
nessita | mandel: are you trying to debug a custom branch with popey? | 12:38 |
mandel | nessita, yes | 12:38 |
nessita | mandel: uh | 12:38 |
nessita | mandel: and does it work for you? | 12:38 |
nessita | :-) | 12:38 |
alecu | popey, "iptables fu to block port 80" -> "port 443" (since that's the port u1 uses) | 12:38 |
nessita | mandel: apparently the UI is not starting, from the 0-sized logs file | 12:38 |
mandel | nessita, yes, it works for me, and the same error 1 was given to alecu, right alecu? | 12:39 |
nessita | ralsina: having this: | 12:39 |
nessita | /* Compensate for border so text doesn't move */ | 12:39 |
nessita | padding-left: 8px; | 12:39 |
nessita | padding-right: 23px; | 12:39 |
nessita | isn't that absolutely dependent of the font size? (ie how many pixels are needed so text do not move) | 12:39 |
mandel | popey, try with: U1_DEBUG=True PYTHONPATH=~/u1/use-qt4-webclient-backend/:~/u1/proxy-tunnel-cookies/:. ./bin/ubuntuone-control-panel-qt | 12:39 |
nessita | mandel: 1 means the app crashed before it finished starting | 12:39 |
popey | alecu: good point :) | 12:39 |
alecu | mandel, right. The "1" is the exit code of the ui process, as far as I debugged yesterday. | 12:40 |
nessita | mandel: are you making popey also run the sso service from ~/u1/use-qt4-webclient-backend ? | 12:40 |
mandel | nessita, yes | 12:40 |
nessita | to run the UI in isolation, you all could: | 12:40 |
nessita | cd ~/u1/use-qt4-webclient-backend/ | 12:41 |
nessita | U1_DEBUG=True PYTHONPATH=. ./bin/ubuntu-sso-login-qt --app_name=foo | 12:41 |
popey | http://paste.ubuntu.com/886250/ | 12:41 |
alecu | gatox, guess who is the winner of a ticket to review two spectacular branches? | 12:41 |
popey | /home/alan/.cache/sso/sso-client-gui.log still empty | 12:41 |
nessita | and that will show how the app is crashing at startup | 12:41 |
* gatox is offline | 12:41 | |
gatox | alecu, jejeje | 12:41 |
gatox | alecu, shoot | 12:41 |
nessita | popey: can you try what I mentioned earlier? please always run the sso service first in another terminal | 12:42 |
nessita | to run the sso service, you should: | 12:42 |
nessita | cd ~/u1/use-qt4-webclient-backend/ | 12:42 |
nessita | U1_DEBUG=True PYTHONPATH=. ./bin/ubuntu-sso-login | 12:42 |
popey | yup, I'm running that in another window | 12:43 |
popey | --app_name=foo <- what goes there? | 12:43 |
nessita | popey: anything, just trying to see the crash | 12:45 |
nessita | ralsina: ping? | 12:45 |
popey | http://paste.ubuntu.com/886256/ nessita | 12:47 |
nessita | popey: je, the error is simple, just run inside that folder: | 12:48 |
nessita | ./setup.py clean build | 12:48 |
nessita | mandel: ^ | 12:48 |
mandel | nessita, why? | 12:48 |
nessita | mandel: he just branched and ui files are not built | 12:48 |
nessita | no? | 12:48 |
mandel | nessita, he did a python setup.py build, right popey ? | 12:49 |
nessita | he did? | 12:49 |
gatox | alecu, ?? the branches? | 12:49 |
alecu | gatox, sorry, I was on mumble. | 12:49 |
gatox | alecu, ah ok | 12:49 |
popey | i did | 12:52 |
popey | http://paste.ubuntu.com/886262/ | 12:53 |
nessita | popey: you're likely missing a package | 12:53 |
nessita | popey: let me give you the list of packages you should have | 12:54 |
nessita | popey: can you please run: | 12:54 |
nessita | apt-get build-dep ubuntu-sso-client ubuntuone-control-panel | 12:54 |
popey | 0 upgraded, 0 newly installed, 0 to remove and 44 not upgraded. | 12:54 |
nessita | hum, we have have a missin dep, let me list the qt4 deps :-) | 12:54 |
mandel | alecu, http://paste.ubuntu.com/886256/ | 12:55 |
nessita | mandel: what's the needed package to build the rc files? | 12:56 |
gatox | ralsina, i just bought "the dig" for the kindle..... it looks pretty cool | 12:56 |
mandel | nessita, you need the uic from PyQt4 | 12:57 |
nessita | mandel: yes, and what's the deb package name? | 12:57 |
mandel | nessita, let me look | 12:57 |
nessita | pyqt4-dev-tools? | 12:57 |
gatox | nessita, you can't just do from PyQt4 import uic ? | 12:57 |
nessita | gatox: not me, another user | 12:58 |
gatox | nessita, if noy... install the dev-tools | 12:58 |
gatox | ah ok | 12:58 |
popey | pyqt4-dev-tools is already the newest version. | 12:58 |
mandel | nessita, popey yes, it should be pyqt4-dev-tools | 12:59 |
alecu | gatox, https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-auth/+merge/97763 | 12:59 |
alecu | gatox, and after that, https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-cookies/+merge/97791 | 12:59 |
nessita | popey: can yout try running the same command as before? (the one with app_name=foo) | 12:59 |
psypher246 | hey all! | 13:00 |
popey | nessita: nothing has changed since I last ran it though | 13:00 |
psypher246 | any progress yet on the ubuntuone login issues? | 13:00 |
alecu | nessita, perhaps not all ENV vars are being passed from the first process to the second? | 13:01 |
popey | ooh, more debug stuff | 13:01 |
alecu | nessita, my bet is on the DISPLAY env var not being passed around. | 13:01 |
mandel | popey, can you ls /usr/lib/python2.7/dist-packages/PyQt4/bin | 13:01 |
mandel | nessita, ^ should be where the rc compiler is | 13:01 |
alecu | nessita, what API is spawning the process? | 13:01 |
gatox | alecu, ack | 13:02 |
alecu | oh, a missing rc file. | 13:02 |
nessita | alecu: did you see the import error? | 13:02 |
alecu | nessita, I've seen it now. | 13:02 |
popey | http://paste.ubuntu.com/886273/ | 13:02 |
nessita | popey: is working!!! | 13:02 |
nessita | mandel: ^ | 13:02 |
nessita | popey: you got an UI, didn't you? | 13:03 |
popey | yes | 13:03 |
mandel | nessita, popey what changed? | 13:03 |
popey | not quite sure | 13:03 |
popey | you sure thats using the proxy? | 13:03 |
nessita | popey: what we did so far was to debug why the UI was not being opened befoe | 13:04 |
nessita | popey: so now, you and mandel can go back to the way you were debugging before | 13:04 |
mandel | nessita, alecu look at line 111 of setup.py in ubuntu-sso-client, /usr/lib/python2.7/dist-packages/PyQt4/bin is wrong, in my system is /usr/lib/python2.7/dist-packages/PyQt4/uic/ | 13:04 |
briancurtin | ralsina: so, how do we begin? | 13:04 |
mandel | nessita, popey so, what changed for the ui to work again? | 13:04 |
mandel | popey, can you try doing what we did last time with proxies? | 13:05 |
popey | hang on | 13:05 |
popey | the UI for the panel _did_ open before, just not the sign up screen | 13:05 |
nessita | mandel: the PATH thing does not matter on Linux, the script is in the PATH already | 13:06 |
nessita | nessita@dali:~$ which pyrcc4 | 13:06 |
nessita | /usr/bin/pyrcc4 | 13:06 |
nessita | mandel: the PATH may be relevant to windows, though | 13:06 |
nessita | ralsina: heeeeellllloooo :-) | 13:06 |
mandel | nessita, probably.. we should check that, maybe is outdated if you install a new version of PyQt on windows.. | 13:07 |
nessita | mandel: right | 13:07 |
mandel | popey, yes, you got stuck in the login one.. which is when everything went bananas | 13:07 |
popey | ah | 13:09 |
mandel | nessita, what did you do to get it working, because alecu was having the same CredentialsError 1 issue | 13:13 |
nessita | mandel: I just saw the trace and aske popey to run ./setup.py clean build | 13:13 |
mandel | nessita, funny, 'cause I know he ran ./setup.py build | 13:14 |
nessita | mandel: didn't you install some dep in between, perhaps? | 13:14 |
mandel | nessita, maybe | 13:15 |
mandel | alecu, can you test that you can launch the ui for login? | 13:15 |
alecu | sure | 13:15 |
mandel | alecu, if it does work, can you test lp:~mandel/ubuntu-sso-client/use-qt4-webclient-backend | 13:16 |
alecu | mandel, I'm testing it on that branch. | 13:17 |
dobey | popey: yes the dup songs from u1 purchases is a known bug. are you using nightlies, or plain ubuntu packages for u1? | 13:17 |
mandel | alecu, make sure that you are on revno 917 | 13:17 |
popey | dobey: plain ubuntu packages | 13:18 |
dobey | popey: also, thanks for killing my computer | 13:18 |
popey | haha | 13:18 |
popey | log less | 13:18 |
popey | :p | 13:18 |
dobey | popey: fix should show up in ubuntu on tuesday or wednesday then. | 13:19 |
popey | yay! | 13:19 |
alecu | mandel, I'm on revno 916. But it worked! | 13:21 |
mandel | alecu, wait, with proxy and all? | 13:21 |
alecu | with proxy and all | 13:21 |
dobey | mandel: what is the spanish translation for the "Getting information..." string in the loading overlay? (if you're using spanish locale) | 13:22 |
mandel | dobey, sorry I don't use spanish | 13:23 |
dobey | spanglish? :) | 13:23 |
mandel | alecu, ok, I'll run the tests and will propose.. | 13:23 |
mandel | dobey, mandelish :) | 13:23 |
dobey | haha | 13:23 |
alecu | dobey, I don't use spanish either, but guess it can't get as uglier than "Getting information..." | 13:24 |
mandel | alecu, dobey it can: Tio, recibiendo la info de las pelotas... | 13:24 |
mandel | :P | 13:24 |
alecu | mandel, that's funnier, not "ugly" | 13:24 |
dobey | heh | 13:25 |
dobey | i'm guessing that text isn't wrapping | 13:25 |
dobey | and is extraneously long anyway | 13:25 |
dobey | We should show a pretty animation and a spinning CD that says "Loading…" next to it, like on XBox games | 13:26 |
alecu | mandel, now that it works: some issues. | 13:28 |
mandel | alecu, shoot! | 13:28 |
alecu | mandel, if I click "save and connect" without entering proxy credentials, crap would be saved on the keyring, and it will get blocked. | 13:29 |
dobey | all the font size discussion reminds me of this: http://www.youtube.com/watch?v=tY6dIxQTaf8 | 13:29 |
alecu | mandel, the same if I just enter the username, without any password. | 13:29 |
mandel | alecu, let me test that.. but there are tests for that.. | 13:30 |
nessita | dobey: "Obteniendo información, por favor espere..." should be | 13:35 |
dobey | yeah, it's too long/formal i think | 13:36 |
nessita | dobey: ah, unrelated, you saw https://bugs.launchpad.net/ubuntu/+source/ubuntuone-client-gnome/+bug/953119 ? | 13:36 |
ubot5 | Ubuntu bug 953119 in ubuntuone-client-gnome (Ubuntu) "valgrind invalid read error" [Undecided,New] | 13:36 |
dobey | but eh, no more string changes this cycle | 13:36 |
dobey | nessita: yes | 13:36 |
nessita | ack | 13:36 |
ralsina | alecu, sorry to drop out but had to reboot, windows had to reboot, then configure updates, then reboot :-/ | 13:40 |
alecu | ralsina, no prob. | 13:41 |
alecu | ralsina, got a minute for mumble now? | 13:41 |
alecu | mandel, too ^ | 13:41 |
ralsina | alecu: sure | 13:41 |
mandel | alecu, sure, lets do it! | 13:41 |
mandel | alecu, I'm already there | 13:42 |
ralsina | gatox: spawn-evil has tests fixed | 13:46 |
gatox | alecu, i have a failure running the tests.... let me know if that is related to your u1-client branch | 13:46 |
gatox | ralsina, ack | 13:46 |
alecu | ralsina, mandel: my mumble seems broken | 13:48 |
ralsina | alecu: you dropped | 13:48 |
mandel | ralsina, mine is the same | 13:48 |
ralsina | mandel: you still appear there | 13:49 |
mandel | ralsina, alecu I cannot hear a thing every few mins.. | 13:49 |
alecu | ralsina, mandel: anyway, I think we were closing the meeting. | 13:49 |
ralsina | mandel: that's the brain damage from rugby | 13:49 |
mandel | ralsina, I think is mumble :P | 13:49 |
mandel | ralsina, alecu so in summary, I fix the use-qt branch and then I disable the ssl stuff, right? | 13:49 |
mandel | ralsina, alecu shall we really disable it, or pass a param to allow pinned certs or something like that? | 13:50 |
ralsina | alecu: yes, actions: 1) if it works at the level of chromium (ssl-to-proxy) I say it's ok to leave it. If it works any worse, drop it. 2) Don't pin certs from u1 | 13:50 |
alecu | ralsina, ssl-to-proxy is not supported by qt. (nor by firefox for that matter) | 13:51 |
ralsina | alecu: then let's just pretend it doesn't exist ;-) | 13:51 |
mandel | alecu, ralsina so I remove it and we are done with it? | 13:52 |
ralsina | mandel: yep | 13:53 |
mandel | ack | 13:53 |
alecu | gatox, failure running the test? | 13:53 |
alecu | gatox, can you paste it? | 13:53 |
mandel | ralsina, alecu I'm off yo have lunch, I should be done with the qt issue in a few mins | 13:53 |
gatox | alecu, it's in the mp | 13:53 |
dobey | hmm | 13:53 |
mandel | alecu, I'll file a bug about the issue when the save creds dialog gets nonesense | 13:53 |
ralsina | mandel: cool | 13:53 |
alecu | mandel, can I review it already? | 13:53 |
alecu | mandel, great. There are a few small bugs we should consider next week. | 13:54 |
mandel | alecu, no yet, I need to fix one test :) | 13:54 |
mandel | alecu, after lunch I'll fix it and will ping you | 13:54 |
alecu | mandel, for instance, socks support is not enabled on ussoc, but is on the sd tunnel. | 13:54 |
alecu | gatox, thanks. | 13:54 |
mandel | alecu, lets 'virtually sit down' and make a list of those small ones, I don't think they are terribly hard ones to fix (socks means changing the proxyfactory) | 13:55 |
alecu | mandel, right! | 13:55 |
alecu | mandel, we can do it after the qt branch lands. | 13:55 |
mandel | alecu, exactly, that is the important one | 13:56 |
mandel | ok, I'm off to get energy from dead animals | 13:56 |
* mandel lunch | 13:56 | |
* rye starts understanding why go does not use exceptions | 13:57 | |
* rye fixes the validation script to deal with different types of errors, it's been a while... | 13:57 | |
gatox | ralsina, +1 | 13:58 |
ralsina | nessita: I am putting the tweaks branch on hold until monday because I need to kick windows into shape today | 13:58 |
* gatox needs to relogin..... brb | 13:58 | |
nessita | ralsina: ack | 13:58 |
ralsina | nessita: thanks for catching that, too! | 13:58 |
ralsina | gatox: thanks | 13:58 |
nessita | ralsina: bad news is that means we're not releasing to ubuntu that... :-/ | 13:58 |
ralsina | nessita: if that's the case, I will try to jump into linux 15 minutes to fix it today | 13:59 |
alecu | gatox, Looks like the ussoc you are using is not trunk. | 13:59 |
alecu | Try updating nightlies or configuring this branch using "--with-sso=/path/to/sso/trunk" | 13:59 |
gatox | alecu, ok | 13:59 |
nessita | ralsina: let me know, otherwise we can explore options | 13:59 |
ralsina | nessita: sure | 13:59 |
ralsina | nessita, gatox: super trivial review https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/no-hack/+merge/97882 | 14:08 |
alecu | ralsina, mandel: On a sunnier note, I've just IRL tested the very latest nightlies, and for a nonauth proxy, SD and SSO are working great. | 14:09 |
ralsina | alecu: \o/ | 14:10 |
briancurtin | ralsina: apparently "u1sdtool -s" doesn't know about the SD it starts. i stepped into the twisted select code and select.select never returns that anything is ready - always ([], [], []) | 14:10 |
ralsina | briancurtin: interesting | 14:10 |
ralsina | briancurtin: blocks on IPC would explain the u1cp locking up too | 14:10 |
ralsina | briancurtin: there are test failures in SSO for IPC, want to take a look at them? Maybe that's the underlyig cause | 14:11 |
briancurtin | ralsina: will do | 14:11 |
mandel | alecu, ok, then I wont jump trough the window ;) | 14:12 |
alecu | mandel, ralsina: http://www.youtube.com/watch?v=-TUt5n3sU14 | 14:13 |
ralsina | alecu, mandel: if you have a little time later today, I may need you to put onyour windows/twisted/IPC hat (remember where you put it?) | 14:13 |
mandel | ralsina, ok, no problem :) | 14:14 |
alecu | ralsina, I think I sat on it. | 14:14 |
alecu | luckily it was not my pointy wizard hat. | 14:14 |
ralsina | alecu: at least there is no h in sat | 14:14 |
alecu | loh | 14:15 |
urbanape | Hi, all. Sorry I'm late. | 14:18 |
ralsina | hi laser-vision dude! | 14:20 |
ralsina | nessita: if trunk is lp:blablah then what's "stable 3"? lp:blabla/stable-3 ? | 14:20 |
nessita | ralsina: yes | 14:21 |
ralsina | urbanape: how's the eyes? | 14:21 |
dobey | ralsina: no, stable-3-0 | 14:21 |
ralsina | urbanape: if you can read this, of course | 14:21 |
urbanape | So, I don't know what they use in metric countries, but I'm guessing I'm 7/5 | 14:21 |
nessita | dobey: right | 14:21 |
nessita | ralsina: yes, sorry, what dobey said | 14:21 |
urbanape | I see at 7m what other see at 5m | 14:21 |
urbanape | so, better than perfect | 14:21 |
ralsina | dobey: ok, and the current one we are releasing is stable-3-0 ? | 14:21 |
urbanape | we use feet here, so I'm 20/15 | 14:21 |
dobey | ralsina: yes | 14:21 |
ralsina | urbanape: yay, so if I have 5 apples, you see 7? Awesome! | 14:21 |
urbanape | little bit of haziness around bright lights, but that should go away in a week or so. | 14:22 |
ralsina | urbanape: that's the pupil being too relaxed | 14:22 |
ralsina | urbanape: I have that twice a year when they check for glaucoma (no luck yet!) | 14:22 |
ralsina | dobey, nessita:thanks so much | 14:23 |
rye | joshuahoover: tomboy sync validation script can now fix xml errors too | 14:24 |
dobey | oh i need to fix the conflict | 14:27 |
ralsina | urbanape: while you were gone I hijacked briancurtin for windows work | 14:28 |
ralsina | urbanape: and I can't release him today, so you will have to hack alone on mac today | 14:28 |
urbanape | will do, though it'd be really keen to get a pair for a few minutes to get me unblocked on a few things. I've been nibbling around the edges on some things, but need a little help getting stuff all tied together. Probably doesn't need to be brian... | 14:30 |
alecu | gatox, did the newer sso help? | 14:32 |
gatox | alecu, i have 2 failures with credentials now | 14:33 |
gatox | alecu, do you want the paste of those failures or i put them in the mp? | 14:34 |
alecu | gatox, I think a paste is better, because otherwise the MP gets huge | 14:35 |
gatox | alecu, ok | 14:36 |
nessita | ralsina: you left a print here: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/no-hack/+merge/97882 | 14:37 |
gatox | alecu, http://paste.ubuntu.com/886399/ | 14:37 |
ralsina | nessita: argh,sorry | 14:37 |
* ralsina is rushig things too much.Time to cool down and meditate. | 14:38 | |
alecu | gatox, spanish? my code did not touch any of those tests, so would you mind running the tests again with an english locale? | 14:39 |
gatox | alecu, ahhhhhhhh sorry.... i'm with an spanish locale now because i'm testing another bug | 14:39 |
ralsina | alecu, gatox: we may have random breakage of tests in spanish. I tried to fix all of them, but surely missed some, specially when translations change | 14:40 |
ralsina | so please run tests with LANG=en_EN.UTF-8 but if you find breakage in other languages, file and assign to me | 14:40 |
ralsina | nessita: removed the print and pushed. Again, sorry, I am a bit too accelerated today, will slow down a bit and doublecheck things now | 14:41 |
nessita | ralsina: thanks! :-) | 14:41 |
dobey | en_US | 14:41 |
dobey | what country is EN? :) | 14:41 |
ralsina | ok, en_UK ;-) | 14:42 |
ralsina | proper english! ;-) | 14:42 |
dobey | what is UK? | 14:42 |
dobey | you mean GB? | 14:42 |
ralsina | en_EN actually defaults to en_US, just like en_XY does ;-) | 14:42 |
ralsina | hahaha | 14:42 |
gatox | alecu, +1, those were the only tests failing... and not related to this.... and the code seems fine | 14:42 |
dobey | en_UK might be interesting though | 14:42 |
dobey | ralsina: everything that fails defaults to en_US because en_US is POSIX C | 14:43 |
alecu | \o/ | 14:43 |
ralsina | "blimey mastah! we 'av'a'issu 'ere!" | 14:43 |
ralsina | dobey: therefore en_EN works, neener neener ;-) | 14:43 |
dobey | ralsina: well, it works sort of. it falls back to C, not en_US.UTF-8 :) | 14:44 |
dobey | also, the fact that we have en_US.po files for things, is frightening | 14:44 |
alecu | gatox, don't go too far away! here's the other one for you :-) https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-cookies/+merge/97791 | 14:44 |
gatox | alecu, yesssssss i was starting with that | 14:45 |
alecu | thisfred, are you around? | 14:48 |
alecu | thisfred, can I ask you to review that branch too? ^^^ | 14:48 |
alecu | gatox, mind that it has the previous branch as a prerequisite. | 14:49 |
dobey | sigh | 14:49 |
ralsina | briancurtin: while you check those test failures, I am cleaning up the windows build starting from your branch (removing the windows-installer stuff and other bits) | 14:50 |
dobey | fix conflicts; commit; update trunk; merge new one in; oh look, more conflicts :( | 14:50 |
ralsina | dobey: what branch? | 14:50 |
thisfred | alecu: I am, though elbow deep in segfaults | 14:50 |
briancurtin | ralsina: sounds good. the only things im seeing on SSO are "Reactor was unclean" test errors fyi | 14:51 |
gatox | alecu, ack | 14:51 |
thisfred | alecu: will review in a sec | 14:51 |
ralsina | briancurtin: yes, those are the ones I saw | 14:51 |
dobey | ralsina: sso system-font | 14:51 |
ralsina | dobey: oh | 14:52 |
dobey | resolving conflicts in qtdesigner ui files is not fun. | 14:52 |
dobey | but i think i got them fixed for now | 14:52 |
dobey | just NOGBODY BETTER BREAK IT AGAIN | 14:52 |
ralsina | NOG WILL NOT BREAK BODY | 14:53 |
ralsina | dobey: ack! | 14:53 |
* mandel back | 14:53 | |
mandel | nessita, u'test-me-more-♥' => lol | 14:54 |
nessita | mandel: we needed complex unicode everywhere :-D | 14:55 |
mandel | me | 15:00 |
gatox | me | 15:00 |
alecu | me | 15:01 |
ralsina | me | 15:01 |
dobey | meh | 15:01 |
briancurtin | me | 15:01 |
dobey | nessita: slacker | 15:01 |
nessita | me | 15:02 |
ralsina | mandel: go | 15:03 |
mandel | DONE: Fixed the qt issue where the wrong webclieng was used. Fixed bug 933081 and bug 956185. | 15:03 |
mandel | TODO: Find a good way to test is qt running. Remove ssl support. 1-1 ralsina, after this. | 15:03 |
mandel | BLOCKED: no | 15:03 |
mandel | gatox, please | 15:03 |
ubot5 | Launchpad bug 933081 in ubuntu-sso-client (Ubuntu Precise) "GTK UI: URL for terms and conditions never finishes loading" [Undecided,In progress] https://launchpad.net/bugs/933081 | 15:03 |
ubot5 | Launchpad bug 956185 in Ubuntu Single Sign On Client "When viewing the term conditions twice an error occurs" [Medium,Fix committed] https://launchpad.net/bugs/956185 | 15:03 |
gatox | DONE: | 15:03 |
gatox | Lots of reviews, all my branches landed, working on some translations issues. | 15:03 |
gatox | TODO: | 15:03 |
gatox | Finish with the translation issues and keep killing bugs from my queue unless someone needs helps with anything else. | 15:03 |
gatox | BLOCKED: | 15:03 |
gatox | No | 15:03 |
gatox | alecu, go | 15:03 |
alecu | 𝕯𝖔𝖓𝖊: branch to use creds from keyring: landed, branch to protect tunnel with a cookie: being reviewed. Found SSL issue. IRL tested SD and SSO thru a nonauthenticated proxy. Always bug #929207. | 15:03 |
alecu | 𝕿𝖔𝖉𝖔: working on branch to pause syncdaemon on credential errors. | 15:03 |
alecu | 𝕭𝖑𝖔𝖈𝖐𝖊𝖉: no | 15:03 |
alecu | 𝕹𝖊𝖝𝖙: ralsina | 15:03 |
ubot5 | Launchpad bug 929207 in ubuntuone-client (Ubuntu Precise) "[FFE] Proxy "tunnel" for syncdaemon" [Undecided,Confirmed] https://launchpad.net/bugs/929207 | 15:03 |
ralsina | DONE: team call, nessita 1:1 dobey 1:1, chipaca 1:1, windows testing, tweaks, reviews, bunch'o'things TODO: kick windows butt, take windows name, chew gum. BLOCKED: windows NEXT dobey | 15:03 |
dobey | λ DONE: team meeting, ping about rb-scope, reviews, bug #951425, fix conflicts in sso system-font branch | 15:03 |
ubot5 | Launchpad bug 951425 in rhythmbox-ubuntuone (Ubuntu Precise) "Ubuntu One plugin has problems when being enable/disabled" [High,Triaged] https://launchpad.net/bugs/951425 | 15:03 |
dobey | λ TODO: bug #934206, bug #932103, bug #953119 | 15:03 |
dobey | λ BLCK: none. | 15:03 |
ubot5 | Launchpad bug 934206 in rhythmbox-ubuntuone (Ubuntu Precise) "track duplication from RB-U1" [High,Confirmed] https://launchpad.net/bugs/934206 | 15:03 |
ubot5 | Launchpad bug 932103 in rhythmbox-ubuntuone (Ubuntu Precise) "Cannot install MP3 playback support; silently fails" [High,Confirmed] https://launchpad.net/bugs/932103 | 15:03 |
ubot5 | Launchpad bug 953119 in ubuntuone-client-gnome (Ubuntu) "valgrind invalid read error" [Undecided,New] https://launchpad.net/bugs/953119 | 15:03 |
dobey | nessita | 15:03 |
nessita | DONE: weekly call, some other random mumbles, lot of reviews and re-reviews, more on more-pages on controlpanel | 15:04 |
nessita | TODO: finish the above | 15:04 |
nessita | BLOCKED: nopes | 15:04 |
nessita | NEXT: ? | 15:04 |
briancurtin | DONE: tried to get windows install working so it can be released | 15:04 |
briancurtin | TODO: make it work... (working on SSO right now) | 15:04 |
briancurtin | BLOCKED: none | 15:04 |
briancurtin | NEXT: nessita | 15:04 |
nessita | sorry briancurtin, seems like I skipped my turn | 15:04 |
gatox | alecu, writes fanzy standups | 15:04 |
alecu | also, urbanape, thisfred: standup! | 15:04 |
alecu | gatox, :-) | 15:05 |
thisfred | oops that shows as one hour ago in my calendar too | 15:05 |
urbanape | me | 15:06 |
thisfred | me | 15:06 |
mandel | uh.. dst, fun! | 15:06 |
thisfred | alecu: didn't we cancel standups, also? | 15:06 |
mandel | thisfred, you didn't get the memo? | 15:07 |
thisfred | DONE: investigate segfaults in u1db | reviews TODO: more of the same BLOCKED: not sure what is causing the segfaults | 15:08 |
thisfred | urbanape: c'est a vous! | 15:09 |
urbanape | DONE: GOT LASER VISION PEW PEW PEW! | 15:09 |
urbanape | TODO: Get brain dump from Jason Foreman (threeve) whose last day is today. | 15:09 |
urbanape | BLOCK: My setup is not letting me run tests. I'm missing something stupidly simple, I'm sure. I'd appreciate a pair for a few minutes sometime today. | 15:09 |
urbanape | eom? | 15:09 |
mandel | ralsina, 1-1? | 15:10 |
ralsina | mandel: sure! | 15:10 |
mandel | ralsina, mumble! | 15:10 |
gatox | alecu, +1 for the other branch too | 15:11 |
mandel | ralsina, im there | 15:12 |
ralsina | mandel: you can't hear me? | 15:12 |
mandel | ralsina, no... stupid mumble, let me restart it.. | 15:12 |
ralsina | mandel: sube el volumen! | 15:12 |
ralsina | mandel: ack | 15:12 |
gatox | nessita, trivial: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/translation-problem/+merge/97901 | 15:16 |
nessita | gatox: ack | 15:17 |
nessita | gatox: doesn' this bug also fixes the other ussoc report about missing translations? | 15:18 |
gatox | nessita, this one is about missing translations... the other one is about encoding problems | 15:19 |
nessita | gatox: ah, ok, thanks | 15:19 |
gatox | nessita, i didn't wanted to put them together, so we can land this one and start with the translation while i fight with the other one | 15:20 |
nessita | gatox: yes, +1 | 15:20 |
=== yofel_ is now known as yofel | ||
* gatox lunch! | 15:34 | |
briancurtin | is it just me or are "reactor was unclean" issues like a needle in a haystack (maybe i just dont get it?) | 15:34 |
alecu | briancurtin, they usually mean that something was not properly closed or disposed of. | 15:36 |
briancurtin | yeah thats what im trying to hunt down | 15:36 |
alecu | briancurtin, do you have a paste for that error? | 15:36 |
briancurtin | alecu: lot of them in sso - https://pastebin.canonical.com/62479/ | 15:37 |
nessita | briancurtin: those are usually caused by a slow or loaded system | 15:44 |
nessita | briancurtin: can you please try either (or both) increasing the suite timeout and/or closing everything in windows and rebooting, to free mem? | 15:45 |
nessita | briancurtin: I bet they pass after that. Those tests require some (real) IPC interaction, and in windows those are *slow* (which is a bummer, yes, but we prefer that and have the whole ipc stack tested) | 15:45 |
briancurtin | nessita: i do need to reboot anyway since this update dialog keeps popping up. i'll try it and see if the timeout needs to be changed | 15:46 |
alecu | nessita, briancurtin: I'm getting those errors too in my windows VM, and I did not usually get them. | 15:46 |
nessita | briancurtin: thanks!!! | 15:46 |
alecu | nessita, so my guess is that something else changed. | 15:46 |
nessita | alecu: reboot... windows mem management sucks (IMHO) | 15:46 |
nessita | alecu: the mem in windows seems to run out just because the OS is running | 15:47 |
alecu | nessita, I've *just now* started windows on this machine. | 15:47 |
nessita | alecu: is your host machine loaded? | 15:47 |
alecu | nessita, and I think it was not even paused. | 15:47 |
alecu | nessita, no. | 15:47 |
nessita | alecu: let me try then | 15:47 |
alecu | nessita, it's got 8 cores, and it's mostly idle. | 15:47 |
nessita | alecu: ack | 15:47 |
nessita | testing in my VM now | 15:48 |
briancurtin | it seems like the same tests consistently fail, so i think its just that something isnt cleaned up properly, but i do need to reboot anyway so brb | 15:48 |
* alecu needs to pick amelia from the new kinder, see you in a few minutes. | 15:48 | |
nessita | gatox: approved | 15:49 |
nessita | anyone could review this trivial (in code, not in semantics) branch? https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/translation-problem/+merge/97901 | 15:50 |
dobey | alright, needs lunch. bbiab | 15:52 |
ralsina | briancurtin: I think you are right, the lock is because syncdaemon is not taking connections | 16:01 |
ralsina | briancurtin: so, u1cp is trying to connect to it and failing miserable | 16:02 |
briancurtin | yeah CP just sits there. if you let it go long enough it'll eventually get some popup message (that i cant remember), but it takes like 10 minutes | 16:02 |
ralsina | the good news is, syncdaemon seems to work | 16:03 |
briancurtin | ralsina: so should i keep looking at that SSO unclean reactor stuff, or dive into SD? | 16:05 |
ralsina | briancurtin: doesn't even seem to be opening its server port, AFAICS | 16:05 |
ralsina | briancurtin: let's try SD | 16:05 |
briancurtin | ralsina: i seem to remember this to be true that in the past you could only open one SD at a time, no? | 16:05 |
ralsina | briancurtin: yep | 16:06 |
briancurtin | because i noticed now if you just keep opening u1sdtools it'll open more SDs | 16:06 |
ralsina | briancurtin: that's not supposed to happen. But if it's failig to open its server port, it's understandable | 16:06 |
ralsina | So I guess tcp activation is broken | 16:06 |
briancurtin | yeah now it makes sense | 16:06 |
ralsina | or something similar | 16:06 |
ralsina | briancurtin: process monitor doesn't show syncdaemon with any LISTENING port | 16:09 |
briancurtin | hmm | 16:09 |
briancurtin | since i don't know where off the top of my head yet, i'm going to step through SD startup and try to find it | 16:11 |
ralsina | briancurtin: it seems it should log "IPC Initialized" when it happens. And I don't have that on my logs | 16:19 |
=== JackyAlcine_ is now known as jalcine | ||
briancurtin | ralsina: yeah, i have a breakpoint in the __init__ where that happens. IPCInterface never gets created | 16:23 |
ralsina | briancurtin: that should be called from interaction_interfaces.py line 1167 | 16:26 |
briancurtin | i was just opening that :) | 16:26 |
ralsina | briancurtin: ok, you follow it, drop breadcrumbs here :-) | 16:27 |
briancurtin | sounds good | 16:27 |
alecu | mandel, ping | 16:28 |
mandel | alecu, pong | 16:28 |
alecu | mandel, I'm about to go to lunch. Any progress on the qt branch? | 16:28 |
alecu | mandel, can I review it already? :-) | 16:28 |
* ralsina goes get a mate working | 16:28 | |
mandel | alecu, got problems updating the proxy when there is an error because of the caching of the network access manager.. | 16:29 |
ralsina | briancurtin: I am going to drop off windows to work on a linux branch, let me know if you need me | 16:29 |
mandel | alecu, it was 'easy' to fix listening to the proxy signal, not that easy in the errback | 16:29 |
briancurtin | ralsina: sounds good. i think we're going in the right direction now | 16:29 |
gatox | nessita, do i need another revision for this? https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/translation-problem/+merge/97901 | 16:29 |
nessita | gatox: I'd say merge | 16:30 |
gatox | nessita, ok | 16:30 |
alecu | mandel, ok. I'll be back in 30 mins aprox, twit me if you need me sooner. | 16:30 |
mandel | alecu, ok | 16:30 |
nessita | ralsina: any idea why controlpanel tests, the qt4 suite, is segfaulting on widnows? (using trunk) | 16:50 |
alecu | mandel, back | 16:59 |
alecu | well, in fact I never left :P | 16:59 |
mandel | alecu, ack | 16:59 |
mandel | alecu, I hate QNetworkAccessManager and its creds cache.. | 17:00 |
alecu | I just had some fruits, now having some soup while risking the keyboard | 17:00 |
alecu | thisfred, ping | 17:06 |
thisfred | alecu: pong | 17:07 |
thisfred | sry, haven't gotten to the branch yet | 17:07 |
alecu | thisfred, ack | 17:07 |
thisfred | will do it now | 17:07 |
alecu | thisfred, let me know as soon as you find any issue, because I'd like for that branch to land today so it makes it into beta 2 | 17:08 |
nessita | ok, lunchtime here | 17:08 |
thisfred | right, switching system to proxy now | 17:08 |
ralsina | nessita: was not segfaulting for me yesterday IIRC | 17:09 |
ralsina | nessita: it was locking up though | 17:09 |
briancurtin | ralsina: ubuntuone/syncdaemon/main.py:113 - creating that ActionQueue causes something to go wrong (trying to figure it out in debugger, it's in some huge twisted loop) | 17:09 |
ralsina | briancurtin: great | 17:10 |
briancurtin | i'm going to grab lunch before i get sucked into this issue and forget to eat | 17:10 |
thisfred | alecu do I need a special version of sso for this or just trunk? | 17:13 |
alecu | thisfred, trunk should be fine. Also the sso in nightlies is fine too. | 17:14 |
alecu | (I think) | 17:14 |
thisfred | kk | 17:14 |
gatox | need to restart | 17:16 |
gatox | no need to restart anymore :P | 17:17 |
thisfred | alecu, also, since this is the client and not the control panel, what exactly do I run? Just shut down u1sdtool, and then run that from t | 17:18 |
thisfred | he branch? | 17:18 |
thisfred | shoot, don't have builddeps and they won't install through the proxy... :S | 17:21 |
alecu | thisfred, I usually run it with: reset;U1_DEBUG=True PYTHONPATH=. bin/ubuntuone-syncdaemon --debug | 17:22 |
thisfred | kthx | 17:22 |
briancurtin | ok, *now* going for lunch (must. stop. debugging.) | 17:24 |
dobey | yay has tickets | 17:25 |
thisfred | tickets for? | 17:26 |
dobey | uds | 17:28 |
mandel | thisfred, ready for a review? | 17:29 |
mandel | thisfred, https://code.launchpad.net/~mandel/ubuntu-sso-client/use-qt4-webclient-backend/+merge/97934 | 17:29 |
thisfred | mandel: in a bit, reviewing alecu's branch right now | 17:30 |
mandel | alecu, puto! puto! puto! ;-) | 17:30 |
mandel | alecu, the told blode is mine! | 17:30 |
mandel | tall... u brainfuck | 17:30 |
alecu | mandel, what's a blode? "A bodyless head with spiky blond hair and black feet" ? | 17:31 |
mandel | alecu, my brain is stuck with the bloody QNetworkAccessManager.. | 17:31 |
mandel | stupid qt! | 17:31 |
thisfred | alecu: so I get 'Nonetype' object has no attribute 'host'. with your branch as well, but may | 17:32 |
thisfred | be I'm not testing the right thing | 17:32 |
dobey | alecu: blode is a one-char typoe of bloke :P | 17:32 |
mandel | thisfred, that is from libsoup.. | 17:33 |
gatox | nessita, ralsina unicode fix: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/unicode-args/+merge/97936 | 17:33 |
alecu | thisfred, where are you getting that error? on the log? | 17:33 |
thisfred | Also: why can't I select and copy text from the cp? | 17:33 |
dobey | thisfred: what text? | 17:33 |
thisfred | alecu no the control panel | 17:33 |
ralsina | gatox: looking | 17:33 |
thisfred | dobey: the error | 17:33 |
dobey | i guess we aren't setting that label to be selectable | 17:33 |
thisfred | alecu: I started sd, and it didn't connect so I figured I needed to use the cp to sign in | 17:34 |
alecu | thisfred, oh, ok. | 17:34 |
alecu | thisfred, try "u1sdtool -c" | 17:34 |
thisfred | kk | 17:34 |
thisfred | alecu does the device need to be registered or not? | 17:35 |
nessita | ralsina, briancurtin: reading a little backlog, it rings the bell that facundo added code in u1client to have 2 queues of commands, where the whole queue (which may be xtra long) is in disk, and a subset of it in memory | 17:35 |
ralsina | nessita: yes, that is breaking somewhere | 17:35 |
nessita | ralsina, briancurtin: so since you mentioned 'AQ seems stucked', perhaps that's it? | 17:35 |
nessita | ralsina, briancurtin: also, another not minor change on AQ was the tunnel adding | 17:35 |
nessita | ralsina, briancurtin: not sure if you talked with alecu about it | 17:36 |
alecu | about what? | 17:36 |
ralsina | nessita: not yet, maybe after briancurtin cmes back from lunch | 17:36 |
alecu | thisfred, if it's not registered then you'll be testing sso too. | 17:36 |
thisfred | alecu same error happens in sso when I'm not registered | 17:36 |
alecu | thisfred, is the proxy authenticated or not? | 17:37 |
nessita | alecu: syncdaemon is not starting in windows, apparently is getting stucked creating the ActionQueue, so I listed the 2 mayor changes we had in that class: the tunnel adding and the split of two queues (disk and in mem) | 17:37 |
thisfred | ok proxy off, reregistering | 17:37 |
thisfred | alecu: it is ralsina's proxy so yes | 17:37 |
ralsina | if anyone needs unauthed proxy I would need his IP | 17:38 |
alecu | thisfred, ok. Try installing a default squid3 with apt-get it will listen on localhost:3128 and ask for no passwords. | 17:38 |
alecu | mandel, reason = 'QNetworkAccessManager is buggy.' !!!!!! | 17:40 |
alecu | mandel, why skipping *so many* tests? | 17:40 |
mandel | alecu, QNetworkAccessManager is buggy. | 17:40 |
mandel | alecu, the stupid network access manager does not like when you change the settings of a proxy while requests are in the way.. | 17:41 |
gatox | nessita, the remove title for reset page: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/remove-title/+merge/97940 | 17:41 |
nessita | gatox: thanks! | 17:41 |
mandel | alecu, then, we (you and me) know that the proxyAuthenticationRequired really does not work, specially with the yield, so the situation is, we get a crazy warning from the auth cache manager and everything goes a la mierda | 17:42 |
mandel | alecu, I've skipped those so I don't block you and I fix the double dialog issue | 17:42 |
thisfred | alecu doesn't seem to want to connect when I do t | 17:43 |
thisfred | hat | 17:43 |
alecu | thisfred, can you paste the sd log? | 17:44 |
thisfred | alecu: it appears I cannot: pastebinit breaks on the proxy too, so I guess network settings is screwed up again | 17:47 |
thisfred | logging out and ing | 17:47 |
alecu | mandel, I'm wary of approving a branch that skips so many tests, at this point. | 17:48 |
alecu | mandel, so let's do something: I'll try IRLing it | 17:49 |
mandel | alecu, I was going to propose the same :) | 17:50 |
thisfred | alecu: http://paste.ubuntu.com/886635/ | 17:51 |
thisfred | alecu: that's ~/.cache/ubuntuone/log/syncdaemon-debug.log | 17:52 |
thisfred | is that the right log? | 17:52 |
ralsina | gotta go pick up the kid, will be back soonish | 17:53 |
thisfred | (pastebinit now works through squid with no password, but the client still does not, it seems | 17:53 |
mandel | alecu, in my IRL if you give the correct creds for the proxy everything goes like a charm | 17:55 |
mandel | alecu, those tests are just when you screw the creds | 17:55 |
alecu | thisfred, can you check for a proxy.log ? | 17:56 |
alecu | mandel, ack. with creds it works. | 17:56 |
alecu | mandel, I'm testing no creds now | 17:56 |
mandel | alecu, hombre de poca fe.. | 17:56 |
mandel | :) | 17:56 |
briancurtin | nessita: actually yeah it's in the tunnel stuff, a problem with a spawnProcess. i closed it to go to lunch but i'll re-run and get the exact spot | 17:57 |
thisfred | alecu: http://paste.ubuntu.com/886643/ | 17:57 |
nessita | briancurtin: awesome that you have that isolated | 17:57 |
mandel | alecu, if you give the wrong creds, it will get stuck | 17:57 |
nessita | briancurtin: please ask alecu for insight about it if you can't debug any further | 17:57 |
alecu | thisfred, thanks, I'll check it out in a few minutes after reviewing mandel's branch. | 17:58 |
thisfred | ok guess I'll switch to that too then | 17:58 |
mandel | alecu, and in the sso, when that happes if will get a QNetworkAccessCache telling that you are not allowed to update settings in a proxy.. | 17:58 |
mandel | alecu, which is due to: https://bugreports.qt-project.org/browse/QTBUG-14850?page=com.googlecode.jira-suite-utilities:transitions-summary-tabpanel | 17:59 |
mandel | but with a factory it is not as easy as that bugs description states.. | 18:00 |
dobey | ugh, jira | 18:01 |
alecu | mandel, have you tried entering wrong u1 credentials? It hangs up indefinitely (both using a nonauth proxy and an authenticated one.) | 18:01 |
mandel | alecu, let me test that | 18:02 |
thisfred | mandel: so how do I test this: run the trunk of CP with this in the PYTHONPATH? | 18:02 |
mandel | thisfred, yes | 18:03 |
alecu | mandel, entering wrong u1 creds also fails with no proxy configured. | 18:04 |
alecu | mandel, luckily you can close the window and retry connecting. | 18:05 |
mandel | alecu, wtf? let me see that | 18:05 |
thisfred | mandel: logging in through non-auth squid works | 18:06 |
alecu | mandel, this seems to happen in trunk too, when using the qt backend | 18:07 |
mandel | alecu, you got me testing on trunk too, and I can confirm that! | 18:07 |
thisfred | dobey: to come back to earlier: why in Zod's name would anyone ever make text non-selectable. | 18:08 |
thisfred | that that option even exists is a bug | 18:08 |
mandel | thisfred, a professor at a uni so that you cannot finish your paper | 18:08 |
dobey | thisfred: because having a cursor drawn somewhere that isn't a text entry, is fugly | 18:08 |
thisfred | why would there be a cursor? | 18:09 |
thisfred | it's not editable | 18:09 |
dobey | sorry, we'll rewrite everything in HTML5 for you | 18:09 |
thisfred | yay! ;) | 18:09 |
dobey | because i didn't write the widget library, so i didn't get to make the decision about what it should draw, and where | 18:09 |
nessita | dobey: needs fixing in https://code.launchpad.net/~dobey/ubuntu-sso-client/system-font/+merge/97264 for a couple of font sizes | 18:09 |
nessita | gatox, briancurtin: may I please have a review? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/default-folders/+merge/97949 | 18:10 |
gatox | nessita, yep | 18:10 |
briancurtin | nessita: will look in a few mins | 18:10 |
thisfred | all I'm saying is that whoever did write it does not get a cookie | 18:11 |
nessita | briancurtin, gatox: thanks! | 18:11 |
thisfred | mandel anything more you want me to test? | 18:12 |
mandel | thisfred, the qt branch, but I'm going to talk first with alecu.. | 18:12 |
mandel | alecu, quick mumble? | 18:12 |
thisfred | kk | 18:13 |
alecu | mandel, the "qtnetwork" hanging up on wrong u1 credentials seems to come from a long time ago. | 18:16 |
alecu | mandel, thing is, we never found out because it was not being used. | 18:16 |
alecu | mandel, I'm on mumble. | 18:16 |
mandel | alecu, on my way.. | 18:17 |
mandel | alecu, my mumble is failing, one sec | 18:17 |
mandel | alecu, sorry, mumble crashed again.. let me update the system | 18:19 |
dobey | nessita: is there some way to make that balloon thing expand and shrink properly for the text? | 18:19 |
alecu | mandel, ack, let me know. | 18:19 |
alecu | thisfred, I'm puzzled by this line on your proxy.log: 2012-03-16 13:50:39,551 - ubuntuone.proxy - ERROR - connection error: Proxy server not found (17) | 18:19 |
nessita | dobey: I'm not sure. But even if there is (which would be great for translations), the font there needs to be smaller than what currently is in your branch... | 18:20 |
thisfred | alecu maybe squid's running on a different port after all? | 18:20 |
thisfred | alecu: but everything else seems to work is the thing | 18:20 |
mandel | alecu, in theory, I'm there.. | 18:20 |
dobey | nessita: yes, but with <small> on my branch instead of normal size, half that balloon is empty and just dead space | 18:21 |
dobey | nessita: on my computer anyway :) | 18:21 |
dobey | nessita: and if someone has set their fonts to a larger size, we still have the same issue | 18:21 |
alecu | thisfred, try: telnet localhost 3128<enter>GET http://slashdot.org HTTP/1.0<enter><enter> | 18:21 |
nessita | dobey: i guess without the <small> you see it completly? | 18:21 |
dobey | nessita: yeah, without the small it looks fine for me; but i'm using tiny fonts | 18:22 |
thisfred | alecu: works | 18:22 |
nessita | dobey: right. So, since I now you have more in your plate, I would say: use <small> to match what we currently have, and let's report the bug about the ballon needing to shrink and expand for larger fonts and/or larger texts | 18:23 |
ralsina | dobey, nessita: +1 those are two separate bugs | 18:27 |
dobey | QWizardPage::addField: Duplicate field 'email_address' | 18:27 |
dobey | what is that all about? | 18:27 |
ralsina | dobey: that has been there forever, nevermind | 18:27 |
ralsina | dobey: basically, on a QWizard you are not supposed to have two fields with the same name | 18:28 |
ralsina | dobey: it's harmless | 18:28 |
dobey | you shouldn't be able to have 2 fields with the same name on anything | 18:28 |
dobey | harmless or not, it's ugly, and makes the tests output ugly, because it gets printed like a billion times | 18:29 |
ralsina | dobey: ok, file a bug I promise I will fix it after the release | 18:29 |
ralsina | dobey: I am not renaming things today ;-) | 18:29 |
dobey | hrmm, the sso tests also seem to be pretty slow | 18:29 |
dobey | ralsina: after the release == after windows .exe building, or after tuesday? | 18:29 |
ralsina | dobey: after tuesday | 18:30 |
ralsina | dobey: since that is crossplatform code | 18:30 |
dobey | heh | 18:31 |
dobey | i /am/ changing the font handling, you know ;) | 18:31 |
ralsina | dobey: sure, what's that have to do with that message? | 18:31 |
dobey | ralsina: i'm pretty sure it's cross platform code | 18:32 |
ralsina | dobey: I have a list of things I expect to get done by today. Changing the fonts? In it. Renaming widgets? Not in it ;-) | 18:32 |
dobey | ralsina: does that include a linux.qss/windows.qss fix for sso as well? | 18:33 |
ralsina | dobey: yes, branch is proposed with yours as prereq | 18:33 |
dobey | nessita: fixed/pushed | 18:34 |
nessita | dobey: ack | 18:34 |
dobey | ralsina: ah ok. didn't see it yet, i'll look now | 18:34 |
ralsina | dobey: you have already approved it :-) | 18:34 |
ralsina | Oh, wait, that's u1cp | 18:34 |
dobey | right, that's cp not sso :) | 18:34 |
ralsina | dobey: I can do one for sso in 10 minutes after yours lands | 18:35 |
mandel | alecu, ping? | 18:35 |
dobey | ok | 18:35 |
briancurtin | ralsina, alecu: so tunnel_runner.py tries to start "ubuntuone-proxy-tunnel" - reactor.spawnProcess chokes on this (i'll figure the exact details out). i modified TunnelRunner.start_process to start "ubuntuone-proxy-tunnel.PY" (note extension) by way of python.exe - it works | 18:35 |
briancurtin | so the SD startup was just hanging around inside of reactor.spawnProcess forever | 18:36 |
ralsina | briancurtin: yay, then this should not be terribly hard to fix! Good job! | 18:36 |
alecu | briancurtin, cool. But please don't land that on trunk yet as it might break the linux release! | 18:36 |
ralsina | alecu: the windows release is supposed to be done from the same tarballs | 18:36 |
ralsina | alecu: so we better make it work for both :-) | 18:37 |
alecu | ralsina, cool, then let's fix it in a sane way for both | 18:37 |
ralsina | alecu: right | 18:37 |
ralsina | alecu: in fact, it's trying to spawn the tunnel even without proxy configured! | 18:37 |
alecu | ralsina, that why I said "not yet" :-) | 18:37 |
briancurtin | i'll work up a patch to build the right command lines and arguments | 18:38 |
alecu | ralsina, that's fine. Qt knows if the proxy is configured. SD should not use the qtreactor. | 18:38 |
briancurtin | also has to consider when it's a Windows install so it'd be a compiled EXE | 18:38 |
ralsina | alecu: why is it ok to spawn a process it won't use? Lost you there | 18:39 |
alecu | ralsina, the tunnel will start, and it will print a port number to stdout if proxy is configured. Or exit if no proxies are needed. | 18:39 |
mandel | alecu, bug 957313 | 18:39 |
ralsina | alecu: ah ok | 18:39 |
ubot5 | Launchpad bug 957313 in Ubuntu Single Sign On Client "If proxy settings are wrong the qt webclient cannot update the creds" [Medium,Triaged] https://launchpad.net/bugs/957313 | 18:39 |
mandel | alecu, I'm leaving yet I have added bug 957317 | 18:45 |
ubot5 | Launchpad bug 957317 in Ubuntu Single Sign On Client "When u1 credentials are wrong and the qt webclient is used the error is not propagated" [Critical,New] https://launchpad.net/bugs/957317 | 18:45 |
mandel | alecu, and a possible hint of the reason, yet I need to know what is the sso backend backend (that is the server side not the sso backend) doing | 18:45 |
alecu | mandel, why is test_anonymous_proxy_is_used also skipped? | 18:45 |
mandel | alecu, proably I screwed up skipping that one, let me update it | 18:46 |
alecu | mandel, I'm not sure it's an sso backend thing, since the libsoup webclient seems to work ok in this case. | 18:47 |
alecu | and by "work ok" I mean, it returns the error expected when u1 credentials are wrong. | 18:47 |
mandel | alecu, ok, I'll look closer on monday | 18:48 |
alecu | ack | 18:48 |
alecu | ralsina, do you have 5 mins for a quick status update? | 18:48 |
ralsina | alecu: sure, mumble? | 18:48 |
alecu | ralsina, perfect | 18:48 |
alecu | mandel, if you have to leave, go ahead. | 18:49 |
mandel | alecu, is close to 8 and I'm been here since my 9 :P | 18:49 |
mandel | alecu, and my head is about to explode | 18:49 |
ralsina | mandel: please go away then ;-) | 18:49 |
alecu | mandel, yesterday my EOD was at my 1 AM. | 18:49 |
mandel | alecu, you crazy! | 18:50 |
mandel | alecu, push branch with the correct tests skipped | 18:50 |
mandel | s/push/pushed | 18:50 |
mandel | alecu, I'll look at the issue during the weekend.. we have waaaaaaaaay to many things interacting, looks like the windows ipc ;) | 18:51 |
* mandel EOD | 18:51 | |
nessita | briancurtin: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/default-folders/+merge/97949 fixed, great catch, thanks!!! | 18:53 |
briancurtin | ralsina: i have a fix for running SD from Python, checking now that i fixed it for a py2exe'd build | 18:56 |
ralsina | briancurtin: awesome | 18:56 |
briancurtin | nessita: looks good, approved :) | 18:56 |
nessita | thanks! :-) | 18:56 |
dobey | nessita: heh. i *just* posted a needs fixing on there too :) | 18:57 |
nessita | let's see | 18:57 |
nessita | dobey: which folders are not supported on XP? I specifically used the function with support on XP | 18:58 |
dobey | nessita: not the function, but the constants. CSIDL_MYVIDEO seems to only exist for 6.0+ (or that documentation is confusing?) | 18:58 |
nessita | dobey: let me re-check then | 18:58 |
briancurtin | ah yeah i think he's right, those are the "libraries" that were added in vista (6.0) | 18:59 |
nessita | ok, will try-except then | 18:59 |
nessita | so they get listed on vista and above | 18:59 |
dobey | right | 18:59 |
nessita | dobey: nice catch with the .config dir | 19:00 |
ralsina | nessita: have 1' for a quick mumble? | 19:03 |
nessita | ralsina: being 110% honest, no. But of course I can if you really need me | 19:03 |
ralsina | nessita: it's ok, ping me when you do | 19:03 |
nessita | ralsina: I'm afraid I'm not gonna make it today with the more-pages branch | 19:03 |
nessita | :-( | 19:04 |
ralsina | nessita: ok | 19:04 |
gatox | nessita, +1 | 19:04 |
nessita | will be closed, but not sure I can land that today | 19:04 |
ralsina | nessita: then we do need to talk a little, will bebrief | 19:04 |
nessita | ralsina: ok, hooking up headset | 19:04 |
dobey | oh mandel is gone | 19:05 |
ralsina | dobey, alecu, nessita, briancurtin, gatox, mandel: because thing have gone all explodey today, we are delaying today's trunk freeze to monday at 6PM UTC | 19:14 |
briancurtin | ralsina: ack | 19:14 |
ralsina | then on tuesday we will do intensive QA and package the things on wednesday | 19:14 |
gatox | ralsina, ack | 19:15 |
alecu | ack | 19:15 |
ralsina | so, rest on the weekend and kick ass on monday, ok? | 19:15 |
dobey | bah. test in production :) | 19:16 |
ralsina | dobey: that's going to be like 50% true next week ;-) | 19:28 |
briancurtin | argh, the py2exe fix didn't work. close, though | 19:29 |
dobey | heh | 19:29 |
dobey | thisfred: what branch were you testing (or trying to test) earlier, involving proxies? | 19:30 |
thisfred | dobey: mandels use qt one and alecu's proxy tunnel one | 19:31 |
dobey | ok | 19:32 |
nessita | dobey: the encoding of the user-dirs file will be sys.getfilesystemencoding()? | 19:35 |
nessita | or that just applies to file *names* (and not content)? | 19:35 |
dobey | that only applies to names | 19:35 |
nessita | dobey: then, have any suggestion? | 19:35 |
nessita | encoding can not be guessed | 19:36 |
dobey | nessita: fail if it's not utf-8? | 19:36 |
nessita | I guess I can try utf-8 and ignore if not | 19:36 |
nessita | right | 19:36 |
dobey | i don't see any reason it shouldn't be utf-8, unless something broke pretty badly | 19:36 |
alecu | ralsina, found the problem (and a solution!) for the bug with the u1 credentials with qtnetwork. | 19:51 |
ralsina | alecu: awesome! | 19:51 |
alecu | ralsina, it seems that the wrong credentials were being retried *forever*. | 19:51 |
ralsina | alecu: ugh | 19:51 |
alecu | a branch to fix it is on the way. | 19:51 |
nessita | dobey: all issues addressed now on https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/default-folders/+merge/97949 | 19:56 |
nessita | dobey: good catch, though I would have enjoyed fixing a lot more if I wasn't with so much rush | 19:56 |
nessita | anyays, import codecs rule | 19:56 |
ralsina | anyone needs a hand, a review, anything? | 19:59 |
ralsina | cofee? cake? | 19:59 |
alecu | cake would do | 19:59 |
ralsina | alecu: on it! | 19:59 |
alecu | you'll start cooking it now? | 19:59 |
ralsina | alecu: http://bit.ly/FOa85J | 20:00 |
ralsina | until we get cake-over-ip that's the best I can do | 20:00 |
* alecu is now officially hungry. | 20:00 | |
briancurtin | ha, a friend of mine started http://foodoverip.org/ | 20:01 |
ralsina | briancurtin: how's the spawner coming along? | 20:01 |
briancurtin | ralsina: still need to figure out py2exe. i can start ubuntu-sso-login.exe and then start SD via Python, then ubuntuone-control-panel-qt.exe, and it seems to work "ok". im pasting a few errors i just saw a minute ago before i go figure them out | 20:02 |
ralsina | briancurtin: cool | 20:02 |
dobey | the cake is a lie | 20:03 |
briancurtin | ralsina: https://pastebin.canonical.com/62504/ and https://pastebin.canonical.com/62500/ | 20:04 |
briancurtin | in any case, it's really close to working. need to add a bunch of logging or something to the py2exe build since it still can't find the right executable to start | 20:05 |
ralsina | briancurtin: that's the same exception seem from both sides of IPC right? | 20:05 |
thisfred | briancurtin: is that friend Benoît, or is he just a very enthousiastic user? :) | 20:06 |
dobey | nessita: +1 | 20:07 |
nessita | dobey: thanks! | 20:07 |
briancurtin | thisfred: yeah its Benoît, the site was started at a sprint at Tarek Ziade's house where the focus became food and wine | 20:08 |
thisfred | heh | 20:08 |
thisfred | I know him a little from my couchd | 20:08 |
thisfred | b efforts | 20:08 |
thisfred | he's nice | 20:08 |
briancurtin | i know tarek much more than benoit, but they're all a good group | 20:09 |
gatox | EOD for me!! bye!! see you on monday | 20:09 |
briancurtin | ralsina: i think it probably is the same exception, i'll see what changed there | 20:09 |
thisfred | see you latox alli | 20:09 |
thisfred | gatox | 20:10 |
dobey | oi, it's 1600 already | 20:10 |
gatox | thisfred, jejee abstract message :P | 20:11 |
thisfred | dobey: and nobody's brought me a mint julep yet! | 20:11 |
alecu | trial is the awesome. Love it. | 20:13 |
thisfred | alecu: is that sarcasm? ;) | 20:13 |
thisfred | I've been looking at py.test again after I saw Holger's pycon talk... | 20:14 |
thisfred | It's nice | 20:14 |
alecu | thisfred, no, seriously. I'm in love with it. I'll say it louder: | 20:15 |
alecu | 𝕀 𝕝𝕠𝕧𝕖 𝕥𝕙𝕖 𝕥𝕨𝕚𝕤𝕥𝕖𝕕.𝕥𝕣𝕚𝕒𝕝 𝕥𝕖𝕤𝕥 𝕣𝕦𝕟𝕟𝕖𝕣! | 20:15 |
thisfred | hehe | 20:15 |
thisfred | I don't dislike it | 20:15 |
thisfred | I like the repeat until failure option | 20:16 |
thisfred | great if you have randomly failing tests | 20:16 |
thisfred | what I like about py.test is that it makes for much less boiler plate in most tests | 20:17 |
thisfred | so they become a lot easier to read | 20:18 |
thisfred | and the failure messages are super too | 20:18 |
alecu | thisfred, can I return a deferred from a py.test ? That's the killer feature for any test runner I use :-) | 20:18 |
dobey | i don't think i'd want to a) convert devtools to py.test b) make py.test work with twisted | 20:19 |
thisfred | I bet you you can, but why would a test return anything? | 20:19 |
dobey | thisfred: because it's twisted | 20:19 |
thisfred | dobey: I'm not proposing that we do | 20:19 |
thisfred | Just trying it out on a hobby project | 20:20 |
dobey | i'm just saying i don't like twisted | 20:20 |
thisfred | I wasn't arguing on that point | 20:20 |
dobey | also, alecu's branch name is making me hungry | 20:20 |
alecu | this one? https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-cookies/+merge/97791 | 20:23 |
dobey | yes | 20:24 |
dobey | also, why the heck were we using a symlink for the music library in rhythmbox | 20:24 |
dobey | also, where is sil so i can yell at him for making it a symlink | 20:25 |
rye | umm why does quickly depend on couchdb-bin? | 20:27 |
rye | ah, desktopcouch | 20:27 |
rye | why does quickly depend on desktopcouch then? | 20:27 |
ralsina | popping back to ubuntu, will be back in 10' or so | 20:27 |
dobey | it used to have CouchGrid widget | 20:27 |
dobey | rye: i think that widget has been removed now. maybe the package deps are not updated? | 20:28 |
rye | dobey: checking... | 20:28 |
rye | data/templates/ubuntu-pygame/project_root/python/hiscores.py:from desktopcouch.records.server import CouchDatabase | 20:29 |
dobey | hmm | 20:30 |
dobey | rye: talk to rickspencer3 about it then | 20:30 |
nessita | dobey: system-font approved | 20:50 |
dobey | yay | 20:54 |
ralsina | thisfred: I have a very small branch for review here https://code.launchpad.net/~ralsina/ubuntuone-control-panel/ubuntu-font-in-windows/+merge/97554 | 21:17 |
thisfred | 5 MLOC of XML later... | 21:17 |
ralsina | dobey: that's *2* lines of XML :-) | 21:18 |
ralsina | oops, I meant thisfred | 21:18 |
=== m_conley_away is now known as m_conley | ||
thisfred | hehe, so we use the ubuntu font except when on Ubuntu? makes sense :P | 21:18 |
ralsina | thisfred: when the ubuntu font is the font in ubuntu, it will make more sense, agreed ;-) | 21:18 |
thisfred | it isn't? Maybe I set it by hand then | 21:19 |
dobey | it is | 21:20 |
dobey | the default is, anyway | 21:20 |
nessita | ralsina: I'm proposing the branch with 2 of the 3 pages added (still pending the more complex and last one, the local to cloud) | 21:20 |
ralsina | dobey: I don't have it but may be because I updated from O? | 21:21 |
ralsina | dobey: as default, I mean | 21:21 |
ralsina | nessita: that's cool | 21:21 |
nessita | ralsina: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/cloud-to-computer-page/+merge/97990 | 21:21 |
ralsina | nessita: easier to test and everything | 21:21 |
nessita | ralsina: well, kinda, I had to adapt the branch to have a page that will not be final to be final in this branch | 21:21 |
ralsina | now that I think about it, our embedded ubuntu font is probably not the latest | 21:21 |
dobey | ralsina: did you set it to something else perhaps? | 21:22 |
ralsina | dobey: not that I recall | 21:22 |
nessita | ralsina: so every branch results in a fully working wizard | 21:22 |
ralsina | nessita: good thinking | 21:22 |
thisfred | ralsina: +1 | 21:23 |
ralsina | thisfred: thanks! | 21:23 |
nessita | thisfred: you dying to do another review, no? :-P | 21:26 |
nessita | this one is fun, you will the premiere about the new wizard pages! | 21:26 |
* nessita tries to be a sellwoman | 21:26 | |
thisfred | nessita: sí! | 21:26 |
nessita | YEAH | 21:27 |
nessita | thisfred: will add IRL instructions and then share the link with you | 21:27 |
thisfred | kk | 21:27 |
nessita | ralsina, thisfred: IRL instructions added to https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/cloud-to-computer-page/+merge/97990 | 21:30 |
ralsina | nessita: thanks! | 21:30 |
* thisfred is on the case | 21:30 | |
dobey | alright all. am off for now. have a good weekend! | 21:30 |
ralsina | nessita: any suggestions about where to put a platform-specific constant on sso's gui? | 21:30 |
thisfred | later dobey | 21:30 |
nessita | and I guess this is all for me today, will hang out for 15 minutes more but then I'm off to pilates and to have some nice night out | 21:30 |
nessita | ralsina: which constant? | 21:31 |
ralsina | nessita: ":/windows.qss" on windows ":/linux.qss" on linux | 21:31 |
ralsina | nessita: to force the ubuntu font only on windows | 21:31 |
nessita | ralsina: let me digg | 21:31 |
ralsina | nessita: I was thinking constant.py.in but I may have to make the setup.py a bitmore complex | 21:32 |
nessita | ralsina: did you added the frozen thing to setup.\py? :-/ | 21:32 |
ralsina | nessita: no | 21:32 |
nessita | ralsina: ah.... relief!!! | 21:32 |
ralsina | nessita: I added it on the runner | 21:32 |
nessita | ralsina: hum... not to get_bin_dir? | 21:33 |
ralsina | nessita: oops, you are right, to get_bin_dir | 21:33 |
nessita | ralsina: is that branch landed already? | 21:33 |
ralsina | nessita: yes | 21:33 |
nessita | ok | 21:33 |
ralsina | nessita: it still needs some fixes by brian to add ".exe" at the end of binary names and such | 21:33 |
nessita | ralsina: who approved? | 21:34 |
ralsina | nessita: hmmm don't recall outof memory | 21:34 |
ralsina | nessita: diego and dobey | 21:34 |
nessita | I was told we never have to use this way: | 21:35 |
nessita | 93 if hasattr(sys, "frozen"): | 21:35 |
nessita | but instead: | 21:35 |
nessita | 93 if getattr(sys, "frozen", False): | 21:35 |
nessita | or | 21:35 |
nessita | 93 if getattr(sys, "frozen", None) is not None: | 21:35 |
ralsina | by chipaca right? | 21:35 |
nessita | yes | 21:35 |
nessita | ralsina: hasattr is very evil | 21:35 |
* briancurtin quietly changes the hasattr i just used for sys.frozen | 21:35 | |
thisfred | nessita: when I check a folder to be synced locally, the screen flashes and then the checkmark is gone | 21:36 |
ralsina | briancurtin: change mine too please ;-) | 21:36 |
ralsina | nessita: briancurtin will undo that, sorry | 21:36 |
nessita | thisfred: yes, the flash is the 'loading' screen, it should appear "the same" in the regular controlpanel folder screen | 21:36 |
nessita | thisfred: and disapperas because is no longer "remote" (is local now that you subscribe to it) | 21:36 |
nessita | briancurtin: thanks :-) | 21:37 |
nessita | ralsina: is ok. SO, going back to the constant thingy | 21:37 |
nessita | there does not seems to be a good place right now, I would advice: | 21:37 |
nessita | ralsina: ubuntu_sso/qt/utils/{__init__.py,linux.py,windows.py, tests/test_linux.py,tests/test_windows.py} | 21:38 |
ralsina | nessita: I could make constants.py smarter by using Template (http://docs.python.org/library/string.html#template-strings) | 21:38 |
thisfred | nessita: I don't understand, but if that is known behavior then I'm fine with it. It looks to me like something didn't work. | 21:38 |
nessita | thisfred: is known, will take notes to talk to design | 21:38 |
nessita | ralsina: at this point, to be consistent, I would prefer the ubuntu_sso/qt/utils/{__init__.py,linux.py,windows.py, tests/test_linux.py,tests/test_windows.py} solution | 21:39 |
ralsina | nessita: ok | 21:39 |
nessita | ralsina: any advantage in the template solution that I'm missing? | 21:39 |
ralsina | nessita: that the next time we need to add a constant that is not @prefix@ we will have to do it anyway | 21:39 |
nessita | ralsina: woudl you be ok to wait till then? and then we may even apply the cleaner fix to controlpanel as well, since now we have the same @prefix@ trick | 21:40 |
ralsina | sure | 21:40 |
thisfred | nessita: also: http://ubuntuone.com/4MqS4Lr2pEscipn54Goup9 I don't understand what 'Sync locally?' without a checkbox means. Seems like I am being asked a question without a way to answer ;) | 21:41 |
ralsina | thisfred: those seem to be collapsed. What happens if you doble cick on "shared by cparrino"? | 21:41 |
thisfred | ralsina: 1. they should have an arrow then, and 2. nothin | 21:42 |
thisfred | g... | 21:42 |
ralsina | thisfred: ok | 21:42 |
thisfred | although I see log lines in the terminal appearing | 21:43 |
ralsina | thisfred: I assume you are already subscribed to everything you got from cparrino and that's why those are not there | 21:43 |
thisfred | ralsina: then neither should the question, right? :) | 21:44 |
ralsina | thisfred: exactly | 21:44 |
thisfred | nessita: want to fix that on this branch or should I approve this? | 21:44 |
nessita | thisfred: looking | 21:44 |
thisfred | no crashes, the close dialog works | 21:45 |
nessita | thisfred: oh nice bug... need to filter group headers with no folder in them :-) | 21:45 |
thisfred | awesome | 21:45 |
nessita | thisfred: will fix in another branch, mainly because UIFe runs | 21:45 |
nessita | ralsina: is that ok with you? ^ | 21:45 |
thisfred | kk, approving this then | 21:45 |
nessita | thisfred: would you have the time to file me the bug as well? | 21:46 |
thisfred | sure | 21:46 |
* nessita asks too much today | 21:46 | |
nessita | thisfred: thanks, really, I need to run to pilates ;) | 21:46 |
ralsina | nessita: I think you shouldnot remove the items when they are checked. That was not the behaviour it had before, and is weird. Will try to say something coherent about it before monday. | 21:47 |
nessita | ralsina: ok... I may come up with something, but that may complicate the branch in a non trivial way... anyway, I haven't thought a lot about that | 21:50 |
thisfred | #957458 | 21:50 |
thisfred | nessita: ^ linked in mer | 21:50 |
thisfred | ge proposal and assigned to youh | 21:50 |
nessita | thisfred: thanks!!! | 21:50 |
ralsina | nessita: it's ok, I will too. May have something for you to look at. | 21:50 |
nessita | ok | 21:51 |
thisfred | now it's the dog's turn | 21:51 |
thisfred | later all | 21:51 |
ralsina | thisfred: just 1 sec | 21:51 |
ralsina | thisfred: nevermind | 21:52 |
nessita | ok, I'm gone | 21:52 |
nessita | bye all! | 21:52 |
nessita | have a nice weekend | 21:52 |
ralsina | bye nessita! | 21:52 |
alecu | so, anybody still up for a small review? https://code.launchpad.net/~alecu/ubuntu-sso-client/fix-wrong-credentials/+merge/97989 | 22:04 |
alecu | anyway... EOW for me | 22:06 |
alecu | bye all!!! | 22:06 |
rye | UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal success_filter=lambda info: info['path'] == path, | 22:11 |
rye | while trying to publish non-ascii file | 22:11 |
rye | what i dislike in twisted is that unless you specifically terminate it, it will keep the reactor running | 22:12 |
rye | even if there is a omg fatal error | 22:12 |
briancurtin | im heading out of here. brain is fried. see everyone next week. | 22:14 |
=== m_conley is now known as m_conley_away |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!