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

Create EFS storage guide for AWS #715

Merged
merged 1 commit into from
Jun 11, 2018
Merged

Conversation

cam72cam
Copy link
Contributor

For #421

@summerswallow-whi
Copy link

Have you done anything with encryption in transit? (e.g. NFS over TLS)

@cam72cam
Copy link
Contributor Author

I have not, not sure how that works with EFS

@cam72cam
Copy link
Contributor Author

https://docs.aws.amazon.com/efs/latest/ug/encryption.html#encryption-in-transit might be a good read, but integrating that into K8S might be a challenge

Copy link
Collaborator

@willingc willingc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @cam72cam. Here are a few suggested changes.


I don't know if the PV needs to be in the namespace, but the arg does not seem to hurt anything. The PVC must be in the namespace or stuff will break in weird ways.

3. Configuring your application to use EFS as it's backing storage
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

its


3. Configuring your application to use EFS as it's backing storage

We now add the following to config.yaml::
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's put config.yaml in backticks.

`config.yaml`



The image setting overrides the default pinned jh base image since it has not yet been updated
to include the CHOWN_HOME setting. This will be fixed in Z2JH 0.7.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zero to JupyterHub v0.7

Let's expand jh to JupyterHub in several places


Setting up EFS storage on AWS
-----------------------------

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add:

**Note: This section is experimental. Do not use in production unless you fully understand the drawbacks and limitations.**

@willingc
Copy link
Collaborator

#707 (comment)

Thanks @cam72cam.

@willingc willingc merged commit f1bbb99 into jupyterhub:master Jun 11, 2018
@manics manics mentioned this pull request Aug 15, 2018
7 tasks
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.

3 participants