Skip to content

doc/man: add missing cephadm shell --mount option#35634

Merged
sebastian-philipp merged 1 commit intoceph:masterfrom
mgfritch:cephadm-manpage-shell-mount
Jun 18, 2020
Merged

doc/man: add missing cephadm shell --mount option#35634
sebastian-philipp merged 1 commit intoceph:masterfrom
mgfritch:cephadm-manpage-shell-mount

Conversation

@mgfritch
Copy link
Contributor

also clean-up --mount help usage

Signed-off-by: Michael Fritch mfritch@suse.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

also clean-up `--mount` help usage

Signed-off-by: Michael Fritch <mfritch@suse.com>
@mgfritch mgfritch requested review from asettle and zdover23 June 18, 2020 02:00
@mgfritch mgfritch requested a review from a team as a code owner June 18, 2020 02:00
parser_shell.add_argument(
'--mount', '-m',
help='file or directory path that will be mounted in container /mnt')
help='mount a file or directory under /mnt in the container')
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@asettle @zdover23 the verbiage here felt incomplete, an alternate suggestion is welcome!

@sebastian-philipp sebastian-philipp merged commit 6dcd0a6 into ceph:master Jun 18, 2020
@mgfritch mgfritch deleted the cephadm-manpage-shell-mount branch June 18, 2020 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants