-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add tenant emergency move to SansShell fdbcli modules #285
Conversation
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.
I'm happy with the grpc interface.
Please add a test for the server before we merge this. The logic around the asynchronous command is complex enough to deserve testing.
…enant-emergency-move
…enant-emergency-move
…enant-emergency-move
Let's add some manual test results to the PR description. |
After merge: update permissions list to restrict which group of people can run these commands
Provide wrappers for new fdbcli command of the form
tenant emergency_move <start|switch|finish|abort|status>
Also add sansshell module
to call and wait for
fdb_move_orchestrator.py
Run fdbmovedata wait 1987524441901984307 to wait for the process to complete
An existing command is already running on the server.
Run fdbmovedata wait 1987524441901984307 to wait for the process to complete
An existing command is already running on the server.
Run fdbmovedata wait 1987524441901984307 to wait for the process to complete
Command execution failure - rpc error: code = Internal desc = Provided ID 1723579472755794943 does not match stored ID 1987524441901984307
Management Cluster FDBCLI Commands: ['/usr/bin/fdbcli', '-C', '/etc/foundationdb/fdb.cluster', '--tls_certificate_file', '/etc/foundationdb/fdb.pem', '--tls_ca_file', '/etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem', '--tls_key_file', '/etc/foundationdb/fdb.pem', '--exec', 'metacluster get awsuswest2temptest000032_jfu_dst_cluster_1 JSON']
....