kklimonda | hmm.. if I change part of file is the whole going to be reuploaded or only the part of it? | 03:27 |
---|---|---|
rye | hi duanedesign | 08:56 |
rye | ok, time to write diagnostic script for .cache/ubuntuone/ file permissions | 09:02 |
rye | bug 491950 - added to ubuntuone-client-diagnose | 09:22 |
ubottu | Launchpad bug 491950 in ubuntuone-client "100% cpu usage calling calcuate_hash on same file over and over (dup-of: 407762)" [High,Triaged] https://launchpad.net/bugs/491950 | 09:22 |
ubottu | Launchpad bug 407762 in ubuntuone-client "ubuntuone-syncdaemon maxes out system resources when trying to hash a file with root permissions" [Medium,Triaged] https://launchpad.net/bugs/407762 | 09:22 |
=== Urda_ is now known as Urda | ||
sebi` | hi, is it possible to make certain files public? | 12:20 |
beuno | sebi`, it will soon. The feature is mostly done, we're figuring out some deployment details | 12:21 |
beuno | stay tuned! | 12:21 |
sebi` | oh, darn | 12:21 |
sebi` | but nice to hear it's going to be implemented \o/ | 12:22 |
sebi` | also, it might sound silly, but will there ever be a non-linux client, as in, for windows XP, or Mac for example? (please don't hurt me for asking that :P) | 12:23 |
beuno | sebi`, sure, there is an on-going effort to build a windows client | 12:24 |
beuno | there will be a sprint during pycon | 12:24 |
sebi` | neat-o | 12:24 |
sebi` | :D | 12:24 |
beuno | sebi`, we're anticipating all your needs | 12:25 |
beuno | what else you got? :) | 12:25 |
sebi` | nothing so far :D | 12:25 |
sebi` | but thank you for your time | 12:25 |
beuno | you're welcome | 12:26 |
rye | ok, I guess we'll need a script that parses the syncdaemon.log of current version to find what state syncdaemon is, since "About a month ago Ubuntu One just stopped working." requires syncdaemon.log to find possible reason. | 12:41 |
beuno | rye, do it! | 12:42 |
rye | beuno, thinking on how to do that atm... | 12:43 |
rye | :) | 12:43 |
beuno | rye, verterok is always a good person to talk to when you want to do funky things, he's an ex-gentoo user ;) | 12:43 |
rye | i just hate repeating "Please upload syncdaemon log", scan it, only to find that it is in STANDOFF_WAITING_FOR_THE_END_OF_THE_WORLD | 12:43 |
* verterok looks | 12:47 | |
rye | verterok, erm, no, it is just an ongoing conversation, nothing to look on so far | 12:48 |
verterok | rye: there are other ways to know the state of syncdaemon :) | 12:50 |
verterok | rye: e.g: dbus-send --session --print-reply --dest=com.ubuntuone.SyncDaemon --type=method_call /status com.ubuntuone.SyncDaemon.Status.current_status | 12:50 |
verterok | rye: that should work in all ubuntu installs :) | 12:50 |
rye | verterok, oh, thanks! That is nice thing, I want to be able to parse log files so that there is clear sequence of events (file spotted, file uploaded, uuid is that one), state changed @ 1 AM, error occured, etc. :) | 12:54 |
rye | oh | 12:54 |
rye | i know i know i know | 12:54 |
rye | syncdaemon needs some method to enable debugging right _now_. | 12:55 |
verterok | rye: parsing the config is a good idea | 12:55 |
verterok | rye: a method? at runtime? | 12:55 |
rye | verterok, yup | 12:55 |
rye | verterok, i.e. i have a bug. syncdaemon does not want to upload files. we instruct the user to enable debug, add more files, then package this into a nice fine tarball and send to us, if this does not help, then restart syncdaemon. it may not be thaaat useful, though. | 12:56 |
verterok | rye: could you file a bug about this requirement? :) | 13:00 |
rye | verterok, which exactly? runtime debug or log parser? :) | 13:11 |
verterok | rye: runtime debug :) | 13:11 |
verterok | rye: regarding the log parser, maybe facundobatista alsready have some scripts :) | 13:12 |
rye | verterok, btw, end users might also want to see what's happening behind the scenes w/o enormous numbers of uuids that don't give much info about filename unless one scans through the logs with a magnifier (or grep) ... | 13:13 |
facundobatista | rye, what you said is logical: people want to see what's going on with the client | 13:14 |
facundobatista | rye, how many files already uploaded/downloaded, how many are still in the queue | 13:14 |
facundobatista | rye, at which speed everything is working, etc | 13:14 |
rye | facundobatista, frankly speaking I'd like to know what's actually happening too :) | 13:14 |
rye | and what Blocked(filename) means... it all is in the sources... thought | 13:15 |
rye | though | 13:15 |
facundobatista | rye, however, referring the user to the logs it's a poor solution: we should provide a GUI for that | 13:15 |
rye | facundobatista, yep, to show the current, aaand a log reader to show the past :) | 13:15 |
facundobatista | rye, mmm | 13:17 |
rye | btw, is anybody here running syncdaemon enabled on session login? | 13:17 |
facundobatista | rye, let's take for example a torrent client... how many times did you want to see the past? | 13:17 |
rye | it looks like syncdaemon eats my harddrive when it starts leading to an increased waiting time... | 13:17 |
facundobatista | rye, in any case... I don't have any scripts for the logs | 13:18 |
rye | facundobatista, ok, but if the torrent client removed the entries for already downloaded file then there should have been some kind of logs | 13:18 |
rye | erm | 13:18 |
facundobatista | rye, or it shouldn't remove the entries for already downloaded files ;) | 13:19 |
rye | torrents not apply here - they stay seeding forever so the only thing that is important is the speed charts, to show that to ISP and convince them that they really have a shaper in place :) | 13:19 |
rye | but files are uploading and we forget about them | 13:19 |
rye | facundobatista, in any way, runtime debug switch seems to be easier to implement | 13:19 |
facundobatista | rye, yeap | 13:20 |
beuno | facundobatista, rye, I hear quickly is great for building quick UIs | 13:20 |
facundobatista | rye, anyway... for GUI enhancements you should talk with Chipaca or beuno | 13:20 |
facundobatista | rye, I just say that logs shouldn't be the way to see what's going on for final users | 13:20 |
rye | facundobatista, beuno, verterok, ok so back to syncdaemon running LR on login - have you experienced any elevated hdd activity and gdm-login-to-usable-session time? | 13:23 |
verterok | rye: it's a known issue that LR takes a lot of time in some cases | 13:23 |
beuno | rye, I showed ubuntu devs my bootchart graph, and they all cursed at u1 | 13:23 |
rye | grrr | 13:24 |
rye | not good | 13:24 |
rye | my session login takes up to 1 minute | 13:24 |
rye | w/ syncdaemon eating ( "D" ) my drive. | 13:24 |
beuno | oh, my boot time is about 11 seconds from power on to desktop :) | 13:25 |
verterok | beuno, rye: that's the applet/nautilus starting syncdaemon earlier than they should | 13:25 |
rye | verterok, ok, what about trunk? | 13:26 |
verterok | rye: you can't start syncdaemon on login using trunk :) | 13:26 |
verterok | rye: there is no applet | 13:26 |
rye | verterok, I can :) | 13:26 |
rye | verterok, so what? | 13:26 |
verterok | rye: hehe, yes you can. | 13:26 |
rye | verterok, applet is no longer the launcher :) | 13:26 |
verterok | rye: who is launching it now? | 13:27 |
beuno | verterok, rye, https://devpad.canonical.com/~beuno/beuno-laptop-lucid-20100212-2.png | 13:27 |
rye | verterok, u1sdtool can launch it but even when syncdaemon itself is launched it starts doing LR which greatly impacts the ability to do something before it calms down | 13:27 |
verterok | rye: yes | 13:28 |
* rye becomes sad because he has 75 seconds of bootcharting | 13:28 | |
verterok | rye: ATM, there is no way to avoid the LR at startup | 13:28 |
rye | verterok, but that can be read in chunks, much like js calls are chained so that browser does not get too sad about executing all that 12 billion lines at once | 13:29 |
rye | verterok, freeing the hd for some time to perform the task... | 13:30 |
verterok | rye: it's being done in chunks :/ | 13:32 |
verterok | rye: one of the options is to change the filesystem manager layer to make it thread safe, so we can do the LR in a different thread | 13:33 |
rye | verterok, i trapped on bug 409972 - could that improve anything? By reading one file instead of tons of small open()s and read()s ? | 13:34 |
ubottu | Launchpad bug 409972 in ubuntuone-client "Use cPickle hack to speed up syncdaemon start" [Medium,Triaged] https://launchpad.net/bugs/409972 | 13:34 |
verterok | rye: no, that's a different issue | 13:34 |
rye | verterok, LR is local rescan, not metadata loading.... | 13:34 |
rye | grrr | 13:35 |
rye | i got it | 13:35 |
verterok | rye: on startup, syncdaemon builds a data structure that holds all the sync states/transitions | 13:35 |
rye | local rescan is traversal of all the files and folder to say "hi, file!" | 13:35 |
verterok | rye: that bug is to pickle that data structure on the first startup, to speedup the next one | 13:35 |
verterok | rye: actually local rescan traverse all files *and* loads metadata | 13:35 |
verterok | rye: the issue with the startup isn't local rescan actually, it's the metadata loading of filesyetem manager | 13:36 |
verterok | rye: FilesystemManager builds an index on startup, and needs to read all the metadata to do it :( | 13:36 |
rye | verterok, With all the buzz around improving startup time we forget that there's still time-to-usable-desktop and that is heavily impacted by running anything disk-intensive right at startup... | 13:37 |
verterok | rye: that's the only block-until-finish bit on syncdaemon startup | 13:39 |
rye | verterok, one seems to be enough :) | 13:39 |
verterok | rye: indeed | 13:40 |
verterok | rye: and we are awere of this problem | 13:40 |
verterok | rye: I did a little experiment, and replaced the current metadata storage with sqlite | 13:40 |
verterok | rye: the result wasn't a big improvement :( | 13:40 |
rye | beuno, was that lucid install mentioned in bootchart a clean one or upgrade from Karmic ? | 13:44 |
beuno | rye, upgrade from karmic to lucid | 13:45 |
beuno | after a tweak | 13:45 |
facundobatista | rye, #476711 could improve LR timings | 13:45 |
beuno | I got it 10 seconds down | 13:45 |
beuno | http://launchpadlibrarian.net/39214382/beuno-laptop-lucid-20100213-2.png | 13:45 |
rye | beuno, any problems w/ ureadahead not generating pack files? | 13:45 |
beuno | rye, not atm, no | 13:45 |
beuno | also, I have an SSD HD | 13:45 |
rye | facundobatista, ah, stat() on every file found? | 13:46 |
rye | beuno, still, my upgrade went not that smooth, i don't have plymouth displaying nice things, ureadahead is broken and tmp is not cleaned on start (IT SHOULD BE TMPFS!) | 13:47 |
facundobatista | rye, yeap | 13:47 |
beuno | rye, wierd. It was flawless for me | 13:48 |
rye | beuno, thanks for SSD hint, is your /tmp in memory or on disk? | 13:49 |
beuno | rye, disk | 13:50 |
aquarius | hey mattgriffin | 13:55 |
beuno | rodrigo_, ping | 13:57 |
beuno | I'm working on the notes branch | 13:58 |
beuno | and I have a questionfor you when you get back | 13:58 |
beuno | rodrigo_, ok, so notes are saving now. Just need to make the flow nice and fix quirks: lp:~beuno/ubuntuone-servers/dont-autosave | 14:06 |
beuno | I'm interested in something you said about revisions and not sending them creating problems | 14:07 |
beuno | aquarius, maybe you can shed some light on it ^ | 14:16 |
rodrigo_ | beuno, cool! | 14:33 |
rodrigo_ | beuno, the problem is that when saving notes yto couchdb, you need to specify the _rev you are updating | 14:34 |
rodrigo_ | beuno, and the autosave code didn't take them into account | 14:35 |
rodrigo_ | beuno, so if there is no autosave, it should be ok, just sending the POST when the user pressed 'Save' should do it correctly | 14:35 |
beuno | rodrigo_, could you enlighten me on why the manual save won't cause this? | 14:38 |
rodrigo_ | beuno, because the autosave code doesn't know the ID of the document AFAIR | 14:39 |
rodrigo_ | the _rev, sorry | 14:39 |
beuno | rodrigo_, so this is what I did to make it work: https://pastebin.canonical.com/27865/ | 14:40 |
rodrigo_ | beuno, not sure if you need the format=json part? but if it works, it looks good to me :-) | 14:41 |
rodrigo_ | beuno, have you submitted the branch for merging? | 14:41 |
beuno | rodrigo_, not yet, it needs a lot of clean up | 14:41 |
beuno | rodrigo_, the json bit | 14:41 |
beuno | is because otherwise the response is an HTML | 14:41 |
rodrigo_ | ah, ok | 14:41 |
beuno | and everything goes BOOM | 14:41 |
beuno | I need to let you go into edit mode, make the buttons pretty, and maybe a spinner while it saves :) | 14:42 |
beuno | how urgent is this? | 14:42 |
beuno | I can split it into 2 branches | 14:42 |
rodrigo_ | if you can, that would be great | 14:42 |
beuno | will do | 14:43 |
rodrigo_ | it's a bit urgent, because the web UI is broken | 14:43 |
rodrigo_ | but if you want to have all in 1 branch, that's ok with me | 14:43 |
beuno | rodrigo_, I'll see how time consuming it is | 14:43 |
beuno | start with the basics | 14:43 |
beuno | and cut off when I get to a reasonable amount of time spent :) | 14:43 |
rodrigo_ | ok :) | 14:43 |
rodrigo_ | thanks! | 14:43 |
beuno | de nada! | 14:44 |
beuno | good thing its a holiday in the US today | 14:44 |
beuno | gave me an extra day before going knee-deep into phone sync | 14:44 |
rodrigo_ | hehe | 14:44 |
beuno | rodrigo_, is this the traceback we're trying to fix with this? https://pastebin.canonical.com/27866/ | 14:53 |
rodrigo_ | beuno, no, but if you get that, I guess we should fix it | 14:54 |
beuno | rodrigo_, I get that with the new implementation :) | 14:54 |
rodrigo_ | beuno, I can fix it if you want, it needs to 'if "Tomboy" in annotations | 14:54 |
rodrigo_ | beuno, right, I changed the code to use dc's get_application_annotations, which doesn't have the has_key | 14:55 |
beuno | rodrigo_, if you can throw a quick patch at me, it would be great | 14:55 |
rodrigo_ | beuno, but since that part wasn't working, I didn't change it it seems | 14:55 |
beuno | or change in your branch and I'll merge | 14:55 |
rodrigo_ | beuno, ok, let me see | 14:55 |
dobey | http://www.youtube.com/results?search_query=ubuntuone&search_type=&aq=f | 14:58 |
rodrigo_ | beuno, ok, patched (but not tested) in my branch, so merge it | 14:58 |
dobey | interesting | 14:58 |
beuno | rodrigo_, merging, will loet you know in a sec | 14:58 |
rodrigo_ | beuno, ok | 14:59 |
beuno | rodrigo_, new error! https://pastebin.canonical.com/27868/ | 15:00 |
rodrigo_ | hmm | 15:01 |
beuno | rodrigo_, that's missing stuff | 15:02 |
beuno | one sec | 15:02 |
rodrigo_ | ah | 15:03 |
beuno | rodrigo_, https://pastebin.canonical.com/27869/ | 15:03 |
rodrigo_ | hmm, I'm lost, that should work :-( | 15:05 |
rodrigo_ | beuno, see line 426, that works, so not sure what the difference is?? | 15:06 |
beuno | hrm | 15:07 |
beuno | rodrigo_, that does: if existing_doc is not None and "_rev" in existing_doc._data: | 15:09 |
beuno | I wonder if that check makes the diff? | 15:10 |
rodrigo_ | that just adds the _rev field if it's an update | 15:11 |
rodrigo_ | but the error says a DC.Record is not serializable | 15:11 |
beuno | right | 15:11 |
beuno | I get that error when editing and saving | 15:11 |
rodrigo_ | aquarius or thisfred should know better what the error means, since it's in DC code | 15:11 |
beuno | so it probably should add the rev as well, no? | 15:11 |
beuno | (even if its unrelated) | 15:12 |
rodrigo_ | but the 2 calls to .put_record use a python list | 15:12 |
rodrigo_ | beuno, if it's a new document, no, you can't make up a revision | 15:12 |
rodrigo_ | couchdb will assign it automatically | 15:12 |
beuno | aha | 15:12 |
beuno | ok, good to know | 15:12 |
=== jamlata-afk is now known as jamalta | ||
rodrigo_ | so, to put_record you pass a Record(list values) and internally DC converts that to a Record, and so it shouldn't be serializing it if it can't be serialized | 15:17 |
rodrigo_ | not sure what's going on, sorry :-( | 15:17 |
rodrigo_ | the 2 calls to put_record seem to use the same format | 15:17 |
beuno | I can't figure it out either :/ | 15:17 |
rodrigo_ | aquarius, ^^ | 15:18 |
* aquarius reads the backscroll | 15:18 | |
beuno | aquarius, the meat of it is: https://pastebin.canonical.com/27869/ | 15:18 |
aquarius | wtf? | 15:18 |
beuno | on saving an edited note | 15:18 |
aquarius | how have you created a nonserialisable dictionary? :) | 15:18 |
rodrigo_ | aquarius, https://pastebin.canonical.com/27869/ | 15:19 |
beuno | aquarius, I'm special that way :) | 15:19 |
rodrigo_ | aquarius, see lib/u1/web/notes/views.py#161 | 15:19 |
aquarius | beuno, can you pastebin the value of doc just before line 161 in web/notes/views.py? | 15:19 |
beuno | sure | 15:20 |
rye_ | beuno, you stuffed mergeable list that you've received originally into the record? | 15:20 |
beuno | aquarius, where would print statements be outputted to? | 15:21 |
beuno | rye_, I haven't changed anything in python | 15:22 |
aquarius | beuno, no idea. I always do raise Exception(str(doc)) :-) | 15:22 |
rye_ | beuno, aquarius - the same at client side: http://paste.ubuntu.com/376910/ | 15:22 |
rye_ | so it IS possible, i have filed a bug, searching... | 15:22 |
aquarius | rye_, hang on, what's that paste? | 15:23 |
aquarius | rye_, ah, that replicates the problem. cheers. | 15:23 |
beuno | aquarius, nice trick: https://pastebin.canonical.com/27871/ | 15:23 |
rye_ | aquarius, bug 510223, though you are already using the server code... | 15:24 |
ubottu | Launchpad bug 510223 in desktopcouch "The new version of update_fields() should be backported from the server code" [Medium,Fix committed] https://launchpad.net/bugs/510223 | 15:24 |
aquarius | weird | 15:24 |
aquarius | thisfred? | 15:24 |
aquarius | CardinalFang? | 15:24 |
aquarius | no thisfred | 15:24 |
aquarius | CardinalFang, ping? | 15:24 |
aquarius | I don't know much about update_fields | 15:24 |
rodrigo_ | aquarius, hmm, so the problem is the doc["application_annotations"] = record.application_annotations? | 15:25 |
rodrigo_ | aquarius, I guess that needs to be converted to a list? | 15:25 |
rye_ | thisfred has told me to do... wait, searching... | 15:25 |
aquarius | something odd about that. It might not be possible to ship a_a around that way | 15:26 |
CardinalFang | aquarius, hi. USians have holiday today. I've seen that bug, and I don't understand it yet. I want to compare the code in py couchdb mod. | 15:26 |
rye_ | old_record.application_annotations["Tomboy"] = application_annotations | 15:26 |
rye_ | record_id = CouchDbNote.notes_db.put_record(old_record) | 15:26 |
rye_ | so no update_fields but my application_annotations is just a regular python dict | 15:27 |
aquarius | CardinalFang, oh, sorry, pal, didn't realise it was a holiday! | 15:27 |
* beuno fixes js while the experts figure this out | 15:29 | |
rye_ | Jan 20 20:11:54 <thisfred> so it needs to be a dictionary :( | 15:30 |
rye_ | Jan 20 20:12:02 <thisfred> you can do this: | 15:30 |
rye_ | Jan 20 20:12:05 <rtgz> sorry, http://paste.ubuntu.com/359656/ | 15:30 |
rye_ | Jan 20 20:12:20 <thisfred> db.update_fields(record_id, { | 15:30 |
rye_ | Jan 20 20:12:20 <thisfred> "application_annotations": application_annotations._data }) | 15:30 |
rye_ | Jan 20 20:12:40 <thisfred> but boy that method stinks | 15:30 |
beuno | hrm | 15:30 |
rodrigo_ | I guess it's better then to do doc["applications_annotations"] = record.application_annotations._data ? | 15:31 |
rodrigo_ | and then just the put_record(Record(data=doc)) ? | 15:31 |
aquarius | db.update_fields(record_id, { "application_annotations": application_annotations._data }) should work | 15:32 |
aquarius | having just tested it | 15:32 |
rodrigo_ | aquarius, but we'll have to list all the fields | 15:32 |
aquarius | mark it with a big UGH THIS IS A WORKAROUND nastiness :) | 15:32 |
rodrigo_ | and wouldn't doc["applications_annotations"] = record.application_annotations._data work also? | 15:33 |
* beuno is prepared to merge | 15:33 | |
rodrigo_ | go beuno go! :D | 15:33 |
beuno | merging! | 15:35 |
beuno | man | 15:35 |
beuno | working on U1 is *so* much more fun than launchpad... | 15:35 |
beuno | you can actually get something done in an hour! | 15:35 |
beuno | rodrigo_, nothing to merge! | 15:35 |
rodrigo_ | beuno, ah, you meant merging from my branch? | 15:36 |
rodrigo_ | let me change that then | 15:36 |
beuno | rodrigo_, anywhere! | 15:36 |
beuno | as long as it has the fix :) | 15:36 |
rye_ | is it possible for syncdaemon to be running as root? | 15:37 |
rodrigo_ | beuno, ok, merge now | 15:38 |
rye_ | i see the config entry im_ok_with_being_root_pretty_please_let_me_be_root.default = False and i think whether it does anything... | 15:38 |
beuno | rodrigo_, works | 15:39 |
beuno | ! | 15:39 |
rodrigo_ | beuno, cool! | 15:39 |
beuno | thank you rodrigo_, aquarius, rye_ | 15:39 |
beuno | on with it | 15:39 |
=== rye_ is now known as rye | ||
rodrigo_ | beuno, let me know when you propose the branch, and I'll review it | 15:40 |
beuno | rodrigo_, will do, I need to fix a few things on the javascript side first so people don't hav to use firebug to edit notes :) | 15:42 |
rye | re "UGH THIS IS A WORKAROUND". I remember having seen "This should not be done this way, it needs to be rewritten ASAP!" in a file that had this since 2004. Nobody knew what was actually wrong there so it was left as is :) | 15:42 |
rodrigo_ | rye, yeah, comments you never revisit don't work :-) | 15:42 |
beuno | ok, I'm off to pay the rent. I'll be back in 30-40' | 15:43 |
aquarius | rye, if you put "FIXME" then pylint bitches at you about it, which is why I do it :) | 15:43 |
=== beuno is now known as beuno-lunch | ||
kjoller | Hi all. I posted quite a lot of messages on a bug this weekend. I would like some feedback/advice on it. | 15:44 |
kjoller | (https://bugs.launchpad.net/bugs/497143) | 15:44 |
ubottu | Ubuntu bug 497143 in desktopcouch "CouchDatabase.execute_view should take an optional extra parameters dictionary" [Low,In progress] | 15:44 |
kjoller | It is basically a minor correction to a solution by aquarius | 15:45 |
beuno-lunch | rodrigo_, I will finish this today, I need to be free tomorrow for phone sync ;) | 15:45 |
rodrigo_ | beuno-lunch, ok, if you finish the js, I can continue on the othjer issues, if you haven't done them | 15:51 |
dobey | aquarius: we don't use pylint on the client, and pyflakes doesn't complain | 15:57 |
dobey | why oh why rhythmbox, did you stick some of my quite old mp3s in "recently added"? | 16:02 |
=== beuno-lunch is now known as beuno | ||
beuno | rodrigo_, sure, I think it's just js now | 16:53 |
beuno | will let you know as I progress | 16:53 |
rodrigo_ | ok | 16:54 |
beuno | rodrigo_, bug 494322 and 501020 feel like dupes | 17:28 |
ubottu | Launchpad bug 494322 in ubuntuone-servers "editing notes in ubuntu one web interface doesn't have a "Save" button" [Low,In progress] https://launchpad.net/bugs/494322 | 17:28 |
ubottu | Launchpad bug 501020 in ubuntuone-servers "Tomboy note sync and web-ui for notes are broken" [High,Confirmed] https://launchpad.net/bugs/501020 | 17:28 |
beuno | what's your take? | 17:28 |
rodrigo_ | beuno, they are indeed | 17:29 |
beuno | I will do the honors | 17:29 |
* rye is slow today, will shutdown now to prevent damage to the source code | 18:08 | |
beuno | rodrigo_, getting closer | 18:14 |
beuno | need to figure out why the save call returns HTML | 18:14 |
beuno | and I think we have a working notes web UI which saves | 18:14 |
beuno | rodrigo_, got a minute? | 18:50 |
beuno | I've traced the remaining issue down to a URL not returning json when it should | 18:51 |
beuno | aquarius, maybe you can help as well :) | 18:52 |
=== teknico is now known as teknico_away | ||
beuno | rodrigo_, branch proposed! | 19:24 |
dobey | http://www.facebook.com/album.php?aid=157659&id=122313196726&saved#!/photo.php?pid=4039110&id=122313196726 | 20:29 |
dobey | whoot. | 20:29 |
beuno | dobey, look at that! one step closer from uninstalling Firefox! | 20:31 |
dobey | too bad I'm not working on a facebook app | 20:32 |
beuno | one step at a time | 20:32 |
dobey | Well, with FB, I don't also have to build the API. It's already mostly all there. :) | 20:33 |
rodrigo_ | beuno, cool, reviewing it now | 20:51 |
beuno | rodrigo_, thanks. I'm unsure about a specific part of it, I mentioned it in the MP | 20:51 |
beuno | rodrigo_, the branch went deeper than I expected, I can tell you that | 20:53 |
rodrigo_ | :) | 20:54 |
beuno | it also made me realize I need to spend 2 or 3 days cleaning up CSS, javascript files and templates | 20:55 |
beuno | they're all clustered into sections | 20:55 |
beuno | the app is still small enough to fix it in a reasonable time frame | 20:55 |
beuno | in 6 months, no way | 20:55 |
* beuno eyeballs jblount to pair with him in the neat future | 20:56 | |
rodrigo_ | beuno, approved! | 21:08 |
beuno | rodrigo_, woooo! thanks, on its way to PQM | 21:09 |
rodrigo_ | beuno, ping some losa when it gets merged, so that it gets deployed asap | 21:10 |
beuno | rodrigo_, this will need to get cherrypicked into production? | 21:10 |
rodrigo_ | beuno, not sure what the process is, it depends what other revisions there are before | 21:11 |
rodrigo_ | beuno, so better ask a losa :-) | 21:11 |
beuno | rodrigo_, will do | 21:11 |
beuno | rodrigo_, what if we let it roll out to edge first, test it, then cherry pick? | 21:15 |
rodrigo_ | beuno, yeah, sounds good | 21:15 |
beuno | or is production broken enough that whatever will be better? | 21:15 |
beuno | super | 21:15 |
rodrigo_ | well, web ui editing of notes is completely broken | 21:15 |
rodrigo_ | but going through edge should be ok, it gets rolled every 4 hours there, I think | 21:16 |
beuno | yeah, I'll let all the moving parts settle, we should have it on edge tomorrow, and have statik and other around to help us with the cherrypick | 21:16 |
rodrigo_ | ok | 21:18 |
=== jamalta is now known as jamalta-afk | ||
=== jamalta-afk is now known as jamalta | ||
=== jamalta is now known as jamalta-afk |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!