FederatedAI / KubeFATE

Manage federated learning workload using cloud native technologies.
Apache License 2.0
423 stars 221 forks source link

fix the issue exsiting claim doesn't work for nodemanger #777

Closed JingChen23 closed 2 years ago

JingChen23 commented 2 years ago

Signed-off-by: Chen Jing jingch@vmware.com

Description

  1. Tell the story why you need to make this change from the user's perspective. Cannot set existing claim for nodemanager. Although there is a problem for 2 replicas sharing one pv for a statefulsets, e.g., the 2 pods must be scheduled to one node, the existingClaim still should be a valid configuration when we set only one replica for node manager.
  2. What will be the pain point if you don't make this change? I will forget to package the helm chart of FUM