pv-migrate icon indicating copy to clipboard operation
pv-migrate copied to clipboard

NodePort type service

Open BonzTM opened this issue 1 year ago • 6 comments

I'm surprised to not have seen this request yet, but my request would be to implement a NodePort type service.

Is your feature request related to a problem? Please describe. LoadBalancer type service is great, if you have IPs to play around with or a L2 implementation in-cluster like MetalLB. Local is great, but unstable and slow with large PVs. A NodePort service implementation would be easier to implement and possibly solve a missing piece between these solutions.

Describe the solution you'd like A NodePort option that creates a service of type NodePort, therefore opening a high-port on each node for incoming traffic (40000 for instance) and forwarding SSH from that port to 22 on the pod.

BonzTM avatar Sep 08 '23 01:09 BonzTM

What would you prefer the rsync target to be? <ip-of-a-random-node>:<nodeport> or something else?

utkuozdemir avatar Sep 26 '23 19:09 utkuozdemir

What would you prefer the rsync target to be? <ip-of-a-random-node>:<nodeport> or something else?

Yes. I think pulling one from kubectl would be good, but requiring the user to input a node IP would also be fine

BonzTM avatar Oct 01 '23 00:10 BonzTM

Hi,

I also was looking on how to setup NodePort.

I suppose this flag should do it:

--helm-set sshd.service.type=NodePort

pavlovnicola avatar Oct 18 '23 09:10 pavlovnicola

@pavlovnicola It wouldn't work unfortunately, it needs explicit support. Setting some helm values can prevent pv-migrate to work correctly, and this is one of the cases - the whole logic depends on service type being LoadBalancer there.

utkuozdemir avatar Oct 18 '23 09:10 utkuozdemir

@utkuozdemir I tried it anyway and you are right. It does not work. Thanks for clarifying.

pavlovnicola avatar Oct 18 '23 13:10 pavlovnicola

Any update on this? I am trying to move data between two self hosted clusters (there isn't any LB on both)

I would use local strategy but I am facing the same error reported on #236

santimar avatar Nov 15 '23 15:11 santimar