/srv/irclogs.ubuntu.com/2011/11/06/#ubuntu-website.txt

cjohnstonsvwilliams: https://lists.ubuntu.com/mailman/listinfo/community-web-projects01:31
cjohnstonmhall119: email20:11
cjohnstonmhall119: nigelb why would a db field be clearing out every 24 hours? its a char field20:58
mhall119cjohnston: in summit?20:59
cjohnstonltp20:59
cjohnstonour override name20:59
mhall119btw, the local_settings in the email looks okay to me, do we know what linaro is using?20:59
nigelbwhat db field?20:59
mhall119cjohnston: is the lpupdate process resetting it?20:59
cjohnstonno i dont20:59
nigelbmhall119: hey, did you see the upgrade thing for django 1.3 for authentication modules?21:00
cjohnstonmhall119: the override name should only be set manually.. lpupdate shouldnt be touching it21:00
nigelb(openid_auth in particular)21:00
cjohnston    override_name = models.CharField(_("Override Imported Team Name"), max_length=80, null=True, blank=True)21:00
nigelbOh, this is LTP.21:01
cjohnstonya21:01
* nigelb gets back to what he was doign.21:01
nigelb:P21:01
cjohnstonnigelb: do some summit work ;-)21:01
nigelbcjohnston: ya, 'm looking at the upgrade21:01
nigelbmhall119: do you have some time to help me?21:02
cjohnstoncool21:02
nigelbafter the upgrade all our tests break. WIIN.21:02
cjohnstonnice21:02
nigelbmhall119: http://paste.ubuntu.com/730405/21:04
nigelbIt still works, but you guys should probably know.21:04
mhall119james_w: happy birthday!21:05
nigelboooh21:05
cjohnstonhappy birthday james_w !21:05
nigelb _   _    _    ____  ______   __  ____ ___ ____ _____ _   _ ____    _ __   __21:06
nigelb| | | |  / \  |  _ \|  _ \ \ / / | __ )_ _|  _ \_   _| | | |  _ \  / \\ \ / /21:06
nigelb| |_| | / _ \ | |_) | |_) \ V /  |  _ \| || |_) || | | |_| | | | |/ _ \\ V /21:06
nigelb|  _  |/ ___ \|  __/|  __/ | |   | |_) | ||  _ < | | |  _  | |_| / ___ \| |21:06
nigelb|_| |_/_/   \_\_|   |_|    |_|   |____/___|_| \_\|_| |_| |_|____/_/   \_\_|21:06
cjohnstontroll!21:06
nigelb 21:06
nigelb:D21:06
nigelbhaha21:06
cjohnstonmhall119: twice last week I WATCHED hauts add the override name of Ubuntu France to his team.. and twice I checked and it was back to ubuntu-fr21:07
cjohnstonI just changed it in the admin area...21:07
nigelbmhall119: Thoughts on this error I see when running tests? http://paste.ubuntu.com/730414/21:07
cjohnstonhttp://paste.ubuntu.com/730415/  why does it add the field and then delete the field21:08
mhall119nigelb: eating lunch,  I'll check it in a few21:13
nigelbmhall119: cool, thanks. I'm digging as well.21:13
mhall119cjohnston: it's gotta be something in the lpupdate then21:14
mhall119nigelb: I recall encountering that wsgi.input error in ISD, I think we just added it into the env dict as an empty string or something21:16
mhall119nigelb: on line 79 of schedule/tests.py, add wsgi.input as an empty string21:22
mhall119should allow the tests to proceed21:22
mhall119nigelb: http://paste.ubuntu.com/730425/ is what is in our test that had the same problem21:22
cjohnstonmhall119: power went out for nigelb.. he says he will look in the morning21:23
mhall119ok21:23
cjohnstonmhall119: could you take a look at mine as well please?21:23
mhall119sure21:24
mhall119cjohnston: it's a migration file, it has a forward() and backward() function21:24
mhall119forward() adds the field21:24
mhall119backward() removed it (for rollbacks, etc)21:25
mhall119I don't see anything in the code that might be re-setting it21:33
mhall119did you verify that it was being saved?21:33
nigelbback!21:37
nigelbyay power21:37
nigelbmhall119: \o/ down to 2 errors21:39
nigelbBoth something to do with SSL handshake.21:39
mhall119nigelb: was lack of power one of the errors?21:39
nigelbheh21:39
mhall119nigelb: bah, I had that before too, something failing trying to connect to Launchpad21:39
mhall119nigelb: is this running lpupdate, or running tests?21:40
nigelbmhall119: just tests21:40
nigelbhttp://paste.ubuntu.com/730435/21:40
mhall119that probably means we should be mocking something outthen21:40
nigelbYeah21:40
nigelbI'm thinking oof creating a launchpad app21:40
mhall119nigelb: might be a useful separation21:41
nigelbI'll do a quick test and propose the move to 1.321:41
nigelband then work on that21:41
mhall119and yeah, that was a quick test I added for a fix for jcastro during sponsorship applications, it should be mocked out properly21:41
nigelbShall I ignore this for now and instead fix it after the move?21:42
mhall119yeah21:42
nigelbcool21:42
mhall119for now, if "manage.py test schedule" all pass, I'm happy21:42
nigelbI'm so glad we have tests :)21:42
nigelbThis migration would have been painful otherwise :)21:42
mhall119yeah21:42
mhall119it may still be painful, we didn't have 100% test coverage21:43
nigelb./manage.py test schedule works perfectly.21:43
mhall119any code we didn't touch in the past 6 months doesn't have tests21:43
mhall119ok, tests on the sponsor app can be fixed later then21:43
mhall119file a bug for the failing test and you can assign it to me21:43
nigelbcool, ok21:44
nigelbI'll file a bug for the launchpad separation as well.21:44
mhall119ok21:44
nigelbcjohnston: Get people to stop playing with that stupid bug.21:44
mhall119well, we were going to make a common app, would it make more sense to just put the launchpad code in there rather than it's own app?21:44
nigelblaunchpad on its own will let us do things like run against staging or local dev21:45
nigelbwe could use a django setting for production, staging, qastaging, and dev.21:45
mhall119we can't do that now with a local_settings.py variable?21:45
nigelbNot easily.21:45
mhall119ok21:45
nigelbWe also use launchpadlib in certain places21:45
mhall119yeah, but I don't think that alone needs a separate app21:46
mhall119LTP has ./common/launchpad.py21:46
nigelbOh.21:46
nigelbThat might work :)21:46
mhall119cjohnston: nigelb: so which of you is going to be project manager this cycle?21:47
nigelbI can do it. Hopefully more effectively than last cycle.21:47
mhall119or should I flip a coin? loser gets the job?21:47
nigelbhaha21:48
mhall119as long as it isn't me, I'll be happy21:48
nigelbheh21:48
nigelbmhall119: So, I've been thinking about this render.py clean up more.21:51
nigelbI wonder, should we render the plenary like we do currently?21:51
nigelbCan the plenary room be one room?21:52
nigelband used like any other room21:52
mhall119nigelb: it's rendered the way it is so that nothing get's scheduled in any other rooms during the plenary time slot21:53
nigelbAhh.21:53
nigelbfuuuu.21:53
mhall119besides, we're going to get a new UI for desktop users21:53
nigelbI realized now that we have to handle that as well.21:53
nigelbwhen is this UI going to be ready?21:53
mhall119so render.py can be relegated to just doing the screens at UDS21:53
mhall119nigelb: I have no idea21:53
nigelbIdeally, I want render.py out.21:54
nigelbWe should be rendering everything using the same code.21:54
mhall119yes, I agree21:54
nigelbAlso, have you used django's sites module?21:54
mhall119but it'll be easier to replace the functionality in separate, clean code, rather than trying to fix the mess that is render.py21:54
nigelbMaybe that's our solution to the linaro hard-coding.21:55
mhall119nigelb: only for testing21:55
nigelbOkay, so I tend to agree with you there.21:55
mhall119nigelb: sites framework would work, yes21:55
nigelbMaybe I'll work on a new class called display.py with new code.21:55
nigelbSo they can coexist while I code.21:56
mhall119that sounds good to me21:58
mhall119I'd like to see a clean separation between the data model that calculates the schedule, and the code that produces the view of it21:59
nigelbI think I'll do it like you suggested.21:59
nigelbGet everything loaded into a class22:00
nigelband then do the looping and stuff in the template22:00
nigelbpossibly using helpers22:00
nigelbmhall119: I've made summit incompatible with Django 1.122:05
nigelbWell, there was no other way :)22:05
cjohnstonmhall119: re: verify if its being saved, we went to /teams and it showed up with the override name22:06
cjohnstonnigelb: for mobile we want the screen to look like guidebook22:09
nigelbcjohnston: dream on.22:09
cjohnstonmhall119: i dont think we really decided too much on a desktop view did we?22:09
cjohnstondream on for the mobile view?22:09
nigelbdream on for making it look like guidebook.22:10
cjohnstonwhy22:10
mhall119nigelb: johnoxton is going to develop the mobile UI for us22:11
mhall119using HTML/CSS22:11
mhall119weren't you on skype for that session?22:11
nigelbYeah22:11
nigelbI think its a better idea to *not* make it look like guidebook.22:11
mhall119oh, ok22:12
cjohnstonnot color and all, but similar layout22:12
cjohnstonhttp://guidebookapp.com/static/corp/img/screenshots/phoneshot-schedule-3.png22:12
mhall119I think it's just going to be a fall-back option, I think we'll still use guidebook next UDS anyway22:12
nigelbThat's easy.22:12
nigelbI can do that22:12
cjohnstonthats what we want22:12
nigelbWe don't eeven need to muck up with render.py for that.22:12
mhall119we're not going to muck up with render.py at all22:13
nigelbcjohnston: can you send me more screenshots from your phone?22:13
mhall119treat render.py as read-only and just write new code22:13
nigelb+122:13
cjohnstonmhall119: i think if we get mobile summit looking better than guidebook, we dont need guidebook... i wonder if we could talk them into giving us 15 minutes after the keynote for a "new summit how-to"22:13
cjohnstonnigelb: itll be a few days22:13
cjohnstonhttp://guidebookapp.com/features/22:13
nigelbcjohnston: sure22:13
mhall119then when we don't need render.py anymore  we'll throw a party with cake and fireworks and play rock-paper-scissors for who gets the honor of running "bzr remove render.py"22:14
nigelbcjohnston: We could a youtube video instead22:14
cjohnstonsend out the video the week before or something?22:14
mhall119+1 for a video, we could link to it on summit.u.c too22:14
nigelbcjohnston: Yeah, we could announce it on uds-announce as soon as its up.22:15
mhall119cjohnston: the benefit of guidebook is that it runs offline with a copy of the schedule, much less network usage22:15
cjohnstonmhall119: but everyone cries about it all week22:15
mhall119and I don't think Canonical paid more than a couple hundred dollars for it22:15
cjohnstonya22:15
cjohnstoni dont like this city search bug22:15
mhall119for LTP?22:16
cjohnstonya22:16
cjohnstonwhere are the cities supposed to come from22:16
mhall119I don't know how we'd go about implementing it without going to an outside source22:16
cjohnstonor make the admins list big cities or something22:16
cjohnstonbut that sounds like a ton of work22:16
mhall119yeah, I'm going to -1 that one22:16
cjohnstonim ok with that22:16
mhall119simplest solution, add a TextField called "keywords" or "search terms" for the admin to fill out22:17
cjohnstonya22:17
nigelbIs that the one where we show venues but no teams?22:17
cjohnstonya22:17
cjohnstonand venues arent linked to teams22:17
cjohnstonmhall119: bug 88693822:17
mhall119I thought the problem was they wanted to enter a city name in the search field, and get the state-wide team in the results22:17
mhall119nothing to do with venues22:18
cjohnstonmhall119: bug #88693822:18
cjohnstonthats what they want22:18
cjohnstonbut they get venues22:18
mhall119oh, ok22:18
cjohnstonmhall119: http://loco.ubuntu.com/teams/ ctrl f "france"22:18
cjohnstonthen tomorrow do the same22:18
cjohnstonwhere is the stupid bug bot22:18
nigelbI don't know what cjohnston has against the French Team :P22:18
nigelbMaybe we'll never know.22:19
* cjohnston slapps nigelb 22:19
nigelbBad vacation there or something? :P22:19
cjohnstonnigelb: if you look right now, its using the override name.. tomorrow, it wont be22:19
cjohnstonhauts went ahead and applied the override name before the blog post and all22:19
mhall119cjohnston: is there any chance that this is just another admin unsetting it?22:20
cjohnstonmhall119: its only a LC field22:21
cjohnstonor atleast is supposed to be only an LC field22:21
mhall119oh yeah, you're right22:21
* cjohnston wants ask a favor and see if we can get loco-directory renamed to loco-team-portal on LP22:22
cjohnstonhttps://blueprints.launchpad.net/loco-directory/+spec/locale-team-support  mhall11922:22
mhall119I'm not sure what that would require22:22
cjohnstoni dont know.. but i figure id ask a favor to a few of our friends in private to see if we could work it out22:22
cjohnstonwould you be against renaming it?22:22
mhall119no22:23
cjohnstoni dont know that i want to do it if we have to create a new project22:23
mhall119I don't think it's a big deal though, since the project name in LP doesn't matter to users22:23
cjohnstoni know..22:23
nigelbmhall119: https://code.launchpad.net/~nigelbabu/summit/django-upgrade/+merge/8142222:27
nigelbcjohnston: https://code.launchpad.net/~nigelbabu/summit/crew-fix/+merge/8115322:28
nigelbThere, work for both of you.22:28
cjohnstondid yall see that i renamed summit in lp22:28
nigelbYeah22:28
* cjohnston is off.. company.. have a good night22:34
nigelbcjohnston: REVIEW THAT CODE!22:34
* mhall119 dinner, no code review23:47

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