Unable to disable log_dir by setting to false
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
puppet-ceilometer |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-congress |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-designate |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-glance |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-heat |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-ironic |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-keystone |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-manila |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-murano |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-nova |
Fix Released
|
High
|
Nikita Karpin | ||
puppet-octavia |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-oslo |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-panko |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-sahara |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-trove |
Fix Released
|
Undecided
|
Nikita Karpin | ||
puppet-watcher |
Fix Released
|
Undecided
|
Nikita Karpin |
Bug Description
Commit 872b1774 broke the ability to set log_dir to false in oslo::log in order to disable logging to a file.
The following code was removed from manifests/init.pp and should have been added to manifests/
- if $log_dir {
- file { $log_dir:
- ensure => directory,
- mode => '0750',
- owner => 'nova',
- group => $::nova:
- require => Package[
- }
- nova_config { 'DEFAULT/log_dir': value => $log_dir;}
- } else {
- nova_config { 'DEFAULT/log_dir': ensure => absent;}
- }
The end result is that setting log_dir to false puts "log_dir=False" in nova.conf, which is interpreted as a directory by the nova processes resulting a failure on startup.
Changed in puppet-nova: | |
status: | New → Triaged |
importance: | Undecided → High |
Changed in puppet-nova: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-congress: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-ceilometer: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-heat: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-oslo: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-panko: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-glance: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-ironic: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-manila: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-designate: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-murano: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-trove: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-watcher: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-keystone: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-sahara: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-octavia: | |
assignee: | nobody → Nikita Karpin (mkarpin) |
Changed in puppet-ceilometer: | |
status: | In Progress → Fix Committed |
Changed in puppet-glance: | |
status: | In Progress → Fix Committed |
Changed in puppet-congress: | |
status: | In Progress → Fix Committed |
Changed in puppet-heat: | |
status: | In Progress → Fix Committed |
Changed in puppet-ironic: | |
status: | In Progress → Fix Committed |
Changed in puppet-keystone: | |
status: | In Progress → Fix Committed |
Changed in puppet-manila: | |
status: | In Progress → Fix Committed |
Changed in puppet-murano: | |
status: | In Progress → Fix Committed |
Changed in puppet-nova: | |
status: | In Progress → Fix Committed |
Changed in puppet-octavia: | |
status: | In Progress → Fix Committed |
Changed in puppet-oslo: | |
status: | In Progress → Fix Committed |
Changed in puppet-panko: | |
status: | In Progress → Fix Committed |
Changed in puppet-sahara: | |
status: | In Progress → Fix Committed |
Changed in puppet-trove: | |
status: | In Progress → Fix Committed |
Changed in puppet-watcher: | |
status: | In Progress → Fix Committed |
Changed in puppet-glance: | |
status: | Fix Committed → Fix Released |
Changed in puppet-keystone: | |
status: | Fix Committed → Fix Released |
Changed in puppet-nova: | |
status: | Fix Committed → Fix Released |
Changed in puppet-ceilometer: | |
status: | Fix Committed → Fix Released |
Changed in puppet-congress: | |
status: | Fix Committed → Fix Released |
Changed in puppet-heat: | |
status: | Fix Committed → Fix Released |
Changed in puppet-ironic: | |
status: | Fix Committed → Fix Released |
Changed in puppet-manila: | |
status: | Fix Committed → Fix Released |
Changed in puppet-murano: | |
status: | Fix Committed → Fix Released |
Changed in puppet-octavia: | |
status: | Fix Committed → Fix Released |
Changed in puppet-oslo: | |
status: | Fix Committed → Fix Released |
Changed in puppet-panko: | |
status: | Fix Committed → Fix Released |
Changed in puppet-sahara: | |
status: | Fix Committed → Fix Released |
Changed in puppet-trove: | |
status: | Fix Committed → Fix Released |
Changed in puppet-watcher: | |
status: | Fix Committed → Fix Released |
Changed in puppet-designate: | |
status: | In Progress → Fix Released |
Fix proposed to branch: master /review. openstack. org/382344
Review: https:/