gardener / etcd-backup-restore

Collection of components to backup and restore the etcd of a Kubernetes cluster.
Apache License 2.0
289 stars 100 forks source link

[Enhancement] Backup-restore should take care of etcd member restart incase of etcd's advertise peerURL found to be updated. #787

Closed ishan16696 closed 1 month ago

ishan16696 commented 1 month ago

Enhancement (What you would like to be added):

  1. Backup-restpre should restart it's corresponding etcd member when etcd's advertise peer URLs found to be updated as it's mention in the etcd documentation that updating peerURLs necessitates restarts of member for the changes to properly propagate throughout the cluster, as it is a cluster-wide operation. More details can be found in the etcd documentation: https://etcd.io/docs/v3.5/op-guide/runtime-configuration/#update-advertise-peer-urls
  2. Backup-restore should only call updateMemberPeerURL API call only when peerURL is found to be updated, currently that's not the case.

Motivation (Why is this needed?): Transition from non-TLS to TLS for PeerURLs is not supported for already deployed etcd HA cluster: https://github.com/gardener/etcd-druid/issues/886

Approach/Hint to the implement solution (optional):

cc @unmarshall

ishan16696 commented 1 month ago

/assign @ishan16696 @unmarshall