Skip to content

Commit

Permalink
CI: separate pebble/jh chart config
Browse files Browse the repository at this point in the history
  • Loading branch information
consideRatio committed Jun 8, 2020
1 parent 310aa5e commit 32838b3
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ install:
- helm repo update

# Install a local ACME server
- helm install pebble jupyterhub/pebble --values dev-config.yaml
- helm install pebble jupyterhub/pebble --values dev-config-pebble.yaml

# Build our images if needed and update values.yaml with the tags
- chartpress
Expand Down
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ __Install Pebble__
```shell
helm repo add jupyterhub https://jupyterhub.github.io/helm-chart/
helm repo update
helm install pebble jupyterhub/pebble --values dev-config.yaml
helm install pebble jupyterhub/pebble --values dev-config-pebble.yaml
```

## 4: Build images, update values, install chart
Expand Down
7 changes: 7 additions & 0 deletions dev-config-pebble.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
# This configuration is meant to map lookups of local.jovyan.org to the
# ClusterIP of JupyterHub's proxy-public service
coredns:
corefileSegment: |-
template ANY ANY local.jovyan.org {
answer "{{ .Name }} 60 IN CNAME proxy-public.{$PEBBLE_NAMESPACE}.svc.cluster.local"
}
8 changes: 0 additions & 8 deletions dev-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -75,11 +75,3 @@ prePuller:

debug:
enabled: true

# A few values for the Pebble Helm chart, allowing us to fool Pebble to resolve
# all .test domains to our proxy-public service.
coredns:
corefileSegment: |-
template ANY ANY local.jovyan.org {
answer "{{ .Name }} 60 IN CNAME proxy-public.{$PEBBLE_NAMESPACE}.svc.cluster.local"
}

0 comments on commit 32838b3

Please sign in to comment.