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

Initial Kubeflow support #91

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

davidspek
Copy link

If the environment variable NB_PREFIX is used, it is assumed rstudio is from kubeflow and thus userconf.sh will set www-root-path to
$NB_PREFIX, set www-port=8888 and it will disable auth. RSTUDIO_VERSION=daily or RSTUDIO_VERSION=preview need to be used (daily not working, preview not tested). On daily, the 404 page loads saying:

RSudio Server
The requested page was not found.
Please contact your system administrator for assistance, or click here to go back.

This was tested with a Dockerfile that exposes port 8888 rather than 8787 (Kubeflow expects 8888).

If the environment variable `NB_PREFIX` is used, it is assumed rstudio is from kubeflow and thus userconf.sh will set `www-root-path` to
 `$NB_PREFIX`, set `www-port=8888` and it will disable auth. `RSTUDIO_VERSION=daily` or `RSTUDIO_VERSION=preview` need to be used (daily not working, preview not tested). On daily, the 404 page loads saying:
```
RSudio Server
The requested page was not found.
Please contact your system administrator for assistance, or click here to go back.
```
This was tested with a Dockerfile that exposes port 8888.
@cboettig cboettig marked this pull request as ready for review April 15, 2021 15:44
@cboettig
Copy link
Member

@davidspek This looks reasonable to me, do you still want this labeled as draft or shall we move forward?

@davidspek
Copy link
Author

@cboettig Let me go over this again quickly. We now have an RStudio Dockerfile for Kubeflow which took a lot of work to get everything working correctly with conda and such. There might be some improvements we found while working on that which will be useful here as well.

@cboettig
Copy link
Member

awesome, thanks, much appreciated

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.

2 participants