Merge pull request #439 from yuvipanda/v1-bump

Bump z2jh to v1.0 beta

.gitignore

@@ -1,3 +1,11 @@
+# Added by 2i2c
+Chart.lock
+**/charts/*.tgz
+
+.env-*
+
+
+
 # Byte-compiled / optimized / DLL files
 __pycache__/
 *.py[cod]

hub-templates/basehub/Chart.yaml

@@ -2,10 +2,9 @@ apiVersion: v2
 appVersion: '1.0'
 description: Deployment Chart for JupyterHub
 name: basehub
-# Let's keep this constant so other charts in this repo can depend on this easily
-version: 0.0.1-n769.hfd04ce8
+version: "0.1.0"
 dependencies:
   - name: jupyterhub
-   # REMEMBER TO CHANGE BASE IMAGE OF images/hub/ WHEN CHANGING THIS
-    version: 0.11.1-n277.h61b0e003
+    # REMEMBER TO CHANGE BASE IMAGE OF images/hub/ WHEN CHANGING THIS
+    version: 1.0.0-beta.1
     repository: https://jupyterhub.github.io/helm-chart/

hub-templates/basehub/values.yaml

@@ -29,10 +29,15 @@ nfsPVC:
 
 jupyterhub:
   custom:
+    singleuserAdmin:
+      extraVolumeMounts:
+        - name: home
+          mountPath: /home/jovyan/shared-readwrite
+          subPath: _shared
     cloudResources:
-      provider: null
+      provider:
       gcp:
-        projectId: null
+        projectId:
       scratchBucket:
         enabled: false
   ingress:
@@ -67,8 +72,6 @@ jupyterhub:
     hook:
       enabled: false
   proxy:
-    https:
-      enabled: false
     service:
       type: ClusterIP
     chp:
@@ -95,11 +98,6 @@ jupyterhub:
       letsencrypt:
         contactEmail: [email protected]
   singleuser:
-    admin:
-      extraVolumeMounts:
-        - name: home
-          mountPath: /home/jovyan/shared-readwrite
-          subPath: _shared
     startTimeout: 600 # 10 mins, because sometimes we have too many new nodes coming up together
     defaultUrl: /tree
     nodeSelector:
@@ -161,7 +159,8 @@ jupyterhub:
                 interfaces:
                   - value: "/tree"
                     title: Classic Notebook
-                    description: The original single-document interface for creating Jupyter Notebooks.
+                    description: The original single-document interface for creating
+                      Jupyter Notebooks.
                   - value: "/lab"
                     title: JupyterLab
                     description: A Powerful next generation notebook interface
@@ -179,7 +178,7 @@ jupyterhub:
           - --Configurator.config_file=/usr/local/etc/jupyterhub-configurator/jupyterhub_configurator_config.py
     image:
       name: quay.io/2i2c/pilot-hub
-      tag: '0.0.1-n769.hfd04ce8'
+      tag: '0.0.1-n781.hf8d0498'
     config:
       JupyterHub:
         authenticator_class: oauthenticator.auth0.Auth0OAuthenticator
@@ -256,7 +255,7 @@ jupyterhub:
 
         class CustomSpawner(ConfiguratorSpawnerMixin, KubeSpawner):
           def start(self, *args, **kwargs):
-            custom_admin = get_config('singleuser.admin', {})
+            custom_admin = get_config('custom.singleuserAdmin', {})
             if custom_admin and self.user.admin:
                 extra_init_containers = custom_admin.get('initContainers', [])
                 extra_volume_mounts = custom_admin.get('extraVolumeMounts', [])
@@ -311,4 +310,4 @@ jupyterhub:
             'PANGEO_SCRATCH': f'{bucket_protocol}://{bucket_name}/$(JUPYTERHUB_USER)',
           }
 
-          c.KubeSpawner.environment.update(env)
+          c.KubeSpawner.environment.update(env)

hub-templates/daskhub/Chart.yaml

@@ -2,10 +2,9 @@ apiVersion: v2
 appVersion: '1.0'
 description: Deployment Chart for a dask-enabled JupyterHub
 name: daskhub
-version: 0.0.1-n2724.h1dfae31
+version: "0.1.0"
 dependencies:
  - name: basehub
-   version: 0.0.1
    repository: file://../basehub
  - name: dask-gateway
    version: "0.9.0"

hub-templates/daskhub/values.yaml

@@ -41,7 +41,6 @@ basehub:
               - port: 22
                 protocol: TCP
       cloudMetadata:
-        enabled: true
         # Don't block access to AWS cloud metadata
         # If we don't, our users can't access S3 buckets / other AWS services
         # without an explicit identity

hub-templates/images/hub/Dockerfile

@@ -1,6 +1,6 @@
 # Should match the hub image used by version of chart in hub/requirements.yaml
 # If that changes, this should be changed too!
-FROM jupyterhub/k8s-hub:0.11.1-n298.hd9d7403b
+FROM jupyterhub/k8s-hub:1.0.0-beta.1
 
 ENV CONFIGURATOR_VERSION ed7e3a0df1e3d625d10903ef7d7fd9c2fbb548db
 

requirements.txt

@@ -1,5 +1,7 @@
 backoff
+chartpress
+six # temp workaround for docker==5.0.0, used by chartpress, that failed to depend on six
 ruamel.yaml
 auth0-python
 jhub-client==0.1.2
-jsonschema
+jsonschema

support/Chart.yaml

@@ -1,6 +1,30 @@
-apiVersion: v1
-appVersion: '1.0'
-description: Support chart for entire cluster
+apiVersion: v2
 name: support
-# Let's keep this constant so other charts in this repo can depend on this easily
-version: 0.0.1
+version: "0.1.0"
+description: Cluster wide depdencies for deployed hubs
+
+dependencies:
+  # Prometheus for collection of metrics.
+  # https://github.com/prometheus-community/helm-charts/tree/main/charts/prometheus
+  - name: prometheus
+    version: 11.15.0
+    repository: https://prometheus-community.github.io/helm-charts
+
+  # Grafana for dashboarding of metrics.
+  # https://github.com/grafana/helm-charts/tree/main/charts/grafana
+  - name: grafana
+    version: 5.6.4
+    repository: https://grafana.github.io/helm-charts
+
+  # ingress-nginx for a k8s Ingress resource controller that routes traffic from
+  # a single IP entrypoint to various services exposed via k8s Ingress resources
+  # that references this controller.
+  - name: ingress-nginx
+    version: 2.15.0
+    repository: https://kubernetes.github.io/ingress-nginx
+
+  # cert-manager for acquisition of TLS certificates
+  # https://github.com/jetstack/cert-manager/tree/master/deploy/charts/cert-manager
+  - name: cert-manager
+    version: v1.0.0-beta.1
+    repository: https://charts.jetstack.io

support/README.md

@@ -0,0 +1,11 @@
+## FIXME: Describe this Helm chart
+
+Describe more about this Helm chart. What are the assumptions made within it for
+example, and what are the assumptions on basehub and daskhub with regards to
+what is available within the k8s cluster. Do they rely on ingress-nginx and
+cert-manager for example?
+
+### Observations
+- This Helm chart contains config specific to 2i2c k8s cluster, as noted by a
+  domain name configured for grafana.
+- This Helm chart contain a template creating a GCP specific k8s StorageClass