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

Fixing error in config for internal-telemetry.md #5672

Merged
merged 4 commits into from
Nov 27, 2024

Conversation

avilevy18
Copy link
Contributor

@avilevy18 avilevy18 commented Nov 22, 2024

Attempted to run collector-contrib with the example config posted on: https://opentelemetry.io/docs/collector/internal-telemetry/#self-monitoring

Found a few errors:

  1. Collector failed with config error:
error decoding 'service.telemetry': decoding failed due to the following error(s):

'traces.processors': source data must be an array or slice, got map
  • OTLP Exporter was trying to send with SSL, to receiver that was not set up with SSL.
warn	[email protected]/clientconn.go:1379	[core] [Channel #1 SubChannel #8]grpc: addrConn.createTransport failed to connect to {Addr: "127.0.0.1:14317", ServerName: "localhost:14317", }. Err: connection error: desc = "transport: authentication handshake failed: tls: first record does not look like a TLS handshake"	{"grpc_log": true}

Adding http:// before localhost in the exporters endpoint, enforces an insecure connection.

Attempted to use insecure: true option but nothing happened.

   traces:
      processors:
      - batch:
          exporter:
            otlp:
              protocol: grpc/protobuf
              endpoint: localhost:14317
              insecure: true

The insecure config options doesn't seem to do anything:
https://github.com/open-telemetry/opentelemetry-collector/blob/f74890a3889a82f57b839a8b316bb43b1e55f53e/service/telemetry/internal/otelinit/config.go#L216

- collector failed with config error:
```
error decoding 'service.telemetry': decoding failed due to the following error(s):

'traces.processors': source data must be an array or slice, got map
```

- OTLP Exporter was trying to send with SSL, to receiver that was not set up with SSL.
```
warn	[email protected]/clientconn.go:1379	[core] [Channel open-telemetry#1 SubChannel open-telemetry#8]grpc: addrConn.createTransport failed to connect to {Addr: "127.0.0.1:14317", ServerName: "localhost:14317", }. Err: connection error: desc = "transport: authentication handshake failed: tls: first record does not look like a TLS handshake"	{"grpc_log": true}
```
@avilevy18 avilevy18 requested a review from a team as a code owner November 22, 2024 22:09
Copy link

linux-foundation-easycla bot commented Nov 22, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: opentelemetrybot / name: OpenTelemetry Bot (3f2cbe3)
  • ✅ login: avilevy18 (1bbca3a, 4305956)
  • ✅ login: svrnm / name: Severin Neumann (e6913ed)

@opentelemetrybot opentelemetrybot requested review from a team and atoulme and removed request for a team November 22, 2024 22:10
@tiffany76
Copy link
Contributor

Hi @avilevy18, could you sign the CLA please?

@open-telemetry/collector-approvers, FYI.

Copy link
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@svrnm svrnm added the CLA: no label Nov 25, 2024
@opentelemetrybot opentelemetrybot requested a review from a team November 25, 2024 19:24
@avilevy18
Copy link
Contributor Author

Signed, sorry for the long wait

@tiffany76 tiffany76 removed the CLA: no label Nov 25, 2024
@tiffany76
Copy link
Contributor

/fix:format

@opentelemetrybot
Copy link
Collaborator

You triggered fix:format action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/12017747156

@opentelemetrybot
Copy link
Collaborator

fix:format was successful.

IMPORTANT: (RE-)RUN /fix:all to ensure that there are no remaining check issues.

@opentelemetrybot opentelemetrybot requested a review from a team November 27, 2024 08:35
@cartermp cartermp added this pull request to the merge queue Nov 27, 2024
Merged via the queue into open-telemetry:main with commit 9485be1 Nov 27, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

7 participants