-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
k8s deploy error - Celery .Values.celery.image.repository is required #2509
Comments
Hi M, could you share that snippet in your values.yaml file where .Values.celery.image.repository is defined? |
Appears I missed configuring the values.yaml. Would there be an example I can refer to. When I look at say the celery section I have no image specified and assuming I need to reference a image repo. Is there any reference to images or Dockerfiles from which I can bootstrap ? |
I'm going to close the issue since the original question was resolved. Celery deprecated their official image in docker hub (https://hub.docker.com/_/celery); in the reasoning, they link to this comment (docker-library/celery#1 (comment)) which recommends using the application image |
@terekete this is the celery Dockerfile we've been using for testing: https://gist.github.com/natekupp/0eb55aec885358dd89ad8572db32877f You can likely do something similar, although you'll probably want to install from PyPI instead of using editable installs. |
thanks both! |
Hi Dagster Community,
Attempting to use the dagster-k8's module and running into an error:
Error: execution error at (dagster/templates/_helpers.tpl:36:3): Celery worker image repository .Values.celery.image.repository is required
helm.go:84: [debug] execution error at (dagster/templates/_helpers.tpl:36:3): Celery worker image repository .Values.celery.image.repository is required
I am just trying a dryrun: helm install dagster --dry-run --debug helm/dagster
Any help is appreciated.
Thanks,
M.
The text was updated successfully, but these errors were encountered: