Skip to content

Commit

Permalink
Merge pull request 2i2c-org#4968 from minrk/oauth-17.1
Browse files Browse the repository at this point in the history
jupyter-health: add experimental hub image with oauthenticator 17.1
  • Loading branch information
consideRatio authored Oct 17, 2024
2 parents b389dff + 33b1fd7 commit 888478e
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 5 deletions.
7 changes: 7 additions & 0 deletions config/clusters/jupyter-health/common.values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,13 @@ jupyterhub:
name: "Jupyter Health"
url: https://www.earthdata.nasa.gov/esds/veda
hub:
# FIXME: Experiment to use oauthenticator 17.1, should be transitioned away
# as part of upgrading to z2jh 4, see
# https://github.com/2i2c-org/infrastructure/pull/4968
#
image:
name: quay.io/2i2c/pkce-experiment
tag: 0.0.1-0.dev.git.10694.h38ededaf
allowNamedServers: true
config:
JupyterHub:
Expand Down
11 changes: 6 additions & 5 deletions helm-charts/chartpress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,12 @@ charts:
REQUIREMENTS_FILE: dynamic-image-building-requirements.txt
contextPath: images/hub
dockerfilePath: images/hub/Dockerfile
- name: support
images:
gcp-filestore-backups:
imageName: quay.io/2i2c/gcp-filestore-backups
valuesPath: gcpFilestoreBackups.image
pkce-experiment:
imageName: quay.io/2i2c/pkce-experiment
buildArgs:
REQUIREMENTS_FILE: pkce-requirements.txt
contextPath: images/hub
dockerfilePath: images/hub/Dockerfile
- name: aws-ce-grafana-backend
images:
aws-ce-grafana-backend:
Expand Down
14 changes: 14 additions & 0 deletions helm-charts/images/hub/pkce-requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# Image lives at quay.io/2i2c/pkce-experiment
# install oauthenticator 17.1,
# which adds PKCE support.
# experiment no longer needed when base chart is updated to z2jh 4.0.0
oauthenticator>=17.1,<18

# jupyterhub-configurator isn't maintained and its not intended to be developed
# further. We are using a branch that has forked from the main branch just
# before a breaking change were made. This allows us to avoid migrating.
#
# ref: https://github.com/yuvipanda/jupyterhub-configurator/commits/main
# ref: https://github.com/yuvipanda/jupyterhub-configurator/commits/backported-jh41-compatibility
#
git+https://github.com/yuvipanda/jupyterhub-configurator@backported-jh41-compatibility

0 comments on commit 888478e

Please sign in to comment.