Skip to content

feat: putting changes#91

Closed
silwyne wants to merge 1 commit into
ClickHouse:masterfrom
silwyne:develop
Closed

feat: putting changes#91
silwyne wants to merge 1 commit into
ClickHouse:masterfrom
silwyne:develop

Conversation

@silwyne

@silwyne silwyne commented Jun 3, 2025

Copy link
Copy Markdown

No description provided.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


smh_tabatabaei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Slach Slach left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

fix CI/CD failures and explain why your changes is needed?

Comment thread README.md
@@ -1,4 +1,4 @@
# Clickhouse Exporter for Prometheus (old clickhouse-server versions)
# Clickhouse Exporter for Prometheus

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

do you know about embedded prometheus endpoint in clickhouse-server?
look details https://clickhouse.com/docs/operations/server-configuration-parameters/settings#prometheus

this project was released for old clickhouse-server versions which not have prometheus endpoints

Comment thread docker-compose.yml
clickhouse:
image: clickhouse/clickhouse-server
clickhouse-exporter:
image: clickhouse-exporter:0.1.1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this image have no sense, we used clickhouse-server for testing

@silwyne silwyne closed this by deleting the head repository Jun 3, 2025
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