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

Quick fix to prevent panic on nil collector result #1508

Merged
merged 2 commits into from Mar 18, 2024

Conversation

nvanthao
Copy link
Member

@nvanthao nvanthao commented Mar 14, 2024

Description, Motivation and Context

Fixes: #1442

Quick fix before we can refractor the collector, I think we should not mutate the result in defer block.

Checklist

  • New and existing tests pass locally with introduced changes.
  • Tests for the changes have been added (for bug fixes / features)
  • The commit message(s) are informative and highlight any breaking changes
  • Any documentation required has been added/updated. For changes to https://troubleshoot.sh/ create a PR here

Does this PR introduce a breaking change?

  • Yes
  • No

@nvanthao nvanthao requested a review from a team as a code owner March 14, 2024 06:13
@nvanthao nvanthao added bug::regression type::bug Something isn't working labels Mar 14, 2024
@nvanthao nvanthao requested a review from banjoh March 17, 2024 21:23
@nvanthao nvanthao merged commit 6f839b3 into main Mar 18, 2024
27 checks passed
@nvanthao nvanthao deleted the gerard/b-panic-run-pod branch March 18, 2024 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug::regression type::bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

potential panic if servicaccount does not exist for run-pod
2 participants