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

Revision history for this message
Soren Hansen (soren) 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.

I believe we do. It's called "outside".

If people can check out trunk and find 7 different sets of deployment tools, none of which work, because none of them are kept up-to-date, everybody loses. The user wanting use install and run Nova doesn't get what they want, and we potentially lose a new user. If people want to get stuff into our trunk, they have to commit to keeping it up-to-date. Getting a nova-core person to vouch for that commitment guarantees that someone who actively follows what goes on in trunk every day looks after it.

A quick glance at this tool reveals that it's already out of date. It's still referencing a bunch of libvirt templates that no longer exist, and the one that remains isn't installed where this tool expects it to. I don't blame the author, because this project moves so fast, it's really, really hard to keep this stuff up-to-date.

« Back to merge proposal