-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add conda lockfile export #23
Conversation
Nice, a side effect will be that all seqera containers will a new checksum 🙄 Did you try to run the snippet |
Yes I thought the same - not sure how else we can do it though?
Yup, looks good I think! Only did a couple of spot checks though, nothing systematic. $ docker run -it community.wave.seqera.io/library/multiqc:1.24.1--789bc3917c8666da sh
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
# micromamba env export --explicit > environment.lock.yml
# tail environment.lock.yml
https://conda.anaconda.org/conda-forge/noarch/rich-13.7.1-pyhd8ed1ab_0.conda#ba445bf767ae6f0d959ff2b40c20912b
https://conda.anaconda.org/conda-forge/noarch/rich-click-1.8.3-pyhd8ed1ab_0.conda#ad6cdc745c76535e7cc56aab865b1fc5
https://conda.anaconda.org/conda-forge/noarch/spectra-0.0.11-py_1.tar.bz2#3979401f3b9bcebf0adba6d091d390e3
https://conda.anaconda.org/conda-forge/noarch/tqdm-4.66.5-pyhd8ed1ab_0.conda#c6e94fc2b2ec71ea33fe7c7da259acb4
https://conda.anaconda.org/conda-forge/noarch/typeguard-4.3.0-pyhd8ed1ab_1.conda#10f49ee1beb82947170c5a5e1a8c0ef3
https://conda.anaconda.org/bioconda/noarch/multiqc-1.24.1-pyhdfd78af_0.tar.bz2#9d85bb8d72c99ef99bb9806fd879c985
https://conda.anaconda.org/conda-forge/noarch/setuptools-72.2.0-pyhd8ed1ab_0.conda#1462aa8b243aad09ef5d0841c745eb89
https://conda.anaconda.org/conda-forge/noarch/wheel-0.44.0-pyhd8ed1ab_0.conda#d44e3b085abcaef02983c6305b84b584
https://conda.anaconda.org/conda-forge/noarch/pip-24.2-pyhd8ed1ab_0.conda#6721aef6bfe5937abe70181545dd2c51
https://conda.anaconda.org/conda-forge/linux-64/procps-ng-4.0.4-hf303867_0.conda#519103dbe25fa37e0b12a85524c3b323 |
Just tested using this file locally and it sort of worked but not quite:
|
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.
Probably shouldn't use a .yml
file extension, as these files are not valid YAML.
wave-utils/src/main/resources/templates/conda/dockerfile-conda-file.txt
Outdated
Show resolved
Hide resolved
wave-utils/src/main/resources/templates/conda/dockerfile-conda-packages.txt
Outdated
Show resolved
Hide resolved
wave-utils/src/test/groovy/io/seqera/wave/util/DockerHelperTest.groovy
Outdated
Show resolved
Hide resolved
wave-utils/src/test/groovy/io/seqera/wave/util/DockerHelperTest.groovy
Outdated
Show resolved
Hide resolved
wave-utils/src/test/groovy/io/seqera/wave/util/DockerHelperTest.groovy
Outdated
Show resolved
Hide resolved
wave-utils/src/test/groovy/io/seqera/wave/util/DockerHelperTest.groovy
Outdated
Show resolved
Hide resolved
wave-utils/src/test/groovy/io/seqera/wave/util/DockerHelperTest.groovy
Outdated
Show resolved
Hide resolved
wave-utils/src/test/groovy/io/seqera/wave/util/DockerHelperTest.groovy
Outdated
Show resolved
Hide resolved
wave-utils/src/test/groovy/io/seqera/wave/util/DockerHelperTest.groovy
Outdated
Show resolved
Hide resolved
Noticed in seqeralabs/wave#172 (comment) you used |
If I add '--name base' to the command
|
I guess we found out why we need the I was testing in complete containers, rather than during the build. So I guess that this is because the command is running before the Should be fine to add |
Closing in favour of #25 |
Generate an explicit lockfile for the created environment.
Exposing this will allow people to exactly replicate the conda build on their own systems.
Warning
I haven't tried running anything here, this is just from me testing outside Wave in the docker images. Please confirm that this works as expected in builds.
'cc @munishchouhan