=== palhmbs_ is now known as palhmbs
JamesTaitGood morning, everyone. :)10:03
nessitahola gente11:59
=== teknico is now known as teknico_away
mandelnessita: ping12:31
nessitamandel: pong12:31
mandelnessita: heelo, I was wondering if you could give me a hand with sso client, I'm getting gconf issues when running the tests12:32
mandelit seems that the dbus seession is not working (I'm testing on M)12:32
nessitamandel: what kind of issues?12:32
mandelnessita: let me patebin it12:32
nessitamandel: sure12:32
* mandel reruns tests12:33
nessitacan I have a second review for https://code.launchpad.net/~evfool/ubuntuone-control-panel/tooltips/+merge/51838 ? (is trivial)12:34
=== teknico_away is now known as teknico
ralsinagood morning!12:38
ralsinanessita: I got it!12:39
mandelnessita: this is mainly what i get paste.ubuntu.com/57442212:40
nessitamandel: ah, that's ok, we've always had that12:41
nessitaI thought you were having crashes12:41
mandelnessita: oh, really? heh I worried for nothing then12:42
=== teknico is now known as teknico_away
mandelnessita: how are you feeling about adding a platform module in ubuntu_sso?12:59
nessitamandel: if it'll work as is working in syncdaemon, pretty bad :-) is very annoying the way it is. What modules do you need to replace?12:59
mandelnessita: so far, keyring and networkstatus at least13:00
mandelnessita: why it it bad in sd?13:00
nessitamandel: from my point of view, is very annoying an confusing to have to code some things inside platform and some other inside the regular syncdaemon dir. It breaks any grouping we can make about related components13:02
nessitamandel: to explain what I've said, imagine you want to use some automatic drawing tool to draw the module depenencies13:03
nessitamandel: by how it's right now, you will get a diagram that does not reflect the truth (at design level I mean). You'll end up with separated modules that are separated just becasue the platform stuff13:04
nessitamandel: does that make sense? we can try to think alternatives13:06
nessitamandel: for instance, if we already have a module graph like:13:06
nessitamandel: http://pastebin.ubuntu.com/574435/13:09
nessitamandel: I think we need to maintain that same graph structure when migrating code to be multiplatform13:10
nessitamandel: let me know when you're here13:12
mandelnessita: I'm here, I'm just thinking :)13:13
nessitamandel: so, what about something like:13:16
nessitamandel: if you need the keyring moduke to have different implementations, I would:13:17
nessita* mkdir keyring13:17
=== alecu_ is now known as alecu
nessita* bzr move keyring.py keyring/linux.py13:17
mandelnessita: I have no issue for the garph since for me both are correct, I do have issues in the way that patching has been implemented13:18
nessita* create a keyring/windwos.py13:18
mandelnessita: I have done that atm, that's why I know that it complicates the testing alot13:18
nessita* create a keyring/__init__.py that import stuff from linux or windows properly13:18
alecunessita, that does not scale13:19
alecunessita, you would need the code to import from linux or windows for each platform dependent module.13:19
nessitamandel: if it complicates testing let's resolve that, but let's no break our "module design graph" just because testing is harder13:19
nessitaalecu: I'm not follwoing you13:19
mandelnessita: when testing is hard, something smells13:20
nessitamandel: yes. Most likely testing is being done the wrong way13:20
alecuand also, that needs each module to have a different python package, and only one platform dependent module.13:20
nessitaalecu: really, I don't see how what you've say apply to what I've said :-)13:21
alecunessita, I mean that you would need the __init__ that selects from linux or windows for each platform python package.13:21
nessitaalecu: you will need the __init__ for each *python module* that you need to have 2 (or more) implementations13:22
nessitano python package13:22
alecunessita, in keyring/__init__.py, "keyring" is a package.13:23
nessitamandel: can you please give me an example of a testing problem with that structure?13:23
nessitaalecu: yes, but I was saying that you only need to transform each (currently being) module that needs more than on impl13:24
nessitaalecu: and usually, and specially using python, those will be very little13:24
mandelnessita: sure, one sec13:24
nessitaalecu: I think that providing multiplatform solutions for ussoc (in particular) should not change the current module dependency diagram (module as in UML lexic)13:26
alecunessita, also: if we do things one way in syncdaemon, I think it is more harder to learn/maintain if we do it totally differently in other parts.13:26
nessitaalecu: I agree for projects inside U1 client, and I thought about that. This is ussoc which is a separated, independent project from u1client13:27
nessitaalecu: I honestly think that having the new platform package breaks good design and under stability of the implementation13:27
mandelnessita: line 164 in test_keyring, when you patch the secrets object13:27
nessitaI meant understability, if such word exists :-)13:28
nessitamandel: looking in one sec13:28
mandelalecu: do you know wich error is raise when the secrets dbus api is not present?13:28
nessitaalecu: what happens if dbus starts working on windows?13:28
alecuWell for practical purposes, it's maintained by the same set of people! :-)13:28
nessitaalecu: you will need to break apis because you no longer will have from ubuntuone.platform.bla13:29
alecunessita, afaik it already does for some kde apps.13:29
alecu(kde apps on windows)13:29
nessitaalecu: or you don't break APIs and leave the same implementation in the platform module which is missleading13:29
alecunessita, ubuntuone.platform.bla are not APIs, they are internal modules13:30
alecuonly our code is supposed to be using that.13:30
mandelnessita: it would be from ubuntuone.platform import13:30
nessitaalecu: no, is API13:30
mandelusing ubuntuone.platform.linux should be not done13:30
alecunessita, in what cases it's API?13:30
nessitamandel: I never said linux ;-)13:30
nessitaalecu: using the dbus interface constants, for instance13:31
alecunessita, what 3rd party software uses those APIs?13:31
nessitaalecu: our control panel broke13:31
nessitaalecu: rye's indicator surely uses the dbus module13:31
mandelnessita: control panel was using a function that started with _13:31
mandeland the dbus api was kept and not touched13:32
alecunessita, if those are DBus constants, they don't belong in a python file.13:32
nessitamandel: that was a test. I'm talking about real functions13:32
nessitaalecu: dbus constants and more13:32
mandelnessita: example?13:32
alecunessita, the only way to define DBus constants is in a documentation file, because other programming languages using DBus won't be able to access constants in python files.13:33
alecunessita, what more?13:33
nessitamandel: don't have one handy. But anyways, I fell pretty strong about not changing the module dep graph, so let's sort your testing issue out13:33
nessitaalecu: we were using the dbus module itself. We were using the tools module, to use SyncdaemonTool which is meant to be use as a helper13:34
nessitaalecu, mandel: I already said this with u1client, let's keep the approach that os.path use13:34
nessitathat's clean and usable13:34
nessitait makes no sense to do: from os.platform.path import mkdir13:35
nessitathe fact that we need 2 implementations for path should be hidden from APIs13:35
nessitamandel: browsing your code now13:36
alecunessita, if the problem is the tools module, then you should have had an empty tools module importing that linux module.13:36
mandelnessita: which code?13:36
alecunessita, and also you don't do "from os.platform.path import mkdir"13:36
alecuI mean13:37
nessitaalecu: exactly, you don't13:37
nessitaalecu: in syncdaemon you have to do that13:37
alecuyou don't do "from ubuntuone.platform.linux import module"13:37
alecunessita, you do "from ubuntuone.platform import module"13:37
nessitaalecu: no, you do: "from ubuntuone.platform import module"13:37
mandelnessita: the example is in trunk, when you patch secrets13:37
nessitafrom my POV, no "platform" word should be needed13:37
alecuso, the platform dependent stuff is hidden.13:38
nessitaalecu: is not. You are exposing the fact that, for now, dbus is platform dependatn13:38
nessitaalecu: what happens when dbus is not platform dependant?13:38
mandelnessita: never is going to happen13:38
nessitamandel: is an example13:38
mandeldbus will not be in windows, as the keyring13:38
nessitamandel: you can't ensure that :-)13:39
nessitamandel: what's the problem with patching the SecretService?13:39
alecuanyway: I don't like building for something that's not there yet.13:39
alecunessita, ^13:39
nessitaalecu: if we're starting fresh, let's do it properly13:39
alecunessita, let's build what's needed right now. And let's do it in a predictable way.13:39
nessitaalecu: I agree.13:40
nessitawhat's needed right now is 2 implementations for the keyring and for NMstate13:40
nessitamaking those modules be packages with 2 implementations is easy and clean13:40
nessitaand no API at all will be broken13:40
mandelnessita: when you patch, you are patching the imported module by the keyring, but that could not be right once you mov the code to linux13:41
nessitamandel: right, but those tests are the linux tests :-)13:41
nessitamandel: since you just said that keyring will have 2 implementations, there will be 2 sets of tests13:41
alecumandel, read in twitter you've been fixing the python keyring module, right?13:41
alecumandel, did you know that that module is horribly broken on linux?13:42
mandelalecu: yes, delete is present, we are also looking at using secrets on linux to fix some issues13:42
mandelAlanBell: funny enough the mantainer is from canonical hehe13:42
mandelnessita: I'll look at fixing those tests..13:43
alecumandel, also, that module is not complete.... some functions are only async, some are only sync...13:43
alecumandel, how do you plan on doing the keyring access on windows? sync or async?13:43
* alecu supposes his lastname changed to Bell recently.13:44
alecuAlanBell, I think mandel meant me.13:44
mandelalecu: atm is sync, but I'm planning of do both, since the encription could be slow on windows13:44
* AlanBell goes back to sleep13:45
mandelAlanBell: hehe stupid tab completion, sorry13:45
alecumandel, assume all keyring operations are slow, because the user may always need to enter a password to unblock it.13:45
nessitaalecu, mandel: I'm open to explore other solutions to implement the multiplatform stuff, but creating a new package platform is, from my POV, a not clean nor clear way of doing it. So maybe my solution is not the proper one, but I would say we should mimic what python itself does with, for example, os.path13:45
nessitaok, water is about to biol, brb13:46
alecumandel, even deletes or simple keyring queries.13:46
* alecu returns to eclipse13:46
mandelalecu: yes, why not, lets be consistent :)13:47
ralsinastandup in 10'13:50
* nessita is back with mate13:52
nessitamandel: so, I'm lost. How are we approaching multiplatform stuff in ussoc then?13:52
mandelnessita: as you say, I have no problem, I'm atm fixing the tests and adding the -i to the runtest script13:53
nessitamandel: ok, I'll be happy to help (at least with reviews, my time is a bit tight this week)13:54
mandelnessita: cool, I'll ping you if needed13:55
nessitamandel: sure13:56
nessitadobey: ping13:58
nessitadobey: hey, question. What projects should I assign this bug to? bug #72687114:00
ubot4Launchpad bug 726871 in ubuntuone-control-panel (Ubuntu) "Ubuntu One Music Store syncing not on by default (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/72687114:00
nessitaralsina, dobey, say me please! :-)14:01
nessitathisfred: would you please?14:01
thisfredDONE: Worked on Bug #702183, Bug #702176, Bug #70217214:01
thisfredTODO: Finish on Bug #702183, Bug #702176, Bug #70217214:01
thisfredBLOCKED: no14:01
thisfredLOVED: http://tinysong.com/3qpb The Antlers - Bear14:01
thisfredNEXT: nessita14:01
nessitaDONE: bug #706034, bug #720650. Chat with cparrino re: control panel UI. A lot of bug status update in u1cp, finally got every single bug reviewed and tagged.14:01
nessitaTODO: meeting with ralsina and cparrino for u1cp UI and string review. Kill more u1cp UI bugs! talk about multiplatform issues in ussoc.14:01
nessitaBLOCKED: nopes14:01
nessitaNEXT: alecu14:01
alecuDONE: getting OAuth tokens from SSO (with hardcoded username/password), and storing in Droid pref storage (bug #725290).14:02
alecuTODO: UI to ask for user/pass, same bug14:02
alecuBLOK: not14:02
alecuHATE: went thru OAuth hell to finally find out that http requests cannot be signed twice by signpost. Found a workaround, got back.14:02
alecuLOVE: los tostados del Beltrán14:02
alecuNOTE: working across the river today, so will be out during my afternoon for bank issues, and will be online later after getting back home.14:02
alecuNEXT: mandel14:02
ubot4Launchpad bug 702176 in ubuntuone-client (Ubuntu) (and 1 other project) "Syncdaemon needs to add an entry to the messaging menu when a folder shared to the user exceeds the owning user's quota (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/70217614:02
ubot4Launchpad bug 702172 in ubuntuone-client (Ubuntu) (and 1 other project) "Syncdaemon needs to send a notification when a folder shared to the user exceeds the owning user's quota (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/70217214:02
ubot4Launchpad bug 702183 in ubuntuone-client (Ubuntu) (and 1 other project) "Syncdaemon needs to open the control panel in the background and change the launcher icon to urgent when the user exceeds their quota (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/70218314:02
ubot4Launchpad bug 706034 in ubuntuone-control-panel (Ubuntu) (and 2 other projects) "Get a better icon for the folder owner in the Cloud Storage panel (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/70603414:02
ubot4Launchpad bug 720650 in ubuntuone-control-panel (Ubuntu) (and 1 other project) "Folder for music I've purchased from the U1 store is not shown cleanly (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/72065014:02
ubot4Launchpad bug 725290 in droidcouch "Get valid OAuth tokens for the logged in user (affects: 1) (heat: 7)" [High,In progress] https://launchpad.net/bugs/72529014:02
mandelDONE: bug triagging, bug #684967,  bug #684978, bug #727680. Ptched python keyring to allow the removal of passwords.14:02
mandelTODO: sort out ubuntu-sso-client tests. Move to next sso erro on windows14:02
mandelBLOCKED: no14:02
mandelralsina, go14:02
ubot4Launchpad bug 684967 in ubuntu-sso-client "SSO does not provide a way to use a keyring according to the platform (affects: 1) (heat: 2)" [Medium,In progress] https://launchpad.net/bugs/68496714:02
ubot4Launchpad bug 684978 in ubuntu-sso-client "There is no Windows keyring implementation (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/68497814:02
ubot4Launchpad bug 727680 in ubuntu-sso-client "There is no network status implementation on Windows (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/72768014:02
ralsinaDONE: 5 calls (4 hours total), few reviews, hotel hunting, taxes, canonicaladmin, etc14:03
ralsinaTODO: 2 or 3 calls, reviews, bugfixing, scheduling14:03
ralsinaBLOCKED: no, tired14:03
dobeydon't ask me a question and then tell everyone to post their standup status14:03
dobeyλ DONE: flights booked, music store testing,14:03
dobeyλ TODO: bug #727556, bug #727558, mp3 install in banshee14:03
dobeyλ BLCK: None.14:03
ubot4Launchpad bug 727556 in libubuntuone "u1ms: links don't open unless store already loaded (affects: 1) (heat: 6)" [High,Confirmed] https://launchpad.net/bugs/72755614:03
ubot4Launchpad bug 727558 in ubuntuone-music-store (and 1 other project) "Need to notify user when Purchased Music folder is not subscribed (affects: 1) (heat: 6)" [Medium,Confirmed] https://launchpad.net/bugs/72755814:03
nessitathisfred: hey! you will be able to bug several bombillas while in buenos aires!14:03
thisfredor bug :)14:03
nessitaralsina: no comments on this side, meeting with cparrino in 514:04
ralsinathisfred: bugs you don; t buy, you catch :-)14:04
thisfredI want a pet chicharra14:04
ralsinathisfred: not chicharra season :-)14:05
thisfreda dead pet chicharra? :)14:05
ralsinaThat can be arranged :-)14:05
thisfredI wonder if they're bigger than the crazy bugs we have here in the park14:05
ralsinaabout the size of your thumb (ok, of my thumb)14:05
thisfredah same then. Maybe it's the same insect too14:06
ralsinaa chicharra is a cicada, I think they are all the same14:06
ralsinamore or less14:06
thisfredwell the european ones are tiny14:06
ralsinaok eom?14:06
thisfredand not nearly as loud14:06
ralsinaoh, the ones here are loud alright :-)14:07
nessitaralsina: do we have hotel yet? (pure curiosity)14:08
dobeyit's the ones in egypt that you really have to watch out for14:08
ralsinanessita: not decided, but surely there will be a hotel ;-)14:08
nessitadobey: so, any ideas about the bug I mentioned?14:09
nessitadobey: shall I assign it to ubuntuone-music?14:09
dobeynessita: i moved it and marked duplicate14:09
nessitadobey: thanks!14:09
nessitadobey: next question: would you know how to make a LinkBUtton 'look' like a regular button but keep functioning like a linkbutton?14:09
nessitain GTK, of course14:10
ralsinaHey, word from Marianna, it's probably going to be http://www.hoteldelcasco.com.ar14:10
dobeya link button?14:10
ralsinathisfred: has a great artisans fair across the street14:10
thisfredoooh nice14:10
ralsinathisfred: so you can buy bombillas ;-)14:10
thisfredI checked the link before, looks like a very nice hotel14:11
dobeynessita: why not use a regular button?14:11
ralsinathisfred: it is really nice.14:11
ralsinaAnd the area is really nice in general. One block from a cathedral, there'sa restaurant zone 2 blocks away, and really peaceful except on sundays14:11
mandelralsina: I hate you for providing a link to a webpage that starts playing a song when I open it14:11
mandelspecially  elevator music14:12
ralsinamandel: come on, Gershwin is not elevator music14:12
dobeymandel: http://html5zombo.com14:12
mandeldobey: I hate you tooo14:13
mandelthat extra o is how much extra hate i have14:13
dobeyralsina: when you visited that hotel, was there Sinatra playing constantly?14:13
nessitadobey: I could, yes, but I was hoping to actually use a linkbutton since that's the functionality I need14:13
dobeymandel: WELCOME! to zombocom14:14
rmcbridethe infinite is possible at zombocom14:14
dobeynessita: it's more code to use a link button and make it look like a normal button, than it is to use a normal button and make it open a link14:14
nessitadobey: right, thanks. I wanted to confirm exactly that.14:15
nessita(maybe I was missing something)14:15
dobeynessita: just do button.connect_swapped ("activate", gtk_show_uri, uri_string)14:16
ryeweird, got notification about my nick mentioned but can't find who/when :-/14:16
dobeynessita: and it should do what you want14:17
dobeyrye: 08:31 < nessita> alecu: rye's indicator surely uses the dbus module14:17
ryeaha, dobey, thanks!14:17
ryealecu, nessita, what about dbus module?14:17
dobeyyeah, /lastlog is win :)14:18
nessitarye: nothing important14:18
ryedobey, oooh, shiny!14:18
nessitathisfred: can you please join #ubuntu-desktop?14:19
thisfredsuret thing14:19
thisfrednessita: eh, why? :)14:23
nessitathisfred: oh, did seb not contact you? :-/14:23
=== teknico_away is now known as teknico
=== zyga is now known as zyga-food
nessitaralsina: would you please file me a bug in u1cp to improve the services tab installation workflow?14:59
nessitaralsina: https://bugs.launchpad.net/ubuntuone-control-panel/+filebug :-)15:00
ralsinanessita: on it!15:00
nessitaralsina: FYI, that will be done after UI freeze (probably during the sprint) since is not an UI bug (but a workflow bug)15:00
ralsinanessita: cool15:00
nessitaralsina: actually, there is a bug for it already bug #72686515:01
ubot4Launchpad bug 726865 in ubuntuone-control-panel (Ubuntu) "Installed desktopcouch to enables services: confusing error and no functionality (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/72686515:01
nessita(I just saw jono's bug)15:01
nessitaI'll affect u1cp project15:01
ralsinaso let's use that15:01
nessitaralsina: can you please add a comment stating the change of the workflow? :-D15:02
* nessita is pedigüeña today15:02
dobeyes, hoy15:06
nessitadobey: está, hoy ;-)15:09
ralsinaOk, I just broke my home's lock15:13
ralsinabe back after I find a locksmith15:13
ralsinaArgh, how do I deduplicate a bug?15:17
duanedesigncan you just leave the field blank and hit ok?15:17
ralsinaOh, found it, thanks15:18
ralsinahey, unity-2d has the progressbar too now :-)15:27
alecuralsina, cool.15:32
alecuralsina, is syncdaemon updating it as well?15:32
ralsinaalecu: yes15:37
ralsinaalecu: noticed it when syncing a folder15:37
nessitaI want that!15:37
nessitadobey: any news on having icons without the -gnome package installed?15:37
ralsinanessita: install unity-2d then :-)15:38
nessitaralsina: I don't have the U1 icon in the launcher, that's my problem :-)15:38
ralsinaadd it!15:39
dobeynessita: not yet15:39
nessitaralsina: is added! I have the entry but iconless, so I have a big question mark for it15:39
nessitaso no progress bar for me15:39
ralsinaOTOH I was running controlpanel, so maybe that's why I had the icon15:40
nessitaralsina: no no. To understand my point, uninstall ubunutone-client-gnome. You'll get the U1 entry in the launcher iconless15:40
ralsinanessita: oh, that15:41
nessitadobey: FYI, setting 'border-relief' to 'half' gives me a linkbutton that looks like a regular button15:41
ralsinaI am triaging bugs in -client-gnome so no ;-)15:41
dobeynessita: well setting the relief to 'normal' it would look more like a regular button that it does by default, but it still doesn't look right15:42
nessitadobey: having relief in normal looked exactly like relief in None15:42
nessitaonly half will show the button border at all times15:42
dobeynessita: either way it is better to use a normal button i think15:43
nessitadobey: why?15:43
nessita(real interest here)15:43
dobeybecause the underline and coloring of the link button look out of place15:43
dobeyi don't know why linkbutton was even put into gtk+ as it is. it makes no sense to me to have it there15:44
=== zyga-food is now known as zyga
dobeylunch time16:24
ralsinanessita: this is the scrollbar thing I mentioned in the call: http://ubuntuone.com/p/fmd/16:42
nessitaralsina: yeah, I saw it. Can you please file a bug for u1cp, attaching the screenshot? I'm about to have lunch and I don't want to drop that package :-)16:44
=== beuno is now known as beuno-lunch
thisfrednessita: alecu: so for the quota notifications/messages/popups: we need to know when a volume that is exceeding quota is a share *from* another user. What is the best way to determine whether it is, do you guys know?17:11
thisfredI have a dictionary like this: {'name': '', 'subscribed': '', 'generation': '', 'other_username': '', 'other_visible_name': '', 'access_level': u'View', 'node_id': '', 'volume_id': u'fake share id', 'free_bytes': '0', 'path': '', 'accepted': '', 'type': u'Share'} (but then with more values filled in presumably ;)17:14
thisfredOr does Share mean it's *always* from another user?17:15
thisfredI guess so huh17:15
thisfredanything I share would just be a folder in my root or a UDF17:16
thisfredas you were ;)17:16
* thisfred loves these socratic monologues we have17:16
alecuthisfred, yes, Shares are always from another user17:21
thisfredcoolness that eliminates a lot of logic :)17:21
alecuthisfred, ShareD are from you to another user, but they won't push that event.17:21
alecuI guess only the Root folder may push the event anyway.17:22
thisfredright, I was wondering about root, but that should be easy to distinguish right? By node_id or something?17:25
thisfredanyway, steaming on17:25
nessitathisfred: yes, sorry I was having linch17:31
thisfredYeah better if the technical lead doesn't actually lynch people ;)17:32
nessitaok, I need to reboot after updates17:36
thisfredalecu: last question: you mentioned that we probably want to throttle the frequency of quota messages/notifications/popunders to once a day. That17:46
thisfreds not happening elsewhere already or is it?17:47
thisfredI mean for other statuses17:47
alecuthisfred, I believe it's already happening for the out-of-quota dialogs that are shown from gsd-plugin.17:48
alecuthisfred, the throttling to 1/day.17:48
thisfredah ok, I'll look there. In which package do they live?17:48
alecuthisfred, beware: that's a gnome-settings-daemon plugin, made with C... it's in u1-client/gsd-plugin17:49
thisfredalecu: I'll probably be fine if I just read the code. If I'm not back in two days, send an exorcist ;)17:49
nessitaok, the reboot was painful :-(17:54
alecuthisfred, probably you'll want to set up a status.aggregator.Timer instance, and not show the message again if the timer is still active.17:54
alecuthisfred, and to see if the timer is still active, create a new method "is_active" in Timer that calls self.delay_call.active()17:58
thisfredalecu: cool, thx17:58
alecuwell, it looks like I need to catch a boat...17:59
alecusea you later, alligators!17:59
=== beuno-lunch is now known as beuno
=== zyga is now known as zyga-afk
=== karni_ is now known as karni
karnisorry, I had irc-lag, let me repeat the question18:41
karnibeuno: I see quite a few handshake timeouts in your log. Should increase the timeout from 30 seconds? One question - was this log submitted from the last link I gave you? I didn't bump the version number, and I don't see the Storage Management piece I was interested in :( I also used your log to detect 2 idle-timeout bugs, so I'm happy we caught that.18:41
beunokarni, I'm not sure what I'm running anymore  :)18:42
karnibeuno: would this be a problem if I gave you a new apk to run?18:42
karniI'd like to see those numbers to fix the 'total space' bug that you have18:42
karnias I recall18:42
beunokarni, not at all18:43
karniI'll bump the version number correctly this time.18:43
beunokarni, just email it to me, it's easier to install that way18:43
karnibeuno: sure, thanks!18:43
=== ivanka is now known as ivanka-train
karnibeuno: sent apk via e-mail19:00
beunothanks karni19:09
nessitacan I have reviews please? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/regular-buttons/+merge/5194119:11
ralsinanessita: lo tengo19:11
nessitaok, I'll restart on classic desktop now19:11
nessitaok, this should be better now19:18
nessitaralsina: button design was given. I don t like it as well19:18
nessitaralsina: was also approved by cparrino19:19
ralsinadid you mention the alignment?19:19
nessitaralsina: I can't align the label only to the left, but only making the whole button small and align that to the ledt19:19
ralsinahmmm... I meant that if it'sgoing to be large, it should align with the label and the other buttons. Or the labels with your name should not be indented19:20
nessitaralsina: the name is supposed to be inside the personal details19:21
nessitaralsina: can you please check the  account.png and account2.png in the email Daniel sent?19:21
nessitaralsina: subject is:19:21
nessitaU1, front screen19:21
ralsinaOk, so it did came from design.19:22
ralsinaOk, let's accept they know what they are doing in taste matters, so +119:22
=== zyga-afk is now known as zyga
nessitaralsina: I will change the name, I just noticed it should read 'Edit account details'19:22
ralsinaOk, I'll hold the approve19:23
ralsinaAlso, Daniel's image doesn't have the text underlined. That looks kinda weird in a "normal" button.19:23
nessitaralsina: but is a linkbutton! it will take you to a browser19:24
ralsinaYes, I know.19:24
nessitaralsina: a completely regular button is not supposed to take you to a browser19:25
ralsinaThat's why I don't ask you to change it :-)19:25
nessitaI want to avoid to have absolutely regular buttons for link stuff19:25
nessitaralsina: fixed and pushed19:31
ralsinanessita: +1oed19:31
nessitaralsina: one of the bugs we talked about today with you and cparrino: bug #72658019:37
ubot4Launchpad bug 726580 in ubuntuone-control-panel (Ubuntu) (and 1 other project) "Control Panel shows me the available storage of people who have shared with me (affects: 2) (heat: 10)" [Low,Triaged] https://launchpad.net/bugs/72658019:37
nessita(just FYI)19:37
ralsinanessita: got it, thx19:37
ralsinacan someone re-approve https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/regular-buttons/+merge/51941 ?19:49
ralsinait 's trivial19:49
beunoI am this >< close to creating an irc bot that will approve branches in my name that anyone can trigger19:50
ralsinabeuno: actually you had approved it already :-)19:50
beunoralsina, I know19:51
dobeyi am this ][ close to writing a bot that will block votes from beuno19:51
ralsinaBut it got a later changeset19:51
beunodobey, it's a challenge, then19:51
ralsinaYou know, the approving vote is kinda trivial using mechanize and a bot construction kit ;-)19:51
dobeyyou know, you all are overreacting at a trivial thing19:52
* ralsina starts creating doppelganger accounts19:52
ralsinadobey: ?19:52
beunowriting a bot will at least feel more productive than having to load pages and click several times19:52
ralsinaI am sure it will save AT LEAST 10 or 15 minutes over the next year.19:52
dobeyralsina: people need to stop being so impatient with launchpad. they ought to be used to it being slow as molasses by now19:53
ralsinaTeam leaders should have a special "I SAY APPROVED" button.19:53
ralsinadobey: it's not all that slow here19:53
ralsinadobey: I am joking, you know19:53
karniralsina: :D19:53
nessitaralsina: so, I think bug #727949 is an Invalid. If you look the window corner (where the scroll bar is chopped off), you will see a triangle that is meant to be used to resize the window20:03
ubot4Launchpad bug 727949 in ubuntuone-control-panel (Ubuntu) (and 1 other project) "The scrollbar is too short (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/72794920:03
nessitaralsina: is clearer in firefox, for example20:03
nessitaralsina: but, my point is that GTK is drawing the scrollbar shorter20:03
nessita"he" may think is fancier :-/20:04
ralsinaI can't see anything visible in that corner in my screenshot20:05
dobeyralsina: that's because there's nothing behind it there20:05
dobeyralsina: but the scrollbar thing is an issue in gtk+ and the theme, not the control panel itself20:06
ralsinaUgly as hell is what it is but yes, Invalid :-(20:06
dobeyoh, well20:07
dobeyif you think that's bad, you haven't seen the half of it yet :(20:07
nessitabeuno: I've got this in teh web ui after trying to publish a file http://ubuntuone.com/p/fow/20:09
nessitacan I have reviews for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/misc-tweaks/+merge/51948 ?20:09
nessitathe merge proposal has screenshots in it :-D20:09
* nessita bribes20:10
ralsinanessita: I'm on it20:10
nessitabeuno: FYI the page was fully loaded20:10
ralsinanessita: U CAN't HAZ CSS :-)20:10
nessitabut but but20:10
nessitaI've been a good girl!20:10
nessitanah, who will buy that20:11
nessitaI've been a almost good girl!20:11
nessitaralsina: do you Invalid or shall I?20:11
ralsinaif you have it handy, please do it20:11
* nessita is all about closing bugs20:11
* beuno looks20:12
beunonessita, nice20:12
beunonessita, it looks like you opened that in a new tab, is that possible?20:13
nessitabeuno: nopes, I was opening another link in the mean time (contacts staging)20:13
nessitamultitasking, vio?20:13
beunonessita, so, it looks like javascript hadn't loaded or something. If it happens again, let me know, it may of been a browser glitch20:14
nessitabeuno: sure20:15
dobeygrr, so many bugs in libu1 :(20:18
lalejandHi, if I want to file a bug about problem syncing contacts on my Ipod, witch package do I have to point ?20:22
dobeyubuntuone-ios-contacts i think?20:23
dobeyor something like that20:23
ralsinanessita: +1ed20:23
beunolalejand, ^20:23
lalejandbeuno : ok thx20:23
nessitaralsina: thanks!20:24
beunonessita, any reason to not uppercase the first lettter for "boommarks", "contacts", etc?20:26
beunoalso, +120:27
nessitano reason, I think is a bug, let me confirm20:29
nessitabeuno: yes, bug, can you please report? I start the branch right now20:29
beunonessita, sure20:30
beunonessita, #72802720:32
nessitabug #72802720:32
ubot4Launchpad bug 728027 in ubuntuone-client "Service names should start with an uppercase letter (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/72802720:32
nessitabeuno: did you +1'd the merge proposal?20:40
beunonessita, I did20:41
nessitabeuno: this one? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/misc-tweaks/+merge/5194820:42
nessitaafter reload I just see r-alsina's20:42
beunoI thought I did...20:42
beunonessita, now I really did20:44
dobeycan i get a couple reviews for https://code.launchpad.net/~dobey/libubuntuone/fix-load-order/+merge/51957 ?20:49
thisfrednessita: I'm thinking about share quota messaging: if we add a message to the messaging menu saying that we20:55
thisfred've exceeded the quota in a share of another user,20:55
thisfredclicking on that message should probably open the shared folder, rather than the control panel20:55
thisfredin the control panel there is nothing the user can do to fix the situation20:56
nessitathisfred: I thought we weren't putting quota notifications in the messaging menu :-(20:58
nessitathisfred: only control panel and notifications20:58
thisfredof course the blueprints and the specs are silent on this point20:58
nessitaralsina, alecu ^20:58
ralsinadobey: I'll do one!20:58
thisfrednessita: then bug #702176 is invalid?20:59
ubot4Launchpad bug 702176 in ubuntuone-client (Ubuntu) (and 1 other project) "Syncdaemon needs to add an entry to the messaging menu when a folder shared to the user exceeds the owning user's quota (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/70217620:59
thisfredIf true that;s fine20:59
thisfredI have not wasted any time on the messaging part yet20:59
nessitathisfred: I would read it as "Syncdaemon needs to add an entry to the notification system..."21:00
nessitathisfred: is the notification part done?21:00
thisfrednessita:  almost, but that's a different bug21:00
thisfredbug #70217221:00
ubot4Launchpad bug 702172 in ubuntuone-client (Ubuntu) (and 1 other project) "Syncdaemon needs to send a notification when a folder shared to the user exceeds the owning user's quota (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/70217221:00
nessitathisfred: I don't want to cause more confusion, let's ask ralsina21:01
thisfrednessita: I wrote both those bugs at the platform rally, but I have not heard since that we weren't going to do it that way21:01
ralsinaI can discuss it in maybe 10'21:01
thisfredsure, no hurry21:01
ralsinaI'm on the phone21:01
thisfredalecu rehi21:08
alecuhi thisfred!21:08
thisfredalecu: http://pastebin.ubuntu.com/574664/21:09
alecuthisfred, did you end up discussing about it?21:14
thisfredalecu: not yet, still waiting on alsina-ray21:14
ralsinamaybe 2' more, sry21:16
nessitaI'm confused about that :-/21:16
nessitaI gotta run in 4 minutes21:16
ralsinaok, let's multitask then21:16
nessitawhatever you decided, +1 from me21:16
ralsinaLast I remember discussing it, we decided what bug #702176 says21:16
ubot4Launchpad bug 702176 in ubuntuone-client (Ubuntu) (and 1 other project) "Syncdaemon needs to add an entry to the messaging menu when a folder shared to the user exceeds the owning user's quota (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/70217621:16
ralsina               And later we discusse using an emblem or however it's called on unity21:17
ralsinaI was not aware of any further discussions21:18
ralsinaDo you see a contradiction between 702176 and 702172?21:20
ralsinathisfred nessita?21:21
thisfredI don't, I was implementing them both21:21
alecuI can't find a good message to put in the messaging menu21:21
alecuthat's both short and descriptive21:21
ralsina"PANIC!" ;-)21:21
thisfredalecu: well, the messaging menu should just have a sender21:21
thisfredalecu: but i was thinking: make the envelope red,21:22
thisfredand open the offending folder, rather than the control paner21:22
nessitaralsina: as far as I know, messaging is oriented to people21:22
ralsina"the owning user's quota" means the receiving user?21:22
alecuthisfred, I like the red icon, but if we open the folder when do we say "out of space"21:22
thisfredralsina: no the user who shared this folder to you21:23
thisfredalecu: yeah, we only have the notification then and if people miss that...21:23
nessitagotta run! sorry, I'll read backlog21:23
alecubye nessita21:23
thisfredalecu does the control panel show quota for shares?21:23
* thisfred checks21:23
ralsinait does21:23
alecuthisfred, I think it does not yet.21:24
thisfredalecu: yep it does :)21:24
alecuralsina, does it? I thought we needed support from the server for that21:24
alecuthat's so cool, then :-)21:24
ralsinaalecu: well, nessita made it work, I have no details on what she had to do :-)21:24
thisfredalecu: right, so, then we open the control panel, which presumably explain wtf is the problem21:24
ralsinaThen why not open the panel? On the panel, you double click on the folder and it opens21:24
ralsinaIf it doesn't explain it right, open a bug for the panel to make it do so21:25
alecuthisfred, yes, I think nessita had a bug for that.21:25
ralsinaRemember not to do this for read-only shares21:25
thisfredalecu: question remains: do we open the control panel directly on the event, or from the messaging menu when a user clicks on the potentially confusing message21:25
ralsinaI think from the menu. Let's not kick the user in the face with control panel21:26
thisfredralsina: yeah, I was wondering: would those trigger quota exceeded events though? Easy enough to put in the check, so I'll do that anyway21:26
ralsina"Shared folder is full" is not a terrible message21:26
alecuralsina, we can do it like the "updates available" window does, opening it in the background, but flashing the taskbar.21:26
dobeyralsina: were you reviewing my branch? :)21:27
thisfredralsina: well, that's what we do when the user hirself runs out of quota, right21:27
ralsinadobey: was going to, got diverted because nessita was leaving21:27
ralsinadobey: will start .... NOW :-)21:27
alecuthisfred, ralsina: the "out of space" event is pushed when uploading files, so this will never should happen on r/o shares.21:28
alecu*no will21:28
ralsinaalecu: that's great21:28
alecuralsina, about the "Shared folder is full"... where would that go?21:29
alecuralsina, in the messaging menu?21:29
ralsinaSo "A shared folder is full" (or whatever the wording is... "cloud folder"? Ask parrino) looks like a good enough message for the messaging menu21:29
ralsinaYou don't need to say which one, because the panel will do that21:29
thisfredalecu: ralsina: so, a pop under with launcher set to urgent rather than a message for this, exactly like the UDF/Root out of quota event? I think that's a good idea. If I can make it work. Both should happen only once per 24h21:29
ralsinaEither is fine for me21:29
dobeygah "cloud" :(21:29
thisfreddobey: cloud cloud desktop cloud21:29
alecudobey, personal cloud folder!21:30
dobeyi hope it's storming in baltaimore21:30
ralsinadobey: there's a chance of cludy clouds, claude, you clod!21:30
thisfredhehe, nope, supersunny today21:30
dobeywell i hope it snows tomorrow21:30
thisfredthat is always a possibility21:30
alecu"personal cloud" makes me remember a cartoon where this raining cloud chased some poor guy21:30
dobeythe depressed kid in peanuts21:31
ralsinathisfred alecu: either solution is good enough for me, flip a coin or something if you don't agree between yourselves ;-)21:31
ralsinadobey: +1, hope that lifts a cloud from your sky! ;-)21:32
ralsinaOk, I will EOD, but if you have anything you need me for, mail me or type here, I'll see it later.21:34
thisfredalecu, what do you think? direct popunder, or message that opens the control panel?21:35
thisfredalecu: note that we do the direct popunder for the user's own quota21:35
thisfredso I like the symmetry there21:35
thisfred(well we will do that when I'm finished ;)21:35
alecuthisfred, +1 for direct popunder. I've never seen any app doing the red icon yet...21:37
thisfredalecu: yeah, that's another thing, I don't even know if that's implemented yet :)21:37
dobeyit should make me popcorn. the ultimate butter kind.21:43
thisfredalecu: nessita: shouldn't I be seeing an Ubuntu One launcher in natty? Or is that something on my todo list? :)22:12
dobeythisfred: it's there by default, do you have ubuntuone-control-panel-gtk installed?22:15
thisfreddobey: do you know how to open an window in the background and attempt to raise it?22:15
thisfreddobey: lemme check22:16
dobeythisfred: window.present_with_timestamp() and pass an old value for the timestamp22:16
thisfreddobey: I do, and I don't see it. Nor do I remember removing it manually, but that's not to say I didn't do that in a flash of temporary insanity22:17
thisfreddobey: so I assume I can't get away with subprocess.popen then.22:17
dobeywell, like i said, i think it is the utterly wrong way to be doing it, but hey what do i know :)22:19
thisfredwe want to do this from ubuntuone-client, where the window belongs to ubuntuone-control-panel22:19
dobeyi know what you want to do :)22:19
thisfreddobey: so, if I want to do that, how do I do it in the least wrong way?22:20
dobeyand i'm guessing you probably want to add a new cmd line option to cp for it, and then do whatever is necessary in there22:20
thisfredthat makes sense22:20
thisfredalso means I might get to dump that on someone else :D22:20
dobeythere's also the other problem of which control panel toa ctually open22:20
dobeybecause you probably don't want to open the gtk+ one under kde (when the qt/kde one gets implemented)22:21
dobeyso you probably need use DBus to do it22:21
thisfredyeah, but we can magic that into platform22:21
thisfredor do it over dbus22:21
dobeyi would just use dbus instead of hardcoding executable names in strings somewhere :)22:21
thisfredI like it22:21
thisfredexcept on windows we will still be out of luck22:22
thisfredbut that's for mandel to figure out :D22:22
dobeywell on windows it will use the windows ipc and do the rgith thing with COM or whatever i guess22:23
mandelthisfred: is 23:23 here and  I hate you both!22:23
mandelI'm going to bed22:23
thisfredmandel: good night22:23
thisfredmandel: PING22:23
thisfredmandel: did you see me say goodnight? :P22:24
mandelyes :D22:24
mandelsee u tom!22:24
thisfredok bye mandel, good night mandel22:24
thisfredsleep well mandel22:24
mandeldobey: later to you too22:24
dobeynight mandel22:24
thisfredif you can't sleep, just count fire engines22:25
thisfredwoohoo I think the quota stuff is done23:21
thisfredat least iteration 123:22
dobeycheers all23:24
nessitaok, I'm back23:46
nessitaand backlog is read23:46
nessitathisfred: any closing comments?23:46
nessitathisfred: +1 to popunder23:48
thisfrednessita: nope, it's almost done, one thing we need to figure out is how to best do the popunder23:49
nessitathisfred: shoot the options23:49
thisfrednessita: the control panel may need an extra switch that opens the window and tries to focus it23:49
nessita(while I start heating the owen)23:49
nessitathisfred: hum... focus what exactly?23:49
thisfredand then we can call that, directly or, as dobey suggested, over dbus23:49
thisfredso we're gnome/kde neutral at least23:50
nessita(shares are in the same tab than folders)23:50
thisfrednessita: itself23:50
thisfrednessita: the way you make the launcher bounce23:50
thisfredis open a window in the background, and then try to give it focus23:50
nessitathisfred: I have no idea how to make the panel bounce, but I can research. Pidgin and many other apps do it23:50
thisfredthat will fail, but alert the user23:50
nessitaI see23:51
nessitawell, please file me a bug, and I'll ask neil about that23:51
thisfredwe'll look tomorrow23:51
thisfredbut we're close23:51

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!