Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use_syslog option fixed #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bungeethefrog
Copy link

Use_syslog option was not working due to typo in template conf file (misspelled variable name) and fact that default value for log_file was set in data/common.yaml and therefore condition on line 46 in manifest/config.pp could never be satisfied with use_syslog set to true.

Alteration made:

  • template conf file:
    • variable name fixed
    • altered logic to set default log file value as needed, based on log_file and use_syslog variables
  • data/common.yaml file:
    • set null as default value for both log_file and use_syslog variables
  • manifest/config.pp and manifest/init.pp files:
    • set use_syslog variable as optional, since it does not have to be set always and in template conf file both 'null' and 'false' are handled as they should be
    • insignificant typo fix in manifest/config.pp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant