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

v4.2 Bad error reporting #156

Open
RoYoMi opened this issue Apr 12, 2022 · 3 comments
Open

v4.2 Bad error reporting #156

RoYoMi opened this issue Apr 12, 2022 · 3 comments

Comments

@RoYoMi
Copy link

RoYoMi commented Apr 12, 2022

Problem

For those of us using the Configmgr Task Sequence Monitor we'd appreciate a better script termination reporting on failures.

For example this is what we see in when the script fails. The message reads InnerTerminatingFailure. The problem is that every single error will report this same message. This requires the user to then crack open the ApplyDriversPackage.log to see exactly what happened.
image

Recommendation

We'd rather see a descriptive message like this - Validation failed with empty list of matched driver packages for 'ThinkPad X1 Yoga 2nd', script execution will be terminated instead of the meaningless InnerTerminatingFailure .
With this type of message we could immediately start reviewing the root cause.
image

Solution

In theory this could be accomplished by make the following changes every where you're calling New-TerminatingErrorRecord
image

@TANielsen
Copy link

Yea, I hate seeing red in my smsts.log

@RoYoMi
Copy link
Author

RoYoMi commented May 1, 2022

No doubt the red is bad, but I hate when the logs lack the necessary details to investigate a problem. There should be a happy compromise in how the script writes these logs.

@stephannn
Copy link

Good call. I added it to our version: #226

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

No branches or pull requests

3 participants