Skip to content

Conversation

@kanwal-kaur
Copy link
Contributor

Description

The command doesn't work as intended. Explanation for the fix:

  1. Windows requires the GUID inside quotes because it's treated as a string (the provider name) in this context. Without quotes, the command parser may interpret the braces or hyphens incorrectly.
  2. Passing 0x0 as the only flag after the provider GUID is valid. Supplying two numbers like 0 0 (as in some bad examples) misleads logman, which tries to interpret the second value as outputFormat.
  3. The -o trace.etl must be at the end and not mistaken for a different parameter like -f (output format), unless you're also explicitly setting -f to text or xml.

Related issues or tickets

Worked out in Case number: 00164653

Reviews

  • Technical review
  • Editorial review
  • Product review

The command doesn't work as intended. 
Explanation for the fix:
1. Windows requires the GUID inside quotes because it's treated as a string (the provider name) in this context. Without quotes, the command parser may interpret the braces or hyphens incorrectly.
2. Passing 0x0 as the only flag after the provider GUID is valid. Supplying two numbers like 0 0 (as in some bad examples) misleads logman, which tries to interpret the second value as outputFormat.
3. The -o trace.etl must be at the end and not mistaken for a different parameter like -f (output format), unless you're also explicitly setting -f to text or xml.
@netlify
Copy link

netlify bot commented Aug 1, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 0ca5b57
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/688ce839da23c8000835815c
😎 Deploy Preview https://deploy-preview-23193--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the area/engine Issue affects Docker engine/daemon label Aug 1, 2025
@thaJeztah
Copy link
Member

Thanks for contributing! @chelnak could you have a peek to verify the change? I'm not on a Windows machine myself, so both "before" and "after" look fine to me 😂 ❤️

@chelnak
Copy link
Contributor

chelnak commented Aug 8, 2025

Hey! I Tested with PowerShell 7, PowerShell 5 and cmd.exe. The new version seems to work in all three.

Whereas previously the example would have only worked in cmd.exe.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for testing!!!!

@thaJeztah thaJeztah merged commit d86d4f7 into docker:main Aug 8, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/engine Issue affects Docker engine/daemon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants