Re: Pmlogger directory permissions

> On Jan 30, 2017, at 3:32 PM, Ken McDonell <[email protected]> wrote:
> 
> Excellent analysis Martins.
> 
> On 31/01/17 05:33, Martins Innus wrote:
>> ... 1.  In the default crontab setup, each day pmlogger_daily runs
>> before pmlogger_check.  While they share similar code, only
>> pmlogger_check has the code  to actually create the directory
>> specified in the control file. Not sure if this should be added into
>> the daily file, or if this was left out intentionally.
> 
> That is an error, that I'll fix.  The order of execution should be immaterial, which means any mkdir/chown activity in pmlogger_check should be replicated in pmlogger_daily.
> 
>> ...
>> 2.  The systemd/init scripts run as root.  So even though the
>> pmlogger startup script calls pmlogger_check, since it runs as root,
>> the directory tree gets created with root permissions.  There is a
>> single chown to fix the leaf directory.  But the parents are still
>> owned by root.
> 
> That is another error that I'll fix.  The mkdir logic needs to be smarter to mkdir/chown each of the missing directory components it is creating.
> 

Great, thanks Ken!

Martins


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links:

You receive all messages sent to this group.

View/Reply Online (#15051): https://groups.io/g/pcp/message/15051
View All Messages In Topic (3): https://groups.io/g/pcp/topic/4288595
Mute This Topic: https://groups.io/mt/4288595?uid=174580
New Topic: https://groups.io/g/pcp/post
-=-=-
pcp mailing list
[email protected]
https://groups.io/g/pcp/messages
-=-=-
Change Your Subscription: https://groups.io/g/pcp/editsub?uid=174580
Group Home: https://groups.io/g/pcp
Contact Group Owner: [email protected]
Terms of Service: https://groups.io/static/tos
Unsubscribe: https://groups.io/g/pcp/leave/354243/563757577/xyzzy
-=-=-=-=-=-=-=-=-=-=-=-