Skip to content

Add capability of outputting DxDiag when running pipelines #1056

Open
joaosaffran wants to merge 20 commits intollvm:mainfrom
joaosaffran:pipeline/run-dxdiag-windows
Open

Add capability of outputting DxDiag when running pipelines #1056
joaosaffran wants to merge 20 commits intollvm:mainfrom
joaosaffran:pipeline/run-dxdiag-windows

Conversation

@joaosaffran
Copy link
Copy Markdown
Contributor

@joaosaffran joaosaffran commented Apr 1, 2026

This patch makes pipelines emit dxdiag information when failing.

@joaosaffran joaosaffran added the test-all When applied to a PR this will opt-in to additional pre-merge test configurations.. label Apr 1, 2026
@joaosaffran joaosaffran changed the title [DO NOT REVIEW] Add DxDiag output when running pipelines Add capability of outputting DxDiag when running pipelines Apr 2, 2026
@joaosaffran joaosaffran marked this pull request as ready for review April 2, 2026 06:03
Comment on lines +108 to +112
run_dxdiag:
description: "Run dxdiag"
required: false
default: false
type: boolean
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Any reason not to just run this all the time? Seems that we'd want to grab the dxdiag output when we see a test run has failed and so it'd be nice if that was always available with the rest of the results.

- name: Check if dxdiag report exists
id: check_dxdiag
if: inputs.OS == 'windows' && inputs.run_dxdiag
shell: powershell
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there a reason this has to be a separate step?

I'm wondering if you ended up doing this to work around dxdiag running asynchronously, which means that the "DxDiag report saved to $output" message gets printed before the output is actually? In which case this would only happen to work by luck and sometimes you'd not get output at all.

If you run dxdiag like this in powershell:

dxdiag /t $output | Out-Null

Then that command will only complete once the file has been written, and so you can safely assume it is there.

@joaosaffran joaosaffran marked this pull request as draft April 2, 2026 17:52
@joaosaffran joaosaffran marked this pull request as ready for review April 2, 2026 20:53
@joaosaffran joaosaffran requested a review from damyanp April 2, 2026 20:53
Copy link
Copy Markdown
Contributor

@damyanp damyanp left a comment

Choose a reason for hiding this comment

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

LGTM, although I wonder if we need to limit this to just failures? If we suspect that a failure may be caused by a driver update then we'd want to compare dxdiag between a successful run and the failing run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-all When applied to a PR this will opt-in to additional pre-merge test configurations..

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants