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

[processor/filterprocessor] Migrate from opencensus library #30734

Closed
Tracked by #29867
codeboten opened this issue Jan 23, 2024 · 1 comment · Fixed by #30736
Closed
Tracked by #29867

[processor/filterprocessor] Migrate from opencensus library #30734

codeboten opened this issue Jan 23, 2024 · 1 comment · Fixed by #30736
Assignees
@codeboten
Labels
help wanted Extra attention is needed priority:p2 Medium processor/filter Filter processor

Comments

@codeboten
Copy link
Contributor

No description provided.

@codeboten codeboten mentioned this issue Jan 23, 2024
Closed
16 tasks
@codeboten codeboten self-assigned this Jan 23, 2024
@codeboten codeboten changed the title processor/filterprocessor [processor/filterprocessor] Migrate from opencensus library Jan 23, 2024
@crobert-1 crobert-1 added the processor/filter Filter processor label Jan 23, 2024
@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners for processor/filter: @TylerHelmuth @boostchicken. See Adding Labels via Comments if you do not have permissions to add labels yourself.

@TylerHelmuth TylerHelmuth added help wanted Extra attention is needed priority:p2 Medium labels Jan 23, 2024
@codeboten codeboten mentioned this issue Jan 23, 2024
Merged
codeboten pushed a commit to codeboten/opentelemetry-collector-contrib that referenced this issue Jan 24, 2024
[processor/filter] mv from opencensus to otel
851c20a 
This moves the telemetry generated by the processor from using opencensus to using otel. Note that the test checks the name of the metric in the format that it is generated using the meter provider, hence the change from `processor_filter_datapoints_filtered` to `processor/filter/datapoints.filtered`. When using the prometheus exporter, the output name will be the same as before.

Fixes open-telemetry#30734

Signed-off-by: Alex Boten <[email protected]>
codeboten pushed a commit that referenced this issue Jan 24, 2024
[processor/filter] mv from opencensus to otel (#30736)
9f6aeb6 
This moves the telemetry generated by the processor from using
opencensus to using otel. Note that the test checks the name of the
metric in the format that it is generated using the meter provider,
hence the change from `processor_filter_datapoints_filtered` to
`processor/filter/datapoints.filtered`. When using the prometheus
exporter, the output name will be the same as before.

Fixes #30734

---------

Signed-off-by: Alex Boten <[email protected]>
@github-actions github-actions bot mentioned this issue Jan 30, 2024
Closed
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this issue Feb 1, 2024
@cparkins
[processor/filter] mv from opencensus to otel (open-telemetry#30736)
b655ceb 
This moves the telemetry generated by the processor from using
opencensus to using otel. Note that the test checks the name of the
metric in the format that it is generated using the meter provider,
hence the change from `processor_filter_datapoints_filtered` to
`processor/filter/datapoints.filtered`. When using the prometheus
exporter, the output name will be the same as before.

Fixes open-telemetry#30734

---------

Signed-off-by: Alex Boten <[email protected]>
@codeboten codeboten moved this from Todo to Done in Collector: v1 Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@codeboten codeboten

Labels
help wanted Extra attention is needed priority:p2 Medium processor/filter Filter processor
Projects
Collector: v1
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[processor/filter] mv from opencensus to otel codeboten/opentelemetry-collector-contrib
3 participants
@codeboten @TylerHelmuth @crobert-1