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
update env in etcd.conf.j2 to reflect the latest naming #5277
Conversation
a4e2766
to
b60188a
Compare
aos-ci-test |
roles/etcd/templates/etcd.conf.j2
Outdated
ETCD_PEER_CERT_FILE={{ etcd_peer_cert_file }} | ||
ETCD_PEER_KEY_FILE={{ etcd_peer_key_file }} | ||
{% endif -%} | ||
#ETCD_PEER_TRUSTED_CA_FILE="" |
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.
should #ETCD_PEER_TRUSTED_CA_FILE=""
be dropped since we give it a value some lines above?
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.
Right, PR updated
b60188a
to
7c96c92
Compare
aos-ci-test |
flags all look good in 3.1 too so [merge] |
Evaluated for openshift ansible merge up to 7c96c92 |
[test]ing while waiting on the merge queue |
Evaluated for openshift ansible test up to 7c96c92 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_openshift_ansible/600/) (Base Commit: 91043b6) (PR Branch Commit: 7c96c92) |
continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_openshift_ansible/984/) (Base Commit: 3384369) (PR Branch Commit: 7c96c92) |
flake on openshift/origin#16023 |
Aligning envs in the
etcd.conf.j2
with the latest envs [1] based onetcd-3.2.6
release.Both
ETCD_CA_FILE
, resp.ETCD_PEER_CA_FILE
are deprecated in favor ofETCD_TRUSTED_CA_FILE
andETCD_CLIENT_CERT_AUTH
, resp.ETCD_PEER_CA_FILE
andETCD_PEER_CLIENT_CERT_AUTH
.[1] https://github.com/coreos/etcd/blob/9d43462d174c664f5edf313dec0de31e1ef4ed47/Documentation/op-guide/configuration.md
Fixes: 1473027