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

clusterctl move fails due to IPAddresses Exists #207

Closed
mcbenjemaa opened this issue Nov 22, 2023 · 4 comments
Closed

clusterctl move fails due to IPAddresses Exists #207

mcbenjemaa opened this issue Nov 22, 2023 · 4 comments
Milestone

Comments

@mcbenjemaa
Copy link
Member

mcbenjemaa commented Nov 22, 2023

Ref kubernetes-sigs/cluster-api#9478

Object already exists, updating IPAddress="mgmt-cluster-control-plane-g6ppb-net0-inet6" Namespace="default"
Retrying with backoff Cause="error updating \"ipam.cluster.x-k8s.io/v1alpha1, Kind=IPAddress\" default/mgmt-cluster-control-plane-g6ppb-net0-inet6: admission webhook \"validation.ipaddress.ipam.cluster.x-k8s.io\" denied the request: spec: Forbidden: the spec of IPAddress is immutable"
@mcbenjemaa
Copy link
Member Author

I tried to pause IPAddresses by annotations as a workaround, but it also failed.

Retrying with backoff Cause="error deleting \"ipam.cluster.x-k8s.io/v1alpha2, Kind=InClusterIPPool\" default/my-cluster-v4-icip: admission webhook \"validation.inclusterippool.ipam.cluster.x-k8s.io\" denied the request: Pool has IPAddresses allocated. Cannot delete Pool until all IPAddresses have been removed."
Deleting InClusterIPPool="my-cluster-v4-icip" Namespace="default"

@schrej
Copy link
Member

schrej commented Feb 5, 2024

This should be working with the current implementation on the main branch, which respects the paused property of clusters. It does require the clusters.x-k8s.io/cluster-name label to be set though, which needs to be specified in the IPAM contract (yet to be documented).

@schrej
Copy link
Member

schrej commented Feb 8, 2024

I've tested this locally and it seems to work fine now.
/close

@k8s-ci-robot
Copy link
Contributor

@schrej: Closing this issue.

In response to this:

I've tested this locally and it seems to work fine now.
/close

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants