Skip to content
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

Boto3 for just tablesnap, tokens and KMS encryption #89

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

juiceblender
Copy link
Contributor

@juiceblender juiceblender commented Oct 24, 2017

Hi guys,

Just thought I would put up another PR this time. This PR was born out of my need to able to use KMS to encrypt backups instead of just SSE but it doesn't look like I am able to do that using boto. So I bumped it to boto3 for just tablesnap only. Boto and boto3 can coexist during the migration so I figured this would be a start: http://boto3.readthedocs.io/en/latest/guide/migration.html

Functionally I've tested md5-on-start (just to be sure etags are right with multiparts...it may not have worked before..?), KMS, SSE, and generating tokens and of course, taking the backup. Let me know if there are any questions/issues/suggestions!

Thanks.

@JeremyGrosser
Copy link
Owner

I've been wanting to do this for a while, but I'm a bit afraid to introduce a dependency on boto3 as there is no system package for it in Debian jessie and quite a few people (including our unit test scripts) haven't moved to stretch yet.

Generally speaking, I think we should probably start working towards a backwards-incompatible major release that moves completely to Python 3 with boto3 and drops support for older systems completely... Ideally that release would also be tested against Cassandra 3.

@JeremyGrosser
Copy link
Owner

...looking into it a bit more, there is a python-botocore package in jessie that contains most of the code from boto3. Maybe we can use that? https://github.com/boto/botocore

@juiceblender
Copy link
Contributor Author

Hi Jeremy,

Sorry you lost me there, how do you mean there is no system package for boto3 in Debian Jessie? If you do an install of boto3 on Debian Jessie it wouldn't work?

I have actually been running my tests on Cassandra 3...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants