| karni | Night everyone! | 00:02 |
|---|---|---|
| alecu | ralsina, cool! | 00:05 |
| ralsina | alecu: then the "watch in | 00:05 |
| alecu | ralsina, I've proposed it for merging, I'll send an email asking for reviews tomorrow. | 00:05 |
| ralsina | ~" kicked in and everything stands still | 00:05 |
| ralsina | alecu: cool | 00:05 |
| alecu | re: "watch-in" great! | 00:05 |
| ralsina | If mandel has the watches fix, I think it's a good point for a release of some sort | 00:06 |
| ralsina | alecu, doesn't your branch mean we now need to do a branch on u1cp to connect only to the signals we need? | 00:08 |
| === yofel_ is now known as yofel | ||
| alecu | ralsina, no, we don't need that. The ipc client is all contained on the ipc_client.py inside syncdaemon, and that's what's used inside u1cp | 00:22 |
| mandel | morning all! | 07:06 |
| === zyga-afk is now known as zyga | ||
| karni | Good morning guys | 10:52 |
| rodrigo_ | hi | 10:54 |
| rodrigo_ | who works on the ubuntuone-installer? | 10:55 |
| karni | mandel ↑ ? | 10:57 |
| rodrigo_ | I should have guessed it was mandel, as it's always him who breaks stuff :D | 10:59 |
| karni | rodrigo_: I meant "mandel, do you know?", I'm sure ralsina or nessita will know, but I don't think they're around yet. | 11:00 |
| rodrigo_ | ah :) | 11:00 |
| karni | :) | 11:00 |
| rodrigo_ | ok, I'll ping them later | 11:00 |
| mandel | rodrigo_, why? | 11:01 |
| mandel | rodrigo_, I'm sure I can give you a hand :) | 11:01 |
| mandel | aunque a un desertor? ni agua ;) | 11:01 |
| rodrigo_ | mandel, the .desktop file is wrong, it has the same categories as the ubuntuone-control-panel-gtk.desktop file, so they both show up on the control center | 11:02 |
| rodrigo_ | also, the X-GNOME-Settings-Panel should be removed from ubuntuone-installer.desktop | 11:02 |
| rodrigo_ | mandel, :D | 11:02 |
| mandel | rodrigo_, in which project is that? in the ubuntuone-control-panel or in the unbutuone-windows-installer? | 11:04 |
| rodrigo_ | hmm, no idea, let me check | 11:04 |
| rodrigo_ | $ dpkg -S /usr/share/applications/ubuntuone-installer.desktop | 11:05 |
| rodrigo_ | ubuntuone-installer: /usr/share/applications/ubuntuone-installer.desktop | 11:05 |
| nessita | good morning! | 11:07 |
| ralsina | dobey: worked on it, but he's on holiday | 11:08 |
| ralsina | and no I am not working yet ;-) | 11:08 |
| mandel | rodrigo_, yep, that is the new installer being added for linux so that we do not ship in the cd... | 11:08 |
| mandel | rodrigo_, we haves sooooo many projects? and I've been told you are porting it to meego, right? | 11:09 |
| mandel | :P | 11:09 |
| rodrigo_ | no, to Windows 3.1 | 11:09 |
| rodrigo_ | the installer shouldn't be on the control center | 11:09 |
| mandel | ralsina, this is for you: https://code.launchpad.net/~mandel/ubuntuone-client/eq-add-ancestors-watches/+merge/73775 | 11:10 |
| mandel | ralsina, can you do a IRL to see if its ok :P | 11:10 |
| mandel | ralsina, that should fix the watches issues | 11:10 |
| mandel | rodrigo_, do not say to high :) | 11:10 |
| rodrigo_ | :) | 11:10 |
| mandel | ralsina, ping | 11:14 |
| facundobatista | Hola! | 11:16 |
| mandel | facundobatista, buenas! | 11:17 |
| mandel | facundobatista, do you need reviews? | 11:17 |
| facundobatista | mandel, not for client, thanks | 11:18 |
| mandel | facundobatista, cool | 11:20 |
| mandel | ralsina, ping | 11:27 |
| * nessita starts the review run | 11:27 | |
| nessita | gatox: ping | 11:30 |
| gatox | nessita, pong | 11:32 |
| nessita | gatox: can you please attach the bug # to https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/reset-password-page/+merge/73522? | 11:32 |
| gatox | nessita, ok | 11:33 |
| nessita | gatox: and one ssmall fix to reset-password in sso: I know this is correct because is alphabetical ordered: | 11:37 |
| nessita | from PyQt4 import QtCore, QtGui | 11:37 |
| nessita | but this causes some times ugly errors like: | 11:37 |
| nessita | basenames = [base.__name__ for base in member.__bases__] | 11:37 |
| nessita | TypeError: C++ type 'QWidget*' is not supported as type | 11:37 |
| nessita | gatox: so, I'm approving, but before landing, change the Qt import to: | 11:37 |
| nessita | from PyQt4 import QtGui, QtCore | 11:38 |
| gatox | nessita, weird....... ok....... changing | 11:38 |
| gatox | nessita, done | 11:39 |
| nessita | mandel: ping | 11:42 |
| nessita | mandel: are you reviewing alecu's branch? | 11:42 |
| mandel | nessita, yes, just added a needs fixing | 11:44 |
| mandel | nessita, I think that adding all the remote_calls is dangerous since the signals are a subset of all the remote calls | 11:45 |
| mandel | nessita, I pushed changes to https://code.launchpad.net/~mandel/ubuntuone-client/add-method-name/+merge/73400 and added the new MP I mentioned in the email, please feel free to hurt my feelings ;) | 11:46 |
| nessita | ok | 11:48 |
| mandel | nessita, I need to walk the dog, any comments can you add them to the MP :) | 11:48 |
| nessita | sure | 11:48 |
| mandel | I'll me 30 min or so | 11:48 |
| * mandel walking beast | 11:48 | |
| nessita | ralsina: ping | 12:14 |
| gatox | nessita, we talk yesterday that the size of the wizard should be: width=736.... and, do you have in mind which should be the proper height? | 12:22 |
| nessita | mandel: did you test IRL the ancestors branch? | 12:31 |
| nessita | gatox: ping | 12:39 |
| nessita | gatox: sorry, I just saw your question | 12:39 |
| gatox | nessita, pong | 12:39 |
| nessita | gatox: ideally, 736x525 | 12:40 |
| ralsina | nessita: pong | 12:40 |
| nessita | gatox: can you please propose a quick branch for ussoc to solve this? | 12:40 |
| nessita | == Python Lint Notices == | 12:40 |
| nessita | ubuntu_sso/qt/tests/test_reset_password.py: | 12:40 |
| nessita | 24: [F0401] Unable to import 'ubuntu_sso.qt.reset_password_ui' | 12:40 |
| nessita | 24: [E0611] No name 'reset_password_ui' in module 'ubuntu_sso.qt' | 12:40 |
| nessita | gatox: nightlies are not building because of that | 12:40 |
| gatox | nessita, ok | 12:40 |
| gatox | nessita, nice!!!!! now the size is: 740x538 | 12:40 |
| nessita | gatox: GREAT! | 12:41 |
| nessita | that's GREAT | 12:41 |
| nessita | gatox: I think there is no need to put more time in this for now | 12:41 |
| gatox | nessita, ok...... working in the branch you mention now | 12:41 |
| nessita | gatox: thanks (just needs pylint disables) | 12:41 |
| ralsina | alecu, mandel, gatox: Who needs a review? I am selling! | 12:45 |
| ralsina | and nessita too, of curse | 12:45 |
| ralsina | of COURSE | 12:45 |
| * ralsina is not cursing | 12:45 | |
| gatox | ralsina, not yet...... almost done!!! | 12:45 |
| nessita | ralsina: we should talk about this https://code.launchpad.net/~sidnei/ubuntuone-windows-installer/sourcedeps/+merge/73728 in the mumble, before reviewing. I'm not sure if the right solution. | 12:46 |
| gatox | ralsina, nessita review (if you want, just 2 lines to disable and enable lint to fix some lint issues): https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/reset-lint-fix/+merge/73800 | 12:50 |
| nessita | gatox: yes! | 12:50 |
| ralsina | gatox: I got it | 12:50 |
| * ralsina suggests gatox that he doesn't ask for reviews until after the diff is visible ;-) | 12:51 | |
| gatox | ralsina, ok...... understood | 12:51 |
| ralsina | looks like my cat is having kittens uder my desktop. Anyone wants one? | 12:52 |
| alecu | ralsina, I'm sure gatox would be interested | 12:52 |
| ralsina | gatox, want some gatitoxs? | 12:53 |
| nessita | alecu: you're on fire today! | 12:53 |
| alecu | k-ching! | 12:53 |
| gatox | ralsina, sorry..... i already have one! | 12:53 |
| nessita | alecu: question, did you run the windows suite for facundobatista's branch from yesterday? | 12:53 |
| nessita | (client branch) | 12:53 |
| gatox | ralsina, and don't play well with others :P | 12:53 |
| alecu | nessita, very good question. I think I did not. | 12:53 |
| alecu | shame on me | 12:54 |
| ralsina | boy launchpad is slow to parse branches today | 12:54 |
| nessita | alecu: yes, because that branch added 7 failures! already debugged and fixed :-) | 12:54 |
| alecu | nessita, sorry about that :/ | 12:54 |
| alecu | nessita, anyway, I'm still not getting all tests passing around here | 12:55 |
| nessita | alecu: I know... | 12:56 |
| nessita | alecu: we need to debug that (timing issues evdently) | 12:56 |
| nessita | alecu: "good news" is that we can use the same box as parameter, seems pretty reproducible | 12:58 |
| nessita | both the non-failures and the failures | 12:58 |
| ralsina | gatox: I am flipping to approved your reset-lint-fix branch | 13:00 |
| gatox | ralsina, okkk | 13:00 |
| gatox | ralsina, i have my branches to fix the installer size ready...... just waiting for launchpad to scan them | 13:00 |
| gatox | standup? | 13:01 |
| gatox | me | 13:01 |
| ralsina | me | 13:01 |
| gatox | alecu, nessita mandel standup? | 13:02 |
| nessita | me | 13:02 |
| nessita | alecu, mandel? | 13:02 |
| alecu | me | 13:03 |
| nessita | gatox: go! | 13:04 |
| gatox | DONE: | 13:04 |
| gatox | Branch to fix Installer sizes, now the size is: w: 740, h: 538. | 13:04 |
| gatox | TODO: | 13:04 |
| gatox | Keep killing UI bugs. | 13:04 |
| gatox | BLOCKED: | 13:04 |
| gatox | No. | 13:04 |
| gatox | ralsina, go | 13:04 |
| ralsina | DONE: implemented proof-of-concept of the systray icon, worked on stderr fix, lots of IRL testing, reviews, call. TODO: finish stderr, finish systray, reviews, testing, package a release candidate. BLOCKED: waiting for all the needed branches to land before packaging of course | 13:04 |
| ralsina | nessita! | 13:04 |
| nessita | DONE: landed branch for bug #814113, meetings (TOO MANY), reviews | 13:04 |
| nessita | TODO: reviews using the new schema (already did 2 hours of review, now switching to coding!). Work on bug #807021 and bug #834766 (I need to solve both to have a clean solution). | 13:04 |
| nessita | BLOCKED: nopes | 13:04 |
| nessita | NEXT: alecu | 13:04 |
| alecu | DONE: proposed a branch to fix the multiple ipc connections issue | 13:04 |
| alecu | TODO: fix some issues that mandel has regarding this branch | 13:04 |
| alecu | BLOCKED: no | 13:04 |
| alecu | NEXT! mandel | 13:04 |
| * nessita adds TODO: fix nightlies builds | 13:05 | |
| gatox | nessita, ralsina this branches are ready for review: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/wizard-sizes/+merge/73803 - https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/wizard-sizes/+merge/73804 | 13:06 |
| nessita | gatox: queueing those up for after lunch | 13:06 |
| gatox | nessita, ok | 13:06 |
| mandel | orry | 13:06 |
| ralsina | gatox, nessita: I'll do a first review of those now | 13:06 |
| mandel | irc issues | 13:07 |
| gatox | ralsina, nessita the one from sso is really trivial | 13:07 |
| nessita | ralsina, gatox: sso one was trivial, so I just approved it | 13:07 |
| ralsina | gatox: yeah, I am setting that one to approved as soon as I see how it looks | 13:07 |
| ralsina | nessita: beat me to it :-) | 13:07 |
| mandel | DONE: fised issues with add-method-name branch. Proposed fix for the watcher issue. Reviews alecus branch. | 13:07 |
| mandel | TODO: get fixes to possible review. Look at bug assigned to me | 13:08 |
| mandel | BLOCKED: no | 13:08 |
| nessita | mandel: did you test IRL the ancestors branch? | 13:08 |
| ralsina | gatox: what the heck is the difference at line 59 about? | 13:08 |
| * gatox looking.... | 13:08 | |
| gatox | ralsina, thats the content of license page...... | 13:09 |
| ralsina | gatox: yes, but what changed? | 13:09 |
| ralsina | I can't parse it :-) | 13:09 |
| gatox | ralsina, ........ nothing....... | 13:09 |
| gatox | qt designer.... | 13:09 |
| gatox | ralsina, do you want me to revert that and try to apply the change of the margin only? | 13:10 |
| nessita | gatox: can we have that in a variable in the code? | 13:10 |
| ralsina | gatox: nah, I don't mind, I just assumed you had changed the text | 13:10 |
| gatox | ralsina, nono | 13:10 |
| ralsina | good idea, and with reasonable HTML? | 13:10 |
| nessita | gatox: so we can avoid stuff like this | 13:10 |
| ralsina | as in not specify font sizes and MS Shell Dlg font | 13:10 |
| gatox | ralsina, nessita, ok...... moving it to a html format | 13:11 |
| ralsina | gatox: cool | 13:11 |
| mandel | nessita, no, I did not IRL, did you find any issues? | 13:11 |
| mandel | nessita, if you did, the tests are wrong then :( | 13:11 |
| nessita | ralsina: can I have a trivial for https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/build-lucid/+merge/73805 ? (fixes a lucid nightlies build failure) | 13:11 |
| mandel | nessita, I can do it though | 13:11 |
| ralsina | nessita: sure! | 13:11 |
| nessita | mandel: I did not, but I would like you to do an extensive IRL test | 13:11 |
| nessita | mandel: I want you to be confident about the fix :-) Ideally, please first reproduce the bug with trunk, and then use your branch to test | 13:12 |
| mandel | nessita, sure, I'll test with my test accound with different UDFs created to ensure that no watches are added | 13:12 |
| nessita | mandel: thanks! (be sure to first reproduce with trunk) | 13:12 |
| ralsina | nessita: +1 and set it to approved | 13:12 |
| mandel | nessita, sure? although I'm confident hehehe | 13:12 |
| nessita | great | 13:12 |
| mandel | alecu, ping | 13:12 |
| nessita | mandel: I'm not :-D | 13:13 |
| gatox | ralsina, where should i put that variable?? gui.py? | 13:13 |
| nessita | mandel: but I trust no one (not even me) | 13:13 |
| ralsina | gatox: yes | 13:13 |
| gatox | ralsina, ok | 13:13 |
| ralsina | that's where we have internationalized stuff, IIRC | 13:13 |
| nessita | gatox, ralsina: or the gui/qt/__init__.py? | 13:13 |
| mandel | nessita, hahaha I told you you were free to hurt my feelings ;) | 13:13 |
| nessita | where all the other _() are | 13:13 |
| ralsina | where they are :-) | 13:13 |
| gatox | nessita, yep.... maybe that is better | 13:13 |
| nessita | mandel: is not personal, I think we all mae mistakes, always | 13:13 |
| nessita | made* | 13:14 |
| ralsina | gatox: +1 on wizard-sizes, it looks great! | 13:18 |
| gatox | ralsina, thanks! | 13:18 |
| ralsina | I mean +1 after I finish running the tests of course | 13:18 |
| alecu | mandel, pong | 13:24 |
| alecu | mandel, I'm working on fixing the issue you commented. | 13:24 |
| === m_conley_away is now known as m_conley | ||
| ralsina | gatox, everything passes, +1 | 13:32 |
| gatox | ralsina, nice | 13:32 |
| gatox | ralsina, nessita i've moved the license content into a variable as we speak...... now i have to run to the bank!!!!!! | 13:38 |
| ralsina | gatox: ok, I'll take a look now | 13:38 |
| ralsina | I approved it before that. Boy, I miss coffee | 13:38 |
| nessita | ralsina: question | 13:41 |
| ralsina | nessita: shoot | 13:41 |
| nessita | ralsina: is there any way to pass custom params to widgets when creating them building them from ui files? | 13:41 |
| nessita | (not sure if the question is clear) | 13:41 |
| ralsina | not sure what you mean | 13:42 |
| nessita | ralsina: I need to make all widgets have the same backend instance, can I create it in the top level widget and pass that to all the children? | 13:42 |
| ralsina | you can add a parameter to the __init__ of the class that calls setupUi | 13:43 |
| ralsina | but then when you add them with designer that's not used, so no | 13:43 |
| ralsina | you would have to make the widget use it from parent() | 13:43 |
| ralsina | like self.backend = parent().backend | 13:44 |
| ralsina | That way it propagates down the widget tree | 13:44 |
| nessita | ralsina: ok, I'll play with that | 13:47 |
| ralsina | nessita: OR you can use app as a "global singleton" | 13:47 |
| ralsina | Since it's guaranteed that QApplication.instance() will always return the same, you can even do a subclass that has the backend | 13:47 |
| nessita | ralsina: hum... | 13:48 |
| nessita | ralsina: I think I prefer the first | 13:48 |
| ralsina | I think the second one is a bit cleaner, but the first one is easier | 13:49 |
| === gatox is now known as gatox_away | ||
| ralsina | mandel, alecu: need any reviews? | 13:52 |
| mandel | ralsina, besides the watches one no? and I'm doing an IRL sets of tests of it | 13:54 |
| ralsina | mandel: ack | 13:54 |
| mandel | alecu, I just wanted to ask you if it made sense? | 13:55 |
| mandel | alecu, the issue I mentioned I mean :) | 13:55 |
| alecu | mandel, yes, it makes sense. I'm working on it already. | 13:56 |
| alecu | ralsina, not right now, thanks. | 13:56 |
| mandel | alecu, cool :) | 13:56 |
| gatox_away | ralsina, i can't go to the bank until lunch..... do you have any bug that you prefer i take?? or i just pick one from the list? | 14:01 |
| === gatox_away is now known as gatox | ||
| ralsina | gatox_away: the page right after login in the installer | 14:01 |
| ralsina | gatox: it should show a "skip setup" button | 14:02 |
| ralsina | gatox: lisette told me about it but I am not sure it has a bug number | 14:02 |
| mandel | ralsina, alecu, nessita, Chipaca, gatox mumble? | 14:02 |
| gatox | ralsina, ok..... i'll take a look at that | 14:02 |
| gatox | mandel, fine by me | 14:02 |
| nessita | yes! | 14:02 |
| ralsina | Chipaca: mumble? | 14:04 |
| Chipaca | going | 14:05 |
| ralsina | aleeeeeecuuuu alecu | 14:07 |
| alecu | uh, uh, uh | 14:07 |
| ralsina | mumble | 14:07 |
| mandel | alecu, ralsina, nessita I've done the IRL AFAIK the bug is fixed, please review whenever you can :) | 14:28 |
| mandel | ok, I have played around and I have my UDFs working!!! | 14:43 |
| * mandel dances | 14:43 | |
| mandel | nessita, alecu please let me know if you have any issues with that branch :) | 14:44 |
| mandel | ralsina, if there is nothing very urgent I'll continued looking at the bugs that have been asigned to me | 14:45 |
| nessita | mandel: what happened with the bug rick had? | 14:47 |
| ralsina | mandel: could you try a share, too? | 14:48 |
| mandel | nessita, I have not been able to reproduce it, I wanted to talk with rick so that we tried again? I'll try a share and will get to it too see if I can reproduce again? maybe with a bigger file | 14:50 |
| mandel | ralsina, waht do you want me to try? | 14:50 |
| ralsina | mandel: subscribe to a share and see if it's replicated correctly | 14:50 |
| mandel | ralsina, ok | 14:50 |
| nessita | mandel: can you please try to coordinate that with rick? | 14:50 |
| ralsina | usually I got things locked before the share could start to download | 14:50 |
| mandel | nessita, yes | 14:50 |
| alecu | mandel, can you please re-review my branch??? | 14:50 |
| nessita | mandel: he should ne starting his work day now | 14:51 |
| mandel | alecu, will do asap | 14:51 |
| alecu | mandel, that should be urgenter than the bugs | 14:51 |
| ralsina | mandel: could you do my laundry? | 14:51 |
| ralsina | ;-) | 14:51 |
| mandel | ralsina, I can do you mu? wont say that to you ;) | 14:51 |
| mandel | alecu, ok, will look at your branh first, let me try shares first :) | 14:51 |
| mandel | ralsina, I have the aranduka books you shared with me working! | 14:52 |
| mandel | \o/ | 14:52 |
| ralsina | cool! | 14:52 |
| mandel | ralsina, if you want to see if it work in your system here is the mp: https://code.launchpad.net/~mandel/ubuntuone-client/eq-add-ancestors-watches/+merge/73775 | 14:53 |
| mandel | alecu, I'm doing your review now | 14:53 |
| alecu | mandel, please, wait. I've just seen some broken tests. | 14:53 |
| alecu | sorry :P | 14:54 |
| mandel | alecu, ok, I was just doing the code review, so feel free to change it :) | 14:56 |
| gatox | ralsina, nessita another branch for review (help button): https://code.launchpad.net/~diegosarmentero/ubuntuone-windows-installer/help-button-removed/+merge/73834 | 14:58 |
| nessita | gatox: you already went to the bank? | 14:59 |
| nessita | (polite version of stop flooding with reviews :-P) | 14:59 |
| gatox | nessita, nop....... | 15:00 |
| gatox | jejeje | 15:00 |
| gatox | nessita, i have to go at lunch :( | 15:00 |
| nessita | ack | 15:00 |
| alecu | mandel, r1125, with the fixed test | 15:01 |
| mandel | ok | 15:01 |
| ralsina | gatox: I don't understand the test for that branch | 15:04 |
| alecu | mandel, ping | 15:04 |
| gatox | ralsina, just checking that the wizard doesn't have the window flag for the help button | 15:04 |
| alecu | mandel, can it possibly be the case that the code in u.p.w.network_manager is butt ugly? | 15:05 |
| ralsina | gatox: why not just flags & QtCore.Qt.WindowContextHelpButtonHint == 0 | 15:05 |
| gatox | ralsina, ............ shame on me | 15:05 |
| alecu | I believe the code in NetworkManager is the thread that's preventing twisted from stopping. | 15:05 |
| ralsina | gatox: there are 10 kinds of programmers. Those who think about doing bitwise operations, and those who don't. | 15:05 |
| gatox | ralsina, right! jeje | 15:06 |
| mandel | alecu, is ugly, we could make it a lot nicer by hooking it to twisted like we did with the fs | 15:07 |
| mandel | alecu, but that is the way to get that the network is up or down, yet ugly | 15:07 |
| mandel | I was planing to clean that | 15:07 |
| alecu | mandel, I believe the pythoncom.PumpMessage() is never stopping | 15:08 |
| alecu | mandel, because it waits for a WM_QUIT that it's never receiving | 15:08 |
| mandel | alecu, that is possible, that code is ugly ugly | 15:08 |
| alecu | mandel, also: we are sending the "SYS_NET_CONNECTED" event at start, all the time. Even when the network is not connected. | 15:09 |
| gatox | ralsina, fixed :P | 15:09 |
| gatox | ralsina, launchpad not updated yey | 15:10 |
| gatox | yet | 15:10 |
| mandel | alecu, always? well that is not a problem due to the sd that will try to connect and twisted would say no no | 15:10 |
| ralsina | gatox: also, line 10 looks wrong | 15:10 |
| mandel | alecu, which is the technique used when there is no NM | 15:10 |
| ralsina | I mean, it works, but if by some miracle the original flags didn; t have that bit set it would break stuff | 15:10 |
| mandel | alecu, but that code certainly needs a second iteration | 15:10 |
| alecu | mandel, right. It's not such a big problem, but that takes some time | 15:11 |
| jo-erlend | On 11.10, I'm still unable to use Desktopcouch in any way. I've been told here, that it's a known issue. Does anyone have a bug number that I can subscribe to? | 15:11 |
| mandel | alecu, nevertheless can you take a look at the watches branch first, there are few hours before my EOD | 15:11 |
| alecu | mandel, sure, sorry. | 15:11 |
| ralsina | gatox: I think the right way is flags & ~QtCore.Qt.WindowContextHelpButtonHint | 15:11 |
| gatox | ralsina, yep..... i was about to do that! | 15:11 |
| ralsina | gatox: cool | 15:11 |
| alecu | mandel, it's just that I saw the bit in ipc where the NM thread was started, and I guessed that was the issue with the dangling threads and the qt reactor not stopping. | 15:12 |
| alecu | mandel, so, I'm running the tests with IPCInterface.test=True, and my guess is that the reactor will stop just fine. | 15:12 |
| mandel | alecu, yeah? we initially had an issue with the looping call of the fs too, we need to hook that to twisted like the fs and listen to the stop event to kill the thread | 15:13 |
| mandel | alecu, no, it won't I dont think I was that smart when I wrote that | 15:13 |
| alecu | mandel, right. We should find a way to stop the thread | 15:13 |
| mandel | alecu, lets file a bug and do this after the release in our 'sprint' in BA | 15:15 |
| alecu | mandel, +! | 15:15 |
| alecu | mandel, +1 | 15:15 |
| mandel | :D | 15:15 |
| alecu | mandel, I think I'll try to fix this sooner. If I don't start the NM thread, I can stop syncdaemon with Ctrl-C | 15:16 |
| gatox | ralsina, done | 15:16 |
| ralsina | gatox: that works without the int() calls? | 15:17 |
| gatox | yep | 15:17 |
| alecu | mandel, I think that the NM thread not stopping takes a toll on the reliability of SD | 15:17 |
| ralsina | then you can remove it in line 23 :-) | 15:17 |
| mandel | alecu, true? it is bad when trying to stop it certainly? I'm nearly done with your review by the way | 15:18 |
| mandel | alecu, but please, review my branch asap, I wont be here next week :( | 15:18 |
| mandel | I'm on holidays | 15:18 |
| alecu | mandel, !!!! | 15:18 |
| alecu | mandel, I did not know | 15:18 |
| alecu | mandel, I'll review it asap, and I'll take a look at this thread next week | 15:19 |
| mandel | alecu, cool thx a lot!!! | 15:19 |
| ralsina | gatox: "then you can remove it in line 23 :-)" just in case you missed it because I didn't say your nick | 15:21 |
| gatox | ralsina, i read that..... i'm checking if it is ok | 15:21 |
| gatox | ralsina, if i do that...... i can't compare it with 0 | 15:21 |
| gatox | ralsina, ....... maybe with QtCore.Qt.QWidget | 15:21 |
| ralsina | Nah, it's ok, use the int() then | 15:22 |
| gatox | ralsina, ok | 15:22 |
| ralsina | gatox: +1 | 15:22 |
| gatox | ralsina, thanks | 15:23 |
| gatox | now is time to go to the bank | 15:23 |
| mandel | alecu, +1 on your branch, I have approved the MP too | 15:27 |
| ralsina | hmmm we are having a serious memory consumption problem in syncdaemon | 15:29 |
| mandel | ein? | 15:33 |
| ralsina | mandel: up to 300MB and rasing about 500K/sec | 15:33 |
| ralsina | mandel: probably not your branch's fault | 15:34 |
| alecu | mandel, revisate ésta: https://code.launchpad.net/~alecu/ubuntuone-client/make-it-stop | 15:34 |
| alecu | mandel, that is, try the branch by running the tests and seeing that they end up properly, and by running syncdaemon, and being able to stop it with Ctrl-C | 15:35 |
| alecu | mandel, if it works for you too we can make a proper branch out of it. | 15:35 |
| alecu | mandel, and don't worry: I'm reviewing your branch, and made that while the tests were running :-) | 15:36 |
| mandel | alecu, hehe I'm not worried, I know you are a man of your word... | 15:39 |
| mandel | ralsina, is this when testin my branch? | 15:39 |
| mandel | ralsina, is that a memleak? | 15:39 |
| ralsina | mandel: yes, but I have seen it on trunk too | 15:39 |
| ralsina | mandel: Iwould guess it is | 15:39 |
| alecu | mandel, and what word would that be? | 15:39 |
| mandel | alecu, lego? | 15:40 |
| mandel | :P | 15:40 |
| alecu | juas | 15:40 |
| mandel | hm? ubuntuone.logger is broken on windows? puto! | 15:48 |
| mandel | ralsina, this is ashot in the dark, but could this be the reason to the memory problems: http://paste.ubuntu.com/680660/ | 15:51 |
| alecu | mandel, I've added a few comments on the style while I finish the review: https://code.launchpad.net/~mandel/ubuntuone-client/eq-add-ancestors-watches/+merge/73775 | 15:52 |
| mandel | alecu, ok, on them | 15:52 |
| ralsina | mandel: I was not getting that | 15:53 |
| ralsina | mandel: I will take a better look once your branch and alecu's are landed | 15:54 |
| mandel | ralsina, I'm getting it on stderr | 15:54 |
| ralsina | but I just killed it and it was using 1.3GB of ram :-( | 15:54 |
| mandel | ralsina, neverthless the handler is broken | 15:54 |
| ralsina | mandel: you think it was keeping the backlog in memory? | 15:54 |
| ralsina | mandel: that doesn't explain 1.3GB of usage :-) | 15:55 |
| === beuno is now known as beuno-lunch | ||
| mandel | ralsina, it could? although 1.3 is waaaaay too much | 15:57 |
| ralsina | ok, lunch for me | 16:09 |
| === ralsina is now known as ralsina_lunch | ||
| mandel | alecu, pushed the style changes | 16:10 |
| mandel | alecu, I need to go for some time since otherwise the stores will be closed, I'll be back on time to make any change required to the review | 16:12 |
| alecu | mandel, I'm approving... but LP is so slow today. | 16:13 |
| mandel | alecu, hurray! | 16:14 |
| mandel | I mean, for the +1 :P | 16:14 |
| karni | rye: was this https://bugs.launchpad.net/ubuntuone-android-files/+bug/806002 related to auto-upload or just manual upload of any file? | 16:37 |
| rye | manual upload, don't think that anybody will want to blindly upload */* from the storage | 16:38 |
| karni | rye: I mean, U1F supports uploading a file of any type. Just touch "plus/add" > File | 16:39 |
| karni | rye: Is that what you meant? | 16:39 |
| karni | rye: We didn't have that before, but I've added that some time ago, and probably forgot to update the bug status. | 16:39 |
| rye | karni, let me check | 16:39 |
| rye | karni, tested on Adobe reader - opened pdf, clicked share - no U1 | 16:42 |
| karni | rye: ack | 16:43 |
| nessita | mandel: I'm doing your review now (ancestors) | 16:43 |
| rye | karni, it works the other way, like you described, but it does not look like it is able to accept arbitrary mimetypes | 16:43 |
| karni | rye: I just uploaded a ".dropsync" file (of unknown to me MIME type) from AndExplorer using Send option (they call it send, it's just share) | 16:44 |
| karni | rye: I'll verify how we register the share option. | 16:44 |
| karni | rye: You're right, thanks! | 16:46 |
| === beuno-lunch is now known as beuno | ||
| gatox | i'm bakc | 16:55 |
| gatox | back | 16:55 |
| karni | rye: any chance you have another app behaving like that? I've registered U1F for */* under both "android.intent.action.SEND" and "android.intent.action.SENDTO" with no luck | 16:55 |
| * nessita -> quick lunch | 16:59 | |
| karni | rye: any chance you have another app behaving like that? I've registered U1F for */* under both "android.intent.action.SEND" and "android.intent.action.SENDTO" with no luck | 17:00 |
| rye | karni, ummm, define:no luck? | 17:00 |
| karni | rye: sorry, stupid me. no luck = Adobe Reader does not display U1F in the 'Share' dialog | 17:02 |
| karni | rye: I can share custom files from other apps | 17:02 |
| karni | rye: I'll sit on in, ignore me. Thanks for reporting that. | 17:02 |
| rye | karni, hm, maybe it is doing something weird there | 17:07 |
| rye | karni, is OI FILE Manager sending it right? or any other file manager out there for Android | 17:09 |
| rye | karni, ah, re-read your message about AndExplorer | 17:09 |
| karni | rye: I'll look into it! :) | 17:13 |
| nessita | mandel: ping | 17:15 |
| === ralsina_lunch is now known as ralsina | ||
| nessita | mandel, alecu: I added a couple of needs fixing to ancestors branch. | 17:20 |
| nessita | ralsina: talking to my office mate, I just remember that I have this Monday off (I swapped it from August 22) | 17:21 |
| mandel | nessita, ok, I'll take a look | 17:21 |
| ralsina | nessita: it's ok | 17:21 |
| mandel | nessita, but will have to be later 2day | 17:21 |
| ralsina | nessita: unless you want to re-swap it? | 17:21 |
| nessita | mandel: please try to fix those today because MOnday I\ m not coming | 17:21 |
| nessita | mandel: drop me an email when you push the changes, so I can re-review | 17:21 |
| nessita | and do not block the brancj | 17:21 |
| mandel | nessita, will do, but is EOD here.. that why I'll do it after I do some shoping :) | 17:22 |
| nessita | ack | 17:22 |
| mandel | nessita, changes pushed? | 17:35 |
| nessita | you tell me? | 17:35 |
| nessita | :-P | 17:35 |
| mandel | nessita, yep, done but I did not have the time to run the tests? so I might have to be back, need to go or I wont have dinner, laters! | 17:35 |
| nessita | ok | 17:36 |
| ralsina | with mandel and alecu's branches, syncdaemon seems to work very smoothly :-) Except for runaway memory usage :-( | 17:42 |
| alecu | ralsina, but "runaway memory usage" is not new, right? | 17:43 |
| ralsina | alecu: no, I think it's old, just we never cared for it until now. I hoped it was related to the IPC bug and go away with it | 17:43 |
| nessita | ralsina: can you see a pattern for that? like, when syncdaemon processes a UDF memory raises | 17:44 |
| nessita | or when is IDLE | 17:44 |
| nessita | or when controlpanel contacts it | 17:44 |
| ralsina | nessita: not u1cp because I did not start it | 17:44 |
| nessita | for example, if there is no controlpanel interaction, does memory increases the same? | 17:44 |
| nessita | ah | 17:44 |
| nessita | ralsina: is there any other pattern you can distinguish? | 17:45 |
| ralsina | I will try in a clean account without UDFs to see what happens | 17:45 |
| ralsina | no UDFs ==> stable memory usage | 17:50 |
| nessita | ralsina: which may indicate that watchs are consuming mem | 17:53 |
| nessita | alecu: is that logical/possible? ^ | 17:53 |
| ralsina | started trunk controlpanel, got this: http://pastebin.ubuntu.com/680758/ | 17:54 |
| ralsina | and this on syncdaemon: http://pastebin.ubuntu.com/680759/ | 17:54 |
| alecu | nessita, not likely: watches are set on "Ubuntu One" too. | 17:56 |
| nessita | ralsina: look like bad credentials | 17:57 |
| nessita | ubuntuone.controlpanel.web_client.WebClientError: ('400', 'Invalid consumer.') | 17:57 |
| nessita | indeed | 17:57 |
| nessita | 2011-09-02 14:53:56,369 - ubuntuone.SyncDaemon.Main - NOTE - ---- MARK (state: <State: 'AUTH_FAILED' (queues WORKING connection 'With User With Network')>; queue: 239; hash: 0) ---- | 17:57 |
| ralsina | but but but | 17:57 |
| ralsina | I did login | 17:57 |
| nessita | ralsina: without further debugging (which may be needed), smells like old controlpanel pycs | 17:57 |
| ralsina | nessita: ok, looking fot that | 17:58 |
| nessita | or pycs mess up | 17:58 |
| alecu | ralsina, when running control panel did you set the PYTHONPATH to use the new u1-client? | 17:58 |
| ralsina | alecu: argh | 17:58 |
| * ralsina is stupid | 17:58 | |
| ralsina | the auth error could explain the stable memory | 18:00 |
| * ralsina gets new credentials and restarts | 18:00 | |
| ralsina | ffffffffffffffffffuuuuuuuuuuuuuuuuuuuuu the wizard's credentials are not valid now :-( | 18:03 |
| * ralsina is going to take 10 minutes for a cofee, bang his head on the table and be right back with a sunnier attitude | 18:04 | |
| nessita | ralsina, alecu: can I haz a review please? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/share-backend/+merge/73862 | 18:06 |
| ralsina | nessita: sure | 18:06 |
| nessita | ralsina, alecu: that branch make widgets share the same backend instance | 18:06 |
| alecu | looking | 18:06 |
| ralsina | nessita: cool | 18:06 |
| gatox | nessita, ping | 18:10 |
| nessita | gatox: pong | 18:10 |
| gatox | nessita, question: why are you using a qstackedwidget in control panel? | 18:10 |
| nessita | gatox: I owe you reviews, I know | 18:10 |
| nessita | gatox: yes | 18:10 |
| gatox | nessita, jejeje no, this time wasn't about that | 18:11 |
| nessita | :-) | 18:11 |
| gatox | nessita, did you read my question? | 18:12 |
| ralsina | nessita: would it be tricky to get a small branch that starts the stackedwidget with the regular u1cp showing instead of the login page? | 18:12 |
| nessita | gatox: sorry, I read "are you using", not "why are you using" | 18:13 |
| nessita | gatox: I'm using it because I need it :-) | 18:13 |
| nessita | gatox: we have 2 screen, that we need to swicth between them | 18:13 |
| gatox | nessita, ahhhh, ok | 18:13 |
| nessita | depending on if the user has credentials or not | 18:13 |
| nessita | gatox: you can try it your self! :-) wanna instructions? | 18:13 |
| gatox | nessita, no... no problem, just wanted to know why that widget was there...... | 18:14 |
| nessita | ah :-) | 18:14 |
| * gatox is fighting with widgets sizes | 18:14 | |
| ralsina | gatox: we discussed that! (you brought up stackedlayout) | 18:14 |
| nessita | ralsina: sorry, not sure what you asked :-) | 18:14 |
| gatox | ralsina, yes, i remember that | 18:14 |
| nessita | ralsina: can you please rephrase? | 18:14 |
| ralsina | nessita: the stackedwidget shows the login page by default. So if sso-client is not running, and credentials take a second to arrive, it flashes | 18:15 |
| nessita | ralsina: yes | 18:15 |
| ralsina | it would seem (to me) more logical that the tabs are shown, and then, if there are no credentials, it switches to the login page | 18:15 |
| nessita | ralsina: hum, that will not work, let me explain why | 18:16 |
| ralsina | nessita: also, +1 on the last branch, with a little golden star sticker. | 18:16 |
| nessita | ralsina: if we show the "management" panel (the non-signin one) first, all the inner panels will try to load the info, and we'll get tons of traces due to credentials not being there | 18:17 |
| ralsina | nessita: yikes | 18:17 |
| nessita | ralsina: does that make sense? | 18:17 |
| ralsina | nessita: yes it does. | 18:17 |
| ralsina | how about a blank page then? | 18:17 |
| nessita | ralsina: we should try not to show anything until the credentials answer is there? | 18:17 |
| nessita | right | 18:17 |
| ralsina | show the window, the overlay, but nothing behind it | 18:18 |
| nessita | ralsina: yes, let me try a simple approach | 18:18 |
| ralsina | cool | 18:18 |
| nessita | if the idea I have works, is easy | 18:18 |
| nessita | ralsina: can you please file a bug in the mean time? | 18:19 |
| ralsina | nessita: sure | 18:19 |
| nessita | ralsina: branch ready! :-P | 18:20 |
| ralsina | nessita: bug is #839795 | 18:21 |
| nessita | ralsina: can you please try IRL? I have almost 0 delay, is hard to tell this is what we want: lp:~nataliabidart/ubuntuone-control-panel/nothing | 18:22 |
| nessita | ralsina: MP is https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/nothing/+merge/73865 | 18:23 |
| nessita | alecu: ping | 18:24 |
| ralsina | seems like the credentials problem is not global, since with another account I can login. | 18:25 |
| ralsina | nessita: checking! | 18:25 |
| alecu | nessita, pong | 18:26 |
| nessita | alecu: I still see issues in this branch https://code.launchpad.net/~mandel/ubuntuone-client/eq-add-ancestors-watches/+merge/73775 but since I'm leaving to uni, and I'm not coming next Monday, I was wondering if I should approve and you add the needs fixing until the test is fixed | 18:26 |
| alecu | nessita, I can do it, yes. | 18:26 |
| nessita | great | 18:26 |
| alecu | nessita, and since mandel goes on vacations next week I can take over that branch | 18:26 |
| nessita | alecu: mandel goes on vacation next week? | 18:27 |
| nessita | I did not know that | 18:27 |
| ralsina | nessita: I forgot to announce it | 18:27 |
| nessita | ralsina: is... kind of... unexpected it | 18:27 |
| ralsina | the same thing hapened with li-sette's we should define a vacation announcement protocol | 18:27 |
| nessita | alecu: so, shall I approve the branch or we're rejecting it (since you may be proposing a new one)? | 18:30 |
| jo-erlend | is the development of Ubuntu One and Desktop-Couch completely closed, or can I follow the work on bugs, etc somewhere? | 18:44 |
| alecu | hello! | 18:47 |
| nessita | alecu: hi! I was telling you: | 18:47 |
| alecu | my laptop has just kernelpaniced | 18:47 |
| nessita | alecu: so, shall I approve the branch or we're rejecting it (since you may be proposing a new one)? | 18:47 |
| alecu | nessita, let's reject it, and I'll fix it | 18:48 |
| nessita | alecu: great, thanks | 18:49 |
| nessita | I'm leaving soon crowd | 18:49 |
| nessita | ralsina, gatox: I'll will have to finish the reviews next week... I guess ralsina can add the branches to the installer without merging those? | 18:50 |
| nessita | ralsina: did the "nothing" branch work? | 18:50 |
| ralsina | nessita: haven't tried it yet | 18:50 |
| gatox | nessita, ralsina also..... i'm finishing with the fixes in control panel (is something really small), but it took me a while to find it | 18:50 |
| ralsina | nessita: works. It's all pink though :-D | 18:51 |
| alecu | nessita, have a nice weekend. | 18:51 |
| nessita | ralsina: no pink, orange I guess | 18:51 |
| ralsina | nessita: could you mail me the list of your branches that should go into the release? | 18:51 |
| ralsina | orange+white = pinkish :-) | 18:52 |
| nessita | ralsina: just the two under review: | 18:52 |
| nessita | https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/nothing/+merge/73865 | 18:52 |
| nessita | https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/share-backend/+merge/73862 | 18:52 |
| ralsina | I think I will just merge "nothing" since it's almost trivial and it works | 18:52 |
| nessita | ralsina: ack | 18:55 |
| nessita | ralsina, gatox, alecu: have a great weekend, let's rest | 18:55 |
| gatox | nessita, byeeee | 18:55 |
| gatox | you too | 18:55 |
| ralsina | off to pick up the kid | 19:10 |
| ralsina | and back | 19:38 |
| karni | pizza, yum | 19:50 |
| ralsina | I am currently having the best ubuntu one run on windows EVER | 19:58 |
| ralsina | everything is working, and memory usage is not running away, and the UI is responsive, and I can see only one ugly bit of UI that needs fixing | 19:58 |
| karni | ralsina: \o/ | 19:59 |
| ralsina | what the heck, let's package it | 20:00 |
| karni | ;D | 20:00 |
| alecu | so, EOW for me | 21:02 |
| alecu | bye all! | 21:02 |
| === m_conley is now known as m_conley_away | ||
| === ralsina is now known as ralsina_away | ||
| gatox | ralsina_away, if you are still around and need something let me know! i'm back | 22:08 |
| ralsina_away | gatox: nah, I packaged it :-) | 23:09 |
| ralsina_away | gatox: You could take a look at bug #839844 but it can wait until monday | 23:10 |
| gatox | ralsina_away, ok | 23:16 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!