[00:01] Have a great afternoon/evening everyone! [00:11] karni: you too! [00:11] I'm off as well [00:29] thisfred: if you're back, and if you're still working, I would strongly appreciate a review for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-folder-mgmt/+merge/65281 [00:29] thisfred: if not possible, is OK :-) [00:30] nessita: will do, and then I'm EOD too, have to cook [00:30] * nessita too [00:30] thisfred: is long, so you're allowed to not to it [00:30] to not to do it* [00:30] nah, I have a little time yet, I'll do it [00:32] thisfred: YOU ROCK, genial! [00:32] :-D [00:32] ok, all emails sent, all branches submitted, all energy wasted [00:32] bye all! [00:32] see you tomorrow! [00:32] :-) [00:33] * nessita realizes that wasted was not the proper term [00:33] "all energy ran out" [00:33] thisfred: is that better? ^ (englishly speaking) [00:33] nessita: yes, wasted implies it was all for nothing :) [00:34] right, and it was not the case (luckily) [00:34] ok, I'm gone [00:34] bye! [01:10] reviewed, few minor issues [01:11] now it's cook or be cooked [02:17] rye: hi [02:18] hi, folks [02:44] * karni EODs, for real this time === avr1l is now known as avril === jamesh_ is now known as jamesh [11:40] morning all [11:41] hi duanedesign === czajkows1i is now known as czajkowski [12:22] * fagan break [12:33] hello all! [12:41] o/ [12:44] hi duanedesign! [12:44] ralsina: you back to duties? :-) [12:46] mandel: good morning. You walking the beast? [13:16] ralsina: ping [13:34] good morning! [13:34] nessita pong [13:34] nessita: yes, I was, but I'm back :) [13:34] ralsina: volviste! [13:35] nessita: back in business, just a tad late because of kid's school parents meeting [13:36] ralsina: when you have a moment, I need a review form you in a branch where I'm having some QT issues [13:36] nessita: right now is a god time :-) [13:36] ralsina: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-folder-mgmt/+merge/65281/comments/141693/+reply [13:36] ah no [13:36] ralsina: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-folder-mgmt/+merge/65281 [13:37] nessita: ok, branching [13:37] ralsina: please read the merge proposal description and run the branch on linux [13:37] nessita: ok [13:38] ralsina, nessita: a review for this will be very appreciated [13:38] https://code.launchpad.net/~mandel/ubuntuone-client/legal_linux_chars/+merge/65285 [13:38] mandel: already approved, running tests still [13:38] mandel: right after nessita's [13:38] ralsina: I need to solve the "QLayout: Attempting to add QLayout "" to SomePanel "Form", which already has a layout" warning, to understand where is coming from, please see the __init__ of the UbuntuOneBin class [13:38] nessita: hmmmm that's pretty harmless [13:38] nessita: cool :) [13:39] ralsina: ... I know the code is not correct, so I would like to know how's the proper way of doing it [13:39] nessita: ok, I'll take a look [13:40] probably you need to set to no layout on designer on the container, or setLayout(None) somewhere [13:40] *or* you are setting the layout of the wrong kind of widget. Will know in 5' [13:42] ralsina: you browse the source, I don't think is that simple, I'm trying to build a widget from an .ui, then make that widget parent of other widgets that are also built from an .ui [13:43] nessita: ping [13:43] cjohnston: pong [13:43] nessita: ok, I am looking now [13:43] ack [13:44] nessita: I was told you may be the correct person to talk to.. I don't know if you have seen status.ubuntu.com yet... On the teams page there are 6 Ubuntu One teams.. Some with only 2 or 3 work items.. I wanted to see about reducing it to just the one core team, and then create topics (like what is seen on the main page of status.ubuntu.com) for any other breakdowns that may be needed. http://status.ubuntu.com/ubuntu-o [13:46] cjohnston: I'm definitely not the person to talk to, you may wanna talk about that with Chipaca`, he is our engineering manager and handles all the "sub teams" (ubuntuone-desktop, ubuntuone-foundations, ubuntuone-web) [13:46] Ok.. Thanks nessita [13:46] cjohnston: that url is giving me 404, btw [13:47] Odd.. it's working just fine for me.. Try http://status.ubuntu.com ? [13:47] nessita: ok, stupid question, qtreactor url please? [13:47] that works, thanks :-) [13:47] ralsina: git clone https://github.com/ghtdak/qtreactor [13:47] Chipaca`: ping.. Same comment/question as I asked nessita ^ :-) [13:48] nessita: thx! [13:50] cjohnston: looking [13:50] ty [13:50] cjohnston: the page looks "interesting", but not particularly useful [13:50] cjohnston: what *should* i be seing? [13:51] nessita: have you considered doing that using promoted widgets instead? [13:51] ralsina: welcome back! [13:51] ralsina: I'm not familiar with promoted widget, so no [13:51] These are burndown charts... If you go to the Teams page, you will see many different Ubuntu One teams.. Chipaca` http://status.ubuntu.com/ubuntu-oneiric/teams.html [13:52] Chipaca`: thanks! [13:52] nessita: basically, you use a placeholder widget on designer, right click and tell it promote to WhateverWidget that is on module whatever [13:52] So for example: http://status.ubuntu.com/ubuntu-oneiric/ubuntuone-hackers.html The ubuntuone-hackers team... (The trendline is off, I'm waiting to a new bzr pull to fix it) [13:52] cjohnston: i'm using chromium, and all i see are dots (and, well, the header with "teams", but clicking it gives me dots also) [13:52] ahh [13:52] ralsina: I don't think that is what I need, from that explanation [13:52] You have a defined font siz [13:52] size [13:52] nessita: and you don't have to load and insert each manually. But since you already did that I will try to make the warning go away :-) [13:53] cjohnston: I have a what? [13:53] alecu: just the person i was looking for! [13:53] nessita: hmmm.... maybe I am not understanding the problem then. Give me 5' more to see if I get it. [13:53] hello Chipaca-prima [13:53] ralsina: I need a base panel to be parent of all the other panels. This parent need to implement some common functionality, and has an empty container to pack another widget in it [13:53] ralsina: sure [13:54] hello also ralsina, everybody! [13:54] dobey, mandel, thisfred, fagan, alecu, ralsina: stand up in 6' [13:54] hi alecu! [13:54] nessita: oooooook I get the idea now. Have never done it this way, so it will take me a bit to untangle it [13:54] alecu: morning! [13:54] Chipaca`: https://bugs.launchpad.net/launchpad-work-items-tracker/+bug/795623 is affecting you.. [13:54] Launchpad bug 795623 in launchpad-work-items-tracker "Can not use minimum font size and can not zoom page (affects: 1) (heat: 5)" [Undecided,Confirmed] [13:54] alecu: 'elo [13:54] alecu: i'm trying to bring up the control panel on windows, and failing. what do i need to fix? [13:55] I'm not sure how to fix that in Chromium.. /me is looking [13:55] Chipaca`, got any error? [13:55] cjohnston: I can look in firefox also :_) [13:55] alecu: "no module named gi.repository" :-/ [13:55] If you could try that.. That might be easier than checking settings, although I did just figure the fix in Chrome if FF doesnt work. :-) [13:56] Chipaca`, oh, yes. I saw that one. Let me check. [13:56] nessita: where are you setting UbuntuOneBin.ui_class ? [13:58] nessita: nevermind, found it [13:58] :-) [13:58] Chipaca`, try this branch: lp:~alecu/ubuntuone-control-panel/run-windows-run [13:58] cjohnston: I'm not sure how these are generated [13:58] cjohnston: but they seem wrong on several axes [13:58] Yes.. I'm waiting on IS to update the config file for me [14:00] alecu: now i got a window, but tracebacks about 'get_root_dir' being missing [14:00] Chipaca`, welcome! [14:00] If you look at the teams page though, there are 6 Ubuntu One teams, that I want to take down to just one team... We can do topics, which is pretty much grouping blueprints together, to futher specify the blueprints into categories.. [14:00] ralsina, nessita just called me; her flat has run out of electrical power. [14:01] alecu: ok [14:01] cjohnston: ok. what do you want from me? [14:01] cjohnston: the only team that should be there is "ubuntuone-team" [14:02] me [14:02] me [14:02] alecu: can I have a review: https://code.launchpad.net/~mandel/ubuntuone-client/legal_linux_chars/+merge/65285 [14:02] dobey: ubuntuone-team only has 22 work items, whereas ubuntuone-hackers has 123.... [14:02] cjohnston: I'm not sure I understand what you expect to be in the burndowns. Currently I'm seeing a bunch of apparently unrelated blueprints [14:02] Chipaca`, sd_client is still missing some bits, and for that my branch needs to comment a few broken bits, and we need to ignore some other broken bits like that. [14:03] cjohnston: for example, why is https://blueprints.launchpad.net/ubuntu/+spec/desktop-o-gtk3-gnome3 under "ubuntu one team"? [14:03] cjohnston: because ubuntuone-hackers includes other people who aren't necessarily on the u1 team [14:03] Because it has a work item that is assigned to a member of the "ubuntu one team" [14:03] ok.. [14:03] and we're back!!! [14:03] me [14:03] dobey: they *all* include other people [14:03] I don't know what's expected, what *should* be in the burndowns [14:03] Chipaca`: ubuntuone-team shouldn't [14:04] me [14:04] ralsina: are we standing up? [14:04] So... dobey, remove all but ubuntuone-team... Do you want any topics, like what is on the overview page? [14:04] dobey: https://blueprints.launchpad.net/ubuntu/+spec/community-o-developer-ubuntu-dot-com-plans is under ubuntuone-team [14:04] nessita, welcome back! [14:04] Chipaca`: joshuahoover made that team explicitly for the purpose of burndown charts [14:04] nessita: sure [14:04] alecu: :-) [14:04] everyone, standup! [14:04] Chipaca`: because rodrigo is in it still? [14:04] DONE: #779851 slight return (rewrote some parts of the branch to make the progress as informative as it can be) TODO: next unity integration bug I guess. BLOCKED: no LIKED: http://outloud.fm/u1rocks NEXT: mandel [14:05] DONE: Fixed bug 800077 bug 798641 bug 799722 bug 799751. Discovered bug 800091 which is very high priority atm because it simply blocks everything. Reviewd alecus branch for bug 799827 [14:05] TODO: bug 800091 and propose SDTool only when all tests pass (includes fixin the previously mentoned bug) [14:05] BLOCKED: no, but I need to have lunch :) [14:05] nessita, please [14:05] and why is yaili there? [14:05] Launchpad bug 800077 in ubuntuone-client "There is no bin that allows to start sd on Windows (affects: 1) (dups: 1) (heat: 12)" [High,In progress] https://launchpad.net/bugs/800077 [14:05] me [14:05] Launchpad bug 798641 in ubuntuone-client "The windows IPC should be using named pipes and not sockets (affects: 1) (heat: 133)" [Medium,In progress] https://launchpad.net/bugs/798641 [14:05] Launchpad bug 799722 in ubuntuone-client "IPC on windows uses the old CredentialsManagementTool and has an import error (affects: 1) (heat: 6)" [Medium,Fix committed] https://launchpad.net/bugs/799722 [14:05] Launchpad bug 799751 in ubuntuone-client "The file system notifications on windows do not convert the legal utf8 chars to ilegal windows chars (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/799751 [14:05] Launchpad bug 800091 in ubuntuone-client "Tritcask has an issue when opening files on windows that were closed in a different session (affects: 1) (heat: 6)" [Critical,In progress] https://launchpad.net/bugs/800091 [14:05] DONE: bug #798964, tons reviews, windows port pushing/metting [14:05] TODO: bug #798413, maybe bug #800161 [14:05] BLOCKED: nopes [14:05] NOTE: will run to university after the morning meetings, I'll be back after lunch [14:05] NEXT: alecu [14:05] Launchpad bug 798964 in ubuntuone-control-panel "Add new folders tweaks (affects: 1) (heat: 24)" [Medium,Triaged] https://launchpad.net/bugs/798964 [14:05] Launchpad bug 798413 in ubuntuone-control-panel "Provide a method in backend to list only devices info without querying local settings from syncdaemon (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/798413 [14:05] Launchpad bug 800161 in ubuntuone-control-panel "Do no allow folder creation in a folder containing an UDF (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/800161 [14:05] DONE: fixed #799827, found #799958 and started working on it [14:05] TODO: fix that bug, keep pushing u1cp-windows forward [14:05] BLOCKED: no [14:05] NEXT: None [14:05] me [14:05] *DONE: fixed bug #799827, found bug #799958 and started working on it [14:06] Launchpad bug 799827 in ubuntuone-client "find_credentials expects a proxy but gets a deferred (windows) (affects: 1) (heat: 6)" [Critical,Fix committed] https://launchpad.net/bugs/799827 [14:06] Launchpad bug 799958 in ubuntuone-client "windows ipc is not using the credentials management tool (affects: 2) (heat: 12)" [Critical,In progress] https://launchpad.net/bugs/799958 [14:06] DONE: nothing, vacation! [14:06] TODO: finding out right now [14:06] NEXT: noone [14:06] ralsina: TODO: styling [14:06] λ DONE: people chasing [14:06] λ TODO: fix more stuff [14:06] λ BLCK: None. [14:06] ralsina: NEXT: fagan [14:06] Chipaca`: ok, will do that after the two reviews I have pending [14:07] Chipaca`: wasn't ralsina our installer dev? [14:07] nessita: that too [14:07] nessita: now there is code I can steal ^w reuse [14:07] Chipaca`, cjohnston: and also, yes, those pages seem wrong on some levels :) [14:07] ralsina: nessita: alecu: mandel: let's have a mumble so we can organize ourselves [14:07] ack [14:07] Chipaca`: sure [14:07] nessita: one last tiny comment on your branch, but I understand why the unicode is there in the first place now :) (I would still like to see a test with a path that is and encoded string with non-ascii chars) [14:07] ralsina, alecu, Chipaca`, mandel: mumble in 8 minutes? (ie a las y cuarto) [14:08] nessita: ok [14:08] ok [14:08] thisfred: we can't build a Qstring with that kind of strings [14:08] nessita: I don't quite understand how you use setupUi on that branch [14:08] ralsina, alecu, Chipaca`, mandel: so we can build mate :-P [14:08] dobey: the shim work got pushed forwards a week [14:08] ralsina: neither do I :-) [14:08] dobey: so, no news on that yet [14:08] nessita: well what happens when the filesystem hands you a path like that then? [14:08] nessita: you are calling it twice on the same widget? You are not supposed to do that :-) [14:08] Chipaca`: by forward you mean back? [14:09] dobey: I'm looking into the future, future is that ways [14:09] already on mumble. [14:09] Chipaca`: ok, so what do we do for lucid? [14:09] because i'm pretty sure the lucid freeze hasn't moved :) [14:09] dobey: for 10.04.3 you mean? nothing [14:10] yeah [14:10] nessita: if the UI in ubuntuonebin.ui has a placeholder, then you have to call setupUi(sel.ui.placeholder) [14:10] nothing is great. i like nothing. [14:10] dobey: we don't have the resources to do what we'd have to do to do it right, and to do things wrong we might as well leave wrong enough alone [14:11] ralsina: I tried that, is does not work [14:11] ralsina: the setupUI expects something with setObjectName method, and the placeholder does not have that [14:12] thisfred: that is handled by QT. The call to getExistingDirectory handles all that [14:12] nessita: alternative is to use a promoted widget to provide the common part of the UI (title, progressbar) in each tab's UI [14:12] thisfred: we're given a QString which inside has a unicode [14:12] thisfred: and syncdaemon expects an unicode encoded with utf8 and transforms all that is needed [14:13] nessita: the *other* alternative is to ignore the warning because it's not a real problem, but I know that's hard ;-) [14:13] thisfred: the QString given is also a unicode encoded with utf8, as far as I understand [14:13] nessita: ok, excellent! [14:13] nessita: unicode is not encoded :) [14:13] you can be one or the other [14:13] thisfred: True, sorry for mixing that up [14:14] nm, branch is great [14:14] alecu: nessita: mandel: any other branch i should merge into u-c-p before showing it around the office? [14:14] i won't show it to mpt, though [14:14] Chipaca`: yeah, several not done yet? [14:14] Chipaca`: is not showable, not close, you know that, do you? [14:14] * mpt pouts [14:14] * nessita freaks [14:15] mpt: I've never seen you go postal, but this would definitely push you towards that edge [14:15] nessita: i need to show it, i will show it, my question is: is there anything else done i should merge before that? [14:16] nessita: tried it, it works [14:16] Chipaca`: the current control panel does not communicate with syncdaemon. LEt's talk in mumble [14:17] nessita: basically, added a placeHolder widget in data/qt/ubuntuonebin.ui and did self.ui.setupUi(self.bin_ui.placeHolder) [14:18] logging into mumble, going slowly! === m_conley_away is now known as m_conley [14:22] sorry was helping my da with his computer and lost track of time [14:23] me [14:23] * fagan writes some notes [14:24] DONE [14:24] * Looked at the code for the keyring in sso to see where exactly its going wrong on my system [14:24] * Looked at the keyring api a bit to figure out how to debug it [14:24] TODO [14:24] * See what I can do to fix it [14:24] Blocked [14:24] * nope === karni_ is now known as karni [14:46] alecu, nessita, ralsina: I'm away for about 20 min for lunch [14:46] mandel: enjoy that! [14:46] mandel, buen provecho! [14:46] * nessita will grab the bolied water and make some tea instead [14:49] mandel: take 40, don't choke, dude :-) [14:55] nessita, mandel, ralsina: hey, all: I need to be afk for 1h30, starting in 30' aprox. Amelia woke up with lots of fever, so she's not going to kinder, and I'll need to take care till the nanny arrives. [14:55] alecu: ack [14:58] alecu: ok === nessita1 is now known as nessita [15:10] damn power supplier === kenvandine_ is now known as kenvandine [15:25] * alecu will be afk for ~ 1h30m [15:37] mandel: were you able to dump all tritcask issues to guillermo? [15:37] mandel: any pending reviews from you to us? [15:37] nessita: yes, he is taking care of them [15:37] great! [15:38] nessita: let me take a look at the reviews, one min [15:38] nessita: "all"? I'm only aware of one [15:38] mandel: I'm then reassigning bug #800091 to verterok [15:38] Launchpad bug 800091 in ubuntuone-client "Tritcask has an issue when opening files on windows that were closed in a different session (affects: 1) (heat: 6)" [Critical,In progress] https://launchpad.net/bugs/800091 [15:38] verterok: yeah, that one, that seems big enough to be a lot :-P [15:38] nessita: yes, windows is broken beyond any possible fix :p [15:39] lol [15:39] * nessita agrees [15:39] nessita: I need a +1 for this https://code.launchpad.net/~mandel/ubuntuone-client/legal_linux_chars/+merge/65285 which I already ask for to ralsina and alecu... [15:39] nah, the fix for all windows problems is to reinstall :) [15:39] mandel: I'll look at it in 5' [15:39] mandel: can you please re ask ralsina and Chipaca`? alecoo said he was afk for a while [15:40] nessita, mandel: how on earth adding a print before an self.assertEqual call might fix a test? [15:40] verterok: timing issue [15:40] lol [15:40] nessita: it's an update of a dict! [15:41] verterok: is it done from a deferred? [15:41] the classic "let's add a print '\a' statement to fix it" [15:41] nessita: no threads, no twisted, no nothing...just plain old python [15:41] verterok: show me the code [15:41] (I'm not buying it then :-P) [15:41] nessita: tests/syncdaemon/test_tritcask.py # KeydirStatsTests.test_remove [15:42] verterok: there can be other twisted-based tests causing timing issues (and by could be, i mean are) [15:42] Chipaca`: cn I get a review for this: https://code.launchpad.net/~mandel/ubuntuone-client/legal_linux_chars/+merge/65285 [15:42] mandel: yes [15:42] * Chipaca` reviews` [15:42] verterok: no clue, windows is a weird thing... [15:42] and python on windows even more [15:43] mandel: looks good to me, of course not actually knowing that remove_illegal_windows_whatever does ;-) [15:44] ralsina: converst ?/ etc in utf8 chars that look the same [15:44] mandel: ok then +1 [15:44] mandel: what else you need reviewed? [15:44] ralsina: did you run the test? [15:45] mandel: you need it on windows, right? [15:45] ralsina: yes [15:46] mandel: don't set to approved then === zyga is now known as zyga-food [15:47] nessita, ralsina: this should be an easy one https://code.launchpad.net/~mandel/ubuntuone-client/use_txnamedpipes/+merge/65317 [15:47] mandel: looking [15:47] mandel: cool, will get to it after this one [15:48] ralsina: while you run mandel's tests, could you please finish https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-folder-mgmt/+merge/65281? (your placeholder fix is now there) [15:48] nessita: sure [15:48] thanks [15:50] Manuel, I get ImportError: No module named ubuntu_sso.main.windows [15:52] mandel: ^ [15:52] ralsina: then you do not have sso installed in the system, do you? [15:53] mandel: looks like it. Just setup.py install from sso? [15:54] ralsina: yes [15:54] mandel: ok, will do that and get back to you in 5' [15:56] nessita, mandel, ralsina: https://code.launchpad.net/~verterok/ubuntuone-client/fix-800091/+merge/65357 [15:57] verterok: on it! [15:57] verterok: well, the fix looks reasonable ;-) [15:57] verterok: is that correct, we do not need to flush ? [15:58] mandel: it should flush on process death regardless of how it dies [15:58] mandel: no need to flush/fsync on read-only files [15:58] and that too, of course [15:58] mandel: ImmutableDataFile instances represents ro files, which are opened using 'rb' [15:58] ok, let me test it in a real life case :) [15:58] verterok: an you please add that clarification as a comment? [15:59] nessita: comment where? it's in the class docstring [15:59] verterok: in the close() implementation, anyone reading that def will wonder why the parent class is not called [16:01] nessita: ok, somethig like: "I don't call parent close, as I'm a read-only file and there is no need to call flush nor fsync (and windows hate it and dies in horrible ways)"? [16:01] verterok: for instance :-D [16:01] verterok: just tested it in a real env (my account syncing) and it works!!! [16:02] mandel: cool [16:02] nessita: added comment & pushed [16:02] nessita, verterok: I thin windows is actually doing something right here, if you flush in a readonly file you should get an exception telling you something like 'you have no w you fool' [16:02] with an ascii mr.t if possible :) [16:03] CardinalFang: could you have a brief look at the pastebin attached to bug #800179 [16:03] Launchpad bug 800179 in desktopcouch "Desktopcouch unable to find listening port? Evolution CouchDB addressbooks still not available [http://askubuntu.com/q/49060/13049] (affects: 1) (heat: 6)" [Undecided,Incomplete] https://launchpad.net/bugs/800179 [16:03] nessita: yes that fixes the stuff with the warning, but have you looked at eric's comment about unicode stuff? Need a hand with that? [16:04] ralsina: seems that was a non-issue [16:04] ralsina: I replied to all, did you read my responses to check if they make sense? [16:04] CardinalFang: Looks like desktopcouch's couchdb crashes because it can't find the remote dbs. Never seen that before... [16:04] nessita: I see no response to the last one [16:05] ralsina: the only one that has not response is "I will set this to approve, to not hold this up in case I'm not around to re-review, but please fix the last two concerns (or explain why I'm wrong about them ;) " [16:05] nessita: but yes, basically, QString(u"whatever") is the correct thing [16:05] nessita: weird, I see no response for "It does, but here you're passing in a unicode string..." [16:06] nessita: then again, I may need to reload the page ;-) [16:06] ralsina: ah I replied to that here [16:06] nessita: ok then :-) [16:06] nessita: yes, your responses about unicode make sense, so +1ng it. [16:07] thanks! [16:07] mandel: the test_dir_create seems to be stuck forever on windows :-( [16:08] ralsina: is not flushing the changes from the chace… do tyou remember my long email... [16:08] ralsina: how are you running the tests? [16:08] mandel: run-tests as usual. No I don't remember the long email, sorry :-( [16:09] ralsina: look at the description of the branch, it tell you which tests to run exactly, the cache issue with the file system is something to fix later… which I dont know how :P [16:09] mandel: ok, will do that [16:11] mandel: ok, tests pass [16:12] :) [16:12] mandel: so, you could set it to approved now [16:12] just did :P [16:15] thisfred, looking. [16:17] thx! [16:17] * ralsina wonders if the windows-installer project should have linux support. Probably not. [16:17] ralsina: yes! [16:18] ralsina: the wizard should be multiplatform unless it delays you a lot [16:18] * ralsina was kidding ;-) [16:18] ralsina: since we will use it on linux as well [16:18] nessita: I am actually doing it on linux, so... [16:18] ralsina: ack [16:18] (great) [16:18] we should rename the project maybe [16:19] ralsina: so, as an FYI, let me share with you that the feelings of the windows team are very sensitive right now, so we need to be careful with the jokes (general notice) [16:19] nessita, ralsina: did you have the chance to look at: https://code.launchpad.net/~mandel/ubuntuone-client/use_txnamedpipes/+merge/65317 [16:20] mandel: yes, runinng tests now [16:20] cool [16:20] nessita: oops, ok, will be sensitive from now on. [16:20] :-) [16:20] mandel: all green, approving. PLease seek a windows review with IRL testing, maybe Chipaca` so ralsina can code :-) [16:21] which branch is it now? [16:21] didn't get to review the last one, mandel, my vm was in use (and i understood ralsina gave you a review, if not please ping) [16:21] Chipaca`: https://code.launchpad.net/~mandel/ubuntuone-client/use_txnamedpipes/+merge/65317 [16:21] Chipaca`: yes, no worries, I ping people as I need :) [16:21] Chipaca`: I would love if you could run thar IRL [16:22] that* [16:22] (on windows, of course) [16:22] lunch, bbiab [16:22] what is "IRL" in this context [16:22] nessita: ? [16:22] in real life. Like grabbing syncdaemon and making it run [16:22] mandel: is thata possible, right? ^ [16:23] um [16:23] mandel: how do i run tests on windows? [16:24] nessita: Chipaca`: using this you can make it run IRL [16:24] Chipaca`: how to run the tests is given in the proposals [16:28] mandel: you're right, apologies [16:28] Chipaca`: no worries, is normal :) [16:30] ugh, no storageprotocol [16:30] * Chipaca` branches [16:33] thisfred, I don't see crashes in that attached pastebin log. [16:34] rye, is there a bug # for the dc brokenness in oneiric? [16:35] CardinalFang: hmm, maybe I was thrown off by the 'Crash report' [16:35] CardinalFang: ok, then it seems it's just running? On 10.10 we didn't have the threeway fork and respective logs yet or did we? [16:36] thisfred, yeah, those are crashes, but normal erlang way of handling errors. Nothing that causes it not to listen on a port. [16:37] right [16:37] thisfred, only two then. [16:37] ok [16:37] one for replication, one for DBus server. [16:40] I guess the dbus is what we need [16:40] rodrigo_: what brokenness? [16:41] Chipaca`, couldn't authenticate errors, rye told me it was broken in oneiric [16:41] Chipaca`, the keyring access, iirc [16:46] rodrigo_: hmm... is this keyring issue different from the ubuntu sso issue around the keyring changing its api? [16:46] Chipaca`, not sure, it might be the same [16:46] Chipaca`, I just get 'unauthenticated' errors when accessing it from evolution [16:52] rodrigo_: right, i don't know what bit is giving you that. We have some issues with several system services changing their api (for no good reason, i must add) [16:53] Chipaca`, yeah :) [16:53] ralsina: can you take a look at https://code.launchpad.net/~mandel/ubuntuone-client/use_txnamedpipes/+merge/65317 [16:53] mandel: tests run and work ok. on to IRL testing ... [16:53] mandel: wasn't chipaca looking at it? [16:54] Chipaca`, ralsina: oh cool, I'll wait then [16:54] ralsina: i am [16:54] mandel: how do i test IRL? [16:55] mandel: just running u1-syncdaemon? [16:55] probably not :) [16:55] Chipaca`: you can do it with the following branchhttps://code.launchpad.net/~mandel/ubuntuone-client/provide_windows_start_script [16:56] Chipaca`: you will need to get your credentials for u1 and put the in the fault, you need sso running for that [16:56] mandel: half through that you started sounding like "no, you can't do IRL testing yet" [16:57] nessita: how important are IRL tests? [16:57] nessita: if I have to do all the above, IRL tests fail :) [16:57] Chipaca`: I'd say very, but I see your point. I guess we can make IRL "mandatory" after that last branch (the one that provides de start script) is in place [16:58] what puzzles me is how we were testing branches before having a start script! :-) [16:58] mandel: how do you run your syncdaemon without the start script? [16:58] nessita: I walways had the start script with the credentials harcoded for test, that is why I create the next branch [16:59] ah... [17:01] mandel: +1 [17:05] dobey: seems like you have u1client nightlies outdated? tarmac fails with: "Instance of 'SyncDaemonTool' has no 'is_autoconnect_enabled' member" and the branch that provides that landed last week [17:05] dobey: can you please confirm and re-approve https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-folder-mgmt/+merge/65281 ? [17:05] * mandel walks dog [17:07] can I have a couple of reviews for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/simpler-devices-backend/+merge/65368 ? [17:10] GASP! [17:10] syncdaemon running on windows [17:11] I don't know if it's *working*, but it's running :) [17:11] Can I have two reviews for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/ui_files_1/+merge/65371 ? Runs only on Linux ATM [17:11] and it certainly looks like it works :) [17:11] nessita: on it! [17:11] ralsina: looking [17:12] nessita: sorry about the length but I did the minimum I could :-( [17:12] ralsina: under 1000 is completly ok [17:12] yours is 809 [17:12] nessita: forget it, it has a lot of cruft in ubuntuone_installer/gui/__init__.py that shouldn't be there [17:12] nessita: reviewing [17:12] ralsina: also [17:13] ralsina: so, your branch needs fixing then? [17:13] nessita: or rather, lots of it *should* be there but I don't know which parts yet [17:13] ralsina: I'm not following, can you please restart from the top? :-) [17:13] nessita: it has too many constants, other than that it should be ok-ish [17:14] nessita: ok, that branch replicates most of the structure of the controlpanel project into windows-installer. I may have copied a bit too much ;-) [17:14] ralsina: right, can you please clean up a bit? [17:15] nessita: I can remove most of ubuntuone_installer/gui/__init__.py. I will ask again in a few minutes [17:15] ralsina: please. Also, we do not versionate (under bzr) the ui.py generated files, so you need to remove those and set a .bzrignore rule [17:15] nessita: ok [17:19] ralsina: you didn't link your branch to a bug [17:19] Chipaca`: oops, will do in 1' [17:20] ralsina: also, no such file "ubuntuone-installer-qt" [17:20] ralsina: bzr ci --fixes=lp:123456 [17:20] ralsina: maybe you forgot to add it? [17:21] Chipaca`: very likely! [17:21] no é así, loco, no é [17:21] :-p [17:21] ralsina: let me know when to re-pull [17:22] Chipaca`: now [17:22] * ralsina is so rusty his chair is getting red [17:23] ralsina: can I also repull? [17:23] nessita: sure [17:23] ralsina: I mean, did you clean up already? [17:23] nessita: yes [17:23] great [17:23] * nessita is leaving in 10 minutes [17:24] ralsina: the ui dir is still there: [17:24] +N ubuntuone_installer/gui/qt/ui/ [17:24] +N ubuntuone_installer/gui/qt/ui/__init__.py [17:24] nessita: yes... shouldn't it be there, not even empty? [17:25] ralsina: nopes [17:25] you can check the controlpanel tree [17:25] nessita@dali:~/canonical/u1/controlpanel/trunk$ bzr st ubuntuone/controlpanel/gui/ui [17:25] nonexistent: [17:25] ubuntuone/controlpanel/gui/ui [17:25] bzr: ERROR: Path(s) do not exist: ubuntuone/controlpanel/gui/ui [17:25] ok, removing it completely then [17:26] ralsina: let me know [17:26] ralsina: import error: from ubuntuone_installer.gui.qt.main import windows: cannot import name windows [17:26] Chipaca`: only on linux right now [17:27] windows in an hour or two after I have a page in it [17:27] nessita: ui folder gone [17:28] ack [17:29] nessita: did you set PYTHONPATH=. ? [17:29] eh? [17:29] ralsina: clicking on cancel does not stop the reactor [17:29] nessita: just finished an apt-get upgrade [17:30] you need to fix that in the same way that controlpanel does [17:30] nessita: this just opens a wizard. I'll file a bug for that one [17:30] ralsina: ack [17:30] dobey: does that mean I can re approve? [17:31] nessita: i just did [17:31] ah, you did [17:31] thanks [17:31] ralsina: can you please clean up all the copied docstrings? """The user interface for the control panel for Ubuntu One.""" :-) [17:31] nessita: filed bug #800282 [17:32] Launchpad bug 800282 in ubuntuone-windows-installer "Closing the window should stop the reactor (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/800282 [17:32] ralsina: grepping -i for "control panel" should give nothing ;-) [17:32] nessita: sure thing! [17:32] ralsina: also remove def uri_hook(uri): [17:32] nessita: if control panel uses it, we probably need it on installer too [17:33] ralsina: don't think so, is too specific [17:33] anyways, is better practice to add the stuff we need when we need it [17:33] nessita: ok [17:33] otherwise we can easily forget and crowd the project with unedeed stuff [17:33] * alecu is back [17:34] hey alecu! [17:36] nessita: removed references to control panel, removed the hook [17:36] yeyyyy [17:39] ralsina: approving but I'm hating the code duplication in logger,py, can you please file a bug to improve that later? :-) [17:39] nessita: we really should have a libary of random useful stuff [17:39] indeed [17:41] nessita: filed bug #800284 [17:41] Launchpad bug 800284 in ubuntuone-windows-installer "logger.py is duplicated from the control panel (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/800284 [17:41] great [17:41] approved [17:42] nessita, ralsina: btw, are we putting a "[] Start Ubuntu One at Log-in" option to cp this week? [17:42] dobey: this week? doubt it [17:42] nessita: thanks! [17:42] hmm [17:43] ok, I'm off to the Uni [17:43] bye all, see ya in a couple of hours! [17:43] bye nessita [17:44] EOD for me I think [17:44] ralsina: did you see that im doing something for josh [17:44] fagan: didn't, please tell me! :-) [17:45] ralsina: im reformatting the FAQ from moin to rst [17:46] fagan: cool [17:46] * ralsina hearts rst [17:46] ralsina: never heard of it [17:46] fagan: check rst2pdf.googlecode.com :-) [17:47] but its not the first time I had to convert stuff, I had to change the quickly docs from html to docbook [17:47] ralsina: nice [17:47] * mandel back [17:47] well, compared to docbook, rst is heaven [17:48] ralsina: I did it by hand :) [17:48] there is something called pandoc that may save you some effort. Not sure it understands moin though [17:49] ralsina: it was a problem with the code we had in the docs. It messed with all the automated tools [17:49] fagan: it happens. Hopefully this time it's easier [17:49] mandel, ping [17:49] and there was some othr formatting bits too that I forget :) [17:50] alecu: pong [17:50] mandel, in provide_windows_start_script, in run_test.bat: how do I make u1trial use the correct reactor? [17:50] ralsina: yeah well its not too hard by the looks of it so I can sort it fast enough and get back to figuring out how the keyring bits are not working in 11.10 [17:50] mandel, in my env it's trying to use the glib reactor :-( [17:50] alecu: you have to use the —reactor parameter [17:50] anway EOD later all [17:51] anyway [17:51] alecu: so, u1trial —reactor=txnp test_path [17:51] bye fagan! [17:51] mandel, perhaps it should be inside the .bat? [17:51] alecu: yes, it should, I must have forgotten to update it [17:51] stupid me... [17:52] alecu: but I need to fix that branch conflicts first, your CredsManagementTool landed first and got some stupid conflicts, easy to solve, but has to be done :P [17:52] mandel, sure, go ahead. [17:53] anyway, I'm getting "the path to test does not exist" now :-( [17:53] alecu: shall we add a bug stating that run-test is bad and we add it to that branch? [17:53] mandel, sure, I'll add it and I'll get to work on it later. [17:54] alecu: superb [17:55] ok, lunch break for me, be back in 30 or so [17:56] mandel, I'll work on in on another branch. Didn't realize that the .bat was there before your branch. [17:57] * alecu brbs [17:57] Chipaca`: can you re-give me the +1 for this https://code.launchpad.net/~mandel/ubuntuone-client/provide_windows_start_script/+merge/65376 [17:58] Chipaca`: I had to resubmit the merge proposal because the dependencie sin the branches were given very strange conflicts [17:58] mandel ther eyou go [17:58] Chipaca`: txh! [17:58] thx :P [18:02] alecu: can you test this branch: https://code.launchpad.net/~mandel/ubuntuone-client/provide_windows_start_script/+merge/65376 [18:02] alecu: is a milestone, it means that in truck we have sd working on windows, we just have to make it easy for people to have the credentials [18:03] mandel, sure, I was already taking a look at that branch. [18:04] dobey: is tarmac merging ubuntuone-windows-installer? [18:04] alecu: awesome, I'm going for rugby (is 7 pm here) but I'll be back in a few hours and will propose the sdtool branch, will try to ping you and nessita about it [18:04] ralsina: no [18:04] mandel, ack [18:04] alecu: if you are ok with that branch, can you approve it so that it lands while I'm at rugby? [18:04] dobey: ok [18:06] dobey: could it? ;-) [18:07] ralsina: maybe? i don't know anything about u1-windows-installer's requirements for landing branches. are there tests we have to run? what else does it require? can it be done on linux? [18:07] dobey: yes to linux, no tests yet, just pep8 and u1lint [18:08] ok [18:08] dobey: requirements same as for control panel [18:08] then i guess we can add it to tarmac, sure [18:08] dobey: cool thx === mandel is now known as mandel_rugby [18:09] hrmm [18:09] i wonder if there is an easy way to do a mass deactivation of milestones on lp [18:09] probably "write a script" [18:10] dobey: cool, right now is only me working on it but that may/will change eventually and tarmac would be handy [18:11] ralsina: i understand. i don't care how many people are contributing to it. all i care about are the requirements for landing branches in it :) [18:12] contrib agreement, tests, etc… stuff [18:12] dobey: ok. Will this be dome soonish (say, next day or two) or should I figure out how to manually merge to trunk? ;-) [18:14] hrmm, need to move the branches too. [18:15] ralsina: i would say by eod today [18:15] dobey: cool then, thanks you so much [18:16] sure [18:18] ralsina: don't propose any other merges to it until i get stuff moved around and tarmac set up though. :) [18:18] dobey: sure thing [18:19] I can push branches but not propose merges, right? [18:20] ralsina: let's hold on that too, but should only take me a few minutes to get this set up [18:21] dobey: ok, sorry then because I pushed one a few minutes ago. No more, I promise [18:21] heh, ok [18:24] dobey: that was quick :-) [18:26] i haven't set up tarmac yet. i merged your branch by hand to avoid a slew of other migration problems :) [18:32] ralsina: do you have a branch that adds run-tests/run-tests.bat which only do the pep8/u1lint checks (preferably using USE_PYFLAKES=1)? [18:32] dobey: don't have it [18:32] dobey: could have it! [18:32] that's the spirit! :) [18:33] ralsina: i've got trunk moved to the right team ownership, and set up tarmac to merge the branches. but there's no run-tests yet, so haven't set it up to run one :) [18:33] ralsina: but as soon as there's a branch adding that, i can add the config option to run it [18:33] dobey: that's ok for today I will have one useful tomorrow [18:34] ok, well it's set up :) [18:35] dobey: cool, thanks! [18:35] np [18:35] now back to evil dbus API changes [18:36] hrmm, and calling this landscaping co. [19:02] hey all, I'm having a configglue issue on windows: http://pastebin.ubuntu.com/630494/ [19:02] anybody has any idea about that? [19:03] Chipaca`, ^ did you have that issue running mandel's branches? [19:30] hrmm. having trouble understanding some of the txsecrets tests [19:36] Can I get two quick linux reviews for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/ui_files_2/+merge/65388 ? [19:46] alecu, nessita: ^ ? pretty please? [19:46] looking [19:48] +1 [19:48] alecu: thx! [19:48] dobey: awesome [19:49] can i steal alecu after he does that review, for a bit? the txsecrets tests in sso are confusing me [19:50] dobey: fine by me if alecu is willing [19:52] ok [19:56] ralsina, I see that you branch has pylint annotations. How are you running pylint on it? [19:56] alecu: have you a few minutes to help with that? [19:56] alecu: manually [19:56] alecu: u1lint ubuntuone_installer [19:57] ralsina, perhaps you should make that part of the run_tests.bat for the installer. [19:57] ralsina, anyway, approving. [19:57] alecu: yes, have a bug for that [19:57] alecu: thanks! [19:57] cool [19:57] dobey, tell me [19:58] alecu: so i'm working on fixing the properties issue in txsecrets (bug #800294), and running the tests i'm getting a few failures/errors, but can't really tell why [19:58] Launchpad bug 800294 in ubuntu-sso-client (Ubuntu) (and 2 other projects) "Support new fdo secrets API property names (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/800294 [19:59] hold on, phone [20:02] I have a weird problem with https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/ui_files_2/+merge/65388 .. I can only set it to work in progress or merged [20:02] alright [20:03] alecu: these are the current errors i have: http://pastebin.ubuntu.com/630510/ [20:03] alecu: i'm not sure why the first one is failing, it seems like it is expecting the wrong thing, from my reading of the code [20:04] ralsina: oh, right, one second, i need to fix up the reviewers [20:04] forgot to do that :) [20:05] dobey, is this happening on your new branch? [20:05] alecu: yes [20:05] ralsina: fixed; you should be able to set it now after you reload the page [20:06] dobey, do you want to push it, so I can take a look? [20:06] dobey: thanks! [20:07] alecu: lp:~dobey/ubuntu-sso-client/new-kr-props [20:13] No rush, trivial branch for review: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/stop_reactor/+merge/65391 [20:15] ralsina: does super() not work right in Qt? [20:16] dobey: I am old fashioned [20:16] well, you are writing Qt and Python, so yes :) [20:20] alecu: any ideas [20:20] ? [20:20] dobey, get_default_collection creates a new collection if it can't find it on the default path. The first test creates a sample collection on the default, then tests get_default_collection. [20:21] dobey, since the collection was already created at the start of the tests (before the first assert), calling get_default_collection should not create a new one. [20:21] dobey, so I think that testing twice for 1 is right. [20:22] alecu: but why would my changes affect that, given i'm not changing that path? [20:23] looking again. [20:29] dobey, I have not followed it properly, but I would suspect it's related to the splitting of LABEL_PROPERTY into ITEM_LABEL_PROPERTY and CLXN_LABEL_PROPERTY [20:32] another trivial branch for review: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800346/+merge/65397 [20:33] alecu: huh [20:34] ralsina: approved [20:34] dobey: thanks! [20:38] ralsina, approved both [20:38] alecu: thanks! [20:38] hmmm [20:42] * nessita is back [20:42] hello nessita [20:42] alecu: i'm not sure, since i also updated the usage of that constant appropriately as well [20:43] nessita: I looked and control panel does have the gui/qt/ui/__init__.py file you made me delete :-) [20:44] ralsina: it has the file but is not versioned [20:44] nessita: you lost me [20:44] apparently you just run into a hole in my bzr knowledge [20:44] I just got here [20:44] :-D [20:45] ralsina: hum, I see what you mean. Then you were right, somehow bzr told me is was not versioned but actually is [20:45] ralsina: sorry for the confusion [20:45] nessita: no problem whatsoever [20:46] dobey, right. But I guess that the mocks rely on both being the same, for looking up the objects in their dicts [20:46] or something like that. [20:47] yet another very simple review: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800330/+merge/65400 [20:47] ralsina: can you please review https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/simpler-devices-backend/+merge/65368 ? [20:47] nessita: sure, trade you ;-) [20:47] sure! [20:48] ralsina: are you using --fixes? [20:48] * nessita guesses not [20:48] nessita: yes, unless I forgot in one branch [20:48] alecu: i haven't found anywhere that suggests that :-/ [20:48] ralsina: the 800330 is not bind to the bug report, can you please ci --fixes --unchanged, please? [20:49] ralsina: and I'm not sure that we're having a GTK version of the installer, aren't we? [20:49] nessita: we are probably never going to have one [20:49] nessita: but better safe than sorry [20:49] ralsina: can we delete all the GTK specific bits? [20:50] we can add them later, and ideally in a way we ran tests for both all the time [20:50] nessita: ok, file a bug and I will do it. I will still keep the directory structure so we can add a gtk/ later [20:50] nessita: bug linked [20:50] ralsina: makes sense about the file dir. But the GTK bits please remove them in this branch [20:50] ralsina: since it will make the diff smaller ;-) [20:51] nessita: ok [20:51] (is not like is big, but I see no point in adding stuff that we will not use) [20:52] nessita: removed and pushed [20:52] thanks [20:53] ralsina: hey! do not assign me to bugs from the installer, assign yourself! :-P for example: bug #800284 [20:53] Launchpad bug 800284 in ubuntuone-windows-installer "logger.py is duplicated from the control panel (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/800284 [20:53] nessita: hey, you own logger.py ;-) [20:53] ralsina: nopes, you own installer till friday at least [20:53] nessita: ok, I will reassign to me [20:53] ralsina: also, please be sure to not leave the bugs in NEW. So, please set importance and status to either TRiaged or In Progress [20:54] already changed assignee [20:54] ralsina: and also, to measure the work done/pending till Friday, we're tagging bugs with u1-zomg-windows, like I mentioned in the first email I send you last week [20:55] nessita: ok, will tag [20:55] that way we can search globally and not only by project [20:55] grr, i don't understand why this is breaking :( [20:55] I thought they changed to in progress when you attached a branch. Oh well. [20:56] ralsina: nope, but change to fix committed when tarmac lands branches that are attached to bugs [20:56] dobey: ok [20:56] ralsina: so given the ideal velocity of our branches, the "in progress" is not as useful if you've already made the branch to fix it :) [20:56] dobey: yeah, in an ideal world ;-) [20:57] well in an ideal world we wouldn't be touching windows :) [20:57] and i wouldn't be writing python ;) [20:58] ralsina: in the script in your last branch does: `which xvfb-run` u1trial --ignore-paths $QT_TESTS_PATH "$MODULE" [20:58] now why is this thing breaking :( [20:58] ralsina: I don't think we want to ignore the wt tests [20:58] qt* [20:59] oops, deleted wrong line, I suppose [20:59] i don't think we want to run the tests on linux? [20:59] dobey: there are no tests yet, but yes, they will have to run [20:59] ralsina: we'll have to run windows installer tests on linux? [21:00] dobey: it's actually a setup wizard [21:00] right. for windows. :) [21:00] dobey: should work just the same on Linux [21:00] dobey: no, for both [21:01] how many control panels do we need? :P [21:01] is it an installer or what? [21:01] ralsina: I would suggest to remove the definition of QT_TESTS_PATH altogether since we don t need that [21:01] dobey: not a control panel. Just a setup wizard. You know they are not the same thing :-) [21:01] dobey: is a wizard to setup Ubuntu One [21:01] nessita: makes sense [21:01] what does "setup" mean here?! install? sign up on sso? [21:02] dobey: you can see the wireframes between our (canonical) google docs [21:02] dobey: choose what to sync, buy storage/streaming, sign in, a few other things. Check the workflow on gdocs [21:02] *facepalm* [21:02] dobey: we really need to get things moving on and we don't have time to argue, sorry [21:03] dobey: not constructive. This has been discussed for months. Facepalms were only allowed until april. [21:03] dobey: you are welcome to send an email to the list, though [21:04] nessita: removed the variable and pushed [21:04] ralsina: ack. Any news on my branch? :-) [21:04] (I'm depending on it) [21:04] nessita: reading it [21:04] thanks! [21:04] nessita: it's a tad longer than mine ;-) [21:05] nessita: do you need any specific testing? [21:05] no, the next one will expose this in the IO [21:05] UI [21:05] nessita: ok [21:06] ralsina: approved and globally approved (we decided with the rest of the windows team that trivial branches can land with one review) [21:06] nessita: cool [21:06] nessita: +1 on your branch [21:06] thanks! [21:06] I just ran the tests and did a quick code review, though [21:07] * ralsina adds bugs for each page on the installer now [21:18] ah-ha [21:18] stupid string literals [21:23] If I wanted to ask U1Files to return a special error code for "out of space", where do I file that bug? [21:28] mterry: ubuntuone-servers i suspect (presuming you mean the REST API) [21:29] hrmm, so it seems like these properties changes work ok with gnome-keyring 2.x also [21:30] dobey, thanks; filed bug 800384 [21:30] ralsina, alecu: do you know if mandel landed everything he had to? does he need reviews? [21:30] mterry: Bug 800384 on http://launchpad.net/bugs/800384 is private [21:31] hrmm, and i wonder if i can make dnsmasq give out both ipv6 and ipv4 to the same MAC [21:31] nessita: not sure. Don't remember him asking for reviews except one that chipaca was doing [21:32] nessita: that was https://code.launchpad.net/~mandel/ubuntuone-client/use_txnamedpipes/+merge/65317 and it's merged now [21:32] right, he also needed to propose the start script [21:33] nessita: checking... [21:33] nessita: that one is approved [21:33] Chipaca`: any idea if the start script for syncdaemon, from mandel branch, is working? asking differently, did he request a review from you for that branch? [21:33] nessita: https://code.launchpad.net/~mandel/ubuntuone-client/provide_windows_start_script/+merge/65376 [21:33] but... HAS NO COMMIT MESSAGE [21:34] oops, sorry about the capshouting [21:35] ralsina: let me set it [21:35] ralsina, nessita: I've just set it [21:35] oh [21:35] nessita, you won. [21:35] oh [21:36] jajaja [21:36] alecu: did you explain to mandel that CredentialsManagementTool is not meant to be used like that? [21:47] ralsina: I hope you don't get tired of this but, you're referencing "2nd page of the installer design" in the bug reports (which are publics) and the design specs are not visible [21:47] nessita: I intend to fill those up with screenshots from the wireframe when I start them [21:47] nessita: I had to file 14 of those :-( [21:48] ralsina: is there any chance to not reference the spec but to describe very shortly what the tab does? Example: "Show licence agreement on 2nd tab, cancel if not accepted" [21:48] "Show folder list so user can subscribe/unsubscribe as s/he wishes" [21:48] etc? [21:48] nessita: yes there is a chance [21:48] nessita: I'll do it, even [21:48] thanks [21:48] ralsina: no need to fix what's done [21:49] ralsina: only what's coming next [21:49] nessita: so only 13 of those ;-) [21:49] :-D [21:49] nessita: don't wprry I will make all the bug reports pretty [21:49] I'm glad I noticed soon enough! [21:49] ralsina: or you can make teh bug reports as you work on the tabs [21:50] nessita: I wanted to create them even if as placeholders so there was a visible goal [21:50] right, makes sense [21:50] nessita: I can flesh them up as I work on them though [21:50] right [22:10] nessita: what's the proper way to start the control panel? I am guessing it's not Popen(['/usr/bin/control-panel-gtk']) [22:10] ralsina: start it from where? [22:10] nessita: the installer, after it finishes [22:10] it shows a button saying "start the dashboard" [22:11] then yes, subprocess.Popen, but you can't hardcode the abspath [22:11] assuming the controlpanel is installed in both systems, they should be in the PATH (the .msi should take care of that in windows) [22:11] I can use shell=True... but that's not usually recommended [22:11] ok [22:11] ralsina: why would you use shell=True? [22:12] nessita: because if you don't do that, the PATH is ignored [22:12] lies [22:12] :-) [22:12] In [1]: import subprocess [22:12] In [2]: subprocess.Popen(('ubuntuone-control-panel-gtk',)) [22:12] works [22:12] it does not work on windows, assuming the panel is in the PATH env? [22:13] * ralsina checks [22:13] shocking! ;-) [22:13] nessita: should be exactly the same on windows. Seems I was just wrong. It happens :-) [22:14] yes, it happens. Anyways, we should confirm this works on W [22:14] OTOH, the *name* ubuntuone-control-panel-gtk is platform dependent (not really but kinda) [22:14] yes, works on windows [22:15] right [22:15] ralsina: you need to use ubuntuone-control-panel-qt on linux and ubuntuone-control-panel-qt.exe on windows [22:15] we should use alternatives to set a ubuntuone-control-panel [22:15] that is what you mean? === m_conley is now known as m_conley_away [22:16] no, the "-gtk" part is currently linux only [22:16] right, that is why I said -qt [22:16] ok then, yes that. [22:16] eventually you can do -qt on linux too, of course [22:17] Can I connect a branch to a bug in a way that doesn't say "this branch fixes the bug"? [22:17] ralsina: no... sorry [22:17] nessita: no problem [22:17] you need to manually set it to In progress [22:17] or create finner grained branches or bugs [22:18] bugs, I mean [22:23] * alecu heads for the pediatric. bbl [22:28] last review request of the day: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix_800383/+merge/65416 [22:32] https://code.launchpad.net/~dobey/ubuntu-sso-client/new-kr-props/+merge/65419 [22:32] my review request :) [22:33] ralsina: you need to add tests :-) [22:33] nessita: yes i do! [22:34] great! we; re in sync then [22:34] ralsina: needinfo about the subprocess.Popen bit :) [22:34] dobey: such as? ;-) [22:34] Should this not be ubuntuone-control-panel-qt, given that we're running it from a qt gui? Also, should we not rely on IPC activation to just start the correct one for the user's environment, instead of running it directly? [22:34] ralsina: our policy is to add tests for all the code we write, not about what we do in designer. Anyways, we don't need to add any complex bit in designer other than layout and widgets packing [22:35] nessita: there is no logic in designer [22:35] dobey: for now we will be running it directly, since we won't be doing a IPC activation for QT [22:36] ralsina: right, which is good [22:36] I was explaining the policy [22:36] dobey: I agree we should open the -qt panel [22:36] nessita: so yes, I will add tests, but I need to get a handle on how and how much and "have the time" :-( [22:36] ralsina: hem... [22:37] ralsina: let's drop the tests when we really need to [22:37] ralsina: I think that today, we can ask you to write tests :-) (still) [22:37] nessita: ok, will try to do it tonight late [22:37] ralsina: thanks, you can check our current test suite to have a guide [22:37] they are not long nor complicated [22:38] but they are a safety net that I think is very important for "it just works" [22:38] my branch already has tests and i updated them ;) [22:38] nessita: agreed [22:39] ok, time to be a dad [22:39] dobey: I will try to do a review later, sorry :-( [22:39] no worries. i'm happy i got the branch working ;) [22:40] :-D [22:40] oops, dogs need to be walked, later all [22:40] see you later/tomorrow all! [22:40] well i'm off. have a good evening all [22:40] ralsina: wait [22:40] ralsina: if you eod, please send the report [22:41] is very important, we're very very tight with the schedule [22:41] dobey: I will review your branch pretty soon