| urbanape | heya, statik | 01:29 |
|---|---|---|
| statik | hey | 01:29 |
| urbanape | so, what's the word in the new world order about merging bindwood branches to trunk? | 01:29 |
| urbanape | now that it's under the auspices of ubuntuone-control-tower | 01:30 |
| statik | you just propose for merge to trunk and set the commit message, and then it gets marked approved after reviews | 01:30 |
| urbanape | k, did that | 01:31 |
| statik | we don't have tarmac in cron yet, so i have to kick it off manually | 01:31 |
| statik | i can do that now | 01:31 |
| urbanape | not sure about the commit message, though. | 01:31 |
| urbanape | does it need to be formatted specially? | 01:31 |
| statik | nope, nothing special | 01:31 |
| statik | just hopefully somewhat descriptive | 01:31 |
| statik | so if you set the commit message here https://code.edge.launchpad.net/~urbanape/bindwood/thorough-debugging/+merge/10928 i can run tarmac on it | 01:32 |
| statik | huh, the ancestry on that branch is odd | 01:33 |
| statik | it looks like it was branched from the packagebranch rather than from trunk | 01:33 |
| urbanape | hrm. | 01:33 |
| urbanape | also, should I be able to edit the commit message? I see no edit icon or link | 01:34 |
| statik | yeah, there should be a pencil icon next to the commit message, at least if you are logged in | 01:34 |
| statik | https://code.edge.launchpad.net/~urbanape/bindwood/thorough-debugging/+merge/10928/+edit-commit-message | 01:34 |
| urbanape | ah, weird. yeah, somehow I got logged out. | 01:35 |
| urbanape | k | 01:36 |
| urbanape | I recall branching after we had done some changes, but I thought it was still cut from trunk. | 01:37 |
| statik | i don't think it will hurt anything | 01:38 |
| statik | packaging branches are basically a long-lived branch from trunk | 01:38 |
| statik | that never gets merged to trunk | 01:38 |
| statik | but trunk is merged to it for every release | 01:38 |
| statik | in this branch, we're merging the packaging branch to trunk but you've deleted the packaging stuff from it | 01:39 |
| statik | i think bzr will be smart enough to handle it | 01:39 |
| statik | urbanape, alright, tarmac has merged it. thanks for the great branch! | 01:40 |
| urbanape | so, for my local copy of trunk, its source is... | 01:41 |
| urbanape | ~ubuntuone-control-tower/bindwood/ubuntu/ | 01:42 |
| urbanape | is that correct? | 01:42 |
| statik | yep, that should be right | 01:42 |
| statik | the shortcut for that is lp:bindwood | 01:42 |
| statik | it's entirely possible that i had already made the branch history funky, i'll run bzr qviz on it tomorrow | 01:42 |
| statik | urbanape, do you know if it's possible to put 8gb of ram in the older macbook pros, or only the new ones? | 01:44 |
| urbanape | how much older? | 01:45 |
| CardinalFang | statik, I'm pushing desktopcouch my changes to lp now. | 01:46 |
| urbanape | the first unibody MBP could take 8, I believe. | 01:46 |
| statik | ah, mine won't work then | 01:46 |
| statik | CardinalFang, awesome! | 01:46 |
| urbanape | the previous MBP I think maxed out at 6GB | 01:46 |
| CardinalFang | urbanape, statik, https://code.edge.launchpad.net/~cmiller/desktopcouch/get_port_not_stale | 01:46 |
| CardinalFang | I think it's close to right. My brain is mush now. | 01:47 |
| CardinalFang | Please eyeball it if you have any cycles left. | 01:47 |
| * CardinalFang sends message to aquarius to see it. | 01:47 | |
| statik | CardinalFang, sure. want to propose it for review? or you just want a pre-review review? | 01:48 |
| CardinalFang | statik, proposed for merge, so reviews are online. | 01:49 |
| CardinalFang | Thx! | 01:49 |
| CardinalFang | Good night, all. | 01:49 |
| statik | gnight | 01:49 |
| CardinalFang | Oh, and beware that teh test suite screws around with couchdb and leaves one running that isn't what you expect. the port-finding code freaks out at it half the time. :( Much time wasted there. :( | 01:50 |
| * CardinalFang collapses. | 01:50 | |
| === CardinalFang is now known as CardinalFangZzzz | ||
| urbanape | What's the best way to run the desktopcouch tests? | 02:28 |
| Chipaca | urbanape: first, you take a young rooster | 02:35 |
| Chipaca | urbanape: I'm not too sure how it goes from there, but you should probably not wear a suit | 02:36 |
| deserted | Chipaca: I think it involves vaseline and a pair of electric shears | 02:40 |
| Chipaca | and... that's probably why I blanked it out | 02:40 |
| urbanape | Hmm, all my Tyvek suits are at the cleaners. From the last time I ran hairy tests. | 02:42 |
| dobey | "trial desktopcouch" | 02:44 |
| urbanape | lots of waiting for couch to start... | 02:57 |
| urbanape | will noodle on it in the morning. | 03:03 |
| urbanape | night, folks. | 03:03 |
| === teknico is now known as teknico_away | ||
| === teknico_away is now known as teknico | ||
| === teknico is now known as teknico_away | ||
| === teknico_away is now known as teknico | ||
| thisfred | goedemorgen | 13:14 |
| === teknico is now known as teknico_away | ||
| === teknico_away is now known as teknico | ||
| jblount | Yo, yo, yo, yo | 14:07 |
| aquarius | yoyo. | 14:13 |
| jblount | aquarius: yo yo, yo? | 14:15 |
| aquarius | jblount, Yo. Yo-yo, ya? | 14:16 |
| jblount | aquarius: yo. :) | 14:19 |
| === teknico is now known as teknico_away | ||
| === teknico_away is now known as teknico | ||
| CardinalFangZzzz | aquarius, has thisfred convinced you that d-c log reading isn't as good? | 14:24 |
| aquarius | CardinalFangZzzz, not quite. I'd like to hear your reasoning too :) | 14:26 |
| CardinalFangZzzz | aquarius, the absolute best approach is to make couchdb emit its info on request. -b starts, -k kills, -I (!) prints JSON information about the running process, including ports and addresses used. TODO: implement -I | 14:26 |
| === CardinalFangZzzz is now known as chad | ||
| === chad is now known as CardinalFang | ||
| aquarius | CardinalFang, yes. The couch people have talked about that, too, but it hasn't happened yet, and it won't for karmic :( | 14:27 |
| thisfred | brb, have to reboot | 14:27 |
| === teknico is now known as teknico_away | ||
| === teknico_away is now known as teknico | ||
| CardinalFang | aquarius, We never know if the port is correct. The log file can be stale; it always exists, regardless of process' existence. We can add the additional check of trying to connect to a port, but we don't even know if we really connect to someone else's couchdb (unless we check auth too). | 14:33 |
| aquarius | but you have to wait to see if couch started up properly *anyway*, before doing anything to establish the port. | 14:34 |
| aquarius | since we need to do that regardless, if you've done that check, then you know the logfile won't be stale | 14:35 |
| === gafton_ is now known as gafton | ||
| === yofel_ is now known as yofel | ||
| CardinalFang | aquarius, I don't know if I need to start couchdb at all. Log file is there. Is it from last month? | 14:36 |
| aquarius | that's what find_pid is for. | 14:36 |
| aquarius | you need to check the pidfile, and see if it's stale, and if it is, then start couch. | 14:37 |
| === teknico is now known as teknico_away | ||
| thisfred | CardinalFang: that's where I messed up: I thought I kept the calling structure intact, and that I checked that everywhere find_port was called, find_pid was called first, but apparently I messed up | 14:38 |
| thisfred | Perhaps I should have let find_port continue to require a pid argument, even though it's not used | 14:39 |
| CardinalFang | thisfred, I added that. | 14:39 |
| === teknico_away is now known as teknico | ||
| thisfred | CardinalFang: aquarius and I discussed earlier that we (I) need to fix the tests, so that they use the find_port and find_pid again, while not starting up the real couchdb, by hacking into local_files in a different way. Have you tested your branch manually, because the current tests don't actually exercise the find_port and find_pid code at all... | 14:44 |
| thisfred | which is how I managed to slip up and create logfilegate | 14:45 |
| aquarius | CardinalFang, my suggested approach is to lie to xdg about where your base directory is. That's what the start_local_couchdb test does. and then everything shoudl work :) | 14:45 |
| thisfred | aquarius: yeah, that part I'm starting a branch for now. My concern is that we should get CardinalFang's branch landed ASAP if we know for sure it works, since the current code doesn't | 14:46 |
| aquarius | agreed | 14:46 |
| aquarius | but...having the code landed doesn't actually help, since what we need to do is get that code into a package and then into a PPA so people can start testing it | 14:47 |
| thisfred | But I do want to know it works. I'm not having much luck with importing find_port | 14:47 |
| aquarius | and I don't want to hassle kenvandine into building a package more than he has to, since he's busy :) | 14:47 |
| thisfred | true | 14:48 |
| thisfred | aquarius: ok, then I'm starting the branch to fix the tests first, since we need to get that landed at the same time, and then package | 14:49 |
| thisfred | aquarius: CardinalFang and we can use it to verify the correctness of any solution to the find_port issue you guys come up with | 14:50 |
| === teknico is now known as teknico_away | ||
| === teknico_away is now known as teknico | ||
| kenvandine | aquarius, thisfred, CardinalFang: if you have some proposed branches... i can merge them together into a ppa build for testing | 14:54 |
| thisfred | aquarius: open question: starting couch before, and stopping it after each test will be too slow, I think. Any good suggestions on a testcase wide 'teardown'? | 14:55 |
| CardinalFang | You didn't like my stochastic method. | 14:56 |
| * CardinalFang hugs the word 'stochastic'. | 14:56 | |
| thisfred | CardinalFang: no, randomness in tests, I dislike :) | 14:56 |
| thisfred | CardinalFang: I prefer having a single test that tests the stopping and restarting | 14:57 |
| thisfred | CardinalFang: then run all the other tests against a running couch | 14:57 |
| thisfred | CardinalFang: also, the stochastic method leaves the couch running at the end of the tests, so it doesn't solve this problem | 14:57 |
| aquarius | thisfred, teardown is just stopping couch. call desktopcouch-stop, that's what it's for | 14:59 |
| thisfred | is it madness to stop couch from the testcase class' __del__ ? | 14:59 |
| CardinalFang | thisfred, yes it is. | 14:59 |
| thisfred | aquarius: teardown is run after each test | 15:00 |
| CardinalFang | thisfred, There's a container for test cases. Can 'trail | 15:00 |
| jblount | MEETING BEGINS | 15:00 |
| CardinalFang | 'trial' use test suite code? | 15:00 |
| jblount | ZOMG! If you are a desktop+ person, this is your time. "me" get's your voice heard, hopefully your voice sounds like "DONE / TODO / BLOCKED" | 15:00 |
| CardinalFang | dagn. | 15:00 |
| CardinalFang | me | 15:00 |
| aquarius | thisfred, what? no it isn't, is it? setup runs at the start, teardown runs at the end | 15:00 |
| thisfred | aquarius: of each test | 15:00 |
| jblount | me | 15:00 |
| aquarius | me | 15:00 |
| teknico | me | 15:00 |
| rodrigo_ | me | 15:00 |
| thisfred | (continue after) | 15:01 |
| statik | me | 15:01 |
| urbanape | me | 15:02 |
| jblount | CardinalFang: It's go time | 15:03 |
| CardinalFang | DONE: desktopcouch replicate to u1. desktopcouch getPort() fixups. desktopcouch pairing fixups. reviewed aquarius' talk notes. | 15:03 |
| CardinalFang | TODO: get reviews. package d-c? | 15:03 |
| CardinalFang | BLOCKED: Nein. | 15:03 |
| CardinalFang | aquarius, if you please.... | 15:03 |
| aquarius | ⚀ DONE: bank holiday, reviewed chad's port branch; work with jan to fix oauth problem | 15:03 |
| aquarius | ⚁ TODO: continue piston oauth in snowy; finish DC talk for Ubuntu Developer Week; look at doing username URL at U1; discussions with thisfred/cardinalfang about DC port stuff | 15:03 |
| aquarius | ⚂ BLOCKED: nothing (woo!) | 15:03 |
| aquarius | doo-do-do-do, doo-doo, doo-doo, you can't touch jblount | 15:03 |
| jblount | DONE: Tiny branch to fix tiny bugs, resolved (and landed) weird issue with pqm not liking my overlays-skin branch, slapping my VMs into shape. | 15:04 |
| jblount | BLOCKED: Nope (although need statik to tell me if I should submit a RT about hooking up blog.ubuntuone.com or some other address setup DNS style) | 15:04 |
| jblount | TODO: Add tabs to website (again) with some re-thinking about nav, get wordpress.com blog into shape | 15:04 |
| jblount | teknico: rocknroll | 15:04 |
| teknico | DONE: worked on a Funambol deployment problem with pfibiger and mattgriffin, implemented the list_all template tag for the contacts CRUD web ui, improved the options handling of the create_couch_contacts.py script | 15:04 |
| teknico | TODO: implement the details view for the contacts CRUD web ui | 15:04 |
| teknico | BLOCKED: none | 15:04 |
| teknico | next: rodrigo_ | 15:04 |
| rodrigo_ | DONE: More tomboy syncing fixes. oAuth request signing in couchdb-glib. Attended some ubuntu developer week talks. Prepared couchdb-glib basic info for Wednesday's aquarius talk | 15:04 |
| rodrigo_ | • TODO: Start upstream discussion for adding social services accounts config to about-me. Talk to Ara about writing mago tests for evo-couchdb. Propose couchdb-glib/evo-couchdb for GNOME 2.29. Store UUIDs for postal addresses. Conflict resolver tool in pair tool. Look at becoming a MOTU (https://wiki.ubuntu.com/UbuntuDevelopers). | 15:04 |
| rodrigo_ | • BLOCKED: none | 15:04 |
| rodrigo_ | next aquarius | 15:04 |
| rodrigo_ | ugh next statik | 15:04 |
| rodrigo_ | or thisfred, sorry :D | 15:04 |
| aquarius | rodrigo_, fail :) | 15:05 |
| statik | DONE: New snapshot package of couchdb 0.10.x with a handful of bugfixes including cascading auth. | 15:05 |
| statik | TODO: desktopcouch bugfix release and packaging branches reorg | 15:05 |
| statik | BLCK: not-blocked-just-yet but anxiously awaiting versions of desktopcouch, bindwood and couchdb-glib which use oauth by default. And a version of couhcdb with SSL! | 15:05 |
| dobey | me | 15:06 |
| jblount | dobey: Your turn | 15:06 |
| dobey | ☭ DONE: Released poauth 0.1, Packaged poauth 0.1, Pushed poauth to REVU, Fixed some small issues in poauth, Fixed ubuntuone-client to use poauth, Fixed ubuntuone-storage-protocol to use poauth | 15:06 |
| dobey | ☭ TODO: Backport poauth to jaunty/hardy, Fix ubuntuone-storage-server to use poauth, Fix #409281 | 15:06 |
| dobey | ☭ BLCK: None. | 15:06 |
| urbanape | erm, okay | 15:06 |
| dobey | you lied | 15:06 |
| urbanape | DONE: Most of the way through the design for Bindwood to talk to Couch via OAuth. | 15:06 |
| urbanape | TODO: Finish that up and push and propose. Also, I'm Face and On-Call Reviewer today. | 15:06 |
| urbanape | BLOCK: Getting my desktopcouch branch to actually expect OAuth. Need a little help. | 15:06 |
| urbanape | standup.next() -> StopIterationError | 15:06 |
| aquarius | urbanape, I can help with that | 15:06 |
| thisfred | aquarius: Zope TestCases have a way of doing testcase wide initialization and cleanup, but I don't think basic python unit tests do. (which is more in keeping with unit testing as it's meant to be, but a pain in cases like this) | 15:06 |
| urbanape | aquarius, I'd like that. | 15:07 |
| jblount | urbanape: hugs, sorry I get confused keeping things in order. | 15:07 |
| urbanape | *sniff* it's okay. | 15:07 |
| jblount | MEETING ENDS (somewhat tentatively) | 15:07 |
| aquarius | oh, I missed: talk to someone who's interested in DC but uses fedora, so there might be some work to make that happen going on over the next week or so. :-) | 15:08 |
| === urbanape changed the topic of #ubuntuone to: Help contact: urbanape | File Sharing for Ubuntu | https://ubuntuone.com | https://launchpad.net/ubuntuone | Current Beta Client Revno is 191, Protocol Revno is 69 | Release 0.93 | ||
| rodrigo_ | aquarius: the opensuse build service builds stuff for fedora also, so we could try finishing my work there and provide packages for all those RPM distros | 15:09 |
| aquarius | rodrigo_, rawk! really? I didn't know t hat. | 15:09 |
| rodrigo_ | aquarius: yes, and for debian and ubuntu also | 15:09 |
| rodrigo_ | although for .deb it needs more work, and it doesn't make sense for us | 15:10 |
| aquarius | rodrigo_, certainly DC doesn't do anything Ubuntu-specific, so it shold work elsewhere, assuming you have a hysterically up-to-date couchdb ;-) | 15:10 |
| rodrigo_ | but for RPMs, we should be able to do one .spec with very few changes for both | 15:10 |
| rodrigo_ | aquarius: yeah, I was a bit blocked on my opensuse packaging work because of erlang and couchdb | 15:10 |
| aquarius | I'm looking forward to the results of someone running DC on fedora to confirm that it works, work out which dependencies are required, etc | 15:10 |
| rodrigo_ | but once that is done, it should be easy to package the client stuff | 15:11 |
| rodrigo_ | aquarius: so when you're going to work on it, let me know and we'll see how to do it there if you want | 15:11 |
| aquarius | rodrigo_, someone else is working on it, I hope, rather than me, but yeah :) | 15:13 |
| rodrigo_ | aquarius: ah, cool, we can just then publish the .spec in OS build service, and have the packages for both distros built there | 15:14 |
| aquarius | nice :) | 15:14 |
| thisfred | aquarius: so, if I poke the right XDG values into env, and force reload xdg.BaseDirectory (siC(amelCase)) it just works? And *that* doesn't mess with the rest of my system? Really? | 15:15 |
| aquarius | it doesn't mess with the rest of your system because the environment you've set goes away once the process ends. (doesn't it? it does, right?) | 15:15 |
| CardinalFang | aquarius, Rgr. | 15:16 |
| thisfred | I hope so, because they're not explicitly unset :) SO, probably we would have found out | 15:16 |
| aquarius | yep. So we're golden. :) | 15:16 |
| thisfred | aquarius: coolness | 15:16 |
| thisfred | that's simple enough, wish I'd seen that | 15:17 |
| CardinalFang | aquarius, this process and all its children, (unless exec* is packed with a explicit environment). | 15:17 |
| aquarius | CardinalFang, that's what I thought. (Sudden clenching fear that I might have been wrong ;)) | 15:17 |
| thisfred | only open problem: how/when to stop the test couchdb | 15:17 |
| aquarius | in shutdown. | 15:17 |
| CardinalFang | thisfred, How evil do you want to be? "import atexit" | 15:17 |
| aquarius | although I admit for that we need test-suite-wide setUp and tearDown. Someone must surely have solved this problem though! | 15:17 |
| thisfred | aquarius: indeed, zope testcase has, perhaps twisted as well | 15:18 |
| thisfred | CardinalFang: evil in tests I can live with | 15:18 |
| aquarius | if it doesn't, then have each setUp call test_suite_wide.setUp, which starts couch and then sets an envar. If the envar is set, don't start couch again | 15:18 |
| CardinalFang | i was just checking for suite setUp and tearDown, and didn't find it. :( | 15:18 |
| thisfred | aquarius: yeah, the starting is easy | 15:19 |
| aquarius | we need to be a little cleverer because you might want to run a subset of the test suite | 15:19 |
| thisfred | aquarius: the *stopping* however | 15:19 |
| CardinalFang | thisfred, "atexit" will let you stop at exit. | 15:19 |
| thisfred | CardinalFang: I'm fine with that | 15:19 |
| aquarius | ah, yeah, stopping is harder because an individual tearDown can't look into the future to know whether there are other tests coming. atexit is probably the way, although it's horrid :) | 15:19 |
| thisfred | I'll leave refining it to people with more developed sensibilities ;) | 15:19 |
| thisfred | being Dutch means never saying sorry, even if you have to :) | 15:20 |
| * aquarius applies for a dutch passport. | 15:20 | |
| aquarius | so...thisfred, you're redoing the tests? | 15:21 |
| thisfred | aquarius: yes, should not be much work, mostly throwing code away | 15:21 |
| thisfred | aquarius: you can never go on holiday again | 15:21 |
| aquarius | thisfred, cool | 15:21 |
| aquarius | thisfred, I am beginning to get that impression. I suck at organising things :( | 15:22 |
| thisfred | aquarius: nah, I should have dug 1 mm farther | 15:22 |
| CardinalFang | (hrmpf! does "couchdb -s" do nothing?) | 15:23 |
| aquarius | dpends on your definition of nothing | 15:24 |
| urbanape | aquarius, so, I'm modifying the script that we had been using that generates the CouchDB port (dbus.sh) to get our overall environment necessary. So that script will spit out the port and the OAuth bits we need to communicate with Couch. | 15:30 |
| aquarius | urbanape, cool. Can you talk to the keyring from the shell? | 15:31 |
| urbanape | I've got your dc-records-oauth branch checked out, but there's nothing that's seeding my keyring. | 15:31 |
| urbanape | yes, I can, but there's no match (obviously) | 15:31 |
| aquarius | ah, you want DC trunk, I think :) | 15:32 |
| urbanape | k, I can do that, too. | 15:32 |
| urbanape | also, I seem unable to kill my system-installed desktopcouch. should I remove the package altogether while this is under development? | 15:33 |
| aquarius | run desktopcouch-stop | 15:34 |
| aquarius | er, /usr/lib/desktopcouch/desktopcouch-stop | 15:34 |
| urbanape | yeah, did that. | 15:34 |
| aquarius | killall beam.smp is the nuclear option :) | 15:34 |
| urbanape | killed it hard and dead | 15:35 |
| CardinalFang | thisfred, aquarius, so what's the state of getPort fixing? Testing, and if pass, merge, and then release as 0.3.1? | 15:41 |
| thisfred | yes, that would have my +1, I'm working on the testing now | 15:42 |
| aquarius | CardinalFang, if we can get the tests in before releasing a 0.3.1 I'd lie that | 15:44 |
| thisfred | aquarius: fail | 15:45 |
| thisfred | aquarius: the xdg stuff looks nice in theory, but it doesn't quite work yet | 15:45 |
| aquarius | what doesn't work? | 15:45 |
| thisfred | everything, including the test_design docs, still talks to .cache | 15:45 |
| thisfred | it's probably an import order problem. | 15:46 |
| aquarius | might have to explicitly override xdg.cache_dir or whatever it is too | 15:46 |
| thisfred | I think local files is loaded before the tests start poking | 15:46 |
| thisfred | and so the dirs are never recomputed | 15:46 |
| thisfred | let's see what effect moving the imports around has | 15:48 |
| thisfred | aquarius: I don't understand: why do you use a mocker if the tests are supposed to talk to a fake database? | 16:03 |
| thisfred | I mean real database, but not the user's | 16:03 |
| dobey | look at the client tests. we override some env vars so stuff gets written to _trial_temp/foo instead of ~/.foo | 16:05 |
| thisfred | dobey: thanks, will do | 16:05 |
| aquarius | thisfred, I'm using the mocker for belt-and-braces security. We'll need to actually start a real couch for testing couch-startup | 16:05 |
| dobey | or rather, our problem was that it was trying to write to /.cache, because the env was empty for the tests, given the way we run them in ubuntuone-client | 16:06 |
| thisfred | aquarius: so the one test that does this poking to start up a new database doesn't actually talk to it, right? | 16:08 |
| aquarius | I knew we'd need it... | 16:08 |
| thisfred | aquarius: oh yeah, and I think it works almost, or sometimes, so I'm happy you did most of the work ;) | 16:09 |
| dobey | eh, i don't think "can we actaully start a db" is a particularly good test | 16:10 |
| thisfred | I just should learn to never ever say again in any context: "should not be much work" ;) | 16:10 |
| dobey | heh | 16:10 |
| thisfred | license revoked :) | 16:11 |
| aquarius | dobey, why? | 16:12 |
| thisfred | dobey: starting a database in itself is a good test, because we do it in a very complicated way. Also: that's not so much at issue here, since that test doesn't actually exist: we need to start a database for all the tests to talk to. | 16:12 |
| dobey | aquarius: because it's going to fail for anyone that doesn't have a specific version of couchdb, and might fail due to configuration differences | 16:13 |
| aquarius | dobey, that's the point. We want to know that the startup scripts properly cope with bad situations, and properly actually start up couch, so we know that we haven't broken them with a checkin. | 16:13 |
| dobey | i am quite happy that in poauth, i completely avoided using a BaseHTTPServer to test the code/protocol | 16:15 |
| thisfred | aquarius: we should totally use what the client does. Shame we can't depend on it, so I'm gonna steal its testcase runner | 16:17 |
| thisfred | dobey: yes of course, faking and mocking is great for unit tests, but we're talking functional/integration tests here, really | 16:18 |
| aquarius | dobey, this isn't testing the protocol (I mock CouchDatabase, and we can continue doing that). This is testing that the scripts that start up couchdb actually work | 16:19 |
| thisfred | aquarius, chad: stupid question: what actually is 'trial' and why do we use that rather than the five million other arbitrary ways to run tests? (I'm not saying we shouldn't, it works just fine, genuinely curious since I hadn't come across it before) | 16:21 |
| thisfred | is it twisted? | 16:22 |
| aquarius | trial is the twisted test runner | 16:22 |
| thisfred | right | 16:22 |
| thisfred | ubuntuone client seems to actually patch that into the regular test runner | 16:23 |
| thisfred | aquarius: I'm also asking since I think we may set the env vars at the start of the test run, and do cleanup at the end, but I need to find out where I can hook in | 16:24 |
| dobey | thisfred: ubuntuone-client has its own test runner which inherits from the twisted one | 16:25 |
| dobey | thisfred: trial has nicer output than just running the standard python unit test thing | 16:26 |
| thisfred | dobey: no, it does not inherit from it, it imports it and uses it though | 16:26 |
| dobey | thisfred: well that's what i mean by "inherits". we don't just run "trial" but imported the twisted test runner independently | 16:27 |
| dobey | and do some other stuff on top | 16:27 |
| dobey | like the env magic and start up a dbus session bus | 16:27 |
| CardinalFang | thisfred, not to be too much of a distraction, but where is the service I use to map OAUTH to usernames or database-names ? | 16:27 |
| thisfred | CardinalFang: this jdo would know :) | 16:27 |
| jdo | CardinalFang, it has not been deployed yet | 16:28 |
| thisfred | CardinalFang: I can also look up the branch in my mail | 16:28 |
| thisfred | ah | 16:28 |
| CardinalFang | jdo, okay. Out of curiosity, what does it yield? | 16:29 |
| jdo | CardinalFang, User info: ID, username, first/last name | 16:32 |
| jdo | 16:32 | |
| CardinalFang | aquarius, SteveA, thisfred: did we decide on a naming scheme for the desktopcouch database names on U1? "%(username)s/%(databasename)s" ? | 16:36 |
| aquarius | CardinalFang, I think it's a bit more complex than that, for easier HTTP sharding. thisfred knows though :) | 16:36 |
| thisfred | aquarius: the http sharding is an open issue. We're using u/xxx/xxx/[userid]/[databaseid] where the xxxs are the first six characters of the hash of the user id. this is to spread the dbs evenly over directories, but I don't know that it will scale out over multiple servers as is | 16:38 |
| thisfred | so anyone have any idea how I can hook into the start of a twisted trial run? are there plugins you can register or something? | 16:39 |
| dobey | thisfred: you probably want to do stuff in setUp() normally, but dbus seems to fail at that | 16:40 |
| thisfred | dobey: but there should be a way to hook into the test run, rather than each individual test, right? | 16:41 |
| dobey | so if you need to start a dbus session bus, then you basically need to do what ubuntuone-client does (which is have its own test script), and run that instead of trial | 16:41 |
| thisfred | yeah, I was afraid that was the answer | 16:41 |
| dobey | you might be able to do less work in a shell script, instead of python, but i'm not sure what all you need to do exactly | 16:42 |
| dobey | and i'm off to get some lunch :) | 16:42 |
| * aquarius ahahahahas at thisfred and CardinalFang over tcole's comment about the logfile on CardinalFang's branch :) | 16:43 | |
| aquarius | so, for reading port info from the logfile: me and tcole. For reading it from /proc: CardinalFang and thisfred. And tcole's got the coolest beard, so he wins. | 16:49 |
| tcole | the main thing is dealing appropriately with stale information | 16:50 |
| tcole | once we get the info from the log file, actually try talking to the process to verify | 16:50 |
| tcole | ironically, I shaved the beard off some time ago | 16:51 |
| thisfred | aquarius: and I'm working on mine, or at least have been too lazy to shave since I moved. Also: I'm totally +1 on logfile, but +10 on what we can get working now. Which in my case seems to be absolutely nothing, mostly. | 16:52 |
| aquarius | CardinalFang, thisfred: it would be interesting to get tcole's perspective on your technical issues with using the logfile | 16:54 |
| thisfred | aquarius: no technical issues, AFAIK: we just reverted to the working proc file, because the logfile code was broken | 16:55 |
| CardinalFang | aquarius, thisfred, I'm happy with using the logfile if you can get it working as well. | 17:08 |
| CardinalFang | Lunch! Back in 1h! | 17:08 |
| aquarius | thisfred, I need to get going -- have to meet up with some people this evening :( | 17:13 |
| thisfred | aquarius: ok, keep you posted by mail | 17:13 |
| aquarius | sorry to have to bail | 17:14 |
| thisfred | aquarius: no problem, I will sort it! I'm sure | 17:19 |
| === yofel_ is now known as yofel | ||
| urbanape | statik, you about? | 17:55 |
| urbanape | or anyone else hip to the new Couch DB + OAuth? | 18:12 |
| CardinalFang | j0! | 18:35 |
| urbanape | heya, CF | 18:36 |
| === teknico is now known as teknico_away | ||
| thisfred | CardinalFang: heads up: I think I'm near a solution for the tests, at least it's now completely broken, and not seemingly working but talking to the wrong db. I understand you will be working on the packaging, so I will tell you when I'm done, and hopefully get some other people to review it. | 18:49 |
| CardinalFang | thisfred, Roger. | 18:49 |
| thisfred | CardinalFang: bah: I think the problem may be: 1. tests poke in env settings to point to test couchdb 2. tests initiate code that talks to couchdb 3. code that talks to couchdb gets info from DBUS 4. DBUS finds couchdb not running and starts it, but at this point I'm not sure it looks at the test ENV anymore | 19:05 |
| thisfred | It's a Kind of Magic | 19:06 |
| CardinalFang | thisfred, well, the dbus call only works because bin/desktopcouch-service is running. | 19:06 |
| thisfred | CardinalFang: ah, right, I could start my own from the tests. That would break the user's normal operations, but at least we don't screw up their real data, though they could screw up our test data, or lose some writes to our temporary db | 19:07 |
| thisfred | Man, this is a fine ball of twine though. | 19:09 |
| thisfred | I want test driven development. At least for everyone else's code! :) | 19:09 |
| CardinalFang | thisfred, I say just kill your running desktopcouch-service and start it where you're testing. Don't get bogged down in making trial work directly. Just make a shell script, "run-tests" that does what you need. | 19:29 |
| kenvandine | thisfred, any eta on a dc release? | 19:30 |
| * gafton_ begs xchat to stop quitting | 19:30 | |
| thisfred | CardinalFang: yeah, I'll do that, I think. | 19:30 |
| thisfred | kenvandine: not sure no, but I'm not sure how much work we'll be bugging you for, as CardinalFang and statik have changed the release process for dc substantially, so probably CardinalFang can answer your question better than I | 19:32 |
| thisfred | kenvandine: if we're all waiting on my tests fix, then, eh, as soon as I get a clue... :( | 19:32 |
| kenvandine | i am mostly concerned with getting it in time to have an uploader still awake | 19:33 |
| thisfred | right | 19:33 |
| kenvandine | statik, CardinalFang: when you guys work on the packaging branch... make sure you merge in my changes in ~ubuntu-desktop | 19:33 |
| kenvandine | thisfred, yeah... it's pitti who is in germany | 19:33 |
| === rmcbride changed the topic of #ubuntuone to: Help contact: urbanape | File Sharing for Ubuntu | https://ubuntuone.com | https://launchpad.net/ubuntuone | Current Beta Client Revno is 194, Protocol Revno is 69 | Release 0.93 | ||
| kenvandine | getting late there now | 19:33 |
| thisfred | CardinalFang: the critical bugs are fixed yes? | 19:34 |
| thisfred | CardinalFang: perhaps we should let perfecting the tests not keep us from releasing then | 19:34 |
| CardinalFang | kenvandine, I think we've done that. Thanks! | 19:36 |
| kenvandine | CardinalFang, good... just make sure desktopcouch depends on python-desktopcouch-records :) | 19:36 |
| CardinalFang | kenvandine, It does. | 19:37 |
| kenvandine | good | 19:37 |
| kenvandine | thisfred, so are the tests at least passing? | 19:41 |
| thisfred | kenvandine: the tests always pass, and test the real code, the only part that is undertested is the stuff that starts couch. That is how the logfile reading bug slipped through. | 19:43 |
| kenvandine | ah | 19:43 |
| thisfred | CardinalFang: can I get an ack on the portnumber issue being done? | 19:44 |
| thisfred | ow, it's still waiting on review | 19:44 |
| thisfred | I wish stuart hadn't abstained. That resulted in everyone else avoiding it | 19:44 |
| thisfred | CardinalFang: I'll do a review, but can you tell me the best way to manually test that the code actually finds the port and if not starts couch? | 19:46 |
| CardinalFang | thisfred, confirming. | 19:48 |
| CardinalFang | ... | 19:48 |
| kenvandine | thisfred, fwiw... i did build a deb of your branch and ran it... seemed fine | 19:52 |
| thisfred | kenvandine: mine, or chad's? | 19:54 |
| kenvandine | oh | 19:54 |
| kenvandine | chad's | 19:54 |
| kenvandine | lp:~cmiller/desktopcouch/get_port_not_stale | 19:55 |
| thisfred | right, that's good news. I'll look through the code, and approve asap, and cajole someone else into doing the same | 19:55 |
| dobey | hrmm | 19:56 |
| CardinalFang | thisfred, Okay: Confirmed that https://bugs.edge.launchpad.net/desktopcouch/+bug/420911 is avoided with /proc searching. | 19:59 |
| ubottu | Launchpad bug 420911 in desktopcouch "Cannot getPort - desktop-couchdb.log doesn't exist" [Critical,New] | 19:59 |
| CardinalFang | Added comment about it. | 19:59 |
| CardinalFang | So, the other bug, hrm.... | 20:00 |
| CardinalFang | https://bugs.edge.launchpad.net/desktopcouch/+bug/422127 Added comment. | 20:02 |
| ubottu | Launchpad bug 422127 in desktopcouch "getPort succeeds when desktopcouch isn't running" [Critical,New] | 20:02 |
| CardinalFang | kenvandine, Are those ^ the two very big bugs we should fix now? | 20:02 |
| kenvandine | yes | 20:02 |
| CardinalFang | Rock. | 20:03 |
| thisfred | CardinalFang: branch is approved as you probably noticed, but if not, now you know | 20:15 |
| thisfred | ;) | 20:15 |
| thisfred | and let me know what else I can do to help | 20:15 |
| CardinalFang | thisfred, Okeh! Dank. | 20:16 |
| CardinalFang | statik, You around? | 20:27 |
| statik | hi CardinalFang, i am now. did my mail go through this time? | 20:32 |
| CardinalFang | statik, Yes. Now I see both. Weird. | 20:33 |
| CardinalFang | Thanks! | 20:33 |
| kenvandine | statik, i need to run out pretty soon... can you prepare the upload for the dc release? | 20:48 |
| kenvandine | and ping pitti if he is still around? | 20:48 |
| CardinalFang | statik, I can't find where to upload the orig.tar.gz . Should it be a link on https://edge.launchpad.net/ubuntu/karmic/+source/desktopcouch ? | 20:49 |
| kenvandine | statik, ok... pitti is still around and awaiting your ping :) | 20:54 |
| kenvandine | when you guys are ready | 20:54 |
| urbanape | I am a horrible Face. | 20:56 |
| kenvandine | statik, i am heading out for a bit.. call my cell if you have any urgent needs | 20:56 |
| CardinalFang | kenvandine, I'm uploading. | 20:57 |
| CardinalFang | Well, I am trying. | 20:57 |
| CardinalFang | kenvandine, statik, "A release requires a corresponding milestone that is not attached to another release." Wtf? | 21:11 |
| dobey | yes | 21:12 |
| dobey | CardinalFang: just make a new milestone called 0.4 or 0.3.1 or whatever you are numbering the release | 21:12 |
| CardinalFang | Okay. | 21:12 |
| thisfred | statik: re:ssl kocolosk reports back that it *does* seem to be a simple oversight, and that a workaround would be 5 minutes work (let's inflate that a lot to include landin testing and packaging) but he's talking to überstream to see what solution mochiweb would prefer | 21:28 |
| CardinalFang | rawk, thisfred. | 21:28 |
| thisfred | CardinalFang: indeed :) | 21:29 |
| statik | kenvandine, james_w had agreed this morning to sponsor, we are going to try doing a merge proposal to lp:ubuntu/desktopcouch | 21:31 |
| statik | will definitely ping pitti if we can't get james_w for some reason | 21:31 |
| statik | thisfred, that is great news | 21:32 |
| * dobey wonders if james_w got to look at poauth today | 21:32 | |
| statik | thisfred, as soon as SSL support is committed to the 0.10.x couchdb branch i will build a new package | 21:38 |
| thisfred | statik: I will immediately dispatch a pigeon when that happens | 21:39 |
| thisfred | forthwith! | 21:39 |
| statik | awesome | 21:40 |
| statik | i'm going to see about backporting erlang and xulrunner and couchdb to hardy for the data center now | 21:40 |
| statik | dobey, any exotic packages you uploaded to karmic that should go on our watchlist for bugs other than icontool? | 21:41 |
| dobey | i'm trying to get poauth uploaded into karmic | 21:43 |
| statik | ok, i've got that on my list | 21:43 |
| dobey | it's still on the fence | 21:43 |
| dobey | statik: i hope we can get it in though | 21:46 |
| statik | me too | 21:46 |
| CardinalFang | dc0.3.1release$ bzr bd -S | 21:49 |
| CardinalFang | dpkg-buildpackage: error: fakeroot debian/rules clean gave error exit status 2 | 21:50 |
| CardinalFang | debuild: fatal error at line 1334: | 21:50 |
| CardinalFang | dpkg-buildpackage -rfakeroot -d -us -uc -S failed | 21:50 |
| CardinalFang | bzr: ERROR: The build failed. | 21:50 |
| CardinalFang | statik, ^ Hrm. | 21:50 |
| statik | CardinalFang, you might need to specify which key to sign with | 21:50 |
| statik | does debuild -S -sa do any better? | 21:50 |
| statik | also, i have DEBSIGN_KEYID=NNNNN set in ~/.devscripts | 21:51 |
| CardinalFang | statik, same. Exactly the same, I thinkj. | 21:51 |
| statik | which overrides which key is chosen to sign with | 21:51 |
| statik | otherwise devscripts tries to find the key based on the email address in the changelog, which doesn't always work the way you want it to | 21:52 |
| statik | you can also specify an unsigned package, since you aren't going to upload it, you're just going to testbuild in pbuilder | 21:53 |
| === teknico_away is now known as teknico | ||
| CardinalFang | statik, it was missing build dep cdbs | 22:08 |
| statik | ah, that would do it | 22:08 |
| * statik hugs apt-get build-dep and sticks his tongue out at rpms | 22:08 | |
| CardinalFang | Okay, now I have source files. | 22:08 |
| statik | yay! the next part is really fun. apt-get install ubuntu-dev-scripts; ln -s /usr/bin/pbuilder-dist ~/bin/pbuilder-karmic;pbuilder-karmic create;pbuilder-karmic build desktopcouchfoo.dsc | 22:09 |
| statik | er, ubuntu-dev-tools not ubuntu-dev-scripts; sorry | 22:11 |
| CardinalFang | Rgr. Building.... | 22:13 |
| CardinalFang | statik, Okeh! I have .debs. | 22:31 |
| statik | CardinalFang, awesome! after a quick testinstall, you are probably ready to propose the spb for merging to lp:ubuntu/desktopcouch, and ping james_w to upload it | 22:32 |
| CardinalFang | statik, You say "testinstall". Is that command somehow, or are you being all German? | 22:33 |
| statik | I do cd ~/pbuilder/karmic_result; sudo dpkg --install desktopcouchfoo-i386.deb;scan the logs, try out /usr/bin/start_local_couchdb.py, etc | 22:34 |
| CardinalFang | statik, looks good. The only problem I see is that couchdb doesn't have authentication on and the file:/// URL we use has a username/password specified, which makes Firefox complain that it's not necessary and may be a phishing attack. | 22:43 |
| statik | CardinalFang, great! | 22:43 |
| CardinalFang | Okay. Uploading and proposing. | 22:44 |
| statik | cool. point me to the merge proposal when you register it, I would like to have a look, i've never seen one before | 22:44 |
| statik | by uploading you mean pushing to lp:~ubuntuone-control-tower/ubuntu/karmic/desktopcouch/spb ? | 22:44 |
| CardinalFang | Yep. | 22:45 |
| CardinalFang | statik, https://code.edge.launchpad.net/~ubuntuone-control-tower/ubuntu/karmic/desktopcouch/spb | 22:47 |
| statik | CardinalFang, it's interesting that the last commit didn't have the comments from the changelog | 22:47 |
| statik | like r53 did | 22:48 |
| CardinalFang | Eek. Hrm. | 22:48 |
| statik | CardinalFang, are the changes to debian/changelog committed there? swapping from UNRELEASED to karmic and adding entries for the two bugs that this upload will close? | 22:49 |
| statik | it's ok for those to be in extra commits | 22:49 |
| CardinalFang | Crap. I reverted that to merge from the bugfix. | 22:50 |
| CardinalFang | Forgot to mv back. | 22:50 |
| statik | no harm done i think | 22:50 |
| CardinalFang | I hope that's the only mistake I made. | 22:51 |
| CardinalFang | statik, reload. | 22:52 |
| * statik waits for launchpad to finish scanning the latest revision | 22:53 | |
| CardinalFang | there. | 22:53 |
| statik | CardinalFang, it looks good. I normally write 'New upstream bugfix-only release' just because i'm super nervous about giving the right impression, but I think this looks good | 22:55 |
| statik | i think it's ready to 'propose for merging' | 22:55 |
| statik | don't forget to set the reviewer to ubuntu-main-sponsors | 22:56 |
| dobey | hrmm | 22:57 |
| dobey | https://edge.launchpad.net/ubuntu/+source/poauth <- not a 404 | 22:57 |
| statik | python-poauth ? | 22:58 |
| dobey | would be the binary package, yes | 22:59 |
| dobey | source is poauth though | 22:59 |
| statik | i usually name my sourcepackages with the python- prefix too, for stuff that is pure python | 22:59 |
| statik | i don't know why though | 22:59 |
| dobey | source should match the tarball name usually | 22:59 |
| dobey | only time it shouldn't really, is if there are conflicts | 23:00 |
| statik | that seems reasonable | 23:00 |
| statik | man erlang takes forever to build | 23:00 |
| dobey | statik: can you go to https://edge.launchpad.net/ubuntu/+source/ubuntuone-client/+subscribe and subscribe "Ubuntu One Hackers"? :) | 23:00 |
| statik | absolutely. i thought i already had, but i must have missed it | 23:00 |
| dobey | thanks | 23:02 |
| dobey | i'm not an admine of ~ubuntuone-hackers, or i'd do it :) | 23:02 |
| CardinalFang | statik, I must go offline, 20 minutes ago. | 23:03 |
| CardinalFang | I can be back in 40min. | 23:04 |
| statik | CardinalFang, thanks for taking care of this, i'll keep an eye out for it too | 23:04 |
| thisfred | ok, I FINALLY got the tests doing something | 23:20 |
| thisfred | glass half full side: I learned a little about xdg | 23:20 |
| thisfred | All tests pass :) except for the mocker... :( | 23:23 |
| statik | that mocker | 23:31 |
| dobey | is it mocking you? | 23:32 |
| thisfred | terribly | 23:40 |
| thisfred | it just sits there | 23:41 |
| thisfred | looking at me | 23:41 |
| thisfred | anyhow: I mailed Stuart, he'll be my knight in shining armor and solve it tomorrow morning, these test fixes missed this alpha already anyway, and I'm glad I got the rest of it working | 23:42 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!