Skip to content
This repository has been archived by the owner on Mar 28, 2020. It is now read-only.

*: make default etcd version to 3.2.10 #1607

Merged
merged 1 commit into from
Nov 18, 2017

Conversation

hongchaodeng
Copy link
Member

@hongchaodeng hongchaodeng commented Oct 31, 2017

@hongchaodeng
Copy link
Member Author

Has TLS issue with 3.2.9:

2017-10-31 18:19:27.991428 I | etcdmain: rejected connection from "10.2.3.149:55104" (tls: "10.2.3.149" does not match any of DNSNames ["*.example-vault-etcd.hongchao-test.svc" "*.example-vault-etcd.hongchao-test.svc.cluster.local"])
...

2017-10-31 18:19:41.334025 W | rafthttp: health check for peer b1b48150f4da7b66 could not connect: dial tcp 10.2.3.149:2380: i/o timeout

@hongchaodeng
Copy link
Member Author

hongchaodeng commented Oct 31, 2017

etcd upstream issues ref:

We have to work around the issue right now by providing FQDN in peer certs.

@hongchaodeng
Copy link
Member Author

hongchaodeng commented Nov 1, 2017

Even after adding cluster domain into peer SAN, etcd server still occasionally rejected connection with error:

etcdmain: rejected connection from "10.60.6.133:60520" (
tls: "10.60.6.133" does not match any of DNSNames
["*.tls-test-5577006791947779410.e2e-etcd-operator-e2eslow-pr-837.svc" 
 "*.tls-test-5577006791947779410.e2e-etcd-operator-e2eslow-pr-837.svc.cluster.local"] 
(lookup 133.6.60.10.in-addr.arpa. on 10.63.240.10:53: no such host))

Still needs more investigation.

@hongchaodeng
Copy link
Member Author

hongchaodeng commented Nov 1, 2017

Also high chance for this error:

2017-11-01 16:31:59.792577 W | etcdserver: could not get cluster response from https://example-0000.example.default.svc:2380:
Get https://example-0000.example.default.svc:2380/members: EOF
2017-11-01 16:31:59.798122 C | etcdmain: cannot fetch cluster info from peer urls:
could not retrieve cluster information from the given urls

@hongchaodeng hongchaodeng changed the title [WIP] *: make default etcd version to 3.2.9 [WIP] *: make default etcd version to 3.2.10 Nov 16, 2017
@hongchaodeng hongchaodeng changed the title [WIP] *: make default etcd version to 3.2.10 *: make default etcd version to 3.2.10 Nov 16, 2017
@hongchaodeng
Copy link
Member Author

All tests fixed

@@ -78,8 +78,9 @@ func NewEtcdRestore(restoreName, version string, size int, restoreSource api.Res
},
Spec: api.RestoreSpec{
ClusterSpec: api.ClusterSpec{
Size: size,
Version: version,
BaseImage: "gcr.io/etcd-development/etcd",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if version is 3.0.x? Will this code find the right image?

maybe we should also pass in the base image as a param.

@fanminshi
Copy link
Contributor

lgtm

@hongchaodeng hongchaodeng merged commit 88c069a into coreos:master Nov 18, 2017
@hongchaodeng hongchaodeng deleted the etcd_32 branch November 18, 2017 00:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v3.2.x compatibility?
2 participants