Code review comment for lp:~nii/nova/nova-deployment-tool

Revision history for this message
Vish Ishaya (vishvananda) wrote :

I don't really see a problem with it going into contrib. We don't
really have a good other location for items like this.

On Wednesday, January 12, 2011, Shigetoshi Yokoyama <email address hidden> wrote:
>> By accepting this into trunk, it's implied that it will be kept up-to-date,
>> given that people checking out trunk will actually be able to find it as part
>> of the checkout. I know that *I* will not be maintaining it. Unless you can
>> find a nova-core person that will volunteer to keep this up-to-date (or at the
>> very least pull it out once it becomes out-dated (which won't be long at the
>> pace this project moves)), I can't approve it. Sorry.
>>
>> Just to be clear, this has nothing to do with the quality of your work (I've
>> only glanced at it). This is about ensuring the future coherence of trunk.
>>
>> Disapproving for now, but if you can find a nova-core person that will promise
>> to keep it up-to-date, I'm very willing to reconsider.
>
> I appreciate that you navigate me, informatively. I think our contribution is
> close to the puppet tool which is put under the contrib directory by some of
> NASA members.
> I have been talking to Jesse and Todd on this deployment tool issues since my
> blueprint proposal. I suppose I should ask them to be the volunteer.
>
>
> --
> https://code.launchpad.net/~nii/nova/nova-deployment-tool/+merge/45335
> You are subscribed to branch lp:nova.
>

« Back to merge proposal