[00:44] https://bugs.launchpad.net/summit/+bug/663168 [00:44] Launchpad bug 663168 in summit "http://summit.ubuntu.com/uds-n/track/other/ returns an error (affects: 1) (heat: 5)" [Critical,Confirmed] [00:44] that sounds like an old django bug to me [00:44] https://docs.djangoproject.com/en/dev/ref/models/querysets/#get [00:44] is the first entry in the section "Methods that do not return QuerySets" [00:45] Mark invalid and tell to reopen if needed [00:45] ? [00:46] o.. that might be prior to initslots being run [00:50] unless somehow the query is somehow returning a QuerySet [02:09] james_w: I marked that bug as critical because they said they put in place a 'bandaid' fix, which I wanted to check up on and clean up if necessary [04:12] cjohnston: I've my hands full with enough things besides dev and I prefer poking proding and gentle harassing [04:12] it's more me === mdke_ is now known as mdke === sipherdee_ is now known as sipherdee [17:29] james_w: cjohnston: nigelb: any objections to me killing the stable series now that trunk is in production? [17:30] go. kill. [17:31] mhall119: I thought you were the one wanting to keep stable so that trunk could be unstable [17:32] No [17:33] we don't let trunk be unstable anymore apparently. [17:40] nigelb: trunk is no longer unstable [17:40] That too. [17:40] I thought you were wanting to allow it to be unstable mhall119 [17:41] cjohnston: when I thought we were going to do major work on render.py this cycle I did [17:41] but that didn't happen, so we're back to just trunk and production [17:41] cjohnston: can you mark the render.py item as POSTPONED [17:41] if/when we end up doing that, we'll make a new unstable branch [17:41] ok [17:41] mhall119: Please make that High priority for next cycle when you discuss at UDS. [17:41] you going to be at the meetings nigelb ? [17:42] and then are you going to help us next cycle? :- [17:42] If its in the first half the day [17:42] :-P [17:42] that will depend on a lot of things. [17:42] But I will finish what I started :) [17:42] I won't take "new" responsibilities [17:42] but I'll take the carried over ones for sure [17:43] nigelb: I'm leaning towards "replace render.py screens" rather than attempting to "refactor render.py" or "cleanup render.py" [17:43] * cjohnston assigns render.py to nigelb for next session [17:43] mhall119: Yes, replace. [17:43] render.py is postponed [17:43] With a lot of templates and macros [17:43] mhall119: We should also improve our infra. [17:44] What do you think of setting up jenkins? ;) [17:44] and if jenkins can do auto-push to the ec2, that'd be splendid. [17:44] how much work is it? [17:44] I can do the work, give me a server :-) [17:44] I did this for $DAYJOB [17:44] or could we just have tarmac run the django test suite before landing [17:44] that too [17:45] nigelb: what does [summit-hackers] on blueprints set the session summary page as the wiki page: TODO mean [17:45] But that would make tarmac slower [17:45] we wanted to pull the session summary from blueprints to summit [17:45] and show it for ical especially [17:45] still doable if you want to poke it [17:46] i thought that was different.. [17:46] thats bug #793018 [17:46] Launchpad bug 793018 in summit "Pull the summary from the launchpad blueprint and push it out via the iCal to Guidebook (affects: 1) (heat: 3)" [High,Confirmed] https://launchpad.net/bugs/793018 [17:46] OR [17:47] It might be pushing the session eitherpad page to launchpad [17:47] But I don't remember that being talked about [17:47] Anyway, we can't do it this cycle. [17:47] im trying to figure out delete vs postpone... if noone knows what it is then delete [17:51] * mhall119 is impatient when it comes to tarmac [17:52] mhall119: I'll take an action item to help with that :) [17:53] make a slap tarmac link that we can hit [17:53] nigelb: you gonna make me more patient? [17:53] mhall119: No. [17:53] I'll give you a link to hit [17:53] I need to add lock files so we don't have simultaneous runs [17:54] mhall119: the imported blueprint is meeting model type blueprint I assume? [17:55] yes [17:55] yes [17:55] jamalta: [18:10] * cjohnston nominates jamalta as new lead for summit [18:10] no way :P [18:10] I propose cjohnston [18:10] nigelb: +1 [18:10] mhall119? [18:10] nigelb: you can't propose the person who assigned you [18:11] It depends [18:11] no it doesnt [18:11] At the end of the majority vote wins as well ;) [18:11] this isnt a democracy [18:11] I thought you were all for demcracy? [18:12] Its a mertocracy. Whoever does the work gets the job. [18:12] :P [18:12] no.. whoever does the work gets to pick [18:15] james_w / mhall119 - relevant to yesterday http://people.mozilla.org/~jdow/deploy.jpg [18:17] ha [18:18] +1 [18:28] is james_w around today? [18:34] cjohnston: nigelb: can one of you review https://code.launchpad.net/~mhall119/summit/fix-tests/+merge/75795 ? [18:37] mhall119: I'll go one up on that. reviewed and landed. [18:38] thanks nigelb [18:38] mhall119: I'm concerned. model_mummy is becoming more painful. [18:38] Is there a way to make it accept defaults [18:39] nigelb: possibly, I'm not sure [18:39] I think it's still worth using though [18:39] Totally worth using! [18:39] But when we change something, lot of tests might break [18:47] nigelb: which tells us that our tests are no longer valid and need to be changed as well [18:47] which is a good thing [18:47] mhall119: at some point I'll exlain how launchpad tests work. [18:48] * AlanBell fixes another bug in https://code.launchpad.net/~alanbell/loco-directory/backbutton/+merge/73526 [18:48] we can just say makeProduct() and it will take some random values. [18:48] I can override any property I wwant to. [18:54] nigelb: is that something the LP devs created, or something they get from using Zope? [18:54] I'll have to check. [18:54] mhall119: created [18:55] mhall119, I am [18:55] mhall119: not too hard either [18:55] we should do something like that [18:56] s/should/could [18:56] Its not eeasy [18:57] mhall119, it's all custom code [18:57] it's better than model_mommy though in my opinion [18:57] james_w: I landed pretty much everythiing and updated the ec2 with teh latest trunk [18:57] model_mommy is *almost* right [18:57] yay, thanks mhall119 [18:57] just that last little bit is *so* wrong [18:57] heh [18:58] james_w: I had to go back and update some test cases to force requires_dial_in=False for older tests [18:58] it should create non-random readable strings by default [18:58] We could wrap around model mummy to create such a factory. [18:58] mhall119, I guess that should be the default on the model? [18:58] nigelb, maybe, yeah [18:58] james_w: it seems that model_mommy doesn't use the default, it still goes random [18:58] mhall119, it's docs say it should use the default if there is one, I'll look in to it [18:59] nigelb, in fact a subclass can likely fix it [18:59] I ran pdb, and it was setting some to True [18:59] keeping the current mode as an optional fuzzer would be good [18:59] james_w: \o/ [18:59] fuzz testing, fun [18:59] We may be hacking around model_mummy in the wrong way. [18:59] deterministic tests are great [18:59] having a fuzzer to find problems when you want is also good [19:01] I might spend my afternoon on that in fact [19:01] let's see what my mail client says about that [19:01] james_w: when I meet you again, I'm buying you a beer! :-) [19:01] james_w: or you can do a production deployment of summit if you want, there's only the one migration, and no major changes [19:01] mhall119, ok, I can do that [19:02] it's Friday what can go wrong! [19:02] who cares, everybody's gone home anyway [19:03] jamalta, nice bug fix! :-) [19:03] I assume it works anyway [19:03] since it calls LP using urllib2, I couldn't write a test case without mocking [19:04] james_w: ha ty [19:04] mhall119: well, should that be re-written to use launchpadlib instead? [19:04] I don't think that particular bit is exposed via launchpadlib [19:04] or at least it wasn't when it was written [19:04] ahh [19:07] appears to work [19:20] ah screwy, model_mommy explicitly ignores defaults on boolean fields [19:20] as django apparently sets a default on them