cinder-manage ignores --sql_connection parameter

[Martin Mágr]

cinder-manage ignores --sql_connection parameter and uses some default. Not sure from where, because this option is commented in cinder.conf:
[para@virtual-rhel packstack]$ sudo cat /etc/cinder/cinder.conf | grep connection
#connection_type=<None>
#sql_connection=mysql://cinder:cinder@localhost/cinder
#sql_connection_debug=0

I've added "print 'used connection: %s' % FLAGS.sql_connection" to cinder/db/sqlalchemy/session.py:113 and output is following:
[para@virtual-rhel packstack]$ sudo cinder-manage --sql_connection=mysql://cinder:66ff39612f704b01@192.168.122.71/cinder db sync
used connection: mysql://cinder:cinder@localhost/cinder
2013-05-02 11:57:48 10457 WARNING cinder.db.sqlalchemy.session [-] SQL connection failed. 10 attempts left.

[Vincent Hou (houshengbo)]

For Glance, it is the same.
--sql_connection is a valid parameter, but it does not work.

[Eric Harney (eharney)]

Currently being worked on at:
https://bugs.launchpad.net/oslo/+bug/1176817
https://review.openstack.org/#/c/28405/

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/29483

[Vincent Hou (houshengbo)]

After I double checked, this bug is still effect, even if the oslo bug (https://bugs.launchpad.net/oslo/+bug/1176817) is resolved, because many parts of cinder are still using FLAGS and the sql_connection should be imported from the cinder config.

[Huang Zhiteng (zhiteng-huang)]

This bug will be fixed once Olso bug is fixed: https://review.openstack.org/#/c/29459/