Skip to content

Commit

Permalink
Merge pull request #66 from scline/1.2.11a
Browse files Browse the repository at this point in the history
Changing php-fpm settings to use apache as socket owner vs root.
  • Loading branch information
scline committed Apr 18, 2020
2 parents cd8949f + b83337a commit e6d5782
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -199,6 +199,10 @@ REPLACE INTO `%DB_NAME%`.`settings` (`name`, `value`) VALUES('poller_type', '2')
```

# Change Log
#### 1.2.11a - 04/18/2020
* Close issue [#64](https://github.com/scline/docker-cacti/issues/64) - FPM initialization failed; Thanks to [kevburkett](https://github.com/kevburkett) for bringing this one up and helping validate the fix
* Close issue [#65](https://github.com/scline/docker-cacti/issues/65) - PHP-SNMP yum commands cause rpmdb error

#### 1.2.11 - 04/17/2020
* Update Docker container to use Centos8 over Centos7
* Close issue [#59](https://github.com/scline/docker-cacti/issues/59) - errors with percona on compose single instance; Thank you [miguelwill](https://github.com/miguelwill)
Expand Down
10 changes: 8 additions & 2 deletions start.sh
Original file line number Diff line number Diff line change
Expand Up @@ -176,9 +176,15 @@ snmpd -Lf /var/log/snmpd.log &

# start php-fpm
echo "$(date +%F_%R) [Note] Starting php-fpm service."
rm -rf /run/php-fpm
# make sure socket and pid files are all cleaned before starting
mkdir /run/php-fpm
php-fpm --allow-to-run-as-root --nodaemonize &
rm -rf /run/php-fpm/*
# change settings in php-fpm file due to https://github.com/scline/docker-cacti/issues/64
sed -i -e "s/;listen.owner = nobody/listen.owner = apache/g" \
-e "s/;listen.group = nobody/listen.group = apache/g" \
-e "s/listen.acl_users = apache,nginx/;listen.acl_users = apache,nginx/g" \
/etc/php-fpm.d/www.conf
php-fpm

# start web service
echo "$(date +%F_%R) [Note] Starting httpd service."
Expand Down

0 comments on commit e6d5782

Please sign in to comment.