Skip to content

Conversation

@alexander-smolyakov
Copy link
Contributor

Devcontainer name:

  • miniconda

Description:

This PR contains the same changes as in #692. The main idea is to make the anaconda and miniconda devcontainers consistent.

Changelog:

  • The pip install and conda install commands now execute against the continuumio/miniconda3 image. (This will allow us to not worry about opt folder permissions and ownership since we can set them up during COPY instruction);

  • The COPY instruction now sets up permissions and ownership for the opt folder. The COPY instruction will not create an additional layer, which should help decrease image size;

  • Previous commands for setting up permissions and ownership were removed.

Checklist:

  • Checked that applied changes work as expected

@alexander-smolyakov alexander-smolyakov requested a review from a team as a code owner September 11, 2023 13:06
@alexander-smolyakov
Copy link
Contributor Author

Additional info:

Before changes

REPOSITORY                                                                           TAG         IMAGE ID       CREATED         SIZE
vsc-miniconda-62fb3f29e4a9189b28ec2500ae0bbb7d3c31676a6ead82d187ffd0f867dab9c6       latest      82708182091f   3 minutes ago   2.06GB

After changes

REPOSITORY                                                                           TAG         IMAGE ID       CREATED         SIZE
vsc-miniconda-62fb3f29e4a9189b28ec2500ae0bbb7d3c31676a6ead82d187ffd0f867dab9c6       latest      504b0619dd23   3 minutes ago   1.67GB

Copy link
Member

@samruddhikhandale samruddhikhandale left a comment

Choose a reason for hiding this comment

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

Looks good, left one comment!

@samruddhikhandale samruddhikhandale merged commit 10c097f into devcontainers:main Sep 11, 2023
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