Code review comment for lp:~jaypipes/glance/use-paste-deploy

Revision history for this message
Rick Harris (rconradharris) wrote :

Nice, it's working for me now.

Small tweaks/suggestions:

* setup.py will need glance-control

* Looks like get_config_file_options is now only referenced in glance-control and tests; it *looks* like we may be able to remove it from glance-control as well as the tests, meaning we could cleanup common/config somewhat. Thoughts?

* Was mentioned earlier as a femto-nit, do we want to change cnf -> conf to match Nova and Swift?

review: Needs Fixing

« Back to merge proposal