-
Notifications
You must be signed in to change notification settings - Fork 2k
add logs command to turbo CLI #827
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
Conversation
cleared with every run and repopulated even for cached tasks, plain format with single hash per line
@chelkyl is attempting to deploy a commit to the Vercel Team on Vercel. A member of the Team first needs to authorize it. |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/vercel/turbo-site/7pfU5uTjoYPiEEUqaf1p9z6JzJdT |
I am liking this idea :) |
start time is ignored in http cache
synthesize output rather than unnecessarily opening logs" This reverts commit 23bb051. Necessary because ui output is not prefixed with package and task
when --output-logs=hash-only
@chelkyl can you say more about your use case for this command? I think we would like to avoid having hashes leak into the UX to the extent possible, so maybe there is another way to address the particular content you are interesting in replaying? |
@gsoltis This command was to address dcherman's comment. The original request was for some way to view the logs that were hidden if you run with I would understand if this isn't a compelling use case! |
I can definitely see the utility in getting the logs for tasks that were run. I think the challenge is going to be in finding a good way to address them. I like the idea of tracking recent runs. That could tie in nicely with eventual better management of the cache. I also like the idea of supplying the same arguments to |
Agreed! The hash seemed to be the best solution, especially because the cached log folders are named by their hash. How do you think task names support should be implemented? My initial thought is to add another file like the last run log that is just a JSON file mapping task name to hash that is only added to during |
I think having each |
Heya @chelkyl! @gsoltis is 100% correct about my opinion. I believe that each run should produce a unique identifier that can be used to go back and look up what happened. Conceptually, |
I see! I hadn't even considered uniquely identifying, storing, and isolating each |
@chelkyl If we aren't using it for a cache hit, which I think we wouldn't be, you could use a uuid. A nice-to-have feature would be something easily time-sortable. |
This one's looking a little stale, folks. Seeing as we're nearly complete with our Rust port, I'm going to close this out as this would shortly end up being deleted code if it were to land. |
Replay logs of most recently run tasks sorted by task execution:

List all hashes found in cache folder:

Replay logs of specific task hashes:

Replay last run's logs sorted by task duration from longest to shortest:

Replay last run's logs sorted by task start time and showing all metadata:


As list:
Related to #822
TODO:
Include remote logsunnecessary because remote logs are fetched during run and it is impractical to fetch all remote logs just for this