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

Fix ebpf maps/progs cleanup test #376

Merged
merged 2 commits into from
Mar 13, 2025

Conversation

Pavani-Panakanti
Copy link
Contributor

Issue #, if available:
With recent changes in NP, every pod will have maps/progs created even if policies are not applied to it. Once we run cyclonus tests and clean up the pods and policies, we run a pod which runs cleanup script to check if there are any leaked maps or progs on the node. As we are creating a new pod for this, there will be maps/progs created for this pod.

Description of changes:
Exclude maps/progs with cleanup pod name while checking for leaked maps/progs. core dns pods will also have default probes attached as it is a deployment. Excluding those too

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Pavani-Panakanti Pavani-Panakanti requested a review from a team as a code owner March 12, 2025 19:02
Copy link
Contributor

@jaydeokar jaydeokar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can keep this as an input arg in future and keep the logic very basic.
Looks good to me for now

@Pavani-Panakanti Pavani-Panakanti merged commit cb01975 into aws:main Mar 13, 2025
4 checks passed
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.

2 participants