Skip to content

diagnostics: switch log archive from zip to tar.gz#14530

Merged
benhillis merged 3 commits intomasterfrom
user/benhill/collect_logs_tar_gz
Apr 28, 2026
Merged

diagnostics: switch log archive from zip to tar.gz#14530
benhillis merged 3 commits intomasterfrom
user/benhill/collect_logs_tar_gz

Conversation

@benhillis
Copy link
Copy Markdown
Member

This change moves from using Compress-Archive to the inbox tar.exe. Compress-Archive can fail for very large files, so this approach will be more reliable.

This change also requires an update to WTI which I've published here: OneBlue/wti#5

Copy link
Copy Markdown
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

Switches WSL diagnostics log archival from Compress-Archive (.zip) to tar.exe (.tar.gz) to improve reliability for very large log sets.

Changes:

  • Generate .tar.gz archives via tar.exe -czf instead of Compress-Archive
  • Add success/failure branching based on native exit code, including cleanup of partial archives
  • Improve user-facing messaging when compression fails

Comment thread diagnostics/collect-wsl-logs.ps1 Outdated
Comment thread diagnostics/collect-wsl-logs.ps1
Comment thread diagnostics/collect-wsl-logs.ps1
Some Windows SKUs may not have tar.exe available. Fall back to
Compress-Archive (zip) when tar fails, so log collection still
works in those environments.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
OneBlue
OneBlue previously approved these changes Apr 6, 2026
@benhillis
Copy link
Copy Markdown
Member Author

@OneBlue - we need to merge this first:
OneBlue/wti#5

The new WTI release adds support for tar.gz log archives, matching
the format change in collect-wsl-logs.ps1.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 22, 2026 21:01
Copy link
Copy Markdown
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

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

Comment thread diagnostics/collect-wsl-logs.ps1
Comment thread .github/actions/triage/action.yml
@benhillis benhillis enabled auto-merge (squash) April 28, 2026 16:06
@benhillis benhillis merged commit 6aa0b0d into master Apr 28, 2026
13 checks passed
@benhillis benhillis deleted the user/benhill/collect_logs_tar_gz branch April 28, 2026 16:06
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