-
Notifications
You must be signed in to change notification settings - Fork 759
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
Adds check-health command to temporal-cassandra-tool and switches start.sh to use that instead of cqlsh #885
Adds check-health command to temporal-cassandra-tool and switches start.sh to use that instead of cqlsh #885
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.
very nice!
return handleErr(fmt.Errorf("unable to establish CQL session:%+v", err)) | ||
} | ||
|
||
defer client.Close() |
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 guess we don't even need to defer
, but this is good regression-proofing. ; )
(PS oh and it might be worthwhile to create a ticket to track adding this to |
…fix_wait_for_cassandra
…al into fix_wait_for_cassandra
There are two changes:
This change will enable auto-setup to work against a Cassandra DB configured with TLS or username/password when launching from the docker-compose file.
Tested auto-setup locally and the check-health command against a deployed Cassandra environment with username/password and mTLS enabled.
Low risk change