fix(enhancement): search for permissions on docker container instead of host and combine it in one command for performance enhancement (#3890)

* fix(enhancement): ensure correct ownership check before setting permissions of profiles

* fix(enhancement): search for permissions on docker container instead of host and combine it in one command for performance enhancement

Resolves #3882

* fix(enhancement): search for permissions on docker container instead of host
This commit is contained in:
Iven Schlenther 2025-08-19 20:39:04 +02:00 committed by GitHub
parent 7ef1b36b90
commit 657a685cbe
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -5,8 +5,8 @@
echo "${_group}Ensuring correct permissions on profiles directory ..."
# Check if the parent directory of /var/vroom/sentry-profiles is already owned by vroom:vroom
if [ "$(stat -c '%U:%G' /var/vroom)" = "vroom:vroom" ]; then
echo "Ownership of /var/vroom is already set to vroom:vroom. Skipping chown."
if [ "$($dcr --no-deps --entrypoint /bin/bash --user root vroom -c "stat -c '%U:%G' /var/vroom/sentry-profiles" 2>/dev/null)" = "vroom:vroom" ]; then
echo "Ownership of /var/vroom/sentry-profiles is already set to vroom:vroom. Skipping chown."
else
$dcr --no-deps --entrypoint /bin/bash --user root vroom -c 'chown -R vroom:vroom /var/vroom/sentry-profiles && chmod -R o+rwx /var/vroom/sentry-profiles'
fi