Issue #3145 has been updated by Tony Garcia.

Confirmed on 0.24.8

Doing via command line with: <code>puppetd 
--logdest=/var/log/puppetd.log</code> works but when attempting to workaround 
with: <code>PUPPET_EXTRA_OPTS="--logdest=/var/log/puppet/puppetd.log"</code> in 
/etc/sysconfig/puppet when running as daemon I only see few notifications:

When starting the daemon:
<pre>
Tue Oct 12 16:33:30 +0000 2010 Puppet (notice): Reopening log files
</pre>

When Stopping the daemon:
<pre>
Tue Oct 12 16:33:30 +0000 2010 Puppet (notice): Starting Puppet client version 
0.24.8
Tue Oct 12 16:33:49 +0000 2010 Puppet (notice): Starting catalog run
Tue Oct 12 16:34:21 +0000 2010 Puppet (notice): Finished catalog run in 31.50 
seconds
Tue Oct 12 16:34:22 +0000 2010 Puppet (notice): Caught TERM; shutting down
Tue Oct 12 16:34:22 +0000 2010 Puppet (notice): Shutting down
</pre>


If I add the <code>--verbose</code> option I'm able to see few more logs(info) 
but only until certain point:
<pre>
Tue Oct 12 16:39:39 +0000 2010 Puppet (info): Loading fact custom1
Tue Oct 12 16:39:39 +0000 2010 Puppet (info): Loading fact custom2
Tue Oct 12 16:39:39 +0000 2010 Puppet (notice): Reopening log files
Tue Oct 12 16:39:39 +0000 2010 Puppet (info): Loading fact custom1
Tue Oct 12 16:39:39 +0000 2010 Puppet (info): Loading fact custom2
</pre>

Adding <code>--debug</code> shows a more complete output but we don't really 
want to have debug messages into the logs.
----------------------------------------
Bug #3145: Logging options are now out of date
http://projects.puppetlabs.com/issues/3145

Author: James Turnbull
Status: Accepted
Priority: Normal
Assignee: James Turnbull
Category: plumbing
Target version: 0.25.6
Affected version: 0.25.4
Keywords: 
Branch: 


No code seems to use the following options:

masterlog, puppetdlog

The following test is the only place that mentions masterlog:

<pre>
        it "should use the masterlog if the process name is 'puppetmasterd'" do
            Puppet.settings.stubs(:value).with(:name).returns "puppetmasterd"
            Puppet.settings.expects(:value).with(:masterhttplog).returns 
"/master/log"

            File.expects(:open).with("/master/log", "a+").returns @filehandle

            @server.setup_logger
        end 

        it "should use the httplog if the process name is not 'puppetmasterd'" 
do
            Puppet.settings.stubs(:value).with(:name).returns "other"
            Puppet.settings.expects(:value).with(:httplog).returns "/other/log"

            File.expects(:open).with("/other/log", "a+").returns @filehandle

            @server.setup_logger
        end 
</pre>

If you reference the code:

<pre>
    def setup_logger
        # Make sure the settings are all ready for us.
        Puppet.settings.use(:main, :ssl, Puppet[:name])

        if Puppet[:name] == "puppetmasterd"
            file = Puppet[:masterhttplog]
        else
            file = Puppet[:httplog]
        end 
</pre>

I think the test means should use masterhttplog.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to