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
Functioning Sec Groups E2E test and refactor AWS infrastructure functions #3261
Functioning Sec Groups E2E test and refactor AWS infrastructure functions #3261
Conversation
@josh-ferrell: This issue is currently awaiting triage. If CAPA/CAPI contributors determines this is a relevant issue, they will accept it by applying the The 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/test-infra repository. |
Hi @josh-ferrell. 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/test-infra repository. |
This PR addresses one use case on #3249 |
/ok-to-test |
/test pull-cluster-api-provider-aws-e2e |
@pydctw let me see if I can determine why that Internet gateway detachment failed. I haven't see this locally. |
@josh-ferrell: The following commands are available to trigger required jobs:
The following commands are available to trigger optional jobs:
Use
In response to this:
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/test-infra repository. |
/test pull-cluster-api-provider-aws-e2e |
bab8625
to
ec742a2
Compare
97caab8
to
b6621be
Compare
/test pull-cluster-api-provider-aws-e2e |
@pydctw I've kicked off this test again. It's passed twice now but I'll wait to see how this goes as I'm not sure why the detach operation for the Internet gateway failed the first time around. Alternatively I can choose not fail on delete operations for the AWS infrastructure components but I don't want to leave a bunch of orphaned resources. Do you know if there is a job to cleanup Amazon resources periodically? |
@josh-ferrell |
/test pull-cluster-api-provider-aws-e2e |
2 similar comments
/test pull-cluster-api-provider-aws-e2e |
/test pull-cluster-api-provider-aws-e2e |
fcf5e11
to
b31f9d0
Compare
@sedefsavas @pydctw This looks to be passing now if it looks good to you |
test/e2e/data/infrastructure-aws/kustomize_sources/external-infrastructure/kustomization.yaml
Outdated
Show resolved
Hide resolved
...structure-aws/kustomize_sources/external-infrastructure/patches/external-infrastructure.yaml
Outdated
Show resolved
Hide resolved
test/e2e/data/infrastructure-aws/kustomize_sources/external-securitygroups/kustomization.yaml
Outdated
Show resolved
Hide resolved
...structure-aws/kustomize_sources/external-securitygroups/patches/external-securitygroups.yaml
Outdated
Show resolved
Hide resolved
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sedefsavas 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 |
@josh-ferrell: you cannot LGTM your own PR. In response to this:
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/test-infra repository. |
LGTM after commits are squashed. |
…ions Add additional filter to getSubnetID function based on clusterName Apply suggestions from code review Co-authored-by: sedefsavas <58447606+sedefsavas@users.noreply.github.com>
58097a7
to
29cdc1d
Compare
/lgtm |
Initial Sec Groups E2E test and refactor AWS infrastructure functions
What type of PR is this?
/kind feature
What this PR does / why we need it:
Adds an E2E test for validating CAPA with managed VPC, subnets and security groups
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Checklist:
Release note: