lhotari commented on code in PR #22054:
URL: https://github.com/apache/pulsar/pull/22054#discussion_r1497240252
##########
docker/pulsar/Dockerfile:
##########
@@ -26,82 +26,84 @@ ADD ${PULSAR_TARBALL} /
RUN mv /apache-pulsar-* /pulsar
RUN rm -rf /pulsar/bin/*.cmd
-COPY scripts/apply-config-from-env.py /pulsar/bin
-COPY scripts/apply-config-from-env-with-prefix.py /pulsar/bin
-COPY scripts/gen-yml-from-env.py /pulsar/bin
-COPY scripts/generate-zookeeper-config.sh /pulsar/bin
-COPY scripts/pulsar-zookeeper-ruok.sh /pulsar/bin
-COPY scripts/watch-znode.py /pulsar/bin
-COPY scripts/install-pulsar-client.sh /pulsar/bin
+COPY scripts/* /pulsar/bin/
# The final image needs to give the root group sufficient permission for
Pulsar components
# to write to specific directories within /pulsar
# The file permissions are preserved when copying files from this builder
image to the target image.
-RUN for SUBDIRECTORY in conf data download logs; do \
- [ -d /pulsar/$SUBDIRECTORY ] || mkdir /pulsar/$SUBDIRECTORY; \
- chmod -R g+w /pulsar/$SUBDIRECTORY; \
+RUN for SUBDIRECTORY in conf data download logs instances/deps; do \
+ mkdir -p /pulsar/$SUBDIRECTORY; \
+ chmod -R g+rwx /pulsar/$SUBDIRECTORY; \
Review Comment:
```suggestion
chmod -R ug+rwx /pulsar/$SUBDIRECTORY; \
chown -R 10000:0 /pulsar/$SUBDIRECTORY; \
```
To cover running with group id != 0, see #22081
--
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]
