Typo in section header in config silently disables all config parsing
Bug #1412798 reported by
George Shuklin
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Glance |
Fix Released
|
Medium
|
Hassaan Pasha |
Bug Description
I know it sounds silly, but I just spend five hours trying to find why glance is not working with swift and printing random erros. At the end I had found it had ignored all debug/log settings, and later I had found the source of the problem - small typo in my config.
If config contains '[[DEFAULT]' instead of '[DEFAULT]' glance ignores all setting in section (i think it is not only for 'default', but 'default' is the most devastating, because it disables logging and logging locations).
Proposed solution: write down a warning to stdout if configuration file contains no '[DEFAULT]' section.
Changed in glance: | |
status: | New → Incomplete |
importance: | Undecided → Medium |
Changed in glance: | |
status: | New → Confirmed |
Changed in glance: | |
assignee: | nobody → Hassaan Pasha (hassaanp) |
Changed in glance: | |
status: | Confirmed → In Progress |
status: | In Progress → Fix Committed |
Changed in glance: | |
milestone: | none → liberty-1 |
status: | Fix Committed → Fix Released |
Changed in glance: | |
milestone: | liberty-1 → 11.0.0 |
To post a comment you must log in.
This doesn't actually work for me on master. On a plain devstack installation, replacing [DEFAULT] with [[DEFAULT] or [xDEFAULT] in /etc/glance/ glance- api.conf causes the service to fail to start. It complains it can't read a certain option from the config file:
2015-01-20 15:28:29.916 5741 TRACE glance BadDriverConfig uration: Driver glance. image_cache. drivers. sqlite could not be configured correctly. Reason: Failed to read image_cache_dir from config
Do you have some steps to reproduce?