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

Error creating processor "memory_limiter" on clean install #34

Closed
andrzej-stencel opened this issue Mar 9, 2021 · 1 comment · Fixed by #39
Closed

Error creating processor "memory_limiter" on clean install #34

andrzej-stencel opened this issue Mar 9, 2021 · 1 comment · Fixed by #39
Assignees

Comments

@andrzej-stencel
Copy link
Member

Helm chart: opentelemetry-collector-0.4.3 (latest at the time of writing)

When installing the chart with:

helm install otc open-telemetry/opentelemetry-collector

the installation succeeds, but the OTC pod crashes with the following error:

Error: cannot setup pipelines: cannot build pipelines: error creating processor "memory_limiter" in pipeline "metrics": checkInterval must be greater than zero
See full OTC pod logs
2021-03-09T12:37:54.144Z	info	service/service.go:411	Starting OpenTelemetry Collector...	{"Version": "v0.20.0", "GitHash": "a10a1a7a", "NumCPU": 8}
2021-03-09T12:37:54.147Z	info	service/service.go:592	Using memory ballast	{"MiBs": 204}
2021-03-09T12:37:54.147Z	info	service/service.go:255	Setting up own telemetry...
2021-03-09T12:37:54.148Z	info	service/telemetry.go:102	Serving Prometheus metrics	{"address": "0.0.0.0:8888", "level": 0, "service.instance.id": "49b6c5c0-07a7-41c2-9a92-b5e01a14b86b"}
2021-03-09T12:37:54.149Z	info	service/service.go:292	Loading configuration...
2021-03-09T12:37:54.149Z	info	service/service.go:303	Applying configuration...
2021-03-09T12:37:54.149Z	info	service/service.go:324	Starting extensions...
2021-03-09T12:37:54.149Z	info	builder/extensions_builder.go:53	Extension is starting...	{"component_kind": "extension", "component_type": "health_check", "component_name": "health_check"}
2021-03-09T12:37:54.149Z	info	healthcheckextension/healthcheckextension.go:40	Starting health_check extension	{"component_kind": "extension", "component_type": "health_check", "component_name": "health_check", "config": {"TypeVal":"health_check","NameVal":"health_check","Port":13133}}
2021-03-09T12:37:54.149Z	info	builder/extensions_builder.go:59	Extension started.	{"component_kind": "extension", "component_type": "health_check", "component_name": "health_check"}
2021-03-09T12:37:54.149Z	info	builder/exporters_builder.go:306	Exporter is enabled.	{"component_kind": "exporter", "exporter": "logging"}
2021-03-09T12:37:54.150Z	info	service/service.go:339	Starting exporters...
2021-03-09T12:37:54.150Z	info	builder/exporters_builder.go:92	Exporter is starting...	{"component_kind": "exporter", "component_type": "logging", "component_name": "logging"}
2021-03-09T12:37:54.150Z	info	builder/exporters_builder.go:97	Exporter started.	{"component_kind": "exporter", "component_type": "logging", "component_name": "logging"}
Error: cannot setup pipelines: cannot build pipelines: error creating processor "memory_limiter" in pipeline "metrics": checkInterval must be greater than zero
2021/03/09 12:37:54 application run finished with error: cannot setup pipelines: cannot build pipelines: error creating processor "memory_limiter" in pipeline "metrics": checkInterval must be greater than zero

Here's the relevant part of the configmap, as installed by Helm:

kubectl describe configmap otc-opentelemetry-collector-agent
# ...
processors:
    batch: {}
    memory_limiter: null
# ...
@dmitryax dmitryax self-assigned this Mar 22, 2021
@dmitryax
Copy link
Member

Can reproduce on the newest helm version staring from 3.5.0. The chart works as expected on helm version 3.4.2 and earlier.

I'm working on a fix...

matej-g added a commit to matej-g/opentelemetry-helm-charts that referenced this issue Oct 20, 2023
* Add hostNetwork

Signed-off-by: Matej Gera <[email protected]>

* Bump chart version

Signed-off-by: Matej Gera <[email protected]>

---------

Signed-off-by: Matej Gera <[email protected]>
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 a pull request may close this issue.

2 participants