puppetlabs/Razor — pull request #117 by tjmcs (2012-07-03 16:47:27+00:00)
Title: Fixing log write errors; take 2

Closed the previous pull request (#112) and modified the fix for the errors that pull request was intended to fix. In short those errors were as follows:

1. Any attempts to write to the log after the daemon process started resulted in a pair of errors being thrown to the razor_daemon.log file (a "log shifting failed. closed stream" message, followed by a "log writing failed. closed stream" message).
2. As a result, no output was sent to the project_razor.log file by the Razor server.
3. While debugging this issue, uncovered a potential bug in the ProjectRazor::Persist::Controller class where a single equals sign was being used for comparison in an "if" statement.

This set of modifications are much smaller, and more closely fit in with a proper usage model for Razor (using the CLI to get information from the 'config' slice rather than using the Razor APIs directly before the Razor server has started). As a result, it is better than (and directly replaces) the changes that were submitted as pull request #112 yesterday.