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

Lack of cleanup of underlying cloud resources #10714

Closed
maxkokocom opened this issue Dec 1, 2023 · 2 comments
Closed

Lack of cleanup of underlying cloud resources #10714

maxkokocom opened this issue Dec 1, 2023 · 2 comments
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.

Comments

@maxkokocom
Copy link

What happened:

When running terraform destroy on existing cluster it's always failing to delete VPC as it has loadbalancers created by nginx-ingress

What you expected to happen:

I would expect there is a way to trigger a cleanup process before terraform destroy of a whole cluster. I asked in terraform EKS module, but they said this should not be done in their module, but via software which created the resources. I would appriaciate a recommanded pattern how to deal with this.

@maxkokocom maxkokocom added the kind/bug Categorizes issue or PR as related to a bug. label Dec 1, 2023
@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Dec 1, 2023
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

If Ingress contributors determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

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.

@strongjz
Copy link
Member

strongjz commented Dec 1, 2023

you can create the load balancer in Terraform, get the IP address, and then assign it in the helm values.

controller.service.loadBalancerIP

https://github.com/kubernetes/ingress-nginx/blob/main/charts/ingress-nginx/values.yaml#L469

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.
Projects
Archived in project
Development

No branches or pull requests

3 participants