-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
etcd-backup to support clusters with enableEtcdTLS
configured
#5064
Comments
CC @justinsb |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
running into the same limitation |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
after add etcd-manager to my cluster spec with this command: |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
@fejta-bot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/reopen |
@rekcah78: You can't reopen an issue/PR unless you authored it or you are a collaborator. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@justinsb can we get this reopened it's certainly a gap for anyone who's running TLS |
/reopen |
@KashifSaadat: Reopened this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Quick ping here - I see some movement in the client implementation of backup etcd-manager (last 18 days i think). Is this something that can be revisited with the recent changes to etcd-backup? If someone can point me in the direction I can also take a stab at it, in my investigations I got as far as buildEtcdBackupManagerContainer and I think some augmentation is needed there to add logic for TLS implementations. (I see a TODO in there). I haven't yet looked at the source in etcd-manager so see what params are needed. |
Hello Guys, I have the same problem in my kops cluster when enableEtcdTLS: true
etcd-backup agent |
Using the following etcdClusterSpec (shortened):
The
etcd-backup
container produces the following logs:Changes would need to be made both to kops and the etcd-manager project to support clusters that have etcd TLS enabled, such as:
The text was updated successfully, but these errors were encountered: