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.

Sign up for GitHub

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

Jump to bottom

[Examples] Change default otel collector for go #218

Merged
merged 1 commit into from
Oct 13, 2021
Merged

[Examples] Change default otel collector for go #218

merged 1 commit into from
Oct 13, 2021

Conversation

miry
Copy link
Collaborator

@miry miry commented Oct 12, 2021 •
edited
Loading

The default port changed for recent version of opentelemetry-collector.

https://github.com/open-telemetry/opentelemetry-collector/blob/5ee979a1bb0d0343e98913cbfb8bff92d43b8378/receiver/otlpreceiver/README.md#L28


    endpoint (default = 0.0.0.0:4317 for grpc protocol, 0.0.0.0:4318 http protocol): host:port to which the receiver is going to receive data. The valid syntax is described at https://github.com/grpc/grpc/blob/master/doc/naming.md.

Closes #217

@miry
[Examples] Change default otel collector for go
219fa7a 
The default port changed for recent version of opentelemtry-collector.
@miry miry self-assigned this Oct 12, 2021
@miry miry requested a review from lucacome October 12, 2021 08:41
@miry miry mentioned this pull request Oct 12, 2021
Closed
@miry miry merged commit 4d26e42 into master Oct 13, 2021
@miry miry deleted the 217-otel-port branch October 13, 2021 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhuizhuhaomeng zhuizhuhaomeng zhuizhuhaomeng approved these changes

@lucacome lucacome lucacome approved these changes

Assignees

@miry miry

Labels
documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

port 55680 is not listen by opentelemetry-collector in example/go/otel
3 participants
@miry @lucacome @zhuizhuhaomeng