/srv/irclogs.ubuntu.com/2011/11/24/#ubuntuone.txt

mterryalecu, dobey: FYI, after a reboot, I still see the dbus error.  Let me know what details I can provide in the bug and I will00:24
=== eu is now known as Guest21052
=== Guest21052 is now known as adorilson
dobeyalecu: btw, your ubuntu-storage-protocol timestamp branches still show as pending review from nessita; did you not change that?01:46
dobeyalecu: is it even possible on the LP page, for you to change that?01:46
alecudobey, I just tried one of those branches, and yes, I was able to change that.01:48
alecudobey, but it's way past my EOD and your EOW, don't worry about that.01:48
alecudobey, have a nice long weekend!01:48
JamesTaitGood morning all! :D09:44
mandelmorning all!09:57
mandelJamesTait, morning!09:57
JamesTaitHey mandel. :)09:58
mandelhm.. JamesTait are europeans the only ones that do not celebrate thanks giving?11:15
JamesTaitmandel: To be honest, I don't know for sure. I thought it was just a US thing.11:20
gatoxhi11:23
mandelgatox, morning!11:23
mandelgatox, I have a happy review for you!11:23
j0nrguys, hi. STILL not all my songs are being processed properly.11:23
mandelgatox, here you go: https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-tests/+merge/8326711:24
mandelthat fixes the sso tests on Windows :)11:24
j0nrMany songs that have been uploaded to U1 are not showing up correctly in Android Music Player. I can search for them and they fall under the unknown artist catagory11:24
mandelj0nr, hav you talk with someone of us before?11:24
j0nrmandel: yes11:24
mandelj0nr, was it karni?11:24
j0nrmandel: I have had someone rescan my library11:24
j0nrmandel: possibly... no manybe beuno and rye11:25
j0nrmaybe*11:25
* karni reads11:25
karniduanedesign: around?11:25
* rye reads11:25
karnirye: IIRC beuno said we'd look at what the server returns for j0nr in case he still gets unknowns after a rescan11:26
ryej0nr, I suppose I was rescanning the songs as well11:26
mandelj0nr, cool we have rye  with us, I'm sure he can give you a hand :)11:26
j0nrThe strange thing is that certain songs, I have batch tagged at the same time in the same software before upload, then only some of them get correctly recognised11:26
karnirye: I believe his music was already rescanned 1-2 times11:26
j0nrrye: not sure what you mean?11:26
ryej0nr, both duanedesign and I triggered song rescanning at different time so it appears to be not working right11:26
j0nrSome of them just get put into 'unknown artist' even though, the file itself definitely has an Artist ID3 tag set11:27
j0nrrye: other times its fine...11:27
j0nrFor e.g., I recently bought something in the U1 Music Store and it shows up fine (quite good actually as it was bought before the release date, so days later, it magically appeared on the release day )11:28
mandelralsina, ping?11:29
* gatox reviewing......11:35
mandelgatox, sweet, take a look closely, the way it works is to have the tearDown to wait until the client and the server are correctly closed. I'll be adding a new TestCase in ubuntuone dev tools that provides that so that we do not have to write it everytime11:39
gatoxmandel, cool!!11:40
mandelgatox, it was a major pita :P11:42
mandelgatox, thankfully once that was fixed the tests passed11:42
gatoxmandel, ping11:54
mandelgatox, pong11:54
mandelgatox, and now that we are talking, I'm getting this on Vista: lp:~mandel/ubuntuone-windows-installer/fix-vista-update11:54
gatoxmandel, your branch that include this: from ubuntuone.devtools.testcases.dbus import DBusTestCase was already merged into trunk?11:54
mandelgatox, sorry, paste fail hehehe11:54
mandelNoneType object has no attribute get_root_dir11:55
mandelgatox, ^11:55
gatoxmandel, i'm getting this: http://paste.ubuntu.com/748106/11:55
mandelgatox, let me check11:55
mandelgatox, you need to update ubuntuone-dev-tools in your system :)11:56
gatoxmandel, mmmmm i've seen before that get_root_dir11:56
gatoxahhhh ok11:56
mandelgatox, and on windows we do not have the update-manager hehehe11:56
mandelgatox,  a simple setup.py install should work, but check the installation path to make sure it works correctly :)11:56
gatoxok11:57
mandelgatox, I cannot believe people still use Vista.. oh my is bad11:57
gatoxmandel, well..... the lead developer or something like that of Vista said the other day that it was THE MOST SECURE OS............ juazzzzzzzzzz11:58
gatoxmandel, he said that it was even more secure that linux and mac11:59
mandelgatox, he might be right, even viruses crash on that system :P12:00
gatoxJAJAJAJAJAJAJA12:00
faganmorning :)12:13
mandelfagan, morning12:16
mandelgatox, can you run sd on windows from trunk12:18
mandel??12:18
mandelgatox, I thing bug 806532 is fixed in trunk already12:18
ubot4Launchpad bug 806532 in ubuntuone-client "Windows: the pyinotify implementation ignores the termination (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/80653212:18
gatoxmandel, give me a sec..... i'm trying to run your other branch :P12:19
mandelgatox, thx12:20
mandelI think that u1 is very broken on Vista.. but I'm using an AD user, so that also might be an issue12:39
mandeljesus christ!12:39
gatoxmandel, you know what i felt now12:39
gatoxjejeej12:39
gatoxtwo weeks using a weird username12:40
mandelgatox, yeah.. I have set up all my network with a single AD username so I don't have to remember more than one password, and that brakes things..12:42
mandelnevertheless is a very common use case.. so lets see what is going on.. I hate debugging py2exe bundle code12:43
gatoxjjejjee12:43
gatox:S12:43
mandelgatox, http://ubuntuone.com/4gqsa87Z4fSIeVnKDxDqkO12:45
mandelgatox, I lost a button!12:46
gatoxmandel, cuak!12:46
gatoxmandel, do you have any relevant trace for that?12:47
gatoxi think that probably the sync is broken12:47
gatoxand that doesn't update the button text12:47
mandelgatox, yeah, is the stupid NoneType error.. I'm trying to find out wtf is going on..12:47
gatoxmandel, if you find it..... or you have a trace show me..... i remember have seeing that bug..... but long time ago12:48
mandelgatox, yeah, I need to talk to ralsina, maybe he did not update the branch he used for the bundle..12:49
mandelI really hope that was the issue..12:49
mandelgatox, is eveyone on holidays?13:09
gatoxmandel, if they are...... they didn't count me in...... :(13:10
mandelgatox, I suppose dobey is because is thanks giving, and nessita took the full week13:10
gatoxjejejjee13:10
gatoxnessita should return today i think13:10
ralsinamandel: ?13:21
ralsinaThe error about None means syncdaemon is not starting13:22
ralsinamandel: and you are not going to debug that with a binary build :-/13:22
ralsinaAnd good morning, sorry I'm late.13:22
gatoxralsina, hi13:24
mandelralsina, yes, I can do it http://wingware.com/doc/howtos/debugging-under-py2exe13:25
mandelralsina, buenos dias :)13:25
Chipacache, mandel, qemu-img resize didn't work for you?13:25
alecumandel, is your branch with the "disconnect tests" fixes already for review?13:25
alecumandel, I would really like to review it.13:25
mandelralsina, can you try the following branch :  https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-tests/+merge/8326713:26
alecugatox, btw: I owe you some tests13:26
mandelalecu, https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-tests/+merge/8326713:26
mandelChipaca, no, no idea why, but the cat trick did the work :)13:26
* ralsina looks13:26
alecumandel, cool, thanks!13:26
gatoxalecu, i'm testing your branches..... i almost finish with maverick..... then all for natty13:26
ralsinamandel: what should I see there?13:26
mandelalecu, ralsina you both need to update the ubuntuone-dev-tools since it was updated and windows does not know about it :)13:26
alecugatox, que grande, gatox!!!13:26
=== yofel_ is now known as yofel
mandelgatox, we should fix your windows machine13:26
mandelfor tests I mean13:27
gatoxmandel, yep...... because you broke it! jejeej13:28
mandelralsina, anyway, I'm testing on a Windows Vista 64 machine and is crap.. we get the notification but the app i snot launched, which does not make much sense13:28
mandelralsina, the only diff is that is execute with the Shell runas thing13:28
mandelgatox, nah, I think you did the setup.py and did not double check where the code was installed, I think DistUtils extra does not work 100% on windows.. I need to check that13:29
ralsinamandel: So, running it manually work, running it with runas doesn't?13:29
mandelralsina, yeah.. I need to look closer into that, I'm setting the dev tools in the vista vm, on the mean time I'm looking at other bugs13:30
ralsinamandel: ok13:30
ralsinamandel: doing it manually with runas doesn't work either?13:30
mandelralsina, I have not tried, let me see13:31
mandelralsina, ha, puto windows vista: runas autoupdate-windows.exe does not work13:34
ralsinamandel: see? Nothing beats IRL tests13:34
mandelralsina, at least with an AD user.. which might be the issue since I have to pass the profile13:34
mandelralsina, I'll try with a non AD user, that means that those users in a corporate network that use AD and Ubuntu One won't be able to upgrade atm13:35
mandelralsina, I'll test it with Windows 7 too and will report a bug if necesary13:35
ralsinamandel: if we can detect and do nothing (or if it fails quietly enough)...13:35
alecumandel, the branch looks very compact, I like it13:35
mandelralsina, it does nothing, yet the command without the runas does work, which means that they get the message and then nothing happens13:36
mandelalecu, thx :)13:36
alecumandel, I'll paste some suggestions in the merge proposal, so using this is shorter for when you move this functionality to u1devtools13:37
ralsinamandel: I am thinking it may be related to security policies13:38
ralsinamandel: like, maybe you have to give a special permission to run autoupdate as AD admin13:38
* ralsina is thinking what windows would use instead of /etc/sudoers13:39
mandelralsina, yeah, I just tested with runas /user:mandel@ad autoupdate-windows.exe which works.. but it asks you the bloody password over the terminal13:39
ralsinawhich, of course, in a subprocess....13:40
ralsinaSo, it is what it is, users will have to do manual updates in AD. Not terrible.13:40
mandelralsina, kinda ugly.. I'll check on 7 maybe is a Vista feature13:40
ralsinamandel: ok, but don't waste too much time on it. It's not worth it.13:41
mandelralsina, will tests first Vista no AD user, if that works we can land the branch ans fix most of the Vista issues we have13:41
mandelralsina, testing on 7 is just launching a vm :)13:41
ralsinamandel: ok, just let's not lose focus13:42
mandelralsina, sure, no problem, I'm closing most of the bugs of my q :)13:44
mandelok, time for lunch13:46
* mandel lunch13:46
gatoxlisette, ping13:47
ralsinaalecu: mumble ping?13:56
alecuralsina, sure. Give me a minute so I can boot my mumble laptop.13:57
alecumandel, I've added a few small suggestions to your branch: https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-tests/+merge/8326713:58
alecugatox, ralsina: those are trial testing style suggestions, and perhaps they might interest you too.13:58
ralsinaalecu: thanks113:59
* gatox lookign.....13:59
alecumandel, gatox, ralsina: I guess it all boils down to "try not to use tearDown, use addCleanup as closely to the initialization as possible"14:01
lisettegatox: pong14:22
gatoxlisette, hi.... how are you?14:22
gatoxlisette, i was looking at this issue: https://bugs.launchpad.net/ubuntuone-windows-installer/+bug/82319514:22
ubot4gatox: Error: Bug #823195 not found.14:22
gatoxand says see attachment..... but i can't find the attachement :P14:23
lisettegatox: oops, i must have forgotten that in the bug frenzy i was in at the time..14:24
gatoxlisette, also..... can you add a little more info about this (if it's not much trouble): List header size and colour14:24
gatox* Check boxes (in the current screen there is no way to unselect a folder)14:24
gatox* Folder icons    ..... or give me the links of the docs where that is defined if you have them please :D14:24
gatoxlisette, i lost my bookmarks :P14:25
gatoxif that were in the docs14:25
* mandel back14:46
mandelalecu, looking14:46
mandelalecu, everything looks reasonable, I'll implemented in a few mins14:47
gatoxme15:00
gatoxalecu, mandel dobey ralsina ping! standup?15:00
ralsinamw15:00
gatoxahhhh team meating today15:00
ralsiname15:00
gatoxor standup?15:00
* ralsina checks the calendar15:00
ralsinateam meeting!15:00
* gatox opening mumble15:01
alecuelopio, ping15:07
alecuelopio, would you be able to join us in a mumble in say, 10 minutes?15:07
elopioalecu, pong15:07
elopioalecu, sure, no problem15:07
alecuelopio, we are having a team meeting right now, I'll ping you back then. thanks!15:08
alecuelopio, would you mind joining mumble?15:32
alecuelopio, channel: U1- Desktop Plus15:32
elopioalecu, ok.15:32
alecuthanks!15:32
ralsinagatox: bug #89443816:08
ubot4Launchpad bug 894438 in ubuntuone-control-panel "The --minimized option is ignored on windows (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/89443816:08
ralsinagatox: it's a one-liner16:08
ralsinagatox: but expect to write about 75 lines of tests for it ;-)16:08
gatoxralsina, jjejejeje nessita??16:09
gatox:P16:09
ralsinagatox: the thing is, I don't think there are any tests for command line options16:10
gatoxralsina, ahhhhhhhh there is the trap!16:10
gatoxjejeej16:10
ralsinagatox: which is why this slipped through our tests16:10
gatoxralsina, ok..... looking at that16:10
* ralsina may be tempted to look the other way as long as minimized=minimized is added to the main() call16:10
gatoxalecu let me know if you receive my message with the u1-client failure..... i'm missing something16:11
gatoxneed to restart... i'm about to lose control of the computer :P mandel please bring me new laptop soon! :P16:16
* gatox lunch..... brb16:20
ralsinaalecu, mandel: I knew there was something else with twisted/gi problems: unity integration! bug #85118716:32
ubot4Launchpad bug 851187 in ubuntuone-client (Ubuntu Precise) (and 7 other projects) "Progress bar does not appear on launcher (affects: 6) (heat: 28)" [Medium,In progress] https://launchpad.net/bugs/85118716:32
alecuralsina: right!16:32
mandelralsina, and we use twisted for that?16:32
ralsinamandel: looks like it16:33
alecumandel, we use gi from syncdaemon, yes.16:33
mandelouch!16:33
dobeyralsina: yes16:36
dobeyalecu: we have broken support for doing some other bits in syncdaemon through gi also; one is pretty easy to fix, though the fix is ugly; the other i am not sure about yet16:37
dobeyand ubuntuone-control-panel is really a mess16:37
ralsinaOk, lunch!16:37
=== ralsina is now known as ralsina_lunch
=== davidcalle_ is now known as davidcalle
mandelalecu, so I implemented the changes but I'm getting lots of mocker errors.. I'm considering getting rid of mocker, what do you think?16:52
gatoxis there anyone free for a review?17:05
mandelgatox, ok, I need someone to scream at :P17:09
gatoxmandel, what happend?17:09
mandelgatox, somos una panda de gilipollas de un nivel desmesurado!!17:09
gatoxmandel, QUE PASO?!17:09
mandelgatox, so, turns out that if you have sso running, the sso tests stop working17:09
mandelgatox, aint that nice :P17:10
mandelI've waisted like an hour on that17:10
gatoxmandel, ahhh..... yes..... i notice that once17:10
dobey?17:10
gatoxmandel, but i never had both things running together17:10
mandelgatox, I did 'til now..17:10
mandeldobey, yeah..17:11
mandelgatox, can you review this guy please: https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-tests/+merge/8326717:11
mandelgatox, if you can run tests on windows :P17:11
dobeymandel: oh, it only happens on windows?17:12
gatoxmandel, i can't run tests on windows yet....... thanks to you! :P jejeje17:12
mandeldobey, yes17:12
gatoxmandel, i'll save the review for later..... i'll try to fix the u-dev-tools problem in a while17:12
mandeldobey, I'll file a bug about that and.. I dont know send it to somewhere so that all know that17:12
dobeymandel: all the IPC we do there is direct P2P right, without an intermediate daemon like dbus?17:12
mandeldobey, yes.. I think it has to do with the activation, but I'm not sure17:13
mandeldobey, it gets the port of the running one or something and brakes17:13
mandelI need to look closer17:13
dobeymandel: right; it seems like we can't solve this the same way as we do for linux17:14
mandeldobey, windows has added a number of 'nice' problems of its own that we need to fix17:16
dobeyanyway, i think i will go make some lunch and get back to this holiday thing17:18
mandeldobey, yeah, is the turkey thing, go an kill a bird (with pun intended :P )17:20
mandelralsina_, ping17:23
dobeymandel: http://www.youtube.com/watch?v=cMVdqrKB60017:24
ralsina_lunchmandel: pong, but I am eating17:24
ralsina_lunchmandel: and I knew about the SSO running problem. I even mentioned it months ago ;-)17:24
mandelralsina_lunch, I was not listening :P17:25
mandelralsina_lunch, so, if a machine was added to a domain using AD the update wont work on Vista no matter if you are a local user or a ad user17:25
ralsina_lunchmandel: hehe. So, what's the pinging for?17:25
ralsina_lunchmandel: ok, so bad luck17:26
mandelralsina_lunch,  the above ^17:26
mandelralsina_lunch, so, the fix works for no ad machines (tested it)17:26
ralsina_lunchok, cool17:26
mandelralsina_lunch, I'm looking at windows 7, but I fear is the same issue17:26
ralsina_lunchI'll commit the installer-side fix17:26
mandelralsina_lunch, same with windows 717:27
mandelralsina_lunch, runas needs the user to be able to do the update and then the password17:28
mandelgatox, ^17:28
mandelI'll file a bug17:28
ralsina_lunchok17:28
mandelralsina_lunch, is my eod, so if gatox annot run https://code.launchpad.net/~mandel/ubuntu-sso-client/fix-tests/+merge/83267 can you do it?17:29
ralsina_lunchmandel: will try17:29
gatoxmandel, i save the link..... so if i can run the tests before my eod., i'll do it17:29
gatoxmandel, i need to run tests!! :P17:30
mandelgatox,  :)17:33
mandelralsina_lunch, here is the bug: https://bugs.launchpad.net/ubuntu-sso-client/+bug/89447117:33
ubot4Launchpad bug 894471 in ubuntu-sso-client "Autoupdate is broken on machines that have joined a domain with AD (affects: 1) (heat: 6)" [Undecided,New]17:33
mandeljoshuahoover, ping?17:33
ralsina_lunchmandel: I am *so* tempted to say "wontfix" but maybe we should ask upstream in bitrock17:36
mandelralsina_lunch, is not bitrock.. is the fact that we use runas, the nice way would be if they fixed the autoupdater to be able to run the check without being root, or set it as root and then check the version with python from our side17:37
mandelralsina_lunch, which makes more sense, right?17:37
ralsina_lunchmandel: we can't set it to run as root if we want to run it periodically17:38
ralsina_lunchmandel: so, we end up getting the XML and parsing it ourselves. Then whay do we need autoupdate.exe for?17:38
mandelralsina_lunch, that is for checking if there is a nwew version, installing needs root, checking for updates does not17:38
ralsina_lunchmandel: yes but it's a single binary17:38
ralsina_lunchmandel: so if I make that binary require root...17:39
mandelralsina_lunch, yes.. I know we need to either do it in python or find a diff way to execute the installer without runas17:39
ralsina_lunchmandel: ok, please put your thoughts in the bug tomorrow ;-)17:40
mandelralsina_lunch, yeah, rugby now, but will fix other bugs first then :)17:43
mandelralsina_lunch, I have approved https://code.launchpad.net/~mandel/ubuntuone-windows-installer/fix-vista-update/+merge/82418 to land in trunk since it fix the bug, I consider the domain bug a diff one17:44
ralsina_lunchmandel: agreed17:44
mandelralsina_lunch, also, when is nessita back?17:45
mandelralsina_lunch, I ahve a number of branches block for here review to fix bug 84565917:45
ubot4Launchpad bug 845659 in ubuntuone-control-panel "Windows: implement the "there is a new version available" notifications (affects: 1) (heat: 11)" [High,In progress] https://launchpad.net/bugs/84565917:45
ralsina_lunchmandel: tuesday I think17:45
mandelralsina_lunch, ok, so I'll remove her need of review if you agree so that we can land the changes and test in irl the check of the new version in a periodic manner, is that ok?17:46
ralsina_lunchyes17:46
=== ralsina_lunch is now known as ralsina
gatoxbrb!!17:47
mandelralsina, cool, then I'll work on landing that tom so that we can do IRL tests on the diff machines17:47
mandelnow, rugby!17:48
* mandel runs to tackle people!17:48
* gatox thinks that ralsina is lying to him, or i can not understand the bug.... everything is working fine with --minimized19:31
alecugatox, bug #59075219:32
ubot4Launchpad bug 590752 in ubuntuone-client "Client tests depends on having Desktopcouch paswords stored (affects: 2)" [High,Fix released] https://launchpad.net/bugs/59075219:32
alecugatox, that's the problem you are having on your maverick vm19:32
gatoxalecu, exactly!!!19:32
gatoxalecu, i'll take a look at that! thakns19:32
alecugatox, you'll probably need to start desktopcouch at least once19:34
alecugatox, by using the "me menu" -> "Ubuntu One..." option.19:34
gatoxalecu, thanks..... i'll try it when i finished with this vm19:35
* gatox soon is going to be able to open more than one vm at the same time19:35
ralsinagatox: I may be lying, but you shouldn't notice ;-)19:36
gatoxralsina, ejejjee...... what should happend if i open the control panel with --minimized????? i expect it to be minimize..... and print the have the --minimzed arg in sys.argv.....19:37
gatoxthat is happening right now..... or i'm missing something?19:37
ralsinagatox: are you trying it in windows?19:37
gatoxralsina, yes19:37
ralsinatrunk or stable?19:37
gatoxtrunk19:37
ralsinaweird19:37
gatoxralsina, believe me..... i can show you screenshots if you want :P19:38
ralsinaIt should not show a window, and only appear a an icon in the notification area. Is that what you get? from current trunk?19:38
gatoxralsina, exactly..... and if i press the icon the window appear19:38
ralsinagatox: *very* weird19:38
gatoxralsina, do you want  a screenshot?19:39
ralsinagatox: give me 5' to see19:39
gatoxralsina, ok!19:39
gatoxralsina, maybe you fix it and you don't remember :P19:39
ralsinagatox: I know I have not merged any branches19:40
ralsinagatox: I feel like I am a crazy person19:44
gatoxralsina, it works for you too?19:44
ralsinagatox: yes19:45
ralsinagatox: but I even remember checking what branch/revision broke it19:45
ralsinagatox: so, I will ust read the IRC backlog and find out ;-)19:46
gatoxweird....19:46
gatoxjejejejeej19:46
ralsinagatox, check https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/import-qt4reactor/+merge/81980 line 50 of the diff19:47
* gatox checking.....19:48
ralsinagatox: and line 4319:48
ralsinaThat function is getting the minimized argument and throwinf it away. And the branch is merged. So I have no idea why that code is not there right now :-)19:48
ralsinaOh, wait19:48
ralsinamark as invalid19:49
ralsinaThe problem is when you have this and another branch for unique instances at the same time, and I owe alecu and natalia an apology19:49
gatoxralsina, ahhhhhhhhh19:50
ralsinaSo, what really has to be done is fix my unique application branch19:50
ralsinaAnd I will do that myself.19:50
* alecu is not sure he understands the problem yet19:51
alecuralsina: is this problem happening because of an unrelated branch?19:51
ralsinaalecu: yes19:51
ralsinaalecu: The other branch broke because of the refactoring and it didn't even produce a conflict19:52
ralsinaalecu: that's what I get for manually merging branches on releases19:52
alecu:-(19:52
ralsinaSo, sorry, alecu!19:54
alecuno problem!~~~19:54
alecunothing to be sorry about !19:54
=== JanC_ is now known as JanC
gatoxeod here!! time to go to the gym! bye20:38
ralsinabye gatox!20:38
gatoxralsina, bye..... see you tomorrow20:38
alecumy house is officially invaded by small kids21:17
* alecu will return in a few hours21:17
mandelalecu, I just pushed the changes for the fix-tests branch :)22:15

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!