Conversation
🦋 Changeset detectedLatest commit: 37a752e The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
wrn14897
left a comment
wrn14897
left a comment
There was a problem hiding this comment.
We need to test out the new build e2e locally first
wrn14897
left a comment
wrn14897
left a comment
There was a problem hiding this comment.
Tested new version E2E
charts/clickstack/values.yaml
Outdated
| clickhousePrometheusEndpoint: | ||
| # Clickhouse database to send logs/traces/metrics to. Defaults to "default" | ||
| clickhouseDatabase: "default" | ||
| # Whether to create legacy schema in Clickhouse. Defaults to "true" |
There was a problem hiding this comment.
just one nit here: what do we mean by legacy? Can we add a little bit more color for the average user? Thanks!
There was a problem hiding this comment.
This means the collector will fall back to letting the clickhouse-exporter create the schema instead of using the optimized used by ClickStack. Let me update the comment
|
My local testing on this new chart was successful. Things look good and I was able to upgrade appropriately! |
4 participants
This PR updates the appVersion in Chart.yaml to
2.19.0.charts/clickstack/Chart.yaml