Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[receiver/jaegerreceiver] Remove deprecated use of host.ReportFatalError #30591

Closed
Tracked by #30501
atoulme opened this issue Jan 16, 2024 · 1 comment · Fixed by #30609
Closed
Tracked by #30501

[receiver/jaegerreceiver] Remove deprecated use of host.ReportFatalError #30591

atoulme opened this issue Jan 16, 2024 · 1 comment · Fixed by #30609
Assignees
@codeboten
Labels
bug Something isn't working receiver/jaeger

Comments

@atoulme
Copy link
Contributor

atoulme commented Jan 16, 2024

Component(s)

receiver/jaeger

Describe the issue you're reporting

Refactor to remove use of host.ReportFatalError, you can use settings.TelemetrySettings.ReportStatus(component.NewFatalErrorEvent(err)) instead.

See #30505 for an example.
@atoulme atoulme added the needs triage New item requiring triage label Jan 16, 2024
@atoulme atoulme mentioned this issue Jan 16, 2024
Closed
21 tasks
@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@crobert-1 crobert-1 added bug Something isn't working and removed needs triage New item requiring triage labels Jan 16, 2024
@codeboten codeboten self-assigned this Jan 16, 2024
@codeboten codeboten mentioned this issue Jan 16, 2024
Merged
codeboten pushed a commit that referenced this issue Jan 16, 2024
[chore] jaeger receiver ReportFatalError -> ReportStatus (#30609)
c314725 
Remove use of deprecated host.ReportFatalError

Linked to
#30501
Fixes #30591

Signed-off-by: Alex Boten <aboten@lightstep.com>
@github-actions github-actions bot mentioned this issue Jan 23, 2024
Closed
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this issue Feb 1, 2024
@cparkins
[chore] jaeger receiver ReportFatalError -> ReportStatus (open-teleme…
9637a30 
…try#30609)

Remove use of deprecated host.ReportFatalError

Linked to
open-telemetry#30501
Fixes open-telemetry#30591

Signed-off-by: Alex Boten <aboten@lightstep.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@codeboten codeboten

Labels
bug Something isn't working receiver/jaeger
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[chore] jaeger receiver ReportFatalError -> ReportStatus codeboten/opentelemetry-collector-contrib
3 participants
@atoulme @codeboten @crobert-1