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

Rename OpenTelemetry Service to OpenTelemetry Collector #352

Closed
8 tasks done
tigrannajaryan opened this issue Sep 17, 2019 · 1 comment
Closed
8 tasks done

Rename OpenTelemetry Service to OpenTelemetry Collector #352

tigrannajaryan opened this issue Sep 17, 2019 · 1 comment
Assignees
@tigrannajaryan
Labels
help wanted Good issue for contributors to OpenTelemetry Service to pick up
Milestone
Alpha v0.2

Comments

@tigrannajaryan
Copy link
Member

tigrannajaryan commented Sep 17, 2019 •
edited by bogdandrutu
Loading

We heard several times that the name "Service" is confusing, is too generic and does reflect well the intent. After discussing with maintainers and other OpenTelemetry community members we decided to rename the "Service" to "Collector" since we believe it better reflects the purpose and is more distinct.

What needs to be done:

  • Rename all references to Service to Collector in the source code, documentation and examples.
  • Rename binary from otelsvc to otelcol.
  • Update documentation where it references to "Collector" as deployment mode to "Standalone" as a deployment mode.
  • Rename this opentelemetry-service repo to opentelemetry-collector.
  • Update source code imports to point to github.com/open-telemetry/opentelemetry-collector instead of github.com/open-telemetry/opentelemetry-service.
  • Announce publicly that repo was renamed so that 3rd party packages who depend on us can update the their dependencies.
  • Update dependencies in opentelemetry-service-contrib.
  • Rename repo opentelemetry-service-contrib to opentelemetry-collector-contrib.
tigrannajaryan pushed a commit to tigrannajaryan/opentelemetry-collector that referenced this issue Sep 17, 2019
Rename OpenTelemetry Service to Collector in docs and comments
291e5d5 
This begins implementation of open-telemetry#352
@tigrannajaryan tigrannajaryan mentioned this issue Sep 17, 2019
Merged
tigrannajaryan pushed a commit to tigrannajaryan/opentelemetry-collector that referenced this issue Sep 17, 2019
Rename OpenTelemetry Service to Collector in docs and comments
06e74e6 
This begins implementation of open-telemetry#352
tigrannajaryan added a commit that referenced this issue Sep 19, 2019
@tigrannajaryan
Rename OpenTelemetry Service to Collector in docs and comments (#354)
7609eaa 
This begins implementation of #352
@tigrannajaryan tigrannajaryan added the help wanted Good issue for contributors to OpenTelemetry Service to pick up label Sep 24, 2019
@tigrannajaryan tigrannajaryan modified the milestones: 0.1.0, Beta Release Sep 24, 2019
tigrannajaryan pushed a commit to tigrannajaryan/opentelemetry-collector that referenced this issue Sep 25, 2019
Rename otelsvc to otcol
5f40f01 
This renames the executable and Docker image.

This implements part of open-telemetry#352
tigrannajaryan pushed a commit to tigrannajaryan/opentelemetry-collector that referenced this issue Sep 25, 2019
Rename otelsvc to otcol
6385168 
This renames the executable and Docker image.

This implements part of open-telemetry#352

Testing done: make, docker-compose up
tigrannajaryan pushed a commit to tigrannajaryan/opentelemetry-collector that referenced this issue Sep 25, 2019
Rename otelsvc to otcol
8b1bd72 
This renames the executable and Docker image.

This implements part of open-telemetry#352

Testing done: make, docker-compose up
@tigrannajaryan tigrannajaryan mentioned this issue Sep 25, 2019
Merged
@songy23 songy23 mentioned this issue Sep 25, 2019
Merged
pjanotti pushed a commit that referenced this issue Sep 25, 2019
@tigrannajaryan
Rename otelsvc to otelcol (#365)
d038801 
* Rename otelsvc to otcol

This renames the executable and Docker image.

This implements part of #352

Testing done: make, docker-compose up

* PR fixes
tigrannajaryan pushed a commit to tigrannajaryan/opentelemetry-collector that referenced this issue Sep 27, 2019
Rename all github paths from opentelemtry-service to opentelemetry-co…
55f5cab 
…llector

This is part of renaming task: open-telemetry#352
@tigrannajaryan tigrannajaryan mentioned this issue Sep 27, 2019
Merged
tigrannajaryan pushed a commit to tigrannajaryan/opentelemetry-collector that referenced this issue Sep 27, 2019
Rename all github paths from opentelemtry-service to opentelemetry-co…
71432b0 
…llector

This is part of renaming task: open-telemetry#352
tigrannajaryan added a commit that referenced this issue Sep 27, 2019
@tigrannajaryan
Rename all github paths from opentelemtry-service to opentelemetry-co…
dc6b290 
…llector (#371)

This is part of renaming task: #352
@tigrannajaryan tigrannajaryan self-assigned this Sep 27, 2019
@pjanotti pjanotti mentioned this issue Oct 3, 2019
Closed
@pjanotti
Copy link
Contributor

pjanotti commented Oct 3, 2019

The last remaining task is tracked via an issue on the contrib repo: open-telemetry/opentelemetry-collector-contrib#23 for the collector itself all issues are done, closing this issue.

@pjanotti pjanotti closed this as completed Oct 3, 2019
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this issue Nov 11, 2021
@evantorrie @jmacd
Add Min() interface - rename MaxSumCount to MinMaxSumCount (open-tele…
fef504d 
…metry#352)

* Add Min() interface - rename MaxSumCount to MinMaxSumCount

Fixes open-telemetry/opentelemetry-go#319

* update stdout exporter to collect and output the minimum value
* update min and max atomically in Aggregator Update
* changed all references to maxsumcount to minmaxsumcount

* Address PR comments
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this issue Jul 5, 2024
@lalitb
fix compile time warnings (open-telemetry#352)
7826040
swiatekm pushed a commit to swiatekm/opentelemetry-collector that referenced this issue Oct 9, 2024
@Allex1 @dmitryax
[opentelemetry-operator] - bump operator/collector images (open-telem…
f4d49db 
…etry#352)

* [opentelemetry-operator] - bump operator/collector images

* bump 0.59

Co-authored-by: Dmitrii Anoshin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tigrannajaryan tigrannajaryan

Labels
help wanted Good issue for contributors to OpenTelemetry Service to pick up
Projects
None yet
Milestone
Alpha v0.2
Development

No branches or pull requests
2 participants
@tigrannajaryan @pjanotti