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

Handle deletion of stale configuration files #29

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

korowa
Copy link
Collaborator

@korowa korowa commented Nov 5, 2024

Now at pre-configure stage, for each directory managed by the role, a task sequence is launched that lists checks for any difference between actual directory contents and expected list of files, and deletes any files not present in expected list.

The behavior is controlled by clickhouse_delete_stale_files role variable, and enabled by default.

Tested via emulation of stale config files presence (side-effect playbook) and additional run of the role on top of it.

Closes #22

Now at pre-configure stage, for each directory managed by the role,
a task sequence is launched that lists checks for any difference
between actual directory contents and expected list of files, and
deletes any files not present in expected list.

The behavior is controlled by `clickhouse_delete_stale_files` role
variable, and enabled by default.

Tested via emulation of stale config files presence (side-effect
playbook) and additional run of the role on top of it.
@azat azat self-assigned this Nov 5, 2024
Copy link
Collaborator

@azat azat left a comment

Choose a reason for hiding this comment

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

Very decent, thank you!

@azat azat merged commit ab439e4 into semrush:main Nov 5, 2024
6 checks passed
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.

Unmanaged configuration files don't get deleted from host
2 participants