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

[DRAFT][WiP][PoC] Operator: IPAM IPAddressClaim & IPAddress request and management #790

Closed
wants to merge 5 commits into from

Conversation

fgiudici
Copy link
Member

This PR introduces an opinionated experimental CAPI IPAM provider resources management in the Elemental Operator:

  • the MachineRegistration Spec now has a network field to hold an optional reference to an InClusterIPPool resource (the user should create the InClusterIPPool and reference it in the MachineRegistration)
  • on registration, the elemental-operator creates an IPAddressClaim from the InClusterIPPool and references it in the MachineInventory it creates
  • the MachineInventory reconcyler references the IPAddress created by the IPAM provider in the new .Status.IPAddressRef as soon as it is available
  • the MachineInventory reconcyler returns the IPAddress on MachineInventory deletion by deleting the associated IPAddressClaim

Resource creation/destruction worked in a Rancher test environment with the CAPI IPAM provider installed.

The idea was than to extend the websocket protocol to allow the client to ask the assigned IP address: the elemental-operator server should check the MachineInventory.Status.IPAddressRef and if available get the info from there: this part is completely missing.

@github-actions github-actions bot added area/operator operator related changes area/tests test related changes labels Jun 28, 2024
@fgiudici
Copy link
Member Author

Superseded by #793 , closing

@fgiudici fgiudici closed this Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/operator operator related changes area/tests test related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant