Skip to content

Commit 0c89358

Browse files
committed
ci: avoid helm repo update for performance reasons
1 parent ad32bf0 commit 0c89358

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

.github/workflows/test.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,15 +78,11 @@ jobs:
7878
- name: Install JupyterHub chart for main tests
7979
if: matrix.test == 'main'
8080
run: |
81-
helm repo add jupyterhub https://jupyterhub.github.io/helm-chart/
82-
helm repo update
8381
./testing/local-binder-k8s-hub/install-jupyterhub-chart
8482
8583
- name: Install JupyterHub chart for auth tests
8684
if: matrix.test == 'auth'
8785
run: |
88-
helm repo add jupyterhub https://jupyterhub.github.io/helm-chart/
89-
helm repo update
9086
./testing/local-binder-k8s-hub/install-jupyterhub-chart --auth
9187
9288
- name: Use chartpress to create the helm chart

testing/local-binder-k8s-hub/install-jupyterhub-chart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,8 @@ with NamedTemporaryFile(mode="w") as tmp:
3838

3939
cmd = ['helm', 'upgrade', '--install', 'binderhub-test']
4040
cmd.extend([
41-
'jupyterhub/jupyterhub',
41+
'jupyterhub',
42+
'--repo=https://jupyterhub.github.io/helm-chart/',
4243
f'--version={_get_jupyterhub_dependency_version()}',
4344
f'--values={tmp.name}',
4445
f'--values={os.path.join(here, "jupyterhub-chart-config.yaml")}',

0 commit comments

Comments
 (0)