/srv/irclogs.ubuntu.com/2011/07/18/#ubuntu-website.txt

mhall119cjohnston: I got a different error00:18
mhall119today_view() got an unexpected keyword argument 'summit_name'00:18
mhall119did you not change the today_view function?00:18
cjohnstoneven when removing summit_name it didnt fix it, and i had no idea what else to do00:19
mhall119cjohnston: where did you get the error?00:24
cjohnstonwhen visiting uds-p/oday00:27
cjohnstonuds-p/today00:28
mhall119I don't have a uds-p00:30
mhall119I went to uds-o/today and didn't get the error00:30
mhall119could the error be caused by data?00:30
cjohnstoni wonder if its because there isnt any info other than just the uds-p and the dates00:32
cjohnstonno items00:32
cjohnstonso mhall119 I can propose that merge then?00:33
cjohnstonany idea why https://code.launchpad.net/~chrisjohnston/loco-directory/608289/+merge/68182 hasnt merged in?00:34
mhall119no merge without fixing today_view to expect a summit name00:39
mhall119cjohnston: either nigel's tarmac is down, or there's a merge conflict perhaps?00:40
cjohnstonmhall119: soI need to remove summit_name from today?00:41
mhall119no, the view function needs to know about it00:42
mhall119hang on, I'll have an MP to your branch00:42
mhall119cjohnston: https://code.launchpad.net/~mhall119/summit/today/+merge/6819900:46
mhall119see if you still get the error with that merged into your branch00:46
cjohnstonDone.. Thanks mhall11900:53
cjohnstonmhall119: you still gonna be able to do reviews tonight?00:53
mhall119what, you want more work out of me?00:53
cjohnstonYes00:54
cjohnstonIt would be nice to get some summit work done :-)00:54
mhall119don't we have a jam scheduled for that?01:00
mhall119man, you've been busy with summit, I thought we gave that to nigel01:01
cjohnstonlol01:01
mhall119at least they're mostly short01:02
cjohnstonthe bzr-apps one is cool01:02
mhall119except it didn't work in LD production01:03
cjohnston?01:03
cjohnstontrue01:03
mhall119something about bzr/ssh local env they were using01:03
cjohnstonforgot about that01:03
cjohnstonwell.. can you figure it out on summit since you have access to that?01:03
mhall119maybe01:04
mhall119I need to dig into the launchpad plugin for bzr to see what it does with lp: branch paths01:05
mhall119also, remind me in the morning to talk to IS about getting the new django-openid-auth available to LD and summit01:06
cjohnstonand the status of django01:07
cjohnstonyou should merge in the bzr apps anyway cause it will make it easier to set it up to start fresh01:10
mhall119yeah01:10
cjohnstonI think there are plenty of MPs waiting for you to merge in tongiht and then push live tonight01:11
cjohnstonnigel broke tarmac01:15
cjohnstonmhall119: https://code.launchpad.net/~chrisjohnston/loco-directory/need-trans/+merge/6820101:16
cjohnstonmhall119: im off for the night.. i expect that all MPs will be reviewed in the AM01:25
cjohnston?01:26
mhall119I expect a hot breakfast and fresh coffee when I wake up01:43
mhall119I guess we'll both be disappointed01:43
=== daker_ is now known as daker
dakermhall119, review pls https://code.launchpad.net/~daker/loco-directory/fix.806005/+merge/6817311:41
mhall119daker: did you get the test running?11:42
dakeryep11:44
mhall119cool, did the test code make sense to you?11:44
dakeryeah i understand now11:44
mhall119great!11:44
mhall119your branch has been approved11:45
dakerthanks!11:59
coalwaterthis is about the 32k bug? what is the max now ?12:25
daker10012:30
* nigelb hugs daker 12:46
* daker hugs nigelb13:21
dakercoalwater, 100 is a big number, i don't think someone will bring 100 guests with him.13:24
nigelbIf needed, we can always increase it.13:25
dakeryep13:25
nigelbmhall119: we should bundle all our deps on our own :D14:00
mhall119IS doesn't like that14:00
mhall119it makes keeping up with security fixes more difficul14:00
nigelbbut what we proposed is almost similar14:01
mhall119this is a temporary thing, once everybody is happy with the stability of the new django-openid-auth, it'll be installed system wide14:02
mhall119in that respect, we're beta-testing it on LD14:02
nigelbah14:04
cjohnstonyay for the community being the driving force on things mhall119  and nigelb14:41
mhall119cjohnston: I just got off a call with stuartm about the SSO deployment14:51
mhall119since we're getting the django-openid-auth locally, we'll be able to require the username for login14:51
mhall119so the only issue there might be is if someone chooses not to pass team membership to LD14:52
cjohnstonok14:52
mhall119if they do that, the only thing that would happen is they wouldn't be able to add meetings or events for their team14:52
cjohnstonwhat about for summit14:52
mhall119once the local django-openid-auth is working for LD, it should be trivial to add it to summit's pythonpath14:52
cjohnstonwas talking about summit with sso14:53
mhall119summit should be in the same situation as ld14:54
mhall119ok, anthony says they're running it on 1.3 in their testing environments with no issue14:55
cjohnstonsweet14:57
cjohnstonLD translation template has been pushed to LP15:04
dakermhall119, with the new SSO deployment, are we going to be able to request the email ?17:20
cjohnstonmhall119: will openid-auth fix currently errored names, or just keep names from breaking in the future17:26
mhall119cjohnston: it should fix them at the next login17:31
cjohnstoncool17:31
dakerfunny account https://launchpad.net/~anonymous-deactivatedaccount-deactivatedaccount-deactivatedaccount17:58
coalwaterlol18:00
cjohnstoncoalwater: did you see my reply to your question on the comment bug?18:00
coalwateryea, i'll work on it when i go home :D18:01
=== daker is now known as daker_
coalwaterok, goin home :D see you all later18:08
cjohnstonmhall119: any chance of creating and hosting a uds-p db?18:09
cjohnston(preferably with the previous uds's that are in uds-o db as well18:10
mhall119cjohnston: you can add data to the one I have on people.u.c and upload it to yours18:12
cjohnstonstill trying to figure out why that track leads migration is trying to delete the m2m18:15
cjohnstoni wonder if it is something with south changing18:15
cjohnstonmhall119: https://code.launchpad.net/~chrisjohnston/summit/tl/+merge/68289   do you see any changes that would cause it to delete an m2m in meeting?18:16
cjohnstonit does it in trunk18:18
cjohnstonhmm18:18
cjohnstonmhall119: I think http://bazaar.launchpad.net/~summit-hackers/summit/trunk/revision/120 is causing my problem18:20
cjohnstonmaybe?18:20
cjohnstonmaybe not tho since its already commented out18:20
cjohnstonmhall119: could you do bzr branch lp:summit test18:22
cjohnstonthen: cd test18:22
cjohnstoncd summit18:22
cjohnston./manage.py schemamigration schedule test --auto18:22
cjohnstonor nigelb ^^18:33
Ronniecjohnston: here it also deletes the M2M table18:54
mhall119have you tried it with South 0.6?18:54
Ronnieno, 0.7.318:56
Ronnieill install 0.618:57
mhall119try it with 0.6, it might be a difference in how they handle things that makes it think the M2M was removed18:57
Ronniemhall119, cjohnston: with 0.6: Unknown command: 'schemamigration'19:02
Ronniewith the command startmigration : http://paste.ubuntu.com/646733/19:05
cjohnstonhmm19:07
cjohnstonRonnie: mhall119 http://south.aeracode.org/docs/tutorial/part2.html#manytomany-fields19:21
cjohnston/31/window number 5519:22
cjohnstonuggh19:22
Ronnietrough is indeed the 'problem'. is the code still working after deleting the M2M table?19:23
cjohnstonthe migrate fails19:23
Ronnieyou probably need to do migrate --fake, but than the other changes in the same merge are not applied too19:31
cjohnstonwhat would that do19:32
Ronniethat would tell the migration database that the migration is executed, but does not execute the migration itself19:34
Ronnieprobably there is no m2m table in the database and the migration tries to delete the (non exisiting) m2m table19:35
cjohnstonRonnie: mhall119 this is what someone just told me:19:47
cjohnstonokay, I think there's something in the NameField that's confused it... maybe just handles it differently because of the  intermediate class, or the intermediate forces some parameter to default differently.  I think I've seen that before in my own work19:47
mhall119cjohnston: someone told you the NameField is causing problems with an M2M table?19:48
cjohnstonmhall119: join #django-south19:48
mhall119no19:49
mhall119you can't make me19:49
cjohnstonlol19:49
cjohnstonnot sure what it's doing with Participants.  I've used the by-name form for through, so it's probably not that.  Is Participant in19:51
cjohnston                    the frozen model of the last migration?19:51
cjohnston15.47.51 < cjohnston> martinm:19:51
cjohnston                      http://bazaar.launchpad.net/~summit-hackers/summit/trunk/view/head:/summit/schedule/migrations/0006_add_static_pad_url.py#L94  ?19:51
cjohnston-------------------------------------------------------------------------------------------------------------------------------------------------------19:51
cjohnston15.50.11 < martinm> the other thing that comes to mind is that I don't think I've used an external relation like that.  and that's interesting, it19:51
cjohnston                    didn't record the through aspect of it at all?19:51
cjohnstonmhall119: he seems to be suggesting to drop old migrations and do all the migrations under 0.720:02
cjohnstonand says that may work20:03
mhall119might not be a bad idea, but let's branch trunk to seek the old series and the new one separate20:03
cjohnstonso do like a v220:04
cjohnstondo you have time to do that anytime soon? im not sure how20:04
cjohnston(both the branch and the droping migrations20:04
mhall119cjohnston: I can do it20:06
cjohnstonmhall119: 16.01.55 < martinm> yeah.  also have to drop the record of the old migrations in south_migrationhistory20:06
cjohnston16.03.32 < martinm> depends partly on what's easier... I think this was an unintended incompatability in the frozen model format that came along with  the parser rewrite.  or maybe it was a bug in 0.6 that just didn't show up20:06
mhall119but first, is there anything that can land now?20:06
cjohnston-------------------------------------------------------------------------------------------------------------------------------------------------------20:06
cjohnston16.05.23 < martinm> ah, I do have that convo - I had a custom field which was being incorrectly frozen in 0.6.  never saw a symptom of it until 0.7,  whcih had fixed it20:06
cjohnstonthat was the issue he had20:06
cjohnstonmhall119: ya.. everything thats sitting https://code.launchpad.net/summit/+activereviews that is mine20:06
mhall119ok, let's land what we can and deploy, then I'll branch trunk to 1.x, then we can scrap the old migrations directories and generate new 0001 migration files for each app using 0.720:08
cjohnstoncool20:08
cjohnstonI'll sit and watch the approval emails come in :-)20:08
mhall119let me review your stuff tonight20:08
cjohnstonboo20:08
cjohnston:-P20:08
mhall119nigelb: would you be able to review any of his summit code?20:08
nigelbmhall119: Well, if he learns some patience, yes.20:09
nigelbmhall119: I can do some reviews this week (before friday)20:09
mhall119that's not likely20:09
nigelbBut no instant gratification, sorry.20:09
mhall119heh20:09
cjohnstoninstant.. there are some mps that are almost a month old20:09
mhall119yeah, only a month20:09
mhall119geez20:09
nigelbYes, UDS isn't tomorrow is it?20:09
cjohnstonno... but im going away before uds20:10
mhall119he's got a point, we're not even supposed to be working on this until the week prior20:10
cjohnstonso i wont be able to play prior to uds20:10
cjohnstonfor being a project manager, your a poor motivator20:10
cjohnstonand a poor planner20:10
mhall119I'm not a project manager20:10
cjohnstonnot you20:11
nigelbI have real life too, you know.20:11
cjohnstonsince when20:11
nigelbsince forever :D20:11
cjohnstonbullcrap20:11
nigelbAlso, summit is not the only webdev project I contribute to. So that's taking time too20:12
mhall119since he started this "sleeping" business20:12
cjohnstonmhall119: +120:12
cjohnstoni say we take his bed away from him mhall11920:12
nigelbcjohnston: https://github.com/mozilla/input.mozilla.org/commits/master20:13
cjohnstonmhall119: yeah, I think you have a reasonable choice, cjohnston: adding the through ought to bring it up to spec for 0.7.  The custom field  stuff was less clear, and would change again after refactoring the custom field out of the app for wider use, and I didn't need  the history of those schema changes, so I punted20:13
cjohnstonnigelb: it isnt summit, or ld, so it isnt important20:14
nigelbIsn't important for you :P20:14
nigelbImportant for me :)20:14
mhall119oh, so now you're sleeping *and* wanting your opinion to count?20:15
mhall119who do you think you are?20:15
nigelblol20:15
cjohnston+2 mhall11920:16
dakerg'night23:07

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