Skip to content

Commit

Permalink
Merge pull request #63 from openinsight-proj/fix_openinsight_chart
Browse files Browse the repository at this point in the history
add more config for otel collector
  • Loading branch information
Frapschen authored Nov 30, 2022
2 parents ff5eba2 + 3d79948 commit 37cc8a0
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/openinsight/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ maintainers:
- name: JaredTan95
- name: Frapschen

version: 0.2.1
version: 0.2.2
appVersion: "0.0.3"

dependencies:
Expand Down
13 changes: 11 additions & 2 deletions charts/openinsight/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ clickhouse:
set -e
clickhouse client -u default --password='changeme' -n <<-EOSQL
CREATE DATABASE otel;
CREATE DATABASE openinsight;
EOSQL
opentelemetry-collector:
Expand All @@ -29,7 +29,7 @@ opentelemetry-collector:
# If you want to use the core image `otel/opentelemetry-collector`, you also need to change `command.name` value to `otelcol`.
repository: ghcr.m.daocloud.io/openinsight-proj/openinsight
pullPolicy: IfNotPresent
tag: "b773134c29942516795a65e463a8bcd0f664cf33"
tag: "4bcfe66f65b9edf15786eafa13418b8b744af19c"
presets:
# Configures the collector to collect logs.
# Adds the filelog receiver to the logs pipeline
Expand Down Expand Up @@ -139,6 +139,8 @@ opentelemetry-collector:
- ${MY_POD_IP}:8888
zipkin:
endpoint: 0.0.0.0:9411
fluentforward:
endpoint: 0.0.0.0:8006
service:
telemetry:
metrics:
Expand All @@ -157,6 +159,7 @@ opentelemetry-collector:
- batch
receivers:
- otlp
- fluentforward
metrics:
exporters:
- logging
Expand Down Expand Up @@ -234,6 +237,12 @@ opentelemetry-collector:
containerPort: 8888
servicePort: 8888
protocol: TCP
fluent-forward:
enabled: true
containerPort: 8006
servicePort: 8006
hostPort: 8006
protocol: TCP

# Resource limits & requests. Update according to your own use case as these values might be too low for a typical deployment.
resources:
Expand Down

0 comments on commit 37cc8a0

Please sign in to comment.