| mandel | morning all!! | 07:50 |
|---|---|---|
| fagan | morning mandel | 07:52 |
| mandel | fagan, morning ;) | 07:54 |
| mandel | fagan, have you heard the news? | 07:54 |
| fagan | mandel: steve dying yeah? | 07:54 |
| fagan | sad :-/ | 07:54 |
| mandel | really sad.. | 07:55 |
| JamesTait | Brrrrr! Good moaning! | 08:41 |
| popey | Morning! | 08:47 |
| popey | I'm testing some code which will use openid (specifically Ubuntu SSO) for auth. | 08:47 |
| popey | is there a staging server for SSO I can use? | 08:47 |
| * popey discovers https://staging.one.ubuntu.com/ | 08:48 | |
| mandel | popey, hm.. good queestion, what are you actually doing? | 08:51 |
| popey | I want to make a simple website which uses launchpad/ubuntu SSO as the auth provider | 08:51 |
| mandel | popey, when I worked porting sso to windows I used the actual server using the + sign in a gmail account.. | 08:52 |
| popey | in the same way that (for example) askubuntu.com does | 08:52 |
| mandel | popey, ok, seems reasonable, if there is no u1 involved you can ignore our code :) | 08:52 |
| popey | right, okay, if that's acceptable | 08:52 |
| mandel | popey, yes, it is more thant acceptable, the service is there for you to use :) | 08:53 |
| mandel | popey what lang are you using, we migh have some example there :) | 08:53 |
| mandel | popey, and let me a sec and I'll as the people that manage sso to see if there are any issues using staging :) | 08:53 |
| popey | php | 08:53 |
| mandel | shit, I use lots of smilies, I must be happy today hehe | 08:53 |
| popey | ☺ | 08:54 |
| mandel | popey, just asked but we might need to wait a little, I think most people from SSO are in the other side of the athlantic, I'll ping you as soon as I get an asnwer | 08:55 |
| popey | ok | 08:56 |
| popey | seems most code I'd find would be python based | 08:56 |
| popey | figured someone must have done an openid client library in php | 08:56 |
| mandel | popey, there should be one, I did some work with js, does that help you? | 08:57 |
| mandel | popey, we can also ping aqurius whenever he is up, he might now about some crazy lib | 08:57 |
| popey | not sure, I'm not the actual coder ☺ | 08:58 |
| popey | will go and have a play and come back later :D | 08:58 |
| mandel | popey, sure, I'll try to get you the info asap | 09:00 |
| popey | thanks! I appreciate the help | 09:01 |
| mandel | popey, I'm here for that :) | 09:03 |
| popey | heh | 09:03 |
| ralsina | good morning. Mandel ping | 09:22 |
| mandel | ralsina, buenos dias! | 09:30 |
| ralsina | buen dia mandel! | 09:30 |
| ralsina | Sorry I am late with that review, ding it now | 09:30 |
| ralsina | doing it | 09:30 |
| mandel | ralsina, I was about to have a coffee, so you have a couple of mins hehe | 09:30 |
| mandel | ralsina, and I moves to the migratin data bug which is nearly done | 09:30 |
| * mandel coffee break | 09:31 | |
| ralsina | mandel: I get test failures, but couldn't capture the log, so am running again | 09:58 |
| mandel | ralsina, weird, please do pate them somewhere | 10:01 |
| mandel | ralsina, there are 4 tests that fail in trunk already | 10:02 |
| ralsina | mandel: could be those | 10:02 |
| mandel | ralsina, yeah, nevertheless please paste and I'll take a look | 10:05 |
| mandel | ralsina, better land that branch late than sorry :) | 10:05 |
| ralsina | mandel: 2 fails: http://pastebin.ubuntu.com/703295/ | 10:05 |
| ralsina | and 13 errors | 10:05 |
| ralsina | let me paste more :-/ | 10:06 |
| mandel | ralsina, exceptions.IOError: [Errno 2] No such file or directory: 'C:\\Users\\ROBERTO\\canonical\\is_path\\data\\logging.conf' | 10:07 |
| ralsina | mandel: yes... | 10:07 |
| mandel | ralsina, you have to do ./autogen.sh & make to get that file :) | 10:07 |
| ralsina | mandel: windows | 10:07 |
| ralsina | if it has to be there for tests, then run-tests should put it there | 10:07 |
| mandel | ralsina, because using autotool for a python project is a good idea :P | 10:08 |
| mandel | ralsina, it has to be done for the tests, so far we have being doing make froma shared folder in a unix system that has the tools | 10:08 |
| mandel | ralsina, which I agree is less that desirable | 10:08 |
| ralsina | mandel: I haven't :-) | 10:08 |
| mandel | ralsina, hahaha | 10:08 |
| mandel | ralsina, the ones that touch the code did hehe | 10:09 |
| ralsina | mandel: there is a logging.cnf in windows folder | 10:09 |
| ralsina | mandel: make run-tests.bat move it if needed | 10:09 |
| mandel | ralsina, how is the progress of removing the c code form ubuntu one client source? | 10:09 |
| mandel | ralsina, sure, I can add that there | 10:09 |
| mandel | let me check if it is correct first | 10:09 |
| ralsina | mandel: stalled, I think | 10:09 |
| ralsina | mandel: it's what we are shipping so it is correct ;-) | 10:10 |
| ralsina | same for clientdefs if needed | 10:10 |
| mandel | ralsina, clientdefs are copied, of that I'm 100% sure | 10:11 |
| ralsina | cool, so same thing :-) | 10:11 |
| mandel | ralsina, give me 2 mins and I'll get that done for you | 10:11 |
| ralsina | awesome | 10:11 |
| mandel | ralsina, so about the c code, are we moving it out, 'cause doing all this autogen crap is a pain | 10:13 |
| mandel | ralsina, I got it working for mac os x, but installing gnome-common for that on a mac just feels stupid | 10:13 |
| ralsina | mandel: dobey was moving it, I think, before he became the linux team | 10:13 |
| ralsina | but yes, using autogen to generate logging.conf is overkill | 10:13 |
| ralsina | specially since we always generate the exact same one anyway | 10:14 |
| mandel | ralsina, indeed, and having all this in a shell script or the setup.py would be great | 10:15 |
| mandel | I'd love to do python setup.py test or something like that hehe | 10:15 |
| mandel | ralsina, I'm testing the changes to run_tests.bat atm | 10:16 |
| mandel | will push as soon as I see that everything is ok | 10:16 |
| ralsina | ack | 10:16 |
| mandel | ralsina, in the main time, can you take a look at https://code.launchpad.net/~mandel/ubuntuone-windows-installer/uninstall-old-app | 10:18 |
| ralsina | mandel: sure | 10:18 |
| mandel | ralsina, that code will uninstall the old msi for the beta in case someone has it, will propose the code that does the data migration in 5/10 min | 10:18 |
| ralsina | mandel: ack | 10:18 |
| mandel | after that I just have one last branch performing the 3 steps adn will do some IRL test with the old-beta and new one | 10:18 |
| ralsina | mandel: that branch cdefines the code but is not called from aywhere? | 10:20 |
| mandel | ralsina, yes, I have done it in small steps, find old app/remove msi/migrate data (next branch)/ put things together (last one) | 10:22 |
| ralsina | ok | 10:22 |
| mandel | ralsina, I wanted to make smaller merge proposals with clear tests so that the reader does not have a 1000 lines to look at :) | 10:22 |
| ralsina | and we all appreciate it ;-) | 10:22 |
| mandel | ralsina, see, I listen to my users hehe | 10:25 |
| mandel | ralsina, tests finished and runtest.bat works as expected, please pull (my finger) when ever you want | 10:27 |
| czajkowski | mpt: ping | 10:46 |
| mandel | czajkowski, I think he is here now | 11:04 |
| czajkowski | mandel: where are you ? | 11:08 |
| mandel | czajkowski, here :) | 11:12 |
| mandel | but if you mean in the world, would be Madrid :) | 11:12 |
| mandel | ralsina, other branch: https://code.launchpad.net/~mandel/ubuntuone-windows-installer/migrate-data/+merge/78387 | 11:12 |
| mandel | ralsina, that one contains just the migration of the data from the old beta to the new one | 11:13 |
| ralsina | mandel: ack. Gotta take the kid to school thouhg | 11:13 |
| mandel | well, new client not new beta :) | 11:13 |
| mandel | ralsina, no probelm | 11:13 |
| gatox | hi everyone | 11:13 |
| mandel | gatox, hola! | 11:14 |
| mandel | gatox, got internet? | 11:14 |
| gatox | mandel, 3g right now... but in a few few hours i should have a proper connection and i'm going to be REALLY HAPPY! :D | 11:15 |
| mandel | gatox, congrats!! | 11:16 |
| gatox | mandel, thanks...... this situation was driving me crazy!!! :P | 11:16 |
| mandel | gatox, I was like a month like that, I understand the pain | 11:16 |
| gatox | mandel, true | 11:17 |
| talios | Morning all - just signed up for ubuntu one music. nice - now to hack an upload tool to run on OSX )() | 11:17 |
| mandel | talios, are you looking to port the daemon that syncs the files, or just an uploader, cause if you want to hack I know the people that can help you for that :) | 11:18 |
| mandel | talios, if it is the daemon, I'm also interesting in doing some work there, but I don't want to have to do the UI hehe | 11:18 |
| czajkowski | mandel: ahhh thought you meant london | 11:19 |
| mandel | czajkowski, oh, no hehe I meant here as in the channel right now | 11:19 |
| talios | mandel: a bit of both :) I actually started working on a personal cloud music tool the other day and am thinking of exploiting Ubuntu One for it. | 11:19 |
| mandel | talios, oh my!!! that sounds great! | 11:21 |
| mandel | talios, so, the one that can help you with the rest api would be urbanape, he did the iOS client | 11:21 |
| mandel | talios, so you might well be able to take advantage of his work | 11:21 |
| mandel | talios, regarding the daemon, I can give you a hand, but only on fridays :( | 11:21 |
| talios | mandel: call me crazy ( every one does ) but I have a map reduce style config for all my iTunes playlists ( heavy smart playlist setup, along with scripts which hack the iTunes meta data ) - I'm trying to write something to ween myself of iTunes :) | 11:22 |
| mandel | talios, I dont see anything crazy about that, I see coolness, do you mind if I get some time (walk the dog) and chat about it in a couple of hours? | 11:24 |
| mandel | talios, we can get urbanape too and see if there is something nice to hack :) | 11:24 |
| talios | mandel: I'd love to but its 12:24 am and I should be heading to bed :) | 11:24 |
| talios | I'm sure I'll be loitering in here a bit as I explore the APIs | 11:25 |
| mandel | talios, superb, please do so, I'm in CET time so you can ping me earlier than this :) | 11:25 |
| mandel | talios, cath you later! | 11:25 |
| * mandel walks dog | 11:25 | |
| talios | now if the android player gave me ID3 from AAC files I'd be immediately happy too :) | 11:26 |
| nessita | good morning everyone! | 11:39 |
| gatox | in less than 2 hours i'm going to have internet!!!! please be patient for 2 hours more :P | 11:54 |
| === yofel_ is now known as yofel | ||
| ralsina | good morning gatox | 12:17 |
| gatox | ralsina, good morning.... i'm here about an hour ago...... but with internet problems...... in less than 2 hours i should have a real connection :D | 12:17 |
| ralsina | good :-) | 12:17 |
| * nessita -> quick errands | 12:20 | |
| * mandel lunch | 12:29 | |
| * nessita is back | 12:42 | |
| ralsina | hola nessita | 12:46 |
| nessita | hi ralsina | 12:46 |
| mandel | ralsina, can you see me? | 12:47 |
| ralsina | mandel: now I can | 12:47 |
| ralsina | mandel: is run-tests.bat updated in that branch? | 12:47 |
| mandel | ralsina, yes, i guess you did not get my msg | 12:48 |
| ralsina | it seems I didn; t | 12:48 |
| ralsina | ok, trying | 12:48 |
| mandel | ralsina, not a. big deal, I also sent you an other message with another branch for re view, did you see that one? | 12:49 |
| ralsina | the one about data migration? Yes, got that one | 12:49 |
| mandel | ralsina, weird cause they were more or less at the same time.. well who cares :) | 12:50 |
| ralsina | mandel, gatox, alecu, nessita, dobey: standup in 9' | 12:51 |
| ralsina | mandel: maybe I just didn't read it. I am very sleepy for some reason | 12:51 |
| mandel | ralsina, or an xchat fail.. :P | 12:51 |
| gatox | brb....... internet dude is here :P | 12:52 |
| mandel | that is what she said!! | 12:52 |
| gatox | jeje | 12:52 |
| mandel | gatox, would have been funnier with a pizza delivery guy... or a plummer :P | 12:52 |
| gatox | jejejjeje | 12:52 |
| dobey | ralsina: lies | 12:53 |
| ralsina | dobey: I am sleepy! | 12:53 |
| dobey | ralsina: no standup | 12:55 |
| dobey | ralsina: mumble instead, no? | 12:56 |
| ralsina | mumble is in 1 hou | 12:56 |
| ralsina | hour | 12:56 |
| mandel | ralsina, no standup? | 12:56 |
| dobey | oh | 12:56 |
| ralsina | so, yes standup, but do it desultorily | 12:56 |
| ralsina | with a slouching posture, maybe | 12:56 |
| ralsina | say meh instead of me | 12:56 |
| dobey | nay | 12:57 |
| mandel | sig | 12:57 |
| mandel | meh | 12:57 |
| ralsina | mandel: still no joy: http://pastebin.ubuntu.com/703365/ | 12:58 |
| nessita | gatox: you reading me in PM? | 12:59 |
| mandel | ralsina, I'll take a closer look aft the stand up, can you put that in the mp? | 12:59 |
| mandel | me | 12:59 |
| ralsina | mandel: sure | 12:59 |
| ralsina | me | 13:00 |
| nessita | gatox: ping | 13:00 |
| nessita | me | 13:00 |
| dobey | We | 13:01 |
| ralsina | nessita: gatox is with the internet guy | 13:02 |
| ralsina | So, mandel, go | 13:02 |
| mandel | DONE: Prosed merges for lp:~mandel/ubuntuone-windows-installer/migrate-data for the code that migrates the data from the old beta. Fixed run-tests for lp:~mandel/ubuntuone-client/is-path-dir-fix so that there is no need to do make on windows. That branch fixes bug 867567 TODO: autoupdate bugs. BLOCK: nope | 13:02 |
| ubot4 | Launchpad bug 867567 in ubuntuone-client "The _path_is_dir function in file system notifications on windows is broken (affects: 1) (heat: 6)" [High,New] https://launchpad.net/bugs/867567 | 13:02 |
| mandel | nessita go | 13:02 |
| ralsina | DONE: administrivia, arranging things for mgmt sprint trip, reading on u1db, some bug triage, some reviewing. TODO: close my damn bug I have been poking for 3 days, bunch'o'reviews, call with you guys BLOCKED: no NEXT: nessita | 13:02 |
| nessita | DONE: finished and landed bug #862991, bug #862540, bug triaging, lots of debugging for Lisette's bug about empty folders not being deleted in the web. Found bug #868661, started a branch for it | 13:02 |
| nessita | TODO: finish bug #868661, lots of reviews I didn't do yesterday, special meeting | 13:02 |
| nessita | BLOCKED: nopes | 13:02 |
| nessita | NEXT: all the dobeys | 13:02 |
| ubot4 | Launchpad bug 862991 in ubuntuone-control-panel "Set a better title for the error dialogs (affects: 1) (heat: 94)" [Low,Fix committed] https://launchpad.net/bugs/862991 | 13:02 |
| dobey | λ DONE: e-mail, more magic, tarmac config stuff | 13:02 |
| dobey | λ TODO: tarmac config updates, more of #865593, call, SRU work | 13:02 |
| ubot4 | Launchpad bug 862540 in ubuntuone-control-panel "Do provide a nice error message when there are issues (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/862540 | 13:02 |
| dobey | λ BLCK: None. | 13:02 |
| ubot4 | Launchpad bug 868661 in ubuntuone-client "u1sdtool is not multiplatform (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/868661 | 13:02 |
| dobey | cest fin? | 13:03 |
| nessita | I guess so | 13:03 |
| mandel | oui oui | 13:03 |
| alecu | good morning! | 13:03 |
| nessita | dobey: would you be able to review https://code.launchpad.net/~nataliabidart/ubuntuone-client/update-sso/+merge/77370 today? | 13:03 |
| nessita | hi alecu | 13:03 |
| ralsina | alecu: you! say me! | 13:03 |
| alecu | me | 13:03 |
| ralsina | alecu, go | 13:03 |
| ralsina | alecu.standup.addCallback(print("EOM")) | 13:05 |
| alecu | ralsina, twisted does not work on python 3 yet! | 13:06 |
| ralsina | alecu: from __future__ import print | 13:06 |
| mandel | ok, I'm off for lunch, wii be back in 30 min more or less | 13:06 |
| ralsina | ;-) | 13:06 |
| ralsina | buen provecho! | 13:07 |
| ralsina | ok, from __future__ import print_function, I was close | 13:07 |
| urbanape | talios: I'd love to see what you come up with. I was working on a simple upload/publish tool, but work keeps getting in the way. | 13:08 |
| alecu | DONE: took morning off, then worked a bit more on server-time sync, on the sd and some fixes on cp. | 13:08 |
| alecu | TODO: finish fixing the tests, propose for reviews | 13:08 |
| alecu | BLOCKED: find out what if we want to use a specific url for this | 13:08 |
| alecu | in fact, let's ping karni | 13:08 |
| alecu | EOM | 13:08 |
| nessita | alecu: what specific url you need? | 13:09 |
| karni | alecu: Yes? | 13:09 |
| karni | Ah | 13:10 |
| karni | alecu: Let me find it | 13:10 |
| alecu | nessita, right now I'm using http://one.ubuntu.com/ but perhaps we want to use *some* other url. | 13:11 |
| alecu | nessita, that's why I'm asking karni, that has already been thru this. | 13:12 |
| alecu | karni, also: what are the non-caching headers that you are using? | 13:12 |
| karni | alecu: I do HEAD https://files.one.ubuntu.com/?status=1 with the header Cache-Control: no-cache | 13:12 |
| karni | alecu: and parse the Date header :) | 13:12 |
| nessita | alecu: sorry I'm a little out of context, I think you discuss this while I was on vacations. You need to hit a url to achieve what exactly? | 13:13 |
| alecu | karni, right. | 13:13 |
| karni | nessita: Find the server time to avoid unauthorized response to OAuth signed calls that used invalid timestamp (invalid, as in: to far from server time) | 13:13 |
| nessita | karni: and then how do you "overwrite" the timestamp? | 13:14 |
| alecu | nessita, I'm getting the server time, so I can tell it the local time is skewed. And then I use that "skew" to adjust the oauth_timestamp parameter. | 13:14 |
| dobey | nessita: a bit of review done | 13:14 |
| karni | nessita: I extended CommonsHttpOAuthConsumer class and override the generateTimestamp method with return Long.toString((System.currentTimeMillis() + timeDriftMillis) / 1000L); | 13:14 |
| nessita | alecu: ah, tricky :-). Do we need to "fix" to many places or (ideally) just ussoc? | 13:15 |
| karni | alecu: Exactly. | 13:15 |
| alecu | karni, just "Cache-Control: no-cache" is enough? I remember that cache headers were a bit more complicated when the browsers had to go thru different transparent proxies and such. | 13:15 |
| ralsina | alecu: Cache-control should override everything IIRC | 13:15 |
| alecu | nessita, I've got five branches | 13:15 |
| karni | alecu: To my best efforts I have confirmed this solved the issue. I still have a bunch of users with the same problem, but it's most probably due to timeouts on /sso-done-so-get-tokens | 13:16 |
| alecu | nessita, storage protocol, sso, control panel, syncdaemon, windows installer. | 13:16 |
| karni | alecu: So I would say, yes. | 13:16 |
| alecu | karni, ralsina: thanks. | 13:16 |
| nessita | alecu: guh, when you have a moment, would you share why? I mean, I would like if all the oauth stuff was isolated in ussoc, so perhaps we can move the logic back to ussoc and re-use from the other locations? | 13:17 |
| karni | alecu: You're welcome. | 13:17 |
| alecu | nessita, those are all the places that are doing oauth signing. (as opposed to getting the tokens used to sign) | 13:18 |
| nessita | dobey: ack, thanks | 13:18 |
| nessita | alecu: in ussoc there is a method that returns signed headers for any http request, can you use that in the other places that sign the urls? | 13:18 |
| dobey | mandel: just for you, We will make everything use imake to build/install | 13:19 |
| ralsina | xmkmf -f was it? | 13:19 |
| karni | nessita: alecu: as to what nessita just wrote - please keep in mind that SSO time may be different than U1 time. I'm fixing the issue in two places. | 13:19 |
| karni | nessita: alecu: With different server URLs | 13:20 |
| nessita | karni: which 2 places? | 13:20 |
| alecu | nessita, I'm taking a look at using the same bit from sso. | 13:20 |
| karni | nessita: When I talk to SSO, I get the Date header from https://login.ubuntu.com . When I talk to U1, I get the Date hear from https://files.one.ubuntu.com/?status=1 | 13:20 |
| nessita | alecu: maybe protocol will have to duplicate code (it does not depend on SSO, and I'm not sure if we want it to depend on that) | 13:21 |
| nessita | karni: are you doing that because you know for a fact that those 2 sites differ in their time? | 13:21 |
| dobey | ralsina: xmkmf -ftw | 13:21 |
| karni | nessita: We had a chat with pindonga and beuno that they need not to check the timestamp (only use it to track the order of requests, as a monotonically increasing integer) | 13:21 |
| karni | nessita: I know for fact I had problems with both. Does that answer your question? | 13:23 |
| karni | nessita: It was the same "UNAUTHORISED" problem. | 13:23 |
| ralsina | if we have two servers with time more that 15' off, right now the desktop client would fail for one of those all the time | 13:24 |
| karni | nessita: And fixing the device time drit/skew did solve the issue for both. | 13:24 |
| nessita | karni: yes, that answers my question, thanks | 13:24 |
| karni | nessita: You are most welcome :) I am happy to help. | 13:24 |
| nessita | :-) | 13:24 |
| dobey | really, all We want to do today, is get results from this call. | 13:26 |
| alecu | ralsina, yes: that would be really weird: we would have that problem on many desktops too if the servers were more than 15' off | 13:26 |
| alecu | karni, ^ | 13:26 |
| karni | alecu: nessita: I also know that SSO has a cluster of servers that are time-synced (with one NTP I believe). We had huge differences in time, because it turned out we were receiving cached responses. So, the Cache-Control: no-cache header solved that for us. In case of U1, I know this is the URL we need to hit. | 13:26 |
| alecu | karni, great. | 13:27 |
| alecu | it makes sense now, that some servers are behind a caching proxy, and some are not. | 13:27 |
| alecu | and also probably the ?status=1 invalidates some proxies | 13:28 |
| karni | alecu: ralsina: I had many users with SSO issues, some claimed they do not want automatic (synced) time set on their Androids, some said they just changed to daylight saving, etc etc. That really sucks. | 13:28 |
| karni | alecu: Most probably yes :) I got that url from beun'o \o/ | 13:28 |
| ralsina | karni: yes, we have the same problem on windows, thus the need for this fix ;-) | 13:28 |
| alecu | karni, yeah, time on cellphones can be set manually very often. And on windows is the same. | 13:28 |
| alecu | ralsina, exactly. | 13:29 |
| * JamesTait pokes talios | 13:30 | |
| alecu | nessita, I'm taking a look at using oauth_headers from ubuntu_sso/utils/__init__.py. It's something that can be done, but I won't be doing it in my current branches. The main issue is that different parts of the code are using a different web client. | 13:31 |
| nessita | alecu: ok, I see. Thanks for checking though! | 13:31 |
| ralsina | alecu: that's the kind of cleanup we should set time apart to do | 13:32 |
| ralsina | alecu: as in "stop using different web clients" :-) | 13:32 |
| karni | ralsina: beuno: Seeing how much resources we've already put into this and will now be fixing for Windows, I'm thinking if they shouldn't drop the timestamp verification (as the specs say they CAN) for request invalidation purposes, and use it only for tracking the order of requests. | 13:33 |
| alecu | ralsina, yup: and we were discussing with mandel a way to replace them all with just one, so proxy support would be easier. I mean feasible. | 13:34 |
| nessita | alecu, ralsina: when implementing proxy support, unifying that will be a must | 13:34 |
| alecu | nessita, exactly. | 13:34 |
| karni | ralsina: beuno: Oh.. I just said "they" thinking of SSO, but we have the very same issue with OAuth on U1. | 13:34 |
| dobey | 'rewrite everything in vala so we can use it from EVERY language without having to write the same code N times' | 13:34 |
| nessita | mandel, gatox: I made all the pending review from you, let me know if I forgot something | 13:34 |
| ralsina | karni: they also use it to discard nonces, I think. jdo gave an explanation of why it couldn't be discarded on that email thread. | 13:34 |
| alecu | dobey, but not on windows, right? | 13:34 |
| dobey | alecu: also on windows. | 13:35 |
| nessita | dobey: fixing update-sso now | 13:35 |
| karni | ralsina: Interesting. Fine, can use to discard nonces, but drop the time lag check. | 13:35 |
| ralsina | dobey: we are not using it from multiple languages. We are just using multiple python libraries to do the same thing on different places | 13:35 |
| alecu | dobey, we'll probably just use qtnet + python. It's a much smaller change. | 13:35 |
| ralsina | karni: yes, yahoo and google don't check it so it can be done, but he... | 13:35 |
| dobey | ralsina: i'm pretty sure we are doing authentication from C/C# apps | 13:35 |
| nessita | ralsina: we should also set time aside so we all migrate to O (I know I didn't do it yet :-/) | 13:36 |
| ralsina | nessita: indeed | 13:36 |
| ralsina | nessita: these are some of the things we need to talk about in ... 24 minutes :-) | 13:36 |
| dobey | crap. | 13:36 |
| alecu | oh, meeting. | 13:36 |
| * ralsina is terrified of O. Last attempt went bad really quick. | 13:36 | |
| * alecu grabs a bite | 13:36 | |
| karni | nessita: migrate to O ? | 13:36 |
| alecu | is that like a 15 times better C? | 13:37 |
| ralsina | karni: all of us are supposed to be on Oneiric since beta at least | 13:37 |
| nessita | karni: our desktops, upgrade to oneiric | 13:37 |
| * alecu will wait for C++ | 13:37 | |
| * alecu will wait for O++ | 13:37 | |
| karni | ralsina: nessita: oh | 13:37 |
| ralsina | alecu: you are thinking of C++++++++++++++ | 13:37 |
| * dobey throws a O(n) at alecu | 13:37 | |
| * JamesTait tried Oneiric from a Live SD Card on an Asus EeePC 701 last night - very nice, I have to say. | 13:38 | |
| ralsina | JamesTait: yes it's pretty! Then I upgraded and it stopped booting. | 13:38 |
| alecu | ouch | 13:38 |
| JamesTait | ralsina: :( | 13:38 |
| dobey | ralsina: does it sit "Waiting for network blah blah" ? | 13:39 |
| ralsina | dobey: no, it kernel-panicked on boot | 13:39 |
| ralsina | dobey: I will try again pinning drivers+kernel first, I suppose | 13:40 |
| dobey | oh | 13:40 |
| dobey | ralsina: hold shift immediately after the BIOS goes away, and boot an older kernel | 13:41 |
| karni | ralsina: And I can only imagine how much more work this will require for you guys to fix that OAuth issue. Can't we really fix it on the server side.. ? Then you would only have to fix the sso part | 13:42 |
| ralsina | dobey: killed the sucker already, will try again on saturday | 13:42 |
| ralsina | karni: that's why I did that email thread. Since server side told us no... | 13:43 |
| karni | ralsina: I'll re-read it in a spare moment. | 13:43 |
| nessita | dobey: question, you mentioned that g_cclosure_marshal_VOID__OBJECT should be VOID__VOID for the CredentialsNotFound signal. I'm not sure if you're suggesting that I should change the marshaller declaration, or if I should add a new marshaller. I would guess you meant the second one (since we need the VOID__OBJECT for the credentials found and credentials error signal), and I did not add it since I read there are a couple of "builting" marshallers alrea | 13:43 |
| nessita | dobey: so, since previously we had no marshaller for STRING_VOID (which was the signature of the auth_denied signal), I would assume there is a builting marshaller for VOID__VOID | 13:44 |
| dobey | nessita: We mean that signal should use a _VOID__VOID marshaller as it is not expecting any return values, nor is it passing any arguments to the callback | 13:44 |
| nessita | dobey: right, but since I removed the signal, I don't have to change that, no? | 13:45 |
| dobey | nessita: right, if you removed the new signal addition, then that line should no longer be in the diff | 13:46 |
| nessita | dobey: ok, I now understand what you meant, thanks | 13:46 |
| gatox | YES! HABEMUS INTERNET! | 13:49 |
| alecu | gatox, WELCOMUS! | 13:49 |
| gatox | alecu, thankus! | 13:49 |
| gatox | jeje | 13:49 |
| nessita | gatox: WHAT A JOY! | 13:49 |
| gatox | nessita, you have no idea :') | 13:50 |
| CardinalFang | I DON'T KNOW WHY WE'RE YELLING! | 13:50 |
| dobey | diez minutos | 13:50 |
| alecu | nessita, ugh, manually marshalling. my condolences. | 13:50 |
| gatox | CardinalFang, jeje | 13:50 |
| nessita | alecu: we needed a firewoman in the team (?) | 13:50 |
| * alecu thinks of firewomans and of poles and of series of really bad jokes | 13:51 | |
| nessita | dobey: fixes pushed to revno 1149 | 13:54 |
| dobey | nessita: cheers | 13:55 |
| nessita | oh meeting! | 13:59 |
| dobey | ugh, 33ms ping | 14:00 |
| ralsina | alecu, nessita, dobey, gatox, thisfred mumble? | 14:00 |
| dobey | and slow upload | 14:00 |
| ralsina | mandel too | 14:00 |
| gatox | ralsina, ack | 14:00 |
| dobey | ralsina: waiting for you | 14:01 |
| thisfred | on my way | 14:01 |
| thisfred | shit. laptop's upstairs, be there shortly | 14:01 |
| mandel | ralsina, what? | 14:03 |
| nessita | mandel: mumble! | 14:04 |
| ralsina | mandel: team mumble | 14:04 |
| mandel | yes, launching it | 14:04 |
| ralsina | mandel: there was mail, and invitation, and it's in your calendar :-) | 14:04 |
| mandel | ralsina, I know, bad phone call at the same time | 14:04 |
| === m_conley_away is now known as m_conley | ||
| === zyga_ is now known as zyga-afk | ||
| mandel | nessita, ping | 15:07 |
| nessita | mandel: pong | 15:07 |
| mandel | regarding https://code.launchpad.net/~mandel/ubuntuone-windows-installer/migrate-data/+merge/78387 I had a lint error when importing from platform, shall I just add an ignore comment? | 15:07 |
| mandel | nessita, ^ | 15:08 |
| nessita | mandel: what lint error? | 15:08 |
| mandel | nessita, let me un do the change and get that for you one sec | 15:09 |
| nessita | mandel: sure | 15:10 |
| dobey | lunch time, bbiab | 15:11 |
| mandel | nessita, the following is when executing runtest in O for that branch: http://paste.ubuntu.com/703438/ | 15:13 |
| gatox | lunch...... brb | 15:13 |
| mandel | nessita, once I have fixed the import, that is | 15:13 |
| nessita | mandel: if you open a python terminal, can you do the import? | 15:16 |
| mandel | nessita, yes, is just lint being an ass on O | 15:18 |
| mandel | nessita, I worry that we will start getting false possitives.. | 15:18 |
| mandel | nessita, http://paste.ubuntu.com/703442/ | 15:19 |
| mandel | including typo hehe | 15:19 |
| nessita | mandel: hum, can you try with the ignore? | 15:20 |
| nessita | mandel: bad news is that I think the pyflakes do not listen to the ignores :-/ | 15:20 |
| mandel | nessita, this is the installer, so we are ok with that, right? | 15:20 |
| mandel | nessita, I'll add the ignores | 15:20 |
| nessita | mandel: eh! wait | 15:20 |
| nessita | mandel: this is installer? | 15:20 |
| nessita | mandel: we certainly should not be using os_help platform stuff outside syncdaemon | 15:21 |
| nessita | mandel: SyncDaemonTool, yes. os_helper, no. | 15:21 |
| nessita | mandel: since this is windows code, you should always use unicode paths, and use the builting functions (eg shutil.move) | 15:22 |
| mandel | nessita, and add the long path prefix too | 15:22 |
| nessita | mandel: well, it depends, the old beta handled long paths? (I would guess no)( | 15:22 |
| mandel | nessita, yes, it did | 15:22 |
| nessita | mandel: but... question | 15:23 |
| mandel | nessita, also, tools comes from platform, why can't we import the rest? | 15:23 |
| nessita | mandel: we won't leaking os_helper outside syncdaemon, otherwise it gets difficult for us to fix/move stuff there | 15:23 |
| nessita | mandel: and outside syncdaemon, the app knows in which paltform it is, so multiplatform FS handling is not needed | 15:24 |
| nessita | mandel: question about log path prefixes: | 15:24 |
| nessita | mandel: hum, no, nothing, just use the builting function passing unicode to them and we'll be fine | 15:25 |
| mandel | nessita, the default funtions you mean, sure I can change that, the installer does not touch that many times the file system | 15:26 |
| nessita | mandel: default functions? | 15:27 |
| mandel | nessita, I'm lost here: 'builting function' I supose you mean the stdlib ones, right? | 15:28 |
| nessita | mandel: yes, sorry for not being clear | 15:29 |
| mandel | nessita, no worries, is irc, no one is ever clear :) | 15:30 |
| mandel | nessita, on question, in ubuntuone-windows-installer, why do we have linux code? | 15:35 |
| mandel | or at least a linux.py? | 15:35 |
| nessita | mandel: because the wizard must work on both platform (is not working yet, but it will) | 15:35 |
| nessita | mandel: the wizard code will be moved to controlpanel, eventually | 15:35 |
| mandel | nessita, an the project will be renamed to unbutone-installer, right? 'cause is confusing :) | 15:36 |
| nessita | mandel: nopes, the project will have only the windows specific bits, like the building script and the migration code | 15:36 |
| mandel | nessita, ah.. ok | 15:37 |
| === beuno is now known as beuno-lunch | ||
| mandel | nessita, https://code.launchpad.net/~mandel/ubuntuone-windows-installer/uninstall-old-app/+merge/77225 is fixed as per review | 15:51 |
| nessita | mandel: ack! | 15:51 |
| mandel | nessita, removed the win32 calls used the get_property func and changed test to patch that function rather than the win32 ones | 15:52 |
| * mandel moves to the other review | 15:52 | |
| mandel | nessita, I'm too close to my EOD to finish with the other reviews, will be done my tom your morning with all the fixes done | 15:55 |
| nessita | mandel: ack | 15:55 |
| mandel | ok, everyone have a great rest of the day! | 15:55 |
| * mandel EOD | 15:55 | |
| === zyga-afk is now known as zyga | ||
| === beuno-lunch is now known as beuno | ||
| dobey | hrmm | 16:34 |
| dobey | nessita: +1 btw on update-sso | 17:01 |
| nessita | dobey: great! question, did you happen to run the musicstore tests using that branch? | 17:04 |
| dobey | nessita: no | 17:07 |
| nessita | dobey: if you have some spare time, could you please do it? I'm using my laptop due a RIP monitor, so I don't have all the deps installed for libubuntuone :-/ | 17:08 |
| dobey | nessita: seems to work | 17:19 |
| nessita | dobey: thanks! (and great news) | 17:19 |
| nessita | I'll mark it as approved then | 17:19 |
| nessita | I know is silly, but I'm happy that we no longer depend on the deprecated service | 17:20 |
| nessita | lucnhtime!!! | 17:33 |
| gatox | nessita, when you have a moment please tell me if this is the correct implementation that you mentioned: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/network-detect/+merge/77048 | 17:34 |
| gatox | lisette, ping | 17:45 |
| lisette | gatox: pong ( i am here 2 more minutes before freediving :) ) | 17:46 |
| gatox | lisette, okok, just a quick question | 17:47 |
| lisette | gatox: shoot | 17:47 |
| gatox | lisette, i was looking at this bug: https://bugs.launchpad.net/ubuntuone-windows-installer/+bug/855067 and i thought that ralsina was talking to you about this, and i wanted to know if you decided something | 17:47 |
| ubot4 | Launchpad bug 855067 in ubuntuone-windows-installer "U1 icons looks ugly in: taskbar, start menu and desktop (affects: 1) (heat: 6)" [Medium,In progress] | 17:47 |
| gatox | lisette, or which is the status of this bug, to know if i should keep looking at this one, or you take another path | 17:48 |
| gatox | lisette, if you are busy i can wait until tomorrow and keep working in something else in the meanwhile | 17:50 |
| * lisette looking | 17:50 | |
| lisette | gatox: ralsina told me the problem only started when he packaged things up, before it looks fine | 17:51 |
| lisette | gatox: so have a chat with him, and email me if you want anything; can look at this tomorrow EARLY if you want | 17:51 |
| gatox | lisette, ahhhhh ok, i'll talk with him about that to see what can we do then | 17:51 |
| lisette | gatox: good luck and let´s chat tomorrow! i am starting to hold my breath now...... | 17:52 |
| gatox | lisette, jeje byeeeeee | 17:52 |
| dobey | where did ralsina go? | 17:56 |
| gatox | dobey, don't know..... | 17:58 |
| nessita | dobey: he mentioned a lunch appointment in the meeting, I think | 18:08 |
| dobey | nessita: he said he had to go to lunch. but that was 2 hours ago :) | 18:08 |
| nessita | dobey: yeah... long lunch :-) | 18:09 |
| nessita | dobey: I can text him if you need/want | 18:09 |
| gatox | nessita, all my branches are up to date now | 18:09 |
| dobey | nah it's not terribly urgent. | 18:09 |
| gatox | nessita, now that i have internet, i'll start flooding branches for review! :P | 18:10 |
| gatox | nessita, i found a problem in the network detect installer branch....... i'm fixing that....... i'll let you know when it is done | 18:23 |
| nessita | gatox: ack | 18:24 |
| ralsina | Sorry ppl, I had a tiny car accident coming back from lunch | 19:44 |
| gatox | ralsina, are you ok? | 19:44 |
| ralsina | fine! | 19:44 |
| dobey | hola ralsina :) | 19:45 |
| ralsina | but my wife fell, so I had to take her to the hospital | 19:45 |
| ralsina | just a bruise, hurts like hell, nothing broken | 19:45 |
| dobey | suck :-/ | 19:45 |
| ralsina | yep | 19:45 |
| gatox | ralsina, :S | 19:45 |
| dobey | that's good at least | 19:45 |
| ralsina | specially on pre-release-day | 19:45 |
| === m_conley is now known as m_conley_away | ||
| alecu | gatox, nessita: if you are still around, please review: https://code.launchpad.net/~alecu/ubuntuone-storage-protocol/timestamp-autofix/+merge/78505 | 21:08 |
| gatox | alecu, on it | 21:09 |
| nessita | alecu: I am! | 21:09 |
| dobey | have a good evening all, We are off. :) | 21:10 |
| alecu | bye dobeys! | 21:10 |
| alecu | gatox, nessita: this branch is the first of five branches, that get the timestamp used for oauth from the server. This one is in storage-protocol; I'm testing the remaining branches for sso-client, syncdaemon, control-panel and windows-installer. | 21:13 |
| nessita | alecu: ack, send me emails with those, I'll complete the reviews tomorrow | 21:13 |
| alecu | they are all dependent, so ideally we would like to land them in a short sequence. | 21:13 |
| alecu | nessita, sure, thanks! | 21:13 |
| alecu | nessita, gatox: so please remember not to set these branches as *globally approved" | 21:14 |
| gatox | alecu, ok | 21:14 |
| nessita | ok, I'm done for the day | 21:19 |
| nessita | bye all! | 21:19 |
| gatox | nessita, bye | 21:21 |
| gatox | alecu, just a REALLY LITTLE detail | 21:21 |
| gatox | alecu, """A mock webserver for testing""" should be """A mock webserver for testing.""" :P | 21:21 |
| alecu | gatox, love those details :-) | 21:21 |
| gatox | alecu, jeje | 21:22 |
| alecu | gatox, they mean that the rest is fine :-) | 21:22 |
| gatox | alecu, tests are ok, and the code seems fine at least for me | 21:23 |
| alecu | gatox, fixed | 21:23 |
| gatox | alecu, +1 | 21:24 |
| alecu | gatox, great, thanks! | 21:25 |
| gatox | :D | 21:25 |
| alecu | gatox, here's the second one, if you are still feeling brave :-) | 21:25 |
| alecu | https://code.launchpad.net/~alecu/ubuntu-sso-client/timestamp-autofix/+merge/78507 | 21:25 |
| gatox | alecu, jeje ok, let me see | 21:26 |
| gatox | alecu, i really need to do that plugin for ninja that listen to xchat and download the branches automatically | 21:26 |
| gatox | alecu, i've been disconnected of everything with this connection problems... i couldn't finish the code completion for sugar also | 21:27 |
| alecu | gatox, I'll do the xchat part, you can work on the ninja part | 21:27 |
| alecu | gatox, oh, right. How was the moving? | 21:27 |
| alecu | gatox, oh, and who did you move with? | 21:27 |
| gatox | alecu, alone... everything is fine now.... the only thing missing is gas.... but i can keep having cold showers and eating tomates jejejej | 21:28 |
| gatox | alecu, let me do the xchat part!! i want to play around with dbus :P | 21:29 |
| alecu | gatox, yup, internet beats gas | 21:29 |
| gatox | alecu, always | 21:29 |
| gatox | jeje | 21:29 |
| alecu | gatox, you should get nessita's usb slippers. Or get a usb stove! | 21:29 |
| gatox | alecu, jejee true..... | 21:29 |
| gatox | alecu, they should connect the gas yesterday/today..... tomorrow i going to call to complain | 21:30 |
| gatox | i'm | 21:30 |
| alecu | and an overclocker's water cooled rig, and use that for bathing! | 21:30 |
| alecu | yup, damn services. | 21:30 |
| alecu | That reminds me... I should call cablevisión tomorrow, to get rid of the modem in my old office. | 21:31 |
| alecu | damn google. Today all of it (gmail, docs, search) has been working really awful for me. | 21:33 |
| alecu | It's probably a problem in telecentro. | 21:33 |
| gatox | alecu, weird... | 21:38 |
| alecu | nobody mentioned something similar, so it's probably my ISP | 21:38 |
| gatox | alecu, lol, the only thing that i found in this other branch is: """A mock webserver for testing""" should be: """A mock webserver for testing.""" | 21:39 |
| gatox | jejeje | 21:39 |
| gatox | everything else seems ok! | 21:40 |
| alecu | damn copypaste! | 21:40 |
| alecu | :-) | 21:40 |
| gatox | jejje | 21:41 |
| gatox | so, +1! (fix that or you will have to face nessita!) :P | 21:42 |
| * gatox returns to work on his bloody branch! :P | 21:43 | |
| SushiDude | Non of my public pictures are showing up | 22:17 |
| SushiDude | none* | 22:17 |
| greg-g | hey all, is bug 847753 atomboy issue or U1? eitherway, it is a mess for oneiric | 22:30 |
| ubot4 | Launchpad bug 847753 in tomboy (Ubuntu) "cannot save synchronisation settings after connecting (affects: 27) (dups: 2) (heat: 92)" [High,Confirmed] https://launchpad.net/bugs/847753 | 22:30 |
| greg-g | ah, it is a duplicate of bug 845321 | 22:35 |
| ubot4 | Launchpad bug 845321 in tomboy (Ubuntu) (and 1 other project) "Unable to setup Ubuntu One sync with Tomboy (affects: 17) (dups: 1) (heat: 96)" [Undecided,Confirmed] https://launchpad.net/bugs/845321 | 22:35 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!