Some errors in log configuration description in configuration reference-icehouse.

Bug #1291096 reported by Maithem
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
openstack-manuals
Fix Released
High
Gauvain Pocentek

Bug Description

When I set the value of log-dir (http://docs.openstack.org/trunk/config-reference/content/list-of-compute-config-options.html), it seems that the variable is not used at all. Although, setting logdir in nova.conf works for me.
===========================(Maithem reported above)=============================
Below is the real problems:
As oslo.config will name the Opt dest via change '-' to '_' in the Opt name.
So only configuration keys written by '_' is effective.
And in the the OpenStack Configuration Reference for Icehouse, there are some logging configuration page using "log-dir", "log-file", "log-formate" and so on.
It misleads users.

Song Li (lisong-cruise)
Changed in nova:
assignee: nobody → Song Li (lisong-cruise)
status: New → In Progress
Revision history for this message
Song Li (lisong-cruise) wrote :

I have reproduced the issue as well as some other cases.
"/usr/bin/python /usr/local/bin/nova-compute --config-file /etc/nova/nova.conf --log-file /var/log/nova" works well.
add "log_file=/var/log/nova" in nova.conf can still works well.
but "log-file=/var/log/nova" can not work.
I will investigate and fix it then.

Revision history for this message
Song Li (lisong-cruise) wrote :

I have browsed the documents of other projects, and they still use the configuration like "log_dir", "log_file" and so on.
Beside, the the sample configuration file on the source code, they use log_dir too.
So I think the document page mentioned should be updated.

Song Li (lisong-cruise)
affects: nova → openstack-manuals
Revision history for this message
Maithem (maithem) wrote :

Glance and Cinder use log_dir/file

Revision history for this message
Song Li (lisong-cruise) wrote :

Yes, we should use '_' in all configuration files, I have browsed the code, and find that only the configuration key using '_' could be effective. So I will change the document of Nova Project. Thanks

Song Li (lisong-cruise)
Changed in openstack-manuals:
status: In Progress → Confirmed
status: Confirmed → In Progress
Song Li (lisong-cruise)
Changed in openstack-manuals:
status: In Progress → Confirmed
Revision history for this message
Song Li (lisong-cruise) wrote :

Hi Mathem, I think the bug you reported is just a manual one, and I changed it to openstack-manual projects, and change the title of the bug.

Changed in openstack-manuals:
status: Confirmed → Invalid
status: Invalid → Confirmed
Song Li (lisong-cruise)
summary: - log-dir variable doesnt work?
+ Some errors in log configuration description in configuration reference-
+ icehouse.
Song Li (lisong-cruise)
description: updated
Revision history for this message
Song Li (lisong-cruise) wrote :

Need the contributors that familiar with the doc changing to improve.
Thanks

Changed in openstack-manuals:
assignee: Song Li (lisong-cruise) → nobody
assignee: nobody → Song Li (lisong-cruise)
Song Li (lisong-cruise)
Changed in openstack-manuals:
assignee: Song Li (lisong-cruise) → nobody
Revision history for this message
Gauvain Pocentek (gpocentek) wrote :

Thanks for the report.

These options are being removed from the projects, and we are in the process of updating the tables to reflect this change. We are having some discussion about how to properly handle all the tables so it might need a few days to get fix, but we're working on it.

Thanks.

Changed in openstack-manuals:
status: Confirmed → In Progress
assignee: nobody → Gauvain Pocentek (gpocentek)
Revision history for this message
Song Li (lisong-cruise) wrote :

Thanks very much Gauvain.

Changed in openstack-manuals:
milestone: none → icehouse
importance: Undecided → High
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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