| nessita | ok, I'm gone for the day | 00:06 |
|---|---|---|
| nessita | bye all! | 00:06 |
| hyperair | hi. i've just rsynced over my entire ~ to a new machine, and have changed the hostname of this machine. how do i re-setup ubuntuone? | 05:53 |
| hyperair | it's still connnected to ubuntuone under the old device nmae | 05:53 |
| hyperair | i actually removed the old device name, and can't add this device to my account now | 05:54 |
| hyperair | when i click "I already have an account!" it just briefly shows another screen and then reverts back | 05:54 |
| duanedesign | hyperair: hello' | 09:06 |
| hyperair | duanedesign: hello. | 09:06 |
| duanedesign | hyperair: you may need to remove the token manually. What version of Ubuntu? | 09:07 |
| hyperair | duanedesign: natty. | 09:07 |
| hyperair | duanedesign: where is the token stored? | 09:08 |
| hyperair | i purged .local/share/ubuntuone, .cache/ubuntuonee, and .config/ubuntuone | 09:08 |
| duanedesign | hyperair: ok, the token is in 'Passwords and Encyption Keys' | 09:08 |
| hyperair | ah i see | 09:08 |
| hyperair | gnome keyring eh | 09:09 |
| hyperair | should have known | 09:09 |
| duanedesign | under the Passwords Tab | 09:09 |
| hyperair | yeah i've found it, thanks | 09:10 |
| duanedesign | cool | 09:10 |
| duanedesign | hyperair: if you r-click and delete it, you should then be prompted to readd the computer | 09:10 |
| hyperair | right click and delete what? | 09:11 |
| duanedesign | hyperair: the 'Ubuntu One' token | 09:12 |
| hyperair | right, i've done all that already | 09:12 |
| hyperair | and added the machine | 09:12 |
| hyperair | thanks | 09:12 |
| duanedesign | hyperair: cool | 09:12 |
| hyperair | case closed =) | 09:12 |
| duanedesign | hyperair: let us know if you get the result you are looking for | 09:13 |
| duanedesign | hyperair: great | 09:13 |
| hyperair | i think it would have been better to still be able to re-add the machine without having issues with the oauth token | 09:13 |
| duanedesign | hyperair: yes. i thought that bug was fixed... | 09:14 |
| hyperair | i use dropbox on this machine as well, and the procedure of removing and re-adding the machine was really smooth. | 09:14 |
| === rodrigo__ is now known as rodrigo_ | ||
| ralsina | morning! | 10:59 |
| clarita | ralsina: muy buenos dias! | 11:11 |
| clarita | I do hate a greeting left unresponded to | 11:11 |
| ralsina | buen dia claire | 11:11 |
| clarita | how were your hols? | 11:11 |
| clarita | or should I say 'were your hols hols? | 11:12 |
| ralsina | well, I spent half of them in brazil against my will, so that was strange. But I am now much more rested and energetic :-) | 11:13 |
| clarita | muito bem! | 11:13 |
| ralsina | And I am programming now, so it's a nice change of pace | 11:14 |
| mandel | ralsina: did the qt reactor fix your problem? | 11:17 |
| ralsina | mandel: yay! | 11:18 |
| * mandel is glad | 11:18 | |
| ralsina | mandel: in fact, the branch (with 4 silly tests) passes now | 11:18 |
| mandel | so we are getting there little by little | 11:19 |
| ralsina | mandel: yep. | 11:19 |
| ralsina | mandel: the "modern" thing is running u1lint instead f pylint, right? | 11:19 |
| mandel | ralsina: yes, that is the deal, is better | 11:20 |
| ralsina | mandel: ok, will change it | 11:21 |
| ralsina | hmmmm --ignore doesn't work with u1int. I need a pylintrc? | 11:22 |
| * mandel away for 15 min | 11:31 | |
| ralsina | off to take the kid to school, will be back in about 1 hour | 11:49 |
| * mandel back | 12:13 | |
| duanedesign | 'lo all | 12:22 |
| nessita | hello everyone! | 12:26 |
| nessita | mandel: ping | 12:33 |
| mandel | nessita: pong | 12:33 |
| nessita | mandel: hi there! question re http://pastebin.ubuntu.com/630651/ | 12:33 |
| nessita | mandel: why is the ussoc that alecu is trying using pb? I thought we replaced that with txn | 12:34 |
| mandel | nessita: one thing is the transport you use, the other is the protocol, protocol is still pb. I understood we were not going to add the shellExtensions for the internal beta we are doing this friday | 12:35 |
| nessita | we are not | 12:35 |
| mandel | nessita: txnamedpipes is just a protocol over namedpipes, then we can change that to txjsonrpc | 12:36 |
| mandel | nessita: exactly, so let get things working with a well known tested protocol, which is pb and later we can merge the changes in the protocol used for the IPC | 12:36 |
| nessita | ok | 12:36 |
| mandel | nessita: I have one question for you, I have problems implementing the start method from SDTool | 12:37 |
| mandel | nessita: shuld I just propose the code as it is so it does not block and implement it in a diff branch? | 12:37 |
| mandel | s/shuld/should | 12:37 |
| nessita | mandel: ah, I guess that answers the question I just asked by replying to your email :-) | 12:39 |
| mandel | nessita: oh, then I read the email :) | 12:39 |
| mandel | nessita: so, I propose the merge and we move on from there, right? | 12:39 |
| nessita | mandel: yeah, please read the email and answer it (so we all are in sync). But regarding to your question, yes, please propose | 12:40 |
| nessita | mandel: besides that branch, do you have any other pending branch? | 12:40 |
| nessita | in the sense of coded branch waiting for something else | 12:40 |
| mandel | nessita: no, we have all the sd methods ready besides start and is just because we have to agree in the best wway to do so, but I'm guessing looking in the reg and using subprocess is the best option | 12:41 |
| mandel | nessita: so I can work on the start method and the examples and give alecu a hand with the sso problem, but it should be easy to fix | 12:42 |
| nessita | mandel: I would advice you to stop using the word "easy" ;-) | 12:42 |
| nessita | past experience has proven nothing is easy so far | 12:42 |
| mandel | nessita: I think that my idea of hard is harder than the average :P | 12:43 |
| nessita | hehe | 12:43 |
| nessita | mandel: ok, please propose the branch ASAP without the start so I can take a look | 12:43 |
| mandel | nessita: I'll do it right now then | 12:44 |
| mandel | gime me a min | 12:44 |
| nessita | thanks | 12:44 |
| ralsina | hello nessita! | 12:45 |
| nessita | hola ralsina. Could you build your tests after mandel's suggestion? | 12:45 |
| ralsina | nessita: yes, they even pass! | 12:45 |
| nessita | lol | 12:46 |
| nessita | ralsina: is the branch ready for re-review? | 12:46 |
| ralsina | nessita: I would like to add another test, and then yes | 12:46 |
| nessita | ralsina: great, let me know | 12:46 |
| nessita | and hopefully, your next branch was TDD'd :-) | 12:46 |
| ralsina | nessita: welll... let's say the next one after the next one ;-) | 12:47 |
| nessita | ... | 12:47 |
| ralsina | nessita: the next one is about changing to u1lint, surely that is not TDD ;-) | 12:49 |
| nessita | ok, deal | 12:49 |
| mandel | nessita: running test on linux and then I propose | 12:52 |
| nessita | mandel: great | 12:52 |
| thisfred | ¡hola a todos! | 12:53 |
| === zyga-food is now known as zyga | ||
| thisfred | Just had a very exciting morning: the greyhound we're dogsitting got away from me in the park. Luckily he stopped running after 500m or so, or I would have some explaining to do... | 12:54 |
| nessita | thisfred: lol | 12:55 |
| nessita | hola | 12:55 |
| thisfred | Poop-for-brains, we affectionately call him | 12:55 |
| thisfred | Makes our own dog seem smart and normal | 12:55 |
| nessita | jajaja | 12:56 |
| thisfred | For those who like electronica: now playing the new fabric CD on u1rocks ;) | 12:56 |
| nessita | mandel: have an ETA? I need to buy some groceries to have breakfast and I wanted to see if I shall wait or leave now :-) | 12:59 |
| mandel | nessita: tests are running, but If you go now I'll take advantage and take the dog out, shall we both go then? | 13:00 |
| nessita | mandel: let's. I'll be back in 15 minutes~ | 13:00 |
| mandel | nessita: I\ll take a little longer but not much more | 13:00 |
| * mandel walks dog | 13:00 | |
| ralsina | nessita: rady for re-review of https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800383/+merge/65416 | 13:02 |
| nessita | ralsina: looking | 13:28 |
| nessita | ralsina: why did you remove ./setup.py build from run-tests? | 13:32 |
| ralsina | nessita: no reason, probably missed it | 13:32 |
| nessita | ralsina: could you please restore? | 13:33 |
| alecu | hello all! | 13:33 |
| ralsina | nessita: sure | 13:33 |
| ralsina | hola alecu! | 13:33 |
| nessita | thanks | 13:33 |
| nessita | hola alecu, how's amelia doing? | 13:33 |
| alecu | nessita, still sleeping. She was fine last night, but during the afternoon she kept crying whenever she coughed. She said "her mouth hurt", but she pointed *inside*, so we went to the doctor to check her throat. | 13:35 |
| alecu | nessita, btw: by removing on_credentials_error_cb, I'll be removing exception NoAccessToken too. | 13:35 |
| alecu | nessita, I think it would be fine, because we are treating all errors returned in the deferred as a NoAccess condition. | 13:36 |
| nessita | alecu: why the exception? I know knows handles it, but we need to be consistent with the linux implementation | 13:36 |
| alecu | (the find_credentials and register deferred) | 13:36 |
| nessita | s/knows/no one | 13:36 |
| alecu | nessita, right. | 13:36 |
| alecu | nessita, then we'll need to turn every exception gotten into a NoAccessTokenException. | 13:37 |
| alecu | nessita, does that sound right? | 13:37 |
| nessita | alecu: hum, I'm a little lost. The linux side alreedy raises a NoAccessToken exception | 13:37 |
| nessita | you say we change it everywhere? | 13:38 |
| alecu | nessita, no. only on windows/ipc.py. | 13:38 |
| nessita | alecu: that's my point: we need to raise the (exact) same exception in both implementations | 13:38 |
| nessita | to provide a consistent API to upper layers | 13:38 |
| alecu | nessita, if any error is found while calling find_credentials and register in ipc.py, we'll wrap it in a NoAccessToken. | 13:38 |
| alecu | nessita, great. But upper layers are not even caring what type of exception it is. | 13:39 |
| alecu | nessita, they just see any exception, and they treat it as an error. | 13:39 |
| nessita | alecu: yes to the first, and yes to the second, but let's not change linux side on this branch | 13:39 |
| alecu | nessita, ok. | 13:40 |
| nessita | so, let's use the same exception in both sides without breaking API for now, let's file a bug for whatever we want to improve | 13:40 |
| nessita | actually, there is one bug... let me find it | 13:40 |
| ralsina | nessita: ok, found the reason setup.py is not on run-tests: I copied it from control panel | 13:40 |
| nessita | ralsina: the control panel does setup.py build | 13:41 |
| alecu | nessita, actually I'll need to change a little bit the linux side: I need to move that exception to a non-platform-dependent place. | 13:41 |
| nessita | alecu: as long as you imported on the l;inux side and remains "importable" from the same path, +1 | 13:41 |
| alecu | ok | 13:41 |
| nessita | alecu: bug #619885 | 13:42 |
| ralsina | nessita: you are right, added and pushed | 13:42 |
| ubot4 | Launchpad bug 619885 in ubuntuone-client "Syncdaemon doesn't handle NoAccessToken exception (affects: 1) (heat: 5)" [Medium,Confirmed] https://launchpad.net/bugs/619885 | 13:42 |
| nessita | alecu: maybe you wanna add something to that bug :-) | 13:42 |
| nessita | ralsina: we kinda use some ad-hoc styling things like: | 13:43 |
| nessita | from ubuntuone_installer.gui.qt.ui import ( | 13:43 |
| nessita | license_ui, | 13:43 |
| nessita | congratulations_ui) | 13:43 |
| nessita | should be: | 13:43 |
| nessita | from ubuntuone_installer.gui.qt.ui import ( | 13:43 |
| nessita | license_ui, | 13:43 |
| nessita | congratulations_ui, | 13:43 |
| nessita | ) | 13:43 |
| ralsina | nessita: ok by me, will change it and re-push | 13:44 |
| nessita | ralsina: thanks. Question: any reason to change self.ui for self.page_ui in LicensePage? | 13:44 |
| ralsina | nessita: to make pylint shutup about it without addding disables. Then I added a pylintrc. I should change it back | 13:44 |
| nessita | ralsina: please, so we remain more consistent between our codes | 13:45 |
| ralsina | nessita: ok | 13:45 |
| nessita | ralsina: next question: are this signed-off texts? "Agree && Install" my concern is the double & | 13:45 |
| nessita | I don't think && is a "word" in english | 13:45 |
| nessita | or QT does some magic with it? | 13:46 |
| * nessita will expect so since she just saw "&Print" | 13:46 | |
| ralsina | nessita: & is used to mark accelerators, && is one & | 13:46 |
| nessita | ralsina: got it, thanks. In that case, shouldn't we add accelerators to "Agree && Install" and "Disagree && Cancel"? | 13:47 |
| ralsina | nessita: yes, but I want to have all the pages up and do a bug "add accelerators" to make sure I am consistent | 13:47 |
| ralsina | nessita: in fact the &print one kinda slipped away ;-) | 13:47 |
| nessita | ralsina: ah, ok | 13:47 |
| ralsina | nessita: pushed with .ui | 13:47 |
| nessita | ralsina: when we started the panel, we agreed we will be pep8 consistent in our code, unless QT forced us otherwise. So, we should be using snake_case for our method names (except when defining callbacks where we may have on_my_widget_valueChanged) | 13:49 |
| ralsina | nessita: ok, will check that | 13:49 |
| nessita | thanks! | 13:50 |
| mandel | nessita: I'm back, pushing the branch right now and will propose the merge | 13:50 |
| nessita | mandel: YEY | 13:50 |
| nessita | standup in 10 crowd | 13:50 |
| ralsina | nessita: thanks for the reviews, you are really helping me! | 13:50 |
| nessita | ralsina: :-) | 13:51 |
| === kenvandine_ is now known as kenvandine | ||
| ralsina | nessita: there, my only non-inherited method is now PEP8 ;-) | 13:52 |
| nessita | lol | 13:52 |
| nessita | ralsina: you missed and ending comma after congratulations_ui, otherwise you will get lint issues. So, final version should be: | 13:53 |
| nessita | congratulations_ui, | 13:53 |
| nessita | ) | 13:53 |
| ralsina | grmbl... ok | 13:53 |
| nessita | ralsina: are initializePage ad | 13:54 |
| nessita | oops | 13:54 |
| nessita | and printDocument inherited methods? | 13:54 |
| ralsina | printDocument is now print_document | 13:54 |
| ralsina | initializePage is inherited | 13:54 |
| nessita | ah, ok :-) | 13:56 |
| * nessita needs more QT foo | 13:56 | |
| ralsina | nessita: noone remembers all the methods, don't worry :-) | 13:56 |
| mterry | Chipaca, hello! Any word on a DD reviewer? | 13:57 |
| Chipaca | mterry: give me 10 minutes plz | 13:57 |
| mterry | Chipaca, sure | 13:58 |
| nessita | ralsina: can you use the sme pylintrc as controlpanel? so, for example, we don't get the report after the test run | 13:59 |
| nessita | and that would be all :-)D | 13:59 |
| ralsina | nessita: sure thing! | 13:59 |
| alecu | me | 14:00 |
| alecu | nessita, mandel, ralsina, dobey, thisfred, fagan, standup! | 14:00 |
| thisfred | me | 14:00 |
| nessita | me | 14:00 |
| * ralsina finishes notes | 14:00 | |
| ralsina | me | 14:00 |
| nessita | thisfred: you go first (my client reports we both said me at the same time) | 14:01 |
| thisfred | DONE: couchdb rererepackaging | reviews TODO: whatever I can to help | unity bugs BLOCKED: no NEXT: nessita | 14:01 |
| nessita | thisfred: but after alecu! :-D | 14:01 |
| thisfred | oops | 14:01 |
| dobey | me | 14:01 |
| nessita | fagan: , mandel! | 14:01 |
| mandel | me | 14:01 |
| nessita | ralsina: is fagan still with us? | 14:01 |
| nessita | alecu: go! | 14:02 |
| alecu | DONE: a branch to fix bug #799958, some reviews | 14:02 |
| alecu | TODO: a few fixes on the above branch, work on bug #800669 | 14:02 |
| alecu | BLOCKED: no kinder today | 14:02 |
| alecu | NEXT: thisfred | 14:02 |
| thisfred | nessita: get a better client, xchat works ;) | 14:02 |
| ubot4 | Launchpad bug 799958 in ubuntuone-client "windows ipc is not using the credentials management tool (affects: 2) (heat: 369)" [Critical,In progress] https://launchpad.net/bugs/799958 | 14:02 |
| ralsina | nessita: he's doing some work for josh | 14:02 |
| ubot4 | Launchpad bug 800669 in ubuntu-sso-client "InsecureJelly exception thrown when SD requests credentials (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/800669 | 14:02 |
| thisfred | DONE: couchdb rererepackaging | reviews TODO: whatever I can to help | unity bugs BLOCKED: no NEXT: nessita | 14:02 |
| nessita | DONE: bug #798413, bug #800362, tons of reviews | 14:02 |
| nessita | TODO: bug #800161, fill missing mobile info in Services tab | 14:02 |
| nessita | BLOCKED: nopes | 14:02 |
| nessita | NEXT: ralsina | 14:02 |
| ubot4 | Launchpad bug 798413 in ubuntuone-control-panel "Provide a method in backend to list only devices info without querying local settings from syncdaemon (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/798413 | 14:02 |
| fagan | crap slept in | 14:02 |
| ubot4 | Launchpad bug 800362 in ubuntuone-control-panel "Refactor devices tab to stop using faked data (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/800362 | 14:02 |
| ubot4 | Launchpad bug 800161 in ubuntuone-control-panel "Do not allow folder creation in a folder containing an UDF (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/800161 | 14:02 |
| ralsina | DONE: | 14:02 |
| ralsina | * Windows catchup call | 14:02 |
| ralsina | * MGMT call | 14:02 |
| ralsina | * Started work on meta-bug #800204 | 14:02 |
| ralsina | * fixed bug #800218 | 14:02 |
| ralsina | * fixed bug #800282 | 14:02 |
| ubot4 | Launchpad bug 800204 in ubuntuone-windows-installer "The installer has no UI files (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/800204 | 14:02 |
| ralsina | * fixed bug #800303 | 14:02 |
| ubot4 | Launchpad bug 800218 in ubuntuone-windows-installer "There is no script to run the installer (affects: 1) (heat: 6)" [Undecided,Fix committed] https://launchpad.net/bugs/800218 | 14:02 |
| ubot4 | Launchpad bug 800282 in ubuntuone-windows-installer "Closing the window should stop the reactor (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/800282 | 14:02 |
| ralsina | * fixed bug #800346 | 14:02 |
| ubot4 | Launchpad bug 800303 in ubuntuone-windows-installer "Add a page to the installer showing the license (affects: 1) (heat: 6)" [Undecided,Fix committed] https://launchpad.net/bugs/800303 | 14:02 |
| ubot4 | Launchpad bug 800346 in ubuntuone-windows-installer "The logger contains controlpanel references (affects: 1) (heat: 6)" [Undecided,Fix committed] https://launchpad.net/bugs/800346 | 14:02 |
| ralsina | * fixed bug #800330 | 14:02 |
| ubot4 | Launchpad bug 800330 in ubuntuone-windows-installer "There is no run-tests script (affects: 1) (heat: 6)" [Undecided,Fix committed] https://launchpad.net/bugs/800330 | 14:02 |
| ralsina | * fixed half of bug #800383 | 14:02 |
| ubot4 | Launchpad bug 800383 in ubuntuone-windows-installer "Implement "congratulations" page to the installer (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/800383 | 14:02 |
| ralsina | * initial work for bug #800381 | 14:02 |
| ubot4 | Launchpad bug 800381 in ubuntuone-windows-installer "Add "sync now or later" page to the installer (affects: 1) (heat: 6)" [Undecided,In progress] https://launchpad.net/bugs/800381 | 14:02 |
| ralsina | * bunch'o'reviews | 14:02 |
| dobey | grr | 14:02 |
| ralsina | TODO: | 14:02 |
| ralsina | * bugs #800357 #800359 #800363 #800364 #800366 #800368 #800374 #800375 #800376 #800377 #800379 #800381 | 14:02 |
| ubot4 | Launchpad bug 800357 in ubuntuone-windows-installer "Add page for "Installing Ubuntu One" (affects: 1) (heat: 6)" [Undecided,Triaged] https://launchpad.net/bugs/800357 | 14:02 |
| ralsina | BLOCKED: NO | 14:02 |
| nessita | ralsina: are you doing aaaaaaall those bugs today? :-) | 14:03 |
| ralsina | nessita: as many as I can ;-) | 14:03 |
| dobey | λ DONE: bug #800294, reviews, u1-win-installer on tarmac, chase people | 14:03 |
| dobey | λ TODO: fix more stuff, chase people | 14:03 |
| dobey | λ BLCK: None. | 14:03 |
| ubot4 | Launchpad bug 800294 in ubuntu-sso-client (Ubuntu) (and 2 other projects) "Support new fdo secrets API property names (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/800294 | 14:03 |
| ralsina | nessita: UI-wise, maybe | 14:03 |
| dobey | mandel: go :) | 14:03 |
| * fagan slept in | 14:03 | |
| mandel | DONE: Finished most of the implementation of SDTool on windows for bug 800671. Looked at how to implement the start and listen to signals methods which are hard to map in the windows case. | 14:03 |
| mandel | TODO: Write an example or examples that use SDTool on windows. | 14:03 |
| mandel | BLOCKED: no | 14:03 |
| mandel | fagan, go | 14:03 |
| ubot4 | Launchpad bug 800671 in ubuntuone-client "There is no SyncDaemonTool windows implementation (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/800671 | 14:03 |
| fagan | TODO | 14:03 |
| fagan | * Porting the FAQ to rst for josh | 14:03 |
| fagan | Blocked | 14:03 |
| fagan | * nope | 14:03 |
| nessita | dobey: I reviewed your branch yesterday, is not fully working on natty for me | 14:03 |
| fagan | Im on AR time for today :D | 14:03 |
| nessita | fagan: you slept in till 2pm your time?!?!?! | 14:04 |
| nessita | any comments, anyone? | 14:04 |
| fagan | nessita: yep | 14:04 |
| fagan | nessita: my alarm didnt go off | 14:04 |
| ralsina | comment: fagan, 2PM dude, not cool. | 14:04 |
| alecu | fagan, your inner alarm? | 14:04 |
| mandel | nessita, alecu, ralsina: https://code.launchpad.net/~mandel/ubuntuone-client/windows_sdtool/+merge/65493 for review | 14:04 |
| ralsina | mandel: will look in 5' | 14:04 |
| nessita | ok, EOM | 14:05 |
| ralsina | nessita: pushed pylintrc | 14:05 |
| dobey | ralsina: can you put all your "DONE" bugs on one line, instead of separate lines so it's easier to read? | 14:05 |
| nessita | ralsina, mandel, alecu, Chipaca: mumble in 10 minutes | 14:05 |
| ralsina | dobey: sure | 14:05 |
| fagan | alecu: hah inner alarms are for people with a regular sleep cycle | 14:05 |
| dobey | pylintrc? | 14:05 |
| mandel | nessita: sure, I'll have lunch after mumble :) | 14:05 |
| Chipaca | mterry: essentially fine; only problem was in startup where one of the times the default in 'storage' was ftp for some reason | 14:05 |
| blajk1 | muffinresearch, lisette, I am ready when you are | 14:06 |
| nessita | ralsina: also, in an ideal world, we list as ToDo what we are "pretty sure" we will be finishing today (of course that unknown issues may arise later) | 14:06 |
| ralsina | dobey: yes pylintrc, I will switch to u1lint later (can't make it ignore folders, and you weren't around to ask ;-) | 14:06 |
| lisette | blajk1: me too | 14:06 |
| nessita | ralsina: if you're pretty sure you'll work on all that, you rock :-P | 14:06 |
| ralsina | nessita: ok, will do that. And I *do* rock. | 14:06 |
| mterry | Chipaca, you mean you had set your backup location to something else and it changed to ftp on you? | 14:06 |
| * ralsina has spent his unfounded bragging allowance for the week | 14:07 | |
| Chipaca | mterry: no, i mean, on first run | 14:07 |
| mterry | Chipaca, that shouldn't happen! ;) | 14:07 |
| Chipaca | mterry: yeah, i figure it's a bug | 14:08 |
| Chipaca | mterry: plus that it's throwing a traceback every time i try to actually back up | 14:09 |
| mterry | Chipaca, ah, well, I wasn't able to reproduce that when you first mentioned it, but I tried again recently and got it | 14:09 |
| mterry | Chipaca, I uploaded a new ubuntuone-couch that fixed that SSL error you saw | 14:09 |
| Chipaca | mterry: but those are not blockers in the "u1 needs to sign off" sense | 14:09 |
| mterry | Chipaca, you should be able to backup now if you have up to date 11.10 | 14:09 |
| Chipaca | oooh | 14:10 |
| * Chipaca gets excited | 14:10 | |
| Chipaca | hmm... same traceback | 14:10 |
| mterry | Chipaca, it was a new upload of python-httplib2 enabling ssl verification by default, which Ubuntu One apparently fails? | 14:10 |
| Chipaca | re-dist-upgrading... | 14:10 |
| mterry | Chipaca, yeah, check your version of ubuntuone-couch | 14:10 |
| Chipaca | 0.2.0+r12-2~oneiric1 | 14:11 |
| nessita | ralsina: big approve! | 14:11 |
| ralsina | nessita: yay! | 14:11 |
| nessita | ralsina: now you need another one :-) | 14:11 |
| Chipaca | mterry: i suspect we need to do some merging of that and 0.2.0-0ubuntu4 | 14:11 |
| mterry | Chipaca, ah... grab distro's version for this | 14:11 |
| mterry | Chipaca, I filed a bzr merge request with the fix | 14:11 |
| mterry | Chipaca, but it isn't accepted yet, as ya'll figure out the right way to fix the ssl error | 14:12 |
| mterry | (mine was just a band-aid) | 14:12 |
| Chipaca | thisfred: ping re ubuntuone-couch ^? | 14:12 |
| * thisfred reads | 14:12 | |
| nessita | I need reviews for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/refactor-devices/+merge/65425 | 14:12 |
| mterry | Chipaca, thisfred: https://code.launchpad.net/~mterry/ubuntuone-couch/no-ssl-validation/+merge/65384 | 14:13 |
| dobey | Chipaca: mterry's "fix" is disable_ssl_validation=True (what a horrible API "disable=True" is). but i think we should do the right thing and require validation, but we need to point at the right CA cert or something | 14:13 |
| mterry | dobey, agreed | 14:13 |
| thisfred | Yeah, so I saw dobey's comment come by, and sort of spaced out: dobey: how do we fix this properly? | 14:14 |
| Chipaca | yeah, they should've gone with disable_not_validating_ssl=False | 14:14 |
| nessita | Chipaca, ralsina, mumble pliz? | 14:15 |
| ralsina | nessita: ack! | 14:15 |
| dobey | well, ideally it would "just work" because the CA certs we're chained against should be picked up by default in Ubuntu, but alas | 14:15 |
| mterry | Chipaca, so are you saying that U1 is comfortable with signing off on DD (except of course for the bugs)? | 14:17 |
| Chipaca | mterry: we're comfortable with signing off on it, period. bugs are bugs are bugs. | 14:17 |
| ralsina | dobey: could re-review https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800383/+merge/65416 ? The Popen stuff is temporary and will file a bug about it. | 14:18 |
| mterry | Chipaca, awesome. You are no longer my blocker. /me whisks away to poke someone else until they give in ;) | 14:18 |
| mterry | Chipaca, thanks! | 14:18 |
| pmatulis | nessita: my u1sdtool setup seems to be working at bootup. i'm still a bit pissed that it created a '~/Ubuntu\ One' folder however | 14:20 |
| dobey | ralsina: can you at least change it to run the -qt version instead of -gtk? pretty sure we're not supporting the latter on windows :) | 14:21 |
| ralsina | dobey: we are not installing -qt anywhere yet, so that would mean it doesn't "work" IRL | 14:21 |
| ralsina | dobey: besides, shouldn't we use alternatives to install one without -qt or -gtk ? | 14:22 |
| dobey | ralsina: and with this it doesn't "work" IRL either, because ther eis no -gtk on windows :) | 14:22 |
| dobey | windows doesn't have alternatives | 14:22 |
| ralsina | dobey: it doesn't work on windows yet ;-) | 14:22 |
| * dobey looks at the project name again "ubuntuone-*windows*-installer" | 14:22 | |
| dobey | and from last night: | 14:23 |
| dobey | 17:36 < nessita> dobey: I agree we should open the -qt panel | 14:23 |
| ralsina | dobey: I agree too. I just say we can't do it *now*. Which is why I will open a bug. | 14:24 |
| dobey | wouldn't it be better to make *this* do what it should be doing, and then file a bug against control-panel that says "install the qt control panel script" instead, that way you don't have to fix it in two places later? i mean, it has to work in 2.5 days anyway, right? :) | 14:27 |
| ralsina | dobey: weeeeeeeeellll | 14:29 |
| ralsina | dobey: pretty please? ;-) | 14:29 |
| dobey | nessita: ^^ what do you think? | 14:30 |
| nessita | dobey: on mumble, one sec | 14:31 |
| lisette | ralsina: nessita: we are ready when you are | 15:03 |
| ralsina | lisette: still on mumble, sorry | 15:03 |
| lisette | ralsina: no worries, we are too, just drag us somewhere when you want us to join | 15:04 |
| ralsina | lisette: can it be in 30 minutes? | 15:04 |
| lisette | ralsina: sure | 15:04 |
| ralsina | ok, at :30 of ehatever hour it is now there :-) | 15:05 |
| lisette | ralsina: we will jump in at 35 past sharp :) | 15:05 |
| ralsina | lisette: ;-) | 15:05 |
| dobey | hmm | 15:09 |
| nessita | pmatulis: the Ubuntu One folder will be there always, is not optional | 15:11 |
| nessita | dobey: regaring your concern, let me ask ralsina this | 15:11 |
| nessita | ralsina: why you don't want to have the -qt version there? | 15:11 |
| ralsina | nessita: because if I put that there you can't see that the button works IRL. | 15:12 |
| nessita | ralsina: you can, if you set the PATH properly | 15:12 |
| ralsina | ok, I will change it; | 15:12 |
| nessita | thanks | 15:12 |
| dobey | :) | 15:12 |
| pmatulis | nessita: too bad. thanks for your help | 15:13 |
| nessita | pmatulis: yq | 15:13 |
| nessita | yw* | 15:13 |
| dobey | well boo. guess my yard work won't get started today thanks to weather :-/ | 15:14 |
| dobey | pmatulis: what is so bad about the folder btw? | 15:14 |
| ralsina | There, pushed it with -qt. And while we are at it, please review https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800381/+merge/65502 (now with tests!) | 15:15 |
| nessita | ralsina: could you please? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/refactor-devices/+merge/65425 | 15:16 |
| ralsina | nessita: on it! | 15:16 |
| nessita | thanks | 15:16 |
| === m_conley_away is now known as m_conley | ||
| * dobey wonders if anyone is going ot package qtreactor at all | 15:18 | |
| gord | hey all, i'm guessing its known that u1 doesn't quite work on O right? | 15:20 |
| dobey | gord: yep | 15:20 |
| gord | cool cool | 15:20 |
| nessita | mandel: there are no tests for SDT on windows :-( | 15:21 |
| ralsina | nessita: +1 | 15:25 |
| nessita | :-) | 15:26 |
| dobey | ralsina: +1 now that it's -qt | 15:26 |
| ralsina | dobey: thx | 15:26 |
| nessita | Chipaca: ping | 15:27 |
| Chipaca | nessita: pong | 15:27 |
| nessita | Chipaca: why do you have ubuntuone-client without a trunk inside, and with several other projects in it? I'm confused | 15:28 |
| Chipaca | nessita: I don't know, I didn't do that | 15:28 |
| Chipaca | nessita: note there's also a Escritorio and such in there | 15:28 |
| dobey | eh? sounds like mail faux pas? | 15:29 |
| mandel | nessita: hmm ok I'll fix that | 15:30 |
| nessita | Chipaca: there is also a Canonical dir in it | 15:30 |
| Chipaca | yeah | 15:30 |
| mandel | nessita: it should pass the same ones as in linux | 15:30 |
| Chipaca | so, not sure | 15:30 |
| nessita | can I remove that? (The whole u1client dir) and start over | 15:31 |
| nessita | mandel: mmmm for most of them, yes, they should. There may be an old one using dbus stuff, maybe | 15:31 |
| Chipaca | nessita: sure | 15:32 |
| nessita | yey | 15:32 |
| alecu | nessita, when you have a minute please re-review https://code.launchpad.net/~alecu/ubuntuone-client/fix-cmt-usage/+merge/65438 | 15:32 |
| Chipaca | nessita: you can nuke all of ~/Canonical and start over fwiw | 15:32 |
| nessita | Chipaca: ack. How can I rm -rf? :-D | 15:32 |
| nessita | alecu: yesmister! | 15:32 |
| Chipaca | nessita: rmdir /a /s AFAIR | 15:33 |
| ralsina | nessita: del /s /q whatever and later rmdir /s /q whatever | 15:33 |
| Chipaca | that | 15:33 |
| Chipaca | dunno what happened to good ol' deltree | 15:33 |
| ralsina | or do it from explorer ;-) | 15:33 |
| ralsina | Chipaca: went to live with xcopy to the old commands home | 15:33 |
| Chipaca | deltree and xcopy, sitting in a tree | 15:33 |
| nessita | ralsina: explorer makes me sick, so let's pick our battles | 15:33 |
| nessita | Chipaca: any reason why we're not using the terminal from cygwin? | 15:34 |
| Chipaca | nessita: not really, other than not wanting to break stuff | 15:34 |
| nessita | ack | 15:34 |
| Chipaca | (cygwin libs have broken build envs form me in the past) | 15:34 |
| nessita | alecu: branch looks great! +1 | 15:35 |
| alecu | cool | 15:35 |
| Chipaca | "a+ programmer! would review again!" | 15:36 |
| nessita | ralsina: doing yours now, I was waiting for the prerequisite to land | 15:36 |
| nessita | mandel: you back? | 15:36 |
| alecu | nessita, the cygwin terminal runs stuff inside it's own env, that is not usually the same as the env that every other program runs in, so, not recommended to use it unless you are developing stuff that will run with cygwin. (which we are not) | 15:37 |
| mandel | nessita: yes | 15:37 |
| alecu | oh, the same that chipaca said a few lines above :P | 15:37 |
| nessita | alecu: I understand. Thanks for un-donkey-me | 15:39 |
| nessita | mandel: great! shall we have the meeting? | 15:39 |
| mandel | nessita: lets do it! | 15:39 |
| nessita | ralsina, lisette, Chipaca, alecu, blak1? | 15:39 |
| mandel | nessita: mumble again? | 15:39 |
| ralsina | nessita: ack, to the bat-mumble! | 15:39 |
| nessita | mandel: yeah, sorry | 15:39 |
| nessita | lisette: shall we? | 15:39 |
| mandel | :) | 15:39 |
| alecu | nessita, you meant blajk1, right? | 15:39 |
| lisette | nessita: cool, let us wrap up web talk | 15:39 |
| alecu | let's crash the web talk!!!! | 15:40 |
| alecu | yeah!!!! | 15:40 |
| nessita | lol | 15:40 |
| nessita | alecu: yes, thanks | 15:41 |
| nessita | Chipaca: can you make it? | 15:41 |
| lisette | clarita, do you wanna mumble? | 15:41 |
| Chipaca | nessita: I always make it | 15:41 |
| clarita | yars will be right there | 15:42 |
| Chipaca | alecu: are you familiar with the phrase "si era hombre era feo"? that's u-c-p on windows right now :) | 15:43 |
| alecu | hahahaha | 15:44 |
| alecu | not familiar with the phrase, tho. | 15:44 |
| === m_conley is now known as m_conley_away | ||
| Chipaca | alecu: ah well. deconstructing it is fun, but not funny :) | 15:45 |
| === m_conley_away is now known as m_conley | ||
| fagan | mumble is a terrible name | 15:47 |
| fagan | wanna mumble just sounds like something completely different :) | 15:47 |
| dobey | haha | 15:54 |
| dobey | Chipaca: que ella eso ;) | 15:55 |
| ralsina | mandel: any hints about installing txnamedpipes on windows? | 15:59 |
| alecu | mandel, when you can, please re-review this branch, because otto is complaining about a new revision that was not reviewed: https://code.launchpad.net/~alecu/ubuntuone-client/fix-cmt-usage/+merge/65438 | 16:00 |
| mandel | ralsina: just use the setup.py and if you hav issues alecu has an experience with that | 16:02 |
| ralsina | mandel: ok | 16:02 |
| mandel | alecu: +1 already | 16:03 |
| dobey | sigh | 16:05 |
| dobey | nessita: where the heck do control panel/sso logs go to? | 16:06 |
| nessita | dobey: .cache/sso/ | 16:10 |
| dobey | ah ok, thanks | 16:10 |
| dobey | meh. | 16:10 |
| dobey | evil secrets api :( | 16:11 |
| alecu | mandel, thanks. | 16:11 |
| nessita | dobey: yw | 16:13 |
| * alecu brbs | 16:14 | |
| nessita | ralsina: finishing your review now... | 16:14 |
| ralsina | nessita: cool | 16:14 |
| nessita | ralsina: can you please use snake_case for all the widget names in the ui files? that way we'll have snake_case for the widgets in out python code | 16:15 |
| ralsina | nessita: ok | 16:15 |
| nessita | thanks | 16:15 |
| nessita | ralsina: in test_sync_now_shows_message there is something I don't fully understand. You're calling "congrats_page.setField("syncNow", True)" and then you're asserting over that with "self.assertEqual(congrats_page.field("syncNow").toBool(), True)". I'm not sure I see the point in that assert | 16:17 |
| ralsina | nessita: well, it didn't work at first! It tests that the radio buttons are mutually exclusive in the page ;-) | 16:19 |
| nessita | ralsina: but you set syncNow and the test for syncNow. If you want to test mutually exclusive, shouldn't you be setting syncNow and tetsing syncLater? | 16:19 |
| ralsina | nessita: yes, right | 16:20 |
| nessita | you fix? :-) | 16:20 |
| ralsina | yes! | 16:21 |
| nessita | ralsina, mandel, dobey: how can I install u1client on windows? python setup.py install does not work :-) | 16:23 |
| ralsina | nessita: I was about to ask the same thing ;-) | 16:23 |
| ralsina | it fails for me because I have no intltool-update | 16:24 |
| nessita | ralsina: you're asking about u1cp | 16:24 |
| nessita | (I think( | 16:24 |
| nessita | )) | 16:24 |
| ralsina | oops, right | 16:24 |
| * ralsina is about to take a break before he does something stupid | 16:24 | |
| dobey | i have no idea | 16:25 |
| dobey | install intltool for one :) | 16:25 |
| mandel | ha.. | 16:25 |
| dobey | but uh, why doesn't setup.py work in client? | 16:25 |
| mandel | dobey: because there is none | 16:25 |
| dobey | is none what? | 16:25 |
| nessita | dobey: in ubuntuone-client, there is no setup.py | 16:26 |
| nessita | dobey: to install all the python libraries | 16:26 |
| dobey | oh right | 16:26 |
| dobey | nessita: why did you say that then | 16:26 |
| nessita | dobey: ? | 16:26 |
| dobey | does not work != there is none | 16:26 |
| nessita | I said: dobey: how can I install u1client on windows? python setup.py install does not work :-) | 16:26 |
| nessita | ah | 16:26 |
| dobey | yes, you implied there was one :) | 16:26 |
| nessita | the commnad does not work | 16:26 |
| nessita | becasue it fails :-P | 16:26 |
| nessita | right | 16:27 |
| nessita | my bad, sorry | 16:27 |
| nessita | alecu: lint issues in your cmt branch | 16:27 |
| nessita | ./ubuntuone/platform/windows/ipc.py: 29: 'Failure' imported but unused | 16:27 |
| ralsina | nessita: I filed bug #800750 for the snake_case and fixed the tests. | 16:27 |
| ubot4 | Launchpad bug 800750 in ubuntuone-windows-installer "use snake_case on widget and field names (affects: 1) (heat: 6)" [Undecided,Triaged] https://launchpad.net/bugs/800750 | 16:28 |
| nessita | ralsina: can you please fixed the syncNow, syncLater and syncSOmething in this branch, since that code is new? | 16:28 |
| ralsina | nessita: ok | 16:28 |
| nessita | dobey: so, how can we install in windows with the current tools? or shall we create a setup.py to install syncdaemon libs? | 16:29 |
| dobey | under cygwin | 16:29 |
| dobey | ask mandel how he does it | 16:29 |
| dobey | i have done no work on u1 on windows :) | 16:29 |
| nessita | dobey: right, but since you knew autotools, you may know how to use it on windows | 16:30 |
| nessita | mandel: shall we create a setup.py for being able to installing the syncdaemon libs? | 16:31 |
| dobey | yes, it works under cygwin, but our stuff isn't set up to build without nautilus/g-s-d/libsyncdaemon stuff currently | 16:31 |
| dobey | i am not sure we should make a setup.py there, but my brain hurts now | 16:32 |
| nessita | dobey: right, and we don't need them as well. From my POV< this is a strong point to separate the syncdaemon lib into another project | 16:32 |
| ralsina | nessita: pushed with snake_case | 16:32 |
| nessita | ralsina: yey! | 16:32 |
| mandel | nessita, dobey: I think we should have the setup.py since it is a bit of an overhead to be use autotools for the python code to generate the logging cong and the clientdefs modules.. | 16:33 |
| alecu | mandel, +1 | 16:34 |
| ralsina | +1 from me too. Purely selfish, but I need to install that to get ubuntune.logger | 16:37 |
| dobey | well you need to set PYTHONPATH | 16:37 |
| dobey | less rushing into things plzkthx | 16:38 |
| dobey | and my spam folder needs to stop filling up with bug reports :-/ | 16:39 |
| ralsina | dobey: silence the u1-zomg-windows tag | 16:39 |
| dobey | that is not why they are going to my spam | 16:40 |
| nessita | dobey: I don't see this a rushing, but as something that needed attention long time ago that we have delayed because we couldn't reach consensus. We have talked about splitting the syncdaemon code into a separate project a lot | 16:40 |
| nessita | and now that we're going nultiplatform this need is more evident | 16:41 |
| nessita | Chipaca: I would like to revisit the splitting u1-syncdaemon into a separated project. Not necesarily before this Friday, but soon | 16:42 |
| dobey | *you* (and maybe chicharros) have talked about splitting syncdaemon code to a seaprate project a lot | 16:42 |
| dobey | that is *not* we | 16:42 |
| nessita | dobey: and Chipaca and ralsina and mandel | 16:43 |
| nessita | dobey: and you were involved as well, in the sense that you were in those discussions as well | 16:43 |
| dobey | no i wasn't | 16:43 |
| nessita | dobey: I have logs! :-) | 16:44 |
| dobey | i was involved in one discussion once a long time ago, and i said it was the wrong soluteion to the problem you were trying to solve then | 16:44 |
| dobey | as it was the wrong solution to your problem | 16:44 |
| nessita | dobey: fair enough. Is your opinion now different? | 16:44 |
| dobey | my opinion is that windows is a different problem and we need to approach it differently, and ideally we should have had this argument 2 months ago :) | 16:46 |
| nessita | dobey: not sure what that means... | 16:49 |
| nessita | ralsina: how can I export PYTHONPATH in windows? | 16:50 |
| dobey | it means i am willing to discuss and implement an optimal solution for this new problem, but we really should have done it 2 months ago, rather than now; as we're going to end up rushing in "band-aid" solutions to cover up the problem | 16:50 |
| ralsina | nessita: set PYTHONPATH=xxx;yyy | 16:51 |
| ralsina | remember that the separator is ";" | 16:51 |
| nessita | thanks! | 16:51 |
| nessita | dobey: well, there is not much we can do about the "we should have done it". I'm interested in the first part. Would you agree to split syncdaemon out of the current source tree? | 16:52 |
| === zyga is now known as zyga-food | ||
| dobey | nessita: i won't blindly agree to that, no. i will agree that the cross-platform issue is an issue that needs more thought and discussion. but moving syncdaemon out of u1client tree means that u1client includes no actual clients, which is just totally lame. :) | 16:53 |
| ralsina | mandel, alecu: can I bother you with reviews for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800715/+merge/65524 | 16:54 |
| dobey | perhaps the solution is move everything out and not have an ubuntuone-client project any longer; but needs thought/discussion | 16:54 |
| dobey | (and i need lunch right now) | 16:54 |
| dobey | otherwise thought isn't going to be very good :) | 16:55 |
| nessita | dobey: or move all non syncdaemon out into a dedicated project is another possibilty | 16:55 |
| ralsina | lunch, I used to have those :-) | 16:55 |
| ralsina | +1 to moving every non-syncdaemon elsewhere | 16:55 |
| nessita | ralsina: how do you build clientdefs on windows? just make? | 16:55 |
| dobey | it's not that simple | 16:55 |
| nessita | no | 16:55 |
| ralsina | clientdefs? | 16:55 |
| nessita | yeah | 16:56 |
| ralsina | it just works, doesn't use much ;-) | 16:56 |
| nessita | ImportError: no module named clientdefs | 16:56 |
| nessita | ralsina: I'm just tyring to run syncdaemon | 16:56 |
| ralsina | nessita: oh, that you hacve to ask mandel | 16:56 |
| nessita | he's gone to run an errand... | 16:56 |
| ralsina | I haven't used syncdaemon on windows this week yet | 16:56 |
| nessita | ralsina: well, but that is no news, clientdefs is there since day 0~ | 16:57 |
| nessita | ralsina: did you recall building it somehow? | 16:57 |
| ralsina | nessita: nope | 16:57 |
| dobey | http://img.chan4chan.com/img/2009-12-10/1260455707052.jpg | 16:57 |
| nessita | dobey: what do you got? | 16:57 |
| dobey | i will look more at the problem and think of possible solutions, as soon as i eat some lunch | 16:59 |
| dobey | so bbiab :) | 16:59 |
| nessita | dobey: thanks | 17:03 |
| hrw | hi | 17:04 |
| hrw | is it normal that U1 ubuntu client says "to login in or create account in ubuntu one, network connection is required" (kind of that - I use Polish in system) even when network is working and one.ubuntu.com is reachable? | 17:05 |
| alecu | ralsina, reviewing | 17:08 |
| alecu | hrw, are you using Oneiric? | 17:09 |
| hrw | alecu: yes | 17:09 |
| alecu | hrw, u1 currently has a bug on Oneiric regarding the network connection. | 17:10 |
| hrw | alecu: I do not use U1 (bug have account) - just checked option in indicator | 17:10 |
| hrw | s/bug/but | 17:10 |
| hrw | alecu: so if release==oneiric then u1_working=false/ | 17:11 |
| hrw | ? | 17:11 |
| hrw | bug 791548? | 17:11 |
| ubot4 | Launchpad bug 791548 in update-manager (Ubuntu) (and 10 other projects) "Oneiric: Never sees NetworkManager connection (affects: 18) (dups: 8) (heat: 108)" [High,Fix released] https://launchpad.net/bugs/791548 | 17:11 |
| alecu | hrw, that seems to be the bug, yes. | 17:14 |
| hrw | ok | 17:15 |
| hrw | will have to remember not to use u1 to share files between machines again | 17:15 |
| alecu | hrw, it's probably fixed on the Ubuntu One nightlies, but I think that there's another related issue where the gnome-keyring changed it's interface. | 17:15 |
| hrw | alecu: is there a way to enable u1 to generate logs usable for u1 devs? | 17:16 |
| alecu | hrw, I don't think logs would be useful in this case, since it's a known issue. | 17:18 |
| alecu | hrw, perhaps you should mark the bug as "affects me also" | 17:18 |
| hrw | alecu: not this case | 17:18 |
| alecu | oh, ok. | 17:18 |
| hrw | alecu: during UDS-O I used u1 to keep firefox profile - got insane amount of u1conflict* files (with just one machine using u1) and that took me out of u1 usag | 17:19 |
| alecu | hrw, interesting use case, we should definitely work to improve that. Let's ask the right people on how to work around that: | 17:20 |
| alecu | rye, ping ^ | 17:20 |
| alecu | facundobatista, verterok: ping as well ^ | 17:21 |
| hrw | I did that way cause laptop was packed and I forgot to copy profile from desktop | 17:21 |
| hrw | this time I will copy all required/useful stuff directly to machine - lost trust in u1 | 17:22 |
| facundobatista | alecu, hrw, we have a bug that could explain that if firefox wrote on those files a lot really fast... though to say without DEBUG logs | 17:23 |
| alecu | facundobatista, hrw is offering to enable the debug logs to help us solve this issue. | 17:23 |
| hrw | during next week I will be in Dublin at ubuntu platform sprint - can use firefox with u1 profile during that time | 17:24 |
| hrw | will curse a lot and reinstall userscripts few times per day but I can do that | 17:24 |
| alecu | hrw, also, there's a project called oneconf that's aimed at that: synchronizing installed applications and application settings using UbuntuOne. | 17:26 |
| alecu | hrw, I don't know what's the support for firefox in it yet. | 17:26 |
| hrw | sounds interesting | 17:27 |
| nessita | alecu: would you how to workaround the IMportError for clientdefs on windows, when trying to run the u1syndaemon? | 17:28 |
| facundobatista | hrw, alecu, actually, we'd need TRACE logs just to prove that is the same issue... it's a very complicated situation that happen in rare situations... that could not be that rare if the same file is written with only milliseconds of difference | 17:28 |
| hrw | facundobatista: I have no idea how it happened and how often firefox is saving its files | 17:29 |
| alecu | nessita, I usually do "gnome-autogen.sh&&make" in linux, because I share the folder with the VM. | 17:29 |
| nessita | ah... | 17:29 |
| nessita | will generate the file locally and copy it | 17:30 |
| nessita | alecu: thanks! | 17:30 |
| alecu | np | 17:30 |
| hrw | facundobatista: for me (as a user) it was really weird - one machine is using u1 and gets multiple per-file conflicts | 17:30 |
| nessita | alecu: how's your debugging going? | 17:30 |
| nessita | ralsina: your branch, approved! | 17:31 |
| facundobatista | hrw, yes, :( | 17:31 |
| ralsina | nessita: thanks! | 17:31 |
| ralsina | I need a second review for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800381/+merge/65502 if anyone can check it... | 17:32 |
| nessita | ok, lunchtime! | 17:35 |
| alecu | dobey, I made fix for a lint warning in this branch, and otto wants me to get more rubberstamps for it: https://code.launchpad.net/~alecu/ubuntuone-client/fix-cmt-usage/+merge/65438 | 17:46 |
| dobey | no, it just wants you to be patient | 17:48 |
| dobey | alecu: you'd set it back to approved before lp had finished rescanning after you pushed the fix | 17:48 |
| alecu | dobey, great, thanks. | 17:50 |
| dobey | facundobatista: btw, did you make any headway with that KeyError issue with new pyinotify? | 17:56 |
| facundobatista | dobey, not yet, but I'm on it | 17:59 |
| dobey | facundobatista: ok, thanks | 18:00 |
| nessita | alecu: did you do the same (build in local linux machine, copy over) with the logging.conf file? | 18:12 |
| alecu | nessita, I just do "make"; it's the same folder for both the linux host and the windows guest, so I guess the logging.conf gets generated too. | 18:17 |
| nessita | ah | 18:17 |
| nessita | alecu: do you have to do something special re: xdg package? | 18:17 |
| dobey | yes logging.conf is generated too | 18:17 |
| nessita | dobey: what for is generated? setting DEBUG if nightlies? | 18:18 |
| alecu | nessita, I got it from lp:~mandel/+junk/.... something. | 18:18 |
| nessita | mandel: ping, ping | 18:18 |
| alecu | nessita, it's on the wikipage | 18:18 |
| nessita | ack | 18:18 |
| dobey | nessita: it's generated to have DEBUG during development cycles, and INFO in stable releases | 18:18 |
| nessita | right | 18:18 |
| nessita | alecu: sorry to bother again, but, would you know where the syncdamon.log file is located? | 18:25 |
| alecu | mandel, ping: http://pastebin.ubuntu.com/630898/ | 18:27 |
| alecu | nessita, let me check... | 18:27 |
| nessita | I'm getting a Failure: twisted.internet.error.ConnectError: An error occurred while connecting 2: ENOENT | 18:28 |
| nessita | when trying to start SD | 18:28 |
| nessita | does that ring a bell? ralsina ^ | 18:28 |
| dobey | alecu: http://www.themacaque.com/?p=885 ? | 18:28 |
| nessita | alecu: mandel sent us a pywin32.dll with the patch applied | 18:29 |
| ralsina | nessita: do you have the patched win32.zip? | 18:29 |
| ralsina | he | 18:29 |
| dobey | mandel: ^^ btw, the link to the bug report in that post is broken :) | 18:29 |
| alecu | nessita, ralsina: I do, and I installed it, but still I get those errors. | 18:29 |
| nessita | ralsina: you mean the pywin32.dll? | 18:29 |
| nessita | ralsina: does that dll have anything to do with the twisted connecterror? | 18:30 |
| alecu | nessita, afaik no | 18:30 |
| ralsina | nessita: it may be a permission problem that DLL helps with | 18:30 |
| ralsina | or I may be completely wrong because I am trying to untangle SSO ;-) | 18:31 |
| nessita | ralsina: the dll fixes a problem when dealing with unicode paths | 18:31 |
| alecu | so, it's not a dll, but the whole c:\python27\lib\site-packages\win32 folder | 18:31 |
| nessita | so I don't think is related | 18:31 |
| ralsina | ok, no idea then | 18:31 |
| alecu | the .zip that mandel sent is to replace that whole folder | 18:31 |
| alecu | but after installing it, the tests keep failing | 18:31 |
| nessita | ralsina: do you have mandel's cell phone? can you please share that with me on private? | 18:32 |
| alecu | ok, I'm having lunch now. bbl | 18:32 |
| ralsina | oh, wait, are you trying to connect to SSO when you get that error? | 18:32 |
| alecu | nessita, do you have sso running when you get that error? | 18:33 |
| alecu | nessita, you should | 18:33 |
| ralsina | if that's the case, you need to start it | 18:33 |
| nessita | ok, no, sso is not running, I'll start it | 18:33 |
| === alecu is now known as alecu-lunch | ||
| nessita | how can I cancel a process? CTRL-C does not work, nor CTRL_Z | 18:35 |
| nessita | or how's the "process manager" executable called? (I can't do ctrl-alt-del) | 18:36 |
| dobey | taskmgr.exe i think | 18:36 |
| facundobatista | nessita, right button in the task bar? | 18:37 |
| nessita | that one! | 18:37 |
| nessita | facundobatista: and then? :-D | 18:37 |
| facundobatista | nessita, choose "task manager" or something | 18:37 |
| nessita | facundobatista: there is none in that menu | 18:37 |
| nessita | but dobey's suggestion kicked in | 18:37 |
| m_conley | hey all - I've got the Oneiric alpha 1 here, and I'm trying to get Ubuntu One set up for some development work I'm doing...and on opening Ubuntu One, it's complaining that an internet connection is required to join / sign in. I definitely have an internet connection working on this machine. | 18:50 |
| m_conley | has anybody seen this? solution or workaround? | 18:50 |
| dobey | m_conley: yes it's known, we're working on getting fixes out | 18:51 |
| m_conley | (this is a fresh install of Oneiric A1) | 18:51 |
| m_conley | dobey: cool. Is there an ETA on that fix? | 18:51 |
| dobey | m_conley: as soon as all the necessary fixes we have to make to get stuff running on oneiric, are fixed :) | 18:53 |
| m_conley | dobey: gotcha, cool - thanks | 18:53 |
| alecu-lunch | nessita, ctrl-shift-esc will open task manager; since you'll be using it often here's the shortcuts: sort by process name, then type the first letter of the process, then Alt-E to finish the process; ESC to close task manager. | 19:14 |
| nessita | alecu-lunch: all those keys re being caught from my "real" desktop | 19:15 |
| === alecu-lunch is now known as alecu | ||
| nessita | alecu-lunch: anyway, run - > taskmgr.exe worked | 19:15 |
| dobey | yay all the sso milestones are set to inactive now. don't know why they weren't before, but i'm guessing some other projects haave similar problem :( | 19:33 |
| nessita | Chipaca, alecu, ralsina: when trying to use sso on xp, I'm getting 'SSLError'. Any clues? | 19:36 |
| alecu | nessita, can you post the whole exception? | 19:37 |
| dobey | nessita: what version of python-httplib2 do you have? | 19:37 |
| nessita | alecu: that;'s all I have | 19:37 |
| dobey | oh wait, sso, nevermind | 19:37 |
| nessita | dobey: no idea | 19:37 |
| dobey | i don't think sso uses httplib2 does it? | 19:37 |
| alecu | nessita, then no clues. | 19:37 |
| nessita | alecu: seems to come from lazr | 19:37 |
| dobey | oh, lazr does | 19:38 |
| nessita | dobey: we use lazr, and that's where the ssl magix happens | 19:38 |
| === zyga-food is now known as zyga | ||
| dobey | nessita: please check the veresion of your httplib2 then. this sounds like the same error that happened recently on oneiric | 19:38 |
| alecu | isn't this lovely? http://pastebin.com/1QBCXf0s | 19:39 |
| nessita | dobey: any clues how can I check that? from a python interpreter? | 19:39 |
| alecu | I've just got my first u1client segfault. Cheers! | 19:39 |
| dobey | nessita: httplib2.__version__ maybe? | 19:39 |
| nessita | looking | 19:39 |
| nessita | alecu: congrats? | 19:39 |
| thisfred | http://www.cheersboston.com/pub/main_cheersfans.html | 19:42 |
| nessita | dobey: httplib2.__version__ -> 0.7.0 | 19:44 |
| * alecu will be afk for an hour or so. | 19:46 | |
| dobey | nessita: https://bugs.launchpad.net/ubuntuone-couch/+bug/797281 | 19:46 |
| ubot4 | Launchpad bug 797281 in ubuntuone-couch (Ubuntu Oneiric) (and 5 other projects) "LP API broken in oneiric with python-httplib2 0.7.0-1 (affects: 3) (heat: 28)" [Undecided,Fix released] | 19:46 |
| nessita | dobey: thanks | 19:47 |
| dobey | sure | 19:47 |
| ralsina | yay, it seems the only thing I still have to do to get SSO working on the installer is reimplement parts of ubuntu_sso.qt.gui :-) | 19:56 |
| nessita | alecu: ping | 19:58 |
| nessita | alecu: were you able to use show_gui in your w7 in a way where you get the QT ui and you enter username and password and all the login dance succeed? | 20:04 |
| ralsina | YEAH! I HAVE SSO EMBEDDED IN THE INSTALLER. I AM A GOLDEN GOD! ermmm... ok, yay! | 20:45 |
| karni | ralsina: What installer :)? | 20:47 |
| dobey | the installer that doesn't install anything | 20:48 |
| dobey | ie, the control panel wizard :P | 20:48 |
| karni | nessita: Remember you told me the PING_URL uses HMAC SHA1 signer? Well, Ubuntu One itself needs a plaintext signer. I haven't tested end-to-end my SSO so I haven't tried the ping url with plaintext signer instead of HMAC, but from what I've seen, the ping_url may be the only using HMAC which would be quite odd. | 20:49 |
| karni | dobey: ah :) | 20:50 |
| nessita | karni: where does it need plaintext??? | 20:50 |
| karni | nessita: I use plaintext message singer when using the Files REST API. | 20:50 |
| nessita | karni: that works, but HMAC should work as well. And you should n ot be using plaintext | 20:51 |
| dobey | karni: they should both work | 20:51 |
| karni | nessita: ok, will you stick around for a while? | 20:51 |
| dobey | karni: well, unless the oauth library you're using to sign the URLs does the HMAC-SHA1 wrong | 20:51 |
| karni | I'll check and be back in few minutes. | 20:51 |
| nessita | karni: yes, at least one more hour | 20:51 |
| karni | dobey: signpost, it's used all over android apps, so I don't think it get's that wrong (but who knows!) | 20:52 |
| karni | nessita: coolio | 20:52 |
| nessita | karni: if it does not work, there may be another bug (it happened to me) | 20:52 |
| ralsina | dobey: ok, the first run wizard. buzzkill! ;-) | 20:52 |
| dobey | karni: well i did a lot of work to fix it to be correct with the 1.0a spec, back in the day, in python-oauth and ubuntu one | 20:53 |
| karni | dobey: heh.. :/ | 20:53 |
| karni | dobey: I mean.. you fixed a lib or u1 ?? | 20:53 |
| karni | kk guys, lemme verify that one again. | 20:53 |
| dobey | karni: i fixed the python-oauth lib, and the u1 server's usage of it | 20:53 |
| karni | dobey: nice :) | 20:54 |
| dobey | karni: no it wasn't :) | 20:54 |
| * ralsina loves the fact that 1.0b is older than the 1.0a spec | 20:54 | |
| dobey | ralsina: what are you yammering about? there is no 1.0b :) | 20:54 |
| ralsina | ok, 1.0 then. I remember it was insane ;-) | 20:55 |
| dobey | yes 1.0 is older than 1.0a | 20:55 |
| ralsina | Well, the *first* 1.0 is | 20:55 |
| ralsina | dobey: this is the *new* 1.0 ;-) http://tools.ietf.org/html/rfc5849 | 20:56 |
| dobey | there is only one 1.0 | 20:56 |
| ralsina | ok there is the 1.0 draft that is superseding 1.0a | 20:56 |
| dobey | ralsina: no, that is 1.0a | 20:57 |
| ralsina | dobey: nope. Read it :-) | 20:57 |
| dobey | yes, i did. and it is 1.0a | 20:57 |
| ralsina | "This specification provides an informational documentation of OAuth | 20:57 |
| ralsina | Core 1.0 Revision A, addresses several errata reported since that | 20:57 |
| ralsina | time, and makes numerous editorial clarifications." | 20:57 |
| alecu | nessita, I'm back. Yes, I got show_gui in w7 to do all the dance, and after I tweaked it a bit I got it to shut down when the auth dance is done. | 20:57 |
| dobey | ralsina: read what you just typed again :) | 20:58 |
| alecu | nessita, I didn't find out why it was not getting the credentials, though I think it should be registering the callbacks first. | 20:59 |
| ralsina | Yes, this has fixes for 1.0a. So, it is not 1.0a by definition ;-) | 20:59 |
| ralsina | dobey: it is also a year newer than 1.0a | 20:59 |
| dobey | ralsina: well yes, the rfc was created a year after 1.0a, but that is 1.0a. "errata and editorial clarifications" is vague and could only be about the document itself, not the protocol | 21:01 |
| dobey | like "fix a spelling error" is an errata for the doucment :) | 21:01 |
| ralsina | dobey: ok, maybe this will convince you: http://oauth.net/core/1.0a/ "This specification was obsoleted by RFC 5849: The OAuth 1.0 Protocol." | 21:01 |
| ralsina | So the versions are 1.0, 1.0 revision A, "RFC 5849 the OAuth 1.0 protocol" | 21:02 |
| dobey | ralsina: as far as i'm concerned any version of OAuth was obsoleted by HTTP Basic Auth, which exist 20 years ago :) | 21:02 |
| ralsina | it's OAuth Core 1.0, Core 1.0 revision A, Oauth 1.0 | 21:02 |
| ralsina | dobey: be a good sport and accept there are two OAuths 1.0 ;-) | 21:03 |
| dobey | i have. there is 1.0 and 1.0a :) | 21:03 |
| ralsina | ok, then there are 3 1.0s ;-) | 21:03 |
| nessita | alecu: thanks. So, in win XP, ussoc crashes very bad with what it looks mem corruption at python.exe level (when using trunk) | 21:04 |
| dobey | well there is 1.0 1.0a and 1.0a published as rfc, but it's still 1.0a :) | 21:04 |
| alecu | nessita, have you unpacked the .zip by mandel? | 21:04 |
| nessita | alecu: if I use r720 (before txnamespipes), it shows the ui but I get the SSLError issue, that dobey mentioned may be the one casued by httplib2 | 21:04 |
| nessita | alecu: no, I have not, but mandel was there in the VM and didn't suggest that can be the casue | 21:05 |
| nessita | alecu: you think it can be related? | 21:05 |
| dobey | it is definitely caused by httplib2 :) | 21:05 |
| alecu | nessita, not. I was suspecting the .zip being the cause of the crash you and I experienced. | 21:05 |
| nessita | dobey: nice, would you know how I can get the fix? (Branching is not an option since bzr branch tries to creates a symlink which fails on XP) | 21:05 |
| ralsina | dobey: ok, I will stop. I am a kind and merciful winner of arguments ;-) | 21:06 |
| nessita | dobey: meaning, would you know if the fix is released? | 21:06 |
| dobey | nessita: eh? bzr should work on windows just fine | 21:10 |
| nessita | dobey: I got this message "attenopting to create symlink bla and failed" or something like that | 21:11 |
| nessita | (I don't have the vm to copy and paste the error) | 21:12 |
| karni | nessita: Sorry it took so long, I had to request removal of one paste haha ;> All right. That's my simple terminal client, notice last two lines https://pastebin.canonical.com/48915/ | 21:13 |
| karni | nessita: This happens with HMAC only, Plaintext singer works well. | 21:13 |
| karni | nessita: oh sweet! I can test this with a "REST console" plugin for Chromium and see if my oauth lib is broken or it's a general issue. | 21:14 |
| nessita | karni: that may be caused by the following: | 21:14 |
| dobey | nessita: weird; you can pull the patch off launchpad, and just apply it by hand i guess? | 21:14 |
| dobey | nessita: or just downgrade httplib2 | 21:15 |
| nessita | karni: you build the resquest foo and you sign it with tokens and HMAC encryption. When our server grabs that, it decrypts the HMAC and gets the tokens from there, and validate. The process of applying the HMAC encryption *depends* on the url being sent, so server side may decruypt it differently depending or url encodes | 21:16 |
| nessita | karni: thisfred is great explaining that ^ | 21:16 |
| karni | nessita: oh :O | 21:17 |
| thisfred | right, so the url on both sides has to be the same | 21:17 |
| karni | nessita: indeed. it might be the issue with encoding the "Ubuntu One @ foobar" get parameter | 21:17 |
| thisfred | which is fun if there's an apache mod rewrite in the mix that insists on decoding/encoding some characters | 21:17 |
| karni | was it get.. | 21:17 |
| nessita | karni: very likely | 21:17 |
| nessita | karni: you should debug that failure with vds | 21:18 |
| dobey | oh right, the new tokens that have spaces | 21:18 |
| dobey | should work though | 21:18 |
| karni | nessita: He's not around. But I'll try to work with this a little. Thanks for the hint. | 21:18 |
| thisfred | karni: could be. Also all of the header param keys have to be lowercased (I think, not 100% sure on that) and alphabetized | 21:18 |
| nessita | karni: is this for the ping only, or all the rest calls? | 21:18 |
| karni | nessita: no, this paste was just call to me() | 21:19 |
| nessita | karni: ... are you mixing our rest api call with SSO's? | 21:20 |
| karni | nessita: eee.. authenticate() actually. | 21:20 |
| nessita | me() is SSO | 21:20 |
| nessita | authenticate is SSO | 21:20 |
| karni | nessita: I'm talking purely about SSO. | 21:20 |
| nessita | karni: then we have no idea if it accepts HMAC or what | 21:20 |
| nessita | :-) | 21:20 |
| karni | nessita: And what singers (plaintext / hmac) I was using | 21:20 |
| * karni bitchslaps himself | 21:20 | |
| karni | nessita: You're right.. I'm so sorry :) | 21:21 |
| nessita | karni: all those needs to be addressed with pindonga | 21:21 |
| karni | nessita: I'll do that. | 21:21 |
| karni | it all works nicely with plaintext signer | 21:21 |
| karni | I guess I wanted to incorporate the Ubuntu One ping() into SSO, which basically is not part of SSO.. | 21:21 |
| nessita | karni: we (me, you) are just a high level client of SSO. Even ussoc is higher level that you since we use lazr and do not do any rest call "by hand" | 21:21 |
| karni | at least not in general sense | 21:21 |
| nessita | karni: no, those are 2 different things | 21:22 |
| nessita | karni: ping is *all* ours | 21:22 |
| karni | nessita: right :) | 21:22 |
| nessita | karni: SSO is all, well, SSO's :-D | 21:22 |
| karni | nessita: one correction though - I'm doing all the low level rest calls by hand :) | 21:22 |
| karni | nessita: ;D | 21:22 |
| dobey | karni: is the url you're signing one.ubuntu.com or login.ubuntu.com? :) | 21:22 |
| karni | dobey: one. | 21:22 |
| karni | dobey: in case of ping() | 21:22 |
| nessita | karni: that is what I mean: Even ussoc is higher level that you since we use lazr <- that "we" is me as in ussoc | 21:23 |
| ralsina | yay, the wizard can sign in and get a credential, all in-window (in windows :-) | 21:23 |
| nessita | karni: then that should be HMAC and it should work | 21:23 |
| karni | nessita: hahahah right :) | 21:23 |
| karni | nessita: thank you so much! | 21:23 |
| nessita | karni: all one. should be HMAC, all login. should be whatever pindonga says | 21:23 |
| karni | nessita: interestingly, we use plaintext singer in the files app.. is this bad? | 21:23 |
| karni | nessita: we used it since evar.. | 21:23 |
| karni | maybe I should change that | 21:24 |
| nessita | karni: I would say very bad, but I'm not an OAuth expert | 21:24 |
| karni | nessita: Well, I've seen other apps use it, so it can't be that bad.. especially that we're using https :) | 21:24 |
| nessita | karni: if I understand correctly, using PLAIN allow anyone to intercept the call and stole the username/password | 21:24 |
| ralsina | nessita: I am taking a break but will put a few more hours late tonight. I am now confident we will have SSO in the wizard today :-) | 21:24 |
| karni | (as in: other apps use it is not a good argument, but you know by common sense what I meant) | 21:24 |
| nessita | ralsina: that's good news! and even better if it's done for today :-) | 21:24 |
| dobey | nessita: well since we're on SSL it shouldn't matter. but you def don't want to send PLAIN over plain HTTP | 21:25 |
| ralsina | nessita: today as in "ready for review tomorrow" | 21:25 |
| nessita | dobey: right | 21:25 |
| nessita | ralsina: great, drop us an email and I'll review when I can | 21:25 |
| karni | nessita: right. so, we're on https, plus oauth uses nonce and timestamps etc etc, it's not easy to do man in the middle with oauth | 21:25 |
| ralsina | nessita: I'm afraid I had to do some not-pretty things because if I import things from SSO at the module level it installs a reactor | 21:25 |
| karni | nessita: I think it's been https that kept us safe. | 21:25 |
| karni | No, I'm sure of it :) | 21:25 |
| karni | Files never used http (it simply can't, because it won't work :) ) | 21:26 |
| nessita | ralsina: that should be fixed instead of ugly-patched (maybe this is not the case), but I can help when I look at the review :-) | 21:26 |
| ralsina | nessita: if it gets fixed it's just copy&paste on my side :-) | 21:26 |
| nessita | ralsina: fixed where? | 21:27 |
| ralsina | nessita: on SSO | 21:27 |
| nessita | ralsina: as far as I know is not broken | 21:27 |
| ralsina | nessita: well, if I import it it installs its own reactor. That seems a bit broken. | 21:27 |
| nessita | ralsina: the reactor should be installed only when running the app, not in the lib modules | 21:27 |
| ralsina | nessita: thus broken ;-) | 21:27 |
| nessita | ralsina: I'll take a look | 21:27 |
| nessita | karni: ok. I repeat I'm not an expert, so I will not say "you're good" :-) | 21:28 |
| karni | nessita: Understood, thanks :-) | 21:28 |
| ralsina | nessita: wait until tomorrow and I'll show you. Or you can look at this branch: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800359 | 21:28 |
| nessita | ralsina: ack | 21:28 |
| dobey | GRRRR | 21:29 |
| dobey | stupid invalid properties | 21:29 |
| dobey | karni: it's really easy to do mitm with oauth PLAIN sig, if you're the mitm :) | 21:30 |
| karni | dobey: point taken | 21:31 |
| dobey | karni: the signature is "token&secret" at that point :) | 21:32 |
| dobey | anyway | 21:33 |
| dobey | thisfred: bah, the Time Bandits scene I was going to twit you in reply with, is not on youtube :( | 21:41 |
| nessita | ralsina: you gone already? | 21:43 |
| ralsina | nessita: not really, just relaxing a bit | 21:43 |
| ralsina | I've been here since 7AM :-) | 21:43 |
| nessita | ralsina: style issues to adrees before proposing: 2 empty lines between all def's at module level | 21:43 |
| ralsina | nessita: ok. pylint doesn't get those? | 21:43 |
| nessita | ralsina: no, pep8 does | 21:44 |
| nessita | ralsina: also, one empty line between import blocks (so from twisted.internet.defer import inlineCallbacks should have an empty line before the from ubuntuone_installer.... block) | 21:44 |
| thisfred | dobey: I promise to watch it on netflix ;) | 21:44 |
| ralsina | you mean on the fix_800359 branch? That's not really clean, it has lots of cruft :-) | 21:44 |
| dobey | thisfred: you haven't seen it? oh, well. you must watch then | 21:44 |
| nessita | ralsina: ok, I was just letting you know. Will ignore then | 21:45 |
| dobey | "I must have fruit!" | 21:45 |
| ralsina | nessita: cool, thanks for letting me know | 21:45 |
| thisfred | dobey: I have, but a long time ago, so it's hazy | 21:45 |
| ralsina | thisfred: are you a Time Bandits virgin? You are in for a treat :-) | 21:45 |
| thisfred | ralsina: the benefit of a leaky memory, you get to have lots of treats ;) | 21:45 |
| ralsina | nessita: that branch was really rough on my Ctrl+C and Ctrl+V keys ;-) | 21:46 |
| ralsina | thisfred: there is a good alzheimer joke there somewhere ;-) | 21:46 |
| dobey | thisfred: unfortunately, having firefox crash randomly is not a treat :) | 21:46 |
| dobey | ah, finally got sso to work right again | 21:47 |
| dobey | now to fix the tests again | 21:47 |
| nessita | ralsina: after a quick review I don't see what problem there may be with having the reactor installed by importing ubuntu_sso modules, but you can show me tomorrow | 21:48 |
| ralsina | nessita: sure thing | 21:48 |
| thisfred | hmm, mouseclicks are mysteriously being ignored. Maybe too many usb thingies | 21:52 |
| alecu | thisfred, yeah, you should unplug that usb beer-cooler. | 21:59 |
| ralsina | the usb foot-fan is overkill | 21:59 |
| thisfred | actually it's the emacs-foot-pedals | 21:59 |
| ralsina | I would like an Esc pedal for vim | 22:00 |
| dobey | thisfred: you have 127 usb devices? | 22:01 |
| * ralsina was a drummer long time ago. Shouldcome natural :-) | 22:01 | |
| * nessita has usb-slippers | 22:01 | |
| thisfred | dobey: considerably less, but after plugging in an external drive, my mouse started misbehaving | 22:01 |
| * alecu was a plumber not so long ago. That's why he tends to look like mario nowadays. | 22:01 | |
| thisfred | maybe it's a faulty splitter | 22:02 |
| dobey | thisfred: or system is having disk i/o issues. i am getting that sometimes without so many devices :) | 22:02 |
| * thisfred used to be dumber, but not by much | 22:02 | |
| thisfred | could be | 22:02 |
| * ralsina is plumper now | 22:07 | |
| alecu | nessita, ralsina, it looks to me that signals are not working right on txnamedpipes | 22:09 |
| alecu | do you guys know anything about that? | 22:09 |
| ralsina | alecu: nope | 22:09 |
| nessita | alecu: no, but may be related to having crashes on sso + xp when using txnamedpipe? | 22:09 |
| alecu | nessita, I get no crashes, but I don't get the signals called either. | 22:10 |
| alecu | ralsina, you have a windows dev env on seven? | 22:10 |
| nessita | alecu: do you have any other specific detail? | 22:10 |
| ralsina | alecu: yes | 22:10 |
| alecu | ralsina, can you try this branch? | 22:11 |
| alecu | https://code.launchpad.net/~alecu/ubuntu-sso-client/broken-signals-questionmark | 22:11 |
| ralsina | alecu: sure | 22:11 |
| alecu | ralsina, start bin\windows-ubuntu-sso-login in one terminal (with set PYTHONPATH=.) | 22:12 |
| alecu | ralsina, and "python ubuntu_sso\qt\tests\show_gui.py" in another term | 22:13 |
| ralsina | alecu: will do it in 2', sorry | 22:13 |
| alecu | no problem | 22:13 |
| alecu | if signals work, then pdb should start. | 22:13 |
| alecu | but it does not, and it's puzzling | 22:13 |
| ralsina | alecu: branching now | 22:18 |
| === m_conley is now known as m_conley_away | ||
| ralsina | alecu: I get the regular SSO screen, no pdb on either terminal | 22:20 |
| alecu | ralsina, perfect. Now try logging in | 22:21 |
| ralsina | alecu: seems successful | 22:22 |
| alecu | ralsina, it should pdb after it returns the credentials, when the signal to return the credentials from the sso to the show_ui gets run | 22:22 |
| ralsina | doesn't | 22:22 |
| alecu | ralsina, right: that's the bug, because it should. | 22:22 |
| ralsina | :-( | 22:22 |
| nessita | alecu: does it work if using r720? (no txn) | 22:23 |
| alecu | nessita, will try it, good idea. | 22:23 |
| ralsina | ok, EOD now, will work late tonight but probably won; t connect to IRC so email if you need anything! | 22:29 |
| alecu | nessita, it does not work either | 22:29 |
| nessita | ralsina: ack | 22:29 |
| nessita | alecu: ouch | 22:29 |
| dobey | grr, rhythmbox | 22:29 |
| alecu | nessita, perhaps I'm connecting the signals in the wrong way... :P | 22:29 |
| nessita | alecu: so, we may be missing something since mandel is sure that the whole SSO dance (from top to bottom) was working | 22:30 |
| nessita | alecu: so, questions: | 22:30 |
| alecu | right | 22:30 |
| nessita | * are the signals within the sso code working? | 22:30 |
| alecu | nessita, but his show_gui.py was not doing the whole dance at all. | 22:30 |
| nessita | alecu: how can you tell? (I mean, what is your way of diagnose) | 22:30 |
| nessita | alecu: I'm asking this becasue if we tell mandel "the dance is not happening" we need to show why, since he's convinced is all working | 22:31 |
| alecu | nessita, look here: http://bazaar.launchpad.net/~alecu/ubuntu-sso-client/broken-signals-questionmark/revision/722 | 22:31 |
| * nessita looks | 22:31 | |
| alecu | nessita, the show_gui.py on trunk is not printing the results of the signal | 22:32 |
| nessita | alecu: :-( | 22:32 |
| nessita | where is the callback_wrapper used? | 22:32 |
| nessita | is it the signal decorator? | 22:33 |
| * nessita browses her code | 22:33 | |
| nessita | ugh that decorator is not pretty | 22:35 |
| alecu | nessita, right. The callback wrapper is inside the @signal decorator | 22:35 |
| alecu | nessita, and that pdb is not being started either | 22:35 |
| nessita | alecu: right, we *could* think that if the callback is called, somehow, in a thread, you will not get the pdb | 22:36 |
| alecu | nessita, so it looks like the signal is not being sent either after and before txnamedpipes | 22:36 |
| nessita | alecu: what about adding a HUGE print? | 22:36 |
| alecu | nessita, will try | 22:36 |
| nessita | or an assert False, or something more visible | 22:36 |
| nessita | alecu: I know this implementation has "several" threads, so I will not count the callback is called in the main thread | 22:37 |
| nessita | alecu: what I don't know is if we will see the print if the callback is being called in another thread :-/ | 22:38 |
| alecu | nessita, don't think so. This is called by twisted, so it should happen on the main twisted thread. | 22:39 |
| alecu | nessita, and loggers always work from every thread. | 22:39 |
| nessita | alecu: everything that is blocking is ran in a Thread | 22:41 |
| nessita | with deferToThread | 22:41 |
| nessita | (see def blocking on windows.py) | 22:41 |
| alecu | nessita, I think I found a clue. The list of connected clients is empty on the server, so the signal is being sent nowhere | 22:44 |
| alecu | nessita, so you may EOD at will, and I'll keep looking there | 22:45 |
| thisfred | gotta walk the hounds | 22:48 |
| nessita | alecu: ok, I'll try to jump in later | 22:50 |
| nessita | my sister is coming to visit, but I'll do my best | 22:50 |
| nessita | alecu: thanks! | 22:50 |
| nessita | alecu: I know! | 22:52 |
| nessita | alecu: you need to call register_to_signals, I think | 22:52 |
| alecu | nessita, just found that! | 22:53 |
| nessita | alecu: that way you get appened to the self.clients list | 22:53 |
| alecu | nessita, don't worry | 22:53 |
| alecu | right | 22:53 |
| alecu | nessita, wonder why show_gui didn't call that :-( | 22:53 |
| nessita | alecu: no idea :-( | 22:54 |
| alecu | and, it's a stupid API, because if the client dies, the server will keep the client registered. | 22:54 |
| nessita | alecu: I'll stay until you confirm is working | 22:54 |
| nessita | urgh | 22:55 |
| alecu | nessita, it workED! | 22:59 |
| alecu | yay! | 22:59 |
| alecu | using txnamedpipes | 23:00 |
| alecu | the whole dance worked fine. | 23:01 |
| alecu | (there's an error when sso is not started, but that's a different one, bug #800887) | 23:02 |
| ubot4 | Launchpad bug 800887 in ubuntu-sso-client "UbuntuSSOClient fails when starting the SSO process (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/800887 | 23:02 |
| dobey | later all | 23:08 |
| nessita | alecu: GREATTTTTTTT | 23:15 |
| nessita | alecu: congrats | 23:16 |
| nessita | I'm off now | 23:16 |
| nessita | bye all, see ya tomorrow | 23:16 |
| alecu | bye! | 23:16 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
