[08:09]  * rye_ logs out for reboot
[08:28] <JamesTait> Happy Friday, folks! :)
[08:49] <rye_> sweet, somebody stole my IRC nickname
[08:54] <rye> looks like i got it back
[08:57] <mandel> rye, are you sure it was not you :)
[08:58]  * mandel mandel
[08:59] <rye> mandel: http://ubuntuone.com/5TVaiFtH9sddWDWTnLwnRT
[08:59] <rye> mandel: running autoupdate binary directly
[09:00] <mandel> rye, ok, looking
[09:00] <mandel> rye, uhhh did we deploy that!
[09:00] <mandel> ???
[09:00] <rye> mandel: yep, 3.0.0 official
[09:01] <mandel> rye, la puta!!! ok, we have to make a new installer, is probable that brian had the trial install when he built it
[09:01] <rye> maaan, my windows vms became so slow it is painful
[09:01] <mandel> rye, is just a small mistake, but makes us look kind of bad :)
[09:01] <mandel> rye, do you have a bug number for that?
[09:01] <rye> mandel: well, it definitely raised a wtf look on my face
[09:01] <rye> mandel: filing one
[09:02] <mandel> rye, please add briancurtin as the person to take care of it, I'll talk with him and ralsina to fix it asap
[09:08] <rye> mandel: bug #986078
[09:08] <mandel> rye, thx!
[11:09] <gatox> good morning!
[11:11] <mandel> gatox, morning, me needs you :)
[11:12] <gatox> mandel, what happend? :P
[11:12] <mandel> gatox, can you please approve the following: https://code.launchpad.net/~mandel/ubuntuone-dev-tools/fix-runtests/+merge/102828
[11:12]  * gatox looking....
[11:12] <mandel> gatox, also, cna you confirm that with the new changes that we have for lp:~urbanape/ubuntu-sso-client/initial-darwin-port we have a segmentation fault?
[11:13] <gatox> mandel, yes..... i confirm that yesterday..... and also a lot of tests fail in windows because an api change..... do you want to see the trace.... i already have them in a paste
[11:13] <mandel> gatox, I think is because glib and qt are being imported at the same time which makes the code import versions 2 and 3 of glib/gtk which brakes things :)
[11:13] <mandel> gatox, no need, as long as it happens to both, I'm ok :)
[11:14] <gatox> mandel, i'm getting this for linux: http://paste.ubuntu.com/937410/ this for windows: http://paste.ubuntu.com/937423/
[11:14] <gatox> mandel, ahhhh :P ok
[11:14] <mandel> gatox, if you approve the ubuntuone-dev-tools we will have that in jenkins too, I recommend you set you irc client to listen for u1-jenkins in u1-internal
[11:15] <gatox> mandel, roger that
[11:15] <mandel> gatox, in xchat is under prefs->alets->nick names to always highlight :)
[11:15] <gatox> good
[11:16] <gatox> mandel, done
[11:16] <mandel> gatox, the approval or the highlight?
[11:18] <gatox> mandel, highlight
[11:18] <gatox> mandel, and now the approval
[11:19] <mandel> gatox, superb!
[11:21] <mandel> gatox, and since is your review day, can I have a review for https://code.launchpad.net/~mandel/ubuntuone-dev-tools/squid-errors/+merge/102825 and for https://code.launchpad.net/~brian.curtin/ubuntuone-client/run-tests-buildout-cleanup/+merge/102394
[11:21] <gatox> mandel, jeje i can't say no :P
[11:22] <mandel> gatox, I know, and also the good thing is that you are the first one usually to arrive :)
[11:33] <mandel> gatox, cna you in the mean time look at the one I gave you from brian?
[11:34] <gatox> mandel, i'm reviewing the both branches you gave me
[11:34] <mandel> gatox, sweet!
[11:39] <gatox> mandel, +1 to your branch
[11:40] <mandel> gatox, great :)
[11:40] <mandel> gatox, when that lands I'll be able why we cannot build on L u1-dev-tools
[11:43] <mandel> gatox, this one https://code.launchpad.net/~mandel/ubuntuone-dev-tools/params-runtests/+merge/102836 passes the parameters to runt-tests and skipslint :)
[11:43] <mandel> gatox, if you do me a quick review I would be very very happy
[11:44] <gatox> mandel, ok, on it...... i was looking at brian's branch making sure that doesn't brake already installed system to run-tests...... but i should install a new vm to fully test that :S
[11:47] <gatox> mandel, ohhhhh the lovely GOTO jeje..... +1 on the last branch
[11:48] <mandel> gatox, yeah... batch, beautiful!
[11:49] <gatox> yeyyyyy..... i already have a cleanvm..... i can fully test brian's branch
[12:34] <mandel> gatox, I'm off to have lunch
[12:34]  * mandel lunch
[12:34] <gatox> mandel, ok
[12:44] <ralsina> rye, mandel: we have a license for it, and yes, that means we have to do a new build and sign it. Ok, let's take it as an opportunity to sign the autoupdater while we are at it.
[12:44] <ralsina> and good morning
[12:44] <ralsina> thisfred: you up already? I want to give you some Qt lessons :-)
[12:45] <gatox> ralsina, good morning
[12:47] <rye> ralsina: from PyQt4.QtLesson import QtSome
[12:47] <ralsina> rye: hehe
[12:48] <thisfred> ralsina sorry yes am up :)
[12:48] <ralsina> thisfred: ok, here's the deal in 6 lines
[12:48] <ralsina> 1) Do the .ui using designer (and add it to the repo)
[12:48] <thisfred> I did the first half :)
[12:49] <ralsina> 2) compile the .ui using pyuic4 from setup.py and don't add the .py to the repo.
[12:49] <ralsina> 3) In your own code, import the generated .py and use setupUi to give you UI
[12:49] <thisfred> I did the first half of that as well :)
[12:49] <ralsina> 4) Then connect stuff using signals and slots
[12:49] <ralsina> 5) ...
[12:49] <ralsina> 6) Profit!
[12:50] <ralsina> thisfred: I have a tutorial on how to do most of this stuf here: http://lateral.netmanagers.com.ar/stories/BBS47.html
[12:50] <thisfred> so what I did (wrong) was throw away the .ui and add the generated code to the repository
[12:50] <thisfred> neat! thx
[12:52] <ralsina> thisfred: you can re-generate the compiled .py manually after you touch the .ui and add it to the repo, too
[12:52] <ralsina> thisfred: but that's bad practice because mybe future versions of the ui compiler are better
[12:52] <thisfred> ralsina, right
[12:52] <ralsina> Quoting myself from that tutorial: "Let me put that in these terms: IF YOU EDIT THE UI FILE (WITHOUT USING DESIGNER) OR THE GENERATED PYTHON FILE YOU ARE DOING IT WRONG! YOU FAIL! EPIC FAIL!. I hope that got across, because there is at least one tutorial that tells you to do it. DON'T DO IT!!!!"
[12:52] <thisfred> so the makefile should take care of generating the code?
[12:52] <ralsina> thisfred: right
[12:53] <ralsina> thisfred: you could do a Makefile or add something in setup.py like we have in u1cp
[12:53] <urbanape> mandel, gatox: yeah, I'll get to the windows API changes this morning.
[12:53] <thisfred> the reason I did not do that was, it lives in u1db for now, and I don't want to pollute that with qt builddeps
[12:53] <urbanape> hi, all
[12:53] <thisfred> but I still want the example code to be runnable
[12:53] <gatox> urbanape, ok
[12:53] <ralsina> thisfred: right
[12:54] <ralsina> thisfred: then maybe adding the compiled code for the time being is not horrible
[12:54] <ralsina> thisfred: alternative: load the .ui file dynamically
[12:54] <thisfred> ok, sounds like  a decent compromise
[12:54] <ralsina> thisfred: and sidestep the whole issue. It's not hard
[12:54] <ralsina> thisfred: it does make it slightly harder to package for windows though
[12:54] <thisfred> ok, that sounds even better, is that in the tutorial?
[12:55] <ralsina> thisfred: I can find you an example
[12:55] <thisfred> that would be awesome
[12:56] <ralsina> thisfred: https://pastebin.canonical.com/64666/
[12:56] <ralsina> thisfred: sorry about the comments in spanish but it's from a book I was writing ;-)
[12:57] <ralsina> thisfred: it assumes the .ui file is in the same folder as the .py that's loading it. Adjust accordingly
[12:57] <ralsina> thisfred: after you do that, all the ui components  are in self.ui.object_name and such
[12:59] <ralsina> thisfred: and that's it.
[13:00] <thisfred> thanks!
[13:02] <thisfred> Good for my technical Spanish :)
[13:02] <thisfred> cargamos los iconos DEL DIABLO!
[13:03] <thisfred> I find putting that at the end of every sentence spices things up a bit.
[13:06] <dobey> hmm
[13:16] <ralsina> thisfred: there's a lot more where that came from: http://nomuerde.netmanagers.com.ar/5.html
[13:17] <ralsina> thisfred: it's a media player, so you will like it ;-)
[13:18] <thisfred> it's not quodlibet, so it has one thing going against it :P
[13:18] <thisfred> heh, the gui is the easy part. I only partly agree with that :)
[13:19] <ralsina> thisfred: hehe
[13:19] <ralsina> thisfred: it's the most minimalistic media player humanly possible. It has one icon and you can only listen to one radio station by default :-)
[13:21]  * dobey wonders when u1db server will be widely usable, and when android u1db will be ready
[13:22] <dobey> eww
[13:23] <dobey> la GUI es la parte no es tan facil
[13:27] <ralsina> dobey: it depens on what GUI and what not-GUI it is
[13:27] <ralsina> dobey: but it's easiest to do a functioning GUI than a functioning backend for almost everything
[13:27] <ralsina> dobey: although a good frontend is harder than most good backends
[13:27] <ralsina> dobey: so, frontend has much more variance
[13:28] <dobey> it's very easy to make a very bad GUI, yes
[13:28] <dobey> but it's also very easy to make a very bad back-end
[13:28] <dobey> making a great GUI is very hard. and harder if you're doing it in English and you're not a native speaker
[13:28]  * ralsina is guessing  thisfred's UI is going to be easier than u1db ;-)
[13:29] <ralsina> dobey: that's why I only use icons in sign language
[13:29] <dobey> yeah, demo app doesn't need great UI
[13:29] <thisfred> yeah, especially since the ui is not the point of the example code at all
[13:29] <dobey> it needs great comments
[13:29] <thisfred> It does :)
[13:29] <ralsina> dobey, thisfred: it needs doctests
[13:30] <ralsina> so it can have truthul comments
[13:30] <ralsina> truthful
[13:30] <thisfred> ralsina: meh, I think doctests are good for API, I find them annoying in docstrings
[13:31] <thisfred> but I'll keep an open mind, maybe they'll come in handy
[13:31] <dobey> doctests are good for stabbing yourself in all your limbs
[13:33] <ralsina> haha
[13:33] <ralsina> doctests that are not exhaustive but are good examples, are awesome
[13:37] <dobey> what you need, is a lintian tool which verifies that everything is properly documented
[13:38] <dobey> of course, python also makes that extremely hard to do 100%
[13:41] <ralsina> dobey: there is no tool in the universe that can verify anything is properly documented. But I understand what you mean.
[13:41]  * mandel back
[13:42] <dobey> well, not necessarily understandably documented, but all inputs/outputs/exceptions are in the documentation for a function. and the spelling/grammar is reasonably correct
[13:42] <mandel> ralsina, briancurtin ping
[13:42] <briancurtin> mandel pong
[13:42] <ralsina> mandel: pong
[13:43] <dobey> mandel: what is all this runtests-params nonsense?
[13:43] <mandel> ralsina, briancurtin, got a very cure bug for you, is bug bug #986078
[13:43] <mandel> briancurtin, ralsina, and that was deployed to the users :)
[13:43] <briancurtin> uh
[13:43]  * ralsina looks at the mail sending brian the license key. Asks for that to be added to the build docs
[13:43] <gatox> briancurtin, hi! i'm trying to test this https://code.launchpad.net/~brian.curtin/ubuntuone-client/run-tests-buildout-cleanup/+merge/102394   i've installed the buildout in a clean vm, and then merge your branch with the one of u1-client inside parts..... how i should run the tests to test this branch?
[13:44] <dobey> heh
[13:44] <briancurtin> mandel, ralsina: i definitely applied that license, so i'm not sure if it went away or what the deal is...looking now
[13:44] <ralsina> briancurtin: ack, please take  a look. But don't worry too much.
[13:44] <dobey> bitrock's "we should respond within 12 hours" thing doesn't hold very well does it :(
[13:44] <ralsina> dobey: about the version logic?
[13:44] <mandel> dobey, so, the run-tests of ubuntuone-dev-tools was not working (not finding python) and was not forwarding the parsm to u1trial, which mean we could not pass the reporter and get nice reports when running the tests on jenkins
[13:44] <ralsina> dobey: they have been responsive for us in the past
[13:44] <dobey> ralsina: about support requests, but that's what i asked for support on yeah :)
[13:45] <ralsina> dobey: ack
[13:45] <mandel> dobey, so now you can do run-tests /skip-lint --reporter=subunit
[13:45] <mandel> dobey, and we have the tests of ubuntuone-dev-tools in jenkins like the other projects /cc ralsina
[13:45] <dobey> mandel: oh, that seems rather nasty
[13:45] <mandel> dobey, is fixed, it was a very simple fix in the .bat
[13:46] <mandel> we can now trust that jenkins will tell us when we broke u1-dev-tools on windows
[13:46] <dobey> mandel: yes i saw the branch. but i mean that fix seems nasty
[13:46] <mandel> dobey, oh, yes, is batch, with gotos and all that..
[13:48] <mandel> ralsina, briancurtin, I would not be to worried, but is something to consider for whenever the .exe comes out of jenkins :)
[13:48] <dobey> we probably need some better magic for that
[13:48] <dobey> oh well
[13:48] <briancurtin> gatox: so if you already have the lp:~brian.curtin/ubuntuone-windows-installer/buildout-env branch merged, make sure you run "env.bat" from the devsetup folder. that sets the path and some other things up. then in u1client you just run "run-tests.bat" and it should work
[13:48] <gatox> briancurtin, great..... i'll test that
[13:50] <mandel> dobey, I wonder how.. but is the runtests.bat, so I expect some ugliness in there as long as it is just in there
[13:50] <mandel> ralsina, 1-1 whenever you want :)
[13:52] <mandel> briancurtin, FYI we have u1-dev-tools in jenkins, I did some hacks to get run-tests.bat there working, would be nice if at some point you can dome some of your magic in that one too :)
[13:53] <briancurtin> mandel: what's the issue there?
[13:54] <mandel> briancurtin, well, there I had to fix the way it finds python and pass parameters to the u1-trial, maybe doing the same approach you have done in the buildout branches is better
[13:55] <briancurtin> mandel: can i see what changes you had to make?
[13:55] <gatox> briancurtin, wow! everything working... nice branch!
[13:55] <gatox> briancurtin, +1 approve
[13:55] <briancurtin> gatox: cool, thanks!
[13:56] <mandel> briancurtin, for the u1-dev-tools, yes here is a diff: https://code.launchpad.net/~mandel/ubuntuone-dev-tools/params-runtests/+merge/102836
[13:57] <gatox> dobey, (are you in your tests day?) can you review this one? https://code.launchpad.net/~brian.curtin/ubuntuone-client/run-tests-buildout-cleanup/+merge/102394   i already run this in a working env... and in a clean env installing everything using the buildout
[13:57] <mandel> briancurtin, approved: https://code.launchpad.net/~brian.curtin/ubuntuone-client/run-tests-buildout-cleanup/+merge/102394
[13:57] <mandel> gatox, I did that one too, so dobey  no need
[13:58] <gatox> dobey, no need..... mandel already approve it
[13:58] <gatox> that! :P
[13:58] <thisfred> ralsina: pushed the changes so the ui is loaded  dynamically. Now 578% nicer!
[13:58] <thisfred> thanks!
[13:59] <ralsina> thisfred: just doing my job! :-)
[13:59] <mandel> gatox, dobey, would be nice if you take a look at https://code.launchpad.net/~brian.curtin/ubuntu-sso-client/run-tests-buildout-cleanup and https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/run-tests-buildout-cleanup
[13:59] <gatox> on it!
[14:00]  * dobey needs to take a vacation
[14:02] <dobey> ralsina: https://launchpadlibrarian.net/102670998/Captura%20de%202012-04-20%2008%3A00%3A00.png <- waht does that say?
[14:03] <ralsina> dobey: it's going to merge a UDF with an existing flder
[14:03] <ralsina> dobey: and what language is that???
[14:04]  * ralsina can kinda read it
[14:05] <dobey> ralsina: it's not spanish?
[14:05] <ralsina> dobey: nope
[14:05] <gatox> dobey, nop
[14:05] <ralsina> dobey: I would guess romanian
[14:05] <mandel> dobey, is catalan :)
[14:06] <ralsina> dobey: or esperanto
[14:06] <dobey> or catalan
[14:06] <ralsina> mandel: same thing ;-)
[14:06] <ralsina> mandel: made up languages all of them
[14:06] <mandel> ralsina, as old as spanish: http://en.wikipedia.org/wiki/Catalan_language#Middle_Ages:_origin
[14:07] <ralsina> mandel: which is also a made up language
[14:07] <dobey> just like all language
[14:07] <ralsina> mandel: we should all be speaking in the original indostanic
[14:07] <mandel> dobey, also, dpm is from valencia, so you should have been able to guess it :)
[14:08] <dobey> ralsina: hrmm, i guess that is a stock qt dialog we're creating via code?
[14:08] <mandel> briancurtin, bad news: https://jenkins.errormessaging.com/view/Windows/job/ubuntuone-client-windows-test/36/console
[14:08] <dobey> also, what a horrible set of choices that is
[14:08] <dobey> "yes/no/cancel"
[14:08] <ralsina> dobey: not sure, would need to check the code.
[14:09] <ralsina> dobey: very not-verbs
[14:09] <dobey> they're not woody at all. rather tinny in fact.
[14:10] <mandel> briancurtin, looks like it cannot find sso, I'll see if I can fix it
[14:12] <briancurtin> mandel: is that running the new run-tests.bat? i think perhaps this grew a bit too quickly, because im not sure how the new run-tests.bat will run without an environment setup from the buildout
[14:12] <briancurtin> mandel: and jenkins already has its own deps and python setup underneath it
[14:13] <briancurtin> oh yeah it is running the new one
[14:13] <briancurtin> ah i know why it fails
[14:13] <gatox> briancurtin, i tested your branches with a env without the buildout
[14:13] <mandel> briancurtin, do you need me to get in the machine to fix it?
[14:13] <gatox> and they work
[14:14] <urbanape> mandel: has your ubuntuone-dev-tools branch with the txwebserver landed?
[14:14] <mandel> urbanape, yes, is in trunk
[14:14] <urbanape> k
[14:14] <briancurtin> mandel: the run-tests.bat does SET PYTHONPATH=..\ubuntu-sso-client;. - so it overwrites the one jenkins setup
[14:15] <mandel> briancurtin, ouch!
[14:15] <briancurtin> mandel: that makes it much easier to run the default case on a local machine, but on jenkins we don't want to touch it
[14:15] <briancurtin> mandel: perhaps we add a --jenkins flag or something, which skips setting of the path?
[14:15] <gatox> mandel, are you sure your devtools is in trunk?? i've just run a buildout, etc..... and it keeps getting the txwebserver issue
[14:16] <dobey> gatox: it is in trunk. it is not in the tarball your buildout is pulling
[14:16] <gatox> ahhhhh it seems that the buildout is not downloading the proper dev-tools
[14:17] <mandel> gatox, http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-dev-tools/trunk/files/head:/ubuntuone/devtools/testing/
[14:17] <mandel> gatox, is in trunk :)
[14:17] <dobey> gatox: dirspec and devtools should be moved to bzr checkouts under [sources] rather than tarball downloads
[14:17] <gatox> mandel, yes yes..... sorry..... the buildout is downloading another thing
[14:17]  * dobey said this the other day
[14:17] <mandel> gatox, I'm going to kill you for this!
[14:17] <mandel> :P
[14:18] <dobey> mandel: you have to destroy the resurrection ship first
[14:18] <mandel> briancurtin, hm.. of a --jenkins or we can append the python path, can't we?
[14:18] <gatox> jejejeee
[14:18] <briancurtin> mandel: i think we'd want to prepend it, to make sure jenkins' stuff is there first
[14:19] <mandel> briancurtin, make sense, can you do a small branch + bug for this
[14:19] <briancurtin> mandel: yep, doing right now
[14:19] <mandel> briancurtin, and do the same for the other two branches
[14:19] <briancurtin> yep
[14:21] <mandel> urbanape, let me know if you need any help with your branch, the segfault is probably due to using glib3 and qt at the same time
[14:25] <ralsina> gatox: could you add 986068 to your queue, set to medium?
[14:25] <gatox> ralsina, ack!
[14:27] <gatox> briancurtin, approve: https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/run-tests-buildout-cleanup/+merge/102392
[14:27] <gatox> briancurtin, and this one only a small need fixing: https://code.launchpad.net/~brian.curtin/ubuntu-sso-client/run-tests-buildout-cleanup/+merge/102393
[14:27] <dobey> mandel: there is no glib3 :P
[14:27] <briancurtin> gatox: ah, merge error. i'll fix it
[14:28] <gatox> briancurtin, both tested in an env with and without the buildout, and everything is ok
[14:28] <mandel> dobey, well you know.. what ever is the new thingy
[14:29] <mandel> dobey, I'l terrible with those things..
[14:29] <dobey> mandel: well qt supports running the glib main loop. so it shouldn't be the cause of any segfaults. there's probably a deeper problem in the tests causing it
[14:31] <mandel> dobey, I'm seen that happening when you have a gi.repository import and import qt but I'm not 100% sure
[14:32] <dobey> mandel: it's possible, but i think that's not necessarily the cause, but merely what you see on top :)
[14:33] <mandel> dobey, yes.. we might need to look deeper. but for that branch makins sure that no gi and qt imports are done when they should not is ok
[14:37] <briancurtin> gatox: fixed your copyright header comment
[14:37] <gatox> briancurtin, great.... approving it
[14:54] <dobey> meh. next 10 minutes are the most agonizing
[14:57] <ralsina> briancurtin, gatox, mandel, dobey, thisfred, urbanape: standup in 4'
[14:57] <gatox> ralsina, ack
[14:57] <mandel> ack
[14:57] <ralsina> alecu is on a conference today
[15:00] <elopio> hey, good morning.
[15:00] <elopio> I have a question.
[15:01] <elopio> the control panel is the wizard and the thing where you connect/disconnect, remove devices, etc.
[15:01] <gatox> me
[15:01] <thisfred> me
[15:01] <elopio> how do you call that thing?
[15:01] <briancurtin> me
[15:01] <urbanape> me
[15:02] <ralsina> me
[15:02] <mandel> me
[15:03] <dobey> me
[15:03] <gatox> DONE:
[15:03] <gatox> Work on mac os (trying to figure it out how to run the tests easier, just using run-tests). Several reviews for mandel and brian. Start working on Bug #986068.
[15:03] <gatox> TODO:
[15:03] <gatox> Finish with Bug #986068, keep doing reviews (review day), try to improve the dev setup for mac.
[15:03] <gatox> BLOCKED:
[15:03] <gatox> No
[15:03] <gatox> thisfred, go
[15:03] <thisfred> DONE: u1db demo app UI iteration 1 TODO: add tags and done status to ui BLOCKED: no NEXT: briancurtin
[15:03] <briancurtin> DONE: got installer automation done, should be able to propose today along with a few other changes to make deployment easier (embedded CRT)
[15:03] <briancurtin> TODO: propose branches, keep an eye on how jenkins responds to changes
[15:03] <briancurtin> BLOCKED: none
[15:03] <briancurtin> NEXT: urbanape
[15:04] <urbanape> DONE: Work on darwin branch. TODO: Fix some tests and shared code. BLOCK: None. NEXT: ralsina
[15:04] <ralsina> DONE: kid's birthday, cake, reviews, alecu 1-1, administrivia, helped thisfred with qt, little coding, lotsa thinking (hey, it's my thinking month), reports TODO: reviews,  more thinking BLOCKED: tired brain, cake make belly hurt NEXT: mandel
[15:04] <ralsina> Oh, and weekly call
[15:04] <mandel> DONE: Fixed bug 985004 bug 986125  and bug 986144 which means that u1-dev-tools in on jenkins. Reviews, worked on a domain socket tcp-activation.
[15:04] <mandel> TODO: more tcp activation. 1-1 with ralsina.
[15:04] <mandel> BLOCKED: no
[15:04] <mandel> dobey, please
[15:04] <dobey> λ DONE: team call, bug #962860, review
[15:04] <dobey> λ TODO: reviews, backports, SRUs, u1db packaging/buildsys
[15:04] <dobey> λ BLCK: none.
[15:04] <dobey> also, the cake is a lie
[15:04] <ralsina> dobey: I beg to differ
[15:05] <mandel> and the spoon does not bend!
[15:05]  * ralsina is eating delicious, delicious lies
[15:05]  * thisfred wants lies
[15:05] <ralsina> thisfred: cake is good for you. There is a lie.
[15:05] <ralsina> So, comments?
[15:05] <gatox_mac> dobey, jejeej the cake is a lie! great game! :D
[15:06] <dobey> ralsina: actually, the organic oreo-like cookies i have are good for you. they have 2g of whole grains per serving, and plenty of antioxidants! :)
[15:06] <ralsina> "Roughly translates to "your promised reward is merely a fictitious motivator"."
[15:06] <dobey> yay green tea + sugar.
[15:06] <mandel> comment, fuck twisted and disconnections! turns out that the stupid domains sockets disconnect in a diff way..
[15:06] <ralsina> dobey: antioxidants are actually bad for you, according to latest medical literature
[15:07] <mandel> sorry, frack twisted
[15:07] <dobey> ralsina: government conspiracies are like that.
[15:07] <ralsina> mandel: goes to show "domain sockets have good support in twisted" is a cake
[15:08] <dobey> haha
[15:08] <ralsina> EOM it is then
[15:08] <dobey> and now. lunch.
[15:08] <dobey> bbiab
[15:08] <ralsina> mandel, mumble?
[15:08] <mandel> ralsina, there already :)
[15:10] <ralsina> mandel: 1 segundo
[15:38]  * gatox quick lunch..... brb!
[16:20] <urbanape> mandel: for the problematic names from ubuntu_sso/main/perspective_broker, should those just be explicitly skipped for pylint?
[16:20] <urbanape> all the ones with studly caps are signal names, not instance methods
[16:21] <urbanape> ah, I see
[16:21] <mandel> urbanape, yes, my comment should have been, skip all those complaints that you consider stupid :)
[16:21] <urbanape> I left that pylint exclusion in the windows module
[16:21] <mandel> urbanape, yes, you just have to move it and you should be fine
[16:22] <urbanape> I think I got it all. pushing.
[16:23] <mandel> urbanape, did you run the tests on linux, I can do it for you if you done have a vm close to you
[16:23] <urbanape> I haven't yet, but can
[16:24] <mandel> urbanape, ok
[16:27] <dobey> mandel: also please see my and sidnei's comments on https://code.launchpad.net/~mandel/ubuntuone-dev-tools/squid-errors/+merge/102825
[16:27] <mandel> dobey, ok
[16:36] <mandel> dobey, new version pushed, I added StringIO as the parent calss of Pipe since it implements a diff API, added \nstdout:\n%s and \nstderr:\n%s for the exception
[16:36] <mandel> dobey, I added the stdout and stderr to be more clear
[16:37] <mandel> urbanape, which revno?
[16:37] <dobey> mandel: cool
[16:37] <urbanape> just pushed. 959
[16:39] <dobey> mandel: and what about the extreme indentation in the diff, now on line 80 of the diff in launchpad?
[16:40] <mandel> dobey, my style, I can put it to the same level of the ( I have no problems
[16:41] <mandel> dobey, I read it better, but I suppose it might be hated by people
[16:41] <ralsina> thisfred: needsinfo on your branch
[16:43] <dobey> mandel: oh i see you do that now on the msg = line. also you apparently typed /n instead of \n :)
[16:43] <mandel> dobey, oh my! that is stupid, fixing
[16:44] <thisfred> ralsina looking
[16:44] <ralsina> thisfred: just suggested some style checking
[16:44] <ralsina> thisfred: other than that, it looks good
[16:44] <mandel> dobey, sorted and added more tranditional indentation
[16:44] <dobey> mandel: k, thanks
[16:45] <mandel> dobey, I guess this in the real purpose of rebase in git, hide how stupid one is at the end of the day :)
[16:45] <thisfred> ralsina: I use flake8 in vim, which says the code is good...
[16:45] <ralsina> thisfred: there is at least a missing docstring around line 224
[16:46] <ralsina> thisfred: also, __init__ should have docstring if it has arguments
[16:46] <thisfred> ralsina. Ah yes. Somehow flake8 strips that out. I wish I knew how to make it not do that
[16:46] <ralsina> thisfred: if pyflakes + pep8 gives different results, then let's not use lake8
[16:46] <ralsina> flake8
[16:47] <dobey> it's too flaky
[16:47] <thisfred> or send patches :)
[16:47] <mandel> urbanape, we have a segfault in the tests of that branch, so needs fixing again :(
[16:47] <mandel> urbanape, sorry
[16:47] <dobey> why didn't the flake8 people think of that? :P
[16:48] <mandel> ok, EOD for me, all have a great weekend!
[16:48] <dobey> mandel: can you wait until we get your squid branch approved?
[16:49] <mandel> dobey, sure, go ahead :)
[16:49] <thisfred> ralsina:
[16:49] <dobey> mandel: or will you be checking it? would like to get it in and see what's happening with lucid :)
[16:49] <thisfred> hmm, my pep8 also does not show that warning  anymore?
[16:49] <urbanape> mandel: yeah, that's okay, I hadn't gotten to that bit yet. Need to follow that up.
[16:50] <ralsina> thisfred: pep8 doesn't complain of that. I expected pyflakes to complain.
[16:50] <thisfred> ralsina: ah
[16:50] <dobey> thisfred: are you doing pep8 --repeat ?
[16:50] <thisfred> ralsina: it does not
[16:50] <thisfred> dobey: yeah
[16:50] <ralsina> thisfred: carp
[16:50] <dobey> hrmm
[16:50] <thisfred> pep8 should though.
[16:50] <dobey> pep8 is what complains about missing docstrings yes
[16:51] <thisfred> it's in pep8
[16:51] <ralsina> really? Ok then.
[16:51] <thisfred> is it an option?
[16:51] <dobey> no
[16:51] <dobey> not afaik anyway
[16:53] <ralsina> thisfred: silly question: are you sure you are checking that file? ;-)
[16:53] <thisfred> dobey: so pep8 should check for that, but it does not, nor does pyflakes
[16:53] <thisfred> pylint does
[16:53] <thisfred> ralsina: ran it with verbose, and checked all the files
[16:54] <dobey> mandel: can you also fix the indentation on line 13 in the diff?
[16:54] <ralsina> thisfred: ok
[16:56] <mandel> dobey, that one is correct, is as close as possible to the upper opening (
[16:56] <mandel> dobey, where yould you like it?
[16:57] <dobey> mandel: it's not correct. it should be +4 of the parent line afaict. if i open emacs and go to that line and press tab, it moves it way over to the left to where i'd expect it to be
[16:57] <dobey> mandel: next line +4 more spaces
[16:58] <dobey> is what i expect when i see a line end with an open paren
[16:59] <mandel> dobey, ok, is style and I'm not very worried about it :)
[17:01] <mandel> dobey, done
[17:03] <dobey> briancurtin: https://code.launchpad.net/~brian.curtin/ubuntu-sso-client/run-tests-buildout-cleanup/+merge/102393/comments/221740
[17:03] <dobey> mandel: cool, thanks
[17:04] <briancurtin> dobey: ah, screwed up merging it, i'll add it back in
[17:04] <mandel> dobey, let me know, I'm got a 'friend' waiting for me to get out of the office :P
[17:05] <dobey> mandel: ok, looks ok to me. i'll bug sidnei to review it again
[17:05] <mandel> dobey, sweet!
[17:06] <mandel> dobey, once that is in place we can debug the L building issue
[17:06] <dobey> yep
[17:06] <dobey> cheers
[17:06] <dobey> ralsina: installbuilder support email is sad
[17:06] <ralsina> dobey: ?
[17:07] <dobey> The AutoUpdate uses incremental integers as versions to avoid issues with version formats (specially dealing with formats like 4.0.1-0beta2).
[17:08] <dobey> so they only support that "version" being an integer :(
[17:09] <ralsina> dobey: so I was accidentally right all along
[17:10] <mandel> dobey, I really need to go now, but I'll follow the branch via phone
[17:11] <dobey> mandel: yeah, enjoy
[17:13] <briancurtin> dobey: that batch file header is now corrected
[17:13] <dobey> ralsina: i suppose the question is what size integer it uses.
[17:13] <dobey> briancurtin: cool. looking at it again
[17:13] <ralsina> dobey: if we limit ourselves to 3 version components, we could live even with 32-bit unsigned uint
[17:14] <ralsina> unsigned int, of course
[17:14] <ralsina> or just use date-based build numbers
[17:14] <ralsina> yyyymmddbb with bb being serials reset by day
[17:15] <dobey> well i was thinking just use a time_t value
[17:15] <dobey> which will hold us until 2038 at least
[17:15] <ralsina> that should have us covered until 4294967296 which means 4294 96 72 96
[17:15] <ralsina> or a time_t yes
[17:16] <elopio> briancurtin, ralsina, I'm late with windows testing this week. What's the most recent version?
[17:16] <ralsina> elopio: 3.0.0 release which we just rolled back
[17:16] <dobey> but i don't want to assume they didn't just use "int" in the code, so it's a signed 16 bit int or something
[17:16] <dobey> because then that would be a problem
[17:16] <ralsina> yeah
[17:17] <ralsina> well, we could set a 2^17 release number and see if it's < 1000
[17:17] <ralsina> or something like tha
[17:17] <dobey> heh, true
[17:17] <ralsina> t
[17:17] <dobey> well testing is easy, yeah
[17:17] <ralsina> ok, my blood sugar is crashing, need to get some food in me
[17:18] <dobey> was just thinking of how to automate generating the xml, and what to do incase we overflow it :)
[17:18] <dobey> buen provecho ralsina
[17:18] <ralsina> unix timestamp should work pending that test
[17:18] <dobey> yeah
[17:18] <ralsina> luckily python can generate unix timestamps on windows ;-)
[17:19] <dobey> well it doesn't matter if we "release" ubuntuone-windows-installer from linux all the time
[17:19] <dobey> which is basically how it's been
[17:19] <dobey> anyway
[17:19] <dobey> go eat
[17:22] <dobey> alecu: should https://code.launchpad.net/~mandel/ubuntuone-control-panel/auto-update-looping-call/+merge/81015 be rejected at this point? :)
[17:33] <gatox> ralsina, can you repeat me please how to reproduce this bug: https://bugs.launchpad.net/ubuntuone-control-panel/+bug/986068 ?? i'm not being able to
[17:34] <gatox> i'm already change the locale from the applicatoin and i get the dialogs in spanish when they are qt strings..... but i want to see that specific dialog
[17:34] <gatox> ralsina, i did it :P
[17:34] <gatox> nevermind
[17:41] <dobey> heh
[17:43] <dobey> gatox: i think i'd be more interested in getting the other issues fixed that i mentioned there
[17:44] <dobey> grr mandel
[17:44] <gatox> dobey, can you open a new bug for that..... because this bug report is actually wrong
[17:45]  * briancurtin lunch
[18:08] <gatox> dobey, did you open a new bug for that or do you want me to do it??..... from my point of view, is not directly related to this bug report, and i can fix it now, for the other issue i want some feedback from the user because something is missing in his machine or from qt translations
[18:09] <dobey> i haven't filed it yet
[18:09] <gatox> dobey, ok, no problem.... i'll do it
[18:10] <dobey> gatox: where are qt translations installed?
[18:11] <gatox> dobey, /usr/share/qt4/translations
[18:11] <ralsina> gatox: what was it then?
[18:11] <ralsina> gatox: missing qt translations from the system?
[18:12] <dobey> hmm
[18:12] <gatox> ralsina, i ask for some information to the user..... because in spanish the dialog is ok... so maybe he is missing some qt translations or qt translations are not complete, or something
[18:12] <dobey> what's the locale for catalan?
[18:12] <dobey> it's ca_ES right?
[18:13] <dobey> it seems there is no translation for catalan
[18:13] <dobey> in qt, anyway
[18:13] <ralsina> dobey: no idea.
[18:13] <gatox> i think that is probably the problem
[18:13] <gatox> qt translation are not "complete"
[18:13] <ralsina> dobey: it's "ca"
[18:13] <dobey> right
[18:13] <dobey> there's no qt_ca.qm here
[18:14] <dobey> no qt_sq.qm either
[18:14] <ralsina> dobey: ca, ca_ES, caES@valencia (WTF?)
[18:14] <ralsina> So, let's shift the bug to the catalan translation team or whatever
[18:14] <dobey> ralsina: well, you know, there are regional variations in spain :)
[18:15] <ralsina> dobey: I just had never seen the @ variation on l10n codes before
[18:15] <dobey> it's not like they all speak es_IT like they do in argentina
[18:15] <ralsina> dobey: their loss! -)
[18:15] <dobey> heh
[18:15] <ralsina> ca_ES@valencia|north
[18:16] <gatox> ralsina, here is my comment: https://bugs.launchpad.net/ubuntuone-control-panel/+bug/986068
[18:16] <ralsina> gatox: saw it
[18:16] <ralsina> gatox: looks like it's justthat Qt is not translated to catalan http://musescore-developer.685061.n2.nabble.com/Catalan-and-Qt-translation-td3840425.html
[18:17] <ralsina> gatox: and http://www.mail-archive.com/debian-bugs-dist@lists.debian.org/msg205958.html
[18:17] <gatox> ralsina, so.... which would be the proper answer? mark as invalid and leave a message saying this is a qt problem?
[18:18] <ralsina> find the team that should handle and assign, I guess
[18:18] <ralsina> we could do a workaround, but I refuse
[18:20] <gatox> that should be: "Ubuntu Translations Coordinators"?
[18:26] <ralsina> gatox: why not
[18:26] <gatox> ralsina, done
[18:26] <ralsina> dobey: yu know better? ^
[18:27] <ralsina> gatox: remember to put a nice comment saying "there seems to be a missing catalan translation for qt" or whatever
[18:27] <dobey> it's already assigned
[18:27] <gatox> ralsina, yes, already did that too
[18:27] <ralsina> gatox: awesome
[18:27] <dobey> eh
[18:27] <dobey> don't assign the control-panel task to them
[18:28] <dobey> assign the translations task to them
[18:28] <dobey> they can't do anything in cp itself either :)
[18:28] <gatox> dobey, ack
[18:28] <dobey> and we can probably delete the cp task
[18:35] <mandel> dobey, good question, I don't think so
[18:35] <mandel> dobey, I'll remove the Mp
[18:36] <dobey> mandel: cool
[18:36] <dobey> mandel: also, can you review the 2 branches from briancurtin that you "claimed" ?
[18:36] <mandel> dobey, sure
[18:36] <dobey> the pythonpath-jenkins ones
[18:38] <mandel> dobey, sorted, is an easy fix, we need to make sure that we don't step on the jenkins path
[18:39] <dobey> yep is trivial, that's why i approved them :)
[18:39] <dobey> but i didn't see you'd "claimed" them until after you left
[18:42] <mandel> dobey, I think I was proposed by someone else :)
[18:42] <briancurtin> i think that might have been my doing. i think i set mandel as the reviewer when i proposed the merge...im not sure if that's a good thing to do though
[18:42] <dobey> ah
[18:42] <briancurtin> i just figured since he was involved to set him in that manner
[18:42] <mandel> dobey, I don't like to claim them because it is blocking :)
[18:42] <dobey> mandel: good :)
[18:43] <mandel> briancurtin, from my point of view, it is a flaw in the workflow of launchpad, is a good idea but should not block things from landing, I'm made the same mistake before
[18:43] <dobey> briancurtin: it's usually better to just leave the default, except for special cases. trivial changes like this aren't special enough for it
[18:44] <dobey> mandel: it's not an LP problem. it's a user choice. by setting it to a specific person, you're saying you want to block on that person reviewing it :)
[18:44] <mandel> dobey, could be
[18:45] <mandel> I wonder if there is a twitter plugin for jenkins, it would more useful than the irc one..
[18:45] <ralsina> setting up a reviewer when proposing means "don't land until this guy sees it"
[18:45] <ralsina> tht's inherently blocking and 90% of the time a bad idea
[18:45] <briancurtin> especially for my trivial one-liner. i'll save the assigned reviewer for big stuff that needs blocking
[18:46] <mandel> well, since I'm not needed any more, I'm off to other japanese and some bad tv
[18:47] <dobey> mandel: there probably is, but it only twits using pictures of cats
[18:47] <mandel> dobey, lol
[18:48] <mandel> briancurtin, ralsina, if you see that jenkins stays for ever running some tests it might mean that it cannot find qt4reactor (it has happened before) so just kill the build and look at what went wrong
[18:48] <mandel> briancurtin, ralsina, the consoler does not give too much info
[18:48] <ralsina> mandel: ack
[18:48] <briancurtin> mandel: ack
[18:49] <ralsina> mandel: and how would we know it's stuck except going to the site and looking?
[18:49] <mandel> ralsina, you have no way.. I wonder if we can set a timeout for that
[18:49] <ralsina> mandel: right
[18:49] <mandel> ralsina, we should never be over 10 mins in tests
[18:49] <ralsina> mandel: I mentioned that. Let's give it a timeout.
[18:50] <ralsina> mandel: if jenkins doesn't have it, a python process that starts things and kills them in 15 minutes is trivial
[18:51] <mandel> ralsina, from a quick look it does not have it as a default config, maybe as an add on, can you add a bug and I'll take a look on monday?
[18:51] <ralsina> mandel: sure. Do we have a place for jenkis bugs?
[18:52] <ralsina> mandel: and -> https://wiki.jenkins-ci.org/display/JENKINS/Build-timeout+Plugin
[18:52] <mandel> ralsina, yep, plugin I can install it as soon as I start on monday
[18:52] <ralsina> mandel: awesome
[18:54] <mandel> ralsina, I might also look at how to run the squid proxy tests there too
[18:54] <ralsina> mandel: awesomer
[18:54] <ralsina> mandel: I saw a one-file http proxy that may be handy
[18:55] <ralsina> mandel: called throxy, pure python
[18:55] <elopio> ralsina: after uninstalling, shouldn't the C:\Program Files\ubuntuone folder be completely removed?
[18:55] <mandel> ralsina, at some point we should start adding more tests for diff proxy settings so that we can test as many scenarios as possible
[18:56] <mandel> ralsina, kind of integration tests
[18:56] <ralsina> elopio: no, it keeps a rolling backup of something
[18:56] <ralsina> elopio: never looked into it, really
[18:56] <ralsina> mandel: that's the future
[18:56] <ralsina> present is, it's 8PM in spain
[18:56] <ralsina> mandel: on a friday
[18:56] <ralsina> mandel: and you are still working
[18:56] <mandel> haha
[18:56] <ralsina> mandel: go away
[18:56] <ralsina> mandel: have a nice weekend ;-)
[18:56] <mandel> ralsina, and in 9  pm ;)
[18:57] <mandel> ok, laters!
[18:57] <ralsina> bye mandel!
[18:57] <elopio> :) bye mandel
[18:57] <elopio> ralsina: I'm just writing updating test case for uninstall, so the expected result should be to keep it or to delete it?
[18:58] <ralsina> elopio: I need to think about it a bit and do some tests
[18:58] <elopio> s/writing/(I need more chocolate)
[18:58] <ralsina> elopio: file a bug asking it to be removed, assign to me
[18:58] <elopio> ralsina: ok, thanks.
[18:58] <ralsina> elopio: but hold on that test for a bit, in case we want to keep it
[19:03] <urbanape> mandel: shouldn't that test (test_qtwisted) be skipped on Linux?
[19:03] <urbanape> it used to be test_windows
[19:04] <gatox> need to reboot...... brb
[19:06] <urbanape> gatox: I pushed a new rev (961) to my branch that should address the problems on Windows and the pylint stuff.
[19:07] <urbanape> also, you should be able to skip test_qtwisted on linux, since it used to be test_windows.
[19:07] <gatox> urbanape, awesome..... i'll run the tests now
[19:26] <gatox> urbanape, the tests are ok in windows..... about linux, i'm still getting the segfault... if qtwisted need to be skipped, shouldn't we modify the run-tests?
[19:26] <dobey> hrmm, i guess squid is just starting very slowly in the lucid buildd
[19:27] <urbanape> gatox: yes, probably.
[19:27] <urbanape> just a sec
[19:27] <ralsina> gatox: yes, the skip is set on run-tests, and if the tests were renamed, that needs to be adjusted
[19:28] <gatox> urbanape, thanks, let me know
[19:28] <urbanape> just pushed 962 which excludes test_qtwisted.py and test_pykeyring.py
[19:29]  * gatox testing....
[19:38] <gatox> urbanape, i'm running trunk tests...... because i have 6 failures and i want to check with trunk
[19:38] <urbanape> k
[19:38] <urbanape> I might have botched a bit of the windows stuff.
[19:38] <gatox> urbanape, i'm getting this: http://paste.ubuntu.com/938784/
[19:38] <gatox> in linux
[19:39] <urbanape> hmm, don't know about the proxy stuff.
[19:40] <gatox> urbanape, i'll rebranch and check......
[19:41] <gatox> grrrrr.... my internet connection is working like crap
[19:47] <gatox> urbanape, everything is ok...... it seems that the merge was wrong
[19:49] <gatox> urbanape, approving the branch! :D
[19:50] <gatox> urbanape, mmmm now i'm having some issues with tcpactivation on windows
[19:50] <gatox> urbanape, http://paste.ubuntu.com/938806/
[19:52] <urbanape> ugh.
[20:14] <gatox> urbanape, i'm in my EOD... but if you make changes to the branch, let me know and i'll run the tests to see if we can merge this today
[20:16] <urbanape> kk
[20:38] <ralsina> EOW for me
[20:38] <ralsina> have a nice weekend!
[20:38] <gatox> ralsina, you too!
[20:39] <briancurtin> enjoy, ralsina
[21:19] <dobey> alright. have a good weekend all