/srv/irclogs.ubuntu.com/2012/04/10/#ubuntuone.txt

=== urbanape` is now known as urbanape
mandelmorning!08:34
JamesTaitGood morning all!08:42
gatoxgood morning11:07
mandelgatox, morning!11:17
mandelgatox, I have been waiting for your evil machine :)11:17
gatoxmandel, jejeejej i know11:18
mandelgatox, you know which branches to use?11:18
mandelgatox, would be lp:~mandel/ubuntuone-dev-tools/mocked-webserver and lp:~mandel/ubuntu-sso-client/fix-activation-tests11:19
gatoxmandel, yep11:23
gatoxmandel, pulling and testing11:23
mandelgatox, thx, I added a wrapper factory that keeps track of all alive connections and removes them at the end by killing the transport, I really hope that it does the trick11:28
=== salgado-afk is now known as salgado
gatoxmandel, i run it a couple of times.... and i get always the same failures: http://paste.ubuntu.com/923135/11:30
mandelgatox, witch revno of ubunutone-dev-tools do you have?11:31
gatoxmandel, i'm using de mocked-webserver revno 7511:31
mandelgatox, ok, so that did not work when the factory is used as a client factory..11:32
mandelgatox, dammed, why doesn't it fail in my machine..11:32
gatoxmandel, it's really weird11:32
=== nessita1 is now known as nessita
ralsina good morning!12:54
mandelralsina, morning!12:55
gatoxralsina, hi12:56
dobeyhmm12:58
mandelok, time for me to go for lunch and take an aspiring.. HTTPClientFactory calls the transport.loseconnection after the deferred is called and that means that we have a connection left there when we leave the test..12:58
* mandel hates this twisted at times..12:58
gatoxmandel, me too :P12:59
* mandel lunch13:01
dobeywhat the heck13:03
ralsinaThere is a reason why twisted is not called straightforward, you know.13:03
dobeyis it 2012 or 1982?13:03
ralsinadobey: let me check... are we invading islands? No: 2012!13:03
dobey21 Jump Street, Three Stooges, Vacation, and Amazon tells me there's new Bonnie Raitt13:03
ralsinadobey: that's a combination of 60s and 90s, so probably it's 1978 or so.13:05
* ralsina checks to see if we are winning any world cups.... no, not that either.13:05
dobey90s?13:06
alecuhello, all!13:10
gatoxalecu, hi!13:11
ralsinadobey: 21 jump street ran from 1987 to 1991. I thught it was later.13:16
ralsinagatox: what's working?13:17
=== m_conley_away is now known as m_conley
gatoxralsina, jejee the problem that i was having with twisted, the client connection when its needed only for ipc, not opening sd for u1sdtool -q and so :P13:18
ralsinagatox: nice13:18
gatoxyeah!13:18
alecuralsina, I'm getting an error while running nessita's last branch: http://pastebin.ubuntu.com/923265/13:26
alecuralsina, do you know if I'm missing anything?13:26
dobeyyay, i love useless stack traces :(13:26
nessitaalecu: hi there. Looks odd, will re-check in a few minutes.13:26
ralsinaalecu: looking13:29
ralsinaalecu: looks like a missing humanize call13:29
alecuralsina, right. I thought that my env may be screwed since tests passed both for you and nessita.13:31
nessitaalecu: do you have nightlies up to date?13:31
ralsinaalecu: and did you merge trunk?13:31
briancurtinralsina: would you mind a few minutes to look this over? https://code.launchpad.net/~brian.curtin/ubuntuone-windows-installer/909389-firewall-exception/+merge/10113713:33
ralsinabriancurtin: sure!13:33
alecuralsina, I did merge trunk. nessita I updated nightlies yesterday, but this also fails if I point PYTHONPATH to sso and u1c trunk13:34
briancurtinhttps://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/c2c-loading/+merge/101246 could also use a look. it's a simple "fix" to move along with windows release - it fixes the Cloud To Computer page13:36
alecubriancurtin, do we have a bug for the TODO in your branch?13:39
ralsinabriancurtin: +1 on the fw exception (tricky stuff about the paths as usual on bitrock)13:40
briancurtinalecu: we don't, but i'll make one13:40
ralsinabriancurtin: I am not happy about merging c2c-loading13:40
alecubriancurtin, please, and add the number to the TODO, and I'll approve.13:40
ralsinabriancurtin: we could add it as a release-patch but if we put that in trunk, we break it on Linux where it works13:41
=== yofel_ is now known as yofel
alecuralsina, good point13:41
briancurtinralsina: i'm not really happy about it either, and don't know about the linux side of it. keeping it as a release patch would work for me to at least get it moving along13:41
briancurtinso should i reject the merge proposal, create the bug, and do it as a release patch?13:42
ralsinabriancurtin: ok, let's do that.13:42
briancurtinwriting up the bug now13:42
ralsinabriancurtin: yes, reject the mp, and we manually merge it13:42
dobeyralsina: https://code.launchpad.net/~dobey/ubuntuone-client/openssl-except/+merge/101308 last one :)14:10
ralsinadobey: I don't believe you :-)14:10
* mandel back14:10
ralsinadobey: you are not deleting the authors in some files14:11
ralsinadobey: lines 605, 638, 671 and so on14:11
dobeyralsina: i'm not deleting the authors in any files?14:11
ralsinadobey: ok, since you were updating copyrights...14:11
ralsinadobey: thought you may as well update to the current guideline14:11
ralsinadobey: but it's ok either way14:11
ralsinadobey: +114:14
dobeyah. i didn't remove the Authors lines in any of these branches. but I will happily do that in the future. Author lines in the comment heading are super annoying14:14
dobeycool14:14
ralsinadobey: should be easy to automate, remove any line that has ^#.*@canonical\.com in it :-)14:14
ralsinaHAHA, if you put a comment that is two spaces and try to send as approve, launchpad ooopses14:15
dobeynice14:16
mandelgatox, ping!14:28
gatoxmandel, pong14:28
mandelgatox, evil machine is required :)14:28
gatoxmandel, i know.....14:28
mandelgatox, hehehe you feel like a tool :P14:28
mandelgatox, is here: lp:~mandel/ubuntu-sso-client/fix-activation-tests/14:28
gatoxmandel, i'm taking note of this..... your assistance will required in the near future jejejej14:29
mandelgatox, attempting to remember the protocol used by the HTTPClientProtocol and force a lose connection in a clean up from the client side :)14:29
mandelgatox, I know.. tic for tac :P14:29
gatoxmandel, which mockserver should i use?? the one from junk or the branch?14:31
mandelgatox, junk please, so I see the stupid prints :)14:32
gatoxokas14:32
gatoxCONGRATULATIONS  mandel !! all the tests pass, and i've run them several times14:43
mandelhurray!14:44
mandeltests on windows are fixed!!!!14:44
briancurtinthree cheers to mandel14:44
mandelif someone does something again to them I'll swim to their country and kill them with a spoon, no joking!14:45
mandelbriancurtin, thx! I'll clean the code a little so I can get reviews and we can move to automate things with jenkins :)14:45
gatoxjejejejejee14:45
gatoxBBC news: "a spanish programmer die in the middle of the atlantic ocean"14:46
gatoxjejejejee14:46
mandelgatox, with the state of the country right now, it would be a surprise :P14:46
gatoxjejee14:46
ralsinagatox, mandel, alecu, thisfred, dobey, briancurtin, urbanape: standup in 6' (who did I forget this time?)14:54
thisfredyou forgot Harry14:54
ralsinaOMG! Harry!14:56
gatoxme15:00
briancurtinme15:01
mandelme15:01
ralsiname15:01
gatoxalecu, thisfred dobey urbanape ?15:02
briancurtinurbanape is on vacation15:02
thisfredme15:02
gatoxbriancurtin, ahhhh right15:02
thisfredDONE: u1db split_words mapping u15:02
thisfredahh15:03
alecume15:03
ralsinagatox: go, dobey is last15:03
gatoxDONE:15:03
gatoxFixed Bug #907479 (opening syncdaemon only when it's necessary). Test in linux: OK, fixing tests on windows.15:03
gatoxTODO:15:03
gatoxFinish with the tests for #907479 and keep working in the other branchs pending.15:03
gatoxBLOCKED:15:03
gatoxNo15:03
ubot5Launchpad bug 907479 in Ubuntu One Client "u1sdtool -q should not start syncdaemon when syncdaemon is not running" [Medium,In progress] https://launchpad.net/bugs/90747915:03
gatoxbriancurtin, go15:03
briancurtinDONE: testing installer, fighting with readonly tests and setup15:04
briancurtinTODO: giving readonly a break for this half day, building a script that will take care of the manual steps of installer building15:04
briancurtinNOTE: today is a half day for me, i have to leave in 2 hours15:04
briancurtinBLOCKED: none15:04
briancurtinNEXT: mandel15:04
mandelDONE: Fixed all tests on windows for ubuntu-sso-client.15:04
mandelTODO: Clean the branches a little (lint, comments, remove debugging prints), propose, beg for reviews. Talk with brian about automation.15:04
mandelBLOCKED: no15:04
mandelralsina, please15:04
ralsinaDONE: caught up, did some reviews, did some IRL testing of windows release candidate. TODO: try to get the windows release out after the linux release, see if I can fix the "remove local device and everything breaks" bug  BLOCKED: feeling kinda sore-throaty NEXT: thisfred15:04
thisfredDONE: u1db split_words mapping in C TODO: refactoring u1db index mappings and extending test suite BLOCKED: no NEXT: alecu15:05
alecuDONE: sifted thru SD bugs, started with security bug updates15:05
alecuTODO: mumble with ralsina, security bug15:05
alecuNEXT:15:05
alecuEOM?15:05
ralsinacrap, security bug15:05
ralsinadobey?15:05
ralsinaalecu: let me call you on the phone, mumble is working strange15:06
dobeyoops15:07
dobeyme15:07
dobeyλ DONE: finish bug #968555, reviews15:08
dobeyλ TODO: releases, uploads, SRUs, finish u1db packaging15:08
dobeyλ BLCK: none.15:08
ubot5Error: Launchpad bug 968555 could not be found15:08
thisfredmandel: you introduced bzr pipelines to the team right?15:11
thisfredI am trying to use them, but have some problems15:12
thisfredmainly pushing fails15:12
thisfredand lp doesn't pick up on the prereq branches15:12
thisfredthe two may be related15:12
mandelthisfred, may I see the error you are getting and the commands used?15:13
thisfredmandel, whenever I push, I get http://pastebin.ubuntu.com/923429/15:14
thisfredmaybe I need to set bzr locations differently?15:14
mandelthisfred,  have you done a bzr sync-pipeline first?15:15
thisfredto work around, I used bzr push lp:~thisfred/project/branch15:15
thisfredmandel: possibly not15:15
mandelthisfred, do that using the first pipe and pointing to lp15:16
thisfredwhen do I do that? when starting the pipeline15:16
mandelthisfred, so, you did a reconfigure-pipeline, right?15:16
thisfredor whenever I need to push?15:16
mandelthisfred, first time you sync, after that you can use push15:16
thisfredmandel: ok, awesome. Yeah I did reconfigure-pipeline15:17
thisfredguess that was the one step missing15:17
mandelthisfred,  :)15:17
thisfredthx!15:17
mandelthisfred, try that and let me know if it fixed the problem15:17
thisfredmandel: http://pastebin.ubuntu.com/923437/15:19
thisfredbut that could be because those branches were pushed manually already?15:20
mandelthisfred, bzr sync-pipeline lp:~thisfred/u1db/c-mappings15:20
thisfredoh ok15:20
* gatox lunch15:25
=== gatox is now known as gatox_lunch
dobeyhttps://code.launchpad.net/~dobey/libubuntuone/update-from-trunk/+merge/10139615:30
dobeyand i am off to get some lunch. bbiab :)15:30
=== salgado is now known as salgado-lunch
=== teknico_ is now known as teknico
mandelalecu, ping15:48
=== gatox_lunch is now known as gatox
alecumandel, pong15:52
mandelalecu, can you give me a hand with a problem I have with the time checker tests?15:53
alecumandel, sure!15:53
mandelalecu, so, the main problem I have is that if I change the url from http://localhost:portnum to http://127.0.0.1:portnum the tests fail with a HTTP Error 504: Gateway Time-out15:54
mandelalecu, do you know why?15:54
mandelalecu, I wanted to remove the need of using localhost if the mockwebserver returns the ip address15:55
alecumandel, can you point me at the branch?15:55
mandelalecu, sure, give me a sec to push it15:55
alecumandel, I'm not quite understanding which url you are referring to15:56
mandelalecu, in lp:~mandel/ubuntu-sso-client/fix-broken-tests revno 957 you will see there is a todo in ubuntu_sso/utils/tests/test_common.py15:59
mandelalecu, that branch depends on lp:~mandel/ubuntuone-dev-tools/mocked-webserver16:01
alecumandel, awesome, I'll take a look now.16:01
mandelalecu, thx!16:01
mandelalecu, in any case I can just pass locahost and get it working, it seems that using the ip address just makes it fail on linux..16:02
mandelgatox, ping!16:10
gatoxmandel, pong16:10
mandelgatox, I made some changes in the brach, it was mainly renaming, but can you check in your machine?16:10
gatoxmandel, okkkkkkk16:11
gatoxmandel, pull only fix-activation?16:11
mandelgatox, lp:~mandel/ubuntu-sso-client/fix-fix-activation-tests and lp:~mandel/ubuntuone-dev-tools/mocked-webserver16:11
=== salgado-lunch is now known as salgado
gatoxmandel, okas16:13
gatoxmandel, bad news :S16:19
mandelgatox, tell me, If I broke them I can revert :)16:19
gatoxmandel, the 13 tests failures are back16:19
mandelgatox, let me revert the changes16:22
gatoxokas16:22
ralsinaLunch time!16:33
alecumandel, the 504 error is a very odd one.16:34
alecumandel, do you happen to have proxies enabled when running those tests?16:34
mandelalecu, no, and it gets fixed by removing the ip for localhost16:35
=== m_conley is now known as m_conley_away
mandelalecu, I need to go (rugby) will be back later to char about it, nevertheless I've proposed the branhces that fix the tests16:36
alecumandel, I'm running the tests on linux, with 127.0.0.1, and all pass :-/16:37
mandelalecu, weird.. the fail in mine, wtf?16:38
alecumandel, that "504" error really sounds like something a proxy would return.16:38
mandelalecu, I know.. I was very puzzled..16:38
alecumandel, for instance: http://www.checkupdown.com/status/E504_es.html16:38
mandelalecu, I read the same page :)16:39
alecumandel, it's the top hit for 504 gateway timeout!16:39
mandelalecu, I need to go, people are waiting, I'll be back and will try to find what is going on16:39
mandelalecu, good news are, tests are fixed and up for reviews \o/16:40
mandelall, I need to go, last week of rugby, laters!16:40
alecumandel, awesome, have a good time.16:40
dobeymaybe i should just turn off spell checking in firefox16:42
dobeyso yeah16:42
dobeycan someone please review https://code.launchpad.net/~dobey/libubuntuone/update-from-trunk/+merge/101396 ?16:42
dobeyhmm, where's nessita?16:43
dobeyralsina: ^^ do i need to do all the releases that nessita normally handles, as well?16:46
briancurtini'm out of here for my half-day, see everyone tomorrow16:56
* alecu runs to the bank, and for some errands.17:10
gatoxbrb17:16
=== gatox is now known as gatox_brb
ralsinadobey: yes, but alecu will give you a hand17:33
ralsinadobey: sadly he doesn't have upload rights17:33
ralsinadobey: I got the review17:33
=== m_conley_away is now known as m_conley
dobeywill be a lot of reviews coming soon17:35
ralsinadobey: when did oauth.c oauth.h get deleted? I don't recall that branch17:35
dobeyralsina: http://bazaar.launchpad.net/~ubuntuone-control-tower/libubuntuone/trunk/revision/15017:36
dobeydidn't you review that branch?17:36
dobeyah no17:36
dobeygatox_brb and thisfred did17:36
thisfredI always approve any deletion17:37
ralsinadobey: ok, approving then!17:38
=== gatox_brb is now known as gatox
ralsinapeople, I feel like crap (fever, sore throat), so I am kinda slow today. Dobey, I will do your reviews, gatox, I just gave you the bug I was working on. I am taking some ibuprofen to see if that helps.17:54
gatoxack17:55
gatoxalecu, ping17:57
gatoxalecu, cancel ping18:02
dobeyhttps://code.launchpad.net/~dobey/ubuntu-sso-client/update-from-trunk/+merge/10142718:05
dobeyralsina: ^^18:05
ralsinadobey: got it18:05
ralsinadobey: +118:07
gatoxalecu, if you can, please, review this branch https://code.launchpad.net/~diegosarmentero/ubuntuone-client/syncdaemon-q/+merge/10098418:12
dobeygatox: nessita will need to review it18:13
dobeyor finish reviewing it18:13
gatoxdobey, yes, i know..... but i would like that alecu review it too, because i discuss part of the implementationn with him18:14
dobeyok18:14
dobeymeh18:31
dobeywhy do we have tests that depend on locale :(18:32
tgall_foourbanape, ping18:44
dobeytgall_foo: he's away this week18:49
dobeytgall_foo: so you probably won't have much luck poking him on IRC. :)18:49
tgall_fooahh thanks dobey18:50
tgall_fooknocked off a number of bugs from the iOS ubuntuone Files app, figured it'd be great to talk to him see what all might be helpful18:51
=== m_conley is now known as m_conley_away
=== m_conley_away is now known as m_conley
gatoxralsina, if you can: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/current-device-removed/+merge/10143219:18
ralsinagatox: sure19:18
gatoxthanks19:18
ralsinaout on errands, will be back in 20/30 minutes19:24
=== ralsina is now known as ralsina_errands
=== ralsina_errands is now known as ralsina
alecugatox, pong20:02
gatoxalecu, no, i made a cancel ping :P20:02
gatoxalecu, but if you can, i would like that you review the branch about the connect sd20:03
alecugatox, reviewing, yes.20:07
alecugatox, http://www.youtube.com/watch?v=hUFR2j7PCbk&feature=youtu.be&t=1h38m47s20:12
gatoxalecu, you are awesome!!!!!20:14
gatoxjejejeje20:14
alecugatox, I still need to find the part where I show up :P20:16
gatoxalecu, don't you have it??20:16
alecugatox, the youtube link I pasted is the full movie, and I know that at 1h38m47s my name shows up onscreen.... but I can't recall at what part of the film my face shows up, if at all :-)20:17
alecugatox, I'm just another extra with a lot of zombie makeup on.20:18
ralsinagatox: +120:22
gatoxralsina, great20:22
gatoxeod here!! see you tomorrow people20:29
gatoxalecu, if you have a minute.... please review this really small branch: https://code.launchpad.net/~diegosarmentero/ubuntuone-control-panel/current-device-removed/+merge/10143220:29
gatoxbyeeeee20:29
dobeyhmm20:49
dobeyhaha20:49
dobeyplaga zombie20:49
dobeyralsina: do we need that branch of diego's, in the release?20:50
ralsinadobey: well, it's a good bugfix20:50
ralsinadobey: there is one other release, right? But it won't be able to go in that one?20:51
dobeyralsina: today is 3.0.020:51
ralsinadobey: so, no more releases20:51
ralsinaright, I missed last week's!20:51
dobeyralsina: there are no more releases before ubuntu 12.0420:51
ralsinadobey: It would be nice to have it20:51
dobeythough we may be able to SRU things if we need to20:52
ralsinadobey: what's your opinion? It's a quite visible bug :-/20:52
dobeywell it's a small enough change20:52
ralsinaok, let's do it20:52
dobeyif it fixes it, land it and we'll get it in20:52
ralsinadobey: it's missing one review20:53
dobeynonsense20:53
dobeybut i'll approve it20:53
ralsinadobey: ack20:53
dobeyi suppose we need to clarify the whole "number of reviews required" thing20:54
dobeyqengho: interesting change of nick, that21:03
qenghodobey, too many misunderstandings.21:03
dobeyheh. no comfy chair?21:04
qenghoRight.  And these kids today (*shakes fist*) have never seen anything older than "Friends" and they think Ruby is cool.21:05
dobeyugh, kids today21:06
dobeyralsina: when will you do the win release? tomorrow?21:11
ralsinadobey: assuming the tarballs are done today, yes21:20
ralsinadobey: or rather, we start it tomorrow, will probably finish it on friday21:20
ralsinabrb21:21
dobeyralsina: ok, i still need to do ubuntuone-windows-installer, ubuntuone-client, and ubuntuone-control-panel21:21
alecuI'm feeling like crap, so I'm calling this an early EOD.21:27
alecubye all!21:27
=== m_conley is now known as m_conley_away
dobeyby alecu21:34
dobeyerr, bye21:34
dobeyralsina: given the time, i think i'll need to get those 3 done in the morning. and uploads tomorrow as well.21:34
dobeythough i might do a couple of the easy no-changes uploads later on tonight21:35
ralsinadobey: there is no point on doing windows-installer anymore, since it installs no code21:36
ralsinaor maybe yes, but it has very few changes21:37
dobeywell it's just a tarball release21:38
dobeyit still contains data and/or scripts for building the actual installer, afaik21:39
ralsinaright21:40
ralsinaEOD for me. dobey, if you propose branches, mail me, I may take a look early tomorrow21:41
ralsinadepending on sleepyness21:41
dobeyright21:41
dobeylater :)21:41
dobeyi'm off as well21:41
ralsinadobey: on second thought, windows-installer needs changes for a stable release (like, setting numbers and versions)21:41
ralsinadobey: so either brian or I have to propose that one21:42
dobeybrian's branch bumped the "203" to "1000" or something21:48
ralsinathat should be enough :-)21:49
=== salgado is now known as salgado-afk

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