=== IAmNotThatGuy is now known as M0hi | ||
failure_ | anyone from the billig team here ? | 09:15 |
---|---|---|
mandel | failure_, not at the moment AFAIK | 09:35 |
mandel | failure_, most of the canonical employees are in the USA timezone, but if you have an issue you can send a support request and they'll get back to you asap | 09:35 |
mandel | failure_, or maybe rye can help you.. | 09:35 |
failure_ | :) | 09:35 |
failure_ | its just a billing issue | 09:36 |
failure_ | i have payed via paypal | 09:36 |
failure_ | and havent got the storage | 09:36 |
failure_ | but i guess no tech can fix that :P | 09:37 |
mandel | failure_, certainly not me, sorry :( | 09:41 |
failure_ | :) | 09:41 |
failure_ | np | 09:41 |
mandel | failure_, let me check if I can find anyone.. | 09:41 |
failure_ | i wait for the billing | 09:41 |
Chipaca | failure_: hi there | 10:20 |
Chipaca | failure_: most of our USA guys are on (national) holiday today | 10:21 |
Chipaca | failure_: I think most but not all, so as soon as come online I'll point them at you, or let you know if they've all gone fishing | 10:21 |
Chipaca | failure_: I presume what happened is you didn't click on paypal's last "click here to confirm payment" link that takes you back to the u1 website? | 10:22 |
Chipaca | failure_: do you have an email from us about that payment? | 10:27 |
Chipaca | failure_: failing that, do you have an email from paypal with a transaction reference? | 10:29 |
gatox | good morning! | 11:04 |
mandel | gatox, morning! | 11:46 |
gatox | mandel, hi | 11:46 |
mandel | gatox, how was the long weekend? | 11:47 |
gatox | mandel, it wasn't long for me.... i was here on friday :P LOTTTTTTT of ninja coding..... the 2.0 is really close........ you? | 11:48 |
mandel | gatox, fine, said god bye to a friend that left to france (got him very very drunk) and put together an ikea furniture.. fun times :P | 11:49 |
mandel | and now.. fighting with a mem leak in the tests.. | 11:49 |
gatox | mandel, jejeje yes, i saw the twits about your friend..... and the pictures | 11:54 |
mandel | gatox, I think he did learn the lesson, vodka is bad :) | 12:03 |
=== Guest10379 is now known as zumbi | ||
gatox | mandel, ah.... i though that lesson was: never go out with mandel and let him take pictures from you | 12:03 |
gatox | jejejee | 12:03 |
=== zumbi is now known as Guest72020 | ||
ralsina | good morning! | 12:04 |
mandel | morning! | 12:07 |
=== Guest72020 is now known as zumbi_ | ||
gatox | ralsina, hi | 12:14 |
* mandel crashed xcode, fuuuuuuu!!! | 12:26 | |
failure_ | Chipaca: transaktions-id-nr.5J143524573852824 | 12:33 |
mandel | ok, lunch time for me, at least I removed the mem leak.. | 12:42 |
* mandel misses GC at times or detests ARC, your choice! | 12:43 | |
mandel | ralsina, Chipaca: http://www.softwareontheside.info/2012/05/kde-ubuntu-one-dolphin-plugin.html | 12:45 |
mandel | in case you did not know, uses ruby, but hey what is it another lang to our ecosystem ;) | 12:45 |
ralsina | mandel: know about it, talked with the guy earlier :-) | 12:45 |
ralsina | mandel: redoing those ruby bits in python is kinda easy | 12:45 |
mandel | ralsina, pido no! ;) | 12:45 |
mandel | ok, quick lunch now and back to fighting with objective-c.. | 12:46 |
ralsina | gonna be afk for a few minutes | 12:53 |
alecu | hey, holas! | 13:29 |
=== m_conley_away is now known as m_conley | ||
gatox | alecu, hi | 13:31 |
* mandel back! | 13:36 | |
=== salgado is now known as salgado-brb | ||
mandel | alecu, may i have you input for: https://code.launchpad.net/~ballogy/ubuntu-sso-client/drop-lazr/+merge/107153 | 13:45 |
alecu | mandel, looking there too. | 13:47 |
ralsina | hola alecu | 13:48 |
mandel | alecu, I think we might as well ask nessita if to confirm that the code is not used in software center.. | 13:48 |
alecu | hola there, boss. | 13:52 |
alecu | mandel, I like the change in ballogy's branch. | 13:55 |
mandel | alecu, me too, I think we just need confirmation from the software-center people just to be absolutely sure | 13:56 |
ralsina | mandel, alecu: and then we remove a bunch of crap from the py2exe script :-) | 13:56 |
alecu | ralsina, do we need people to agree to the contributor agreement if the branch is all red? | 13:56 |
ralsina | alecu: he has it signed anyway | 13:57 |
mandel | ralsina, alecu I'm pinged nessita if she cannot think or anything that could go wrong I'd say lets kill it :) | 13:57 |
alecu | ralsina, oh, right: it's a LP team now. Awesome. | 13:57 |
ralsina | mandel, alecu: now is like, the best time to break things, if it breaks ;-) | 13:58 |
Chipaca | failure_: you still there? | 13:58 |
mandel | herb, I love how Chipaca talks with failure after that comment hehehe | 13:58 |
Chipaca | failure_: is that from the papypal email, or from ours? | 13:58 |
* Chipaca wonders where herb came into it | 13:59 | |
ralsina | mandel: you understand there is a he-rb in the channel and you ing him twice a day, right? ;-) | 14:00 |
Chipaca | mandel: and he' | 14:00 |
Chipaca | s awesome :) | 14:00 |
mandel | up, shit, stupid auto complete! | 14:00 |
mandel | gatox, look what you did, you infected me! | 14:01 |
gatox | mandel, eejejej | 14:01 |
alecu | mandel, I've approved it. | 14:01 |
mandel | alecu, +1 for https://code.launchpad.net/~alecu/ubuntuone-client/signature-dict/+merge/107265 I'll set it to approve and will block the one from dobey_ since its the same, right? | 14:03 |
gatox | i'm trying to change this: MUSIC_REAL_PATH = u'.ubuntuone/Purchased from Ubuntu One' to use the os separator.... and if i do that..... runninng the tests python dies.... WEIRD! | 14:09 |
alecu | mandel, right, you can dismiss the other branch. | 14:09 |
alecu | mandel, thanks for the review!!! | 14:09 |
mandel | alecu, ralsina, only problem about this https://code.launchpad.net/~ballogy/ubuntu-sso-client/drop-lazr/+merge/107153 is that it does not drop lazr in the setup.py and there is an import of decimal in bin/ubuntu-sso-login that if the comment is true we don't longer need after that branch | 14:10 |
mandel | alecu, ralsina shall I approve and a bug for that? | 14:10 |
ralsina | mandel: we can either do a merge of this and a later branch | 14:11 |
ralsina | or ask for a needsfixing | 14:11 |
ralsina | I think a needsfiing or that is kinda rude | 14:11 |
ralsina | grmbl. "For that" | 14:11 |
ralsina | mandel: but it fails lint, doesn't it? | 14:11 |
mandel | ralsina, yeah, and coming from the community I'm more inclined to approve it | 14:11 |
Chipaca | failure_: pindonga | 14:11 |
Chipaca | pindonga: failure_ | 14:11 |
mandel | ralsina, no, lint is ok | 14:11 |
Chipaca | now, talk! :) | 14:12 |
ralsina | mandel: so, let's approve it | 14:12 |
ralsina | mandel: and if you file it, I will clean it later | 14:12 |
ralsina | mandel: so you don't switch to it | 14:12 |
mandel | ralsina, no no, I've got enough with fsevents, I'm just doing by round of reviews atm :) | 14:13 |
ralsina | mandel: ok then | 14:13 |
gatox | alecu, ping..... this "path" MUSIC_REAL_PATH = u'.ubuntuone/Purchased from Ubuntu One' in ubuntuone/controlpanel/gui/__init__.py should contain the OS separators, or there is any particular reason why the "/" is hardcoded? | 14:15 |
gatox | or ralsina mandel ^ if any of you guys now | 14:16 |
alecu | gatox, probably because it's only used on linux? | 14:17 |
mandel | gatox, the important questions is, is that a path in the server side or not, if it is in the server side then is correct to use / | 14:17 |
ralsina | gatox: I suspect it's serverside | 14:17 |
mandel | gatox, if it is not in the server side, we have a problem | 14:17 |
ralsina | gatox: if it were clientside, that would be ~/blah, no? | 14:17 |
alecu | mandel, gatox: do we have a "music" folder on windows at all? | 14:17 |
gatox | it's being use to check something in a test that create a local path | 14:17 |
alecu | gatox, on linux, that's the folder where the music I purchased thru the music store shows up. | 14:19 |
gatox | it's being use to comparet 2 paths, in add_folder inside LocalFolders ui | 14:19 |
alecu | gatox, on windows we don't have a music store. | 14:19 |
mandel | ralsina, bug #1005544 and bug 1005548 | 14:19 |
ubot5 | Launchpad bug 1005544 in Ubuntu Single Sign On Client "Lazr is deployed on windows when it is not longer needed" [Medium,Triaged] https://launchpad.net/bugs/1005544 | 14:19 |
ubot5 | Launchpad bug 1005548 in Ubuntu Single Sign On Client "Decimal imported but not used in ubuntu-sso-login" [Low,Triaged] https://launchpad.net/bugs/1005548 | 14:19 |
gatox | alecu, so.... should i skip that test for windows | 14:19 |
gatox | ? | 14:19 |
mandel | alecu, but the udf should be added, right? or can be added I should say | 14:19 |
ralsina | mandel: yay :-) | 14:19 |
alecu | mandel, I think the UDF is automatically added by the music store. | 14:20 |
alecu | gatox, what's the test where this shows up? | 14:20 |
gatox | alecu, "test_add_music_folder_adds_the_folder" in test_folders | 14:20 |
ralsina | on windows, we may someday need to add that UDF from the client side, but not now | 14:21 |
ralsina | mandel: slightly confused, you said you would do those two, or I should? | 14:21 |
mandel | alecu, ralsina, is the last failing test in jenkins: https://jenkins.errormessaging.com/view/Windows/job/ubuntuone-control-panel-windows-test/57/testReport/ubuntuone.controlpanel.gui.qt.tests.test_folders/LocalFoldersPanelAddFolderTestCase/test_add_music_folder_adds_the_folder/ | 14:22 |
mandel | ralsina: | 14:22 |
mandel | <ralsina> mandel: and if you file it, I will clean it later | 14:22 |
mandel | <ralsina> mandel: so you don't switch to it | 14:22 |
mandel | <mandel> ralsina, no no, I've got enough with fsevents, I'm just doing by round of reviews atm :) | 14:22 |
ralsina | mandel: so you? ;-) | 14:22 |
mandel | ralsina, hehehe | 14:22 |
ralsina | mandel: you are not clear, dude :-) | 14:22 |
ralsina | gatox: switching to path separator should not break anything, or we skip on windows, I say alecu's call | 14:23 |
gatox | alecu, ralsina, switching to path separator...... BREAKS PYTHON running the tests :S | 14:23 |
mandel | ralsina, well, you said, 'so you don't switch to it', is this a http://i0.kym-cdn.com/photos/images/original/000/001/384/Atrapitis.gif | 14:23 |
ralsina | But you said "no no no" :-) | 14:24 |
ralsina | gatox: WAT | 14:24 |
gatox | ralsina, yap | 14:24 |
mandel | ralsina, is like a no*3 :) | 14:24 |
gatox | ralsina, shocking! | 14:24 |
ralsina | mandel: ok, answer with one word: are you doing it? ;-) | 14:24 |
mandel | ralsina, alecu, gatox that test does not fail on a normal windows env, we can always do skipIfJenkins | 14:24 |
mandel | ralsina, no as per managers orders! | 14:24 |
ralsina | mandel: ack! | 14:24 |
pindonga | failure_, hi there, I'm looking at what happened with your payment | 14:24 |
* mandel is starting to get confused.. | 14:25 | |
pindonga | failure_, pm | 14:25 |
gatox | mandel, it's failing for me | 14:25 |
ralsina | mandel: I may be the manager, but no means no ;-) | 14:25 |
mandel | gatox, hm.. really, I don;t recall it failing in my system.. | 14:25 |
gatox | mandel, yes.... it's failing here.... that's how i'm debugging it | 14:25 |
ralsina | gatox's evil freedom-hating machine strikes again! | 14:26 |
gatox | mandel, but only if you run the whole suit.... if you just execute that file it doesn't fail | 14:26 |
gatox | ralsina, but it's failing on jenkins too | 14:26 |
mandel | gatox, uh, wtf! | 14:26 |
gatox | soooo..... the question remains..... to skip or not to skip.... that is the question? | 14:26 |
alecu | gatox, mandel: I'm opposed to skipping this test on jenkins... I think we can make it work. | 14:27 |
gatox | is this test relevant for windows? | 14:27 |
alecu | gatox, the test is relevant, yes. | 14:27 |
alecu | gatox, think of a linux user that has bought songs, and that also uses windows. | 14:27 |
gatox | alecu, ok, then..... i think i should change that path to use os separator, and fix the tests that brake python | 14:27 |
alecu | gatox, wait. | 14:27 |
mandel | alecu, yes, if it is failing on both normal machines + jenkins we should fix it | 14:28 |
gatox | alecu, what? | 14:28 |
alecu | gatox, before changing to use path separator we need to know if the path is a server path or a local path. | 14:28 |
ralsina | alecu: but on the windows side it's just a plain UDF, I think this is a server side path | 14:28 |
mandel | alecu, with no ~ feels like server | 14:28 |
ralsina | alecu: so that the music store creates a consistently-named UDF on the server | 14:28 |
alecu | mandel, if it whizzes and vibrates slightly then yes, it feels like the server. | 14:29 |
alecu | mandel, try not to sit on top of it :-) | 14:29 |
mandel | lol | 14:29 |
alecu | ralsina, right. | 14:29 |
gatox | alecu, it's being use to compare with this u'C:\\Users\\tester\\.ubuntuone\\Purchased from Ubuntu One'.... after extracting the home_path | 14:30 |
gatox | alecu, also.... it's being use in a comparison when trying to add a local folder..... inside the LocalFolders ui | 14:31 |
alecu | gatox, so, I have a folder that shows up like "Purchased Music" in the web ui and on the control panel. On disk it's called "/home/alecu/.ubuntuone/Purchased from Ubuntu One", and the UDF is called "~/.ubuntuone/Purchased from Ubuntu One". | 14:32 |
alecu | gatox, ralsina: question is: "how should this folder show up on windows?" | 14:33 |
gatox | alecu, actually..... if the comparison was working..... it would show: MUSIC_DISPLAY_NAME = _('Purchased Music') | 14:33 |
ralsina | gatox: yes, the u1cp hides this ugly path. So, let's find a similar appropriate windows ugly path for it? | 14:34 |
gatox | ralsina, sorry.... didn't understand that | 14:35 |
ralsina | gatox: usually u1cp displays the path to UDFs. For this one it says "Purchased Music" | 14:36 |
gatox | right now.... the test is failing because in the _process_name method is comparing: u'.ubuntuone\\Purchased from Ubuntu One' against u'.ubuntuone/Purchased from Ubuntu One' | 14:36 |
gatox | and that make it use wrong values later | 14:38 |
ralsina | gatox: that should compare the server side paths of the UDF, why is it using \\ ? | 14:39 |
alecu | gatox, I've started the control panel on windows, on a pristine vm with the latest installer. On the control panel the folder shows up as ".ubuntuone\Purchased from Ubuntu One", so there's probably a bug there. | 14:39 |
alecu | ralsina, ^too | 14:39 |
ralsina | looks like it's using separators inconsistently | 14:40 |
gatox | alecu, yes, the problem is with the _process_name method using MUSIC_REAL_PATH | 14:40 |
ralsina | and noone noticed since we never looked for it | 14:40 |
gatox | which has / hardcoded | 14:40 |
gatox | and we are using a windows path to compare that | 14:40 |
gatox | which contains \\ everywhere | 14:40 |
alecu | ralsina, right: I bought U1 tracks using my main account; but I never test in the windows VMs with that account, always with a tests one. | 14:41 |
alecu | gatox, I think it's a mistake to compare there using the local path; the user may have chosen any path for the UDFs. | 14:42 |
ralsina | gatox: so either we are comparing the local path and shuld compare the server side path instead, or you need to change the separators in the constant | 14:42 |
alecu | gatox, the only "stable" path is the server path. | 14:42 |
alecu | gatox, and by "stable" I mean, that's the path that the music store will always use to deliver purchases. | 14:43 |
gatox | ralsina, alecu i think its comparing the local with the server path, to obtain how that folder should be shown in the control panel for the user, and the items shows that display name, but it contains the real windows path for that folder | 14:44 |
ralsina | gatox: right, in volume_info, it has the server side path as well, it should use that one | 14:44 |
alecu | gatox, using the local path (the real windows path) is wrong imho. | 14:45 |
gatox | alecu, yes, i agree with that..... showing the real windows path it's the bug | 14:45 |
alecu | gatox, it's ok to be showing the windows path on the control panel. | 14:46 |
gatox | and sorry to repeat myself jeje but it's showing that.... because it's failing in the comparison of the paths..... the _process_name method, receive the windows path WITHOUT the home.... so it's the same as the server, but the problem right now is that it has different separators | 14:46 |
alecu | gatox, but we should be comparing the server path to that string (".ubuntuone/Purchased...") to see if we need to replace the local name for the nice string "MUZIK!" | 14:46 |
gatox | alecu, ok..... can we have a quick mumble?? :P | 14:47 |
alecu | gatox, sure! | 14:47 |
alecu | gatox, so, I understand that we have some issue. I've just been describing what the solution should look like! | 14:48 |
alecu | gatox, I think that might be why we are sounding like crazy people :-) | 14:48 |
gatox | alecu, on mumble! | 14:48 |
gatox | alecu, jejej maybe..... but i want to double check ejje | 14:48 |
ralsina | team standup in 3 | 14:57 |
ralsina | it's just mandel, gatox, alecu and I though | 14:58 |
mandel | how I imaging gatox and alecu on mumble: http://www.youtube.com/watch?v=SiHSxi2eiNc | 14:59 |
mandel | ralsina, ack | 14:59 |
alecu | "esa no que la tengo repe!" | 15:00 |
mandel | jeje | 15:00 |
gatox | me | 15:01 |
ralsina | me | 15:01 |
mandel | me | 15:01 |
ralsina | "para gila, de tu amigo, san antonio" :-D | 15:01 |
ralsina | writing notes, damn short standup :-) | 15:02 |
failure_ | poff and thanks for the great support as always! <3 one | 15:02 |
gatox | mandel, did you saw my twit about some spaniard guy dancing in madrid? jeje | 15:03 |
mandel | gatox, nop, I'll take a look :) | 15:03 |
gatox | mandel, wait.... it's old, i'll give you the link | 15:03 |
mandel | ok, so do we start? | 15:03 |
ralsina | go! | 15:03 |
gatox | mandel, http://youtu.be/0Rqa1VXmhw8 | 15:03 |
gatox | DONE: | 15:03 |
gatox | Reviews, stress tests for fsevents on mac, debugging u1-cp failing test. | 15:03 |
gatox | TODO: | 15:03 |
gatox | Create a new bug for the problem that we are having in u1-cp, and FIX IT. Keep working on mac fsevents. | 15:03 |
gatox | BLOCKED: | 15:03 |
gatox | No | 15:03 |
gatox | ralsina, go | 15:03 |
ralsina | gatox, you go, I mean | 15:03 |
ralsina | DONE: finishing get_rootdir bug, reviews, helped around, most of a natl holiday, TODO: merge this thing, fix the cleanup manuel reported, tech leads, general VMs cleanup BLOCKED: no | 15:04 |
ralsina | nest mandel! | 15:04 |
mandel | DONE: reviews, reviews, reviews. Found a mem issue in the fsevents tests and got it fix. | 15:04 |
mandel | TODO: Propose merges on fsevents. More tests and add more support on python. | 15:04 |
mandel | BLOCKED: no | 15:04 |
mandel | alecu, please! | 15:04 |
gatox | look at this video, it's a spanish guy that is funnier than mandel : http://youtu.be/0Rqa1VXmhw8 jejejeej | 15:07 |
mandel | gatox, hehe is one of my fav shows :) | 15:08 |
gatox | jeje really? | 15:09 |
gatox | the old guy is funny too | 15:09 |
mandel | gatox, yes, and that step is really hard | 15:09 |
* gatox imagine mandel dancing like that when he is drunk jeje | 15:09 | |
mandel | gatox, they now have a 'party' in which they gather 10000s of homosexuals in a ani-gay village is spain :P | 15:10 |
mandel | gatox, I'd love to be able to dance like that lol | 15:11 |
gatox | jejeje | 15:11 |
alecu | uh, I missed standup. | 15:17 |
* alecu writes notes | 15:17 | |
alecu | DONE: nat holiday | 15:20 |
alecu | TODO: a fix for sec patches, sort some SD bugs, get tests running on mac | 15:20 |
alecu | BLOCKED: no | 15:20 |
elopio | ralsina: ping. I'm having a little trouble starting SSO with testability. Do you have a sec? | 15:28 |
ralsina | elopio: sure! | 15:28 |
elopio | ralsina: when I start sso from command line, I can see it attached to the testability driver server. But when it is opened from the u1 wizard, testability doesn't notice it. | 15:29 |
ralsina | elopio: could be the envirnment variable is not set | 15:30 |
ralsina | elopio: are you setting it in a terminal and then starting the wizard from that same terminal? | 15:30 |
elopio | ralsina: I exported TESTABILITY=True, and ran the sso from the same terminal. It works. | 15:30 |
elopio | ralsina: then exported TESTABILITY=True and ran ubuntu control panel from the same terminal, and it doesn't. | 15:31 |
gatox | alecu, it this ok? https://bugs.launchpad.net/ubuntuone-control-panel/+bug/1005574 | 15:31 |
ubot5 | Ubuntu bug 1005574 in Ubuntu One Control Panel "Adding folders, doesn't work as expected with UDFs" [High,In progress] | 15:31 |
ralsina | elopio: linux? | 15:31 |
elopio | ralsina: yes. Ubuntu precise, with nightlies. | 15:31 |
ralsina | elopio: ok, I can try it myself... where can I get testability? | 15:32 |
elopio | ralsina: http://pad.ubuntu.com/u1-testability | 15:34 |
ralsina | elopio: thanks | 15:34 |
ralsina | elopio: will get back to you in 10' with some idea :-) | 15:35 |
elopio | ralsina: the easiest way to see what the server can do is through the testability visualizer, but the one from the gerboland ppa crashes. I had to compile it from git: http://gitorious.org/tdriver/pages/Home | 15:36 |
elopio | ralsina: great. Let me know if I can help. | 15:36 |
ralsina | elopio: sure! | 15:36 |
* gatox lunch | 15:36 | |
=== gatox is now known as gatox_lunch | ||
ralsina | elopio: since sso is activated via dbus, it's not getting the environment variable | 15:39 |
ralsina | elopio: so you will have to activate it manually :-( | 15:40 |
ralsina | elopio: OR you have to set it on .bashrc maybe | 15:40 |
elopio | ralsina: I did a workaround so testabiity run the sso, and it works more or less. It has a little trouble with the long command I used. And this is ugly. | 15:44 |
elopio | I think that setting it from bashrc might be better. | 15:44 |
ralsina | elopio: or maybe from one of the X startup scripts | 15:45 |
elopio | ralsina: yes. I'll ask sidnei what he thinks is better for jenkins. But that sounds like the way to go. | 15:46 |
elopio | ralsina: thanks. | 15:46 |
ralsina | elopio: de nada! | 15:46 |
ralsina | elopio: good news is it should work on windows :-) | 15:46 |
elopio | ralsina: yes, I'll try it today. Good news should come in the afternoon :) | 15:47 |
ralsina | elopio: awesome :-) | 15:49 |
=== gatox_lunch is now known as gatox | ||
ralsina | gatox: can you give me a hand on windows? | 16:56 |
gatox | ralsina, yep! shoot | 16:56 |
ralsina | gatox: ok, I want to see if there is a way to force the get_root_dir error | 16:56 |
gatox | ralsina, not having syncdaemon running? | 16:56 |
ralsina | gatox: could you try deleting syncdaemon and then starting u1cp and ee if that does it? | 16:56 |
gatox | ralsina, running from sources or with an installed u1? | 16:57 |
ralsina | gatox: I don't know | 16:57 |
gatox | ralsina, ok... i'll try from sources and then installed if needed | 16:57 |
ralsina | gatox: thing is, it doesn't happen in ubuntu ever, and don't want to reboot because I *am* installing the VM right now :-) | 16:57 |
gatox | ralsina, mmmm i'm not being able to reproduce it.... i'm getting other errors... | 16:59 |
ralsina | gatox: darn | 17:00 |
gatox | ralsina, do you want to reproduce it? or just force it??..... because you could hardcoded some piece of code to do that | 17:00 |
gatox | doing the backend None i think it will do the trick | 17:00 |
ralsina | gatox: thing is, if sd crashes, we end up (on windows) with backend set to None | 17:00 |
ralsina | gatox: so I want to make the UI notice that and fail | 17:00 |
ralsina | gatox: I think the code is done, but I want to see if it works IRL :-) | 17:01 |
gatox | ralsina, right..... let me try a couple of things | 17:01 |
gatox | ralsina, i'm trying to reproduce it.... but no luck so far :S...... just get u1-cp getting stuck | 17:15 |
gatox | but no showing any kind of error | 17:15 |
ralsina | gatox: daemn | 17:19 |
ralsina | so my fix is mostly theoretical | 17:19 |
gatox | ralsina, i was trying to kill sd and sso in different ways..... but same result | 17:19 |
gatox | ralsina, let me try just one more thing | 17:20 |
gatox | ralsina, nop..... no luck | 17:22 |
ralsina | ok, I'll propose anyway, and we'll see | 17:23 |
ralsina | I wrote a lot of code I threw away to come up with a tiny branch though :-) | 17:23 |
ralsina | gatox, mandel: a branch I am not very happy with! https://code.launchpad.net/~ralsina/ubuntuone-control-panel/decrypt-errors/+merge/107667 | 17:25 |
gatox | ralsina, on it | 17:26 |
mandel | ralsina, assertRaises :) | 17:26 |
ralsina | mandel: he | 17:26 |
ralsina | mandel: fixing | 17:26 |
ralsina | mandel: fixed && pushed | 17:30 |
ralsina | Oh, lunch! | 17:36 |
mandel | ralsina, what I think is a better fix is to go to /ubuntuone/platform/ipc/perspective_broker.py line 112 and check if port is Non and if it is throw the nicer exception | 17:36 |
ralsina | mandel: could be | 17:37 |
mandel | ralsina, that way we fix the issue at the root and not in the client side (control panel) | 17:37 |
ralsina | mandel: can't find that file... | 17:38 |
gatox | ralsina, all the tests are ok....... and the code seems ok (nice comments from mandel here), but +1 from me | 17:38 |
ralsina | mandel: ok, found it. Let me think a sec | 17:40 |
mandel | ralsina, also, before we both go, I think that the error should not longer occur when we are using endpoints | 17:40 |
mandel | ralsina, look at http://twistedmatrix.com/trac/browser/tags/releases/twisted-12.0.0/twisted/internet/endpoints.py#L254 | 17:41 |
ralsina | mandel: yes, but we are going to be doing windows releases from stable-3-0 fr a while and endpoints is not going to be merged there | 17:41 |
ralsina | mandel: this has a chance of being merged, though | 17:41 |
mandel | ralsina, oh.. interesting, well as long as we don't carry that to trunk :) | 17:42 |
mandel | ralsina, specially because by looking at endpoints, you will get a deferred: http://twistedmatrix.com/trac/browser/tags/releases/twisted-12.0.0/twisted/internet/endpoints.py#L131 | 17:42 |
ralsina | mandel: we can do this on trunk, and when we use endpoints we remove it | 17:42 |
ralsina | mandel: or leave it since it will be a noop | 17:42 |
mandel | ralsina, we already have enpoints in trunk :) | 17:42 |
ralsina | mandel: really? ;-) | 17:42 |
ralsina | mandel: I am out of date! | 17:43 |
mandel | ralsina, yes, I made the changes so that we already have all the ipc in domain sockets | 17:43 |
ralsina | ok, so I should really aim this branch at stable-3-0 | 17:43 |
ralsina | for which I need to reject and re-propose. Fun | 17:43 |
mandel | ralsina, which brings to mind: https://code.launchpad.net/~mandel/ubuntuone-client/unify-tools/+merge/107226 | 17:43 |
mandel | ralsina, sorry :( | 17:44 |
ralsina | gatox: this may be why we are not getting that error anymore on windows, BTW :-) | 17:44 |
ralsina | mandel: no problem at all | 17:44 |
gatox | ralsina, :P | 17:45 |
mandel | ralsina, I need to walk the beast, let me know via any transport if you need any help :) | 17:45 |
mandel | that includes domain sockets hehehe | 17:45 |
mandel | gatox, if you can review: https://code.launchpad.net/~mandel/ubuntuone-client/unify-tools/+merge/107226 I would be very happy | 17:45 |
gatox | mandel, of course! | 17:45 |
mandel | gatox, that should mean we are able to launch cp on mac, but I have not tried it | 17:46 |
mandel | ok, I'm off laters! | 17:47 |
gatox | mandel, wow! i'll try :P | 17:47 |
mandel | gatox, but it will not start sd, is a TODO from mmcc whenever he has the packaging | 17:47 |
* mandel runs | 17:47 | |
ralsina | argh, and tests from stable-3-0 don't run on nightlies ... ok, thisis going to take a second or two | 17:48 |
ralsina | I made a salad of merge proposals, but I think it's straight now | 17:59 |
ralsina | https://code.launchpad.net/~ralsina/ubuntuone-control-panel/decrypt-errors-3-0/+merge/107671 | 17:59 |
ralsina | but I think that one should wait for dobey, anyway | 18:00 |
ralsina | oh, right, I was going to hve lunch | 18:17 |
ralsina | failed at at that I guess | 18:17 |
ralsina | alecu: any thoughts on bug #1005189 | 18:30 |
ubot5 | Launchpad bug 1005189 in Ubuntu One Client "File Sync fails to connect on a native IPv6 client with NAT64" [Undecided,New] https://launchpad.net/bugs/1005189 | 18:30 |
alecu | ralsina, "who uses ipv6, anyway"? | 18:33 |
ralsina | alecu: diplomacy +1, roll again! | 18:33 |
ralsina | ;-) | 18:34 |
alecu | ralsina, sorry. So... yes, we should set up some dev envs for ipv6, and support it. | 18:34 |
ralsina | alecu: ok, marking as veryverylow ;-) | 18:34 |
alecu | ralsina, and learn a bit about ipv6 in the way... | 18:34 |
ralsina | alecu: worse, this requires we get a ipv6-only LAN with nat64 in the router. I don't think I can even set that up here | 18:35 |
alecu | ralsina, probably playing with VMs we can do it... | 18:35 |
alecu | ralsina, what about "patches welcome"? would that be a better answer? | 18:36 |
ralsina | I will say something like "we don't have any similar environments available, so we would love some help getting this done", how about it? | 18:36 |
ralsina | gatox: there is a unicode bug you reported and never fixed! I'm taking away your üñî©ódè badge! bug #994057 | 18:55 |
ubot5 | Launchpad bug 994057 in Ubuntu Single Sign On Client "qtnetwork failing sometimes on login or registration" [Undecided,Triaged] https://launchpad.net/bugs/994057 | 18:55 |
gatox | ralsina, yes.... i see it now | 19:09 |
ralsina | gatox: looks like it never happens for anyone else, so I am setting to low | 19:10 |
ralsina | gatox: are you getting this consistently? | 19:10 |
gatox | ralsina, i don't remember exacly how to reproduce it... i'll test it again as soon as i finish with this one | 19:11 |
ralsina | gatox: ack | 19:11 |
ralsina | gatox: probably the right person to fix it is alecu, but if you are the only one who sees it, you will have to team up | 19:12 |
gatox | ack | 19:13 |
alecu | gatox, I'm off to kinder right now, but I'll take a look when I return. | 19:15 |
gatox | alecu, ack | 19:17 |
gatox | ralsina, to see the "Purchased Music..." in u1..... do i need to upgrade the account? am i right? | 19:47 |
gatox | should i talk with beuno? | 19:47 |
ralsina | gatox: you need to buy 1 song | 19:56 |
gatox | ok..... eod for me.... alecu i might need your help tomorrow to reproduce this.... i'm not getting the purchased folder in windows for some reason.... | 20:20 |
gatox | bye all | 20:20 |
=== m_conley is now known as m_conley_away |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!