-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[docdb] Add --certs_dir plumbing to yb-ts-cli #2877
Comments
@srivignessh Did you want to look into this? I can assign it to you. Let me know if you'd like some more pointers on where to get started. We document how to start clusters with TLS in our docs. One simple command you can do is try to change flag values:
This should fail if TLS is enabled on the cluster you're targeting. |
@bmatican Sure. Thanks, you can assign it to me. I will work on it. Can you point me to the docs for tablet server cli? (yb-ts-cli). |
@srivignessh Oh, good question, I think we do not have docs yet for I think for the purposes of this task, any command you run from it should fail without certs plumbed through, as they will all try to send RPC requests to the servers. That's why I just referenced one example command above. Also, a useful commit for reference might be 9739e4a, which added support for |
Add --certs_dir plumbing to yb-ts-cli from yb-admin. Fixes yugabyte#2877 TODO: Unit test
Summary: Porting PR: #2953 Test Plan: New unit test was introduced ./yb_build.sh --java-test org.yb.pgsql.TestSecureCluster Reviewers: bogdan, sergei Reviewed By: sergei Subscribers: ybase Differential Revision: https://phabricator.dev.yugabyte.com/D8252
Currently, we support
--certs_dir
for the servers, to setup TLS.We also support
--certs_dir_name
foryb-admin
, to be able to send secure RPCs to the servers.We need to add support for this to
yb-ts-cli
.See 9739e4a
The text was updated successfully, but these errors were encountered: