deajan commented on PR #10723: URL: https://github.com/apache/cloudstack/pull/10723#issuecomment-2810163596
The potential problem here is that if `/usr/bin/logger` does not exist on the target OS, the script must fallback to log() function which would require `read` command so it can accept getting stdout /stderr through pipes. AFAIK `read` is a bash builtin, so the script would become interpreter dependant. I tend to use that kind of log function so I always get my logs, regardless of the environment. This would only be a problem of course if `/usr/bin/logger` is not present by default on all Cloudstack supported distros. It is definitly on AlmaLinux 9.5. I can't say for the others. @DaanHoogland So, do I switch to logger with a fallback to log() knowing this ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
