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

[docs] Add man-page of subcommand mounted #242

Merged
merged 1 commit into from
Dec 5, 2018

Conversation

eramoto
Copy link
Contributor

@eramoto eramoto commented Nov 20, 2018

Add a man-page of subcommand mounted that is added by commit 1075a73.
Add new lines to prevent the section SUB-COMMANDS of containers-storage.md from crushing when previews as markdown.

Signed-off-by: ERAMOTO Masaya eramoto.masaya@jp.fujitsu.com

Add a man-page of subcommand mounted that is added by commit 1075a73.
Add new lines to prevent the section SUB-COMMANDS of containers-storage.md
from crushing when previews as markdown.

Signed-off-by: ERAMOTO Masaya <eramoto.masaya@jp.fujitsu.com>
@rhatdan
Copy link
Member

rhatdan commented Nov 20, 2018

LGTM
@TomSweeneyRedHat @giuseppe @nalind PTAL

@rhatdan
Copy link
Member

rhatdan commented Nov 27, 2018

@TomSweeneyRedHat @giuseppe @nalind @umohnani8 Someone please review so we can merge this.

containers-storage mounted - Check if a file system is mounted

## SYNOPSIS
**containers-storage** **mounted** *LayerOrContainerNameOrID*
Copy link
Member

Choose a reason for hiding this comment

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

Should this be "LayerOrContainerNameOrID" or "layerOrContainerMountpointOrNameOrID"? @giuseppe or @nalind do you know?

Copy link
Member

Choose a reason for hiding this comment

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

The current logic doesn't attempt to look up a layer by mount point, so LayerOrContainerNameOrID looks like the correct description.

@umohnani8
Copy link
Member

LGTM

1 similar comment
@giuseppe
Copy link
Member

giuseppe commented Dec 5, 2018

LGTM

@rhatdan rhatdan merged commit c3682b3 into containers:master Dec 5, 2018
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.

6 participants