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

Remove "Pod ... has started." log from KubernetesWorker #17174

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

kevingrismore
Copy link
Contributor

Closes #17141

The "Pod has started." log seems unnecessary since we're logging pod statuses if it exists, and otherwise logging that it didn't start. As it was, the log was repeated for every event from the pod.

Checklist

  • This pull request references any related issue by including "closes <link to issue>"
    • If no issue exists and your change is not a small fix, please create an issue first.
  • If this pull request adds new functionality, it includes unit tests that cover the changes
  • If this pull request removes docs files, it includes redirect settings in mint.json.
  • If this pull request adds functions or classes, it includes helpful docstrings.

@github-actions github-actions bot added the bug Something isn't working label Feb 18, 2025
@kevingrismore kevingrismore merged commit 77026f9 into main Feb 18, 2025
18 checks passed
@kevingrismore kevingrismore deleted the remove-extra-k8s-log branch February 18, 2025 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kubernetes Worker logs duplicate pod start messages for the same pod
2 participants