Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
icinga-migration opened this issue Jan 5, 2016 · 9 comments
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Jan 5, 2016

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:

@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Jan 13, 2016

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
Copy link
Member Author

@icinga-migration icinga-migration commented Jan 14, 2016

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
Copy link
Member Author

@icinga-migration icinga-migration commented Jan 20, 2016

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

  • Status changed from Feedback to Closed

Works correctly.

@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Jan 26, 2016

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
Copy link
Member Author

@icinga-migration icinga-migration commented Jan 26, 2016

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

  • Target Version set to 2.4.2
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Jan 26, 2016

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
Copy link
Member Author

@icinga-migration icinga-migration commented Feb 22, 2016

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

  • Relates set to 10653
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Feb 23, 2016

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

  • Backport? changed from Not yet backported to Already backported
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Feb 23, 2016

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.