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

Implement RemoteMachine pooling #334

Closed
jnummelin opened this issue Nov 9, 2023 · 0 comments · Fixed by #357
Closed

Implement RemoteMachine pooling #334

jnummelin opened this issue Nov 9, 2023 · 0 comments · Fixed by #357
Assignees
Labels
cluster-api Something that relates to k0smotron acting as Cluster API provider

Comments

@jnummelin
Copy link
Member

To be able to use RemoteMachines as part of scalable control planes, we need to implement some sort of pooling for them. CAPI control plane objects always refer to XYZMachineTemplates and as such those are kinda impossible to implement for RemoteMachines.

So instead we could implement some sort of pooling for the RemoteMachines.

We could create a new kind for a pooled machine, say PooledMachine:

kind: PooledRemoteMachine
pool: mypool
spec:
  address: 1.2.3.4
  sshKey: mykey
  ...

Then in the RemoteMachineTemplate we could refer to a pool, something like:

kind: RemoteMachineTemplate
spec:
  pool: mypool

When various CAPI controllers would create a RemoteMachine out from a RemoteMachineTemplate we could have an mutating webhook to:

  • select one un-used PooledRemoteMachine, we need to track the usage status in status fields
  • copy over the address etc. details to the created RemoteMachine
@jnummelin jnummelin added the cluster-api Something that relates to k0smotron acting as Cluster API provider label Nov 9, 2023
@makhov makhov self-assigned this Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cluster-api Something that relates to k0smotron acting as Cluster API provider
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants