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

[confmap] Remove deprecated WithErrorUnused #9484

Closed
mx-psi opened this issue Feb 6, 2024 · 0 comments · Fixed by #9503
Closed

[confmap] Remove deprecated WithErrorUnused #9484

mx-psi opened this issue Feb 6, 2024 · 0 comments · Fixed by #9503
Assignees
@atoulme
Labels
area:confmap release:required-for-ga Must be resolved before GA release
Milestone
go.opentelemetry....

Comments

@mx-psi
Copy link
Member

mx-psi commented Feb 6, 2024

It's been deprecated for a couple versions now, so we can remove it.
@mx-psi mx-psi added release:required-for-ga Must be resolved before GA release area:confmap labels Feb 6, 2024
This was referenced Feb 6, 2024
Merged
Merged
Merged
bogdandrutu pushed a commit to open-telemetry/opentelemetry-collector-contrib that referenced this issue Feb 7, 2024
@atoulme
[chore] remove use of confmap.WithErrorUnused (#31104)
d922b66 
**Description:** 
remove use of confmap.WithErrorUnused

See
open-telemetry/opentelemetry-collector#9484
mx-psi pushed a commit to open-telemetry/opentelemetry-collector-contrib that referenced this issue Feb 8, 2024
@atoulme
[chore] remove deprecated calls with confmap.WithErrorUnused, now e…
3c64ddc 
…nabled by default (#31090)

**Description:**
Remove all calls to `confmap.WithErrorUnused` ahead of its removal.

**Link to tracking Issue:**
open-telemetry/opentelemetry-collector#9484
mx-psi added a commit that referenced this issue Feb 8, 2024
@atoulme @mx-psi
[confmap] Remove deprecated confmap.WithErrorUnused (#9503)
cf51a35 
**Description:**
Remove deprecated `confmap.WithErrorUnused`

**Link to tracking Issue:**
Fixes #9484

---------

Co-authored-by: Pablo Baeyens <pablo.baeyens@datadoghq.com>
Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
@crobert-1 crobert-1 mentioned this issue Mar 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@atoulme atoulme

Labels
area:confmap release:required-for-ga Must be resolved before GA release
Projects
Collector: v1
Archived in project
Milestone
go.opentelemetry.io/collector/confmap...
Development

Successfully merging a pull request may close this issue.

[confmap] Remove deprecated confmap.WithErrorUnused atoulme/opentelemetry-collector
2 participants
@atoulme @mx-psi