[dev.icinga.com #12098] include_recursive should gracefully handle inaccessible files

[icinga-migration]

This issue has been migrated from Redmine: https://dev.icinga.com/issues/12098

Created by gbeutner on 2016-07-05 10:51:49 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2016-07-05 13:45:09 +00:00)
Target Version: 2.5.0
Last Update: 2016-07-05 13:45:09 +00:00 (in Redmine)

Icinga Version: 2.4.10
Backport?: Not yet backported
Include in Changelog: 1

---

include_recursive should log a warning for files which are not readable (e.g. because they were deleted).

Changesets

2016-07-05 13:40:49 +00:00 by mfriedrich 5836b5b

Fix: include_recursive should gracefully handle inaccessible files

fixes #12098

[icinga-migration]

Updated by mfriedrich on 2016-07-05 13:26:27 +00:00

• Status changed from New to Assigned

  mbmif /usr/local/icinga2/etc/icinga2/conf.d (master) # chmod 600 users.conf
  mbmif /usr/local/icinga2/etc/icinga2/conf.d (master) # icinga2 daemon -C
  information/cli: Icinga application loader (version: v2.4.10-513-g63f83e8; debug)
  information/cli: Loading configuration file(s).
  warning/ConfigCompiler: Cannot compile file '/usr/local/icinga2/etc/icinga2/conf.d/users.conf': Error: Function call 'std::ifstream::open' for file '/usr/local/icinga2/etc/icinga2/conf.d/users.conf' failed with error code 13, 'Permission denied'

  information/ConfigItem: Committing config item(s).
  information/ApiListener: My API identity: mbmif.int.netways.de
  critical/config: Error: Validation failed for object 'passive!passive' of type 'Notification'; Attribute 'users': Object 'icingaadmin' of type 'User' does not exist.
  Location: in /usr/local/icinga2/etc/icinga2/tests/passive.conf: 14:3-14:27
  /usr/local/icinga2/etc/icinga2/tests/passive.conf(12): object Notification "passive" {
  /usr/local/icinga2/etc/icinga2/tests/passive.conf(13): host_name = "passive"
  /usr/local/icinga2/etc/icinga2/tests/passive.conf(14): users = [ "icingaadmin" ]
  ^^^^^^^^^^^^^^^^^^^^^^^^^
  /usr/local/icinga2/etc/icinga2/tests/passive.conf(15): command = "none"
  /usr/local/icinga2/etc/icinga2/tests/passive.conf(16):

  critical/config: 1 error

[icinga-migration]

Updated by mfriedrich on 2016-07-05 13:45:09 +00:00

• Status changed from Assigned to Resolved
• Done % changed from 0 to 100

Applied in changeset 5836b5b.