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

Kubernetes example config is using the old TLS config #4247

Closed
jpkrohling opened this issue Oct 22, 2021 · 6 comments
Closed

Kubernetes example config is using the old TLS config #4247

jpkrohling opened this issue Oct 22, 2021 · 6 comments
Assignees
Labels
bug Something isn't working easy Complexity: Easy good first issue Good for newcomers

Comments

@jpkrohling
Copy link
Member

The Kubernetes example config is currently using the old squashed TLS configuration, which causes pods to fail with:

2021-10-22T11:44:16.540Z	info	service/collector.go:303	Starting otelcol...	{"Version": "v0.33.0-50-g0594aa1a", "NumCPU": 6}
2021-10-22T11:44:16.540Z	warn	service/collector.go:312	`mem-ballast-size-mib` command line option has been deprecated. Please use `ballast extension` instead!
2021-10-22T11:44:16.540Z	info	service/collector.go:242	Loading configuration...
Error: cannot load configuration: error reading exporters configuration for otlp: 1 error(s) decoding:

* '' has invalid keys: tls
2021/10/22 11:44:16 collector server run finished with error: cannot load configuration: error reading exporters configuration for otlp: 1 error(s) decoding:

* '' has invalid keys: tls

Steps to reproduce:

@jpkrohling jpkrohling added bug Something isn't working good first issue Good for newcomers easy Complexity: Easy labels Oct 22, 2021
@shree007
Copy link
Contributor

May i proceed to work on it?

@jpkrohling
Copy link
Member Author

It's yours!

@jpkrohling
Copy link
Member Author

Sorry, my diagnosis was really wrong. I should have paid more attention to the error message. The actual problem is that the image being used by that example seem to be using an older code base, which still didn't have the TLS changes I mentioned earlier. The example has been updated to use the new TLS option.

So, the solution really is to use another more recent image, like otel/opentelemetry-collector:0.37.1 (or perhaps even latest) instead of otel/opentelemetry-collector-dev:latest.

You'll notice that it will fail due to a command line parameter that has been removed. Just change the command to remove it.

@shree007
Copy link
Contributor

shree007 commented Oct 27, 2021

Hey Juraci
No problem. Let me check once.

@shree007
Copy link
Contributor

Hi Juraci
So I have replaced the image from otel/opentelemetry-collector-dev:latest to otel/opentelemetry-collector:0.37.1
and removed the "--mem-ballast-size-mib=165" and "--mem-ballast-size-mib=683" based on following error msg
"mem-ballast-size-mib command line option has been deprecated." that i got while running.

Please review #4278

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working easy Complexity: Easy good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants