-
Notifications
You must be signed in to change notification settings - Fork 190
Encryption At Rest: fixed issues and added an enhancement #128
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
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.
Nice catch! LGTM!
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.
Minor spelling/grammar and a comment. Otherwise LGTM.
LGTM, thanks! |
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.
Ugh, sorry forgot to change to Approve, not just comment - now approved :) thanks!
When can we expect this to be merged into a release? We're running into this issue and now it's not possible to update the encryption at rest key without recreating the whole cluster. |
@rvdh barring any last minute issues we plan to release 0.4.0 today. We'll be posting two blogs and I'll link them in the issues section once live as one is an upgrade guide regarding the Project IP Whitelist changes. In between releases please feel free to beta coming changes by building from the master branch. Instructions are in the README file. |
@themantissa Thats great news! :) Thanks. |
closes: #74 , #80 , #93