Skip to content

debugging: add a bunch of tracing around archive#1784

Closed
chriskrycho wants to merge 2 commits intomainfrom
chriskrycho/debug-1744
Closed

debugging: add a bunch of tracing around archive#1784
chriskrycho wants to merge 2 commits intomainfrom
chriskrycho/debug-1744

Conversation

@chriskrycho
Copy link
Copy Markdown
Contributor

@chriskrycho chriskrycho commented Jul 8, 2024

Background/Notes

This is not really intended to be merged as is, but it should allow folks to do some testing and see if we can get more info for #1744.

In the interest of possibly being able to build on this for merging it if we find that useful in the future, I used trace! rather than debug! for this extreme level of output, and added a --very-verbose flag for this mode.

Note: This is based on v1.1.1, not on the latest main, and I have intentionally marked it as a draft until we get more info from #1744.

To Test

git clone https://github.com/volta-cli/volta.git
cd volta
git switch chriskrycho/debug-1744
cargo build --release
mv $HOME/.volta/bin/volta $HOME/.volta/bin/volta-backup
cp ./target/release/volta $HOME/.volta/bin/volta
volta --verbose --very-verbose

That final line should print the normal help output preceded by this line:

[very verbose] log level: VeryVerbose

(If you have a non-standard installation location, you will want to figure out where it is with which volta and do the same basic thing for the final two steps, but with the appropriate directory changes.)

Once you have that set up, you should be able to get much more info about where in the archive/unpacking process things are going wrong.

This is not really intended to be merged as is, but it should allow
folks to do some testing and see if we can get more info for #1744.

In the interest of *possibly* being able to build on this for merging it
if we find that useful in the future, I used `trace!` rather than
`debug!` for this extreme level of output, and added a `--very-verbose`
flag for this mode.
@chriskrycho
Copy link
Copy Markdown
Contributor Author

Closing this as we have (at least theoretically!) resolved the issue it was intended to help debug: #1744. The useful-for-the-future parts have separately been extracted to #1793 and #1815.

@chriskrycho chriskrycho deleted the chriskrycho/debug-1744 branch July 30, 2024 16:56
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.

1 participant