Skip to content

Conversation

@tbauriedel
Copy link
Member

The current '/etc/corosync/service.d' path is not mandatory. Better use '/etc/corosync' instead

@tbauriedel tbauriedel self-assigned this Jan 10, 2023
Copy link
Member

@martialblog martialblog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beware, that in /etc/corosnyc there might be a authkey file with some secrets...

@tbauriedel
Copy link
Member Author

The specified path in relevantPaths is not collected. It is only checked if it exists.

@martialblog
Copy link
Member

well then, LGTM

@tbauriedel tbauriedel merged commit 89d97aa into main Jan 10, 2023
@tbauriedel tbauriedel deleted the corosync-adjust-relevant-paths branch January 10, 2023 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants