| jo-erlend | I would love to have an RSS feed of the status page so that I don't have to keep looking for new information about the couchdb issues. | 00:22 |
|---|---|---|
| karni | Have a great night pplz! | 00:38 |
| mandel | morning all! | 08:38 |
| karni | Morningses! | 08:43 |
| duanedesign | fougout | 09:01 |
| duanedesign | LOGOUT | 09:01 |
| === zyga-afk is now known as zyga | ||
| * mandel afaik for 30 min | 10:40 | |
| === mandel is now known as mandel_bbl | ||
| * fagan needs some tea | 10:41 | |
| === teknico is now known as teknico_away | ||
| fagan | :/ they delivered my package to somewhere else | 10:56 |
| fagan | Just not my week so far | 10:56 |
| nessita | hello everyone! | 11:34 |
| fagan | hey nessita | 11:34 |
| nessita | hi fagan | 11:35 |
| nessita | mandel_bbl: when you come back, let me know | 11:35 |
| nessita | ralsina: you around? | 11:37 |
| fagan | nessita: he didnt say morning so id say not yet | 11:37 |
| nessita | fagan: thanks | 11:37 |
| mandel_bbl | nessita: ping | 11:39 |
| === mandel_bbl is now known as mandel | ||
| nessita | mandel: pong | 11:40 |
| nessita | mandel: hola! | 11:40 |
| mandel | fagan: did say morning, but you were not here ;) | 11:40 |
| mandel | nessita: buenas! | 11:40 |
| fagan | mandel: I meant ralsina didnt say morning :) | 11:40 |
| mandel | fagan: oh, hehe | 11:40 |
| fagan | mandel: had to move my computer and shout at some people about losing my package | 11:40 |
| * fagan angry | 11:41 | |
| mandel | fagan: was out of context, sorry :P | 11:41 |
| nessita | mandel: so, I have a couple of questions re the fix-os-helper branch. Did you re-propose that already? | 11:41 |
| fagan | mandel: its cool :) | 11:41 |
| nessita | mandel: so I can base my question in a LP diff (if not, no problem at all) | 11:41 |
| mandel | nessita: no, is basically the one you have, so shall we use yours or mine? | 11:41 |
| mandel | nessita: I can propose it if you want… is no problem waht so ever | 11:42 |
| nessita | mandel: mine, I added some aesthetic changes as well, fixed some typos, and the like | 11:42 |
| nessita | mandel: but I (maybe) wanna request some changes based on some questions | 11:42 |
| mandel | nessita: I used yours and recreated all branches based on that, so all your changes should be there | 11:43 |
| mandel | nessita: did you get my morning report? | 11:43 |
| nessita | mandel: I did, I read it, and that's why I'm asking: | 11:43 |
| nessita | did you propose fix-os-helpers? I mean, that branch has a good size now to propose | 11:44 |
| mandel | nessita: no, lets do it now | 11:44 |
| mandel | nessita: if you have comments I can fix them there and pump the changes up to the other branches | 11:45 |
| nessita | mandel: yeah, I have some comments. I'll be using my branch to base comments on, is that ok? | 11:46 |
| mandel | nessita: lp just gave me this error: 'There is already a branch merge proposal registered for branch lp:~nataliabidart/ubuntuone-client/fix-os-helper to land on lp:ubuntuone-client that is still activ' | 11:47 |
| nessita | mandel: that is your proposal, since my branch was built on top of yours | 11:47 |
| nessita | o no? | 11:48 |
| nessita | I never proposed that branch for review | 11:48 |
| nessita | mandel: you proposed my branch for review! :-) | 11:49 |
| mandel | nessita: no idea, it looks like I propose it for you… can you do that in lp? | 11:49 |
| mandel | that is weird... | 11:49 |
| nessita | mandel: seems like it | 11:49 |
| nessita | let's remove that proposal | 11:49 |
| nessita | mandel: so. you branch from my branch, push to a location of yours, and re-propose | 11:49 |
| mandel | nessita: on it | 11:50 |
| nessita | mandel: you removed the old proposal??? | 11:50 |
| nessita | old == https://code.launchpad.net/~mandel/ubuntuone-client/fix-os-helper/+merge/68227 | 11:50 |
| mandel | yes | 11:51 |
| nessita | mandel: never remove proposal with comments in it, just mark it as rejected | 11:51 |
| mandel | nessita: ok, did not know that... | 11:51 |
| mandel | nessita: the new is here: https://code.launchpad.net/~mandel/ubuntuone-client/fix-os-helper/+merge/68530 | 11:51 |
| nessita | great, looking | 11:52 |
| nessita | mandel: ok, my first concern is the name of the method remove_utf8_chars | 11:54 |
| mandel | not correct? | 11:54 |
| nessita | mandel: the name is misleading, if I understood correctly, it should be something like remove_unicode or map_unicode_to_utf8_chars | 11:54 |
| nessita | mandel: since the method grabs unicode objects and replace it with bytes encoded with utf8 | 11:55 |
| mandel | nessita: what about, readd_illegla_chars? | 11:55 |
| mandel | illegal* | 11:55 |
| nessita | restore_illegal_chars can work | 11:55 |
| mandel | nessita: and the other one would be, remove_illegal_chars | 11:56 |
| nessita | exactly | 11:56 |
| mandel | so we know one those the opposite of the other | 11:56 |
| nessita | yes | 11:56 |
| mandel | nessita: ok, np | 11:57 |
| nessita | mandel: so, next question is in the file filesystem_notifications.py. Why the ex-longpath decorator was removed from _add_single_watch and rm_path? | 11:57 |
| nessita | and from add_watch as well? | 11:57 |
| nessita | mandel: is making me 'noise' that the decorator was there before, and now, after the rename to windowspath, is not longer used in those methods | 11:58 |
| mandel | nessita: let me look at the context to give you a decent answer | 11:58 |
| nessita | thanks | 11:58 |
| mandel | the issue is that rm_watch is called by the watch manager rm_watch, that has two problems, one, that we are mangling the path twice, two that our asserts will compain because in the second pass of the decorator we are passing unicode and not bytes | 12:03 |
| * fagan brea k | 12:04 | |
| nessita | the assert complaining is not a problem but a relief, since mangling the path twice is bad | 12:04 |
| nessita | mandel: I don't understand why rm_watch is called twice | 12:05 |
| mandel | nessita: so we have to rm_watch, one from FileSystemMonitor and the other from WhatchManager, FileSystemMonitor is the public interface used by sd, therefore passes get byte, the other is internal | 12:06 |
| mandel | which is the watchmanager and has no decorator, before the asserts there was no problem, now when we have the decorator in both the asserts fail | 12:07 |
| j0nr | hey folks, is it possible to publish photos synced from my phone and some how grab all the public url's? So I can chuck a simple html file in the front to show a gallery of my photos | 12:07 |
| nessita | mandel: ok, but are we calling rm_watch twice for the same path? | 12:07 |
| nessita | j0nr: hi there, let me ping someone that knows a lot about that | 12:08 |
| nessita | karni: you around? | 12:08 |
| karni | nessita: always! | 12:08 |
| karni | ;) | 12:08 |
| nessita | karni: hola! can you please give j0nr a hand? | 12:08 |
| karni | sure | 12:08 |
| j0nr | thanks nessita | 12:08 |
| karni | lemme see | 12:09 |
| nessita | j0nr: you're welcome! | 12:09 |
| nessita | mandel: so, not sure if you understand my question | 12:09 |
| mandel | nessita: no, we are calling FileSystemMonitor.rm_watch which calls WatchManager.rm_watch | 12:09 |
| mandel | nessita: yes, why we removed the decorator, answer: because the decorator is applied on FileSystemMonitor method | 12:09 |
| j0nr | I may be thinking outside the 'storage' box here, but I have done simple php scripts to fetch all images in a directory and create a simple gallery. Just wondering if I could somehow do the same with U1 so my gallery is always showing latest pics synced | 12:09 |
| mandel | whic follows the FileSystemMonitor interface from linux, which has more than just calling the wathcmanager | 12:10 |
| nessita | mandel: so, you said that FileSystemMonitor receives bytes, right? | 12:10 |
| karni | rye: Could you give j0nr the link to your U1 gallery script please? | 12:10 |
| karni | j0nr: ↑ I think this is closed to what you have in mind | 12:11 |
| j0nr | karni: but the random url of published photos seems to make this difficult... its not like I can loop throu photos ammending their names to www.ubuntuone.com/j0nr/photos/... | 12:11 |
| mandel | nessita: yes, and passes unicode to WatchManager | 12:11 |
| karni | j0nr: People complained about easily guessable urls. We can't have it that easy. | 12:11 |
| mandel | so WatchManager.rm_path should not get byte but unicode | 12:11 |
| nessita | mandel: where is that code? | 12:11 |
| j0nr | karni: oh ok cool. so can I run php scripts on U1? | 12:11 |
| karni | j0nr: That's not what I meant :) | 12:11 |
| nessita | mandel: I mean filename and approx line | 12:12 |
| karni | one moment | 12:12 |
| mandel | nessita: ubuntuone.platform.windows.filesystem_notifications line 806 | 12:12 |
| mandel | that would be the FileSystemMonitor method | 12:12 |
| nessita | mandel: the conversion to unicode is in the decorator? | 12:13 |
| karni | j0nr: rye will help you when he's around. He has a script that you can run locally, to generate a nice gallery. | 12:13 |
| karni | I think ralsina also had one! :) | 12:13 |
| mandel | nessita: yes | 12:13 |
| mandel | nessita: so that when we move to unicode, we remove the decorator | 12:14 |
| nessita | mandel: ok, I see. SO why the add_watch right next to rm_watch does not have the decorator? | 12:14 |
| nessita | line 822 | 12:14 |
| nessita | mandel: the inconsistency is what is killing me :-) | 12:14 |
| nessita | (in order to process this0 | 12:14 |
| j0nr | karni: cheers... does it involve having to have the photos synced to a local machine? | 12:16 |
| mandel | nessita: hehe, I know you… and you are right, there the problem is that Watch in the __init__ is performing the same operation, lets make the init do not care and add a decorator there, does that sound better? | 12:16 |
| karni | j0nr: It's one-time run thing. It's not umm.. dynamic. | 12:16 |
| karni | j0nr: And I'm not sure about the answer to your question. | 12:16 |
| karni | j0nr: I think it's possible. But once you generate it, you can simply put the page on a server | 12:17 |
| j0nr | karni: ok thanks :) | 12:17 |
| karni | and it will access your files via public urls | 12:17 |
| karni | So just hunt these people ;) | 12:17 |
| j0nr | karni: so I take it there is a way to interogate U1 files to discover if they have a public url. Can this be done via command line? | 12:18 |
| nessita | mandel: indeed | 12:18 |
| j0nr | can I sync U1 to a server? I.e. no GUI | 12:18 |
| mandel | nessita: ok, lets add that in the comments and lets take care of it! | 12:18 |
| nessita | mandel: same question for freeze_being vs freeze_rollback and freeze_commit | 12:19 |
| karni | j0nr: run u1sdtool in termnal and read up on --info=PATH switch, maybe it contains the public url :) | 12:19 |
| nessita | mandel: I'll start adding the comments to the MP, but I have more things I want to talk with you from this branch | 12:19 |
| karni | j0nr: you can certainly publish from command line, I never grabbed public link sthat way | 12:19 |
| mandel | nessita: ok, lets talk about everything before I start typing like a mad man | 12:20 |
| nessita | mandel: yes. So, whta about freezes? | 12:20 |
| nessita | "same question for freeze_being vs freeze_rollback and freeze_commit" | 12:20 |
| j0nr | karni, nice one :) | 12:21 |
| mandel | nessita: events should have paths in byte, so commit which takes events should not have a decorator, agreed? | 12:22 |
| nessita | ok | 12:23 |
| nessita | same for rollback and begin, but begin has the deco | 12:23 |
| mandel | yes, and that should be wrong… since its calling the shared class by windows and linux which is in sd and should be using byte... | 12:24 |
| nessita | mandel: ^ | 12:24 |
| === teknico_away is now known as teknico | ||
| mandel | nessita: ^^ | 12:25 |
| mandel | nessita: two up, or which ever got first | 12:25 |
| mandel | nessita: we have not been bitten by a bug there because the notification from windows are a little smarter regarding moves, which is what the freeze is for | 12:25 |
| mandel | so, as I'd say in spanish, pura suerte | 12:25 |
| nessita | mandel: ok, so I think this class needs a careful review of decorators, I'll add that to the MP, yes? next question: | 12:26 |
| nessita | os_helper.py | 12:26 |
| mandel | nessita: completely agree | 12:26 |
| nessita | _get_syncdaemon_valid_path, I would expect this class, that assert over the param to be a unicode, to return bytes | 12:26 |
| mandel | nessita: let do this, rule of thum, Watch and Watch manager do nothing with paths, FileSystemNotifications take care of it | 12:26 |
| nessita | mandel: makes sense, can we add tests? I see that in this branch we removed some decorator and tests did not change | 12:27 |
| mandel | nessita: yes, we should, this is probably the most important branch we have, if we get it right lots of bugs will be fix, lets take our time and add as many tests as possible if needed | 12:28 |
| mandel | nessita: I'm 110% sure all the encoding and \\?\ things are the main root of issues in the port | 12:29 |
| mandel | visteme despacio... | 12:29 |
| nessita | right | 12:29 |
| nessita | mandel: so, I'm starting again with the next question to avoid confusions: in os_helper.py: _get_syncdaemon_valid_path, I would expect this class, that assert over the param to be a unicode, to return bytes | 12:30 |
| j0nr | can i install u1sdtool on a server? | 12:31 |
| nessita | mandel: ex-remove_utf8_chars only replaces unicode chars, but does not transform to bytes | 12:32 |
| nessita | mandel: that method (_get_syncdaemon_valid_path) should do the .encode('utf8') | 12:33 |
| mandel | nessita: it is asserting that it gets unicode, it is returning unicode, I just did a grep and this is only used in the filesystem notifications to get the path of the even (not to be confused with the pathname) which is never used, yet it should be using bytes, issue here is that wath should be using unicode in self._path, so we have to remove the \\?\ and the unicode chars and be able to perform the correct operation in line 212 in filesystem_notificati | 12:34 |
| mandel | nessita: sorry for the terrible spelling, I was looking at the vm while I typed... | 12:34 |
| nessita | mandel: the thing is the name of the method. A method with that name, *should* return bytes since is meant to be used by syncdaemon | 12:35 |
| mandel | nessita: I'd remove that method and use reinstantiate_illegla_chars instead | 12:35 |
| nessita | mandel: restore_illegal_chars? :-) | 12:35 |
| mandel | eso :P | 12:35 |
| nessita | ok | 12:35 |
| nessita | mandel: also, what do you think about changing _validate_windows_path to _assert_path_is_bytes? | 12:36 |
| mandel | nessita: yes, sounds good, is more direct | 12:37 |
| nessita | mandel: and then you can use that in the implementation of _valiate_output_path, reaplacing the call to | 12:38 |
| nessita | assert isinstance(path, str), 'All paths returned to sd should be bytes' | 12:38 |
| nessita | mandel: yes? | 12:38 |
| ralsina | morning! | 12:38 |
| mandel | nessita: yes | 12:38 |
| nessita | ralsina: hey there! you ok? :-) | 12:39 |
| nessita | mandel: and I think that's all I have :-) | 12:39 |
| mandel | nessita: great review :) | 12:39 |
| nessita | mandel: :-) | 12:39 |
| ralsina | nessita: had another bout of stomach ache last night but much better than the night before | 12:40 |
| nessita | mandel: I'll add all this to the MP | 12:40 |
| ralsina | nessita: probably will not have another one tonight :-) | 12:40 |
| mandel | nessita: if we have everything in comments I'll start working on it right now, I really think this is coming very nice, even if we need some many cycles to land it | 12:40 |
| mandel | ralsina: there is an age where you can only dinner raisins… | 12:41 |
| * mandel hides | 12:41 | |
| ralsina | mandel: and I hate raisins :-) | 12:42 |
| * fagan slaps mandel | 12:42 | |
| nessita | mandel: all comments added | 12:43 |
| nessita | mandel: let me know as soon as those are fixed | 12:43 |
| nessita | mandel: and I will start charging you and ralsina a dollar per each needs fixing :-P (j/k) | 12:44 |
| ralsina | nessita, mandel, alecu: possible hotel for the sprint http://www.hoteljacaranda.com.ar | 12:44 |
| nessita | ralsina: does that work for you? | 12:45 |
| ralsina | nessita: 15 minutes by car | 12:45 |
| ralsina | nessita: it's quite far fr alecu though | 12:45 |
| nessita | ralsina: have a googlemap references? | 12:45 |
| ralsina | http://is.gd/Pct7ly | 12:46 |
| mandel | nessita: it should be calculate by the size of the branch.. otherwise is not fair! | 12:48 |
| nessita | ralsina: a taxi from AEP would be like $100? | 12:48 |
| nessita | mandel: lol | 12:48 |
| ralsina | nessita: more or less, yes | 12:48 |
| nessita | ralsina: do taxis go there or shall I book a remis? | 12:49 |
| nessita | ralsina: I know that taxis from capital charges extra for going into the provincia | 12:49 |
| ralsina | taxis usually go there, but since it's outside CABA they may charge you a bit extra (say, $10) | 12:49 |
| ralsina | or they may not, usually you arrange that before starting the trip | 12:50 |
| fagan | 10 mins to standup | 12:50 |
| nessita | ralsina: ok | 12:50 |
| ralsina | mandel OTOH is going to have close to the longest taxi ride possible in buenos aires :-) | 12:50 |
| nessita | mandel: did you add your travel details to the wiki page? | 12:51 |
| mandel | ralsina: so no night out… :( | 12:51 |
| mandel | nessita: no, will do after the stan up | 12:51 |
| ralsina | mandel: nah, it's les than 1 hour http://is.gd/0PbzjH | 12:51 |
| mandel | ralsina: that in spain is like goign to a diff city... | 12:52 |
| ralsina | mandel: at least on a sunday it's less than 1 hour | 12:52 |
| ralsina | mandel: here too! It's just that the cities are next to each other ;-) | 12:52 |
| nessita | ralsina: I moved the sprint page to https://wiki.canonical.com/OnlineServices/Sprints/SprintJuly2011 | 12:52 |
| nessita | ralsina: I'll let marianna know | 12:52 |
| mandel | ralsina: well, I guess is better for you guys that I'm not too close :) | 12:53 |
| ralsina | mandel: you will be 15 minutes from my home, we'll show you around if you want | 12:54 |
| mandel | ralsina: good :) | 12:54 |
| ralsina | mandel: and check the satellite view, the hotel is 2 blocks from the private piers. In the "national capital of sailing" | 12:55 |
| fagan | moi | 13:00 |
| mandel | me | 13:00 |
| nessita | me | 13:00 |
| fagan | ralsina dobey | 13:00 |
| dobey | me | 13:00 |
| ralsina | me | 13:00 |
| fagan | ill go | 13:01 |
| fagan | DONE | 13:01 |
| fagan | * tested alecu's script | 13:01 |
| fagan | * set up a usability phone call for tomorrow with matthew revell about lp | 13:01 |
| fagan | * finished the 3 months of backlog | 13:01 |
| fagan | TODO | 13:01 |
| fagan | * More unassigned bugs (if nothing else is needed) | 13:01 |
| fagan | * reply to a lecturer about something with college should take a bit | 13:01 |
| fagan | BLOCKED | 13:01 |
| fagan | * Nope | 13:01 |
| fagan | mandel: go | 13:01 |
| mandel | DONE: Rebased all branches in the work done by nessita. One of the branch has a small inconsistent test failing. Talk with nessita about improving fix-os-helper. | 13:01 |
| mandel | TODO: Perform changes in fix-os-helper according to review. Add travel details to wiki. Talk with verterok about local_rescan failing test. | 13:01 |
| mandel | BLOCKED: no | 13:01 |
| mandel | nessita, go go go | 13:01 |
| nessita | DONE: bug #810662, tons of reviews, meetings, chasing people | 13:01 |
| nessita | TODO: more of the same! :-) | 13:01 |
| nessita | BLOCKED: nopes | 13:01 |
| nessita | NEXT: dobey | 13:01 |
| ubot4 | Launchpad bug 810662 in ubuntuone-control-panel "QT UI: "Delete" button in the devices tab does not work (affects: 1) (heat: 15)" [High,Triaged] https://launchpad.net/bugs/810662 | 13:01 |
| dobey | λ DONE: reorg plan, converted protocol and cp to dh_python2 in ubuntu | 13:01 |
| dobey | λ TODO: reorg work, shim code | 13:01 |
| dobey | λ BLCK: None. | 13:01 |
| dobey | ralsina | 13:01 |
| ralsina | DONE: windows call, mgmt call, reviews, closed bug #810521, worked on bug #811111, published an installer, worked on bug #800376 | 13:01 |
| ralsina | TODO: work on #800376, close #811111 and work a lot of minor ones | 13:01 |
| ubot4 | Launchpad bug 810521 in ubuntuone-client "Typo in signal name (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/810521 | 13:01 |
| ubot4 | Launchpad bug 811111 in ubuntuone-windows-installer (and 1 other project) "There is no visual indication that the form is being processed (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/811111 | 13:01 |
| ubot4 | Launchpad bug 800376 in ubuntuone-windows-installer "Add "syncing your computer to the cloud" page to the installer (affects: 1) (heat: 8)" [High,In progress] https://launchpad.net/bugs/800376 | 13:02 |
| ralsina | and I am the last one. Comments? | 13:02 |
| nessita | did alecu say me? | 13:02 |
| fagan | Nope my call is around the same time as standup tomorrow but im a good multitasker | 13:02 |
| dobey | nessita: no | 13:02 |
| fagan | alecu isnt online | 13:02 |
| nessita | ah, ok | 13:03 |
| nessita | no comments here | 13:03 |
| * mandel lunch | 13:03 | |
| === m_conley_away is now known as m_conley | ||
| fagan | alecu: hey, tested that script I can give you the output if you want | 13:04 |
| fagan | it worked I think it gave out a big long id thingy | 13:04 |
| alecu | fagan, please do | 13:04 |
| fagan | alecu: kk | 13:05 |
| fagan | 5 mins ill have it in a pastebin | 13:05 |
| dobey | alecu: standup :) | 13:05 |
| alecu | me | 13:06 |
| dobey | alecu: it is your turn, we already went :P | 13:06 |
| * alecu is writing notes | 13:06 | |
| fagan | http://pastebin.ubuntu.com/648224 | 13:06 |
| fagan | alecu: ^ | 13:07 |
| fagan | Thats for my main account do you need another account too | 13:07 |
| fagan | I tested it and it was something pretty similar | 13:07 |
| alecu | DONE: started working on #803669, many reviews | 13:07 |
| alecu | TODO: finish branch, work on #803672 | 13:07 |
| alecu | BLOCKED: no | 13:07 |
| fagan | alecu: ok here is the second http://paste.ubuntu.com/648228 | 13:10 |
| nessita | bug #803669, bug #803672 | 13:14 |
| ubot4 | Launchpad bug 803669 in ubuntuone-client (and 1 other project) "Get the port numbers from the users' registry (affects: 1) (heat: 6)" [Low,Triaged] https://launchpad.net/bugs/803669 | 13:14 |
| ubot4 | Launchpad bug 803672 in ubuntuone-client (Ubuntu) (and 2 other projects) "Do not start two instances of this process (affects: 1) (heat: 6)" [Undecided,Invalid] https://launchpad.net/bugs/803672 | 13:14 |
| === teknico_ is now known as teknico | ||
| * mandel back | 13:23 | |
| mandel | nessita: I'm starting with the changes | 13:23 |
| nessita | mandel: great! | 13:24 |
| ralsina | alecu, nessita, mandel, dobey, vds, CardinalFang, thisfred: I just sent you your performance reviews. Sorry about the loooooooong delay, it was my own fault. | 13:27 |
| thisfred | thx, just saw it! | 13:27 |
| mandel | ralsina: ack | 13:27 |
| ralsina | mail got misfiled without me seing it, and I first expected it to appear, then forgot about it, then assumed it was never sent | 13:28 |
| mandel | so who sai in my peer review that I'm extremely attractive? | 13:28 |
| mandel | :P | 13:28 |
| nessita | mandel: fagan after drinking tequilas? :-) | 13:28 |
| ralsina | mandel: you attract jellyfish! | 13:29 |
| jo-erlend | can someone confirm a bug for me? If you add a file to a synced folder, then immediately choose to publish the file, then afterwards, it's no longer possible to either publish or unpublish the file from nautilus. That means you have to move the file somewhere else and then back in order to do things with it. | 13:29 |
| mandel | hehehe | 13:29 |
| jo-erlend | I think that could be fixed by not providing the menu option to publish a file until it's been synced. | 13:29 |
| * mandel goes are reads the report... | 13:29 | |
| mandel | ralsina: update my fligth details in the wiki | 13:32 |
| ralsina | mandel: is that an order? ;-) | 13:32 |
| mandel | ralsina: does marianna know I'm staying til the 2nd | 13:32 |
| mandel | ralsina: was not an order, it was a missing ed :P | 13:33 |
| ralsina | mandel: if it's in the wiki, she knows. She may not knw canonical is paying for it though, but I'll tell her that. If she asks you, tell her it's approved by me and Chipaca | 13:33 |
| ralsina | at least that's how I understood Chipaca in the mumble call :-) | 13:33 |
| mandel | ralsina: ok, if not I can pay, unless is too expensive | 13:34 |
| ralsina | mandel: I have a couch | 13:34 |
| ralsina | mandel: And I have a key for a nice furnished appartment downtown as well ;-) | 13:35 |
| mandel | ralsina: now, that sounds like a plan… | 13:35 |
| alecu | fagan, thanks for the pastebins, was just what I expected. | 13:36 |
| lisette | nessita: hi | 13:37 |
| fagan | nice alecu | 13:37 |
| nessita | lisette: hi there! | 13:38 |
| lisette | nessita: got your mail; working on it :) | 13:38 |
| nessita | lisette: thanks! | 13:38 |
| nessita | lisette: alecu added a comment to the bug report, just FYI | 13:38 |
| lisette | i saw | 13:39 |
| nessita | lisette: great! | 13:39 |
| ralsina | mandel, alecu, nessita, chipaca: looks like I broke my headset. Can we do the mumble a little later today? | 13:40 |
| nessita | ralsina: sure by me, what time? | 13:40 |
| Chipaca | ralsina: yeah, i'll be on late today | 13:41 |
| ralsina | I would say after lunch so I can go buy a new ne | 13:41 |
| * mandel read that as 'I broke my head...' | 13:41 | |
| ralsina | maybe 13:30 art? | 13:41 |
| ralsina | mandel, is that too late for you? | 13:41 |
| mandel | ralsina: is never too late :) | 13:41 |
| nessita | ralsina: weren't you maybe traveling for banking issues today? | 13:42 |
| ralsina | was a maybe, but I am not :-) | 13:43 |
| ralsina | nessita: found other solutions | 13:43 |
| nessita | ralsina: oh good! | 13:43 |
| nessita | ralsina: after lunch sounds ok, but maybe is too late for Chipaca? | 13:43 |
| Chipaca | nessita: give me a time | 13:45 |
| nessita | Chipaca: 2pm ART, 5pm UTC | 13:45 |
| ralsina | Chipaca: 13:30 ART? | 13:45 |
| nessita | ralsina, alecu^ | 13:45 |
| ralsina | ok, 2pm | 13:45 |
| Chipaca | I can do 6pm, yes :) | 13:45 |
| nessita | great! is a date! (?) | 13:45 |
| alecu | ack | 13:47 |
| ralsina | Chipaca: we'll keep it short, promise! | 13:47 |
| mandel | nessita: so, when do we meet? | 13:47 |
| nessita | mandel: 5pm UTC | 13:48 |
| nessita | mandel: 7pm your time | 13:48 |
| mandel | nessita: ok | 13:48 |
| vendix | good day | 13:55 |
| mandel | nessita: regarding the os_helper branch, I'm going to add the validate_input_path in the WatchManager and Watch methods to ensure that those just get unicide and not bytes, do you agree on that? | 13:56 |
| mandel | nessita: that way we force FileSystemNotifications to always pass the paths correctly to the more platform dependent code | 13:56 |
| nessita | mandel: yes, but you mean the assert_path_is_bytes, right? | 13:57 |
| nessita | mandel: since we rename it (in our heads) | 13:57 |
| mandel | nessita: yes.. just did the change, and yes _assert_path_is_bytes is the one | 13:58 |
| nessita | mandel: yes to the first question then | 13:58 |
| mandel | nessita: but that is the method name not the decorators, should I rename the decorator too? | 13:58 |
| nessita | mandel: yeah, I think it makes sense | 13:58 |
| mandel | nessita: ok, search and replace on the way! | 13:59 |
| vendix | what happened to Ubuntu One? i can't login. this what i get to when i try to log into Ubuntu One account. http://unknown.lt/screen.png | 14:01 |
| dobey | vendix: are you on 11.04 or 11.10? | 14:02 |
| vendix | donno | 14:02 |
| vendix | dunno | 14:02 |
| vendix | how to see that? | 14:02 |
| dobey | vendix: if you are on 11.04 with gnome3 ppa installed, the gnome3 stuff broke various things | 14:03 |
| dobey | vendix: /etc/lsb-release i think | 14:03 |
| mandel | nessita: other change validate_output_path assert_output_path_is_unicode | 14:04 |
| vendix | dobey: give me different command | 14:05 |
| nessita | mandel: but... validate_output_path returns bytes, no? | 14:05 |
| vendix | to see the version of ubuntu | 14:05 |
| dobey | vendix: that isn't a command, it is a file that contains the ubuntu version information | 14:05 |
| mandel | nessita: sorry yes, s/unicode/bytes | 14:06 |
| nessita | mandel: thing is that validate_output_* also check the long prefix not there | 14:06 |
| nessita | mandel: that's why I did not suggested the renaming | 14:07 |
| nessita | mandel: I will leave the function as is, fixing the typo in def _valiate_output_path(path): | 14:07 |
| vendix | 11.04 | 14:07 |
| vendix | how to upgrade to 11.10? | 14:07 |
| mandel | nessita: lets split it in two, so for when ever we need to use unicode we simply remove that decorator and not the check of the long path prefix | 14:08 |
| mandel | checking for \\?\ will always be needed | 14:08 |
| nessita | mandel: ok, sounds good (I hope the branch does not grow that much) | 14:08 |
| mandel | nessita: yeah, seems to be an accordeon branch | 14:09 |
| dobey | vendix: if you *really* want to upgrade to the development release, you can run "update-manager -d" and it should offer you the option to upgrade | 14:14 |
| ralsina | I'm having a horrible time concentrating on writing code today :-( | 14:56 |
| ralsina | anyone needs any reviews? | 14:56 |
| alecu | ralsina, I can use a review here: https://code.launchpad.net/~alecu/ubuntuone-client/unbreak-connect/+merge/68309 | 14:57 |
| alecu | nessita, you might want to re-review the above branch, too. | 14:57 |
| nessita | alecu: sure! | 14:57 |
| alecu | thanks! | 14:57 |
| ralsina | alecu: this needs any sort of IRL testing? | 14:58 |
| nessita | alecu: quick tweak, can you please add a space after the : in lambda _:reply_handler()? | 14:58 |
| alecu | ralsina, just try stopping and starting again the syncdaemon from the control panel. | 14:58 |
| alecu | nessita, sure | 14:58 |
| nessita | alecu: code looks great! running tests as soon as you push that | 14:59 |
| ralsina | alecu: ack | 15:00 |
| alecu | nessita, pushed | 15:01 |
| nessita | alecu: yey | 15:02 |
| ralsina | nessita: replied to your mail about what to do with the installer UI and ussoc. Basically, I'm ok with that, debugging inside ussoc is much harder, and things are just weird :-( | 15:04 |
| nessita | ralsina: ack, I'll start working on that after alecu's gives his +1, if he agrees, of course :-p | 15:06 |
| nessita | ok, I need to go to the meat shop otherwise my half orange will starve\ | 15:07 |
| * nessita -> brbs | 15:07 | |
| * mandel small break to get some air in to the brain | 15:11 | |
| rye_ | has anybody pinged me while I was half-away | 15:17 |
| rye_ | ? | 15:17 |
| * ralsina ---> cook | 15:19 | |
| === ralsina is now known as ralsina_cooking | ||
| nessita | alecu: all green, approving | 15:26 |
| alecu | nessita, great. I didn't understand your comment about "alecu's gives his +1" | 15:27 |
| alecu | nessita, is this related to splitting the sso? | 15:27 |
| alecu | nessita, I mean, not running the installer on the sso process? | 15:27 |
| alecu | nessita, I'm +1 to not running the installer on the sso process. | 15:27 |
| nessita | alecu: I answered to ralsina_cooking's report, last night, proposing moving the installer process outside sso | 15:27 |
| nessita | alecu: what about the idea I proposed? does it make sense? | 15:27 |
| alecu | nessita, I read it, but I have some doubts: | 15:29 |
| nessita | alecu: I'm here to answer | 15:29 |
| nessita | alecu: I'm here to answer "i don't know" :-P | 15:29 |
| alecu | nessita, what about the "registration": will we be duplicating all the registration steps on the installer? | 15:29 |
| alecu | nessita, the captcha and validation code, and all that? | 15:30 |
| alecu | nessita, or will the installer use the _ui.py modules from sso? | 15:30 |
| nessita | alecu: the registration is kinda duplicated already, since all the QT side of SSO is "wierd" | 15:30 |
| nessita | alecu: let me explain a bit more | 15:30 |
| alecu | right :P | 15:30 |
| alecu | yes, please explain | 15:31 |
| nessita | alecu: the wizard re-uses the following: | 15:31 |
| nessita | * qt widgets | 15:31 |
| nessita | * qt controllers for that widgets, which I *think* provides all the logic, for example, validation and error management from sso backend | 15:31 |
| nessita | I know ralsina_cooking had to add some code in the wizard layer as a glue between those controllers and widgets, but basically no, the registration will not be duplicated | 15:32 |
| * mandel back | 15:32 | |
| alecu | nessita, cool. | 15:33 |
| alecu | nessita, mandel, ralsina_cooking: I have a serious concern about the "client re-org" that's been mentioned on ubunet-discuss. | 15:34 |
| nessita | alecu: can you reply to the email? | 15:34 |
| alecu | nessita, mandel, ralsina_cooking: I'm not sure doing this a few days before the sprint is such a great idea. | 15:34 |
| nessita | alecu: yes, I mentioned more or less the same but focusing on the assets | 15:35 |
| alecu | nessita, well, I can. But ideally I'd like to discuss this during the mumble. | 15:35 |
| nessita | alecu: can you please bring this up in mumble? | 15:35 |
| nessita | ah, that | 15:35 |
| alecu | sure | 15:35 |
| mandel | alecu: I agree, doing it now means doing two big things at once… and I'm not 100% confident that we will be running tests in both platforms, I'd go against it | 15:37 |
| alecu | mandel, exactly | 15:37 |
| mandel | alecu: I don't know if you have this in ar: 'el que mucho avarca, poco aprieta' | 15:40 |
| DiegoSarmentero | mandel, yes | 15:40 |
| mandel | DiegoSarmentero: that is why I like smaller wom... | 15:41 |
| * mandel stops before someone brings the code of conduct | 15:41 | |
| alecu | mandel, we have it, but with a "b" in "avarca" | 15:41 |
| alecu | :-) | 15:41 |
| mandel | alecu: va, I hate that b/v thing, we pornounce them the same and so I always get them confused | 15:42 |
| alecu | jajajaja | 15:42 |
| mandel | also, remove the bloody h from the spanish lang! | 15:42 |
| alecu | mandel, and let's get rid of "z and c". "s and k" should be enough for everybody! | 15:43 |
| mandel | alecu: j and g, is the same a jato and a gato | 15:43 |
| mandel | :P | 15:43 |
| * alecu starts sharpening his str.translate() skills | 15:44 | |
| * DiegoSarmentero is away: Not here... send me an e-mail! :P | 15:46 | |
| dobey | ok, time to get lunch. bbiab | 15:52 |
| mandel | nessita: I'm out walking the dog oherwise wont make it to mumble, I have added some extra decorators in file system notifiactions so that it brakes when we do not get the correct type of path, will run slower but for debugging is way better, will tell you more later | 15:53 |
| nessita | mandel: ack! | 15:55 |
| * mandel dog whispering | 15:55 | |
| nessita | ok, lunch! | 16:04 |
| === teknico is now known as teknico_away | ||
| Uber_Geek | Ello all, been a while. | 16:27 |
| Uber_Geek | any update on the Windows Version of U1? | 16:28 |
| nessita | Uber_Geek: we're working like crazy :-D | 16:31 |
| * mandel back | 16:31 | |
| Uber_Geek | :) | 16:31 |
| mandel | Uber_Geek: yes, like crazy… you wont believe it, but the new version will be waaaay better that the horrible beta I wrote :) | 16:32 |
| * mandel runs after dog that ran away, bastard!!! | 16:35 | |
| === ralsina_cooking is now known as ralsina | ||
| nessita | ralsina: ping | 16:37 |
| ralsina | nessita: pong | 16:37 |
| * mandel back | 16:38 | |
| nessita | ralsina: I think you reviewed yesterday a branch of mine, but I think you did not set the approve | 16:38 |
| nessita | https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/letmeremove/+merge/68444 | 16:38 |
| ralsina | nessita: I got tracebacks with that branch when deleting devices | 16:40 |
| nessita | ralsina: right, and alecu said we should not get those anymore after a branch of his | 16:41 |
| dobey | hrmm | 16:41 |
| ralsina | nessita: right, so I assumed this brach should go in after alecu's? | 16:41 |
| dobey | moving ubuntuone.credentials is a bit more complex than i thought | 16:41 |
| nessita | dobey: ah, speaking of that, we're evaluating not having our stuff moved before our sprint... :-/ | 16:42 |
| nessita | dobey: since we need everything working next week since we're sprinting | 16:42 |
| nessita | dobey: unless you can confirm everything keeps working on windows? :-) | 16:42 |
| nessita | ralsina: well, is working in linux... so seems like a windows only issue, and not related to the code itself I'm addding | 16:43 |
| ralsina | nessita: ok, +1 then | 16:43 |
| dobey | nessita: i wouldn't break windows; and the merge request for removing stuff after moving it over, would have to get tested on windows :) | 16:43 |
| nessita | dobey: well, as long as you make sure everything is tested in windows (by you or by us), I think it can work | 16:44 |
| dobey | nessita: right. and i thought we made that commandment at UDS anyway :) | 16:45 |
| nessita | dobey: we did, yes, but since we're in such a hurry, we sometimes forget... | 16:45 |
| ralsina | I am starting to feel really sick. I will go to the doctor after the mumble. | 16:47 |
| === beuno is now known as beuno-lunch | ||
| nessita | ralsina: really sick how? :-/ | 16:52 |
| ralsina | nessita: gastritis, same as the last two nights | 16:53 |
| ralsina | nessita: and I did not even eat anything :-( | 16:53 |
| nessita | ralsina: you know not eating is the worse things you can do for gastritis, right? | 16:53 |
| nessita | one of the* | 16:53 |
| nessita | ralsina: you need to eat, every 2 hours | 16:53 |
| ralsina | nessita: I ate pure de zapallo y arroz blanco. That's what I meant by "not eat anything" :-) | 16:54 |
| nessita | ralsina: so you did eat! and healthy food | 16:54 |
| ralsina | nessita: special, gastritis-approved food, too | 16:54 |
| nessita | yum! | 16:55 |
| ralsina | nessita: and it's starting to hurt anyway. So, I am going to see the doctor and ask why | 16:55 |
| nessita | ralsina: did you drink mate, or coffee or tea? | 16:55 |
| ralsina | nessita: nope, just water | 16:55 |
| nessita | ralsina: what a embole | 16:55 |
| ralsina | I have had a few of these, I know the routine | 16:55 |
| ralsina | And usually I get better in a day or two, but it's incredibly annoying | 16:56 |
| nessita | ralsina: yes, I've had gastritis in the past | 16:56 |
| nessita | alecu: you available for a review? or two? :-) | 16:57 |
| alecu | nessita, sure | 16:58 |
| ralsina | nessita: can I point you at u1-internals? There's a guy saying ussoc doesn't work on oneiric | 16:59 |
| nessita | alecu: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/letmeremove/+merge/68444 thanks! | 16:59 |
| ralsina | nessita: your or dobey, I assume, is the right person to ask about that | 16:59 |
| nessita | ralsina: sure! | 16:59 |
| mandel | nessita, ralsina, alecu, Chipaca: mumble? | 17:00 |
| nessita | yes! | 17:00 |
| Chipaca | mumble doesn't like me right now | 17:01 |
| nessita | mumcle is rejecting my (correct) password | 17:01 |
| mandel | nessita: same here... | 17:01 |
| ralsina | nessita: mine too | 17:01 |
| Chipaca | ditto | 17:01 |
| ralsina | so... | 17:01 |
| mandel | joder | 17:01 |
| nessita | skype! | 17:01 |
| nessita | Chipaca: can call us? | 17:02 |
| nessita | mandel, ralsina, alecu: skype | 17:02 |
| ralsina | am there already | 17:02 |
| mandel | nessita: I'm there already :) | 17:02 |
| mandel | the internet gods are against us today... | 17:02 |
| mandel | @ping | 17:03 |
| ubot4 | pong | 17:03 |
| ralsina | nessita mandel Chipaca alecu: want me to call? | 17:04 |
| mandel | alecu: can you speak louder? | 17:07 |
| mandel | nessita, alecu: will be pushing the new version in 10/15 min or so | 17:22 |
| mandel | nessita, alecu: I'm going to run the test on linux, if everything is ok I'll let you know | 17:35 |
| nessita | mandel: thanks | 17:36 |
| === beuno-lunch is now known as beuno | ||
| ralsina | ok, off to the doctor. I'll check the backlog every now&then, may be back later depending on how it goes | 17:42 |
| === m_conley is now known as m_conley_away | ||
| === m_conley_away is now known as m_conley | ||
| nessita | ralsina: good luck! | 17:45 |
| nessita | alecu, mandel: when you're available, could you please (also) review https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/go-to-web-button/+merge/68573 ? | 17:46 |
| alecu | nessita, ack | 17:47 |
| alecu | damn. | 17:47 |
| alecu | nessita, I forgot to bring up the splitting issue during the meeting. | 17:47 |
| nessita | alecu: right!!! | 17:48 |
| nessita | alecu: I mentioned something to dobey a while ago, we agree he will ask review from us for every branch, and we should test that in windows... | 17:48 |
| alecu | nessita, great. | 17:48 |
| mandel | alecu, nessita: I have a question about the split, so the nautlius extension is going to a diff project? | 17:49 |
| nessita | mandel: YES :-D | 17:49 |
| mandel | nessita: hurray!! | 17:50 |
| mandel | nessita: does that mean we get a setup.py and all the python things, right? | 17:50 |
| mandel | so my evil question is, how will clientdefs.py be generated? 'cause that has the client version number etc… right? | 17:51 |
| dobey | don't worry about it | 17:51 |
| dobey | ;) | 17:51 |
| mandel | dobey: ok :) | 17:51 |
| mandel | nessita, alecu: branch is ready for review, I'm going to make it an EOD but if you send me an email with comments I'll read it and will fix the branch asap, if you are still here I'll ping you, is that ok? | 17:55 |
| nessita | mandel: sure | 17:56 |
| mandel | ok, laters everyone! | 17:56 |
| nessita | alecu: ping | 19:58 |
| alecu | nessita, pong | 20:02 |
| alecu | nessita, approved go-to-web-button branch | 20:52 |
| nessita | alecu: yey! | 20:52 |
| nessita | alecu: you ok if I land this with only one review? | 20:53 |
| alecu | I'm ok, looks harmless. | 20:53 |
| * nessita agrees | 20:53 | |
| === m_conley is now known as m_conley_away | ||
| dobey | have a good evening all | 21:14 |
| nessita | ok, I'm off crowd | 21:28 |
| nessita | see ya all tomorrow! | 21:28 |
| tntc | joshuahoover: hey, I'd just like to say thanks again! | 22:01 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
