Possible FOG installer/updater chmod typo?

[utopia]

Server

• FOG Version: 1.4.0-RC-5
• OS: Ubuntu 14.04.01 LTS

Description

The following looks like a possible typo in the fog installer/updater for chmod.

“chmod: cannot access ‘/var/log/php*fpm’: No such file or directory”

It should probably be set to “/var/log/php*fpm.log”
instead of the current string “/var/log/php*fpm”.

It’s possible this is just a remnant from when my server was updated from php5 to php7.0, since I did have problems with that as outlined in the already solved topic found at https://forums.fogproject.org/post/83191 . However, I thought I would post it anyway since it’s possible others could be affected as well, and it will apparently prevent php logging for FOG. Can anyone else confirm this error in their …/fogproject/bin/error_logs/fog_error_…log file? Or is it just me?

root@Fog01:/# date
Sat Apr 15 16:35:59 2017
root@Fog01:/# tail -5 /git/fogproject/bin/error_logs/fog_error_1.4.0-RC-5.log 
 * Exporting directories for NFS kernel daemon...
   ...done.
 * Starting NFS kernel daemon
   ...done.
chmod: cannot access ‘/var/log/php*fpm’: No such file or directory
root@Fog01:/# ls -l /var/log | grep php
-rw-------  1 root              root         0 Nov 13 13:13 php5-fpm.log
-rw-------  1 root              root       291 Nov 13 12:52 php5-fpm.log.1
... (unnecessary output removed) ...
-rw-------  1 root              root       123 Sep 13  2016 php5-fpm.log.9.gz
-rw-------  1 root              root         0 Nov 13 13:17 php7.0-fpm.log
-rw-------  1 root              root         0 Mar 24 20:14 php7.1-fpm.log

[Tom Elliott]

That command is correct, except I think it’s as you said (leftovers).

I’ll update the installer to hopefully correct for this as well.

The reason for the command is to update a folder named php-fpm (though it could be anything like php5-fpm php7.0-fpm etc…

[utopia]

So within the /var/log folder I should have a subfolder named php7.1-fpm. Then what should it’s ownership, permissions, and contents be if I choose to try to create it manually? Or should I just wait for a dev-branch update to do this for me?

[Tom Elliott]

@utopia wait for update. It wasn’t really hitting anything so not the end if the world I don’t think.

[utopia]

Will do. I agree, not an issue for me. Thanks for letting me know.