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/jmx/internal/subprocess] Enable goleak check #31773

Merged
merged 19 commits into from
May 22, 2024

Conversation

crobert-1
Copy link
Member

@crobert-1 crobert-1 commented Mar 14, 2024

Description:

This originally was failing in CI/CD, but has passed 10+ times in a row, I think it's safe to merge.

Link to tracking Issue:
#30438

@github-actions github-actions bot added the receiver/jmx JMX Receiver label Mar 14, 2024
@crobert-1 crobert-1 marked this pull request as ready for review March 22, 2024 21:12
@crobert-1 crobert-1 requested a review from a team as a code owner March 22, 2024 21:12
@crobert-1 crobert-1 requested a review from dashpole March 22, 2024 21:12
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Apr 19, 2024
@crobert-1 crobert-1 removed the Stale label Apr 19, 2024
Copy link
Contributor

github-actions bot commented May 7, 2024

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label May 7, 2024
@crobert-1 crobert-1 removed the Stale label May 7, 2024
@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label May 19, 2024
@codeboten codeboten merged commit 91e4b9c into open-telemetry:main May 22, 2024
168 of 169 checks passed
@github-actions github-actions bot added this to the next release milestone May 22, 2024
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jul 11, 2024
…elemetry#31773)

This originally was failing in CI/CD, but has passed 10+ times in a row,
I think it's safe to merge.

**Link to tracking Issue:**

open-telemetry#30438
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Code review completed; ready to merge by maintainers receiver/jmx JMX Receiver
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants