Skip to content

Commit

Permalink
add grpcio 1.57.0 pin to provided helm chart images (#26630)
Browse files Browse the repository at this point in the history
Summary:
Attempt to resolve reported memory leak versions with . Apparently the
underlying issue is also resolved in python 3.13, so once we officially
support that in dagster, we could potentially move these images to 3.13
and remove this.

Test Plan:
Deploy the helm chart locally with these built images

Changelog:
Added a pin to the grpcio version used by the Dagster Helm chart daemon
and webserver images, to a version that has been reported to avoid
memory leaks.

## Summary & Motivation

## How I Tested These Changes

## Changelog

> Insert changelog entry or delete this section.
  • Loading branch information
gibsondan authored Dec 22, 2024
1 parent 3f5a73a commit 0099d83
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,5 @@ RUN python -m uv pip install \
-e dagster-k8s \
-e dagster-celery[flower,redis,kubernetes] \
-e dagster-celery-k8s \
-e dagster-webserver
-e dagster-webserver \
grpcio==1.57.0 # https://github.com/grpc/grpc/issues/38327
Original file line number Diff line number Diff line change
Expand Up @@ -16,4 +16,5 @@ RUN python -m uv pip install \
dagster-celery-k8s \
dagster-gcp \
dagster-graphql \
dagster-webserver
dagster-webserver \
grpcio==1.57.0 # https://github.com/grpc/grpc/issues/38327
Original file line number Diff line number Diff line change
Expand Up @@ -12,4 +12,5 @@ RUN python -m uv pip install \
-e dagster-postgres \
-e dagster-k8s \
-e dagster-aws \
-e dagster-webserver
-e dagster-webserver \
grpcio==1.57.0 # https://github.com/grpc/grpc/issues/38327
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,5 @@ RUN python -m uv pip install \
dagster-aws \
dagster-gcp \
dagster-graphql \
dagster-webserver
dagster-webserver \
grpcio==1.57.0 # https://github.com/grpc/grpc/issues/38327

0 comments on commit 0099d83

Please sign in to comment.