-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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: support vpc-<uuid> in target group binding vpc-id validation #3845
feat: support vpc-<uuid> in target group binding vpc-id validation #3845
Conversation
Hi @khizunov. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
0c17b0e
to
17c5075
Compare
/ok-to-test |
@shraddhabang tests seems to be passed, what are the further steps? |
/lgtm |
@shraddhabang sorry for bothering you, do you know who should approve? |
@oliviassss @M00nF1sh Can you please take a look at this PR? |
Hello @oliviassss @M00nF1sh, just checking in to see if you’ve had a chance to review this PR. Let me know if there’s anything I can improve. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: khizunov, oliviassss The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…ubernetes-sigs#3845) * support vpc-<uuid> in target groups binding validation * add VpcID validation associated tests
Issue
#3839
Description
Due to the changes introduced in this PR, the AWS Load Balancer Controller has dropped support for AWS API-compatible solutions. The current PR specifically addresses a use case that involves supporting the VPC ID format as vpc-.
In addition:
Enhanced the VpcID validation in targetgroupbinding_validator.go by introducing specific error messages and validations for different VpcID formats. Added new test cases in targetgroupbinding_validator_test.go to verify the correctness and robustness of the validation logic.
Checklist
README.md
, or thedocs
directory)BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯