=== jalcine is now known as JackyAlcine === gatox_swim is now known as gatox [02:39] Anyone here? I'm having issues with U1 === hloeung_ is now known as hloeung [09:38] morning all! [10:14] Good morning all! :D [11:11] good morning [11:28] gatox, morning! [11:28] mandel, hi [11:29] gatox, oh, got a professional question for you here, what is the nice way that ralsina wrote to set up a new windows devel machine? [11:32] mandel, you need to install a clean machine [11:32] gatox, yes, I have that already... [11:33] mandel, then python, pyqt.... [11:33] gatox, I had some issue with the vmware machine I was using.. [11:33] mandel, let me look for the email [11:33] gatox, thx [11:36] mandel, http://paste.ubuntu.com/799251/ [11:37] gatox, thx! [11:37] mandel, look at that..... ask me if you need something else [11:37] mandel, bazaar is only needed if you commit or branch code from your vm.... i didn't install it because i do it from linux [11:37] ok [12:15] gatox, is amazing how terrible are the windows updates... [12:15] gatox, what a waste of time! [12:16] hello everyone! [12:16] nessita, morning! [12:16] nessita, hi [12:17] mandel, totally agree! [12:24] @ping [12:24] pong [12:25] gatox: so, I need to review for you network-detect and fix-links? [12:25] nessita, i'm finishing with fix-links right now.... the other one is ready [12:25] ack [12:37] gatox, agh. I need the vs2008 for the script to work, right? [12:37] stpuid pycrypto! [12:37] mandel, mmmmmmm i think you shouldn't [12:38] mandel, ahhhhhh ralsina gave me a different configuration file...... [12:38] i don't know if he put it anywhere else.... i'll email you that file [12:38] gatox, please :) [12:42] gatox: why are you adding # pylint: disable=W0212? not sure what W0212 is warning for [12:42] * gatox looking.... [12:43] nessita, access to a protected member [12:44] _next_id from the wizard [12:44] ah, thanks [12:48] gatox: reviewed done, added a couple of nf [12:49] nessita, ok [12:49] nessita, i'm testing fix-links.... because if i run just the file with u1trial it works.... but if run run-tests it fails..... so i'm look what might be going on [12:50] gatox: ack [12:58] gatox: ah, and seems like ralsina's branch from yesterday had some pep8 issues, could you please fix them in your branch? [12:58] ./scripts/conf.py:32:66: E202 whitespace before ')' [12:58] ./scripts/setup.py:300:54: E202 whitespace before '}' [12:59] nessita, project? [12:59] gatox: windows-installer [12:59] nessita, roger that! [13:00] thanks! [13:16] nessita, do you know the name of the dirspec project? [13:16] mandel: dirspec :-) [13:16] https://code.launchpad.net/dirspec/ [13:18] nessita, oh, that was easy.. [13:18] I expected something hard to guess.. [13:18] hehe [13:46] ok, time for lunch [13:46] * mandel lunch [13:53] damn, it's so hot here. [13:59] alecu: +10 [14:01] nessita, I've started with your review late yesterday. I'm currently in the middle of it. [14:01] alecu: awesome. Anything to fix? [14:01] back!! now i have 30mb! :D [14:02] gatox: wtf? is it iplan? [14:02] nessita, yes, a few things. Let me paste what I have so far in the mp. [14:02] nessita, nop.... fibertel evolution [14:02] gatox: does it fly? [14:02] is it optic fiber? [14:02] nessita, yes..... [14:03] * gatox cries.... [14:03] jejeje [14:03] nessita, there: https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/unify-signal-broadcaster/+merge/87759/comments/189874 [14:03] gatox: how much, if I may? [14:04] gatox, what's the upload speed? [14:04] nessita, if you have cablevision (i have it for consorcio.. but i don't use it :S) 300.... if you don't have cablevision 400 [14:04] alecu, 3mb [14:04] :P [14:04] alecu: we can't have a self inside a self [14:04] nessita, if you want to access the "upper" self, then you can't. But if you don't need to access it, then you can. [14:05] nessita, also.... you have corporate fibertel or something like that.... that is 8mb download, 1mb upload.... but constant and it works better that common users [14:05] alecu: I need to access it, for the self.app_name [14:05] alecu: that class is a terrible hack [14:05] alecu: that in the branch I'm working on will fly [14:05] nessita, no, the self.app_name is not used there. [14:05] away, away [14:05] a ver.... [14:06] alecu: you're right, I will change that (one of the versions of that class had it :-)) [14:06] nessita, also, I think that *that* "__init__ with lambdas" should be replaced with a few methods, as I said just below it. [14:06] hmm [14:07] alecu: I'm reading that... but since the class is so aweful, I would like to have it small. But I don't feel strong about that [14:10] alecu, mandel, gatox, dobey: 2 parochial announcements [14:10] 1- ralsina texted me and he says that the whole are he's in is "out of" 3g, he's looking for alternatives [14:10] 2- I have a medical appointment 15 minutes before the standup, so I will not be here for it. I'll paste it now: [14:10] DONE: tons of reviews, helped brian get tickets for the sprint (we succeeded!), started with bug #839877 [14:10] TODO: bug #839877 to the death! follow up on reviews from yesterday [14:10] BLOCKED: nopes [14:10] Launchpad bug 839877 in ubuntuone-windows-installer (and 2 other projects) "When creating a new account, the credentials stored are invalid. (affects: 1) (heat: 7)" [Critical,In progress] https://launchpad.net/bugs/839877 [14:10] nessita, ack! [14:10] nessita, should we be buying tickets too? [14:11] by we I mean facundo and me. [14:12] alecu: I guess so, but I would confirm with ralsina. You can ceratinly request options from the travel agency, but wait for ralsina before confirming [14:12] alecu: specially because I'm not sure how many days you can/want/canonical-wants-you-to attend [14:13] nessita, right. [14:14] I'll be leaving in 17 minutes, let me know if you need anything from me. I may be back on hour after that approx [14:15] where is ralsina anyway that he needs to use 3g? [14:15] dobey: mar del plata [14:25] alecu: did you get a chance to look at my twisted branch btw? [14:26] dobey, not yet, sorry. I'm still in the middle of reviewing nessita's big branch. [14:26] alecu: ok, thanks [14:27] ok, I'm gone for a while, I'll be back! [14:27] alecu, gatox: send me a sms if you need anything [14:27] you or somebody else, of course :-) [14:28] nessita, ok [14:45] che, anyone seen ralsina? [14:45] Chipaca: he reportedly has no 3g access and is looking for an alternative connection [14:46] Chipaca, 3g issues [14:46] ah....... what dobey says :P [14:47] dobey: gatox: thanks [14:48] Chipaca, he texted nessita about it [14:48] nessita: could you hop on mumble at 12? [14:48] Chipaca => 2- I have a medical appointment 15 minutes before the standup, so I will not be here for it. I'll paste it now [14:48] Chipaca: nessita just left for a medical appt. :) [14:48] poop [14:49] Chipaca, she did not mention if it was about that... too many details [14:49] hehehe [14:49] jejeje [14:49] Chipaca: what do you need? [14:50] a managerialish overview :) [14:50] never mind [14:50] "There's nothing to see here, move along." :) [14:51] "This is not the droid you are looking for..." [14:51] i hear we're updating twisted in P? [14:52] it's already been updated to 11.1.0, but yes i am working on getting the new reactor in [14:52] gireactor that is [14:52] rockzilla [14:52] rockzillantastic, even [14:52] and there are changes to gtk2reactor in trunk, which we need to get in === m_conley_away is now known as m_conley [14:54] but new gireactor also requires some changes to gtk2reactor, to get accepted upstream, which i was working on yesterday; and unfortunately ran into issues with, otherwise would be ready to re-propose [14:57] dobey, gatox, alecu do we do the stand up? [14:57] we have 3 mins to decide :P [14:57] mandel, sure, let's do it [14:57] you know my answer :) [14:57] dobey, which is the same as main ;) [14:58] s/main/mine [14:58] 2/3 majority! [14:58] my standup is really short..... all the morning fighting with the same thing [14:58] oh i guess there are 4 of us [14:58] standup +1 [14:58] 2/2 [14:58] :P [14:58] ah crap [14:58] gatox, you vote counts 1/2, you are to young to have a full vote [14:59] lol [14:59] good morning! [14:59] also, short too :P [14:59] lol [14:59] ralsina, hito_jp0 [14:59] ralsina, hi [14:59] jeje autocomplete [14:59] gatox, hahaha I read hijo_pu0 [14:59] me too! [14:59] jejjeje [15:00] that wasn't my intention! [15:00] me [15:00] so, what did I miss? [15:01] ralsina, you missed nessita standup, she had to go to the doctor [15:01] me [15:01] ralsina, also, Chipaca was looking for you [15:01] me [15:01] me (no notes) [15:01] alecu, dobey ? [15:02] i cast my vote already! [15:02] gatox, I've already said "me" twice. [15:02] alecu, yes..... i see it now [15:02] :P [15:02] meh [15:02] DONE: [15:02] Fixed some branches, start working on network detect in u1-client, fighting with a weird bug in the tests of u1-client-fix-links [15:02] TODO: [15:02] Fix the tests in that branch, keep working on network detect for u1-client [15:02] BLOCKED: [15:02] No [15:02] alecu, go [15:02] I've been told that cordoba residents like standups.. [15:03] jeje [15:03] DONE: bureaucracy, fought against buildout, merged windows packaging! TODO: reviews, organize scopes contractor, mgmt call, review resumes for mac position blOCKED: was without internet a long while [15:03] DONE: fixes for proxy-integration-tests on P, power outage, reviews [15:03] TODO: nessita's big branch review, take a look at dobey's twisted branch, use-restful-client branch [15:03] BLOCKED: a few of my upcoming branches are blocked on the nessita's branch landing, so more reviews appreciated: https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/unify-signal-broadcaster/+merge/87759 [15:03] REVIEWS NEEDED: https://code.launchpad.net/~alecu/ubuntu-sso-client/silence-dbus-logger/+merge/87815 [15:03] RE_REVIEWS NEEDED: https://code.launchpad.net/~alecu/ubuntu-sso-client/proxy-integration-tests/+merge/87538 [15:03] mandel: it makes them feel taller if they stand, while everyone else is sitting down :) [15:03] NEXT: gatox [15:03] alecu, i was the first one [15:04] DONE: Wasted a huge amount of time fighting with vmware to recover my devel win 7 machine.. Manage to do it, running tests for fix for bug 907511. Did some reviews. [15:04] TODO: Ensure bug 907511 is fixed. Move to proxy work. [15:04] BLOCKED: no [15:04] ralsina, go [15:04] Launchpad bug 907511 in ubuntuone-client "Windows: events from directories where no watch has been added should be ignored (affects: 1) (heat: 31)" [High,In progress] https://launchpad.net/bugs/907511 [15:04] mandel: already posted ;-) [15:04] jejejeje this was a mess [15:04] ralsina, che, you jumped the q! [15:04] gatox, doh [15:04] mandel: sorry! [15:04] dobey, go [15:04] λ DONE: dirspec MIR, mostly done with gireactor [15:04] λ TODO: write proposal for u1 packageset, finish gireactor work, start music store/libsyncdaemon work [15:04] λ BLCK: gireactor/gtk2reactor went boom. [15:05] any comments anyone? :P [15:05] gatox, in fact I said "me too" a few seconds before you said "me"!!!! [15:05] dobey: went boom? [15:05] twsited and python hate me. [15:05] alecu, ahhhh but that wasn't for the standup! so doesn't count :P [15:05] dobey: you started it! [15:06] ralsina: yes, was working fine. i refactored (split the code into a base that both derive from), and all of a sudden it no longer works right [15:06] lol [15:06] ralsina, I can confirm what dobey says... [15:06] ralsina, I will anyway take a look at the code to see if I can spot anything obvious. [15:07] dobey, alecu; I believe you. I also know it's probably your fault. Or your computers are nondeterministic now ;-) [15:07] dobey, the twisted reviewers requested the refactoring? [15:08] alecu: yes, because most of the code between the two reactors is exactly the same [15:09] ralsina: well i am not fond of python using 2.9G of RES for no apparent reason :) [15:09] ralsina, I have a problem with configglue on windows, which version are we suppose to me using, I know you had already this problem [15:09] mandel: we are using release, IIRC [15:10] should be 1.0 i guess [15:10] there haven't been any changes to it since then [15:10] ack [15:11] mandel, did you see my reply to your latests comment in my branch? [15:11] mandel, also, let's try to move forward the review of nessita's branch, since my next branch is blocked on that. [15:13] * gatox lunch...... brb === gatox is now known as gatox_lunch [15:17] alecu, which is nessitas branch? [15:17] alecu, and, no I did not see you comment, let me take a look [15:17] mandel, it's in my standup notes above [15:18] * mandel reads [15:19] alecu, nessitas branch has a needs fixing from you, right? [15:19] alecu: +1 on silence-dbus-logger [15:20] mandel, a small one, while I finish the whole review. But it's a huge branch so more eyeballs are welcome. [15:20] alecu, sure [15:20] mandel, also, that branch is a big help (but a big blocker too) for upcoming proxy branches. [15:21] mandel, so it should be our priority right now to help get it landed. [15:21] mandel: btw, what'd you want with dirspec? [15:21] alecu, I just looked at https://code.launchpad.net/~alecu/ubuntu-sso-client/silence-dbus-logger/+merge/87815 ralsina was faster, yet I agree with dobey that is a good idea to move that to the DBusTestCase in ubuntuone-dev-tools so that we clean all the projects [15:22] make sense [15:22] dobey, I just did not have it and wanted to install it (dirspec) [15:22] oh [15:22] apt-get install python-dirspec :) [15:22] dobey, windows [15:22] oh [15:24] mandel: I don't agree with moving that to DBusTestCase: it should be done on a testcase-by-testcase basis, so we don't mask possible problems. [15:24] mandel, it makes sense on the txsecrets tests, since they are testing two versions of the same API but that receive different parameters [15:25] mandel, so it makes sense to silence it there. But I believe we should not be doing it globally. [15:25] alecu, well, it seems to be useful code that might be reused by other tests [15:27] alecu: you're going to mask possible problems regardless of where you put it, and copying the code around just makes it worse :) [15:28] dobey, well, I disagree with this case. It should be exceptional to mask this kind of errors. [15:29] alecu: we could put the bulk of the code in DBusTestCase, and just require test cases that want to use it to explicitly call self.block_dbus_logging_crack() or something [15:29] hahaha [15:29] but whatever. i think we shouldn't do it anywhere, and instead just trap them all and log them to a file [15:30] i'm sure there are worse issues with some of our tests right now, so i don't really feel like arguing about it :) [15:31] dobey, arguing is part of the human nature, embrace it! [15:33] DiegoSarmentero, dobey, ralsina, nessita: mandel has just volunteered to give a small "bzr pipeline" lesson this friday, so we end up the cycle of megabranches. [15:34] what mega branches? [15:34] alecu: I have already had his explanation about pipeline, but happy to get a refresh :-) [15:34] alecu great! [15:34] let's wait till nessita returns from her doctor appointment so we can agree on a time [15:34] dobey, the >5k-line ones [15:35] dobey, like the one I mentioned on my standup notes. [15:35] i don't think pipeline itself really helps with that does it? it's not like bzr understands the logic in the code, and how to split it appropriately [15:36] and reviewing pipelined changes is a pain [15:37] dobey i think it doesn't hurt to listen what mandel has to tell us... maybe it can help with some cases [15:38] dobey, it makes the splitting easier, but of course is the developers responsability to split the code in a sensible manner :P [15:38] dobey, well, as long as pipelining helps with making sets of smaller branches, I'm willing to give it a try. [15:38] and some branches just have to be that big [15:39] ok, it's lunchtime for me. === alecu is now known as alecu_lunch [15:39] eh [16:09] alright, bbiab; lunch and errands === alecu_lunch is now known as alecu === gatox_lunch is now known as gatox [16:17] alecu, ralsina, nessita I did a first pass to https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/unify-signal-broadcaster/+merge/87759 [16:17] that is as far as I got before I started confusing the signal names [16:31] mandel, lol [16:50] i'll be back in a while.... i need to take my cat to her doctor appointment [16:54] * nessita is back! [16:54] mandel: what did you confused with what? :-) [16:54] mandel: I removed a lot of code to try to ease the reading :-D [16:55] @ping [16:55] pong [16:55] nessita, yeahm I was confusing the names because of the amount of them [16:56] nessita, I did a review, I think there is some code that we can remove since is a little repetitive [16:58] mandel: such as? (+1 to remove code) [16:58] alecu: what did you need for to agree on? :-) [16:58] nessita, it in the review :) [16:58] mandel: ack [17:00] ralsina: got confirmation from the agency for brian's plane ticket, I will add his details to the wiki page [17:00] ralsina: so, I guess alecu and facundo should be booking now [17:00] nessita: awesome [17:00] nessita: writing email! [17:00] nessita, I don't understand "what did you need for to agree on?" [17:00] alecu: I read the backlog, and at some point you said [17:00] (12:34:34 PM) alecu: let's wait till nessita returns from her doctor appointment so we can agree on a time [17:02] "nessita: mandel has just volunteered to give a small "bzr pipeline" lesson this friday, so we end up the cycle of megabranches." [17:09] there was a power micro-cut [17:09] alecu: sorry, I did not see any answer from you due my connection getting stuck [17:09] "nessita: mandel has just volunteered to give a small "bzr pipeline" lesson this friday, so we end up the cycle of megabranches." [17:09] my last messages were: [17:10] (02:01:45 PM) nessita: alecu: 47.7C sensación termica is burning my brain [17:10] alecu: how will bzr pipeline solve the mega branches? [17:11] nessita, by making it easier to have smaller branches that depend on each other, and to pull changes from previous unlanded branches [17:11] nessita, it's a tool to make smaller branches, not a magic bullet [17:11] alecu: right, but the problem with this kind of big branches is that I can't build a branch smaller that will not break trunk [17:12] alecu: so, if I make smaller branches, we'll have trunk broken for the period of time until the next one lands [17:14] nessita, perhaps we can do the review on the smaller branches, and only land the changes from the last branch [17:14] alecu: can be, yes... [17:17] nessita, anyway, I heard that bzr pipeline is a nice tool for these cases, and I could certainly use it for my latests branches, so I talked mandel into doing a small session on it, this friday. [17:18] alecu: +1 [17:18] nessita, do you agree with the comment in the branch? [17:18] alecu: why this is not ok? "Provides an utility" [17:19] nessita, whether you use "a" or "an" depends on the sound of the following word. [17:19] nessita, "an umbrella", "a utility method" [17:20] nessita, alecu: we can do reviews on all the pipeline and land it quickly. 7 reviews of 1000 are easier than 1 review of 7000 [17:20] mandel: a little... though you have some valid points about duplication in tests, I'm not complete convinced is worth the effort of unifying that. But I will certainly try and see if what I can do is acceptable. Regarding the "d.addCallback(lambda a: reply_handler(*a))", if you follow up the code where that is used, you will always, 100% if the times, get a tuple there [17:21] alecu: you sure about that? I thought the rule was "an" always before a word that starts with a vowel [17:22] ralsina: you sure? even when the changes are related? how can you keep track that you're modifying everything that needs to be modified? I mean, you see that something should fixed and is not, and you can not tell either if the fix is coming on another branch or is something that was forgotten... [17:22] dobey: would you teach me some English please? when shall we use "an" instead of "a"? [17:22] nessita, yes, I looked it up: http://answers.yahoo.com/question/index?qid=20070517154920AAABFVA [17:22] alecu: thanks for the link [17:23] nessita: I suppose it depends on the branches, really [17:23] ralsina, re: the "7 reviews of 1000", also, those 7 reviews could be done by different people. [17:23] ralsina: yes... I agree long branches sucks. Big time. But on the other hand, we need to do some serious refactorings... so yes, I guess is a case by case [17:24] nessita: lol :) [17:24] nessita: if it's really a branch that can not be split in any meaning ful way, pipelines are not going to help [17:24] dobey: lol? it wasn't a joke :-) [17:24] nessita: "an" is used when the following word starts with a vowel sound, and "a" used when it starts with a consonant [17:24] dobey: how do you define vowel sound? [17:25] why utility does not start with a vowel sound? :-) [17:25] it does [17:25] or it doesn't [17:25] dobey: ? [17:25] uhm, sorry, i am not an english teacher; vowel sound is the wrong term [17:26] nessita: http://owl.english.purdue.edu/owl/resource/540/01/ [17:26] a + singular noun beginning with a consonant sound: a user (sounds like 'yoo-zer,' i.e. begins with a consonant 'y' sound, so 'a' is used); a university; a unicycle [17:27] nessita: so the initial u is often preceded by "a" instead of "an" if it sounds "iu" [17:27] "yuniversity", "yunicycle" [17:27] heh [17:28] right. u is special because the long pronounciation of it is like y [17:28] but it's "an hour". [17:28] english sucks [17:28] and y is special because sometimes it is a vowel :) [17:28] mandel: got my reply? [17:28] it's certainly pronounced like two spanish vowels in "yuniversity" [17:29] ralsina: it's all the fault of the british [17:30] nessita, I'm the last two files of the megareview. [17:30] i think i'll read some megatokyo [17:33] alecu: you are an angel [17:35] nessita, my father is "Miguel Ángel", I'm just an incubus. [17:35] lol [17:36] nessita, no.. [17:36] mandel: repeating now: [17:36] mandel: a little... though you have some valid points about duplication in tests, I'm not complete convinced is worth the effort of unifying that. But I will certainly try and see if what I can do is acceptable. Regarding the "d.addCallback(lambda a: reply_handler(*a))", if you follow up the code where that is used, you will always, 100% if the times, get a tuple there [17:36] got it now [17:37] mandel: already working on the decorator for logging signals [17:39] mandel, so, is my answer enough in this MP? https://code.launchpad.net/~alecu/ubuntu-sso-client/proxy-integration-tests/+merge/87538 [17:40] nessita, also, can you please re-review that branch? [17:40] nessita, I added a reply to your comment too. [17:41] alecu: yes, sure! [17:42] nessita, dobey: are we having some issue with our tarmac? [17:42] not that I know of [17:42] alecu, yes it is, I'm looking a second time, but it looks ok in a first pass [17:42] are we? [17:42] i don't know of any :) [17:42] dobey, nessita: it seems it's not picking up this branch to do the merging: https://code.launchpad.net/~alecu/ubuntu-sso-client/silence-dbus-logger/+merge/87815 [17:42] alecu: i'll look [17:43] thanks [17:44] looks like something crashed it, and left a lock held [17:45] * alecu sees and then hears the thunderstorm [17:45] alecu: should grab it on the next run in ~5 min [17:46] damn climate. I should have gotten into the pool before lunch :-( [17:46] alecu: if it's not merged in ~10 min, ping me again [17:46] dobey, great, thanks. [17:46] ack [17:49] huzzah [17:49] now we can fix all the usage of the xdg hacks from sso, to use dirspec [17:52] nessita, Needs Fixing!!!! [17:52] alecu: another one? :-) [17:53] nessita, very small fixes, really. [17:53] alecu: I already fixed and push you first batch, so I will do that shortly [17:53] cool [17:53] * alecu now tries to focus on the GI branch [17:57] focusing is hard :-/ [17:58] ok, EOD for me, later! [17:59] mandel, bye [18:00] ralsina: brian's flight details added to wiki [18:09] meh [18:10] alecu: btw, http://twistedmatrix.com/trac/ticket/4558#comment:15 is the trac review commentary [18:10] looking [18:22] ralsina: ping [18:25] ok guys, I will have a really quick lunch, brb [18:31] dobey, have you seen this in the trial log? exceptions.AttributeError: 'GIReactor' object has no attribute '_POLL_DISCONNECTED' [18:32] http://pastebin.ubuntu.com/799677/ [18:32] hmm, no, i haven't seen that [18:33] grumble [18:33] dobey, it keeps being written to the log while the memory is chewed up, I found out by stracing it. [18:35] ah [18:35] people.... i'm having power issues too...... [18:35] gatox, damn power plants. I didn't have any today... [18:36] * alecu crosses fingers [18:36] alecu, nessita left for something or power issues for her too? [18:36] alecu: thanks! [18:37] dobey, you are welcome! [18:37] gatox, she said "really quick lunch" [18:38]