Skip to content

Conversation

@execreate
Copy link
Contributor

Summary

ClickHouse ingestion user needs CREATE DATABASE permissions, because on startup collector tries to execute CREATE DATABASE IF NOT EXISTS. Not having CREATE DATABASE permissions will cause the collector to fail on startup.

Checklist

ClickHouse ingestion user needs `CREATE DATABASE` permissions, because on startup collector tries to execute `CREATE DATABASE IF NOT EXISTS`. Not having `CREATE DATABASE` permissions will cause the collector to fail on startup.
@execreate execreate requested a review from a team as a code owner November 19, 2025 07:58
@vercel
Copy link

vercel bot commented Nov 19, 2025

@execreate is attempting to deploy a commit to the ClickHouse Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Nov 19, 2025

CLA assistant check
All committers have signed the CLA.

@execreate
Copy link
Contributor Author

I am not sure what is the correct way to go here:

  1. updating docs to have the correct grants
  2. or updating the collector logic so that it does not try to execute CREATE DATABASE IF NOT EXISTS

Creating a PR to update the docs is easier, so here we are :) but I'd personally prefer the second option. What do you think? I can submit a PR for the second option later when I have a bit more time.

@vercel
Copy link

vercel bot commented Nov 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
clickhouse-docs Ready Ready Preview Nov 19, 2025 2:48pm
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
clickhouse-docs-ru Ignored Ignored Preview Nov 19, 2025 2:48pm
clickhouse-docs-zh Ignored Ignored Preview Nov 19, 2025 2:48pm

@gingerwizard
Copy link
Collaborator

(1) @execreate for now.

although (2) ill raise with the dev team

@gingerwizard gingerwizard merged commit a176f9e into ClickHouse:main Nov 19, 2025
14 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.

3 participants