Skip to content

Conversation

michi-covalent
Copy link
Contributor

@michi-covalent michi-covalent commented May 2, 2025

probably easier to review commit by commit:

  • remove unused K8sWatcher: 14a80a5
  • remove unused k8s client code: 533ecae

@michi-covalent michi-covalent force-pushed the pr/michi/no-more-watcher branch from 0306c19 to a9beb02 Compare May 2, 2025 20:44
@michi-covalent michi-covalent added the release-note/misc This PR makes changes that have no direct user impact. label May 2, 2025
@michi-covalent michi-covalent force-pushed the pr/michi/no-more-watcher branch 2 times, most recently from 93e0cfe to 631dc70 Compare May 2, 2025 22:03
@michi-covalent michi-covalent changed the title does this work watcher / k8s cleanup May 2, 2025
Tetragon now uses controller-runtime to manage informers. Delete unused
K8sWatcher struct and Watcher interface.

Signed-off-by: Michi Mutsuzaki <[email protected]>
Tetragon now uses controller-runtime to access Kubernetes resources, so
we don't need to generate the client code.

Signed-off-by: Michi Mutsuzaki <[email protected]>
@michi-covalent michi-covalent force-pushed the pr/michi/no-more-watcher branch from 631dc70 to 533ecae Compare May 2, 2025 22:05
@michi-covalent michi-covalent marked this pull request as ready for review May 2, 2025 22:13
@michi-covalent michi-covalent requested a review from a team as a code owner May 2, 2025 22:13
@michi-covalent michi-covalent requested a review from tixxdz May 2, 2025 22:13
@michi-covalent michi-covalent merged commit 9067213 into main May 3, 2025
50 checks passed
@michi-covalent michi-covalent deleted the pr/michi/no-more-watcher branch May 3, 2025 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants