Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

Override configuration and user settings instead of replacing #72

Merged
merged 2 commits into from
Jan 12, 2022
Merged

Override configuration and user settings instead of replacing #72

merged 2 commits into from
Jan 12, 2022

Conversation

AntonFriberg
Copy link
Contributor

Solves #70

@AntonFriberg
Copy link
Contributor Author

Note that this change is probably not backward compatible since the default config.xml and users.xml would have been replaced by previous deployments. Not sure either if conf.d or config.d is applied first. I tried it by having a local vagrant environment that I could deploy a fresh deployment on.

@AntonFriberg
Change to override instead of replace for user config
651b59f 
Solves AlexeySetevoi#70

Change-Id: I2229aed59c09b426e8f1d33d214aa7e1dc3f0c22
@AntonFriberg
Fix paths and config_files in molecule tests
33cd661 
Change-Id: I7b6198fe4e1fb5ad055fe7f67e3d253449e5be17
@AlexeySetevoi AlexeySetevoi merged commit 411e76f into AlexeySetevoi:master Jan 12, 2022
AntonFriberg added a commit to AntonFriberg/ansible-clickhouse that referenced this pull request Mar 25, 2022
@AntonFriberg
Override configuration and user settings instead of replacing (Alexey…
d6ed110 
…Setevoi#72)

* Change to override instead of replace for user config

Solves AlexeySetevoi#70

Change-Id: I2229aed59c09b426e8f1d33d214aa7e1dc3f0c22

* Fix paths and config_files in molecule tests

Change-Id: I7b6198fe4e1fb5ad055fe7f67e3d253449e5be17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AntonFriberg @AlexeySetevoi