Code review comment for lp:~usrleon/nova/small_fixes
Revision history for this message
| Vish Ishaya (vishvananda) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
I agree in general with Jay's point. You can always specify the file on the command line as well.
The LDAP change seems fine, since match seems to have that weird issue where 'California' != 'California' for some reason.
Vish
On Apr 26, 2011, at 7:36 AM, Jay Pipes wrote:
> Hi!
>
> I'm not sure that I agree with changing the default location of nova.conf from the current working directory to /etc/nova. Generally, I think these kind of things belong more to the realm of packaging or configuration management, and specifying a location other than the current working directory assumes a certain setup. Granted, /etc/nova is standard deployment for configuration files, but I still think we should leave that up to the configuration management tool and/or packaging scripts...
>
> The LDAP change I will politely defer to Ryan Lane or someone other LDAP expert (not me!) :)
>
> Thoughts?
> --
> https:/
> You are subscribed to branch lp:nova.