Skip to content

KRA-1228 remove helmclient package and migrate to new helm client im…#29

Merged
matteogastaldello merged 5 commits intomainfrom
28-integrate-shared-helm-library
Mar 16, 2026
Merged

KRA-1228 remove helmclient package and migrate to new helm client im…#29
matteogastaldello merged 5 commits intomainfrom
28-integrate-shared-helm-library

Conversation

@matteogastaldello
Copy link
Contributor

…plementation

  • Deleted mock tests and related files for helmclient.
  • Removed helmclient specification and utility functions.
  • Refactored helm client initialization in main.go to use new helm client with caching and CRD informer.
  • Updated tracer to ensure thread-safe resource capturing.
  • Added comprehensive tests for the new tracer implementation.

…plementation

- Deleted mock tests and related files for helmclient.
- Removed helmclient specification and utility functions.
- Refactored helm client initialization in main.go to use new helm client with caching and CRD informer.
- Updated tracer to ensure thread-safe resource capturing.
- Added comprehensive tests for the new tracer implementation.
@matteogastaldello matteogastaldello linked an issue Mar 16, 2026 that may be closed by this pull request
@matteogastaldello matteogastaldello changed the title refactor: remove helmclient package and migrate to new helm client im… KRA-1228 remove helmclient package and migrate to new helm client im… Mar 16, 2026
@matteogastaldello matteogastaldello merged commit 84945bd into main Mar 16, 2026
3 checks passed
@codecov
Copy link

codecov bot commented Mar 16, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrate Shared Helm Library

1 participant