Skip to content

Commit

Permalink
[FIXED JENKINS-26982] MacOSX: re-use jenkins user default group for g…
Browse files Browse the repository at this point in the history
…roup own�$B!D�(Bership permissions

Porting
lacostej/jenkins@66cb50d
  • Loading branch information
kohsuke committed Apr 29, 2015
1 parent 1f9ef5c commit 26b1323
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions osx/scripts/postinstall-launchd-jenkins
Expand Up @@ -44,7 +44,13 @@ else
dscl . -append /Groups/@@ARTIFACTNAME@@ GroupMembership @@ARTIFACTNAME@@
fi

find "$JENKINS_HOMEDIR" \( -not -user @@ARTIFACTNAME@@ -or -not -group @@ARTIFACTNAME@@ \) -print0 | xargs -0 chown @@ARTIFACTNAME@@:@@ARTIFACTNAME@@
# identify the real default group name for user jenkins
groupid=`dscl . read /Users/@@ARTIFACTNAME@@ PrimaryGroupID | awk '{print $2}'`
gname=`id -n -g $groupid`

echo "Using @@ARTIFACTNAME@@:${gname} as file owner and group for @@ARTIFACTNAME@@ daemon files"

find "$JENKINS_HOMEDIR" \( -not -user @@ARTIFACTNAME@@ -or -not -group ${gname} \) -print0 | xargs -0 chown @@ARTIFACTNAME@@:${gname}

# Add defaults for heap sizing
arch=$(uname -m)
Expand All @@ -67,17 +73,17 @@ defaults write $DEFAULTS_PLIST httpPort @@PORT@@
JENKINS_TMPDIR="$JENKINS_HOMEDIR/tmp"
defaults write $DEFAULTS_PLIST tmpdir $JENKINS_TMPDIR
mkdir -p $JENKINS_TMPDIR
chown @@ARTIFACTNAME@@:@@ARTIFACTNAME@@ $JENKINS_TMPDIR
chown @@ARTIFACTNAME@@:${gname} $JENKINS_TMPDIR

# Create log directory, which can be written by Jenkins daemon
mkdir -p /var/log/@@ARTIFACTNAME@@
chown @@ARTIFACTNAME@@:@@ARTIFACTNAME@@ /var/log/@@ARTIFACTNAME@@
chown @@ARTIFACTNAME@@:${gname} /var/log/@@ARTIFACTNAME@@

# Enable log rotation by newsyslog
cat <<_EOT_ > /etc/newsyslog.d/@@ARTIFACTNAME@@.conf
# logfilename [owner:group] mode count size when flags [/pid_file] [sig_num]
# Rotate @@PRODUCTNAME@@ log at midnight, and preserve old logs in 3 days
/var/log/@@ARTIFACTNAME@@/@@ARTIFACTNAME@@.log @@ARTIFACTNAME@@:@@ARTIFACTNAME@@ 644 3 * \$D0 J
/var/log/@@ARTIFACTNAME@@/@@ARTIFACTNAME@@.log @@ARTIFACTNAME@@:${gname} 644 3 * \$D0 J
_EOT_

# Load and start the launch daemon
Expand Down

2 comments on commit 26b1323

@daniel-beck
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice commit message 😉

@kohsuke
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like some kind of unicode glitch. Oh well.

Please sign in to comment.