| daker | nigelb, mhall119, cjohnston http://www.micahcarrick.com/gedit-as-a-django-ide-for-linux.html | 11:09 |
|---|---|---|
| YoBoY | thanks daker :D | 11:21 |
| daker | yw ツ | 11:21 |
| === ryanakca_ is now known as ryanakca | ||
| mhall119 | daker: cool, thanks | 11:56 |
| nigelb | james_w: HA, code deletion bug. My favorite type :) | 15:23 |
| james_w | :-) | 15:23 |
| james_w | someone added code to summit to delete meetings when the blueprints are deleted? | 17:07 |
| james_w | hmm, no, that's always been there | 17:08 |
| james_w | why is it breaking now? | 17:08 |
| nigelb | Because summit is awesome. | 17:08 |
| nigelb | :) | 17:08 |
| james_w | actually looks like it may be something to do with caching | 17:10 |
| mhall119 | james_w: corrupt launchpadlib cache? | 17:10 |
| james_w | seems that requesting +temp-meeting-export doesn't always give you the same thing | 17:10 |
| james_w | so it's creating meetings one run | 17:10 |
| mhall119 | james_w: were you ever able to get my new unit tests failing properly? | 17:10 |
| james_w | then deleting them the next run | 17:10 |
| james_w | mhall119, haven't tried yet, sorry | 17:10 |
| nigelb | Oh god. | 17:10 |
| nigelb | Why launchpad why. | 17:10 |
| mhall119 | james_w: was the delete cron being run regularly? | 17:10 |
| nigelb | Is that launchpad caching? Maybe we're hitting multiple app servers | 17:11 |
| mhall119 | could be, but if that's the issues then there's a bug in LP | 17:12 |
| nigelb | Well, to be fair on LP +temp-meeting-export is fairly heavy on the DB. | 17:12 |
| mhall119 | james_w: have you tested http://ec2-50-16-76-22.compute-1.amazonaws.com/ | 17:12 |
| mhall119 | nigelb: yeah, but it should be consistent | 17:13 |
| nigelb | agreed. | 17:13 |
| james_w | nigelb, mhall119: https://bugs.launchpad.net/summit/+bug/849331 | 19:22 |
| james_w | that's the cause and fix for the cache issue | 19:22 |
| james_w | we're going to want a deploy to fix that quickly | 19:24 |
| james_w | so if I were to say I was going to do that tonight, how scared would you be? | 19:24 |
| nigelb | as long as its checked in, minimal :) | 19:24 |
| james_w | oh, of course :-) | 19:25 |
| james_w | just wondered if you were scared about having the code in trunk in production | 19:26 |
| james_w | I know I need to to fix the autoscheduler for private rooms first | 19:26 |
| james_w | private sessions I mean | 19:26 |
| nigelb | james_w: you can just merge to the stable branch | 19:26 |
| james_w | though the autoscheduler isn't yet running | 19:26 |
| james_w | oh, cool! | 19:26 |
| nigelb | which is probably more stable. | 19:26 |
| james_w | what a great idea! | 19:26 |
| james_w | that makes my life easier, thanks :-) | 19:27 |
| nigelb | heh | 19:27 |
| nigelb | mhall119, cjohnston - tarmac set for stable branch as well. Should run. | 19:36 |
| nigelb | (summit) | 19:36 |
| mhall119 | nigelb: great, but I'll be killing 1.x as soon as trunk is ready to be merged into production | 19:44 |
| nigelb | mhall119: Dont' | 19:45 |
| nigelb | I'm sure situations like these will pop up | 19:45 |
| mhall119 | james_w: explain that one to me? | 19:45 |
| mhall119 | wouldn't we get the same thing if we're hitting the cache, rather than different things? | 19:46 |
| mhall119 | james_w: yeah, check this into 1.x series | 19:46 |
| mhall119 | and it'll be easy to merge it into production and deploy | 19:46 |
| james_w | mhall119, there are two caches | 19:52 |
| james_w | and so it's 50% chance of getting each | 19:53 |
| james_w | yay, failing tests! | 20:00 |
| james_w | plus ViewDoesNotExist: Could not import summit.sponsor.views. Error was: No module named restfulclient.errors | 20:01 |
| james_w | which is likely a version mismatch | 20:01 |
| james_w | yep, that's fixed too | 20:04 |
| james_w | NoReverseMatch: Reverse for 'summit.schedule.decorators.inner' with arguments '('',)' and keyword arguments '{}' not found. | 20:05 |
| james_w | getting that on two tests | 20:05 |
| mhall119 | james_w: yeah, that's fixed in another MP | 20:10 |
| mhall119 | in trunk | 20:10 |
| james_w | cool | 20:10 |
| mhall119 | is the private scheduling one failing properly now? | 20:10 |
| * james_w pulls | 20:10 | |
| james_w | yeah, looks like it | 20:10 |
| mhall119 | cool | 20:10 |
| james_w | just getting down to a "clean" test run first | 20:10 |
| james_w | mhall119, https://code.launchpad.net/~mhall119/summit/fix-etherpad-edit-links/+merge/75061 that one? | 20:11 |
| mhall119 | yeah | 20:13 |
| james_w | thanks | 20:13 |
| james_w | FAIL: test_no_available_private_room (summit.schedule.tests.PrivateSchedulingTestCase) | 20:14 |
| james_w | \o/ | 20:14 |
| mhall119 | yay | 20:15 |
| mhall119 | now this one is tricky, because we want to let the admins schedule a private meeting in a public room if necessary, but we don't want the auto-scheduler doing it | 20:16 |
| james_w | I don't know | 20:18 |
| james_w | http://paste.ubuntu.com/688601/ | 20:18 |
| james_w | passes tests | 20:18 |
| james_w | there's only one callsite - the autoscheduler | 20:18 |
| mhall119 | that'll work | 20:19 |
| mhall119 | we can look at adding the ability to auto-schedule private meetings into private rooms another time | 20:19 |
| mhall119 | make an MP and I'll approve it | 20:20 |
| mhall119 | target this one to trunk though | 20:20 |
| james_w | yep | 20:21 |
| mars | mhall119, nigelb, I made a graph of the summit website model object structure, thought you may find it useful: http://people.canonical.com/~mars/summit-models.png | 20:44 |
| mhall119 | mars: did you use the django-extensions for that? | 20:45 |
| nigelb | mars: Neat! | 20:45 |
| mars | mhall119, yep. Pretty cool. I find pictures really helpful for exploring and discovery | 20:51 |
| mhall119 | mars: yeah | 20:52 |
| mars | mhall119, nigelb, if you like it, feel free to pass it on to the other devs | 20:52 |
| mhall119 | I did that graphing thing for a project at my last job.....it was bad | 20:52 |
| james_w | hey mars | 20:52 |
| mars | Hey james_w | 20:53 |
| mars | james_w, how's the weather up by Owen Sound? Fall colours out yet? | 20:53 |
| nigelb | mhall119: badly designed models? | 20:54 |
| mhall119 | nigelb: yeah, and lots of them | 20:54 |
| nigelb | heh | 20:54 |
| james_w | mars, not quite | 20:55 |
| mars | james_w, your team may find this useful as well: http://people.canonical.com/~mars/summit-models.png | 20:56 |
| james_w | yeah, thanks mars | 20:56 |
| james_w | very useful when you are getting to know a project | 20:57 |
| mars | yeah | 20:57 |
| mars | This explains why model_mommy was helpful for building things | 20:57 |
| mhall119 | james_w: were you going to make a change to the stable branch today? | 20:58 |
| james_w | mhall119, I want to | 20:58 |
| mhall119 | what's stopping you? | 20:58 |
| james_w | time | 20:58 |
| mars | heh | 20:58 |
| james_w | https://code.launchpad.net/~james-w/summit/dont-autoschedule-private/+merge/75256 | 20:58 |
| mhall119 | ok, I'll check it | 20:59 |
| james_w | is it lp:summit/stable ? | 20:59 |
| james_w | I can't load the code page to get the list unfortunately | 20:59 |
| mhall119 | lp:summit/1.x | 20:59 |
| james_w | oh, I know | 20:59 |
| mhall119 | oh wait, yes, it is stable now | 20:59 |
| mhall119 | forgot we changed the series name | 20:59 |
| james_w | cool, thanks | 20:59 |
| james_w | going to work on https://bugs.launchpad.net/summit/+bug/849331 next | 21:00 |
| james_w | I'd like to deploy https://code.launchpad.net/~chrisjohnston/summit/linaro-link-fix/+merge/74877 soon too, but that one isn't breaking anything | 21:01 |
| james_w | after that then I want to look at importing from multiple Launchpad sprints for a single summit | 21:02 |
| mhall119 | james_w: that was was targetting trunk wasn't it? | 21:02 |
| mhall119 | trunk has several menu changes | 21:02 |
| nigelb | Definitely need to remove the "summit" hilight. | 21:02 |
| nigelb | james_w just pinged me with everyline. | 21:02 |
| james_w | the link fix? yeah | 21:02 |
| james_w | heh | 21:02 |
| james_w | summit | 21:02 |
| james_w | summit | 21:02 |
| james_w | summit | 21:02 |
| james_w | I love summit! | 21:02 |
| nigelb | oh god | 21:02 |
| mhall119 | james_w: with your fix to the autoscheduler, I think trunk is ready to be merged into /production | 21:03 |
| james_w | ooh | 21:03 |
| james_w | exciting | 21:03 |
| mhall119 | yeah | 21:03 |
| mhall119 | james_w: I'll get you lp import fix landed in stable and pushed to production, then we'll work on getting trunk into production | 21:04 |
| james_w | ok | 21:04 |
| mhall119 | nigelb: what's taking tarmac so long? | 21:04 |
| nigelb | when did you approve the MP? | 21:05 |
| mhall119 | there's 3 of them | 21:05 |
| mhall119 | maybe they don't have commit messages | 21:05 |
| * nigelb looks | 21:05 | |
| nigelb | ah, right. | 21:05 |
| nigelb | james_w: heh, so for all the bugs cjohnston and mhall119 fixed, you buy them a drink per bug :P | 21:06 |
| nigelb | mhall119: Overall status is Needs review | 21:06 |
| james_w | deal | 21:06 |
| mhall119 | nigelb: I thought an approved review message was enough for tarmac | 21:06 |
| nigelb | No | 21:06 |
| mhall119 | ok | 21:06 |
| nigelb | neither it is for PQM. | 21:06 |
| nigelb | I'm fairly sure there's lots of inspiration from PQM for tarmac. | 21:07 |
| nigelb | so I'd expect similar behavior. | 21:07 |
| mhall119 | nigelb: do they still need commit messages too? | 21:07 |
| cjohnston | yes | 21:07 |
| mhall119 | well, look who decided to show up | 21:08 |
| cjohnston | hush | 21:08 |
| cjohnston | id love to have 5 minutes to work on something | 21:08 |
| nigelb | mhall119: he heard about the drink per bug deal :P | 21:08 |
| cjohnston | let alone take a breath | 21:08 |
| cjohnston | But.. I was filing RTs today from class.. so SHUT UP! | 21:08 |
| cjohnston | :-P | 21:08 |
| nigelb | mhall119: let me know when you've fixed all the MPs | 21:09 |
| mhall119 | cjohnston: speaking of, was that resolved? | 21:09 |
| nigelb | I'll run tarmac manually for ya | 21:09 |
| mhall119 | I pinged Ng in -sysadmin | 21:09 |
| cjohnston | mhall119: yes.. ty | 21:09 |
| mhall119 | nigelb: ready | 21:09 |
| mhall119 | cjohnston: cool | 21:09 |
| * nigelb runs | 21:09 | |
| cjohnston | mhall119: has the % thingie been fixed | 21:09 |
| cjohnston | mhall119: james_w nigelb I believe we still need to make the conflict resolver NOT resolve private meetings, but move public meetings according to private meeting schedule | 21:11 |
| cjohnston | mhall119: did we ever try to get the cranberry memory error RT moved up? | 21:11 |
| nigelb | Try - yes. Actually get it done - No. | 21:11 |
| james_w | cjohnston, the conflict resolver will not move a private meeting as far as I can see | 21:11 |
| nigelb | mhall119: Done. | 21:11 |
| cjohnston | james_w: what about move public meetings if there is a reqired participant scheduled in a private meeting/ | 21:12 |
| james_w | cjohnston, also, hi :-) | 21:12 |
| cjohnston | hey bud | 21:12 |
| james_w | cjohnston, I think it will do that, but I would need to look to be sure | 21:12 |
| cjohnston | wanna fix some status bugs while your at it? ;-) | 21:12 |
| nigelb | lol | 21:12 |
| cjohnston | nigelb: any chance you can do a quick fix to https://code.launchpad.net/~nigelbabu/summit/668528-last-refresh/+merge/63448 | 21:13 |
| cjohnston | jussi: who was it we needed to bug about getting the bug bot back? | 21:13 |
| nigelb | cjohnston: I bugged already. | 21:13 |
| nigelb | cjohnston: Not ATM. Working. | 21:14 |
| cjohnston | bugged about what | 21:14 |
| nigelb | the bot. | 21:14 |
| mhall119 | cjohnston: I tried on that RT, jcastro said he doesn't have authority, and jono didn't respond | 21:14 |
| cjohnston | ok | 21:14 |
| mhall119 | bbl, kids' karate | 21:29 |
| james_w | gah, can't register the merge now | 21:30 |
| james_w | well, the fix is in https://code.launchpad.net/~james-w/summit/cache-control/ | 21:42 |
| james_w | doesn't seem like I'm going to be able to request a merge against stable | 21:42 |
| james_w | so either someone will have to bypass tarmac for me | 21:42 |
| james_w | or we can merge to trunk and go with deploying the whole thing | 21:42 |
| nigelb | james_w: launchpad hates you. | 21:43 |
| nigelb | I just proposed a merge | 21:43 |
| james_w | yes | 21:43 |
| james_w | it's doing it to a non-default branch that breaks it I think | 21:43 |
| nigelb | I did it against stable | 21:43 |
| nigelb | https://code.launchpad.net/~james-w/summit/cache-control/+merge/75265 | 21:43 |
| james_w | well, ok | 21:44 |
| james_w | thanks :-) | 21:44 |
| nigelb | I think launchpad has inbuilt burn out prevention :P | 21:44 |
| james_w | what did you put in the box? | 21:44 |
| nigelb | Nothing :p | 21:44 |
| nigelb | description? | 21:44 |
| james_w | no, the target box | 21:44 |
| nigelb | I clicked stable | 21:44 |
| james_w | ok | 21:44 |
| james_w | I didn't have that as an option | 21:45 |
| nigelb | did you do it differently? AH. | 21:45 |
| james_w | thanks | 21:45 |
| james_w | well, we clearly don't want that print in there | 21:45 |
| * james_w fixes | 21:45 | |
| james_w | argh, I can't get a migration going for a new model | 22:12 |
| james_w | I just want the new table so I can test that it works at all! | 22:22 |
| james_w | and what do you know, complaining made me stumble on the answer | 22:24 |
| james_w | missing Meta | 22:24 |
| nigelb | that is so Meta :P | 22:25 |
| james_w | wahey, found another regression | 23:08 |
| nigelb | james_w: in cache? | 23:09 |
| nigelb | I just approved that one. | 23:10 |
| james_w | nope | 23:10 |
| james_w | in importing | 23:10 |
| nigelb | Ah. | 23:10 |
| james_w | it now crashes if there is a "." in the meeting name | 23:10 |
| nigelb | I thought I fixed that. | 23:10 |
| * nigelb clearly remembes fixing that. | 23:10 | |
| james_w | you mean that you broke it? :-) | 23:10 |
| james_w | it was a side effect of your fix | 23:11 |
| nigelb | Argh ;( | 23:11 |
| nigelb | :( | 23:11 |
| james_w | confused me for a few minutes as I couldn't see how I had caused that error | 23:11 |
| nigelb | let me guess, the reverse lookup doesn't work | 23:11 |
| james_w | easy to fix though, not a problem | 23:11 |
| nigelb | phew | 23:11 |
| james_w | nope, in this case it crashes immediately on importint | 23:11 |
| nigelb | sigh. | 23:11 |
| james_w | as you changed one place where it uses the name from launchpad, but not the other | 23:12 |
| nigelb | ahhh! | 23:12 |
| james_w | so it can't find the thing that it just put in the database | 23:12 |
| nigelb | Right! | 23:12 |
| nigelb | Needs test cases. | 23:12 |
| nigelb | we should be able to Mock Launchpad. | 23:12 |
| james_w | well, it just needs some xml | 23:13 |
| nigelb | Hrm. That's right. | 23:14 |
| james_w | yay, my change works | 23:20 |
| nigelb | \m/ | 23:21 |
| mhall119 | we need to figure out how to allow . in the url.py regex | 23:41 |
| james_w | why's that? | 23:48 |
| mhall119 | that's the reason we've been going around replacing . with _ or - everywhere, because it breaks the reverse url lookups | 23:50 |
| james_w | well, this is replacing it in the same place :-) | 23:51 |
| james_w | but if there are still remaining places then that would be good | 23:51 |
| james_w | though we should do it soon, otherwise it will cause dataloss | 23:51 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
