| *** dendrobates is now known as dendro-afk | 00:24 | |
| *** westmaas has joined #openstack-meeting | 01:16 | |
| *** dendro-afk is now known as dendrobates | 02:07 | |
| *** westmaas has quit IRC | 02:31 | |
| *** thatsdone has joined #openstack-meeting | 02:34 | |
| *** dendrobates is now known as dendro-afk | 02:55 | |
| *** dendro-afk is now known as dendrobates | 02:56 | |
| *** thatsdon_ has joined #openstack-meeting | 08:08 | |
| *** thatsdone has quit IRC | 08:10 | |
| *** thatsdone has joined #openstack-meeting | 09:36 | |
| *** thatsdon_ has quit IRC | 09:39 | |
| *** thatsdon_ has joined #openstack-meeting | 12:00 | |
| *** thatsdone has quit IRC | 12:00 | |
| *** thatsdon_ has quit IRC | 12:20 | |
| *** thatsdone has joined #openstack-meeting | 12:20 | |
| *** thatsdon_ has joined #openstack-meeting | 12:38 | |
| *** thatsdone has quit IRC | 12:40 | |
| *** dabo has joined #openstack-meeting | 13:30 | |
| *** thatsdone has joined #openstack-meeting | 13:48 | |
| *** thatsdon_ has quit IRC | 13:50 | |
| *** anticw_ has joined #openstack-meeting | 14:04 | |
| *** anticw has quit IRC | 14:05 | |
| *** dendrobates has quit IRC | 14:08 | |
| *** dendrobates has joined #openstack-meeting | 14:09 | |
| *** thatsdon_ has joined #openstack-meeting | 14:09 | |
| *** thatsdone has quit IRC | 14:10 | |
| *** dabo has quit IRC | 14:19 | |
| *** dabo has joined #openstack-meeting | 14:22 | |
| *** dendrobates is now known as dendro-afk | 14:40 | |
| *** thatsdon_ has quit IRC | 14:50 | |
| *** dendro-afk is now known as dendrobates | 14:59 | |
| *** dendrobates has joined #openstack-meeting | 14:59 | |
| *** thatsdone has joined #openstack-meeting | 15:23 | |
| *** westmaas has joined #openstack-meeting | 15:46 | |
| *** bcwaldon has joined #openstack-meeting | 15:50 | |
| *** thatsdone has quit IRC | 15:52 | |
| *** troytoman has joined #openstack-meeting | 15:56 | |
| *** cynb has joined #openstack-meeting | 16:36 | |
| *** thatsdone has joined #openstack-meeting | 16:53 | |
| *** thatsdon_ has joined #openstack-meeting | 16:54 | |
| *** thatsdone has quit IRC | 16:58 | |
| *** nixpert has joined #openstack-meeting | 17:47 | |
| *** nixpert has quit IRC | 18:33 | |
| *** nixpert has joined #openstack-meeting | 18:34 | |
| *** thatsdon_ is now known as thatsdone | 18:49 | |
| *** thatsdone has quit IRC | 19:05 | |
| *** GasbaKid has joined #openstack-meeting | 19:11 | |
| *** dendrobates is now known as dendro-afk | 19:18 | |
| *** bcwaldon has quit IRC | 19:27 | |
| *** glenc has joined #openstack-meeting | 19:38 | |
| *** westmaas has quit IRC | 19:40 | |
| *** westmaas has joined #openstack-meeting | 19:41 | |
| *** bcwaldon has joined #openstack-meeting | 19:43 | |
| *** dendro-afk is now known as dendrobates | 19:43 | |
| *** creiht has joined #openstack-meeting | 19:54 | |
| *** nixpert has quit IRC | 20:00 | |
| *** gustavomzw has joined #openstack-meeting | 20:02 | |
| *** gustavomzw has quit IRC | 20:05 | |
| *** gustavomzw has joined #openstack-meeting | 20:05 | |
| *** gustavomzw has quit IRC | 20:08 | |
| *** henrichrubin has joined #openstack-meeting | 20:15 | |
| *** bcwaldon has quit IRC | 20:17 | |
| *** groo has joined #openstack-meeting | 20:21 | |
| *** groo has quit IRC | 20:25 | |
| *** DigitalFlux has joined #openstack-meeting | 20:27 | |
| *** groo has joined #openstack-meeting | 20:29 | |
| *** bcwaldon has joined #openstack-meeting | 20:32 | |
| *** groo has left #openstack-meeting | 20:36 | |
| *** johnpur has joined #openstack-meeting | 20:46 | |
| *** User613 has joined #openstack-meeting | 20:47 | |
| *** gholt has joined #openstack-meeting | 20:47 | |
| *** User613 has quit IRC | 20:47 | |
| *** User613 has joined #openstack-meeting | 20:47 | |
| *** colinnich has joined #openstack-meeting | 20:47 | |
| *** gustavomzw has joined #openstack-meeting | 20:48 | |
| *** Tushar has joined #openstack-meeting | 20:49 | |
| *** spectorclan has joined #openstack-meeting | 20:50 | |
| *** ttx has joined #openstack-meeting | 20:50 | |
| *** masumotok has joined #openstack-meeting | 20:54 | |
| *** gustavomzw is now known as groo | 20:55 | |
| *** jk0 has joined #openstack-meeting | 20:55 | |
| spectorclan | I'm here for a bit | 20:56 |
|---|---|---|
| *** jaypipes has joined #openstack-meeting | 20:57 | |
| *** dragondm has joined #openstack-meeting | 20:58 | |
| ttx | ~o~ | 20:59 |
| dendrobates | o/ | 20:59 |
| glenc | \o | 20:59 |
| zul | heylo | 20:59 |
| spectorclan | o | 20:59 |
| dabo | \o | 20:59 |
| * ttx feels a bit older today. | 20:59 | |
| * dragondm waves | 20:59 | |
| termie | ([) | 20:59 |
| *** kpepple has joined #openstack-meeting | 21:00 | |
| *** vishy has joined #openstack-meeting | 21:00 | |
| ttx | ooook, let's get started, we have a large agenda today | 21:00 |
| *** koji-iida has joined #openstack-meeting | 21:00 | |
| ttx | #startmeeting | 21:00 |
| openstack | Meeting started Tue Jan 18 21:00:48 2011 UTC. The chair is ttx. Information about MeetBot at http://wiki.debian.org/MeetBot. | 21:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic. | 21:00 |
| *** eday has joined #openstack-meeting | 21:00 | |
| ttx | Welcome to our weekly OpenStack team meeting... | 21:00 |
| ttx | Today's (long) agenda is at: | 21:01 |
| soren | o/ | 21:01 |
| ttx | #link http://wiki.openstack.org/Meetings | 21:01 |
| ttx | #topic Actions from previous meeting | 21:01 |
| ttx | * ttx to document simplified exception procedure in wiki: DONE | 21:01 |
| ttx | #topic Current release stage: QA | 21:01 |
| ttx | #info We are now passed FeatureFreeze and well into QA stage. The goal is to test, find and fix as many bugs as we can before GammaFreeze. | 21:02 |
| ttx | After GammaFreeze the idea is to restrict bugfixing to release-critical issues only, in order to avoid introducing regressions... a known bug being better than an unknown regression | 21:02 |
| ttx | That said, on the release schedule, GammaFreeze is EOD on Tuesday, January 25. I wonder if we shouldn't push that back to EOD Thursday, January 27. | 21:02 |
| *** sandywalsh_ has joined #openstack-meeting | 21:02 | |
| ttx | Given the number of new things that landed in this release, maximizing random bugfix time sounds like a good idea for Nova... | 21:02 |
| *** ewanmellor has joined #openstack-meeting | 21:03 | |
| ttx | 3 business days between GammaFreeze and RCFreeze should be enough to spot and fix regressions. | 21:03 |
| ttx | Thoughts ? | 21:03 |
| jaypipes | hopefully... | 21:03 |
| dendrobates | only if we stop new merges | 21:03 |
| ttx | dendrobates: new merges ? | 21:03 |
| devcamcar | would be nice to have til 27th for gamma | 21:04 |
| devcamcar | how are we supposed to fix bugs if we can merge? :) | 21:04 |
| dendrobates | ttx: there are still some outstanding questions that we are going to discuss later in the meeting. | 21:04 |
| dendrobates | ttx: I just mean we need to decide today and not draw anything out | 21:04 |
| ttx | ok, maybe revisit that question at the end of the meeting then. | 21:05 |
| ttx | Moving on | 21:05 |
| ttx | #topic Release status | 21:05 |
| ttx | #link http://wiki.openstack.org/releasestatus/ | 21:05 |
| ttx | We still have 2 feature branches in the FeatureFreezeException track: | 21:05 |
| ttx | https://code.launchpad.net/~citrix-openstack/nova/xenapi-glance-2/+merge/45977 | 21:05 |
| ttx | jaypipes: could you give us a quick summary of the situation ? | 21:06 |
| *** DigitalFlux has quit IRC | 21:06 | |
| jaypipes | sure.. | 21:06 |
| ttx | or maybe ewan | 21:06 |
| ewanmellor | right here | 21:06 |
| ttx | ewanmellor: could you give us a quick summary of the situation ? | 21:06 |
| jaypipes | xenapi-glance-2 was delayed because of Glance packaging issues. that has been fixed, but there were other bugs in Glance that showed up. the one right now that is a showstopper is appparent large file upload limitations... | 21:07 |
| jaypipes | So, while citrix' code is fine, glance has bugs that may be showstoppers for merging that code in. | 21:07 |
| ttx | jaypipes: and we are still a few days away from getting those ironed out, right | 21:08 |
| ewanmellor | We've worked around those problems by registering images by URL rather than by upload | 21:08 |
| jaypipes | ttx: and since I'm currently the only one working on glance bugs, it's a little tough to predict if I can get them all fixed in time . | 21:08 |
| jaypipes | ewanmellor: sure, but that's a limitation for sure.. | 21:08 |
| *** DigitalFlux has joined #openstack-meeting | 21:08 | |
| ttx | So from a release management perspective, I want us to stop merging new code asap... | 21:08 |
| ttx | But it's up to the nova-core team to determine if it's ready to go in or not. | 21:09 |
| jaypipes | ttx: like I said, it's not the xenapi-glance-2 code that's buggy, it's glance... | 21:09 |
| ewanmellor | There's no reason to block the Nova half of the code because of a bug in Glance. | 21:09 |
| ewanmellor | jaypipes: Thanks ;-) | 21:09 |
| ttx | jaypipes: so should we merge the code ? | 21:10 |
| ttx | the code is fine, the feature is not, IIUC | 21:10 |
| devcamcar | ewanmellor: agree, lets merge the branch and let the glance bug be a glance bug | 21:10 |
| jaypipes | ttx: that is true, actually, there are 2 workarounds... one for glance (use x-image-meta-location) and one is just use nova-objectstore... | 21:10 |
| ewanmellor | I can get someone to help with the Glance bugs. | 21:10 |
| jaypipes | ttx: I don't want to make the final call, because it's my bugs that are the issue... | 21:10 |
| * jaypipes hides sheepishly... | 21:11 | |
| ttx | any other nova-core dev wanting to chime in ? | 21:11 |
| jaypipes | ttx: devcamcar says proceed... | 21:11 |
| soren | jaypipes: Are the glance problems likely to be resolved any time soon? | 21:11 |
| jaypipes | soren: hopefully, yes. | 21:11 |
| dendrobates | I say merge it | 21:11 |
| soren | +1 | 21:11 |
| devcamcar | +1 | 21:11 |
| jaypipes | soren: there were, like 6 bugs, and I've fixed all but 1 of them.. | 21:11 |
| xtoddx | are we talking about merging a branch we can't test? | 21:11 |
| jaypipes | xtoddx: no. | 21:11 |
| xtoddx | then merge | 21:12 |
| jaypipes | xtoddx: it has unit tests and has functional testing ability (with the workarounds mentioned above) | 21:12 |
| ttx | ok, then please one of you set the branch to approved now :) | 21:12 |
| jaypipes | devcamcar: go for it. | 21:12 |
| creiht | jaypipes: If the only problem is the large file uploading one, that should be fixable | 21:12 |
| ttx | second feature branch left: | 21:13 |
| ttx | https://code.launchpad.net/~termie/nova/db_migration/+merge/46073 | 21:13 |
| jaypipes | ttx: I'll try to get that remaining bug resolved, and in the unlikely event I cannot resolve it, we'll make a note in the release about the workarounds. | 21:13 |
| *** bcwaldon has quit IRC | 21:13 | |
| ttx | termie: could you give us a quick summary of the situation, though I guess that will quickly bring us to the next topic... | 21:13 |
| ttx | jaypipes: works for me | 21:13 |
| jaypipes | creiht: yeah, that's the only bug left. | 21:13 |
| termie | the branch is essentially done, it has just been being updated to track trunk changes to db schema | 21:13 |
| termie | the latest tracking brought me to the changes introduced by live_migration | 21:13 |
| *** tonywolf has joined #openstack-meeting | 21:14 | |
| termie | which is up for some debate and the db_migration branch is stalled on it | 21:14 |
| soren | Can you explain how db_migration is stuck because of it? It's not entirely clear to me. | 21:14 |
| jaypipes | no debate in my mind... I've voted for a reversion of live-migration and a merge of db-migration... | 21:14 |
| *** enigma has joined #openstack-meeting | 21:14 | |
| termie | largely in that the schema changes it suggests do not appear to be completely thought out | 21:14 |
| ttx | ok, let's switch to the next topic then | 21:15 |
| ttx | #topic Live-Migration patch | 21:15 |
| ttx | vishy/termie: floor is yours. | 21:15 |
| *** bcwaldon has joined #openstack-meeting | 21:15 | |
| termie | sure, some discussion has already happened over on https://code.launchpad.net/~termie/nova/revert_live_migration/+merge/46660 | 21:15 |
| vishy | basically there are a number of issues with the patch | 21:16 |
| ttx | termie: wouldn't it be simpler at this point to just fix the remaining issues ? | 21:16 |
| termie | the gist of it is that i think the code review of the live_migration branch failed to identify issues | 21:16 |
| vishy | there are two options: keep the merge in and fix the issues | 21:16 |
| termie | ttx: not nearly so simple as reverting the patch, no | 21:16 |
| termie | ttx: there are no tests in the patch | 21:16 |
| vishy | or revert the merge and let the original author fix the issues | 21:16 |
| ttx | termie: "revert+fix+rereview" is not simpler than "fix" | 21:17 |
| termie | ttx: yes it is when there are no tests | 21:17 |
| termie | ttx: and we are only talking about revert here | 21:17 |
| soren | termie: I can't help but wonder why you didn't raise these issues while this patch was being reviewed? | 21:17 |
| termie | soren: i didn't review the patch | 21:17 |
| termie | soren: i don't believe i am expected to review them all? | 21:17 |
| soren | Are you in nova-core? | 21:18 |
| *** glenc_ has joined #openstack-meeting | 21:18 | |
| jaypipes | OK guys, let's cool it. | 21:18 |
| vishy | I think we have failed as a team in terms of reviewing | 21:18 |
| ttx | termie: if we revert it, that means we push it back to Cactus. We can't wait for another review cycle. | 21:18 |
| dragondm | couldn't we implement some test-coverage requirement hook/test in hudson?? | 21:18 |
| jaypipes | A number of us were on vacations, let's be more reasonable here.. | 21:18 |
| termie | ttx: i think it should be pushed back to after bexar, yes | 21:18 |
| ttx | termie: I'm fine with that, just explaining what "reverting" exactly means | 21:18 |
| dendrobates | vishy: I agree we have failed | 21:18 |
| eday | dragondm: we already have coverage checks, we're just not enforcing anything. but, yes, we could | 21:18 |
| soren | dragondm: Current trunk would fail such a test, probably. | 21:19 |
| vishy | we need to have a discussion about nova-core and reviews. We proposed to the community that they could submit patches before BMPF and that we would help them prepare for FF | 21:19 |
| vishy | we clearly did not do that well for this patch | 21:19 |
| *** glenc has quit IRC | 21:19 | |
| vishy | and most of the external patches were probably under-reviewed | 21:19 |
| jaypipes | agreed, for this specific patch. most other patches had a lot of review, however (xenapi-glance-2, for instance..) | 21:20 |
| vishy | soren and I hit most of them, but we did it too late and too quickly imo | 21:20 |
| *** glenc_ is now known as glenc | 21:20 | |
| jaypipes | vishy: sorry, what did you mean by "external patches"? | 21:20 |
| termie | jaypipes: i think he means patches not coming from core team members | 21:20 |
| vishy | jaypipes: patches done by non- nova-core members | 21:21 |
| jaypipes | ah, k. | 21:21 |
| termie | jaypipes: we tend to talk about our own patches amongst ourselves more | 21:21 |
| devcamcar | yea i did notice a number of patches that weren't reviewed by nova-core but were still approved | 21:21 |
| soren | vishy: At some point, I did switch into "alright, I think this is `good enough`, we can fix the remaining issues after the freeze" sort of mode. | 21:21 |
| devcamcar | is there some way to enforce that? | 21:21 |
| soren | vishy: Perhaps not entirely consciously. | 21:21 |
| jaypipes | I think some non-core patches were very heavily reviewed (all of the citrix ones at least). the reason is because we were able to go through 5-6 cycles for each of the patches.. | 21:21 |
| *** westmaas has quit IRC | 21:21 | |
| ttx | In the end it's a chief architect / nova-core decision. Is it good enough ? I just set the deadline. | 21:21 |
| vishy | soren: me too | 21:22 |
| soren | devcamcar: Whuh? | 21:22 |
| soren | devcamcar: Example? | 21:22 |
| jaypipes | devcamcar: that's not possible AFAIK... | 21:22 |
| *** westmaas has joined #openstack-meeting | 21:22 | |
| ttx | devcamcar: only nova-core can approve | 21:22 |
| vishy | it is possible if someone from nova-core doesn't wait for the second approval | 21:22 |
| devcamcar | i could have sworn i saw a few, maybe i'm wrong | 21:22 |
| dendrobates | adding to nova core and having more people doing reviews would be helpful | 21:23 |
| jaypipes | vishy: yes, that is true. | 21:23 |
| sandywalsh_ | dendrobates, +1 | 21:23 |
| soren | To me, being in nova-core means you *must* do reviews. | 21:23 |
| ttx | soren: +1 | 21:23 |
| vishy | +1 | 21:23 |
| ttx | nova-core is a duty, not a right | 21:23 |
| soren | Membership in nova-core is.. | 21:23 |
| soren | right, precisely. | 21:23 |
| *** spectorclan has quit IRC | 21:23 | |
| dendrobates | soren: I am working on a definition of core dev for the wiki | 21:24 |
| vishy | people that don't have time to review sould remove themselves from nova-core so we know how many active reviewers we have | 21:24 |
| jaypipes | Guys, I think we should keep some perspective here... granted, this live-migration patch was a mess, but overall, we reviewed dozens of patches in the last week or so... | 21:24 |
| dendrobates | should be published tonight | 21:24 |
| soren | vishy: Fully agreed. | 21:24 |
| *** pvo has joined #openstack-meeting | 21:24 | |
| termie | if nova-core means must then require sizeof(nova-core) approvals | 21:24 |
| vishy | jaypipes: true...its the big ones that no one wants to touch | 21:24 |
| creiht | We should ask lp to have a popup when you hit the approve button to confirm that all unit tests were run and that they did indeed read the code | 21:25 |
| creiht | ;) | 21:25 |
| termie | creiht: we should ask lp to have line-by-line code review | 21:25 |
| jaypipes | vishy: not necessarily :) I reviewed some pretty big ones...but just did not get around to live-migrations for some reason.. | 21:25 |
| vishy | in any case we can go on about fixes to the process, but we need to take action on this particular patch, so what should we do? | 21:25 |
| soren | termie: No, we shouldn't. I don't want to use a web UI to do code review. | 21:25 |
| soren | Fix the patch. | 21:25 |
| soren | IMO | 21:26 |
| devcamcar | it can be tough to review the larger patches solo | 21:26 |
| jaypipes | is this a vote then? revert vs. fix patch? | 21:26 |
| termie | revert | 21:26 |
| jaypipes | revert | 21:26 |
| devcamcar | it would be valuable to have a way to review virtually as a group | 21:26 |
| vishy | i feel that it was our (nova-core's) failure and to show good faith to the community we should fix it ourselves. | 21:26 |
| termie | devcamcar: standard line-by-line code review handles group interaction well | 21:26 |
| vishy | fix | 21:27 |
| dendrobates | fix, basically because we messed up. | 21:27 |
| devcamcar | termie: yes it does but lp doesn't do it =x | 21:27 |
| devcamcar | revert | 21:27 |
| xtoddx | revert, and fix in a branch ourselves? | 21:27 |
| soren | Just to be clear: Revert means revert the patch and refuse the feature for bexar? | 21:27 |
| xtoddx | definitely revert though | 21:27 |
| ttx | xtoddx: there is no time left for that option | 21:28 |
| ttx | (revert+fix) | 21:28 |
| devcamcar | just playing devil's advocate - why is it our job to fix as opposed to authors | 21:28 |
| xtoddx | ttx: for cactus then | 21:28 |
| jaypipes | soren: yes | 21:28 |
| soren | devcamcar: We approved it. | 21:28 |
| ttx | xtoddx: ack | 21:28 |
| dendrobates | we shoudn;t have merged it, but once we did, we all own it. | 21:28 |
| ttx | soren: yes | 21:28 |
| soren | If people hated the patch so much, they should have said something. Really. | 21:28 |
| vishy | devcamcar: and we didn't review it in a timely manner as outlined for the process for getting stuff in. | 21:28 |
| devcamcar | ok, change my vote to fix | 21:28 |
| devcamcar | otherwise its not going in this release | 21:29 |
| termie | soren: you should have reviewed it better, you can't blame that on somebody else | 21:29 |
| ttx | I'm not nova-core, but I prefer to "fix", not because otherwise it won't make it into this release, but to be fair to the proposer | 21:29 |
| soren | termie: I'm not. | 21:29 |
| jaypipes | termie: it's not all on soren. we all played a role. | 21:29 |
| soren | termie: How am I blaming anyone else? | 21:29 |
| ttx | since he proposed well before the branchmergeproposalfreeze | 21:29 |
| termie | soren: "if people hated the patch so much..." etc | 21:30 |
| soren | termie: If I were proposing to revert, *then* I'd be a hypocrite. | 21:30 |
| termie | soren: we depend on the people who review something to do it correctly | 21:30 |
| termie | otherwise we _all_ have to look at eevery patch | 21:30 |
| devcamcar | termie, soren: take it to pms | 21:30 |
| vishy | termie: I'll take credit for being lazy with the review | 21:30 |
| soren | Me too, for sure. | 21:31 |
| ttx | I take my part in the blame -- I pushed for getting stuff merged | 21:31 |
| xtoddx | we don't need to assess blame, we just need to move on | 21:31 |
| jaypipes | xtoddx: ++ | 21:31 |
| ttx | (though I also pushed for getting stuff reviewed :) | 21:31 |
| soren | I decided to let it in and fix things after the fact. | 21:31 |
| ttx | it's about half/half votes | 21:31 |
| soren | I figured it'd be faster, and the really tricky bits seemed ok. | 21:31 |
| vishy | forget blame, I'm taking credit...laziness is an asset! :) | 21:31 |
| dendrobates | I think we can agree to fix the process and not let this happen again. | 21:31 |
| jaypipes | hehe | 21:31 |
| creiht | at least it was pep8 compliant ;) | 21:31 |
| ttx | sounds like it's time for some chief architect fiat | 21:31 |
| jaypipes | creiht: had to go there, didn't ya? :P | 21:32 |
| ttx | creiht: it wasn't until the last review | 21:32 |
| ttx | :P | 21:32 |
| creiht | >:) | 21:32 |
| johnpur | who is volunteering to make the fix and get it reviewed if we vote for fix? | 21:32 |
| xtoddx | we have a policy of piece-meal commits to keep size down, so approving a half-baked commit should be expected occasionally. | 21:32 |
| dendrobates | ttx: what was the vote? | 21:32 |
| termie | johnpur: looks like me | 21:32 |
| ttx | "fix" vs. "revert and defer to cactus" | 21:32 |
| termie | there are no tests though | 21:32 |
| termie | so it can't be fixed | 21:32 |
| johnpur | termie: how much time will this take? | 21:32 |
| ttx | I don't tjhink it would just fall on termie | 21:33 |
| dendrobates | soren: you should help to. | 21:33 |
| johnpur | can we get tests from the original author | 21:33 |
| vishy | masumotok can probably help | 21:33 |
| vishy | johnpur: lets ping him for some tests | 21:33 |
| termie | tests usually take longe rot write than the code, for the record | 21:33 |
| johnpur | termie: a sad fact! | 21:33 |
| vishy | meanwhile we can clean up formatting issues and such | 21:34 |
| masumotok | how much time ca I have to add the test? | 21:34 |
| johnpur | ttx: how long can we reasonably take? | 21:34 |
| xtoddx | lets aim for landing the fix before gamma freeze? | 21:34 |
| ttx | johnpur: depends on how much that blocks db-migration | 21:34 |
| ttx | I want db-migration in | 21:34 |
| soren | Me too. | 21:34 |
| johnpur | me too | 21:35 |
| ttx | since its 1k lines still needing to hit trunk | 21:35 |
| termie | ttx: they are self-contained lines | 21:35 |
| vishy | masumotok: hi, didn't know you were here | 21:36 |
| termie | ttx: so it doesn't affect much by waiting, i'd like it in too but we will have to change that code again once the new changes happen | 21:36 |
| termie | so either i update it to fit with current and then make another branch after this fixes branch goes in | 21:36 |
| jaypipes | isn't gamma freeze in 8 days? can we get tests in next 2 days and reviews in an additional day? | 21:36 |
| ttx | OK. Let's give the db-migration branch until the end of the week | 21:36 |
| johnpur | ttx: agree | 21:36 |
| xtoddx | do we want to aim for tests from masumotok by end of week, leaving the weekend and until tuesday to land fixes to that branch and migrations before gamma freeze? | 21:36 |
| ttx | since it was expected to be the last branch anyway, by design | 21:36 |
| termie | masumotok: i am happy enough to work wiht you to write tests, i don't think this will be a very easy thing to test | 21:37 |
| jaypipes | xtoddx: that's pushing it a bit thin IMHO. | 21:37 |
| termie | masumotok: but i have worked with testing a lot so maybe can offer assistance | 21:37 |
| dendrobates | down to the wire again :( | 21:37 |
| johnpur | dendrobates: that's how we roll! | 21:37 |
| xtoddx | jaypipes: then lets make a post-gamma exception and get it in before release candidate freeze? | 21:37 |
| ttx | xtoddx: no way | 21:37 |
| xtoddx | we want both features, what the other option? | 21:38 |
| jaypipes | xtoddx: no, I was suggesting the other way :) reviews complete by end of week, tests done by Thursday. | 21:38 |
| ttx | I think we have enough time | 21:38 |
| dendrobates | if the tests don;t arrive at that late date, what would we do? | 21:38 |
| dendrobates | revert right before release | 21:38 |
| dendrobates | ? | 21:38 |
| jaypipes | masumotok: think you can get your team to get tests done by Thursday end of day UTC? | 21:39 |
| ttx | dendrobates: the end of this week is not right before release | 21:39 |
| termie | i would say revert now if matsumotok is on board and he can just work on cleaning the branch up more | 21:39 |
| masumotok | jaypipes: yes | 21:39 |
| dendrobates | ttx: I meant post gamma | 21:39 |
| *** MrKohonen has joined #openstack-meeting | 21:39 | |
| johnpur | dendrobates: let's make sure that doesn't happen.... sounds like there is a lot of help ready to write the tests | 21:39 |
| eday | I'm thinking we revert and merge db migration today. if we can get it fixed up before freeze, it makes it, if not, it's not a big revert at the last minute | 21:39 |
| ttx | dendrobates: no way I'll let that overflow to next week, lest postgamma | 21:39 |
| termie | eday: +1 | 21:39 |
| jaypipes | termie: I'm on the fence with this one... | 21:39 |
| dendrobates | eday: that is starting to seem more sane to me | 21:40 |
| jaypipes | eday: that was termie's original suggestion, and what I voted for originally... | 21:40 |
| vishy | eday: I'm ok with that proposal if masumotok is ok with it | 21:40 |
| eday | jaypipes: yup | 21:40 |
| johnpur | jaypipes: time to get off the fence, let's decide | 21:40 |
| dendrobates | I think we are waiting for masumotok | 21:40 |
| ttx | I feel comfortable merging db-migration late... but not so much merging live-migration late | 21:40 |
| dendrobates | ttx: why?> | 21:41 |
| *** pvo is now known as pvo_away | 21:41 | |
| jaypipes | johnpur: revert live-migrations, merge db-migrations, masumotok to make tests by thursday, nova-core to get reviewed by end of this week. | 21:41 |
| *** pvo_away is now known as pvo | 21:41 | |
| masumotok | End of the thursday I submit the test code, do I? | 21:41 |
| masumotok | UTC | 21:41 |
| ttx | because db-migration is relatively self-contained | 21:41 |
| jaypipes | masumotok: yes, along with the rest of the patch, which we'll revert to get the db-migrations patch in. | 21:41 |
| masumotok | jaypipes: that is fine. | 21:42 |
| dendrobates | ok, done. | 21:42 |
| jaypipes | masumotok: and you have nova-core to assist with you in the tests as you need it. | 21:42 |
| *** MrKohonen has quit IRC | 21:42 | |
| dendrobates | not really, who is going to handle reverting? | 21:42 |
| jaypipes | dendrobates: it's already proposed by termie.. | 21:42 |
| ttx | dendrobates: termie has a branch up for it | 21:42 |
| termie | dendrobates: already submitted the merge prop | 21:42 |
| xtoddx | dendrobates: termie has a branch | 21:43 |
| ttx | 4x | 21:43 |
| dendrobates | easy then | 21:43 |
| jaypipes | heh | 21:43 |
| soren | ttx: Can you post a summary of this discussion? | 21:43 |
| soren | ttx: On that bpm? | 21:43 |
| johnpur | all: nice conclusion | 21:43 |
| soren | bmp, even. | 21:43 |
| ttx | soren: I will. Tomorrow | 21:43 |
| *** annegentle has joined #openstack-meeting | 21:43 | |
| xtoddx | haha, i read that as "eday then" which was quite the non-sequitur. | 21:43 |
| ttx | #action ttx to summarize the decision on live-migration | 21:43 |
| *** diegoparrilla has joined #openstack-meeting | 21:43 | |
| dendrobates | ttx: what, you can't work all night? | 21:43 |
| ttx | dendrobates: that depends how much more you can pay me | 21:43 |
| ttx | #topic D release naming | 21:44 |
| ttx | OK, funnier discussion now. | 21:44 |
| ttx | As we move closer to Bexar release, we need to discuss the name of the "D" release. | 21:44 |
| ttx | I summed up the rules of the game at: | 21:44 |
| ttx | #link http://wiki.openstack.org/ReleaseNaming | 21:44 |
| ttx | There aren't so many cities and counties in California starting with a D, but some people added new ones | 21:44 |
| ttx | Please dump your preference on the channel for a highly scientific popularity contest ! | 21:44 |
| dendrobates | diablo!!!!! | 21:45 |
| ttx | Sounds like it boils down to dublin vs. Diablo | 21:45 |
| zul | diablo | 21:45 |
| jaypipes | Denver. | 21:45 |
| xtoddx | I like Dublin. Mostly because of Dr Pepper | 21:45 |
| ttx | Dublin ! | 21:45 |
| glenc | Dunsmuir | 21:45 |
| termie | i think danville is pretty funny | 21:45 |
| termie | as a california native | 21:45 |
| kpepple | Downieville (http://en.wikipedia.org/wiki/Downieville,_California) | 21:45 |
| sandywalsh_ | diablo | 21:45 |
| ttx | stephenspector asked me to vote "Diablo" for him | 21:45 |
| soren | Diablo sounds awesome. | 21:45 |
| _cerberus_ | Diablo | 21:45 |
| pvo | diablo | 21:45 |
| * jaypipes was joking... diablo sounds good to me. | 21:45 | |
| xtoddx | Dublin does double duty as a tx city and a ca city | 21:45 |
| diegoparrilla | Diablo, no doubt | 21:46 |
| * creiht like Dublin | 21:46 | |
| termie | easier to drink beer from dublin to celebrate, also | 21:46 |
| * vishy votes for Dorris | 21:46 | |
| jaypipes | diegoparrilla: ah, hola. :) | 21:46 |
| eday | so we are ok with diablo but not corpus-christi? :) | 21:46 |
| soren | Yes. :) | 21:46 |
| ttx | eday :) | 21:46 |
| creiht | and don't forget dublin sugar dr pepper :) | 21:46 |
| soren | Absolutely. | 21:46 |
| dabo | Diablo has more non-city meaning than Dublin | 21:46 |
| * sandywalsh_ looks for the lightning clouds | 21:46 | |
| jaypipes | corpus christi isn't in California. :P | 21:46 |
| termie | diablo just makes me think of the video game | 21:47 |
| dabo | Hard to explain it's supposed to be a city reference | 21:47 |
| * creiht thinks diablo is a dumb name for a release | 21:47 | |
| termie | clicking on pale yellow artifacts | 21:47 |
| * dabo votes for Dublin | 21:47 | |
| eday | jaypipes: for the C release we shot it down due to religious implications | 21:47 |
| eday | diablo++ | 21:47 |
| jaypipes | eday: yes, I know. :) I was pulling your chain. | 21:47 |
| ttx | so let's count | 21:47 |
| zul | diablo is not religious? ;) | 21:47 |
| soren | eday: What's that? diablp? | 21:47 |
| dabo | eday: Diablo should go for the same reason | 21:47 |
| creiht | How about... wait for it... 1.0? :) | 21:47 |
| creiht | dabo++ | 21:48 |
| * jaypipes kickbans creiht. | 21:48 | |
| diegoparrilla | jaypipes: hi! | 21:48 |
| xtoddx | creiht: pffft. thats so 1990s | 21:48 |
| creiht | :) | 21:48 |
| xtoddx | creiht: did you just vote to name it dabo? | 21:48 |
| dabo | creiht: 1.0 doesn't start with 'D' | 21:48 |
| creiht | lol | 21:48 |
| creiht | doh1.0 | 21:48 |
| creiht | ! | 21:48 |
| dabo | Yeah, let's call it dabo!! :) | 21:48 |
| termie | http://www.google.com/imgres?imgurl=http://www.stanleysteamers.com/vapor-trails/saca-danville/danville1-x.JPG&imgrefurl=http://www.stanleysteamers.com/danville.htm&usg=__SDFY1sLWIcd9U-QADxdsJFF8cuo=&h=571&w=947&sz=82&hl=en&start=0&sig2=0R-fcWbp2eE9pJorsEjH6A&zoom=1&tbnid=tiE6zFbS-8gg9M:&tbnh=120&tbnw=199&ei=pwo2TZm7CJPWtQOStvC1BQ&prev=/images%3Fq%3Ddanville%26um%3D1%26hl%3Den%26safe%3Doff%26sa%3DN%26biw%3D1156%26bih%3D930%26tbs%3Disch:1 | 21:48 |
| vishy | deuce? | 21:48 |
| ttx | so it looks like it's Diablo vs. Dublin | 21:48 |
| vishy | we'll create a new city | 21:48 |
| ttx | with Diablo leading a bit, but having strong advocates against it | 21:49 |
| johnpur | Drytown | 21:49 |
| jaypipes | Dublin then is my vote...since it's more identifiable as a city.. | 21:49 |
| sandywalsh_ | Dumas (dumbass) | 21:49 |
| annegentle | Danville LOL. Just got it. I'm slow. | 21:49 |
| * creiht thinks it is funy that you drop Corpus Christi for religious reasons, but don't for diablo | 21:49 | |
| * vishy still thinks Dorris is incredible | 21:49 | |
| creiht | sandywalsh_: ++ | 21:49 |
| dabo | jaypipes: ...even if most won't know it's in CA | 21:49 |
| pvo | ttx: surveymonkey? | 21:50 |
| soren | http://www.googlefight.com/index.php?lang=en_GB&word1=diablo&word2=dublin | 21:50 |
| soren | :( | 21:50 |
| ttx | pvo: yeah, that dodn't exactly turned out to be the global consensus I hoped :) | 21:50 |
| eday | I say DIablo, and during the diablo summit, we take a field trip and hike mt diablo | 21:50 |
| soren | Of course we want the one with the *fewest* google hits for better ranking. :) | 21:50 |
| johnpur | Death Valley? | 21:51 |
| creiht | release del diablo | 21:51 |
| ttx | johnpur: rha | 21:51 |
| dabo | johnpur: yeah, that's the vibe we want! | 21:51 |
| vishy | soren: http://www.googlefight.com/index.php?lang=en_GB&word1=diablo&word2=dorris | 21:51 |
| xtoddx | johnpur: i like it, its edgy | 21:51 |
| sandywalsh_ | Dumas is just down the road from Cactus actually | 21:51 |
| ttx | dendrobates: do you have enough momentum for Diablo to decide it, or should we move to a more scientific voting technique ? | 21:52 |
| pvo | science. meh | 21:52 |
| dendrobates | I say put up a voting | 21:52 |
| pvo | yea, I do too | 21:52 |
| ttx | ok, I'll condorcet it | 21:52 |
| dendrobates | let everyone vote, not just those on IRC now. | 21:52 |
| termie | http://www.google.com/imgres?imgurl=http://www.cccprc.org/puppies/images/group/bowling_0106b.jpg&imgrefurl=http://www.cccprc.org/puppies/group-pictures.html&usg=__7KQUKlmiJU3PJ5yeT1eKvI8o_YA=&h=300&w=450&sz=42&hl=en&start=25&sig2=AfHh8A3S53s-EjiV7Y660g&zoom=1&tbnid=7cEhJQUS1fIdBM:&tbnh=141&tbnw=196&ei=wgs2TaT7NYz0swPav7DiBQ&prev=/images%3Fq%3Ddanville,%2Bca%26um%3D1%26hl%3Den%26safe%3Doff%26biw%3D1156%26bih%3D930%26tbs%3Disch:10,631&um= | 21:53 |
| termie | how can you vote against puppies? | 21:53 |
| eday | shoudl we really spend more time on this? setting up a vote and whatnot? dendrobates, just pick one :) | 21:53 |
| ttx | #action ttx to turn his failed IRC attempt to a real vote | 21:53 |
| ttx | eday++ | 21:53 |
| sandywalsh_ | so, top three choices are Diablo, Death Valley and Dublin? | 21:53 |
| creiht | anyone but diablo | 21:53 |
| creiht | :) | 21:53 |
| annegentle | Danville, Dumas, Death Valley, Dublin? | 21:54 |
| *** kpepple has quit IRC | 21:54 | |
| devcamcar | vote = dublin | 21:54 |
| termie | i can hook us up with some danville locals if we want to celebrate in danville | 21:54 |
| ttx | ok, dendrobates said "put up a vote", I'll do that unless someone else feels up to it | 21:54 |
| ewanmellor | Please not Death Valley. Two-word release names are really irritating. | 21:55 |
| ttx | #topic open discussion | 21:55 |
| creiht | So is it still in question on weather or not diablo is in the running? | 21:55 |
| ttx | ewanmellor: rules of the game say "single word, 10 char max" | 21:55 |
| eday | dendrobates: dublin or diablo, pick one :) | 21:55 |
| dendrobates | damn, lay off! | 21:55 |
| creiht | lol | 21:55 |
| eday | haha | 21:55 |
| * notmyname votes dublin | 21:55 | |
| creiht | dublin | 21:56 |
| xtoddx | yea, more dublin momentum, keep it coming! | 21:56 |
| * ttx makes mental note: Never discuss release naming on IRC again | 21:56 | |
| dendrobates | corpus christie was not officially rejected, so I see no reason that diablo should be | 21:56 |
| eday | funny thing is, ultimately, no one really cares ;) | 21:56 |
| ewanmellor | ttx: lol | 21:56 |
| xtoddx | tts: you should know how the internet works by now | 21:56 |
| jk0 | eday: ++ | 21:56 |
| ttx | eday: exactly | 21:56 |
| dendrobates | true | 21:56 |
| johnpur | Darwin? | 21:56 |
| xtoddx | johnpur: to apple | 21:56 |
| johnpur | lol | 21:56 |
| jaypipes | eday: exactly. the only release anyone ever cares about is the one that isn't released yet... | 21:57 |
| ttx | "Downey" seems a bit dangerous for a production release | 21:57 |
| creiht | Disney? | 21:57 |
| sandywalsh_ | hah | 21:57 |
| creiht | :) | 21:57 |
| xtoddx | ttx: it'll probably develop a drug habbit | 21:57 |
| johnpur | ooooh.... Dulzura? | 21:57 |
| ttx | "The Disneyland release" | 21:57 |
| * jaypipes picked a bad week to stop sniffing glue. | 21:57 | |
| * eday goes back to real work | 21:57 | |
| ttx | #endmeeting | 21:57 |
| openstack | Meeting ended Tue Jan 18 21:57:48 2011 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:57 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack-meeting/2011/openstack-meeting.2011-01-18-21.00.html | 21:57 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack-meeting/2011/openstack-meeting.2011-01-18-21.00.txt | 21:57 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack-meeting/2011/openstack-meeting.2011-01-18-21.00.log.html | 21:57 |
| pvo | thank you | 21:57 |
| jaypipes | And I'm spent. | 21:58 |
| ttx | Thanks everyone, keep them coming :) | 21:58 |
| johnpur | ha ha | 21:58 |
| creiht | Duarte | 21:58 |
| *** creiht has left #openstack-meeting | 21:58 | |
| *** jk0 has left #openstack-meeting | 21:58 | |
| *** eday has left #openstack-meeting | 21:58 | |
| *** ewanmellor has left #openstack-meeting | 21:59 | |
| sandywalsh_ | Devine | 21:59 |
| *** bcwaldon has left #openstack-meeting | 21:59 | |
| *** koji-iida has left #openstack-meeting | 21:59 | |
| *** henrichrubin has left #openstack-meeting | 21:59 | |
| vishy | oh doh I had one other thing that i had...taking it to openstack | 21:59 |
| *** groo has left #openstack-meeting | 22:00 | |
| *** westmaas has quit IRC | 22:03 | |
| soren | annegentle: Don't feel bad. I *still* don't see what's funny about "Danville". | 22:03 |
| *** dragondm has left #openstack-meeting | 22:03 | |
| annegentle | soren: it's funny to me because of watching Phineas and Ferb - that's where they live. Sorta like the Simpson's Springfield. | 22:04 |
| soren | annegentle: Oh! | 22:04 |
| ttx | Oh! | 22:04 |
| annegentle | soren: there's one in every state I suppose :) | 22:04 |
| soren | annegentle: I've only seen two or three episodes. :( Not enough for that sort of trivia to stick :) | 22:05 |
| soren | Our S release should totally be Springfield. | 22:05 |
| annegentle | heh, yeah :) | 22:05 |
| *** annegentle has left #openstack-meeting | 22:08 | |
| *** gholt has left #openstack-meeting | 22:08 | |
| *** dendrobates is now known as dendro-afk | 22:13 | |
| *** dendro-afk is now known as dendrobates | 22:13 | |
| *** colinnich has left #openstack-meeting | 22:20 | |
| *** masumotok has quit IRC | 23:02 | |
| *** johnpur has left #openstack-meeting | 23:24 | |
| *** diegoparrilla has left #openstack-meeting | 23:29 | |
| *** pvo is now known as pvo_away | 23:37 | |
| *** troytoman has quit IRC | 23:37 | |
| *** zul has quit IRC | 23:37 | |
| *** GasbaKid has quit IRC | 23:45 | |
| *** dendrobates is now known as dendro-afk | 23:48 | |
| *** enigma has left #openstack-meeting | 23:50 | |
| *** Tushar has quit IRC | 23:51 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
