[dev.icinga.com #10932] Cluster config sync: Ensure that /var/lib/icinga2/api/zones/* exists

[icinga-migration]

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

Created by purplecarrot on 2016-01-05 13:31:37 +00:00

Assignee: gbeutner
Status: Resolved (closed on 2016-01-26 09:50:03 +00:00)
Target Version: 2.4.2
Last Update: 2016-02-23 10:55:38 +00:00 (in Redmine)

Icinga Version: 2.4.1
Backport?: Already backported
Include in Changelog: 1

---

As requested by Tobias, I'm opening this bug because the logs do not show any error message when this directory doesn't exist.

Changesets

2016-01-26 09:45:03 +00:00 by (unknown) 1eecc72

Use MkDirP instead of MkDir when syncing zone configs

fixes #10932

2016-02-23 08:28:37 +00:00 by (unknown) 5c23beb

Use MkDirP instead of MkDir when syncing zone configs

fixes #10932

---

Relations:

• relates #10653

[icinga-migration]

Updated by mfriedrich on 2016-01-13 16:30:22 +00:00

• Category set to Cluster
• Status changed from New to Feedback
• Assigned to set to purplecarrot

Can you please provide a little more context an when and why this happens? There already is error logging when mkdir() fails e.g. on incorrect permissions.

[icinga-migration]

Updated by tobiasvdk on 2016-01-14 07:24:33 +00:00

I tested it myself. As dnsmichi already mentioned there is an error message on the satellite:

[2016-01-14 07:22:01 +0000] warning/JsonRpcConnection: Error while processing message for identity 'my-master'
Error: Function call 'mkdir' for file '/var/lib/icinga2/api/zones/my-zone' failed with error code 2, 'No such file or directory'

[icinga-migration]

Updated by tobiasvdk on 2016-01-20 16:06:47 +00:00

• Status changed from Feedback to Closed

Works correctly.

[icinga-migration]

Updated by gbeutner on 2016-01-26 09:43:25 +00:00

• Status changed from Closed to Assigned
• Assigned to changed from purplecarrot to gbeutner

Er, surely we could just call MkDirP for this.

[icinga-migration]

Updated by gbeutner on 2016-01-26 09:45:37 +00:00

• Target Version set to 2.4.2

[icinga-migration]

Updated by Anonymous on 2016-01-26 09:50:03 +00:00

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

Applied in changeset 1eecc72.

[icinga-migration]

Updated by mfriedrich on 2016-02-22 15:39:12 +00:00

• Relates set to 10653

[icinga-migration]

Updated by gbeutner on 2016-02-23 09:58:37 +00:00

• Backport? changed from Not yet backported to Already backported

[icinga-migration]

Updated by mfriedrich on 2016-02-23 10:55:38 +00:00

• Subject changed from Add an error message when /var/lib/icinga2/api/zones/ doesn't exist to Cluster config sync: Ensure that /var/lib/icinga2/api/zones/* exists