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

Docker image tweaks #6466

Merged
merged 2 commits into from
Jun 7, 2024
Merged

Docker image tweaks #6466

merged 2 commits into from
Jun 7, 2024

Conversation

danielhollas
Copy link
Collaborator

Some tweaks to Docker images

  • Don't pin mamba version, I think we always want the latest version (similar to pip). Relatedly, the latest versions of conda actually use the libmamba resolver so there's probably little benefit of using mamba over conda now. But I haven't explored it in detailed yet so I did not change it in this PR.
  • set PIP_USER=1, which will make the --user flag to pip default. We want users to install to ~/.local since the home directory is preserved on a volume when the container stops. We do the same in aiidalab-docker-stack
  • xz-utils package was being installed twice

@danielhollas danielhollas requested review from unkcpz and sphuber June 6, 2024 23:33
@@ -22,8 +22,7 @@ RUN mamba install --yes \
# Install erlang.
RUN apt-get update --yes && \
apt-get install --yes --no-install-recommends \
erlang \
xz-utils && \
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Already installed in aiida-core-base

Copy link

codecov bot commented Jun 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.83%. Comparing base (ef60b66) to head (26b9689).
Report is 112 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6466      +/-   ##
==========================================
+ Coverage   77.51%   77.83%   +0.33%     
==========================================
  Files         560      561       +1     
  Lines       41444    41767     +323     
==========================================
+ Hits        32120    32505     +385     
+ Misses       9324     9262      -62     
Flag Coverage Δ
presto 73.15% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@sphuber sphuber left a comment

Choose a reason for hiding this comment

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

Thanks @danielhollas

@sphuber sphuber merged commit 82bba13 into aiidateam:main Jun 7, 2024
11 checks passed
@danielhollas danielhollas deleted the docker-tweaks branch June 7, 2024 06:06
mikibonacci pushed a commit to mikibonacci/aiida-core that referenced this pull request Sep 3, 2024
* Remove pin of mamba version
* Set `PIP_USER` as default
* Remove double install of `xz-utils`
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.

2 participants