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

Fix dicts.j2 template #62

Merged
merged 1 commit into from
Sep 3, 2021
Merged

Fix dicts.j2 template #62

merged 1 commit into from
Sep 3, 2021

Conversation

sysolyakin
Copy link
Contributor

@sysolyakin sysolyakin commented Aug 17, 2021

Hello!

I've tried to deploy the role with a list of dictionaries in clickhouse_dicts variable and got this error:

2021.08.17 14:32:32.316330 [ 15692 ] {} ExternalDictionariesLoader: Failed to load config file '/etc/clickhouse-server/auto_dictionary.xml': Poco::Exception. Code: 1000, e.code() = 0, e.displayText() = Exception: Failed to merge config with '/etc/clickhouse-server/conf.d/clickhouse_remote_servers.xml': Exception: Root element doesn't have the corresponding root element as the config file. It must be , Stack trace (when copying this message, always include the lines below):

According to documentation it must use tags <yandex></yandex> instead of <dictionaries>.

So, could you merge this PR, please?

@AlexeySetevoi AlexeySetevoi merged commit ad0452b into AlexeySetevoi:master Sep 3, 2021
@sysolyakin sysolyakin deleted the fix-dicts-template branch September 3, 2021 14:52
AntonFriberg pushed a commit to AntonFriberg/ansible-clickhouse that referenced this pull request Mar 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants