-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat: Add pattern for VPC CNI native network policy support #1781
feat: Add pattern for VPC CNI native network policy support #1781
Conversation
Hi @valentinwidmer! Thanks for the PR. I’ll review and get back to you soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @valentinwidmer!!
Please update the docs/patterns
using the same standards of the other patterns here.
Also, run the pre-commit scripts against your files to fix any formatting and lint for the CI tests. pre-commit run --files ./patterns/aws-vpc-cni-network-policy/*
Added documentation and executed pre-commit scripts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome!
Description
Motivation and Context
How was this change tested?
pre-commit run -a
with this PRAdditional Notes