-
Notifications
You must be signed in to change notification settings - Fork 50
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
[ENH] - Improve build logs for users #806
Comments
Unfortunately I don't think I think the only way would be to contribute something in |
I think this should be moved to the Action items:
Long reply to @kcpevey's questions (feel free to skip if you don't need context):
Yes, these are written only after an action succeeds. It should be possible to write iteratively to the output stream.
conda-store uses many tools internally and we print logs from them without any processing. The actual channel priority is set via
These strings are part of actions that are printed in
build_conda_environment . That the main build pipeline. There are also actions concerning creation of build artifacts, see the tasks here
create_build .
Yes, I agree. Most of this should be solved by flushing stdout/stderr.
What version of conda-store are you running? Either it's a network failure or you're running an old version of conda-store that doesn't have fixes for package cache corruption, see this PR #745 |
I'm in agreement with @nkaretnikov that at the very least there are issues here which need to first be addressed on the conda-store-server side of things. IMO improving the logging situation there is the first step, then we can think about how to implement user-facing feedback in the UI next. I think for this reason it makes sense to first move this to https://github.com/conda-incubator/conda-store. With respect to the individual issues:
@kcpevey Looks like Can we insert a shim for
It sounds like it might be possible to get more information about what TLDR: there are a few things we can start to take action on.
If we can't make progress with (1) above, we'll probably need to make upstream changes to |
I actually think this one has to do with a bug that had been recently introduced where "queued" builds were showing up as "building". So this can be disregarded. As for reasonable output given what I know now, I propose the logs simply output 2 additional pieces of information:
|
Peyton, Chuck, and I discussed this today. It was agreed that I'll focus on log-flushing since I have the most context. We'll split this into self-contained issues and close this one to avoid getting sidetracked: |
Feature description
As an end user, if I have a build that fails, I expect to use the logs (via the UI, not Loki) to help me debug. I find that the logs available to me are very frequently insufficient to help me debug.
starting build of conda environment 2024-03-21 14:52:09.332184 UTC
. Perhaps they are not flushed frequently enough?channel_priority: flexible
which shouldn't be true for conda-store (we have checked this many times).starting build of conda environment 2024-03-21 14:52:09.332184 UTC
..conda
file. I don't know what conda-store is running so I can't try to reproduce it on my local machine. That said, I'm also not sure if there is any additional logging from conda-lock that would be helpful in debugging?I will also say the conda output that I expect as an end user IS available, but only after a successful build and it looks a little different given that we're building from a conda-lock file. At the very least, I think if an error happens during the build, there should be a stdout/stderr flush.
Value and/or benefit
End users who can debug failed builds and have visibility into the progress of a build.
Anything else?
Example failure:
The text was updated successfully, but these errors were encountered: