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

[DONT MERGE], only for windows perf log #112980

Closed
wants to merge 1 commit into from

Conversation

klensy
Copy link
Contributor

@klensy klensy commented Jun 23, 2023

Trying to collect etw log from windows builder to check what's going on.

yaml escaping looks sus.

r? @ghost

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Jun 23, 2023
@klensy klensy force-pushed the etw-win-ci branch 9 times, most recently from 38f17f2 to cc5f88d Compare June 23, 2023 21:08
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@klensy klensy force-pushed the etw-win-ci branch 4 times, most recently from fb60ddd to f9c578b Compare June 24, 2023 09:58
@klensy
Copy link
Contributor Author

klensy commented Jun 24, 2023

run: https://github.com/rust-lang/rust/actions/runs/5360795506
First thing found in first seconds of run is EdgeUpdate, actively writing to disk. Yeah, expected.

TODO:

@klensy klensy force-pushed the etw-win-ci branch 2 times, most recently from 7bdc0ba to bdd4e07 Compare June 26, 2023 14:14
@rust-log-analyzer

This comment has been minimized.

@klensy klensy force-pushed the etw-win-ci branch 2 times, most recently from a854e1b to 52b6668 Compare June 29, 2023 17:42
@bors
Copy link
Contributor

bors commented Jul 29, 2023

☔ The latest upstream changes (presumably #114148) made this pull request unmergeable. Please resolve the merge conflicts.

@Dylan-DPC
Copy link
Member

Closing this at it was an experiment

@Dylan-DPC Dylan-DPC closed this Nov 2, 2023
@Dylan-DPC Dylan-DPC removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants