lhotari commented on code in PR #22613:
URL: https://github.com/apache/pulsar/pull/22613#discussion_r1583601316
##########
docker/pulsar/Dockerfile:
##########
@@ -59,16 +59,16 @@ RUN apk add --no-cache \
musl-dev \
libffi-dev \
py3-pip \
- py3-grpcio \
py3-yaml
RUN pip3 install --break-system-packages \
kazoo
ARG PULSAR_CLIENT_PYTHON_VERSION
+# Once PyPI provides the grpcio-aarch64 package, we can remove
https://wheel-index.linuxserver.io/alpine-3.19/
RUN pip3 install --break-system-packages \
- pulsar-client[all]==${PULSAR_CLIENT_PYTHON_VERSION}
-
+ pulsar-client[all]==${PULSAR_CLIENT_PYTHON_VERSION} \
+ --prefer-binary --find-links
https://wheel-index.linuxserver.io/alpine-3.19/
Review Comment:
> This solution comes from
https://github.com/grpc/grpc/issues/27512#issuecomment-1030884361
>
> Non Alpine official solution.
What makes this solution "official"?
I think a better solution is to lower the required grpc version in
pulsar-client-python to 1.59.3 so that no external solution wouldn't be
required at all.
The grpc version should match apache-bookkeeper-client's grpc version to
ensure compatibility of generated stubs.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
