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] Fix failing test case for flinkmetrics #23143

Merged

Conversation

MovieStoreGuy
Copy link
Contributor

Description:

On failure to read job metrics, this type can be nil causing a panic.

Link to tracking Issue: NA

Testing:
NA

Documentation:
NA

@MovieStoreGuy MovieStoreGuy requested a review from a team June 6, 2023 00:37
@MovieStoreGuy
Copy link
Contributor Author

I have yet to add tests to help get the release out sooner; happy to follow up in a future PR or leave it to code owners in an issue.

@codeboten codeboten merged commit e4c3488 into open-telemetry:main Jun 6, 2023
@github-actions github-actions bot added this to the next release milestone Jun 6, 2023
MovieStoreGuy added a commit to atlassian-forks/opentelemetry-collector-contrib that referenced this pull request Jun 6, 2023
On failure to read job metrics, this type can be nil causing a panic.
Caleb-Hurshman pushed a commit to observIQ/opentelemetry-collector-contrib that referenced this pull request Jul 6, 2023
On failure to read job metrics, this type can be nil causing a panic.
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