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

logging into stdout is not supported #510

Closed
mqmr opened this issue Apr 9, 2024 · 1 comment
Closed

logging into stdout is not supported #510

mqmr opened this issue Apr 9, 2024 · 1 comment
Labels
invalid This doesn't seem right

Comments

@mqmr
Copy link

mqmr commented Apr 9, 2024

[NOT A CONTRIBUTION]

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 7.23.0
  • Ruby: 3.1.2p20
  • Distribution: Debian 12
  • Module version: 10.0.0

How to reproduce (e.g Puppet code you use)

What are you seeing

What behaviour did you expect instead

Output log

Any additional information you'd like to impart

Redis supports sending logs to stdout, which the modules does not support because of "log_dir" parameter.

# Specify the log file name. Also the empty string can be used to force
# Redis to log on the standard output. Note that if you use standard
# output for logging but daemonize, logs will be sent to /dev/null
logfile ""

(source: https://github.com/redis/redis/blob/7.2.4/redis.conf#L352C1-L355C11)

Would it be possible to support logging into stdout?

Thank you

@mqmr
Copy link
Author

mqmr commented Apr 10, 2024

I found a workaround with using the "extra_config_file" property.

@mqmr mqmr closed this as not planned Won't fix, can't repro, duplicate, stale Apr 10, 2024
@evgeni evgeni added the invalid This doesn't seem right label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants