-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Manage certificates dynamically in kv store #2411
Manage certificates dynamically in kv store #2411
Conversation
Hello @dahefanteng . Many thanks for this very useful PR. WDYT about adding end-to-end test for the main KV stores (ETCD and Consul)? |
801aeab
to
ef66605
Compare
@dahefanteng Could you please fix integration tests and resolve conflicts |
4b1c9f0
to
1e5deb2
Compare
@nmengin I have fix integration tests and resolve confilicts. |
fbc9013
to
e0a1dfd
Compare
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.
Thank you for the tests.
I just added test with Etcd V3 Key store.
WDYT about adding a description of this feature into the documentation?
e0a1dfd
to
aabc4cd
Compare
2bb2a9f
to
e5b8ae7
Compare
@nmengin I have added some docs about this,although not too much. |
e5b8ae7
to
9738d16
Compare
9738d16
to
ba4107c
Compare
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.
Many thanks for this PR and your responsiveness!!
Sure this PR will be very useful for the cluster mode.
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
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
What does this PR do?
manage tls cert file and key file in the kv store dynamically,so we can add https service without restart traefik
Motivation
almost do the same thing with PR #2401, but this will looks consistent with the other code, and support multi kv store.
More