-
Notifications
You must be signed in to change notification settings - Fork 101
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
wait for etcd to be ready before continuing with etcd api call #628
Conversation
I kindof disagree with the PR description ... please mention about the root cause this PR is solving which is |
Updated the PR description and release note, @aaronfern please check. hope that’s fine. |
Thanks @ishan16696! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
…ner#628) * wait for etcd to be ready before continuing with memberControl object creation * address review comments
…ner#628) * wait for etcd to be ready before continuing with memberControl object creation * address review comments
What this PR does / why we need it:
We notices that etcd backup restore does not wait for its corresponding etcd to be ready before making any etcd api call like
UpdateMemberPeerURL
which results in error in case of peerURL failed to get update withnonTLS -> TLS
.etcd-backup-restore/pkg/server/backuprestoreserver.go
Line 203 in 96e810f
This PR adds changes for
etcd-backup-restore
to wait for etcd to get ready before backup-restore try to update peerUrl for corresponding etcd.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note: