Skip to content
This repository has been archived by the owner on Dec 11, 2020. It is now read-only.

User home #142

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

User home #142

wants to merge 4 commits into from

Conversation

delgadom
Copy link
Member

@delgadom delgadom commented Aug 11, 2020

Workflow

Summary

This re-organizes the notebook setup files to create a pre-home directory which will be copied into a user's directory. The idea is to make the setup of the user's directory more transparent and easier for new contributors.


# should probably pick one of these!!! The second is new, but is implied by the
# cp /pre-home below, and we actually only read the version in ~ in rhg_compute_tools.
cp -r -v /pre-home/worker-template.yml /home/jovyan/.config/dask/
cp -r -v /pre-home/worker-template.yml /home/jovyan/
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we now drop this b/c it is implied by the copy of the full folder below?

@@ -4,17 +4,13 @@ set -x

echo "Copy Dask configuration files from pre-load directory into home/.config"
mkdir -p /home/jovyan/.config/dask
Copy link
Collaborator

Choose a reason for hiding this comment

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

We can drop this line right? It will be implied by the copy of the folder contents?

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

Successfully merging this pull request may close these issues.

2 participants