Skip to content

Add integration test for Pod delete #569

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Supreeth095
Copy link

@Supreeth095 Supreeth095 commented Jun 9, 2025

Issue #, if available:

Description of changes:

Test passes:


Branch ENI Pods Test Network Connectivity on Delete and Recreation of Pod creating statefulset with network connectivity when statefulset is created with network connectivity requirements should have all pods running with network access
/home/supreeet/workspaces/vpc-cni/amazon-vpc-resource-controller-k8s/test/integration/perpodsg/perpodsg_test.go:561
STEP: creating the namespace if not exist @ 06/10/25 09:06:16.007
STEP: creating security group policy @ 06/10/25 09:06:16.018
STEP: creating the security group policy @ 06/10/25 09:06:16.018
STEP: creating statefulset @ 06/10/25 09:06:16.028
STEP: waiting for statefulset pods to be ready @ 06/10/25 09:06:16.041
STEP: verifying network connectivity of all pods @ 06/10/25 09:06:21.142
STEP: getting the pod belonging to the deployment @ 06/10/25 09:06:31.142
STEP: getting the branch ENI from the pod's annotation @ 06/10/25 09:06:31.143
STEP: Verifying for ENI ID: eni-0707914300fef8c34 ENI IP: 192.168.11.171 Pod Name: per-pod-sg/network-test-0 Pod's IP: 192.168.11.171 @ 06/10/25 09:06:31.143
STEP: getting the security group for the ENI from AWS EC2 @ 06/10/25 09:06:31.143
STEP: getting the branch ENI from the pod's annotation @ 06/10/25 09:06:31.254
STEP: Verifying for ENI ID: eni-07f8b5b81c6d69179 ENI IP: 192.168.36.27 Pod Name: per-pod-sg/network-test-1 Pod's IP: 192.168.36.27 @ 06/10/25 09:06:31.254
STEP: getting the security group for the ENI from AWS EC2 @ 06/10/25 09:06:31.255
STEP: force deleting one pod to verify recreation @ 06/10/25 09:06:31.366
STEP: waiting for pod to be recreated @ 06/10/25 09:06:31.385
STEP: verifying network connectivity after pod recreation @ 06/10/25 09:06:31.386
STEP: getting the pod belonging to the deployment @ 06/10/25 09:06:41.386
STEP: getting the branch ENI from the pod's annotation @ 06/10/25 09:06:41.386
STEP: Verifying for ENI ID: eni-08814d938bcc167d2 ENI IP: 192.168.30.67 Pod Name: per-pod-sg/network-test-0 Pod's IP: 192.168.30.67 @ 06/10/25 09:06:41.386
STEP: getting the security group for the ENI from AWS EC2 @ 06/10/25 09:06:41.386
STEP: getting the branch ENI from the pod's annotation @ 06/10/25 09:06:41.497
STEP: Verifying for ENI ID: eni-07f8b5b81c6d69179 ENI IP: 192.168.36.27 Pod Name: per-pod-sg/network-test-1 Pod's IP: 192.168.36.27 @ 06/10/25 09:06:41.497
STEP: getting the security group for the ENI from AWS EC2 @ 06/10/25 09:06:41.497
STEP: deleting the statefulset @ 06/10/25 09:06:41.607
STEP: deleting security group policy @ 06/10/25 09:06:41.616
STEP: deleting the namespace @ 06/10/25 09:06:43.626
• [33.628 seconds]


@Supreeth095 Supreeth095 requested a review from a team as a code owner June 9, 2025 12:19
@Supreeth095 Supreeth095 force-pushed the master branch 2 times, most recently from f0a003f to 0fdc1b5 Compare June 10, 2025 09:17
@Supreeth095
Copy link
Author

Integration Test:
Integration_Test.txt

@yash97
Copy link
Contributor

yash97 commented Jun 21, 2025

Can you make PR title/Commit message more descriptive? Thanks.

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