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][receiver/haproxy] Fix scraperhelper name #30925

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

mx-psi
Copy link
Member

@mx-psi mx-psi commented Jan 31, 2024

Description:

Same as #30917, before this change an empty name could be passed to the scraperhelper.

I think this is the only other instance of this pattern

Link to tracking Issue: This is needed to make contrib tests pass on open-telemetry/opentelemetry-collector/pull/9414

@mx-psi mx-psi merged commit 8f066f4 into open-telemetry:main Jan 31, 2024
96 checks passed
@mx-psi mx-psi deleted the mx-psi/haproxy-scraper branch January 31, 2024 19:45
@github-actions github-actions bot added this to the next release milestone Jan 31, 2024
mx-psi added a commit that referenced this pull request Feb 1, 2024
**Description:** Similar to #30917 and #30925, the ID was empty before.

**Link to tracking Issue:** This is needed to make contrib tests pass on
open-telemetry/opentelemetry-collector#9414
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Feb 1, 2024
**Description:** 

Same as open-telemetry#30917, before this change an empty name could be passed to the
scraperhelper.

I think this is the only other instance of this pattern

**Link to tracking Issue**: This is needed to make contrib tests pass on
open-telemetry/opentelemetry-collector/pull/9414
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Feb 1, 2024
**Description:** Similar to open-telemetry#30917 and open-telemetry#30925, the ID was empty before.

**Link to tracking Issue:** This is needed to make contrib tests pass on
open-telemetry/opentelemetry-collector#9414
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.

4 participants