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.

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

[chore] remove tests checking for component.ErrNilNextConsumer #31301

Merged

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Feb 16, 2024

Description:
We are looking to deprecate component.ErrNilNextConsumer and have pipelines check it rather than set it the expectation on every component that the next component may be nil.

See open-telemetry/opentelemetry-collector#9526 for context.

@dmitryax dmitryax merged commit 21d12c5 into open-telemetry:main Feb 16, 2024
143 checks passed
@github-actions github-actions bot added this to the next release milestone Feb 16, 2024
XinRanZhAWS pushed a commit to XinRanZhAWS/opentelemetry-collector-contrib that referenced this pull request Mar 13, 2024
…telemetry#31301)

**Description:**
We are looking to deprecate component.ErrNilNextConsumer and have
pipelines check it rather than set it the expectation on every component
that the next component may be nil.

See open-telemetry/opentelemetry-collector#9526
for context.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants