-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
[DNM] Debug conda-libmamba-solver + conda-build #158
Conversation
@conda-forge-admin, please rerender |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
…nda-forge-pinning 2023.11.16.09.46.51
@conda-forge-admin, please rerender |
…nda-forge-pinning 2023.11.16.09.46.51
This is very interesting 🤔 If I disable the build by adding some mockup |
Indeed, very interesting 😱 Is |
@@ -79,7 +85,8 @@ if [[ "${BUILD_WITH_CONDA_DEBUG:-0}" == 1 ]]; then | |||
# Drop into an interactive shell | |||
/bin/bash | |||
else | |||
conda mambabuild "${RECIPE_ROOT}" -m "${CI_SUPPORT}/${CONFIG}.yaml" \ | |||
# export CONDA_VERBOSITY=3 | |||
conda build --debug "${RECIPE_ROOT}" -m "${CI_SUPPORT}/${CONFIG}.yaml" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that's big boy log file we get XD
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like it only fails with
2023-11-22T20:46:38.5859860Z + python -c 'import lsst'
2023-11-22T20:46:38.5860312Z Traceback (most recent call last):
2023-11-22T20:46:38.5868623Z File "<string>", line 1, in <module>
2023-11-22T20:46:38.5869792Z ModuleNotFoundError: No module named 'lsst'
which is expected due to the exit 0
in build_stackvana.sh
.
# TEMPORARY: | ||
pip install -U https://github.com/jaimergp/conda-build/archive/debug-cls-keyerror.tar.gz --no-deps | ||
pip install -U git+https://github.com/conda/conda-libmamba-solver.git@conda-build-offline --no-deps | ||
pip install -U https://github.com/jaimergp/conda/archive/investigate-cb-key-error.tar.gz --no-deps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this gives any trouble, you could also do the
python -c "import pathlib, conda.plan; src = pathlib.Path(conda.plan.__file__); text = src.read_text().replace('stack_context_default', 'stack_context'); src.unlink(); src.write_text(text)"
spiel.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
investigate-cb-key-error
also has some extra logging in case we hit a KeyError, and does not raise so the build can proceed without the report. I think the patch does work (you can see the repodata not being re-downloaded). The error is now about import lsst
in the tests 🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I saw conda/conda-libmamba-solver#393 (comment). Ok, reverting and rerunning.
Look, it's passing! |
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)