dheeraj12347 commented on PR #12967:
URL: https://github.com/apache/cloudstack/pull/12967#issuecomment-4273035240
@DaanHoogland I’ve updated this PR based on the feedback:
– For management/server/agent I’ve kept the existing CONSOLE appender on
SYSTEM_OUT and added a CONSOLE_ERR appender on SYSTEM_ERR with a timestamped
pattern so *.err now gets timestamped stderr lines, in line with #12887.
– ERROR/FATAL events now go to CONSOLE_ERR (and thus *.err) instead of
*.out, which makes .err the primary place for timestamped error diagnostics;
I’m happy to adjust in a follow‑up if you’d prefer duplication in both streams.
– For Usage I’ve added a CONSOLE_ERR appender targeting SYSTEM_ERR with
%d{ABSOLUTE}{GMT} ... and updated the PR description to mention the Usage
config and the actual patterns used.
– I also addressed the whitespace-only comments that Copilot pointed out.
From my side this is ready for another review and, if you’re happy with the
behaviour, ready to merge.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
