Commit Graph

145 Commits

Author SHA1 Message Date
DavidSpek 2e5eaa1dc6 Remove the f-strings in the Jupyter web app's backend (kubeflow/kubeflow#5680) 2021-06-03 09:10:37 -07:00
Jaeyeon Kim 2a0d0c7857 [fix]: Make jupyter-web-app parse workspace volume MountPath (kubeflow/kubeflow#5952)
* [fix]: Make jupyter-web-app parse workspace volume MountPath

- workspace volume path was fixed with "/home/jovyan"
- it should be enable to parse from jupyter-web-app-config's data

* change parsing key correctly
2021-06-03 08:53:36 -07:00
DavidSpek e7c924013e Update jupyter, rstudio and vs code notebook images + allow rstudio in iframe (kubeflow/kubeflow#5961)
* Update jupyter, rstudio and vs code notebook images

* Remove sed and use substring removal
2021-06-03 08:13:37 -07:00
DavidSpek c1a4f6659f Update base notebook for Renovate compatibility (kubeflow/kubeflow#5955)
* Update base notebook for Renovate compatibility

* Separate kubectl and S6 arch
2021-06-03 06:38:36 -07:00
Saffa Alvi 1e7fbce505 feat(jupyter): add fonts as assets to service (kubeflow/kubeflow#5691)
* cwa(front): Ignore font files in assets

* feat(jupyter): add fonts as assets to service

* CRUD: fonts in common

* CWA: Remove link to css file

* jwa(front): Remove font assets from jupyter

Co-authored-by: Wendy Gaultier <wvgaultier@gmail.com>
2021-06-02 10:44:35 -07:00
juliusvonkohout 2406692558 FIX The number of gpu must be set as string in Kubernetes/Openshift (kubeflow/kubeflow#5891)
* Update form.py

* Update form.py

* Update form-default.component.ts

* fix whitespace issue
2021-06-02 10:28:35 -07:00
Kimonas Sotirchos 0a7ba3170b JWA: Don't override assets with logos ConfigMap (kubeflow/kubeflow#5942)
* jwa(front): Add static logos in the app

The app does not contain the logos' svgs in its source code/static
files. This results in the icons to not show when developing locally.

This commit adds the svgs found in the logos ConfigMap to the static
files of the app as well.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Change logos fetch url

Change the URLs of the logos from `static/assets/*` to
`static/assets/logos`.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(manifests): Don't override assets with logos

Mount the ConfigMap under the `static/assets/logos` directory to not
override the contents of the entire assets dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-05-31 01:36:03 -07:00
DavidSpek 4b59c008b9 tensorboard-controller: fix binding issue (kubeflow/kubeflow#5925) 2021-05-25 07:30:09 -07:00
Kimonas Sotirchos 33fea6f503 Add CI format checks for the Volumes web app (kubeflow/kubeflow#5820)
* vwa(front): Add npm script to check the formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update the package-lock.json

Run `npm install` to bring the package-lock.json up to date

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Remove unused dockerignore file

We have created a global dockerignore file for all the web apps in the
parent dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Don't copy node_modules in dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(make): Don't include dockerignore

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(vwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Copy only necessary files for build

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: Use docker context instead of cd ..

Don't use a `cd ..` and copy dockerignore files back and forth. Instead
we should use the Docker context and the global dockerignore file we
have for all the web apps.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-05-17 08:25:31 -07:00
James Liu 51d0acb344 chore: Add comment to reference RStudio license. (kubeflow/kubeflow#5884) 2021-05-06 04:39:05 -07:00
DavidSpek 0563160233 notebook-servers: Update JupyterLab and add Git Extension (kubeflow/kubeflow#5846) 2021-04-27 10:15:14 -07:00
DavidSpek fe2789b9d4 Make notebook limits configurable with a multiplication factor (kubeflow/kubeflow#5815)
* Make notebook limits configurable with a multiplication factor

* Make limits configurable under advanced section

* run prettier to format frontend code

* fix formatting and add rounding in backend

* Return error if limit is smaller than request

* Allow disabling limitFactor by setting it to none

* review: remove camelCase in python backend

* fix: update spawner_ui_config.yaml in manifests directory

* review: fix setting limits backend

* review: remove unnecessary check from backend
2021-04-13 08:40:11 -07:00
DavidSpek 2570a8eea4 Remove Logo Trademarks from the Jupyter Web App and make logos configurable (kubeflow/kubeflow#5823)
* rebase: Make logos configurable in configmap and remove trademark references

Rebased to remove the changes to the package-lock.json

* review: add suggested changes and add image group section to README
2021-04-13 05:04:04 -07:00
Ilias Katsakioris 92ca8a2f84 tensorboard-controller: Fix scheduling unbound PVCs (kubeflow/kubeflow#5819)
When the TB controller attempts to schedule a RWO PVC it checks its
accessModes in the PVC status. The controller panics if the list is
empty.

This commit adds a check to ensure the list is not empty.

Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>
2021-04-08 17:27:03 -07:00
Mathew Wicks 500f5c61ef initial notebook server images README.md (kubeflow/kubeflow#5818) 2021-04-08 10:19:03 -07:00
Kimonas Sotirchos 149a986d3d Add CI format checks for the Jupyter web app (kubeflow/kubeflow#5811)
* jwa(front): Add npm rule for checking the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci: Add common tasks for format checks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(jwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix formatting

Run `npm run format:write` on frontend

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Include prettier in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Don't copy node_modules in Dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Add global dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Remove unused dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(make): Don't include dockerignore and cleanup

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-04-08 03:51:02 -07:00
DavidSpek a0069797df Update image tags and python packages in pytorch and tensorflow full dockerfiles (kubeflow/kubeflow#5817) 2021-04-08 03:50:02 -07:00
DavidSpek e78477ec5d Update second layer docker images to new tags and some python deps (kubeflow/kubeflow#5809) 2021-04-08 02:11:02 -07:00
DavidSpek 686667f5bd Update notebook server base images (kubeflow/kubeflow#5804) 2021-04-06 04:22:00 -07:00
DavidSpek 94390858bc Specify commonLabels for tensorboard-controller (kubeflow/kubeflow#5780) 2021-03-26 03:35:46 -07:00
DavidSpek 50445138fe (manifests): Remove unused params that break newer kustomize for web apps (kubeflow/kubeflow#5768) 2021-03-24 08:04:45 -07:00
DavidSpek 4842c53f7a Update manifests to use ECR and fix fieldPath in kustomization files (kubeflow/kubeflow#5765)
* Update manifests to use ECR and latest image tags

* remove duplicate value in central-dashboard kustomization.yaml
2021-03-24 07:35:45 -07:00
Kimonas Sotirchos facf6073e5 web-apps: Introduce an APP_SECURE_COOKIES env var (kubeflow/kubeflow#5764)
* web-apps(back): Introduce APP_SECURE_COOKIES var

Expose a new APP_SECURE_COOKIES env variable that will configure whether
the web apps should set Secure cookies or not.

This will allow the admins to configure the web apps to work when
Kubeflow is exposed over localhost/http.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(back): Switch CSRF checks order

The order the backend makes the CSRF checks should be the following:
1. check if the CSRF cookie is present
2. check if the CSRF header is present
3. check if the CSRF cookie and header have the same value

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Set APP_SECURE_COOKIES to false in dev

When running the web apps via the makefiles in dev mode we will need to
explicitly set the APP_SECURE_COOKIES env var to False, since the app
will be served over http.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-03-24 05:16:45 -07:00
DavidSpek b89ce2ce21 (JWA): Add RStudio trademark statement in Notebook Spawner UI (kubeflow/kubeflow#5758)
* Add RStudio trademark statement

* move tooltip position to the right of the RStudio button

* fix labels of icons

* RStudio trademark tooltip on index page, remove title jupyter-icon
2021-03-23 13:05:44 -07:00
DavidSpek 165d7d7241 (JWA): update manifests with latest notebook image tags (kubeflow/kubeflow#5757) 2021-03-23 12:30:44 -07:00
DavidSpek 713f0bc2d3 Add notebook-server-rstudio-tidyverse dockerfile and CI/CD (kubeflow/kubeflow#5711)
* Add notebook-server-rstudio-tidyverse dockerfile and CI/CD

* cleanup rstudio tidyverse notebook image

Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
2021-03-23 10:28:44 -07:00
Mathew Wicks 447c63bb03 set TZ in rstudio notebook image (kubeflow/kubeflow#5756) 2021-03-23 09:53:44 -07:00
Kimonas Sotirchos a144fcae46 Update the manifests for the Jupyter web app (kubeflow/kubeflow#5741)
* manifests: Add variable for the app's prefix

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* manifests: Remove Rok variables

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* manifests: Use AWS image

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* manifests: Update the spawner-config.yaml file

* Add the ImagePullPolicy setting
* Don't name the config `jupyter-web-app-jupyter-web-app-config`

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* manifests: Permissions to start/stop nbs

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Add latest jupyter images

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-03-23 08:50:44 -07:00
Kimonas Sotirchos 0fe8bf5463 Tensorboards web app manifests: Don't use specific namespace in base (kubeflow/kubeflow#5753)
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-03-23 08:46:44 -07:00
DavidSpek 1d9c808db7 Add notebook-server-rstudio dockerfile and CI/CD (kubeflow/kubeflow#5624)
* Add R-Studio image

* cleanup rstudio notebook server image
2021-03-23 07:37:44 -07:00
DavidSpek 8bff0c78b5 Fix conda activation for JupyterLab in full images + cleanup (kubeflow/kubeflow#5748) 2021-03-22 10:09:17 -07:00
DavidSpek 93b9c01880 Fix conda activation for JupyterLab + cleanup (kubeflow/kubeflow#5743) 2021-03-22 08:45:17 -07:00
DavidSpek 9cb33192f1 Update notebook dependency + cleanup (kubeflow/kubeflow#5744) 2021-03-22 08:10:17 -07:00
DavidSpek c934536e2c Fix conda activation in notebook-server-base for JupyterLab (kubeflow/kubeflow#5742) 2021-03-22 07:48:18 -07:00
DavidSpek 90a8e02fc1 (JWA): Add server type selector for jupterlab, vs-code and r-studio (kubeflow/kubeflow#5646)
* jwa(back): ability to setting annotations on NB resource

* jwa(back): update spanwer yaml, dump logo from yaml if file doesn't exist

* jwa(front): add annotations and VSCode/RStudio image types/config

* jwa(front): add server type toggle to UI

* jwa(front): set annotations in notebook request based on server-type

* jwa(front): add server type column to index page

* review: improve button toggle formatting

* jwa(back): set rstudio-tidyverse image in spawner_ui_config

* review: move rewrite and headers to backend

* review: add logo SVGs and set them in environment*.ts

* review: fix how allowing custom images works

* review: add server type logo to index
2021-03-22 05:09:18 -07:00
DavidSpek 7bde2e7e71 Add notebook-server-codeserver-python dockerfile and CI/CD (kubeflow/kubeflow#5700)
* Add notebook-server-codeserver-python dockerfile and CI/CD

* cleanup codeserver python image

Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
2021-03-22 03:58:17 -07:00
DavidSpek ca6323c7d5 Add notebook-server-jupyter-scipy dockerfile and CI/CD (kubeflow/kubeflow#5699)
* Add notebook-server-jupyter-scipy dockerfile and CI/CD

* cleanup jupyter scipy image

Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
2021-03-22 00:32:17 -07:00
DavidSpek f0aaa691d3 Add notebook-server-jupyter-tensorflow-full dockerfiles and CI/CD (kubeflow/kubeflow#5632) 2021-03-22 00:16:17 -07:00
Kimonas Sotirchos ca44b1c4ee Manifests for Tensorboard controller (kubeflow/kubeflow#5730)
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-03-21 14:28:17 -07:00
DavidSpek 149393118e Add listing nodes to JWA ClusterRole for GPU vendor detection (kubeflow/kubeflow#5732) 2021-03-21 13:48:17 -07:00
DavidSpek 98d9c5b152 Add notebook-server-codeserver dockerfile and CI/CD (kubeflow/kubeflow#5625)
* Add Code-Server (VS-Code) image

* cleanup codeserver notebook image

Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
2021-03-21 12:20:17 -07:00
DavidSpek 85859c8998 Add notebook-server-tensorflow dockerfiles and CI/CD (kubeflow/kubeflow#5627)
* Add TensorFlow CPU base image

* cleanup jupyter tensorflow notebook images

Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
2021-03-21 11:48:17 -07:00
DavidSpek 321d60d916 Add notebook-server-jupyter-pytorch-full dockerfiles and CI/CD (kubeflow/kubeflow#5631) 2021-03-21 10:48:17 -07:00
DavidSpek 8908bde330 Add notebook-server-jupyter-pytorch dockerfiles and CI/CD (kubeflow/kubeflow#5626)
* Add pytorch base images

* cleanup jupyter pytorch notebook image

* rename jupyter pytorch dockerfiles

* fix typo

Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
2021-03-21 09:07:17 -07:00
DavidSpek 3a7bd2d716 Add notebook-server-jupyter dockerfile and CI/CD (kubeflow/kubeflow#5623)
* Add base dockerfile for all jupyter based images

* cleanup jupyter notebook image

Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
2021-03-21 08:11:17 -07:00
DavidSpek fe7e662392 Add notebook-server-base dockerfile and CI/CD (kubeflow/kubeflow#5622)
* Add base dockerfile for all Web-IDE images (jupyter, r-studio, vs-code)

remove sudo from image

Add S6-overlay

Change naming and add CD

Add ci build test

change naming in prow config to avoid character limit

Add OWNERS file

rename folder

rename folder (again)

remove labels

Rename to the final folder

* cleanup base notebook image Dockerfile

Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
2021-03-21 04:50:17 -07:00
Kimonas Sotirchos 8db0c6120d Manifests for the Volumes web app (kubeflow/kubeflow#5728)
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-03-19 16:22:17 -07:00
Kimonas Sotirchos 0aeda30689 Manifests for Tensorboards web app (kubeflow/kubeflow#5727)
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
2021-03-19 16:21:17 -07:00
Yannis Zarkadas 22e4cecf56 fix notebook controller manifests (kubeflow/kubeflow#5729)
* notebook-controller: Remove manager from gitignore

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* notebook-controller: Add missing manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>
2021-03-19 16:09:17 -07:00
Yannis Zarkadas ae3b53f8d2 Notebook Controller: Consolidate manifests (kubeflow/kubeflow#5723)
* notebook-controller: Modify kubebuilder manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* notebook-controller: Set storageVersion to v1

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* notebook-controller: Fix RBAC

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* notebook-controller: Regenerate manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* notebook-controller: Remove unused kubebuilder manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>
2021-03-19 10:22:16 -07:00