Skip to content
This repository was archived by the owner on Sep 4, 2025. It is now read-only.

Conversation

@macqueenism
Copy link
Contributor

Summary

• Moved CoreV1Api client instantiation to module level to prevent creating new instances on every pod phase change event
• Eliminates memory leak caused by accumulating HTTP connection pools and associated resources
• Single API client instance is now reused across all events

Test plan

  • Deploy the controller and monitor memory usage over time with multiple pod failures
  • Verify that memory usage remains stable instead of continuously growing

🤖 Generated with Claude Code

Move CoreV1Api client instantiation to module level to prevent creating
a new client instance on every pod phase change event. This eliminates
the accumulation of HTTP connection pools and associated resources.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@macqueenism macqueenism self-assigned this Aug 8, 2025
@macqueenism macqueenism merged commit 00460d8 into main Aug 8, 2025
5 checks passed
@macqueenism macqueenism deleted the fix-memory-leak-api-client branch August 8, 2025 15:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants