Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide option to disable clickhouse-instance.yaml #427

Open
somnusfish opened this issue Apr 11, 2024 · 5 comments
Open

Provide option to disable clickhouse-instance.yaml #427

somnusfish opened this issue Apr 11, 2024 · 5 comments
Labels
chart:clickhouse Issue related to clickhouse helm chart question Further information is requested

Comments

@somnusfish
Copy link

Hi,

I use signoz chart as a dependency for my build. I need to do some customization (set up different s3 endpoint for different namespace) in clickhouse-instance.yaml when using signoz chart. Right now I have to fork the whole clickhouse chart locally because ClickHouseInstallation got some unexpected behavior when deployed twice using override , could you please provide an option to disable the clickhouse-instance.yaml so I can override it easily?

@grandwizard28
Copy link
Contributor

Hi @somnusfish,
Can you explain with a snippet what would you like?

@grandwizard28 grandwizard28 added chart:clickhouse Issue related to clickhouse helm chart question Further information is requested labels Sep 19, 2024
@garbelini
Copy link

This would also help me since we already have a Clickhouse operator installed in the cluster and the SigNoz chart would conflict with that

@grandwizard28
Copy link
Contributor

Hi @garbelini,
Would it be possible to provide a short snippet or explain the ask in detail?

@garbelini
Copy link

@grandwizard28 The signoz helm installs, through the clickhouse chart, the CRDs for the Altinity clickhouse operator.

Since I already have that operator and the same CRDs installed on my cluster I can't use the signoz chart since, besides version conflicts, helm will not allow me to maintain the same CRDs through 2 separate charts.

@grandwizard28
Copy link
Contributor

@garbelini does using clickhouse.enabled = false not serve your use-case when using the SigNoz chart?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart:clickhouse Issue related to clickhouse helm chart question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants