Skip to content
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

Delete index-cache at the end of container creation #358

Merged
merged 1 commit into from
Mar 29, 2023
Merged

Conversation

dbast
Copy link
Member

@dbast dbast commented Mar 28, 2023

This hopefully is the last attempt to enable libmamba as solver:

The container created from current master branch still fails on s390x:

docker run -ti --rm --platform linux/s390x continuumio/anaconda-pkg-build:master /bin/bash -c "conda install numpy" with RuntimeError: Could not read JSON repodata file (/opt/conda/pkgs/cache/8943ae43.json) parse error line 1

But is successful with docker run -ti --rm --platform linux/s390x continuumio/anaconda-pkg-build:master /bin/bash -c "conda clean --index-cache --yes && conda install numpy --yes" ... Thus adding the index-cache clean command at the end of container creation.

Copy link

@ELundby45 ELundby45 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dbast dbast merged commit 3f8fbe3 into master Mar 29, 2023
@dbast dbast deleted the index_clean branch March 29, 2023 09:12
@dholth
Copy link

dholth commented Mar 29, 2023

Since 23.1 conda writes <cache key>.state.json with the http headers, alongside the full, unmodified repodata. It contains a certain set of keys. libmamba requires a different set of keys, and crashes if they are not there; they want to make all keys optional to avoid this problem, but I don't know which release that would be.

We might be able to hack in a conda feature that writes libmamba's expected keys possibly as "" so that this problem does not continually happen.

@dholth
Copy link

dholth commented Mar 29, 2023

We checked conda 23.3's .state.json format and it looks much closer to the libmamba format. Hopefully this will work great after the March release.

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.

None yet

4 participants