-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[CI:DOCS] rootless_tutorial: Remove incorrect advice regarding volume… #14905
[CI:DOCS] rootless_tutorial: Remove incorrect advice regarding volume… #14905
Conversation
$ podman run --rm -v ./dir1:/dir1:Z docker.io/library/fedora:36 touch /dir1/file |
ade9e1b
to
1c96a0b
Compare
I added one of the advices back (now also mentioning relative paths starting with a dot). |
1c96a0b
to
8c14f91
Compare
…s and mount points * Remove the instruction "The mount point must exist in the container" as it is incorrect which is shown by this example ``` $ mkdir dir1 $ podman run --rm -v ./dir1:/dir1:Z docker.io/library/fedora:36 touch /dir1/file $ ls dir1 file $ podman run --rm docker.io/library/fedora:36 ls -l /dir1 ls: cannot access '/dir1': No such file or directory $ podman --version podman version 4.1.1 $ ``` * Rewrite the advice "You should always give the full path to the volume you'd like to mount" so that it also mentions relative paths starting with a dot. Signed-off-by: Erik Sjölund <[email protected]>
8c14f91
to
b8a7e27
Compare
LGTM |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: eriksjolund, rhatdan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…s and mount points
"You should always give the full path to the volume you'd like to mount"
"The mount point must exist in the container"
as they are incorrect as shown by this example
Signed-off-by: Erik Sjölund [email protected]
Does this PR introduce a user-facing change?