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

admin_guide/encrypting_data: add note about required etcd version #5923

Merged
merged 1 commit into from Oct 25, 2017

Conversation

php-coder
Copy link

To reduce confusion of the users who are still on etcd2 but would like to try data encryption, I added a note that this feature only works on etcd3.

The identical changes have been already made for Kubernetes: kubernetes/website#5521

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1502767

PTAL @smarterclayton @openshift/team-documentation
CC @simo5

@php-coder
Copy link
Author

BTW do we need to also warn users that data encryption is in alpha state?

@mburke5678
Copy link
Contributor

@php-coder LGTM. Probably should add an alpha note.
Other places we use: This is an alpha feature and may change in future

@php-coder
Copy link
Author

@mburke5678 Done. PTAL.

@mburke5678
Copy link
Contributor

@php-coder LGTM. Ready to merge? This is for 3.6 and forward?

@php-coder
Copy link
Author

Ready to merge?

Yes, please.

This is for 3.6 and forward?

Yes, exactly.

@mburke5678
Copy link
Contributor

[rev_history]
|xref:../admin_guide/encrypting_data.adoc#admin-guide-encrypting-data-at-datastore[Encrypting Data at Datastore Layer]
|Added a note that etcd v3 or later is required in order to use data encyption.
%

@php-coder php-coder deleted the data_encryption_and_ectd3 branch October 25, 2017 14:04
@php-coder
Copy link
Author

@mburke5678 Thanks!

I'm wondering why it has only enterprise tags, while it also available in origin, for example?

@mburke5678
Copy link
Contributor

mburke5678 commented Oct 25, 2017

@php-coder

while it also available in origin

Origin is based off the master branch. As soon as a PR is merged to master it will go to Origin.

mburke5678 pushed a commit to mburke5678/openshift-docs that referenced this pull request Oct 25, 2017
…uired etcd version and feature state.

(cherry picked from commit b7c06e7) xref:openshift#5923
mburke5678 pushed a commit to mburke5678/openshift-docs that referenced this pull request Oct 25, 2017
…uired etcd version and feature state.

(cherry picked from commit b7c06e7) xref:openshift#5923
@mburke5678 mburke5678 modified the milestones: Next Release, Staging, Published - 11/03/17 Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants