-
Notifications
You must be signed in to change notification settings - Fork 741
Conversation
@@ -17,5 +17,6 @@ package cluster | |||
import "errors" | |||
|
|||
var ( | |||
errNoBackupExist = errors.New("no backup exist for disaster recovery") | |||
errNoBackupExist = errors.New("no backup exist for disaster recovery") | |||
errInvalidMemberName = errors.New("member's Name has the wrong format") |
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.
the format of member's name is invalid.
@@ -64,3 +71,12 @@ func podsToMemberSet(pods []*api.Pod, selfHosted *spec.SelfHostedPolicy) etcduti | |||
} | |||
return members | |||
} | |||
|
|||
func findID(name string) (int, error) { |
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.
probably move this to etcdutil/member
member.NameID()
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.
The logic of naming convention all sits in pkg/cluster:
https://github.com/coreos/etcd-operator/blob/master/pkg/cluster/cluster.go#L297
Can we put it in the same pkg right now? I have another PR ontop.
We would centralize such logic, probably in etcdutil/member later.
fix error message. |
@etcd-bot retest this please An etcd pod keeps pending. Seems like a normal cluster flake:
|
Merging this. |
No description provided.