MonkeyCanCode commented on code in PR #3597:
URL: https://github.com/apache/polaris/pull/3597#discussion_r2738585326
##########
getting-started/telemetry/docker-compose.yml:
##########
@@ -65,7 +65,12 @@ services:
- CLIENT_SECRET=s3cr3t
volumes:
- ../assets/polaris/:/polaris
- entrypoint: '/bin/sh -c "chmod +x /polaris/create-catalog.sh &&
/polaris/create-catalog.sh"'
+ entrypoint: "/bin/sh"
+ command:
+ - "-c"
+ - >-
+ chmod +x /polaris/create-catalog.sh;
Review Comment:
Well, that is very good point. Upon checking, we can just remove chmod line
completely as the file is 755 already and here is the mounted file permission:
```
/ # ls -l /polaris/create-catalog.sh
-rwxr-xr-x 1 root root 2559 Dec 20 20:51
/polaris/create-catalog.sh
```
So we should just move chmod line.
--
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]
