Skip to content

Add trace for output returned from resource#1447

Merged
SteveL-MSFT merged 1 commit intoPowerShell:mainfrom
SteveL-MSFT:output-trace
Mar 26, 2026
Merged

Add trace for output returned from resource#1447
SteveL-MSFT merged 1 commit intoPowerShell:mainfrom
SteveL-MSFT:output-trace

Conversation

@SteveL-MSFT
Copy link
Member

PR Summary

While helping to investigate an issue with a new resource where dsc errors that the schema didn't match, it required adding traces to the resource to emit the JSON it was returning. This change adds the raw output from a resource as a trace in DSC to help diagnose JSON schema issues with output from resources.

image

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds more detailed tracing around JSON verification for command-based DSC resources to help diagnose schema mismatches in resource output.

Changes:

  • Emit a trace log when verifying JSON from a resource, including the raw JSON content.
  • Update the localized verifyJson message to include the JSON payload.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
lib/dsc-lib/src/dscresources/command_resource.rs Changes JSON verification logging to trace and includes the JSON string in the message.
lib/dsc-lib/locales/en-us.toml Updates the verifyJson localized string to include a %{json} placeholder.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SteveL-MSFT SteveL-MSFT added this pull request to the merge queue Mar 26, 2026
Merged via the queue into PowerShell:main with commit 35ea75b Mar 26, 2026
23 checks passed
@SteveL-MSFT SteveL-MSFT deleted the output-trace branch March 26, 2026 21:53
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

Successfully merging this pull request may close these issues.

3 participants