Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move template-sync sidecar to datahub secrets #2009

Merged
merged 1 commit into from
Dec 14, 2020

Conversation

yuvipanda
Copy link
Contributor

We override hub.extraContainers in secrets/staging.yaml
and secrets/prod.yaml to include the fetch-course-emails
container along the hub. But this overrides what's in
config/common.yaml destructively, since it just replaces
the list rather than add to it.

I've filed jupyterhub/kubespawner#468
to fix this. Until then, I've manually moved the extraContainers
config to the secret files.

We override `hub.extraContainers` in `secrets/staging.yaml`
and `secrets/prod.yaml` to include the fetch-course-emails
container along the hub. But this overrides what's in
`config/common.yaml` destructively, since it just replaces
the list rather than add to it.

I've filed jupyterhub/kubespawner#468
to fix this. Until then, I've manually moved the extraContainers
config to the secret files.
@yuvipanda yuvipanda merged commit 1744db3 into berkeley-dsep-infra:staging Dec 14, 2020
@yuvipanda yuvipanda deleted the fix-extracontainer branch December 14, 2020 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant