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

support encrypted EBS volumes #114

Merged
merged 4 commits into from
Apr 6, 2015
Merged

Conversation

scalp42
Copy link
Contributor

@scalp42 scalp42 commented Feb 25, 2015

I remember seeing PRs related to the right_aws gem, so this is for aws-sdk one, now that we don't depend on the old one anymore.

Per:

http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/EBSEncryption.html

and

http://docs.aws.amazon.com/sdkforruby/api/Aws/EC2/Client.html#create_volume-instance_method

@otakup0pe
Copy link

LGTM 🍸

@mrsipan
Copy link

mrsipan commented Feb 25, 2015

+1

1 similar comment
@arnaud036
Copy link

+1

@callmeradical
Copy link

+1 This is a feature very much needed.

Do you also need to supply the encryption key when making a volume encrypted?

@scalp42
Copy link
Contributor Author

scalp42 commented Mar 6, 2015

@callmeradical I'll push the added param. Good catch!

@scalp42
Copy link
Contributor Author

scalp42 commented Mar 6, 2015

@callmeradical should have the extra KMS bits now!

@callmeradical
Copy link

+1 for great justice

@DrMerlin
Copy link

DrMerlin commented Apr 6, 2015

+1

@scalp42
Copy link
Contributor Author

scalp42 commented Apr 6, 2015

@cwebberOps @someara @jtimberman any chance to review this PR please?

Sorry for the ping but it's for a good cause (stop the forks).

@scalp42
Copy link
Contributor Author

scalp42 commented Apr 6, 2015

Rebased on 2.6.6

someara pushed a commit that referenced this pull request Apr 6, 2015
@someara someara merged commit 65564e1 into sous-chefs:master Apr 6, 2015
@callmeradical
Copy link

Thanks Obama, I mean @someara

@DrMerlin
Copy link

DrMerlin commented Apr 6, 2015

Stop the forks!

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.

None yet

8 participants