[database] connection is not present in glance-api.conf - icehouse

Bug #1305764 reported by Artem Rozumenko
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
openstack-manuals
Fix Released
High
Tom Fifield

Bug Description

in /etc/glance/glance-api.conf there is section:

# SQLAlchemy connection string for the reference implementation
# registry server. Any valid SQLAlchemy connection string is fine.
# See: http://www.sqlalchemy.org/docs/05/reference/sqlalchemy/connections.html#$
sql_connection = mysql://glance:password@controller/glance

But in doc is mentioned:
[database]
connection = mysql://glance:GLANCE_DBPASS@controller/glance

and there are not [database] section at all:
root@controller:~# grep database /etc/glance/glance-api.conf
# to the database.
# Whether to automatically create the database tables.
# Enable DEBUG log messages from sqlalchemy which prints every database
# If your database tables do not have charset utf8 you will
# only relevant if your database engine is MySQL.

-----------------------------------
Built: 2014-04-10T08:38:58 00:00
git SHA: 5a2c0eaacddb47ef7bc6f937fdb80f6feaa271a7
URL: http://docs.openstack.org/trunk/install-guide/install/apt/content/glance-install.html
source File: file:/home/jenkins/workspace/openstack-install-deploy-guide-ubuntu/doc/install-guide/section_glance-install.xml
xml:id: glance-install

Tags: manuals
Revision history for this message
ZhiQiang Fan (aji-zqfan) wrote :

you can read this: https://github.com/openstack/glance/blob/master/glance/openstack/common/db/options.py#L32

the problem is that, glance project has not update its sample config file about this config option, and the old option is deprecated for a long time, but it still works for such a long time too, even thoug it is not recommended any more.

usually, a deprecated thing will not be supported in two release cycle, which means this option may not be supported in Juno release

my advice is following the manual's guide, and use the new style

Changed in openstack-manuals:
status: New → Incomplete
Revision history for this message
Artem Rozumenko (artem-rozumenko) wrote :

@ZhiQiang,
Please advice how the fact of adding will work for deprecated and not commented option in here https://github.com/openstack/glance/blob/master/etc/glance-api.conf

Should be it at least commented or ignored or mentioned in docs to comment?

Revision history for this message
Andreas Jaeger (jaegerandi) wrote :

Artem, this is currently discussed here:

http://lists.openstack.org/pipermail/openstack-dev/2014-April/031612.html

The documentation team is waiting on whether glance will update its configuration for the final Icehouse release or stay as is - and then document accordingly.

Changed in openstack-manuals:
status: Incomplete → Confirmed
importance: Undecided → High
assignee: nobody → Tom Fifield (fifieldt)
Revision history for this message
Gauvain Pocentek (gpocentek) wrote :

A patch to regenerate the config file is proposed here: https://review.openstack.org/#/c/83327

Revision history for this message
Tom Fifield (fifieldt) wrote :
Revision history for this message
Tom Fifield (fifieldt) wrote :
Changed in openstack-manuals:
status: Confirmed → In Progress
Revision history for this message
Andreas Jaeger (jaegerandi) wrote :

fixed now

Changed in openstack-manuals:
status: In Progress → Fix Released
milestone: none → icehouse
Revision history for this message
Matt Kassawara (ionosphere80) wrote :

Tom,

Does the presence of this line in the new default configuration files impact anything?

[database]
sqlite_db = /var/lib/glance/glance.sqlite

Revision history for this message
Tom Fifield (fifieldt) wrote :

sqlite_db by itself is fine, I believe.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.