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

luminous: doc: Put command template into literal block #25001

Merged
merged 1 commit into from
Nov 27, 2018
Merged

luminous: doc: Put command template into literal block #25001

merged 1 commit into from
Nov 27, 2018

Conversation

MrStupnikov
Copy link

ceph-deploy osd create --data {device} {ceph-node} command
is not displayed properly because of missing double colons. As a result, double en dash is translated to single en dash [1]

[1] http://docs.ceph.com/docs/luminous/start/quick-ceph-deploy/

@tchaikov
Copy link
Contributor

@MrStupnikov i noticed that this change is actually a backport of #24999 . could you do instead a "git cherry-pick -x bff5a21" onto luminous and the repush the fix?

we enforce this approach to ensure that the fixes on master are properly populated to supported branches. normally, a tracker ticket is created to make sure the fix are tracked.

Copy link
Contributor

@smithfarm smithfarm left a comment

Choose a reason for hiding this comment

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

missing (cherry-picked from ...) line - see #25001 (comment)

@smithfarm
Copy link
Contributor

@MrStupnikov No tracker ticket is needed in this case, just cherry-pick -x and repush.

``ceph-deploy osd create --data {device} {ceph-node}`` command
is not displayed properly because of missing double colons.

Signed-off-by: Alexey Stupnikov <aleksey.stupnikov@gmail.com>
(cherry picked from commit bff5a21)
@MrStupnikov
Copy link
Author

@tchaikov @smithfarm thank you for your reviews. Pushed proper commit message.

@smithfarm smithfarm merged commit 252165f into ceph:luminous Nov 27, 2018
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.

3 participants