Skip to content

Add a method to remove a single pod from the fake watcher#4996

Merged
andrewstrohman merged 1 commit into
cilium:mainfrom
mstemm:pr/mstemm/add-pod-remove-fake-watcher
May 13, 2026
Merged

Add a method to remove a single pod from the fake watcher#4996
andrewstrohman merged 1 commit into
cilium:mainfrom
mstemm:pr/mstemm/add-pod-remove-fake-watcher

Conversation

@mstemm
Copy link
Copy Markdown
Contributor

@mstemm mstemm commented May 13, 2026

This is useful for frameworks utilizing FakeK8sWatcher that need to remove individual pods.

Description

Add a method to the fake watcher watcher.RemovePod that can be used to remove a single previously added pod. This is useful for frameworks using fake watcher that need to change the set of pods.

Changelog

Add a method `(watcher *FakeK8sWatcher) RemovePod` that can be used to remove a previously added pod. This is useful for frameworks using fake watcher that need to change the set of pods.

@mstemm mstemm requested a review from a team as a code owner May 13, 2026 02:21
@mstemm mstemm requested a review from kevsecurity May 13, 2026 02:21
@mstemm mstemm force-pushed the pr/mstemm/add-pod-remove-fake-watcher branch from c668b6c to 60413ad Compare May 13, 2026 05:16
@tpapagian tpapagian added the release-note/misc This PR makes changes that have no direct user impact. label May 13, 2026
Comment thread pkg/watcher/fake.go Outdated
A pod is considered a match if the name and namespace match.

This is useful for frameworks utilizing FakeK8sWatcher that need to
remove individual pods.

Signed-off-by: mstemm <mstemm@isovalent.com>
@mstemm mstemm force-pushed the pr/mstemm/add-pod-remove-fake-watcher branch from 60413ad to 9c65bc8 Compare May 13, 2026 14:47
@netlify
Copy link
Copy Markdown

netlify Bot commented May 13, 2026

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 9c65bc8
🔍 Latest deploy log https://app.netlify.com/projects/tetragon/deploys/6a048f1b78610100087cb516
😎 Deploy Preview https://deploy-preview-4996--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@andrewstrohman andrewstrohman merged commit ab20129 into cilium:main May 13, 2026
59 of 61 checks passed
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.

4 participants