-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat(windows-agent): Redirect agent logs to LocalAppData #379
Conversation
a661b9c
to
3fca65e
Compare
Printing them when tests succeed causes too much spam when running with the verbose flag (yes, it's too verbose for verbose mode).
It fixes a couple issues in the logs and makes it more similar to the new logWindowsAgentJournal.
1b9e20e
to
e7c1c31
Compare
Rebased main, no new changes. |
and rename it to a more accurate logWslProServiceOnError.
and rename it to a more accurate logWindowsAgentOnError.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍🏽
Thanks for considering my point on the parameter. I hope didrocks won't disagree :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have nothing to add on top of this.
Very useful for end-to-end testing.
UDENG-1732