Skip to content
Snippets Groups Projects

Docker: update-permissions, only touch registry if exists

Merged Jason Plum requested to merge 2074-update-permissions-abort-when-no-registry into master
All threads resolved!

Add an exists check to the update-permissions script, so that it only attempts to set the permissions on the registry (/var/opt/gitlab/gitlab-rails/shared/registry) if it actually exists. This removes the spurious "fail" on setting permissions on things that don't exist, and allows for better scripting compatibility.

Set the /var/opt/gitlab/redis permissions to gitlab-redis:git if the git user exists. This brings the settings in-line with what is expected in the environment for the git user to be able to access the Redis instance.

Closes #2074 (closed)

Edited by Jason Plum

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jason Plum resolved all discussions

    resolved all discussions

  • Jason Plum added 1 commit

    added 1 commit

    • 811210f2 - Docker: update-permissions - alaways check path exists

    Compare with previous version

  • assigned to @marin

  • Jason Plum mentioned in merge request !1706 (merged)

    mentioned in merge request !1706 (merged)

  • Marin Jankovski approved this merge request

    approved this merge request

  • Marin Jankovski mentioned in commit cb7d860a

    mentioned in commit cb7d860a

  • Please register or sign in to reply
    Loading