openstack/puppet-openstack-integration
Code Issues Proposed changes

Don't hardcode user:group ownership after the "jenkins" name

Browse Source 
This will prevent permissions setting from failing when run under
another user which can be expected when running in other
environments or as openstack-infra phases out jenkins.

Change-Id: Id4cf39e2d4fd21604d359eb6ce6a3716b6bba931
Depends-On: I58f6ee3ce22a3826fadfb019d3c67be710ab95d7
This commit is contained in:
David Moreau-Simard 2016-06-22 16:42:58 -04:00 committed by David Moreau Simard
parent 04e82b16cf
commit a51051bc1a
1 changed files with 1 additions and 1 deletions

2
copy_logs.sh

@ -177,7 +177,7 @@ set -e
# Also make sure zuul can rsync all the logs and configs
sudo find $LOG_DIR -type d -execdir sudo chmod 755 '{}' \;
sudo find $LOG_DIR -type f -execdir sudo chmod 644 '{}' \;
sudo chown -R jenkins:jenkins $LOG_DIR
sudo chown -R "$(id -u):$(id -g)" $LOG_DIR
# do not try to save symlinks because source files might not have
# the right permissions to let jenkins user to upload them on log servers.