=== yofel_ is now known as yofel [08:28] morning all! [09:02] * mandel update reboot [09:43] verterok: morning [11:01] good morning! [11:27] mandel, i add a comment in the MP [11:28] gatox, ok, checking :) [11:28] gatox, god I hate that pyflakes does not have disable comments :( [11:29] yap [11:29] gatox, branch approved.. [11:29] mandel, thx [11:29] mandel, how was your weekend? [11:29] mandel, and hi! jeje [11:30] gatox, I might ask you for a review later, the windows tests are broken when I changed the API of the TCP activation in sso so I'm fixing and adding domain sockets in u1-client and u1-cp [11:30] gatox, weekend was good, went to buy furniture and not much more :P [11:30] gatox, I was at the gym on saturday worked out too much and was tired for the rest of the weekend hehe [11:31] mandel, jejejejee [11:33] gatox, how was yours, lots of kinet? [11:33] gatox, is it worth it? [11:33] gatox, I mean, is it fun to play etc.. 'cause I've seen you killing zombies.. and I would have not uploaded that video to the internet :) [11:34] mandel, lot of ninja coding, and on saturday we went to my house in carlos paz (perrito, nessita, matiasb and some others)..... in we ate a lotttttttt of asado! [11:34] jejeje [11:34] gatox, nice! [11:34] mandel, yes, the kinect worth it..... is really fun [11:34] gatox, I'd love to be there.. specially when my country is going down to hell.. putos politicos [11:34] mandel, also.... you can plug it to your computer, and make your own programms using the kinect [11:35] mandel, jejeje well i promise an asado if you come to cordoba :P [11:35] gatox, would be good to remove the need of a remote.. that way the dog will not distroy it [11:35] gatox, I'm jotted down that offer :) [11:36] mandel, ahhh my talk in pycon australia was rejected..... so...... not spain soon :( [11:36] mandel, i'm planning to take some vacations in japan :D [11:36] gatox, oh :( [11:36] gatox, I'm going to thailand this year, japan might be nexts holidays [11:37] mandel, when you go to visit some new places...... you hire some tourist guide or something..... or tourist package...... or arrange everything by your own? [11:38] * gatox is trying to decide which is more fun [11:38] gatox, I usually go crazy if I go alone, I usually book a hostal because there you are going to meet students and young people (means going out which is a good thing to do) and then once I have 'friends' I improvise :) [11:39] gatox, so for me, student hostals are the way to go [11:39] jejeej [11:39] yap..... i wanted something more like a tour for japan..... [11:41] gatox, I'd buy a guide and then use the train seems to be the best thing to do [11:42] yes..... probably [11:42] gatox, in my trip to thailand I'm getting there and booking everything from there but knowing a little before hand, for example I'll stay 3 days in the jungle in an elephant trip :) [11:42] mandel, wowwwwww [11:42] pictures! [11:43] gatox, sure, I'm even considering getting eye surgery done so I don't longer need the glasses [11:43] gatox, I don't want to get lost in the jungle loose then and get eating by predator :P [11:43] jejejeje sounds logic jejeje [11:55] gatox, so the ipc on u1-client is using mocker.. I'm very tempted to remove it so that I can add tests for domain sockets too, what do you think? [11:55] mandel, the rule is: if you are fixing/adding some tests where it contains mocker now..... REMOVE IT! [11:56] (whenever you can) [11:56] gatox, well, I think I do have to touch the tests because they are old right now.. :) [11:56] gatox, also, I made changes to the code and the tests pass which is not a very good sign.. [11:56] jejeje [12:17] Good morning [12:19] ralsina, morning! [12:19] Hola mandel, how's things? [12:20] Is the mac port done yet? [12:20] ;-) [12:21] ralsina, lol I wish, I'm fixing the windows tests and in the same branch adding support for domain sockets for u1-client and u1-cp which gets us a step closer :) [12:21] ralsina, buenas [12:22] mandel: ack, saw jenkins whining a bit ago, which is a good thing [12:22] ralsina, then, more fsevents code to increase tests and add integration [12:22] ralsina, yeah, this started late my friday (at least first time I got it) I'll try to set up the mac jenkins this afternoon (I have to play with my rooter, ouch!) [12:23] mandel: I just found out I can't use the mini yet because I have no mice [12:23] mandel: so I have to buy one and it's raining :-/ [12:24] ralsina, he, your new cat must be very good :P [12:24] mandel: como se llama el ave que incendia los campos mexicanos? [12:25] ralsina, ni idea, como? [12:25] mandel: (deci rapido) el pajaro quema maiz [12:25] rye: FWIW, fresh comments appeared under bug #879342 [12:25] Launchpad bug 879342 in Ubuntu One Servers "Got status code 302 or 401 using Ubuntu One" [Undecided,Triaged] https://launchpad.net/bugs/879342 [12:26] ralsina, jajajaa [12:26] ralsina, uf, tan malo como, como se dice autobus (colectivo) en aleman? [12:26] mandel: subanaprietesetrujenbajen? [12:27] ralsina, si :) [12:27] mandel: that one was old when I was not [12:27] ;-) [12:27] karni: slow network connection, timestamp checking. same thing [12:27] ralsina, last one, como se dice cuanto cuesta un pajaro en vasco? [12:27] mandel: you told me already [12:27] mandel: acuantolacacatua? [12:27] rye: Could we mark it as 'in progress' perhaps? Or let them know we're working on it. [12:27] ralsina, es cara la cacatua? [12:28] ralsina, I need to get new jokes :( [12:28] karni: in progress [12:28] rye: thanks [12:28] mandel: OTOH, the one I did about a song with diego in twitter right now? I am fairly proud of that one! [12:28] And enough jokes, let's fix a bug or two [12:28] :) [12:28] :P [12:28] hey ya! hey ya' all! [12:29] alecu, hi [12:29] * rye injects https://code.launchpad.net/~rye/ubuntuone-client/ignore-in-close-write-in-dirs/+merge/106612 - nothing to fix, just test under Ubuntu ^_^ [12:29] hola alecu! [12:29] rye: looking [12:30] because if i write an automatic test for this you will all hate me. Setting up ecryptfs directories in tarmac, managing mounts... [12:30] rye: is this the cause of all the weird eCryptfs errors? [12:30] or at least some of them? [12:30] rye: haha [12:30] ralsina: no, of only one of them: when you cd to ~/Private, syncdaemon explodes and won't synchronize anything until restart [12:30] rye: yes, we will not add tests for that :-) [12:31] ok, I'm off to have lunch [12:31] * mandel lunch [12:31] Anyone other than rye has branches for review I should look at? [12:32] rye, no need to do an integration test for this; but a unit test would be nice. [12:33] alecu: the thing is that i can't cause pyinotify to trigger this [12:33] rye, so, we should at least build a pair of tests that try to send the IN_CLOSE_WRITE and that check whether it was pushed or not. [12:34] ralsina, there are a few but I think I went over all of them.. [12:34] alecu: another thing - we don't seem to have any test that tests for filters like this [12:34] mandel: ok [12:34] rye, we should send a synthetic event. But don't worry, I can take care of those tests. [12:34] rye, thanks a lot for finding this and for your fix. [12:35] alecu: it has been sitting in LP for 6 months and today joshuahoover forwarded me a ticket with exactly this problem and I recalled I promised to get it in in some of my blog posts [12:35] alecu: so you will branch from that branch? [12:35] ralsina, right [12:36] alecu: ok, cancelling review [12:44] alecu: so should I unpropose the branch? [12:49] huh? [12:53] rye, I'm working on the test, then you can add it from my branch so the branch with the fix is still the one you did. [12:53] ralsina: https://code.launchpad.net/~dobey/poauth/test-runner/+merge/105874 still has no reviews :) [12:53] dobey: on it [12:55] alecu: oh, shiny! I get all the credits! [12:56] rye, and the karma, yes. [12:57] * rye loves bzr --author switch [13:01] dobey: line 107 of the diff, why? [13:01] dobey: same thing in other places, too [13:02] ralsina: you mean the new line? [13:02] line 187 in the diff isn't a change, either [13:02] dobey:107 not 187 [13:02] oh, 107 [13:02] doh [13:02] because python3 [13:02] dobey: you are using list() on a generator [13:02] no [13:02] dobey: both are valid on python 3 [13:02] am using list() on a thing that isn't a list [13:03] .items() does not return a list on python3 [13:03] dobey: yes items() on py3 is a generator. You can iterate over it just fine [13:04] dobey: https://pastebin.canonical.com/66420/ [13:05] tell that to 2to3 [13:05] dobey: do not use 2to3, 2to3 is dumb :-) [13:06] dobey: or, better, explain me the problem. So, if you use 2to3 on that without the list() it fails? [13:06] 2to3 just outputs a diff of suggested changes. one of those suggested changes is to include the list() there [13:07] dobey: ok, that one doesn't seem necessary, and is kinda ugly [13:07] i can change it to not have the list() i guess [13:08] dobey: please [13:10] dobey: that change only is useful when you are using items in a non-iterative fashion, which it seems is not the case in any place here [13:10] pushed [13:11] dobey: you can even tell 2to3 to not suggest those changes with -x dict (but that disables a bnch of other stuff too) [13:11] ralsina: also, i guess we do need to backport some things for windows-installer, and maintain a stable branch, if we are going to ever have nightlies, beta, and stable releases for windows [13:11] dobey: yeah [13:11] dobey: it makes sense to have a stable branch that builds from stable and keep trunk building from trunk [13:13] i guess i'll do that then [13:14] dobey: should be just a small fix in conf.py [13:18] erm... how come i have conflicts in https://code.launchpad.net/~rye/ubuntuone-client/ignore-in-close-write-in-dirs-stable-3-0/+merge/106613 ??? [13:18] oh [13:18] i know [13:18] wow that is a long branch name [13:19] proposed against trunk [13:19] dobey: if it breaks, i found a bug :) [13:20] rye: not really. unix sockets have a maximum filename length [13:21] rye: so if it breaks, your branch name is too long :) [13:29] yeah, i guess 128 characters [13:34] rye: why no unit test? [13:34] dobey: alecu is working on that since he knows where events can be injected from [13:35] switched to WIP [13:36] dobey: +1 [13:39] morning briancurtin [13:39] morning dobey [13:40] briancurtin: are any of the last 3 changes on https://code.launchpad.net/~ubuntuone-control-tower/ubuntuone-windows-installer/trunk necessary for doing the 3.0.1 release for winodws? [13:41] dobey: nope, those aren't needed for it [13:42] ok, cool [13:42] rye, https://code.launchpad.net/~alecu/ubuntuone-client/ignore-in-close-write-in-dirs [13:42] rye, (still running the tests on it, though) [13:47] * mandel back [13:48] briancurtin, ralsina: made the 'release' of windows-installer 3.0.1 with updated xml to change the version to 3.0.1 and build to 1001 [13:48] dobey: awesome, thanks [13:49] so you should be able to build a 3.0.1 release for windows now [13:51] alecu: wow, i spent a couple of days ago trying to come up with the test... and all I was thinking - how to make filesystem send these events [13:51] briancurtin: what dobey said^ [13:52] ralsina: how should that affect the 3.0.1 installer i already built? i was going to do some last testing this morning then ask if rick got a chance to look at it, then get everything signed [13:53] briancurtin: well, since you already built it, in no way I guess :-) [13:53] i thought that was a "candidate" and we'd build a proper thing after the stuff was released [13:54] dobey: the idea of a candidate is that if it's approved, it's shi[pped [13:54] ralsina: but the idea of having consistent releases across platforms is that everything is built from things that are released :) [13:56] dobey: yeah, we have a conflict of paradigms here ;-) [13:57] dobey, briancurtin: we need to decide on a worklow here [13:57] flow* [13:57] dobey, briancurtin: and I am sure you two can come up with a nice one [13:58] For this release, just ship the one that's done, for the next one, we have a defined checklist and steps to follow [13:58] dobey: plus, the one that was built on friday should be the exact same code as this one we could build now, really [13:59] since "creating the release" in LP is something dobey does, at least for now, i'd hate to have him go through any of that, then come back today and say no now we need to include these two branches, do another candidate, come back with another branch, then we finally release [13:59] briancurtin: the main problem being that we keep needing magic branches [14:00] briancurtin: which is something we should aim to get rid of for the next release, anyway [14:00] id certainly love to get away from that [14:00] well, what is the current 3.0.1 installer built from exactly? [14:01] briancurtin: we are all in agreement there. We just need to get it fixed. So, you and I will do it before the next release. Plus, with the nightlies from jenkins, we can get have QA test before we do a "real" release, and avoid all this silly work you have to do now. [14:01] * ralsina is -1 on english today :-/ [14:02] msinly, i think we should also strive to have reproducible builds on windows as well as linux. [14:02] dobey: it's from stable 3.0.0 plus https://code.launchpad.net/~brian.curtin/ubuntuone-control-panel/c2c-loading and https://code.launchpad.net/~brian.curtin/ubuntuone-client/fix984407-windows-is_root [14:02] err mainly, not msinly [14:02] briancurtin: ok i think we definitely need a new installer then [14:02] briancurtin, dobey: isn't windows-is_root in 3-0 now? [14:02] crap...do we still need mandel's extra logging SSL branch [14:03] briancurtin: no, that's merged [14:03] yeah, i have no idea what we're doing then [14:03] briancurtin, is there :) [14:03] ralsina: actually i don't think the extra logging is in stable-3-0 [14:03] dobey: oops, it should, shouldn't it? [14:03] i do the release from stable-3-0 on the 3.0.0 tag, plus the two branches listed above [14:03] ralsina, dobey, briancurtin, it is in trunk.. maybe not in an stable branch [14:04] so mandel's branch isn't in the stable, so i forgot that one... [14:04] ugh [14:04] there are lots of other branches that should be included in the 3.0.1 release [14:04] briancurtin: ok, let's do one from 3.0.1 tag instead. dobey: we should get mandel's branch in stable, too :-( [14:04] briancurtin: anyway, we probably can't get QA today because it's accounts-releas-day-again [14:05] ralsina: no problem. i will take a look today and see what i can find on my own, then get them a release whenever they can take it [14:07] mandel: please restate bug #1002208 as a bug? :-) [14:07] Launchpad bug 1002208 in Ubuntu One Client "Unix domain sockets should be used for the IPC on darwin" [Medium,In progress] https://launchpad.net/bugs/1002208 [14:08] ralsina, fuuu ok, on it, I wish my todos would use bug titles.. [14:13] i wish bitrock had good version comparison support [14:16] mandel: thing is, a bug should describe something that is wrong, not how we want to fix it [14:16] ralsina, briancurtin: i think we can probably do qa on windows in between new accounts work...don't hold up on it for qa...if need be i'll help test again [14:16] ralsina, I know, I just keep writing things as my todo list.. [14:16] mandel: so "Darwin is using wrong transport" is a bug title, specially with a description explaining why [14:17] mandel: I used to do the same, but it' tricky to hand bugs over when they are done as todos :-) [14:18] joshuahoover: ack [14:18] joshuahoover: we have some minor branch selection troubles to clear first, but we'll have something today [14:18] ralsina, I know.. sorry [14:18] mandel: np [14:20] * briancurtin rebooting since windows update keeps nagging me [14:20] ralsina: can you update the rt ticket to let IS know it's not coming today and give them an estimate on when they should expect it? i'm thinking wednesday or thursday at this point, but i'll leave that up to you [14:21] joshuahoover: I think it can go out today, if QA can do a minimal run of tests [14:21] joshuahoover: at least put the binaries for signing late today [14:22] ralsina: oh, ok...i guess it depends on when we get the installer today...but yeah, assuming we get it before the end of the day today then we should be able to turn it around with testing pretty quick [14:22] joshuahoover: we'll try to get it ASAP and then we'll see [14:23] joshuahoover: worst case it will be uploaded tomorrow morning [14:23] ralsina: ok, sounds good [14:28] * mmcc finishes reading backlog [14:28] morning all [14:29] ralsina, gatox, can I have a very easy review for the following: https://code.launchpad.net/~mandel/ubuntu-sso-client/darwin-ac-cmdline/+merge/106635 I just moved the code out to the correct function and allowed it to be used at package level [14:29] mmcc, hi [14:29] mmcc, morning!! [14:29] mmcc, of course [14:29] gatox: sure [14:29] YAY 3rd IPC in U1 for Mac OSX [14:29] mmcc: good morning! [14:29] mmcc, I did a review I owed you, there are some small things to fix [14:29] ohhhh the review was for mandel [14:29] mandel, of course [14:29] jeje [14:29] gatox, te odio -_- [14:29] :P [14:29] * gatox has the ability to ping always the wrong person [14:30] ok mandel, still going thru emails... [14:31] * rye goes to the last spanish lesson [14:33] mandel: +1 [14:33] ralsina, thx! [14:37] mandel, +1 [14:38] gatox, thx, we need to change that to work in a decent manner on darwin but the current imp is ok for what we need atm [14:42] briancurtin, dobey: want to talk about what is and is not in stable-3-0 and what should be in 3.0.1? [14:42] yes [14:43] briancurtin: good, talk with dobey and I will listen and chime in when needed :-) [14:45] so at the very least, what i've been using has been stable-3-0 plus the is_root fix (the main point of the release), the c2c fix (allows c2c page to, at the very least, not display "getting information" forever), and it should have included mandel's SSL logging branch [14:45] i'm certain there is more to release, but i'm not sure if this release is just intended to fix the is_root problem since that is starting to build up support tickets, or if we want to release everything else that has been added since 3.0.0 release [14:46] if we're going to release it as 3.0.1, then it should include what's in 3.0.1 (and i guess the ssl logging branch if necessary) [14:47] there are definitely other important fixes in the 3.0.1 tarballs which i think should get pushed out in the windows release as well [14:47] that makes sense to me, i just didnt know what else constituted 3.0.1 [14:48] since i already forgot the mandel branch, i need to create another installer anyway, so i can do it from 3.0.1 plus our select branches (probably just C2C then) [14:48] briancurtin, dobey: seems to me we rushed the windows bit and should instead have held back until we had 3.0.1 "done" [14:48] ralsina: right [14:48] agreed. i only had my windows hat on, so when we had windows ready, i was ready to roll [14:48] mandel, mmcc can we move the fsevents mumble from today to tomorrow?? i'm taking half-day sick after the standup because i'm with a really bad headache [14:48] hopeully this release will be awesome enough that we'll feel no rush to do the next one [14:48] rather, that one windows fix ready [14:49] gatox, I'm ok with it [14:49] mandel, thanks! mmcc ? [14:49] yeah, no problem, feel better gatox! [14:49] mandel, mmcc thanks! i'll update the event [14:49] I was sick all weekend, I feel your pain :\ [14:50] mmcc, bummer..... [14:50] briancurtin: the thing is, i don't think we had windows ready, necessarily :) [14:50] mmcc, mandel i sent the update [14:50] mandel, just pushed your suggested changes. [14:50] dobey: i dont *actually* think that either, we just got that is_root branch in so for most people (customers), it was ready in that they'd actually be able to use it [14:51] brb in time for standup... [14:51] briancurtin: right. assuming that it didn't break for them on a couple of other bugs in u1-client :) [14:52] briancurtin: anyway, it sounds like you need to use the release-3_0_1 tag now, and also include that c2c branch, and the extra ssl logging branch from mandel [14:52] mmcc, ok, will take a look in a few mins [14:53] dobey: so it's that tag from the stable-3-0 *branch*, correct? (just making sure i have the right terminology) [14:53] briancurtin: right [14:58] dobey, briancurtin: we could think of something for "patch releases" like we have in ubuntu, where we do a release from a tag + a patch. But numbering gets crazy. [15:00] me [15:00] me [15:01] me too [15:01] me [15:01] mandel, ralsina dobey ? [15:01] me [15:01] me [15:02] me [15:02] DONE: [15:02] Friday: lot of improves in the freaky friday project. Today: Debug Filesystem notifications (not root implementation) tests, a little bit closer. [15:02] TODO: [15:02] Rest (Take half-day sick leave). [15:02] BLOCKED: [15:02] No [15:02] mmcc, go [15:02] DONE: read up on sandboxing, delinted network detection, made decision doc for osx client [15:02] TODO: daemon on 10.6, packaging [15:02] BLCK: NO [15:02] NEXT: alecu [15:02] DONE: finishing security work, some tests for rye's branch [15:02] TODO: mumble with mandel, mmcc, gatox: OSX notifications, wrap up security patches [15:02] BLOCKED: no [15:02] NEXT: briancurtin [15:02] DONE: half day, release stuff [15:02] TODO: make a new release based on what we just talked about, test it, etc [15:02] NOTE: in an hour i have a PSF Board meeting so i'll be around but on the phone [15:02] BLOCKED: none [15:02] NEXT: mandel [15:02] DONE: Reviews, reviews, reviews. Bug 1002307. Worked on bug 1002208 and bug 1001296 to get ipc on windows back on u1-client and add domain sockets support for darwin. [15:02] TODO: More on 1002307 and 1001296 (making the ipc tests with pb run on linux and over both transports), more reviews, fix u1-cp ipc for the same reasons as the above mentioned bugs. [15:02] BLOCKED: no [15:02] ralsina, please [15:02] Launchpad bug 1002307 in Ubuntu Single Sign On Client "The darwin implementation of get_activation_cmdline is missing" [Medium,Fix committed] https://launchpad.net/bugs/1002307 [15:02] Launchpad bug 1002208 in Ubuntu One Client ""Darwin is using wrong transport" [Medium,In progress] https://launchpad.net/bugs/1002208 [15:02] Launchpad bug 1001296 in Ubuntu One Client "Tests are broken on Windows" [High,In progress] https://launchpad.net/bugs/1001296 [15:02] DONE: bled from my nose all over my notebook (looks like blood is not conductive!), reviews, minor coding, figuring out what bug to hit first, bug triaging, askubuntu answering, canonicaladmining, etc. TODO: fix a bug BLOCKED: no [15:03] dobey! [15:03] λ DONE: 1-1 review meeting, 3.0.1 backports, 3.0.1 releases [15:03] λ TODO: finish bug #853060, 3.0.1 SRUs, u1-installer 3.0.1 release/SRU [15:03] λ BLCK: none. [15:03] Launchpad bug 853060 in ubuntuone-installer "ubuntuone-installer crashed with GError in function(): Failed to execute child process "ubuntuone-control-panel-gtk" (No such file or directory)" [Critical,In progress] https://launchpad.net/bugs/853060 [15:03] ralsina, ugh, nosebleed! are you all right? [15:03] alecu: yeah [15:03] me [15:03] alecu: I have been having sinus issues all morning, and finally blew my nose once too many [15:04] alecu, i didn't see you in the guest list of the mumble meeting.... i talked with mandel and mmcc if we can move that for tomorrow.... because i'm about to take half-day sick leave now...... because i'm with a really bad headache [15:04] DONE: productive u1db handover sprint TODO: lots, first thing: u1db combine() mapping BLOCKED: no [15:04] alecu, did you receive the update of the calendar event? [15:04] gatox, no [15:05] gatox: but now I see it updated, yes [15:05] alecu, it's ok with you to do it tomorrow? [15:07] gatox: oh, yes. No problem. [15:07] alecu, thx...... i'm off for today now....... [15:07] see you tomorrow! [15:07] gatox: bye! [15:08] no comments? [15:09] EOM [15:13] * alecu will be back in 10' [15:21] ok, need to get lunch. and then lots of SRU uploads and finishing up this installer bug [15:21] bbiab [15:40] mandel: I have shared with you jam's windows environment which should be everything required to run u1db tests on windows [15:40] thisfred, via email? [15:41] thisfred, ok, got it :) [15:43] mandel: via this filesyncing service I heard about ;) [15:43] briancurtin, you are aware of the branch that mandel is about to propose that fixes ipc on windows, right? [15:43] alecu: no [15:44] briancurtin, are you releasing from trunk or from the tar.gz? [15:44] alecu: stable-3-0 branch, release-3_0_1 tag [15:44] briancurtin, the branch is for trunk, since one of the recent branches that changed ipc to use unix domain sockets for osx broke ipc on windows. [15:45] briancurtin, we should make sure that this issue is not present on 3-0-1 [15:45] mandel, what would the symptoms for this be? just ipc not working? [15:45] it should be easy to tell :-) [15:46] alecu: i'm building the installer right now so i'll know in a few minutes what happens [15:46] briancurtin, great. [15:46] alecu: that brach never made it to stable-3-0 AFAIK [15:47] ralsina, great. And I guess it did not make it to 3-0-1 either. [15:47] alecu, I believe as ralsina said that this broke trunk and not stable-3-0 but let me check since it is code in sso [15:47] * alecu sighes relieved. [15:47] * mandel takes a look [15:47] alecu: shouldn't [15:49] briancurtin, ralsina, mandel: what about the branch that logs the ssl details? Why are we manually including it instead of merging it into trunk or stable? [15:49] ralsina, briancurtin, alecu, if we are using lp:ubuntu-sso-client/stable-3-0 to build the installer there should be no issues with the release [15:49] mandel, great with that. [15:49] alecu, is in trunk, I guess it was not moved to the stable branch, no idea why [15:49] alecu: I assumed that branch was merged into stable, seems I assumed wrongly [15:51] alecu: tech leads in 10' [15:51] ralsina, thanks. [15:52] ugh, can't find the bug i reported about sso not working, which i think was related to mandel's IPC issue. LP sucks. [15:52] briancurtin, was that in production? 'case the ipc is broken but was never released AFAIK [15:53] mandel: no it was just on my machine using trunk [15:53] ralsina: any ideas on what to tell windows users getting the valicert error? [15:53] i was trying to debug something the other day and while tests always worked, actually starting up CP and everything else wasnt working [15:53] briancurtin, ok, let me know when you get the but num and I'll take a llook asap [15:55] oh here it is, mandel: https://bugs.launchpad.net/ubuntu-sso-client/+bug/1000407 [15:55] Ubuntu bug 1000407 in Ubuntu Single Sign On Client "AttributeError on new install" [Undecided,Confirmed] [15:55] i dont know how, if at all, that is related to your work...but its something that broke recently [15:55] briancurtin, https://bugs.launchpad.net/~brian.curtin/+reportedbugs ? [15:56] joshuahoover: we have one of those? [15:56] ralsina: more than one [15:56] ralsina: at least 2 that i'm looking at right now [15:56] joshuahoover: ok, control panel, certificates, search for the valicert one, "enable for all purposes" [15:56] joshuahoover: specific steps may vary according to windows version [15:56] briancurtin, ok, looking [15:57] ralsina: k, thanks! [15:57] briancurtin, and that one is completely new to me.. :( [16:50] all, EOD here, alecu I'll propose the fix for windows tom, I'm getting dirty reactor exceptions on windows and will need to refactor the code to use the u1-devtools code correctly [16:52] mandel, great. [16:53] alecu, I should be able to propose the cp branch too and we will have all ipc on domain sockets too, but we geed to get the security right as you mentioned last time, I suppose setting the rights of the socket correctly should be more than enough [16:53] mandel, should be enough, but not more :-) [16:54] alecu, linguistic freedom in my part :) [17:01] I think I will have a lunch [17:01] only one of them! :) [17:02] so, i wonder how to make this code actually fail for me [17:02] dobey: yes :-) [17:35] so i have a branch which i think fixes this installer issue [17:36] * mmcc going to find a lunch to have [17:46] bah, where is this stupid error coming from, or not as the case may be :( [17:55] dobey: want me to test it? [17:55] briancurtin: any update on the windows installer to test? :) [17:56] joshuahoover: good timing. i'm wrapping up a quick smoke test with it right now before i upload it, will have a link shortly [17:57] briancurtin: smoke test? sounds unnecessary ;) [17:57] joshuahoover: just need to make sure i pulled in the right accessory branches, since i screwed that up last time [17:58] ...and its good (holds hands up like a touchdown) [17:58] ralsina: if you could; lp:~dobey/ubuntuone-installer/fix-aptdaemon [17:59] dobey: on it [17:59] dobey: will make sure I am in such a situation that current package fails, first, then try that oe [17:59] one* [17:59] ok [18:01] joshuahoover: http://ubuntuone.com/7cnc3EWLWTSblGPPPOV7DK [18:01] sorry for the delay, i got stuck on the phone way too long earlier [18:02] briancurtin: cool, thanks [18:02] rmcbride, elopio: ^^ [18:03] great. [18:03] (cool [18:05] damn you apt-get for being slower than everything else in my computer [18:16] dobey: branch looks good, but released -installer has decided to work correctly so could not test IRL that it fixes the bug [18:19] :( [18:20] dobey: considering the volume of failure, if it fixes it, we will know [18:21] I know I soud like House MD there, though [18:21] ralsina: but we need to be able to tell the SRU team how to test it [18:22] dobey: we can still use the "network disabled" test, right? [18:23] hmm, let me try on my laptop without network [18:25] dobey: did you commit a partial fix a while ago? I may hve been using nightlies [18:26] ralsina: no; i made a fix for a related and similar problem, but apparently not the same one [18:26] dobey: I may have been triggering that one then :-( [18:27] * briancurtin back in a few minutes, need to pick up a sandwich [18:29] ralsina: i doubt it; that was a long time ago. no way you were triggering it last week :) [18:30] oh fun [18:30] dobey: 12.04 user with gnome 3 installed is getting this error when trying to open u1-installer...any ideas/pointers? http://paste.ubuntu.com/999508/ [18:30] it installed the cached version [18:31] joshuahoover: it would appear they have a broken qt or pyqt install [18:32] joshuahoover: also, that error is from the control panel :) [18:33] dobey: yeah, heh...just going by what the user told me [18:34] hrmm, stupid error messages and label widget. [18:35] joshuahoover: broken system, do apt-get dist-upgrade [18:43] so it does seem to fix the no network case at least [18:44] would like to fix the "clicked cancel in the auth dialog" bug too, but i can't figure out where exactly there error is popping up through [18:45] dobey: closing the installer window half-way through leaves things in a broken state, maybe that's the same thing? [18:45] no, that's different [18:46] dobey: ok [18:50] * briancurtin back [19:05] ralsina: https://code.launchpad.net/~dobey/ubuntuone-installer/fix-aptdaemon/+merge/106693 [19:05] dobey: got it [19:11] dobey: +1 [19:13] anyone else want to review that? :) [19:19] kinder for me! [19:24] dobey, I took a look at your mp but I've never written any gtk so don't have much context for judgement. looks reasonable - I'm curious about the bare integers for apparently identifying pages in the notebook, eg set_current_page(2). Is there a reason not to use a variable with a readable name there? [19:26] less typing than defining constants for all the pages [19:30] okay - well I guess you still need a more knowledgeable reviewer, unless you want to coach me through testing it out :) [19:35] i can't even fully test it myself :) [19:35] i can only test the case where failure happens due to network being disconnected [19:36] but whatever else is causing problems for users, i can't seem to find a way to cause it [19:36] and users aren't particularly helpful here. "it just didn't work." [19:42] dobey: ah ok so this is just to get better debugging info [19:43] well, it fixes the problem of not doing any error handling at all [19:43] which should generally fix the problem for everyone that's having this specific problem [19:44] since they'll be able to see the error, fix it, and try again [19:44] there are a couple more error cases which i can reproduce, but which I haven't figured out yet either; but they are separate bugs and happen much less often [19:57] dobey, your commit message mentions trying again - how does that work? it looks like if you hit 'cancel', __got_response calls main_quit() - am I getting that right? [20:03] mmcc: if you hit close it quits, yes. if you hit install, it tries to install again [20:03] mmcc: the error page shows [Install] [Close] as the dialog buttons [20:06] dobey, ok. would it be a lot of work to switch that to [Try Again] [Close] or something? Admittedly it's a nitpick [20:07] yes [20:08] well, it's a string change [20:08] = translation? [20:08] and the whole point of having it just be install there is to avoid the string change [20:08] = pain to sru, yeah [20:09] and i don't think having it be something else really helps clarify what it does [20:10] changing it is only to satisfy pedantry really. it's still going to do the exact same thing that clicking on the install button did in the first place. which is install the stuff, or show the error page again [20:11] fair enough. I don't feel too strongly about it. any complaints with me approving this then? I understand it now. [20:12] no complaints from me, no :) [20:13] ok done, thanks for holding my hand there. figured I ought to try to get used to more parts of the code (eg. gtk) [20:14] mmcc: good attitude! [20:15] * mmcc pats self on back [20:15] mmcc: BTW, have you claimed a review day in the calendar yet? [20:15] * mmcc tweaks shoulder [20:15] ralsina, yes - Tuesday! despite appearances to the contrary, it is not today [20:15] mmcc: hehe [20:16] I think we need to tell mandel to get two half-days instead of a day, because he never has reviews on monday morning [20:16] oh, wait he had 3 or 4 today. Ok then [20:27] ralsina: remind me again, this 3.0.1 release will have the fix for "Please don't run the syncdaemon as root", correct? [20:27] yes [20:27] joshuahoover: yes [20:27] heh, thanks dobey and ralsina :) [20:27] joshuahoover: I expect we will cause you a lot less pain starting with this release :-) [20:28] ralsina: i'm hoping! [20:40] I am going to take a slightly early EOD because I feel like crap. briancurtin, the RT is in your hands as discussed, I am now about to have like a dozen ibuprofens or something :-/ [20:41] ralsina: i was going to wait until i got the go-ahead from rick, leo, or joshua to know they've run their test suite or whatever [20:41] "a dozen ibuprofens" ...sounds like the dosage you'd give to a grizzly bear [20:41] joshuahoover: well, I am half grizzly [20:41] father's side [20:41] so just 6 [20:41] elopio, rmcbride: any updates on testing the latest 3.0.1 release? [20:41] heh [20:41] briancurtin: when you get word, go ahead, and i'll cross ingers [20:41] fingers [20:41] joshuahoover looking good so far. things are working, changes are syncing [20:42] good [20:43] hey, feel better soon ralsina. [20:43] mmcc: will try! [20:43] bye! [20:44] joshuahoover: I've just started my testing. No problems so far. [20:45] rmcbride, elopio: once you guys are done running through the test cases, please let briancurtin know the results, thanks! [20:47] will certainly do so [21:24] dobey: i heard from a (turkish) user experiencing the valicert problem on ubuntu and the workaround in bug #997326 didn't work, it said the file already existed [21:24] Launchpad bug 997326 in Ubuntu One Client "Users in Turkey can't connect due to invalid Valicert certificate" [High,Triaged] https://launchpad.net/bugs/997326 [21:24] joshuahoover: oh fun [21:25] dobey: yeah...sigh [21:25] * dobey recalls a TMBG song [21:26] joshuahoover: on the other hand. if the file exists for that user, i wonder if it's a broken symlink or something [21:27] dobey: hmmm...could be, i suppose i could ask him to delete the current link and try creating it again? [21:28] joshuahoover: well, that's assuming the file it's linking to isn't a broken symlink as well [21:28] true [21:30] on my system it's a symlink that points to a symlink that points to a cert file [21:30] rmcbride, joshuahoover, briancurtin, smoke works here. [21:31] elopio: thanks! [21:31] elopio: joshuahoover: briancurtin: yea I'd say smoke and a bit more work fine. I have been messing around with various aspects for a while now [21:34] thanks for taking a look. now that we have at least 3 people successfully using it i'll go ahead and start getting the binaries signed. let me know if anything does pop up so i can halt the process [21:41] elopio, rmcbride: did we capture these latest test results in moztrap? [21:41] joshuahoover: you know, I did not. I'm working from the windows box doing this, and I hadn't set up the whole browserID thing in it yet [21:42] (I also forgot Moztrap was working again) [21:43] I'll get that set up in this location so that it can be recorded properly after I get dinner started [21:43] joshuahoover: not all the tests are in moztrap yet. [21:43] but I made the runs and recorded some results. [21:44] elopio: ah, ok...maybe we can make that part of testing windows next time? either you and/or rmcbride could probably knock out the remaining tests pretty quick next time around [21:45] binaries are off to be signed [21:45] thanks brian! [21:46] alright kids. have a good evening! [21:48] joshuahoover: sure. I'm copying tests every day. I hope that by the end of the next week all the things from the wiki are updated and on moztrap. [21:49] elopio: cool...i would've helped last week but you know, moztrap wasn't available ;) [21:52] I know. but don't worry, I'll leave the mac test cases to you so you are not left without the joy of documentatio ;). [23:07] I'm heading out for the night. Today's rathole brought to you by: homebrew, py2app, SIP and PyQt.