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

Temporarily disable traffic check in SP #498

Conversation

timdengyun
Copy link
Contributor

@timdengyun timdengyun commented Jan 23, 2024

This patch is to temporarily disable traffic check and named ports
check in Security Policy e2e test.

In VPC mode, currently, it's unable to create a subnetport in k8s env,
so, we can not check pod running and pod traffic.

Also, creating named port related security policy will fail as well
because there are no running pods created for the named port security policy.

We still check securitypolicy creation, update and deletion for e2e test.

@timdengyun timdengyun force-pushed the Temporarily-disable-traffic-check-in-sp branch from 35d549f to f7c52f2 Compare January 23, 2024 09:46
@timdengyun timdengyun force-pushed the Temporarily-disable-traffic-check-in-sp branch 3 times, most recently from 29932b7 to 30497ad Compare January 24, 2024 07:18
@timdengyun
Copy link
Contributor Author

/e2e

@timdengyun timdengyun force-pushed the Temporarily-disable-traffic-check-in-sp branch 2 times, most recently from 60a7945 to 93bf870 Compare January 24, 2024 15:01
@zhengxiexie
Copy link
Contributor

/e2e

This patch is to temporarily disable traffic check and named ports
check in Security Policy e2e test.

In VPC mode, currently, it's unable to create a subnetport in k8s env,
so, we can not check pod running and pod traffic.

Also, creating named port related security policy will fail as well
because there are no running pods created for the named port security policy.

We still check securitypolicy creation, update and deletion for e2e test.
@timdengyun timdengyun force-pushed the Temporarily-disable-traffic-check-in-sp branch from 93bf870 to fb49cbf Compare January 25, 2024 02:58
Copy link
Contributor

@zhengxiexie zhengxiexie left a comment

Choose a reason for hiding this comment

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

LGTM

@timdengyun timdengyun merged commit d79c8f9 into vmware-tanzu:main Jan 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants