Skip to content

fix: add custom admin profile to clickhouse users#419

Closed
DecFox wants to merge 1 commit intomainfrom
fix/admin-profile
Closed

fix: add custom admin profile to clickhouse users#419
DecFox wants to merge 1 commit intomainfrom
fix/admin-profile

Conversation

@DecFox
Copy link
Copy Markdown
Contributor

@DecFox DecFox commented Apr 29, 2026

This diff adds a custom admin profile to clickhouse users.xml server, since we have an admin user entry but no profile for it

@github-actions
Copy link
Copy Markdown

Ansible Run Output 🤖

Ansible Playbook Recap 🔍



Ansible playbook output 📖success

Show Execution

$ ansible-playbook playbook.yml --check --diff -i ../tf/modules/ansible_inventory/inventories/inventory-dev.ini
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'
[ERROR]: the role 'geerlingguy.docker' was not found in /home/runner/work/devops/devops/ansible/roles:/home/runner/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/home/runner/work/devops/devops/ansible
Origin: /home/runner/work/devops/devops/ansible/deploy-testlists.yml:16:7

14         node_exporter_host: "0.0.0.0"
15         node_exporter_options: ""
16     - role: geerlingguy.docker
         ^ column 7

Pusher @DecFox
Action pull_request
Working Directory
Workflow .github/workflows/check_ansible.yml
Last updated Wed, 29 Apr 2026 14:55:10 GMT

@aagbsn
Copy link
Copy Markdown
Contributor

aagbsn commented Apr 29, 2026

should this go in group_vars?
https://github.com/idealista/clickhouse_role#users-and-roles
this also seems to distinguish between users in the database vs xml configs. I don't see an example for _xml for profiles, however:
https://github.com/idealista/clickhouse_role/blob/main/defaults/main.yml

@DecFox
Copy link
Copy Markdown
Contributor Author

DecFox commented Apr 29, 2026

So this is actually a no-op. I now see that the group vars already contain the notebook server and the vars configure the admin profile correctly. I am going to close this PR.

@DecFox DecFox closed this Apr 29, 2026
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