Skip to content
This repository has been archived by the owner on Feb 9, 2024. It is now read-only.

Avd2.2.0 cvp3.0.0 debian #21

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

ankudinov
Copy link

A proposal for the new AVD container setup.
Most important changes:

  • AVD and CVP collections are now part of the container. It's still possible to override that with the custom version by using bind mount.
  • No env variable to install custom requirements or ansible. If required, override by using custom Dockerfile or devcontainer.json
  • CMD is replaced with ENTRYPOINT to work consistently between different use cases: devcontainer/cli
  • entrypoint script is simplified and enhanced to detect various use cases and react accordingly
  • gitconfig is only processed by entrypoint script if container is called directly. In case of devcontainer, native vscode workflow will be used
  • minor enhancement to the Dockerfile format
  • optimised use of user accounts

changed vscode friendly entrypoint
remove list of images to simplify maintenance
that are not used in entrypoint script any more
as they are part of the image now
as it's part of container
customisation with postCreateCommand and bind mount
@ankudinov ankudinov added the enhancement New feature or request label Jul 1, 2021
@ankudinov ankudinov requested a review from titom73 July 1, 2021 09:35
@titom73 titom73 marked this pull request as draft July 1, 2021 15:28
@titom73 titom73 force-pushed the master branch 10 times, most recently from defed2b to 937f376 Compare August 28, 2021 13:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant