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

Changing option -v for --mount in bind mount declaration for azure cli docker run command #4879

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Charystag
Copy link

Changing option -v for --mount in bind mount declaration

Bind mount was declared with the option -v, it is now declared with the option --mount according to the recommendations that can be found in the docker reference and the docker engine manual

Explanation

  1. In the volumes section of the docker engine manual where it is written that : "In general, --mount is preferred. The main difference is that the --mount flag is more explicit and supports all the available options."
  2. in the bind mount section of the docker engine manual where the same sentence is written
  3. in the bind mount section of the docker run reference where it is written that : "Even though there is no plan to deprecate --volume, usage of --mount is recommended."

bind mount was declared with the option `-v`, it is now 
declared with the option `--mount` according to the 
recommendations that can be found on the docker docs website 

1. in the volumes section of the docker docs : https://docs.docker.com/engine/storage/volumes/#syntax
2. in the bind mount section of the docker run command doc : https://docs.docker.com/reference/cli/docker/container/run/#mount

Where it is written that the `--mount` syntax should be preferred to the `-v` syntax and that it supports all the mounting options
Copy link
Contributor

Learn Build status updates of commit 2766e56:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/run-azure-cli-docker.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

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.

1 participant