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

Update IP Spaces Floating IPs comment #382

Open
srekkas opened this issue Nov 12, 2024 · 2 comments
Open

Update IP Spaces Floating IPs comment #382

srekkas opened this issue Nov 12, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@srekkas
Copy link

srekkas commented Nov 12, 2024

Describe the bug

IP comment show bad info from non existing deleted cluster.
image

Reproduction steps

  1. Create cluster
  2. Create service with Load balancer
  3. Get External IP
  4. Delete that cluster
  5. Create new
  6. Create service with Load Balancer and specify IP which was used on cluster before

Expected behavior

Comment contains new cluster and service info.

Additional context

No response

@srekkas srekkas added the bug Something isn't working label Nov 12, 2024
@arunmk
Copy link
Collaborator

arunmk commented Nov 12, 2024

CPI will not automatically delete LBs when a cluster is deleted. This is likely common to all CPIs.

Even if not, we do so by design. We delegate the responsibility uninstall LB applications to the cluster owner before deleting the cluster. We cannot auto-delete such resources since these may be sensitive (someone may have many DNS rules set on an IP, for example).

@srekkas
Copy link
Author

srekkas commented Nov 13, 2024

CPI will not automatically delete LBs when a cluster is deleted. This is likely common to all CPIs.

Even if not, we do so by design. We delegate the responsibility uninstall LB applications to the cluster owner before deleting the cluster. We cannot auto-delete such resources since these may be sensitive (someone may have many DNS rules set on an IP, for example).

But i think it must be an option, because everything what is created dynamical, must be option to delete it dynamically.
So if we or customer create cluster from webui or something like terraform, create multiple pvc, services with load loadbalancers, and maybe everything dynamicaly with gitops. After that cluster get deleted and many do not think to cleanup everything. So Loadbalancers can use limited external IPs, which have some cost and inconveniences to cleanup everything.

And in this case, i see some comment on that IP, which says IP is used by non existing cluster, it complicates manual cleanup even further. Maybe comment can be refreshed every so often?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants