Thanks for the merge proposal; I reworked it a bit as we don't want to drop the sql connection patch - and after a little thought and testing I think that checking for:
is quicker and more effective than what you proposed; this ensures that if someone comments out the default connection url provided in the packaging its not detected by mistake.
We should apply this consistently across the packages - raising tasks for cinder, nova and glance as well - neutron might not need this.
Corey
Thanks for the merge proposal; I reworked it a bit as we don't want to drop the sql connection patch - and after a little thought and testing I think that checking for:
grep -q "^connection. *sqlite. *" /etc/keystone/ keystone. conf
is quicker and more effective than what you proposed; this ensures that if someone comments out the default connection url provided in the packaging its not detected by mistake.
We should apply this consistently across the packages - raising tasks for cinder, nova and glance as well - neutron might not need this.