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

test: add create endpoint info policy test #3141

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

QxBytes
Copy link
Contributor

@QxBytes QxBytes commented Nov 12, 2024

Reason for Change:

Adds testing coverage to test if network policies passed in from the networking configuration make their way into the proper NetworkPolicies field and EndpointPolicies field of the endpoint info struct. Also tests to ensure that each endpoint info struct has a distinct network policies and endpoint policies slice, separate from other endpoint infos.

Issue Fixed:

Requirements:

Notes:

@QxBytes
Copy link
Contributor Author

QxBytes commented Nov 12, 2024

/azp run Azure Container Networking PR

@Azure Azure deleted a comment from azure-pipelines bot Nov 12, 2024
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@QxBytes QxBytes self-assigned this Nov 12, 2024
@QxBytes QxBytes added cni Related to CNI. ci Infra or tooling. labels Nov 12, 2024
@QxBytes QxBytes force-pushed the alew/add-create-epinfo-policy-test branch from 97ca9c9 to af0504a Compare November 12, 2024 23:02
@QxBytes QxBytes force-pushed the alew/add-create-epinfo-policy-test branch from af0504a to 515df56 Compare November 13, 2024 04:05
@QxBytes
Copy link
Contributor Author

QxBytes commented Nov 13, 2024

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@QxBytes QxBytes marked this pull request as ready for review November 13, 2024 20:01
@QxBytes QxBytes requested a review from a team as a code owner November 13, 2024 20:01
Copy link
Contributor

@paulyufan2 paulyufan2 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks for adding uts for ACL/outbound policies

@paulyufan2 paulyufan2 added this pull request to the merge queue Nov 13, 2024
Merged via the queue into master with commit 21708a1 Nov 13, 2024
22 checks passed
@paulyufan2 paulyufan2 deleted the alew/add-create-epinfo-policy-test branch November 13, 2024 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Infra or tooling. cni Related to CNI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants